MNT remove redundant code; see init_catalog()

[prjemian]

The tiled_client is not used in startup.py now. This is handled by apsbits.core.init_catalog() now.

[coveralls]

Pull Request Test Coverage Report for Build 21046559579

Details

• 0 of 0 changed or added relevant lines in 0 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.08%) to 84.507%

---

Totals
Change from base Build 20969680449:	0.08%
Covered Lines:	1440
Relevant Lines:	1704

---

💛 - Coveralls

[MDecarabas]

Can we please add more documentation for what to place where in the iconfig so that tiled will function.

[prjemian]

Sure, I can add. Where would you recommend the new content? Perhaps docs/source/guides/setting_iconfig.rst? Other places (under docs/source/) might need updates for this shift from databroker to tiled.

[MDecarabas]

setting_iconfig sound great

[prjemian]

I'm not making any changes to demo_scripts/qs_host.sh.

[MDecarabas]

I'm not making any changes to demo_scripts/qs_host.sh.

?

[prjemian]

I'm not making any changes to demo_scripts/qs_host.sh.

These lines:

BITS/src/apsbits/demo_scripts/qs_host.sh

Lines 42 to 48 in e2e0b30

	if [ "${DATABROKER_CATALOG}" == "" ]; then
	if [ -f "${ICONFIG_YML}" ]; then
	DATABROKER_CATALOG=$(grep DATABROKER_CATALOG "${ICONFIG_YML}" | awk '{print $NF}')
	# echo "Using catalog ${DATABROKER_CATALOG}"
	fi
	fi
	DEFAULT_SESSION_NAME="bluesky_queueserver-${DATABROKER_CATALOG}"

[MDecarabas]

I see, I didn't remember this snippet. Now that you bring it up, shouldn't we. Isn't the goal to move towards tiled servers for the qs as well. At the very least have a conditional that if data broker is not defined try tiled, and if that is not defined either start a local temp tiled server?

[prjemian]

Turns out this is easier than anticipated.

[prjemian]

@MDecarabas Take another look now.

• Docs added to the file described above.
• Bash script qs_host.sh updated to consider TILED_PROFILE_NAME.
• Pre-commit run enough times that it stopped complaining.