Unknown CA Authority for Github Enterprise

Hi,
I have set the local hub.host but when I try to do a 'hub pull-request', I am getting this:
Post https:myhost/mypath: x509: certificate signed by unknown authority
I tried adding the certificate manually to global git setup, like:
git config --global --add http.sslCert path-to-pem
git config --global --add http.sslVerify false
They don't seem to work.
My regular git works fine.
What else can I try?
Thanks.

Hi, thanks for reporting.
Your GitHub Enterprise host seems to be using a self-signed certificate that you need to explicitly approve somehow. Doing the steps you tried above might solve it for git push/pull, but hub doesn't communicate with GitHub over any of the git protocols. Instead, it does regular HTTPS requests to GitHub API. These HTTPS requests are subject to certificate verification and that's why your unverified host fails.
If you tell us which OS you're on, which hub version do you use, and how did you obtain hub (precompiled binary or compiled yourself?), we could try help you investigate how to add custom root certificates.
@jingweno Do you have an idea where Go net/http draws it root certificates from? Is it bound to system OpenSSL? On OS X, that would mean that you could simply import a new root certificate to your Keychain and configure it so it's always trusted by default.

Hello,
Thanks a lot for the help.
I am on MacOS, and I got hub from the binaries downloads section here.
Also, just mention that I do not need to any of that with regular Mac git, without doing the extra config mentioned above. All I did is setup the Enterprise github account to include my certificate.
I'd say that hub should somehow be able to do the same, if it is to work with Enterprise github, unsure though.
Thanks again.

Going back to git transfer protocol, this piece of git config might be dangerous:
# not recommended
git config --global --add http.sslVerify false

You're turning off SSL certificate verification for all git operations across all hosts. That means when you try to pull git changes from any host, including GitHub.com, the operation will be performed insecurely (that is, the identity of the host will never be verified). I would recommend removing that configuration, and first exploring how you can set http.sslCert to point to a custom certificate that makes it possible for you to git clone/pull/push to your Enterprise instance. Then, when that's sorted out, let's figure out how we can enable the same for hub operations (which depend on the GitHub HTTPS API).

Yes, I guessed that's what that would be doing.
I tried to set that option if it had an effect on hub, but it seems it didn't.
Regular git was working for me with custom certificate on github Enterprise.
I never needed the http.sslCert option for that either, I tried to set it if it solved the hub issue, but it did not.

I tried to set that option if it had an effect on hub, but it seems it didn't.

Almost none of the git options have any effect on hub, because they are separate programs and communicate with GitHub differently. I understand how this is confusing.

Regular git was working for me with custom certificate on github Enterprise. I never needed the http.sslCert option for that either

You never needed that option because you disabled cert verification with http.sslVerify false. In my above comment I encouraged you to disable that configuration (remove it from ~/.gitconfig or set it back to true) and see if git pull/push still work for you. This is not related to hub yet, but it's an important first step to understand that we're at least able to configure the custom root cert for git protocol.

I never had those two options in regular git usage, and git always worked fine.
I added them to see if it had an effect on hub.

@mislav:

Do you have an idea where Go net/http draws it root certificates from? Is it bound to system OpenSSL? On OS X, that would mean that you could simply import a new root certificate to your Keychain and configure it so it's always trusted by default.

Go uses the root cert from the host if not specified. We always verify cert in current implementation. We could make it optional though by setting InsecureSkipVerify to false based on a environment varilable.

I never had those two options in regular git usage, and git always worked fine.

Ok, good to know.
When your company set up the GitHub Enterprise instance on the custom domain, did you ever have to set up a custom SSL certificate to access it in the web browser?

I have the exact same issue, on Mac OSX 10.10, using hub installed via Macports
$ port installed | grep hub
hub @2.2.0_0 (active)

The certificate is signed by my company's root CA, and it is stored in the OS X Keychain; i.e. if I browse the site on a web browser I see a correct, green, verified connection.  But I can't tell hub to look into the Keychain for the certificate. Or is there a way?

It seems that Go should look up certificates in Keychain on OS X. Which Go version is the Macports hub compiled with?
Unfortunately, if it doesn't work, I don't know what else you could try. Maybe we need a fix in hub's own source.

It's go @1.4.2_0.  Thanks for looking into this!

That is a very recent version of Go. I can't tell why a root cert isn't being discovered from OS X Keychain.

I'm having this same problem.
hub create
<asks for username and password>
Post https://<GH Enterprise URL>.com/api/v3/authorizations: x509: certificate signed by unknown authority

Go version
go1.4.2 darwin/amd64
hub version
git version 2.4.5
hub version 2.2.1-gca7939b
OS X 10.10.3
Gray lock icon in browser location bar.
2 factor is on for my regular Github account and for my Github Enterprise account.
I was able to use the gist tool to access this same instance of Github Enterprise.
I was able to use hub to create a repo on my regular Github account.

@christianmlong Do you remember where you installed the certificate for your Enterprise repo system-wide so you can access the site from your web browser and other programs?

Same issue, same versions as @christianmlong. I've installed the certificates to my keychan and I still get issues.

I'm having the same issue.  Based on the source of golang @ https://github.com/golang/go/blob/master/src/crypto/x509/root_darwin.go  It looks like the only place that Go checks for certs is in the System Root Certificates keychain.  It should be checking the System and the Login keychains for certs as well, but isn't.
It's not possible from the Keychain Access gui to add certificates to this keychain.  I've tried playing with the command line security tool, but so far no luck.

@stevehedrick Awesome detective work. Do you think we could/should replicate this logic to look up the cert in Login keychains from hub ourselves? Or should we add hub configuration to specify the location of custom CA bundle on disk?

Hi!
I have exactly the same problem on Windows XP and not Enterprise git:
git version 2.7.1.windows.2
hub version 2.2.3
Regular git works fine, however i had to configure it with:
git config --system http.sslcainfo "C:\Program Files\Git\mingw32\ssl\certs\ca-bundle.crt"

@mislav FYI, the git-lfs project has solved a similar issue. I imagine the solution for hub would be similar: https://github.com/github/git-lfs/issues/308#issuecomment-197964308

Thanks for that reference! That's really useful.
This is a really interesting starting point for OS X, for example. Thanks for doing this hard work in git-lfs, @sinbad!

No problem - you actually want to use this as a starting point: https://github.com/github/git-lfs/commit/cb889413872b9f94e172fe15fe4b005e84c33624#diff-9938c5ad789d4db49e660fe24b343282, the commit above was a WIP. HTH!

Patch that makes go look for certs in system.keychain is landing in 1.8.

Hi -- running into this issue at the moment, wondering what the solution is on the user's end? I'm using Hub 2.2.3 on OSX, installed via Homebrew.
I'm able to pull/push to our enterprise config over HTTPS and SSH. But if I try and open a PR with Hub, I get prompted for my login and password, then hit with an error: Post https://github.<our domain>.com/api/v3/authorizations: x509: certificate signed by unknown authority

@tconroy Unfortunately, there is no user workaround right now. We need to replicate the git-lfs functionality (linked above) into hub first.

@mislav I see -- very unfortunate. :( Do you have any rough/approximate ETA for when this feature will be fixed? honestly the primary draw of hub for me is the ability to do tasks such as open PR's from the command line.

Looks like most people have been trying this on OSX. I have also not been able to get this to work on RHEL 7.
Thanks.

I am having this same issue, and out GitHub Enterprise cert is signed by a real, trusted external root CA:
Post https://github.ourcompany.com/api/v3/authorizations: x509: certificate signed by unknown authority
The cert chain is:
Entrust Root Certificate Authority—G2 (System Roots keychain)
-- Entrust Certificate Authority ‐ L1K (login keychain)
-- Company GitHub Enterprise cert
~|⇒  hub --version
git version 2.6.4 (Apple Git-63)
hub version 2.2.3

@joedj That's strange. I would have hoped that Go discovers the certificates stored in your system or login keychains. In any case, the version that you're running is really old and I encourage you to upgrade to at least v2.2.9. It might not solve your certificate problem, but it fixes a lot of other bugs.

Thanks @mislav
Unfortunately I can confirm that v2.2.9 does not resolve this issue.
For reference, it looks like the relevant golang issue is https://github.com/golang/go/issues/14514 (which has been closed).
There is apparently a workaround for El Capitan users, which I have not tried yet:

Disable System Integrity Protection
Add the new CA to SystemRootCertificates.keychain

I am on Windows 10 and am running into this same issue when attempting to connect to an enterprise GitHub account.  (I notice that https is NOT enabled when I navigate there in the browser.)

I get the same error "x509: certificate signed by unknown authority".
I use the git config --global --add http.sslVerify false option and the git command line works for me. Is it in anyway possible to get hub to skip the ssl verification too?
I understand that there is a security vulnerability here however given that git already has this feature we are only asking hub to do exactly what git is doing (and not weaken the security model).

Any updates on this issue?

Yes this is quite a blocker for GitHub Enterprise usage!

@jsok I was able to resolve this myself and am able to use hub with github enterprise. You will have to compile hub yourself with Golang with CGO disabled. You can follow the following steps:

Install go with CGO disabled. If go is already installed you can verify this by checking the output of the command go env and search for 'CGO_ENABLED' flag. If it's "0" then disabled (obviously). You can install golang without CGO using brew command brew install go --without-cgo
Clone hub and compile the standalone binary
Import the certificate for your github enterprise into the "System" keychain
Add your github enterprise URL into .gitconfig by executing git config --global --add hub.host my.example.org

You should be able to use hub with your github enterprise now :)

@rage28 Thank you for sharing your workaround!
Changing a local installation of go is indeed a nuclear option that I wouldn't suggest to everyone unless they really know what they are doing. I would much rather see us add code to hub that allows reading an extra certificate from a file or something.

I'm also experiencing this issue working with Github Enterprise:

Fails with same certificate error with hub pr ..., hub issue ..., hub ci-status, hub release
hub pull-request works.

Same issue. GitHub Enterprise, valid cert, getting the warning on hub checkout <pr_url>

I can confirm that running the Homebrew-installed hub (2.5.0) on High Sierra will work against Enterprise instances using my enterprise CA when the "System" Keychain is set to explicitly "Always Trust" that CA. That is, I am now able to run hub commands pointing at an Enterprise repository with no complaints about the certificates I'm using.
Disclaimer: In my case, it's an enterprise CA that I'm explicitly trusting, so I consider this slightly less risky than explicitly trusting a self-signed certificate, but everyone can/should do that calculation for themselves.

This is still failing for me on MacOS Mojave 10.14.1 and Hub 2.6.0 using a private enterprise CA. The cert is trusted in my system keychain, but Hub continues to stop me at every turn with x509: certificate signed by unknown authority. Major showstopper.

@forresthopkinsa There was a bug a for a while in Go (not sure if it is fixed) where it has issues accessing the certificates from System or SystemRoot, a workaround is to copy the certificate to the login keychain associated with your user and give it full trust there.
Open Keychain Access and navigate to the certificate in question under System and then drag and drop it onto login to initiate the copy, at which point you need to provide sudo capable credentials. Once that is done, double click the certificate to view it, then expand the Trust arrow and give it System-wide Always Trust and you should be able to talk using golang tools like hub without getting this error.
Let me know if this works for you, currently I'm on 10.13.6, but I imagine I'll find out soon enough when they push the Mojave upgrade to my machine.

Ran into this on Mojave, brew upgrade go and hub resolved it.


Update to Error Prone 2.3.2

Only a few fixes required.
I had to disable analysis of generated code since AutoValue-generated code doesn't pass the UnnecessaryParentheses check.  I tried updating AutoValue to fix this, and decided to keep the update.

Actually, I don't get why the tests are failing. It seems to be something about IllegalAccessError in this line: compilationHelper = CompilationTestHelper.newInstance(NullAway.class, getClass());
But, even on the current error-prone master, that method is public.

@lazaroclapp @subarnob what is going with passing -cp here?
https://github.com/uber/NullAway/blob/237a6ef87b14414de3a4a12469cc0b35e8a7a763/nullaway/src/test/java/com/uber/nullaway/NullAwayTest.java#L1078-L1079
I wonder if it's breaking these tests with this update by overriding the test classpath.

@lazaroclapp @subarnob what is going with passing -cp here?
NullAway/nullaway/src/test/java/com/uber/nullaway/NullAwayTest.java
Lines 1078 to 1079 in 237a6ef
"-cp",
"../jar-infer/test-java-lib-jarinfer/build/libs/test-java-lib-jarinfer.jar"))
I wonder if it's breaking these tests with this update by overriding the test classpath.

Not entirely sure.
This is already a test dependency for nullaway: https://github.com/uber/NullAway/blob/master/nullaway/build.gradle#L57 But removing the -cp argument causes the tests to fail with missing errors.
I believe the code that scans for jars with models here: https://github.com/uber/NullAway/blob/master/nullaway/src/main/java/com/uber/nullaway/handlers/InferredJARModelsHandler.java#L110 looks at the -cp (or similar) option of the javac command, and can't see the way in which CompilationTestHelper makes the outer classpath available.

I believe the code that scans for jars with models here: https://github.com/uber/NullAway/blob/master/nullaway/src/main/java/com/uber/nullaway/handlers/InferredJARModelsHandler.java#L110 looks at the -cp (or similar) option of the javac command, and can't see the way in which CompilationTestHelper makes the outer classpath available, hence the requirement to explicitly add that jar on the javac command.

Can we do something like this trick to avoid looking at -cp?

I believe the code that scans for jars with models here: https://github.com/uber/NullAway/blob/master/nullaway/src/main/java/com/uber/nullaway/handlers/InferredJARModelsHandler.java#L110 looks at the -cp (or similar) option of the javac command, and can't see the way in which CompilationTestHelper makes the outer classpath available, hence the requirement to explicitly add that jar on the javac command.

Can we do something like this trick to avoid looking at -cp?

👍 Oh, nice find!
It seems to work, at least as far as NullAway's own tests are concerned. See #243 . I'd still like to test this a bit internally before merging that PR, though :)

Ok, cool, once #243 is merged I'll rebase on that and hopefully we'll be good to go with this one


Add Related persons to patient registration

Fixes #184
Type
Choose one: Feature
Checklist

[ ] I have written Unit tests for any new feature(s) and edge cases for bug fixes
[ ] I have added any strings visible on UI components to the strings.xml file
[ ] I have updated the  CHANGELOG.md file for any notable changes to the codebase
[ ] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the project's style guide
[ ] I have built and run the fhircore app to verify my change fixes the issue and/or does not break the app

Depends on #356

@ekigamba what's blocking this?

@ekigamba what's blocking this?

@pld It's dependent on this PR https://github.com/google/android-fhir/pull/566 being merged and release with the work.

@ekigamba what's blocking this?

@pld It's dependent on this PR google/android-fhir#566 being merged and release with the work.

w/566 merged, is this ready for review?

@ekigamba what's blocking this?

@pld It's dependent on this PR google/android-fhir#566 being merged and release with the work.

w/566 merged, is this ready for review?

Not yet, we still have another issue blocking further work on this here https://github.com/opensrp/fhircore/issues/398

@ekigamba what's blocking this?

@pld It's dependent on this PR google/android-fhir#566 being merged and release with the work.

w/566 merged, is this ready for review?

Not yet, we still have another issue blocking further work on this here #398

Looks like 398 is now closed

@ekigamba what's blocking this?

@pld It's dependent on this PR google/android-fhir#566 being merged and release with the work.

w/566 merged, is this ready for review?

Not yet, we still have another issue blocking further work on this here #398

Looks like 398 is now closed

That's correct @pld . I'm currently working on this to get this done

Pull Request Test Coverage Report for Build #262

0 of 0   changed or added relevant lines in 0 files are covered.
9 unchanged lines in 2 files lost coverage.
Overall coverage increased (+1.7%) to 89.102%

Files with Coverage Reduction
New Missed Lines
%

src/main/java/org/smartregister/fhircore/viewmodel/QuestionnaireViewModel.kt
2
95.0%

src/main/java/org/smartregister/fhircore/activity/QuestionnaireActivity.kt
7
87.78%

Totals

Change from base Build #260:
1.7%

Covered Lines:
1349

Relevant Lines:
1514

💛  - Coveralls


Think about signing the built container images

Investigate, if sigstore can be used for signing the images and if the idea makes sense.

The signature can be created with GitHub Action and then stored in the container registry.
Quay already supports the sigstore, as it is based on OCI, however only from version 3.6 and quay.io yet wasn't upgraded to this version (but will be).
When the quay.io supports the sigstore we could sign our containers, and ship them signed.
The authentication of the signer is done via the OpenID connect

Thanks for the ping.  Yes, we are working on integrating sigstore/cosign support into the container tools (Skopeo, Podman, Buildah).


Update CA default root cert TTL to 10 years

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: wattli
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

security/OWNERS [wattli]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@wattli: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
462b533c4ea38f88e149fa5c1bc6cb033f8527ef
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

This made it into 1.1.8


aspid update

connected missed external airlocks
replaced water tank on hight-capacity water tank at jani room

the dreaded lf ops

Janitors rejoice


Slashes in chapter name prevents chapter from downloading

Describe the bug
If the chapter contains a slash Mango attempts to open the file in a subfolder, which dosent exist and the download freezes.
[ERROR]   2020/06/08 20:13:44 | Error opening file with mode 'w': '/mnt/extrafiles/mango_lib/library/Dogeza De Tanondemita/Ch.40 Coming of Age/Adult's Day.cbz': No such file or directory
To Reproduce

Download series 30590 chapter 40

Expected behavior
I would expect the slashes to be filtered out, as to prevent such issues, same with ".."
Environment (please complete the following information):

OS: Manjaro Linux 20.0.3 - Lysia
Browser: Firefox 77.0.1
Mango Version 0.5.2

Thanks for the bug report! The issue is fixed in the development branch by replacing illegal characters with _. I will make a new release soon.

👍


How to set Tool Tip Color same as Legend Color for multi-line chart?

I am trying out multi line chart with 3 datasets Dataset 1, Dataset 2, Dataset 3. When hover Dataset 1 only Dataset 1 should value should shown in Tooltip whereas on hover Dataset 2 or Dataset 3, both Dataset 2 & Dataset 3 value should be shown. I achieved showing multi values on Tooltip but the actual problem is Dataset 2 Legend color is shown for both Dataset 2 & Dataset 3 tool tip color on hover Dataset 2 and  Dataset 3 Legend color is shown for both Dataset 2 & Dataset 3 tool tip color on hover Dataset 3. Let me know some solutions to achive it. Thanks in advance!

I tested this on chart.js v2.9.3 (latest release version) and this works fine. Without a jsfiddle test case I can't provide more help, sorry.


Permission denied error

Hi - I’m trying to use this for a basic react app. When it runs, it uploads everything correctly, but then fails at the very end with the following error:
...
Completed 38.6 MiB/38.7 MiB (4.1 MiB/s) with 2 file(s) remaining
Completed 38.7 MiB/38.7 MiB (4.1 MiB/s) with 2 file(s) remaining
upload: build/static/media/socicon.a35b6574.svg to s3://***/static/media/socicon.a35b6574.svg
Completed 38.7 MiB/38.7 MiB (4.1 MiB/s) with 1 file(s) remaining
upload: build/static/js/2.4a354a39.chunk.js.map to s3://***/static/js/2.4a354a39.chunk.js.map
sh: /: Permission denied

The IAM user has full S3 access, so permissioning shouldn’t be an issue.
This is the code block in our Github actions workflow:
    - name: Deploy to S3
      uses: jakejarvis/s3-sync-action@master
      with:
        args: --acl public-read --follow-symlinks --delete
      env:
        AWS_S3_BUCKET: ${{ secrets.AWS_PRODUCTION_BUCKET_NAME }}
        AWS_ACCESS_KEY_ID: ${{ secrets.AWS_ACCESS_KEY_ID }}
        AWS_SECRET_ACCESS_KEY: ${{ secrets.AWS_SECRET_ACCESS_KEY }}
        AWS_REGION: ${{ secrets.AWS_REGION }}
        SOURCE_DIR: "build"

@ankitgoyal100 how did you solve it? I am facing the same issue.

@ankitgoyal100 how did you solve it? I am facing the same issue.

@MaiKaY i ended up creating a new bucket and it worked

@MaiKaY i ended up creating a new bucket and it worked

@ankitgoyal100 I just found my issue. It was because my S3 bucket was configured as "private". As soon as I turned off the Block all public access it works for me.

@ankitgoyal100 I just found my issue. It was because my S3 bucket was configured as "private". As soon as I turned off the Block all public access it works for me.

I'm having these error, and my s3 bucket is fully public, also IAM user has full S3 access


Error in building Chocolatey package

This showed up in the 2.4.1 release of the .NET 4.6.2 Pluggable Agent but appears to be a problem either in the recipe itself or in cake.tool 4.0.
Two URLs in the built package were mangled, dropping the name of the GitHub repository. This caused the submitted package to fail verification at chocolatey.org. I fixed the immediate error by editing the package manually and this issue is for identification and correction of the underlying fault that caused the failure.
Expected output, as included in the nuspec file:
<licenseUrl>https://raw.githubusercontent.com/TestCentric/net462-pluggable-agent/main/LICENSE.txt</licenseUrl>
...
<bugTrackerUrl>https://github.com/TestCentric/net462-pluggable-agent/issues</bugTrackerUrl>

Actual output in the built package:
<licenseUrl>https://raw.githubusercontent.com/TestCentric//main/LICENSE.txt</licenseUrl>
...
<bugTrackerUrl>https://github.com/TestCentric//issues</bugTrackerUrl>

This may have something to do with how we are combing values in the nuspec with the settings we pass to cake.tool or it could be a fault in cake.tool itself.
Important to fix this for the 1.1.0 release and indeed before we do any more releases of pluggable agents.

This is caused by the fact that the BuildSettings have not been initialized at the time we assign values in constants.cake. We should remove any values from constants.cake which depend on BuildSettings and defihne them at the point of use.

:tada: This issue has been resolved in version 1.1.0 :tada:
The release is available on:

GitHub Release
NuGet Package


Reverse Infinite Scroll Feature

Hello,
Do you have anything planned for such a feature?
Something like replicating the Facebook Messages system. Basically, instead of triggering the event on scrolling down, it would trigger it on scrolling up.
Thanks

@IvoPereira, this question is not to me. This package is based on https://github.com/webcreate/infinite-ajax-scroll, so it supports only features that are present in this plugin :)


Make ctlplane backward compatible

As a pull request owner and reviewers, I checked that:

[X] Appropriate testing is done and actually running

/hold
if we get https://github.com/openstack-k8s-operators/openstack-operator/pull/639 in, we shouldn't need this patch.

Other patch merged, dropping this one.


Added functionality to merge two singly linked list

Alright. Done
I have some doubts about my code i don't understand,. Whenever i try to delete oldhead; ( line 45 & 39) it some how changes the l1 list after the merge (line 98) even though i am passing it by const reference and copying it in the next line. When i don't
delete oldhead it works the way it should be.
why is that?

@aayushjain7  and @Arsenic-ATG  everything works perfectly now.
Please check...

Hi @LightMonarch,
Nice work, and welcome to our team

The invitation link

Thank you. Good to be part of the community


fix install.el script

Currently, package freezing fails because of the repo in the install.el script is not correct

it seems it's resolved in the develop branch


[Bug] Patient Age is lost from Lifepoint HL7 messages

Describe the bug
The HHS protect feed is not receiving age of the patient from LifePoint and Ellume
Impact
About 700k Ellume test results do not have patient age.
To Reproduce
I was able to reproduce the bug locally.
Steps to reproduce the behavior:

Run a local version of ReportStream
gradle run
Get a Lifepoint HL7 message
=> Notice that the PID segment has patient birthdate.
Upload the Lifepoint HL7 message

> curl -X POST -H "Content-Type:application/hl7-v2"  --data-binary @result_files/lifepoint/lifepoint-covid-19-0000acfe-98e6-4a95-876a-5792f6e5fbf0-20220124002222.hl7 'http://localhost:7071/api/reports?client=lifepoint'
{
  "id" : "124517ec-f89b-40ff-83cc-cb90cbcf0475",
  "timestamp" : "2022-02-09T14:30:21.667412Z",
  "topic" : "covid-19",
  "reportItemCount" : 1,
  "sender" : "lifepoint.default",
  "destinations" : [ {
    "organization" : "HHSProtect",
    "organization_id" : "hhsprotect",
    "service" : "elr",
    "sending_at" : "2022-02-09T09:31-05:00",
    "itemCount" : 1
  } ],
  "destinationCount" : 1,
  "warningCount" : 5,
  "errorCount" : 0,
  "errors" : [ ],
  "warnings" : [ {
    "scope" : "item",
    "message" : "Report 1: Invalid field equipment_model_name; please refer to the Department of Health and Human Services’ (HHS) LOINC Mapping spreadsheet for acceptable values.",
    "itemNums" : ""
  }, {
    "scope" : "item",
    "message" : "Report 1: Invalid field equipment_model_id_type; please refer to the Department of Health and Human Services’ (HHS) LOINC Mapping spreadsheet for acceptable values.",
    "itemNums" : ""
  }, {
    "scope" : "item",
    "message" : "Report 1: Invalid field ordered_test_encoding_version; please refer to the Department of Health and Human Services’ (HHS) LOINC Mapping spreadsheet for acceptable values.",
    "itemNums" : ""
  }, {
    "scope" : "item",
    "message" : "Report 1: Invalid field test_kit_name_id_cwe_version; please refer to the Department of Health and Human Services’ (HHS) LOINC Mapping spreadsheet for acceptable values.",
    "itemNums" : ""
  }, {
    "scope" : "item",
    "message" : "Report 1: Invalid field test_performed_system_version; please refer to the Department of Health and Human Services’ (HHS) LOINC Mapping spreadsheet for acceptable values.",
    "itemNums" : ""
  } ]
}⏎

Look at the resultant internal file
=> No patient age
Look at the resultant hhs protect file
=> No patient age
Expected behavior
The patient age should be transmitted to HHS Protect
Should be easy to find the code problem. Most likely to be caused by the ingestion of HL7.

Found at the same time as #4257


Access Policy with System Assigned Management Identity

I follow this document to configure WebApp for Container with System Assinged Management Identity, however, it is not obvious how to configure access policy for system assigned management identity. I can eventually configure it to search the Application that has same name as webapp, however, I can't find it on the official doc, instead, I can guess that from the Jeff's post. https://medium.com/statuscode/getting-key-vault-secrets-in-azure-functions-37620fd20a0b . I can figure out, however, it took some time. It might be helpful if you add note for the key vault access policy configuration part.

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

ID: 7feaa3d0-7233-a3cb-cb9b-a9a55b1f0d7a
Version Independent ID: f41f9d51-52f8-1f45-1043-1344a3b74458
Content: Key Vault references - Azure App Service
Content Source: articles/app-service/app-service-key-vault-references.md
Service: app-service
GitHub Login: @mattchenderson
Microsoft Alias: mahender

@TsuyoshiUshio Thanks for bringing this to our attention. Your feedback has been shared with the content owner for further review.

The system-assigned identity is an extension of the resource it is assigned to, in this form:
'/////providers/Microsoft.ManagedIdentity/Identities/default'.
As an example, you would declare the resource id of the system-assigned identity like so (update for your resource type):
"variables": {
"sfAppSystemAssignedIdentityResourceId": "[concat(resourceId('Microsoft.ServiceFabric/clusters/applications/', parameters('clusterName'), parameters('applicationName')), '/providers/Microsoft.ManagedIdentity/Identities/default')]"
}
And then declare an access policy entry like so:
under 'resources':
{
	"type": "Microsoft.KeyVault/vaults/accessPolicies",
	"name": "[concat(parameters('keyVaultName'), '/add')]",
	"apiVersion": "2018-02-14",
	"properties": {
		"accessPolicies": [
		{
				"name": "[concat(parameters('clusterName'), '/', parameters('applicationName'))]",
				"tenantId": "[reference(variables('sfAppSystemAssignedIdentityResourceId'), '2018-11-30').tenantId]",
				"objectId": "[reference(variables('sfAppSystemAssignedIdentityResourceId'), '2018-11-30').principalId]",
				"dependsOn": [
					"[variables('sfAppSystemAssignedIdentityResourceId')]"
				],
				"permissions": {
					"secrets": [
						"get",
						"list"
					]

(looks like my text got truncated) The first line should read:
'/subscriptions/{subscription}
/subscriptions/{subscription id}/resourcegroups/{resource group name}/providers/{provide name}/{resource type}/{resource name}/providers/Microsoft.ManagedIdentity/Identities/default'.

Don't forget if doing this to also update the dependsOn to include a reference to vaults/accessPolicies that your MSI is in. Otherwise this may not get set in the correct order.

Ignore my comment above

fixed in https://github.com/MicrosoftDocs/azure-docs-pr/pull/244872.
#please-close


Suggestion: Players can protect their buildings from being schemed by others

Not all players enjoy their builds being copied in multiplayer, I think a system like the one that is in place for scheming a building would work perfect for protecting buildings from being copied. It would be great if users could add others as their friends (different from team), so that only they can copy their structures if wanted.
While we are here, some kind of 'chunk claiming' and 'land protection' might go well with this feature.

I don't see a major need for protecting intellectual property in a sandbox game. If someone likes your design and uses it, you should be proud and feel honoured! =)
If you're looking for world protection and claims in general, there are plenty of server plugins and mods out there already. Have a look at FTB Utilities for example.

I replied on #29  :)

Imagine this:

You come to your small friendly server and someone (maybe some kid), just because thought it was funny, scanned a mountain, put a chest beside a Schematicannon, put all of their stone which came from mining in it and build a mountain on top of your house with little to no effort whatsoever.

Please add another 'Paper and Quill' item which when in hand, allows you to created a prohibited area for scanning. Only you have the control to disable it temporarily to scan something and then you can re-enable it for protection. People cannot create or scan anything in those areas. Forget chunk claiming, this is much more realistic and requires effort on players behalf, it's not just a click and bam, you have claimed a 16x16 worth of block for any height anywhere on the map.
Please please please, this is really necessary.


Edit Channels Backend

Screens have been create for creating and joining channels.
These screens do not have full code behind them. They will need to implement their intended functionality.

Implemented the create and join code.
Suggestion for further work would be a channel manage screen which shows all your channels, and the ability to add more and delete them.


Stop crawling and accept crawled records

In current state, when I stop crawling anything will be missed.
I need to stop crawled records and save them.

+1

Any updates on this?


Show AM health based on mesh status

Conditionally return OK or NOT HEALTHY based on
whether the mesh is settled or not.
This is a coarse indicator. In the future, a
more detailed check of the health of an instance
can be implemented.
Discussed this briefly with @grobie
This check wasn't added to the ready endpoint because a running alertmanager should always be ready to accept alerts (taking the kubernetes terminology), but health indicates its current running status.

I don't think an unhealthy mesh should cause the Alertmanager to be killed, during a network partition you want it to keep running.

I agree with @brian-brazil. This is even the opposite: those are the moments you want AM to work.

that's fair. I'm wondering if there's any benefit to exposing this directly through an endpoint rather than just through metrics.


prepending zero

I am trying to use this widget for a simple 2-digit countdown and also for a clock. I need 2-digits shown at any time (ie 11, 10, 09, 08, 07...). I can prepend '0' for values that are <10 but I will lose the animation for that zero that's added. Any way to have this feature?
My current workaround:
AnimatedFlipCounter(
  value: time.toDouble(),
  prefix: time<10 ? '0' : '',
),

Good idea. This has been added in 0.2.0 with a new property called wholeDigits. For example, wholeDigits: 4 means it will pad 48 into 0048. Please help test this new feature and let me know how it goes. Thanks!


change the aspect ratio of a donut/pie chart

Thanks for the great work on this library.
I would love it if I could make the aspect ratio of a canvas with a pie or donut chart 1:1. The extra padding on the left and right makes it harder to add a legend.

:+1:

There is workaround to set
Chart.defaults.global.maintainAspectRatio = false;
and set width/height of container

None of these workarounds really work for responsive charts.

Well?! What is the correct way to deal with this?
The pie chart occupies an area much smaller than it's container's width.

Hello, today I am closing all issues that are only affecting version 1 of Chart.js as WONTFIX.
If this issue does affect version 2 as well, I apologize for the error. Please create a test case against Chart.js 2 using on of the below websites and we will be happy to reopen the issue and update its classification:

http://jsbin.com/
http://jsfiddle.net/
http://codepen.io/pen/
Premade template: http://codepen.io/pen?template=JXVYzq

Hi I'm using version: 2.7.1 and I have this problem, the canvas size is rectangle instead of squared, how to fix this?

aspectRatio: 1 in the chart options

aspectRatio: 1 in the chart options
please mention in docs for future users. wasted nights on working around this

@sht5, @simonbrunel
Could you please tell, where exactly should I place aspectRatio: 1 in the chart options?
this didn't work for me:
var myChart = new Chart(ctx, {
  type: 'pie',
  data: {
    labels: ["Green", "Blue", "Gray", "Purple", "Yellow", "Red", "Black"],
    datasets: [{
      backgroundColor: [
        "#2ecc71",
        "#3498db",
        "#95a5a6",
        "#9b59b6",
        "#f1c40f",
        "#e74c3c",
        "#34495e"
      ],
      data: [12, 19, 3, 17, 28, 24, 7]
    }]
  },
  options: {
    aspectRatio: 1,
    legend: {
      display: false,
    },
  }
});

thank you in advance!

@amatiyko your snippet should work but maybe you are not using the latest version?
@sht5 feel free to submit a PR with the docs updated (responsive.md)

@simonbrunel  the reason was in version, thank you very much!

This PR adds the docs update https://github.com/chartjs/Chart.js/pull/5756 :)


Ability to enable fine scale OSD even with a built-in display only

Before opening the issue, have you...?

[X] Searched for existing issues

Is your feature request related to a problem? Please describe
On my MacBook Pro M1 2021, when I change brightness using system F1/F2 keys it changes brightness to -+ 6% or something aka one bar step:

So each this bar changes -+ steps too much for me, I'd like to have an option to change -+ 1% using F1/F2 keys
Describe the solution you'd like
Add option to set value of % in which each F1/F2 key press will change value of brightness.
Describe alternatives you've considered
In my NativeDisplayBrightness app there is such option, it changes brightness more precise, so just by adding less then -+6% for each step.

Anything else?
No response

Hi @KAMIKAZEUA
The option you're looking for is called "Use fine OSD Scale" located here in the app settings

Hey @the0neyouseek this option didn't worked for me. Using MacBook Pro M1 2021.
https://user-images.githubusercontent.com/3391456/162590269-2ecae7d5-0315-4a94-b598-8d29c2c6fc70.mov

Hey @KAMIKAZEUA - are you using an external display or only the built-in display?

Only built in at the moment. Just I'm not in office. While pressing Shift+Option changes smoothly.

All right. The problem probably is that MonitorControl disengages from the keyboard completely when only the internal display is connected. This is intentional and is a feature actually (there were some requests towards this, I don't recall why exactly though). This might be finetuned in a way that this does not happen when the fine OSD scale is enabled.

I think this was the reason:
https://github.com/MonitorControl/MonitorControl/issues/456

I see, but this was OLD macOS. A lot was changes since that time. It's now working well, since it detects built-in display correctly.

Hmm, look what I found, if you just uncheck/check Brightness it will start to change smoothly:
https://user-images.githubusercontent.com/3391456/162591584-aee10f71-f7cf-4ccf-b83f-f67b35c41867.mov
Even if I restart app, it will continue to work. But if I disable OSD scale option and then quit app - it will not work after next start, if I enable OSD scale again. But I can just uncheck/check Brightness again)
Bug = feature)

And yet, still it changes mostly for 2% per step.. but I'd like to have 1% per step max.. :p

Hmm. Something is off there obviously. I can convert this into a feature request.

@waydabber also, while using OSD scale for built-in display, I noted one thing: if auto-brightness enabled, it changes brightness obviously, after that if I press F1/F2 - screen will blinks in brightness level.
So, it's not reading current value of display brightness, before apply changes to it.

Hi @KAMIKAZEUA - yes, if the system changes the brightness, it does not inform MonitorControl about it. MC however polls the system brightness periodically and if it detects a change, it will alter the slider and update its knowledge about the brightness level + tries to poll faster for a while. You'll see this - if you change the brightness using the ambint light sensor (for example by covering it with your hands), you'll see your screen dim and the MC brightness slider follow it after a while.
If you however try to change the brightness using MC before MC updated its internal understanding of current brightness, the screen will flash.

Hey @waydabber Just used this over almost all day at work. In 80% of cases it blinks brightness on F1/F2 key, which is really annoying. I'm sitting near window and looks like brightness is changing a lot, clouds are moving so it's really dynamic.
I think this should be like maybe another bug report, since MC should read system brightness when I press F1/F2 key.
But since I'm currently using like bug=feature, this is perhaps should be considered and tested while implementing current request. If this is possible of course.

But yes, you should create a separate bug report about this one. :) Thank you!

Yes, you are right, I added a note to that bug about the fact that the problem is deeper than known. Honestly I use my macbook in clamshell mode mostly plus I have the ambient light sensor disabled so I did not notice the issue. Sorry about that + thank you!


The cost definition for SASD

Excuse me, May i ask you where is the definition of the cost function for SASD， i cannot find it in the file of “pcdet/models/detectors/second_net_iou.py”
def forward(self, batch_dict):
    batch_dict['dataset_cfg'] = self.dataset.dataset_cfg
    for cur_module in self.module_list:
        batch_dict = cur_module(batch_dict)

    if self.training:
        weights = batch_dict.get('SEP_LOSS_WEIGHTS', None)
        loss, tb_dict, disp_dict = self.get_training_loss(weights)

        ret_dict = {
            'loss': loss
        }
        return ret_dict, tb_dict, disp_dict
    else:
        pred_dicts, recall_dicts = self.post_processing_multicriterion(batch_dict)
        return pred_dicts, recall_dicts

When the fellowing part is in the file of “tools/cfgs/da-nuscenes-kitti_models/secondiou_st3d/secondiou_st3d++_ros_car.yaml”
SELF_TRAIN:
SRC:
USE_DATA: True
USE_GRAD: True
LOSS_WEIGHT: 1.0
SEP_LOSS_WEIGHTS: [0.0,1.0,0.0]
TAR:
USE_DATA: True
LOSS_WEIGHT: 1.0
SEP_LOSS_WEIGHTS: [1.0,1.0,1.0]
Thank you very much！

https://github.com/CVMI-Lab/ST3D/blob/2634561684dfbafce6ed86e50c6f70f51988593c/tools/train_utils/train_st_utils.py#L44-L61


Packaging request: GDC

Project description
A GCC frontend for the D programming language.
GDC has some quirks in regards to its Dlang syntax validation an application I'd like to package relies on. Attempting to use another D compiler throws alot of errors and fixing the source to not rely on GDC-specific, old behaviour would require more than just afew changed lines. Hence, and with the recently added GCC 9 being the first version to integrate GDC, I'd like to see it being added to nixpkgs :slightly_smiling_face:
I gave it a shot some moons ago by adopting how gfortran is handled, but neglected it since then. I remember the compilation got stuck at .libs/libgdruntime.so.76.0.3 aborting with alot of errors similar to
relocation R_X86_64_32 against '.rodata.str1.8' can not be used when making a shared object; recompile with -fPIC

I'm hoping someone else might be better at figuring out how to get it working.
Metadata

homepage URL: https://gdcproject.org/ (possibly outdated since being integrated into upstream GCC)
source URL: same as GCC
license: GPL 3 or later (according to wiki)
platforms: Linux, Darwin, Windows (according to wiki)

this new template selection thing seems pretty nice, :+1:

Closed by #69144


Combining queries on mobile does not work

Expected Behavior
When I combine queries like this:
"""tasks not done (due before tomorrow) OR (scheduled before tomorrow) OR (tags include todo) OR (heading includes todo) shorten """
Combined tasks with this query show up on PC
Current behaviour
On Obsidian mobile I get response
"do not understand query:..."
Steps to reproduce
Just open app on android
Which Operating Systems are you using?

[X] Android
[ ] iPhone/iPad
[ ] Linux
[ ] macOS
[ ] Windows

Obsidian Version
1.4.1
Tasks Plugin Version
1.16.0
Checks

[X] I have tried it with all other plugins disabled and the error still occurs

Possible solution
No response

some formatting got messed up, query is written like this:
not done
(due before tomorrow) OR (scheduled before tomorrow) OR (tags include todo) OR (heading includes todo)
shorten

Soory, cannot reproduce this with the same versions on Android. Officially it's "short mode" not "shorten", but the latter works as well.

"do not understand query:..."

The text that it shows after that message is the all-important bit! What does it say?
Also, are you sure that you are using the same version of the plugin version on all devices? Please check.

it automagically works now. sorry for opening an issue.

it automagically works now. sorry for opening an issue.

Great! Thanks for letting us know.


Datepicker minDate and maxDate don't work properly with ng-model-options timezone

Bug description:
When using date-picker-popup with an ng-model-options timezone set, the minDate and maxDate don't seem to be working properly. For example, if I set minDate to be 2016-12-31T00:00:00Z it allows selection of Dec 30th 2015. If I set maxDate to be 2016-01-15T00:00:00Z it only allows selection up to Jan 14th 2016.
Link to minimally-working plunker that reproduces the issue:
http://plnkr.co/edit/NheeTWdqKf4eFDNPV5uT?p=preview
Notice how you are allowed to select outside of the minDate and maxDate that is set.
Version of Angular, UIBS, and Bootstrap
Angular: 1.5.x
UIBS: 2.1.3 (originally encountered on 1.3.3)
Bootstrap: 3.x

After further investigation it seems that the ngModelOptions isn't propagating from the popup to the picker. The workaround would be to put the ngModelOptions on a parent of the input field (the <div class="col-md-6"... in the linked plnkr) in which case Angular propagates that down to all child models.


problem with Light Mode and Dark Mode

good job! this GitHub Action is very interesting!
I have a question:
in README it's possible to insert an image with a tag that indicates in which mode it should be viewed:
![Light-Mode](./img/img5-light.png#gh-light-mode-only)
![Dark-Mode](./img/img5-dark.png#gh-dark-mode-only)

if my pc has light mode, only the first image is chosen otherwise the second. Problem is that both images are inserted in the pdf. solution?

I haven't yet supported this feature - it was new to GH fairly recently. I can look into whether it's possible but it might be an idea of simply using custom CSS:
(presuming you only want the light theme picture)
img[src*="gh-dark-mode-only"] {
    display: none;
}

Thank you for your answer!
I have tried many time your code but it doesn't work...it was very helpful and I found this solution:
images are inserted in the html in base64 and the tag gh-dark-mode-only is lost. So I use a class in img tag:
.dark-mode {
    display: none;
}

this is a possibile solution


New Rule: Test-NetConnection should be used instead of Ping

In the same vein as the rule to use CIM instead of WMI, Test-NetConnection should be used over Ping.
Test-NetConnection is a direct Ping.exe replacement that returns an object instead of text.

One of the potential issues for that is that the cmdlet is only available in Windows 8.1/2012R2 and up, so it would break anything downlevel of that.

This is also the case of the example I gave of using CIM instead of WMI. The current PS version can be checked and ignored if not applicable.

Other than the non-availability of the cmdlets on downlevel machines, I don't see any other issues with this approach.
Is perf an issue? - I see a report of this being slower than the standard ping.
http://windowsitpro.com/blog/tricks-test-connection

@dcrreynolds first just want to say thanks for engaging with us and offering up your contribution. The more users we have like you, the better. :)
Unfortunately, I'm of the opinion that we should not take this. I get that Test-NetConnection mostly does what ping does, but for many scenarios where you simply need to fire off a few packets, ping is going to be waaaaay faster.
Ideally, we should have some sort of -Basic option for Test-NetConnection that does nothing but what ping does (and as fast), but until then, I feel uncomfortable saying that this is a "best practice".

In light of Test-Connection and Test-NetConnection being less performant, I understand how there is not a clear winner between the many ways to ping a machine. I will close the PR and leave this in a branch in case it comes up again in the future.

Thanks!


Doc typo

You can write a plugin for Lapce with any programing language that complies to WASI

Probably meant "compiles".

I can't find it so I assume it's fixed now


Calling startActivity() from outside of an Activity context requires the FLAG_ACTIVITY_NEW_TASK flag

This happens in our React Native app. Possibly because the following method fails since the context is a react context and not an activity:
https://github.com/twitter/twitter-kit-android/blob/38aeae6c046fe2ba1074a1f0cbfaf7746a8c76ce/twitter-core/src/main/java/com/twitter/sdk/android/core/IntentUtils.java#L34
Steps to reproduce the problem

Implement TwitterTweet or TwitterTimeLine in your app
Open app on Android 6 or earlier
Share a tweet (press share icon)

Expected behaviour
Twitter app intent chooser to share the tweet
Actual behaviour
App crashes:
android.util.AndroidRuntimeException: Calling startActivity() from outside of an Activity  context requires the FLAG_ACTIVITY_NEW_TASK flag. Is this really what you want?
Android API Version
Android 6 and prior
This can easily by adding the FLAG_ACTIVITY_NEW_TASK to the intent defined here:
https://github.com/twitter/twitter-kit-android/blob/c148862c612c66d0f2b50cac40adf4e3d287cb8b/tweet-ui/src/main/java/com/twitter/sdk/android/tweetui/AbstractTweetView.java#L248

I have created a PR which resolves this issue. To reproduce you can use
context.getApplicationContext() instead of context when creating the intent. This mimics the same behaviour as reactContext.
Please review my code review. In our application we have about 50.000 Android users with Android 6 or lower. We really need this fix.

@efrohnhoefer Any progress on this? As I've said, we have a lot of users affected by this bug.

Just do this:

create a folder in your project
put the *.aar files of the twitter project in there
add "implementation fileTree(dir: '<your_folder_here>', include: ['*.aar'])" to your build.gradle

works like charm, and fixes your problem until they decide to implement/use your fix.


MountVolume.SetUp failed for volume "bgp-config-path" : configmap "bgp-config" not found

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
mp host-proc-sys-net]: timed out waiting for the condition
Warning  FailedMount       35m (x12 over 43m)    kubelet            MountVolume.SetUp failed for volume "bgp-config-path" : configmap "bgp-config" not found
Warning  FailedMount       34m                   kubelet            Unable to attach or mount volumes: unmounted volumes=[bgp-config-path], unattached volumes=[etc-cni-netd host-proc-sys-net hostproc host-proc-sys-kernel clustermesh-secrets xtables-lock cilium-bootstrap-file-dir lib-modules kube-api-access-jd48v cilium-run cilium-cgroup bgp-config-path cni-path tmp bpf-maps]: timed out waiting for the condition
Warning  FailedMount       25m (x8 over 26m)     kubelet            MountVolume.SetUp failed for volume "bgp-config-path" : configmap "bgp-config" not found
Warning  FailedMount       2m43s                 kubelet            Unable to attach or mount volumes: unmounted volumes=[bgp-config-path], unattached volumes=[tmp host-proc-sys-kernel clustermesh-secrets cni-path bgp-config-path host-proc-sys-net bpf-maps lib-modules xtables-lock etc-cni-netd cilium-bootstrap-file-dir cilium-cgroup cilium-run kube-api-access-jd48v hostproc]: timed out waiting for the condition
Warning  FailedMount       33s (x10 over 4m44s)  kubelet            MountVolume.SetUp failed for volume "bgp-config-path" : configmap "bgp-config" not found
Warning  FailedMount       27s                   kubelet            Unable to attach or mount volumes: unmounted volumes=[bgp-config-path], unattached volumes=[host-proc-sys-kernel hostproc clustermesh-secrets cni-path tmp host-proc-sys-net bgp-config-path bpf-maps cilium-bootstrap-file-dir cilium-run cilium-cgroup kube-api-access-jd48v etc-cni-netd xtables-lock lib-modules]: timed out waiting for the condition
Cilium Version
v1.13.0
Kernel Version
5.15
Kubernetes Version
v1.26.1
Sysdump
No response
Relevant log output
No response
Anything else?
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

As mentioned in https://docs.cilium.io/en/v1.13/network/bgp/, it seems like the configmap must be created before enabling bpg.

BGP support is enabled by providing the BGP configuration via a ConfigMap and by setting a few Helm values. Otherwise, BGP is disabled by default.

Heads up that this is happening again, and the 1.14 documentation does not mention the ConfigMap. https://docs.cilium.io/en/v1.14/network/bgp-control-plane/

@Apocrathia Thanks for the reporting, but the document you mentioned is not the equivalent of the one in the original issue. The same sentence @sayboras mentioned still exists on the v1.14 as well.
https://docs.cilium.io/en/v1.14/network/bgp/


Install RStudio Professional Drivers

Description: https://docs.rstudio.com/pro-drivers/installation/#installing-rstudio-professional-drivers

LGTM, just fix the linter errors!


panic when submitting TaskRun

Expected Behavior
TaskRun to complete successfully
Actual Behavior
Controller panics
Steps to Reproduce the Problem

This happens when I create a TaskRun that references a ClusterTask.

Additional Info

Kubernetes version:

$ kubectl version
Client Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.3", GitCommit:"2d3c76f9091b6bec110a5e63777c332469e0cba2", GitTreeState:"clean", BuildDate:"2019-08-19T11:13:54Z", GoVersion:"go1.12.9", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"14+", GitVersion:"v1.14.10-gke.17", GitCommit:"bdceba0734835c6cb1acbd1c447caf17d8613b44", GitTreeState:"clean", BuildDate:"2020-01-17T23:10:13Z", GoVersion:"go1.12.12b4", Compiler:"gc", Platform:"linux/amd64"}

Tekton Pipeline version:

v0.10.1

The panic from the controller logs:
panic: runtime error: index out of range [5] with length 5

goroutine 258 [running]:
github.com/tektoncd/pipeline/pkg/pod.(*stepStateSorter).changeIndex(...)
	github.com/tektoncd/pipeline/pkg/pod/status.go:386
github.com/tektoncd/pipeline/pkg/pod.(*stepStateSorter).Swap(0xc000caad40, 0x2, 0x1)
	github.com/tektoncd/pipeline/pkg/pod/status.go:393 +0x514
sort.insertionSort(0x1b7c1a0, 0xc000caad40, 0x0, 0x5)
	sort/sort.go:28 +0x57
sort.quickSort(0x1b7c1a0, 0xc000caad40, 0x0, 0x5, 0x6)
	sort/sort.go:209 +0x201
sort.Sort(0x1b7c1a0, 0xc000caad40)
	sort/sort.go:218 +0x79
github.com/tektoncd/pipeline/pkg/pod.sortTaskRunStepOrder(0xc000fea240, 0x5, 0x8, 0xc000ab8000, 0x7, 0x8, 0x2, 0x0, 0x0)
	github.com/tektoncd/pipeline/pkg/pod/status.go:357 +0xb5
github.com/tektoncd/pipeline/pkg/pod.MakeTaskRunStatus(0x0, 0x0, 0x0, 0x0, 0xc000828630, 0x2c, 0x0, 0x0, 0xc0008286f0, 0x29, ...)
	github.com/tektoncd/pipeline/pkg/pod/status.go:177 +0x4f7
github.com/tektoncd/pipeline/pkg/reconciler/taskrun.(*Reconciler).reconcile(0xc000106750, 0x1b8e3a0, 0xc0001f01e0, 0xc000aadb80, 0xed5fc68ab, 0x27380c0)
	github.com/tektoncd/pipeline/pkg/reconciler/taskrun/taskrun.go:360 +0x115e
github.com/tektoncd/pipeline/pkg/reconciler/taskrun.(*Reconciler).Reconcile(0xc000106750, 0x1b8e3a0, 0xc0001f01e0, 0xc0009bfc80, 0x56, 0xc0000b9e00, 0x1b8e3a0)
	github.com/tektoncd/pipeline/pkg/reconciler/taskrun/taskrun.go:153 +0x841
knative.dev/pkg/controller.(*Impl).processNextWorkItem(0xc0002e4600, 0x0)
	knative.dev/pkg@v0.0.0-20191111150521-6d806b998379/controller/controller.go:335 +0x654
knative.dev/pkg/controller.(*Impl).Run.func1(0xc0009224f0, 0xc0002e4600)
	knative.dev/pkg@v0.0.0-20191111150521-6d806b998379/controller/controller.go:285 +0x53
created by knative.dev/pkg/controller.(*Impl).Run
	knative.dev/pkg@v0.0.0-20191111150521-6d806b998379/controller/controller.go:283 +0x1ac

I've managed to modify the ./pkg/pod/status_test.go file (TestSortTaskRunStepOrder) to recreate the error:
diff --git a/pkg/pod/status_test.go b/pkg/pod/status_test.go
index 9215e116..3f96a41a 100644
--- a/pkg/pod/status_test.go
+++ b/pkg/pod/status_test.go
@@ -635,6 +635,10 @@ func TestSidecarsReady(t *testing.T) {
 func TestSortTaskRunStepOrder(t *testing.T) {
 	steps := []v1alpha1.Step{{Container: corev1.Container{
 		Name: "hello",
+	}}, {Container: corev1.Container{
+		Name: "extra-1",
+	}}, {Container: corev1.Container{
+		Name: "extra-2",
 	}}, {Container: corev1.Container{
 		Name: "exit",
 	}}, {Container: corev1.Container{

go test ./pkg/pod --run TestSortTaskRunStepOrder -v

I can reproduce with the modifications to the unit test you mentioned but I'm curious if you see this with all ClusterTasks?  I'm not able to reproduce with the trivial ClusterTask example in examples/v1beta1/taskruns/clustertask.yaml.
I have a fix that seems to work ok for the unit test example but I'd really like to see an example of a ClusterTask / TaskRun that's hitting the same error so that I can confirm my fix works for that too. Can you nail down what specifically in the ClusterTask configuration is the source of the problem or provide your ClusterTask for me to test?

(Even just copy/pasting the output of describe taskrun would be helpful - I'm trying to understand where the mismatch between number of steps and number of step states is coming from)

So it looks like the problem happens when a step fails.

Woo this is a doozy.
The image digest exporter (part of the Image Output Resource) is configured with "terminationMessagePolicy": "FallbackToLogsOnError",.  When a previous step has failed the exporter emits the following log line: 2020/03/13 12:03:26 Skipping step because a previous step failed.  This gets read by the Tekton controller which is looking for JSON in the termination message. It fails to parse the message from the image digest exporter and stops trying to read step statuses (here).  This results in a mismatch in the length of the list of steps and the length of the list of step statuses and finally our sort method panics with an out of bounds error because it assumes the lengths of the two lists are the same.
I'm working on a couple of fixes for this and will make a PR later today.

Reopening until this is backported to 0.10

Marking as closed now that we've backported to 0.10 and released in 0.11.


Support for Honeywell D6 Pro

Just purchased a new D6 Pro thermostat for ductless split. Works well, but when being discovered by Homebridge I get the following messages:
[12/3/2020, 1:25:36 AM] [HoneywellHomeThermostat] Total Locations Found: 1
[12/3/2020, 1:25:36 AM] [HoneywellHomeThermostat] Getting devices for name_removed...
[12/3/2020, 1:25:36 AM] [HoneywellHomeThermostat] Total Devices Found at name_removed: 1
[12/3/2020, 1:25:36 AM] [HoneywellHomeThermostat] A LLC Device has been discovered with a deviceModel that does not start with T5 or T9
I am willing to do test work to add support for this device.

@fabianoflnsp I believe we are good to close this right?
I will release a new version soon, but in the mean time you can stay on the beta.


WIP: Deis hephy fixup

This has at least one thing still missing...
Some credentials to be added for querying Kubernetes management APIs (GCS and Azure)
There's a place where a GUID representing the GCS KEY should be, but instead it's an ID that just says GCSKEY... that's an incorrect diff
All of the GUIDs in here are "safe for git" -- they are just pointers at credentials that are locked inside of Jenkins credentials binding plugins (we should also have them in a password safe, I guess?)

Supersedes #1


feat: BaseCarousel

BaseCarousel

Usage

<BaseCarousel>

Slide 1

Slide 2

Slide 3

</BaseCarousel>

Maybe it will be worth to implement Navigation, Pagination, ProgressBar, ScrollBar but thinking it might be bloated, usually we have simple carousel with some arrows (open to opinions)

[AUTOMATED] Latest version has been pushed to preview.
Preview deployed at https://preview-8ecbaf0.dev.nextjs-boilerplate.jam3.cloud.jam3.net

[AUTOMATED] Latest version has been pushed to preview.
Preview deployed at https://preview-37a9115.dev.nextjs-boilerplate.jam3.cloud.jam3.net

This is more of a thought, but should we be moving component into the next-react-ui repo? I feel like component wise we're getting too bloated. Even though I know we use a lot of Carousels on projects sadly


[WIP] storage create and edit modal in workbenches

RHOAIENG-7681
Description
This PR is on top of the #3320
Create storage modal: Updated the ManageStorageModal

Attach storage modal:

Recording:
https://github.com/user-attachments/assets/66d102c8-a962-475b-8ac5-eac85f849f48
How Has This Been Tested?

Test Impact

Request review criteria:

Self checklist (all need to be checked):

[ ] The developer has manually tested the changes and verified that the changes work
[ ] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[ ] The developer has added tests or explained why testing cannot be added (unit or cypress tests for related changes)

If you have UI changes:

[ ] Included any necessary screenshots or gifs if it was a UI change.
[ ] Included tags to the UX team if it was a UI/UX change.

After the PR is posted & before it merges:

[ ] The developer has tested their solution on a cluster by using the image produced by the PR to main

Closing this PR, reopening it with the independent branch.


why not write executor in go?

Issues are for bugs in syzkaller itself, for everything else please contact<EMAIL_ADDRESS>mailing list.
Before filing an issue please check the existing issues and the mailing list (https://groups.google.com/forum/#!forum/syzkaller).
Please provide syzkaller revision, Go version (output of go version), relevant information about host and target systems.
If syz-manager is not working overall, please provide output of syz-manager with -debug flag.

Hi @joeyjiaojg, please use the mailing list<EMAIL_ADDRESS>for discussions. Thanks.


Update README.md

Codecov Report

:exclamation: No coverage uploaded for pull request head (readme-patch@23d6e44).

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 747cc5d...23d6e44. Read the comment docs.


Move Vm SSH authentication away from copy_throughput benchmark

For Private Keys, we must call vm.PushHostFile instead of vm.PushFile. AuthenticateVM in linux_virtual_machine already has that, so it's better to keep that functionality there.

LGTM


update gisce/ooui to v2.6.4

Automated changes by create-pull-request GitHub action

:tada: This PR is included in version 2.10.7 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.11.0-alpha.10 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Show 'publicly' who subscribed in space

Leak some semi-privacy thing

function could be same for this https://github.com/yangwao/progressbar-coweb/issues/14


Debian release for Humble

Is it possible to release this package to the OSRF package server?

If you can help, why not :wink:


python310Packages.dvc-data: 0.1.5 -> 0.1.13

Description of changes
Update to latest upstream release 0.1.13

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 185450 run on x86_64-linux 1

2 packages marked as broken and skipped:

dvc
dvc-with-remotes

4 packages built:

python310Packages.dvc-data
python310Packages.dvc-objects
python39Packages.dvc-data
python39Packages.dvc-objects


[Feature Request] List of connections

Hello,
We have 4 environments (REC, INT, PPROD and PROD) and we would like to be able to switch easily from one environment to the other. So the feature would be to have a list of connections (in my case 4) registered somewhere. Right now, we have to disconnect, then connect to the other environment.
Cobra

Hi @Cobra5197, sorry for picking with such delay. We are thinking of moving neodash from a single to multiconnections, but this requires a huge engeneering effort in terms of refactoring. I'll keep you posted when something starts moving.


''Failed to update the output theme. Please reload the page.''

I get this message whenever I open a kata. Reloading the page changes nothing. And if I try to test my code I get another message: ''Failed to connect to the runner. Please reload the page.'' I had no such issue until this morning. I tried using a different browser.. I also tried deleting my cache and all that. Nothing fixes it. I appreciate the help in advance.

Thanks for reporting. This is caused by a new deployment.
Are you using the light theme? I forgot to test with it and was able to reproduce it.

No.. I use the default dark theme. But i tried in light theme now.. same problem still there.

Thanks, I'm unable to reproduce it with the dark theme, but #2269 was also reported so it seems like the connection in general is buggy. I'll close this one because it's a subset of #2269.


Write a Kotlin program to find square of a number

Description
Write a Kotlin program to find square of a number
Input  : 5
Output : 25

How to contribute

Save the solution in FindSquareOfANumber.kt file
Add FindSquareOfANumber.kt file in find-square-of-a-number folder

!assign

Hey @saiprakash255, this issue is already assigned to @darkbits018 !!!
Please choose another issue.
Thanks for your interest in contributing to this project.


what is the dynamic range of the input pixels that the network accepts?

do it take images in range of 0-1, 0-255, or some preprocessed?

0~255


Publish a new version of NPM package containing recent fixes?

It has been a while since last release. Important fixes like #6 were not included, making webpack unable to bundle the package correctly.

Temporary workaround for webpack:
{
  resolve: {
    alias: {
      'leaflet.awesome-markers$': 'leaflet.awesome-markers/dist/leaflet.awesome-markers.js'
    }
  }
}

fixed with 2.0.5 release


MemDB does not reuse free nodes

MemDB cannot reuse free nodes, which may lead to mem leak in some cases. If a DML writes many keys and it has been retried multiple times due to pessimistic lock error, MemDB would re-alloc nodes for each try. Refer to the following demo code, the mem usage of allocator keeps growing.

/report customer

Fixed in ART based membuffer


lose focus in multiple inputbox of a row

I'm submitting a ...  (check one with "x")
[ ] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request => Please do not submit support request here, post on Stackoverflow or Gitter

Current behavior
I have implement multiple inputbox in a row.
when I type in a inputbox, after type a character I lose the focus from input box.
Expected behavior
I expect to keep the focus in input box.
Reproduction of the problem
This is my code snippet.
          <ngx-datatable-column name="urls" [flexGrow]="5" headerClass="kartList-header">
            <ng-template let-karts="column" let-sort="sortFn" ngx-datatable-header-template>
              Url
            </ng-template>
            <ng-template let-rowIndex="rowIndex" let-value="value" let-row="row" ngx-datatable-cell-template>

{{url}}

                  <input type="text" class="editable-url" [(ngModel)]="row.urls[i]">

            </ng-template>
          </ngx-datatable-column>

What is the motivation / use case for changing the behavior?
Please tell us about your environment:

Operating System : MacOs, IDE : WebStorm 2017.2.4, PM: NPM

Table version: 0.8.x

"@swimlane/ngx-datatable": "10.2.3"

Angular version: 2.0.x

Angular Version : 4.3.5

Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

Language: [all | TypeScript X.X | ES6/7 | ES5]

+1

+1


Saving Data - entity framework SQL Insert statement creates a null value for a field with a value.

Describe what is not working as expected.
I have an application that is posting payments.  It writes new records to two tables and updates a third.  When saving the changes, the generated insert statements work fine for several hundred inserts and then changes one of the parameters to a null.  This then violates the database integrity which throws an error.
Turning on logging shows the last few inserts, the second to last insert to the Vouchers table shows that the second parameter, the BORROWER_ID has a value.  The Last insert has a null where the Borrower_ID parameter should be:
INSERT [dbo].[VOUCHERS]([BATCH_NUM], [BORROWER_ID], [InvestorID], [MICInvestorID], [ReceiptTypeValue], [PAYMENT_DATE], [PaymentMethod], [PMT_METHOD], [PMT_AMT], [RENEWAL_FEE], [CHG_AMT], [TX_TypeID], [FromTrustAmount], [USER_ID], [Notes], [Change], [X1], [X2], [X5], [X10], [X20], [X50], [X100], [CashSubtotal], [REFERENCE], [ChequeAmount], [DraftNumber], [DraftAmount], [TX_ID])
VALUES (@0, @1, NULL, NULL, @2, @3, NULL, NULL, @4, @5, @6, NULL, @7, NULL, NULL, @8, @9, @10, @11, @12, @13, @14, @15, @16, NULL, @17, NULL, @18, NULL)
SELECT [VOUCHER_NUM], [ReceiptType], [Version], [RowGUID]
FROM [dbo].[VOUCHERS]
WHERE @@ROWCOUNT > 0 AND [VOUCHER_NUM] = scope_identity()
-- @0: '12680' (Type = Int32)
-- @1: '2224' (Type = Int32)
-- @2: '1' (Type = Byte, Size = 1)
-- @3: '01/06/2019 12:00:00 AM' (Type = DateTime2)
-- @4: '1396.2400' (Type = Decimal, Precision = 19, Scale = 4)
-- @5: '0' (Type = Decimal, Precision = 19, Scale = 4)
-- @6: '0' (Type = Decimal, Precision = 19, Scale = 4)
-- @7: '0' (Type = Decimal, Precision = 19, Scale = 4)
-- @8: '0' (Type = Decimal, Precision = 10, Scale = 4)
-- @9: '0' (Type = Int16)
-- @10: '0' (Type = Int16)
-- @11: '0' (Type = Int16)
-- @12: '0' (Type = Int16)
-- @13: '0' (Type = Int16)
-- @14: '0' (Type = Int16)
-- @15: '0' (Type = Int16)
-- @16: '0' (Type = Decimal, Precision = 19, Scale = 4)
-- @17: '0' (Type = Decimal, Precision = 19, Scale = 4)
-- @18: '0' (Type = Decimal, Precision = 19, Scale = 4)
-- Executing at 06/09/2019 1:30:10 PM -07:00
-- Completed in 8 ms with result: SqlDataReader
UPDATE [dbo].[Payments]
SET [PaymentPostingStatus] = @0, [VoucherID] = @1
WHERE (([PADID] = @2) AND ([Version] = @3))
SELECT [RowGUID], [Version]
FROM [dbo].[Payments]
WHERE @@ROWCOUNT > 0 AND [PADID] = @2
-- @0: '1' (Type = Byte, Size = 1)
-- @1: '30856' (Type = Int32)
-- @2: '2004' (Type = Int32)
-- @3: 'System.Byte[]' (Type = Binary, Size = 8)
-- Executing at 06/09/2019 1:30:10 PM -07:00
-- Completed in 8 ms with result: SqlDataReader
INSERT [dbo].[VOUCHERS]([BATCH_NUM], [BORROWER_ID], [InvestorID], [MICInvestorID], [ReceiptTypeValue], [PAYMENT_DATE], [PaymentMethod], [PMT_METHOD], [PMT_AMT], [RENEWAL_FEE], [CHG_AMT], [TX_TypeID], [FromTrustAmount], [USER_ID], [Notes], [Change], [X1], [X2], [X5], [X10], [X20], [X50], [X100], [CashSubtotal], [REFERENCE], [ChequeAmount], [DraftNumber], [DraftAmount], [TX_ID])
VALUES (@0, NULL, NULL, NULL, @1, @2, NULL, NULL, @3, @4, @5, NULL, @6, NULL, NULL, @7, @8, @9, @10, @11, @12, @13, @14, @15, NULL, @16, NULL, @17, NULL)
SELECT [VOUCHER_NUM], [ReceiptType], [Version], [RowGUID]
FROM [dbo].[VOUCHERS]
WHERE @@ROWCOUNT > 0 AND [VOUCHER_NUM] = scope_identity()
-- @0: '12680' (Type = Int32)
-- @1: '1' (Type = Byte, Size = 1)
-- @2: '01/06/2019 12:00:00 AM' (Type = DateTime2)
-- @3: '2329.9200' (Type = Decimal, Precision = 19, Scale = 4)
-- @4: '0' (Type = Decimal, Precision = 19, Scale = 4)
-- @5: '0' (Type = Decimal, Precision = 19, Scale = 4)
-- @6: '0' (Type = Decimal, Precision = 19, Scale = 4)
-- @7: '0' (Type = Decimal, Precision = 10, Scale = 4)
-- @8: '0' (Type = Int16)
-- @9: '0' (Type = Int16)
-- @10: '0' (Type = Int16)
-- @11: '0' (Type = Int16)
-- @12: '0' (Type = Int16)
-- @13: '0' (Type = Int16)
-- @14: '0' (Type = Int16)
-- @15: '0' (Type = Decimal, Precision = 19, Scale = 4)
-- @16: '0' (Type = Decimal, Precision = 19, Scale = 4)
-- @17: '0' (Type = Decimal, Precision = 19, Scale = 4)
-- Executing at 06/09/2019 1:30:10 PM -07:00
A first chance exception of type 'System.Data.SqlClient.SqlException' occurred in System.Data.dll
A first chance exception of type 'System.Data.SqlClient.SqlException' occurred in System.Data.dll
A first chance exception of type 'System.Data.SqlClient.SqlException' occurred in System.Data.dll
A first chance exception of type 'System.Data.SqlClient.SqlException' occurred in System.Data.dll
-- Failed in 122 ms with error: The INSERT statement conflicted with the CHECK constraint "CK_VOUCHERS". The conflict occurred in database "ArmadaProduction", table "dbo.VOUCHERS".
The statement has been terminated.
CK_Vouchers is checking to ensure that either the Borrower_ID has a value or the InvestorID has a value.  Of the 160+ records being written, this is close to the end.  I check the values to be written to see if any have a null Borrower_ID and all of them have values.
This fragment of the code that creates a Voucher is:
Steps to reproduce
   Private Function createVoucher(batch As Batch, pmtVM As PaymentViewModel, Amount As Decimal) As Voucher
        Dim vchr As Voucher = DBCTX.DatabaseContext.Vouchers.Create()
        With vchr
            .BorrowerID = pmtVM.BorrowerID
            '.Borrower = pmtVM.Borrower
            .Batch = batch
            .PaymentDate = pmtVM.PaymentDate
            .Amount = Amount
            .ReceiptType = CommonResources.Receipts.ReceiptTypes.Payment
            .UserID = Me.CurrentUserID
            If pmtVM.TakeFromTrustAmount.HasValue Then .FromTrustAmount = pmtVM.TakeFromTrustAmount.Value
            .PaymentMethod = pmtVM.PaymentMethod
            If .PaymentMethod.HasValue Then
                .PaymentMethodDescription = EnumFunctions.GetEnumName(Of PaymentMethods)(.PaymentMethod.Value)
            End If
            .Reference = pmtVM.ReferenceNumber
        End With
        batch.Vouchers.Add(vchr)
        DBCTX.DatabaseContext.Vouchers.Add(vchr)
        Return vchr
    End Function

.
.
.
Dim missingBorrowers As List(Of Voucher) = pmtsTOProcess.Where(Function(p) Not p.PaymentEntity.Voucher.BorrowerID.HasValue).Select(Function(p) p.PaymentEntity.Voucher).ToList
RecordSaved = DBCTX.SaveChanges()
The missingBorrowers list returns 0 entries. The BorrowerID in the PMTVM is defined as an Integer and is non-nullable.
DBCTX is a synchronized wrapper for the database context called ArmadaDBEntities which is generated from the Entity Framework model.  Scanning the records to be updated shows that the records in question all have a valid value for the BorrowerID.
 Could you please post a complete and runnable, but minimal, project/solution that reproduces the behavior you are seeing.

EF Team Triage: Closing this issue as the requested additional details have not been provided and we have been unable to reproduce it.
BTW this is a canned response and may have info or details that do not directly apply to this particular issue. While we'd like to spend the time to uniquely address every incoming issue, we get a lot traffic on the EF projects and that is not practical. To ensure we maximize the time we have to work on fixing bugs, implementing new features, etc. we use canned responses for common triage decisions.


Multiple previews and resolving different values for a singleton

Been hurting my brain trying to figure out the best approach to this and hoping there's a simple solution I've just not thought of :-)
In a SwiftUI preview I have a number of previews I want to return with different values. For example (made up):
struct Hello_Preview: PreviewProvider {
static var previews: some View {
        Container.share.user.register { NoUser() }
        HelloView()
            .previewDisplayName("No user")
        Container.share.user.register { SubscribedUser() }
        HelloView()
            .previewDisplayName("Subscribed user")
    }

Now I'm sure you're about to tell me this won't work and it doesn't because both instances of HelloView resolve the user much later and get the same subscribed user.
I can't just pass the user in because as much as I'd like to, HelloView is a complicated view with many sub views in it and I'd end up writing a lot of boilerplate to pass the user down which would defeat the purpose of using Factory in the first place.
So my question is - is there a way to setup factory's user registration so that the correct user is returned when the factory resolves for a preview without having to change anything in the injections littered throughout the view hierarchy?
My currently (ugly) solution to all of this is to create multiple PreviewProvider structs instead of just one and that can become messing with a lot of cut-n-paste. So I'd really like just Ione :-)

First, check out the Multiple Previews section in the docs.
https://hmlongco.github.io/Factory/documentation/factory/previews

I have a case that does not work with the multiple preview as you describe

contenteview with @Injected service1
child of content view that is init in the body also uses @Injected service1

As body happens after init no matter what you do the child could potentially get a different service is in between init of content view and the body call the registration for service1 changes. To
Solve it you would pass service1 in the body to
The child on init. But that beets the purpose of using factory to inject.
So I'm a bit stuck with that.

I'd have to think about this, but the key to the issue would probably focus around some mechanism that allows Container.shared to return a different container for each scene.

Every instance of @Injected triggers s new factory resolution. Have you tried using a .shared scope?


[BUG] Unable to start topology configuration

Describe the bug
Unable to initiate topology configuration. Status says Not configured, steps 5 and 6 are greyed out, clicking start pops up a window with "No WebOS devices detected. To configure your devices, ..."
What I've tried:

Uninstalling and installing the app
Installing 3.5 version
Clearing config.json
Restarting TV, turning off other displays
Trying with different Quick Start+, Always Ready combinations
Changing FriendlyNames, removing them, adding them to other subkeys (results in companion finding 2nd device with null ip)

How to reproduce the bug?
It might've been after I renamed TV in device manager with FriendlyName method with regedit. I noticed this feature was off after quite some time, so many factors could be at play.
What was the expected behavior
Self-explanatory.
What is your system?
Win 10 22H2, build 19045.4412
LG OLED C2, WebOS 23 13.30.56, (TV Version 8.3.0-2301)
Log file and screenshots

Log: https://www.prcl.dev/51c2wx3d2ac3rqg
Config: https://www.prcl.dev/5ar0iytp80xgmg9

Hi @maqzek thanks for the detailed bug report. You are already on to the explanation for this issue. Below is the relevant code snippet that fails unexpectedly when the device friendly name has been changed:
if (FriendlyName.find(L"lg tv") != std::wstring::npos)
So as you see "lg tv" is hardcoded, which is fine for all situations except when the user has manually changed the friendly name. It's not difficult to allow for a user configurable option but it will add complexity (for the user). I'll see if I can make this matching functionality smarter.
For now the workaround is to ensure that the "lg tv" is a substring of the friendly name

Different things. Please see the screenshot below. #1 can be anything, #2 must contain "[LG]" for the scan to work and #3 must contain "LG TV" for the topology feature to work

Looks allright I think. Probably need then to add a few lines of debug code to understand what is going wrong. Are you on discord? PM me please

Excellent @maqzek thanks! 👍


[PDI-15628] Results from jobs run in parallel should not be mixed in log files

[PDI-15628] Results from jobs run in parallel should not be mixed in log files
-added checking object name in LoggingObject equals()
@mbatchelor @mchen-len-son Could you please review and merge it?

Build Completed
:white_check_mark: This pull request has passed all validations.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f core/build.xml clean-all resolve jacoco && ant -Dtestreports.dir=bin/reports/integration-test -f core/build.xml jacoco-integration checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  core/src/org/pentaho/di/core/logging/LoggingObject.java
  core/test-src/org/pentaho/di/core/logging/LoggingObjectTest.java

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.core.logging.LoggingObject

Branch Change:  + 3.00%
Complexity Change:  + 4.43%
Instruction Change:  + 5.35%
Line Change:  + 5.84%
Method Change:  + 9.09%

I fixed checkstyle violations.
@mbatchelor @mchen-len-son Could you please review and merge it?


Headers struct behind non-default headers feature gate

PR'ing for visibility and any comment.  The changes here are best summarized by the change log additions:

The default compat feature is no longer optional, as its unlikely that anyone would be using recent versions without this feature. The feature gate name is retained for now, but has no effect.

Place the legacy Headers struct under a new non-default headers feature  gate. Note that use of this type is no longer required nor recommended for parsing and serialization of typed headers. See the rewritten typed header doc examples.  Consider replacing with the http crate HeaderMap and the TypedHeaders extension trait introduced here in 0.15.0.

The motivation for this is to steer new users in the best direction (using http::HeaderMap instead of Headers when possible), avoiding such issues as #17, and a minor compile time and size improvement when the headers feature is not enabled.

Merged to master in 09035b6b1


[Bugs] LGPL module is used in CHF

Describe the bug
The LGPL-licensed module "github.com/juju/fslock" is used in CHF.
Walkthrough (Option)
This module is used by the field CdrFileLock in the struct ChfUe in the file "NFs/chf/internal/context/ue_context.go".

Hi @ShouheiNishi
This issue just been fixid by https://github.com/free5gc/chf/pull/27
Thanks for your information.


Enhance the overall UI

I would want to enhance the UI of the website for a better user-experience.
@rahulsainlll Please assign me this issue under GSSoC-extd 2024.
Thankyou

Hey,
Thanks for your interest! You’re assigned to this issue under GSSoC-extd 2024. Feel free to improve the UI, but make sure it maintains that old-timer aesthetic. We want a modern touch while preserving the classic feel.
Looking forward to seeing your updates!

I would want to enhance the UI of the website for a better user-experience.
@rahulsainlll Please assign me this issue under GSSoC-extd 2024.
Thankyou

what do you want to enhance ?

will try to make more attractive UI for this website.....


pass-secret-service: unstable-2020-04-12 -> unstable-2022-03-21

Description of changes

Update to the latest upstream version of pass-secret-service that includes systemd service files.
Add patch to fix use of a function that has been removed from the Python Cryptography library in NixOS 22.05
Install systemd service files in the Nix package.
Add NixOS test to ensure the D-Bus API activates the service unit.
Add myself as a maintainer to the package and NixOS test.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[x] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[x] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Rebased against recent master.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/984


Custom Filters and extra job fields

How do I add extra fields to the filters? I would like to add an extra filter for "Salary From" which exists in the Bullhorn backend dashboard but I would like it on the Career Portal aswell.
Is this possible? and where would I look? Thanks

Hello Sudar,
You are able to add additional filters to the sidebar, but there will need to be several modifications made in different areas, first you will want to adjust the sidebar controller and template to include your additional fiels, then, you will want to modify the search service as well.
#46 and #138 are two examples of people attempting to utilize additional or customized filters on their portal

Do you know which files these are in? Are they in the various js files?
Thanks for the quick reply

You'll be able to find the sidebar files in the sidebar directory found here and the search service in the services directory.

Thanks so much. I will take a look at this shortly
Is there also a way to display the "Salary From" details on the job listings on the right hand panel?

You are able to do that by adding the data you would like to add to the list template  using angular interpolation.
What you will add is something like {{::job.salary}}

Ok great - thank you very much

I will take a proper look at the code I have when I get home as I dont remember seeing all these HTML files in the version of the Career Portal that I have. The examples you have shown seem to have a lot more HTML pages than the one I have

It is possible that you are taking a look at the built version of the career portal (found on the releases page) you will instead want to clone down the career portal and start from there.

ah - ok great. Thanks for all your help. I think that will be the version I have because it was easy to put it onto my clients shared hosting account


Update xsalsa20poly1305 requirement from 0.7.1 to 0.9.0

Like #100 but with the required API change

Thanks @tarcieri


Kafka consumer is slow for real time data binding on UI if we are querying 4-5 topics.

In our use case we want to show some charts, metrices and grid based on Kafka topics data.( All Topics are already loaded with large amount of Json data from different systems )
We are planning to use Kafka connect and will sync topics data to Cassandra database.
Based on some trigger like any new data in Kafka topic will re-load UI and read same data from Cassandra (Via Dot Net core APIs) and display it on UI.
So is it good idea to use Kafka connect and sync data to Cassandra and query on Cassandra to load UI data Realtime.
Note : Reading data directly from Kafka topics and display on UI using Dot net Kafka consumer is very slow as in our use case we need to query 4-5 topics to get required data.
Using below consumer code but it seems to be slow -
private List<Dictionary<string, object>> RetrieveDataWithGenericRecordKey(CachedSchemaRegistryClient schemaRegistryClient, string topicName, CancellationTokenSource cts)
{
var processedResult = new List<Dictionary<string, object>>();
using (var consumer = new ConsumerBuilder<GenericRecord, GenericRecord>(consumerConfig)
.SetKeyDeserializer(new CustomAvroDeserializer(schemaRegistryClient).CustomAsSyncOverAsync())
.SetValueDeserializer(new CustomAvroDeserializer(schemaRegistryClient).CustomAsSyncOverAsync())
.SetErrorHandler((_, e) => Console.WriteLine($"Error: {e.Reason}"))
.Build())
{
        Dictionary<string, object> results = new Dictionary<string, object>();

        consumer.Subscribe(topicName);

        try
        {
            while (true)
            {
                try
                {
                    // consumer.Subscribe(topicName);

                    ConsumeResult<GenericRecord, GenericRecord> consumeResult = consumer.Consume(cts.Token);

                    if (consumeResult.IsPartitionEOF)
                    {
                        return processedResult;
                    }

                    Dictionary<string, object> obj = new Dictionary<string, object>();
                    var result = ProcessRetrievedResult(consumeResult);
                    processedResult.Add(result);

                }
                catch (ConsumeException e)
                {
                    Console.WriteLine($"Error occurred: {e}");

                }
            }
        }
        catch (OperationCanceledException)
        {
            // Ensure the consumer leaves the group cleanly and final offsets are committed.
            consumer.Close();
        }
    }

    return processedResult;
}

it is very resource intensive to create a new kafka consumer.
it looks like you want to write a function to read all data from a single partition topic.
to do that, you should create just one consumer instance over the lifetime of your application, and re-use it.
also, do not subscribe to the topic, assign to it instead (starting at offset 0).
i think this should perform adequately.

Thanks @mhowlett and any suggestions over using Kafka connect and sync data to Cassandra Db for UI binding ?

that sounds ok. i'm not sure what the read latency is on cassandra (would depend on consistency settings). you could also just use postgres or mysql - they can scale very well.

thanks for your input, need one more suggestion on Trigger part, for example if we sync data to Cassandra then we need one trigger point to load UI real time based on any new data in topic, so what's your view on same ?

it depends on how much data and how frequently...
you could also store data in memory in your application (use an LRU cache type arrangement), adding to it as you read memory off the kafka topic, displaying directly from memory. if you get data for a topic that's out of the LRU cache, then read the entire topic again.

ok, our topics are loaded with Json structured messages and each message  is having 30-40  objects\fields , and there could be 1000 * n no of  of messages in each topic and we need to query different topics to get desired data. so that's we are are planning to use kafka connect  + Cassandra so that we can read data from there ? so is it good idea ? or any better approach for same amount of data (For real time databinding on UI)


Use tokens for colors and fonts

Those tokens are already present in the TfBase component.
May be in the future, we will implement theming to allow personalisation of those tokens.


Alternative Backends | Support open table projects like Apache Iceberg

Hi,
First of all, thank you for the great project !
I was wondering if under "Alternative Backends" - can integrations with Open Table format like Apache Iceberg can be considered / added to the roadmap?

Hi @ramkumarkb ,
The next integration for prql-query will probably Polars. I would like to support Apache Iceberg and initially that will probably come through the DuckDB backend (who I believe are working on this) and possibly also Data Fusion.


update extension package to v 0.1.5

This PR

updates the extension version from  0.1.4 to 0.1.5 for package release to insiders

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.002%) to 42.47%

Totals

Change from base Build<PHONE_NUMBER>:
-0.002%

Covered Lines:
26784

Relevant Lines:
57118

💛  - Coveralls


Raincatch 1106 coveralls integration

Motivation
This PR is used to implement coveralls integration with raincatcher-core to provide a tracking system of our code coverage over time. An new command npm run test-ci is provided for running tests on Travis that will automatically upload.
Description

Edit multiple package.json files to produce coverage reports
Add lcov-result-merger to combine multiple reports - the lcov report contents will be uploaded by coveralls
Add coveralls integration to upload combined coverage report to coveralls
Edit .gitignore files to ignore coverage report folders
Add npm-run-all to main package.json to reduce the script length and provide script isolation. Composite scripts can be created and steps to tasks can be triggered easily
Edit .travis.yml file to run new test-ci rather than test, to allow test to be used for local development without trying to merge reports and upload to coveralls
Add removal of coverage_report folders to clean commands
Add Build and Coverage badges from Travis and Coveralls to README.md

Progress

[x] Edit package.json in multiple repos to provide support for lcov coverage output
[x] Added lcov-result-merger to combine multiple files
[x] Added ability to upload to coveralls in package.json
[x] Edited .gitignore files to ignore coverage report folders
[x] Added npm-run-all as dev dependency and created some new scripts and composite scripts in main package.json
[x] Edited .travis.yml
[x] Updated clean commands where necessary
[x] Added badges to README.md
[ ] Documentation

Additional Notes

Changes Unknown when pulling 168adbb93d3e5db047e68e219a28b45dc9343b22 on twnolan:RAINCATCH-1106_coveralls_integration into ** on feedhenry-raincatcher:master**.

Great work that actually can be seen in this PR comment made by coveralls extension.
All changes are ok. I think we just need to organize some of commands into scripts or grunt.

Changes Unknown when pulling 61c7b36b4bc1f3094bc47d6ff644bbf96307b35f on twnolan:RAINCATCH-1106_coveralls_integration into ** on feedhenry-raincatcher:master**.

@twnolan - comment about scripts is optional.

@wtrocki I agree that we should create a ticket to cover the scripts. You have suggested the use of either scripts or grunt here, so we should really know which way we are going before doing more work on this issue.

@twnolan - Ping me on ticket so I can add more details.
Feel free to squash this change back to repo!
Really good work.

Changes Unknown when pulling a8603b81616f0ca073e0ed6de4773399cabd5350 on twnolan:RAINCATCH-1106_coveralls_integration into ** on feedhenry-raincatcher:master**.