Skip to content

examples fix#25

Merged
ChrisLisbon merged 2 commits intomainfrom
examples_fix
Aug 7, 2025
Merged

examples fix#25
ChrisLisbon merged 2 commits intomainfrom
examples_fix

Conversation

@ChrisLisbon
Copy link
Owner

Moving mnist download added and iterations results for noise experiments

@ChrisLisbon ChrisLisbon requested a review from nicl-nno August 7, 2025 15:42

1. Clone the repository:
```bash
git clone https://github.com/yourusername/TorchCNNBuilder.git
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yourusername?

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment on lines 82 to 88
# iterations_path = 'iterations - batchnorm L1 loss'
# iterations_path = 'iterations - batchnorm SSIM loss'
# iterations_path = 'iterations - batchnorm BCE loss'

# iterations_path = './iterations - none L1 loss'
# iterations_path = './iterations - none SSIM loss'
# iterations_path = './iterations - none BCE loss'
# iterations_path = 'iterations - none L1 loss'
# iterations_path = 'iterations - none SSIM loss'
# iterations_path = 'iterations - none BCE loss'
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Можно как докстиринг оформить, чтобы не выглядело как закомментированный код.

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Сделала докстрингом

@ChrisLisbon ChrisLisbon requested a review from nicl-nno August 7, 2025 16:38
@ChrisLisbon ChrisLisbon merged commit 8288022 into main Aug 7, 2025
1 check passed
@ChrisLisbon ChrisLisbon deleted the examples_fix branch November 10, 2025 09:19
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