260 show extractor metadata #274
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…rate-extractor-parameter-forms' into 252-use-rjsf-to-dynamically-generate-extractor-parameter-forms
nothing shows yet
… does not display anything
very basic, only works for non-nested metadata
…etadata collapsible
Closed
Contributor
Author
|
I realize I need to add the tab for datasets - right now it just works for files. Should be done soon. |
Contributor
Author
|
Extractor metadata added for datasets now. |
lmarini
changed the base branch from
main
to
252-use-rjsf-to-dynamically-generate-extractor-parameter-forms
lmarini
approved these changes
Jan 17, 2023
lmarini
merged commit 593c1a8
into
252-use-rjsf-to-dynamically-generate-extractor-parameter-forms
lmarini
added a commit
that referenced
this pull request
Jan 17, 2023
* correctly model the extractorInfo * plugin rjsf * rjsf working now * pass in datasetId and fileId to Listeners cannot click on listener unless a file or dataset is selected * dataset and fileid undefined in submitextraction. * moving dispatch * submit extractor now gets parameters correctly. * submit does not work, sending requestBody * adding model for EventListenerSubmit parameters not currently included * extraction does not submit - currently not sending parameters * changing - sending in requestBody but still does not submit * adding back name, does not submit * redeem config change and put example in script * new entry in uvicorn * match codegen * fix submit button and action * correct the file/extract route request params * fix file extraction * fix dataset extraction submission as well * routing key doesn't have extractor.dot there * codegen * fix frontend * fix condition * revert run uvicorn * change 'extractor' to 'listener' in build metadata * add stepper back * close the modal will start over * add listenerInfo * add padding * 253 implement extractor filtering by category (#264) * seperate out the search box of listeners to another component * backend add filter capability * add category filter to the endpoint * add mock filter * automatically generate the fitler options * filter works * frontend handle interaction between searchbox and filters * add an endpoint to get all the distinct categories field * connect frontend with category endpoint * add endpoint to list distinct default labels * add codegen * properly wire the filter * 260 show extractor metadata (#274) * fixing uvicorn, this got overwritten with local (i think by me) * new tab, placeholder * gets metadata with listeners nothing shows yet * adding separate component for each listener metadata entry, currently does not display anything * adding component for contents and listener agent very basic, only works for non-nested metadata * working on nested metadata, only displays top level correctly * json displays properly, need to put each metadata in a box and make metadata collapsible * toggle. can hide or view metadata now * formatting. pipenv run black app * adding new tab to dataset for extractor metadata * changing tab order, tab names, and setting metadata json open by default * metadata from extractors appears in solid boxes. change in font for listener agent. * changing tab order, names for dataset * Added card to extracted metadata entries and fixed grid around them. Co-authored-by: Luigi Marini <[email protected]> Co-authored-by: toddn <[email protected]> Co-authored-by: Todd Nicholson <[email protected]> Co-authored-by: Luigi Marini <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
With this pull request, metadata from extractors/listeners is visible under a new tab. Right now the data can be viewed, with each entry having a toggle to view/hide metadata.
For metadata, the following file includes some sample extractor metadata. Some of the fields are not meaningful but were included to make the json nested.
file_metadata.txt