Feature request: Additional trigger method for MQTT snapshots

Is your feature request related to a problem? Please describe.
Addition of a separate binary sensor that triggers upon the completion of a new snapshot sent via mqtt mainly for notification automations.
Describe the solution you'd like
Currently there doesn't seem to be an effective means of triggering an automation based solely on the arrival of a new mqtt snapshot. Attempting to use the mqtt snapshot topic as a trigger method is completely unreliable, it's erratic even causing the automation to periodically fire when there is no event. Using the events topic is the only means we currently have but is still not ideal because it often triggers sending duplicate snapshots.
Describe alternatives you've considered
For people who do not expose HA or frigate to the web and do not host frigate on the same machine options become very limited pretty quickly. Unconventional means of sending notifications that first require an image to be downloaded (such as snapshots through mqtt) are rather difficult to automate due to required addition of balancing inserted delays and juggling what is available through the events topic.
Additional context
Another potential option would be to add an additional unique identifier indicating a new image was sent via mqtt to the "update" and "end" events. ("new" would not work due to the event firing well before any image could be sent). It would have to be something unique to each event due to there currently not being any reproducible means of separating multiple "update" events from one another.

@SonomaGTS

It would have to be something unique to each event due to there currently not being any reproducible means of separating multiple "update" events from one another.

You mean like the event ID?

@SonomaGTS

It would have to be something unique to each event due to there currently not being any reproducible means of separating multiple "update" events from one another.

You mean like the event ID?

No something unique for each event fired within the same event ID. a clear indicator that a new snapshot has been sent/triggered and not just the same snapshot repeated with each "update" etc.
{   "type": "update",   "before": {     "id": "1607123955.475377-mxklsc",     "camera": "front_door",     "snapshot": "new-01"   },   "after": {     "id": "1607123955.475377-mxklsc",     "camera": "front_door",     "snapshot": "new-02"   } }

Can you describe how you want your notifications to work?

My current automation uses the events topic as the trigger, which has a condition that looks for the "new" type in the payload so that it can only fire once, my first action is a delay to give frigate time to send a mqtt snapshot and followed by a camera snapshot service to save the image which in turn leads to another delay to give time to save the image locally to be able to pipe that image into a notification. It's not a very efficient means of sending notifications with images but those in my scenario have no other choice due to our limitations. If two separate events were to happen one after another in a short enough time frame even with the current restrictions I have set sometimes the second notification I get is in fact the very last image sent by frigate of the first event.
I believe a separate topic/sensor (for each camera) that changes state between 0 > 1 > 2 > 3 might work out best for most people in similar situations as I.
state : 0 - indicates no new image
state: 1 - indicates new first snapshot image of an event
state: 2 - indicates updated snapshot image of same event
state: 3 - indicates last  snapshot image of the same event.
This could be used a stand alone trigger for an automation or used for a wait action to know precisely when a mqtt snapshot was changed.

You are describing what you implemented technically. Something like this:
I want to get a notification on my phone through the homeassistant app when an object is detected and I want to update the image in the notification when a new one is published for the tracked object. Once the event has ended, I want to update the notification with a final picture and a link to the video.

@hunterjm Has a blueprint that does just exactly that.

@hunterjm Has a blueprint that does just exactly that: https://community.home-assistant.io/t/frigate-mobile-app-notifications/311091

My blueprint (and the mobile app integration in general) requires HA to be exposed to the internet. It also just replaces an existing notification with the a new one on update or end event, so it doesn’t matter if the thumbnail is the same or not. That actually sounds more like what you want - the ability for your notification service to replace snapshots. Otherwise you can compare the before/after event MQTT message and see if the accuracy % of the label is higher in after than before. That means a new best image was found and the snapshot replaced.

Otherwise you can compare the before/after event MQTT message and see if the accuracy % of the label is higher in after than before. That means a new best image was found and the snapshot replaced.

There is a considerable random amount of time between when the event is fired and the MQTT image is updated. I had always thought the snapshot was separate from the Best image.  It would make sense for the time delay to be due to formatting the Best image to crop, add timestamp etc. I had not thought about this and directly addresses my second post. I will give it a try but ideally it would still be nice to have some means of reliably detecting exactly when a MQTT snapshot has been updated with a new image.

Can you describe what you are trying to accomplish with your notifications generally? I know that you don't expose HA of frigate. What do you use for notifications? Previously you have only explained how you want something to work technically but without enough additional context. Pretend you are explaining this to a completely non-technical person. I think it's likely there is a better way to get what you want.

I use smtp to send notifications via sms. I do this for several other reasons and it works extremely well for what I need it to do. However in this particular case it does add additional delay and more steps to accomplish what I need but the added delay and the more steps is something I am ok with. Of all the options of retrieving an image to send as a notification I really like the one the MQTT snapshot produces.
my current automation is as follows,
events topic as a trigger
condition against after top score above 0.70
condition against type "new"
condition against camera name
my first action is a delay of 5 seconds to give frigate time to send the first mqtt snapshot
my second action is using the camera snapshot service to save the mqtt snapshot camera image as a file
my third action is another delay of 3 seconds to give time for the file to be saved
my fourth action is to send the notification with attached snapshot file as a payload
I have two cameras that I use with Frigate currently, one of them being a doorbell camera and is the one that most of this is about.
I have rewritten this post so many times cause you guys got me questioning that there must be something staring me in the face that i am missing. I am certainly going to give what hunterjm mentioned a try and see if that works however I still feel like if there was a seperate sensor/topic like what I listed above that would help give more flexibility in creating automations but also help stubborn mules like myself who really only want to use those mqtt snapshots for notifications an easier time.
what i am trying to accomplish is getting that first mqtt image sent on a new event as quickly as possible and depending on certain conditions I would like to get updated images. I personally have no use for the end "best" image. while we are at home one notification an event is perfect. if we are not at home and someone is lingering at the door the one notification every 60 seconds or so would be nice. that is really all I am trying to accomplish.

That helps. I definitely think you can accomplish what you want with existing features. Can you post the actual yaml for the automation?

`
alias: Doorbell Snapshot
description: ''
trigger:

platform: mqtt
topic: frigate/events
condition:
condition: template
value_template: '{{ trigger.payload_json.after.top_score > 0.70 }}'
condition: template
value_template: '{{ trigger.payload_json.type == ''new'' }}'
condition: template
value_template: '{{ trigger.payload_json.after.camera == ''doorbell'' }}'
action:
delay:
hours: 0
minutes: 0
seconds: 5
milliseconds: 0
service: camera.snapshot
data:
filename: >-
/share/frigate_snapshots/snapshot_{{ trigger.payload_json.after.id
}}.jpg
target:
entity_id: >-
camera.{{ trigger.payload_json.after.camera }}_{{
trigger.payload_json.after.label }}
delay:
hours: 0
minutes: 0
seconds: 3
milliseconds: 0
choose:

conditions:

condition: time
after: '08:00'
before: '23:00'
sequence:
service: notify.mms_smtp
data:
data:
attachments:
- >-
/share/frigate_snapshots/snapshot_{{
trigger.payload_json.after.id }}.jpg
message: >-
{{ trigger.payload_json.after.label }} - {{
now().strftime("%_I:%M%p %x") }}
title: '{{ trigger.payload_json.after.camera }}'
default:

service: notify.mms_smtp_noma
data:
data:
attachments:
- >-
/share/frigate_snapshots/snapshot_{{
trigger.payload_json.after.id }}.jpg
message: >-
{{ trigger.payload_json.after.label }} - {{ now().strftime("%_I:%M%p
%x") }}
title: '{{ trigger.payload_json.after.camera }}'
mode: single
max: 10
`

Any idea? Still think something like a sensor is going to be the only real means of working.

Sorry. I have been busy with other things.
I don't recommend using the camera.snapshot service for notifications. You will never be able to accurately ensure the snapshot is for the current detection. Use the downloader service to download the image by id from frigate like this:
Add this to your HA config:
downloader:
  download_dir: /share/frigate_snapshots

Update your automation like this:
alias: Doorbell Snapshot
description: ''
trigger:
  - platform: mqtt
    topic: frigate/events
condition:
  - condition: template
    value_template: '{{ trigger.payload_json.after.top_score > 0.70 }}'
  - condition: template
    value_template: '{{ trigger.payload_json.type == ''new'' }}'
  - condition: template
    value_template: '{{ trigger.payload_json.after.camera == ''doorbell'' }}'
action:
  - delay:
      hours: 0
      minutes: 0
      seconds: 5
      milliseconds: 0
  - service: downloader.download_file
    data_template:
      url: 'http://<frigate_ip>:5000/api/events/{{trigger.payload_json["after"]["id"]}}/snapshot.jpg'
      filename: 'snapshot_{{ trigger.payload_json["after"]["id"]}}.jpg'
      overwrite: true
  - delay:
      hours: 0
      minutes: 0
      seconds: 3
      milliseconds: 0
  - choose:
      - conditions:
          - condition: time
            after: '08:00'
            before: '23:00'
        sequence:
          - service: notify.mms_smtp
            data:
              data:
                attachments:
                  - >-
                    /share/frigate_snapshots/snapshot_{{ trigger.payload_json["after"]["id"]}}.jpg
              message: >-
                {{ trigger.payload_json["after"]["label"] }} - {{
                now().strftime("%_I:%M%p %x") }}
              title: '{{ trigger.payload_json["after"]["camera"] }}'
    default:
      - service: notify.mms_smtp_noma
        data:
          data:
            attachments:
              - >-
                /share/frigate_snapshots/snapshot_{{ trigger.payload_json["after"]["id"]}}.jpg
          message: >-
            {{ trigger.payload_json["after"]["label"] }} - {{ now().strftime("%_I:%M%p
            %x") }}
          title: '{{ trigger.payload_json["after"]["camera"] }}'
mode: single
max: 10

I have used the downloader before. I have found the timing is more predictable (in regards to using the delays) using the camera snapshot not to mention since the image was already being sent anyways it's miniscule less resource intensive. I have tried many elaborate ways to achieve what I would like to have, even using folder watcher to know when an image was in fact created but in the end I settled on what I posted above because it's less code that tackles multiple cameras and objects in the same automation. Using folder watcher was the closest I could get to detecting new images but using that was jumping a lot of hoops to achieve what I think could be more easily solved and more universally used.
The main issue stems from the inability to use the snapshot mqtt as a trigger. That is a HA issue from what little I am managed to research and not something you can fix directly through frigate which prompted this feature request to bypass that but my suggestions above would also allow even more freedom and options beyond using an mqtt as a trigger anyways. I was hoping there was something I had missed that would tackle what I wanted but I been beating my head on this for several months now and just broke down to make this request.

Are you saying the image isn't guaranteed to be saved to disk immediately after using the downloader service and you need a delay before you use it? If that's true, then you could try a shell integration and call curl.
What I posted is the same amount of code and still works for all cameras. What you are trying to achieve isn't any different than every other notification automation I have seen. It just uses email and mms instead of telegram or the HA app.

I am saying it took took a little longer to complete. I think my initial delay was something like 7 or 10 vs the current 5. I even tinkered and ran it as two automations. one being the event topic trigger using the downloader to download the image then a separate automation using the download complete event as a trigger to fire off the notification. It was kind of messy imo, it worked, was not as flexible but ultimately I didn't like it and went back to what I am using now.  Using folder watcher was very similar results.

The snapshot.jpg endpoint from frigate works instantly in the current version, even before the snapshot file is saved to disk. Others using telegram to send notifications must use the downloader too, and they don't need to add a delay after calling the downloader service.

I was referring to the time it took to download an image from a remote source vs saving an image already present in the mqtt buffer. My setup is HA on an rpi4 and Frigate on an older I7 and it just took longer for me to fetch the images any other means. I feel like we are kind of straying off topic though. unless there is some means of using the snapshot endpoint as a trigger?

Not off topic. Even if you could use the snapshot topic to trigger automations, I still think it's a bad approach. I would expect that you can remove the delay after the downloader service call and everything will work. If the file is not available immediately after calling that service, I would consider it to be a bug. For others using the downloader service, it is available immediately.

it would still yield the same results tho. not every event offers a new image plus you would get a double of the same at the end of the event.

Not the same result. There will never be a way to know which event the mqtt image is connected to, or guarantee that it was updated before the other automation fires.
I can add an attribute to the events topic with the frame time of the current best snapshot of the event. That will allow you to know if the frame used for the event changed, and that will tell you when to fetch an updated snapshot with the downloader service with the event id.
You still shouldn't be using the camera snapshot service from the mqtt camera though. There is no way to guarantee that the image was updated there before you save the snapshot.

I can add an attribute to the events topic with the frame time of the current best snapshot of the event. That will allow you to know if the frame used for the event changed, and that will tell you when to fetch an updated snapshot with the downloader service with the event id.

That would help. If you can make it something we can condition against that will ensure the next image fetched was updated that is what this topic was all about.
I will swap it back over to use the downloader later this evening. The camera snapshot was just ideal cause it not only handled the saving locally but the image already came formatted the way I want it. not to mention I use that snapshot already as a quick glance on the dashboard.

@blakeblackshear would you mind transferring this to https://github.com/blakeblackshear/frigate ? If I'm following the proposed remedy correctly, I think it's on the frigate backend side.

This should already be resolved in the beta anyway


Slight changes

fix polymer lib path
automate bower install after npm install
retina enable circle-timer canvas

Sorry for missing this, I've properly set up notifications now :(
LGTM, except that maybe it should use window.devicePixelRatio to do the scaling. There are devices out there that have a dPR of 3, for example.

OK, fixed using DPR. Can you check again?

LGTM! There is a merge conflict, but feel free to merge when that's sorted.


Travis failure Feb 2019

@muschellij2 do you know why is Travis failing lately?
https://travis-ci.org/dorianps/LESYMAP
At first I thought it was complaining about not findings ANTsR (which also fails Travis it seems), but in the last try to start the build there is a timeout error.

Looks like github downloads were really slow for some reason and travis
times out
On Tue, Feb 12, 2019 at 11:38 AM dorianps<EMAIL_ADDRESS>wrote:

@muschellij2 https://github.com/muschellij2 do you know why is Travis
failing lately?
https://travis-ci.org/dorianps/LESYMAP
At first I thought it was complaining about not findings ANTsR (which also
fails Travis it seems), but in the last try to start the build there is a
timeout error.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dorianps/LESYMAP/issues/14, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABBnriC3OeLHRzIS86quz86lpm2eOM6Gks5vMzROgaJpZM4a4G45
.

--
John

Restarted the build and failed again. I can see an error about ANTsR missing, and another one about tarballs of noise.gz files.

Sorry - don't have time to look into this for at least a few weeks.
John
On Tue, Feb 12, 2019 at 5:55 PM dorianps<EMAIL_ADDRESS>wrote:

Restarted the build and failed again. I can see an error about ANTsR
missing, and another one about tarballs of noise.gz files.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dorianps/LESYMAP/issues/14#issuecomment-463047430,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABBnrqw_2cFfrxPxBM9GDdNVv3AOM3S4ks5vM4zAgaJpZM4a4G45
.

@muschellij2 , thanks for the previous help, now the build works on Ubuntu, but exceeding compile time in OSX. Any idea why it is trying to install everything (ITKR/ANTsRCore/ANTsR) in OSX but not in linux?
https://travis-ci.org/dorianps/LESYMAP/builds/507502255

It's because devtools is grossly outdated for OSX binary builds :
https://cran.r-project.org/web/packages/devtools/index.html, see Package
source: devtools_2.0.1.tar.gz
Windows binaries: r-devel: devtools_2.0.1.zip, r-release:
devtools_2.0.1.zip, r-oldrel: devtools_2.0.1.zip
OS X binaries: r-release: devtools_1.13.6.tgz, r-oldrel: devtools_1.13.6.tgz
and not getting installed correctly.
John
On Mon, Mar 18, 2019 at 11:24 AM dorianps<EMAIL_ADDRESS>wrote:

@muschellij2 https://github.com/muschellij2 , thanks for the previous
help, now the build works on Ubuntu, but exceeding compile time in OSX. Any
idea why it is trying to install everything (ITKR/ANTsRCore/ANTsR) in OSX
but not in linux?
https://travis-ci.org/dorianps/LESYMAP/builds/507502255
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dorianps/LESYMAP/issues/14#issuecomment-473958139,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABBnrk8vv5RepSKXYa9rmipFistsKHDYks5vX6-6gaJpZM4a4G45
.

Ok, so I'll ignore for now.


Fix Auth Header parsing for values that don't contain spaces between commas

Changes Introduced
Fixed parsing of Auth headers that don't have spaces between commas
Applicable linked issues
#387
Checklist (check all that apply)

[X] This change maintains backwards compatibility
[X] I have introduced tests for all new features and changes
[ ] I have added documentation covering all new features and changes
[X] This pull-request is ready for review

Wonderful, @etspaceman. Yeah, your solution seems good since you leverage on the fact that everything else was already in place 😄 Thank you.

Codecov Report
Base: 88.70% // Head: 88.63% // Decreases project coverage by -0.06% :warning:

Coverage data is based on head (6f05e97) compared to base (fd75b54).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #388      +/-   ##
==========================================
- Coverage   88.70%   88.63%   -0.07%
==========================================
  Files          93       93
  Lines        3001     3001
  Branches      170      170
==========================================
- Hits         2662     2660       -2
- Misses        339      341       +2

Impacted Files
Coverage Δ

...rc/main/scala/kinesis/mock/KinesisMockRoutes.scala
89.01% <ø> (ø)

src/main/scala/kinesis/mock/cache/Cache.scala
83.85% <0.00%> (-0.33%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Sd card not found on Mega2560

I am trying your great AVR logger example.
Runs on Uno but when I move to Mega (using pins 50 to 53 for SPI)
#define PIN_SPI_SS    (53)
#define PIN_SPI_MOSI  (51)
#define PIN_SPI_MISO  (50)
#define PIN_SPI_SCK   (52)
it fails to find the sd card
Any suggestions for debugging further?

I connected an Adafruit MicroSD breakout to these pins and ran the AvrAdcLogger example without modification.
Here is the output:

Type any character to begin.

Unused stack: 6923

type:
b - open existing bin file
c - convert file to csv
l - list files
p - print data to Serial
r - record ADC data

Opening: AvrAdc00.bin
Allocating: 100 MiB
Sample pins: 0 1 2 3 4
ADC bits: 10
ADC clock kHz: 500
Sample Rate: 5000.00
Sample interval usec: 200.00
Logging - type any character to stop

Truncating file
Max write latency usec: 1068
Record time sec: 12.306
Sample count: 61530
Overruns: 0
FIFO_DIM: 96
maxFifoUse: 2
Done

Odd, I am using the same Adafruit breakout board.
I will try again.

OK found the problem.
You need to connect MOSI -> DI and MISO -> DO on adafruit board.


Is it posible to flash html somehow?

Hi,
i will be very happy, when if the possibility of adding html into message could be implemented.
Thanks!

I'll look into this, generally injecting html has severe security implications, but will look into possible options


gen-bundle-image: refactor to accomodate microshift preset

This PR refactor around set_bundle_variables, generate_image and sign_bundle_files functions to make sure image can be generated for microshift bundle also.

/cherry-pick release-4.14


Consider removing ability to invert questionnaire questions/answers

Describe the issue and expected behaviours. If this is a bug, describe the steps to reproduce the problem, what you observe, and what you expected to observe.
Acceptance Criteria

[ ] Remove the inverted flag from the questionnaire data types
[ ]  Relevant questions are rephrased to indicate that they are no longer inverting
[ ] This will require updated translations of relevant questions and answers

design team will come up with english flow and tagging logic, hand off to devs, and in the meanwhile prepare to send to translations

SOLUTION: change to-
Question: Which of the following important Canadian documents do you still need?
Answers:
I need a Social Insurance Number
I need a Canadian bank account
I need a Permanent Resident Card
I need to register for MSP (Medical Services Plan)
I need a BC Services Card (health card)
I need a BC Driver’s License
I am not sure what I need
Related taxonomies should not change since we have accidentally been showing topics backwards anyways, except for the last option. The additional option of 'i'm not sure what i need' should lead users to see all of the above options.

I expected to see in both BC and Manitoba the new question as  "Which of the following important Canadian documents do you still need?" (question 2 on BC & 4 on MB), with the answer options being as follows:
Answers (BC):
I need a Social Insurance Number
I need a Canadian bank account
I need a Permanent Resident Card
I need to register for MSP (Medical Services Plan)
I need a BC Services Card (health card)
I need a BC Driver’s License
I am not sure what I need
Answers (MB):
I need a Social Insurance Number
I need a Canadian bank account
I need a Permanent Resident Card
I need to register for Manitoba Health Care Coverage
I need a Manitoba Health Card
I need a Manitoba ID card or a Manitoba Driver's Licence
I am not sure what I need
BUT instead, I the old question that say's "Do you have any of the following docuements?" and answers that are not inversed, instead they show "INV" in capitols next to the original statements.
Please refer to the screenshots to fix.


USHIFT-1861: Add verbosity to router-default

Which issue(s) this PR addresses:

Closes #

/test microshift-metal-tests

/test microshift-metal-tests

/test microshift-metal-tests

/test microshift-metal-tests

/test microshift-metal-tests

/test microshift-metal-tests

/test microshift-metal-tests

/test microshift-metal-tests

/test microshift-metal-tests

As a general idea I'd prefer to make sure tests leave clean environment on teardown, rather than next test making sure of that but for now looks good to me
/lgtm

Metal test failed because this change was not yet merged: https://github.com/openshift/microshift/pull/2725
/test microshift-metal-tests

Overriding the rebase as the failure relates to a patch that needs an update and that is handled in a different PR.
/override ci/prow/test-rebase

/cherry-pick release-4.15


valid FormsyText will overwrite underlineFocusStyle border-color style with default color

It seems that the style set inside underlineFocusStyle will be overwritten with default color when the FormsyText validation is passed. Very annoying bug since the default is bright blue.
Any workaround, or will I have to disable underline in the meantime?
Using :
"react": "15.4.2",
"formsy-material-ui": "^0.5.5",
"formsy-react": "^0.18.0",

Hello, is this issue being acknowledged?

To try and be helpful, adding this code in line 152 of app/Main.js inside the example code will let you see the bug:
underlineFocusStyle={{borderColor:'green'}}
You will see that when you click on the Name field, the color shows green (as material-ui intended). But when you enter text into this field, it turns blue, because formsy somehow defaults the color back to material-ui's blue color upon validation.
Any quick fix for this while it's being looked at?

Just my initial thoughts because I don't have time to dig into this right now:
We recently accepted a change that added the underlineStyle and underlineFocusStyle props, see here. It probably has to do with this.validationColor() being checked for a value before deferring to underlineStyle. There might be a theme variable for material-ui that sets the validation color. So there may be 3 or 4 total color settings you have to account for: basic, focused, and valid, invalid. Just speculating though.

Guys this needs attention, lots of project use the lib in production. Bugs been happening for a while

@kdichev your first sentence--there's no place for that here. This is an open source project, not a product that you paid for. Every bit of progress happens on a voluntary basis. If you're using this in production and a bug pops up that you don't think is being fixed quickly enough--dig into the library and offer some insight, a PR, or a question.
Aside from that, thanks for pointing out how to use the theme provider for those not familiar. I reckon there is a simple fix, but I have higher priorities this week. Maybe Saturday if this is still open, I'll jump in.

Hey I totally did not have an intend to sound like that, demanding... I'm a young dev so I don't know the codex. I'll checkout the source and try to fix it :)

Hey thanks for acknowledge the bug. I was worried I was the only one experiencing it since nobody else had raised the issue, as it seems to be quite damaging.
I don't have experience fixing libraries, but I'll follow this issue closely and hopefully learn how the eventual fix goes.


Remove undeplopyed version

Remove undeployed versions
Description

Remove undeployed version

Federation
Bridge
AllowTokens

How to Test

Run both ganache-cli
Enter into the bridge directory and run all the tests

Case 1

Go to bridge directory

$~ cd bridge

Run all the tests

$~ npm run test

Expected Result

It should pass the test

Run the first ganache-cli

$~ npm run ganache

Open another shell and run the ganache mirror

$~ npm run ganache-mirror

Go to the federator directory

$~ cd ../federator

Run the integration test

$~ npm run integrationTest

Expected Result

It should pass the test

Run the NFT integration test

$~ npm run nftIntegrationTest

Expected Result

It should pass the test

Checklist
Bridge Directory cd bridge

[x] Lint is clean npm run lint
[x] Test is passing npm run test
[x] Contracts are compiling npm run compile

Federator Directory cd federator

[x] Lint is clean npm run lint
[x] Test is passing npm run test
[x] Typescript is compiling npm run build

Coverage remained the same at 94.826% when pulling 91f541a70ef82a9de5592a135f3b472bc73ed611 on remove-undeplopyed-version into 10a8479c0d645d9094de5b484c5c318e3f24ab60 on nftbridge.


Using typer.prompt with choices from Enum doesn't display choices

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the Typer documentation, with the integrated search.
[X] I already searched in Google "How to X in Typer" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to Typer but to Click.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
import typer
from enum import Enum

class ChoiceEnum(Enum):
    option_one = "opt1"
    option_two = "opt2"
    option_three = "opt3"

app = typer.Typer()

@app.command()
def subcommand(argument_option: ChoiceEnum = typer.Option(ChoiceEnum.option_two.value,
                                                          prompt="argument option",
                                                          show_choices=True)):
    prompt_option: ChoiceEnum = typer.prompt("prompt option",
                                             ChoiceEnum.option_three.value,
                                             show_choices=True)
    print(f"Argument choice: {argument_option}")
    print(f"Prompt choice: {prompt_option}")

if __name__ == "__main__":
    app()

# The result of running the above script results in the following output:
#
# > argument option (opt1, opt2, opt3) [opt2]:
# > prompt option [opt3]:
# > Argument choice: ChoiceEnum.option_two
# > Prompt choice: opt3

Description
Using the typer.prompt function isn't displaying the choices of an enum the same way that an Option does.
Am I doing something wrong here? Is there some other way I should be doing the prompt in order for it to display the choices as is done with the argument choices?
Also, why are the return values different (the prompt gives me the string "opt3", while the argument returns the enum value itself).
Operating System
macOS
Operating System Details
No response
Typer Version
0.6.1
Python Version
Python 3.10.8
Additional Context
No response

Managed to figure out that I needed to add type=ChoiceEnum to the prompt call to get the return value to be of the same type as the argument. Still doesn't fix showing the choices.

Managed to find a workaround for this by not using an enum and instead use the click.Choice type:
    click_choice = click.Choice(['opt1a', 'opt2a', 'opt3a'])
    click_prompt_option: click.Choice = typer.prompt("click prompt option",
                                                     "opt1a",
                                                     show_choices=True,
                                                     type=click_choice)

Still think that the typer behavior needs to be looked at to make the behavior a bit more predictable.

I solved the issue by creating a new Choice class for (Int)Enums:
class IntEnumChoice(click.Choice):
    def __init__(self, enum_type: EnumType, case_sensitive: bool = True) -> None:
        choices = [f"{value.value}: {value.name}" for value in enum_type]
        super().__init__(choices, case_sensitive)
        self._enum_type = enum_type

    def convert(self, value: t.Any, param: t.Optional["Parameter"], ctx: t.Optional["Context"]) -> t.Any:
        try:
            return self._enum_type(int(value))
        except ValueError:
            choices_str = ", ".join(map(repr, self.choices))
            self.fail(
                ngettext(
                    "{value!r} is not {choice}.",
                    "{value!r} is not one of {choices}.",
                    len(self.choices),
                ).format(value=value, choice=choices_str, choices=choices_str),
                param,
                ctx,
            )

You can easily adapt it for string Enums.

I'm looking for a typer way to replace:
parser.add_argument('move', choices=['rock', 'paper', 'scissors'])

It would be great to have a similar feature in the core of typer.


Update how-atp-safe-links-works.md

Added a note in the end to tell customers that it is normal to see a few seconds of delay at boot time to verify that the user's protected by Safe Links.

@nenonix @JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: ruiwensu sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


E2E taking too long to run

At the moment running the complete test suite takes ~1h. This prevents us from running the test suite on "pull request" basis.
We would like to take a deeper look into this and hopefully improve test time as much as possible.
the completed runs:
https://circleci.com/gh/kubermatic/machine-controller/2738

for reference see also:
#159
#129

Closing, as this got fixed within #159


DO NOT TEST - first version of component tokens with updates

Here are my notes on all of the small bug fixes: https://goa-dio.atlassian.net/wiki/spaces/DS/pages/3552313784/A+lot+of+little+bug+fixes


Allow binding other swipe events

I want to bind a swipeup event for a different function, "swipeup"or "swipedown"appears to be disabled in swiper.
How can I enable it?

I already resolved, You can use tapup or tapdown event intead of swipeup and swipedown.


chore: create .dockerignore file

This pull request adds  .dockerignore file to streamline the Docker build process by excluding unnecessary files and directories.
File exclusions:
node_modules
docs
logs
build
cache

@coleam00 it was my hope that this would have been merged at the same time as https://github.com/coleam00/bolt.new-any-llm/pull/71 In that h only reason I didn' add it there was  @ajshovon had already crated it, and to subvert the PR workflow would have been unhelpful.

Thanks for pointing that out @hillct! I've merged it now, thank you @ajshovon!


Decode SafeToL2Migration contract transactions

What is needed?
The migrateToL2 call of the SafeToL2Migration contract is not decoded. It is necessary to analyse the best way to add the way to decode contract operations as SafeToL2Migration by default.

looking good to me buen señor


Origin/develop

What is the purpose of the change
XXXXX
Brief changelog
XX
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Coverage decreased (-0.2%) to 48.754% when pulling e67a8d2ba7dae40232c9fe98b8ef171f6c9d2895 on daokuang:origin/develop into bbb723fb805357d0fb5caf2282c499d89469e994 on apache:develop.

@daokuang As this pr has travis ci problem, I will close this pr temporarily, please feel free to reopen it.


feat: dkn in docker

Motivation
I am trying work on the docker container with DKN that will able to use the container from ollama and other resources.
We need to build a script that work with these feature that also enable connect the internet via sock5 and https protocol.
work in progress.

services:
    dknmine:
      image: whatwhat/dkn-compute-node:latest
      container_name: dkn_compute
      environment:
        - 'DKN_WALLET_SECRET_KEY=xxxxxxxxxxxxxxxxxxxxxx'
        - 'http_proxy=http://host.docker.internal:7890'
        - 'https_proxy=http://host.docker.internal:7890'
        - 'all_proxy=socks5://host.docker.internal:7891'
      extra_hosts:
         - "host.docker.internal:host-gateway"
         - "<IP_ADDRESS>:<IP_ADDRESS>"
      volumes:
        - /srv/dkn:/dkn:rw
      working_dir: /workspace
      ports:
        - "19880:9880"
      shm_size: 16G
      deploy:
        resources:
          reservations:
            devices:
                - driver: nvidia
                  device_ids: ['0']
                  capabilities: [gpu]
      stdin_open: true
      tty: true
      restart: unless-stopped
      networks:
        - nvidanetgroup
    ollama:
        volumes:
          - /srv/models/ollama:/root/.ollama
        container_name: ollama
        pull_policy: if_not_present
        tty: true
        restart: unless-stopped
        image: ollama/ollama:latest
        ports:
          - 11434:11434
        networks:
          - nvidanetgroup
        deploy:
          resources:
            reservations:
              devices:
                - driver: nvidia
                  device_ids: ['0']
                  capabilities: [gpu]
volumes:
  valkey_data2:
networks:
  nvidanetgroup:
    name: mlnetgroup
    external: true

have you tried our compose.yml?


Style/MutableConstant autocorrect doesn't properly correct with %w() word list

Given class definition
# frozen_string_literal: true
class MyClass
  STRINGS=%w(a b c)
end

Expected behavior
Run auto correct
# frozen_string_literal: true
class MyClass
  STRINGS=%w(a b c).freeze
end

Actual behavior
Run auto correct
# frozen_string_literal: true
class MyClass
  STRINGS=[%w(a b c)].freeze
end

At this point STRINGS in the new (autocorrected) class is an array with an array.  Not the same as a flat array of strings.
Steps to reproduce the problem
Save MyClass from the top as my_class.rb
Run
rubocop -a my_class.rb
check the diff
RuboCop version
$ rubocop -V
0.47.1 (using Parser <IP_ADDRESS>, running on ruby 2.3.3 x86_64-darwin15)

I can't reproduce this problem.
$ ls -a
.  ..  my_class.rb

$ cat my_class.rb
# frozen_string_literal: true
class MyClass
  STRINGS=%w(a b c)
end

$ rubocop -a my_class.rb
Inspecting 1 file
C

Offenses:

my_class.rb:2:1: C: [Corrected] Add a space after magic comments.
class MyClass
^
my_class.rb:3:10: C: [Corrected] Surrounding space missing for operator =.
  STRINGS=%w(a b c)
         ^
my_class.rb:3:11: C: [Corrected] Freeze mutable objects assigned to constants.
  STRINGS=%w(a b c)
          ^^^^^^^^^

1 file inspected, 3 offenses detected, 3 offenses corrected

$ cat my_class.rb
# frozen_string_literal: true

class MyClass
  STRINGS = %w(a b c).freeze
end

$ rubocop -V
0.47.1 (using Parser <IP_ADDRESS>, running on ruby 2.4.0 x86_64-linux)

Do we need any configurations(.rubocop.yml) for reproduce?

This was fixed in https://github.com/bbatsov/rubocop/pull/3962, which isn't released yet.

Thanks.  I look forward to the release.


Add eslint

Used https://github.com/Shopify/eslint-plugin-shopify with some rule overrides in the .eslintrc file. All of the starter file branches have also been updated. Code snippets on https://github.com/Shopify/developer-website are next.
🎩ed on all browsers. Updated @shopify/koa-shopify-auth to fix ITP issues, still need to make this change on Starter branches. Updated server.js to reinclude webhooks.

Going to stop pushing changes until we can talk about rule overrides as a group.

@shopify/koa-shopify-auth has been updated on all of the starter branches

FYI waiting to merge this at the same time as the updates on the developer site. Starting those today.


Add mypy to .pre-commit-config.yml

Fix https://github.com/ml-explore/mlx-examples/issues/829

Ah. Circle CI calls pre-commit. So we have to fix all mypy errors before enabling mypy in pre-commit.

Oh yes.. that might take some time :. I wonder if we could point it to a list of files to start and add more over time 🤔

Sure. I am making a list and will paste to https://github.com/ml-explore/mlx-examples/issues/829


fetchgit cannot handle some tags that look like hashes anymore

With #104714 merged nix-prefetch-git cannot fetch all tags anymore: https://github.com/NixOS/nixpkgs/issues/114439#issuecomment-798801869
This isn't the fault of #104714 (only modifies checkout_hash()) but now it's a problem that checkout_ref() cannot fetch some tags.
In the case of #114439: checkout_ref() will be called with hash=20210208 and ref=. and ref_from_hash 20210208 will not return a reference.
Relevant code parts:
https://github.com/NixOS/nixpkgs/blob/6779902b32a208f39ab5d073714a12a7ac72bcf1/pkgs/build-support/fetchgit/nix-prefetch-git#L230-L235
https://github.com/NixOS/nixpkgs/blob/6779902b32a208f39ab5d073714a12a7ac72bcf1/pkgs/build-support/fetchgit/nix-prefetch-git#L167-L191
https://github.com/NixOS/nixpkgs/blob/6779902b32a208f39ab5d073714a12a7ac72bcf1/pkgs/build-support/fetchgit/nix-prefetch-git#L113-L117
Possible solutions:

Merge checkout_hash() and checkout_ref()

Likely the most difficult solution and there might be a reason why we need both (haven't looked at that yet)
But IMO this might be the best solution as Git should also detect any ambiguities (e.g. if a tag and branch with the same name exist).

Fix ref_from_hash()

The sed magic is too much for me (\,$hash\t, { s,\(.*\)\t\(.*\),\2,; p; q}) but git ls-remote origin could certainly correctly return the desired tag

Extend checkout_hash() to handle `"$object_type" == "tag" as well

Should be the easiest solution but it would be a dirty hack and we should IMO avoid it.

I can offer two more possible solutions:

Merge rev and hash parameters into one

Something like rev_from_hash would always be attempted, but might just return the original value if no matching hash is found
git fetch would need to fail safely if given a hash as the last argument

Add a separate tag (and possibly head, for branches) parameter to fetchgit

This might be a painful change
It seems unlikely(?) that anyone wants to treat tags, hashes, and branches interchangeably when fetching sources

Also, #115145 might be a case where checkout_hash should handle a tag rev but doesn't. I agree that for just this case, adding support for the tag object type would be a hack.


Improve readme + Minor beautify of code + Update dev dependencies

Q
A

Is bugfix?
❌

New feature?
❌

Breaks BC?
❌

Fixed issues
-

👍

👍


Implemented SuggestionsManager, ButtonStep, and StepManager in compliance with our CRC cards design

created the SuggestionsManager, ButtonStep, and StepManager classes in compliance with the entities layer of our clean architecture design. This allows functionality of our program to analyze the transcript and be able to find an optimal talk step when called by the use case interactor. It is significant in that we are following the Single Responsibility Principal for each class and having our implementations in the higher level entities of SuggestionManager and StepManager depend on abstractions of TalkStep and Scorable rather than concrete implementations.

See https://github.com/edwardhuahan/tli-speakfluid/pull/69


[BUG]: Unable to get a copy of the innovation backlog working

Describe the bug
I have created a copy of the innovation backlog app. When I run this copy, data for ideas already submitted is not appearing.
Component (please tell us which flow or app you are experiencing issues with):

Innovation Backlog
1.17
Innovation Backlog

To Reproduce
Steps to reproduce the behavior:

Create a copy of the innovation backlog app
Launch this copy
Data is not seen within the Ideas gallery even though the data can be seen using the Innovation Backlog app within the managed solution

Expected behavior
User should be able to view the data within the Ideas entity using an app created from the managed Innovation Backlog app  as a copy
Screenshots
If applicable, add screenshots to help explain your problem.
Managed Innovation Backlog App

Copy of the Innovation Backlog App

Additional context
Add any other context about the problem here.

Can you please let me know how you created the copy? Was it just a save-as into the same environment?

Yes, it was.

Created a new solution.
Added the existing Innovation Backlog app.
Opened the app to edit it.
"Save As" in the same environment.

Do you see errors too? As shown in #879?

Yes, I see those errors as well.

Gotcha. This is the same issue as #879 then. I will track with that one so please follow it to find resolution.

Duplicate of #879


Issue for creating a service that references a variable that does not exist

Version: main
UI: dev-f2d563b
Commit ID: e8abf74ef7061965340ac1985a2d6d8a08747bdd
Env: AWS EC2 Ubuntu 22.04 amd64
Test Steps:

Create a service that references a variable that does not exist. It shows failed to create service

2023-07-28T01:29:24.362Z	E	api	error requesting /v1/services?projectID=471301531866497031: missing variables: [host-ip]
github.com/seal-io/seal/pkg/apis/runtime.Erroring.func1
	github.com/seal-io/seal/pkg/apis/runtime/middleware_error.go:40
github.com/gin-gonic/gin.(*Context).Next
	github.com/gin-gonic/gin@v1.9.0/context.go:174
github.com/seal-io/seal/pkg/apis/runtime.Recovering.func1
	github.com/seal-io/seal/pkg/apis/runtime/middleware_recovery.go:44
github.com/gin-gonic/gin.(*Context).Next
	github.com/gin-gonic/gin@v1.9.0/context.go:174
github.com/seal-io/seal/pkg/apis/runtime.Observing.func1
	github.com/seal-io/seal/pkg/apis/runtime/middleware_observe.go:78
github.com/gin-gonic/gin.(*Context).Next
	github.com/gin-gonic/gin@v1.9.0/context.go:174
github.com/gin-gonic/gin.(*Engine).handleHTTPRequest
	github.com/gin-gonic/gin@v1.9.0/gin.go:620
github.com/gin-gonic/gin.(*Engine).ServeHTTP
	github.com/gin-gonic/gin@v1.9.0/gin.go:576
net/http.serverHandler.ServeHTTP
	net/http/server.go:2947
net/http.initALPNRequest.ServeHTTP
	net/http/server.go:3556
net/http.(*http2serverConn).runHandler
	net/http/h2_bundle.go:5910
2023-07-28T01:29:24.362Z	D	api	500 | HTTP/2.0 |      385 B |       88 B |   18.284019ms |   <IP_ADDRESS> | POST    /v1/services?projectID=471301531866497031

Remove the variable and save again. It shows dupilicated

Check the service list. The service was actually created in step 1.

Internal processing flow:

create the service db object first
check variables before creating k8s job
So after saving the service, the service is created wether the variables existed or not.
The same goes for modifying a service with the wrong variable.

Environment

Walrus version: main b51994e3fdba0aaeaba9439f6c609c105e5cbb92
UI version: dev-3b160ec

Test Result:

Reference a variable that does not exist. --> pass

Reference a service output variable that does not exist. --> failed

After dissusion, we deside not to validate service output. Otherwise, we can't clone batch of services that are connected.

Environment

Walrus version: main 7dddda0f4b8d278b4731528af87dbf93f8a17d97
UI version: dev-ec890a0

Test Result: pass, no error is returned while creating a services.

Environment

Walrus version: main 7bb79389ab0436969f4c4214e38918ee6c6e9070
UI version: dev-7885278

Test Result: This issue can be reproduced in main.
https://github.com/seal-io/walrus/assets/26084756/e77d58a5-cf11-4944-a8c1-fa645304135c

Environment

Walrus version: main fa6e56aca10e61259243c938130c8e63dd19ba27
UI version: dev-4218197

Test Result:
This error is also returned when upgrading.

Environment

Walrus version: main 0a0f275bc92d1f7f379e5d97337b3362096fe508
UI version: dev-87c5bd5

Test Result: pass


cask/audit: disallow new cask to have token in tap_migrations.json

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?
[x] Have you successfully run brew man locally and committed any changes?

This audit already exists for formulae (but not for casks): https://github.com/Homebrew/brew/blob/master/Library/Homebrew/formula_auditor.rb#L688-L699
Related: https://github.com/Homebrew/homebrew-cask/pull/98719

Review period will end on 2021-01-27 at 09:20:43 UTC.

Review period will end on 2021-01-27 at 09:20:43 UTC.

Review period ended.

Review period ended.

Thanks @SeekingMeaning!

Thanks @SeekingMeaning!


forms need sensible starting point that lands in the study area

coordinates in form template assumed required becasue we need a spatial object for gpkg to burn to file correctly
currently template is made from skeena project file putting a point way out of bounds for many study areas.
Could perhaps grab centroid of watershed group used to clip layers and make spatial object based on that. or maybe its not worth it for the customization.  REally blows up the screen when you zoom to project in mergin or zoom to layer (form) in Q

is it not possible to use a gpkg with no records?
with vector tiles as source for background layers, the background layers should display nicely in scale specific styles

I tested burning to blank geopackage with R (started with spatial object then sliced to 0 rows) but it showed as a non-spatial table in Q only and was not edible in mergin. Could def be another way though....
cool beans with the vtiles


test move 2 cases from p2 to p0

Proposed changes
releated: #36787

test_text_garbled_file
test_hive_statistic_timeout
test_hive_to_array
test_hive_statistic
test_hive_default_partition

run buildall

run buildall

run buildall


ENH Postgres - Add support for Postgres Table Inheritance

Consider this in the 6.x timeframe.

I'm closing this for now.


cfcache directories not deleted from venv

When a V2 module is installed (non-editable install) and the compiler runs against it, a __cfcache__ a directory is created in the model directory of the installed package. When the module is later on deleted from the venv, the __cfcache__ directory is not deleted from the venv.
 ~/.virtualenvs/inmanta-core/lib/python3.6/site-packages/inmanta_plugins/elaboratev2module $ find .
.
./model
./model/__cfcache__
./model/__cfcache__/_init.2020_6.cfc

Duplicate created in #4360. Closing this one because the duplicate covers breaking implications in addition to the behavior observed here.


Vocareum: Autograder that accepts multiple files

From @jesseilev on December 4, 2015 19:36
Need to modify script to allow passing in multiple source files, e.g.
check50 2015.fall.pset5.speller dictionary.h dictionary.c

Copied from original issue: Launch-Code/cs50x-live-2016#28

duplicate of #12


[Idea wanted] Add centralised link list in the network management [long loading time]

Short description

This is the (probably) easiest and laziest implementation of centralised link list and the classical problem exists here too: the loading time is unacceptably long.
This PR is not intended as final proposal and opened to demonstrate the problem and to ask for ideas/suggestion how to solve it.

Do you see any possibility to improve loading time with this implementation?
If we structure the centalised link checker in another way (different from region link list), what do you imagine?

See this cool insctuction for how to test with realistically large data.
Proposed changes

Reuse the template and view of region link list

Side effects

Extremely long loading time

Resolved issues

Fixes: #1443

Pull Request Review Guidelines

I have done some optimizing and pushed everything to a branch: feature/global_link_list_performance
Originally, the global link list did not load at all for me, and with this branch it takes about 7 seconds.
Feel free to merge if it makes sense to you :)
The actual database queries only take about 500ms, so there is still a lot of potential for optimization, but I think that would require a bit more code restructuring. The lowest hanging fruit it probably removing the hot python loop in filter_urls. Maybe it would be possible to use something like https://docs.djangoproject.com/en/5.0/ref/models/expressions/#window-functions for partitioning everything in a query, i will take a look at the the next days.

The main problem with the current approach was that the python loop in filter_urls performed a database query for every single of those ~100k urls, which was way too slow. Moving everything into a single query causes the page to load for me with production data. Also not calling prefetch is not required when loading all links, and removing that also gave a nice performance boost.

As succeeded by #2971


Add TAG proposals

@zhaoyue-zephyrus
Please list the THUMOS performance: proposal recall and mAP here for reference.


Improve NeomakeInfo

Similar to SyntasticInfo, it would be great to add a function to Neomake to check which makers are active for the current buffer (and other information as well, maybe which preconfigured makers could be found on the user's PATH, etc.).

We have NeomakeInfo by now.
Does it work for you?
Feel free to suggest enhancements to it.

which preconfigured makers could be found on the user's PATH

Can you elaborate? Does Syntastic have that?

@blueyed I'm no longer using neomake, I've switched to Spacemacs. Best of luck!

Thanks for reporting back.
Spacemacs is tempting.. ;)


Moving test_util.py to pytest

Description
Refactored test_util.py from unittest style test to pytest.
Part of #4156

I added the class structure back.


Display.checkEvents(): don't keep processing events on QUIT

On my test setup, continuing to process events when the QUIT event is
received results in the following output on stderr:
ERROR:
Traceback (most recent call last):
  File "/Users/keunwoo/Documents/cvplay/ENV/lib/python2.7/site-packages/SimpleCV/examples/detection/barcode_reader.py", line 29, in <module>
    while display.isNotDone():
  File "/Users/keunwoo/Documents/cvplay/ENV/lib/python2.7/site-packages/SimpleCV/Display.py", line 689, in isNotDone
    return not self.isDone()
  File "/Users/keunwoo/Documents/cvplay/ENV/lib/python2.7/site-packages/SimpleCV/Display.py", line 667, in isDone
    self.checkEvents()
  File "/Users/keunwoo/Documents/cvplay/ENV/lib/python2.7/site-packages/SimpleCV/Display.py", line 630, in checkEvents
    self.pressed = pg.key.get_pressed()
error: video system not initialized
Cleaned up camera.

The error message appears to be benign, but it would be cleaner not to
print it; at a minimum, it is confusing/alarming for new users.
This commit is based on the answer here:
http://help.simplecv.org/question/1505/error-with-example-from-page-25/
I literally just unpacked and ran simplecv for the first time a few
minutes ago, so I don't know if this is a correct fix, but it does get
rid of the error message.  (Naively, it seems like you shouldn't be
processing additional events after quit, but I'm not familiar with the
pygame framework.)
If accepted, this fix should probably also be backported to the develop
branch.

nobody cares about this; abandoning


Mmc

What is the intent with this PR?

Please re-submit in multiple PRs for individual subsystems and changes. Your PR is changing default system behaviour and cannot be merged this way.

Sorry . I just learning about development  base on PIX4 Flight Stack . Hope someday I can distribute to it and development with you . I need more time to learn about Git .

No worries, please keep the pull requests coming! I saw that you did a new driver - do you maybe want to send only the driver as a pull request?


Side effects in useMemo

First I just want to say great work with this!
I was reading through the code and noticed that you're calling  world.createRigidBody inside useMemo which breaks some React rules since world.createRigidBody has side effects.  useMemo isn't the best choice for this anyway because React can decide to throw away the value at any time and recompute it.
https://github.com/pmndrs/react-three-rapier/blob/a4b9f3fb2ca40abea8f5dcd6d4c44068f60fe398/packages/react-three-rapier/src/hooks.ts#L94-L116
In my own integration I ended up using a getter which as far as I can tell should be React 18+ safe. The approach is described in the bottom of https://github.com/reactwg/react-18/discussions/18
https://github.com/alexandernanberg/playground/blob/973c512aa192a6500a503a15aea6d6dd1b83d2be/src/components/physics.tsx#L265-L272

Thanks for this! Indeed I will have to move everything into effects. It's just not nice working with current all the time, and I was hoping to avoid it wherever I could, but I was not aware of these drawbacks.
Your adaption is really nice. Would you like to help flesh out r3f/rapier?
🇸🇪👋

@wiledal : To make the DX nicer, what about proxying the rigid body API calls?
Before:

:x: Uses useMemo
✅  Nice DX without .current

// Package
export const useRigidBody = <O extends Object3D>(
  // ...
  const rigidBody = useMemo(() => {
    // ...
    const body = world.createRigidBody(rigidBodyDesc);
    return body;
  }, []);
  // ...
  return [ref as MutableRefObject<O>, rigidBody];
};

After:

✅  Uses useEffect
✅  Nice DX without .current

export const useRigidBody = <O extends Object3D>(
  // ...
  const rigidBodyRef = useRef();
  useEffect(() => {
    // ...
    const body = world.createRigidBody(rigidBodyDesc);
    rigidBodyRef.current.body;
  }, []);
  // ...
  const api = createBodyApi(rigidBodyRef);
  return [ref as MutableRefObject<O>, api];
};

function createBodyApi(bodyRef) {
  return {
    applyImpulse: (...args) => {
      bodyRef.current.applyImpulse(...args);
    },
    applyTorqueImpulse: (...args) => {
      bodyRef.current.applyImpulse(...args);
    },
    // ...
  };
}

Would allow the same public API.
For example:
// Usage
  const [box, rigidBody] = useCuboid(
    {
      position: [-4 + Math.random() * 8, 10, 0],
    },
    {
      args: [0.5, 0.5, 0.5],
    }
  );

  useEffect(() => {
    rigidBody.applyImpulse({ x: 0, y: 0, z: 0 }, true);
    rigidBody.applyTorqueImpulse({ x: 0, y: 0, z: Math.random() * 0.1 }, true);
  }, []);

Would be fun, I'm a bit short on time right now but if I get some spare time later I'll for sure see where I can contribute!

No worries! I might lift some approaches from your repo, as you have it so neatly laid out -- with your permission of course 🙏

function createBodyApi(bodyRef) {

I was thinking of doing this, but the extra work put me off for now. But, I will probably do it anyways because interacting with the api is annoying right now as you have to convert between Rapier's Vector3, Three's Vector3 back and forth for all interactions.

you can't pass down the rigidbody in context to the child

Hm, supposedly we could initialize the rigidbody in useLayoutEffect to get around that...

with your permission of course

Sure, feel free to take anything that is useful 😄

Hm, supposedly we could initialize the rigidbody in useLayoutEffect to get around that...

I think I had to create both the rigidbody and collider in a useLayoutEffect to avoid "layout shift" (not sure what it's called in a 3d context), was some time ago so don't remember exactly why I had to do it that way

Sure, feel free to take anything that is useful 😄
✨

I've been doing some experiments, and from what I can tell it works okay initializing in a useLayoutEffect and returning a proxy api.
I have one strange useLayoutEffect cleanup function that fires prematurely for some reason, but probably unrelated.
I feel there's big win to only have to deal with Three vector types, so I'll probably continue on this path.

@alexandernanberg you are right, useLayoutEffect did not work as we also needed the world to be initiated the same.
After more fiddling I believe a combination of these approaches is hitting the goldilocks sweetspot ✨

Use the RefGetter (MutableRefObject<() => T>) approach for "lazy" (or rather "eager"!) initiation of objects (world, rigidbody, collider, joint), as mentioned in the original post.
In a memoized proxy, use the RefGetter for all api forwarding

const createSomeApi = (refGetter) => ({
  someFunc() => refGetter.current()!.someFunc()
})

const useSomething = () => {
  const ref = useRef()
  const refGetter = useRef(() => {
    if (!ref.current) {
      ref.current = new Something()
    }
    return ref.current
  })

  const api = useMemo(() => createSomeApi(refGetter), [])

  return api
}

The result is an immediately accessible api, with eagerly initiated objects.
const [ref, api] = useRigidBody(...)

useEffect(() => {
  api.applyImpulse(new Vector3(0,0,1))
}, [])

  useFrame(() => {
    refs.current.forEach((ref) => {
      const p = ref.current.translation();
      // since p is proxied to be a Vector3, we can easily apply all Threejs math on it
      p.normalize().multiplyScalar(-0.02);
      ref.current.applyImpulse(p);
    });
  });

https://github.com/pmndrs/react-three-rapier/pull/14

I'm still not convinced useMemo is the right choice here, since React can throw away the value at any time. A lazy state setter might be better (useState(() => createSomeApi(refGetter))), but technically it might still violate the React rules since you're setting/getting the ref value in render (React 18 compatibility is hard 😅)


hide impl classes in Navigate > Class

@bjansen would it be possible to eliminate the e.g. Sequence$impl classes in the result list?

I can hide them, but then they won't be visible from Java files either. I think they might be useful when we try to implement a Ceylon interface in Java. I'll have to dig deeper.

I think they might be useful when we try to implement a Ceylon interface in Java.

Well, perhaps, but it's probably a bad idea to do this, especially since the $impl classes are going away, and for sure we don't do this anywhere right now, except in the language module which is a special case.

I'll have to dig deeper.

That's fine, but in the meantime, I still think we should just hide them.

If we hide them, it means they're not in the index, which means the model loader won't be available to find them, which means List$impl won't be found, which means every piece of code that uses iterables will have (invalid) red highlights :(

WDYM "code that uses iterable"? I don't understand.

A compromise solution might be to add a Weigher so that the fall to the bottom of the list.

for sure we don't do this anywhere right now

Just found a place where we do this.

well, this code is temporary and will probably change. But it's right that Java code that implements a Ceylon interface with some implementation methods has to manage $impl elements explicitly.

Just found a place where we do this.

Then I suggest that we fix that code.

But it's right that Java code that implements a Ceylon interface with some implementation methods has to manage $impl elements explicitly.

No. It's not right. $impls are going away, and any code which uses them directly will break.

Moving to 1.3.5 because it's more an inconvenience than a critical bug.


Message priority

Hello
Ask me please, how I can use message priority added in RabbitMq 3.5 ???

Install composer req enqueue/amqp-bunny and do:
<?php

use Enqueue\AmqpBunny\AmqpConnectionFactory;

$factory = new AmqpConnectionFactory('amqp://guest:guest@localhost:5672/%2F');
$context = $factory->createContext();

$queue = $context->createQueue('a_queue_name');
$queue->setArguments(['x-max-priority' => 4]);
$context->declareQueue($queue);

$message = $context->createMessage('a body');

$producer = $context->createProducer();

$producer
    ->setPriority(3)
    ->send($queue, $message)
;

Hello,
you must specify x-max-priority argument when declaring the priority queue:
use Bunny\Client;

$client = new Client();
$client->connect();
$channel = $client->channel();

$channel->queueDeclare("my_queue", false, true, false, false, false, ["x-max-priority" => 4]);

After that, you can publish messages with property priority:
$channel->publish("message", ["priority" => 3]);

thank you, jakubkulhan and you makasim


implement ReadLine functionality

Implement ReadLine functionality in a a way that works well with REPL sessions and separates history for prompts vs the REPL session.  PoC: #311
This will live in the CommandDotNet.Console package

PoC @ #311

closing as won't fix.  can reopen later if someone wants to take it on.


US National Mode should no longer use parenthesis

I'm hoping the community will comment heavily on this. According to E.123, http://en.wikipedia.org/wiki/E.123#Example_formats , parentheses are used to indicate "digits that are sometimes not dialed". As all of the US is now on 10 or 11 digit dialing, should area codes in the US still have parentheses?

I'll close this as it's not an issue with the library. Good question though to bring up on
the discussion
group?


Master

@CodiumAI-Agent /improve

PR Code Suggestions ✨
No code suggestions found for the PR.

@CodiumAI-Agent /add-doc

@CodiumAI-Agent /describe

Title
Master

PR Type
enhancement, configuration changes

Description

Introduced a configuration management system using Viper, allowing configuration through files and environment variables.
Implemented CUE file generation with options to fetch disc information from MusicBrainz and GNUDB.
Added functionality to fetch cover art and handle concurrent disc information retrieval.
Defined data structures for handling disc and release information.
Included utility functions for generating TOC and managing file paths for caching.
Updated module dependencies and checksums in go.mod and go.sum.

Changes walkthrough 📝
Relevant filesConfiguration changes1 files

config.goAdd configuration management using Viper

config/config.go
Introduced a new configuration package using Viper.  Set default values for configuration parameters.  Added support for loading configuration from files and environment variables.  Implemented error handling for configuration loading.

+62/-0

Enhancement8 files

cue.goImplement CUE file generation and disc info retrieval

cue/cue.go
Implemented CUE file generation functions.  Added functions to fetch disc information using MusicBrainz and GNUDB.  Included error handling for disc information retrieval.  Supported overwriting existing CUE files.

+132/-0

discinfo.goAdd cover art fetching and concurrent disc info retrieval

cue/discinfo.go
Added functions to fetch cover art if needed.  Implemented concurrent fetching of disc information from GNUDB and MusicBrainz.  Prioritized disc information source based on availability.

+104/-0

gnudb.goImplement GNUDB querying and metadata extraction

gnudb/gnudb.go
Implemented functions to query and fetch disc information from GNUDB.  Added error handling for GNUDB queries.  Parsed GNUDB response to extract disc metadata.

+138/-0

main.goImplement main function for CUE generation

main.go
Implemented main function to parse flags and generate CUE files.  Added command-line options for MusicBrainz ID and disc ID.

+28/-0

musicbrainz.goFetch and convert MusicBrainz release data

musicbrainz/musicbrainz.go
Implemented functions to fetch release information from MusicBrainz.  Converted MusicBrainz release data to internal disc information format.

+67/-0

types.goDefine data structures for disc and release info

types/types.go
Defined data structures for disc information and MusicBrainz releases.

+25/-0

discutils.goAdd utility functions for TOC and disc ID

utils/discutils.go
Added utility functions to generate TOC and disc ID.  Implemented functions to retrieve MusicBrainz TOC.

+50/-0

fileutils.goImplement file utilities for caching

utils/fileutils.go
Implemented file utility functions for playlist and cover art caching.  Added functions to check and create necessary directories.

+35/-0

Dependencies2 files

go.modDefine module and add dependencies

go.mod
Added module definition for the project.  Included dependencies for Viper and other libraries.

+30/-0

go.sumAdd dependency checksums

go.sum

Added checksums for module dependencies.

+66/-0

💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information


az connectedk8s proxy: ModuleNotFoundError: No module named 'azure.cli.core.auth.msal_authentication' on 2.65.0

Describe the bug
I have a GitHub workflow that runs az connectedk8s proxy as part of our Kubernetes deployment. The exact same workflow that was working ~14 hours ago that used az cli version 2.64.0 is now failing on version 2.65.0 with the error ModuleNotFoundError: No module named 'azure.cli.core.auth.msal_authentication'
Related command
az connectedk8s proxy -n ${{ inputs.arc_cluster }} -g ${{ inputs.arc_rg }} --kube-context ${{ inputs.arc_cluster }}
Errors
ERROR: The command failed with an unexpected error. Here is the traceback:
ERROR: No module named 'azure.cli.core.auth.msal_authentication'
Traceback (most recent call last):
  File "/opt/az/lib/python3.11/site-packages/knack/cli.py", line 233, in invoke
    cmd_result = self.invocation.execute(args)
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/az/lib/python3.11/site-packages/azure/cli/core/commands/__init__.py", line 564, in execute
    self.commands_loader.load_arguments(command)
  File "/opt/az/lib/python3.11/site-packages/azure/cli/core/__init__.py", line 526, in load_arguments
    self.command_table[command].load_arguments()  # this loads the arguments via reflection
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/az/lib/python3.11/site-packages/azure/cli/core/commands/__init__.py", line 321, in load_arguments
    super(AzCliCommand, self).load_arguments()
  File "/opt/az/lib/python3.11/site-packages/knack/commands.py", line 104, in load_arguments
    cmd_args = self.arguments_loader()
               ^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/az/lib/python3.11/site-packages/azure/cli/core/commands/command_operation.py", line 125, in arguments_loader
    op = self.get_op_handler(self.op_path)
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/az/lib/python3.11/site-packages/azure/cli/core/commands/command_operation.py", line 59, in get_op_handler
    handler = import_module(mod_to_import)
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/az/lib/python3.11/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<frozen importlib._bootstrap>", line 1204, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1176, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1147, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 690, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 940, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/opt/az/azcliextensions/connectedk8s/azext_connectedk8s/custom.py", line 47, in <module>
    import azext_connectedk8s._clientproxyutils as clientproxyutils
  File "/opt/az/azcliextensions/connectedk8s/azext_connectedk8s/_clientproxyutils.py", line 21, in <module>
    from azure.cli.core.auth.msal_authentication import ServicePrincipalCredential
ModuleNotFoundError: No module named 'azure.cli.core.auth.msal_authentication'
To check existing issues, please visit: https://github.com/Azure/azure-cli/issues

Issue script & Debug output
.
Expected behavior
The command works as expected and sets up a proxy
Environment Summary
azure-cli                         2.65.0
core                              2.65.0
telemetry                          1.1.0
Extensions:
azure-devops                       1.0.1
connectedk8s                       1.9.2
Dependencies:
msal                              1.31.0
azure-mgmt-resource               23.1.1
Python location '/opt/az/bin/python3'
Extensions directory '/opt/az/azcliextensions'
Python (Linux) 3.11.8 (main, Sep 25 2024, 11:33:44) [GCC 11.4.0]
Legal docs and information: aka.ms/AzureCliLegal
Additional context
No response

Thank you for opening this issue, we will look into it.

Also facing this issue, but with "az connectedk8s connect". Running just installed az cli (2.65.0).

Same also getting this issue

Same issue. Was working with 2.64.0 and now breaking with 2.65.0

Same issue here with deploying AIO with Codespaces

Also getting this issue deploying in vscode

Same issue here with command    az connectedk8s connect in AIO deployment

Please upgrade the connectedk8s extension cli to the most recent version(v1.9.3) which contains the fix for this issue.

my connectedk8s version is 1.10.0, but still got that issue, is it only working on v1.9.3

Please upgrade the connectedk8s extension cli to the most recent version(v1.9.3) which contains the fix for this issue.
Cmd to upgrade: az extension update --name connectedk8s

@wuqiten, the fix was applied only to the publicly available CLI version. The latest publicly available connectedk8s CLI version is v1.9.3.
I have now patched version v1.10.0 by merging the fix from the upstream branch. Could you please download the new whl file for v1.10.0 and try again?

Issue fixed thanks!

@wuqiten, the fix was applied only to the publicly available CLI version. The latest publicly available connectedk8s CLI version is v1.9.3.
I have now patched version v1.10.0 by merging the fix from the upstream branch. Could you please download the new whl file for v1.10.0 and try again?


ADDED REACT GROUP ALBANIA

Updated the document and added the link to the React.js User group meetup page for Albania.

@olsiseferi thanks!

@lex111 It's a pleasure :)


fix loading of yaml templates with CFN tags

Ref!/Sub!/etc will no longer fail to load

The test is failing on error: botocore 1.8.44 is installed but botocore==1.8.45 is required by set(['awscli']), but that new explicit version dependency doesn't conflict with stacker's botocore>=1.6.0

Another option here would be to use cfn_flip, which is already pulled in by troposphere.
I didn't like its helper as much though because it's just a yaml loader which ends up making an object with a bunch of OrderedDicts . Not sure if adding some translation on top of that (instead of just using the awscli helper) is better.

I was thinking that as well @troyready - I'm honestly more hesitant with what you have here just because of the reliance on awscli - which I think folks might conflict with quite a bit.

Codecov Report

Merging #538 into master will increase coverage by <.01%.
The diff coverage is 81.25%.

@@            Coverage Diff             @@
##           master     #538      +/-   ##
==========================================
+ Coverage   87.54%   87.55%   +<.01%
==========================================
  Files          94       95       +1
  Lines        6072     6098      +26
==========================================
+ Hits         5316     5339      +23
- Misses        756      759       +3

Impacted Files
Coverage Δ

stacker/blueprints/raw.py
66.12% <100%> (ø)
:arrow_up:

stacker/actions/diff.py
57.03% <25%> (ø)
:arrow_up:

stacker/awscli_yamlhelper.py
88.46% <88.46%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update af5916a...25cbce6. Read the comment docs.

I definitely like the idea of using cfn_flip more, but I'm not sure how to implement it in a way that won't be kinda gross.
Accordingly, I just did the vendoring.

Added the wrapper function


Ensure language of every question/piece of copy is as it should be

Page id: cosigner court contact info
Was: Make sure you have {cod}’s permission to put this on the form.
Change to: Contact information you list will go on the Motion to Dismiss. Ask {cod} for permission to give this contact information to the court.
Page id: how to send to codefendant
First 2 bullets--> can we replace "they" with the codefendants name? like in the 3-5th bullets? and replace "draw their signature" with  "sign"
In the last bullet--> Add a"." after interview and delete "and"
Page id: x.which_device
Current: Do you want to sign on this computer or on a phone or tablet instead?
I am guessing this is a basic question, but I don't think we need the word instead.  It's just, do you want to sign on a computer or phone or tablet.  I also think many people have never signed on a screen so we want a line like,
Question: |
Where do you want to sign?
Subquestion:|
You can draw your signature on your screen with:

your finger if you have a touchscreen or
-your mouse.

Then displayed are the choices that say "Computer" and "Phone and tablet" (So I don't think we have to say the words computer or phone in the question as they are in the icons.)
Andddd Final screen. I am creating an issue just so you can show me if there are other options #107 and will need to review after #17 implemented.

We are waiting for penultimate Master once all current MVP PRs are merged to run through to see how flow works


Disable jacoco report gen

Purpose

Describe the problems, issues, or needs driving this feature/fix and include links to related issues.

Fixes #
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, releated PRs, TODO items or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@d3d355f). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master   #15647   +/-   ##
=========================================
  Coverage          ?   68.85%
=========================================
  Files             ?        5
  Lines             ?      122
  Branches          ?       21
=========================================
  Hits              ?       84
  Misses            ?       38
  Partials          ?        0

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d3d355f...f514560. Read the comment docs.


chore(deps): update dependency org.jasig.portlet.courses:courses-portlet-dao to v1.0.1

This PR contains the following updates:

Package
Update
Change

org.jasig.portlet.courses:courses-portlet-dao
patch
1.0.0-M3 -> 1.0.1

Renovate configuration
:date: Schedule: At any time (no schedule defined).
:vertical_traffic_light: Automerge: Disabled by config. Please merge this manually once you are satisfied.
:recycle: Rebasing: Whenever PR is stale, or if you modify the PR title to begin with "rebase!".
:no_bell: Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

Available now for Enterprise: Renovate Pro with real-time webhook handling and priority job queue.

PR has been edited
:construction_worker: This PR has received other commits, so Renovate will stop updating it to avoid conflicts or other problems. If you wish to abandon your changes and have Renovate start over you may click the "rebase" checkbox in the PR body/description.


New feature : New events can be added to EventRelay via the new optio…

This pull request includes a

[ ] Bug fix
[x] New feature
[ ] Translation

The following changes were made

new Options: relayablePluginsEvents [String]
EventRelay : relayEvents is now extendable with relayablePluginsEvents [String]

Why ?
It give the possibility to add relayable custom plugins events without manually overriding EventRelay.
Related to
#4446
Example
Example of usage with a plugin that trigger an event when the result search didn't found anything.
// Can be improved by adding a method 'add' to Defaults.prototype.
$.fn.select2.defaults.set('relayablePluginsEvents', $.fn.select2.defaults.defaults.relayablePluginsEvents.concat('plugin:noresult'));

$.fn.select2.amd.require([
    'select2/results',
  ], function(Results) {
    Results.prototype._bind = Results.prototype.bind;
    Results.prototype.bind = function(container, $container) {
      var self = this;
      this._bind.apply(this, arguments);
      // After results:select
      container.listeners['results:select'][1] = function(e) {
        var $highlighted = self.getHighlightedResults();

        if ($highlighted.length === 0) {
          self.trigger('plugin:noresult', {
            originalEvent: e,
            data: {}
          });
        }
      };
    };
  });

 var select2 = $(".js-example-basic-single").select2();
 select2.on('select2:plugin:noresult', function() {
  console.log('no result found');
 });

My end goal is to actually rewrite how this is handled, so plugins for Select2 can define additional events which should be relayed.


crosslink rewards typo

One more layer to unpack to loop around all the index inside state.shard_committees_at_slots[:EPOCH_LENGTH]

Thanks. The good news is that shard_committees_at_slots (as well as persistent_committee_reassignments, validator_registry_delta_chain_tip) will likely be removed from state.


Tasks are missing NPM dependencies

Some of the tasks are missing the necessary dependencies for NPM

Cookbook Version
Environment Cookbook Version constraint
VSTS Environment variables

This has been fixed in version 1.2.63


extract submitted date to support slack reader reqs

Hi, thanks for hosting this feed.
As in #3 I was also failing to sub to this in Slack and trying to get it to work. (I thought https://github.com/snarfed/granary/pull/847 would solve it but I was wrong)
My next hunch for what might help comes from https://github.com/GoogleChrome/developer.chrome.com/issues/2808 --

extracting and including item publish dates when possible
sorting items by publish date decreasing

Hopefully this will make it subbable in Slack 🤞
Thanks again!

Thank you for your contribution, @isnbh0! I will merge and deploy this now.

This change is now deployed at https://jamesg.blog/hf-papers.json. Let me know if things work as you expect!

It worked! The Slack RSS reader was finally able to successfully fetch items 🎉 Thanks again!!

Yay! I'm glad everything works!


got below error restarting comfyui (import failed)

Traceback (most recent call last):
File "/opt/conda/lib/python3.10/site-packages/transformers/utils/import_utils.py", line 1535, in _get_module
return importlib.import_module("." + module_name, self.name)
File "/opt/conda/lib/python3.10/importlib/init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1050, in _gcd_import
File "", line 1027, in _find_and_load
File "", line 1006, in _find_and_load_unlocked
File "", line 688, in _load_unlocked
File "", line 883, in exec_module
File "", line 241, in _call_with_frames_removed
File "/opt/conda/lib/python3.10/site-packages/transformers/models/auto/processing_auto.py", line 27, in
from ...image_processing_utils import ImageProcessingMixin
File "/opt/conda/lib/python3.10/site-packages/transformers/image_processing_utils.py", line 28, in
from .image_transforms import center_crop, normalize, rescale
File "/opt/conda/lib/python3.10/site-packages/transformers/image_transforms.py", line 47, in
import tensorflow as tf
File "/opt/conda/lib/python3.10/site-packages/tensorflow/init.py", line 38, in
from tensorflow.python.tools import module_util as _module_util
File "/opt/conda/lib/python3.10/site-packages/tensorflow/python/init.py", line 42, in
from tensorflow.python.saved_model import saved_model
File "/opt/conda/lib/python3.10/site-packages/tensorflow/python/saved_model/saved_model.py", line 20, in
from tensorflow.python.saved_model import builder
File "/opt/conda/lib/python3.10/site-packages/tensorflow/python/saved_model/builder.py", line 23, in
from tensorflow.python.saved_model.builder_impl import _SavedModelBuilder
File "/opt/conda/lib/python3.10/site-packages/tensorflow/python/saved_model/builder_impl.py", line 26, in
from tensorflow.python.framework import dtypes
File "/opt/conda/lib/python3.10/site-packages/tensorflow/python/framework/dtypes.py", line 39, in
_np_float8_e5m2 = pywrap_ml_dtypes.float8_e5m2()
TypeError: Unable to convert function return value to a Python type! The signature was
() -> handle
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/mnt/data/software/comfyui/nodes.py", line 1890, in load_custom_node
module_spec.loader.exec_module(module)
File "", line 883, in exec_module
File "", line 241, in _call_with_frames_removed
File "/mnt/data/software/comfyui/custom_nodes/ComfyUI_LayerStyle/init.py", line 60, in
imported_module = importlib.import_module(".py.{}".format(name), name)
File "/opt/conda/lib/python3.10/importlib/init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1050, in _gcd_import
File "", line 1027, in _find_and_load
File "", line 1006, in _find_and_load_unlocked
File "", line 688, in _load_unlocked
File "", line 883, in exec_module
File "", line 241, in _call_with_frames_removed
File "/mnt/data/software/comfyui/custom_nodes/ComfyUI_LayerStyle/py/Qwen_image2prompt.py", line 3, in
from transformers import AutoModel, AutoProcessor, StoppingCriteria, StoppingCriteriaList
File "", line 1075, in _handle_fromlist
File "/opt/conda/lib/python3.10/site-packages/transformers/utils/import_utils.py", line 1526, in getattr
value = getattr(module, name)
File "/opt/conda/lib/python3.10/site-packages/transformers/utils/import_utils.py", line 1525, in getattr
module = self._get_module(self._class_to_module[name])
File "/opt/conda/lib/python3.10/site-packages/transformers/utils/import_utils.py", line 1537, in _get_module
raise RuntimeError(
RuntimeError: Failed to import transformers.models.auto.processing_auto because of the following error (look up to see its traceback):
Unable to convert function return value to a Python type! The signature was
() -> handle
Cannot import /mnt/data/software/comfyui/custom_nodes/ComfyUI_LayerStyle module for custom nodes: Failed to import transformers.models.auto.processing_auto because of the following error (look up to see its traceback):
Unable to convert function return value to a Python type! The signature was

Try upgrade transformers dependency package

Try upgrade transformers dependency package

Thanks for post, transformers has already been updated to the latest. I doubt if the version is too new?
--
Name: transformers
Version: 4.41.2
Summary: State-of-the-art Machine Learning for JAX, PyTorch and TensorFlow
Home-page: https://github.com/huggingface/transformers
Author: The Hugging Face team (past and future) with the help of all our contributors (https://github.com/huggingface/transformers/graphs/contributors)
Author-email<EMAIL_ADDRESS>License: Apache 2.0 License
Location: /opt/conda/lib/python3.10/site-packages
Requires: filelock, huggingface-hub, numpy, packaging, pyyaml, regex, requests, safetensors, tokenizers, tqdm
Required-by: adaseq, auto-gptq, clip-interrogator, compel, image-reward, ms-swift, optimum, pai-easycv, peft, text2sql-lgesql, transformers-stream-generator, vllm

Thanks for post, transformers has already been updated to the latest. I doubt if the version is too new?

Perhaps the version is too high？ The 4.40.2 I am currently using is working properly.


Suggestions for some modules

Contributing guidelines

[X] I have read CONTRIBUTING.md
[X] I have read CODE_OF_CONDUCT.md

Module(s)
-- To be done --
Description
It would be great if mini.nvim could provide some alternatives for:

[ ] https://github.com/andymass/vim-matchup
[ ] https://github.com/tpope/vim-sleuth
[ ] https://github.com/ziontee113/color-picker.nvim
[ ] https://github.com/simrat39/symbols-outline.nvim

Thanks for the suggestions!
Right now, the priorities of which module to write next are a mix of "I am using it right now" and "There is an interesting design/implementation idea I want to write". None of these fall into any of the categories, so it will be a while.
Right now I'd say that:

'symbols-outline.nvim' will probably not happen, as it is based on LSP and I'd like to stay away from it if possible.
Color picker might be on the horizon, but definitely not close.
Others might happen.

Perfect, yes I totally understand and expected that to be the case.
Just wanted to get them under your radar 😉


Go to definition needs to be pressed twice

Go to definition on an object
Get the message "Definition of xx" could not be found
Go to definition again and it works
This seems to occur at the start of opening a workspace/folder and occurs a short time after.
Using the previous python-vscode version of 2021.2.636928669 makes the issue go away.
Environment data

VS Code version: 1.54.3
Extension version (available under the Extensions sidebar): 2021.3.658691958
Python version (& distribution if applicable, e.g. Anaconda): 2 & 3
Windows build: Windows 10 Pro 1909
Value of the python.languageServer setting: Pylance/JediLSP/Microsoft

Bug does not appear when any one of the following is set:

python-vscode being on version 2021.3.65869195
python.languageServer is set to 'Jedi' (Not 'JediLSP')

Steps to reproduce:

You list all language servers. So are you saying this happens on all three of them?

Yes, I have same issues after upgrade new python extension version.
Happened on Python.

@djplt What is the OS?

@djplt What is the OS?

Mine is MacOS

You list all language servers. So are you saying this happens on all three of them?

So it doesn't happen when the language server is set to 'Jedi', the other three ('Microsoft', 'JediLSP' & 'Pylance') cause the bug to appear.

@djplt What is the OS?

Windows 10 Pro 1909
I've updated the information in my original post.

Found that the commit that caused this is 3288ba40fed754ae08fb85f479000cae03415e2b
I've created a branch that appears to fix this issue (just above) - by removing the setting of the telemetry event names in languageClientMiddleware.ts (I assumed that since telemetry has been disabled for LSP this is no longer needed).

Yep, that's the bug and I probably committed that. The ExP code removal changed the code path and the sendTelemetry change meant we dropped the result. @bschnurr was debugging this from the Pylance side and will submit something for this.

Hi everyone, we released a new version of the extension that includes the fix above, could you give it a try and see if this solves the issues you've been encountering?
Thank you!

Thank you for reporting back! Closing this issue as resolved.


📝 Update includes in docs/de/docs/tutorial/dependencies/classes-as-dependencies.md

Not sure if the tip about annotated version should be removed...

Thanks for the effort! :cake:
This was handled in another PR, so I'll close this one now. :coffee: