Issue with connecting to server

I'm trying to build the latest version of software and seem to have difficulties connecting to server.
I've shown the error and set up below:
$ ./app/acvp_app --all_algs --sample
Using the following parameters:

ACV_SERVER:     demo.acvp.nist.gov
ACV_PORT:       443
ACV_URI_PREFIX: acvp/v1/
ACV_CA_FILE:    certs/acvp.nist.gov.crt
ACV_CERT_FILE:  certs/CSR_athorath-corsec.cer
ACV_KEY_FILE:   certs/privateKey.pem

***ACVP [ERR][acvp_curl_http_post:323]--> Curl failed with code 6 (Couldn't resolve host name)
***ACVP [STATUS][log_network_status:885]--> POST Login...
Status: 0
Url: https://demo.acvp.nist.gov:443/acvp/v1/login
Resp:
(null)
***ACVP [STATUS][acvp_register:1396]--> Login Send Failed
***ACVP [ERR][acvp_run:2142]--> Failed to register with ACVP server

I tried to use "acvp/" for ACV_URI_PREFIX (excluded /v1/) but still can't connect.

$ ./app/acvp_app --all_algs --sample
Using the following parameters:
ACV_SERVER:     demo.acvp.nist.gov
ACV_PORT:       443
ACV_URI_PREFIX: acvp/
ACV_CA_FILE:    certs/acvp.nist.gov.crt
ACV_CERT_FILE:  certs/CSR_athorath-corsec.cer
ACV_KEY_FILE:   certs/privateKey.pem

***ACVP [ERR][acvp_curl_http_post:323]--> Curl failed with code 6 (Couldn't resolve host name)
***ACVP [STATUS][log_network_status:885]--> POST Login...
Status: 0
Url: https://demo.acvp.nist.gov:443/acvp/login
Resp:
(null)
***ACVP [STATUS][acvp_register:1396]--> Login Send Failed
***ACVP [ERR][acvp_run:2142]--> Failed to register with ACVP server

I've run the client before and there's no change in my keys or certs.
Curl version and LD_LIBRARY_PATH shown below:

$ echo $LD_LIBRARY_PATH
/home/amritha/cavp_tool/curl-7.59.0/lib:/home/amritha/ssl/lib
$ curl -V
curl 7.59.0 (x86_64-pc-linux-gnu) libcurl/7.59.0 OpenSSL/1.0.2p zlib/1.2.8 librtmp/2.3
Release-Date: 2018-03-14
Protocols: dict file ftp ftps gopher http https imap imaps ldap ldaps pop3 pop3s rtmp rtsp smb smbs smtp smtps telnet tftp
Features: AsynchDNS IPv6 Largefile NTLM NTLM_WB SSL libz TLS-SRP UnixSockets HTTPS-proxy

Curl failed with code 6 (Couldn't resolve host name)
Because your ACV_SERVER should be demo.acvts.nist.gov not demo.acvp.nist.gov
I think it is wrong in the nist_setup.sh script, I'll fix that.

@bfussell : Thanks for the comment. I've done that and I still an issue.
amritha@thorath-vm:~/libacvp/libacvp-master$ ./app/acvp_app --all_algs --sample
Using the following parameters:

ACV_SERVER:     demo.acvts.nist.gov
ACV_PORT:       443
ACV_URI_PREFIX: acvp/
ACV_CA_FILE:    certs/acvp.nist.gov.crt
ACV_CERT_FILE:  certs/CSR_athorath-corsec.cer
ACV_KEY_FILE:   certs/privateKey.pem

***ACVP [ERR][acvp_curl_http_post:323]--> Curl failed with code 60 (Peer certificate cannot be authenticated with given CA certificates)
***ACVP [STATUS][log_network_status:885]--> POST Login...
Status: 0
Url: https://demo.acvts.nist.gov:443/acvp/login
Resp:
(null)
***ACVP [STATUS][acvp_register:1396]--> Login Send Failed
***ACVP [ERR][acvp_run:2142]--> Failed to register with ACVP server

I tried to see if I can access the URL https://demo.acvts.nist.gov:443/acvp/login on browser and it says "Forbidden" and I'm guessing this has something to do with the email Jason sent out this morning after maintenance?

You definitely got passed the DNS name issue and now have a CA cert issue.
That CA cert doesn't work for me either. I always use the mozilla CA cert from the
certs/ directory

@bfussell : Thanks for the pointer about Mozilla cert.
I think I got past that error. And now I see this:

$ ./app/acvp_app --all_algs --sample
Using the following parameters:
ACV_SERVER:     demo.acvts.nist.gov
ACV_PORT:       443
ACV_URI_PREFIX: acvp/
ACV_CA_FILE:    certs/mozzila_trust_anchors.pem
ACV_CERT_FILE:  certs/CSR_athorath-corsec.cer
ACV_KEY_FILE:   certs/privateKey.pem

***ACVP [STATUS][log_network_status:885]--> POST Login...
Status: 404
Url: https://demo.acvts.nist.gov:443/acvp/login
Resp:
ErrorNot Found
***ACVP [STATUS][acvp_register:1396]--> Login Send Failed
***ACVP [ERR][acvp_run:2142]--> Failed to register with ACVP server

I'm trying to request sample files for all algs. Is there anything wrong with the args I'm passing which caused the issue?

You forgot to change ACV_URI_PREFIX back

BTW, the best way to get a full set of vectors is to use the new offline mode options.
/app/acvp_app --all_algs --vector_req req.json
This way the client won't bother trying to process them.

Oops my bad, I've corrected the ACV_URI_PREFIX.
So I've tried to use the command you gave in previous comment and I see:

{
"acvVersion" : "1.0",
"error" : "The supplied one time password could not be verified"
}

I know sometime back there was an issue with seed generation and null byte not being handled or something on those lines. I'm checking with NIST to see if the TOTP seed I received a few months back is still valid.
Meanwhile I wanted to check I didn't make any other mistake.
I tried to export ACV_TOTP_SEED with and without quotes.

I fixed my issue by restarting my system. The NTP time was not in sync.
Thanks for your help!!


Ford is moving too slowly and needs your help

Describe the bug
Ford's progress is too slow. I really hope you can expedite the process. We can provide Ford Edge, Mondeo, and Lincoln Nautilus 2020-2023 as test vehicles. I will try and report the test results. I have a friend who bought the C3X model MKX from China, but he can't get support. In the end, the equipment was damaged and I sent it to you for repair. A large number of Fords in China did not receive your support, all of which were Q3, requests from modification shops.
Provide a route where the issue occurs
ac88836411233dff
openpilot version
0.961
Additional info
No response

This isn't a bug - check out our Discord or GitHub Discussions instead.


Explicitly set csharp interpreter for mac packaging

The test are fixed by changing the VS version in #2114


Issue 718 refactor

Resolves issue #718 and enhances documentation of each refactored module.
Tests running successfully on refactored codebase, with the exception of the misimplemented (missing) test for the AI module

@captivus also, please fix the merge conflicts :)

@captivus congrats on the merge - great work!


Favicon on all the pages

Hey @nisha331, I would like to open this issue under JWOC. The favicon appears only on the main page. I'll like to add it on all the pages.

ok work on it


keeps saying error code 429 and failed to load sensapi.dll

But the thing about sensapi.dll is that when i delete bloxstrap roblox is fine and the 429 looks like    Error code 429

an error was encountered during authentication roblox

That's Roblox ratelimiting you, not a Bloxstrap problem.

But what do I do?

You wait a while. Back to this.

and failed to load sensapi.dll

Does this happen without Bloxstrap?

Nope only with bloxstrap

Let me clear up something - are you getting both errors with Bloxstrap? The message for Error Code 429 wouldn't show if the one for SensApi.dll shows. Either way, it might be worth forcing a Roblox reinstall? Try setting your channel to 'ZNext', and see if that changes anything.

Yeah those errors only happen with blox strap

where do i set my channel to znext

Does it work?

Yeah fixed thanks


add support for additional habitat packages on managed supervisor

User Story
As an Automate User, I will use this work when I want to run effortless config/effortless compliance.
Currently when running Automate 2, it manages its own supervisor and rejects any additional habitat packages. A requirement of our organization is that we have our effortless config (chef-client habitat package) managing the node that Automate is running on.
This causes a conflict as Automate will immediately unload the effortless habitat package.
Definition of Done
Have the ability to specify, or manually override the removal of habitat packages.
Demo Script / Repro Steps
Spin up Automate 2, try to hab svc load ...

We think this is a good idea and agree that we want to support people who want to use Habitat for other services (effortless config, log gathering services etc) on the same machine as Habitat.
Addressing this requires that we answer a few questions:

[ ] Are users OK with Chef Automate maintaining control of the hab-sup and hab-launch versions? Currently, we carefully manage these versions as Chef Automate requires specific Habitat behavior.

[ ] Are users OK with Chef Automate potentially restarting their other services on upgrade?  Right now, we control the systemd unit file for hab-sup and occasionally need to restart the entire hab-sup process tree when updates to the unit file are required:  https://github.com/chef/automate/blob/master/components/automate-deployment/pkg/target/local_target.go#L62-L84

[ ] Proxy variables.  Right now we set our HTTP_PROXY variables in the systemd unit file to ensure they are universally applied and to ensure that hab-sup itself always respected those variables.  My guess is that users who need an HTTP proxy for automate would also need it for other services so perhaps this isn't much of an issue.

[ ] How would we handle users who want to join their hab-sup to a larger habitat ring?

These are all great points, likely boils down to accepting the level of risk of running A2 anyways.
I can't answer for the team, but I can provide insight into the organizations that I've worked with in engagements where habitat is heavily used.
For the first two, some level of support would definitely be appreciated especially for the orgs that I've worked with. Anytime we need to run agents on A2 right now, we have to do some horrific templating of a systemd service where we wrap the packages run hook as the ExecStart -- something I would never want to maintain.
As for proxies, in most cases I've deployed A2 in environments where we use the air-gap bundle as opposed to opening up the proxy we rarely have to deal with that. However, a well written no_proxy should take care of most proxy related issues.
Finally, ring is very interesting and perhaps controlled with deployment config?

@stevendanna has there been anything else determined/designed on this?

@skylerto Unfortunately not.  I'll try to bring this up internally to get some movement on it.

transferred to aha idea


[Process RFC] Clarify Community Strategy Decision Process

rendered

+1

I'm in favor of this. A healthy community needs progress and keep updated to the latest trend, and a super majority vote is for now the best way we could think of to balance the need for stability while being against stagnation

+1

I was a bit confused, and would love to clarify my points first, and ask @leandron to clarify some of your points :-)

It might become too costly for some to keep up with gigantic puzzle of features that are just too complicated to make work.

To clarify, according to my read, this RFC is particularly designed to be extremely conservative that a decision making should get super majority (2/3) of the votes to pass. It means the will of a single person or identity is insufficient to push forward anything unless they convince most of the community members.
Intuitively, if a super majority of the community firmly believe in a progress to be made via formal voting, I would agree that this is a technical direction that the community believes in and should invest in.
Of course, upon any single commit of any code change, there will be less active contributors feeling left behind, and we do need to care about those contributors even if a super majority think the contrary. Therefore, I would propose to attach a guideline: any legacy code path, if conflicts with a new direction, it has to be preserved in a separate branch. To be clear, there is no case yet that a decision introduces incompatibility against the existing ones, but as PMC members, I should give sufficient emotional support to those who could not catch up.

On the flip side, many see value in having a stable architecture with a set of components we can evolve long term in a coherent way, that would require us to be mindful when bringing new alternatives to solutions we already have on the stack

I share the same sentiment with you that I wish some of my softwares never got updated, for example, LLVM breaks upon almost every release, and Cython just breaks recently which is a huge headache. Stability is indeed an important issue to me, and therefore, if a “stable architecture” already copes with new technologies out of box, I would personally prefer not to upgrade, just to save my own time.
Meanwhile, as a PMC member, I would take responsibility to a) hear the voice of super majority, and b) make concrete progress to keep the software updated to the latest challenge, which further becomes neither my personal will, nor stability concern wishing for zero change or zero break, but a grave concern on a) community stagnation; b) being out of touch from major usecases.

I understand the fact a subset of the community wants to bring changes faster into the TVM stack for the benefit of "a field in fast development" or to attract more contributors, etc.

I apologize in advance if I misread as a non-native speaker, but correct me if I was wrong, as a PMC member just like you, I was slightly surprised about your wording on this particular scenario. Given the context that this is an extremely conservative proposal that requires super majority to push forward any technical decision, I would love to have you re-examine the following assumption:

A super majority is described as "a subset of the community". Do you want to clarify if it's you believe super majority is representative in your reasoning?
The motivation of having new features, as you described, is to "attract more contributors" instead of merely coping with latest needs in the field. Do you want to clarify if you believe there is any need to cope with latest needs?

To summarize, @leandron, do you agree with me that we value community over code? I'm really grateful to have such a helpful and collaborative super majority of the community, and we concrete deliver for them all the time, making our solution as fast, robust and easy-to-use, solving community problems at large scale.

+1

The RFC is about how we operate as a community, and it's not necessarily related to "gigantic puzzle of features". To clarify, according to my read, this RFC is particularly designed to be extremely conservative that a decision making should get super majority (2/3) of the votes to pass. It means the will of a single person or identity is insufficient to push forward anything unless they convince most of the community members.
As a community of diverse needs, of course, we have different opinions on how we could drive our technical needs, which is a process issue of any community operation. If a super majority of the community firmly believe in one approach of how, I would agree and support it no matter what I personally believe in. The RFC is just about that the community needs a way to drive technical resolution.

I wasn't aware of these motivations, I think they should be stated in the RFC text.

I share the same sentiment with you that I wish some of my softwares never got updated, for example, LLVM breaks upon almost every release, and Cython just breaks recently which is a huge headache (and meanwhile, they constantly bring in coolest new features!). Stability is indeed an important issue to me, and therefore, if a “stable architecture” already copes with new challenges out of box, I would personally prefer not to upgrade, just to save my own time.

We are getting confused with the difference between a stable architecture (components that interact with each other), with API, which is the specific function calls and classes design to make something work. What I'm arguing is that we should prioritise evolving our existing components e.g. IRs, rather than come up with something new every couple years.
Cython and LLVM are great software with mature communities, being used in many production cases and - I'd argue - very stable.
This is probably one of the reason their contribution guides are clear w.r.t changes, for example Cython mention the development larger features in branches before they get into the main codebase (source) and LLVM has this note in their development guide: "we need to strike a balance between being inclusive of new ideas and people and the cost of ongoing maintenance that new code requires. As such, we have a general support policy for introducing major new components into the LLVM world, depending on the degree of detail and responsibility required. Core projects need a higher degree of scrutiny than peripheral projects, and the latter may have additional differences."
So, if said projects keeping control of their architectures break sometimes, imagine for project that are less careful with that, how would that be?

Meanwhile, as a PMC member, I would take PMC responsibility to a) hear the voice of super majority, and b) make concrete progress to keep the software updated to the latest challenge, which further becomes neither my personal will, nor stability concern wishing for zero change or zero break, but grave concern accordingly on a) community stagnation; b) being out of touch from major usecases.

I this this paragraph misinterpret the role of the PMC, which is not to "hear the voice of super majority". It says, instead, among other things: "to further the long-term development and health of the community as a whole, and to ensure that balanced and wide scale peer review and collaboration takes place." (source).
I think it is not welcoming and correct to say that there is this "super majority" that crushes everyone else's opinion. At least this is not the way I think it should be.

I understand the fact a subset of the community wants to bring changes faster into the TVM stack for the benefit of "a field in fast development" or to attract more contributors, etc.

I apologize in advance if I misread as a non-native speaker, but correct me if I was wrong, as a PMC member just like you, I was slightly surprised about your wording on this particular scenario. Given the context that this is an extremely conservative proposal that requires super majority to push forward any technical decision, I would love to have you re-examine the following assumption:

A super majority is described as "a subset of the community". Do you want to clarify if it's you believe super majority is representative in your reasoning?

A "subset" is exactly what it is.

The motivation of having new features, as you described, is to "attract more contributors" instead of merely coping with latest needs in the field. Do you want to clarify if you believe there is any need to cope with latest needs?

Sorry, I'm not aware of statistics in this area. I'm just representing a view I took from previous numerous discussions around this topic, that I referred in my previous post.

To summarize, @leandron, do you agree with me that we value community over code? I'm really grateful to have such a helpful and collaborative super majority of the community, and we concrete deliver for them all the time, making our solution as fast, robust and easy-to-use, solving community problems at large scale.

This text is already off-topic by a lot, so I suggest rather than posting passive-aggressive questions such as whether long term contributors of this project agree with "community over code", we focus on clarifying the RFC text, so that the community understands what is it that we're changing once this goes to vote.

It is clear from current conversations and past conversations that there are different opinions on how we should operate as a community.
These include what we should prioritize (e.g., “prioritize evolving our existing components, e.g., IR”), how we evolve core components, and how to “ensure long-term development and health of the community as a whole.”
It is also clear there are different interpretations of the same text; for example, some opinions about what approach qualifies as “ensure long-term development” or “evolving existing core components.”
This proposal comes with the observation that there is no single, dogmatic way or interpretation to approach all situations. There are many ways a project can be successful. There should be holistic consideration from a technical perspective, the current state of the AI/ML ecosystem, and the demand from the community.  This RFC do not suggest a specific path, or interpretation of the how.
Unfortunately, some of the hows, or interpretation of hows, are exclusive to each other when it comes down to the operational process. By stagnating, we are implicitly taking a path that is favored by some, but dismiss the opinions of others, sometimes the majority of the community.
It shall be the decision of the community collectively to choose the how(and path) to take after conversations.  That means each one of us should do our best to convey our idea of how (technical strategies with concrete technical reasonings, approach of supporting emerging needs, approach to maintain existing code) to the community. The community collectively chooses.
We would anticipate everyone to put the factors (such as long-term development, evolving our existing components, community growth, the need of stability on some modules) into consideration. Of course everyone would have their own interpretation of how these goals boils down to a concrete decision.
In short, this is a question of process for the community to collectively make decisions on how (and implicitly their interpretation to some extent), and that is what this RFC is about.

As a diverse community, there are clearly different opinions on rules and guidelines of community operation, and it's not unusually that people agree to disagree. As PMC members, it's our responsibility to take actions to navigate the way the community operates, hearing voices from the community, keeping our technical decisions up-to-date to the latest challenges in industry, and that is exactly why we need this process RFC - Clarify the rules and guidelines of community operation.

I think this paragraph misinterpret the role of the PMC, which is not to "hear the voice of super majority". It says, instead, among other things: "to further the long-term development and health of the community as a whole, and to ensure that balanced and wide scale peer review and collaboration takes place." (source).

Textual interpretation, just as @tqchen said, is usually subject to personal opinions. This is why we need a process RFC to clarify how decision making process could potentially incorporate community needs. This is how I interpret "community as a whole".

I think it is not welcoming and correct to say that there is this "super majority" that crushes everyone else's opinion. At least this is not the way I think it should be.

I see what you mean. Thanks for clarifying your opinion as a PMC member.

+1! I really like this proposal! The ability to move fast is crucial in adapting to the ever-evolving AI infra. I like the concept of lazy majority of binding decisions, it could facilitate the decision-making process and make sense to me.

Hi @leandron! Would you kindly consider posting the text changes you would like to see in order for you to lift your changes requested? This may help keep discussions concise as well as help others in the community understand your desires most directly.
If convenient to you, it could help discussion for these to be submitted as code changes suggestions in the github review. Reading from the document changes your propose would be helpful to avoid inflation of discussion on irrelevant topics that can frequently happen in committee and discussion threads.
Thanks for considering

I fully support this RFC to require a 2/3 majority to make strategic decisions in TVM.
I'm in favor of the RFC text as it is now. It's clear and concise, while precise and effective.

+1 for this proposal
This will allow the project and community to move fast given the nature of the space that the project is in.
Very clear description on what the expectation from the community and the project.
NOTE: I saw a comment about having "value in having a stable architecture with a set of components", this is can be achieve via Git management and releases. We should NOT block progress for OSS projects as if they are internal enterprise based projects that need permissions on some VPs or business leaders to make decisions.
Reminder that PMC IS the leadership of any ASF Top Level Projects. So member of PMC needs to be able to balance and switch hats as they are contributing to the success of the project.

We are getting confused with the difference between a stable architecture (components that interact with each other), with API, which is the specific function calls and classes design to make something work. What I'm arguing is that we should prioritise evolving our existing components e.g. IRs, rather than come up with something new every couple years.
Cython and LLVM are great software with mature communities, being used in many production cases and - I'd argue - very stable.

I do Not think "we" are confused. Coming up with something new every couple of years actually should be happening for open source projects. API stability is another dimensions of evolution and progression that we could manage via versioning and release management.
I would not recommend to compare Apache TVM, or ASF, with other OSS projects and ecosystem. They are different and we will never try to operate and follow other communities on how to operate.

+1
I support this proposal. We need to be agile to be competitive in this fast moving space of AI / ML ecosystem, while at the same time ensure we are building robust, stable software architecture. 2/3rd majority strikes the right balance between those two options.

Thank you everyone for your inputs so far. We have had many related conversations over the past year and get collective inputs on different views to approach this process. Based on the inputs so far over the past year, I opened a vote to bring forward the version that most in the discussion thread prefers so we can bring clarity to this process.
https://github.com/apache/tvm/issues/15521
Thank you

+1
I think this is a substantially improved proposal compared with the similar RFC from last year, being more concise and using a 2/3rds majority which feels appropriate.

+1

Link to the voting result thread
https://github.com/apache/tvm/issues/15618
https://lists.apache.org/thread/bd0ph9j98r6sjm3wtp9174zgqqfhskt6


Adding super apply conversion

Fixes #27

This has been included in 7.0.3. Let me know if any issues with the release. It has been awhile.


[delegations] Create a reusable workflow that can create a delegations metadata file and sign

Description
Consume this workflow in an external repository.
Ensure that this workflow can

Checkout a configured repository & branch  (root-signing and the ceremony branch)
Create the metadata file (or update)
Sign with a key
Create a PR with the updated the delegation file against the configured repository & branch

cc @kommendorkapten

Closed, because will be handled by the delegation

this should be decently handled by tuf-on-ci-delegate (to create the delegation) and theupdateframework/tuf-on-ci/actions/online-sign-targets action for signing in delegated metadata in a workflow


Fix markdown header in README

Fixes a typo introduced in https://github.com/actions-runner-controller/actions-runner-controller/commit/39484063746373ebe088796e57bcbaeb6fd4812e

LGTM


Add an API for drawing a box

Something like
box(WIDTH, HEIGHT, x, y, "TEXT")

where

box(10, 10, 3, 5, "Hello")

would render (assume X's are spaces):

XXXXXXXXXX
XXXXXXXXXX
XXXXXXXXXX
XXXXXXXXXX
XXXXXXXXXX
XXXHelloXX
XXXXXXXXXX
XXXXXXXXXX
XXXXXXXXXX
XXXXXXXXXX

This would be useful in the robot example but also for something like, say, a "screensaver", like a text string that bounces around the console.
But this is low priority, and honestly may be a very niche case.

If we want overlaying text, e.g. a GAME OVER message drawing over the screen, this can be gnarly, because of needing to understand and update colors, e.g. above if the X's are yellow and Hello is red, we need to change the color back to yellow.
This probably means this feature is too custom and is kind of a deal breaker, unless we come up with some way to render text to an intermediate string and then we can merge those two together.

It's possible we could come up with an API similar to bordered which just got added today.

I don't think this has ever come up for me needing it, so I'm not going to add an API for it.
However, if you need it, you can delegate to justified:
justified(CENTER, minWidth = 10) {
   repeat(4) { textLine() }
   text("Hello")
   repeat(5) { textLine() }
}


Missing Upsert on Update Entity

Using Entities connected with a relationships, JAORM execute an upsert if entity is not updated.
Same behavior should also be used (if feature is enabled)

Closed in #99


Errors not detected when g++ is not in system path (Windows)

I have correctly the Exec Path to match the location of g++ (mingw-w64) on my Windows 10 machine:
C:/Program Files/mingw-w64/x86_64-5.1.0-posix-seh-rt_v4-rev0/mingw64/bin/g++

However, the linter always reports "No Issues."
Alternatively, if I startup Atom with the g++ directory in the system path, the linter correctly detect errors. It seems that the Exec Path setting is not working?

I see you have a space in your path. That could be the issue. Could you try setting ExecPath again, wrapping your path in double quotes?

If I add double quotes around the ExecPath setting, it reports "No Issues" even when g++ is also in the system path. Weird.
I have tested the following values:
"C:/Program Files/mingw-w64/x86_64-5.1.0-posix-seh-rt_v4-rev0/mingw64/bin/g++"
C:/Program\ Files/mingw-w64/x86_64-5.1.0-posix-seh-rt_v4-rev0/mingw64/bin/g++
"C:\\Program Files\\mingw-w64\\x86_64-5.1.0-posix-seh-rt_v4-rev0\\mingw64\\bin\\g++"
C:\\Program Files\\mingw-w64\\x86_64-5.1.0-posix-seh-rt_v4-rev0\\mingw64\\bin\\g++
C:\\Program\ Files\\mingw-w64\\x86_64-5.1.0-posix-seh-rt_v4-rev0\\mingw64\\bin\\g++
"C:\Program Files\mingw-w64\x86_64-5.1.0-posix-seh-rt_v4-rev0\mingw64\bin\g++"
C:\Program Files\mingw-w64\x86_64-5.1.0-posix-seh-rt_v4-rev0\mingw64\bin\g++

Let me know if I can provide any more testing to help fix this problem.

I think I might know what the problem is. It's possible g++ is being given invalid parameters. Could you check the parameters reported in the javascript console? You can open the console using Ctrl+Shift+I on linux, but I'm not sure how to do it on windows. Could you report the parameters reported in the console? Also, try combining the parameters you see in the console and run them on cygwin or something and make sure g++ is happy with them. If g++ is given invalid parameters, it'll just say 'unrecognized option' and you won't get any errors reported by the linter.

I think you are onto something. The parameters are "-Wall", "-std=c++11", "-fmax-errors=15", "-I ", and the last one is the full path to the source file. When I run g++ from the command line using these parameters, that one that gives me trouble is "-I ". When that parameter is included, I get the following error message: g++: fatal error: no input files. If I remove the -I, the last parameter is properly detected as the input file.
I am using g++ version 5.1.0 if that makes a difference. Thanks.

Ok I think I know what the problem is. It looks like your include paths parameter is comprised only of spaces. To test this, could you make sure your include paths is completely empty and see if you get linting? One way to do this is to press Ctrl+A with the cursor inside the 'include paths' text box and delete everything. Once we know this is the problem, I will push an update that trims leading and trailing spaces from this parameter.

Interestingly, having "-I " as a parameter to g++ on linux doesn't cause problems - the linting still works fine. In any case, I will update the code to do the trimming.

Ok just pushed the update. Now if you look at the parameters, you'll find that having an empty 'include paths' or one comprised only of spaces means the "-I" parameter is left out. I would be grateful if you could update to the latest version and confirm that it's working so I can close this issue.

Thank you for your quick work! However, I just tried your update and have the same problem. The errors/warnings are displayed when g++ is in the system path, but not when it is just in the Exec Path. Also, the Exec Path must not contain double quotes around it. Here is the output from the developer console (main.js:148 and 149):
C:\Program Files\mingw-w64\x86_64-5.1.0-posix-seh-rt_v4-rev0\mingw64\bin\g++
["-Wall", "-std=c++11", "-fmax-errors=15", "C:\Source\Demos\FileDemo\createFile.cpp"]

Also, I checked to make sure the "Gcc Include Paths" field is empty (not spaces or paths).

This is interesting. Clearly, the "-I" parameter is no longer there, but you're still having problems. Could you try running the command by stitching together the parameters in the console? It all looks fine to me really, and it should work.

I think have have tracked to problem down. g++ needs access to its libraries, which are located in the same directory as g++. Therefore, if I call g++ without this directory in my path, g++ complains about missing libraries (dlls).
The issue is really a side effect of my system and not your linter. Sorry for sending you on a few rabbit trails there.
My setup for g++ using mingw-w64 is fairly common. A nice additional feature for other Windows users (and me) might be to have a setting to specify the path to g++'s directory, rather than g++ itself. Then the linter would call g++ in an environment that includes that whole directory. I don't know enough about the Atom (or the linter package) API to say that my suggestion is possible.
Thank you for being so responsive.

I think I have a solution for you. g++ has an option "-L" which allows you to add library directories. Could you try using that option to remedy the library issue? I think it's probably a good idea for me to add this option to the linter anyway. Please let me know how you get on.

The -L did not work in my tests. I think this parameter lets g++ know where to look for libraries that the linker needs to include in the executable, but the problem is that g++ does not have access to dynamic libraries that it needs to run.
From the command line, I need to update the path before I run g++
SET PATH=C:\Program Files\mingw-w64\x86_64-5.1.0-posix-seh-rt_v4-rev0\mingw64\bin;%PATH%;
Most Window's programmers open up a "developers command prompt" that adds the appropriate additions to the path at startup. I am not sure how you could do something like that from a plugin.

Hello again, and sorry for disappearing for a little while. I don't think it's a good idea to make the plugin modify the user's path variable. I think I'll add a note in the README for Windows users informing them that g++/gcc needs to be in the their path so it can find the libraries it needs. What do you think?

I agree, the plugin should not modify the user path. I was hoping g++ could execute with a custom path, without modifying the user path. I know you can modify the path for a specific command-line instance and I see there is a node.js variable, process.env, that supports this, but I am not sure the linter API is that flexible. Again, I have never written an Atom package.
At minimum, adding a note to the README is a great idea. It would also be nice to put a note for Windows users right in the plugin setting by the exec path.

Hello. It looks like the issue you brought up is slightly more complicated than we thought. Linter-gcc, like some other linters, uses a helper npm package called atom-linter to execute GCC and parse the errors. There has been a discussion on atom-linter's issue page about this, and in the end it was decided to wait for the issue to be fixed in Atom itself, as the existing solutions are not ideal. The atom-linter issue is here and the main Atom discussion is here.
Having this issue corrected in Atom itself makes much more sense, because a lot of packages run external commands, and all of these packages would be affected by the inability to use the  $PATH variable.
In any case, I will push an update this evening that cleans up linter-gcc's settings, and I will add a note to the execpath setting for Windows/OSX users. Also, I am reopening the issue. Let's leave it open until this issue is resolved in Atom.

Sounds good! Thank you for the links to the other relevant issues.

Hello. Just wanted to let you know that I've just pushed an update that makes linter-gcc detect command-line errors such as invalid flags. I was thinking this would help with the path issue because it should show up as an error. Let me know if it doesn't work. To trigger it, you can simply enter an invalid flag in the C++ flags field in the package settings.

That is very helpful, thank you!
Could the linter also output message if the GCC Executable Path is incorrect?
When g++ cannot be found by the linter, I get "No Issues." However, I notice that when linter-gcc tries to execute g++, the following message is processed as output:
"'g++' is not recognized as an internal or external command,
operable program or batch file.
"

My guess is that it does not trigger an error, because it does not have the word "error" in the string.

Hmmm that's interesting. On Linux, entering an invalid executable path triggers an atom error notification (see here for info). Internally, linter-gcc uses an npm package called atom-linter to run GCC and parse its output, so this should really be handled by atom-linter. Having said that, however, I will try to incorporate this functionality using another package to detect the presence of an executable in a platform-independent way.

@hebaishi atom-linter is just an interface to the underlying BufferedProcess and BufferedNodeProcess API exposed by Atom. and those classes too rely on Node core to determine if a file/folder was found or not.
If I had to guess, the user's shell is not passing correct exit codes

@hebaishi Just a side note that if it was a $PATH related problem then it's already been work-arounded in the latest atom-linter, https://github.com/AtomLinter/atom-linter/blob/master/CHANGELOG.md#420

@hebaishi Just a side note that if it was a $PATH related problem then it's already been work-arounded in the latest atom-linter, https://github.com/AtomLinter/atom-linter/blob/master/CHANGELOG.md#420

@steelbrain thanks for that. I will update linter-gcc's dependency list so the latest version of atom-linter is installed.

@steelbrain thanks for that. I will update linter-gcc's dependency list so the latest version of atom-linter is installed.

It works! In Windows when I don't have g++ in my system path, I now get a popup error message stating that g++ was not found, instead of a silent failure.

It works! In Windows when I don't have g++ in my system path, I now get a popup error message stating that g++ was not found, instead of a silent failure.

$PATH expansion now works correctly both on Windows and Linux and when linter-gcc fails to find the executable, a helpful error message is displayed.


Create traute-maven

Came to the conclusion that it's not worth to create a Maven plugin as Maven provides rather a restrictive model.
Adapted the test-suite to the Maven environment


add method getBoundingBox

I needed a method to create the data-structure "BoundingBox" as used in the Elasticsearch bounding_box-filter. Maybe this is of use for other people also?
Thank you for this library!

Thank you for your contribution. However, I think the "BoundingBox" is Elasticsearch only. This repo is only about the geohash, and should not include any other thing. Thank you. But this PR should not be merged.


Feature request: add commands: /gm1 /gm2 /gm3 /gm4

Added here https://github.com/0PandaDEV/Nextron/commit/65134961ae42fffa5afd83c4304b2248edb87aea


feat(foundryup): warn users if already installed via cargo

Motivation
Users often get confused if they previously installed forge/cast via cargo and now run foundryup to seemingly have no effect (because cargo bin takes priority)

Solution
Warn users if the location of the forge/cast binaries have cargo in them

Let me know where to send the fruit basket!


Resize h tags?

Why only resize the p tags with a media query and not the h tags? Should this be included?

We've noticed that the header sizes especially on smaller devices are not ideal, so we'll likely be revising those in the media queries. Thanks for calling this out.

These have been revised to hopefully not be quite so extreme on small devices. 6e10151b092b66a02152b543e87e8ab4eaa7a90d


Added Spongebob headshot and quote

yo u did not add a changelog comment , pls add it and I will publish accept your pull request.


[prototype] consteval maps

Superseeded by https://github.com/userver-framework/userver/commit/e0936b514c0669992ed7edb52dad3d08e8e7850b


peer auth: Only set permissive mTLS on the webhook port.

Changes
Rather than making mTLS permissive on all ports, only open up the one
port that needs to be accessed by the control plane. This places the
metrics and profiling endpoints back under mTLS being required.
This technically enables the minimal carve out needed, and has been how
we have been running the KNative webhooks successfully.

/kind enhancement
From my perspective it technically improves security - at least as far as auditing is concerned.

The purpose is to reduce the surface area of things running in the cluster that is not covered by mTLS. Whitelisting one port is better than whitelisting the whole service.
Release Note

Make mTLS permissive only on the webhook port. Other ports will go back to using the cluster default setting.

Docs
N/A?

Sorry - default email and CLA email are different and I always forget for Google repos.

Sorry - default email and CLA email are different and I always forget for Google repos.

/ok-to-test
/assign @ZhiminXiang @JRBANCEL @arturenault

/ok-to-test
/assign @ZhiminXiang @JRBANCEL @arturenault

Test failure looks unrelated (?):

2021/01/29 02:22:04 Error during command execution: unknown flag: --mesh

Test failure looks unrelated (?):

2021/01/29 02:22:04 Error during command execution: unknown flag: --mesh

Yeah, https://github.com/knative-sandbox/net-istio/pull/498 is fixing the issue now.

Yeah, https://github.com/knative-sandbox/net-istio/pull/498 is fixing the issue now.

/retest

/retest

/lgtm


Description of mptable.rs file

Please open this PR in https://github.com/codenet/understanding-rust-vmm/pulls


onOutputReceived getIsOutgoing and getChangeAmount is false

Hello
My use case is listening to onOutputReceived while also using the same wallet in monero-wallet-gui... when I deposit to my address onOutputReceived returns reasonable results
but when I withdraw via monero-wallet-gui onOutputReceived runs with getIsOutgoing and getChangeAmount false
I assume this is the expected behaviour and there is some other way to achieve what I am trying to do, can anyone point me in the right direction?

I haven't tried with this setup, but did you check if onOutputSpent() is called? This would be the outgoing spend.

seems to me onOutputSpent is not called at all when I do withdraw in monero-wallet-gui - I solved my case by avoiding use of monero-wallet-gui and will be using monero-wallet-cli instead with the sweep_all command, as this avoids creation of change and therefore no calling of onOutputReceived

Yeah I suspect because the keys are loaded in separate wallets which don't have access to each other's local state, this is expected behavior from monero-project.
A workaround could be to manually fetch the corresponding transaction in onOutputReceived() and check if it has an outgoing transfer, then you know the incoming output is change or an internal transfer.


Release should be automatically filled with build

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Spot -> Spot Consolidation

Tell us about your request
Karpenter should consolidate underutilized Spot nodes and replace them with smaller Spot nodes.
Tell us about the problem you're trying to solve. What are you trying to do, and why is it hard?
Currently, Karpenter does not support Spot->Spot consolidation. This can result in unnecessary costs, if a large spot instance is provisioned, and its workload diminishes over time.
Are you currently working around this issue?
Increased costs :(
Additional Context
The reason Spot->Spot is currently disallowed is because Karpenter uses the PCO strategy (price-capacity-optimized). Choosing the cheapest instance is problematic, since it may cause increased interruption rates. The PCO strategy helps Karpenter choose the cheapest instance that is unlikely to be interrupted imminently.
There may be a middle ground to strike. One thought is define an improvement threshold (e.g. 25%) to control this middle ground.

Consolidation looks for instances that could be replaced by cheaper nodes
We compare the potential cost reduction to the improvement threshold
If the improvement is enough better, we trigger consolidation.

This allows PCO to operate as intended, while avoiding the worst edge cases (e.g. m5.24xlarge that could be an m5.large)
This approach has nice effects for on-demand as well, as replacing a node with a marginally cheaper node may not be worth the disruption. Minimally, we'll need to do some investigation into what the right default value is. We could consider exposing this threshold to users as a follow-on in the future.
Attachments
No response
Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Closing as dup of https://github.com/aws/karpenter/issues/2741


Null pointer when using JunctionTable definition

Hi,
I get a null pointer exception (see below) when upserting an entity with a junction table.
@Entity
public abstract class AbstractPerson implements Parcelable, Persistable {

  @Key
  int id;

  @ManyToMany
  @JunctionTable(type = AbstractPersonPicture.class)
  List<AbstractPersonPicture> pictures;
}

@Entity
public abstract class AbstractPicture implements Persistable {
  @Key
  int id;
  String url;
}

@Entity
public abstract class AbstractPersonPicture implements Persistable {

  @Column(name = "referenced_user")
  @ForeignKey(
      delete = ReferentialAction.CASCADE,
      update = ReferentialAction.CASCADE,
      references = AbstractPicture.class)
  @Key
  int pictureId;

  @ForeignKey(
      delete = ReferentialAction.CASCADE,
      update = ReferentialAction.CASCADE,
      references = AbstractPerson.class)
  @Key
  int personId;

  String additionalField = "additionalField";
}

      Person person = new Person();
      person.setId(1);
      Picture picture = new Picture();
      picture.setId(1);
      person.getPictures().add(picture);
      database.upsert(person).blockingGet();

Where database is a ReactiveEntityStore.
java.lang.NullPointerException: Attempt to invoke virtual method 'boolean java.lang.Class.isInterface()' on a null object reference
    at java.lang.Class.isAssignableFrom(Class.java:545)
    at io.requery.sql.EntityWriter.updateAssociation(EntityWriter.java:679)
    at io.requery.sql.EntityWriter.updateAssociations(EntityWriter.java:625)
    at io.requery.sql.EntityWriter.insert(EntityWriter.java:441)
    at io.requery.sql.EntityWriter.upsert(EntityWriter.java:483)
    at io.requery.sql.EntityDataStore.upsert(EntityDataStore.java:280)
    at io.requery.reactivex.WrappedEntityStore$8.call(WrappedEntityStore.java:138)
    at io.reactivex.internal.operators.single.SingleFromCallable.subscribeActual(SingleFromCallable.java:35)
    at io.reactivex.Single.subscribe(Single.java:2517)
    at io.reactivex.Single.blockingGet(Single.java:2001)

I also tried settig type to PersonPicture.class instead of  AbstractPersonPicture.class but this resulted in the compile error
Error:Execution failed for task ':app:compileDebugJavaWithJavac'.
> java.lang.IllegalArgumentException: part '<any?>' is keyword

Hi thanks for reporting. It would be nice to be able to use PersonPicture.class in the type, however it can cause some strange issues with the compiler's processing framework as that class itself is being generated in that same processing round, however it may be possible.

Thanks for fixing this. Would it be also possible to insead of using int id in the Junction class, the following definition
@Entity
public abstract class AbstractPersonPicture implements Persistable {

  @Column(name = "referenced_user")
  @ForeignKey(
      delete = ReferentialAction.CASCADE,
      update = ReferentialAction.CASCADE,
      references = AbstractPicture.class)
  @Key
  @OneToOne
  AbstractPicture pictureId;

  @ForeignKey(
      delete = ReferentialAction.CASCADE,
      update = ReferentialAction.CASCADE,
      references = AbstractPerson.class)
  @Key
  @OneToOne
  AbstractPerson personId;

  String additionalField = "additionalField";
}

So far this would throw a mapping exception (afaik?), as one side is mapped with ManyToMany and the other with OneToOne. Or an exception will be thrown if no mapping annotation is found (leaving out OneToOne).
Because if we use the junction table with int we loose the ability to just say personPicture.getPersoon(), right?

There isn't a OneToOne relationship between PersonPicture.pictureId and PictureId, you can have many of the same PictureIds present in the PersonPicture table.

Thanks for the reply. The problem I am trying to solve is:
I want to mimic the behaviour of requeries automatic junction table generation, but with having full control/an extra column in the junction table.
So I was trying to have the comfortable ability to be able to call personPicture.getPerson() and not having to manually join/select the Person via the id in PersonPicture.class (suggested in my first post). My previous post was just me trying to achieve that and asking if you know how that could work.
I hope I made my question a bit clearer.

Sure understand you are looking to use an object reference in that junction table instead of the id.

Yes. But is that possible at the moment with requery?

Currently not, although this just a simple findByKey once you have the id

This fix is in version 1.2.0 thanks


Expose topology differences

Expose hwloc's ability to compute diffs of topologies that are similar but not quite identical: https://hwloc.readthedocs.io/en/v2.9/group__hwlocality__diff.html

I must admit that the use case for this (topology diff that only works in simple cases and errors out otherwise) is a bit unclear to me at the moment. That makes it hard to find motivation for binding it and inspiration for API ergonomics. If you use or would want to use this feature, please speak up and describe your use case!


ES6 template strings are not being highlighted

This is what I'm seeing:

I've tried this with a handful of themes using version c907b15. Is anyone else seeing highlighted template strings? It looks like this feature was added in #51.

Yep, this is working fine for me:

You have to alter your colorscheme for it to pick up javaScriptTemplate* ids introduced in #51:

This is a custom scheme I'm using so it will be different for you, but you get the idea.

just submitted a patch for tomorrow-night in case anyone is using that: https://github.com/chriskempson/vim-tomorrow-theme/pull/44

Ah, interesting. Thanks.
Rather than patching all of our themes why not highlight javaScriptTemplateString inside this repo? I added a line here to highlight template strings with String and it's working nicely.
Would be happy to open a pull request if that approach makes sense.

If it's helpful for anyone else who ends up here, adding this to your vimrc should take care of the highlighting for most themes: https://gist.github.com/sirbrillig/fc0630baf46a2c10667f


fix image problem + some editing

Fixes #7744

@CESARDELATORRE this still doesn't solve the image problem. Can I resize the images to make them fit better?

Sure! No issues at all about re-sizing the images! :)

I'll try to fix it adding some responsiveness, so it fits better to the device's screen.

@mairaw it's actually a css problem from the table the current markup generates, it can be solved with a little html/css and it would also be responsive, so the table would colapse to a single column for small devices.
NOW:

FIXED:

Is it ok to have html/css in docs? (ask because I saw the submitted one was removed)

BTW, this markup is different from the original one,

Thanks @mvelosop but that didn't fix the issue. See the page at https://review.docs.microsoft.com/en-us/dotnet/standard/microservices-architecture/container-docker-introduction/docker-defined?branch=pr-en-us-7777.
We avoid HTML as much as we can in the articles and I think resizing would probably be better in this case.

Couldn't get to see the preview (unauthorized). 🙁
Anyway, resized the images an reverted to regular markdown. If this doesn't work we could just go to a two rows table, instead of a two column one.
Best.

Yep, thanks @mvelosop!


I can provide additional diagnostics if there are no breaking changes that could do that and that I missed.

I could only think about the bbox change as a change which would hit the performance (8617a6b5d5ff147718051a7fc582c54f603aa90f). But this change is only applied in very edge cases and should not effect the normal flow (and even only in FF).
I would be very thankfull if you could track that down. Maybe you test commit by commit?
Note that some commits there are not stable because I missed to add a file - well - you will see it.

I saw lots of bbox-related stuff in the profiler. Let me reproduce it...

Meanwhile, you can try the script for yourself: https://gist.github.com/agladysh/df5d17ad7a0141a63e74
You can comment out update() call on line 943 to prevent it from hanging right away. Press Update button to render when you are ready.

Ugh. Now my profiler crashes :( Anyway, tell me if you want me to git bisect svg.js on my side. I should be able to do that this week.

To check for the bbox calls just insert a console.log into the new added try-catch-blog (commit above).
Do you deal with lots of elements in the defs when calling bbox?
Since I am busy with the textmorphing feature I have no time to investigate that further.
If you find the time a git bisect would be really cool!

Check out the latest commit which maybe fix this performance issues (0b5b0eb4842c0bad045a69981d00cba7010f8d31)

Fix worked, thanks!


Change servicePath in serverless.yml

This is a Feature Proposal
Description
For feature proposals:
For the case i would like to use bable to compile my code to another dest like "dist" folder.
and library can get the compiled source code and execute it.
Additional Data
like: babel services --out-dir dist/services

Please comment on my feature request and code.
This is my first open source pull request. = ]

@yoyeung thanks for contributing! @gertjvr and I have talked about this before. It's definitely a feature we need.
I've added some comments to your PR.

@johncmckim actually i am think about to integrate babel to this plugin.
is easy to add and watch babel logic.
in my design is look like use babel.transform to monitor src and dist folder.
Other config will use .babelrc provided by user.
Another step is monitor the serverless.yml.
When developer go to update js it will live compile
if developer change serverless.yml, it will live refresh.
Is it more close to the project goal? = ]

Hi @yoyeung
Firstly thanks for adding the service path change.
I have been using serverless-webpack combined with service path change in offline previously to achieve this.
I dont think plugins should have a single purpose, pulling in the babel or webpack isn't a foxus of this plugin.

@gertjvr Good to hear the more clear vision for this plugin

@gertjvr after research  serverless-webpack plugin. But it live compile and watch the src.
Am I overlooked??

Thanks for contributing @yoyeung. I've merged your PR. Will release an update soon.
I'm not exactly sure I understand your question. But if webpack or babel is re-building your dist file on changes, it will automatically use the newly built code. Please let us know if that doesn't answer your question.

@johncmckim yup ur correct.

Hi @johncmckim @gertjvr ,
I am not sure if I understand this correctly. But setting
simulate:
services: docker-compose.yml
dist: dist
My folder structure is /home/jenkins/git/src & /home/jenkins/git/dist
The dist folder contains src/index.js.
But when I do
./node_modules/.bin/sls simulate  --stage dev apigateway -p 5000 --lambda-port 4000
I always uses the servicePath to be /home/jenkins/git.
The config value dist does not take affect.
Thanks in advance.
-Ali


Set tcc source to mob branch

Hello the version that is downloaded for tcc is like 3 years old. I suggest to download the sources directly from the mob branch which is what people use for production nowadays.
https://repo.or.cz/w/tinycc.git
This is a special case because over the years numerous bugs we fixed and since this is a compiler, it has no excuses to have bugs if you are building anything meaningful with it. This project has gone into mob state long time ago and nobody knows if the maintainer (grischka) will ever do any releases. But he is very much active. So even if the mob comes to disarray state most of the problems get fixed very quickly. But somehow we still get people now and then reporting bugs that are past decade long on the mailing list because of using the old version.
--

Also, since this is a source based distro, that should give it even more incentive to pull the latest version.

Also, since this is a source based distro, that should give it even more incentive to pull the latest version.

For more about what is mob read here: https://repo.or.cz/h/mob.html

For more about what is mob read here: https://repo.or.cz/h/mob.html

To reach the maintainer of the package you have to ping them.
@ $(kiss-maintainer tcc)

To reach the maintainer of the package you have to ping them.
@ $(kiss-maintainer tcc)

I don't know their github username, I'll send him an email. But if not I guess @git-bruh is the curator here while dylan is gone? Or you mean something else by that @ $(kiss-maintainer tcc)? Kiss maintainer only gives me his email.

I don't know their github username, I'll send him an email. But if not I guess @git-bruh is the curator here while dylan is gone? Or you mean something else by that @ $(kiss-maintainer tcc)? Kiss maintainer only gives me his email.

 Should I do it myself? It's really not that hard to update the checksum and pick some git commit hash base off of.

@git-bruh And if he never responds? Should I do it myself? It's really not that hard to update the checksum and pick some git commit hash base off of.

@kyx0r yeah you can do it yourself and send a PR if the maintainer doesn't respond

@kyx0r yeah you can do it yourself and send a PR if the maintainer doesn't respond

@kyx0r I've been away from KISS linux for a while now, do you want to take over tcc maintainership?

@kyx0r I've been away from KISS linux for a while now, do you want to take over tcc maintainership?


Document AtlasTexture not supporting repetition

See #5660.
Note: Opened against the 3.2 branch as the way texture flags are handle in master is different.

Thanks!


System encountered a BMC error that policy table could not understand

I think we have an issue with the policy table or missing information to interpret.
Customer system got a SEL event:
12535  | Unknown       | 2020-02-11 13:56:24  | Unknown      | Critical  | error: Not found in policy table: Sensor.Device.Error.ReadFailure||
But it really is not telling us what sensor. When I go into the elog file I see the following for this event:
DICT_ENTRY "oa{sa{sv}}" {
OBJECT_PATH "/xyz/openbmc_project/logging/entry/12535";
ARRAY "{sa{sv}}" {
DICT_ENTRY "sa{sv}" {
STRING "org.freedesktop.DBus.Peer";
ARRAY "{sv}" {
};
};
DICT_ENTRY "sa{sv}" {
STRING "org.freedesktop.DBus.Introspectable";
ARRAY "{sv}" {
};
};
DICT_ENTRY "sa{sv}" {
STRING "org.freedesktop.DBus.Properties";
ARRAY "{sv}" {
};
};
DICT_ENTRY "sa{sv}" {
STRING "com.ibm.Logging.Policy";
ARRAY "{sv}" {
DICT_ENTRY "sv" {
STRING "EventID";
VARIANT "s" {
STRING "None";
};
};
DICT_ENTRY "sv" {
STRING "Description";
VARIANT "s" {
STRING "An internal BMC error occurred"
So I would expect the log message to say: Internal BMC error occurred.  That would help the field on how to proceed with troubleshooting.
I did notice that customer has the 1.14 openbmctool version. Could not get them to update the tool to see if error symptom changed, they got upset and is now demanding a new system.

@thalerj  Can you see if this is covered in the new policy table?

@yesyjim This case is not covered in the policy table as the expectation is that a sensor name or address would be provided here. @spinler How do you feel about rolling this into a generic event of an internal BMC error when we fail a policy table match?

@yesyjim do you also have the output from the elogall.log file that mentions the Sensor.Device.Error.ReadFailure?

@spinler  Sure I have the openbmc data in ecurep for that customer. 58119,057,649.
https://ecurep.mainz.de.ibm.com/ae5/#id=58119%2C057%2C649&path=58119%2C057%2C649%2Fmail20200212-000517-Joseph_Chen%2F
58119-elogall_log.txt


Git - generate commit message in editor

Add support for generating a commit message when the editor is being used to author the commit message.

This would be very useful for people attempting to automate git workflows :)

@lszomoru the closed issue mentions gitlens, but would this be a general commit message generator?
Also, what does "github.copilot.git.generateCommitMessage" provide currently? Where does that setting apply?  Is it the version control in the UI?


Add helpers to simplify and demonstrate functionality

add helpers that accomplish common tasks. This will serve as both demonstration and convenience. Derived from work of @st-wong

found out delegated harvesting is not really implemented yet. This is what is probably needed on our side as well https://github.com/nemtech/nem2-sdk-java/blob/master/sdk-core/src/main/java/io/nem/sdk/model/message/PersistentHarvestingDelegationMessage.java#L54


Update JUnitSuite.scala

Update JUnit test example. Fix example with updated references

@jackhclee  Looks good, thanks for catching it! @bvenners do you mind to review it for approval?
Thanks.


HttpClient should honor user-installed certificates on Android

See https://github.com/dart-lang/sdk/issues/48056 for context.

I've been experimenting with moving Flutter to use package:http.
That would allow the user to use package:cronet_http, which does honor user-installed certificates.
There are two other approaches that I can think of to solve this:

use JNI to ask the OS to resolve the certificate
make certificate resolution the responsibility of the embedder (i.e. Flutter)

That would allow the user to use package:cronet_http, which does honor user-installed certificates.
I think this will be problematic for people using Dart/Flutter on an embedded Android platform, that typically does not support Google Play Serives.

There are two other approaches that I can think of to solve this:

use JNI to ask the OS to resolve the certificate
make certificate resolution the responsibility of the embedder (i.e. Flutter)

Option 2 seems to be preferable as it opens up solutions for other platforms/emebedders.

I started a discussing on this in the hackers-framework-🔩 Discussion. So far the feedback is that depending on package:http seems like a bad idea.

I've updated the examples in cupertino_http and cronet_http to use package:cached_network_image.
At this point it should be possible to use those packages to with user-installed certificates.

I wrote a design doc suggesting that we recommend package:http as an HTTP client. Feedback welcome!

I presented this at the Dash forum. It's basically asking for feedback for the design doc. There seemed to be consensus so I'm moving forward on it.

There is still work to do in terms of documentation but think that we already have a solution in place.

Would like to know if this is in the roadmap at all?

Is there a bounty I can pay to get this feature? There are a few downstream apps that use your libraries and I'm running them completely offline.  Why can't I trust my own installed CAs just like the device does? Happy to donate 100 bucks just point me where I can donate.

There is still work to do in terms of documentation but think that we already have a solution in place.

@brianquinlan Is there any progress on this?

Any progress?

@spacekitteh @dasRicardo
I think that using package:cronet_http will solve your issue. Could you please try that?


Research: Feature Flags

Goal
Research how we can make use of feature flags to hide new features of the Lifecycle Controller.
Technical details
OpenFeature is a CNCF project that solves feature flagging in a vendor agnostic fashion with a reference implementation for a FF provider. We should look into the project and check how we can make use of it.
Acceptance Criteria

[ ] Evaluation at KeptnApp level should be enabled only when the FF evaluation-app is true.

DoD

[x] A PoC with some features behind feature flags.

References

https://github.com/open-feature
https://docs.openfeature.dev/
https://docs.openfeature.dev/docs/tutorials/overview
https://github.com/open-feature/flagd
https://github.com/open-feature/go-sdk

During a one day hackaton we had a chance to test and research OpeanFeature for KLT.  PoC #585 has been created as reference. A feature flag to enable different level of span has been implemented.
Research outcomes

Operator installation is straightforward as in the docs
The sdk is quite easy to use and the changes required for the operator ar only the followings:

Enabling open feature with an annotation
Initialising the provider in main
Setting up client and context in the controllers where the feature is needed
Preparing the config file with rules
To setup rules one can use any kind of json operator, it is useful to rely on https://jsonlogic.com/operations.html
As per today the only way to change a feature are either with rules ore changing the default value

Proof of concept to add ff for metric server https://github.com/thschue/lifecycle-controller/pull/8

this research is completed


Getting JWT Error while update profile and create new Article.

Hello there,
thanks for your contribution
i am new in Vuejs, for learning purpose i setuped this repo with laravel realworld backend
i am getting this error while update profile or create new article.
{"errors":{"message":"JWT error: Token is absent","status_code":401}}
Thanks

Hi @thebleshbanz
It seems that a problem somewhere in the laravel app (I am not a PHP developer, so I can not help more accurately)
Token is sent OK from the front-end


Add Basic Component - Quote

Request style guide for Quote component.

Current demo:

Not sure if we should add quote mark like this:

TASK COMPLETE!
The components are divided into two categories: Alignment and Type to clearly show the difference in style.
⤷ Here's the currently file in Figma.

Some annotation about Playground

Currently, the Navbar Menu and Quote have added Playgrounds to display the previewed components. Will present how to use it to everyone in the next meeting.

Request style guide for Quote component.
Current demo:
Not sure if we should add quote mark like this:


Compatible with Codec & Wasm32

Can you fix the merge conflicts please.

obsolete. Applied these changes to the snowbridge branch


[test] Try to reduce safari flakyness

Safari 12 is quite flaky right now. Keeping this PR open for a while with the occasional re-run to see if this change reduces flakyness.

Increased disconnect tolerance to 3 (up from 1)
Reduced no activity timeout to 10s (down from 5 minutes)

Superseded by https://github.com/mui-org/material-ui/pull/23292


AKS-RBAC to API server

Is there an existing issue for this?

[ ] I have searched the existing issues

Description
No response
New or Affected Resource(s)/Data Source(s)
No response
Potential Terraform Configuration
No response
References
No response

PR created: Integrate aks with azure rbac #35


Experimental support for PowerShell 7 and Linux (for non-build jobs)

Experimental Support
Setting the repo setting "shell" to "pwsh", followed by running Update AL-Go System Files, will cause all PowerShell code to be run using PowerShell 7 instead of PowerShell 5. This functionality is experimental. Please report any issues at https://github.com/microsoft/AL-Go/issues
Setting the repo setting "runs-on" to "Ubuntu-Latest", followed by running Update AL-Go System Files, will cause all non-build jobs to run using Linux. This functionality is experimental. Please report any issues at https://github.com/microsoft/AL-Go/issues

Running end-2-end tests with these changes here: https://github.com/microsoft/AL-Go/actions/runs/3682647177

End-2-End tests running Ubuntu here: https://github.com/microsoft/AL-Go/actions/runs/3684994328


Make mypy ignore CoreSchema's typing

But now we can't use mypy properly on pydantic.
I'm afk, but I might have another solution.


Enviroments without MySQL password can't install postleaf

Issue Summary
MySQL password is not always required to access to database And it even needs to be 8 chars length.
MySQL servers not always are password protected, into developing enviroments MySQL don't have a default password also (XAMPP, etc...).
In short, don't allow users install PostLeaf if their database dont have a password is kinda weird (this dont happens in most of CMS, Wordpress for example).
The restriction of request an 8 chars length MySQL password has also nosense, since this password is not setup by postleaf is a server side decision.
Steps to Reproduce
Simply install and leave the database password field empty.
Additional info

Postleaf version: 1.0.0 beta3
PHP version: 7
Affected browsers: All tested (Chrome, FireFox, Opera, etc)
Operating system: Tested on Windows and Linux (Windows running XAMPP and Linux running Apache+MariaDB+PHP7).

The eight character requirement is for the user's password, not the database password.
Although I strongly disagree with using a blank database password even in a dev environment, I understand that in some instances it may be appropriate or even beyond the developer's control, so I'll remove the check for the database password.
You can wait for the next patch version 1.0.0-beta4 or simply remove db-password from the array in install.php and run the installer again.


Some Bugs and Issues and Enhancements

Hello,
I found some bugs and enhancements that could be done:
Bugs:

There is a bug with the Connect-SPOService that the orgname does not correctly be inputted in the URL.
FIX:

Write-Output "Connecting to SharePoint Service"
        Connect-SPOService -Url https://$org_name-admin.sharepoint.com

Not using "" because this is not needed.
Enhancements:

Add a new module (this does not have to be installed when the other modules are installed! This is for Security & Compliance where also audits are available:

Write-Output "Connecting to IPPSSession..."
        Connect-IPPSSession

Add the feature to define the script folder. Now you are required to execute the M365Inspector tool within the folder and the Inspectors folder must be alongside.

Thank you,
I do like the addition of the Security and Compliance Center for additional visibility and inspectors - we can certainly add that and begin building out new features based upon that.
We can look into adding a custom script path as well for Inspector execution.
We can also remove the quotes from the URL string, but we have not seen any issues with the organization name not being input correctly, could you provide an example or error logs to show when/how that failed?

[UPDATE 15-03-2022: 16:50 GMT+1]
Here a list of bugs I found when the report was generated:

It seems that the first object get printed out twice. Not sure if this is because of me. But I have to look further into it.
Affected O365 Objects: frequently misses values or seem to have weird values. This is tested with custom inspectors. I do not know what the result would be with the regular inspectors and what their expected output would be. I need to research that as well. I made the custom inspectors conform the template that you presented.

Would you be able to give me the name of the finding for the first result? This changes based on the tenant's configuration, but if one of the inspectors is causing a duplicate value to be returned that should be an easy fix, or I may be able to explain why it appears to be a duplicate.

There are some more multiple bugs I found after fixing some issues: I can't get a specific script to work. It seems to throw this error:
Cannot find an overload for "Replace" and the argument count: "2".

====================Truncated=================

This is the error that also breaks the script. I do not know if it is because of the script itself.

But It happens when I have this script ran:

function Audit-GlobalAdmins{
$rolegba = Get-MsolRole -RoleName "Company Administrator"
$count = (Get-MsolRoleMember -RoleObjectId $rolegba.objectid)
if ($count.Count -ile 1 -or $count.Count -igt 4){
return $count.ToString()
}
return $null
}
return Audit-GlobalAdmins

It does output correctly when not running the inspector.

This function is already included in the tool under the ProperAdminCount.ps1 script - https://github.com/soteria-security/365Inspect/blob/main/Inspectors/ProperAdminCount.ps1
You are seeing this error because your script returns a System.Object array. You're casting an array of object properties to a string and overloading the replace switch in the conversion to the html report.

You would want to pick a property to return - eg, $count.DisplayName

Thank you!
I have merged the PR with the changes to the 365Inspect script to allow for the Security Center, a handful of inspectors will be inbound in the coming weeks to make use of that functionality.
At this time, we feel like a custom script path may introduce added complexity and will not be adding that to the base tool. This may change in the future.
The issues listed above also appear to be a result of the custom script written to include in the local copy of the tool in your environment, and not an issue with any of the existing scripts or the tool itself.


⚠⚠ FCC VIOLATION ⚠⚠ Recaptcha is being abused by clients to thwart downloading.

I've raised this issue many times with Envato. They use Google Recaptcha to limit the amount you're able to download. It isn't time based or how fast you download. It's based on the amount downloaded. For instance. If I download over 50 items in one day, they start to require a recaptcha every 15 items, and then it's reduced to 10, and then 5, and then every download requires a recaptcha.
I asked them about this, and they asked "why are you downloading so much?" What are you talking about? You have stock photos, videos, and fonts. They don't offer a way to download a preview, and they have no way of viewing fonts before you try them. Google Fonts would be ideal for them. So I am forced to download things that MIGHT work, only to be blocked by a recaptcha.
And it would be fine if it was a regular recaptcha but it isn't. It will start to slow down to the point that it takes on average 3 minutes to complete one recaptcha.
Envato lists "UNLIMITED DOWNLOADS" and that in itself doesn't matter in the fact that websites cannot throttle your data usage if your activities are legitimate. Which they are. I also use Google DNS, and since Google is an ISP, they aiding in an effort to throttle data which is extremely against FCC regulations.
Clear motive:

you are not utilizing any means other than traffic to determine your recaptcha.
you are selling a false product as it does not actually detect bots.
you are selling a product, and using the customers of clients to train your own neural network.
you are extorting both clients and their customers as well as allowing clients to throttle data usage.

The FCC's open internet rules protect and maintain open, uninhibited access to lawful online content.
The rules specifically prohibit:
• Blocking: Broadband providers may not block access to lawful content, applications, services
or non-harmful devices.
• Throttling: Broadband providers may not deliberately target some lawful internet traffic to be
delivered to users more slowly than other traffic.
• Paid prioritization: Broadband providers may not favor some internet traffic in exchange for
consideration of any kind. ISPs are also banned from prioritizing content and services of their
affiliates.
Since Google, Amazon, and some other ISP offer the broadband availability of these websites from their data center servers, they are in fact INDIRECTLY violating the rules set forth by the FCC.
Issue description

Environment

OS name and version:
PHP version:
Web server name and version:
google/recaptcha version:
Browser name and version:

Reproducing the issue

URL (optional):
Code (optional):

User steps

1. Visit page...

FCC Ticket: 5126910

Repo is just for PHP client issues. This is about the reCAPTCHA service.


[Bug] - Updated skip-nav Link

Signed-off-by: Adithya Krishna<EMAIL_ADDRESS>Description:
Fixed skip-nav component link at https://ux.redhat.com/components/

Thank you @adithyaakrishna !


Changed sleep time in multiThreadTryWriteLockTest

Fixes #1254
Changes proposed in this pull request:
Changed sleep time from 5 seconds to 1 second in cn.hippo4j.config.toolkit.SimpleReadWriteLockTest#multiThreadTryWriteLockTest

Check mailbox configuration when submitting. Contributor Guide

@tungbq I migrated the directory, which may cause conflicts, the develop branch under Merged should be fine

@tungbq I migrated the directory, which may cause conflicts, the develop branch under Merged should be fine

Thanks @magestacks, I've resolved the conflicts in latest patch, please help review again!


x-choyce randomly stops working

Hello,
installed xchoyce from latest sources and using openbox with compton
after reboot I start x:choyce in command line and everything works fine.
However after a while (dont know exactly, let's say 20 minutes) it stops working, meaning that pressing keyboard combination does not start up the selection. There is nothing in terminal, only the pressed buttons:
user@user:~$ x:choyce
^[	^[

Killing and restarting x:choyce does not help. Only hard reboot helps.
Any clue where to look?
Thanks

I have a guess, that there is something wrong with catching the shortcut;
I tried to set it to something that is surely not bind to anything else like "Alt+q"
I tried to XF86LogGrabInfo while pressing the xchoyce shortcut, this is the output
[  7126.177]   Printing all registered grabs of client pid 17764 x:choyce
[  7126.177]   grab 0x4540007b (core), type 'KeyPress' on window 0x119
[  7126.177]     detail 24 (mask 0), modifiersDetail 8 (mask 0)
[  7126.177]     device 'Virtual core keyboard' (3), modifierDevice 'Virtual core keyboard' (3)
[  7126.177]     core event mask 0x3
[  7126.177]     owner-events false, kb 1 ptr 1, confine 0x0, cursor 0x0
[  7126.177]   Printing all registered grabs of client pid 17771 xdotool key XF86LogGrabInfo
[  7126.177] End list of registered passive grabs

Because of how the gnome project evolves (in a more and more unix unfriendly way) it would be really cool if this project would get a new maintainer. Maybe I will try to promote the app itself, because there are many benefit even if you copy the gnome workflow 1:1! :) Nevertheless, thanks for the effort you put already into it!


FEATURE: Add new tag

Recently I've been noticing I've had to do a lot of updates to the README and CONTRIBUTING and such, which are not features. rather chores. I propose updating CONTRIBUTING with this new issue/branch type ("CHORES") so to avoid confusion with actual necessary features

All good


Feature FPCdoc: documenting the parallel model checker

This is a meta-issue for documentation of the parallel model checker. All issues on that feature are labeled with FPCdoc.

The current version of the parallel checker is massively outdated. We are planning another batch of work on parallelization, which requires the server mode.


Update csi-release-tools

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:
NONE

/lgtm
/approve


Highlight single node at top of page.

The note in the middle of the page highlights a very important limation for using Azure Disk with Kubernetes. As many users will want multiple nodes! I think this should be moved up into the intro paraph.
An Azure disk can only be mounted with Access mode type ReadWriteOnce, which makes it available to only a single AKS node. If needing to share a persistent volume across multiple nodes, consider using Azure Files.

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

ID: d1ac572f-c613-71fc-580a-2805b740b793
Version Independent ID: b4aeef15-7b0f-abc7-872e-28051e10d6ca
Content: Use Azure Disk with AKS
Content Source: articles/aks/azure-disks-dynamic-pv.md
Service: container-service
GitHub Login: @neilpeterson
Microsoft Alias: nepeters

@MSIH Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

@MSIH - thanks for the suggestion. I will work something up.

There is still a misstake in the note. The Azure Disk is mountable to multiple pods on one node, like as @MSIH already sayed in his initial issue message.
But currently the text says: "[...] which makes it available to only a single pod in AKS. If you need to share a persistent volume across multiple pods, use Azure Files."
It should be more: "[...] which makes it available to only pods on the same node in AKS. If you need to share a persistent volume across multiple nodes, use Azure Files."


New routing table interceptor

requires #10 to be merged in to https://github.com/kedacore/http-add-on/pull/206 before this

mistakenly opened this against kedacore/http-add-on, sorry! I need to open this against my fork.


Plugin error when starting Neovim on WSL2

Describe the bug
Starting neovim on WSL2 Ubuntu with noice.nvim installed with packer and received an error:
Currently using the default packer snippet to manage the plugin:
-- Packer
use({
  "folke/noice.nvim",
  event = "VimEnter",
  config = function()
    require("noice").setup()
  end,
  requires = {
    -- if you lazy-load any plugin below, make sure to add proper `module="..."` entries
    "MunifTanjim/nui.nvim",
    "rcarriga/nvim-notify",
    }
})

What version of Nevoim are you using?
neovim version: 0.8.0

To Reproduce
Start neovim in WSL2 Ubuntu
Expected behavior
Expected to have no error message on starting neovim
Screenshots
errors:

Noice log
output from running  :Noice log

That's odd. Please report an issue upstream for this.

It really seems you're not on 0.8.0.
vim.ui_attach does not exist in 0.7 and vim.cmd.whatever neighter.
Both are new additions to 0.8.0

Can you do :version inside Neovim?

Hi, I tried building neovim from source to use the latest version and it works now. It seems that 0.8.0-dev is behind 0.8.0? Either way, thanks for the help!


[FLOC-3021] Backport ZFS 0.6.5. fix to release 1.3.0

This patch needs to be applied to the 1.3 branch in order for tests to pass, so that we can release 1.3.1 for other significant bugs.

The change looks good. I'm happy for this to be merged when the tests pass.


Wrong return type for BINARY input

Hi,
This function should return 'the same type as string_expression' but for BINARY input I get VARCHAR as a result. Is it documentation or product issue?
declare @vb varbinary(10) = 0x01
declare @res sql_variant = REPLICATE(@vb, 2)
SELECT sql_variant_property(@res, 'BaseType')
Expected type is 'varbinary', the same type as @vb but actual result is 'varchar'. Is it possible to get VARBINARY result from REPLICATE function?

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

ID: 826d7873-b880-f00e-3bf6-5c2fc2b6ec91
Version Independent ID: dfe47828-8fd7-4f0a-6d40-25688eea46b4
Content: REPLICATE (Transact-SQL) - SQL Server
Content Source: docs/t-sql/functions/replicate-transact-sql.md
Product: sql
Technology: t-sql
GitHub Login: @MikeRayMSFT
Microsoft Alias: mikeray

@Al-ex-and-er Thank you for the feedback, assigning this to the author for review.
For product-related issues, please start a thread here.

@Al-ex-and-er , @msebolt , and @MikeRayMSFT : this is definitely a bug, but not of the return type. Hence this cannot be fixed by merely changing the return type to VARBINARY, or stating in the doc that VARBINARY will be returned as VARCHAR. I believe the issue to be that VARCHAR is returned only due to the input value being converted to VARCHAR within the function, or on the way in.
-- DB collation = Latin1_General_100_BIN2_UTF8
DECLARE @3bytes VARBINARY(5) = 0x41DC21,
        @4bytes VARBINARY(5) = 0x41DC21DC;

SELECT @3bytes AS [@3bytes],
       DATALENGTH(@3bytes) AS [num bytes @3],
       CONVERT(VARCHAR(10), @3bytes, 0) AS [@3 to VC],
       REPLICATE(@3bytes, 3) AS [repl @3bytes],
       DATALENGTH(REPLICATE(@3bytes, 3)) AS [num bytes repl @3], -- 9

       '--' AS [---],
       @4bytes AS [@4bytes],
       DATALENGTH(@4bytes) AS [num bytes @4],
       CONVERT(VARCHAR(10), @4bytes, 0) AS [@4 to VC],
       REPLICATE(@4bytes, 3) AS [repl @4bytes],
       DATALENGTH(REPLICATE(@4bytes, 3)) AS [num bytes repl @4]; -- 9?? not 12!!!
                              -- or even 11: A�!�A�!�A�! (i.e. 0x41DC21DC41DC21DC41DC21)

returns:
@3bytes    @3 num   @3 to   repl         num bytes --- @4bytes     @4 num   @4 to   repl       num bytes
           bytes    VC      @3bytes      repl @3   ---             bytes    VC      @4bytes    repl @4

0x41DC21   3        A�!    A�!A�!A�!   9        -- 0x41DC21DC    4       A�!   A�!A�!A�!   9

As you can see, for @4bytes, REPLICATE repeated 0x41DC21 instead of 0x41DC21DC. If the conversion to VARCHAR happened upon being returned, then at most the final byte (invalid UTF-8 sequence of 0xDC) would have been truncated, but not the intermediate instances of that byte as those are followed by valid UTF-8 sequences (hence why that same byte shows up as � when followed by "!").
Hence, the two options appear to be:

Fix REPLICATE so that a VARBINARY input value is not converted to VARCHAR internally, has all bytes replicated, and is returned as VARBINARY (much preferred), or
Update the doc to state that binary input values will be converted into VARCHAR based on the code page associated with the current database's default collation, and that the output might not be a replication of the exact sequence of input bytes. (easier to implement, but far worse user-experience for both functionality and documentation).

Take care,
Solomon...
https://SqlQuantumLift.com/
https://SqlQuantumLeap.com/
https://SQLsharp.com/

@srutzky Thank you for such a detailed analysis. You confirmed my fears. How to raise it as a bug?
I think it is worst to mention this in documentation anyway.

@Al-ex-and-er , @msebolt , and @MikeRayMSFT : Unfortunately, I need to correct my previous statement(s). I have done additional testing across most other string functions and have found that this behavior is common among most of them.  To summarize, this behavior is:

If there is one "string" parameter and VARBINARY is passed in, it is assumed to be VARCHAR in the code page associated with the database's default collation.
If there are multiple "string" parameters:

If all string params are passed in as VARBINARY, they are assumed to be VARCHAR
If one or more params are passed in as VARCHAR and none as NVARCHAR, the VARBINARY is assumed to be VARCHAR
If any string params are passed in as NVARCHAR, the VARBINARY is assumed to be NVARCHAR

As far as I can tell, the only string function that will return VARBINARY given VARBINARY input is SUBSTRING.
So the problem is really that the documentation isn't specific enough. The statement:

string_expression can be either character or binary data.

is not incorrect. It just doesn't state that binary data will be implicitly converted to VARCHAR. If this were to be stated, then getting VARCHAR output for VARBINARY input makes sense, and thus the statement:

Returns the same type as string_expression.

is also not incorrect.

@MikeRayMSFT It appears the link contains the information you need, the second comment (shown below) has more to consider as well for a minor update:
I suspect that REPLICATE is expecting a string not a binary type, and an implicit conversion is taking place from VARBINARY to VARCHAR. If you update your example like so you get the expected type:
declare @vb nvarchar(10) = '#'
declare @res sql_variant = REPLICATE(@vb, 2)
SELECT sql_variant_property(@res, 'BaseType')
Even though it accepts a binary input, the spec does specifically state that parameter is a "string expression" which is why I suspect type coercion is happening before the function is actually run. By "expected type" I am referring to it not being fixed as VARCHAR irrespective of input type which might be implied when looking in your result on its own. So I think the documentation is correct, but perhaps not perfectly clear: the function returns the same [N][VAR]CHAR() type that is receives, and in your test it is receiving a VARCHAR() due to implicit conversion from VARBINARY to VARCHAR.

Thanks, everyone for the detailed research and analysis (and suggestions) here. @juliemsft Could you please look more into this issue? Thank you! #reassign:juliemsft

Hello, because this is a design change request, instead of entering a comment on docs, it's best to enter it on UV at https://aka.ms/sqlfeedback as an improvement suggestion.
As other users recognize the usefulness of such requested change, and vote on it, we'll accrue the necessary support to move it fwd. Thank you

@Al-ex-and-er Hello, thank you again for contacting Microsoft. We have updated the document to reflect the current behavior. If you would like to see this behavior changed please submit your suggestion at https://feedback.azure.com/forums/908035-sql-server. I will now close this issue. The @notification mechanism continues to work even after an item is closed, in case further communication is needed. We hope to hear from you again.

#please-close


Permit the passing of config from a variable (or pipe)

Don't force to have a config file (default .weberarc) if needed data can be provided as arguments.
Related: #10

Being able to pass the configuration as a string should suffice for the time being.


[WIP] Refactoring telemetry

Making metrics direct part of the library
Separate traces/metrics/logging

@phatboyg Looking for initial feedback


cymbling, busing (logical-conflict)

This pull request was generated by the 'mq' tool

/trunk merge


[release request] 3.0.0

If the update is ready, please release the ipa for us :)

@khanhduytran0 @hugeBlack Now that many serious bugs have been fixed and it is very smooth to use, I think it is time to release the version 3.0.0


feat: add new column for activation id to the list and details page

Pull Request Summary
This PR adds Activation ID column to Rulebook Activation page and details page for each activations:

This Pull request provides

Enhancement

Additional information
Currently, the only way to check the Activation ID is to look at the number in the URL in the Web UI or use the API.
The Activation ID is important information when looking for Activation Job Pods or Service resources created by the EDA Server, or when calling some process with the API, and it would be nice if it were displayed in an easy-to-see location in the Web UI.
However, since this information is not necessary for all users, it is displayed in a less conspicuous position in this PR, especially on the list page.
I know this is not necessarily a change that will be acceptable to all, so feel free to reject this if you don't like this change 😃

@jamestalton @lgalis
Hi, thanks for moving my PR forward!
Do you have any ideas as to the cause of the failure of the CI?: https://github.com/ansible/ansible-ui/actions/runs/5841715214?pr=777
I am not very familiar with Cypress, but I find it hard to believe that the failure was caused by the changes I made to my PR.
Anyway, I have rebased my branch. Thanks in advance.

The Cypress tests are failing because of GitHub security settings that are keeping the test jobs from access repository secrets. We will review, verify and then we can merge it.
Thanks for the PR!


Add TableA7ValidatorTest to the code base

TableA7Validator needs unit tests written.

Test is written and complete.  This ticket is no longer needed.


Run integration tests on dependabot updates so that every bump gets updated

Look at chaining workflows to run integration tests more frequently than nightly - maybe for every PR.

Closing this.
If pyrsia/pyrsia-integration-tests#35 is solved, this would be also implemented.


Fix HDFS node for Integrated auth

Fixes https://github.com/microsoft/azuredatastudio/issues/12905 - the casing was different so the check wasn't being done properly.

Pull Request Test Coverage Report for Build 305216443

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.9%) to 41.622%

Totals

Change from base Build 304822178:
1.9%

Covered Lines:
19319

Relevant Lines:
42569

💛  - Coveralls


Add instant search / search shortcut

Kind of issue

[x] Feature
[ ] Bug
[ ] Design
[ ] Other, namely:

Description
We can make searching easier on the website by adding either of the following:

Automatically start searching when the user starts typing (letters or numbers) anywhere on the website.
Add a key/key combination to focus the search input. This could be Ctrl+f but unless we support exact search I think we should keep support for browser search.

I personally like the former. Any opinions from others (@simple-icons/maintainers)?

I'm highly against highjacking of the built-in functions of browsers. Very disrespectful to the user imho.
I like the idea that the user can just type anywhere and it will search.
One more unobtrusive variant would be to search by typing "/" anywhere first – just like GitHub does.

owing to an apparent disinterest in this feature, can we close this ticket?
I personally think that it is neat, but really not necessary. It's a flourish that we can easily live without.


prometheus-redis-exporter: 1.37.0 -> 1.43.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/redis_exporter/versions.
meta.description for prometheus-redis-exporter is: "Prometheus exporter for Redis metrics"
meta.homepage for prometheus-redis-exporter is: "https://github.com/oliver006/redis_exporter"
meta.changelog for prometheus-redis-exporter is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.43.0 with grep in /nix/store/5ab4m8sbwc4vz36gvj1jn4s1akzrxvix-redis_exporter-1.43.0
found 1.43.0 in filename of file in /nix/store/5ab4m8sbwc4vz36gvj1jn4s1akzrxvix-redis_exporter-1.43.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
prometheus-redis-exporter

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/5ab4m8sbwc4vz36gvj1jn4s1akzrxvix-redis_exporter-1.43.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A prometheus-redis-exporter https://github.com/r-ryantm/nixpkgs/archive/f71a3a13bf158dea6e8969fc1bc2f66c5830f3e8.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/5ab4m8sbwc4vz36gvj1jn4s1akzrxvix-redis_exporter-1.43.0
ls -la /nix/store/5ab4m8sbwc4vz36gvj1jn4s1akzrxvix-redis_exporter-1.43.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

prometheus-redis-exporter

Maintainer pings
cc @eskytthe @srhb for testing.

addressed by #186495


Missing Info & Enum classes

In fact the documentation tells the following

Four types of metric are offered: Counter, Gauge, Summary and Histogram.

Nevertheless a few lines below it shows the classes Enum and Info, which are not available for me, so from prometheus_client import Info results in ImportError: cannot import name {Info, Enum}. Am I missing something?

A release hasn't been made with these changes yet.


It does not work with an item like a totem of undying in the offhand

Right-clicking with a totem of undying for example does absolutely nothing, so it should work with the mod. Currently you have to remove the totem from the offhand so both hands are empty and then you can copy the text

Patched in v1.2 (forked)