8000 Feature: Additional AE fields · Issue #35 · Gilead-BioStats/gsm.mapping · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Feature: Additional AE fields #35

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.

Sign up for GitHub

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

Jump to bottom
Open
jonthegeek opened this issue Feb 28, 2025 · 4 comments
Open

Feature: Additional AE fields #35

jonthegeek opened this issue Feb 28, 2025 · 4 comments
Milestone
v1.0.2

Comments

@jonthegeek
Copy link
Contributor
jonthegeek commented Feb 28, 2025 •
edited
Loading

Feature Details

We need these for gsm.ae. I'm including additional information to help set them up for gsm.datasim as well, plus future information for #7 in case we ever sort that out.

    aeongo:
      type: character
      label: Ongoing
      values: "ONGOING", "" (which should encode as "NOT ONGOING")
    aerel:
      type: character
      label: Relatedness
      values: "NOT RELATED", "RELATED"
    mincreated_dts:
      type: timestamp
      label: Minimum Created Time

As far as I can find, this is everything that's needed in AE for the current iteration of gsm.ae.
@jonthegeek jonthegeek mentioned this issue Feb 28, 2025
Open
@jonthegeek jonthegeek changed the title Feature: Additional AE field(s) Feature: Additional AE fields Feb 28, 2025
@jonthegeek
Copy link
Contributor Author

Update: It looks like we need mincreated_dts for both the "Data Quality" panel and the "Timelines" panel, so I definitely think we need it even if we also have a derived "time to AE entry" value.

@samussiah
Copy link
Contributor

You got it - we need mincreated to calculate the time between date of the visit following the event and the date it was recorded in the system.

@jonthegeek
Copy link
Contributor Author

I also need mincreated_dts even for panel 1; one of the things that has been requested is counts of AE/SAE "since last snapshot", which means I need to know when they were recorded. It's probably good that we don't have it right now; it's forcing me to think about how to have the user tell me what counts as "new". But we're going to need it for our own things, so it makes sense to include it in our demo data and standard mapping.

@jonthegeek
Copy link
Contributor Author

This is actually a datetime (or timestamp, I'm not sure if we have any of these to define exactly what we want to call it for ingestion), not a Date.

@jonthegeek jonthegeek mentioned this issue Apr 29, 2025
Open
@lauramaxwell lauramaxwell added this to the v1.0.2 milestone May 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v1.0.2
Development

No branches or pull requests
3 participants
@samussiah @lauramaxwell @jonthegeek
0