Repository navigation
Build client-only Engine DJ harmonic/BPM visualization app (React + Vite + sql.js + D3) - #1
Conversation
Agent-Logs-Url: https://github.com/samsta/KeyBpmMap/sessions/c0f558bc-e1a9-4945-80b0-fb7e2ddfbe74 Co-authored-by: samsta <3915450+samsta@users.noreply.github.com>
|
@copilot in order to review and test before merge it would be good to deploy PRs as well, not just on main. |
Agent-Logs-Url: https://github.com/samsta/KeyBpmMap/sessions/f857f5bc-e007-47b0-9497-70c01e47ca6d Co-authored-by: samsta <3915450+samsta@users.noreply.github.com>
Agent-Logs-Url: https://github.com/samsta/KeyBpmMap/sessions/f857f5bc-e007-47b0-9497-70c01e47ca6d Co-authored-by: samsta <3915450+samsta@users.noreply.github.com>
Updated in e4f5ad2. PRs targeting |
|
@copilot I can't deploy even though I set copilot/create-dj-database-visualization-app as the deployment branch. What's the matter? And if we can only have one branch deploying, how do we do testing without affecting the live version before merging? |
Agent-Logs-Url: https://github.com/samsta/KeyBpmMap/sessions/e78b03c4-9939-4bef-9718-fb83f07af170 Co-authored-by: samsta <3915450+samsta@users.noreply.github.com>
Fixed in 588a8dc. The failing PR deploy was coming from |
Agent-Logs-Url: https://github.com/samsta/KeyBpmMap/sessions/5800cbb1-2129-4949-8b59-0124cb8cccab Co-authored-by: samsta <3915450+samsta@users.noreply.github.com>
Agent-Logs-Url: https://github.com/samsta/KeyBpmMap/sessions/5800cbb1-2129-4949-8b59-0124cb8cccab Co-authored-by: samsta <3915450+samsta@users.noreply.github.com>
Agent-Logs-Url: https://github.com/samsta/KeyBpmMap/sessions/5800cbb1-2129-4949-8b59-0124cb8cccab Co-authored-by: samsta <3915450+samsta@users.noreply.github.com>
| <h2>Top playlist transitions</h2> | ||
| <ul className="simple-list"> | ||
| {topTransitions.length > 0 ? ( | ||
| topTransitions.map((transition) => ( | ||
| <li key={transition.id}> | ||
| <div> | ||
| <strong>{transition.label}</strong> | ||
| <span>{transition.count} chained mix points</span> | ||
| </div> | ||
| </li> | ||
| )) | ||
| ) : ( | ||
| <li>No playlist transitions remain after the current filters.</li> | ||
| )} | ||
| </ul> |
There was a problem hiding this comment.
@copilot please remove this. We might re-introduce it once I have wrapped my head around a more useful feature along those lines
There was a problem hiding this comment.
Removed in c6a632b. Screenshot: /tmp/keybpmmap/insights-without-transitions-panel.png
Agent-Logs-Url: https://github.com/samsta/KeyBpmMap/sessions/5800cbb1-2129-4949-8b59-0124cb8cccab Co-authored-by: samsta <3915450+samsta@users.noreply.github.com>
|
Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
Agent-Logs-Url: https://github.com/samsta/KeyBpmMap/sessions/b935ceee-0146-452b-8d98-995d4a464519 Co-authored-by: samsta <3915450+samsta@users.noreply.github.com>
|
Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
Agent-Logs-Url: https://github.com/samsta/KeyBpmMap/sessions/3bbe7cd3-239e-4cc7-86f3-c9ce760a9282 Co-authored-by: samsta <3915450+samsta@users.noreply.github.com>
|
@copilot add a version field that's displayed somehwere on the page. Is it easier to set that version manually, or should we use a git tag for this? I think we should just start at 0.1 and bump the minor in every PR |
|
Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
Agent-Logs-Url: https://github.com/samsta/KeyBpmMap/sessions/c8e26471-412f-4aac-ba26-e4f809f916b7 Co-authored-by: samsta <3915450+samsta@users.noreply.github.com>
Agent-Logs-Url: https://github.com/samsta/KeyBpmMap/sessions/c8e26471-412f-4aac-ba26-e4f809f916b7 Co-authored-by: samsta <3915450+samsta@users.noreply.github.com>
Agent-Logs-Url: https://github.com/samsta/KeyBpmMap/sessions/c8e26471-412f-4aac-ba26-e4f809f916b7 Co-authored-by: samsta <3915450+samsta@users.noreply.github.com>
Implemented in 3daf4a4. I went with a manually bumped Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
@copilot sounds great. Can you please add some instructions for copilot reviews to ensure the version is updated in each PR, and it is monotonically increasing |
Agent-Logs-Url: https://github.com/samsta/KeyBpmMap/sessions/f0d861be-38c4-4937-a76e-76759a62930e Co-authored-by: samsta <3915450+samsta@users.noreply.github.com>
Agent-Logs-Url: https://github.com/samsta/KeyBpmMap/sessions/f0d861be-38c4-4937-a76e-76759a62930e Co-authored-by: samsta <3915450+samsta@users.noreply.github.com>
Agent-Logs-Url: https://github.com/samsta/KeyBpmMap/sessions/f0d861be-38c4-4937-a76e-76759a62930e Co-authored-by: samsta <3915450+samsta@users.noreply.github.com>
Added in c30078e. I created |
Original prompt