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

Added dataset in JSON format #6

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

gunarto90
Copy link

As Python 2 is not supported anymore by Jan 2020, some people including myself cannot access the pickled dataset. Moreover, the dataset can only be opened using a Linux environment which may hinder a Windows user to access it. Thus, I created a "JSON" format for the dataset as it might be easier to understand the dataset and because the original data are based on the dictionary of dictionaries, JSON format is suitable for representing them.

Gunarto Sindoro Njoo added 4 commits June 3, 2020 11:17
…oring done

1. Refactor the snippet for max aggregation (i.e., line 86-90) using default dict for a simpler logic
https://github.com/vonfeng/DeepMove/blob/0acff5d88d70d31a4fcf580648e0056e103dc6cf/codes/train.py#L86

2. Refactor the snippet for avg aggregation (i.e., line 106-116) using Counter for a simpler logic
https://github.com/vonfeng/DeepMove/blob/0acff5d88d70d31a4fcf580648e0056e103dc6cf/codes/train.py#L106

3. Move the assignment of "history_count" inside the if logic to reduce redundancy
https://github.com/vonfeng/DeepMove/blob/0acff5d88d70d31a4fcf580648e0056e103dc6cf/codes/train.py#L124

4. Refactor the variable "loc_tim" in "generate_input_long_history2" since it is unnecessary

5. Refactor the history, count extraction, and torch_trace

6. Fixed the indentation for "generate_input_long_history2"

7. Added commentary for foursquare dataset structure
Fixed indentation for generate_input_long_history2 and refactoring
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.

None yet

1 participant