-
Notifications
You must be signed in to change notification settings - Fork 75
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Ggr0921 #530
base: master
Are you sure you want to change the base?
Ggr0921 #530
Conversation
'Stof' AS source_concept_class_id, | ||
NULL AS standard_concept, | ||
'Ingredient' AS source_concept_class_id, | ||
'S' AS standard_concept, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks suspicious - making source ingredient standard
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just started, there are too many changes. not even sure how to go through them.
maybe we need a call with Violetta, Eddy and I.
|
||
LEFT JOIN SOURCES.GGR_SAM sam ON sam.mppcv = mpp.mppcv | ||
WHERE OUC = 'C' | ||
and sam.hyr_ not like 'LA%' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what does it mean?
GGR/auto_init.sql
Outdated
* limitations under the License. | ||
**************************************************************************/ | ||
/* | ||
SELECT devv5.FastRecreateSchema(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is not a part of load_stage or create input
GGR/auto_init.sql
Outdated
/* | ||
SELECT devv5.FastRecreateSchema(); | ||
/**************************************************************************/ | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the block is OK, should stay there.
also autors should be added, see how it's done in other vocabularies
WHERE mpp.ouc = 'C';--OUC means *O*ne, m*U*ltiple or pa*C*k | ||
LEFT JOIN SOURCES.GGR_SAM sam ON mpp.mppcv = sam.mppcv | ||
WHERE mpp.ouc = 'C' | ||
and mpp.hyr_ not like 'LA%';--OUC means *O*ne, m*U*ltiple or pa*C*k |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what is this change? new condition? need comment
); | ||
update |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
still don't know the convention(
* See the License for the specific language governing permissions and | ||
* limitations under the License. | ||
**************************************************************************/ | ||
/**************************************************************************/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the header was fine, add the author and year, see in other vocabs how it's done in load_stage
No description provided.