Pulse pass manager bugfix

[nkanazawa1989]

Summary

Pulse pass manager skeleton was added by #11743. This PR fixes bugs.

Details and comments

1: Pass must be hashable.

Due to the original implementation of Qiskit PassManager, passes with the identical hash are considered as idempotent and execution of the duplicated (same hash) passes are omitted by design. There was long discussion in #10127 but we decided to remain this behavior.

qiskit/qiskit/passmanager/base_tasks.py

Lines 97 to 101 in 6ebb7aa

	if self not in state.workflow_status.completed_passes:
	ret = self.run(passmanager_ir)
	run_state = RunState.SUCCESS
	else:
	run_state = RunState.SKIP

The base passes introduced in #11743 didn't implement hash, and crashes the pass manager since a pass instance cannot be added to the completed pass set.

2: Bug fix of schedule block -> IR conversion.

A small test added in #11743 was unintentionally passed due to

qiskit/qiskit/passmanager/passmanager.py

Lines 220 to 221 in 6ebb7aa

	if not self._tasks and not kwargs and callback is None:
	return in_programs

Although the pass and conversion logic had bugs, round trip test unintentionally succeeded because nothing was really executed (pass manager run just returned input object as-is, since no pass was added to the flow controller).

[qiskit-bot]

One or more of the the following people are requested to review this:

• @Qiskit/terra-core
• @nkanazawa1989

[TsafrirA]

Looks good.
You can merge this first and I'll fix #11951.