Rohit Malhotra

Results 104 comments of Rohit Malhotra

Fixed merge issues, please double check. Thanks for the discussions and feedback so far!

@openhands please fix lint issues on both the backend and frontend

Ah maybe this can be triggered behind an env variable then, which is kept off by default

Okay this is interesting, including the timestamp resulted in 53% resolve rate compared to the baseline 50% (no timestamps)

I'm fairly happy with the current changes, so I'd like to get a review on this. The PR contains many changes, but a few notable areas are 1. [openhands/server/config/openhands_config.py](https://github.com/All-Hands-AI/OpenHands/pull/5371/files#diff-136d34f31c20675994b2c87979c4245a79a9343bafbe8643aca0ad50193bc701) contains...

Yeah definitely! I'm in agreement with everything said so far Ideally we open a separate issue so that 1. We can openly discuss what an ideal taxonomy of settings looks...

Hmm this is a bit weird, will take a look into this soon

Hey @josx! I'd recommend taking a look at [`Openhands/openhands/resolver`](https://github.com/All-Hands-AI/OpenHands/tree/main/openhands/resolver). All of the resolver logic is contained within this folder. The files [`Openhands/openhands/resolver/issue_definitions.py`](https://github.com/All-Hands-AI/OpenHands/blob/main/openhands/resolver/issue_definitions.py) (downloads issues) and [`Openhands/openhands/resolver/send_pull_request.py`](https://github.com/All-Hands-AI/OpenHands/blob/main/openhands/resolver/send_pull_request.py) (uploads code) use Github...

Closing issue as solved by #6458