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

UI update and integration with revamped backend #7

Merged
merged 9 commits into from
Aug 3, 2021

Conversation

mukheshpugal
Copy link
Contributor

No description provided.

@rsnk96
Copy link
Member

rsnk96 commented Jul 22, 2021

Lgtm. Let's merge this after the issue at thermal_base are closed, since there'll be some changes to the styling required here too post that

@renilthomask can you try running this once before we merge it, provide recommendations on improving it? You can test the version available at https://github.com/mukheshpugal/Thermal_Image_Analysis

@mukheshpugal
Copy link
Contributor Author

Added features discussed in yesterday's meeting

@rsnk96
Copy link
Member

rsnk96 commented Jul 26, 2021

@mukheshpugal - now that thermal_base's PR is merged, will the import structure here have to change?

@mukheshpugal
Copy link
Contributor Author

Fixed the import structure.

@rsnk96 rsnk96 changed the title UI update UI update and integration with revamped backend Aug 3, 2021
@rsnk96 rsnk96 merged commit 87b8e72 into detecttechnologies:master Aug 3, 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