merge develop nach GSOC24 #2525
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Merge develop to stable == 9.0.0
Merge stable to develop
Occasionally the test runs on macos-13 are super slow and there is generally a lot of variation in the time required to run the test suite on those runners. I think they genuinely need more time sometimes, so this increases the timeout to accommodate.
* Run flake8 for all .py files tracked in git * Fix flake8 errors
this should keep it archived also when a server sets the threshold to a few years
* Add a codespell lint job * Fix misspellings found by codespell
* updated data base migration to current release * deprecation for manual migration added
`rtd.io` is on longer operational, according to [the project page](https://readthedocs.org/projects/mss/) the correct domain is either `rtfd.io` or the full `readthedocs.io`. Also switch to https.
* Fixed an issue with units being ignored in vertical axis * Fixed an issue with *assuming* that vertical axis is in pressure if no standard name "air_pressure" was available * Fixed an issue with extrapolating data * Fixes an issue with interpolating in pressure instead of log-pressure * Fixes an issue with ignoring jumps/extrapolation in data for missing longitudes (now it is consistent with vertical section driver)
* doc updated * indenting level fixed * add details about the wsgi modules
Merge stable to develop
…n-MSS#2431) Enhancing the functionality of the MSColab server to allow users to upload small images which would replace generic initials as identifiers during login. This feature personalizes user accounts and significantly enhances the visual aspect of user identification and interaction within the MSColab platform. Co-authored-by: Aryan Gupta <[email protected]>
On import of mslib.mscolab.server initialize_managers was called once, hooking events up to handlers in a SocketsManager instance and initializing the Flask app with the Flask-SocketIO extension. Then, in handle_start, initialize_managers was called a second time, creating another SocketsManager object, overwriting the instance saved with the SocketIO object, and trying to initialize the Flask app with it again. The last step does not work though, a Flask app can only be initialized once with each extension (i.e. the sockio.init_app(app) call can only be done once per Flask app object). What this led to is that the Flask-SocketIO extension was using the first SocketsManager object to handle incoming events, while the server side was using the second SocketsManager object when it called methods on sockio.sm. This changes the startup process to do the initialization just once.
Previously test_sockets_manager.py communicated with a MSColab server running in a different process. This made it impossible to assert anything on the server state (e.g. on attributes of the SocketsManager object) due to the process boundary. Using a test client also simplifies the test setup quite a bit.
Merge stable to develop
* changed to videos/ * use mss style on rtfd
With this flask-migrate is used on application setup to automatically migrate the configured database to the latest revision shipped with MSColab. This will make manual migrations in production environments unnecessary in the future. Also included is an upgrade path from databases which were previously manually managed. This entails generating a new database and copying over all data.
Bumps [benc-uk/workflow-dispatch](https://github.com/benc-uk/workflow-dispatch) from 1.2.3 to 1.2.4. - [Release notes](https://github.com/benc-uk/workflow-dispatch/releases) - [Commits](benc-uk/workflow-dispatch@v1.2.3...v1.2.4) --- updated-dependencies: - dependency-name: benc-uk/workflow-dispatch dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [benc-uk/workflow-dispatch](https://github.com/benc-uk/workflow-dispatch) from 1.2.3 to 1.2.4. - [Release notes](https://github.com/benc-uk/workflow-dispatch/releases) - [Commits](benc-uk/workflow-dispatch@v1.2.3...v1.2.4) --- updated-dependencies: - dependency-name: benc-uk/workflow-dispatch dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
validate xml content
When inserting rows into PostgreSQL tables and explicitly setting values for e.g. auto-increment columns, like the IDs, the associated sequences aren't automatically updated. This means that without explicitly fixing the sequences PostgreSQL would try to insert new rows with existing values for the IDs, failing in the process due to unique constraints. This issue doesn't affect MariaDB nor SQLite. The former automatically updates auto-increment counters on insert, the latter simply uses the last rows ID as a starting point. This change also extends the tests to try adding a new user after the migration, which should catch this kind of issue.
Created enum types should also be dropped on downgrade. Dropping tables must be committed to become visible.
* removing pinned xmlschema (Open-MSS#2117) * Stop ignoring some test files (Open-MSS#2291) * Reconfigured tutorial videos for RTFD (Open-MSS#2458) * changed to videos/ * use mss style on rtfd * fix urls and text (Open-MSS#2460) * Bump benc-uk/workflow-dispatch from 1.2.3 to 1.2.4 (Open-MSS#2450) Bumps [benc-uk/workflow-dispatch](https://github.com/benc-uk/workflow-dispatch) from 1.2.3 to 1.2.4. - [Release notes](https://github.com/benc-uk/workflow-dispatch/releases) - [Commits](benc-uk/workflow-dispatch@v1.2.3...v1.2.4) --- updated-dependencies: - dependency-name: benc-uk/workflow-dispatch dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Limit video width to the containing block's width (Open-MSS#2463) * pinning of libtiff is not needed but blocks a more recent libxml2 and others (Open-MSS#2469) * fixed the model.Message description and tests now using op_id (Open-MSS#2470) * bug fix for incomplete xml imports (Open-MSS#2468) validate xml content * prepare v9.2.0 (Open-MSS#2481) --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: Matthias Riße <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
This reverts commit 2c89c79.
Merge stable to develop
Enhancing the functionality of the MSColab server to allow users to upload small images which would replace generic initials as identifiers during login. This feature personalizes user accounts and significantly enhances the visual aspect of user identification and interaction within the MSColab platform. This includes real-time tracking of active users per operation, enhancing the chat window, seeing who performed what changes and separating system message from user messages. --------- Co-authored-by: Aryan Gupta <[email protected]>
Show in About a new version available. --------- Co-authored-by: Matthias Riße <[email protected]>
removed the xmlsec binary selector based on CONDA_PREFIX provided separate config for msidp separate dir for mscolab server data and work asynchronously by msolab ui --------- Co-authored-by: Matthias Riße <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Purpose of PR?:
Fixes #
Does this PR introduce a breaking change?
If the changes in this PR are manually verified, list down the scenarios covered::
Additional information for reviewer? :
Mention if this PR is part of any design or a continuation of previous PRs
Does this PR results in some Documentation changes?
If yes, include the list of Documentation changes
Checklist:
<type>: <subject>