search

pophealth / popHealth

popHealth - An Open Source Population Health Reporting Prototype
http://projectpophealth.org
Apache License 2.0
126 stars 154 forks source link

Installation instructions using "latest" instead of specific versions is just asking for trouble #269

Closed vais closed 7 years ago

vais commented 10 years ago

Please please please update the installation instructions to use specific version numbers of popHealth and the measures bundle that are last known to work as expected with each other. Pinning installation instructions to "latest" of everything is causing grief to your users.

petercyli commented 10 years ago

Was there a specific issue encountered with the latest installation instruction?

vais commented 10 years ago

@petercyli Yes, at least for me - please see #268 and see if you can reproduce this. Since downgrading to an older popHealth commit AND an older bundle (2.4.0) is what fixed the issue for me, I can only assume the fault is somewhere in the combination of latest commit and latest bundle.

petercyli commented 10 years ago

Will try to reproduce.

I installed popHealth couple weeks ago using the first part of the V.2.1 instruction, and then switched to V.3 instruction after I found out I needed to use the so called “latest” instruction to install V.3. I didn’t encounter your problem.

Anyway, I should redo it based on V.3 instruction to duplicate your issue.

Thanks.

Peter

From: Vais Salikhov [mailto:notifications@github.com] Sent: Wednesday, October 15, 2014 6:26 AM To: pophealth/popHealth Cc: petercyli Subject: Re: [popHealth] Installation instructions using "latest" instead of specific versions is just asking for trouble (#269)

Yes, at least for me - please see #268 https://github.com/pophealth/popHealth/issues/268 and see if you can reproduce this. Since downgrading to an older popHealth commit AND an older bundle (2.4.0) is what fixed the issue for me, I can only assume the fault is somewhere in the combination of latest commit and latest bundle.

— Reply to this email directly or view it on GitHub https://github.com/pophealth/popHealth/issues/269#issuecomment-59186008 . https://github.com/notifications/beacon/1581933__eyJzY29wZSI6Ik5ld3NpZXM6QmVhY29uIiwiZXhwaXJlcyI6MTcyODk4Nzk4MywiZGF0YSI6eyJpZCI6NDUzOTY0NTR9fQ==--99ec08c437b9f400460f7a699c6a0f0e59e8fecd.gif

vais commented 10 years ago

Could the error be specific to a particular CQM? I am not sure how the map/reduce code works, but I believe that's implemented in javascript for each measure? The CQMs I was hitting are: 50, 165, 68, 65, 22, 69, 138, 166, and 155.