Skip to content

Conversation

@wildintellect
Copy link
Contributor

@wildintellect wildintellect commented Oct 24, 2025

Edits based on #15 , some are not quickly fixable or need more discussion, only the easy changes are part of this.

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@wildintellect wildintellect marked this pull request as ready for review October 28, 2025 17:53
@wildintellect wildintellect requested a review from hrodmn October 28, 2025 17:54
@wildintellect
Copy link
Contributor Author

@hrodmn I fixed most of the easy things from #15 , there are a few enhancements to consider for later, and other things for discussion that I think we can apply later.

Copy link
Contributor

@hrodmn hrodmn left a comment

Choose a reason for hiding this comment

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

Thanks for making these updates @wildintellect ! I had a few ideas for little tweaks we could make but nothing big.

@wildintellect wildintellect requested a review from hrodmn October 31, 2025 15:58
Copy link
Contributor

@hrodmn hrodmn left a comment

Choose a reason for hiding this comment

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

I stepped through all of the notebooks on the binder hub, everything seems to be working well!

@wildintellect wildintellect merged commit 144aa5e into main Oct 31, 2025
1 check passed
@wildintellect wildintellect deleted the feat/oct25 branch October 31, 2025 20:22
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.

3 participants