Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feature: Adds tree view #223

Merged
merged 21 commits into from
Sep 7, 2020
Merged

feature: Adds tree view #223

Changes from 1 commit
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
6df7014
Add back highlighting to CPU legend if not focused
ClementTsang Sep 3, 2020
8c4ea05
Get *very* basic tree-mode working
ClementTsang Sep 4, 2020
94d9267
Fix husky
ClementTsang Sep 4, 2020
d4d489a
Dang conflicts
ClementTsang Sep 4, 2020
8023f11
Merge branch 'master' into forestry
ClementTsang Sep 4, 2020
94dfafe
Add some comments to taste
ClementTsang Sep 4, 2020
7f7c8ee
Switch to using pid_t
ClementTsang Sep 4, 2020
cf8c01c
Apparently windows doesn't like that
ClementTsang Sep 4, 2020
c2b6065
Rudimentary trees work and look ok...
ClementTsang Sep 4, 2020
86f921e
Fix line generation, basic version of filtering
ClementTsang Sep 5, 2020
e44c025
Now we have sorting in tree mode
ClementTsang Sep 6, 2020
78046b3
Windows implementation
ClementTsang Sep 6, 2020
d8cc0b3
Let sort apply to top level as well
ClementTsang Sep 6, 2020
f442c50
attempt to fix conflict
ClementTsang Sep 6, 2020
243a743
Simplify tree orphan step
ClementTsang Sep 6, 2020
53ad270
Merge branch 'master' into forestry
ClementTsang Sep 6, 2020
5a403e1
Fix merge error
ClementTsang Sep 6, 2020
9ab2f2d
Update docs
ClementTsang Sep 6, 2020
0229261
Update changelog
ClementTsang Sep 7, 2020
d43b913
Update widths for now
ClementTsang Sep 7, 2020
3aba4b4
Fix sort being reversed for trees
ClementTsang Sep 7, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Fix sort being reversed for trees
  • Loading branch information
@ClementTsang
ClementTsang committed Sep 7, 2020
commit 3aba4b4337700b51ba61421d7b4c73f96ada9102
11 changes: 8 additions & 3 deletions src/data_conversion.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -470,6 +470,8 @@ pub fn tree_process_data(
single_process_data: &[ConvertedProcessData], is_using_command: bool,
sort_type: &ProcessSorting, is_sort_descending: bool,
) -> Vec<ConvertedProcessData> {
// TODO: [TREE] Allow for collapsing entries.

// Let's first build up a (really terrible) parent -> child mapping...
// At the same time, let's make a mapping of PID -> process data!
let mut parent_child_mapping: HashMap<Pid, IndexSet<Pid>> = HashMap::default();
Expand Down Expand Up @@ -596,9 +598,11 @@ pub fn tree_process_data(
// Now let's sort the immediate children!
sort_vec(&mut to_sort_vec, sort_type, is_sort_descending);

// Need to reverse what we got, apparently...
if let Some(current_mapping) = parent_child_mapping.get_mut(&current_pid) {
*current_mapping = to_sort_vec
.iter()
.rev()
.map(|(pid, _proc)| *pid)
.collect::<IndexSet<Pid>>();
}
Expand Down Expand Up @@ -657,10 +661,9 @@ pub fn tree_process_data(
)
}),
ProcessSorting::Pid => {
// Note we only sort if is_sort_descending as otherwise it's repeated.
if is_sort_descending {
to_sort_vec.sort_by(|a, b| {
utils::gen_util::get_ordering(a.1.pid, b.1.pid, is_sort_descending)
utils::gen_util::get_ordering(a.0, b.0, is_sort_descending)
});
}
}
Expand Down Expand Up @@ -691,7 +694,9 @@ pub fn tree_process_data(
is_sort_descending,
)
}),
ProcessSorting::Count => {}
ProcessSorting::Count => {
// Should never occur in this case.
}
}
}

Expand Down