Skip to content

Purge dim name and distutils #1321

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

calad0i
Copy link
Contributor

@calad0i calad0i commented Jul 2, 2025

Description

Get rid of dim names in the whole repository, based on the DA PR.

Type of Change

  • Cleaning up

Tests

N/A

Checklist

  • all

@calad0i calad0i added please test Trigger testing by creating local PR branch and removed please test Trigger testing by creating local PR branch labels Jul 2, 2025
@calad0i calad0i force-pushed the purge_dim_name branch 3 times, most recently from b3d3cc7 to 1d9b0f5 Compare July 2, 2025 21:57
@calad0i calad0i added please test Trigger testing by creating local PR branch and removed please test Trigger testing by creating local PR branch labels Jul 2, 2025
@calad0i calad0i added please test Trigger testing by creating local PR branch and removed please test Trigger testing by creating local PR branch labels Jul 3, 2025
@vloncar
Copy link
Contributor

vloncar commented Jul 3, 2025

Now you're adding unrelated commits to this PR 😄

@calad0i
Copy link
Contributor Author

calad0i commented Jul 3, 2025

Now you're adding unrelated commits to this PR 😄

Let me move the distutil one to separate PR? It is deprecated and breaks on python 312 and 313.

@JanFSchulte
Copy link
Contributor

I think that was about the change to the bit-exact algorithm which probably needs to go the DA PR :-D

@calad0i calad0i changed the title Purge dim name Purge dim name and distutils Jul 4, 2025
@calad0i calad0i added please test Trigger testing by creating local PR branch and removed please test Trigger testing by creating local PR branch labels Jul 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
please test Trigger testing by creating local PR branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants