Skip to content

feat: peaks to nmrium endpoint#100

Merged
vcnainala merged 4 commits intodevelopmentfrom
feat/peaks-to-nmrium-endpoint
Feb 18, 2026
Merged

feat: peaks to nmrium endpoint#100
vcnainala merged 4 commits intodevelopmentfrom
feat/peaks-to-nmrium-endpoint

Conversation

@vcnainala
Copy link
Member

No description provided.

Add detailed descriptions, request/response examples, and proper HTTP
response codes across all routers. Add OpenAPI tag metadata and a rich
API description to the FastAPI app configuration.
Add POST /spectra/parse/publication-string endpoint that resurrects an
NMR spectrum from an ACS-style publication string. The endpoint accepts
the publication string as a plain text body and invokes the nmr-cli
parse-publication-string command via Docker exec.

Uses StreamingResponse with Content-Disposition attachment header to
prevent Swagger UI from hanging on the large spectrum JSON response.
Also improves OpenAPI docs for existing spectra endpoints.
Add a new POST /spectra/parse/peaks endpoint that converts a list of
NMR peaks (chemical shift, intensity, width) into a full NMRium-compatible
spectrum object. The endpoint delegates to a new `peaks-to-nmrium` CLI
command in nmr-cli which uses `peaksToXY` from nmr-processing to generate
the simulated 1D spectrum data.
@vcnainala
Copy link
Member Author

fixes #96

@vcnainala vcnainala merged commit 060a27e into development Feb 18, 2026
1 check passed
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.

2 participants

Comments