Scoped MediatR dependencies not available when switching thread

Hi there!
Currently working on a project that contains a console application. That console app subscribes and listens to a SignalR feed, and whenever a message comes in it is handled by the appropriate event handler.
Whenever the application is started, MediatR and all it's necessary dependencies are registered on the main thread (startup thread). And after that, my SignalR connection is configured adding the necessary event handlers and starting the connection. In the background, SignalR will process incoming messages on seperate threads and let the event handler process the messages.
This is where I get into trouble, because in one of my event handlers I implement IMediator to do some commands and queries. However, I get the following error message:
Cannot access a disposed object. Object name: 'IServiceProvider'.
This is because IMediator uses scoped dependencies which, in the separate thread that has now been created, are not available anymore.
Is there any way to change the lifetime of the dependencies registered by MediatR?
UPDATE
If I inject the command handlers into my event handler directly, in stead of using IMediator, it works like a charm.

Yeah - don't call Mediator from inside a handler, things get weird.

Hey @jbogard, I just ran into this same issue yesterday when calling IMediator from inside a handler. Do you have any alternative suggestions when such need happens?!

Hi,
I ran into that issue also.
Is it ok to publish an event from inside a handler ?

Same issue here.
Do you have any solution, or should i not user mediator at all in this case ?

Hi @jbogard, what should we do instead of calling a mediator from inside a handler? For some reasons I have to keep my mediator with scoped lifetime. There's a mediator that send 4 parallel commands and from inside of some handlers we'll call others mediator commands. So we have the same issues telling that  IServiceProvider is disposed.

Hi,
I ran into that issue also.
Is it ok to publish an event from inside a handler ?

Yes.

Hi @jbogard, what should we do instead of calling a mediator from inside a handler? For some reasons I have to keep my mediator with scoped lifetime. There's a mediator that send 4 parallel commands and from inside of some handlers we'll call others mediator commands. So we have the same issues telling that  IServiceProvider is disposed.

Create your own abstraction here instead. You don't want nested behavior pipelines.


SharePoint IRM versus Sensitivity labels (AIP)

Is SharePoint IRM still updated and therefore recommended by Microsoft?
OR shall we move to sensitivity labels?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: b29df28b-0065-a059-f22d-3433c1dd65a0
Version Independent ID: 83b44475-2a39-5747-fc3c-2a92d0d958f2
Content: Set up Information Rights Management (IRM) in SharePoint admin center - Microsoft 365 Compliance
Content Source: microsoft-365/compliance/set-up-irm-in-sp-admin-center.md
Service: o365-seccomp
GitHub Login: @KCCross
Microsoft Alias: krowley

@teomonda Thank you for your feedback.
While the option to use IRM in Sharepoint admin center is still available, Office 365 sensitivity labels are the preferred way to protect content stored in SharePoint Online and OneDrive for Business. They are more flexible and powerful than the traditional approach of protecting SharePoint libraries with IRM.
Hope this helps!
Thanks
Sri

Thanks for your info, Sri.
Does it make sense that we add a note or something in the article, pointing out to the sensitivity labels?

@teomonda Thank you for your feedback. We have now added documents relating to sensitivity labels to the recommended content section of this document.
Hope this helps!
Thanks
Sri

@yogkumgit This issue can be closed.

@msbemba Thank you very much for the contribution and sharing this explanation. @teomonda Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Update docs to automated libdds build

doc(README): Improve C++ library build documentation
Add build and test instructions for C++ library. These commands are
based on build process initiated by Makefile.
doc(README): Improve dependency installation
Dependencies should be held in one place. The requirements.txt offers a
natural place to maintain all requirements which can be installed using
a pip command.
doc(README.gcp): Remove C++ library instructions
The C++ library instructions became outdated with changes to automate
submodule build. The automation takes care of C++ library build process
which allows the cloud build skip directly to the python part of
instructions.
There is a small addition to use libdds-build target before trying local
server. This makes sure automate C++ library build process will be done
before attempting to use the library.

Thanks! I'll look in a couple of hours.
We should add a make target to start the local server. Then libdds can be a prerequisite, as it is for the test target.


Consortium RPCs

Similar to https://github.com/DeFiCh/jellyfish/issues/48 This tracks all the RPC methods and categories being implemented in jellyfish-api-core. Please tag update this if you want to work on a feature.
What would you like to be added:
...
Why is this needed:

/triage accepted
/kind feature
/area packages
/priority important-soon

/assign @infinia-yzl to manage and assign to BR_Hydra, closely related to upcoming work.
/priority urgent-now
For reference, check out https://github.com/JellyfishSDK/jellyfish/issues/1773


[BUG] Error thrown on the overview page when top queries are not enabled

What is the bug?
When top queries are not enabled, Observed an error is thrown on the overview page.

TopNQueries.tsx:170 Failed to retrieve settings: TypeError: Cannot read properties of undefined (reading 'insights')

How can one reproduce the bug?
See above
What is the expected behavior?
No errors should be thrown if the feature is not enabled.
What is your host/environment?
MacOS
Do you have any screenshots?
See above
Do you have any additional context?
Add any other context about the problem.

We need to fix this in the MVP release.


CAN't connect to my account

Issue Type: Feature Request
I want to connect my visual studio code to my azure account but its not working.pls help
VS Code version: Code 1.46.0 (a5d1cc28bb5da32ec67e86cc50f84c67cc690321, 2020-06-10T09:03:20.462Z)
OS version: Windows_NT x64 10.0.18363

I want to connect my visual studio code to my azure account

What are you trying to achieve?
In 1.46 the new Accounts icon only comes to life with a few extensions. See https://code.visualstudio.com/updates/v1_46#_accounts-management
It does more in the Insiders release.


Make glc will be able to select any history

https://github.com/kachick/dotfiles/commit/d03a314daa4caa895eca5ff02caca89e06e29883
Getting last is not reasonable if I typed git status, or typo
Using atuin may help these cross-history operations https://github.com/kachick/dotfiles/pull/266

https://github.com/junegunn/fzf/issues/1308#issuecomment-394552841

https://github.com/kachick/dotfiles/pull/493/files#diff-2e6095ac9bebf267e8ed8fddcb830959644cbbf44ca56ec0f34908f102cb7666R187-R193
fc -rl 1


[Question] How to match inline-code in comments?

I am trying to match inline code in comments, e.g. the "foobar" in here:
// the function `foobar` should not be called to do this and that
function foobar () {
}

Following the example in the readme, I tried this:
("text" @markup.raw.markdown_inline
 (#match? @markup.raw.markdown_inline "`.+`"))

Which correctly highlights `foobar` but not `foobar()` or `foo.bar`. I also tried variations like "`[a-zA-Z\.]+`" for the regex, but those also did not work. I assume there are some limitations to the regex syntax, but in the docs I could find, I could not find clues?

Hi, exposing the text nodes from the parser was more like a hack in order to support uppercase only annotations (that don't end with :), issues, PRs, and usernames references (!, #, @). I've been considering stop exposing text nodes and just expose nodes for uppercase tags without :, and maybe nodes for<EMAIL_ADDRESS>If you are trying to highlight Markdown in your comments, what you probably want to do is just inject the Markdowm parser in the comments (instead of or in addition to this parser).
Anyway, if you still want to manually match that, the problem is that `foobar()` produces several text tokens, `foobar, (, ), `. The reason is that those are basically mark a stop word, so things like (TODO: I'm a tag) work.

If you are trying to highlight Markdown in your comments, what you probably want to do is just inject the Markdowm parser in the comments (instead of or in addition to this parser).

Oh, that's a good idea! Is it possible to make "global injections", meaning for every filetype? Like, for instance, I got this one in to inject markdown into lua files, but it feels a bit tedious to add such injections for every single filetype.
; queries/lua/injections.scm
(comment
  (comment_content) @injection.content (#set! injection.language "markdown"))

Also, I kind of wonder whether this might have an performance impact to inject markdown into every comment?

Anyway, if you still want to manually match that, the problem is that foobar() produces several text tokens, foobar, (, ), . The reason is that those are basically mark a stop word, so things like (TODO: I'm a tag) work.

In case there is no straightforward way of making global injection, matching might nevertheless be the better way. So I see the problem with the tokens—but how do I deal with that problem though?

Oh, that's a good idea! Is it possible to make "global injections", meaning for every filetype? Like, for instance, I wrote this to inject markdown into lua files, but it feels a bit tedious to add such injections for every single filetype.

That's the way of doing it. If you are using Neovim, you may be able to override some internal lua function, maybe.

but how do I deal with that problem though?

You need to write several queries for each case. Maybe something like this
(("text"  @_start "text"* @content "text" @_end)
 (#match? @_start "^`")
 (#match? @_end "^`"))


Add redfish /Systems/identifier/Processors API for UCS

Background
To enable redfish API data access with UCS.
Details

Implemented details to identify Dell/Cisco and other hardware platforms
Implement /Systems/identifier/Processors API for UCS

Paired with: @bbcyyb
@anhou @iceiilin @yaolingling @mcgG @larry-dean

test this please


Are webhooks no longer calling back in southcentralus?

I originally thought I was getting the stuck "NotStarted" issue again, but it looks like they are completing, but not calling the webhook back.
The last call back I got was
12/30/2020, 4:27:37.027 AM
Any insight on why a webhook would stop calling back?

Could you please let me know the identity of your webhook, or the name of your subscription? This would be me to get your service logs and look at the issue.

What you can do yourself is to check the state of your webhook by doing a GET on it.
The service sets it to failed if there were too many failures while calling into your callback. If it's in state "Failed" you can simply update the url to the same value to try enabling it again (with a PATCH).

Thank you - here is the info:
We have two, but the key one:
https://southcentralus.api.cognitive.microsoft.com/speechtotext/v3.0/webhooks/80f96f25-3087-469d-990a-b18e268ea895
Details->
"displayName": "Transcription web hook",
"self": "https://southcentralus.api.cognitive.microsoft.com/speechtotext/v3.0/webhooks/80f96f25-3087-469d-990a-b18e268ea895",
"links": {
"ping": "https://southcentralus.api.cognitive.microsoft.com/speechtotext/v3.0/webhooks/80f96f25-3087-469d-990a-b18e268ea895/ping",
"test": "https://southcentralus.api.cognitive.microsoft.com/speechtotext/v3.0/webhooks/80f96f25-3087-469d-990a-b18e268ea895/test"
"displayName": "Transcription Grader web hook",
"self": "https://southcentralus.api.cognitive.microsoft.com/speechtotext/v3.0/webhooks/47481178-9122-4794-b94a-c3f58cd44716",
"links": {
"ping": "https://southcentralus.api.cognitive.microsoft.com/speechtotext/v3.0/webhooks/47481178-9122-4794-b94a-c3f58cd44716/ping",
"test": "https://southcentralus.api.cognitive.microsoft.com/speechtotext/v3.0/webhooks/47481178-9122-4794-b94a-c3f58cd44716/test"

Here is a result of a Get - status appears to be "Succeeded"
{
"displayName": "Transcription web hook",
"self": "https://southcentralus.api.cognitive.microsoft.com/speechtotext/v3.0/webhooks/80f96f25-3087-469d-990a-b18e268ea895",
"links": {
"ping": "https://southcentralus.api.cognitive.microsoft.com/speechtotext/v3.0/webhooks/80f96f25-3087-469d-990a-b18e268ea895/ping",
"test": "https://southcentralus.api.cognitive.microsoft.com/speechtotext/v3.0/webhooks/80f96f25-3087-469d-990a-b18e268ea895/test"
},
"properties": {
"apiVersion": "v3.0"
},
"webUrl": "https://func-snasr-prod-001.azurewebsites.net/api/callback",
"events": {
"transcriptionCreation": true,
"transcriptionProcessing": true,
"transcriptionCompletion": true
},
"createdDateTime": "2020-11-05T21:08:32Z",
"lastActionDateTime": "2020-11-05T21:08:33Z",
"status": "Succeeded"
}

The queue was processing unusually slow. We restarted it and you should see the notifications appear until it catches up.

Awesome!  I have logging on my end to catch this sooner now.
If this should appear again, is raising an issue here the best way to address?

The system has fully catched up processing should be fast again.
Yes, filing an issue in GitHub is a good way to ask for support.

We also tweaked the thresholds for our internal monitoring to catch this kind of issue faster (threshold was too relaxed)

Thank you both

@bexxx or @rarera any chance the queues are running behind today (southcentralus)?  We're starting to see items waiting for over 1 hour - just curious.  Not sure the best way to ask about this

Hi @kleindev, yes we are looking at an issue with higher latencies than usual. We are sorry for the inconvenience.
You can report issues here or submit a support request in the Azure Portal.

@chlandsi - sorry to bother, but it seems as though there still may be latencies?

Hi @kleindev, yes, unfortunately we are still seeing this issue. We hope we'll have a more permanent situation in the next few days. If you have the possibility to switch your traffic to a different region, this will very likely improve the situation; US South Central is more affected than other regions.

Which region is least impacted?  I would need to move/retrain all data - not necessarily a quick task, but possible

Hi @kleindev, all other (US and non-US) regions should be better. We do, however, expect to improve the situation also in USSC next week.

Ok our audio traffic tapers off after Friday 6 pm central - so fingers crossed we can hobble through the rest of today and tomorrow.  I appreciate the transparency and help

seems like it is in a better state today (so far) - just curious if the fix/improvement you had in mind was able to be put in place already?

Hi @kleindev, sorry for the delayed response - yes, we deployed a first batch of fixes on 03/22 around 9:0 UTC, and that has improved delays according to our metrics. A second batch of improvements is currently being rolled out, which will reach US South Central end of this or beginning of next week. Please reach out if you continue to see problems!

Looks like I have about 800+ jobs stuck last 24 hours...
I need to find a way to work around this issue

i see a lot of webhook events in the queue, it is being cleaned up slowly.

I don't see any new requests..  we are pulling people in to manually type

It is only the webhooks which are showing delays, transcriptions are being processed. Please try polling the API as a fallback.

Meaning the webhook on my azure subscription is not responding ?

We would need the webhook id to check on that particular webhook, but I don't think this was the problem. We have made a change so that queued up webhook invocations are processed quickly now. Still, it would be good if your application would fall back on polling the API if there is no response on the webhook within the expected timeframe.

it will take approx an hour for the queue to clean up

Ok I'll check over the next hour, I don't see them currently.   We'll work on adding something to poll the API as well

Do there happen to be any queue or web hook issues in south central ?

Web hooks function as expected in south central. The last request to your web hook (80f96f25-3087-469d-990a-b18e268ea895) was on 04-07-2021 at 05:21 UTC. Do you trigger any job after that time?

Perfect - that helps me troubleshoot the issue - thank you

We are seeing a few items that don't appear to have been called back since ~9 am central.  We are working on changes to poll the service instead, but that is not yet in production -  Curious if there is any queue backlog

Good morning - just wanted to do a quick health check - last call back I see is 4/20/2021, 10:25:16.173 PM (cst) - do thinks look clear on the southcentral side?

Hi kleindev, on our side everything looks fine, queues are emptied ...
it looks like we get ServiceUnavailable when we try to send the webhook requests.

@rarera did it say where service unavailable is coming from?  Here is what I show

I'm confused where the service unavailable comes from - Azure functions here have no trace of 503

I've pinged and tested from and they were both accepted.  Is there something else I need to do to reset this?  Again I don't see any trace of service unavailable

I've deleted the webhooks, re-added, the web hooks now have a status of success.  I submitted a few and it was working - but now I believe it has stopped again (last call back over 1 hour ago).

Hi kleindev, our queues are empty in southcentral us, so from that perspective it looks good.
I am now checking if I find again that we received Service Unavailable

Hi again, I went again through our logs and dashboards. Our service looks healthy, all errors I find are about timeout (no answer from the remote server). In some cases I see also service availability problems on the remote server. If restart on your side helped, that might be an indicator that you need to look into the health of your side as well. You might want to look into your Azure function, and check the usage, see if it is pegged.
Most of the problems I see with remote sites is when the remote server does not answer, and we get a timeout.


go: why enterprise chose java

What's this PR do?

[x] Add an article about why enterprise chose java

Looks good, imo, It would be more concrete if we put a comparison with prevailing languages like C/C++/C#/COBOL... e.g: some comparisons between Java and C++:

https://qr.ae/p2CQ8L
https://qr.ae/p2CQyN
It could be any other languages besides C++, wdyt @thangnt294

make sense, I'll update @fuatto


Range selection core data

(Required) Version Number:
Description
Steps To Reproduce
Expected Behavior
Additional Context

Hello my problem is that I am not so experienced on swift, already made work the JTAppleCalendar on my App, currently it saves just one given date on core data. I am deciding to change this to a range, but I am having lots of trouble making it work with a for loop and to save on core data. I wanted to know how to print the whole range of days selected on xcode

calendarView.selectedDates = all the selected dates

Ok I can print the range of days : Found a way of saving each day individually onto the calendar using a for loop as I said before:
for i in 0..<daysArray.count {
let project = NSEntityDescription.insertNewObject(forEntityName: "ProjectCD", into: context)
project.setValue(daysArray[i], forKey: "days")
}
do {
try managedContext.save()
} catch  {
print("Could not save. (error)")
}
I truly believe that I am close but when I decide to edit the project (should have the same info as the one that was saved) it shows just the last date of the range only

Ok I can print the range of days : Found a way of saving each day individually onto the calendar using a for loop as I said before:
for i in 0..<daysArray.count {
let project = NSEntityDescription.insertNewObject(forEntityName: "ProjectCD", into: context)
project.setValue(daysArray[i], forKey: "days")
}
do {
try managedContext.save()
} catch {
print("Could not save. (error)")
}
I truly believe that I am close but when I decide to edit the project (should have the same info as the one that was saved) it shows just the last date of the range only. But how could I make the range of selected days appear when I decide to edit the info of the project? 🤔

But how could I make the range of selected days appear when I decide to edit the info of the project?

unclear what you are asking

Ok, User has to specify some information for the project he is intending to create, one of the fields is to select a range of days on which this user is working on the project.
This app has a swipe action for the new project that enables the user to edit the info, so first step is to fetch data from Core Data and “overwrite” the data once again.
So this edit window should show the range of days that User selected before while creating the peoject.
The problem is that the 📆 after creating the range of days. It shows me the last day of the whole range selected, not the range itself man
Get Outlook for iOShttps://aka.ms/o0ukef

From: JTAppleCalendar<EMAIL_ADDRESS>Sent: Sunday, August 11, 2019 11:34:40 AM
To: patchthecode/JTAppleCalendar<EMAIL_ADDRESS>Cc: Conoize<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [patchthecode/JTAppleCalendar] Range selection core data (#1114)
But how could I make the range of selected days appear when I decide to edit the info of the project?
unclear what you are asking
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/patchthecode/JTAppleCalendar/issues/1114?email_source=notifications&email_token=AGCOKKH6WTISKCH7CQAOC5DQEAWRBA5CNFSM4IKZ7GNKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4BDD4Y#issuecomment-520237555, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGCOKKE5ESR3GHP5IUMCNLTQEAWRBANCNFSM4IKZ7GNA.

...so basically all you are asking is how to select a range of dates when a user taps a button?

Mmm not exactly, all I am asking is that the calendar to show the whole range of days while on the edition window, as I chose on the projects window before
I dont know if that is a core data problem or a calendar related problem, either way I know that should be the correct approach for the user (he sees the same range of days on the project window as well as the edit window).
Note: Im not an expert on ios developement, this is just a bold move of mine...😞
Get Outlook for iOShttps://aka.ms/o0ukef

From: JTAppleCalendar<EMAIL_ADDRESS>Sent: Monday, August 12, 2019 9:45:32 AM
To: patchthecode/JTAppleCalendar<EMAIL_ADDRESS>Cc: Conoize<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [patchthecode/JTAppleCalendar] Range selection core data (#1114)
...so basically all you are asking is how to select a range of dates when a user taps a button?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/patchthecode/JTAppleCalendar/issues/1114?email_source=notifications&email_token=AGCOKKCUHSWIGTAN5EYP2A3QEFSPZA5CNFSM4IKZ7GNKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4CSKWI#issuecomment-520430937, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGCOKKBKGDWY7PGTU4SEIALQEFSPZANCNFSM4IKZ7GNA.

See, what i am trying to get you to do is break down what exactly you want.

all I am asking is that the calendar to show the whole range of days

What does this mean? Does this mean that you want the calendar to display a number of selected days right? Or does this mean that you want want some label (other than the calendar) to display this? You need to be clear, because right now you are a little vague.

while on the edition window, as I chose on the projects window before

So basically, on the screen, you'll tap some button called "EDIT", and then it navigates to a next screen which should display a range of dates.
Where are the dates saved? In the Database
Where do you want the dates displayed?

The range of dates are saved onto core data, and I want the dates (all) to be displayed on the edit window , right now I only get to display just one date (last date of the dates selected)
Get Outlook for iOShttps://aka.ms/o0ukef

From: JTAppleCalendar<EMAIL_ADDRESS>Sent: Monday, August 12, 2019 4:17:12 PM
To: patchthecode/JTAppleCalendar<EMAIL_ADDRESS>Cc: Conoize<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [patchthecode/JTAppleCalendar] Range selection core data (#1114)
See, what i am trying to get you to do is break down what exactly you want.
all I am asking is that the calendar to show the whole range of days
What does this mean? Does this mean that you want the calendar to display a number of selected days right? Or does this mean that you want want some label (other than the calendar) to display this? You need to be clear, because right now you are a little vague.
while on the edition window, as I chose on the projects window before
So basically, on the screen, you'll tap some button called "EDIT", and then it navigates to a next screen which should display a range of dates.
Where are the dates saved? In the Database
Where do you want the dates displayed?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/patchthecode/JTAppleCalendar/issues/1114?email_source=notifications&email_token=AGCOKKC73FLRBPQEN7W5HZDQEHAMRA5CNFSM4IKZ7GNKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4DWJXI#issuecomment-520578269, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGCOKKEYFOJWZMLE7KGJ7ITQEHAMRANCNFSM4IKZ7GNA.

Ok. Since you want dates which are saved in core data to be displayed on a window (and not on the calendar itself), then your question relates entirely to core data. I'll have to tell you that i know nothing about core data; it is not my expertise.

Yeah I thought so too, all right thanks for the info!
Get Outlook for iOShttps://aka.ms/o0ukef

From: JTAppleCalendar<EMAIL_ADDRESS>Sent: Tuesday, August 13, 2019 8:04:24 AM
To: patchthecode/JTAppleCalendar<EMAIL_ADDRESS>Cc: Conoize<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [patchthecode/JTAppleCalendar] Range selection core data (#1114)
Ok. Since you want dates which are saved in core data to be displayed on a window (and not on the calendar itself), then your question relates entirely to core data. I'll have to tell you that i know nothing about core data; it is not my expertise.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/patchthecode/JTAppleCalendar/issues/1114?email_source=notifications&email_token=AGCOKKH27ONO5BLQAARIFULQEKPMRA5CNFSM4IKZ7GNKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4FODSA#issuecomment-520806856, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGCOKKBHIIMV6AVWCRE6TQTQEKPMRANCNFSM4IKZ7GNA.

cool. Will close this issue then as the original one was answered.


Shipping method not getting updated after changing the zip code.

Bug report
Title
Shipping method not getting updated after changing the zip code.
Issue Description
Shipping method not getting updated after changing the zip code.
Preconditions
1. framework Version.-1.1.1

Steps to reproduce
1. step1- Proceed to checkout for a product and update the zip code.

Expected result
shipping method should display according to updated zip code.
Actual result

fixed


refreshing snap from 1.13/stable to 1.15/stable fails

The following sequence results in reproducible failures with the following inputs / outputs.
root@devstand:~# snap switch --channel=1.15/stable microk8s
"microk8s" switched to the "1.15/stable" channel

root@devstand:~# snap refresh --list
Name      Version  Rev  Publisher   Notes
microk8s  v1.15.3  825  canonical✓  classic
consider re-refresh of "microk8s"
root@devstand:~# snap refresh --classic microk8s
error: cannot perform the following tasks:
- Run configure hook of "microk8s" snap if present (run hook "configure":
-----
+ source /snap/microk8s/825/actions/common/utils.sh
+ '[' '!' -d /var/snap/microk8s/825/certs ']'
+ '[' '!' -f /var/snap/microk8s/825/certs/csr.conf.template ']'
+ cp /snap/microk8s/825/certs/csr.conf.template /var/snap/microk8s/825/certs/csr.conf.template
+ grep requestheader-client-ca-file /var/snap/microk8s/825/args/kube-apiserver
--requestheader-client-ca-file=${SNAP_DATA}/certs/ca.crt
+ grep -E -- '--(requestheader-allowed-names|requestheader-extra-headers-prefix|requestheader-group-headers|requestheader-username-headers|proxy-client-cert-file|proxy-client-key-file)=' /var/snap/microk8s/825/args/kube-apiserver
+ echo 'Enabling Enable the aggregation layer'
Enabling Enable the aggregation layer
+ echo ''
+ echo --requestheader-allowed-names=front-proxy-client
+ echo --requestheader-extra-headers-prefix=X-Remote-Extra-
+ echo --requestheader-group-headers=X-Remote-Group
+ echo --requestheader-username-headers=X-Remote-User
+ echo '--proxy-client-cert-file=${SNAP_DATA}/certs/front-proxy-client.crt'
+ echo '--proxy-client-key-file=${SNAP_DATA}/certs/front-proxy-client.key'
+ systemctl restart snap.microk8s.daemon-apiserver
+ grep -e requestheader-client-ca-file=/var/snap/microk8s/.../certs/ca.crt /var/snap/microk8s/825/args/kube-apiserver
+ mkdir -p /var/snap/microk8s/825/var/lock/
+ '[' '!' -f /var/snap/microk8s/825/credentials/client.config ']'
+ echo 'Patching client config location'
Patching client config location
+ mkdir -p /var/snap/microk8s/825/credentials/
+ cp /snap/microk8s/825/client.config /var/snap/microk8s/825/credentials/
cp: cannot stat '/snap/microk8s/825/client.config': No such file or directory
-----)

root@devstand:~# snap tasks 32
Status  Spawn               Ready               Summary
Done    today at 14:52 PDT  today at 14:53 PDT  Ensure prerequisites for "microk8s" are available
Undone  today at 14:52 PDT  today at 14:53 PDT  Download snap "microk8s" (825) from channel "1.15/stable"
Done    today at 14:52 PDT  today at 14:53 PDT  Fetch and check assertions for snap "microk8s" (825)
Undone  today at 14:52 PDT  today at 14:53 PDT  Mount snap "microk8s" (825)
Undone  today at 14:52 PDT  today at 14:53 PDT  Run pre-refresh hook of "microk8s" snap if present
Undone  today at 14:52 PDT  today at 14:53 PDT  Stop snap "microk8s" services
Undone  today at 14:52 PDT  today at 14:53 PDT  Remove aliases for snap "microk8s"
Undone  today at 14:52 PDT  today at 14:53 PDT  Make current revision for snap "microk8s" unavailable
Undone  today at 14:52 PDT  today at 14:53 PDT  Copy snap "microk8s" data
Undone  today at 14:52 PDT  today at 14:53 PDT  Setup snap "microk8s" (825) security profiles
Undone  today at 14:52 PDT  today at 14:53 PDT  Make snap "microk8s" (825) available to the system
Undone  today at 14:52 PDT  today at 14:53 PDT  Automatically connect eligible plugs and slots of snap "microk8s"
Undone  today at 14:52 PDT  today at 14:53 PDT  Set automatic aliases for snap "microk8s"
Undone  today at 14:52 PDT  today at 14:53 PDT  Setup snap "microk8s" aliases
Undone  today at 14:52 PDT  today at 14:53 PDT  Run post-refresh hook of "microk8s" snap if present
Undone  today at 14:52 PDT  today at 14:53 PDT  Start snap "microk8s" (825) services
Done    today at 14:52 PDT  today at 14:53 PDT  Clean up "microk8s" (825) install
Error   today at 14:52 PDT  today at 14:53 PDT  Run configure hook of "microk8s" snap if present
Hold    today at 14:52 PDT  today at 14:53 PDT  Run health check of "microk8s" snap
Done    today at 14:52 PDT  today at 14:53 PDT  Consider re-refresh of "microk8s"

......................................................................
Run configure hook of "microk8s" snap if present

2019-09-11T14:53:26-07:00 ERROR run hook "configure":
-----
+ source /snap/microk8s/825/actions/common/utils.sh
+ '[' '!' -d /var/snap/microk8s/825/certs ']'
+ '[' '!' -f /var/snap/microk8s/825/certs/csr.conf.template ']'
+ cp /snap/microk8s/825/certs/csr.conf.template /var/snap/microk8s/825/certs/csr.conf.template
+ grep requestheader-client-ca-file /var/snap/microk8s/825/args/kube-apiserver
--requestheader-client-ca-file=${SNAP_DATA}/certs/ca.crt
+ grep -E -- '--(requestheader-allowed-names|requestheader-extra-headers-prefix|requestheader-group-headers|requestheader-username-headers|proxy-client-cert-file|proxy-client-key-file)=' /var/snap/microk8s/825/args/kube-apiserver
+ echo 'Enabling Enable the aggregation layer'
Enabling Enable the aggregation layer
+ echo ''
+ echo --requestheader-allowed-names=front-proxy-client
+ echo --requestheader-extra-headers-prefix=X-Remote-Extra-
+ echo --requestheader-group-headers=X-Remote-Group
+ echo --requestheader-username-headers=X-Remote-User
+ echo '--proxy-client-cert-file=${SNAP_DATA}/certs/front-proxy-client.crt'
+ echo '--proxy-client-key-file=${SNAP_DATA}/certs/front-proxy-client.key'
+ systemctl restart snap.microk8s.daemon-apiserver
+ grep -e requestheader-client-ca-file=/var/snap/microk8s/.../certs/ca.crt /var/snap/microk8s/825/args/kube-apiserver
+ mkdir -p /var/snap/microk8s/825/var/lock/
+ '[' '!' -f /var/snap/microk8s/825/credentials/client.config ']'
+ echo 'Patching client config location'
Patching client config location
+ mkdir -p /var/snap/microk8s/825/credentials/
+ cp /snap/microk8s/825/client.config /var/snap/microk8s/825/credentials/
cp: cannot stat '/snap/microk8s/825/client.config': No such file or directory
-----

There is another directory that is not created on refresh
...
+ /snap/microk8s/860/bin/sed -i s/NAME/controller/g /var/snap/microk8s/860/credentials/controller.config
+ /snap/microk8s/860/bin/sed -i /username/d /var/snap/microk8s/860/credentials/controller.config
+ /snap/microk8s/860/bin/sed -i s/AUTHTYPE/token/g /var/snap/microk8s/860/credentials/controller.config
+ /snap/microk8s/860/bin/sed -i s/PASSWORD/WUlqUTdqbndhUHZ1dHdobGg0R3p5Y2RqckxKc3V0OFMvN3hlazZkTlAzVT0K/g /var/snap/microk8s/860/credentials/controller.config
+ refresh_opt_in_config kubeconfig '${SNAP_DATA}/credentials/controller.config' kube-controller-manager
+ local opt=--kubeconfig
+ local 'value=${SNAP_DATA}/credentials/controller.config'
+ local config_file=/var/snap/microk8s/860/args/kube-controller-manager
+ local 'replace_line=--kubeconfig=${SNAP_DATA}/credentials/controller.config'
++ grep -qE '^--kubeconfig=' /var/snap/microk8s/860/args/kube-controller-manager
+ sudo /snap/microk8s/860/bin/sed -i '$ a --kubeconfig=${SNAP_DATA}/credentials/controller.config' /var/snap/microk8s/860/args/kube-controller-manager
+ '[' -e /var/snap/microk8s/860/credentials/callback-tokens.txt ']'
+ skip_opt_in_config master kube-controller-manager
+ local opt=--master
+ local config_file=/var/snap/microk8s/860/args/kube-controller-manager
+ sudo /snap/microk8s/860/bin/sed -i /--master/d /var/snap/microk8s/860/args/kube-controller-manager
+ '[' -e /var/snap/microk8s/860/credentials/callback-tokens.txt ']'
+ refresh_opt_in_config use-service-account-credentials true kube-controller-manager
+ local opt=--use-service-account-credentials
+ local value=true
+ local config_file=/var/snap/microk8s/860/args/kube-controller-manager
+ local replace_line=--use-service-account-credentials=true
++ grep -qE '^--use-service-account-credentials=' /var/snap/microk8s/860/args/kube-controller-manager
+ sudo /snap/microk8s/860/bin/sed -i '$ a --use-service-account-credentials=true' /var/snap/microk8s/860/args/kube-controller-manager
+ '[' -e /var/snap/microk8s/860/credentials/callback-tokens.txt ']'
+ refresh_opt_in_config token-auth-file '${SNAP_DATA}/credentials/known_tokens.csv' kube-apiserver
+ local opt=--token-auth-file
+ local 'value=${SNAP_DATA}/credentials/known_tokens.csv'
+ local config_file=/var/snap/microk8s/860/args/kube-apiserver
+ local 'replace_line=--token-auth-file=${SNAP_DATA}/credentials/known_tokens.csv'
++ grep -qE '^--token-auth-file=' /var/snap/microk8s/860/args/kube-apiserver
+ sudo /snap/microk8s/860/bin/sed -i 's;^--token-auth-file=.*;--token-auth-file=${SNAP_DATA}/credentials/known_tokens.csv;' /var/snap/microk8s/860/args/kube-apiserver
+ '[' -e /var/snap/microk8s/860/credentials/callback-tokens.txt ']'
+ systemctl restart snap.microk8s.daemon-controller-manager
+ for dir in "${SNAP_DATA}/credentials/ ${SNAP_DATA}/certs/ ${SNAP_DATA}/args/ ${SNAP_DATA}/var/lock"
+ chmod -R ug+rwX /var/snap/microk8s/860/credentials/ /var/snap/microk8s/860/certs/ /var/snap/microk8s/860/args/ /var/snap/microk8s/860/var/lock
+ chmod -R o-rwX /var/snap/microk8s/860/credentials/ /var/snap/microk8s/860/certs/ /var/snap/microk8s/860/args/ /var/snap/microk8s/860/var/lock
+ getent group microk8s
+ getent group microk8s
+ chgrp microk8s -R /var/snap/microk8s/860/credentials/ /var/snap/microk8s/860/certs/ /var/snap/microk8s/860/args/ /var/snap/microk8s/860/var/lock/
+ '[' -f /var/snap/microk8s/860/args/flanneld ']'
+ cp -r /snap/microk8s/860/default-args/cni-network/flannel.conflist /var/snap/microk8s/860/args/cni-network/
cp: cannot create regular file '/var/snap/microk8s/860/args/cni-network/': Not a directory
-----)

Can you please try again:
sudo snap refresh microk8s --channel 1.15/edge

Great! It works. Thanks so much.


Display Subcommands in Help Menu

I can't quite figure out a way to populate subcommands/groups into the built-in help menu. I've defaulted to manually listing the subcommands in the help text, but that obviously requires manual upkeep. Maybe there's a way to use help_text_fn to accomplish this? Would greatly appreciate if anyone has some direction here (or telling me it's not possible with the current implementation)

Indeed, the built-in help menu only prints top-level commands. But support could be added behind a new HelpConfiguration::include_subcommands: bool flag. Not sure how the rendered text should look like. Suggestions welcome
In the meantime, you could make your own help command. Commands are accessible via ctx.framework().options().commands. Example:
let mut text = String::new();
for command in &ctx.framework().options().commands {
    text += &format!("{}: {}\n", command.name, command.description);
    for command in &command.subcommands {
        text += &format!("    {}: {}\n", command.name, command.description);
    }
}
ctx.say(text).await?;

Appreciate the help! I'll play around with some formatting/optionality and see what works well and report back.

Implemented in ec869bbd7cc97a9d9e8e0d83c239802b142b464f


fix(store): avoid prototype pollution

for (const key in patchToApply) {
  if (!patchToApply.hasOwnProperty(key)) continue
  ...
}

if (!patchToApply.hasOwnProperty(key)) continue

always return false,can't avoid prototype pollution

This breaks the existing behavior. Do you have a repro?

https://codesandbox.io/p/github/Debbl/pinia-template/draft/keen-river?file=%2Fsrc%2FApp.vue

App.vue file. The userStore does not have these attributes username and level, but it is working.

It's for intherited properties: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/hasOwnProperty

Thank you. It was my mistake.


adds snappy dependency in order to turn on avro snappy codec

Codecov Report

Merging #9 (0661ccd) into main (ba5d401) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main       #9   +/-   ##
=========================================
  Coverage     82.17%   82.17%
  Complexity      298      298
=========================================
  Files            63       63
  Lines          1021     1021
  Branches         62       62
=========================================
  Hits            839      839
  Misses          148      148
  Partials         34       34

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 ba5d401...0661ccd. Read the comment docs.


Don't apply :hover if disabled

I found Radium doesn't support :disabled so far but this might be a good idea since :active, :focus and :hover are already supported.

https://github.com/FormidableLabs/radium/tree/master/docs/faq#how-do-i-use-pseudo-selectors-like-checked-last-before-or-after

@ianobermiller I see. But actually :disabled is quite different than other pseudo selectors.
For example, the "disabled" attribute of "reset" button is controlled by a state that if other inputs/textareas are all empty. In this case a click on the reset button will clean all other inputs/textareas box and cause itself disabled.
In traditional CSS, the style :hover, :active and :focus can be overridden by :disabled. But since we are using JS events to simulate the behavior, at least the :hover behavior is still working and there's no way to override it.

In another hand... I think my problem actually is: should :hover be disabled when the element has the attribute "disabled"?

That seems fair!

Check this PR, #689
I had the same problem and made some workaround. I hope this change will help you.

Resolved by #689


Please fix issues reported by adapter checker

[E518] "@alcalzone/release-script" is used, but ".releaseconfig.json" not found
[W171] "common.title" is deprecated in io-package.json
[W105] "common.titleLang" should be translated into all supported languages (en, de, ru, pt, nl, fr, it, es, pl, uk, zh-cn)
[W109] "common.desc" should be translated into all supported languages (en, de, ru, pt, nl, fr, it, es, pl, uk, zh-cn)
https://www.iobroker.dev/adapter-check

Done!


[CLOSED] Email alerts, first pass

 Issue by christophwong
Wednesday 30 March 2016 at 13:08 GMT
Originally opened as https://github.com/alphagov/specialist-publisher/pull/687

This PR is the first pass of speciality publisher integrating with email-alert-api.
Trello Card
This PR adds feature that "On publishing a document with a major update, email-alert-api is triggered to send out notification emails.
I couldn't find an updated documentation of what the email-alert-api consumes so there isn't a contract test for what the payload should look like.
It currently has the keys subject, body, links, and document_type.
Testing I’ve done apart from the automated test, I ran email-alert-api locally and tailed the sidekiq logs, and on "publish with a major update for a finders document” I can see that it triggers an async-job to talk to GovDelivery (that’s where I had to stop without staging credentials for govdelivery).
still need to implement:

documents of type "medical-safety-alerts" uses a different template -- reference
documents of type "drug-safety-update" should NOT send email-alerts on publish or major updates.

christophwong included the following code: https://github.com/alphagov/specialist-publisher/pull/687/commits

 Comment by gpeng
Thursday 31 March 2016 at 10:07 GMT

Is there are reason why all the payload files, methods etc use "key" => val syntax rather than the >1.9 key: val?

 Comment by christophwong
Thursday 31 March 2016 at 10:19 GMT

@gpeng the payloads are in the old style to look more like JSONs

 Comment by gpeng
Thursday 31 March 2016 at 10:23 GMT

I thought it would be something like that. I still think they should probably be in the preferred syntax but not a problem if you disagree unless anybody else has an opinion. One other thing... Have the email templates you've used just been copied from the master or are they new? Have they been tested?

 Comment by christophwong
Thursday 31 March 2016 at 10:27 GMT

@gpeng All the email templates are copied over from master.
In terms of testing, the templates and full end-to-end with GovDelivery are not done, as I have no access to GovDelivery for this app.

 Comment by gpeng
Thursday 31 March 2016 at 10:34 GMT

Ok... Fair enough. If there are no changes from master then all should be well in the world. I'm happy to merge unless anybody else has an opinion on the Hash syntax.

 Comment by jackscotti
Thursday 31 March 2016 at 10:40 GMT

I think you'll need to remove the style from the HTML files and add a stylesheet where required.

 Comment by christophwong
Thursday 31 March 2016 at 10:45 GMT

@jackscotti regarding those html files, I believe the reason why the styles are inline is because they aren't rendered in our app, but will be sent as a string to GovDelivery to send out.

 Comment by christophwong
Thursday 31 March 2016 at 10:50 GMT

@tuzz the email templates are copied from master branch.
Let's do an end-to-end-test, then we can merge it.
Thank you everyone

 Comment by tuzz
Thursday 31 March 2016 at 10:45 GMT

Has the HTML email been constructed as part of this work or is it copied over from somewhere else? If it's been constructed, has it been tested to work in the majority of email clients?
The integration with Email Alert API looks ok. The fields you're passing through look correct (https://github.com/alphagov/email-alert-api/blob/master/app/controllers/notifications_controller.rb#L12-L16).
If you'd like to do a full end-to-end test on staging, I can help with providing access to govdelivery (I'm on 2ndline this week)


⚠ Bofrost.at has degraded performance

In 0002b7e, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2795 ms

Resolved: Bofrost.at performance has improved in bf514a2.


V2 API (top-companies API Endpoint): Random data is fetched when companies have same count for claManagersCount/corporateContributorsCount/projectsCount

Background

Fetch the response for https://api-gw.dev.platform.linuxfoundation.org/cla-service/v4/metrics/top-companies
Observed that random records are fetched when same count exists for multiple companies

Pull Request : https://github.com/communitybridge/easycla/pull/864
If count is same then sorting metrics by CompanyName

Triggered lambda function to get latest metrics and verified that each time records are fetched as below

topCompaniesByClaManagers are fetched based on claManagersCount
When few companies among top 5 companies of topCompaniesByClaManagers have same claManagersCount then records are fetched based on claManagersCount and ordered by company name
topCompaniesByProjectCount are fetched based on projectsCount
When few companies among top 5 companies of topCompaniesByProjectCount have same projectsCount then records are fetched based on projectsCount and ordered by company name

Could not test topCompaniesByCorporateContributors as I cannot contribute to existing companies that fetched in the top 5 companies of topCompaniesByCorporateContributors. Also I do not have more than 50 github accounts to test this issue

Verified metrics returning record by checking dynamo records for cla-dev-metrics and cla-dev-signatures tables

@dealako  to look at it & make a decision.
This API doesn't seem to be used by Corporate Console.

We are closing this issue for now. If the issue is observed, this ticket can be re-opened or a new ticket created.


notNull hints

Variants of: https://github.com/ndmitchell/hlint/blob/c9bfda7de8c2872d0d207212b123345756084e98/data/hlint.yaml#L210-L211 https://github.com/ndmitchell/hlint/blob/c9bfda7de8c2872d0d207212b123345756084e98/data/hlint.yaml#L811-L812

Sounds like it would be a good idea to add not (null ...) versions too, as only people who use HLint will see the notNull hints fire

I don't think explicit not (null ...) variants are needed:
    - warn: {lhs: not (null (concat x)), rhs: any notNull x}
    - warn: {lhs: not (null (filter f x)), rhs: any f x}
    - warn: {lhs: "not (null [x])", rhs: "True", name: Evaluate}
    - warn: {lhs: "not (null [])", rhs: "False", name: Evaluate}

because such expressions would already be addressed by other, existing rules (e.g., an Evaluate rule for null [] -> True plus one for not True -> False).

Ah, very true!


getting below errors after socket opened and when i try to subscribe my payload from Server sometimes

1st Error: handleCallbackError: java.util.TreeMap cannot be cast to java.lang.Throwable
2nd error:  @Override
public void handleCallbackError(WebSocket websocket, Throwable cause) throws Exception {
super.handleCallbackError(websocket, cause);
Log.i(TAG, " handleCallbackError: " + cause.getMessage());
}
handleCallbackError: java.lang.RuntimeException: Can't create handler inside thread Thread[WritingThread,5,main] that has not called Looper.prepare()
because of 2nd error sometimes i could not able to subscribe my payload and no updates from server

It would greatly increase fix chance if you provide a code snippet to reproduce the issue(and use some markdown).
Because now it seems that problem is not in this lib:
handleCallbackError: java.lang.RuntimeException: Can't create handler inside thread Thread[WritingThread,5,main] that has not called Looper.prepare()
There is no Looper in codebase and so such error message


Cannot get it working with Pi 2 hardware

Hi, I've used this excellent library on a Pi 1, however I cannot get it working at all on the Pi 2. I cannot even get the examples to work. The ofxRPiCameraVideoGrabber library works fine on the Pi 2, so I don't think it's a camera issue. Are you able to help me, please? Thank you.

Hi,
I've actually recently got my PI 2 and it works.
I did encounter some issues, these were with the OpenCV libraries themselves,
not the camera or the driver code I'm using (which is why ofxRPiCameraVideoGrabber works).
You can see some details on this issue here:
http://forum.openframeworks.cc/t/unable-to-link-opencvexample-on-armv7/14786
Long story short, until the OF's OpenCV static libraries get updated,
you can download the static libraries Aldo Hoeben recompiled from here:
https://github.com/fieldOfView/openFrameworks/tree/a1077f19ca7129cd7a5fbdea2dca1934875dcdc2/addons/ofxOpenCv/libs/opencv/lib/linuxarmv7l
Replace the old ones from the addons/ofxOpenCv/libs/opencv/lib/linuxarmv7l/
Let me know how this goes or if you run into any other issues.
Good luck,
George

Hello George,
Excellent – thank you for such a fast response!
I will follow your advice – and confirm how I fare.
Thanks again – your advice is very much appreciated!
Luke
From: George Profenza
Reply-To: orgicus/ofxCvPiCam
Date: Thursday, 2 April 2015 16:18
To: orgicus/ofxCvPiCam
Cc: Luke Butcher
Subject: Re: [ofxCvPiCam] Cannot get it working with Pi 2 hardware (#1)
Hi,
I've actually recently got my PI 2 and it works.
I did encounter some issues, these were with the OpenCV libraries themselves,
not the camera or the driver code I'm using (which is why ofxRPiCameraVideoGrabber works).
You can see some details on this issue here:
http://forum.openframeworks.cc/t/unable-to-link-opencvexample-on-armv7/14786
Long story short, until the OF's OpenCV static libraries get updated,
you can download the static libraries Aldo Hoeben recompiled from here:
https://github.com/fieldOfView/openFrameworks/tree/a1077f19ca7129cd7a5fbdea2dca1934875dcdc2/addons/ofxOpenCv/libs/opencv/lib/linuxarmv7l
Replace the old ones from the addons/ofxOpenCv/libs/opencv/lib/linuxarmv7l/
Let me know how this goes or if you run into any other issues.
Good luck,
George
—
Reply to this email directly or view it on GitHubhttps://github.com/orgicus/ofxCvPiCam/issues/1#issuecomment-88942393.

Perfect – that’s made my day!
Thank you VERY much!
Best wishes,
Luke
From: George Profenza
Reply-To: orgicus/ofxCvPiCam
Date: Thursday, 2 April 2015 16:18
To: orgicus/ofxCvPiCam
Cc: Luke Butcher
Subject: Re: [ofxCvPiCam] Cannot get it working with Pi 2 hardware (#1)
Hi,
I've actually recently got my PI 2 and it works.
I did encounter some issues, these were with the OpenCV libraries themselves,
not the camera or the driver code I'm using (which is why ofxRPiCameraVideoGrabber works).
You can see some details on this issue here:
http://forum.openframeworks.cc/t/unable-to-link-opencvexample-on-armv7/14786
Long story short, until the OF's OpenCV static libraries get updated,
you can download the static libraries Aldo Hoeben recompiled from here:
https://github.com/fieldOfView/openFrameworks/tree/a1077f19ca7129cd7a5fbdea2dca1934875dcdc2/addons/ofxOpenCv/libs/opencv/lib/linuxarmv7l
Replace the old ones from the addons/ofxOpenCv/libs/opencv/lib/linuxarmv7l/
Let me know how this goes or if you run into any other issues.
Good luck,
George
—
Reply to this email directly or view it on GitHubhttps://github.com/orgicus/ofxCvPiCam/issues/1#issuecomment-88942393.

Awesome!  Glad it works on your PI 2.
Hopefully the updated static libraries will make it to the main OF repo
and this step won't be necessary (otherwise a script might shave off a few minutes)
Thanks for the update, I'll close this issue now.


Feature/add timeout to info notifications

Add a 20s expiry time to all notifications that AREN'T errors aka info notifications and warn notifications, discussed in #1618
This PR will conflict with #1636 but will push a conflict fixed version depending on which goes first

This looks good too, but think if I accept it you pushing the merge conflict fix will cause the review to become undone, so may as well wait and accept it after that.
As an aside, is it worth me creating an issue about the notification size?
The boxes seem much bigger for me now:

Looks like there is some wasted space for me, though I am also not sure if that is because of my monitor res or scaling etc (4K monitor with 150% scaling and Windows isn't so great at scaling so it wouldn't surprise me if it was due to that).

@CrossR the max width on the notifications is set to 30% of the available space which since I work primarily of a laptop seemed to always be squarish but I guess on a hi-res large monitor 30% could easily be enormous might be worth me using 30rem instead which should roughly equate to 30characters wide based on the current font size, can add that to this once I push the merge fixes

might be worth me using 30rem instead which should roughly equate to 30characters wide based on the current font size, can add that to this once I push the merge fixes

Yes, +1 to using a measurement like this - the 30% is pretty huge on my monitor too. 👍

Updated 👍, dont actually have a large screen to hand but 25rem should be a more constant value than the percentage

Looks a lot more reasonable to me!

@bryphe managed to get it working with the observables 😄 🎇 and added the api method as well as removed the component side effect

Sweeet, nice find @Akin909 ! Change looks great - approving now.

Thanks @bryphe was nice to get the opportunity to use redux-observable, seems I took down one of the unit tests but just fixed it


Puppeteer hangs for some websites (Navigation Timeout Exceeded: 30000 ms exceeded.)

Puppeteer hangs at some websites, throwing following error:
pyppeteer.errors.TimeoutError: Navigation Timeout Exceeded: 30000 ms exceeded.
This is the example:
async def main():
    browser = await launch({'headless': False})
    page = await browser.newPage()
    await page.goto('https://www.ferragamo.com/shop/us/en/women/silk-bijoux/shawls-and-stoles')
    await browser.close()
asyncio.get_event_loop().run_until_complete(main())

I have tried 'waitUntil': 'networkidle0', 'waitUntil': 'networkidle2'. It does not help for this particular website.

@deinega What version do you use? Puppeteer 1.3 works just fine for me:
const puppeteer = require('puppeteer');

(async () => {
  const browser = await puppeteer.launch();
  const [page] = await browser.pages();
  await page.goto('https://www.ferragamo.com/shop/us/en/women/silk-bijoux/shawls-and-stoles');
  await browser.close();
})();

Thank for you reply.
I am actually using python version of puppeteer called pyppeteer (version 0.0.17):
https://github.com/miyakogi/pyppeteer
According to their documentation, it is based on 1.0.0:
https://miyakogi.github.io/pyppeteer/changes.html#version-0-0-17-2018-04-02

@deinega If it works using the 1.3 then your issue was probably already fixed since the 1.0.0.
You should probably create an issue to the other repository asking them to update their version of puppeteer instead.

I am using version 1.3.0 on Heroku and sometimes I get this error too through Sentry error reporting.


GH-37090: [C++] Add sum_checked aggregate function

Rationale for this change

The sum aggregate function doesn't check for overflow.
What changes are included in this PR?

Add a sum_checked function that returns error on overflow.
Are these changes tested?

Yes.
Are there any user-facing changes?
No.

Closes: #37090

By the way, do we expect AVX2 or AVX512 to generate more optimized code on overflow-checking operations?
I tested locally (gcc 12.3.0, AMD Ryzen 9 3900X) and AVX2 makes things worse here:
$ ARROW_USER_SIMD_LEVEL=none python -m timeit -s "import pyarrow as pa, pyarrow.compute as pc; a = pa.array([42]*1_000_000, type='int32')" "pc.sum_checked(a)"
500 loops, best of 5: 462 usec per loop
$ ARROW_USER_SIMD_LEVEL=avx2 python -m timeit -s "import pyarrow as pa, pyarrow.compute as pc; a = pa.array([42]*1_000_000, type='int32')" "pc.sum_checked(a)"
100 loops, best of 5: 2.46 msec per loop

Note that AVX2 does improve performance on non-checked sum operation:
$ ARROW_USER_SIMD_LEVEL=none python -m timeit -s "import pyarrow as pa, pyarrow.compute as pc; a = pa.array([42]*1_000_000, type='int32')" "pc.sum(a)"
2000 loops, best of 5: 120 usec per loop
$ ARROW_USER_SIMD_LEVEL=avx2 python -m timeit -s "import pyarrow as pa, pyarrow.compute as pc; a = pa.array([42]*1_000_000, type='int32')" "pc.sum(a)"
5000 loops, best of 5: 82.6 usec per loop

(this CPU can't run AVX512, sorry)
If we can't exhibit any performance gain, then we should remove the AVX* specializations.

@pitrou I've removed AVX2 and AVX512 for int types. They are kept for float types because sum_checked is the same as sum for floats, so in theory there should be a speedup. But my machine is ARM. Could you help verify this? Thanks!

I've tried to create a few benchmarks by hand and the SIMD speedups seem preserved (with AVX2; I don't have a AVX512 machine).
That said, we don't have any benchmarks for Sum and Mean in the repo, which makes comparisons tedious.

That said, we don't have any benchmarks for Sum and Mean in the repo, which makes comparisons tedious.

There is one for Sum [1] but it lives under Acero. I also added one for SumChecked

Hmm, can we move those benchmarks back under the kernels directory? (but Acero benchmarks should remain in the acero directory)

Scalar aggregate benchmarks are moved back to compute/kernels and hash aggregate ones are kept under Acero.

@pitrou Hi, would you mind taking another look at this PR?

@js8544 can you split this PR in two? One that does all the code movement in preparation to the implementation and then the kernel implementation itself? Because as I review I have to do that mentally myself which is very tricky.


Windows 10 v0.9.4 installs but doesn't launch

IPFS command line is installed in a non-standard location.  The IPFS daemon executes successfully along with the Web UI.
Not seeing any events or log file entries to reference to provide more details on why the IPFS Desktop isn't launching.

Hello @cadayton! Thanks for your report!
I just installed 0.9.4 on a fresh Windows 10 install and it works with no issues:

What do you mean? What is not working?

IPFS command line is installed in a non-standard location.

We add the CLI tools to the PATH as most applications on Windows do. There is no standard location for Windows to put binary files (as opposed to Unix where /usr/local/bin is the default).

When I double click on the desktop icon the application doesn't start.  Also tried running the .exe file directly too.
Craig Dayton
<EMAIL_ADDRESS>360-466-8778
paypal.me/CraigDayton
Sent with ProtonMail Secure Email.
‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
On Sunday, October 6, 2019 1:01 PM, Henrique Dias<EMAIL_ADDRESS>wrote:

Hello @cadayton! Thanks for your report!
I just installed 0.9.4 on a fresh Windows 10 install and it works with no issues:
image
What do you mean? What is not working?

IPFS command line is installed in a non-standard location.

We add the CLI tools to the PATH as most applications on Windows do. There is no standard location for Windows to put binary files (as opposed to Unix where /usr/local/bin is the default).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

My bad.  I didn't notice IPFS desktop was running in the system tray.

No problem at all. It's good you figured out what was wrong. We are actually thinking about starting to open the window the first time we double-click the icon: https://github.com/ipfs-shipyard/ipfs-desktop/issues/1153#issuecomment-536587390


Target region(s)

[x] Concurrent deploy
[x] Re-use zip
[ ] Stack support
[ ] Config

35s for all 12 regions from this starbucks connection


Fix dependency misspelled

According to @fians, the "Inc" repository never existed...
Switching to the correct dependency (Waves).

Thank you. :+1:

Thanks :+1:


chore: upgrade deps

#2376

As tests are saying, you should remove useBuiltins option
https://github.com/vuejs/vue-cli/blob/dev/packages/%40vue/babel-preset-app/index.js#L44

Needs help, It's beyond my ability.

Closing in favor of https://github.com/vuejs/vue-cli/pull/2642


Add a new v5 transaction script verification API to zcash_script and Zebra

Motivation
We want to modify ZIP-244 to include all UTXO values and scripts in each input's sighash.
This requires updates to zcashd's script module, zcash_script, zebra-script, and zebra-consensus.
Specifications
ZIP 244 - Changes to transparent component of signature digest
TODO: quote consensus rules after the ZIP PR merges
Designs
Zebra changes:

update zcashd's script module with a v5 script verification precompute function, which takes the the concatenated serialized bytes of all the spent outputs for a transaction
update zcash_script's submodule to include the new v5 function
update zebra-script with a safe interface to the new v5 function
update zebra-consensus to wait for all the UTXOs, then pass them to the new zebra-script v5 function

API consistency:

also add a non-precomputed function to zcash_script and zebra-script

Hey team! Please add your planning poker estimate with ZenHub @conradoplg @dconnolly @jvff @oxarbitrage @teor2345 @upbqdn

Split the remaining part into https://github.com/ZcashFoundation/zebra/issues/3520.


Allow dom-repeat for array-like Objects

Implement a further check on objects passed through dom-repeat to check if they have the required properties and methods (https://www.polymer-project.org/1.0/docs/devguide/properties#array-mutation) to allow stamping out of array-like objects on a template dom-repeat.
Reference Issue
https://github.com/Polymer/polymer/issues/3811

The problem is a little bit more subtle as the code assumes that items is always an array. I think instead it should switch to [Symbol.iterator] instead to also be compatible with Map and Set.

Closing per above comment. Feel free to publish a new PR that uses [Symbol.iterator] as a check. Right now there a lot of internal checks in dom-repeat that rely on the value being an Array, so the PR might grow a lot bigger.


Logo image contains non-free embedded ICC profile

The Groestlcoin logo - same content at the two paths  src/qt/res/icons/about.png and src/qt/res/icons/groestlcoin.png - contains embedded ICC profile which is copyright Hewlett Packard and lack license.
Please strip that ICC profile, to legally permit redistribution of that/those file(s).

ICC are removed


South Park corrupted downloads, something about hls features

(using the latest version, read the faq, etc.)
southpark.cc.com download used to work very well. Today it started to warn about unsupported hls feature in the native parser, but ffmpeg shows a lots of errors. The result download are playable to some extents, but there are jumps every few seconds, like for a corrupted file. The output log is very long, I have pasted it somewhere else: https://paste.fedoraproject.org/paste/ds7s3t6e~y5qFe5pwS7ifA
I have tried a few episodes, and it appears to be an issue for all of them.

Install pycrypto or update ffmpeg.


Add support for milliseconds in timestamp (allowed by the ISO 8601-format)

Added support for milliseconds in timestamp - used by the ISO 8601 Extended Format implemented by Javascript as Date.toISOString().

Your version of master is out of date.

Will reresubmit.


Create SECURITY.md

Why:
Closes [issue link]

What's being changed (if available, include any code snippets, screenshots, or gifs):

Check off the following:

[x] I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

Thanks for opening a PR @AalOsman! We're not going to add a security.md at this time, so I'm going to close this out.
Thank you for your interest in improving the GitHub docs ✨


latest master changes

@hendrikmuhs
This PR contains latest master changes

Coverage increased (+0.01%) to 88.215% when pulling 388d0c7429c985efdf48bdecc88e692750aef10c on narekgharibyan:0_2_cleanup into d9de2d31b44057fa95c8b670ab87a0ba32aa0549 on cliqz-oss:0_2_cleanup.


Marker not highlighted on iOS. Samething works perfect on Android

I have different types of marker placed on map. The issue is when i want to move map camera to another location with marker infowindow. Infowindow shows correct but marker is not showing in front like android, its showing behind . Android is working perfectly. You can check video link below.
  I change map camera position with marker infowindow with below code:

  Marker m = _markers.elementAt(index);
  _mapController
      .animateCamera(CameraUpdate.newLatLngZoom(m.position, mapZoom));
  Future.delayed(const Duration(milliseconds: 100), () {
    _mapController.showMarkerInfoWindow(m.markerId);
  });

Check video reference

Hi @MdFarhanRaja
What does your maker icon look like? Also, please try to provide the information indicated below so that we can address this issue properly.

google_maps_flutter version you are using
a completed and minimal reproducible code sample
The output of flutter doctor -v

Thank you!


Add individual options for amount of clicks to open files and folders

What's the Problem?
There is currently an option to open items with a single click, some users might want additional customization in this area to configure files and folders differently.
Solution/Idea
Adding a second option to "Open files with a single click" should resolve this issue.
Alternatives
na
Priorities

Capability
Priority

This proposal will change the current setting to only apply to folders
Must

This proposal will add an option to open files with a single click
Must

Files Version
v2.0.15
Windows Version
Windows 11
Additional comment
No response

Correct, I would like to have single click for folders (Column view) but still double click for files. I think it is the same behavior with the Mac OS Finder. Also when I move the keyboard arrows up and down, it should open the folder. But this seems another issue, probably.


"File not found" Error even when provided correct PDB path.

Hi,
This application fails to read any PDB files with either of the following commands
pdbex * factorio.pdb -o factorio.h
pdbex * ./factorio.pdb -o factorio.h

After reading the source code, I realized that this application depends on a msdia140.dll file.
Adding this file directly into the working directory seems to be a workaround for the issue.

However the error message instead implies that the application cannot find the .pdb file.
I recommend either removing the dependency on msdia140.dll or improving the error message.

It would be great to know which file is not found. In my case I'll need to probably run pdbex from source to figure it out


Fixed redirect.html

-Reactivated redirecting functionality in redirect.html

It looks like you've got some conflict with our dev branch. Ping me on Slack if you're looking for how to go after those.


Multisample framebuffer

This is an implementation proposal for framebuffer multisample feature.
When using multisampling with framebuffers, you need 2 OpenGL FBOs :

a mutisample FBO used to capture the scene. This one can't be rendered directly or used as regular textures.
a non multisample FBO where resolved samples will be copied to textures and then usable as regular textures.

There was several ways to implementing it. There was an old PR where GLFrameBuffer was handling 2 OpenGL framebuffers. This overcomplexify GLFrameBuffer code with a lot of code duplication. It also makes it more complex to configure and introduce confusion : (eg. which one is bound when calling bind method). So i decided to not take that route.
This PR use an approach closer to the OpenGL API. As you can see in the test, user code have to create 2 FrameBuffers and call the transfer method to copy multisamples into the other FBO. Another benefit is that in some case (hardware specific) it's not possible to have a texture for the stencil buffer or the packed depth and stencil buffer. With this approach, you can render a scene using both depth and stencil buffers, and later transfer only depth pixels to a texture. It also allows internal format conversion (eg. rendering colors into a multisample float buffer and transfer to an RGBA8 texture).
Note that, it could be possible to have texture attachments with multisample enabled. That allows to get hand on samples in order to compose with other multisampled textures and handle antialiasing manually. These textures can't be rendered directly neither. This feature (a very advanced case imho) would requires more changes and i thought it didn't worth it at the moment. An exception is thrown when using multisample FBO and texture attachement together.
Transfer code is a bit complex because of OpenGL API apparently designed for non MRT context. glBlitFramebuffer is the command that perform the copy, it allows to copy depth, stencil and one color buffer in a single call. But if you have multiple color attachement, it's necessary to call it again for other color buffers.
There are surely some stuff to improve, i'd like to have your opinion on the API proposal.

When running the GL31FrameBufferMultisampleMRTTest on Windows Desktop I am just seeing a black screen. Is this normal?

No, it should looks like this :

Maybe on Windows it requires glEnable(GL_MULTISAMPLE); ? from this tutorial :

On most OpenGL drivers, multisampling is enabled by default so this call is then a bit redundant, but it's usually a good idea to enable it anyways.

Could you try to do that ? it should be done in desktop code, because it's not part of GLES : glEnable(GL13.GL_MULTISAMPLE);

No luck, I did try that yesterday too. Tried again just to be sure, tried both GL13 ENUM as well as just plugging its integer value in directly.
Gdx.gl.glEnable(GL13.GL_MULTISAMPLE);
Gdx.gl.glEnable(0x809D);
I also tried putting GLProfiler into the test, but not getting any error messages out.
On Windows 11 with Nvidia 3080

Narrowed the problem down. If I remove Gdx.gl.glEnable(GL20.GL_DEPTH_TEST); in render method of test, it works.

Narrowed the problem down. If I remove Gdx.gl.glEnable(GL20.GL_DEPTH_TEST); in render method of test, it works. Or I can keep the depth test, but I must add true to the first ScreenUtils clear method to clear depth as the first one does not pass true.
ScreenUtils.clear(Color.CLEAR, true);

Good catch! it's fixed. I remember now that default frame buffer clearing have difference between windows and linux. Thanks.

Narrowed the problem down. If I remove Gdx.gl.glEnable(GL20.GL_DEPTH_TEST); in render method of test, it works. Or I can keep the depth test, but I must add true to the first ScreenUtils clear method to clear depth as the first one does not pass true.
ScreenUtils.clear(Color.CLEAR, true);

Good catch! it's fixed. I remember now that default frame buffer clearing have difference between windows and linux. Thanks.

Just checking back in. Your latest changes are working for me on the test. I submitted my other two concerns/observations as part of the GH review process.

Any updates?)

Hey folks.  I check in every week to see if they are any updates on this PR.  I am sure many are also waiting on this.
I know its summer and we are all busy.   A friendly reminder that the community appreciates the work you do.  Cheers!

What can the community do to help the libGDX contributors reduce the cycle time for PRs to get merged?  This is an honest and sincere question.

What can the community do to help the libGDX contributors reduce the cycle time for PRs to get merged? This is an honest and sincere question.

I don't know enough about this domain to be able to adequately review this PR. I think we're kind-of short on people who do have that domain knowledge, much like we are with people who understand Tiled PRs.

@mgsx-de It'd be great if you could have a look at @JamesTKhan review comment https://github.com/libgdx/libgdx/pull/7185#discussion_r1345826056 when you have the chance, considering the interest we should aim at getting it merged in 1.12.2.

@JamesTKhan i think i addressed all your points (thank you for the review), it seems that you need to review again unblock the merge.
@obigu I think it's ready to be merged.
Thank you all for your patience


Update to dev branch of tchannel-go

I looked through the commits on the dev branch since the last hash we pinned. A lot of stuff looked like tweaks / bug fixes, but a few things stood out:

"Don't close connections on errors": https://github.com/uber/tchannel-go/commit/b9dc4c162aad44e2f01508cf0c6e81032675b4cf
"Close connections if the RelayHost returns a ProtocolError": https://github.com/uber/tchannel-go/commit/fef3a3f5eb009e12c7d4446979a425aa2435a760
"Set DiffServ (QoS) bit on Outbound connections": https://github.com/uber/tchannel-go/commit/6b8d075c6f47e574ee33513fe1375648fd9a0651

Changes Unknown when pulling d7735df291357dd3fbf855e3e746658668c6c331 on richardartoul:ra/bump-tchannel-go into ** on m3db:master**.

Changes Unknown when pulling d7735df291357dd3fbf855e3e746658668c6c331 on richardartoul:ra/bump-tchannel-go into ** on m3db:master**.

@richardartoul looks good. before landing, I'd suggest deploying m3dbnode on some hosts that don't share shards and monitoring perf changes for a bit

@prateek Looks like Chao has already tested this version and its good to go. Gonna land this so we can get C.I working again!

LGTM


Problems updating to 0.19.1

Trying to update to the latest version, from 0.18.9, and I'm seeing this error in typescript compliation:
ts/src/index.ts:176:35 - error TS2551: Property 'levels' does not exist on type 'Readonly<LoggerConfig>'. Did you mean 'level'?

176     level: Logger.DEFAULT_OPTIONS.levels[config.logLevel || 2],
                                      ~~~~~~

Is this expected? Where are the log levels exposed now?

@kjin I ran into some issues in nodejs-error-reporting as well with the logger :/  I'm not entirely familiar with the changes you made in there - would you mind taking a gander?

I would expect the array with the log levels to be exposed somewhere.
I tried: Logger.LEVELS[config.logLevel as number] but still saw (about) the same error:
ts/src/profiler.ts:239:21 - error TS2339: Property 'LEVELS' does not exist on type 'typeof Logger'.

239       level: Logger.LEVELS[config.logLevel as number],

I actually already addressed this in #157 but I didn't think it needed to be released right away. Sorry about that, I'll open a release proposal. In the meantime, I believe lowercase logger.LEVELS still works.

@nolanmar511 v0.19.2 has been released, it should contain the fix.

Thank you @kjin!


Change app icon

Change the app icon
issue: https://github.com/VikasPandey121/WildCare/issues/15
This time i've added the .png on drawable folder, then using AndroidStudio I added  the .png with new Image Asset manager as Icon Launcher (Adaptative and Legacy) using the previous icon name (ic_launcher and derivates)
I didn't change manifest.xml because i've maintained the previous icon name (ic_launcher)

Here in this PR, you can see that in the AndroidManifest.xml android:icon="@drawable/ic_launcher_background" due to which the android application will look for the icon with the given path and file name only which is there in the drawable folder with file name ic_launcher_background.
Please follow these steps to do the needful changes:

Add the animal-care.png file in the drawable file without changing its name.
In the AndroidManifest.xml file change android:icon="@drawable/ic_launcher_background" with<EMAIL_ADDRESS>Don't create another PR.
Squash your commits into one. Here, this page might be helpful for it.

Feel free to ask either here or in the Gitter channel.
I am closing this PR for now. Please work upon suggested changes and reopen the same PR.

no problem, you can assign to another one, thank you. o/


multiple upload

hi
how about multiple upload?
I've add use-custom-dropzone-options=true and in my options uploadMultiple but how can I use successmultiple event of dropzone?

Hi @Bahram9
I've just released version 2.1.7 to npm which should support the successmultiple and sendingmultiple events. Check out the docs . Let me know if you run into any issues.
Cheers
Rowan

hi
it's good to hear that, thank you.
Cheers
Bahram
On Tue, Feb 14, 2017 at 1:40 PM, Rowan Winsemius<EMAIL_ADDRESS>wrote:

Closed #36 https://github.com/rowanwins/vue-dropzone/issues/36.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rowanwins/vue-dropzone/issues/36#event-961098457, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AH2eVmCQ0io7s63tUPz0UnVMT0zOvY1uks5rcX2UgaJpZM4L7Co-
.


Plugin fails to work with Safari Version 11.0.2 (136<IP_ADDRESS>.3)

Images and Videos are invisible on Version 11.0.2 (136<IP_ADDRESS>.3) of Safari

closing since it seems to be specific to a patch version


Unify default hash lenght

Relates: https://github.com/openshiftio/saas/pull/236

CC @vpavlin

we first need to fix scripts which build images and have the number 6 harcoded as well. As we are not able to coordinate this across the board, the easiest and safest way to do it is to set hash_length for all services. Once all of them use length 7, we can remove the hash_length from all of them and apply this patch.


Infinite loop occurred at linked list

then "pOption" and "pOption->next" are same pointer address.
https://github.com/Lobaro/lobaro-coap/blob/d7a4c79ad2217f744ac9757d2b4bc15599442450/src/coap_options.c#L423-L425
Is the free list correct?
https://github.com/Lobaro/lobaro-coap/blob/d7a4c79ad2217f744ac9757d2b4bc15599442450/src/interface/mem/coap_mem.c#L1015-L1024

I'm sorry.
It was a problem due to multi-threaded exclusive control on the application side.
Close the problem.


Seems like the model outputs are disp image rather than depth image.

Thanks for the great work!
I noticed that the model outputs the disp image instead of depth image, how can I get the depth image from the model?

I have noticed that the code in metric-depth provides depth image, but I found the results were worse and many floater can be observed in the results. Since I don't need the absolute scaled depth and a relative one is okay, is there a way to easily convert the results from the original model to a relative depth map?

@InFistLee i have a same problem, do you have any idea recently?

@InFistLee same issue here, i just can't find the code to generate the depth image like this :

Does anyone have the solution to this problem?

@InFistLee To generate the depth map image (depth.png), and @Liuhsinlun to generate the coloured depth map (depth_colored.png) use this code:
import cv2
import numpy as np
from PIL import Image
import matplotlib.pyplot as plt
import torch

from depth_anything_v2.dpt import DepthAnythingV2

DEVICE = 'cuda' if torch.cuda.is_available() else 'mps' if torch.backends.mps.is_available() else 'cpu'

model_configs = {
    'vits': {'encoder': 'vits', 'features': 64, 'out_channels': [48, 96, 192, 384]},
    'vitb': {'encoder': 'vitb', 'features': 128, 'out_channels': [96, 192, 384, 768]},
    'vitl': {'encoder': 'vitl', 'features': 256, 'out_channels': [256, 512, 1024, 1024]},
    'vitg': {'encoder': 'vitg', 'features': 384, 'out_channels': [1536, 1536, 1536, 1536]}
}

encoder = 'vitl' # or 'vits', 'vitb', 'vitg'

model = DepthAnythingV2(**model_configs[encoder])
model.load_state_dict(torch.load(f'checkpoints/depth_anything_v2_{encoder}.pth', map_location='cpu'))
model = model.to(DEVICE).eval()

raw_img = cv2.imread('path2RGB.png')
depth = model.infer_image(raw_img) # HxW raw depth map in numpy

depth_image = Image.fromarray(depth.astype(np.uint16))
depth_image.save('depth.png')

depth_normalized = (depth - depth.min()) / (depth.max() - depth.min()) * 255.0
depth_normalized = depth_normalized.astype(np.uint8)
colored_depth = (plt.cm.plasma(depth_normalized / 255.0)[:, :, :3] * 255).astype(np.uint8)

colored_depth_image = Image.fromarray(colored_depth)
colored_depth_image.save('depth_colored.png')

@AbedSoleymani   Thank you sooo much!!!

@InFistLee To generate the depth map image (depth.png), and @Liuhsinlun to generate the coloured depth map (depth_colored.png) use this code:
import cv2
import numpy as np
from PIL import Image
import matplotlib.pyplot as plt
import torch

from depth_anything_v2.dpt import DepthAnythingV2

DEVICE = 'cuda' if torch.cuda.is_available() else 'mps' if torch.backends.mps.is_available() else 'cpu'

model_configs = {
    'vits': {'encoder': 'vits', 'features': 64, 'out_channels': [48, 96, 192, 384]},
    'vitb': {'encoder': 'vitb', 'features': 128, 'out_channels': [96, 192, 384, 768]},
    'vitl': {'encoder': 'vitl', 'features': 256, 'out_channels': [256, 512, 1024, 1024]},
    'vitg': {'encoder': 'vitg', 'features': 384, 'out_channels': [1536, 1536, 1536, 1536]}
}

encoder = 'vitl' # or 'vits', 'vitb', 'vitg'

model = DepthAnythingV2(**model_configs[encoder])
model.load_state_dict(torch.load(f'checkpoints/depth_anything_v2_{encoder}.pth', map_location='cpu'))
model = model.to(DEVICE).eval()

raw_img = cv2.imread('path2RGB.png')
depth = model.infer_image(raw_img) # HxW raw depth map in numpy

depth_image = Image.fromarray(depth.astype(np.uint16))
depth_image.save('depth.png')

depth_normalized = (depth - depth.min()) / (depth.max() - depth.min()) * 255.0
depth_normalized = depth_normalized.astype(np.uint8)
colored_depth = (plt.cm.plasma(depth_normalized / 255.0)[:, :, :3] * 255).astype(np.uint8)

colored_depth_image = Image.fromarray(colored_depth)
colored_depth_image.save('depth_colored.png')

Hello, I would like to ask a question, why is the size of the depth value opposite? For example, in the following picture, the closer the street is to the inside, the larger the value should be, but the value in the numpy array is smaller.
According to normal intuition, the greater the depth, the larger the value should be.

@520jz I have had the same question too. And I figured it out in their original paper sec 5.2

Similarly, our models produce affine-invariant inverse depth

So the output is inverse depth, the real depth should be 1 / output.

May I ask what the unit of this depth value is, meters


{doc} Add extension versioning doc

Related command

Description

Testing Guide

History Notes

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

doc

The word "previous" is used in the first paragraph, and the word "last" is used in the second paragraph. Is this intentional?


TRAPI 1.0 parameterization

The TRAPI 1.0 query below provides an example for COHD, with default query parameters used in cases when a user does not specify them. COHD based their specs on those that Hao used for the ICEES+ API. However, I cannot remember what the defaults are.
Here are the parameters that are included in the Swagger interface:

Here is a set of parameters that I thought we were going to include:
"query_options": {
"max_results": 100,
"method": "ChiSquare",
"maximum_p_value": 1,
"correction": true,
"method": "bonferroni"
"biolink_only": true,
"ontology_targets": null
}
Here is a COHD TRAPI 1.0 example
{
"message": {
"query_graph": {
"nodes": {
"n00": {
"id": "MONDO:0007254"
},
"n01": {
"category": "biolink:Drug"
}
},
"edges": {
"e00": {
"predicate": "biolink:correlated_with",
"subject": "n00",
"object": "n01"
}
}
}
},
"query_options": {
"max_results": 50,
"method": "obsExpRatio",
"dataset_id": 3,
"local_oxo": true,
"biolink_only": true,
"ontology_targets": null
}
}

From the documentation on the ICEES+ KG query (not a TRAPI 1.0 query):

KG overlay endpoint:

KG one-hop endpoint:


myth -j creates massive log file

Symptom
Calling myth with -j on my project (300 LOC in total) took very long time and generated over 90mb of json object dump
Version
0.16.31
Steps to recreate
git clone https://github.com/makoto/blockparty.git
cd blockparty
git checkout 1c2884cc9262c5fc8aba026720b289620c00bceb
myth -j ./statespace.json contracts/Conference.sol --max-depth 8
$ ls -ltr statespace.json
-rw-r--r--  1 makoto  staff  92582854  5 May 20:52 statespace.json

Thanks Makoto, we'll try to address this soon.

There is probably a state explosion due to:
	function attend(address[] _addresses) external onlyAdmin onlyActive{
		for(uint i=0;i<_addresses.length;i++){
			address _addr = _addresses[i];
			require(isRegistered(_addr));
			require(!isAttended(_addr));
			emit AttendEvent(_addr);
			participants[_addr].attended = true;
			attended++;
		}
}

Could this be related to #111? It seems that loops don't always trigger the depth limit..

Not sure, I'll look into it

@b-mueller Yes, it seems to be the same problem. There is a state explosion due to of the inverted jumpi condition branch

Second example from #111:
The loop in getTransactionCount() causes the interpreter to enter an endless loop if max-depth>=10.
    function getTransactionCount(bool pending, bool executed)
        public
        constant
        returns (uint count)
    {
        for (uint i=0; i<transactionCount; i++)
            if (   pending && !transactions[i].executed
                || executed && transactions[i].executed)
                count += 1;
    }

I assume this will be solved if we implement the wordlist algorithm instead of recursive _sym_exec.


Diagnostics in Problems tab do not use monospaced font

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.84.2
OS Version: macOS 13.6.2

Context:
I'm the developer of https://github.com/elixir-lsp/vscode-elixir-ls and I'm trying to make elixir compiler error messages look nice in VSCode. I noticed that Problems tab uses non monospace font which breaks rendering of preformatted messages.
Steps to Reproduce:

Use an extension that emits LSP diagnostics that are preformatted

Result:
The same diagnostic renders correctly with monospaced font in hover panel in editor but renders badly in Problems tab

This is the message that was used in the screenshots above
** (MismatchedDelimiterError) mismatched delimiter found on lib/abc.ex:5:3:
    error: unexpected reserved word: end
    │
  4 │     elem({2}, 1
    │         └ unclosed delimiter
  5 │   end
    │   └ mismatched closing delimiter (expected ")")
    │
    └─ lib/abc.ex:5:3
    (elixir 1.16.0-rc.0) lib/kernel/parallel_compiler.ex:428: anonymous fn/5 in Kernel.ParallelCompiler.spawn_workers/8

For reference the LSP Diagnostic message is a plain string and does not accept markdown

/duplicate of #54272

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!