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

Develop #179

Closed
wants to merge 97 commits into from
Closed

Develop #179

wants to merge 97 commits into from

Conversation

jreps
Copy link
Collaborator

@jreps jreps commented May 4, 2023

No description provided.

jreps and others added 30 commits March 28, 2023 14:56
-starting to restructure sccs
- adding a component module for the input selection and selected view
- testing component in the meta module
- updating incidence rate table
- adding conditional panel to meta results
Adding markdown to Depends of Description
fixing check issues
fixing test errors
- updated SCCS module
- fixes multiple selection in input select component
fixing checks
- adding extra char types
fixing bug with diagnostic due to missing database_id in join
adding covariate name to sccs diag
fixing big with sccs diagnostic single view
jreps and others added 29 commits June 13, 2023 14:04
- replaced plp design and model buttons with action dropdown button
- updated tests
- renamed tablePrefix to plpTablePrefix in Prediction to prepare for shared settings object across modules
- renamed mySchema to schema in Prediction to standardized and prepare for shared settings object across modules
… and connection handlers, small update to components
adding some updates to main to phevaluator before merging into develop
Adding phevaluator and datasources modules to develop. Also improving tests and components module.
- renamed description to  characterization
- renamed estimation to cohort-method
- updated inputs to prediction
- updating modules so they all use the same inputs in resultDatabaseSettings (this can now be shared across modules)
- updating names to match analysis package names
updated interface
updated interface
- minor updates to cohort diagnostic to enable shared database table
updating version
* Index event breakdown report added to select input navigation

* database selection in cd time distribution tab

* database name in cohort diagnostics cohort counts simple tab
* Started large table component - mostly working.

TOODO:

- Unit tests
- Inclusion in modules (characterization and cohort diagnostics)

- Download data functionality (implement promises callback?)

- Make appearance as close to other tables as possible

- Allow row selection callback

- Allow as many customisable reactable options as possible (beyond search and filters which will not easily work)

- Make more compatible with query namespeaces to automatically generate advanced column filters

* Tests and fully operational tables

* Tests and fully operational tables

* docs

* docs

* example fixed

* example fixed

* example fixed

* example fixed

* Changes to appearance

* Changes to appearance

* Minor fixes for appearance

* Minor fixes for appearance

* Columns now parameter to compnenent not data object, convenient creation function for ldts

* Columns now parameter to compnenent not data object, convenient creation function for ldts

* first CD ldt now mostly working - just some minor tweaks on first table to add

* first CD ldt now mostly working - just some minor tweaks on first table to add

* fixed typo in tests

* fixed typo in tests

* fixed typo in tests

* fixed typo in tests

* Minor changes to table

* Minor changes to table

* SOlution for query that works but not if database t1 is empty

* SOlution for query that works but not if database t1 is empty

* selected descedning results by default

* selected descedning results by default

* working tables

* working tables

* test change

* test change

* docs fix
- standardizing cohort method
- standardizing sccs
fixing merge of standardization
@jreps jreps closed this Aug 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants