Skip to content

fix docker compose to setup dashboard#387

Open
prashantpandeygit wants to merge 1 commit intoopenclimatefix:mainfrom
prashantpandeygit:dockerfix
Open

fix docker compose to setup dashboard#387
prashantpandeygit wants to merge 1 commit intoopenclimatefix:mainfrom
prashantpandeygit:dockerfix

Conversation

@prashantpandeygit
Copy link
Contributor

@prashantpandeygit prashantpandeygit commented Dec 18, 2025

Pull Request

Description

the docker build was broken, the docker-compose up --build failed and threw the error attached
Screenshot 2025-12-18 172258

this happens as the docker-compose.yml file overrides the CMD command that is already at the bottom of the DockerFile which is responsible for the allocation of port and streamlit launch command, the change made now help the image to build using the docker-compose.yml and launches the port and streamlit using DockerFile causing no issues

To Reproduce error

  1. clone repo
  2. cd analysis-dashboard
  3. create .env and secrets.toml as per readme
  4. docker-compose up --build

How Has This Been Tested?

rebuilt the image as per the readme, and the streamlit app is ready on the port

If your changes affect data processing, have you plotted any changes? i.e. have you done a quick sanity check?

  • Yes

Checklist:

  • My code follows OCF's coding style guidelines
  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked my code and corrected any misspellings

@prashantpandeygit
Copy link
Contributor Author

@peterdudfield can you have a look at this too? thanks!

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.

1 participant