Madison Swain-Bowden

Results 175 comments of Madison Swain-Bowden
trafficstars

Oh, it looks like we'll also want to add that condition here too: https://github.com/WordPress/openverse/blob/c0105eb1b89aae9d3548f42ef69be52c5e863609/catalog/dags/providers/provider_api_scripts/inaturalist.py#L331 Apologies for missing that! Using the Variable there as well will prevent the dataset from being...

@madewithkode thanks for making those changes! I was able to test this locally, and opted to make two other changes: - Set the default of the parameter to `False`, since...

@munch-lax actually, I think a better option would be the approach outlined in #2295! I'm going to close this issue in favor of that. Do you feel like you'd be...

Aaaaaand everything is passing today 😅 Sorry about the false alarm!

> I question the idea that we should deliberately duplicate tags and therefore boost any result with machine tags. I also don't fully understand why in the case of deduplication,...

Thanks for your thoughts folks 🙂 I'll wait for @obulat to chime in before making any revisions or changes.

Drafting while I revise this based on the clarification round!

Based on a short synchronous chat that several maintainers had regarding this project during our recent priorities meeting, I'm going to move forward with the same approach laid out in...

Document has been revised, and ready for a decision!

> I missed the extra case. It sounded not so complicated when you mentioned it in the sync discussion. I ended up adding an extra caveat to an existing section...