Skip to content

finish book chapter move#228

Merged
lazarusA merged 18 commits intomainfrom
la/f_book_chapter
Feb 19, 2026
Merged

finish book chapter move#228
lazarusA merged 18 commits intomainfrom
la/f_book_chapter

Conversation

@lazarusA
Copy link
Member

continue #216

@gemini-code-assist
Copy link

Summary of Changes

Hello @lazarusA, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request finalizes the migration of the 'Synthetic Data Analysis' example from a standalone book chapter project into the core documentation as a new, accessible tutorial. This refactoring improves the organization of examples and ensures they are properly built and presented within the project's documentation. Additionally, it includes a minor update to project dependencies and a small code clarity improvement in an import statement.

Highlights

  • Documentation Structure: The 'Synthetic Data Analysis' example, previously located in the projects/book_chapter directory, has been moved and integrated into the main documentation as a new tutorial.
  • New Tutorial: A new tutorial titled 'Synthetic Respiration' has been added to the documentation, showcasing hybrid model training with Q10 temperature sensitivity.
  • Dependency Management: The Statistics package has been added as a new dependency in the main Project.toml file.
  • Code Refinement: The import statement for ComponentArrays in src/EasyHybrid.jl was refined for explicit module and type import.

🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console.

Changelog
  • Project.toml
    • Added Statistics package as a dependency.
  • docs/literate/tutorials/synthetic_respiration.jl
    • Introduced a new tutorial for synthetic data analysis, showcasing hybrid model training with Q10 temperature sensitivity.
  • docs/make.jl
    • Integrated the new 'Synthetic Respiration' tutorial into the documentation's 'Research' section.
  • projects/book_chapter/Project.toml
    • Deleted the Project.toml file previously associated with the book chapter project.
  • projects/book_chapter/example_synthetic.jl
    • Eliminated the example_synthetic.jl script, as its content was moved to a new tutorial.
  • src/EasyHybrid.jl
    • Modified the ComponentArrays import to be more specific, importing ComponentArrays and ComponentArray directly.
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

Copy link

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request moves a book chapter example into the main documentation as a tutorial. The changes include adding the new tutorial file, updating the documentation build configuration, removing the old project files, and adding a dependency compatibility entry.

My review focuses on the new tutorial file. I've suggested a few improvements for code clarity and quality, such as simplifying a redundant call, using more descriptive variable names, and adopting a more concise function syntax. I also noted the presence of TODO comments in the new tutorial, which should ideally be resolved or tracked as issues before being published as part of the documentation.

# ### DimensionalData
using DimensionalData
mat = Array(Matrix(dfnot)')
da = DimArray(mat, (Dim{:col}(Symbol.(names(dfnot))), Dim{:row}(1:size(dfnot, 1))));

Choose a reason for hiding this comment

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

medium

The dimension names :col and :row are a bit generic. For better clarity and consistency with common practices in DimensionalData.jl, consider using more descriptive names like :variable and :observation. This would make the data structure's intent clearer to readers of the tutorial.

da = DimArray(mat, (Dim{:variable}(Symbol.(names(dfnot))), Dim{:observation}(1:size(dfnot, 1))));

@codecov
Copy link

codecov bot commented Feb 19, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 50.31%. Comparing base (ea7f378) to head (c2da981).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #228   +/-   ##
=======================================
  Coverage   50.31%   50.31%           
=======================================
  Files          27       27           
  Lines        1932     1932           
=======================================
  Hits          972      972           
  Misses        960      960           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@BernhardAhrens
Copy link
Collaborator

The bookchapter example should go under research as we try to replicate existing science

@lazarusA
Copy link
Member Author

yes, currently in the menu bar is going there already.

Do you mean: we should create a new folder, research and put it there? make sense...

@BernhardAhrens
Copy link
Collaborator

Ah I did not see that it already goes there. I think the research folder is a good idea

@lazarusA
Copy link
Member Author

I will do the new folder 📁.

@lazarusA lazarusA merged commit 1df1aaa into main Feb 19, 2026
12 checks passed
@lazarusA lazarusA deleted the la/f_book_chapter branch March 6, 2026 07:59
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