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

setting latest tag for COLMAP_GIT_COMMIT instead of dev or main #2432

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

behnamasadi
Copy link

If the user doesn't specify any commit id (COLMAP_GIT_COMMIT) the Dockerfile set COLMAP_GIT_COMMIT to the latest tag, i.e. 3.9.1, otherwise it uses the user provided one
Example
docker build --build-arg CUDA_ARCHITECTURES=75 .
docker build --build-arg COLMAP_GIT_COMMIT=main --build-arg CUDA_ARCHITECTURES=75 .

…ockerfile set `COLMAP_GIT_COMMIT` to the latest tag, i.e. 3.9.1, otherwise it uses the user provided one

Example
`docker build --build-arg CUDA_ARCHITECTURES=75 .`
`docker build --build-arg COLMAP_GIT_COMMIT=main --build-arg CUDA_ARCHITECTURES=75 .`
@behnamasadi behnamasadi changed the title setting latest tag for COLMAP_GIT_COMMIT) instead of dev setting latest tag for COLMAP_GIT_COMMIT instead of dev or main Feb 20, 2024
Copy link
Contributor

@ahojnnes ahojnnes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thx

@behnamasadi
Copy link
Author

Hello Johannes @ahojnnes,
The docker build failed due to credentials, I don't have user and password for that, can you please have a look or tell me what can I do to pass it? Regards

@ahojnnes ahojnnes enabled auto-merge (squash) February 24, 2024 10:21
@ahojnnes
Copy link
Contributor

@behnamasadi Seems like there is also still some other issue? The CI build still fails.

@behnamasadi
Copy link
Author

Sorry for my late reply, will investigate it this weekend, thanks for your patience

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.

None yet

3 participants