feat(package): Add gridfm-graphkit Nexus package#157
Conversation
romeokienzler
force-pushed
the
add_gridfm_graphkit-0.8.0rc3
branch
from
d792985 to
7234664
Compare
|
@romeokienzler let's wait until a proper release of both gridfm-graphkit and gridfm-datakit are available. |
|
@romeokienzler just to give you a feedback on the rest. I see you have not defined any models with the package. Is this the intended outcome? Note that models are required to have a HuggingFace ID |
|
@romeokienzler please merge with the latest master and then run git restore --source origin/main -- uv.lock
uv lockThen add the new uv.lock and commit. Make sure the pre-commit hooks are installed becuase they will re-generate the requirement files. |
|
@romeokienzler could you please have a loot at this PR? See my latest comment for instructions. Thanks. |
romeokienzler
force-pushed
the
add_gridfm_graphkit-0.8.0rc3
branch
from
7234664 to
db2e966
Compare
christian-pinto
left a comment
christian-pinto
left a comment
There was a problem hiding this comment.
If something is not working on your PR, just send a message here and we'll see what can be done. This is related to the changes i to the secrets file as well as the constraint-dependencies.
| "results": { | ||
| "requirements-ecosystem.txt": [ | ||
| { | ||
| "hashed_secret": "655a749bbf386524a927cbd38dd8896628d38d96", | ||
| "is_verified": true, | ||
| "line_number": 575, | ||
| "type": "Secret Keyword", | ||
| "verified_result": null | ||
| } | ||
| ] | ||
| }, |
There was a problem hiding this comment.
please revert this change
There was a problem hiding this comment.
still there. This file should be untouched
Signed-off-by: Romeo Kienzler <romeo.kienzler1@ibm.com>
romeokienzler
force-pushed
the
add_gridfm_graphkit-0.8.0rc3
branch
from
ceedc7e to
c9c61c3
Compare
uv lock produced no changes to the lock file. This means the uv.lock on origin/main already contained everything needed |
Since gridfm-graphkit has no published Hugging Face checkpoints to reference, the nexus.yaml correctly contains only the package.name field with no models: section. All other packages in the repo (bmfm-targets, terratorch, tokamind, terrakit) are in the same situation — none of them declare models in their nexus.yaml either. Did I get something wrong here? |
|
Also can't access the pipelines, so no idea what the problem is |
| "results": { | ||
| "requirements-ecosystem.txt": [ | ||
| { | ||
| "hashed_secret": "655a749bbf386524a927cbd38dd8896628d38d96", | ||
| "is_verified": true, | ||
| "line_number": 575, | ||
| "type": "Secret Keyword", | ||
| "verified_result": null | ||
| } | ||
| ] | ||
| }, |
There was a problem hiding this comment.
still there. This file should be untouched
terratorch and tokamind both define models inside the models folder within the nexus package. Deinifing models is not mandatory. In your case all is good then. |
Some issues with secrets detection. Revert that change on the secrets file and we try again |
2 participants
Description
Adds the
gridfm-graphkitalgorithm package (version0.8.0rc3) to theecosystemvariant of Algorithm Nexus, with the corresponding Nexus package underpackages/gridfm-graphkit/.Additional Information
gridfm-graphkitdoes not declarevllmas a dependency, so it belongs to theecosystemvariant only.gridfm-graphkit==0.8.0rc3pinsgridfm-datakit==1.0.5rc1(a prerelease) to skip1.0.4's brokengridfm-datakit-winblockmarker. To let this transitive prerelease pin resolve,prerelease = "allow"was added to[tool.uv]. This can be dropped once a stablegridfm-datakit/gridfm-graphkitships.gridfm-datakitis pulled in transitively viagridfm-graphkit— it is not declared as a dedicated dependency.uv.lockandrequirements-ecosystem.txtwere regenerated via the documenteduv add/uv exportsteps.I need help with this PR
N/A — dependency resolution succeeds with
prerelease = "allow"enabled.Related Issues
See gridfm/gridfm-graphkit#81 for the upstream
gridfm-datakitprerelease context.