Skip to content

SNOW-2094085: Limit supported range for snowpak-ml-python in modin dependency#3338

Merged
sfc-gh-nkumar merged 1 commit intomainfrom
nkumar-SNOW-2094085-doc-fix
May 7, 2025
Merged

SNOW-2094085: Limit supported range for snowpak-ml-python in modin dependency#3338
sfc-gh-nkumar merged 1 commit intomainfrom
nkumar-SNOW-2094085-doc-fix

Conversation

@sfc-gh-nkumar
Copy link
Copy Markdown
Contributor

  1. Which Jira issue is this PR addressing? Make sure that there is an accompanying issue to your PR.

    Fixes SNOW-2094085

  2. Fill out the following pre-review checklist:

    • I am adding a new automated test(s) to verify correctness of my new code
      • If this test skips Local Testing mode, I'm requesting review from @snowflakedb/local-testing
    • I am adding new logging messages
    • I am adding a new telemetry message
    • I am adding new credentials
    • I am adding a new dependency
    • If this is a new feature/behavior, I'm adding the Local Testing parity changes.
    • I acknowledge that I have ensured my changes to be thread-safe. Follow the link for more information: Thread-safe Developer Guidelines
    • If adding any arguments to public Snowpark APIs or creating new public Snowpark APIs, I acknowledge that I have ensured my changes include AST support. Follow the link for more information: AST Support Guidelines
  3. Please describe how your code solves the related issue.

    Error in Build Doc ci job

    docs: install_deps> bash ./scripts/tox_install_cmd.sh sphinx './.[modin-development]'
    error: resolution-too-deep
    sphinx ./.[modin-development]

× Dependency resolution exceeded maximum depth
╰─> Pip cannot resolve the current dependencies as the dependency graph is too complex for pip to solve efficiently.

This PR fixes the issue by limiting the supported range for snowflake-ml-python in the modin dependency.

@sfc-gh-nkumar sfc-gh-nkumar requested review from a team as code owners May 7, 2025 18:51
@sfc-gh-nkumar sfc-gh-nkumar added the NO-CHANGELOG-UPDATES This pull request does not need to update CHANGELOG.md label May 7, 2025
@github-actions github-actions bot added snowpark-pandas and removed NO-CHANGELOG-UPDATES This pull request does not need to update CHANGELOG.md labels May 7, 2025
@sfc-gh-nkumar sfc-gh-nkumar added NO-PANDAS-CHANGEDOC-UPDATES This PR does not update Snowpark pandas docs NO-CHANGELOG-UPDATES This pull request does not need to update CHANGELOG.md labels May 7, 2025
@sfc-gh-nkumar sfc-gh-nkumar merged commit 83c4d3e into main May 7, 2025
88 of 97 checks passed
@sfc-gh-nkumar sfc-gh-nkumar deleted the nkumar-SNOW-2094085-doc-fix branch May 7, 2025 19:32
@github-actions github-actions bot locked and limited conversation to collaborators May 7, 2025
@sfc-gh-snowflakedb-snyk-sa
Copy link
Copy Markdown

sfc-gh-snowflakedb-snyk-sa commented May 7, 2025

🎉 Snyk checks have passed. No issues have been found so far.

security/snyk check is complete. No issues have been found. (View Details)

license/snyk check is complete. No issues have been found. (View Details)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

NO-CHANGELOG-UPDATES This pull request does not need to update CHANGELOG.md NO-PANDAS-CHANGEDOC-UPDATES This PR does not update Snowpark pandas docs snowpark-pandas

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants