Comments (24)
@dantp-ai I have made some changes , but idk if it is correct or wrong , so let me send a pull request shortly and kindly can you go through it ?
from tianshou.
@dantp-ai check #1125
Seems like i kinda messed up , T_T
from tianshou.
@DarkTechPirate wanna have a look at this one? It's a fairly small thing and a good way to get started with contributing :)
from tianshou.
@MischaPanch yea sure
from tianshou.
Cool, thanks!
from tianshou.
can this also be changed into len(batch.get_keys()) == 0
from tianshou.
Not sure what you mean. How is this related to is_empty()
?
from tianshou.
in 5th line we are using is_empty()
to check it right!?
if we remove that fun it wont work , so we need to replace it with something right
from tianshou.
Yes, you're right, hadn't seen it. I think a full-text search of is_empty
should reveal all usages, in case the find usages of your IDE misses some
from tianshou.
@MischaPanch I'd be glad to assist with this one for review in PR.
from tianshou.
@MischaPanch I'd be glad to assist with this one for review in PR.
Sounds good, thanks!
from tianshou.
@DarkTechPirate do you have an ETA for when you could submit a PR?
from tianshou.
@MischaPanch so can we have a goggle meet or something as soon all my doubts clear i can finish it in 1 or 2 hours , let me know is it even possible ?
from tianshou.
@dantp-ai hey sure , we shall also have a meeting when you are free , lmk
from tianshou.
I won't have time until the end of next week, but @dantp-ai knows what this issue is about, so if the two of you have time to talk to each other, that would be the fastest option :). It's not a big change
from tianshou.
@DarkTechPirate I'd be happy to help. We can look at it tomorrow.
from tianshou.
@MischaPanch Okay sure , we will finish it off asap
from tianshou.
Thank you two, highly appreciate it!
from tianshou.
@dantp-ai how to contact you , discord or you people use something else !?
its my first time working with someone , so sorry for inconvenience!
from tianshou.
@DarkTechPirate Once you have a solution (a work-in-progress is fine, though a complete solution is greatly appreciated), please open a PR and we can review the code together here on GitHub.
The above description of the issue is instructive and should contain all the hints. If you have any doubts, feel free to write them down here and I will be able to help you.
I assume you have already read the document on contributing to the codebase?
from tianshou.
@DarkTechPirate How can I help?
from tianshou.
Yes, let's do like that. Looking forward to the PR.
from tianshou.
@DarkTechPirate I didn't see the PR. If you made some changes let's review them together in the PR even if you are unsure of the changes.
Pls add me as reviewer. Thank you!
from tianshou.
okay
from tianshou.
Related Issues (20)
- MPO Implementation HOT 1
- Improve interface of BasePolicy.compute_action HOT 1
- Suggestion - Redesign RayEnvWorker for Improved Performance HOT 12
- tianshou v1.0.0 failed to install on python 3.12.4 HOT 2
- How to add a obs with multiple dimensions to the buffer?
- TypeError: isinstance() argument 2 cannot be a parameterized generic HOT 1
- Make Batch.__eq__ work also for scalar values (0-dimensional arrays) HOT 2
- EOFError on training HOT 8
- ModuleNotFoundError: No module named 'tianshou.highlevel' HOT 3
- Cannot unify float64 and array(float64, 1d, C) for 'gae.2'
- Discrete distributions are broken HOT 2
- Actor and distribution function should not be specified independently HOT 1
- Learning rate scheduler confusion HOT 1
- Handling LR scheduling when episodes instead of steps are collected
- Release 1.1.0 to pypi HOT 1
- Refactoring: make Trainer more flexible
- Support custom scores for Trainer in high-level interfaces
- Prioritized Replay Buffer: Anneal Beta or not? HOT 5
- Towards Release 2.0.0 HOT 11
- Question about Actor's outputs and selected actions
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from tianshou.