cBLUE-dev-team / cBLUE.github.io

cBLUE is a tool to calculate the total propagated uncertainty of bathymetric lidar data.
https://noaa-rsd.github.io/cBLUE.github.io/
GNU Lesser General Public License v2.1
12 stars 1 forks source link

Added Chiroptera, Hawkeye to dropdown #50

Closed fpcorcoran closed 2 years ago

fpcorcoran commented 3 years ago

This PR addresses the need to include support for other lidar systems - namely the Leica Chiroptera 4x and HawkEye 4x systems. In order to complete this PR, we need:

1) Add dropdown menu items for each new system 2) Pass system name to application backend in order to query corresponding lookup table

fpcorcoran commented 3 years ago

This PR addresses the need to include support for other lidar systems - namely the Leica Chiroptera 4x and HawkEye 4x systems. In order to complete this PR, we need:

  1. Add dropdown menu items for each new system
  2. Pass system name to application backend in order to query corresponding lookup table

Note: DO NOT MERGE YET

fpcorcoran commented 2 years ago

Ok, so I think I managed to connect the front and back ends of the sensor model selection. I don't want to merge this until it's VERY well tested though. Might require some data from different models + known TPU calculations (ideally).

cacwad commented 2 years ago

I would like to use this for the Chiroptera and Hawkeye 4x systems. Is this ready to merge or are there still needed updates? Thanks! @fpcorcoran btw this is Colin Cooper

fpcorcoran commented 2 years ago

I would like to use this for the Chiroptera and Hawkeye 4x systems. Is this ready to merge or are there still needed updates? Thanks! @fpcorcoran btw this is Colin Cooper

Hi Colin. Please do NOT merge yet. We are still working out some major bugs. Feel free to use this branch and report any issues/bugs though. Thanks!

fpcorcoran commented 2 years ago

@cacwad. I fixed the bugs in the multithreading so you should be able to consider this branch reliable for Chiroptera 4X. I'm still working on getting the correct beam divergence for Hawkeye, so that feature isn't available yet. Once I have that information and I can make the proper changes I'll merge this branch.

As always if you get any major errors with Chiroptera let me know.

--Forrest

fpcorcoran commented 2 years ago

I removed Hawkeye 4X from the options menu for the time being. Once the correct beam divergence is known for that system it can be added back in easily. Considering this branch stable now and merging to master.