Skip to content

Remove PPSTM_simple.py and migrate to TOML-based configuration (part 1)#49

Merged
GianmarcoMidena merged 4 commits into
masterfrom
33-removing-duplicates-ppstm_simple
Jun 15, 2026
Merged

Remove PPSTM_simple.py and migrate to TOML-based configuration (part 1)#49
GianmarcoMidena merged 4 commits into
masterfrom
33-removing-duplicates-ppstm_simple

Conversation

@GianmarcoMidena

@GianmarcoMidena GianmarcoMidena commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Changes

  • [root] Removed PPSTM_simple.py
  • [TOAT] Replaced PPSTM_simple.py script with TOML config for fixed voltage scans
  • [TOAT] Migrated dIdV_test to TOML config for relaxed state scans
  • [Si_7x7] Migrated STM_test to TOML configuration

Related

Partially solves #33

…les with TOML config for fixed voltage scans
- Port dIdV_test_TOAT.py script to TOML configuration format
- Support relaxed state scans with s and pxy tip orbitals
@GianmarcoMidena GianmarcoMidena linked an issue Jun 12, 2026 that may be closed by this pull request
1 task
@GianmarcoMidena GianmarcoMidena changed the title 33 removing duplicates ppstm simple Remove PPSTM_simple.py and migrate to TOML-based configuration (part 1) Jun 13, 2026

@ondrejkrejci ondrejkrejci left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

1 small change and it is good to go. Sorry for the delay!

Comment thread examples/Si_7x7/run_all_tests.sh Outdated
@GianmarcoMidena GianmarcoMidena force-pushed the 33-removing-duplicates-ppstm_simple branch from 1373752 to cf87678 Compare June 15, 2026 05:45

@ondrejkrejci ondrejkrejci left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good to me

@GianmarcoMidena GianmarcoMidena merged commit 96665be into master Jun 15, 2026
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.

Removing duplicates PPSTM_simple

2 participants