Skip to content
Snippets Groups Projects

Point LALSuite wheels to solar_system_ephemerides package for ephemeris files

Merged Karl Wette requested to merge (removed):lalpulsar-ephem-pypi into master
All threads resolved!

Description

This MR remove the LALPulsar ephemeris files from the LALSuite wheels (PyPI packages). Instead these files are sourced from the solar_system_ephemerides package. People who need these files can run

pip install lalsuite[lalpulsar]

to install them with LALSuite.

API Changes and Justification

Backwards Compatible Changes

  • This change does not modify any class/function/struct/type definitions in a public C header file or any Python class/function definitions
  • This change adds new classes/functions/structs/types to a public C header file or Python module

Backwards Incompatible Changes

  • This change modifies an existing class/function/struct/type definition in a public C header file or Python module
  • This change removes an existing class/function/struct/type from a public C header file or Python module

If any of the Backwards Incompatible check boxes are ticked please provide a justification why this change is necessary and why it needs to be done in a backwards incompatible way.

Review Status

cc @matthew-pitkin @david-keitel @adam-mercer

Edited by Karl Wette

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • David Keitel
  • Karl Wette added 1 commit

    added 1 commit

    • c5911eb8 - Point LALSuite wheels to solar_system_ephemerides package for ephemeris files

    Compare with previous version

  • Karl Wette changed the description

    changed the description

  • Karl Wette resolved all threads

    resolved all threads

  • Author Maintainer

    @adam-mercer This should be ready for your approval. Once merged it should save ~18 MB from the PyPI packages.

  • Karl Wette changed the description

    changed the description

  • Adam Mercer approved this merge request

    approved this merge request

  • Karl Wette mentioned in commit de270339

    mentioned in commit de270339

  • merged

  • Please register or sign in to reply
    Loading