Skip to content
This repository has been archived by the owner on Dec 10, 2023. It is now read-only.

Integration of WIP Branch to Main Branch #11

Merged
merged 13 commits into from
Jan 25, 2022
Merged

Integration of WIP Branch to Main Branch #11

merged 13 commits into from
Jan 25, 2022

Conversation

georgetayqy
Copy link
Owner

Updates to App

  • Restructured App Files and Functions
  • Refined File Download Button
  • Improved Efficiency with spaCy, pandas and Python Lists
  • Fixed an issue whereby rerunning the app after the source code has been edited would result in an ValueError Exception raised as Python was unable to find the function in the List iterable used to store the functions
  • Initial Introduction and Final Deprecation of use of Session State in app

@georgetayqy
Optimisation and Cleanup
99760e5
@georgetayqy
Optimisation of Sentiment Analysis Module
3d2368e 
UI/UX Changes were also made to the app
@georgetayqy
Ignore PyCharm Config Files
e2401b5
@georgetayqy
Added Statefulness to LCV
74ea84c 
Deprecated the use of HTML buttons and instead opted for Streamlit's own download button to download data

Statefulness will be progressively rolled out to the other apps over the course of the week
@georgetayqy
Fixed the Excel File Downloaders
6a08d42
@georgetayqy
Added Statefulness to the Other Apps
d7fc97b 
Statefulness is still being tested as so far it does not allow for the remembering of state for more than 3 nested buttons
@georgetayqy
Shuffled Options Selection Around
aedbeb0 
Tests on the state of the app has not been conducted yet, will be done in due time
@georgetayqy
Reversion of Statefulness
ec14b82 
Error with ensuring state across NLP Toolkit resulted in the reversion back to the non-stateful verison of the app. Max Message Size has been set to 100 GB to account for larger datasets that may be required to display to the screen and for large file downloads.
@georgetayqy
Merge branch 'wip' of https://github.com/asdfghjkxd/ArticPy into wip
6220b92
@georgetayqy
Removed PyCharm Artifacts
d616ae5
@georgetayqy
Amended some parts of the code
8a8f2e7
@georgetayqy
Updated App Structure
6ccb4ba 
Removed app state from main app functions, added new configs and options to allow for a greater amount of data to be displayed to screen
@georgetayqy
Ignore XML files generated by PyCharm
2f750fe
@georgetayqy georgetayqy added the enhancement New feature or request label Jan 25, 2022
@georgetayqy georgetayqy self-assigned this Jan 25, 2022
@georgetayqy georgetayqy linked an issue Jan 25, 2022 that may be closed by this pull request
WIP: Fixes for Finals #6
Closed
@georgetayqy georgetayqy merged commit e94b28f into main Jan 25, 2022
@georgetayqy georgetayqy deleted the wip branch January 25, 2022 13:27
@georgetayqy georgetayqy restored the wip branch January 29, 2022 05:30
@georgetayqy georgetayqy deleted the wip branch January 29, 2022 05:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@Jungstershark Jungstershark Awaiting requested review from Jungstershark

Assignees

@georgetayqy georgetayqy

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

WIP: Fixes for Finals
1 participant
@georgetayqy