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

Making scrunch compatible with Python 3.11 #450

Merged
merged 1 commit into from
Feb 12, 2024
Merged

Making scrunch compatible with Python 3.11 #450

merged 1 commit into from
Feb 12, 2024

Conversation

shaikh-ma
Copy link
Contributor

@shaikh-ma shaikh-ma commented Jan 23, 2024

As discussed in the issues section of the scrunch repository here, this Pull Request attempts to make scrunch compatible with the Python version 3.11.

The adjustments are made only in the modules, dataset.py, order.py & expressions.py.

These include version specific checks for importing methods from the collection module and accessing of values of the ast.Subscript object.

Python version Test Result
2.7 image
3.6 image
3.11 image

@alexbuchhammer
Copy link

Thanks @shaikh-ma!

@shaikh-ma
Copy link
Contributor Author

Hi @jjdelc,

As discussed, here's the Pull Request which supports 3.6 & 3.11.

Please could verify it on your end and let us know if it's good to be merged. :)

@jamesrkg
Copy link

jamesrkg commented Feb 9, 2024

Hi @jjdelc,

Can you please let us know how/when this PR can progress? This is a significant impediment to our efforts to update our code base at the moment. If there is anything we can do to further help resolve the PR, please just let us know.

Cheers,
Jamie

@jjdelc
Copy link
Contributor

jjdelc commented Feb 9, 2024

I'll look at this now

@jjdelc jjdelc merged commit e75284c into Crunch-io:master Feb 12, 2024
1 of 4 checks passed
@jjdelc
Copy link
Contributor

jjdelc commented Feb 12, 2024

Thanks @shaikh-ma !!

@shaikh-ma
Copy link
Contributor Author

Thank you too @jjdelc !!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants