Feedback Paper Submission#37
Conversation
- Extension of author list, courtesy of @ndaelman-hu Changes to be committed: new file: paper/draft.tex new file: paper/parse-patrol.png new file: paper/ref.bib
-- TODO: add spepcifics on the solutions we provide + results observed Changes to be committed: modified: draft.tex
-- TODO: update statement on future work Changes to be committed: modified: draft.tex
Changes to be committed: modified: draft.tex
Changes to be committed: modified: draft.tex
Changes to be committed: modified: draft.tex
-- Add svg file for editing with InkScape Changes to be committed: modified: parse-patrol.png new file: parse-patrol.svg
-- AI generate overview of Git PR authorship Changes to be committed: modified: draft.tex new file: git-contriubtions.tex
- Save edits InkScape Changes to be committed: modified: draft.tex modified: parse-patrol.svg
|
The author contributions are missing atm. It is one of the manuscript requriements. |
They should be at the bottom of \section*{Author Contributions}
\textbf{N.D.}: Conceptualization, Software, Original Code Draft, Visualization, Writing - Editing,
\textbf{C.E.}: Conceptualization, Software, Visualization, Video Editing, Writing - Original Manuscript Text Draft, Figure, Editing
\textbf{R.M.}: Implementation asynchronous servers, Extension Testing Infrastructure - Conceptualization Revision
\textbf{S.K.}: Rollout Parser Support |
yeah, it was the overleaf version where it was missing. Just saw that 1 min ago. Sorry 🙈 |
|
I really like the new version of the manuscript. It has much more soul than the very technical version, that I had drafted 🤣 There are just
|
How about: "Parse-Patrol combines various community parsers into a single cohesive interface for AI and human developer alike." |
- Add empty citation markers Changes to be committed: modified: draft.tex
|
this sounds good to me! |
QuantumChemist
left a comment
There was a problem hiding this comment.
My idea how to cite the MCP server "original work"
- Elaborate on the community issues and impact -- TODO: add paragraph on AI in the intro - Swap "dependency" for the more Pythonic "module" Changes to be committed: modified: draft.tex
Added references for Python Package Index and MCP Servers Registry.
|
btw I would be in favour of merging this into main in the end to have our contribution to the manuscript as part of the repo 😄 |
Updated references to MCP Registry and PyPi with hyperlinks.
…-submission - Correct typos Changes to be committed: modified: draft.tex
Changes to be committed: modified: draft.tex
- Use figure terminology - Broaden title to include scientific parsers in general Changes to be committed: modified: draft.tex
- Unify the figure text box between both branches Changes to be committed: modified: draft.tex modified: parse-patrol.png
-- fold bulletpoints back into text -- remove redundant phrases - Extend coverage of agentic pitfalls Changes to be committed: modified: draft.tex
- Merge paragraph on database and parser communities - Revise phrasing
- Add manual polishing Changes to be committed: modified: draft.tex
- Extend caption along the entire page width - Change type setting caption - Trim paragraph Changes to be committed: modified: draft.tex
avoid naming specific software, package, db support and focus on the concept instead.
- Remove unused references Changes to be committed: modified: draft.tex modified: ref.bib
Clarified language regarding LLMs and MCP, improving readability and precision in the explanation of parser infrastructure and its integration with AI tools.
RubelMozumder
left a comment
There was a problem hiding this comment.
It looks really great.
-- clarify phrasing on I/O dependency, how a lack of docs is handled, why we introduce MCP servers as modules too -- add reference to MCP intro -- explain where the Git link points to and which version was originally submitted Changes to be committed: modified: draft.tex modified: ref.bib
Changes to be committed: modified: draft.tex
- Shorten text to meet length requirements by removing redundant phrasing - Replace "outsources" with "sidesteps" and "input/output" with "source/target" - Add "however" transition and clarify component registration flow - Refine dual-mode explanation: introduce tool schemas and explain interface evolution - Standardize terminology: "predefined prompts", capitalize mode names, remove "hosts" Changes to be committed: modified: draft.tex
- Correct caption: capitalization, phrasing Upper branch, boldness sections - Correct phrasing figure: "Parser Output conform User-Defined Schemas" Changes to be committed: modified: draft.tex modified: parse-patrol.png modified: parse-patrol.svg
|
Okay, the commit marked at this timestamp is the submitted version. Unless prompted by FAIRmat, this version does NOT get modified further. |
Updated the README to include a link for the Model Context Protocol, a suggested in #37 .
This is just a tool for getting feedback. It does NOT get merged into
main.