Fix frozen dataclass instance error in apply_to_collection

[abhi-rf]

What does this PR do?

changed the set_attribute function in in the apply_to_collection for recursively applying a function to all elements of a certain dtype in utilities/apply_func.py to Work with frozen dataclass Also now.

Fixes #10873

Does your PR introduce any breaking changes? If yes, please list them.

Before submitting

• Was this discussed/approved via a GitHub issue? (not for typos and docs)
• Did you read the contributor guideline, Pull Request section?
• Did you make sure your PR does only one thing, instead of bundling different changes together?
• Did you make sure to update the documentation with your changes? (if necessary)
• Did you write any new necessary tests? (not for typos and docs)
• Did you verify new and existing tests pass locally with your changes?
• Did you list all the breaking changes introduced by this pull request?
• Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review

Anyone in the community is welcome to review the PR.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

• Is this pull request ready for review? (if not, please submit in draft mode)
• Check that all items from Before submitting are resolved
• Make sure the title is self-explanatory and the description concisely explains the PR
• Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun? Yes

Make sure you had fun coding 🙃

[justusschock]

Sorry, but I feel I need to block this for now. I think this violates the core idea of a python frozen dataclass (which is being immutable) and therefore it should not be hacked around this. Let's discuss in #10873 .

If I'm outvoted I will happily open this PR again. Just want to prevent merging until a common decision was made

[abhi-rf]

Sorry, but I feel I need to block this for now. I think this violates the core idea of a python frozen dataclass (which is being immutable) and therefore it should not be hacked around this. Let's discuss in #10873 .

If I'm outvoted I will happily open this PR again. Just want to prevent merging until a common decision was made

Sure, makes sense. I did not think of it as a way to prevent python from doing its job.

[abhi-rf]

Raised error instead of forcing the apply

[justusschock]

@abhi-rf Would you mind to also add a test case for this?

[abhi-rf]

@abhi-rf Would you mind to also add a test case for this?

Sure, I can surely give it a try!!

[carmocca]

@abhi-rf You can find its existing tests in tests/utilities/test_apply_func.py

[tchaton]

LGTM !

[stale]

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. If you need further help see our docs: https://pytorch-lightning.readthedocs.io/en/latest/generated/CONTRIBUTING.html#pull-request or ask the assistance of a core contributor here or on Slack. Thank you for your contributions.

The blocking review is outdated. The original error is respected