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

Request - Add min and max depth to flat #8576

Open
1 of 10 tasks
happiah-madson opened this issue Feb 27, 2025 · 6 comments
Open
1 of 10 tasks

Request - Add min and max depth to flat #8576

happiah-madson opened this issue Feb 27, 2025 · 6 comments
Labels
Curatorial Search Function-SearchOrDownload Priority-Normal (Not urgent) Normal because this needs to get done but not immediately.

Comments

@happiah-madson
Copy link

happiah-madson commented Feb 27, 2025

Help us understand your request (check below):

  • search or data request (help with SQL or provide the specific fields you are trying to retrieve for your collection)
  • add a new Github user to the Arctos Users team
  • a new barcode series
  • a new term for data entry or management
  • a change or new report or label
  • update to an Agent (split, merge, or other type of change)
  • bulkload changes (to Agents, Identifiers, etc that are not already available as a bulkloading tool)
  • new feature or otherwise a good idea to make Arctos even better
  • quarantine taxon name and suggested replacement (please supply links)
  • other

Describe what you're trying to do

Per #8524, I am seeking to add some fields to flat. Elevation fields are part of flat, but depth fields are not. Would it be possible to add locality_min_depth and locality_max_depth and locality_depth_units? I actually don't need all 3, I just want the min and max depths and then the units can be there, too? IDK what happens magically in the background.

@dustymc @mkoo

@happiah-madson happiah-madson changed the title Request - Request - Add min and max depth to flat Feb 27, 2025
@happiah-madson happiah-madson added Priority-Normal (Not urgent) Normal because this needs to get done but not immediately. Function-SearchOrDownload Curatorial Search labels Feb 27, 2025
@dustymc
Copy link
Contributor

dustymc commented Feb 27, 2025

I think this needs significant additional clarification.

I actually don't need all 3

The concatenation is in flat


arctosprod@arctos>> select json_locality[0] ->'depth' from flat where guid='DMNS:Inv:26912';
     ?column?     
------------------
 "19000-19000 ft"

It's not clear to me if that's sufficient for the user (can we ask them?), or if they need to be some other structure (or what that might be - is this request implicitly for the 'best' locality only, or ?? ), or ????????

locality_min_depth and locality_max_depth and locality_depth_units

None of those exist in this context.

@mkoo
Copy link
Member

mkoo commented Feb 27, 2025

I think what's being said here is that already exists @happiah-madson
HOWEVER it is not available in the options in the customized Results (as far as I can see-- currently JSON wrapped in Locality). So I think the request is to have depth as an option for viewing and seeing in the summarize results etc etc

@dustymc
Copy link
Contributor

dustymc commented Feb 27, 2025

currently JSON wrapped in Locality

yes

depth as an option for viewing and seeing

That's technically plausible as long as the need is the depth from the (one) 'selected' locality, but I'll need details. Costs could be:

  1. However many columns this involves will not be available for later expansion (we're currently using ~15% of the capacity)
  2. Some processing (minimal since we're already pulling data from the relevant table)
  3. Some storage (I think little impact)

@dustymc dustymc added this to the Needs Discussion milestone Feb 28, 2025
@happiah-madson
Copy link
Author

I think this needs significant additional clarification.

My apologies, I should have used many different words.

HOWEVER it is not available in the options in the customized Results (as far as I can see-- currently JSON wrapped in Locality). So I think the request is to have depth as an option for viewing and seeing in the summarize results etc etc

Yes, this is exactly what I was trying to communicate, but did so very poorly.

That's technically plausible as long as the need is the depth from the (one) 'selected' locality

Yes. I assume that's the same way that it is done for elevation? As a user, I would assume this would work the same way.

@happiah-madson
Copy link
Author

Ha! I found a better way to ask my question, I think:

Can filtered_flat.min_depth_in_m and filtered_flat.max_depth_in_m be added to the search result via the customize profile bit?

Image

@sharpphyl
Copy link

Yes, it would be very helpful to have the Depth option available in the customize options (especially Results) the same way Elevation can be selected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Curatorial Search Function-SearchOrDownload Priority-Normal (Not urgent) Normal because this needs to get done but not immediately.
Projects
None yet
Development

No branches or pull requests

4 participants