Trouble with “Actually used Certificate” if more than one vaccination certificate is available…

Avoid duplicates

[X] Bug is not mentioned in the FAQ
[ ] ?: Bug is specific for Android only, for general issues / questions that apply to iOS and Android please raise them in the documentation repository
[X] Bug is not already reported in another issue

Technical details

Device name:  Samsung Galaxy Note 10, SM-N970F/DS
Android version:  Android 11 (One UI 3.1) with Android security update: 01. August 2021
App version:  2.8.0

Describe the bug
Problems after selecting the "Currently used Certificate" („Aktuell verwendetes Zertifikat“) because then another vaccination certificate is the current one...
If more than one vaccination certificate for a person has been scanned into the CWA, the following selection results, for example:

If you now tap on the "Currently used Certificate" („Aktuell verwendetes Zertifikat“) you can see the details of this certificate; including the URN. For example, my URN ends with "5".
After tapping on the arrow at the top left [<-] you get to the previous screen (see picture above).
If you look closely, you can see that after tapping on [<-] the positions of the two 2/2 certificates are swapped.
If you now tap on the "Currently used Certificate" („Aktuell verwendetes Zertifikat“) again and look at its details, the position swap of the two certificates is confirmed. In my example, the certificate with the URN "5" is no longer displayed, but my certificate with the URN "Q" at the end.
Steps to reproduce the issue

Start CWA.

Select "Certificates".

Select a person with multiple vaccination certificates.

Scroll down the screen; see screenshot above.

Select the "Currently used Certificate" („Aktuell verwendetes Zertifikat“) and pay attention e.g. to the URN under certificate identifier ("Unique Certificate Identifier").

Tap on [<-] (top left) - the two vaccination certificates 2 of 2 are visibly reversed in the order.

Select the "Currently used Certificate" („Aktuell verwendetes Zertifikat“) again and verfify the URN under Certificate ID ("Unique Certificate Identifier"). It is now the other certificate.

Steps 6 and 7 can now be repeated as required; the 'other' certificate is displayed in each case.

Possibly incomplete Example:

Notes:

If in steps 5 and 7 you do not select the "Currently used Certificate" („Aktuell verwendetes Zertifikat“) but rather the other one ('Currently NOT used Certificate' 2/2), the order of the certificates is not changed; i.e. the same certificate is always displayed here. Depending on the initial situation, in my case either the one with the URN ending "5" or "Q".

All of the above steps and results can be transferred 1:1 to certificates 1/2. When selecting the upper 1/2 certificate and taping on [<-] (top left), it is also visibly exchanged with the other (here second) 1/2 certificate in the sequence shown.
When you tap on the lower 1 of 2 certificate (also several times) - as with the 'Currently NOT used Certificate' above - the order of the 1/2-certificates is not changed either, which means that always the same certificate is displayed.

Changing the order of certificates 2/2 does not affect the order of certificates 1/2 and vice versa.

(Hint: This means that the affiliations of the certificates (2/2 & 1/2) with regard to the displayed order are no longer given.)
Expected behaviour
Is that so useful / desired? I don't think so.
This can become problematic and confusing if the 'country-specific checks of the vaccination certificates' are different for several existing valid certificates; e.g. because of different DCC schema versions or other.
(See "Additional Context" below.)
Possible Fix
I suspect it would make sense to be able to set a certificate as the standard certificate, which is then also used for the 'country-specific check of the vaccination certificate' - until the standard certificate is explicitly changed, right?
Of course, this assumes that a standard certificate can be set manually.
Additional context
As a result for CWA v 2.8.0, the workaround for sorting the vaccination certificates no longer works; see: https://github.com/corona-warn-app/cwa-documentation/issues/679#issuecomment-896324471.
Cross links:

Latest issued Vaccination DCC 2/2 does not have highest priority.
Check for different schema versions for certificates stored for the same person.

PS: I don't think we need to have another discussion here about whether or not it makes sense to have multiple valid certificates for one person.
PS2: A revision of the certificate handling is likely to be necessary if 3rd / 4th / ... vaccinations or their certificates have to be processed in the CWA in the future.
Best regards, Joachim.
Internal Tracking ID: EXPOSUREAPP-8756
[Android] Latest issued Vaccination DCC 2/2 does not have highest priority

@Jo-Achim are these real certificates? I don't see the vaccinatedOn date in your screenshot?

@mtwalli, yes, these are real, valid certificates.
I don't quite understand your question. What information should my screenshot have included? Do you have a screenshot with the missing data? Further data are only visible after selecting one of the certificates.

I think Mohamed means, that instead of your real name "Joachim Mustermann" and your real birthdate "29.02.1963" there are just placeholders "Firstname Surname" and "dd.mm.yyyy".
So, question is, whether the certificates are self-created ones, or if they had been issued officially with valid data, but you replaced your personal data in the screenshot for privacy reasons.
By the way, the described behaviour does not appear with CWA 2.6.1, so it must have been introduced after.

@Jo-Achim  As @vaubaehn  already clarified. I want to understand the data in the certificate is real or you created it. Valid does not mean it is real, right?. I assume that these certificates some of them have the same vaccinatedOn date which Vaccination certificates are sorted by it. If you more than certificate with the same vaccinatedOn date it might be the cause to the ordering you experienced. In real cases no user will take more than one dose in the same day, is that your case ?

@vaubaehn, @mtwalli, the screenshots are scans from real original "EU Digital Covid Vaccination Certificates"!
The entries "Firstname", "Name" and "dd.mm.yy" in the screenshots above have been constituted by me for data protection reasons.
The different certificates for the same two vaccinations comes from different downloads (25.06.2021 (QR code on the left of the text, validity: "21.06.22 12:35 Uhr") and 31.07.2021 (QR code on the right side of text, validity: "30.07.22 12:01 Uhr")).
The reason why I scanned both the 'old' and the 'new' certificate is because the newer certificate has a completely different, significantly larger QR code; so, I think, it also contains more data.
(Even if it shouldn't be probable, I don't want to rule out that these two QR code versions of the same vaccinations lead to different results when checking the certificates.)

By the way, the described behaviour does not appear with CWA 2.6.1, so it must have been introduced after.

Right, that's how I see it too.

@mtwalli
I'm not @Jo-Achim , but can answer some questions.

In real cases no user will take more than one dose in the same day, is that your case ?

There are many users with 4 vacc certs: 1 pair 1/2 + 2/2 issued by pharmacy in mid-June (DCC schema version 1.0.0) and 1 pair 1/2 + 2/2 issued by federal government/download portal from July on (DCC schema version 1.3.0). The vaccination dates for pairs with schema 1.0.0/1.3.0 will be the same then. See also: https://github.com/corona-warn-app/cwa-app-android/issues/3838
A use case that could make it necessary to import the newer certs are business rules: Like the Netherlands published business rules for schema 1.3.0 some weeks ago, validation with vacc certs with schema 1.0.0 resulted in status 'open'. While NL downgraded their rules meanwhile, a similar problem pops up with France currently, which may force users to import 'freshly issued' certs additionally.

I assume that these certificates some of them have the same vaccinatedOn date which Vaccination certificates are sorted by it.

I have 2 vacc certs 2/2 with same vaccination date imported (schemas 1.0.0 and 1.3.0), but in CWA 2.6.1 the described issue is not present.

@vaubaehn, thank you! I couldn't have gotten to the point better.

Okay , Thanks for the clarifications, since you mentioned that 2.6.0 did not have this behaviour. I think 🧐 this behaviour was introduced in 2.7.0 where certificate priority  is  also affected by Validity State. We will investigate it further, thanks for your help!

Wouldn't the “Technisches Ablaufdatum” ("Technical expiry date") be helpful for sorting multiple vaccination certificates from one person?
It could ensure that the most current certificate (with the latest expiration date) is used by default and is displayed at the top. (So in my example above: 30.07.22.)
This default could be restored when the CWA is started. This would give the option of selecting a different certificate during a CWA session, which will be used until the CWA is closed.

This problem also exists in CWA 2.10.1.

@dsarkar could you please link the internal ticket of this issue ?

Internal Tracking ID: EXPOSUREAPP-8756
[Android] Latest issued Vaccination DCC 2/2 does not have highest priority

"Latest issued Vaccination DCC 2/2 does not have highest priority" is logged as https://github.com/corona-warn-app/cwa-app-android/issues/3838.

PR: https://github.com/corona-warn-app/cwa-app-android/pull/4123

@Jo-Achim
I can reproduce your issue with CWA 2.10.1 on Android 8. The certificate with the label "Currently used certificate" moves up and down in the overview when selected and when there is more than one certificate for the same vaccination date. The label "Currently used certificate" is not automatically applied to the certificate with the latest issue date.
The latest build in development from the release/2.11.x branch contains the fix from PR https://github.com/corona-warn-app/cwa-app-android/pull/4123 "Latest issued Vaccination DCC 2/2 does not have highest priority (EXPOSUREAPP-8756)".
I tried out the release/2.11.x branch (commit https://github.com/corona-warn-app/cwa-app-android/commit/292f97cc6bd56e3ca7b053bd14efbe9bb0d916d1).
With the release/2.11.x branch, the certificate which was issued latest is always labelled with the text "Currently used certificate", for instance the latest 2/2 certificate. So that fixes the issue https://github.com/corona-warn-app/cwa-app-android/issues/3838.
The behavior is still a bit strange though. If an older 2/2 certificate is shown at the top of the overview list and I tap on it, then return to the overview, the newest 2/2 has moved to the top of the list (with the correct text "Currently used certificate"). The older 2/2 certificate has moved down one place.
Now if I tap on the newest 2/2 certificate at the top of the overview list, then return to the overview, the newest certificate has moved downwards one place.
Perhaps @mtwalli could look at this to determine if there is still an open issue which needs to be addressed?

@MikeMcC399 Thanks for thorough analysis, forwarded to internal ticket.

@dsarkar

Thanks for thorough analysis, forwarded to internal ticket.

I added some screenshots to make it easier to understand visually.

@MikeMcC399 internal ticket updated with your screenshots

@MikeMcC399
These are two different issues PR #4123 fixed the priority issue.  PR https://github.com/corona-warn-app/cwa-app-android/pull/4136 should fix the re-ordering issue.
There are two different things happening in this list:
1- Finding the highest priority Cert
2- Ordering the rest of the list (TC, VC, VC)

@mtwalli

PR #4136 should fix the re-ordering issue.

I agree! Thank you! 👍🏻

@Jo-Achim
Do you use Android Studio? If yes, you could try out the fix yourself on a pre-release version.
It should be fixed in the next release 2.11.x! 🙂

@MikeMcC399,
thanks for info.
And no, I don’t use / work with Android Studio.

@Jo-Achim

no test possible so far, as version 2.11 has not yet arrived.

I am in the same position.
The release has been delayed due to a hotfix according to https://github.com/corona-warn-app/cwa-website/issues/1842#issuecomment-938474245.

CWA 2.11.2 has meanwhile arrived and, if I have tested correctly, the problem 'Trouble with “Actually used Certificate” if more than one vaccination certificate is available…' is solved.
It should be noted that this workaround for sorting the digital EU vaccination certificates no longer seems to work:

But there is a workaround to sort the EU certificates:
1. Delete the relevant certificates.
2. Scan the certificate that should be the standard certificate (1/2 and 2/2) first.
3. Scan the 'old' certificate(s).

Result: the EU certificate scanned first is shown as the 'Standard Certificate'.
This works under CWA 2.6.1 as well as under CovPass 1.28.7.


Incorrect documentation page title

Bug report
Incorrect documentation page title
Describe the bug
The documentation getting started page has "tailwind css" in it title. This is because the code docs website was focked from tailwind css documentation repo.
Steps to reproduce the behavior

Go to https://tenseijs.com/docs/getting-started
Check the page title

Expected behavior
Title surfix should be " - Open source Node.js Headless CMS"
Screenshots

@Ethical-Ralph thanks very much for taking the time to log this. If you have some time, we would really appreciate a pull request to the documentation repository to fix this issue. But this will definitely be addressed very soon. Thanks again !

@Ethical-Ralph thanks very much for taking the time to log this. If you have some time, we would really appreciate a pull request to the documentation repository to fix this issue. But this will definitely be addressed very soon. Thanks again !


Module fails with: "ModuleNotFoundError: No module named 'einops'"

Installed on Slicer-5.2.1, downloaded sample data set, downloaded model, clicked Apply - then disappointment :(
Traceback (most recent call last):
  File "C:/Users/andra/AppData/Local/NA-MIC/Slicer 5.2.1-monaitest/NA-MIC/Extensions-31317/MEMOS/lib/Slicer-5.2/qt-scripted-modules/MEMOS.py", line 290, in onApplySingleButton
    logic.runSingle(tempVolumeFile, self.modelPathSingle.currentPath, tempOutputPath)
  File "C:/Users/andra/AppData/Local/NA-MIC/Slicer 5.2.1-monaitest/NA-MIC/Extensions-31317/MEMOS/lib/Slicer-5.2/qt-scripted-modules/MEMOS.py", line 381, in runSingle
    import einops
ModuleNotFoundError: No module named 'einops'

I am not replicating this on my end with fresh installations. This is windows right?

Hi @lassoan can you let me know what happens when you manually install with slicer.util.pip_install('einops')

I've found a serious issue with how the module starts. The module calls installLibs in the module object constructor! This is a significant problem even when only minor packages are installed, but for extremely large packages like pytorch then it is essentially it prevents Slicer to start up for potentially tens of minutes (if the required pytorch version was not already installed and available in the pip download cache on the computer). The user of course does not know that Slicer is actually still starting up, he restarts Slicer, so now we have a Slicer running in the background trying to install a number of packages, while another Slicer instance is still running. The result is quite likely partially installated Python packages (which can cause the "no module named einops" error reported above).
To fix this:

only do trivial operations in the module class constructor (what is shown in the module template), absolutely no pip install (it would block Slicer application startup if there is no or slow network or installing large packages)
do not rely on the user reading documentation in advance and installing pytorch using PyTorch extension, but make PyTorch extension a dependency of MEMOS and use it to install pytorch
move installation of required library at the point of first use (e.g., when the user clicks Apply) - luckily in TotalSegmentator we have a robust example of how to install pytorch and other libraries (tested by almost 2000 people on Windows, 200 linux, 300 macos; we got only a handful of error reports, and we could resolve most of them). I would recommend to copy the implementation from here: https://github.com/lassoan/SlicerTotalSegmentator/blob/d9b4f6e7a2eae05d099836aa8d314a59df066262/TotalSegmentator/TotalSegmentator.py#L571-L626

Thanks @lassoan we were hesitant to add an unnecessary dependency, but I think this makes a lot of sense.

PyTorch extension is a very lightweight dependency, it just installs light-the-torch, a tiny Python package that can determine what pytorch version is compatible with the system and can install it.

Fixed with adding pytorach dependency


Add Cloud Foundry routes as a source

This is a proposal to add Routes from Cloud Foundry as an optional source.
Delivers a ubiquitous Developer Experience to use External DNS open source project for both Kubernetes as well as Cloud Foundry. This streamlines application publishing using single, automated  DNS management across multiple application platforms and multiple clouds. Currently, this functionality does not exist across multiple Cloud Foundry foundations (or AZs).
General idea here is that you would point external-dns to a Cloud Foundry foundation/endpoint and it would map all routes under all domains back to the DNS zone you would like to manage with external-dns. This allows for routes registered in Cloud Foundry to have their DNS managed in an external provider (Route 53, Google Cloud DNS, etc).

Thank you for your PR!
You may want to fix formatting first:
pkg/apis/externaldns/types.go:186:33:warning: missing ',' before newline in composite literal (deadcode)
pkg/apis/externaldns/types.go:186:33:warning: missing ',' before newline in composite literal (golint)
Additionally to that you have to make sure dep has all your new importing packages otherwise the build will fail.
After fixing we'll have a look :slightly_smiling_face:.

Thanks for taking a look @njuettner. I've fixed those 2 issues but getting stuck on something else with the build. It seems like it's not finding the new config options I've added even though they have been added to source/store.go. Looking at the version of the project that is pulled into the vendor directory during the build?
Let me know if I'm missing something. Thanks!

From the TravisCI output it looks like there is a missing method to implement in the test and other issues:
# github.com/kubernetes-incubator/external-dns/source [github.com/kubernetes-incubator/external-dns/source.test]
source/route.go:41:31: rs.client.Config undefined (type *cfclient.Client has no field or method Config)
source/store_test.go:64:25: cannot use mockClientGenerator (type *MockClientGenerator) as type ClientGenerator in argument to ByNames:
	*MockClientGenerator does not implement ClientGenerator (missing CloudFoundryClient method)
source/store_test.go:73:25: cannot use mockClientGenerator (type *MockClientGenerator) as type ClientGenerator in argument to ByNames:
	*MockClientGenerator does not implement ClientGenerator (missing CloudFoundryClient method)
source/store_test.go:83:25: cannot use mockClientGenerator (type *MockClientGenerator) as type ClientGenerator in argument to ByNames:
	*MockClientGenerator does not implement ClientGenerator (missing CloudFoundryClient method)
source/store_test.go:92:19: cannot use mockClientGenerator (type *MockClientGenerator) as type ClientGenerator in argument to ByNames:
	*MockClientGenerator does not implement ClientGenerator (missing CloudFoundryClient method)
source/store_test.go:95:18: cannot use mockClientGenerator (type *MockClientGenerator) as type ClientGenerator in argument to ByNames:
	*MockClientGenerator does not implement ClientGenerator (missing CloudFoundryClient method)
source/store_test.go:98:18: cannot use mockClientGenerator (type *MockClientGenerator) as type ClientGenerator in argument to ByNames:
	*MockClientGenerator does not implement ClientGenerator (missing CloudFoundryClient method)
source/store_test.go:107:19: cannot use mockClientGenerator (type *MockClientGenerator) as type ClientGenerator in argument to ByNames:
	*MockClientGenerator does not implement ClientGenerator (missing CloudFoundryClient method)
FAILED   /home/travis/gopath/src/github.com/kubernetes-incubator/external-dns/vendor/github.com/kubernetes/repo-infra/verify/go-tools/verify-govet.sh	31s
The command "vendor/github.com/kubernetes/repo-infra/verify/verify-go-src.sh -v --rootdir $(pwd)" exited with 1.

Do you think you can work at fixing those issues?

thanks @Raffo will work through these issues

Issues are fixed but still missing a few tests - will work on adding those

@dgrizzanti is this ready for review?

@Raffo yes - it should be all set. Only comment would be that I opened this PR more as a proposal for adding the feature. The code works, but I probably need to update the docs on how to use it/intention, etc. I wanted someone to at least give the 👍 on the approach prior putting together a full set of docs.
I'm fine with adding all of that after the fact in if you think I'm good but wanted to get through that checkpoint first.
Thanks!

@dgrizzanti we switched to go modules, could you please rebase from master?

@njuettner rebased and updated with go modules

/lgtm
/approve


114 | Flatten Binary Tree to Linked List | Python

Title -
What will change - Add solution to 114. Flatten Binary Tree to Linked List in Python.
Type of Issue -
Please add/delete options that are not relevant.

[x] Adding New Code

Programming Language
Please add/delete options that are not relevant.

[x] Python

Self Check

Ask for issue assignment and wait to get assigned to it before making Pull Request.
Add your file in the proper folder
Clean Code and Documentation for better readability
Add Title and Description of the program in the file

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Discord server ✉ - https://discord.gg/CXksGdn
Happy Coding,

Could you please assign me for this task ?


0.2.14 HLS Chrome problem

HLS stream is not being played in Chrome with HLS.js.
Plays well with Safari and FF. It loads segments and playlist file but results in black screen. No JS errors.

@AlexRevin can u give me this source url?

Have you got a link to the page with the issue? Will be easier to debug.
Out of those browsers the only one where hlsjs will actually be chosen is chrome.

Yeah, checkout http://lj.ee/broadcasts/bc-758b06c1100bcd843e3e for eample

When I disable mediasources api in Chrome it gracefully falls back to FlashHLS and works well then.

Yeh I have the same issue in chrome on windows. The links has stopped working for me now though.

same here, not working

I have updated the link, because the broadcasting stopped, those streams are not permanent, you know. You can try http://lj.ee/broadcasts/bc-09468478f78d292fc79e for eample, should be live for now.
Or from the frontpage enter any SopCase url and it should start broadcasting.

ok, it's not playing on hls.js demo page also (http://dailymotion.github.io/hls.js/demo/?src=http://rtmp.lj.ee:3800/hls/bc-09468478f78d292fc79e.m3u8, you'll need to run chrome with --disable-web-security to get rid of access control errors) so we should report this issue to @mangui. If you want to go back to flashls while we figure out what's going on, follow what is described on #607

@AlexRevin just wondering what's the segmenter you're using, is nginx-rtmp?

@flavioribeiro it won't be playing on their demo page as CORS headers are not set-up wide open I guess.

if u run chrome with --disable-web-security you'll not face cors related problems

Yeah, right. I've tried running the stream from their demo page with disable-web-security flags applied and it did not work too.
I am using http://arut.github.io/nginx-rtmp-module/ for providing HLS live streams so I shouldn't be the one with such problems I guess.
Unfortunately I cannot provide a permanent link for testing, as they are not permanent but it's obviously related to HLS.js

@AlexRevin Try testing directly on http://dailymotion.github.io/hls.js/demo/, if you still see problems, we should put @mangui on this loop

@flavioribeiro have just tested it, still not working. It is definitely a problem with hlsjs, I've tried getting into details and it seems like I'd better move the issue to their tracker.
Thanks for your help!

I have the same problem with a nginx-rtmp generated stream.

@cowai did you try to replicate this at http://dailymotion.github.io/hls.js/demo/ ? if yes, we should open this issue on hls.js
One thing you can do is force flahshls for a while:
plugins: {playback: Clappr.FlasHLS},

@leandromoreira
The figured out my problem. My manifest file didn't have the "CODECS=" directive. And I was using LC AAC codec for audio, and hls.js was automatically declaring he-aac since I did not have an CODECS directive. So after I recompiled ffmpeg with he-aac, it worked fine and at the same gave me much better sound on lower bitrate :)

thanks, good to know ;)

Thanks! Will give it a try.
However I came to conclusion, that using hls.js is a dead-end solution, encoding streams to HLS and MPEG-DASH for browsers demanding mp4.

@AlexRevin did you know clappr already supports dash? also, soon MSE will be supported in more browsers.

@AlexRevin You still need to support hls for iOS for the coming feature, don't you?

Guys, there's a #550 ticket which I am waiting to be closed. It is more efficient to transcode into two streams on the server rather than re-transcode on the client side.


Cards can only be moved when manual sorting is active

Hi,
In my tasks.md installation  I only can move cards when sorting is set to "Manually".

For all other options drag and drop seems to be disabled somehow. My understanding is, that with fixed sorting a move of cards between columns should still be possible, correct?
Thanks and BR

To be honest I didn't paid attention to this detail and simply disabled the card so you can't sort them. But you're right, it should be possible to move them between lanes. I'm planning to submit a new release this week, so I'll try to include the fix for this issue too

thanks for the quick reaction and this great piece of software, as well!

Implemented on #119. I'm going to publish the new release with the fix in a few minutes. If you encounter a problem related to the solution feel free to open the issue again


Widget border color based on device theme

It would be great if the color of the border of the photo widget would change according to the device theme.
White for light theme and black for dark.
Currently I have to change it everytime.
Also if we can set custom border radius  for the widget it would be awesome.. But that's just a suggestion.

There's some mandatory radius on modern Android, so I'm unlikely to play with that.
However, I'll take a look at providing the system dependent black/white. I wonder how reactive it would be though. By the way, on your device, does it switch between dark/light theme on schedule, or you often manually switch between the two?
And I just thought, maybe I could also provide a colour option that takes the dynamic material you colour. Not sure whether that would be used.

Yes I have set a schedule to change between dark/light mode. Or when battery saver mode is turned on it automatically turns dark mode on as well. Most of my widgets on homes screen follow system theme.
Dynamic colors is also a good idea. Some widgets on my home screen have some parts like texts with dynamic colors and remaining area follows light/dark theme.

Thanks alot Man


how to set remote postgreSQL server?

I'm trying to split history-tools server and DBMS server.
What should I do? Is there any config options in config.ini ?

Use the PG* environment variables. If you can get psql to work remotely using them, then fill-pg should also work.


Still seeing secure score

I am still seeing the secure score page, it has been 5 months, which is much longer than the few weeks mentioned above. Has something changed in the plans to deprecate that page?

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

ID: e17e8994-077f-4042-b478-edbbc23a5687
Version Independent ID: 847bfd6e-eaf0-acf3-d57e-f5c1fbcdad70
Content: Overview of Secure score in Microsoft Defender Security Center - Windows security
Content Source: windows/security/threat-protection/microsoft-defender-atp/overview-secure-score.md
Product: w10
Technology: windows
GitHub Login: @mjcaparas
Microsoft Alias: macapara

@DeanGross - Thank you for submitting feedback.
I will get this issue over to the Win10 ITPro writing team for investigation.
Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.

@mjcaparas - Can you please share your insights on this issue?
Thank you.
cc: @kenwith

@DulceMontemayor please see comments

@DeanGross The article has been updated.
The issue is being closed since your question seems to be answered. Feel free reach out again if you feel that it hasn't been answered or that there are further suggestions to improve the documentation itself.
Thanks.


make a better repr for pipelines

closed by https://github.com/otosense/lined/commit/48ffa301c9504c7a48d46ef0db323e710f0a27b9


Name clash in bower

There is a name clash between this package purescript-domparser and purescript-DOMParser which hasn't been touched for three years.  Unfortunately, the latter one is what bower install finds.
I'm not sure what the best solution is - possibilities I can think of are 1)negotiate removing purescript-DOMParser from the bower registry, 2)rename this package, and 3)negotiate making this module part of purescript-dom.  In the meantime, of course, a work-around for users is referencing this repo explicitly, for example in bower.json:
  "dependencies": {
    "purescript-domparser": "https://github.com/toastal/purescript-domparser.git#^1.0.0"
  }

Good catch. I'm not actually in control of any of these pieces of software for the registry, so I don't actually know what's best to do.

Rename purescript-domparser -> purescript-dom-parser maybe?

@8084 moved it in v4.0.0


Added changes created by user ARENOUX that allow nested objects are p…

…arsed as part of the payload.
Added changes to this parser so that the right zuora object types get the correct type definition, in this case ammendoptions and the action element.
Ensured that ammendment type of objects do not get a zObject element, as this is not allowed by Zuora's API.
This functionality is tested and works as expected.

Thanks Yousefcodes. I'm going to merge it in.
A small thing, but this request is using tabs, and the rest of the module uses 2-spaces. It would be great if you could switch that around, but I can merge it and fix it later too.

Hi! Thanks! Let me see if I can untabify it really quick

Done!


Rewrite all iri reference attribute ids

Handle all defs elements that have iri capable attributes as defined by
w3c in http://www.w3.org/TR/SVG/linking.html#processingIRI
Chrome also has problems with patterns, so why not just handle all
iri-reference-capable attributes instead of adding a third loop for just
patterns

Good idea! Thanks for the PR. Let me try this out and I'll get it merged.

Cool. Yeah, I had to make modifications to handle patterns and realized it was probably just better and easier to handle all of the possible IRI attributes in one go instead of adding additional loops every time someone had an issue with another attribute.


crates.io 0.3.3 missing?

As the title says, is they any reason 0.3.3 is not present on crates.io?
Not a big problem but just found out this when updating deps in my project and prost-wkt stopped working with prost 0.11.
Switched for now to git with patch.crates.io

It has not been released yet, but it will be in the next few days!

Ah, thanks! Wrongly assumed it is just a mistake. Looking forward to it :)


Add Lean code for computing the transitive closure

This needs some discussion before it can be merged
Closes #29

awaiting-review


Alternative key bindings for up,down,esc

I added alternative key bindings for Up, Down, Esc
Up: Ctrl-k
Down: Ctrl-j
Esc: Ctrl-c
I find these bindings more intuitive than the current ones, when you have a very keyboard focused workflow. The suggested bindings allows your hands to stay in the center of the keyboard while working.

LGTM


"c" icon next to the checkmark on an account

I'm confused by the "c" icon next to the checkmark on some accounts.  The checkmark means the automatic import is working, but what does the "c" mean?
YNAB support was stumped.  I turned off the toolkit, and the "c" icons went away, so I think the answer is in the toolkit somewhere.
I looked over all the account options a few times, but nothing mentions the "c" icon.
If you hover over an account with a checkmark, and an account with a checkmark and "c" icon, you get the same result.
Transaction imports: Good
Last checked 2 hours ago
Learn More button
Any ideas?

I believe that is the "Uncleared Account Highlight" feature, which:

Adds a small indicator next to account balances on the sidebar to indicate not all transactions are cleared.

The setting can be set from the General tab of the extension Settings.

That was it!  I didn't expect the setting to be in the general tab :-).
Since I always have a mix of cleared/uncleared transactions, there is no
point for the indicator.  I turned it off, and it looks all "normal"
again.  Thank you!
On Sat, Apr 10, 2021 at 9:43 PM Stephen Brown II @.***>
wrote:

I believe that is the "Uncleared Account Highlight" feature, which:
Adds a small indicator next to account balances on the sidebar to indicate
not all transactions are cleared.
The setting can be set from the General tab of the extension Settings.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/toolkit-for-ynab/toolkit-for-ynab/issues/2357#issuecomment-817246570,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADETQFZLNJSAPXHUVBBAEH3TIESGNANCNFSM42XFTTRQ
.

I really like the "c" feature since I used cleared/uncleared to balance my credit card accounts on a monthly basis. However, in Chrome browser the "c" appears on the left hand side of the YNAB table, which I found extremely helpful. But in Firefox browser, it appears on the right side of the YNAB table. I do not like the location on the right side because it is too difficult to see.
What accounts for the different locations in the different browsers?

@Chancycat Sounds to me like you haven't got the same settings turned on/off for the two browsers. The easy way to fix it is to export the settings from Chrome and then import them into Firefox. To access the export/import button, open the Settings page then click on the Import/Export Settings button located in the lower lefthand corner of the page. Right-click the highlighted text and select Copy in the pop-up menu. Then do much of the same in Firefox but instead of right-clicking the highlighted text hit the backspace key then paste the settings into the box you just cleared. See there, easy peasy. 😎

Thank you!

On Apr 23, 2021, at 7:27 AM, David Baldon @.***> wrote:

@Chancycat Sounds to me liThankyuke you haven't got the same settings turned on/off for the two browsers. The easy way to fix it is to export the settings from Chrome and then import them into Firefox. To access the export/import button, open the Settings page then click on the Import/Export Settings button located in the lower lefthand corner of the page. Right-click the highlighted text and select Copy in the pop-up menu. Then do much of the same in Firefox but instead of right-clicking the highlighted text hit the backspace key then paste the settings into the box you just cleared. See there, easy peasy. 😎
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Pagination next button does not work

Pagination Next button is not working.
Also, Instead of 'start' and 'end' parameters. Should be 'height' and 'rows', with a sane max row return. Probably 2000 since that is what we use elsewhere. Make default 12 ( 1 hours worth ).

Completed in https://github.com/dcrdata/dcrdata/pull/128/


[vcpkg_configure_make] Fix z_prepend_include_path when var contains colon

vcpkg_host_path_list(PREPEND) will report error when compilation-related environment variables contain colon separators in UNIX:
Host path separator (:) in path; this is unsupported.

This is due to the fact that z_vcpkg_env_backup_${var} uses string as a backup and doesn't take this case.
So we have to preprocess these variables to call vcpkg_host_path_list correctly before using backup.
Fixes #24138.

cc @BillyONeal @Neumann-A @dg0yt

Already checked other scripts, only use backup here.

When can this get merged?

With the macro now strongly simplified, I no longer see why we just don't use vcpkg_host_path_list instead. That function operates on env vars, handles the originally-empty state, and uses VCPKG_HOST_PATH_SEPARATOR. And it is unit tested.

With the macro now strongly simplified, I no longer see why we just don't use vcpkg_host_path_list instead. That function operates on env vars, handles the originally-empty state, and uses VCPKG_HOST_PATH_SEPARATOR. And it is unit tested.

Because variables that require repend may contain colons as separators that vcpkg_host_path_list does not support.

With the macro now strongly simplified, I no longer see why we just don't use vcpkg_host_path_list instead. That function operates on env vars, handles the originally-empty state, and uses VCPKG_HOST_PATH_SEPARATOR. And it is unit tested.

Because variables that require repend may contain colons as separators that vcpkg_host_path_list does not support.

No. Supporting colons for VCPKG_HOST_PATH_SEPARATOR is the whole point of vcpkg_host_path_list. The trigger for https://github.com/microsoft/vcpkg/issues/24138 is that z_prepend_include_path was using it wrongly in a host-unaware foreach loop over a cmake variable.
More on expected behaviour with : separator:
https://github.com/microsoft/vcpkg/blob/8f1d60f2014729c80d1c7a95947c30cdfd7aa4ea/scripts/test_ports/unit-test-cmake/test-vcpkg_host_path_list.cmake#L68-L125
So tell me why replacing
z_prepend_include_path(INCLUDE)

with
vcpkg_host_path_list(PREPEND ENV{INCLUDE} "${CURRENT_INSTALLED_DIR}/include")

wouldn't work, given the macro's implementation in this PR.

@dg0yt You can prepend the value contains :, but the var shouldn't contains ::
set(var "c:d")
vcpkg_host_path_list(PREPEND var "a:b")

This will report error:
Host path separator (:) in path; this is unsupported.

here:
https://github.com/microsoft/vcpkg/blob/8f1d60f2014729c80d1c7a95947c30cdfd7aa4ea/scripts/cmake/vcpkg_host_path_list.cmake#L43
This is becuase:
https://github.com/microsoft/vcpkg/blob/8f1d60f2014729c80d1c7a95947c30cdfd7aa4ea/scripts/cmake/vcpkg_host_path_list.cmake#L40
lst is the origin var, it contains :.

@JackBoosY Your usage is wrong. vcpkg_host_path_list expects a cmake list of paths (separated by the cmake list operator ;) not a string containing : which would be a list of paths for environment variables of the host.

@Neumann-A Yes I agree with you.
However, the user's system may have those environment which contains :, and they will be carried into here.
See https://github.com/microsoft/vcpkg/issues/24138#issuecomment-1122319595.
I discussed with @BillyONeal , instead of preprocessing this separator before using vcpkg_host_path_list, use set directly.

You can prepend the value contains :, but the var shouldn't contains ::
set(var "c:d")
vcpkg_host_path_list(PREPEND var "a:b")

This will report error:
Host path separator (:) in path; this is unsupported.

I'm already point at the unit test. What else can I do?
set(VCPKG_HOST_PATH_SEPARATOR ":")
set(ENV{var} "a:b")
unit_test_check_ENV{var}iable_equal(
     [[vcpkg_host_path_list(PREPEND ENV{var} d e)]]
     ENV{var} "d:e:a:b"
 )
unset(ENV{var})
unit_test_check_ENV{var}iable_equal(
    [[vcpkg_host_path_list(PREPEND ENV{var} d e)]]
    ENV{var} "d:e"
)

However, the user's system may have those environment which contains :, and they will be carried into here.
See #24138 (comment).

Again, there were carried by the original foreach loop over ${z_vcpkg_env_backup_${var}}. This circumvented the correct handling which vcpkg_host_path_list implements for ENV variables. (Baseline: VCPKG_HOST_PATH_SEPARATOR is used for environment variables, not for cmake variables).

I'm already point at the unit test. What else can I do?
set(VCPKG_HOST_PATH_SEPARATOR ":")
set(ENV{var} "a:b")
unit_test_check_ENV{var}iable_equal(
     [[vcpkg_host_path_list(PREPEND ENV{var} d e)]]
     ENV{var} "d:e:a:b"
 )
unset(ENV{var})
unit_test_check_ENV{var}iable_equal(
    [[vcpkg_host_path_list(PREPEND ENV{var} d e)]]
    ENV{var} "d:e"
)

I can't believe our pipeline passed this test because I tested this code locally, it failed.

I can't believe our pipeline passed this test because I tested this code locally, it failed.

Uff. It is not run because the feature is published as host-path-list but the if clause uses add-to-path. :cry:
Now I am no longer surprised that unit_test_check_ENV{var}iable_equal syntax didn't fail.
CC @strega-nil-ms

https://github.com/microsoft/vcpkg/blob/2f0872b7d592635e8e4919d178650d549c211464/scripts/test_ports/unit-test-cmake/vcpkg.json#L12
https://github.com/microsoft/vcpkg/blob/2f0872b7d592635e8e4919d178650d549c211464/scripts/test_ports/unit-test-cmake/portfile.cmake#L142-L144
Should be
 if("host-path-list" IN_LIST FEATURES)

After some exploration, I don't think this PR addresses the intended issue. vcpkg_host_path_list should have dealt with the problem that the environment variable in the first parameter contains a colon, and the original issue should not be caused by this problem.


Opening another socket or listening to inbound external calls outside Discord

Is it possible to do something like this:
https://blog.wyrihaximus.net/2015/03/reactphp-sockets/
My scenario:
I want a REST API or a communication possibility with the bot outside of Discord. So I can send a command from an external server to the bot either using a socket or through HTTP.
Because I assume you cannot send a message with an external headless server client through Discord itself to the bot.
But since DiscordPHP has abstracted ReactPHP I'm not sure how to open another socket connection and listen to connections and read the payload coming through.
Any tips?

I think you have to open the socket from inside discord and then check it if a change comes through.
My solution to that Problem is pretty dirty but I have a loop that checks a mysql Database every couple of minutes - if i want to send a command to the bot i put it in the mysql DB and the Bot can then do what I want.

@Omegatcu I use a message queue for that. But I wan't to open another socket and point Nginx to it.

@Zyles would you happen to have an example of how you're using a message queue with this?  I've been trying to get this to read from a queue and post messages in the chat for a while, but I've only managed to get it to post once.  It'll loop just fine, but can only post in the channel once and then stops sending...

@GameCharmer Depends what your framework looks like. I have mine in classes.
        // How often to sleep between checks (seconds)
        $interval = 60;

        $this->discord->loop->addPeriodicTimer($interval, function () use (Discord $discord) {
            	// Check MQ for messages
            	$this->checkMessages();
        });

@Zyles You're a saint! Thank you!

Check out ReactPHP HTTP server. Will interact with the event loop. Closing due to age.


parse Dockerfiles for _env(x) to get substituted with parent environment variables or with the --env switch

I'm interested in understanding why you didn't implement this in the end – would you mind sharing some insight into this? Was it because you implemented support for secrets coming from env vars, or some other reason?

Hi @timriley! Thanks for the feedback. Because of security reasons we decided to go for the secret service. If we substitute host environment variables this can be potentially dangerous. This was a feature we wanted to use in our Buildgrid buildsystems over at Cloud 66, but the secrets coming from the secret service giving us more control. If you have a good use-case, please share. We can always reopen the issue if more people want this ;-)


KGQA code

Thanks. For question answering, would you like to share the data processing process and the code for fine-tuning and inference of the QA pairs?

Hi, thanks for your interest. I will be cleaning and adding the code soon, however meanwhile please check out issue #11 for a temporary solution for QA finetuning. Please let me know if you face any issues.


Add emoji support

Add emoji support.
I already have something really specific in mind (supporting short codes as well as supporting custom emojis, like :FooBar.png:), but I won't dive into it too lengthy since I'm going to implement it soon anyways.

test: :thumbsup:
Turns out GitHub supports not only the official list of short codes, but their aliases as well.

This is implemented now; you can update to version v1.17.0 to access this feature (e.g. by doing pip3 install --upgrade gh-md-to-html>=1.17.0).


fix serialization of a string that looks like a valid YAML float

refs gh-156
(but isn't a valid ruby float)

after massaging a bit more, this actually fixes parsing of valid YAML floats that look like this (it's a variant of what is already handled of no fractional piece, but with no exponent)


Jump to latest message

Please do all this work in a branch called 'issue-013.'
Currently the chat messages populate downwards, but the at a certain point, the user needs to start scrolling down to see responses from the AI. Add a function to let the page jump to the latest AI message.

Issue resolved in branch issue-013.


Updated README.md File

Description
As assigned, I have made the README.md file in a more captivating way.
Checklist

[x] I am making a proper pull request, not a spam.
[x] I've checked the issue list before deciding what to submit.

Related Issues or Pull Requests
#96
Add relevant screenshot or video (if any)

Hey @nilisha-jais ,
Kindly Review my PR as soon as possible !

Hey @niloysikdar ,
Kindly Review my PR !


Implement leave-one-out CV on polar volumes/file

One way of implementing this effectively would be to add 'pvol_id' attribute to RadarDataset. Then we can loop over polar volumes and leave one out each time. What do yout think?

I will work on moving it from the other repo on top of #96


A better API for handling input that has already been parsed

The Naga crate uses hexf_parse to handle hex float literals in WGSL. Our front end has already verified that the text conforms to WGSL's grammar for hex float literals, but since that grammar has subtle differences from this library's grammar, we have to use format! to reassemble the pieces into what parse_hexf64 expects.
It'd be nicer if this library could accept a struct like this:
struct HexFloatParts<'a> {
    negative: bool,
    integer: Option<&'a str>,
    fraction: Option<&'a str>,
    exponent: Option<&a str>,
}

Do you think splitting parse to 4 sub functions can be a proper approach?
Any suggestion or contribution will be welcome

@youknowone What would you think of having parse produce a type like this?
/// The parsed form of a floating-point number.
#[derive(Debug, Eq, PartialEq)]
pub struct Parsed {
    negative: bool,
    integral: u64,
    fractional: u64,
    num_fractional_digits: isize,
    exponent: isize,
}

Adapting the tests is quite a lot of not-so-interesting work, so I thought I'd check here before proceeding.

See #23 for what this might look like.

There is this PR https://github.com/aldanor/fast-float-rust/pull/25 for fast-float-rust (which got its internals into rust's std lib) that might be a source of inspiration API wise. They seem to be going for:
pub fn parse_from_parts<T: FastFloat, S: AsRef<[u8]>>(integral: S, fractional: S, exponent: i64, negative: bool) -> T;

I'm not sure what's best here (for instance why is the exponent an i64 instead of another S) but so far the Parsed struct approach feels like it's leaking some internals (for instance users of the API would have to handle errors that hexf was previously handling).
What do you guys think about the following signature (it's similar to the initial proposal without the Options since we can just pass in empty &strs)?
pub fn parse_from_parts_hexf64(negative: bool, integral: &str, fractional: &str, exponent: &str) -> f64;

As to the suggested signature, here's why I moved away from &str to u64:
It seems desirable that this new API should be suitable for use by the hexf_parse crate itself, to connect its parsing and float-assembly steps. If the API isn't good enough for hexf_parse itself to use, then it won't be good enough for some its clients.
The thing about passing &str is that it means you need to first parse out the strings of digits - possibly making a copy in a temporary buffer if you want to skip separators - and then make a pass over them again to find their value. It's very easy to accumulate the bits as you parse the digits.

users of the API would have to handle errors that hexf was previously handling

This is a fair point. Not optimal.
But as long as the function that consumes Parsed structures fully validates the values received there, then the only error checking the caller actually needs to do is making sure it doesn't drop bits. It looks to me like the error checks in hexf_parse::parse fall into two categories:

syntax errors, which parsing front ends want to handle themselves - that's the whole point, we have our own lexer with its own rules

bit loss errors, which are not that hard to catch.

What I had in mind is extracting parts of the original parse function that could run on substrings of the original input; then the original parse function and also the new parse_from_parts function could use those (this would remove the need of having to go through each character more than once). I briefly looked at the code and this seems quite doable.
Regarding the separator, if we'd like to add that as part of the parse_from_parts, it could be a new Option<char> arg.
@jimblandy @youknowone what do you think?


Feature Request: Client Grouping

On the "clients" screen of the uchiwa frontend, having an option in the "Actions" menu to "Group clients" is highly desired. My organization wants to switch to Sensu (from Zabbix), but won't do it if every single client machine and name clutters the screen. We have almost 1000 client machines we are monitoring. They'd rather group all machines of one product or domain to make things cleaner to look at.
The way I would see this working, on the "clients" page:

select checkboxes of clients (via filtering or manually) to be included in the group
select "actions" (dropdown menu) -> "group clients"
a small box asking for the group name would pop up, you type the name and click "ok"
those clients "fold" into the group
the group would unfold/fold on click and display all of the same information about clients as normal
the group would display the same warning level as the most critical warning in the group
there would also be a "remove group" action, which would return things to the way they were

I have been hacking away at this, attempting to accomplish it, but I have close to zero web development experience so decided to post here. If you don't like the idea of this feature maybe you could respond with some hints/ideas on how I could accomplish it.

The problem I see here is that it would require some kind of backend storage in order to have persistency and the ability to share these groups.
I feel like the grouping should be done at the Sensu level with subscriptions and these could be displayed on a new view which shows the status of each subscription based on their clients status.

I've long wanted sensu to have a services group feature, which would enable both this front-end feature and a couple of other things: subscription groups and cleaner aggregates in particular.
But it seems to me that the notion of a service is strictly different to subscriptions: for example our database servers are all subscribed to both "systems" and "database", but almost everything is subscribed to "systems" so that's not a particularly useful group to surface.
Additionally, having a service notion would allow you to aggregate checks from multiple sources -- including jit sources -- into a nice view.

The idea that palourde presented would be sufficient in solving this problem, I think. All we'd have to do is have a separate view where the clients are folded into groups based on subscriptions. If the subscription name was the title of the group, then my managers would probably be happy.
Honestly... now that think about it, why not just do this with the current clients view? Instead of seeing a list of (~1000 for me) clients that I have to filter through, why not just have clients folded into groups based on what subscriptions they fall under?
Yes, some clients would fall under multiple subscription groups and sometimes those groups would not be very useful to us (like quodlibetor's "systems" example), but we could simply use the "client.json" of each client to put them into groups that are useful to us. It removes the need for backend storage, as palourde suggested. Even if no checks point at a group, we could still use made-up subscriptions to make uchiwa more user friendly at large scale.
Finally, if we want to solve the "systems" problem quodlibetor mentioned instead of ignoring it, we could put an optional field in the "uchiwa.json" called "invisible_subscriptions" (or something similar) that causes uchiwa to not present the listed groups on the clients page. This would mean that any machines that fell only under an invisible group show up not grouped at all. No code change would be needed on the sensu side for this.
I can think of another solution, but it would require a small change in the way sensu parses the client.json "subscriptions" field. Just add a "-" like so:

"subscriptions": [ "cat", "-dog"]
Sensu would have to parse "-dog" and "dog" the same (so checks with subscribers: dog will get run on them) , but "-dog" would force uchiwa to not put that client into that group when displaying it on the clients page. This would probably be a neater solution in terms of how things turn out, but might require more work to accomplish.

Solving that problem isn't really mandatory, though, it just cleans up things a bit further. Even in Zabbix we currently have groups that are divided into subgroups, and we rarely care about seeing the entire large groups as a whole. If the default state of these groups was "folded" on the uchiwa clients screen and the little green "OK!" bar shows in front of them all, management is generally happy.

Honestly... now that think about it, why not just do this with the current clients view? Instead of seeing a list of (~1000 for me) clients that I have to filter through, why not just have clients folded into groups based on what subscriptions they fall under?

It doesn't currently do the fancy collapsing that you recommend, but there is a subscriptions filter in Uchiwa on the clients page. At least in 0.9+, I'm not sure when it was introduced. It's pretty helpful.

Yeah, I mentioned the filter to my management, and they said it wasn't sufficient.
For me, the way that sensu/uchiwa does this now is perfectly fine. I rather prefer seeing the empty "Events" screen with no events if there are no problems.
For them, they want a dashboard to go look at that has product names (groups) that combines all of that status of it's child machines with it's own status marker next to it. That way, when they get a customer calling about the status of a product we offer, they can just see the product and answer the question. They don't want to have to do any filtering or ...thinking... to get the UI the way "it is supposed to be" (which I realize is just an aftereffect of having used Zabbix for a long time). Plus, if a product ends up being hosted on more machines than the current screen holds they either have to increase the quantity per screen or filter further... etc.
I know there are ways around it, but doing something like this would quite drastically clean/organize the uchiwa UI clients page, and that should be reason enough.

Just to update, I have made significant headway on this in a testing environment.

Having no web development experience before this is causing the project to move slower than I'd wish.
How I did it: I basically just used the bootstrap ui accordion that was already included in the code to make the accordion, and added a single function in controllers.js that updates the overall subscription status based on children statuses.
I'm going to clean up the CSS and behavior of the page a bit and try to figure out how to make this a toggle-able option (in the options menu, maybe). Any thoughts or recommendations?

@traviswdev Thanks for your PR. Unfortunately I won't be able to look at it before the end of August since I'm on vacation for the next weeks.
I feel like it should be enabled/disabled with a toggle or moved to a dedicated view since it might not easily scale with hundred and hundred of clients but again I only saw the screenshots and didn't actually played with it.
@amdprophet, if you have some free time and you are interested, feel free to go ahead with this PR!

I would also love something like this. Since some of us don't use subscriptions and Sensu supports arbitrary metadata it'd be nice to be able to pre-define keys to use as groups in the Uchiwa config.

I updated my pull request to reflect the requested changes. The "Groups" view now exists in my fork and is below the clients view. My pull request (with pictures and more detailed info) is at https://github.com/sensu/uchiwa-web/pull/60

Hey, are there any plans to progress this forward?
I think it'd be great to be able to provide Uchiwa with a configuration option to specify an attribute to group clients by; with the default being no grouping.
We use an arbitrary attribute that'd we'd love to group clients by.

I'm :+1: to add the ability to do this, I am :-1: on requiring an external persistent store dependency. Can this be some new metadata field we provide alongside subscriptions in the config?

+1


Cant start service on RHEL based system

Good day can you create a counter part script of probe.tpl for RHEL based system?

Thanks for your report, currently we don't have RHEL in our test stack but we are looking into it. PRs are welcome though :P


Fix MiqWorker service_base_name to match unit file prefix

After we renamed the systemd services to have "manageiq-" and "manageiq-providers-" prefixes the service_base_name wasn't updated to match the new names.
Also adds a spec test to ensure all workers have systemd unit files and that there are no extra unit files without workers.

In splitting up https://github.com/ManageIQ/manageiq/pull/20983 and https://github.com/ManageIQ/manageiq/pull/21120 the commit that did this service file rename got dropped in the shuffle


Added function to calculate the raw delta (delta without any extrapol…

…ation) for gauge and counter time series

Thanks for your PR. Functions like these have been extensively discussed previously, and I have no desire to have multiple subtlety different functions that will confuse users and this functions will also underestimate the true values. If you have more complex use cases you can build your own bespoke calculations using idelta and offset.

Thanks for your quick answer. Excuse me, but I guess I missed the discussion on this topic ;)


About some performance comparison

Hi, thank you very much for sharing your great work.
It is greatly interesting!
I have a bit question about Table 1 and 2.
It seems that MolCLR for MUV (in Table 1) and GLOVER for physical chemistry (in Table 2) would possibly be better when I read the original papers.
I might misunderstand, but is there some difference in experimental conditions from the original ones?
I want to understand a little more precisely how each method performs in the same conditions.
Thank you in advance.

Thank you for your interest in our paper. We would like to provide further clarification regarding the results presented in Tables 1 and 2.
All the results reported in the two tables were obtained by independently running the official code provided by each paper three times and then averaging the results. Specifically, we used the MolCLR code from https://github.com/yuyangw/MolCLR and the GROVER code from https://github.com/tencent-ailab/grover to ensure that we followed the same setting as the original papers.
However, it appears that the reproduced results are indeed lower than those reported in the original papers.
Thank you again for your interest in our work, and please let me know if you have any further questions or concerns.

@ZJU-Fangyin
Thank you for your kind reply. I now understand the experimental condition.
I can see full of possibility for the future in your research.
Have a nice day!

Thank you for your positive feedback and interest in our research. Have a great day too! ☺


Build failure: noto-fonts-color-emoji (python311Packages.brotlipy)

Steps To Reproduce
Steps to reproduce the behavior:

sudo nixos-rebuild switch --flake .#hostname with pkgs = import nixpkgs {}; in the config (config used)

Build log
error: builder for '/nix/store/zmc1rwraqz5yk8h59c2rkm2bsyi00drn-python3.11-brotlipy-0.7.0.drv' failed with exit code 1;
       last 10 log lines:
       > creating build/temp.linux-x86_64-cpython-311/build/temp.linux-x86_64-cpython-311
       > gcc -Wsign-compare -DNDEBUG -g -fwrapv -O3 -Wall -I/nix/store/i5lrswq5mg2i2aljpw35sma87idq3d7s-libxcrypt-4.4.36/include -fPIC -Ilibbrotli -Ilibbrotli/include -I/nix/store/slyq4a5c0vrng83y8ng4kq2pa6x85np0-python3-3.11.9/include/python3.11 -c build/temp.linux-x86_64-cpython-311/_brotli.c -o build/temp.linux-x86_64-cpython-311/build/temp.linux-x86_64-cpython-311/_brotli.o
       > creating build/lib.linux-x86_64-cpython-311
       > creating build/lib.linux-x86_64-cpython-311/brotli
       > gcc -shared -L/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-bzip2-1.0.8/lib -L/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-expat-2.6.2/lib -L/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-libffi-3.4.6/lib -L/nix/store/i5lrswq5mg2i2aljpw35sma87idq3d7s-libxcrypt-4.4.36/lib -L/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-mpdecimal-4.0.0/lib -L/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-ncurses-6.4/lib -L/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-openssl-3.0.14/lib -L/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-sqlite-3.45.3/lib -L/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-xz-5.4.7/lib -L/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-zlib-1.3.1/lib -L/nix/store/k8mqmpby1i5i50vayix3gc5qzi06y9ma-tzdata-2024a/lib -L/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-gdbm-1.23/lib -L/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-readline-8.2p10/lib -L/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-bzip2-1.0.8/lib -L/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-expat-2.6.2/lib -L/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-libffi-3.4.6/lib -L/nix/store/i5lrswq5mg2i2aljpw35sma87idq3d7s-libxcrypt-4.4.36/lib -L/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-mpdecimal-4.0.0/lib -L/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-ncurses-6.4/lib -L/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-openssl-3.0.14/lib -L/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-sqlite-3.45.3/lib -L/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-xz-5.4.7/lib -L/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-zlib-1.3.1/lib -L/nix/store/k8mqmpby1i5i50vayix3gc5qzi06y9ma-tzdata-2024a/lib -L/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-gdbm-1.23/lib -L/nix/store/eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee-readline-8.2p10/lib build/temp.linux-x86_64-cpython-311/build/temp.linux-x86_64-cpython-311/_brotli.o -L/nix/store/slyq4a5c0vrng83y8ng4kq2pa6x85np0-python3-3.11.9/lib -Lbuild/temp.linux-x86_64-cpython-311 -llibbrotli -lstdc++ -llibbrotli -o build/lib.linux-x86_64-cpython-311/brotli/_brotli.abi3.so
       > /nix/store/bfhcdsrgpq5qsifizkfw7q76hhcjpqn2-binutils-2.41/bin/ld: cannot find -llibbrotli: No such file or directory
       > /nix/store/bfhcdsrgpq5qsifizkfw7q76hhcjpqn2-binutils-2.41/bin/ld: cannot find -llibbrotli: No such file or directory
       > collect2: error: ld returned 1 exit status
       > error: command '/nix/store/j1bhwjzi56w6v81q1rbvrk5b4kysknd8-gcc-wrapper-13.2.0/bin/gcc' failed with exit code 1
       > /nix/store/m165kv9lrqjdrbxc01abid0rrfjgc12g-stdenv-linux/setup: line 1579: pop_var_context: head of shell_variables not a function context
       For full logs, run 'nix log /nix/store/zmc1rwraqz5yk8h59c2rkm2bsyi00drn-python3.11-brotlipy-0.7.0.drv'.
error: 1 dependencies of derivation '/nix/store/hrnpij7c9cb2vm2b6k549ld65ni5rxw0-python3-3.11.9-env.drv' failed to build
error: 1 dependencies of derivation '/nix/store/im7nfg1c61vsbq05w3spyqrbsixvy142-noto-fonts-color-emoji-2.042.drv' failed to build
error: 1 dependencies of derivation '/nix/store/nrczl8x60kglvzjxqx5d67q9jq702q1g-fc-00-nixos-cache.conf.drv' failed to build
error: 1 dependencies of derivation '/nix/store/digzfbssd8lbqazxm84lbfhgds6b5zfs-fontconfig-conf.drv' failed to build
error: 1 dependencies of derivation '/nix/store/07jgh1nky3cjfs2hzl3c041zifap6f5c-fontconfig-etc.drv' failed to build
error (ignored): error: cannot unlink '/tmp/nix-build-tuigreet-0.9.0.drv-1/build/source/target/x86_64-unknown-linux-gnu/release/deps': Directory not empty
error: 1 dependencies of derivation '/nix/store/g88zfdsm3bgc242q6j6a0x911dl9sqvp-etc.drv' failed to build
error (ignored): error: cannot unlink '/tmp/nix-build-cargo-c-0.9.31.drv-1/build/cargo-c-0.9.31+cargo-0.78.0.tar.gz/target': Directory not empty
error (ignored): error: cannot unlink '/tmp/nix-build-spidermonkey-115.9.1.drv-1/build/firefox-115.9.1/gfx': Directory not empty
error (ignored): error: cannot unlink '/tmp/nix-build-rustc-1.77.2.drv-1/build/rustc-1.77.2-src/vendor/tikv-jemalloc-sys': Directory not empty
error (ignored): error: cannot unlink '/tmp/nix-build-clang-17.0.6.drv-2/build/clang-src-17.0.6/clang': Directory not empty
error (ignored): error: cannot unlink '/tmp/nix-build-rustfmt-nightly-1.77.2.drv-1/build/rustc-1.77.2-src/vendor/icu_locid': Directory not empty
error: 1 dependencies of derivation '/nix/store/q29ln3lpfbhs03dmf9m6hgk0r5gh8w1j-nixos-system-NxNORTH-24.05.20240719.0c53b6b.drv' failed to build

Additional context
noto-fonts-color-emoji is defined in fontconfig-default-fonts.nix (?) by default, so I cannot build my system right now at all as far as I know. nix-shell -p noto-fonts-color-emoji does work.
Notify maintainers
@nbraud
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
 - system: `"x86_64-linux"`
 - host os: `Linux 6.6.41, NixOS, 24.05 (Uakari), 24.05.20240719.0c53b6b`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.5`
 - nixpkgs: `/nix/store/qszplw617r895nbcprgyj139c9a3r0xs-source`

Add a :+1: reaction to issues you find important.

Can't repro on 0c53b6b8c2a3e46c68e04417e247bba660689c9d, you sure you don't have some override or overlay in your config?

Can't repro on 0c53b6b8c2a3e46c68e04417e247bba660689c9d, you sure you don't have some override or overlay in your config?

I do not. But something seems to be wrong with that host anyway, since the same config works on a different machine.
Also everything is compiled from source on that host even though I disabled enableParallelBuildingByDefault. It still does not use Hydras cache from cache.nixos.org
What could cause this?
Since this is obviously not a code issue, you may close this issue (or should I do it?)

I deleted the whole nix cache and It started using cache.nixos.org again :+1:


Implement some indicator of the version of sqld used by turso dev

Today it's not easy to know what version of sqld is being used with turso dev.  It would be good to have a --version flag for that, and also something in the message on stdout when it's launched.

fixed in #580


Document the horizontal property on ListView

ListView inherits all the properties of a scrollView. horizontal being one such property of scrollView.
IMO, It's more scalable to document listview as a child of scrollview rather than documenting individual properties of scrollview in listview.

Oh neat, I didn't know that ListView had all the properties of ScrollView. It does seem like some properties are already duplicated tho, for example, removeClippedSubviews.
It seems like horizontal is a property of ListView tho, it's internally used. It's also a really important property, so I don't know what the best approach is here.
If the documentation states that ListView has all the properties of ScrollView, you have to jump to ScrollView to see how to list things horizontally (which seems kind of weird?)

@arianitu - any suggestions on how we can improve the documentation for this without having to duplicate the properties?

We could have the docs generator link to inherited props. It would find ...ScrollView.propTypes and know to link to the props section of the ScrollView docs. That'd be a great PR.

...ScrollView.propTypes and know to link to the props section of the ScrollView docs.

Hmm, it should already do that. If it doesn't anymore then it's a bug

Yeah it looks broken (no ScrollView reference on https://facebook.github.io/react-native/docs/listview.html)

@arianitu - could you investigate why those props aren't being pulled out for docs?


V2 - Select doesn't select the default value if it's a number

Short description of the problem:
Create a simple select where values are numbers, set the default value to one of them, it's not working
Steps to reproduce:
@Page({
    templateUrl: 'build/pages/page1/page1.html',
})
export class Page1 {
    playerId:number = 0;
    players = [
        {
            id: 0,
            name: 'None'
        },
        {
            id: 1,
            name: 'Foo'
        },
        {
            id: 2,
            name: 'Bar'
        }
    ];
}

  <ion-item>
    <ion-label>Players</ion-label>

    <ion-select [(ngModel)]="playerId">
        <ion-option
                *ngFor="#player of players" value="{{player.id}}"
                checked="{{player.id == playerId ? 'true' : 'false'}}">
            {{player.name}}
        </ion-option>
    </ion-select>
  </ion-item>

if playerId is set as a string, it works:
playerId:string = '0';

Ionic Version: 2.x
Browser & Operating System: iOS / Android / Chrome
**ionic info: **
Your system information:

Cordova CLI: 6.0.0
Gulp version:  CLI version 3.9.1
Gulp local:
Ionic Version: 2.0.0-beta.1
Ionic CLI Version: 2.0.0-beta.17
Ionic App Lib Version: 2.0.0-beta.8
ios-deploy version: 1.8.5
ios-sim version: 5.0.6
OS: Mac OS X El Capitan
Node Version: v5.6.0
Xcode version: Xcode 7.2.1 Build version 7C1002

Thanks for the descriptive issue! This is a duplicate of #5495. It will be fixed with the next release (beta.2). :smile:


[FEATURE] gcc nopch on CI

GCC warnings are not similar to clang warnings and we completly miss them when compiling with a PCH.
OR: we can make PCH build produce warnings. I asked in a cmake channel on how to do it, since googling is not obvious.

Maybe, if we define CMAKE_PCH_PROLOGUE="" this will help.

Tested and it doesn't change the generated PCH header :/

Tested and it doesn't change the generated PCH header :/

Do you think it should? Could you test if maybe warnings show up?


Add Nicer cards

Sorry. Don't have time to do a full test

Vercel tests it, and the build failed.

you don't modify global.css, you modify themes.css and App.svelte

Attempted to close a  that was not open...

Im redoing


chore: support CJS

this PR changes the CI to run two protobuf generators, one for each, plus another for dts. currently untested.

tested working on livekit-server-sdk (which breaks for a different reason, will be handled there, this part is fine)

tested building downstream with esbuild (agents + tsup)


Fix yandex music extractor

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[ ] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[x] Bug fix
[x] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
Explanation of your pull request in arbitrary form goes here. Please make sure the description explains the purpose and effect of your pull request and is worded well enough to be understood. Provide as much context and examples as possible.
Add track version (such as remix or edit) to title;
Avoid duplicates when downloading 150+ tracks playlist;
Fix http 414 error when downloading large playlists (500+ tracks).

Read coding conventions.
Provide clear explanation with example URLs of what this is supposed to fix.
Add tests.


Content crafting and publishing: Contribution/Documentation page

User Story

As an IBM.com Library adopter,

I need guidance on how to contribute Documentations to the IBM.com Library website,

so that I can support the growth of the IBM.com Library.

Additional information

Work on the content by following the publishing process as much as possible and as needed.
Craft content based on the idealization outcome of the Ideation: Contribution Detail page task on the ideation board.
Create the website design with the Sketch App as needed and the Sketch publishing template can be found on the BOX folder. Once complete, save the revised Sketch file in the Guideline folder / sub-folder here.
Once the content is reviewed (with stakeholders or SME of the topic) and finalized.
Copy the content from the Sketch file into the .mdx file for the markdown publishing process to start.
Once complete, create the Pull Request and review, using the Preview link, to be sure it matches the Sketch file design
When ready to deploy, include Jenny as a PR Reviewer and Approver.
Jenny will review and merge the PR

Acceptance criteria

[x] Content crafting is done.
[x] Asset image creation is done (if needed)
[x] Page design is done (if needed)
[x] Publishing with markdowns is done.
[ ] QA'd by Jenny
[ ] Web page preview is approved by Linda, Jeff, and Wonil
[ ] Web page is published

@ljcarot Can you take a look at the Documentation page under the Contribution section and let me know if any edits needed? The preview can be found in PR 744This page is very simple and will be linked to the production guideline I am working on via task issue #722.

Done and approved

Thanks, Linda.


Code Style

@Mottie I see your style for whitespace (particulary the function argument spacing) is quite different from mine, which is not a bad thing but it will lead to inconsistencies.
I placed an .eslintrc which is meant to enforce a particular style, which you can check by first installing ESLint (npm i -g eslint) and then running eslint *.js. As you're so far the prime contributor, I'd suggest you adapt the rules in .eslintrc to suit you needs. I'm using a plugin for SublimeText which highlights any linter errors, so it's easy for me to adapt once the rules are fixed :)

No worries, I'm adaptable :grin:

Very nice of you :+1:


Txt2AL Generated xlf files are unstable.

The Txt2AL program generates different XLF files each time it's run.
Please sort them so we can see the differences between runs.

I get a different content each time.
It appears that the order of the labels are randomised due to the txt2al program being a multi-threaded program which completes conversion of various objects in an unpredictable order.
This also causes object numbers for the extension objects to be assigned in an apparently random order.

After the strange stuff I experienced with #3026 and #3013 I executed two further runs with
PS C:\Windows\system32> Convert-ModifiedObjectsToAl using the same unchanged database from the same container (version 13.0.23474.0-DE) and I can confirm that this happens. The content of the xlf file changes tremendously with each run. Even the file size differs.
Attached are 3 versions (for de-DE), which really should be identical.
XLIFFVariations.zip

Actually, there should not be anything in the 70000000 range in the file, that it another bug I reported with #2987.

Thanks for your input, we will look into fixing this issue

The fix for this issue has been checked in to the master branch. It will be available in the bcinsider.azurecr.io/bcsandbox-master Docker image starting from platform build number 25582.
If you don’t have access to these images you need to become part of the Ready2Go program: aka.ms/readytogo
For more details on code branches and docker images please read:
https://blogs.msdn.microsoft.com/nav/2018/05/03/al-developer-previews-multiple-releases-and-github/
https://blogs.msdn.microsoft.com/freddyk/2018/04/16/which-docker-image-is-the-right-for-you/


Take metadata from form instead of the file section

Needed to make the changes in mass-project/mass_api_client#23 also work for reports.

Codecov Report

Merging #39 into master will increase coverage by 0.16%.

@@            Coverage Diff             @@
##           master      #39      +/-   ##
==========================================
+ Coverage   60.01%   60.17%   +0.16%
==========================================
  Files          61       61
  Lines        1973     1971       -2
  Branches      158      157       -1
==========================================
+ Hits         1184     1186       +2
+ Misses        750      747       -3
+ Partials       39       38       -1

Impacted Files
Coverage Δ

mass_flask_api/resources/scheduled_analysis.py
46% <ø> (+1.76%)
:white_check_mark:

mass_flask_core/models/sample.py
66.83% <ø> (+0.99%)
:white_check_mark:

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 20d2704...b05fcc3. Read the comment docs.


[CONNECT] Service account name is not properly set

The chart ignores the service account name provided in chart values and is always set to the result of rstudio-connect.fullname template.
The following line seems to be incorrect: https://github.com/rstudio/helm/blob/f7bf50ca6a29a455122e75edf47fd5102fddb16b/charts/rstudio-connect/templates/rbac.yaml#L4
Based on the Helm documentation for default function, its syntax is default DEFAULT_VALUE GIVEN_VALUE.
Therefore to take properly into account the service account name specified in Values, the line above should rather be:
{{ $serviceAccountName := default (include "rstudio-connect.fullname" .) .Values.rbac.serviceAccount.name }}

Similar issue is present here:
https://github.com/rstudio/helm/blob/f7bf50ca6a29a455122e75edf47fd5102fddb16b/charts/rstudio-connect/templates/deployment.yaml#L80

Thank you for filing this! I'll take a look today and get this resolved. Confirmed that the service account cannot be set.
Note that this is only true when the launcher is enabled:
» helm template ./ --set launcher.enabled=true --set rbac.serviceAccount.name=asdf --set sharedStorage.create=true | grep serviceAccount
   ...
      serviceAccountName: release-name-rstudio-connect

vs w/o the launcher we get the correct behavior
» helm template ./ --set rbac.serviceAccount.name=asdf | grep serviceAccount
      serviceAccountName: "asdf"