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.

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

AllSkyAI - Additonal options #72

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

TheCrowsNestAstro
Copy link
Contributor

Added Disable Upload
Added Upload Frequency
Added a default allskyai.json file even if the image was not classified to keep variables accessible

Added Disable Upload
Added Upload Frequency
Added a default allskyai.json file even if the image was not classified to keep variables accessible
@Alex-developer
Copy link
Collaborator

Christian

We do not allow PR into master anymore :-( could you please change the PR so that its going into dev. I have a few other module changes in the 61-test branch and once i ham done with those I will merge them into dev and then merge dev into master.

If the changes are urgent and cannot wait then please let me know and I will sort something.

There are a couple of other changes it would be good if you can make. I am currently adding a couple of new entries to the metaData. One for bus information and the other for changelogs. If you look in the 61-test-all-external-devices-for-error-handling branch I have started to change a lot of the modules, a good example is the dewheater module.

These changes are to allow more information to be displayed in the module manager and ensure that things like the i2c bus are enabled on the pi before a user attempts to install modules that needs it.

    "businfo": [
        "i2c"
    ],    
    "changelog": {
        "v1.0.0" : [
            {
                "author": "Alex Greenland",
                "authorurl": "https://github.com/allskyteam",
                "changes": "Initial Release"
            }
        ],
        "v1.0.1" : [
            {
                "author": "Damian Grocholski (Mr-Groch)",
                "authorurl": "https://github.com/Mr-Groch",
                "changes": [
                    "Added extra pin that is triggered with heater pin",
                    "Fixed dhtxxdelay (was not implemented)",
                    "Fixed max heater time (was not implemented)"
                ]
            }
        ],
        "v1.0.2" : [
            {
                "author": "Alex Greenland",
                "authorurl": "https://github.com/allskyteam",
                "changes": "Initial Release"
            }
        ],
        "v1.0.4" : [
            {
                "author": "Alex Greenland",
                "authorurl": "https://github.com/allskyteam",
                "changes": "Add AHTx0 i2c sensor"
            },            
            {
                "author": "Andreas Schminder",
                "authorurl": "https://github.com/Adler6907",
                "changes": "Added Solo Cloudwatcher"
            }
        ],
        "v1.0.5" : [
            {
                "author": "Alex Greenland",
                "authorurl": "https://github.com/allskyteam",
                "changes": "Added OpenWeather option"
            }
        ]                                        
    }

Thanks

Alex

@TheCrowsNestAstro
Copy link
Contributor Author

Ok! I'll fix this. Thanks Alex!

@Alex-developer
Copy link
Collaborator

Bump ;-)

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