-
-
Notifications
You must be signed in to change notification settings - Fork 13
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
Feature Request - drop table ctabundance #7691
Comments
@dustymc I'm a bit confused. We have ctabundance which is controlled, but the data you provide look like it's coming from 'abundance' in ctattribute_type - ??? I checked some of the MVZ records and those are from ctattribute_type = 'abundance' So we have two ways of saying the same thing. I'm ok with dropping ctabundance but we need to keep abundance as a record attribute type. Can you send a CSV of values that use ctabundance? |
@ccicero you got it, two things here:
I'm not proposing doing anything with the attribute. I am proposing either
Data are in the original comment. |
@dustymc The data in original comment looks like it's coming from ctattribute_type#abundance - no? The column is ATTRIBUTE_TYPE = 'abundance' and the values are not controlled, so what are the records with data from ctabundance? That's unclear to me. |
As far as I know there are none. |
Hmmm, ok. Then I vote for getting rid of ctabundance and just keeping ctattribute_type#abundance. |
APSU only has one record on this issue list so, small beans, but upon
inspecting I'm not even sure what that number is in relation to the record
so APSU is good to go with whatever is decided. -Jessica
…On Fri, Apr 19, 2024 at 2:42 PM dustymc ***@***.***> wrote:
records with data from ctabundance
As far as I know there are none.
—
Reply to this email directly, view it on GitHub
<#7691 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ARAJP3HBDKAHR6KETV6K3ADY6FQP3AVCNFSM6AAAAABGPKNKB6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANRXGEYDKOBYGY>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Bare integers aren't the most ridiculous things in these data! #6179 - I am increasingly convinced that we wildly underestimate the costs of simply having lots of THINGS (eg some poor overwhelmed student puts A-thing into Z-container and it's lost). |
I agree 100 as the kids say these days...
…On Fri, Apr 19, 2024 at 3:00 PM dustymc ***@***.***> wrote:
not even sure what that number is
Bare integers aren't the most ridiculous things in these data! #6179
<#6179> - I am increasingly
convinced that we wildly underestimate the costs of simply having lots of
THINGS (eg some poor overwhelmed student puts A-thing into Z-container and
it's lost).
—
Reply to this email directly, view it on GitHub
<#7691 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ARAJP3EOO6KKQUDN4HJPGK3Y6FSTZAVCNFSM6AAAAABGPKNKB6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANRXGEZDSOJSGI>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Is your feature request related to a problem? Please describe.
Abundance is an uncontrolled attribute. I think perhaps the code table is a failed experiment?
Describe what you're trying to accomplish
Clean
Describe the solution you'd like
Drop the table.
Describe alternatives you've considered
Use the table.
Additional context
Some of these data are probably lost, #6179 is related.
Priority
Critical, I'll drop the code table unless someone has a use for it.
Data
temp_abundance.csv.zip
Usage
Ping
@DerekSikes
@StefanieBond
@jtgiermakowski
@ccicero
@mkoo
@mvzhuang
@acdoll
@Jegelewicz
@camwebb
@gradyjt
@ehalverson26
The text was updated successfully, but these errors were encountered: