fix(jssp): invalid value passed as duration to zero & sink ops (#430)

- Improve op printing in main
- Fix typo:
- Format & clippy fixes
- Fix invalid operation intialization

<!-- If applicable - remember to add the PR to the EA Rust project (ONLY
IF THERE IS NO LINKED ISSUE) -->

## Description

<!-- Please describe the motivation & changes introduced by this PR -->

## Linked issues

<!-- Please use "Resolves #<issue_no> syntax in case this PR should be
linked to an issue -->

## Important implementation details

<!-- if any, optional section -->

examples/jssp/logging.rs

@@ -1,4 +1,7 @@
-use std::{path::{Path, PathBuf}, collections::HashMap};
+use std::{
+    collections::HashMap,
+    path::{Path, PathBuf},
+};
 
 use log4rs::{
     append::{console::ConsoleAppender, file::FileAppender},
@@ -35,7 +38,7 @@ pub fn init_logging(log_files: &HashMap<String, PathBuf>) -> Result<log4rs::Hand
                     Logger::builder()
                         .appender(event_name)
                         .additive(false)
-                        .build(event_name, log::LevelFilter::Info)
+                        .build(event_name, log::LevelFilter::Info),
                 );
         }
     }

examples/jssp/main.rs

@@ -58,7 +58,7 @@ fn run_with_ecrs(instance: JsspInstance, _args: Args) {
 
 fn run() {
     let args = cli::parse_args();
-    
+
     util::assert_dir_exists(args.output_dir.as_ref());
     let event_map = util::create_event_map(args.output_dir.as_ref());
     if let Err(err) = logging::init_logging(&event_map) {
@@ -67,8 +67,10 @@ fn run() {
 
     // Existance of input file is asserted during cli args parsing
     let instance = JsspInstance::try_from(&args.input_file).unwrap();
-    for op in instance.jobs.iter() {
-        info!("{op:?}");
+    for job in instance.jobs.iter() {
+        for op in job {
+            info!("{op:?}");
+        }
     }
     run_with_ecrs(instance, args)
 }

examples/jssp/problem/population.rs

@@ -20,8 +20,8 @@ impl JsspPopProvider {
 
         // Prepare mock operations
         // TODO: Shouldn't the duration be set to 0?
-        let mut zero_op = Operation::new(0, usize::MAX, 0, None, Vec::new());
-        let sink_op = Operation::new(dim + 1, usize::MAX, 0, None, Vec::from_iter(0..=dim));
+        let mut zero_op = Operation::new(0, 0, 0, None, Vec::new());
+        let sink_op = Operation::new(dim + 1, 0, 0, None, Vec::from_iter(0..=dim));
 
         // Shift all ids by 1 && and job 0 & n + 1
         instance.jobs.iter_mut().for_each(|job| {

examples/jssp/problem/probe.rs

@@ -11,6 +11,7 @@ impl JsspProbe {
         Self {}
     }
 
+    #[allow(dead_code)]
     // TODO: This has either been not working as expected or the solver runs so bad.
     // TODO: Verify whether the diversity is better on other problems
     fn estimate_pop_diversity(population: &[JsspIndividual]) -> f64 {

examples/jssp/util.rs

@@ -1,6 +1,10 @@
 #![allow(dead_code)]
 
-use std::{collections::{HashSet, HashMap, hash_map}, fmt::Display, path::{PathBuf, Path}};
+use std::{
+    collections::{hash_map, HashMap, HashSet},
+    fmt::Display,
+    path::{Path, PathBuf},
+};
 
 pub fn print_hash_set<T: Display>(set: &HashSet<T>) {
     for elem in set {
@@ -23,7 +27,7 @@ pub fn create_event_map(base_dir: &Path) -> HashMap<String, PathBuf> {
         ("newbest".to_owned(), base_dir.join("event_newbest.csv")),
         ("bestingen".to_owned(), base_dir.join("event_bestingen.csv")),
         ("popgentime".to_owned(), base_dir.join("event_popgentime.csv")),
-        ("iterinfo".to_owned(), base_dir.join("event_iterinfo.csv"))
+        ("iterinfo".to_owned(), base_dir.join("event_iterinfo.csv")),
     ])
 }
 
@@ -33,7 +37,7 @@ pub fn assert_dir_exists(dir: &Path) {
     }
 
     match std::fs::create_dir_all(dir) {
-        Ok(()) => return,
-        Err(err) => panic!("Failed to create outuput directory with error {err}"),
+        Ok(()) => (),
+        Err(err) => panic!("Failed to create output directory with error {err}"),
     };
 }