use required component + remove some unused imports or obsolete comments

Author: Vrixyz

examples/interpolate_custom_schedule.rs

@@ -12,7 +12,7 @@
 
 use bevy::{
     color::palettes::{
-        css::{ORANGE, RED, WHITE},
+        css::WHITE,
         tailwind::{CYAN_400, RED_400},
     },
     ecs::schedule::ScheduleLabel,
@@ -82,6 +82,7 @@ fn main() {
     // Run the app.
     app.run();
 }
+
 /// Eases the translations of entities with linear interpolation.
 fn ease_translation_lerp(
     mut query: Query<(&mut Transform, &TranslationEasingState)>,
@@ -147,8 +148,8 @@ fn setup(
     let mesh = meshes.add(Rectangle::from_length(60.0));
 
     commands.spawn((
-        TaskToRenderTime::default(),
         Timestep {
+            // Set the fixed timestep to just 5 Hz for demonstration purposes.
             timestep: Duration::from_secs_f32(0.5),
         },
         TaskResults::<TaskWorkerTraitImpl>::default(),
@@ -309,7 +310,6 @@ pub mod task_user {
             substep_count: u32,
         ) -> Vec<(Entity, Transform, LinearVelocity, AngularVelocity)> {
             let simulated_time = timestep * substep_count;
-            let to_simulate = simulated_time.as_millis() as u64;
             // Simulate an expensive task
             std::thread::sleep(Duration::from_millis(thread_rng().gen_range(200..201)));
 

src/background_fixed_schedule.rs

@@ -1,13 +1,8 @@
 use bevy::ecs::schedule::{LogLevel, ScheduleBuildSettings, ScheduleLabel};
-use bevy::ecs::world;
-use bevy::log::tracing_subscriber::fmt::time;
 use bevy::prelude::*;
 use bevy::tasks::AsyncComputeTaskPool;
-use bevy::{log::trace, prelude::World, time::Time};
+use bevy::{prelude::World, time::Time};
 use crossbeam_channel::Receiver;
-use rand::{thread_rng, Rng};
-use std::default;
-use std::slice::IterMut;
 use std::{collections::VecDeque, time::Duration};
 
 ///
@@ -67,7 +62,7 @@ pub struct BackgroundFixedUpdatePlugin<T: TaskWorkerTrait> {
 impl<T: TaskWorkerTrait> Plugin for BackgroundFixedUpdatePlugin<T> {
     fn build(&self, app: &mut App) {
         app.add_systems(
-            bevy::app::prelude::RunFixedMainLoop, // TODO: use a specific schedule for this, à la bevy's FixedMainLoop
+            bevy::app::prelude::RunFixedMainLoop,
             FixedMain::run_schedule::<T>,
         );
 
@@ -120,14 +115,24 @@ pub struct TaskToRenderTime {
 }
 
 /// Difference between tasks and rendering time
-#[derive(Component, Default, Reflect, Clone)]
+#[derive(Component, Reflect, Clone)]
 pub struct Timestep {
     pub timestep: Duration,
 }
 
+impl Default for Timestep {
+    fn default() -> Self {
+        Self {
+            timestep: Duration::from_secs_f64(1.0 / 64.0),
+        }
+    }
+}
+
 /// Struct to be able to configure what the task should do.
-/// TODO: extract first, then do work.
+// TODO: This should also require `TaskResults` and `WorkTask`
+// but their type parameter not enforcing `Default`  makes the require macro fail. This should be a bevy issue.
 #[derive(Clone, Component)]
+#[require(TaskToRenderTime, Timestep)]
 pub struct TaskWorker<T: TaskWorkerTrait> {
     pub worker: T,
 }

src/interpolation.rs

@@ -6,7 +6,6 @@
 
 use crate::*;
 use bevy::{
-    app::FixedMain,
     ecs::schedule::{InternedScheduleLabel, ScheduleLabel},
     prelude::*,
 };