Skip to content

[WIP] Drive per-sample external source with batch size from batched ones.#2680

Draft
mzient wants to merge 1 commit into
NVIDIA:mainfrom
mzient:PerSampleExternalSourceVarBatch
Draft

[WIP] Drive per-sample external source with batch size from batched ones.#2680
mzient wants to merge 1 commit into
NVIDIA:mainfrom
mzient:PerSampleExternalSourceVarBatch

Conversation

@mzient

@mzient mzient commented Feb 16, 2021

Copy link
Copy Markdown
Contributor

Signed-off-by: Michał Zientkiewicz mzient@gmail.com

Why we need this PR?

Pick one, remove the rest

  • It fixes a bug: per-sample external source don't adjust to dynamic batch size

What happened in this PR?

Fill relevant points, put NA otherwise. Replace anything inside []

  • What solution was applied:
    • Run batched External Source first
    • Use actual batch size from batched ES in per sample ES
  • Affected modules and functionalities:
    • pyhon: Pipeline, ExternalSource
  • Key points relevant for the review:
    • N/A
  • Validation and testing:
    • Python unit tests
  • Documentation (including examples):
    • N/A

JIRA TASK: N/A

Signed-off-by: Michał Zientkiewicz <mzient@gmail.com>
@mzient mzient requested a review from a team February 16, 2021 10:40
@awolant awolant self-assigned this Feb 16, 2021
@mzient mzient changed the title Drive per-sample external source with batch size from batched ones. [WIP] Drive per-sample external source with batch size from batched ones. May 12, 2021
@mzient mzient marked this pull request as draft May 12, 2021 08:42
@JanuszL JanuszL changed the base branch from master to main May 19, 2021 12:23
@klecki klecki removed their assignment Jul 23, 2021
@awolant awolant removed their assignment Sep 21, 2021
@NVIDIA NVIDIA deleted a comment from dali-automaton Jan 27, 2022
@dali-automaton

Copy link
Copy Markdown
Collaborator

CI MESSAGE: [3840108]: BUILD FAILED

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.

5 participants