Shoaib Burq

Results 48 comments of Shoaib Burq

@Roy-Mao as much as I appreciate you taking interest in the gem I would like us to make sure that the gem stays simple and easy to maintain. So the...

> Can I have write access to this repository so that I can create a new branch for this improvement? @Roy-Mao let's do a joint review first. Just let me...

@Roy-Mao yes that would be awesome let's do a call. Can you send me meeting request for later next week via https://calendly.com/sabman and we can do a paired code review

@chrisedington I was waiting for @Roy-Mao to add tests. But seems he's busy now. I'll take a look next week.

I am thinking of adding support for `.env` and/or `ENV` variables to overwrite the defaults. Thoughts?

Yes we can probably add other items too. Only thing I'd wanna ensure is avoid accidental name clashes. So maybe namespacing the environment variables with `FCM_VAR_XXX` might be the way...

@derhuerst is the main blocker here the need to have `geometry` types instead of `geography` types? Shall we look into a flag to set the types in PostGIS? Thanks!

@ashish-jajoria thanks for this. Can you please add the tests and also reference the documentation on Firebase for this change? thanks!

checkout https://github.com/daniel-cortez-stevenson/aws-athena-udfs-h3/blob/main/src/main/resources/sql/create_planet_h3.sql

@RaczeQ Thanks for creating this issue. I'd like to see if I can contribute. I am assuming this is in reference to the training loop for the embedding models? If...