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

testing application data set and steps #23

Open
cambro opened this issue Oct 11, 2016 · 3 comments
Open

testing application data set and steps #23

cambro opened this issue Oct 11, 2016 · 3 comments

Comments

@cambro
Copy link
Member

cambro commented Oct 11, 2016

this needs to be added. currently, running the test application gives this:
cursor.execute("SELECT * FROM stringed_instruments_sentences_nlp352;") psycopg2.ProgrammingError: relation "stringed_instruments_sentences_nlp352" does not exist LINE 1: SELECT * FROM stringed_instruments_sentences_nlp352;

The reason is because the tables defined by the steps to import the testing set are not correct. I would make the testing set data = 1 data set for all steps involved here. So, let's make this data from "Getting Started":

curl -o example_input.zip https://geodeepdive.org/dev_subsets/example_input.zip unzip -j example_input.zip -d ./input rm example_input.zip

Be the same data that is used by the example application, and perhaps get rid of weird table names etc. in the postgres database.

@cambro
Copy link
Member Author

cambro commented Oct 11, 2016

@iross When fed-up with ingest documentation and processing overhaul, do a "from scratch" app-template follow through as if you were newbie. This helps find errors.

@iross
Copy link
Member

iross commented Oct 11, 2016

Part of the issue is that the script we use to read the example_input into tables was looking for config instead of config.yml, which has been fixed.

The other issue is that getting the example_input into the database is accomplished via a make local_setup, which gets a little buried in the 'Getting started' guide. While it'd be nice to have one copy/paste block to run the example app, there's always going to be the step of modifying the credentials to match the local postgres setup. But I do think there's value in having a more streamlined quickstart/how to run the example app wiki page (and I think we've had a placeholder for it at https://github.com/UW-Deepdive-Infrastructure/app-template/wiki/Running-the-example-application)

@cambro
Copy link
Member Author

cambro commented Oct 12, 2016

Yes, that's the idea. This placeholder page is what caught my attention and led me to try and run the example app. As you say, it would really help if this were streamlined for users (config. for db. connection is a separate issue that IS in fact addressed in the documentation).

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

No branches or pull requests

2 participants