feat: load_family() returning elements

[vitspec99]

PR for issue-2954

Description

New function load_family2() introduced.
It loads Family and returns list[DB.FamilySymbol] of loaded Family or empty list of error occured.

---

Checklist

Before submitting your pull request, ensure the following requirements are met:

• Code follows the PEP 8 style guide.
• Code has been formatted with Black using the command:

  pipenv run black {source_file_or_directory}

• Changes are tested and verified to work as expected.

---

Related Issues

• Resolves revit.create.load_family2() to return family symbols like revit.query.get_family() #2954

---

Thank you for contributing to pyRevit! 🎉

[devloai]

Unable to trigger custom agent "Code Reviewer"You have run out of credits 😔
Please upgrade your plan or buy additional credits from the subscription page.

[Copilot]

Pull request overview

This PR introduces a new function load_family2() to provide an alternative family loading method that returns a list of loaded family symbols instead of a boolean result.

Key changes:

• Added load_family2() function that loads a family file and returns all family symbols from the loaded family
• Returns an empty list on error or if the family is already loaded, rather than a boolean

[Copilot]

Comparing boolean values using != True is redundant. Use if not res: instead for more idiomatic Python code.

Suggested change

	if ( res != True ):
	if not res:

[jmcouffin]

this should be fixed @vitspec99

[Copilot]

Corrected spelling of 'Cann't' to 'Cannot' and 'It's may be' to 'It may be'.

Suggested change

	mlogger.debug("Cann't load Family from file=%s. It's may be OK if Family already loaded.", family_file)
	mlogger.debug("Cannot load Family from file=%s. It may be OK if Family already loaded.", family_file)

[jmcouffin]

Thinking out loud here:
Why not modifying the original load_family function adding a default arg to trigger a different output?

@vitspec99

[vitspec99]

It was my thought too. But I decide to start new function firstly and if function idea usefull and code is OK original load_family() interface/code may be changed.

[jmcouffin]

Can you make a pass on copilot's comments please @vitspec99

[jmcouffin]

Please implement directly this change to hte original function load_family and do not add a new function
The original one seems to be used once in the entire codebase here

pyRevit/pyrevitlib/pyrevit/revit/db/ensure.py

Line 59 in b4bb7a8

if create.load_family(family_file, doc=doc):

so if you fix the above and its usage in the ensure module, this should be good enough for me to review and test
also, for good measure, provide a minimal example of its usage and output so that it becomes easier to test