Recommend adding an includes/menu.xml file to improve navigability

[saulakravitz]

No description provided.

[saulakravitz]

Submitted a pull request with a menu that should work.
I can't get the build process to work on my laptop (running out of heap space despite reconfiguring docket desktop),...

[preston]

@neuronsong Could you take a look at this? I'm not familiar with it.

[neuronsong]

Are you doing a docker build... or running the publisher manually? If you're running the publisher manually I have had good success with the following:
java -Xss1g -Xms2g -Xmx4g -jar org.hl7.fhir.igpublisher.jar -ig .

I'm taking a look at the changes now.

[neuronsong]

I just built the branch... and the menu is not working. The menu is broken visually and throwing off 404 errors.

[preston]

For Docker builds, the docker build ... needs a --memory=4g switch, as well as the daemon allowed to malloc at least that amount. For macOS and Windows, this is globally set in the UI settings panel.

[saulakravitz]

BUilding successfully with java directly. SHould have a new pull request that actually works shortly. Thanks for your patience.

…

________________________________ From: Preston Lee <[email protected]> Sent: Thursday, April 23, 2020 5:39 PM To: logicahealth/covid-19 <[email protected]> Cc: Kravitz, Saul A. <[email protected]>; Author <[email protected]> Subject: [EXT] Re: [logicahealth/covid-19] Recommend adding an includes/menu.xml file to improve navigability (#15) For Docker builds, the docker build ... needs a --memory=4g switch, as well as the daemon allowed to malloc at least that amount. For macOS and Windows, this is globally set in the UI settings panel. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub<#15 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AI36FE4V7JHMOZM4HWVN733ROCYSDANCNFSM4MNRC7JA>.

[saulakravitz]

Still can't build....how does one get the ig build tool to NOT run sushi...it keeps clobbering my menu.xml file. I've committed the correct one, with correct links to the relevant subsections of the IG that should be of interest to a reader. I can't get it to build, but there is not a lot there to go wrong (famous last words).

…

________________________________ From: Kravitz, Saul A. <[email protected]> Sent: Thursday, April 23, 2020 6:34 PM To: logicahealth/covid-19 <[email protected]>; logicahealth/covid-19 <[email protected]> Cc: Author <[email protected]> Subject: Re: [EXT] Re: [logicahealth/covid-19] Recommend adding an includes/menu.xml file to improve navigability (#15) BUilding successfully with java directly. SHould have a new pull request that actually works shortly. Thanks for your patience.

________________________________ From: Preston Lee <[email protected]> Sent: Thursday, April 23, 2020 5:39 PM To: logicahealth/covid-19 <[email protected]> Cc: Kravitz, Saul A. <[email protected]>; Author <[email protected]> Subject: [EXT] Re: [logicahealth/covid-19] Recommend adding an includes/menu.xml file to improve navigability (#15) For Docker builds, the docker build ... needs a --memory=4g switch, as well as the daemon allowed to malloc at least that amount. For macOS and Windows, this is globally set in the UI settings panel. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub<#15 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AI36FE4V7JHMOZM4HWVN733ROCYSDANCNFSM4MNRC7JA>.

[preston]

If it detects the fsh source directory it automatically runs. Is this something that needs to be fixed in the IG publisher?

[saulakravitz]

Got it working. They key is DELETING the input/* and output directories prior to each run of the Java code. I've built and tested the menu. It ain't perfect, but it works. Give it a whirl!

…

________________________________ From: Preston Lee <[email protected]> Sent: Friday, April 24, 2020 10:54 AM To: logicahealth/covid-19 <[email protected]> Cc: Kravitz, Saul A. <[email protected]>; Author <[email protected]> Subject: [EXT] Re: [logicahealth/covid-19] Recommend adding an includes/menu.xml file to improve navigability (#15) If it detects the fsh source directory it automatically runs. Is this something that needs to be fixed in the IG publisher? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub<#15 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AI36FE7VHXVPBVZJ6OHNA4TROGR3PANCNFSM4MNRC7JA>.