Skip to content
This repository has been archived by the owner on Nov 20, 2024. It is now read-only.

add neon backend #11

Closed
wants to merge 1 commit into from
Closed

add neon backend #11

wants to merge 1 commit into from

Conversation

JarbasAl
Copy link
Member

@JarbasAl JarbasAl commented Oct 4, 2022 •
edited
Loading

follow up to #10

NeonMQ Backend

Python client library for interaction with several supported backends under a single unified interface

Backend Overview

API Offline Personal Selene OVOS Neon
Admin yes [1] yes no no no
Device yes [2] yes yes yes [4] yes [4]
Metrics yes [2] yes yes yes [4] yes
Dataset yes [2] yes yes yes [4] yes [4]
OAuth yes [2] yes yes yes [4] yes [4]
Wolfram yes [3] yes yes yes yes
Geolocate yes yes yes yes yes
STT yes [3] yes yes yes yes
Weather yes [3] yes yes yes yes
Email yes [3] yes yes yes yes 
[1] will update user level mycroft.conf
[2] shared json database with personal backend for UI compat
[3] needs additional configuration (eg. credentials)
[4] uses offline_backend functionality

@JarbasAl JarbasAl added the enhancement New feature or request label Oct 4, 2022
@JarbasAl JarbasAl requested review from AIIX and NeonDaniel October 4, 2022 13:17
This was referenced Oct 4, 2022
Merged
Merged
JarbasAl
JarbasAl commented Oct 10, 2022
View reviewed changes
ovos_backend_client/backends/neon.py
units (str): metric or imperial measurement units
lat_lon (tuple): the geologic (latitude, longitude) of the weather location
"""
raise NotImplementedError()
Copy link
Member Author

@JarbasAl JarbasAl Oct 10, 2022 •
edited
Loading

Choose a reason for hiding this comment

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

@NeonDaniel are these available in neon_mq or only the other weather endpoints?

JarbasAl
JarbasAl commented Oct 10, 2022
View reviewed changes
ovos_backend_client/backends/neon.py
# Metrics API
def metrics_upload(self, name, data):
""" upload metrics"""
raise NotImplementedError() # TODO
Copy link
Member Author

Choose a reason for hiding this comment

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

@NeonDaniel could you please implement these missing methods since you are the most familiar with the mq messages?

@JarbasAl JarbasAl force-pushed the feat/neon_ovos_backends branch from 9c25203 to df048fe Compare October 10, 2022 16:35
@JarbasAl JarbasAl changed the title add neon + ovos backends add neon backend Oct 11, 2022
@JarbasAl JarbasAl force-pushed the feat/neon_ovos_backends branch from df048fe to f807046 Compare October 11, 2022 19:17
@JarbasAl JarbasAl mentioned this pull request Dec 6, 2022
Closed
This was referenced Jan 24, 2023
Closed
Closed
@JarbasAl JarbasAl mentioned this pull request Feb 8, 2023
Closed
@JarbasAl JarbasAl mentioned this pull request Sep 29, 2023
Closed
36 tasks
@JarbasAl JarbasAl closed this Jun 13, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@AIIX AIIX Awaiting requested review from AIIX

@NeonDaniel NeonDaniel Awaiting requested review from NeonDaniel

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@JarbasAl