Repository navigation
Spec 015 T070: node answers an event's BioPAX - #754
Merged
Merged
Conversation
/exporter/event/{id}.biopax is answered from the library service's
/biopax/ (T069), as research R6 decided: application/rdf+xml, a
{stId}.biopax download, SBML's 404 and 400 in the envelope, the body
streamed, its own share of two apart from SBML's and the deep pool, and a
503 when the service cannot answer, never a fallback. SBML's answer is
generalised to a format each; the event endpoint dispatches by extension
and leaves the rest to Java. The id's kind in parameters.ts takes
.biopax, so the answer is in standard form and cached per release. Not
routed yet: T071 does that once T049 and T050 are recorded.
Also from #753's review: T032 says it was next-release.sh's cleanup that
removed release 97, T072's 16 minutes cites its measurement, and the
dependencies say T072 needs a copy at beta's release.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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
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.
Spec 015 (specs/015-graph-exports), T070.
What it does.
/ContentService/exporter/event/{id}.biopaxis answered from the library service's/biopax/{stId}, which T069 adds. It follows research R6:application/rdf+xml, downloaded as{stId}.biopax;How:
exportAnswer, with one format for each export./exporter/event/{id}dispatches by extension:.biopaxto BioPAX,.sbmlas before, and anything else to Java as before.SBML_SERVICE, which now names the library service for both exports.parameters.ts's kind for the id now takes.biopax, so the answer is in standard form and cached per release (spec 011).Not routed yet. T071 adds the route once T049 and T050 are recorded. Until T069 merges, the service has no
/biopax/and would answer it with a 404.Tests:
biopax-export.spec.ts, 10 tests, against a stub service. Each of six mutants made at least one fail:.owldownload name;text/plaintype;.biopaxleft out of standard form.SBML's own 14 tests pass unchanged, and so do all 965 of content-node's.
Also, from #753's review (tasks.md):
next-release.sh cleanupthat removed release 97.🤖 Generated with Claude Code