Refactor get_session_path() BIDS util to rely only on BIDS Layout

[alyssadai]

Currently, bids_utils.get_session_path relies on an extra parameter bids_dir to construct the path for no session layer. Given advice in bids-standard/pybids#978, we should simplify the corresponding if block to use layout.root for all cases and just append a session name or empty string depending on whether the session param is None.

Tests may also need updating following this change.

See

bagel-cli/bagel/cli.py

Lines 268 to 273 in 6763e09

	session_path = butil.get_session_path(
	layout=layout,
	bids_dir=bids_dir,
	bids_sub_id=bids_sub_id,
	session=session,
	)

[github-actions]

We want to keep our issues up to date and active. This issue hasn't seen any activity in the last 30 days.
We have applied the stale-issue label to indicate that this issue should be reviewed again and then either prioritized or closed.

[github-actions]

We want to keep our issues up to date and active. This issue hasn't seen any activity in the last 30 days.
We have applied the stale-issue label to indicate that this issue should be reviewed again and then either prioritized or closed.

[github-actions]

We want to keep our issues up to date and active. This issue hasn't seen any activity in the last 75 days.
We have applied the _flag:stale label to indicate that this issue should be reviewed again.
When you review, please reread the spec and then apply one of these three options:

• prioritize: apply the flag:schedule label to suggest moving this issue into the backlog now
• close: if the issue is no longer relevant, explain why (give others a chance to reply) and then close.
• archive: sometimes an issue has important information or ideas but we won't work on it soon. In this case
  apply the someday label to show that this won't be prioritized. The stalebot will ignore issues with this
  label in the future. Use sparingly!