FHIR Chat · This needs to be checked · IG creation

Stream: IG creation

Topic: This needs to be checked


view this post on Zulip David Pyke (Jun 03 2021 at 16:07):

I'm building a new IG and I'm getting:
Reclaiming memory...
## Memory (MB): Use = 132, Free = 363, Total = 496, Max =8124
Exception in thread "main" java.lang.Error: This needs to be checked for profile path
at org.hl7.fhir.igtools.publisher.Publisher.fixSearchForm(Publisher.java:5190)
at org.hl7.fhir.igtools.publisher.Publisher.generate(Publisher.java:5059)
at org.hl7.fhir.igtools.publisher.Publisher.createIg(Publisher.java:901)
at org.hl7.fhir.igtools.publisher.Publisher.execute(Publisher.java:745)
at org.hl7.fhir.igtools.publisher.Publisher.main(Publisher.java:8577)
What is "this"?

view this post on Zulip David Pyke (Jun 03 2021 at 17:03):

The source is here...

view this post on Zulip David Pyke (Jun 03 2021 at 17:05):

@Grahame Grieve

view this post on Zulip Jose Costa Teixeira (Jun 03 2021 at 17:23):

seems that the code is looking for a searchform.html

view this post on Zulip David Pyke (Jun 03 2021 at 17:24):

So do I need to create one or is there a setting that needs to be set?

view this post on Zulip Jose Costa Teixeira (Jun 03 2021 at 17:25):

did you mess with the template? Any change to the pagebegin.html?

view this post on Zulip David Pyke (Jun 03 2021 at 17:25):

I touched nothing

view this post on Zulip Jose Costa Teixeira (Jun 03 2021 at 17:25):

oh ok

view this post on Zulip Jose Costa Teixeira (Jun 03 2021 at 17:25):

is there a repo for this?

view this post on Zulip David Pyke (Jun 03 2021 at 17:25):

Copied it over from MHD and that was all

view this post on Zulip David Pyke (Jun 03 2021 at 17:25):

IHE/PDQm

view this post on Zulip Jose Costa Teixeira (Jun 03 2021 at 17:26):

perhaps IHE has invented a copy protection mechanism for IGs... who knows...

view this post on Zulip Jose Costa Teixeira (Jun 03 2021 at 17:26):

which repo did you copy (and which branch)?

view this post on Zulip David Pyke (Jun 03 2021 at 17:33):

Work with the master branch of the PDQm. I created it from scratch and pulled the template from the MHD master

view this post on Zulip Lloyd McKenzie (Jun 03 2021 at 17:57):

This isn't something I've messed with. @Grahame Grieve?

view this post on Zulip John Moehrke (Jun 03 2021 at 18:15):

note that MHD still builds just fine.

view this post on Zulip Jose Costa Teixeira (Jun 03 2021 at 18:16):

@David Pyke can you send a zip with what you have?

view this post on Zulip John Moehrke (Jun 03 2021 at 18:16):

is it checked into https://github.com/IHE/ITI.PDQm.git

view this post on Zulip David Pyke (Jun 03 2021 at 18:16):

Everything I have is checked into the github repo

view this post on Zulip John Moehrke (Jun 03 2021 at 18:19):

unknown tag 'insert'

view this post on Zulip David Pyke (Jun 03 2021 at 18:19):

ITI.PDQm.zip

view this post on Zulip John Moehrke (Jun 03 2021 at 18:19):

Jekyll: [31m  Liquid Exception: Liquid syntax error (C:/Users/johnm/Git/ITI.PDQm/temp/pages/_includes/template-page-md.html line 6): Unknown tag 'insert' included in actors_and_transactions.html←[0m (00:59.0917)
Jekyll: [31m                    ------------------------------------------------←[0m (00:59.0925)
Jekyll: [31m      Jekyll 4.2.0   Please append `--trace` to the `build` command ←[0m (00:59.0926)
Jekyll: [31m                     for any additional information or backtrace. ←[0m (00:59.0926)
Jekyll: [31m                    ------------------------------------------------←[0m (00:59.0927)
Jekyll: C:/Ruby27-x64/lib/ruby/gems/2.7.0/gems/liquid-4.0.3/lib/liquid/document.rb:23:in `unknown_tag': Liquid syntax error (C:/Users/johnm/Git/ITI.PDQm/temp/pages/_includes/template-page-md.html line 6): Unknown tag 'insert' included  (Liquid::SyntaxError) (00:59.0927)
Jekyll: from C:/Ruby27-x64/lib/ruby/gems/2.7.0/gems/liquid-4.0.3/lib/liquid/document.rb:11:in `block in parse' (00:59.0928)
...

view this post on Zulip Jose Costa Teixeira (Jun 03 2021 at 18:22):

yeah, insertdoes not really exist

view this post on Zulip David Pyke (Jun 03 2021 at 18:22):

So who inserted insert?

view this post on Zulip Jose Costa Teixeira (Jun 03 2021 at 18:22):

the same person that did not include inlcude

view this post on Zulip John Moehrke (Jun 03 2021 at 18:23):

:-)

view this post on Zulip David Pyke (Jun 03 2021 at 18:24):

So where do I make a change to make it work?

view this post on Zulip John Moehrke (Jun 03 2021 at 18:24):

you didn't specify the template the same way I did in MHD

view this post on Zulip John Moehrke (Jun 03 2021 at 18:24):

you must follow the Jose magjic exactly

view this post on Zulip John Moehrke (Jun 03 2021 at 18:24):

#ihe-template-no-numbers

view this post on Zulip David Pyke (Jun 03 2021 at 18:25):

Okay, I'll try that

view this post on Zulip John Moehrke (Jun 03 2021 at 18:25):

im trying that right now

view this post on Zulip John Moehrke (Jun 03 2021 at 18:25):

nope

view this post on Zulip John Moehrke (Jun 03 2021 at 18:25):

didn't work

view this post on Zulip David Pyke (Jun 03 2021 at 18:26):

No difference

view this post on Zulip Jose Costa Teixeira (Jun 03 2021 at 18:28):

mine builds ok

view this post on Zulip John Moehrke (Jun 03 2021 at 18:28):

ah, so it is Jose magjic

view this post on Zulip David Pyke (Jun 03 2021 at 18:29):

So you checked it out of the repo or used the zip above. I'm not above wiping my dir and re-checking it out if it will work

view this post on Zulip John Moehrke (Jun 03 2021 at 18:30):

I freshly cloned

view this post on Zulip John Moehrke (Jun 03 2021 at 18:31):

is not working for me

view this post on Zulip John Moehrke (Jun 03 2021 at 18:31):

why are there .index files in the ihe-template-no-numbers folder?

view this post on Zulip David Pyke (Jun 03 2021 at 18:31):

Dunno, I didn't put them there

view this post on Zulip John Moehrke (Jun 03 2021 at 18:32):

well, deleting them didn't help

view this post on Zulip Jose Costa Teixeira (Jun 03 2021 at 18:32):

ready for one more? (sorry, now it's the footer that had a typo)

view this post on Zulip John Moehrke (Jun 03 2021 at 18:32):

but I don't have them in MHD

view this post on Zulip David Pyke (Jun 03 2021 at 18:32):

I'm ready for anything, Jose!

view this post on Zulip Jose Costa Teixeira (Jun 03 2021 at 18:32):

actually that was a misplaced message

view this post on Zulip John Moehrke (Jun 03 2021 at 18:32):

raise our hopes

view this post on Zulip Jose Costa Teixeira (Jun 03 2021 at 18:32):

delete your local template cache

view this post on Zulip Jose Costa Teixeira (Jun 03 2021 at 18:33):

in your fhir packages, the fhir base template.

view this post on Zulip Jose Costa Teixeira (Jun 03 2021 at 18:33):

(just to make sure that is not the issue)

view this post on Zulip Jose Costa Teixeira (Jun 03 2021 at 18:33):

and the IHE template as well

view this post on Zulip John Moehrke (Jun 03 2021 at 18:33):

wouldn't my MHD then fail?

view this post on Zulip John Moehrke (Jun 03 2021 at 18:34):

my MHD succeeds on this same machine

view this post on Zulip David Pyke (Jun 03 2021 at 18:35):

No difference

view this post on Zulip David Pyke (Jun 03 2021 at 18:46):

On a fresh clone, I get the liquid error you got, John

view this post on Zulip Jose Costa Teixeira (Jun 03 2021 at 18:51):

liquid error about the insert?

view this post on Zulip David Pyke (Jun 03 2021 at 18:51):

yep

view this post on Zulip David Pyke (Jun 03 2021 at 18:52):

So, my local gets the "This needs to be checked" error and a fresh checkout gets the liquid "insert" error. So exciting!

view this post on Zulip Jose Costa Teixeira (Jun 03 2021 at 18:52):

Unknown tag 'insert' included in actors_and_transactions.html

view this post on Zulip David Pyke (Jun 03 2021 at 18:52):

Yes, that one

view this post on Zulip Jose Costa Teixeira (Jun 03 2021 at 18:52):

that says

view this post on Zulip John Moehrke (Jun 03 2021 at 18:53):

should have been include, not insert

view this post on Zulip John Moehrke (Jun 03 2021 at 18:53):

I just found that

view this post on Zulip Jose Costa Teixeira (Jun 03 2021 at 18:53):

you have a tag insert in your actors_and_transactions

view this post on Zulip Jose Costa Teixeira (Jun 03 2021 at 18:53):

and you should not insert insert, rather include include (sorry)

view this post on Zulip John Moehrke (Jun 03 2021 at 18:54):

yup, it now builds

view this post on Zulip David Pyke (Jun 03 2021 at 18:56):

ANd now it builds.

view this post on Zulip David Pyke (Jun 03 2021 at 18:56):

Life goes on as normal

view this post on Zulip John Moehrke (Jun 03 2021 at 18:57):

thanks @Jose Costa Teixeira

view this post on Zulip David Pyke (Jun 03 2021 at 18:57):

Thanks @Jose Costa Teixeira !

view this post on Zulip Grahame Grieve (Jun 03 2021 at 21:55):

This needs to be checked for profile path

How to I reproduce that ?

view this post on Zulip David Pyke (Jun 04 2021 at 13:21):

Grab the zip file above

view this post on Zulip Grahame Grieve (Jun 04 2021 at 14:10):

I did. I didn't have the problem

view this post on Zulip David Pyke (Jun 04 2021 at 14:11):

THere must be something special about my setup that was causing it

view this post on Zulip Grahame Grieve (Jun 04 2021 at 14:11):

well, can you still reproduce it?

view this post on Zulip David Pyke (Jun 04 2021 at 14:15):

Hold on, let me see

view this post on Zulip David Pyke (Jun 04 2021 at 14:25):

No, it seems to be gone. But there was work done on the IHE template, that may have fixed it


Last updated: Apr 12 2022 at 19:14 UTC