Refactor player filtering logic in app.py to correct mask application for removing players. Updated the remove mask condition to ensure accurate filtering of lineups based on user-defined criteria, enhancing functionality and maintainability.

app.py

@@ -1322,12 +1322,14 @@ with tab2:
                         include_mask = parsed_frame[player_columns].apply(
                             lambda row: all(player in list(row) for player in conditional_include_players), axis=1
                         )
+
+                        parsed_frame = parsed_frame[include_mask]
                             
                         remove_mask = parsed_frame[player_columns].apply(
-                            lambda row: all(player in list(row) for player in conditional_remove_players), axis=1
+                            lambda row: not all(player in list(row) for player in conditional_remove_players), axis=1
                         )
                             
-                        parsed_frame = parsed_frame[include_mask & remove_mask]
+                        parsed_frame = parsed_frame[remove_mask]
 
                         st.session_state['working_frame'] = parsed_frame.sort_values(by='median', ascending=False).reset_index(drop=True)
                         st.session_state['export_merge'] = st.session_state['working_frame'].copy()
@@ -1342,11 +1344,13 @@ with tab2:
                             lambda row: all(player in list(row) for player in conditional_include_players), axis=1
                         )
                         
+                        parsed_frame = parsed_frame[include_mask]
+
                         remove_mask = parsed_frame[player_columns].apply(
-                            lambda row: all(player in list(row) for player in conditional_remove_players), axis=1
+                            lambda row: not all(player in list(row) for player in conditional_remove_players), axis=1
                         )
                             
-                        parsed_frame = parsed_frame[include_mask & remove_mask]
+                        parsed_frame = parsed_frame[remove_mask]
                         
                         st.session_state['export_base'] = parsed_frame.sort_values(by='median', ascending=False).reset_index(drop=True)
                         st.session_state['export_merge'] = st.session_state['export_base'].copy()