Skip to content
This repository has been archived by the owner on Dec 4, 2024. It is now read-only.

meta data is overwirtten with constant data #6

Open
najtin opened this issue Jul 30, 2021 · 11 comments
Open

meta data is overwirtten with constant data #6

najtin opened this issue Jul 30, 2021 · 11 comments

Comments

@najtin
Copy link

najtin commented Jul 30, 2021

Hi, i just stumbled upon this project so i am not sure if i understand correctly.

To me this seems wrong:
https://github.com/detecttechnologies/thermal_base/blob/main/thermal_base/thermal_base.py#L110-L125

I am going to try to built it myself and report back.

@najtin
Copy link
Author

najtin commented Jul 30, 2021

It seems like someone had at least partial success in reverse engineering the important meta data: https://exiftool.org/forum/index.php?topic=11401.0

@rsnk96
Copy link
Member

rsnk96 commented Jul 30, 2021

Hey @najtin

Yes absolutely, it would be better if we can extract the formula params from the metadata itself for the DJI encoding too. Would you be interested in contributing the same?

@najtin
Copy link
Author

najtin commented Jul 31, 2021

Yes i will take a look into that. Though i have no experience in reverse engineering and it would be a side project for me. :D

@rsnk96
Copy link
Member

rsnk96 commented Aug 2, 2021

Haha well, all the best, hope you figure it out! 🙂

@omritz
Copy link

omritz commented Sep 2, 2021

Hi, Regarding the same issue, DJI released SDK that allows extracting the information from images taken by H20T and XTS but does not yet support M2EA. According to this thread, one of their developers says they will release a version that supports M2EA in the second half of 2021. The formula that DJI used to calculate the temperatures in deg C is different than the one that you should use for FLIR data.

@rsnk96
Copy link
Member

rsnk96 commented Sep 6, 2021

Thanks for sharing @omritz . Will look into integrating that!

@rsnk96
Copy link
Member

rsnk96 commented Oct 5, 2021

Hey @omritz - the DJI SDK has been integrated by @agarwalkunal12 for fetching the raw sensor values 😃

@rsnk96
Copy link
Member

rsnk96 commented Oct 5, 2021

Can you test it out, and share how it compares to using the DJI Thermal Analysis tool?

@omritz
Copy link

omritz commented Dec 29, 2021

Thanks for the update. When I test it with the new M2EA support and adjust the same parameters, the results are not the same as the DJI thermal analysis tool.
For the same image:
DJI Thermal Analysis - max=55.3, min=16.8, avg=35.1
Thermal_base - max=38.2, min=26.7 , avg=32.2

@rsnk96
Copy link
Member

rsnk96 commented Dec 30, 2021

@agarwalkunal12 - any thoughts on this? Could this be because of the assumptions made for the missing metadata?

@nrontsis
Copy link

Perhaps this could be because DJI's SDK is not called when M2EA images are processed by thermal_tools?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants