Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix regex patterns and add minimum version for scikit-image #198

Merged

Conversation

tabedzki
Copy link
Contributor

This pull request includes two commits. The first commit fixes regex patterns in the code by providing raw annotations for strings with unsupported escape regex sequences. The second commit adds a minimum version requirement for scikit-image in the setup.py file. These changes ensure that the code functions correctly in Python 3.12 and is compatible with the required dependencies.

@tabedzki
Copy link
Contributor Author

@ethho would you be able to review this?

@ethho
Copy link

ethho commented Aug 13, 2024

@ttngu207 or @kushalbakshi Do you have time to review? I'm not as familiar with this project.

cc: @tabedzki

@kushalbakshi
Copy link
Collaborator

Thanks for bringing this to my attention, @ethho. I should be able to take a look today.

@kushalbakshi kushalbakshi self-requested a review August 14, 2024 20:07
Copy link
Collaborator

@kushalbakshi kushalbakshi left a comment

Choose a reason for hiding this comment

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

Thanks @tabedzki. I was able to test the tutorial notebook through Codespaces with your changes and it ran through without any issues. I'll merge this right now but hold off on a release since we're planning for one in the coming weeks. You will still be able to use this by installing the repository directly.

@kushalbakshi kushalbakshi merged commit 27c56ea into datajoint:main Aug 14, 2024
7 checks passed
@ttngu207
Copy link
Contributor

Thanks @tabedzki @kushalbakshi @ethho!

@tabedzki tabedzki deleted the adding-raw-strings-and-package-minimum branch October 3, 2024 17:50
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.

4 participants