Update molAOP Builder entry: repository renamed, version bump, API auth flag - #299
Closed
marvinm2 wants to merge 5 commits into
Closed
Update molAOP Builder entry: repository renamed, version bump, API auth flag#299marvinm2 wants to merge 5 commits into
marvinm2 wants to merge 5 commits into
Conversation
Contributor
|
Oh, the chaos... doing this manually... :/ |
Contributor
|
Manually merged in. |
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
Housekeeping on
docs/service/molaopbuilder.jsonafter renaming the upstream repository. JSON only — the generator workflow regeneratesmolaopbuilder.mdfrom it.Changes
urlmarvinm2/KE-WP-mappingmarvinm2/molAOP-builderdeployment_docs.../KE-WP-mapping#quick-start.../molAOP-builder#quick-startrelease_notes.../KE-WP-mapping/releases.../molAOP-builder/releasesinstance.sourcemarvinm2/KE-WP-mappingmarvinm2/molAOP-builderinstance.docker.../KE-WP-mapping/pkgs/container/ke-wp-mapping.../molAOP-builder/pkgs/container/molaop-builderinstance.version2.3.02.8.0access.logintruefalseRepository rename.
marvinm2/KE-WP-mappingis nowmarvinm2/molAOP-builder. The old name dated from when the tool only mapped Key Events to WikiPathways; it now covers WikiPathways, GO and Reactome. GitHub redirects the old URLs, so nothing was broken — but the catalog should name the real repository.Container link was wrong in two ways. Both the repository and the package name were stale: the published package is
molaop-builder, notke-wp-mapping. Verified the corrected target is real and public via an anonymous registry pull:access.login. The service's public REST API (/api/v1) requires no authentication — it is open, rate-limited to 100 req/h/IP. Only the mutation endpoints (submitting mappings and proposals) need a login, sotrueoverstated the barrier for anyone browsing the catalog for reusable APIs.Version. The deployed release is 2.8.0 per the repository CHANGELOG.
Notes for maintainers
docs/service/ke-wp-mapping.mdlooks like an orphan: it is an autogenerated page with no correspondingke-wp-mapping.json, left over from before this service was renamed tomolaopbuilder. It duplicatesmolaopbuilder.mdand still carries the pre-v1.5 description ("gene overlap, text similarity"), which no longer reflects how the tool ranks suggestions. I have not touched it, in case the path is intentionally kept for existing links — but it is stale content either way.doiempty. The dataset now has a Zenodo concept DOI (10.5281/zenodo.20184643), but the field is documented as the DOI of a related article/preprint, which is not out yet. Happy to add it if the field is meant more broadly.