search

Gilead-BioStats / gsm

Good Statistical Monitoring R Package
https://gilead-biostats.github.io/gsm/
Apache License 2.0
39 stars 9 forks source link

Bugfix: Don't force `SnapshotDate` in `Report_MetricTable()` #1764

Closed jonthegeek closed 2 months ago

jonthegeek commented 2 months ago

Expected Behavior

If dfResults doesn't include a SnapshotDate, don't display it.

Current Behavior

If dfResults doesn't include a SnapshotDate, Report_MetricTable() adds today's date, which seems weird.

Possible Solution

Don't add it when it's missing. In filter_by_latest_SnapshotDate() specifically.

Steps to Reproduce

1. 2. 3. 4.

Context (Environment)

Possible Implementation

Additional Comments