docs, probes: simplify doc

What this PR does / why we need it:
The cloud init user-config configuration is modified to start the guest
agent and create the readiness file for the probe.
As a result, the instructions to manually perform this after the VM's
boot was removed.
Special notes for your reviewer:
Must still assure this works; will update the PR in the coming days.
Release note:

NONE

Rebased latest main on this forced push.

Re-written commit msg in this forced push.

Above the VM YAML, we have a note:
**Note:** The Fedora image used in this example does not have the qemu-guest-agent available by default. We need to install and enable it through the console until this is resolved.
The cloud-init defined below will take care of the install for us, but we still need to enable it.

Perhaps it should also be removed. Also looks good to me.

Looks great now!
\lgtm

Looks great now!
\lgtm

@marceloamaral I think (understatement) the bot does not recognize backslash :)


French kerning

Description
I'd like to request an option in XeLaTeX and LuaLaTeX to mimic the French kerning option. This is available in regular LaTeX through the microtype package, but not XeLaTeX. Essentially it adds a full space between the previous letter and a colon, and a half space between the previous letter and a semicolon, exclamation mark and question mark. I suspect this could be done by modifying the XeLaTeX config file but I am too much a newbie to know how to do that on a glyph-by-glyph basis. And it would be good to have it available for everyone.
Motivation: When French kerning has come up on stack exchange there are howls of protest that it is just wrong. I disagree. I have looked at many well-typeset books from the past and "French" kerning is the standard for publications in English, and was so well into the 20th Century. In my own case I want to produce in XeLaTeX a typographically similar version of a philosophical text from David Hume, so that a modern reader gets the feel of what it was like to read the book as the author approved it in 1768.
This is my first post so apologies in advance if it breaks protocol in any way. Thank you for fontspec -- it is a great piece of work.

Thanks for writing. In an ideal world I believe this would be a font option, but I've never heard of a font designer implementing this. Unfortunately while fontspec is cross-platform (both XeTeX and LuaTeX) there is no built-in way this could be done. It would almost certainly be possible in LuaLaTeX by adjusting glyph metrics but I don't have code to demonstrate... In XeTeX the best approach (I think) would be using inter char classes. (See package interchar.) Again, not something that fontspec can manage internally. So I will have to close the issue here, but good like finding a good solution!

I know that the font Junicode provides French kerning via a style set. However, I don’t know of any other font that does this.


added wrapper function for json parsing

Notes for Reviewers
This PR fixes #8496
Added a wrapper function for json parsing
Signed commits

[x] Yes, I signed my commits.

🚀 Preview for commit 00eeb6b19fc07dbe9239ebf8aef56ef31ab929ac at: https://64e31b7b7de90c60886c42f1--meshery-docs-preview.netlify.app

@leecalcote so I have to wrap every JSON.parse except the docs, right?

@leecalcote so I have to wrap every JSON.parse except the docs, right?

@EraKin575 you have to execute on the original request by making changes to Meshery UI.

🚀 Preview for commit 2e9e30bef184f91c0369085ca7490b68ab9d39aa at: https://64e49194dafe34027b6e5472--meshery-docs-preview.netlify.app

@EraKin575 Let's discuss this on Meshery Dev call. Please add this as an agenda item in the meeting minutes if you would. :)

sure thing @vishalvivekm

@EraKin575 merge conflict...

🚀 Preview for commit efd06baf552c9ea5e951adc67c8f3f8c5379edb1 at: https://64fcba2f130f941068f25ddf--meshery-docs-preview.netlify.app

🚀 Preview for commit fc508dba5b7e312a7ec1919671b5879bdbbbea24 at: https://64fd8f6bfa00dc0452fbc0af--meshery-docs-preview.netlify.app

@aabidsofi19

🚀 Preview for commit 9a5c41eee00c1852ec781f566ff19b7bd7560b91 at: https://6502c3245cd4a732161e7251--meshery-docs-preview.netlify.app

@EraKin575 there is still docs changes which is not needed, revert any change made to docs

🚀 Preview for commit d9b11d4a41f69e9ed9794acf0b164c41b53d4507 at: https://6506b49b0c4f2c173f04bda9--meshery-docs-preview.netlify.app

🚀 Preview for commit 0ce4909521c326927b1ec38b671714555d473585 at: https://6506b57c30d1f517158a9d19--meshery-docs-preview.netlify.app

🚀 Preview for commit b13aeb390bd02e547ab857e63f1490c2ad429932 at: https://6506c49dfbb37b1e97cf0b5d--meshery-docs-preview.netlify.app


Implement Motorbike and Truck Classes

Add new classes Motorbike and Truck, extending the Vehicle class, and ensure they include unique attributes and actions as specified in the project requirements.
Acceptance Criteria:
Motorbike class should be fully functional with prompts for details specific to motorbikes.
Truck class should allow for additional functionality unique to trucks.

The Motorbike and Truck classes have been implemented as extensions of the Vehicle class, each with unique attributes and actions as specified. Here are the key points from testing:

Motorbike Class: The Motorbike class is fully functional with prompts for details specific to motorbikes, ensuring accurate data collection and handling.
Truck Class: The Truck class includes additional functionality unique to trucks, as required, enabling users to access truck-specific features seamlessly.

All acceptance criteria have been met, and the new classes work as expected. Based on these results, I am closing this issue as resolved.


[Flake] pubsub batching_publisher_connection_test failed (macOS)

https://source.cloud.google.com/results/invocations/10e831ca-7a07-4639-bdf6-5fa69cfe4480
134/333 Test #169: pubsub_internal_batching_publisher_connection_test ........................***Failed    0.85 sec
Running main() from gmock_main.cc
[==========] Running 12 tests from 1 test suite.
[----------] Global test environment set-up.
[----------] 12 tests from BatchingPublisherConnectionTest
[ RUN      ] BatchingPublisherConnectionTest.DefaultMakesProgress
../../../../google/cloud/pubsub/internal/batching_publisher_connection_test.cc:54: Failure
Mock function called more times than expected - returning default value.
    Function call: AsyncPublish(56-byte object <08-44 B7-01 01-00 00-00 00-00 00-00 00-00 00-00 00-00 00-00 00-00 00-00 01-00 00-00 04-00 00-00 F0-9C 40-CA D2-7F 00-00 A0-4D 60-CA D2-7F 00-00 00-00 00-00 FE-7F 00-00>)
          Returns: 16-byte object <00-00 00-00 00-00 00-00 00-00 00-00 00-00 00-00>
         Expected: to be called once
           Actual: called twice - over-saturated and active
unknown file: Failure
C++ exception with description "Operation not permitted on an object without an associated state." thrown in the test body.
[  FAILED  ] BatchingPublisherConnectionTest.DefaultMakesProgress (282 ms)

https://source.cloud.google.com/results/invocations/71ac87ad-b0b3-43f2-b1cf-8ab368bf5d4e

A similar looking error:
132/332 Test #165: pubsub_internal_batching_publisher_connection_test ........................***Failed    0.70 sec
Running main() from gmock_main.cc
[==========] Running 12 tests from 1 test suite.
[----------] Global test environment set-up.
[----------] 12 tests from BatchingPublisherConnectionTest
[ RUN      ] BatchingPublisherConnectionTest.DefaultMakesProgress
../../../../google/cloud/pubsub/internal/batching_publisher_connection_test.cc:54: Failure
Mock function called more times than expected - returning default value.
    Function call: AsyncPublish(<
topic: "projects/test-project/topics/test-topic"
messages {
  data: "test-data-1"
}
>)
          Returns: 16-byte object <00-00 00-00 00-00 00-00 00-00 00-00 00-00 00-00>
         Expected: to be called once
           Actual: called twice - over-saturated and active
unknown file: Failure
C++ exception with description "Operation not permitted on an object without an associated state." thrown in the test body.

https://source.cloud.google.com/results/invocations/c40e3036-9c07-4216-809c-d194916db410


Use-case icons in left colum too large

^ these should have the same size and margins as the post/search icons or the title-icon above (pic below for reference)

we can tackle this issue once the original icons dont include padding within the svg


Problem with overridden properties that depend on other properties

I have a following-ish setup in my project.
An abstract class with some virtual properties:
[JsonObject(MemberSerialization.OptIn)]
    public abstract class Base: IInterface1, IInterface2
    {
        [JsonProperty(PropertyName = "id")]
        public virtual string IdString { get { return IdInt.ToString(); } }

        [JsonProperty]
        public virtual int IdInt { get; set; }
    }

And a class that inherits it:
[JsonObject(MemberSerialization.OptIn)]
 public class NewClass : Base
 {
     [JsonProperty(PropertyName = "id2")]
     public override string IdString { get { return IdInt.ToString(); } }

     [JsonProperty("id1")]
     [TypeConverter(typeof(IdContentPickerConverter))]
     [UmbracoProperty("ItemLink")]
     public override int IdInt { get; set; }
 }

After mapping IPublishedContent to NewClass, what I would expect is to have IdInt and IdString output as the same value, but one as an int and the other as a string. However, unfortunately that is not what happens. The output looks like that:
{
id2: "0",
id1: 1613
}

If I instead create a new property instead of IdString in the NewClass or change the modifier from override to new (which I obviously don't want to do), the IdString value is output correctly.
I'm using Ditto v0.8.3 and Umbraco v7.3.7.

@Slowacki Thanks for raising this.  Ditto shouldn't be attempting to do anything with the IdString property, since it doesn't have a setter method.  But I'll set up a unit-test to try out this scenario.

@Slowacki OK, I can reproduce the error, it seems to be related to using the virtual properties within the abstract Base class.
We'll probably need @JimBobSquarePants to take a look at this too. Here's a gist of the unit-test I made (branched off from "0.8.4" tag)...
https://gist.github.com/leekelleher/5985a13e6716d77c2b5d0790b7db21d2

@leekelleher What does v0.9.0 do in this scenario?

@JimBobSquarePants I haven't tried it yet... will do (between CG16 sessions) :metal:

@JimBobSquarePants Tried it... it's the same issue on v0.9.0.
(No rush on this though, we're at CG16 after all :smirk_cat:)

@JimBobSquarePants you had chance to look at this yet?

Shall I include the failing unit-test I made, (see gist) in a PR? Then we can assess it further

I've finally added the unit-test in its own branch/PR; for further review.

Ah great. Still think this setup is weird though.

@JimBobSquarePants I know what you mean, it's not a typical set-up that I'd use myself either. However when I was setting up the unit-test, I set the assertions to be what I'd expect the results should be...
https://github.com/leekelleher/umbraco-ditto/blob/d9bd32178c86a6b654e2802de3fb0c2cca638d9a/tests/Our.Umbraco.Ditto.Tests/Issue177Tests.cs#L48
...and those expectations failed.  Whether that's the right or wrong way, we can discuss over on the PR.

@leekelleher I'll see what I can do asap.

Should we close this ticket down, as this should be solved with PR #191 right?

I agree, closing this ticket. Any discussion can be carried on PR #191.


Added dockerfile for web and updated docker-compose

Type of Change

[ ] Bugfix
[ ] New feature
[x ] Enhancement
[ ] Refactoring
[ ] Dependency updates
[ ] Documentation
[ ] CI/CD

Description

Additional Changes

[ ] This PR modifies the API contract
[ ] This PR modifies the database schema
[ x] This PR modifies application configuration/environment variables

Motivation and Context

How did you test it?

I manually tested the changes by spinning up the Docker Compose environment and verifying that the web client and control center services are properly integrated with the existing services. All services were able to communicate with each other as expected.
Checklist

[ ] I formatted the code cargo +nightly fmt --all
[ ] I addressed lints thrown by cargo clippy
[ ] I reviewed the submitted code
[ ] I added unit tests for my changes where possible
[ ] I added a CHANGELOG entry if applicable

Closing this in favor of #4197.


Parsing problems on newest node 0.12.*

When using ng-annotate on newest node 0.12.*, there is a problem with parsing code like this:
angular.module('module',[])
.filter('filternName', function(selected, items) {
if (!selected || !selected.length) {
return items;
} else {
return items.filter(notSelected);
}
})
;
Notice the items.filter code fragment. This causes ng-annotate to stop working and triggers and error.
I cannot trace the original error because i installed 0.10.* node version, on which ng-annotate works.

I just tested your example with node-v0.12.0-darwin-x64 and ng-annotate 0.15.4 and it works fine. If you're able to repro again then please re-open and upload error message and versions you're using.


Function arguments parsing broken when using streaming

When using functions combined with streaming completions, the ChatFunctionCallArgumentsSerializerAndDeserializer is used to deserialize arguments. This is broken because quotes get stripped from the chunks that get returned. This leads to a JSON deserialization exception of the combined chunks (when the arguments JSON gets parsed).
The token ":{" is decoded to :{ and then decoded again, so losing all quotes in the process. This leads to a Jackson parse exception later when the aggregate of the chunks is (attempted to be) deserialised into the arguments.
I tried to use my own ChatFunctionCallMixIn but failed to plug that in. Also: there seems to be no way to override the mapper in OpenAiService which is kind of bummer because then I could have just plugged in a fix (in stead of patching the source below)
Would be great if this (or a better fix) is present in the next release.
Thanks a bunch; really appreciate your work!
com/theokanning/openai/service/ChatFunctionCallArgumentsSerializerAndDeserializer.java

    @Override
    public JsonNode deserialize(JsonParser p, DeserializationContext ctxt) throws IOException {
      String json = p.getValueAsString();
      if (json == null || p.currentToken() == JsonToken.VALUE_NULL) {
        return null;
      }

      // ADDED
      // encode to valid JSON escape otherwise we will lose quotes
      json = MAPPER.writeValueAsString(json);
      // END ADDED

      try {
        JsonNode node = null;

        try {
          node = MAPPER.readTree(json);
        } catch (JsonParseException ignored) {
        }
        if (node == null || node.getNodeType() == JsonNodeType.MISSING) {
          node = MAPPER.readTree(p);
        }
        return node;
      } catch (Exception ex) {
                ex.printStackTrace();
        return null;
      }
    }
  }

Just hit the same issue

Hello! I also ran into the same issue.

I ended up migrating to the azure open ai client library.


Reconnect Port change is not covered

SerialPort version: 4.0.7 (same on 4.0.6)
NodeJS Version: v6.9.2
Operating System and Hardware Platform: Ubuntu 14.04LTS x64
Are you using an alternative NodeJS runtime? (eg Electron): no
there is no function which just shows ports in Use. E.g. as the arduino IDE does.
so I tried to iterate over the list of ports and and Open each single Port to see if
port.('open') happens but this actually does open(on linux) a port where no device is connected.
A reconnect of a device can always cause a port switch, is there any clever solution to this ?

will look in the pnpID stuff. do you know why portOpen  Events happen on linux withou a device being connected ? maybe it would be good ebough to only try to connect to devices which caused such an event !?


Broadcom 4335/4339 Didn'twork

Hi
I install package from magisk and installed
I try to test it . opened terminal and run : airmon-ng start wlan0
Its says :
Found 1 processes that could cause trouble.
Kill them using 'airmon-ng check kill' before putting
the card in monitor mode, they will interfere by changing channels
and sometimes putting the interface back in managed mode
PID Name
31197 wpa_supplicant
PHY     Interface       Driver          Chipset
phy0    p2p0            bcmsdh_sdmmc    Broadcom 4335/4339
phy0    wlan0           bcmsdh_sdmmc    Broadcom 4335/4339
ERROR adding monitor mode interface: command failed: Operation not supported (-95)
And its didn't work
Can you fix it?
And I have TL-WN321G adapter WiFi that not in the supported list
Thank you

Same for me, bcm4354 don't work

Added 4335, 4339, 4354.
Future ref: https://packages.debian.org/stretch/firmware-brcm80211


test: fix GTest CI issues

Having different projects (vcxproj) w/ the same project GUID produces issues at CI. Old projects (2005/2010) were dropped to resolve ambiguity.

@dvrogozh,

I've used exactly the same copy as we have at GitHub
My point is certainly same - we have to use 3d party dependencies as-is without any modifications.

@gkrivor, could u please explain current CI issues w/ googletest to Dmitry R.?

@dvrogozh I agree with you here. @gkrivor I understand your point, but this is third-party projects. We only mess things up and trying to fix somebody else's code. I believe we should completely whitelist and ignore non-critical problem in third-party code. That would be much easier for us just to keep clean copy of external repo. At least for this problem we can limit rules related to vcxproj files only for our codebase

Let's go from other point of view. How you will use existing solutions in our code? It requires to upgrade compiler version/windows sdk version.
In this case I have a suggestion: mandatory forbid any changes in any thirdparty code.
Then I suggest @dmitry-gurulev to do not use projects/solutions/cmake/non-code-things from this folder and then we may add exclusion rule for whole folder: projects will be ignored, but usage of projects will be also forbidden in repo. In case of copying one of existing project - it GUID should be regenerated, because it will be new project/solution.
Is it looks like a solution?

@gkrivor well, absence of VS 2017/2019 solution may be considered as a problem. But again, let's treat third-party code as not our's code, i.e. we do not fix this until it is not critical / security bugs and deleting some project files looks like unnecessary action.

@mgonchar, current CI and my requirement - do not introduce any duplication/misunderstandings. How it will be achieved - we have many choices. One of that - removing unused entities.
Currently, we are blocking @dmitry-gurulev work by no clear decision about 3rd party code.
I can help with fixing CI in case we will not introduce duplication. If we will not use "meta"-objects from 3rd party - I don't have any concerns with adding exceptions. In case projects will be used in our solutions - I don't see any difference between upgrading projects/regenerating GUIDs and removing projects.

@gkrivor : contrib/googletest is a 3rd party code taken as-is for convenience. It is taken as-is to simplify its support: basically we are not responsible for this code and are not going to modify it, we just intend to use it. If update will be needed it will be taken in full.
I don't understand your talk about support of VS 2005. If mediasdk on Windows does not support it any more and you want to drop that support - that's totally ok, do that if that's your intent. But how this relates to gtest copy?! Just don't use VS 2005 project from gtrest and use more recent one, like https://github.com/Intel-Media-SDK/MediaSDK/tree/master/contrib/googletest/googlemock/msvc/2015. We do NOT advertise using and installing of our own gtest copy. Instead people should go to google original repository for that. This copy is for mediasdk use only for self-testing and nothing more.
This is not a problem which actually needs any solving since there is no problem.

@dvrogozh, please forget about support. It is side-effect. Original issue - duplicated GUIDs in projects. It should be solved.
As I mentioned before - one of way is removing unused projects.
I heard only one motivation - we don't want to touch 3rd party components. If we will use any of existing projects in our solution - we have to upgrade them, this action will change the files. What's the difference with removing? No difference.
I suggested alternate solution - mandatory rule to do not change anything in 3rd party (even folder structure) - it is reasonable. In this case - @dmitry-gurulev may copy project to our codebase and regenerate GUID.
Do you agree with such approach? Do you have any other solutions?

You are saying that GUIDs are duplicated. Can you, please, point me to what is actually duplicated? Please, provide links to project files and post GUID here.

@dvrogozh,
https://github.com/Intel-Media-SDK/MediaSDK/blob/48a4add341ba7e91dabcb5796aa01707971d167b/contrib/googletest/googlemock/msvc/2015/gmock.vcxproj#L22
https://github.com/Intel-Media-SDK/MediaSDK/blob/48a4add341ba7e91dabcb5796aa01707971d167b/contrib/googletest/googlemock/msvc/2005/gmock.vcproj#L6
https://github.com/Intel-Media-SDK/MediaSDK/blob/48a4add341ba7e91dabcb5796aa01707971d167b/contrib/googletest/googlemock/msvc/2010/gmock.vcxproj#L22

ok. And how/why this managed to affect CI?
Where is the solution which is being built? Do we managed to include few different gtest project files in the same solution?
I honestly don't understand why this spark any problem. My understanding is that we just use only one of these projects not all 3 at once.

We have "CI check for unique GUID's" in repos. This requirement is broken by googletest.
I don't want to block Dmitry's work. So I vote for a strong rule. If rule have any exceptions - then this PR might be merged as exception. If not - let's work with that as "totally untouchable".
Anyway, do you plan "sometimes" change forked 3rd party source, "if needed"? If yes, why we cannot change it now, due to our requirements to not have duplicates?
If you don't want to remove files - I may suggest to regenerate GUIDs, another way we will still have a conflict because entities will have duplicates in OpenSource repo.
If we still have some misunderstanding of this situation - I would like suggest to make a call/meeting.

From the development stand point I don't see an issue having the same GUID for the mutually excluding projects. That's exactly the case. No one will ever use 2 gtest VS projects targeting different VS versions in the same solution. I understand that the intent of the CI check was to make sure we don't reuse same guid for different not mutually excluding projects within mediasdk, i.e. that we did not copy project and forgot to change guid. However, I think that we will die if we will start to apply our CI checks for the external projects and address such failures patching external projects. I think that we might need to patch external project only if there is runtime or build failure which we can't fix with the project update to the most recent version. In this case we will need to report bug against this project and might temporary apply our own fix.
So, in this particular case my recommendation is to avoid merging in this PR and request update of CI check to exclude contrib/googletest folder from it.

@dvrogozh, we shouldn't apply CI rules for external repos. I agree.
I really appreciate you keep "your garden" in order and you do it awesome. We are trying do similar things.
And I would like support your point of "read-only 3rd party code", but I don't understand: it should be really read-only, or it should be read-only by mood.
Actually, this change was created as a workaround for fail from existing CI check. You had duplicated googletest. And all of you missed/skipped it, but CI check has found it. But I see disagreement with that.
GUID is Globally Unique IDentifier. Globally. Unique. ProjectGUID. Represents non-identical entities. And here is an issue in original googletest - identifiers are not unique. Isn't it? My original suggestion was: regenerate GUID's, if you still want to keep duplication of code.
But if you decided to have more gogletests in repos - I would like keep an order in projects/solutions while we don't have better check. To be able understand: which project should be used in selected solution. To automatically prevent a blind deleting/renaming/moving projects/solutions and to prevent appearing a lost projects/solutions.
And as I said face2face to @dmitry-gurulev: this is partial solution, anyway we need to regenerate GUID's in remained folder, otherwise CI check will fail. Because last ones weren't changed: as I understand they will be changed in a future commit.
@dmitry-gurulev is waiting for final solution. You don't want to change "3rd party code". We don't want to keep duplicates.
@dvrogozh , @mgonchar could you please think about solution from this point?
If it will be solved by removing duplicates and will be regenerated only for one folder - it will be applicable for me. I don't think about this code as "3rd party" because it was copied into our repo and became part of it. And we have to fix existing issue with non-unique GUID's. You agree to fix existing issues even in 3rd party code if it won't be fixed in original source. It is an issue. But it is not applicable for you because change... is too big? Am I right?
In this case I suggest to make a smaller change: regenerate GUID's. Change in source will be much smaller and will be easy traceable in case of any update.
Even we will close our eyes on duplication, we will need to solve requirement with updating solutions to new version. How we will solve it? Inventing yet another workaround to keep "3rd party code" formally non-changed? Why did we forked it in this case? To introduce some new requirements for ourselves? Or to be able to change it, if needed?
I only ask to not block our colleague by long discussions. If folder should be excluded - ok, but go til the end: provide alternate solution to automatically prevent duplications and allow easy moving/linking/renaming.
Let's sync if needed.

I do not try to block anyone and/or keep a long discussion. But I see a PR which needs a review and I see that I don't understand the nature of the problem neither from the commit message nor from the existing comments. Hence I ask questions. This is a review process. I am trying to help find the best solution.
If we will consider support of contrib/googletest from the perspective that it should be unchanged upstream copy of googletest, then it seems that there was some step out from this idiom. I mean that there were some other change(s) which already got merged into the repo: when I look into upstream googletest I simply don't see any VS projects, only bare cmake - see https://github.com/google/googletest/tree/master/googlemock, there is no msvc folder as we have here: https://github.com/Intel-Media-SDK/MediaSDK/tree/master/contrib/googletest/googlemock/msvc
How this did happen and when (give link to closed PR, please)? It seems that I missed some earlier PR otherwise this issue would spark earlier.
And if I understand correctly, gtest is intended to have only cmake build system both on Windows and Linux. If so, we need to consider how we build gtest on Windows for the mediasdk needs and where we place VS projects if we pursue this way? From my perspective they need to be outside of contrib/googletest. Maybe just directly in contrib.

@gkrivor,

Sorry, I have to recall my approval because of updated information from @dvrogozh.

So, u suggest me to wait me until we switch to googletest's master branch?
@dvrogozh,
Switching to master would be the best option, but AFAIR we're using of googletest's release 1.8.1 for some reasons, see
https://github.com/Intel-Media-SDK/MediaSDK/pull/842
https://github.com/Intel-Media-SDK/MediaSDK/pull/856
https://github.com/Intel-Media-SDK/MediaSDK/issues/836

@gkrivor as for me, I just vote for completely read only third-party (the only exception is security/critical bugfixes, which are not merged to third-party projects yet for some reason)

@dvrogozh @mgonchar Could we review two approaches?

remove msvc/* folders as part of preparing to moving on new version/master (it already removed in googletest master)
update googletest to the master

master branch might be unstable. I strongly suggest to take latest stable release and document tag name from which we've taken code.
Here, on GitHub I don't see any CI checks over GUID which fail...
Anyhow, do we have any VS projects in the current code base on the GitHub (already merged) which depend on gtest? In other words, do we have at least 1 gtest based test which can be built on Windows?


fix: remove event filter for nodepool hash controller

Fixes #N/A
Description
If a user removes the hash from the nodepool, the hash won't be added again until another event occurs, this could prevent drift from happening
There are four other controllers that filter out update events that don't modify the spec, which are all the correct behavior.
How was this change tested?
tested manually
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
9 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.09%) to 79.981%

Files with Coverage Reduction
New Missed Lines
%

pkg/controllers/disruption/drift.go
2
87.34%

pkg/controllers/provisioning/scheduling/preferences.go
7
86.67%

Totals

Change from base Build<PHONE_NUMBER>:
-0.09%

Covered Lines:
7667

Relevant Lines:
9586

💛  - Coveralls


support for experimental extension?

Hi,
are you planning to add support for the experimental extensions (GTX)? I am trying to port a c++ code to python which uses the glm::decompose function.
Also, thanks for this project. Makes my work a lot easier right now. :)

Hi there hschwane,
I'm sorry, but porting every single GTX extension over to Python would be way too much effort (especially with the other data types, such as the other int matrices and dual quaternions).
However, implementing single extensions, especially single methods is quite simple in most cases.
I try to port GTX extensions only when requested, because I don't have a lot of time to work on PyGLM these days.
The decompose extension should make it into the next version - which will hopefully be released soon.
Either way, thank you for using my library (:

sure, that would be a lot of work. Totally understandable. Looking forward to the next version. I found a way around using decompose until then, so don't stress yourself ;)


NATS issue with accepting org invitation with disparate BPIs

Disparate BPIs are unable to accept organization invitation tokens and are therefore unable to join a workgroup with a remote BPI

Issue related to NATS-server. Need to configure NATS server using verifying key that lives in the vault instance owned by the signer.


raster tester can't test for polygon overlays

Currently raster plots and vector plots are handled separately. however, often we combine the two via overlays in our assignments.
if the RasterTester object inherited from the VectorTester, then all methods would be available for testing!

@nkorinek can you please test this against the homework assignment that you are working on?

this has been implemented


Move rewriteUrlClosure into the web/chromecom.js file

Given that this is only used for the Chrome-addon, we can move this code into the relevant file instead.

/botio integrationtest

Looks good!


Fix func variable in uprobe

"func" is treated as usym in uprobe. Therefore pid is needed to be
stored along with the address.
Closes #977

Does it seem tests fail due to #75?

I didn't look into why, but that test failure isn't your fault. I've merged this PR manually to fix conflicts and adjust that test to not fail (lazy, I know). d864f18c6f1b44a245d5cb44ea7065ddac46ec68


Cocoon agent needs to check attached iOS device in a new way

Currently Cocoon agent relies on idevice_id that found in $PATH env var to check an iOS device is connected.
As Flutter ships iOS tools with it now, we'd better stop installing the iOS tools manually, and let Cocoon agent checks attached iOS device in a new way.
Option 1: Call flutter devices.
Option 2: Utilize the following wrapper script from @christopherfujino:
#!/bin/bash
# run-ios-dependency.sh

names=(libimobiledevice libplist ios-deploy ideviceinstaller openssl usbmuxd)
FLUTTER_PATH="$HOME/git/flutter"
LIB_PATH=''
for name in ${names[@]}; do
  LIB_PATH="$FLUTTER_PATH/bin/cache/artifacts/$name:$LIB_PATH"
done

DYLD_LIBRARY_PATH=$LIB_PATH "$@"

@digiter any plans to fix this bug?

Here's an example of using option 1 in the devicelab https://github.com/flutter/flutter/pull/59907

Not sure about the best way, LUCI still uses idevice_id in bot_config.py to populate dimensions for iPhones.

Not sure about the best way, LUCI still uses idevice_id in bot_config.py to populate dimensions for iPhone testbeds.

I highly recommend calling flutter devices --machine in case future changes to either the flutter tool, ios, or libimobiledevice regress this behavior.


My car

Add solution for simplifying car selection.
User shoudn't has to choose the same car over and over.

Propose last chosen car

Maybe we could use vue-multiselect for this?

How about storing last chosen car id in session storage?

This should be easy enough to do by basing off work on similar feature implemented here https://github.com/CodeForPoznan/pah-fm/pull/348

Relates to #383

In which PR it was fixed?

I believe it was done in https://github.com/CodeForPoznan/pah-fm/pull/361.
I closed it because the description states that:

User shoudn't has to choose the same car over and over.

implies that this issue should be resolved by remembering the selected car, which is implemented and works. It would be nice to use vue-multiselect as we did for Passenger selector, but that wasn't specified in first post, so it wasn't an requirement.


spawn_npc.py Crashes the Simulator

The demo spawn_npc.py used to work, but in the latest commit cf58f7a2c, it will crash the simulator.
This is output on the terminal prior to the crash:
[2019.05.15-13.08.35:729][ 89]LogCarla: Spawning actor 'vehicle.nissan.patrol'
[2019.05.15-13.08.35:734][ 89]LogCarla: Spawning actor 'vehicle.gazelle.omafiets'
[2019.05.15-13.08.35:737][ 89]LogCarla: Spawning actor 'vehicle.mini.cooperst'
[2019.05.15-13.08.35:738][ 89]LogBlueprintUserMessages: [Mini_C_0] MaterialNotFound
[2019.05.15-13.08.35:738][ 89]LogCarla: Spawning actor 'vehicle.dodge_charger.police'
[2019.05.15-13.08.35:739][ 89]LogCarla: Spawning actor 'vehicle.bmw.grandtourer'
[2019.05.15-13.08.35:740][ 89]LogCarla: Spawning actor 'vehicle.chevrolet.impala'
[2019.05.15-13.08.35:743][ 89]LogCarla: Spawning actor 'vehicle.lincoln.mkz2017'
[2019.05.15-13.08.35:744][ 89]LogCarla: Spawning actor 'vehicle.nissan.patrol'
[2019.05.15-13.08.35:745][ 89]LogCarla: Spawning actor 'vehicle.diamondback.century'
[2019.05.15-13.08.35:747][ 89]LogCarla: Spawning actor 'vehicle.nissan.patrol'
Failed to find symbol file, expected location:
"/home/david/carla/Dist/0.9.5-95-gcf58f7a/LinuxNoEditor/CarlaUE4/Binaries/Linux/CarlaUE4.sym"
Assertion failed: IsInGameThread() [File:/home/david/UnrealEngine_4.22/Engine/Source/Runtime/Engine/Classes/Animation/AnimInstance.h] [Line: 1241]
Signal 11 caught.
Malloc Size=65538 LargeMemoryPoolOffset=65554
CommonUnixCrashHandler: Signal=11
Malloc Size=65535 LargeMemoryPoolOffset=131119
Malloc Size=103376 LargeMemoryPoolOffset=234512
[2019.05.15-13.08.35:772][ 89]LogCore: Error: appError called: Assertion failed: IsInGameThread() [File:/home/david/UnrealEngine_4.22/Engine/Source/Runtime/Engine/Classes/Animation/AnimInstance.h] [Line: 1241]
[2019.05.15-13.08.35:778][ 89]LogCore: === Critical error: ===
Unhandled Exception: SIGSEGV: invalid attempt to read memory at address 0x00000000000040b1
[2019.05.15-13.08.35:778][ 89]LogCore: Assertion failed: IsInGameThread() [File:/home/david/UnrealEngine_4.22/Engine/Source/Runtime/Engine/Classes/Animation/AnimInstance.h] [Line: 1241]
0x0000000004e5aeef CarlaUE4!UnknownFunction(0x4c5aeee)
0x0000000004e2bca0 CarlaUE4!UnknownFunction(0x4c2bc9f)
0x0000000004b20bac CarlaUE4!UnknownFunction(0x4920bab)
0x0000000004cbc180 CarlaUE4!UnknownFunction(0x4abc17f)
0x000000000249f64b CarlaUE4!UnknownFunction(0x229f64a)
0x00000000024a5f5a CarlaUE4!UnknownFunction(0x22a5f59)
0x0000000005bcdfdf CarlaUE4!UnknownFunction(0x59cdfde)
0x00007ffb25a21830 libc.so.6!__libc_start_main(+0xef)
0x0000000002495029 CarlaUE4!UnknownFunction(0x2295028)
[2019.05.15-13.08.35:790][ 89]LogExit: Executing StaticShutdownAfterError
[2019.05.15-13.08.35:800][ 89]LogCore: Warning: Unable to statfs('/home/david/carla/Dist/0.9.5-95-gcf58f7a/LinuxNoEditor/CarlaUE4/Saved/Crashes/crashinfo-CarlaUE4-pid-16561-0BD4434808D6D914E966EB883C33B792/CarlaUE4.log'): errno=2 (No such file or directory)
Malloc Size=512385 LargeMemoryPoolOffset=746913
Engine crash handling finished; re-raising signal 11 for the default handler. Good bye.
Segmentation fault (core dumped)

I'm not able to reproduce, have you tried with the last nightly build?

The demo spawn_npc.py used to work, but in the latest commit cf58f7a, it will crash the simulator.
This is output on the terminal prior to the crash:
[2019.05.15-13.08.35:729][ 89]LogCarla: Spawning actor 'vehicle.nissan.patrol'
[2019.05.15-13.08.35:734][ 89]LogCarla: Spawning actor 'vehicle.gazelle.omafiets'
[2019.05.15-13.08.35:737][ 89]LogCarla: Spawning actor 'vehicle.mini.cooperst'
[2019.05.15-13.08.35:738][ 89]LogBlueprintUserMessages: [Mini_C_0] MaterialNotFound
[2019.05.15-13.08.35:738][ 89]LogCarla: Spawning actor 'vehicle.dodge_charger.police'
[2019.05.15-13.08.35:739][ 89]LogCarla: Spawning actor 'vehicle.bmw.grandtourer'
[2019.05.15-13.08.35:740][ 89]LogCarla: Spawning actor 'vehicle.chevrolet.impala'
[2019.05.15-13.08.35:743][ 89]LogCarla: Spawning actor 'vehicle.lincoln.mkz2017'
[2019.05.15-13.08.35:744][ 89]LogCarla: Spawning actor 'vehicle.nissan.patrol'
[2019.05.15-13.08.35:745][ 89]LogCarla: Spawning actor 'vehicle.diamondback.century'
[2019.05.15-13.08.35:747][ 89]LogCarla: Spawning actor 'vehicle.nissan.patrol'
Failed to find symbol file, expected location:
"/home/david/carla/Dist/0.9.5-95-gcf58f7a/LinuxNoEditor/CarlaUE4/Binaries/Linux/CarlaUE4.sym"
Assertion failed: IsInGameThread() [File:/home/david/UnrealEngine_4.22/Engine/Source/Runtime/Engine/Classes/Animation/AnimInstance.h] [Line: 1241]
Signal 11 caught.
Malloc Size=65538 LargeMemoryPoolOffset=65554
CommonUnixCrashHandler: Signal=11
Malloc Size=65535 LargeMemoryPoolOffset=131119
Malloc Size=103376 LargeMemoryPoolOffset=234512
[2019.05.15-13.08.35:772][ 89]LogCore: Error: appError called: Assertion failed: IsInGameThread() [File:/home/david/UnrealEngine_4.22/Engine/Source/Runtime/Engine/Classes/Animation/AnimInstance.h] [Line: 1241]
[2019.05.15-13.08.35:778][ 89]LogCore: === Critical error: ===
Unhandled Exception: SIGSEGV: invalid attempt to read memory at address 0x00000000000040b1
[2019.05.15-13.08.35:778][ 89]LogCore: Assertion failed: IsInGameThread() [File:/home/david/UnrealEngine_4.22/Engine/Source/Runtime/Engine/Classes/Animation/AnimInstance.h] [Line: 1241]
0x0000000004e5aeef CarlaUE4!UnknownFunction(0x4c5aeee)
0x0000000004e2bca0 CarlaUE4!UnknownFunction(0x4c2bc9f)
0x0000000004b20bac CarlaUE4!UnknownFunction(0x4920bab)
0x0000000004cbc180 CarlaUE4!UnknownFunction(0x4abc17f)
0x000000000249f64b CarlaUE4!UnknownFunction(0x229f64a)
0x00000000024a5f5a CarlaUE4!UnknownFunction(0x22a5f59)
0x0000000005bcdfdf CarlaUE4!UnknownFunction(0x59cdfde)
0x00007ffb25a21830 libc.so.6!__libc_start_main(+0xef)
0x0000000002495029 CarlaUE4!UnknownFunction(0x2295028)
[2019.05.15-13.08.35:790][ 89]LogExit: Executing StaticShutdownAfterError
[2019.05.15-13.08.35:800][ 89]LogCore: Warning: Unable to statfs('/home/david/carla/Dist/0.9.5-95-gcf58f7a/LinuxNoEditor/CarlaUE4/Saved/Crashes/crashinfo-CarlaUE4-pid-16561-0BD4434808D6D914E966EB883C33B792/CarlaUE4.log'): errno=2 (No such file or directory)
Malloc Size=512385 LargeMemoryPoolOffset=746913
Engine crash handling finished; re-raising signal 11 for the default handler. Good bye.
Segmentation fault (core dumped)

I got the same problem when I was using nvidia driver 384 with 2 GTX 1080 GPUs. And this is fixed after updating the nvidia driver from 384 to 410.
I hopt it helps!
By the way, when the computer is updating nvidia driver, I went to dinner with my friend. What unbelievable is that, we found that all the files of UE4 and carla are deleted from my computer. I still do not find out why it just happens.

I ended up with downgrading to carla 0.9.5 version from the nightly build. The problem was solved.
If I notice anything new, I'll post it here.


Make data_vault server cd-copyable string on dataset page

If I'm looking at /dataset/foo/bar looking at a dataset, it would be great to have a copy-able string I can put in my data analysis code: ['', 'foo', 'bar']

could possibly just have a button that uses clipboard.js

Or would it make more sense to instead display the directory in a the ['', 'foo', 'bar'] format?

I could see both ways. Having the console format for the navigation bar at the top would be kinda awkward maybe.

While we're at it, is it worth having regular directory format available, or anything else? If we're designing a UI element for this, it may as well be done right the first time.


Remove all warnings in

RR4 beta still fails <StrictMode> because it uses the legacy context API. We should consider moving to create-react-context to prepare for future React releases.

We actually are using create-react-context, but we are also using the legacy context API for the sake of people who might have (against our recommendation) been using this.context.router directly.
If it's going to be a show-stopper, we can just remove the old context code. But I was hoping to ship 4.4 with both and then just print a bunch of warnings if you access the old stuff.
Another option might be to only run that code if we're running an older build of React...
if (!React.createContext) {
  Route.childContextTypes = ...
}

Going to cherry-pick the <StrictMode> tests from #6287 to make sure we get rid of this problem once and for all.

@mjackson
What about legacy context api? If possible please remove old context stuff. If you have a time please look issue #6348, I think its related to legacy context.

I closed this prematurely. Will keep looking into it.

So, I'm trying to trigger the <StrictMode> warning about legacy context usage so I can make a test that we don't trigger that warning, but I can't get the warning to fire no matter what I do.
Here's the test for the warning message I'm trying to trigger in React core: https://github.com/facebook/react/blob/4773fdf7cdf5d6d775ad0960f23ee1a830e7b82b/packages/react/src/tests/ReactStrictMode-test.internal.js#L819-L911
And here's the code I'm using to try to cause that warning to fire: https://codesandbox.io/s/k265vz8z8v
I've checked to make sure that <StrictMode> fires on other violations, like using componentWillReceiveProps, and that works fine. But I can't trigger the error with legacy context for some reason...

Hey @jaredpalmer, I decided to just go ahead and run all of our tests in <StrictMode> to make sure we weren't triggering warnings anywhere, and we're good now on master. I'm going to cut another beta today. Please let me know how it goes!


REQUEST: Ignore files on pattern, not only on type

Hi, I'm using simple tab groups and this extention backups its config every day. This backup is shown in the status bar as download of<EMAIL_ADDRESS>Since FF is almost always running, it fills my status bar. I've tried to add stg-backup*.* as type but that doesn't work
Thanks Marc

This should work as of 1.10.0


What was it like to learn Ruby?

We’ve recently started a project to find the best way to design our tracks, in order to optimize the learning experience of students.
As a first step, we’ll be examining the ways in which languages are unique and the ways in which they are similar. For this, we’d really like to use the knowledge of everyone involved in the Exercism community (students, mentors, maintainers) to answer the following questions:

How was your experience learning Ruby? What was helpful while learning Ruby? What did you struggle with? How did you tackle problems?
In what ways did Ruby differ from other languages you knew at the time? What was hard to learn? What did you have to unlearn? What syntax did you have to remap? What concepts carried over nicely?

Could you spare 5 minutes to help us by answering these questions? It would greatly help us improve the experience students have learning Ruby :)
Note: this issue is not meant as a discussion, just as a place for people to post their own, personal experiences.
Want to keep your thoughts private but still help? Feel free to email me at<EMAIL_ADDRESS>Thank you!

How was your experience learning Ruby?

Learning Ruby was a lot of fun. The thing I like about Ruby, is that it is very expressive. You don't need much code to do lots of things. Also, running tests in Ruby is just blazingly fast (when compared to a compiled language like C#).

What was helpful while learning Ruby?

The Ruby docs were somewhat helpful. I mostly learned Ruby the right way from the mentoring I received and by looking at other people's solutions.

How did you tackle problems?

The usual mixture of trial and error, StackOverflow, blog posts and browsing the official documentation. And getting a lot of help from mentors.

In what ways did Ruby differ from other languages you knew at the time?

There was more emphasis on the productivity of the programmer than in other language I knew. This was most apparent in the huge number of built-in functions.

What was hard to learn?

Nothing in particular.

What concepts carried over nicely?

Most of the object-oriented features I knew from C# carried over relatively nicely: classes, inheritance, mutation, etc.

What did you struggle with?

I found it hard to determine how one should write idiomatic Ruby, as Ruby is so expressive that things can be solved in tons of different ways. Sometimes, the Exercism mentors themselves also seemed to disagree :)
I also found that Ruby has such a rich set of built-in functionality, that it was often best to start with scanning the full list of available functions, to see if Ruby didn't already built-in the desired functionality.
My natural tendency was to write quite functional Ruby code (modules over classes, immutability over mutability), whereas Ruby appears to favor a more object-oriented approach.
Admittedly, I also struggled with not having a full-featured IDE. Especially when new to language/platform, the ability to just discover functions and types by merely typing and pressing a dot key every now and then should not be underestimated! This was replaced with lots of switching between my IDE and my browser, which listed the available APIs.
The official documentation was not as good as I had hoped. The interface in particular feels like it was coded in the 90s, with the corresponding user-friendliness. Also, searching for an API not uncommonly lead to an official Ruby documentation page, but for an older Ruby version.

What did you have to unlearn?

That an if should always be defined before the corresponding expression (Ruby allows it to be defined after it). Also, as said, Ruby tends to do things in a very object-oriented way, so I had to unlearn my FP tendencies. As I'm quite interested in performance but Ruby focuses on readability, I also had to stop myself from worrying too much about performance. Indenting with 2 spaces was a first for me too :) Finally, as Ruby is dynamic, I could rely less on my IDE to help me.

What syntax did you have to remap?

Braces are used differently than e.g. in C#, where they are defining scope.

We're closing this issue as it was part of our research for the v3 version of Exercism which has just been released.
Thanks everyone for chipping in! It has been greatly appreciated.


tmpop: fix node that does not start

This change is

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


fix hardhat & add foundry examples

Hi @m4sterbunny @0xlucian @sukanyaparashar please have a look at this PR:

Edited Hardhat Example
Added Foundry Example
Add the foundry example to sidebar.js

@mnedelchev-vn There are conflicts on this branch since merging other branches PTAL

@m4sterbunny the conflict in this branch is made by you, because you've changed file static/data/link-list.json which was recently updated in the main branch. I guess undo your change and it include it in a separated PR.

@mnedelchev-vn confused! GitHub says that these are the conflicting files:

none of which are a json file

@mnedelchev-vn I have fixed the conflicts -- there are 2 items pending with you. PTAL and ping me when done.

@m4sterbunny there was some wrong stuff with using_hardhat.mdx file, for whatever reason the content was a real mess. I made a new commit where I recreated the steps to deploy thru hardhat. Please check Hardhat & Foundry pages and lmk if we missed smth.


Error when installing on Apple M1 machine (building from source)

Issue Summary
Looks like npm looks for a wrong location when looking for stdarg.h (I believe on a Mac its under Xcode package). The main error is:
fatal error: cannot open file '/usr/local/include/stdarg.h': Permission denied
Relevant logs or output
Full log:
✘ m@macname  ~/Documents/temp/googleauth  npm install sqlite3 --build-from-source
npm ERR! code 1
npm ERR! path /Users/m/Documents/temp/googleauth/node_modules/sqlite3
npm ERR! command failed
npm ERR! command sh -c node-pre-gyp install --fallback-to-build
npm ERR! CC(target) Release/obj.target/nothing/../node-addon-api/nothing.o
npm ERR!   LIBTOOL-STATIC Release/nothing.a
npm ERR!   ACTION deps_sqlite3_gyp_action_before_build_target_unpack_sqlite_dep Release/obj/gen/sqlite-autoconf-3380400/sqlite3.c
npm ERR!   TOUCH Release/obj.target/deps/action_before_build.stamp
npm ERR!   CC(target) Release/obj.target/sqlite3/gen/sqlite-autoconf-3380400/sqlite3.o
npm ERR! Failed to execute '/opt/homebrew/Cellar/node/18.4.0/bin/node /opt/homebrew/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js build --fallback-to-build --module=/Users/m/Documents/temp/googleauth/node_modules/sqlite3/lib/binding/napi-v6-darwin-unknown-arm64/node_sqlite3.node --module_name=node_sqlite3 --module_path=/Users/m/Documents/temp/googleauth/node_modules/sqlite3/lib/binding/napi-v6-darwin-unknown-arm64 --napi_version=8 --node_abi_napi=napi --napi_build_version=6 --node_napi_label=napi-v6' (1)
npm ERR! node-pre-gyp info it worked if it ends with ok
npm ERR! node-pre-gyp info using<EMAIL_ADDRESS>npm ERR! node-pre-gyp info using<EMAIL_ADDRESS>| darwin | arm64
npm ERR! node-pre-gyp info build requesting source compile
npm ERR! gyp info it worked if it ends with ok
npm ERR! gyp info using<EMAIL_ADDRESS>npm ERR! gyp info using<EMAIL_ADDRESS>| darwin | arm64
npm ERR! gyp info ok
npm ERR! gyp info it worked if it ends with ok
npm ERR! gyp info using<EMAIL_ADDRESS>npm ERR! gyp info using<EMAIL_ADDRESS>| darwin | arm64
npm ERR! gyp info find Python using Python version 3.9.13 found at<EMAIL_ADDRESS>npm ERR! gyp info spawn<EMAIL_ADDRESS>npm ERR! gyp info spawn args [
npm ERR! gyp info spawn args   '/opt/homebrew/lib/node_modules/npm/node_modules/node-gyp/gyp/gyp_main.py',
npm ERR! gyp info spawn args   'binding.gyp',
npm ERR! gyp info spawn args   '-f',
npm ERR! gyp info spawn args   'make',
npm ERR! gyp info spawn args   '-I',
npm ERR! gyp info spawn args   '/Users/m/Documents/temp/googleauth/node_modules/sqlite3/build/config.gypi',
npm ERR! gyp info spawn args   '-I',
npm ERR! gyp info spawn args   '/opt/homebrew/lib/node_modules/npm/node_modules/node-gyp/addon.gypi',
npm ERR! gyp info spawn args   '-I',
npm ERR! gyp info spawn args   '/Users/m/Library/Caches/node-gyp/18.4.0/include/node/common.gypi',
npm ERR! gyp info spawn args   '-Dlibrary=shared_library',
npm ERR! gyp info spawn args   '-Dvisibility=default',
npm ERR! gyp info spawn args   '-Dnode_root_dir=/Users/m/Library/Caches/node-gyp/18.4.0',
npm ERR! gyp info spawn args   '-Dnode_gyp_dir=/opt/homebrew/lib/node_modules/npm/node_modules/node-gyp',
npm ERR! gyp info spawn args   '-Dnode_lib_file=/Users/m/Library/Caches/node-gyp/18.4.0/<(target_arch)/node.lib',
npm ERR! gyp info spawn args   '-Dmodule_root_dir=/Users/m/Documents/temp/googleauth/node_modules/sqlite3',
npm ERR! gyp info spawn args   '-Dnode_engine=v8',
npm ERR! gyp info spawn args   '--depth=.',
npm ERR! gyp info spawn args   '--no-parallel',
npm ERR! gyp info spawn args   '--generator-output',
npm ERR! gyp info spawn args   'build',
npm ERR! gyp info spawn args   '-Goutput_dir=.'
npm ERR! gyp info spawn args ]
npm ERR! gyp info ok
npm ERR! gyp info it worked if it ends with ok
npm ERR! gyp info using<EMAIL_ADDRESS>npm ERR! gyp info using<EMAIL_ADDRESS>| darwin | arm64
npm ERR! gyp info spawn make
npm ERR! gyp info spawn args [ 'BUILDTYPE=Release', '-C', 'build' ]
npm ERR! warning: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/libtool: archive library: Release/nothing.a the table of contents is empty (no object file members in the library define global symbols)
npm ERR! Release/obj/gen/sqlite-autoconf-3380400/sqlite3.c:341:10: fatal error: cannot open file '/usr/local/include/stdarg.h': Permission denied
npm ERR! #include <stdarg.h>     /* Needed for the definition of va_list */
npm ERR!          ^
npm ERR! 1 error generated.
npm ERR! make: *** [Release/obj.target/sqlite3/gen/sqlite-autoconf-3380400/sqlite3.o] Error 1
npm ERR! gyp ERR! build error
npm ERR! gyp ERR! stack Error: `make` failed with exit code: 2
npm ERR! gyp ERR! stack     at ChildProcess.onExit (/opt/homebrew/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:194:23)
npm ERR! gyp ERR! stack     at ChildProcess.emit (node:events:537:28)
npm ERR! gyp ERR! stack     at ChildProcess._handle.onexit (node:internal/child_process:291:12)
npm ERR! gyp ERR!  System Darwin 21.5.0
npm ERR! node-pre-gyp ERR! command "/opt/homebrew/Cellar/node/18.4.0/bin/node" "/Users/m/Documents/temp/googleauth/node_modules/.bin/node-pre-gyp" "install" "--fallback-to-build"
npm ERR! node-pre-gyp ERR! cwd /Users/m/Documents/temp/googleauth/node_modules/sqlite3
npm ERR! node-pre-gyp ERR! node -v v18.4.0
npm ERR! node-pre-gyp ERR! node-pre-gyp -v v1.0.9
npm ERR! node-pre-gyp ERR! not ok

npm ERR!      /Users/m/.npm/_logs/2022-06-25T07_24_07_847Z-debug-0.log

Version
5.0.8
Node.js Version
18.4.0
How did you install the library?
npm install sqlite3 --build-from-source

@daniellockyer Mac OS X version 12.4.
 Extracting by manifest.
470 silly tarball no local data for sqlite3@https://registry.npmjs.org/sqlite3/-/sqlite3-5.0.9.tgz. Extracting by manifest.
471 timing reifyNode:node_modules/imurmurhash Completed in 110ms
472 timing reifyNode:node_modules/object-assign Completed in 109ms
473 timing reifyNode:node_modules/ip Completed in 110ms
474 timing reifyNode:node_modules/wide-align Completed in 109ms
475 timing reifyNode:node_modules/has-unicode Completed in 110ms
476 timing reifyNode:node_modules/brace-expansion Completed in 110ms
477 timing reifyNode:node_modules/abbrev Completed in 110ms
478 timing reifyNode:node_modules/@gar/promisify Completed in 110ms
479 timing reifyNode:node_modules/promise-inflight Completed in 110ms
480 timing reifyNode:node_modules/infer-owner Completed in 111ms
481 timing reifyNode:node_modules/@npmcli/move-file Completed in 110ms
482 timing reifyNode:node_modules/chownr Completed in 110ms
483 timing reifyNode:node_modules/minipass-pipeline Completed in 110ms
484 timing reifyNode:node_modules/minipass-collect Completed in 110ms
485 timing reifyNode:node_modules/minipass-flush Completed in 110ms
486 timing reifyNode:node_modules/wrappy Completed in 110ms
487 timing reifyNode:node_modules/path-is-absolute Completed in 110ms
488 timing reifyNode:node_modules/once Completed in 111ms
489 timing reifyNode:node_modules/inflight Completed in 111ms
490 timing reifyNode:node_modules/ms Completed in 111ms
491 timing reifyNode:node_modules/webidl-conversions Completed in 114ms
492 timing reifyNode:node_modules/ansi-regex Completed in 114ms
493 timing reifyNode:node_modules/is-fullwidth-code-point Completed in 114ms
494 timing reifyNode:node_modules/clean-stack Completed in 113ms
495 timing reifyNode:node_modules/string-width Completed in 114ms
496 timing reifyNode:node_modules/strip-ansi Completed in 114ms
497 timing reifyNode:node_modules/aggregate-error Completed in 114ms
498 timing reifyNode:node_modules/signal-exit Completed in 114ms
499 timing reifyNode:node_modules/indent-string Completed in 114ms
500 timing reifyNode:node_modules/console-control-strings Completed in 114ms
501 timing reifyNode:node_modules/aproba Completed in 115ms
502 timing reifyNode:node_modules/set-blocking Completed in 115ms
503 timing reifyNode:node_modules/fs-minipass Completed in 115ms
504 timing reifyNode:node_modules/@tootallnate/once Completed in 115ms
505 timing reifyNode:node_modules/humanize-ms Completed in 114ms
506 timing reifyNode:node_modules/p-map Completed in 115ms
507 timing reifyNode:node_modules/lru-cache Completed in 115ms
508 timing reifyNode:node_modules/inherits Completed in 115ms
509 timing reifyNode:node_modules/env-paths Completed in 115ms
510 timing reifyNode:node_modules/npmlog Completed in 115ms
511 timing reifyNode:node_modules/make-dir Completed in 115ms
512 timing reifyNode:node_modules/util-deprecate Completed in 116ms
513 timing reifyNode:node_modules/string_decoder Completed in 117ms
514 timing reifyNode:node_modules/color-support Completed in 117ms
515 timing reifyNode:node_modules/balanced-match Completed in 117ms
516 timing reifyNode:node_modules/yallist Completed in 117ms
517 timing reifyNode:node_modules/fs.realpath Completed in 117ms
518 timing reifyNode:node_modules/unique-slug Completed in 122ms
519 timing reifyNode:node_modules/node-gyp/node_modules/npmlog Completed in 119ms
520 timing reifyNode:node_modules/minizlib Completed in 122ms
521 timing reifyNode:node_modules/ssri Completed in 122ms
522 timing reifyNode:node_modules/is-lambda Completed in 122ms
523 timing reifyNode:node_modules/http-cache-semantics Completed in 122ms
524 timing reifyNode:node_modules/which Completed in 122ms
525 timing reifyNode:node_modules/minimatch Completed in 122ms
526 timing reifyNode:node_modules/safe-buffer Completed in 125ms
527 timing reifyNode:node_modules/socks-proxy-agent Completed in 124ms
528 timing reifyNode:node_modules/graceful-fs Completed in 124ms
529 timing reifyNode:node_modules/rimraf Completed in 124ms
530 timing reifyNode:node_modules/delegates Completed in 125ms
531 timing reifyNode:node_modules/concat-map Completed in 126ms
532 timing reifyNode:node_modules/encoding Completed in 126ms
533 timing reifyNode:node_modules/http-proxy-agent Completed in 125ms
534 timing reifyNode:node_modules/debug Completed in 125ms
535 timing reifyNode:node_modules/detect-libc Completed in 124ms
536 timing reifyNode:node_modules/isexe Completed in 128ms
537 timing reifyNode:node_modules/safer-buffer Completed in 127ms
538 timing reifyNode:node_modules/are-we-there-yet Completed in 128ms
539 timing reifyNode:node_modules/nopt Completed in 127ms
540 timing reifyNode:node_modules/err-code Completed in 129ms
541 timing reifyNode:node_modules/node-gyp/node_modules/are-we-there-yet Completed in 127ms
542 timing reifyNode:node_modules/promise-retry Completed in 128ms
543 timing reifyNode:node_modules/glob Completed in 128ms
544 timing reifyNode:node_modules/emoji-regex Completed in 131ms
545 timing reifyNode:node_modules/negotiator Completed in 131ms
546 timing reifyNode:node_modules/whatwg-url Completed in 131ms
547 timing reifyNode:node_modules/make-dir/node_modules/semver Completed in 131ms
548 http fetch GET 200 https://registry.npmjs.org/minipass/-/minipass-3.3.4.tgz 114ms (cache miss)
549 timing reifyNode:node_modules/minipass Completed in 132ms
550 timing reifyNode:node_modules/mkdirp Completed in 134ms
551 timing reifyNode:node_modules/agentkeepalive Completed in 133ms
552 timing reifyNode:node_modules/minipass-sized Completed in 135ms
553 timing reifyNode:node_modules/https-proxy-agent Completed in 134ms
554 timing reifyNode:node_modules/minipass-fetch Completed in 135ms
555 timing reifyNode:node_modules/agent-base Completed in 135ms
556 timing reifyNode:node_modules/node-fetch Completed in 136ms
557 timing reifyNode:node_modules/depd Completed in 136ms
558 timing reifyNode:node_modules/retry Completed in 138ms
559 timing reifyNode:node_modules/tr46 Completed in 140ms
560 timing reifyNode:node_modules/gauge Completed in 139ms
561 timing reifyNode:node_modules/make-fetch-happen Completed in 139ms
562 timing reifyNode:node_modules/node-gyp/node_modules/gauge Completed in 139ms
563 timing reifyNode:node_modules/unique-filename Completed in 142ms
564 timing reifyNode:node_modules/smart-buffer Completed in 143ms
565 timing reifyNode:node_modules/cacache Completed in 144ms
566 timing reifyNode:node_modules/@npmcli/fs Completed in 145ms
567 timing reifyNode:node_modules/readable-stream Completed in 148ms
568 timing reifyNode:node_modules/tar Completed in 149ms
569 timing reifyNode:node_modules/node-addon-api Completed in 149ms
570 timing reifyNode:node_modules/@mapbox/node-pre-gyp Completed in 150ms
571 timing reifyNode:node_modules/semver Completed in 152ms
572 timing reifyNode:node_modules/socks Completed in 158ms
573 timing reifyNode:node_modules/iconv-lite Completed in 159ms
574 timing reifyNode:node_modules/node-gyp Completed in 169ms
575 http fetch POST 200 https://registry.npmjs.org/-/npm/v1/security/advisories/bulk 344ms
576 timing auditReport:getReport Completed in 348ms
577 silly audit report {}
578 timing auditReport:init Completed in 0ms
579 timing reify:audit Completed in 348ms
580 http fetch GET 200 https://registry.npmjs.org/sqlite3/-/sqlite3-5.0.9.tgz 1333ms (cache miss)
581 timing reifyNode:node_modules/sqlite3 Completed in 1352ms
582 timing reify:unpack Completed in 1355ms
583 timing reify:unretire Completed in 0ms
584 timing build:queue Completed in 2ms
585 timing build:link:node_modules/@mapbox/node-pre-gyp Completed in 3ms
586 timing build:link:node_modules/color-support Completed in 3ms
587 timing build:link:node_modules/mkdirp Completed in 3ms
588 timing build:link:node_modules/make-dir/node_modules/semver Completed in 2ms
589 timing build:link:node_modules/nopt Completed in 3ms
590 timing build:link:node_modules/rimraf Completed in 3ms
591 timing build:link:node_modules/node-gyp Completed in 3ms
592 timing build:link:node_modules/semver Completed in 3ms
593 timing build:link:node_modules/which Completed in 2ms
594 timing build:link Completed in 3ms
595 info run<EMAIL_ADDRESS>install node_modules/sqlite3 node-pre-gyp install --fallback-to-build
596 info run<EMAIL_ADDRESS>install { code: 1, signal: null }
597 timing reify:rollback:createSparse Completed in 98ms
598 timing reify:rollback:retireShallow Completed in 0ms
599 timing command:install Completed in 10514ms
600 notice
600 notice New minor version of npm available! 8.12.1 -> 8.14.0
600 notice Changelog: https://github.com/npm/cli/releases/tag/v8.14.0
600 notice Run npm install -g<EMAIL_ADDRESS>to update!
600 notice
601 verbose stack Error: command failed
601 verbose stack     at ChildProcess.<anonymous> (/opt/homebrew/lib/node_modules/npm/node_modules/@npmcli/promise-spawn/lib/index.js:63:27)
601 verbose stack     at ChildProcess.emit (node:events:527:28)
601 verbose stack     at maybeClose (node:internal/child_process:1092:16)
601 verbose stack     at Socket.<anonymous> (node:internal/child_process:451:11)
601 verbose stack     at Socket.emit (node:events:527:28)
601 verbose stack     at Pipe.<anonymous> (node:net:709:12)
602 verbose pkgid<EMAIL_ADDRESS>603 verbose cwd /Users/m/Documents/org
604 verbose Darwin 21.5.0
605 verbose node v16.15.1
606 verbose npm  v8.12.1
607 error code 1
608 error path /Users/m/Documents/org/node_modules/sqlite3
609 error command failed
610 error command sh -c node-pre-gyp install --fallback-to-build
611 error CC(target) Release/obj.target/nothing/../node-addon-api/nothing.o
611 error   LIBTOOL-STATIC Release/nothing.a
611 error   ACTION deps_sqlite3_gyp_action_before_build_target_unpack_sqlite_dep Release/obj/gen/sqlite-autoconf-3390100/sqlite3.c
611 error   TOUCH Release/obj.target/deps/action_before_build.stamp
611 error   CC(target) Release/obj.target/sqlite3/gen/sqlite-autoconf-3390100/sqlite3.o
611 error Failed to execute '/usr/local/bin/node /opt/homebrew/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js build --fallback-to-build --module=/Users/m/Documents/org/node_modules/sqlite3/lib/binding/napi-v6-darwin-unknown-arm64/node_sqlite3.node --module_name=node_sqlite3 --module_path=/Users/m/Documents/org/node_modules/sqlite3/lib/binding/napi-v6-darwin-unknown-arm64 --napi_version=8 --node_abi_napi=napi --napi_build_version=6 --node_napi_label=napi-v6' (1)
612 error node-pre-gyp info it worked if it ends with ok
612 error node-pre-gyp info using<EMAIL_ADDRESS>612 error node-pre-gyp info using<EMAIL_ADDRESS>| darwin | arm64
612 error node-pre-gyp info build requesting source compile
612 error gyp info it worked if it ends with ok
612 error gyp info using<EMAIL_ADDRESS>612 error gyp info using<EMAIL_ADDRESS>| darwin | arm64
612 error gyp info ok
612 error gyp info it worked if it ends with ok
612 error gyp info using<EMAIL_ADDRESS>612 error gyp info using<EMAIL_ADDRESS>| darwin | arm64
612 error gyp info find Python using Python version 3.9.13 found at<EMAIL_ADDRESS>612 error gyp info spawn<EMAIL_ADDRESS>612 error gyp info spawn args [
612 error gyp info spawn args   '/opt/homebrew/lib/node_modules/npm/node_modules/node-gyp/gyp/gyp_main.py',
612 error gyp info spawn args   'binding.gyp',
612 error gyp info spawn args   '-f',
612 error gyp info spawn args   'make',
612 error gyp info spawn args   '-I',
612 error gyp info spawn args   '/Users/m/Documents/org/node_modules/sqlite3/build/config.gypi',
612 error gyp info spawn args   '-I',
612 error gyp info spawn args   '/opt/homebrew/lib/node_modules/npm/node_modules/node-gyp/addon.gypi',
612 error gyp info spawn args   '-I',
612 error gyp info spawn args   '/Users/m/Library/Caches/node-gyp/16.15.1/include/node/common.gypi',
612 error gyp info spawn args   '-Dlibrary=shared_library',
612 error gyp info spawn args   '-Dvisibility=default',
612 error gyp info spawn args   '-Dnode_root_dir=/Users/m/Library/Caches/node-gyp/16.15.1',
612 error gyp info spawn args   '-Dnode_gyp_dir=/opt/homebrew/lib/node_modules/npm/node_modules/node-gyp',
612 error gyp info spawn args   '-Dnode_lib_file=/Users/m/Library/Caches/node-gyp/16.15.1/<(target_arch)/node.lib',
612 error gyp info spawn args   '-Dmodule_root_dir=/Users/m/Documents/org/node_modules/sqlite3',
612 error gyp info spawn args   '-Dnode_engine=v8',
612 error gyp info spawn args   '--depth=.',
612 error gyp info spawn args   '--no-parallel',
612 error gyp info spawn args   '--generator-output',
612 error gyp info spawn args   'build',
612 error gyp info spawn args   '-Goutput_dir=.'
612 error gyp info spawn args ]
612 error gyp info ok
612 error gyp info it worked if it ends with ok
612 error gyp info using<EMAIL_ADDRESS>612 error gyp info using<EMAIL_ADDRESS>| darwin | arm64
612 error gyp info spawn make
612 error gyp info spawn args [ 'BUILDTYPE=Release', '-C', 'build' ]
612 error warning: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/libtool: archive library: Release/nothing.a the table of contents is empty (no object file members in the library define global symbols)
612 error Release/obj/gen/sqlite-autoconf-3390100/sqlite3.c:14044:10: fatal error: cannot open file '/usr/local/include/stdio.h': Permission denied
612 error #include <stdio.h>
612 error          ^
612 error 1 error generated.
612 error make: *** [Release/obj.target/sqlite3/gen/sqlite-autoconf-3390100/sqlite3.o] Error 1
612 error gyp ERR! build error
612 error gyp ERR! stack Error: `make` failed with exit code: 2
612 error gyp ERR! stack     at ChildProcess.onExit (/opt/homebrew/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:194:23)
612 error gyp ERR! stack     at ChildProcess.emit (node:events:527:28)
612 error gyp ERR! stack     at Process.ChildProcess._handle.onexit (node:internal/child_process:291:12)
612 error gyp ERR! System Darwin 21.5.0
612 error gyp ERR! command "/usr/local/bin/node" "/opt/homebrew/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "build" "--fallback-to-build" "--module=/Users/m/Documents/org/node_modules/sqlite3/lib/binding/napi-v6-darwin-unknown-arm64/node_sqlite3.node" "--module_name=node_sqlite3" "--module_path=/Users/m/Documents/org/node_modules/sqlite3/lib/binding/napi-v6-darwin-unknown-arm64" "--napi_version=8" "--node_abi_napi=napi" "--napi_build_version=6" "--node_napi_label=napi-v6"
612 error gyp ERR! cwd /Users/m/Documents/org/node_modules/sqlite3
612 error gyp ERR! node -v v16.15.1
612 error gyp ERR! node-gyp -v v9.0.0
612 error gyp ERR! not ok
612 error node-pre-gyp ERR! build error
612 error node-pre-gyp ERR! stack Error: Failed to execute '/usr/local/bin/node /opt/homebrew/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js build --fallback-to-build --module=/Users/m/Documents/org/node_modules/sqlite3/lib/binding/napi-v6-darwin-unknown-arm64/node_sqlite3.node --module_name=node_sqlite3 --module_path=/Users/m/Documents/org/node_modules/sqlite3/lib/binding/napi-v6-darwin-unknown-arm64 --napi_version=8 --node_abi_napi=napi --napi_build_version=6 --node_napi_label=napi-v6' (1)
612 error node-pre-gyp ERR! stack     at ChildProcess.<anonymous> (/Users/m/Documents/org/node_modules/@mapbox/node-pre-gyp/lib/util/compile.js:89:23)
612 error node-pre-gyp ERR! stack     at ChildProcess.emit (node:events:527:28)
612 error node-pre-gyp ERR! stack     at maybeClose (node:internal/child_process:1092:16)
612 error node-pre-gyp ERR! stack     at Process.ChildProcess._handle.onexit (node:internal/child_process:302:5)
612 error node-pre-gyp ERR! System Darwin 21.5.0
612 error node-pre-gyp ERR! command "/usr/local/bin/node" "/Users/m/Documents/org/node_modules/.bin/node-pre-gyp" "install" "--fallback-to-build"
612 error node-pre-gyp ERR! cwd /Users/m/Documents/org/node_modules/sqlite3
612 error node-pre-gyp ERR! node -v v16.15.1
612 error node-pre-gyp ERR! node-pre-gyp -v v1.0.9
612 error node-pre-gyp ERR! not ok
613 verbose exit 1
614 timing npm Completed in 10566ms
615 verbose unfinished npm timer reify<PHONE_NUMBER>542
616 verbose unfinished npm timer reify:build<PHONE_NUMBER>908
617 verbose unfinished npm timer build<PHONE_NUMBER>908
618 verbose unfinished npm timer build:deps<PHONE_NUMBER>909
619 verbose unfinished npm timer build:run:install<PHONE_NUMBER>914
620 verbose unfinished npm timer build:run:install:node_modules/sqlite3<PHONE_NUMBER>914
621 verbose code 1
622 error A complete log of this run can be found in:
622 error     /Users/m/.npm/_logs/2022-07-14T17_58_31_493Z-debug-0.log

Unfortunately it's still not clear to me what the issue is 😕

I know it's not a real solution but v5.1.1 now comes with ARM64 binaries, so you shouldn't need to build any longer: https://github.com/TryGhost/node-sqlite3/releases/tag/v5.1.1

I'm going to close this as we ship Darwin ARM binaries and I still can't reproduce the issue. Happy to re-open if it is still an issue down the line

Maybe you should specify target_arch when build
npm install sqlite3 --build-from-source --target_arch=arm64 --fallback-to-build
This command works for me under M1.


Change Management Module

add Change Management Module (ticketing)

marked this issue as related to #142

added 1m of time spent

marked this issue as related to #96

marked this issue as related to #88


Ramp-up EKS APIs & existing TF usage

┆Issue is synchronized with this Jira Task by Unito
┆friendlyId: K8SSAND-1457
┆priority: Medium

Tracking as epic in cloud-readiness to identify task breakdown.


Intermittent failure of test CxfWsTrustTest.programmaticStsNotSet

Bug description
2023-06-14T03:36:28.4259208Z Caused by: java.net.ConnectException: ConnectException invoking http://localhost:8081/jaxws-samples-wsse-policy-trust-sts/SecurityTokenService: Connection refused

Note the URL has port 8081. Which probably  comes from here:
https://github.com/apache/camel-quarkus/blob/main/integration-test-groups/cxf-soap/cxf-soap-ws-trust/src/main/resources/AsymmetricSAML2Policy.xml#L50-L52
The resulting exception is thrown from AbstractSTSClient:
trust}sayHello has thrown exception, unwinding now: org.apache.cxf.ws.security.trust.TrustException: Exception when trying to retrieve/process a WSDL via WS-MEX
2023-06-14T03:47:00.2152933Z 	at org.apache.cxf.ws.security.trust.AbstractSTSClient.configureViaEPR(AbstractSTSClient.java:612)
2023-06-14T03:47:00.2153531Z 	at org.apache.cxf.ws.security.trust.STSUtils.getClientWithIssuer(STSUtils.java:166)
2023-06-14T03:47:00.2154101Z 	at org.apache.cxf.ws.security.trust.STSTokenRetriever.getToken(STSTokenRetriever.java:68)
2023-06-14T03:47:00.2155000Z 	at org.apache.cxf.ws.security.policy.interceptors.IssuedTokenInterceptorProvider$IssuedTokenOutInterceptor.handleMessage(IssuedTokenInterceptorProvider.java:147)
2023-06-14T03:47:00.2155767Z 	at org.apache.cxf.phase.PhaseInterceptorChain.doIntercept(PhaseInterceptorChain.java:307)
2023-06-14T03:47:00.2156286Z 	at org.apache.cxf.endpoint.ClientImpl.doInvoke(ClientImpl.java:528)
2023-06-14T03:47:00.2156730Z 	at org.apache.cxf.endpoint.ClientImpl.invoke(ClientImpl.java:439)
2023-06-14T03:47:00.2157149Z 	at org.apache.cxf.endpoint.ClientImpl.invoke(ClientImpl.java:354)
2023-06-14T03:47:00.2157580Z 	at org.apache.cxf.endpoint.ClientImpl.invoke(ClientImpl.java:312)
2023-06-14T03:47:00.2158031Z 	at org.apache.cxf.frontend.ClientProxy.invokeSync(ClientProxy.java:96)
2023-06-14T03:47:00.2158506Z 	at org.apache.cxf.jaxws.JaxWsClientProxy.invoke(JaxWsClientProxy.java:140)
2023-06-14T03:47:00.2158915Z 	at jdk.proxy2/jdk.proxy2.$Proxy102.sayHello(Unknown Source)

What's interesting is that test only fails some of the time. Meaning that the exception is probably being swallowed or ignored when the test passes.

Reopening as this appears to still not be fixed.


Scaffolder: ensure cookiecutter intermediateDir

Hey, I just made a Pull Request!
This change ensures that the intermediateDir is created before running cookiecutter.
This fixes scaffolding in Linux where the directory is not created automatically(probably due to a difference in how Docker handles file system mounting in OSX/Linux).
fixes #4330
:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)

Tugboat has finished building the preview for this pull request!
Link:

https://pr4373-nxzaedotg7sbwpgmxropum9k2mzy697h.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/601a6a9098552d8316550fcb


Corrected debug-level printing

there was a clash between errp() functions

these overloads are legal in C++
yet might be considered ODR in C (assuming the language uses the term)

the printing function must be stashed away

and passed in

great, thanks!


Cannot refresh access token without explicit scope

The OAuth2 specification states that providing a scope is optional when refreshing an access token:
scope
      OPTIONAL.  The scope of the access request as described by
      Section 3.3.  The requested scope MUST NOT include any scope
      not originally granted by the resource owner, and if omitted is
      treated as equal to the scope originally granted by the
      resource owner.

Attempting to do this against a Spring Security OAuth2 authorization server results in DefaultOAuth2RequestValidator throwing the following exception:
InvalidScopeException: Empty scope (either the client or the user is not allowed the requested scopes)

See this test to reproduce the issue: https://github.com/markhobson/spring-security-oauth-issue/

Still an issue with spring-security-oauth2 2.3.5. Is there any appetite to fix this or is all effort going into Spring Security 5 OAuth?

@markhobson Thanks for the report. This is a bug as you have determined. To workaround this issue you can provide a custom OAuth2RequestValidator via AuthorizationServerEndpointsConfigurer.requestValidator().
We are quite busy working on features for the upcoming Spring Security 5.2.0 release so yes all our efforts are focused there. I've scheduled this fix for next release but that won't be for a while given we recently released 2.3.5.
Would you be interested in submitting a PR for this fix?

Hi @jgrandja, thanks for the reply. I'm happy to take a look at creating a PR, although I thought it was more involved than updating the OAuth2RequestValidator as we need to obtain the originally granted scopes when absent.

@markhobson

we need to obtain the originally granted scopes when absent

Yes, this is required as part of the fix. If you look at AuthorizationEndpoint, you will notice it stores AuthorizationRequest in the session using the attribute AuthorizationEndpoint.AUTHORIZATION_REQUEST_ATTR_NAME.

@jgrandja Your initial workaround of providing a more lenient OAuth2RequestValidator was indeed sufficient. Allowing empty scopes results in DefaultTokenServices creating an OAuth2Authentication with the originally granted scopes. Note that I raised spring-boot#17014 to ease configuring a custom OAuth2RequestValidator.
If I understand correctly, your comment regarding AuthorizationEndpoint storing the AuthorizationRequest in the session shouldn't be relevant since refreshing is performed by the TokenEndpoint? They do both use the OAuth2RequestValidator so I trust that changing its behaviour for token refresh won't inadvertently affect authorization.
I'll create a PR so you can review my fix so far.

PR: https://github.com/spring-projects/spring-security-oauth/pull/1697

On reflection I think this PR needs some more work.
Both authorization requests and token requests are passed through the OAuth2RequestValidator. At these points requests with empty scopes have been set to the client's scopes. Thus the only case for the validator to allow empty scopes is for a refresh request when the client has no scopes defined. Authorization requests must always resolve to some scope. I'll update the validator to only guard against empty scopes in validateScope(AuthorizationRequest, ClientDetails).
It is a little misleading that for refresh requests empty scopes are translated to client scopes prior to validation, and then reset back to the requested scopes afterwards. This doesn't give the validator a clear view of the actual request.
I believe this whole issue can be worked around if the client is registered with scopes. In my case it wasn't, so coupled with the empty scope refresh request it highlighted this edge case.

Updated PR to only relax token request validation. @jgrandja Let me know your thoughts, thanks.


Shap value doesn't match model output

First off, thanks so much for shap! I love it. So I'm having an issue where the shap values are on a much different scale than my model output. I only come across this issue when I run lightgbm in random forest mode - when I use lightgbm in boosting mode or use sklearn random forest, shap works as expected. Here is a minimal reproducible example of my error.
code:
import lightgbm as lgb
import shap

# load JS visualization code to notebook
shap.initjs()

# load data
X, y = shap.datasets.boston()

# train lightgbm random forest
model = lgb.LGBMRegressor(boosting_type = 'rf',
                          n_estimators = 500,
                          max_depth = 7,
                          num_leaves = 2**7,
                          subsample_freq = 1,
                          subsample = 0.6,
                          colsample_bytree = 0.5,
                          n_jobs = -1,
                          random_state = 123,
                          importance_type = 'gain',
                         )

model.fit(X, y)

# calculate shap values
explainer = shap.TreeExplainer(model)
shap_values = explainer.shap_values(X)

# visualize the first prediction's explanation
shap.force_plot(explainer.expected_value, shap_values[0,:], X.iloc[0,:])

# get the first prediction from the model
model.predict(X.iloc[0:1])

output:
visualizer's prediction is 14340.97
the model prediction is 28.68
Any help is appreciated as I much prefer lightgbm's implementation of random forest over sklearn's.
Thanks!

There is a transform between the margin of the model and the output of predict so run model.predict(X.iloc[0:1], raw_score=True) and you get the right answer.

Okay thanks for that information. From what I can tell, the raw score is just the model prediction multiplied by the number of trees (500 in this example). When I divide both explainer.expected_value and shap_values by the number of trees the model outputs match exactly and I get shap values and a force_plot that are similar to the shap values and force_plot when using sklearn.ensemble.RandomForestRegressor. I guess my question then is what advantage is there in having shap values represented in the untransformed space for lightgbm random forest?

Interesting. It is really driven by the choice of LightGBM to encode their RF trees that way. It is a bit odd that they would use a raw_score that is not averaged and then only average the trees as a post-processing step. We could get back to the more expected behavior by adding a special case for LightGBM RF models that divides all leaf values by the number of trees. I'll leave a todo tag here in case anyone wants to tackle that.

Oh I see. Okay thanks again for your help and all your work on shap!

I also faced the same issue of sum(Shap values) not matching the predict_proba() output. However, dividing by some constant does not solve the problem in my case.
Its the logit transformation that did the job; Predict_proba = 1/(1+e^(-raw score))
Now I am worried that shap values are getting learned from raw_scores which is not a linear transformation of Predict_proba.

@jatiniitr I am facing the same issue where for most examples  sum(shap_values) do not match preds or raw_scores for LGBM binary classifier.
Any solution?

Facing the same issue with LGBM binary classifier


feat: get swagger instance from register

Describe the PR
Problem starts if our swagger instance generated by CI and registered.
In main program if we want to modify some parameters inside of that instance is it not possible because code not in there.
Getting swagger instance with name from register it is easy to work with it, without need any workaround.
Swagger is an interface so we should do some casting but we know the our code and it should be this type.
if s, ok := swag.GetSwagger("swagger").(*swag.Spec); s != nil && ok {
	s.Version = "1.0.0"
	s.Title = "Test"
}

Relation issue
#1240

@rytsh Thanks for your contribution.


Wounds greater than 16 error

https://github.com/ThePants999/Yellowscribe/blob/af80deece46eb0a824d911c9d778d75f61aa99e9/lua_modules/MatchedPlay_Constants.lua#L55
When playing with the Triumph of Saint Katherine (18 wounds), the decrement wounds function fails because there are only 16 valid wound colors. When decrementing with the hotkey, the error "Indexing a NiL value" comes up.
This can be resolved by adding 2 more entries to the WOUND_COLOR_CUTOFFS dictionary.
local WOUND_COLOR_CUTOFFS = {     {g=0}, -- 1     {g=1}, -- 2     {g=2, o=1}, -- 3     {g=2, o=1}, -- 4     {g=3, o=1}, -- 5     {g=4, o=2}, -- 6     {g=5, o=2}, -- 7     {g=6, o=3}, -- 8     {g=6, o=3}, -- 9     {g=7, o=4}, -- 10     {g=7, o=4}, -- 11     {g=8, o=4}, -- 12     {g=8, o=4}, -- 13     {g=8, o=5}, -- 14     {g=9, o=5}, -- 15     {g=9, o=5}, -- 16     {g=9, o=5}, -- 17     {g=9, o=5} -- 18 }
Additionally, a more robust way of dealing with more wounds than 18 may need to be thought of, but the proposed solution should work for all models up to 18 wounds.

Damn, I thought I'd done this already, I had Triumph complaints weeks ago - guess I'm hallucinating :D  Thanks.

Happy to help!

But I've checked in 2376bcd1e38afa98dd9b259a46153b2017bd9eac anyway as a workaround.


Dataset Request

Hi, I requested access to the Mapillary Vistas Dataset ten days ago, but have yet received any relay. Is the dataset still available for the public?

I'm in the same situation. I sent them an email to ask for guidance, but received nothing yet.
Did you get any response from them?

I'm in the same situation. I sent them an email to ask for guidance, but received nothing yet.
Did you get any response from them?

I'm in the same situation. I sent them an email to ask for guidance, but received nothing yet.
Did you get any response from them?

not yet.

I'm in the same situation. I sent them an email to ask for guidance, but received nothing yet.
Did you get any response from them?

not yet.

Hi @michaelowenliu and @amwfarid!
Apologies for not getting back on this earlier, however, we have just upgraded the dataset (see this blog post), and made access easier through direct download. Just go to vistas.mapillary.com, fill out the request form again and you will immediately receive an email with download links and instructions how to unpack. HTH

Hi @michaelowenliu and @amwfarid!
Apologies for not getting back on this earlier, however, we have just upgraded the dataset (see this blog post), and made access easier through direct download. Just go to vistas.mapillary.com, fill out the request form again and you will immediately receive an email with download links and instructions how to unpack. HTH

@pkontschieder Thank you so much! I didn't realize there was an upgrade going on. I confirm that I got the download links.
Thank you so much for the hard efforts in making this dataset possible!

@pkontschieder Thank you so much! I didn't realize there was an upgrade going on. I confirm that I got the download links.
Thank you so much for the hard efforts in making this dataset possible!


Fix asan issue for soci::use

Description of the Change
Pull request fixes asan issue (see https://gist.github.com/luckychess/f830ab0d50d31ac4f340d5a4fd61bbc9 for details).

Benefits
No more issue :-)

Possible Drawbacks
None.

Usage Examples or Tests
Run irohad under asan and then run python example - no issues should be reported.
Similar issues and docs:

https://github.com/SOCI/soci/pull/659/files (same issue fix)
http://soci.sourceforge.net/doc/3.2/exchange.html#bind_local - check Object lifetime and immutability section

P.S.: I don't understand why an old code even compiles.

Please merge it after #12


Refactor sunlight logic into modules.

Move the service call into the sundataservice.  getSunData will return a promise that will be fulfilled with the converted data.


renderCustomComponent with avatar is not working

according to the docs the 3rd param should be a Boolean, but
renderCustomComponent(Component, props, true) doesn't seem to work

Fix in new version v2.1.2! 😄


Capture is ignoring the amount property

Without it, we can't implement partial captures.
const capture = (opts, body) =>
  request.post(opts, routes.transactions.capture(body.id), {})

https://github.com/pagarme/pagarme-js/blob/master/lib/resources/transactions.js#L79

https://github.com/pagarme/pagarme-js/pull/53

reopening to close when we merge the PR :)


ModuleNotFoundError: No module named 'debug_toolbar'

Following the Installation with SSL I got the error below when running docker run --env-file=shynet.env shynet-ssl:latest. I built the image from master. I then rebuilt the image with v0.10.0 and that didn't produce the error.
Traceback (most recent call last):
  File "/usr/src/shynet/./manage.py", line 21, in <module>
    main()
  File "/usr/src/shynet/./manage.py", line 17, in main
    execute_from_command_line(sys.argv)
  File "/usr/local/lib/python3.9/site-packages/django/core/management/__init__.py", line 419, in execute_from_command_line
    utility.execute()
  File "/usr/local/lib/python3.9/site-packages/django/core/management/__init__.py", line 413, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/usr/local/lib/python3.9/site-packages/django/core/management/base.py", line 354, in run_from_argv
    self.execute(*args, **cmd_options)
  File "/usr/local/lib/python3.9/site-packages/django/core/management/base.py", line 393, in execute
    self.check()
  File "/usr/local/lib/python3.9/site-packages/django/core/management/base.py", line 419, in check
    all_issues = checks.run_checks(
  File "/usr/local/lib/python3.9/site-packages/django/core/checks/registry.py", line 76, in run_checks
    new_errors = check(app_configs=app_configs, databases=databases)
  File "/usr/local/lib/python3.9/site-packages/django/core/checks/urls.py", line 40, in check_url_namespaces_unique
    all_namespaces = _load_all_namespaces(resolver)
  File "/usr/local/lib/python3.9/site-packages/django/core/checks/urls.py", line 57, in _load_all_namespaces
    url_patterns = getattr(resolver, 'url_patterns', [])
  File "/usr/local/lib/python3.9/site-packages/django/utils/functional.py", line 48, in __get__
    res = instance.__dict__[self.name] = self.func(instance)
  File "/usr/local/lib/python3.9/site-packages/django/urls/resolvers.py", line 598, in url_patterns
    patterns = getattr(self.urlconf_module, "urlpatterns", self.urlconf_module)
  File "/usr/local/lib/python3.9/site-packages/django/utils/functional.py", line 48, in __get__
    res = instance.__dict__[self.name] = self.func(instance)
  File "/usr/local/lib/python3.9/site-packages/django/urls/resolvers.py", line 591, in urlconf_module
    return import_module(self.urlconf_name)
  File "/usr/local/lib/python3.9/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1030, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1007, in _find_and_load
  File "<frozen importlib._bootstrap>", line 986, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 680, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 850, in exec_module
  File "<frozen importlib._bootstrap>", line 228, in _call_with_frames_removed
  File "/usr/src/shynet/shynet/urls.py", line 18, in <module>
    import debug_toolbar
ModuleNotFoundError: No module named 'debug_toolbar'

Fixed in v0.11.0! Thanks.


opt add a variable to control varchar length in ctas (#37069)

pick from master #37069
add a new session variable: use_max_length_of_varchar_in_ctas
In CTAS (Create Table As Select), if CHAR/VARCHAR columns do not originate from the source table, whether to set the length of such a column to MAX, which is 65533. The default is true.

run buildall

run compile


Two Tables on the Same page

@Sayyiditow Did you close this issue because you figured out how to display more than one table on the same page? I'm stuck with a related issue. If I conditionally render one table at the time (always just one <Datatable> visible at any given time), it works fine. Once I try to display two tables simultaneously, it crashes -- wonder if it is related to the use of stores. Did you figure out a way around it?

@Sayyiditow Did you close this issue because you figured out how to display more than one table on the same page? I'm stuck with a related issue. If I conditionally render one table at the time (always just one <Datatable> visible at any given time), it works fine. Once I try to display two tables simultaneously, it crashes -- wonder if it is related to the use of stores. Did you figure out a way around it?


turbo - doesnt work

i downloaded the Text Encoders：
CLIPs: OpenCLIP-ViT/G, CLIP-ViT/L, context length 77 tokens
T5: T5-xxl, context length 77/256 tokens at different stages of training
and the checkpoint model.
but it doesnt work.
i have a 3090rtx.
FLUX works
and many other SDXL models work fine.

It looks like you're using Automatic1111's stable-diffusion-3.5, which doesn't use this repository. A1111 doesn't support 3.5 at the moment but will likely support it soon. I'd keep a tab on their issues page to see when they'll support it.


test on current python/django versions, linter config

This PR continues the wider platform and version upgrades work I am looking at across DC repos.
Key topics on this one are bumping to current/applicable python/django versions and bringing linter setup into line with other DC projects.
Commit log should be pretty descriptive for this one. I will also leave some comments inline on the diff with a bit of further explanation.

TODO: once this is approved, cut a release after merging

I tested this by installing it locally in Website. Is the plan to remove pyproject.toml from upstream repos or is there further configuration needed to designate which pyproject.toml should be used in the event that more than one is found?

I tested this by installing it locally in Website. Is the plan to remove pyproject.toml from upstream repos or is there further configuration needed to designate which pyproject.toml should be used in the event that more than one is found?

Chris and I had a chat about this and he explained more about how pyproject.toml  works so all sorted now.


'noexcept' : unknown override specifier

Visual Studio won't compile my project with the json.hpp header because of 92 errors, mostly this one.
Am I missing something?

You should use Visual Studio 2015, which support noexcept keyword. Moreover this lib still doesn't compile under windows. You may check out this PR https://github.com/nlohmann/json/pull/88

Hi @ajmetal,
as @kepkin said, Visual Studio support is still suboptimal, and I am investigating how to get the code running without making too many changes.
Sorry for the inconvenience,
Niels


How get the response object

Actually im send a post with aplication/urlencoded but only response me the body object and I need the response object.

Check out the resolveWithFullResponse option. Hope this helps.

@analog-nico thanks, its working well 👍


Added build step to prepare script

Added build step to prepare script and Removed redundant calls of build script

Why:
Closes #222

What's being changed:

Check off the following:

[ ] I have reviewed my changes in staging (look for the latest deployment event in your pull request's timeline, then click View deployment).
[x] For content changes, I have completed the self-review checklist.

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

[ ] This pull request impacts the contribution experience

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

I have added npm run build to the prepare script in package.json. As a result, I have attempted to remove all redundant calls of npm run build.
However, there are some calls that I am not sure how to handle. Here are they:

https://github.com/FarazzShaikh/docs/blob/b08a65a90373d6cd92604b7ca63d6fd78b5a1265/.github/workflows/sync-single-english-algolia-index.yml#L51
https://github.com/FarazzShaikh/docs/blob/b08a65a90373d6cd92604b7ca63d6fd78b5a1265/.github/workflows/test-windows.yml#L64
https://github.com/FarazzShaikh/docs/blob/b08a65a90373d6cd92604b7ca63d6fd78b5a1265/.github/workflows/test.yml#L69
https://github.com/FarazzShaikh/docs/blob/b08a65a90373d6cd92604b7ca63d6fd78b5a1265/script/enterprise-server-deprecations/archive-version.js#L110
https://github.com/FarazzShaikh/docs/blob/b08a65a90373d6cd92604b7ca63d6fd78b5a1265/Dockerfile#L58

And maybe other that I might have missed. These calls perform some operations after npm ci and before npm run build.

@FarazzShaikh Thanks so much for opening a PR! I'll get this triaged for review ⚡

Thanks for opening this PR @FarazzShaikh! Sometimes we need to run packages without making a build, having to build to install would be a deal breaker for some processes, so we can't accept this PR.
Thanks so much for your interest and passion in improving the GitHub docs 💛

@janiceilene thank you for your review. You should close #222 as well if this cannot be implemented.

@FarazzShaikh We really appreciate the PR you offered up. 🙇🏻♂
Unfortunately, when the larger team reviewed your PR, one of the valid issues mentioned was that this change (adding a prepare script to run the npm run build script) would've forced us to always run the build, even when we don't want or need to, and thus slowing down certain processes (e.g. in many of our Actions workflows). That was a perspective that I hadn't considered when I encouraged you to create this PR. 😓
Since npm does not currently provide us a mechanism to opt-out of that behavior (other than --ignore-scripts, which also ignores all of the dependencies' lifecycle scripts, too 😬 🙅🏻♂), we decided it was better for us to stick with the opt-in behavior that we currently have. While we understand that does put a slight burden on contributors to execute that one extra step, we believe it is something that is easily enough learned and worked around. 🤷🏻♂
Thank you especially for your thoroughness in taking the extra step of updating the contributor documentation, the Codespaces configuration, and the Actions workflows as well. ✨
We are thankful for your involvement and effort. We would love to see additional contributions from you in the future. ❤

Understood, Thanks for letting me know!  😄


Incorrect colorization after using triple quoted string

I added triple quotes around code, and colorization is incorrect. Reopening the file fixes it.

May be related to https://github.com/Microsoft/PTVS/issues/4236

I retested this with latest from master to see if today's fixes took care of it, but no, not quite.

Also reported at https://developercommunity.visualstudio.com/content/problem/275331/python-triple-quotes.html

This is still an issue in VS2019 version 16.2.1
It seems that when you use triple quotes, the following line is in red (as it's supposed to be) but all subsequent lines after that, within the quotes, aren't.


Refactor menu and main() for better performance, error handling and readability, work on structure.

The current main function copies many blocks such as try and except, it makes code slower and inefficient. We should work on a code structure and add helper function to reduce repetitive input checks.
Context: This would simplify our error handling and will make our code easier to read. The well-designed structure will make our code more readable.

I would like to work on this issue. Please assign it to me.