-
Notifications
You must be signed in to change notification settings - Fork 11
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
Merge staging changes into production for 2024.11.1 #686
Merged
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
Main difference - remove m2m between target and project and add fk to project in target model. Quite a lot of implications to security and target object retrieval. Locally operations know to me (LHS and RHS upload, download) work. Next step, test in my stack
Fix ComputedMolecule name format (issue 1524)
The problem was caused by faulty SiteObservation query, cases where observation longcode was the same between multiple targets are affected.
Fixed (probably) data loss on metadata.csv uploads (issue 1535)
Instead of previous 'New', each upload gets tagname from version (upload_[N])
feat: versioned auto-assigned upload tags
fix: correct source paths
Allow uploading same target to multiple projects (issue 1514)
fix: add code path to handle old uploads (with task_id) (issue 1514)
fix: a bug in migration script (issue 1514)
Implements 2 points outlined in spec (m2ms/fragalysis-frontend#1527) - original sdf download - custom pdb download TODO: - additional data to sdf
RHS data download (issue 1527, partial)
Tested on prod directory tree and confirmed working, safe do deploy to prod. NB! tarball 'A71EV2A_upload_1_2_161024.tgz' seems to be missing!
Another fix to data structure migration script.
Added timestamp to auto-upload tag (issue 1517)
Single sdf in downloads has correct structure separators (issue 1555)
kaliif
approved these changes
Nov 4, 2024
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.
No description provided.