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

Typeguard requirement is for old version #2043

Closed
jakeBass opened this issue Dec 4, 2023 · 5 comments
Closed

Typeguard requirement is for old version #2043

jakeBass opened this issue Dec 4, 2023 · 5 comments
Assignees
Labels
enhancement New feature or request

Comments

@jakeBass
Copy link

jakeBass commented Dec 4, 2023

Hi folks! It appears that Ax still has pinned the typeguard requirement to an ancient version (from Dec 2021). It's getting increasingly hard to align this requirement with requirements from other OSS for a more modern version of typeguard. Are there any plans to update the typeguard requirement, or to remove it?

danielcohenlive pushed a commit to danielcohenlive/Ax-1 that referenced this issue Dec 5, 2023
Summary: Attempt to solve/mitigate facebook#2043

Differential Revision: D51853729
@danielcohenlive
Copy link

Hi @jakeBass I'm working on seeing if we can upgrade, but there are some internal complications. In the meantime, if you run pip install typeguard==4.1.5 after installing ax, the only problem will be a bunch of false warnings in the form of

ax.utils.common.kwargs: `<SOME_CLASS>` expected argument `SOME_ARG` to be of type SOME_CLASS. Got SOME_CLASS(...).

danielcohenlive pushed a commit to danielcohenlive/Ax-1 that referenced this issue Dec 5, 2023
Summary:
Attempt to solve/mitigate facebook#2043

We stopped pinning typeguard to 2.13.3, but gracefully failing if it doesn't have that version manually changes the version.  This is important so ax can install in a standard way without conflicting with user's other dependencies.

Differential Revision: D51853729
danielcohenlive pushed a commit to danielcohenlive/Ax-1 that referenced this issue Dec 5, 2023
)

Summary:

Attempt to solve/mitigate facebook#2043

We stopped pinning typeguard to 2.13.3, but gracefully failing if it doesn't have that version manually changes the version.  This is important so ax can install in a standard way without conflicting with user's other dependencies.

Reviewed By: mpolson64

Differential Revision: D51853729
danielcohenlive pushed a commit to danielcohenlive/Ax-1 that referenced this issue Dec 5, 2023
)

Summary:

Attempt to solve/mitigate facebook#2043

We stopped pinning typeguard to 2.13.3, but gracefully failing if it doesn't have that version manually changes the version.  This is important so ax can install in a standard way without conflicting with user's other dependencies.

Reviewed By: mpolson64

Differential Revision: D51853729
facebook-github-bot pushed a commit that referenced this issue Dec 6, 2023
Summary:
Pull Request resolved: #2044

Attempt to solve/mitigate #2043

We stopped pinning typeguard to 2.13.3, but gracefully failing if it doesn't have that version manually changes the version.  This is important so ax can install in a standard way without conflicting with user's other dependencies.

Reviewed By: mpolson64

Differential Revision: D51853729

fbshipit-source-id: b131d2751474ca3d9303853119744e5148fe8cec
@danielcohenlive danielcohenlive added the enhancement New feature or request label Dec 6, 2023
@danielcohenlive
Copy link

@jakeBass the previous comment still applies, but #2044 will simplify this for you next release.

@jakeBass
Copy link
Author

jakeBass commented Dec 6, 2023

Thanks so much @danielcohenlive, really appreciate the quick response!

Do you know when 0.3.6 will be released? Or is there somewhere I can follow along?

@danielcohenlive
Copy link

@jakeBass sorry, I thought I responded Friday, but we just released 0.3.6 2 days ago. I'm going to close the issue for now but feel free to reopen or open a new one if you have further questions.

@jakeBass
Copy link
Author

That's wonderful, thanks so much again @danielcohenlive!

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

No branches or pull requests

3 participants