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.

Sign up for GitHub

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

Jump to bottom

Major PR [DO NOT MERGE] #50

Merged
merged 70 commits into from
Sep 11, 2024
Merged

Major PR [DO NOT MERGE] #50

merged 70 commits into from
Sep 11, 2024

Conversation

ypriverol
Copy link
Member

This PR includes multiple refinements about the structure of quantms.io, the fields, the metadata and the documentation.

  • All the structures were moved into an Ansidoc page README.adoc
  • All the schemas were updated to have documentation for every field and attribute.
  • Non-necessary fields were removed.
  • Flowchart about the relation between all formats and models was created.

zprobot and others added 30 commits June 5, 2024 16:31
@zprobot
minor fix: psm
48acdf4
@zprobot
fixed: error
613572b
@zprobot
update: doc
e8f683c
@zprobot
update: doc,query
1c7132b
@zprobot
update: index
74af8ab
@zprobot
minor change
2af8197
@ypriverol
Merge branch 'main' of https://github.com/bigbio/quantms.io into dev
1a911bd
fixed: condition
09de3d9
@ypriverol
Merge branch 'dev' of https://github.com/bigbio/quantms.io into dev
f137f25
update: pep
f9a58fc
@mobiusklein
use correct name for MANIFEST.in to allow setuptools to find it
b99a3c5
@mobiusklein
Make unit tests runnable from the root, decouple test file location f…
70c2773 
…rom `__package__`
@mobiusklein
Appease Codacy
57df7c8
@ypriverol
Merge pull request #47 from mobiusklein/feature/test-suite-running
4419f0e 
feature: make test suite runnable from repository root
@ypriverol
Merge branch 'dev' of https://github.com/bigbio/quantms.io into dev
c5a6db9
fix test
4ee96d2
@mobiusklein
fix: move tests outside package, fix test CI failure
c2ee5b1 
This change moves all the test code and test data out of the Python
package, which means that it no longer takes up space in the built
distribution, and is no longer installed with the package on end-user
machines.

It also fixes the CI script that assumed tests ran inside the package,
which depended upon the local imports that were removed in the last
batch of changes.
@mobiusklein
pytest paths
763a15a
@zprobot
feat：maxquant
c4b1d13
@zprobot
update: doc
56c5dc5
@ypriverol
Merge pull request #48 from mobiusklein/fix/move-tests
dac2cb1 
fix: move tests outside package, fix test CI failure
@zprobot
update: command
d60bfae
@zprobot
update: diann_convert
63d55eb
@zprobot
update: maxquant
4a58418
@zprobot
fixed: doc
4e870da
@ypriverol
Merge branch 'dev' of https://github.com/bigbio/quantms.io into dev
3692136
@ypriverol
small change in command
c5cdd85
@ypriverol
small change in command
5a402a7
@ypriverol
Major change in the PSM view.
31add29
@ypriverol
Major change in the PSM view.
8c875bd
@ypriverol ypriverol mentioned this pull request Sep 8, 2024
Closed
@ypriverol ypriverol linked an issue Sep 10, 2024 that may be closed by this pull request
CamelCase vs snake_case #54
Closed
@ypriverol ypriverol linked an issue Sep 11, 2024 that may be closed by this pull request
additional_scores as Record in parquet instead of list of String #52
Closed
@ypriverol ypriverol merged commit a3eff82 into main Sep 11, 2024
10 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zprobot zprobot Awaiting requested review from zprobot

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

CamelCase vs snake_case additional_scores as Record in parquet instead of list of String
3 participants
@ypriverol @zprobot @mobiusklein