[BUG] Analysis repeatedly attempts to process albums with "skipped" songs.

[TJ09]

Describe the bug
Several albums in my library have "songs" that are either silent or too short to analyze. The analysis step correctly skips over these (e.g. Skipping track [silence] by TV on the Radio as analysis returned None.), but the next time analysis is run, these same albums are analyzed again, rather than being skipped as having complete analysis.

To Reproduce
Steps to reproduce the behavior:
0. Have an audio file that is completely silent. This can be easily generated and added to any album for testing.

1. Run analysis.
2. See that the album was launched and analyzed (log reports "analysis complete").
3. Run analysis again.
4. See that the album was again launched and analyzed.

Expected behavior
After the first analysis, subsequent runs are no-ops since all valid files have been analyzed (barring actual errors such as decoding failures).

Environment (please complete the following information):

• Deployment: Docker Compose
• AudioMuse-AI Version: v0.7.12-beta

Additional Context

To be clear, I don't expect any meaningful analysis to occur on short/silent songs, but re-analyzing the album on each run is wasted effort that ideally can be avoided.

[NeptuneHub]

Thanks for the feedback. Sometimes happen that a songs is not analyze but a new analysis sovle the problem due to a momentaneus issue. If I mark a skipped songs as skip forever, it will never be analyzed again.

If are just few songs I think is not a big problem. If instead you found out multiple songs that are not possible to be analyzed I suggest to move in a different library and then use the MUSIC_LIBRARIES paramer as documented in the readme of this project to select only the library that you want to analyze. In this way you can say which library need to be analyzed and which not.

Maybe for the future a better implementation could be check in the code if the same song is skipped multiple time (let's say 3+?) to avoid both the cases. If you think it could be a good balance, I can keep in the enanchment for the future (touch the analysis is something that need extra attention).

[TJ09]

Maybe for the future a better implementation could be check in the code if the same song is skipped multiple time (let's say 3+?) to avoid both the cases. If you think it could be a good balance, I can keep in the enanchment for the future (touch the analysis is something that need extra attention).

Yeah, if something fails repeatedly I imagine that's unlikely to change (though perhaps it should be reset each version because it's possible a code change does fix it).

This is probably a separate request, but right now analysis failures aren't surfaced to the user to know they're happening (you mostly have to read the logs). A possible future solution could be to show failed albums/songs in the UI and let people choose to clear the failure themselves so that the next run will try again.