Apurva Banka

Results 22 comments of Apurva Banka

@cleder I can start working on this issue. Can you go ahead and assign this issue to me?

@cleder Created pull request for the same. Please review the PR. Didn't include the tests cases for fastkmltime.py as I see there is a refactor issue for the same. Another...

@geekygirlsarah I would like to contribute. Can you assign this to me?

@cleder Can you assign me this issue?

@cleder This change requires to install "xmldiff". Should this be added to the requirement.txt or the pyproject.toml.? For pyproject.toml, how do we add the dependency? I have used poetry in...

@cleder Trying to understand what we are comparing here. I tried to create a KML file after using the Parse function on "Document-clean.kml" file. I see there are components that...

@cleder Created the issue https://github.com/cleder/fastkml/issues/390. Can you verify if any fields are missing?

@bh2smith I would like to contribute. Can you assign me this issue?

List of Functions that need to be revamped. ************* Module dune_client.client_async dune_client/client_async.py:232:4: R0917: Too many positional arguments (7/5) (too-many-positional-arguments) dune_client/client_async.py:266:4: R0917: Too many positional arguments (7/5) (too-many-positional-arguments) dune_client/client_async.py:356:4: R0917: Too...

@bh2smith How to handle params for create_table function? https://github.com/duneanalytics/dune-client/blob/7d7d14ea950845fc9d715dada33d29c574c237ad/dune_client/api/table.py#L55-L62 We can create a dict with all the params and pass it in line 75. https://github.com/duneanalytics/dune-client/blob/7d7d14ea950845fc9d715dada33d29c574c237ad/dune_client/api/table.py#L75-L81 Thoughts?