Skip to content

Conversation

@davidsbatista
Copy link
Collaborator

Related Issues

Proposed Changes:

How did you test it?

  • unit tests + manual verification

Checklist

  • I have read the contributors guidelines and the code of conduct
  • I have updated the related issue with new insights and changes
  • I added unit tests and updated the docstrings
  • I've used one of the conventional commit types for my PR title: fix:, feat:, build:, chore:, ci:, docs:, style:, refactor:, perf:, test: and added ! in case the PR includes breaking changes.
  • I documented my code
  • I ran pre-commit hooks and fixed any issue

@davidsbatista davidsbatista marked this pull request as ready for review July 5, 2025 12:17
@davidsbatista
Copy link
Collaborator Author

@ivyleavedtoadflax the idea here is to remove this big dependency, pandas which was also bringing lots of other stuff, like numpy.

Since we're using pandas only to output a DataFrame, I think outputting a .csv is better, which can easily be loaded into a DataFrame if the user wishes so, and then nervaluate becomes self contained no dependencies.

@ivyleavedtoadflax
Copy link
Collaborator

Great idea, totally agree. Pandas is huge.

@davidsbatista davidsbatista merged commit dcd54d6 into main Jul 14, 2025
3 checks passed
@davidsbatista davidsbatista deleted the remove-pandas branch July 14, 2025 10:57
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.

Removing pandas as a dependency

3 participants