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

PM5 Part 1 #15

Merged
merged 5 commits into from
Apr 7, 2023
Merged

PM5 Part 1 #15

merged 5 commits into from
Apr 7, 2023

Conversation

Bluesy1
Copy link
Contributor

@Bluesy1 Bluesy1 commented Apr 6, 2023

Alongside initial pass at my portion of the dashboard, I've also fixed the python file import issue by renaming the code directory to scripts.

  • While I was doing that, I also sorted/reformatted imports to follow PEP 8's Imports Guide for consistency

Other notes:

  • As referenced in Feedback Session # 2 #14 during project feedback hours, I've added a non geographic reliant plot to help explain part of my timeseries data.
  • I've added a .gitattributes to help with diffs and merges with certain files, and to make sure git hopefully auto normalizes EOL characters
  • I've updated the main README.md file to mention a few extra optional dependencies that can help with speedups, that I've found helpful and are suggested in the docs of pandas.

Bluesy1 added 4 commits April 5, 2023 18:59
Also update all analysis notebook imports to account for this
- While doing this, updated all imports in the project to follow PEP8 ordering
During the last feedback session, I was asked to create a non geographical plot

Refs: #14
@Bluesy1 Bluesy1 added this to the PM5 milestone Apr 6, 2023
@Bluesy1 Bluesy1 requested a review from theHDarian April 6, 2023 03:42
@Bluesy1 Bluesy1 self-assigned this Apr 6, 2023
@theHDarian theHDarian merged commit e6ea776 into main Apr 7, 2023
@Bluesy1 Bluesy1 deleted the Gavin-PM5 branch April 7, 2023 19:20
@Bluesy1 Bluesy1 mentioned this pull request Apr 8, 2023
3 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants