Skip to content

Conversation

@LaSi5002
Copy link
Contributor

rapidyaml==0.8.0 is incompatible with Python 3.12. Caused error on pip install -e .
from fourc_webviewer.fourc_webserver import FourCWebServer was removed 2 days ago. Probably accidentally.

Copy link
Contributor

@c-p-schmidt c-p-schmidt left a comment

Choose a reason for hiding this comment

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

Thank you for this fix! We try to establish some first actual tests of the webviewer soonish, such that these things should not occur anymore.

@gilrrei
Copy link
Collaborator

gilrrei commented May 23, 2025

Hey @LaSi5002, thanks for the fixes. Regarding the requirements, your fix is absolutely correct. However, I don't get this error message 🤔 Which operating system are you using?

@LaSi5002
Copy link
Contributor Author

Hi @gilrrei im on Windows 11. But it seems weird that it worked before on your systems. Are you sure that you ran it with the 3.12 interpreter?

@c-p-schmidt
Copy link
Contributor

Yes, we use python 3.12 on an Ubuntu 24.04 operating system.

@gilrrei
Copy link
Collaborator

gilrrei commented May 24, 2025

@LaSi5002

Hi @gilrrei im on Windows 11. But it seems weird that it worked before on your systems. Are you sure that you ran it with the 3.12 interpreter?

4C and most of the subprojects are only fully tested and supported on Ubuntu 24.04. There is a strong dependency of the OS on Python packages. This is exactly what you observed; in this instance, it has nothing to do with the Python version.

The fix you provided is only required for Windows and macOS. Once fourcipp (which is the dependency that requires rapidyaml) is updated, this fix is no longer required.

@c-p-schmidt So let's merge this as is, and revert it once the dependency is sorted out in fourcipp,

@c-p-schmidt c-p-schmidt merged commit 0c69146 into 4C-multiphysics:main May 24, 2025
1 check passed
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