Chris Knoll

Results 131 comments of Chris Knoll
trafficstars

I added a comment to the PR, I don't think the PR addresses this since you can't remove `file = ` and have the value passed in as the first...

We should have better documentation for these reports.

I believe the issue is that when you import a characterization, it doesn't overwrite the existing one, but rather creates a new characterization using the imported design. Is that correct?...

@TomWhite-MedStar , I checked in with our techs and they reminded me that we took some steps to reduce the usage of CTEs because of the challenges of their performance/optimization...

I was going to suggest that we implement the 'not in' logic on race and ethnicity concepts so you could say 'not in' {the other list of concepts), but we...

I can check, but my impression that it would order by the proportional value first, showing you the most prevalent features in the data above others.

Fair point, we should remove the styling on the cohort name if it is read only.

It would be difficult to implement this in the tool, you're basically asking for a winmerge visualization (left-side, right side) comparison of the cohort entry which is a complicated feature...

Hi, @mrechkem . Thank you for the well thought out proposal. I think eliminating the different reusables is something that would simplify things, but at the same time could have...

Ok, i thought the +/- buttons were styled as html links, but we should put some style there that can change the mouse pointer and possibly apply a color that...