Skip to content

Comments

Update CUSTOM_CYCLER to have 10 distinct line styles for plotting#151

Open
nemo794 wants to merge 1 commit intoisce-framework:mainfrom
nemo794:cycler_update
Open

Update CUSTOM_CYCLER to have 10 distinct line styles for plotting#151
nemo794 wants to merge 1 commit intoisce-framework:mainfrom
nemo794:cycler_update

Conversation

@nemo794
Copy link
Collaborator

@nemo794 nemo794 commented Jan 22, 2026

This closes #150

This update to nisarqa's CUSTOM_CYCLER global impacts the QA PDF plots which use line styles for all eight L1/L2 products, although GCOV QA PDF reports are the only which need all 10 distinct line styles.

As an example output from this update, for quad pol GCOV products, the Freq A backscatter histograms will now be plotted with 10 distinct line styles like this:

Screenshot 2026-01-22 at 2 11 12 PM

@gshiroma
Copy link
Contributor

Thank you, @nemo794 , for issuing this PR. The new color palette solves the problem, but I don't think it looks good. There's too much going on, maybe too many colors. One way to improve this might be to use fewer colors and introduce different line styles instead. For example, you could keep the first four colors and use three line patterns, which would give you twelve distinct combinations before they start repeating. You can test some different combinations.

I’d suggest moving the green color to be second in the sequence. The HV polarization is usually the second element, and cross-polarimetric terms are often associated with vegetation. This convention would apply to both RSLC and GSLC products as well.

Also, is there a way to emphasize the diagonal terms? It looks like they’re plotted first and then get buried beneath the other curves. At the same time, it's nice that they appear first in the legend. Some ways to fix it:

  • Use [zorder](https://matplotlib.org/3.1.1/gallery/misc/zorder_demo.html).
  • Change the plotting order, and keep the legend order by setting it manually (e.g., using handles in the call to plt.legend().
  • Increase the linewidth for the first elements in the cycler (or decrease it for the later ones).

Thanks again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update CUSTOM_CYCLER to have 10 line styles

2 participants