Support a better string representation for `NDict`

[SagiPolaczek]

Now we can get the tree-like representation of the NDict also a string.

Useful for logging the cfg structure using logging package. For example:

@hydra.main(version_base="1.2")
def main(cfg: DictConfig) -> None:
    cfg = hydra.utils.instantiate(cfg)
    log = logging.getLogger("Fuse")
    log.info("\n"+NDict(cfg).get_tree(True))

[SagiPolaczek]

@CodiumAI-Agent

[CodiumAI-Agent]

PR Analysis

• 🎯 Main theme: Adding a new method to get a string representation of the NDict object in a tree-like structure

• 🔍 Description and title: Yes

• 📌 Type of PR: Enhancement

• 🧪 Relevant tests added: Yes

• ✨ Minimal and focused: Yes, the PR is focused on adding a new method to the NDict class and updating the relevant tests.

• 🔒 Security concerns: No, the changes made in this PR do not introduce any security concerns.

  PR Feedback

• 💡 General PR suggestions: The PR is well-structured and follows good practices. The new method is properly documented and the tests are updated accordingly. However, the test could be improved by checking the actual output instead of just checking if the output is in a list of expected outputs.

• 🤖 Code suggestions:

  • relevant file: fuse/utils/tests/test_ndict.py suggestion content: Instead of checking if the output is in a list of expected outputs, it would be better to check the actual output. This will make the test more robust and less prone to false positives. [important]

How to use

Tag me in a comment '@CodiumAI-Agent' to ask for a new review after you update the PR. You can also tag me and ask any question, for example '@CodiumAI-Agent is the PR ready for merge?'