Skip to content

Feature docker#11

Merged
ghareea merged 6 commits intomainfrom
feature-docker
Apr 18, 2026
Merged

Feature docker#11
ghareea merged 6 commits intomainfrom
feature-docker

Conversation

@ghareea
Copy link
Copy Markdown
Contributor

@ghareea ghareea commented Apr 18, 2026

Docker installation and improved JSON parsing

ghareea added 2 commits April 17, 2026 15:12
  - Add Docker support (Dockerfile, .dockerignore) for a self-contained,
    cross-platform runtime; update README with Docker setup instructions
  - Add .env.example and wire up python-dotenv so .env files are actually
    loaded at startup (was documented but never implemented)
  - Migrate from futurehouse_client to edison_client, renaming
    FUTUREHOUSE_API_KEY → EDISON_API_KEY and updating all platform URLs
    to platform.edisonscientific.com
  - Fix JSONDecodeError in experimental_assay() by extracting JSON from
    LLM responses bustly, handling markdown code-fenced output
  - Add CLAUDE.md with codebase guidance for future development
@ghareea ghareea requested a review from mskarlin April 18, 2026 00:25
Comment thread robin/assays.py Outdated
Comment thread robin/configuration.py
Comment thread robin/configuration.py Outdated
Comment thread Dockerfile
Comment thread pyproject.toml Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
- Improve JSON parsing in assays.py to use re.search fallback
- Rename _fh_client/_fh_client property to _edison_client/edison_client
- Add comment clarifying get_default_llm_config() key injection pattern
- Move jupyterlab to [dev] extras in pyproject.toml; remove from Dockerfile
- Pin edison-client>=0.11 in pyproject.toml
- Update README to remove outdated closed beta/Finch references

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Comment thread README.md
Copy link
Copy Markdown
Contributor

@SamCox822 SamCox822 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm once ci passes

@ghareea ghareea merged commit 9dd64b7 into main Apr 18, 2026
1 of 3 checks passed
@ghareea ghareea deleted the feature-docker branch April 18, 2026 06:33
@ghareea ghareea removed the request for review from mskarlin April 18, 2026 06:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants