Export file reading functionality - #190
Open
johncmerfeld wants to merge 3 commits into
Open
Conversation
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
ejoranlienea
approved these changes
Jul 7, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Provide a file reader that can be imported and used with compiled Earthmover configs to turn an input path into a dataframe. Runway needs this in order to manipulate source files consistently with Earthmover's logic.
read_compiled_sourcetakes in a named Earthmover source (i.e. not a path, but something likeinput) and a compiled Earthmover file and returns a dataframe (the caller can ask for an unmaterialized Dask dataframe). Requiring that the config file be compiled makes the code lighter and more predictable, and seems like a reasonable setup step, especially for our intended use case.Also exposes
read_file, which takes in a path and a dict of Earthmover config. This is the primitive thatread_compiled_sourceuses. We could leave it out of the init file but callers would still be able to use it if they wanted to.Open to other file naming / organization.
You can test the functionality out with something like this: