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

[FEA] more docs, image cleanup, module renames #313

Merged
merged 34 commits into from
Oct 18, 2021

Conversation

trxcllnt
Copy link
Collaborator

@trxcllnt trxcllnt commented Oct 15, 2021

  • Rename modules @nvidia/ -> @rapidsai/.
  • Keep VSCode Remote Containers server in .vscode/server in the workspace.
  • Add fixguid to the devel container so it can be launched w/ a different UID than it was built with.
  • Move the .cache dir up one level so we can do COPY modules modules in the build
  • Remove the source + build after npm pack to reduce the intermediate packaging container image size.
  • Split out the merge PR github action into separate jobs to maximize concurrency in CI, add custom image builder action to reduce duplication.
  • Fix image tags to remove extra -:
    - ghcr.io/rapidsai/node:21.12.00-runtime-node-16.10.0-cudagl11.4.2-ubuntu20.04-main-amd64
    + ghcr.io/rapidsai/node:21.12.00-runtime-node16.10.0-cudagl11.4.2-ubuntu20.04-main-amd64
  • Add more docs on:
    • Setting up a local development environment
    • Starting and connecting to the devel container from VSCode
    • Which images are available from the github container registry
    • How to use and/or extend the runtime images for app code
    • Launching the preinstalled demos in the demos container

@nicolasgere
Copy link

Do we know when a package would be publish? We would like to try this package to replace our headless-gl setup

@AjayThorve AjayThorve mentioned this pull request Oct 16, 2021
@trxcllnt trxcllnt force-pushed the fea/keep-vscode-remote-server branch from 7112781 to b3e7e72 Compare October 17, 2021 19:51
@trxcllnt trxcllnt merged commit 1cba9f8 into rapidsai:main Oct 18, 2021
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.

2 participants