PSC-PublicHealth / HERMES

0 stars 0 forks source link

add_loops_experiment: no transport component added; get failure error message #65

Open jennBakal opened 7 years ago

jennBakal commented 7 years ago

image

failure error message: image

then next page doesn't have anything (obviously) but you can still keep clicking Next.... image

image

shots47s commented 7 years ago

I cannot reproduce this error with the beta-release branch, everything works as expected. Leila, can you try and confirm Jenn's error?

Shawn

On Mon, Aug 14, 2017 at 3:07 PM, jennBakal notifications@github.com wrote:

[image: image] https://user-images.githubusercontent.com/3708686/29286833-6e5a1f70-8101-11e7-8977-3327f45c2f29.png

failure error message: [image: image] https://user-images.githubusercontent.com/3708686/29287018-0e104bf2-8102-11e7-93ee-ab8b5153823d.png

then next page doesn't have anything (obviously) but you can still keep clicking Next.... [image: image] https://user-images.githubusercontent.com/3708686/29287053-2a23727e-8102-11e7-8118-50b34e40ffcd.png

[image: image] https://user-images.githubusercontent.com/3708686/29287072-344788c6-8102-11e7-989b-01f16511e908.png

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/PSC-PublicHealth/HERMES/issues/65, or mute the thread https://github.com/notifications/unsubscribe-auth/AAAl96LVj5qmobetC_WlaY8xC0ngA7ldks5sYJr2gaJpZM4O2xX3 .

shots47s commented 7 years ago

I tried the following in the transport loop experiment workflow and I did not run into the errors in Jenn's screenshots. It seemed to work fine, except two unrelated issues: You run into costing errors if you try to run the model immediately (because no per diem policies are assigned to the new routes). Also, it assigned a shipping interval of 20 days to the loops even though 28 day months are now the default (and are used in this particular model).

Using the attached model, I:

On Mon, Aug 14, 2017 at 8:41 PM, Shawn T. Brown shawntbrown@gmail.com wrote:

I cannot reproduce this error with the beta-release branch, everything works as expected. Leila, can you try and confirm Jenn's error?

Shawn

On Mon, Aug 14, 2017 at 3:07 PM, jennBakal notifications@github.com wrote:

[image: image] https://user-images.githubusercontent.com/3708686/29286833-6e5a1f70-8101-11e7-8977-3327f45c2f29.png

failure error message: [image: image] https://user-images.githubusercontent.com/3708686/29287018-0e104bf2-8102-11e7-93ee-ab8b5153823d.png

then next page doesn't have anything (obviously) but you can still keep clicking Next.... [image: image] https://user-images.githubusercontent.com/3708686/29287053-2a23727e-8102-11e7-8118-50b34e40ffcd.png

[image: image] https://user-images.githubusercontent.com/3708686/29287072-344788c6-8102-11e7-989b-01f16511e908.png

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/PSC-PublicHealth/HERMES/issues/65, or mute the thread https://github.com/notifications/unsubscribe-auth/AAAl96LVj5qmobetC_WlaY8xC0ngA7ldks5sYJr2gaJpZM4O2xX3 .

shots47s commented 7 years ago

Ah, ok, i can fix these.

On Aug 15, 2017 7:18 AM, "Leila Haidari" leila@psc.edu wrote:

I tried the following in the transport loop experiment workflow and I did not run into the errors in Jenn's screenshots. It seemed to work fine, except two unrelated issues: You run into costing errors if you try to run the model immediately (because no per diem policies are assigned to the new routes). Also, it assigned a shipping interval of 20 days to the loops even though 28 day months are now the default (and are used in this particular model).

Using the attached model, I:

  • Selected District as the level at which to start the loops
  • Selected Health Center as the level the loops will distribute to
  • Specified 7 locations per loop (Jenn, this obviously isn't related to the issue, but I realized we should change this from 6 to 7)
  • Kept provincial Hilux selected as the vehicle

On Mon, Aug 14, 2017 at 8:41 PM, Shawn T. Brown shawntbrown@gmail.com wrote:

I cannot reproduce this error with the beta-release branch, everything works as expected. Leila, can you try and confirm Jenn's error?

Shawn

On Mon, Aug 14, 2017 at 3:07 PM, jennBakal notifications@github.com wrote:

[image: image] https://user-images.githubusercontent.com/3708686/29286833-6e5a1f70-8101-11e7-8977-3327f45c2f29.png

failure error message: [image: image] https://user-images.githubusercontent.com/3708686/29287018-0e104bf2-8102-11e7-93ee-ab8b5153823d.png

then next page doesn't have anything (obviously) but you can still keep clicking Next.... [image: image] https://user-images.githubusercontent.com/3708686/29287053-2a23727e-8102-11e7-8118-50b34e40ffcd.png

[image: image] https://user-images.githubusercontent.com/3708686/29287072-344788c6-8102-11e7-989b-01f16511e908.png

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/PSC-PublicHealth/HERMES/issues/65, or mute the thread https://github.com/notifications/unsubscribe-auth/AAAl96LVj5qmobetC_WlaY8xC0ngA7ldks5sYJr2gaJpZM4O2xX3 .

shots47s commented 7 years ago

These have both been addressed in the beta-release branch. It should be safe to pull.

Shawn

On Tue, Aug 15, 2017 at 7:21 AM, Shawn T. Brown shawntbrown@gmail.com wrote:These have

Ah, ok, i can fix these.

On Aug 15, 2017 7:18 AM, "Leila Haidari" leila@psc.edu wrote:

I tried the following in the transport loop experiment workflow and I did not run into the errors in Jenn's screenshots. It seemed to work fine, except two unrelated issues: You run into costing errors if you try to run the model immediately (because no per diem policies are assigned to the new routes). Also, it assigned a shipping interval of 20 days to the loops even though 28 day months are now the default (and are used in this particular model).

Using the attached model, I:

  • Selected District as the level at which to start the loops
  • Selected Health Center as the level the loops will distribute to
  • Specified 7 locations per loop (Jenn, this obviously isn't related to the issue, but I realized we should change this from 6 to 7)
  • Kept provincial Hilux selected as the vehicle

On Mon, Aug 14, 2017 at 8:41 PM, Shawn T. Brown shawntbrown@gmail.com wrote:

I cannot reproduce this error with the beta-release branch, everything works as expected. Leila, can you try and confirm Jenn's error?

Shawn

On Mon, Aug 14, 2017 at 3:07 PM, jennBakal notifications@github.com wrote:

[image: image] https://user-images.githubusercontent.com/3708686/29286833-6e5a1f70-8101-11e7-8977-3327f45c2f29.png

failure error message: [image: image] https://user-images.githubusercontent.com/3708686/29287018-0e104bf2-8102-11e7-93ee-ab8b5153823d.png

then next page doesn't have anything (obviously) but you can still keep clicking Next.... [image: image] https://user-images.githubusercontent.com/3708686/29287053-2a23727e-8102-11e7-8118-50b34e40ffcd.png

[image: image] https://user-images.githubusercontent.com/3708686/29287072-344788c6-8102-11e7-989b-01f16511e908.png

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/PSC-PublicHealth/HERMES/issues/65, or mute the thread https://github.com/notifications/unsubscribe-auth/AAAl96LVj5qmobetC_WlaY8xC0ngA7ldks5sYJr2gaJpZM4O2xX3 .