Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Visual Reasoning Collection #1298

Closed
wants to merge 100 commits into from
Closed

Visual Reasoning Collection #1298

wants to merge 100 commits into from

Conversation

tkornuta-nvidia
Copy link
Contributor

For now - a draft only, just to check CI

@tkornuta-nvidia
work in progress: simplification of structured configs
3832512 
Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
@tkornuta-nvidia
test with defaults
d4b1125 
Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
@tkornuta-nvidia
work in progress - issues with set_hyperparams
f32805b 
Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
@tkornuta-nvidia
Merge branch 'main' of github.com:NVIDIA/NeMo into main-vis-res
f004a66
@tkornuta-nvidia
commented hyperparameters
8d1e652 
Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
@tkornuta-nvidia
upgraded trainer config to PTL 0.9
3936344 
Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
@tkornuta-nvidia
format
d60c5d6 
Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
@tkornuta-nvidia
work on xmnist
d1306d3 
Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
@tkornuta-nvidia
Switching between different MNIST tasks works - by setting _target_
3ce7730 
Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
@tkornuta-nvidia
minor comments
db4cde4 
Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
@tkornuta-nvidia
training operational, still not using structured configs for transforms
31ea524 
Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
@tkornuta-nvidia
Refactored and simplified the LeNet-5 example, introduced CV Model, c…
17ef4b6 
…leaned up the code

Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
@tkornuta-nvidia
missing config
698a89f 
Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
@tkornuta-nvidia
init of porting of resnet50 example
c08dbbd 
Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
@tkornuta-nvidia
CIFAR 10 classification working, configuration for CIFAR10, ported Im…
5807209 
…ageEncoder module, ImageEncoder config

Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
@tkornuta-nvidia
minor cleanups
b878763 
Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
@tkornuta-nvidia
CV documentation fixes
aeb723a 
Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
@tkornuta-nvidia
tracking down improper size of CIFAR10 images
184bb82 
Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
@tkornuta-nvidia
CIFAR10: fix of a dummy mistake in the target class
c17f305 
Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
@tkornuta-nvidia
fixed ResNet50 model, several minor fixes, model converging
4405f13 
Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
@tkornuta-nvidia
added transforms to documentation
826e98f 
Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
@tkornuta-nvidia
CLEVR dataset ported and operational, working on the first, text-base…
15ad639 
…d baseline

Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
@tkornuta-nvidia
VIS docs update
02a97c6 
Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
@tkornuta-nvidia
vis intro
1030e29 
Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
@tkornuta-nvidia
minor description update
04fa0d6 
Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
@tkornuta-nvidia
progress on qtype - text transforms
285734b 
Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
@tkornuta-nvidia
Unified hydra runner
cab9660 
Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
@tkornuta-nvidia
refactor, SentenceEmbeddings operational
0c5f871 
Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
@tkornuta-nvidia
fixes in instantiate_dataloader (collate), fixes in clevr, passing sa…
9e2644a 
…mples to embeddings operational

Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
@tkornuta-nvidia
clevr dataset preprocessing operational
bddedfd 
Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
@tkornuta-nvidia
CV integration fix - MNIST + CIFAR10
69ad548 
Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
@lgtm-com
Copy link

lgtm-com bot commented Oct 16, 2020

This pull request introduces 16 alerts and fixes 4 when merging 69ad548 into 910caf6 - view on LGTM.com

new alerts:

  • 12 for Unused import
  • 2 for Unused local variable
  • 2 for 'import *' may pollute namespace

fixed alerts:

  • 3 for Unused import
  • 1 for 'import *' may pollute namespace

@tkornuta-nvidia
LGTM fixes, added __all__ to vis/feature_loaders
10aadf2 
Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
@tkornuta-nvidia
LGTM fixes
76390b4 
Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
@tkornuta-nvidia
jenkins fixes for CV
3750f09 
Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
@lgtm-com
Copy link

lgtm-com bot commented Oct 16, 2020

This pull request introduces 11 alerts and fixes 4 when merging 3750f09 into 910caf6 - view on LGTM.com

new alerts:

  • 8 for Unused import
  • 2 for Unused local variable
  • 1 for 'import *' may pollute namespace

fixed alerts:

  • 3 for Unused import
  • 1 for 'import *' may pollute namespace

@tkornuta-nvidia
LGTM fixes cnt
c2ea20a 
Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
@tkornuta-nvidia
CV integration tests: root fix
386787e 
Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
@lgtm-com
Copy link

lgtm-com bot commented Oct 17, 2020

This pull request introduces 5 alerts and fixes 4 when merging 386787e into 910caf6 - view on LGTM.com

new alerts:

  • 3 for Unused import
  • 2 for Unused local variable

fixed alerts:

  • 3 for Unused import
  • 1 for 'import *' may pollute namespace

@tkornuta-nvidia
LGTM fixes cnt
928fdee 
Signed-off-by: Tomasz Kornuta <tkornuta@nvidia.com>
@lgtm-com
Copy link

lgtm-com bot commented Oct 17, 2020

This pull request introduces 2 alerts and fixes 4 when merging 928fdee into 910caf6 - view on LGTM.com

new alerts:

  • 2 for Unused local variable

fixed alerts:

  • 3 for Unused import
  • 1 for 'import *' may pollute namespace

@mcdavid109
Adding CATER dataset
76d0332 
Signed-off-by: tuananhn <tuananhn@nvidia.com>
@lgtm-com
Copy link

lgtm-com bot commented Oct 17, 2020

This pull request introduces 7 alerts and fixes 4 when merging 76d0332 into 910caf6 - view on LGTM.com

new alerts:

  • 5 for Unused import
  • 2 for Unused local variable

fixed alerts:

  • 3 for Unused import
  • 1 for 'import *' may pollute namespace

@mcdavid109
Modifying CLEVRER to not always save frames
54ac8f2 
Signed-off-by: tuananhn <tuananhn@nvidia.com>
@lgtm-com
Copy link

lgtm-com bot commented Oct 17, 2020

This pull request introduces 8 alerts and fixes 4 when merging 54ac8f2 into 910caf6 - view on LGTM.com

new alerts:

  • 6 for Unused import
  • 2 for Unused local variable

fixed alerts:

  • 3 for Unused import
  • 1 for 'import *' may pollute namespace

@mcdavid109
Fix question_subtype issue
2536455 
Signed-off-by: tuananhn <tuananhn@nvidia.com>
@lgtm-com
Copy link

lgtm-com bot commented Oct 17, 2020

This pull request introduces 8 alerts and fixes 4 when merging 2536455 into 910caf6 - view on LGTM.com

new alerts:

  • 6 for Unused import
  • 2 for Unused local variable

fixed alerts:

  • 3 for Unused import
  • 1 for 'import *' may pollute namespace

@mcdavid109
Remove unneeded flags in GQA
85f75c5 
Signed-off-by: tuananhn <tuananhn@nvidia.com>
@mcdavid109
Remove unneeded flags in GQA
9bf1661 
Signed-off-by: tuananhn <tuananhn@nvidia.com>
@lgtm-com
Copy link

lgtm-com bot commented Oct 17, 2020

This pull request introduces 8 alerts and fixes 4 when merging 9bf1661 into 910caf6 - view on LGTM.com

new alerts:

  • 6 for Unused import
  • 2 for Unused local variable

fixed alerts:

  • 3 for Unused import
  • 1 for 'import *' may pollute namespace

@mcdavid109
Add Resnet101, Faster-RCNN to CV collection
7876566 
Signed-off-by: tuananhn <tuananhn@nvidia.com>
@lgtm-com
Copy link

lgtm-com bot commented Oct 18, 2020

This pull request introduces 10 alerts and fixes 4 when merging 7876566 into 910caf6 - view on LGTM.com

new alerts:

  • 7 for Unused import
  • 2 for Unused local variable
  • 1 for Suspicious unused loop iteration variable

fixed alerts:

  • 3 for Unused import
  • 1 for 'import *' may pollute namespace

@mcdavid109
Fixing some missing typos
bfb3970 
Signed-off-by: tuananhn <tuananhn@nvidia.com>
@mcdavid109
Fixing code styles
e6027fc 
Signed-off-by: tuananhn <tuananhn@nvidia.com>
@mcdavid109
Fixing missing typos
a6a1102 
Signed-off-by: tuananhn <tuananhn@nvidia.com>
@lgtm-com
Copy link

lgtm-com bot commented Oct 18, 2020

This pull request introduces 10 alerts and fixes 4 when merging a6a1102 into 910caf6 - view on LGTM.com

new alerts:

  • 7 for Unused import
  • 2 for Unused local variable
  • 1 for Suspicious unused loop iteration variable

fixed alerts:

  • 3 for Unused import
  • 1 for 'import *' may pollute namespace

@mcdavid109
remove static dimension
79f9e6f 
Signed-off-by: tuananhn <tuananhn@nvidia.com>
@lgtm-com
Copy link

lgtm-com bot commented Oct 19, 2020

This pull request introduces 10 alerts and fixes 4 when merging 79f9e6f into 910caf6 - view on LGTM.com

new alerts:

  • 7 for Unused import
  • 2 for Unused local variable
  • 1 for Suspicious unused loop iteration variable

fixed alerts:

  • 3 for Unused import
  • 1 for 'import *' may pollute namespace

@okuchaiev okuchaiev closed this Jul 19, 2021
@ericharper ericharper deleted the main-vis-res branch September 22, 2022 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@tkornuta-nvidia @okuchaiev @mcdavid109