Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deprecation and/or removal of resdata applications #1035

Open
berland opened this issue Feb 14, 2025 · 0 comments
Open

Deprecation and/or removal of resdata applications #1035

berland opened this issue Feb 14, 2025 · 0 comments

Comments

@berland
Copy link
Contributor

berland commented Feb 14, 2025

The applications in resdata/applications/resdata installed through setup.py serves various purposes. Some of these should be deprecated and/or possibly removed:

  • CF_dump: Any user of this is better of using res2df and its rft module, that extracts the same information and more
  • convert.x: This might have some use cases, but could be deprecated in favour of a recommentation to use resfo instead.
  • rd_pack.x/rd_unpack.x: This is used by subscript's restartthinner. If subscript uses resfo instead, we can deprecate it.
  • grdecl_grid: Should be deprecated or outright removed. It parses input decks, but very poorly. And then it does nothing.
  • grdecl_test: Times the loading of a grdecl-file. Limited use for this, deprecate.
  • grid_dump.x/grid_dump_ascii.x: Prints xyz for all 8 corners of every cornerpoint-cell. Has probably limited use-cases. Deprecate.
  • grid_info.x: Will print some simple aggregated numbers for a grid. Attainable in short lines of python from res2df also. Can also compare two grids. for equivalence. Can probably be replaced by res2df and pandas.
  • kw_extract: Can seemingly extract summary vectors (?) from one file and inject it into another file. Very unclear why anyone would like to do that. Propose to deprecate.
  • make_grid: Makes a dummy cartesian grid. Only useful in a test-suite. Deprecate.
  • ri_well_test: Loads a file "similar" to how resinsight is loading it. Would fit as an integration test, but resinsight is not using this codebase any longer so serves no purpose. Remove.
  • segment_info: This data is much better presented by res2df rft. Deprecate.
  • select_test.x: Seemingly a dummy binary that does nothing (could be part of the test suite). Remove.
  • summary.x: The most famous application from resdata, and might still have users. Recommended to use res2df summary instead. Deprecate.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

No branches or pull requests

1 participant