transition from paperspace to gradient #103841
Closed
+162
−47
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ofborg
bot
added
6.topic: python
8.has: clean-up
8.has: package (new)
freezeboy
force-pushed
the
fix-paperspace
branch
from
bd5b388 to
9282ac6
Compare
|
Now hyperopt has been updated, this PR should be building correctly |
thoughtpolice
requested changes
Nov 16, 2020
There was a problem hiding this comment.
Thanks, but you'll definitely need to retarget this to be based on the staging branch instead, because it will effectively rebuild the world. We can't merge this until then, so I'm just putting in a soft Request Changes — other than that your changes look fine to me, but someone like Jon or Frederik should take a look.
Note that GitHub handles moving PRs onto new base branches very poorly (so poorly, in fact, that I cannot even describe how to do it properly; everybody messes it up completely for a reason, I suppose!) and it often ends up pinging hundreds of maintainers in the process, so you might actually want to just close this PR and open a new one...
|
Wouldn't it be enough to rebase on staging on my side then push force here? |
freezeboy
force-pushed
the
fix-paperspace
branch
from
9282ac6 to
98a93d3
Compare
freezeboy
requested review from
alyssais,
Ericson2314,
matthewbauer,
stigtsp and
ttuegel
as code owners
|
Ok rebased on staging, looks good to me in terms of PR content It seems numpy world-rebuild was already part of staging, so it vanished for this PR |
ofborg
bot
added
6.topic: nixos
ofborg
bot
added
10.rebuild-darwin: 501+
10.rebuild-darwin: 5001+
10.rebuild-linux: 501+
10.rebuild-linux: 5001+
10.rebuild-darwin: 11-100
10.rebuild-linux: 11-100
and removed
10.rebuild-darwin: 11-100
10.rebuild-linux: 11-100
6.topic: GNOME
|
Result of 13 packages failed to build:
45 packages built:
|
|
|
||
| # tries to use /homeless-shelter to mimic container usage, etc | ||
| doCheck = false; | ||
|
|
|
|
||
| # tries to use /homeless-shelter to mimic container usage, etc | ||
| doCheck = false; | ||
|
|
|
Please either execute tests for those packages or add |
|
Result of 12 packages failed to build:
46 packages built:
|
|
Result of 4 packages marked as broken and skipped:
11 packages failed to build:
41 packages built:
|
SuperSandro2000
added
the
2.status: merge conflict
|
I marked this as stale due to inactivity. → More info |
stale
bot
added
the
2.status: stale
ghost
closed this
This pull request was closed.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Motivation for this change
This PR contains some changes to resolve the reverse dependencies broken by the automatic PR #103814.
Upstream paperspace has worked to rewrite its SDK which involves an update of the complete stack. The dangerous this here is this update includes
wheelandnumpy(even though for this one, it is a revision update).If you want to run nixpkgs-review, prepare for a big build I guess, I compiled the packages inside this PR only.
This will not compile before PR #103814 is merged.
Things done
sandboxinnix.confon non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"./result/bin/)nix path-info -Sbefore and after)