Skip to content
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

New option/config showconcise #170

Merged
merged 3 commits into from
Mar 15, 2022
Merged

Conversation

navinrathore
Copy link
Contributor

To control the display of fields of matchType "DONT_USE" during Labelling and updateLabelling, following has been added.
a) --showConcise cli option
b) showconcise config parameter

#169

@navinrathore navinrathore changed the title Z labeller New option/config showconcise Mar 14, 2022
@sonalgoyal sonalgoyal merged commit a64f944 into zinggAI:main Mar 15, 2022
@navinrathore navinrathore deleted the zLabeller branch April 4, 2022 12:14
@tomdavidson
Copy link

@navinrathore when I use the showConcise param in the labeling phase, I get the following:

$ podman run -it -v $(pwd):/zingg/project zingg  --conf config.json --phase label --showConcise
 19:04:38.18 
 19:04:38.19 Welcome to the Bitnami spark container
 19:04:38.20 Subscribe to project updates by watching https://github.com/bitnami/bitnami-docker-spark
 19:04:38.20 Submit issues and feature requests at https://github.com/bitnami/bitnami-docker-spark/issues
 19:04:38.20 

2022-04-12 19:04:40,656 [main] WARN  org.apache.hadoop.util.NativeCodeLoader - Unable to load native-hadoop library for your platform... using builtin-java classes where applicable
 2022-04-12 19:04:40,971 [main] INFO  zingg.client.Client - 
 2022-04-12 19:04:40,979 [main] INFO  zingg.client.Client - ********************************************************
 2022-04-12 19:04:40,987 [main] INFO  zingg.client.Client - *                    Zingg AI                          *
 2022-04-12 19:04:40,995 [main] INFO  zingg.client.Client - *               (C) 2021 Zingg.AI                      *
 2022-04-12 19:04:41,009 [main] INFO  zingg.client.Client - ********************************************************
 2022-04-12 19:04:41,015 [main] INFO  zingg.client.Client - 
 2022-04-12 19:04:41,023 [main] INFO  zingg.client.Client - using: Zingg v0.3.3
 2022-04-12 19:04:41,031 [main] INFO  zingg.client.Client - 
 2022-04-12 19:04:41,046 [main] WARN  zingg.client.ClientOptions - Could not recognize argument [email protected]
 2022-04-12 19:04:41,056 [main] WARN  zingg.client.ClientOptions - zingg.sh [options]
zingg.sh --phase train --conf config.json
zingg.sh --phase findTrainingData --conf config.json --zinggDir /location 
zingg.sh --phase label --conf config.json --zinggDir /location 
zingg.sh --phase trainMatch --conf config.json --email [email protected]
zingg.sh --phase findAndLabel --conf config.json --zinggDir /location
options
         --preprocess:          convert files to unix format
         --phase:               train|match|trainMatch|findTrainingData|label|link|generateDocs|updateLabel|findAndLabel
         --h:                           print usage information
         --showConcise:         Display only fields that are used to make model, true/false  
         --license:             location of license file
         --zinggDir:            location of Zingg models, defaults to /tmp/zingg
         --jobId:               database job id for logging
         --help:                print usage information
         --conf:                JSON configuration with data input output locations and field definitions
         --email:               notification email id. Can be an alias
         --collectMetrics:              collect analytics, true/false  
         --format:              format of the data
         --modelId:             model identifier, can be a number 

 2022-04-12 19:04:41,063 [main] WARN  zingg.client.Client - Apologies for this message. Zingg has encountered an error. Do not recognize option [email protected]

But only when using --showConcise. This is v0.3.3.

@sonalgoyal
Copy link
Member

sonalgoyal commented Apr 13, 2022 via email

@tomdavidson
Copy link

@sonalgoyal yep, that's it. Seems a bit funny ux, but clearly my not reading the instructions well enough.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants