Skip to content

[WIP] Updating the API client#1982

Open
esoteric-ephemera wants to merge 10 commits intomasterfrom
modernize
Open

[WIP] Updating the API client#1982
esoteric-ephemera wants to merge 10 commits intomasterfrom
modernize

Conversation

@esoteric-ephemera
Copy link
Collaborator

Completed:

  • Reorganize client into modules for easier editing
  • Replace some quasi-deprecated packages in favor of those already in the stack:
    • ujson in favor of orjson (req'd by pymatgen)
  • Align the syntaxes of the main API client and the MPC client (i.e., consistent naming of class kwargs)

Planned:

  • Updating client structure + lessening dependency stack to be more future proof
  • Pydantic models for returned data types? Possibly just pandas

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