Skip to content

io.ascii read dtypes, because why not (fixes #3534)#3611

Merged
ThomasLecocq merged 9 commits intomasterfrom
feat_ascii_slist
Dec 24, 2025
Merged

io.ascii read dtypes, because why not (fixes #3534)#3611
ThomasLecocq merged 9 commits intomasterfrom
feat_ascii_slist

Conversation

@ThomasLecocq
Copy link
Contributor

@ThomasLecocq ThomasLecocq commented Aug 27, 2025

io.ascii now allows the "dtype" column to be INT* or FLOAT* matching numpy dtypes.

Why was it initiated? Any relevant Issues?

fixes #3534

PR Checklist

  • Correct base branch selected? master for new features, maintenance_... for bug fixes
  • If the PR is making changes to documentation, docs pages can be built automatically.
    Just add the build_docs tag to this PR.
    Docs will be served at docs.obspy.org/pr/{branch_name} (do not use master branch).
    Please post a link to the relevant piece of documentation.
  • All tests still pass.
  • Any new features or fixed regressions are covered via new tests.
  • Significant changes have been added to CHANGELOG.txt .
  • Add the yellow ready for review label when you are ready for the PR to be reviewed.

Now allows the "dtype" column to be INT* or FLOAT* (startswith("INT") | startswith("FLOAT"))
@ThomasLecocq ThomasLecocq added this to the 1.5.0 milestone Aug 27, 2025
@ThomasLecocq
Copy link
Contributor Author

Remaining failed tests are unrelated (shapefile).

Ready ?

@ThomasLecocq ThomasLecocq added ready for review PRs that are ready to be reviewed to get marked ready to merge build_docs Docs will be automatically built and deployed in github actions on pushes to the PR labels Aug 27, 2025
@ThomasLecocq
Copy link
Contributor Author

as such, the code (and tests) don't verify that the items read can actually be casted to the right format... but yay...

@ThomasLecocq ThomasLecocq requested a review from megies December 2, 2025 10:27
@ThomasLecocq ThomasLecocq merged commit 0897b68 into master Dec 24, 2025
31 of 32 checks passed
@ThomasLecocq ThomasLecocq deleted the feat_ascii_slist branch December 24, 2025 10:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build_docs Docs will be automatically built and deployed in github actions on pushes to the PR io.ascii ready for review PRs that are ready to be reviewed to get marked ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Reading integer slist format

1 participant