pillargg / pillar_algos

Finds best timestamps to cut at
https://docs.pillar.gg/pillar_algos/
GNU General Public License v3.0
1 stars 0 forks source link

Automate AWS api #16

Closed pomkos closed 3 years ago

pomkos commented 3 years ago

Connect to AWS bucket, retrieve filenames with optional save. Retrieve n random names. Retrieve top 16 names (top 5 smallest size, top 5 largest, and the median 6).

Also some bug fixes related to algo1, 2. Limit variable to limit how many timestamps are returned from each algo.

chand1012 commented 3 years ago

@chand1012 may be able to help with cicd for incrementing the version number. Not 100% sure what goes into that though.

CI/CD for publishing to PyPI will be done in the coming days on a different branch, version numbers are generally done manually but I can look into automating those as well.

pomkos commented 3 years ago

@chand1012 Is command line merging preferred over using the github webgui?

chand1012 commented 3 years ago

@chand1012 Is command line merging preferred over using the github webgui?

No, but GitHub says it cannot merge automatically nor can you resolve it in the web GUI (which is actually kind of strange, GitHub is usually really good about these kind of things).

pomkos commented 3 years ago

I successfully merged, but I can't push the merge. You may have to take this one!

Total 6 (delta 4), reused 0 (delta 0)
remote: Resolving deltas: 100% (4/4), completed with 3 local objects.
remote: This repository moved. Please use the new location:
remote:   https://github.com/pillargg/twitch_chat_analysis.git
remote: error: GH006: Protected branch update failed for refs/heads/main.
remote: error: At least 1 approving review is required by reviewers with write access.
To https://github.com/pomkos/twitch_chat_analysis
 ! [remote rejected] main -> main (protected branch hook declined)
error: failed to push some refs to 'https://github.com/pomkos/twitch_chat_analysis'