M-Labs/thermostat
9
0
Fork 5
Code Issues 9 Pull Requests 5 Releases Wiki Activity

PyThermostat: Add entry points to runnable scripts #149

Merged
sb10q merged 2 commits from atse/thermostat:pytec-entry-points into master 2024-11-25 10:56:45 +08:00
Conversation 1 Commits 2 Files Changed 2 +123 -108
2 changed files with 123 additions and 108 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
pythermostat/pythermostat/plot.py
View File

@ -5,6 +5,8 @@ import matplotlib.animation as animation
from threading import Thread, Lock from threading import Thread, Lock
from pythermostat.client import Client from pythermostat.client import Client
def main():
TIME_WINDOW = 300.0 TIME_WINDOW = 300.0
tec = Client() tec = Client()
@ -119,7 +121,7 @@ def animate(i):
margin_y = 0.01 * (max_y - min_y) margin_y = 0.01 * (max_y - min_y)
ax.set_ylim(min_y - margin_y, max_y + margin_y) ax.set_ylim(min_y - margin_y, max_y + margin_y)
global legend nonlocal legend
legend.remove() legend.remove()
legend = ax.legend() legend = ax.legend()
@ -129,3 +131,7 @@ ani = animation.FuncAnimation(
plt.show() plt.show()
quit = True quit = True
thread.join() thread.join()
if __name__ == "__main__":
main()

9
pythermostat/setup.py
View File

@ -9,4 +9,13 @@ setup(
license="GPLv3", license="GPLv3",
install_requires=["setuptools"], install_requires=["setuptools"],
packages=find_packages(), packages=find_packages(),
entry_points={
"gui_scripts": [
"thermostat_plot = pythermostat.plot:main",
],
"console_scripts": [
"thermostat_autotune = pythermostat.autotune:main",
"thermostat_test = pythermostat.test:main",
]
},
) )
atse marked this conversation as resolved Outdated
sb10q commented 2024-11-18 19:26:16 +08:00
Outdated
Review
Copy Link

Looks suspicious to me that ARTIQ also has several entry points but does not use py_modules and the entry points are prefixed with artiq..

Looks suspicious to me that ARTIQ also has several entry points but does not use ``py_modules`` and the entry points are prefixed with ``artiq.``.
atse commented 2024-11-25 10:11:02 +08:00
Outdated
Review
Copy Link

Fixed, no need for py_modules now but depends on #155.

Fixed, no need for `py_modules` now but depends on #155.