FHIR Chat · ValueSet.expansion.parameter · terminology

Stream: terminology

Topic: ValueSet.expansion.parameter


view this post on Zulip Jack Bowie (Aug 01 2018 at 14:50):

Pardon the rather basic question, but in R4 what is returned in the expansion.parameter elements? The documentation says they are "all the parameters that affect the contents of the expansion". This is a pretty broad statement. So are these the attributes that are named in the VS expression or are they duplicates of the $expand In Parameters like filter and activeOnly? If the latter, do only those parameters with non-default values need be specified? The example does not help. Thanks.

view this post on Zulip Grahame Grieve (Aug 01 2018 at 20:00):

not things from the VS expression

view this post on Zulip Grahame Grieve (Aug 01 2018 at 20:01):

and only parameters with nondefault values

view this post on Zulip Rob Hausam (Aug 01 2018 at 22:07):

the tracker and resolution for GF#15144 states that it should include "all of the parameters, including server defaults"

view this post on Zulip Grahame Grieve (Aug 01 2018 at 22:08):

we should clarify this then - server defaults that are not default values

view this post on Zulip Rob Hausam (Aug 01 2018 at 22:10):

yes - it isn't clear to me exactly what is meant by "server defaults that are not default values"

view this post on Zulip Grahame Grieve (Aug 01 2018 at 22:20):

well, the default value for activeOnly is false. So we don't echo this in the expansion parameters if the client doesn't specify it, or specifies it as false, or if the server default value is false.

view this post on Zulip Grahame Grieve (Aug 01 2018 at 22:20):

but if the server default is true then that should go in the expansion parameters

view this post on Zulip Peter Jordan (Aug 01 2018 at 23:17):

I need to implement that! BTW should the name of the expansion parameter containing the version of the Value Set be 'valueSetVersion' (mirroring the name of the input parameter) or simply 'version'?

view this post on Zulip Jack Bowie (Aug 02 2018 at 00:03):

And count and offset I presume?

view this post on Zulip Grahame Grieve (Aug 02 2018 at 00:11):

yes to count and offset. @Peter Jordan there's special parameters defined in R4 for version

view this post on Zulip Peter Jordan (Aug 02 2018 at 00:34):

Thanks - I can only see special parameters relating to the version of a Code System, but I guess the Value Set version would be superfluous as that's contained elsewhere. BTW: what's the difference between expansion.offset and the offset expansion.parameter?

view this post on Zulip Grahame Grieve (Aug 02 2018 at 00:44):

ah yes I was thinking of the code system versions. the value set version - yes - elsewhere

view this post on Zulip Grahame Grieve (Aug 02 2018 at 00:45):

what's the difference between expansion.offset and the offset expansion.parameter?

oops. have we (I) documented both?

view this post on Zulip Rob Hausam (Aug 02 2018 at 00:50):

I get the point about "So we don't echo this in the expansion parameters if the client doesn't specify it, or specifies it as false, or if the server default value is false" - but wouldn't it be simpler and clearer if we just explicitly included the values of the parameters regardless of whether they are the default value or not?

view this post on Zulip Grahame Grieve (Aug 02 2018 at 01:06):

most implementers would regard that as needless verbosity

view this post on Zulip Rob Hausam (Aug 02 2018 at 01:07):

possibly - I'm fine with not doing that if there's general agreement

view this post on Zulip Jack Bowie (Aug 02 2018 at 01:40):

Including only non-defaults works for me.

view this post on Zulip Peter Jordan (Aug 02 2018 at 03:37):

what's the difference between expansion.offset and the offset expansion.parameter?

oops. have we (I) documented both?

ValueSet.expansion.offset..."If paging is being used, the offset at which this resource starts. I.e. this resource is a partial view into the expansion. If paging is not being used, this element SHALL NOT be present."

ValueSet.expansion.parameter["offset"]..."Paging support - where to start if a subset is desired (default = 0). Offset is number of records (not number of pages)" - this echoes the description of the offset input parameter to an $expand request.

view this post on Zulip Grahame Grieve (Aug 02 2018 at 05:00):

sigh better fix that

view this post on Zulip Jack Bowie (Aug 02 2018 at 15:58):

Should a DateTime data type be supported in ValueSet.expansion.parameter to accommodate a 'date' parameter in the expansion request, or is the ValueSet version in the result sufficient?

view this post on Zulip Grahame Grieve (Aug 02 2018 at 18:56):

it certainly should be

view this post on Zulip Grahame Grieve (Aug 02 2018 at 18:56):

@Rob Hausam this is the sort of emergency finding I absolutely hate... we have to fix this one

view this post on Zulip Rob Hausam (Aug 02 2018 at 23:21):

agree - this is a rather glaring omission

view this post on Zulip Jack Bowie (Aug 03 2018 at 00:39):

Sorry, guys. I'll just go back to coding ;-) Best.

view this post on Zulip Grahame Grieve (Aug 03 2018 at 01:00):

@Jack Bowie it's clearly your fault that we forgot to add dateTime as a type

view this post on Zulip Grahame Grieve (Aug 03 2018 at 01:10):

so, @Rob Hausam what are our procedural options here? Can I just add it and get approval later?

view this post on Zulip Rob Hausam (Aug 03 2018 at 03:23):

I think we need to add a tracker and pre-apply it now, and then get it on the Vocab agenda for approval next week. That should work.

view this post on Zulip Grahame Grieve (Aug 03 2018 at 03:35):

GF#17616

view this post on Zulip Grahame Grieve (Aug 03 2018 at 03:37):

added the type

view this post on Zulip Michael Lawley (Aug 03 2018 at 03:52):

oh, I thought you'd fix it by removing support for the date parameter in the request :-/

view this post on Zulip Grahame Grieve (Aug 03 2018 at 04:11):

lol.


Last updated: Apr 12 2022 at 19:14 UTC