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

Restructured app #66

Merged
merged 1 commit into from
Nov 2, 2024
Merged

Restructured app #66

merged 1 commit into from
Nov 2, 2024

Conversation

minimalparts
Copy link
Member

This PR includes a lot of restructuring, meant to simplify the PeARS index. IDs linking database entries with numpy matrix rows are now included under the 'vector' attribute of the Urls model.

The PR also includes more minor changes:

  • some styling of the admin database;
  • direct indexing of URLs, without writing the information submitted by the user to a file first. NB: this very nearly means we could get rid of the userdata directory, but it is still needed to index pdfs, which are first (temporarily) downloaded and processed page-by-page;
  • improved admin access to database entries: the admin can now make changes to a number of fields in the database entry, automatically triggering appropriate changes in the pod structure and matrix shapes.

@nandajavarma
Copy link
Member

Thanks a lot for adding these cahnges @minimalparts ! Shall we merge this change?

@minimalparts
Copy link
Member Author

Thanks for reviewing, @nandajavarma ! Yes, I'll merge now.

@minimalparts minimalparts merged commit dd4527a into main Nov 2, 2024
1 check passed
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.

2 participants