🖋️ Scribe: Update README to list pandas as required#710
🖋️ Scribe: Update README to list pandas as required#710
Conversation
- Update installation instructions to recommend `pip install "imednet[pandas]"` - Add a note clarifying that pandas is required for core functionality despite being optional in config - Add journal entry to `.jules/scribe.md` documenting the hidden dependency This prevents new users from encountering `ImportError` when installing the base package. Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com>
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
1 participant
💡 What: Updated
README.mdto change the default installation command topip install "imednet[pandas]"and added a warning note. Added an entry to.jules/scribe.md.🎯 Why: The base installation
pip install imednetresults in anImportErrorbecausepandasis imported unconditionally in the initialization chain, despite being listed as optional inpyproject.toml.🧠 Cognitive Impact: Fixes a "broken path" for new users who follow the installation instructions and immediately face a crash. Saves time debugging missing dependencies.
📖 Preview:
PR created automatically by Jules for task 12132033040977034608 started by @fderuiter