Skip to content

fix: Use executorlib with context#122

Merged
ltalirz merged 6 commits intofeat/api-slurmfrom
debug/api-slurm
Feb 9, 2026
Merged

fix: Use executorlib with context#122
ltalirz merged 6 commits intofeat/api-slurmfrom
debug/api-slurm

Conversation

@jan-janssen
Copy link
Copy Markdown
Contributor

No description provided.

@jan-janssen jan-janssen marked this pull request as draft February 9, 2026 06:19
# Use TestClusterExecutor for local - it supports wait=False
# (SingleNodeExecutor does not support wait=False)
from executorlib.executor.single import TestClusterExecutor
from executorlib.api import TestClusterExecutor
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.


if TYPE_CHECKING:
from executorlib.executor.single import TestClusterExecutor
from executorlib.api import TestClusterExecutor
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 9, 2026

Coverage

Coverage Report
FileStmtsMissCoverMissing
__init__.py190100% 
io_utils.py574128%76–85, 106–111, 135–142, 144, 170–172, 174–179, 181–187
mass.py40100% 
neighbors.py340100% 
shared.py16756%25–27, 61, 64, 67, 69
structure.py36515756%106–107, 144–145, 268, 270–271, 273–275, 278, 280–281, 305–306, 308–309, 325–327, 339–340, 361–362, 369–370, 376, 380–382, 388–390, 393–394, 400–401, 410–411, 416–417, 419–420, 422–423, 507, 526–527, 538–539, 552–553, 689–691, 700–701, 707, 714–715, 721–722, 727, 754, 760, 777, 779–780, 782–783, 786, 788–790, 793, 795–796, 799, 801, 803–806, 809–813, 816, 819–820, 823, 851–854, 856–857, 859–860, 862–864, 867–869, 871–873, 875–878, 880, 882–889, 891–895, 898, 900, 949, 951, 954, 964, 976, 978–982, 985–986, 988–991, 994–1000, 1002–1004, 1007, 1009
analysis
   __init__.py00100% 
   bond_angle_distribution.py322618%52, 54, 62–83, 89–90
   cavities.py191142%59–60, 63, 65–66, 69–70, 73–74, 77, 79
   cte.py302710%57, 71–78, 80–81, 83–88, 90–91, 93–94, 126–128, 131–132, 134
   qn_network_connectivity.py30293%103–104
   radial_distribution_functions.py685322%57–60, 62–63, 65–67, 69–71, 73–77, 79, 119, 121–123, 125–126, 129–131, 134–136, 139, 142–143, 146–147, 150–152, 154, 156–159, 162–163, 165–167, 169–172, 174
   rings.py403025%71–76, 78–79, 81, 83–85, 87–88, 90–94, 96, 121–130
potentials
   __init__.py00100% 
   bjp_potential.py342914%45–46, 49, 56–58, 60, 68, 71–74, 77, 85–93, 95–99, 101, 103
   pmmcs_potential.py383410%56–57, 60–63, 65–66, 68–69, 71–73, 75, 84, 86–89, 91, 99–105, 109–113, 117, 119
   potential.py13838%28–35
   shik_potential.py847313%67–69, 71–73, 80–81, 83–86, 88–93, 95, 111–115, 117–118, 128–129, 131–132, 134–135, 138–143, 145–147, 150, 161–162, 164–165, 167–170, 173–180, 182, 184–187, 189, 191–195, 197–199, 201
workflows
   __init__.py00100% 
   cte.py20518410%28–29, 32–33, 36–37, 40, 42, 117–118, 121, 148–150, 152, 154, 159, 176–178, 180–187, 189–192, 194, 210–212, 214–219, 221–226, 228–233, 235–240, 242–247, 337, 340, 343, 345–346, 349, 352, 366, 369, 376, 383, 390, 399–410, 439–447, 449–451, 453–454, 462–465, 467–473, 475–483, 485–492, 494, 520–522, 547–548, 550–553, 555, 659, 662, 667, 670, 673, 676, 679–680, 683, 686, 702, 718–720, 723–726, 729–730, 733, 749–751, 754, 765–777, 780–781, 783
   elastic_mod.py806617%84–85, 88, 91, 119, 121, 143–144, 146, 149, 151, 193–196, 199–202, 204, 230–233, 235–238, 240–241, 243, 311, 313–314, 322, 327, 342, 346–348, 350, 353, 367–368, 371–377, 380–387, 389, 391, 394–397, 399
   md.py211338%76–77, 80, 83, 111, 113, 166, 168–169, 177, 182, 197, 199
   meltquench.py372824%82–83, 86, 90–91, 93–94, 96, 98, 100, 127, 129, 154–156, 159, 161, 221–223, 225, 228–231, 234, 251, 253
   meltquench_protocols.py372921%54, 65, 75, 83, 92, 101, 108, 129, 140, 151, 160, 170, 179, 186, 207, 218, 227, 237, 242, 252, 261, 271, 279, 332–335, 337–338
   shared.py11109%21–25, 27–29, 31–32
   structural_analysis.py24219619%113–114, 116–118, 121, 123–128, 130–132, 134, 155–160, 162–163, 165–167, 169–176, 178, 180, 197–198, 201–204, 206–209, 211, 213–221, 223–227, 229, 245, 248–251, 253, 255–259, 261, 263–267, 269, 271–275, 278–281, 283, 285–286, 289–293, 297–301, 303–304, 307–308, 310, 314–321, 323, 340–344, 346, 359–365, 367, 384–390, 392, 406–410, 412, 428, 431–435, 449, 454–458, 460, 475–476, 478–479, 481–484, 486, 501–502, 532–533, 536, 562, 564–567, 570, 579–582, 585–588, 591–594, 597–601, 603
   viscosity.py1474271%80–81, 84, 87, 115, 117, 166, 168–169, 177, 182, 197, 212, 225, 227, 269, 315–316, 318–319, 321–322, 326–331, 337, 349, 361, 370–372, 377–378, 382, 384, 409, 430–431, 505
TOTAL1663106635% 

@jan-janssen jan-janssen changed the title debug: API with SLURM support fix: Use executorlib with context Feb 9, 2026
@ltalirz
Copy link
Copy Markdown
Contributor

ltalirz commented Feb 9, 2026

I'll merge this and continue in the other PR. Thanks for your help!

@ltalirz ltalirz marked this pull request as ready for review February 9, 2026 09:11
@ltalirz ltalirz merged commit b1b0fb9 into feat/api-slurm Feb 9, 2026
3 of 5 checks passed
@ltalirz ltalirz deleted the debug/api-slurm branch February 9, 2026 09:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants