spin_orbit_tilt no longer an output parameter

[maxbriel]

In the default population_params.ini spin_orbit_tilt is one of the default output options.
However with PR #129, this attribute has been removed from the SingleStar and been replaced with:
spin_orbit_tilt_first_SN and spin_orbit_tilt_second_SN

The default should be changed or spin_orbit_tilt option should output both.

What is the preferred behaviour here?

[maxbriel]

The naming of the attributes are also different in their initialisation compared to their where they're set?

In SingleStar.py:

if not hasattr(self, 'spin_orbit_tilt_first_SN'):
      self.spin_orbit_tilt_SN1 = None
if not hasattr(self, 'spin_orbit_tilt_second_SN'):
      self.spin_orbit_tilt_SN2 = None

In step_SN.py, one of the sets:

binary.star_1.spin_orbit_tilt_first_SN = tilt
binary.star_2.spin_orbit_tilt_first_SN = tilt

Would this be sufficient as a fix? Commit on max_synpop_population branch for: attribute names - 590e440 population_params_default.ini: 2355763

[mpgalleg]

I prefer spin_orbit_tilt option outputting both.

We use S1, S2, CC1, CC2 etc so it is inconsistent to use first_SN and second_SN. If we can reduce that, especially in the output, that would be best I think.