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

Instruction for installation in Windows #216

Merged
merged 4 commits into from
Feb 13, 2024

Conversation

Ettrig
Copy link
Contributor

@Ettrig Ettrig commented Feb 8, 2024

Existing instruction Docker-installation.md is not fully compatible with Windows. This PR adds adds an instruction that is.

Copy link
Member

@kshitijrajsharma kshitijrajsharma left a comment

Choose a reason for hiding this comment

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

It is okay to modify TODO now ! Let me know if there are more questions

docs/Docker-install-windows.md Outdated Show resolved Hide resolved
docker exec -it api bash
```

Once bash is promoted run the following commands, one at a time: ***TODO*** How do we know when bash is promoted? What does "promoted" mean in this context?
Copy link
Member

Choose a reason for hiding this comment

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

When you see new screen saying welcome . its a terminal inside docker container !

- Create environment variable TRAINING_WORKSPACE

Training workspace is the directory where fAIr will store its training files
for example. ***TODO*** Is this about a new checkpoint after fine-tuning? The name implies that this is for temporary files.
Copy link
Member

Choose a reason for hiding this comment

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

This is about a base folder where all information about training will be saved including checkpoints , It will also save preprocessed data & labels.geojson

nvidia-smi
```

You need to see your graphics card details here. ***TODO*** Does this mean that if this command shows details about my GPU, then it is OK for fAIr?
Copy link
Member

Choose a reason for hiding this comment

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

Yes if it shows details about your GPU that means you can use your nvidia graphics to fAIr which is ideal


1. Check your graphics card

It is highly recommended to use a graphics card to run fAIr. It might not work with CPU only. (You can set up and test from bottom of this document). ***TODO*** What does this mean? NVIDIA graphics cards are tested. The bottom of this document is about a tile server, whatever that is. But it is not a GPU test?! ***TODO*** Does this mean that it works for NVIDIA cards but not for cards from other brands?
Copy link
Member

Choose a reason for hiding this comment

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

Tile server is different , this is solely meant for serving your local geotiff to fAIr Read more here : https://developmentseed.org/titiler/

On this doc , When we say NVIDIA cards are tested it means : Developers have tested fAIr with NVIDIA Graphics . fAIr basemodels might not work with other graphics or might work we haven't tested them yet.

It doesn't mean it won't work , it can solely be setup in CPU only too but may not be ideal for trainings ! I am pretty sure to be able to setup basemodels like RAMP in other graphics card will be hell of pain

Copy link
Contributor Author

Choose a reason for hiding this comment

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

At least normally, I put the TODO comments after the text that I think is problematic. Here my problem is with the text "(You can set up and test from bottom of this document)." To me it looks like it is saying "At the bottom of this text there is an instruction for how to set up and test your graphics card." But there is no such instruction at the bottom of this text. So I am clueless, and I think many other readers will also find this difficult to understand.

(I understand the other part.)

Copy link
Member

Choose a reason for hiding this comment

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

You are right , we can include following link for setup https://www.nvidia.com/Download/index.aspx
For testing nvidia-smi will be enough if it returns you have set up your drivers correctly if not either you don't have graphics card or drivers are not setup

@kshitijrajsharma
Copy link
Member

This is nice @Ettrig ! Can you verify this completely works on windows on clean installation ? Once you verify that I can merge this PR !

@Ettrig
Copy link
Contributor Author

Ettrig commented Feb 13, 2024

I followed it verbatim, to the best of my ability. It took 29 minutes. It worked. I did not install Docker. I did not do new setup in OSM.

@kshitijrajsharma kshitijrajsharma merged commit a025ffd into hotosm:master Feb 13, 2024
2 checks passed
jeafreezy pushed a commit to jeafreezy/fAIr that referenced this pull request Sep 18, 2024
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