search

arXiv / arxiv-submission-ui

User interface of NG submit system.
MIT License
2 stars 6 forks source link

Request PDF from FileManager for PDF-only submission, not from compiler #128

Closed bdc34 closed 5 years ago

bdc34 commented 5 years ago

This change to process.py makes it so a PDF submission will get the PDF to preview from the FileManager, not the compiler.

While change this fixes the issue, it may be better to have the Compiler service respond with a redirect to the FileManager when the product is requested. This would prevent downstream services from containing any knowledge about how different types of submissions are turned into PDFs.

@DavidLFielding Is it correct to handle PDFTEX submission types this way?

I plan to squash this due to the WIP commit.

DavidLFielding commented 5 years ago

I will swing back to review this change. We should probably double check the document flow/path in the legacy system. I seem to recall pointing out that the PDF-only stamp happens in the AutoTeX module but I would need to doublecheck. If this is the case, it might make sense for the request to go to the compiler which fetches the PDF from file manager and stamps it. Or one could duplicate stamping code elsewhere.

bdc34 commented 5 years ago

PDFTEX indicates a compiled-from-source paper such that the generated PDF is located in the compilation service.

When I attempt to upload and compile a PDFTEX submission (1906.05476) I'm able to retrieve a PDF when I click on the View PDF button but the PDF is of some other paper. Might be interesting to determine what is going on here.

@DavidLFielding I've removed PDFTEX from this PDF from file manager type of preview.