Display results of proximity analysis to user #57
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
Co-Authored-By: j-aslarus <85576329+j-aslarus@users.noreply.github.com>
commit 427e51e Merge: a267410 590b56e Author: Peihua Huang <32581282+phuang00@users.noreply.github.com> Date: Wed Jul 21 19:35:28 2021 -0400 Merge pull request #46 from dhmit/create_single_corpus_component Create Single Corpus Component commit 590b56e Author: Peihua Huang <nemesis2126@gmail.com> Date: Wed Jul 21 19:35:18 2021 -0400 updated map function such that the data is first filtered then mapped commit 2d88e42 Author: Peihua Huang <nemesis2126@gmail.com> Date: Wed Jul 21 19:32:13 2021 -0400 updated heading tags and loading states commit 584c092 Merge: ddfa24d ead126f Author: Peihua Huang <nemesis2126@gmail.com> Date: Mon Jul 19 17:31:48 2021 -0400 Merge branch 'main' into create_single_corpus_component commit ddfa24d Merge: f2fd0e1 e0d24bc Author: Peihua Huang <nemesis2126@gmail.com> Date: Mon Jul 19 17:22:06 2021 -0400 Merge branch 'main' into create_single_corpus_component commit f2fd0e1 Author: Peihua Huang <nemesis2126@gmail.com> Date: Fri Jul 16 17:03:50 2021 -0400 italicized the title of documents and fixed linter warnings commit 9e1e249 Author: Peihua Huang <nemesis2126@gmail.com> Date: Fri Jul 16 16:58:01 2021 -0400 update corpus page to show list of documents in corpus and allows users to use a modal to update documents in the corpus commit 057d88d Author: Peihua Huang <nemesis2126@gmail.com> Date: Fri Jul 16 15:27:17 2021 -0400 update API url and the way we check if a corpus contains a document commit a8d3e18 Author: Peihua Huang <nemesis2126@gmail.com> Date: Thu Jul 15 16:32:18 2021 -0400 update the way documents list is passed into api commit f3dc588 Author: Peihua Huang <nemesis2126@gmail.com> Date: Thu Jul 15 16:07:09 2021 -0400 update function naming typo commit b474af5 Merge: 593547e 9b828d9 Author: Peihua Huang <nemesis2126@gmail.com> Date: Thu Jul 15 15:26:21 2021 -0400 Merge branch 'main' into create_single_corpus_component commit 593547e Author: Peihua Huang <nemesis2126@gmail.com> Date: Wed Jul 14 15:46:04 2021 -0400 added list of documents that can be added to corpus commit 2f2209a Author: Peihua Huang <nemesis2126@gmail.com> Date: Wed Jul 14 14:55:55 2021 -0400 added path and get route to backend commit 7eed0c0 Author: Peihua Huang <nemesis2126@gmail.com> Date: Wed Jul 14 14:47:27 2021 -0400 created corpus component
…of extraneous variables
…analysis_web into proximity_frontend
…analysis_web into proximity_frontend
(and deleted all references to it in `Corpora.js`)
in `Corpora.js` and `Corpus.js`
…analysis_web into proximity_frontend
Also resolved some linter/syntax errors
Testing the overall flow of data across the network
…analysis_web into proximity_frontend
…/gender_analysis_web into proximity_frontend" This reverts commit 039f649, reversing changes made to 594a549.
…om/dhmit/gender_analysis_web into proximity_frontend"" This reverts commit f1595f4.
To test overall flow of data
The API endpoint works! Now all that's left to do is work out a good display.
(Displaying anonymous authorship and adding an ellipsis)
Co-Authored-By: irinazoccolini <71565864+irinazoccolini@users.noreply.github.com>
Co-Authored-By: irinazoccolini <71565864+irinazoccolini@users.noreply.github.com>
(Based on description lists) Co-Authored-By: irinazoccolini <71565864+irinazoccolini@users.noreply.github.com>
Co-Authored-By: irinazoccolini <71565864+irinazoccolini@users.noreply.github.com>
Includes only showing the 10 most common words for each NLTK part-of-speech tag Co-Authored-By: irinazoccolini <71565864+irinazoccolini@users.noreply.github.com> Co-Authored-By: Watermelanie <46464201+Watermelanie@users.noreply.github.com> Co-Authored-By: yaroluchko <85574367+yaroluchko@users.noreply.github.com> Co-Authored-By: j-aslarus <85576329+j-aslarus@users.noreply.github.com> Co-Authored-By: Ashar Farooq <ashar744@gmail.com>
commit 0fdfcc6 Merge: 1276da0 b10cfa9 Author: Peihua Huang <32581282+phuang00@users.noreply.github.com> Date: Fri Jul 23 15:48:02 2021 -0400 Merge pull request #25 from dhmit/transfer_frequency_module_PR1 Transfer frequency module pr1 commit 1276da0 Merge: 657b20f b5a9482 Author: YifanWang0 <47550739+YifanWang0@users.noreply.github.com> Date: Fri Jul 23 15:38:12 2021 -0400 Merge pull request #54 from dhmit/transfer_frequency_model Added FrequencyAnalysis model commit b5a9482 Author: Yifan Wang <yifan.wang0422@gmail.com> Date: Fri Jul 23 14:08:54 2021 -0400 added FrequencyAnalysis model commit b10cfa9 Author: Peihua Huang <nemesis2126@gmail.com> Date: Fri Jul 23 13:41:17 2021 -0400 update run_analysis to use primary keys instead of labels and update result dictionary to key by gender object commit 1196fc1 Author: Peihua Huang <nemesis2126@gmail.com> Date: Fri Jul 23 13:36:54 2021 -0400 Revert "update frequency result to key by gender primary key instead of label" This reverts commit 151a4bc. commit 151a4bc Author: Peihua Huang <nemesis2126@gmail.com> Date: Fri Jul 23 13:34:11 2021 -0400 update frequency result to key by gender primary key instead of label commit 0684a02 Author: Peihua Huang <nemesis2126@gmail.com> Date: Thu Jul 22 16:43:50 2021 -0400 got frequency analysis working with a corpus commit 2003ce6 Author: Peihua Huang <nemesis2126@gmail.com> Date: Thu Jul 22 16:28:27 2021 -0400 convert run_analysis to run_single_analysis, such that the function only takes in one document and update docstrings and added test commit 0f817ba Merge: dc21391 6619e68 Author: Peihua Huang <nemesis2126@gmail.com> Date: Thu Jul 22 14:54:15 2021 -0400 Merge branch 'main' into transfer_frequency_module_PR1 commit dc21391 Author: Yifan Wang <yifan.wang0422@gmail.com> Date: Wed Jul 7 16:44:55 2021 -0400 fixed small mistakes according to pr comments commit a852cd3 Author: Yifan Wang <yifan.wang0422@gmail.com> Date: Mon Jun 28 16:42:28 2021 -0400 added docstrings commit 32731f4 Author: Yifan Wang <yifan.wang0422@gmail.com> Date: Mon Jun 28 16:01:57 2021 -0400 fixed minor bug with _run_analysis function commit 233e2e6 Author: Yifan Wang <yifan.wang0422@gmail.com> Date: Fri Jun 25 16:52:31 2021 -0400 moved over run analysis but still need to fix bug with helper function
joshfeli
assigned
farooqashar,
joshfeli,
Watermelanie,
irinazoccolini,
yaroluchko and
j-aslarus
(not sure if this is needed)
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.
This PR presents an MVP for displaying the results of running
proximity.run_analysisto the user. It features a modal to run the analysis, some copied-and-pasted code from thecreate_API_endpoint_for_proximityAnalysisbranch for an API endpoint, some extra state variables to hold the results of the analysis, and some basic styling for showing the most frequently occurring words and indenting.