oss-aspen / 8Knot

Dash app in development to serve open source community visualizations using GitHub data from Augur. Hosted app: https://eightknot.osci.io
MIT License
46 stars 59 forks source link

Bot filtering #541

Closed cdolfi closed 9 months ago

cdolfi commented 9 months ago

This PR implements bot filtering. Decisions still need to be made around the styling of the filter switch itself and the overall app changes from the dash bootstrap components update

https://community.plotly.com/t/dbc-nav-styling-change-with-update/80094/7

cdolfi commented 9 months ago

Note: we need to look over the overall styling changes before merging this. Check out the hosted instance next to this one to see the difference @JamesKunstle

cdolfi commented 9 months ago

@JamesKunstle Ready for re-review. _bot.py is not in its final form as this solution causes that query to be called for every visualization and each time its rerun. Let me know how you think it should be changed to handle that

JamesKunstle commented 9 months ago

@JamesKunstle Ready for re-review. _bot.py is not in its final form as this solution causes that query to be called for every visualization and each time its rerun. Let me know how you think it should be changed to handle that

@cdolfi commented above

JamesKunstle commented 9 months ago

Note: we need to look over the overall styling changes before merging this. Check out the hosted instance next to this one to see the difference @JamesKunstle

I see the differences. I don't really mind them honestly. I'd like to merge this and then create a follow-up issue to fix styling rather than dragging this PR through that fix.

cdolfi commented 9 months ago

@JamesKunstle Sounds good, the zoom in bothers me a lot more than the darkened tabs but not a huge fan of either. Ill open an issue for that. Once the heatmap is merged ill rebase on that. Besides that this pr is good to go for me. Let me know if you still have issues with the query and we can figure it out. Not sure whats happening as its working for me

JamesKunstle commented 9 months ago

@JamesKunstle Sounds good, the zoom in bothers me a lot more than the darkened tabs but not a huge fan of either. Ill open an issue for that. Once the heatmap is merged ill rebase on that. Besides that this pr is good to go for me. Let me know if you still have issues with the query and we can figure it out. Not sure whats happening as its working for me

I actually don't have a zoom-in render on my end. I wonder if it's a screen-size thing?

JamesKunstle commented 9 months ago

@cdolfi once the conflict is fixed I'll merge!

oh also make sure to squash your commits

cdolfi commented 9 months ago

@JamesKunstle Ready to go! (cant squash any further without messing up relevant history)

JamesKunstle commented 9 months ago

@cdolfi there's a rebase conflict in color.css- if you fix that you can just force-push this branch into dev. If you want me to do it that's okay too

cdolfi commented 9 months ago

@JamesKunstle on my side that conflict is fixed. Let me know if its still showing for you Screenshot 2023-11-16 at 7 29 56 AM

JamesKunstle commented 9 months ago
Screenshot 2023-11-16 at 12 06 50 PM

lol github what are you doing

JamesKunstle commented 9 months ago

@cdolfi go ahead and merge it yourself, idk why the github UI is giving me a hard time