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 pre-commit and RTD build #42

Closed
wants to merge 3 commits into from
Closed

Conversation

sphuber
Copy link
Contributor

@sphuber sphuber commented Oct 26, 2023

No description provided.

The older version fails during install for newer versions of poetry.
Add the new required `build.os` key and fix it to Ubuntu 22.04
@sphuber
Copy link
Contributor Author

sphuber commented Oct 26, 2023

@chrisjsewell this fixes two issues with the CI pipeline, however, tests are still failing. Seems like some dependency down-stream got a new release which causes the diffs to be different. For some reason the md5s of images are getting changed and I have no idea who is responsible for writing this output or where it is being written:

## replaced /cells/13/outputs/0/data/image/png:
-  iVBORw0K...<snip base64, md5=f94a82072d4a539b...>
+  iVBORw0K...<snip base64, md5=d4cb03e0b7c8a788...>

## replaced /cells/14/outputs/0/data/image/jpeg:
-  /9j/4AAQ...<snip base64, md5=780504154e4118c5...>
+  /9j/4AAQ...<snip base64, md5=63dc79d972a66eab...>

## replaced /cells/15/outputs/0/data/image/png:
-  iVBORw0K...<snip base64, md5=f94a82072d4a539b...>
+  iVBORw0K...<snip base64, md5=d4cb03e0b7c8a788...>

## replaced /cells/15/outputs/3/data/image/jpeg:
-  /9j/4AAQ...<snip base64, md5=780504154e4118c5...>
+  /9j/4AAQ...<snip base64, md5=63dc79d972a66eab...>

Any ideas?

@sphuber
Copy link
Contributor Author

sphuber commented Nov 9, 2023

@chrisjsewell just a quick ping. This is blocking a number of packages to release a Python 3.12 compatible version. Could you please have a look? Thanks!

@chrisjsewell
Copy link
Owner

all fixed cheers

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.

None yet

2 participants