Skip to content

(fix): fix reset data idx.#76

Merged
lkevinzc merged 1 commit intoaxon-rl:mainfrom
PanAndy:fix/reset_idx
Aug 28, 2025
Merged

(fix): fix reset data idx.#76
lkevinzc merged 1 commit intoaxon-rl:mainfrom
PanAndy:fix/reset_idx

Conversation

@PanAndy
Copy link
Contributor

@PanAndy PanAndy commented Aug 27, 2025

self.idx = random.randint(0, len(self.dataset) - 1) may get idx=len(self.dataset), which will cause out of index error.

    def randint(self, a, b):
        """Return random integer in range [a, b], including both end points.
        """

        return self.randrange(a, b+1)

@lkevinzc lkevinzc self-requested a review August 28, 2025 00:26
@lkevinzc lkevinzc merged commit 799ee80 into axon-rl:main Aug 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants