Skip to content

Spec 015: BioPAX's exporter work ticked; the website's BioPAX planned (2) - #751

Closed
adamjohnwright wants to merge 2 commits into
mainfrom
s015/biopax-tasks-2
Closed

adamjohnwright wants to merge 2 commits into
mainfrom
s015/biopax-tasks-2

Conversation

@adamjohnwright

Copy link
Copy Markdown
Contributor

Spec 015 (specs/015-graph-exports). This replaces #750, carrying its commit and the fixes from its review.

From #750, unchanged:

  • T045–T048 and T051 are ticked against BioPAX3Exporter's own spec 001.
  • T031 is withdrawn.
  • T049 and T050 are narrowed to the website's part.
  • T052 becomes T069–T072, on SBML's terms.
  • R10 is amended: BioPAX joins the library service in deploy/java-library/.

From #750's review:

  • T069 build: the service's pom sets kotlin.version to the library's. Spring Boot 2.7 would otherwise bring Kotlin's standard library down to 1.6.21. The jar check now also covers biopax3-exporter-lib, kotlin-stdlib and paxtools-core.
  • T069 memory and time:
    • Measured at the worst case the shares allow: two BioPAX and two SBML conversions at once.
    • The largest pathway's cold conversion must finish within node's 290 s.
    • The service's 404 is a backstop.
  • T072: headers and errors are checked against R6, not against the legacy service's, which differ on purpose.
  • T049:
    • The comparison is against T004's answers, at the graph's release.
    • Content differences go in BioPAX3Exporter's differences.md.
    • What is left goes to BioPAX3Exporter.
    • The one-event file's ontology comment is an expected difference.
  • Corrections: T020 merged as BioPAX3Exporter#79, not Add first names to contributors page #78. The citations for T047 and T048 are fixed. T035 is withdrawn.
  • Other files: the Dependencies and strategy sections, and lines in spec.md, plan.md and checks.md, no longer say SBML alone or T052 alone. T071 is under the render budget. R10 notes that the BioPAX library has no licence file.

🤖 Generated with Claude Code

adamjohnwright and others added 2 commits October 29, 2026 16:12
BioPAX3Exporter's own spec 001 has done T045-T048 and T051, and the species
half of T049 and T050; each is ticked with its pull requests, and T031 is
withdrawn for its ci/compare.py, as T030 was for SBML. T049 and T050 keep the
website's part: the library's per-event files against the legacy service's.
T052 becomes T069-T072, on SBML's terms: the operation in the library
service, node's endpoint, the route, and the check on beta. Research R10 is
amended: BioPAX joins the service here, with node as its client.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…easured

T069's pom sets kotlin.version to the library's (Boot 2.7 would bring
Kotlin's standard library down to 1.6.21), and its jar check covers the
BioPAX library, kotlin-stdlib and paxtools-core. Its memory is measured at
the worst case the shares allow, two BioPAX and two SBML conversions at
once, and the largest's cold time against node's 290 s. T072 checks R6's
headers rather than the legacy service's; T049 compares with T004's
answers at the graph's release and sends what is left to BioPAX3Exporter.
T020 merged as BioPAX3Exporter#79, not #78. T035 is withdrawn, the
citations of T047 and T048 corrected, and the dependencies, strategy,
spec, plan and checks no longer say SBML alone or T052 alone.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@adamjohnwright

Copy link
Copy Markdown
Contributor Author

Replaced by the next PR, which carries this one's commits and the fixes from its review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant