Keybase install error on MacOS

Hi, I have been receiving this error message while doing a fresh install of Keybase on the Mac: "There was an error trying to run the install (2). Please run 'keybase log send' to report the error." The options Quit and Ignore are presented. Tapping ignore just leaves the GUI with the keybase logo and 'loading...'.
Also, it does look like the CLI tool isn't installed at the moment.
My log id: 4e01722c030895f43f19481c
Any help would be really appreciated.

Hi, I'm struggling with the same problem as @jithinrk mentioned above.
While doing an install of Keybase an error window occurred.

"Ignoring" this message leads to the following screen, which is loading endless.

Furthermore the CLI tool doesn't work (also as mentioned above).
Operating System: macOS 10.13.6 High Sierra
Thanks for your help.

cc @strib

@max why the cc?  The redirector isn't mentioned here.
@Firni11 if you run keybase log send and post the resulting log ID here, someone can look in more detail.

@strib
I tried, but "keybase: comand not found".

can anyone help?

It seems like you haven't fully installed the app.  Did you drag it into your Applications folder and launch it from there?

I have the same issue, and have been trying to run the Keybase app from the Applications folder (the Keybase command isn’t installed on my system either)

I'm afraid I don't know anything about the mac command line installation.  @cjb do you happen to know why that wouldn't work?

@strib
Yeah. I dragged the app into the application folder. From there I started the app and the window (which i mentioned above) popped out. Also all „keybase“ commands are impossible to run in the terminal. Terminal gives back:“command not found“.

Huh, can't find anything for that log ID.
My guess would be something like: you have /usr/local/bin already created, but it's owned by a different user, and your current user can't put /usr/local/bin/keybase into it.  Could that be it?
(Alternatively: you're out of disk space?)

I have the same issue -- has anyone resolved this on their own?

@rridolfo  please keybase log send

I've done this twice.   The resulting terminal output is:
MACC02VW06XHTDF:MacOs rich$ ./Keybase log send
No installer.json file
Version: 2.5.1-20180830164425+a18ce51fa9
2018-09-11 10:03:28.424 Keybase[3842:257927] *** WARNING: Textured window <AtomNSWindow: 0x7fdbdceb5190> is getting an implicitly transparent titlebar. This will break when linking against newer SDKs. Use NSWindow's -titlebarAppearsTransparent=YES instead.
Exec path not found (or accessible as executable): /Applications/Keybase.app/Contents/SharedSupport/bin/keybase
The result of running this is that it launches the client, which indicates "Loading..." and never progresses.

I also can't install keybase, when I attempt to send in logs I get a segfault 11
keybase log send
Segmentation fault: 11


Issue on homepage (Cannot read property 'members' of undefined)

After I set this up locally on my computer and on Glitch , I get the following error on the Home page

Bro did you add server id in config?
On Sat, Aug 7, 2021 at 1:37 AM Kane. @.***> wrote:

After I set this up locally on my computer, I get the following error on
the Home page
[image: image]
https://user-images.githubusercontent.com/50002439/128565729-600d6455-fdb2-48ec-b44f-8e80f56cbcbf.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Disbotsxyz/Disbots/issues/3, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AU3NHRXMV5KOK7R2KDSUZJLT3Q6HVANCNFSM5BWQ4TEQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

My bad, didn't saw the last few lines.
After testing, maintanance doesn't really work as it should! It's not possible to remove it...

Bro maintenance mode has error so if you turn on maintenance it will show
redirect u many times, so go to mongodb and remove maintenance and if you
have any more questions pls join our discord server thankyou! Invite link:
https://discord.gg/ YhTU6Akzmy
On Sat, Aug 7, 2021 at 3:27 PM Kane. @.***> wrote:

My bad, didn't saw the last few lines.
After testing, maintanance doesn't really work as it should! It's not
possible to remove it...
[image: image]
https://user-images.githubusercontent.com/50002439/128596383-32fe3b8e-12a5-46f8-a328-315ae6f2ee19.png
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Disbotsxyz/Disbots/issues/3#issuecomment-894633170,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AU3NHRVCHFZ6RMVK2IFMD6LT3T7PJANCNFSM5BWQ4TEQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

It worked before the changes where made 5 hours ago.
https://cdn.discordapp.com/attachments/865182535118094356/873538418842415154/unknown.png

Fixed.....


Read workflow definition and run from code

Is it possible to execute  a definition read using Workflow reader from code.

Hello @rmanjusha! No, that's not possible with the SDK alone.
You can however do that with Synapse, which uses this SDK.


send alert without series data

Hello,
I am using TICK Stack to monitor network devices by snmp and send the resulting alerts (high bandwidth utilization) to an alerta console.  I noticed that every event has the following json series list data attached to it and I wonder how can I get rid of it, which means I want to disable the forwarding of the raw measurement values when I send out an alert. Is it possible and if so how ? I was not able to find anything related in the docs...
thanks in advance,
best regards,
Tom
{
"series": [
{
"columns": [
"time",
"inbytes.ifHCInOctets",
"inbytes.ifHCOutOctets",
"inbytes.ifHighSpeed",
"inbytes.ifName",
"inbytes.ifSpeed",
"inbytes.in_percent",
"inbytes.rate",
"inbytes.rate_bits",
"outbytes.ifHCInOctets",
"outbytes.ifHCOutOctets",
"outbytes.ifHighSpeed",
"outbytes.ifName",
"outbytes.ifSpeed",
"outbytes.out_percent",
"outbytes.rate",
"outbytes.rate_bits",
"stateduration_inpercent",
"stateduration_outpercent"
],
"name": "interface.tsy",
"tags": {
"hostname": "my_hostname",
"ifDescr": "my_ifDescr"
},
"values": [
[
"2020-11-18T13:40:09Z",
45994261355,
11415107218,
4,
"Gi8.100",
4092000,
92.00348321928968,
470597.81666666665,
3764782.533333333,
45994261355,
11415107218,
4,
"Gi8.100",
4092000,
11.443297491039427,
58532.46666666667,
468259.73333333334,
10,
-1
]
]
}
]
}

Hello,
I was finally able to drop the series data within alerta itself. Solved!
best regards,
Tom


Install kibana 4.5.0 by default (requires elasticsearch 2.3.0, updated config).

This PR updates this cookbook to install the recently released kibana 4.5.0, which goes with the recently released elasticsearch 2.3.0.
Please note that the format of the kibana configuration file (see attributes/config.rb) has changed; you should release a new major version of this cookbook if you decide to merge this PR.

One note: readme - autogenerated file (from doc/overview.md)

Thank you very much for the new release :)


Question: is the precondition on init(shape:scalars:) meant to be permanent?

Hello,
this may be not a genuine TF question, but just for my information, I've noticed that
let w2 = Tensor<Float>(shape: [N, N], scalars: giveMeAnArrayNxN(N))
won't compile, which I think relates to the initialiser design here:
https://www.tensorflow.org/api_docs/swift/Structs/Tensor#initshapescalars
i.e. there's a precondition on the [Scalar] size. In other words, it looks like the code snipet is never going to work, since  giveMeAnArrayNxN() returns [Scalar].
What would be your recommendation? Any hopes to dynamically allocate Tensors?

I'm not sure what the return type of giveMeAnArrayNxN is, but you may be interested in another Tensor initializer which accepts [[Float]]†:
let matrix = Tensor<Float>([[1, 2], [3, 4]])
// matrix represents [[1.0, 2.0], [3.0, 4.0]]

†: All Float values passed to the initializer must be literals. This is because under the hood, the ExpressibleByFloatLiteral and ExpressibleByArrayLiteral initializers for Tensor are called. The conformances of Tensor to those two protocols are defined in Tensor.swift.

That’s what I suspected with the precondition, but that’s a big limitation. This means that everywhere in the code, tensors have a fixed size.
So dynamic size allocation isn’t possible, nor planned in future releases?

Hi Dan,
sorry, maybe my question was confusing, the idea is to have the same flexibility that exists with arrays in the definition of Tensors. That is, an array size can be defined at run time.
Let's take an example
//n is an Int32, which value is know at run time.

let foobar:[Float] = (0..<n*n).map { (tt) -> Float in
                return Float(tt);
              }

let w2 =   Tensor<Float>(shape: [n, n], scalars: foobar)

I understand this is not possible yet. My question is will this ever be possible, or will a Tensor instantiation always require literals?
Reason: I have to deal with objects which phase space is known at run time only, if Tensors must be allocated with literals, I'll have to pre-allocate a zoo of tensors of various shapes and sizes and pick one that will most closely fit the current said object, reassign all values to zero, and assign new values where appropriate. Very far from being workable.
I understand why it makes sense in ML to have Shape/Sizes with literals (fixed number of active nodes), but in my case it's highly inefficient.
Thanks for your help.

Taking your example:
import TensorFlow
var x = Tensor<Float>([2])
// x represents [2.0] and has initial shape [1]
x += Tensor<Float>(ones: [2, 2])

can this ever work:
import TensorFlow
var x = Tensor<Float>([2])
// x represents [2.0] and has initial shape [1]
x += Tensor<Float>(ones: [n, n])

where n is know at run time?

Hi @Mamonaku , currently Swift enforces "constant expressions" for TensorShape. This is to allow Swift code work uniformly with all platform that TensorFlow supports, including CPUs, GPUs and TPUs. TPU execution through XLA compilation currently requires tensor shapes to evaluate to constants. We will consider relaxing this constraint!

array algebra is almost impossible to optimise if the number and distribution of elements are unknown at compile time.

Actually, let me take this back... What's costly is dealing with arrays which size grows unpredictably, but arrays of fixed capacity, even allocated at runtime, are handled just fine, and optimisations were routinely enforced (Eg. never had to recompile anything just to change an Array length)
What would I break if I was to change
  precondition(scalars.count == contiguousSize,
               "The number of scalars does not match the shape.")

by
  assert(scalars.count == contiguousSize,
               "The number of scalars does not match the shape.")

Assuming that scalars.count == contiguousSize is always true?
(Sorry, I don't mean to be nosy, even if this looks like it)

mmm... Something, somewhere, got lost, and it's on me... Let me take a more careful look.
thanks for your patience, and help, I'm all set.


Add params_to_tune for catboost models

Before submitting (must do checklist)

[x] Did you read the contribution guide?
[x] Did you update the docs? We use Numpy format for all the methods and classes.
[x] Did you write any new necessary tests?
[x] Did you update the CHANGELOG?

Proposed Changes
Look #1184.
Closing issues
Closes #1184.

Codecov Report

:exclamation: No coverage uploaded for pull request base (automl-2.0@d964bda). Click here to learn what that means.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##             automl-2.0   #1185   +/-   ##
============================================
  Coverage              ?   7.85%
============================================
  Files                 ?     177
  Lines                 ?   10389
  Branches              ?       0
============================================
  Hits                  ?     816
  Misses                ?    9573
  Partials              ?       0

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Update swiftui tests for iOS 16.1

Sectioned table views are now implemented using a collection view rather than a table so those need to be updated. A few behavioral changes in 16.0 appear to have reverted back to the pre-16 behavior as well.

Very nice, finally fixing this and get a green check in the CI.
Two things, seems like the matching for app.buttons.matching(identifier: "Reorder"). is not working in the latest iOS releases.
Also, I tested this in an iPhone 8 which seems to be the chosen simulator for running this tests and some table cells count are not equal to the one in the test, because of the screen size.
I'm testing on an iPhone 8 (16.1).

The cell count depending on screen size is super awkward. I guess I need to make it check screen size and adjust the expected amount based on that. Another option might be to use a really small text size for the test app size so that they all fit as it doesn't actually need to be usable.

I've updated the tests to hopefully be screen-size independent.


[Extensibility Request] local to global Codeunit 5407 "Prod. Order Status Management" - OutputExists

Hello, we need the following currently Internal procedure to be set as Global. Can you please check this? Thank you!
[Extensibility Request] Codeunit 5407 "Prod. Order Status Management" - Procedure from Internal to Global
local procedure OutputExists(ProdOrderLine: Record "Prod. Order Line"): Boolean
    var
        ItemLedgEntry: Record "Item Ledger Entry";
    begin
        ItemLedgEntry.SetCurrentKey("Order Type", "Order No.", "Order Line No.");
        ItemLedgEntry.SetRange("Order Type", ItemLedgEntry."Order Type"::Production);
        ItemLedgEntry.SetRange("Order No.", ProdOrderLine."Prod. Order No.");
        ItemLedgEntry.SetRange("Order Line No.", ProdOrderLine."Line No.");
        ItemLedgEntry.SetRange("Entry Type", ItemLedgEntry."Entry Type"::Output);
        if ItemLedgEntry.FindFirst() then begin
            ItemLedgEntry.CalcSums(Quantity);
            if ItemLedgEntry.Quantity <> 0 then
                exit(true)
        end;
        exit(false);
    end;

@JesperSchulz Can you please check this? Thank you!

This is done but request was not closed due to sync issue.

We had some issues with our automation which reports back progress from our internal systems to these GitHub issues. Some issues may not have been updated, such as this one - even though they were successfully processed! We will look at getting that fixed in the next weeks. Sorry about the inconvenience!


Inject vendor bundle globally

The case:

There is old school angular application.
There is index.html, that contains a list of vendor dependencies (jquery, jquery-ui, moment, lodash, angular and so on). After vendor dependencies there is a huge list of angular components, related to the application itself.
All vendor dependencies are minified.

I'm trying to use webpack to build all the stuff and create two bundles vendor.js and application.js. I don't want to use not minified versions of the vendor scripts, because of two reasons:

it will increase the build time, since I'll have to at lease uglify them for production.
probably, vendor knows how to uglify sources better, than me. For example, uses Google Closure compiler with advanced flag.

I've played around combination of externals, ProviderPlugin, exports, imports, expose-loaders, raw, but my solution look like crutch.
Is there some straightforward way to include all minified vendor files in the global scope, as it would be if I would have included them in the document in <script> tag?

I would create 2 builds:
module.exports = {
  entry: {
    vendor: './src/vendor.js',
    application: './src/app.js',
  },
  output: {
    path: './dist/',
    filename: '[name].js'
  }
}

then include both script tags as needed.
You can expose a library globally with window.$ = window.jQuery = require('jquery')
If you need to dynamically add a script tag:
var script = document.createElement('script')
script.src = '/path/to/script.js'
document.body.appendChild(script)

or last resort is the script-loader which uses execScript/eval to simulate a script being added like it was placed with a script tag. The script loader has some drawbacks with source maps and some security concerns though.


[build-script-impl] Enable llbuild's Swift bindings

This is no longer a dependency problem because llbuild now builds after
Swift.

depends on https://github.com/apple/swift-llbuild/pull/224

@swift-ci please smoke test


feat: let user plugins add plugins in config hook

Description
Todo: Explain stuff
Todo: Support worker.plugins

New public utils
This PR also adds a few helper functions for Vite plugin arrays:

flattenVitePlugins
Encapsulates the asyncFlatten call with plugin filtering based on apply and truthiness
sortVitePlugins
Sort a plugin array in-place using the enforce property of each plugin
splitVitePlugins
Split a plugin array into 3 arrays: pre, normal, post

Back when I originally implemented this, these helper functions were something I needed in Saus, but not anymore, so we can make them private if you think that's best.

What is the purpose of this pull request?

[ ] Bug fix
[x] New Feature
[ ] Documentation update
[ ] Other

With Plugin initializers being able to return arrays of plugins and async support for initializers, i'm not sure there is a need for this.
The only reason would be that you want to add a plugin based on a condition that can only be evaluated within the config hook, not within vite.config.js
To keep the current behavior of config hooks being called exactly once for each plugin and each plugin gets the complete list of plugins that is going to be in the final config, you would have to recursively recreate all plugins until the list of plugins has stabilized. and then recreate them again a final time.
I'm not seeing that happening in the implementation here, so you are changing the conditions of what plugins can and can't expect from the config passed in (or even if their config hook was called more than once)
This is a can of worms imho and unless there is a great usecase, i'd rather it not be opened

@dominikg has a good point. As we didn't see others' requests for this feature, I think it may be better if we close this PR for the time being until we gather enough use cases to justify this feature.


Remove s2n_map

Issue # (if available):
https://github.com/awslabs/s2n/issues/879
Description of changes:
Remove all map references
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

I apologize for not commenting on #879 earlier! I was planning on using this data structure to implement #3 . I have not submitted a PR for this yet, but have this code in a branch that I am working on and will be submitting a few PRs soon. Would you be okay with leaving s2n_map in for now or is there another reason why you would like to remove it?

I was just going though recent issues and looking for easy stuff to clean up. I'll update this to only remove the one import.


Add WiNS data

[ ] upload logo in good resolution / vector format to assets/images/organizations_logos
[x] add markdown file with metadata to content/english/organizations

I'm still looking for a HQ logo, nobody has it ahah


Word boundary (\b)

This is my code:
public OnPlayerText(playerid, text[])
{
    new Regex:r = Regex_New("\b(word1|word2|word3)\b", REGEX_ICASE),
        RegexMatch:m,
        startpos, pos;

    if(Regex_Search(text, r, m, pos, startpos))
    {
        SendClientMessage(playerid, 0xF154ACFF, "Detected");
        return 0;
    }

    Regex_Delete(r);
    return 1;
}

This is how it should work:
https://i.imgur.com/wtbb9Wi.png
When I type word1 or word2 or word3 ingame, nothing happens. But, when I remove '\b' from regex pattern it works, but not the way I want. Without using '\b' it will capture word even if it's mixed within another word.
Like this:
https://i.imgur.com/1LdPxtl.png

Regex_New("\\b(word1|word2|word3)\\b"

Regex_New("**\b(word1|word2|word3)**\b", REGEX_ICASE)

Thanks, sir :+1:


Plugin causing indexing to hang

A recent update to the plugin (not sure which) has caused IntelliJ's indexing process to hang when indexing the purescript files in the spago libraries in my project. I've confirmed it's the plugin specifically: it happens reliably every time with the plugin enabled, but with the plugin disabled indexing runs fine.
Oddly, the spago libraries in my project are listed as excluded directories, so I'm not sure why IntelliJ is even trying to index them with the plugin enabled; I'm not sure if this specific aspect is related, though, or if this is an unrelated issue on IntelliJ's side.

@jhilyar https://plugins.jetbrains.com/plugin/9738-purescript/versions/stable/347370 should fix this, please reopen if your issues are not fixed by this version

Unfortunately I am still seeing the same issue, yeah: I've installed 0.79.5, but the index update is still hanging on my Spago libraries. There doesn't seem to be any consistency in which file it hangs on either? Each time, the file it's indexing when getting stuck has been different, and there hasn't appeared to be anything out of the ordinary about any of them that I've looked at.

Can you publish your dependencies, and their versions, so that i can use it as a test to see if I have fixed your issue?

Actually, while grabbing those for you, I think I may have found the issue? We have a few dependencies where we've forked and tweaked the original package, referencing the forked repo as the repo in packages.dhall and the commit hash as the version (as documented here). So far it's always been one of those particular packages that the plugin is hanging on. Could using this package override definition be confusing the plugin?

@i-am-the-slime do you have the same experience (now after the fix). That you don't have issues with normal packages.

This is a closed project at my organization, unfortunately. I'll check if there's any issue with sharing our dependency list, though I think that should be fine.

If you can't. Then it would be interesting If you could create a new empty project and see if you have the same issues in that one.
And if not see if just adding the dependencies to it makes the bug appear.

@jhilyar What I managed to do was to click on the message about indexing and see the file it got stuck on. Then I could comment out parts of the file in binary search style fashion until I found the offending bit of code. Maybe that's an option for you too. In my case it was this:
newtype UseDrawRectangle hooks = UseDrawRectangle
  ( UseRef Instant
      ( UseState DrawState
          ( UseMotionValue Number
              ( UseMotionValue Number
                  ( UseMotionValue Number
                      ( UseMotionValue Number
                          ( UseRef (Maybe DrawModifier)
                              ( UseRef
                                  ( Maybe
                                      { current ∷ { x ∷ Number, y ∷ Number }
                                      , start ∷ { x ∷ Number, y ∷ Number }
                                      }
                                  )
                                  hooks
                              )
                          )
                      )
                  )
              )
          )
      )
  )

The example above is now parsed quickly (under 10 ms on my machine). @jhilyar try this new version https://plugins.jetbrains.com/plugin/9738-purescript/versions/stable/358231

@Neppord Looks like it did, yep! Thanks for the fix!


Type hinting and PHPDoc

In another issue, a question was raised about a function's PHPdoc - in particular, the function had been like this
/**
 * @param string $a
 * @param array $b
 */
public function foo (string $a, $b) {...}

I removed "@param string $a" in order to pass the inspection "no_superfluous_phpdoc_tags" (which is implied by @symfony in the distributed .php_cs.dist file). That raised a question among the reviewers and the statement that one reviewer would like to see PHPdoc type hints for all parameters.
I'm creating this issue as a locus for any follow-up conversation.
If we want to see PHPdoc type hints for all parameters, we would have to follow one of these courses:

remove php-native type hints (which would reduce code reliability)
provide comments on each variable (which tends to generate a lot of repetitive and uninformative text)
override the configuration and set "no_superfluous_phpdoc_tags" to false

I can live with any choice that makes the code and tests comply with the inspection profiles we ship alongside. But I prefer the configuration we have. In my mind, the desired ideal is that nearly all variables are self-documenting and are constrained by php-native types. With the current codebase, we can verify that things like
ETL::load(Loader $loader, string $output, $options = []): Etl {}
can be refactored to
ETL::load(Loader $loader, string $output, array $options = []): Etl
It seems implied by the code, but since that could introduce a breaking change, so it also seems worth explicitly verifying.
There are about as dozen places in the code where the variable type is mixed. With the current inspection profiles, those variables would need to retain their PHPdoc. In all other cases, PHPdoc would be provided for variables only when the variable name was not felt to be adequately self-documenting.

There hasn't been any comment in this, but bits and pieces in other cards lead me to believe we want to avoid writing redundant PHPdoc when we have type hints for properties with PHP 7.4 and when we make that release we'll be removing stuff that becomes redundant.
Does anyone disagree?

I personally have a preference for adding PHPDoc tags for all parameters. I have no authority regarding this package, so my opinion might not be worth much :) But I prefer this way because, even through it seems redundant, it's easier to scan for missing/incomplete PHPDocs if the number of tags matches the number of parameters. I believe it helps reduce the number of mistakes.


Idea for new ruleset: check if recordId is saved as record and not as String

Since some releases, it is possible to pass the recordId as record to a screenflow by record page or quick action. This saves you one DML operation and you can directly access the whole record. So it would be helpful to check the flows based on this setting. I assume it is an easy one for you to check if the flow contains and return true/false
<variables>
        <name>recordId</name>
        <dataType>String</dataType>
        <isCollection>false</isCollection>
        <isInput>true</isInput>
        <isOutput>false</isOutput>
    </variables>

What can I do for you? Is this article helpful? https://salesforcetime.com/2021/10/20/using-an-action-to-pass-the-entire-record-to-a-flow/

@fkramer690 for this one, I have been thinking of scenarios where you might want to pass the record id to a subflow(perhaps even geneneric). I can see in some orgs it might be Best Practice, but in others maybe not, so instead of adding it to the default Im considering if this should be a custom rule. Id love to hear your thoughts

@fkramer690 I just realize I totally misunderstood this and now see you can use a record as starting point, sorry!! Would this rule be only to visual flows?

@RubenHalman auto launched flows would also work but not record triggered flows


inconsistency between loading relationship "normally" and eager loading it

Laravel Version: 5.4.19
PHP Version: 7.0.15-0ubuntu<IP_ADDRESS> (my tests are in artisan tinker)
Database Driver & Version: sqlite

Description:
i have a belongsToMany relationship, with a union (my goal was using belongsToMany to override a default value via a left join)
it gives the result i expect when called "normally" (ie get the model and call the relationship). but when i try to eager load the relationship it ignores the union
public function rules() {
    	$game = $this;
    	$nonMatches = Rule::whereNull('game_rule.value')->leftJoin('game_rule', function($join) use ($game){
    		$join->on('rules.id', '=', 'rule_id')->where('game_id', $game->id);
    	});
    	return $this->belongsToMany(Rule::class)->withPivot('value')->withTimestamps()->union($nonMatches)->orderBy('id');
}

I realize my use case is beyond what belongsToMany was designed for, but as far as im aware eager loading should give the same results, just with "normally" less queries

This is not a bug. Use the forums or the slack channel to get your issue resolved. :)


Merge with latest from upstream master

Work towards #57.

Sorry, I merged #97 which I think needs to be brought in here before you can merge :/
Since this is a big change, I'll probably diff the modified files against master to make sure we're in sync.

Done! PTAL.

Hmm. Author consent check failed. @scolsen, can you check it both emails you used for commits are all registered with GitHub?

Oh, I think I just manually give my consent and you can merge it by applying a label?

It's up to you to confirm consent of all the commit author(s), set the cla label to yes (if enabled on your project), and then merge this pull request when appropriate.

At any rate I can register my other address if that helps.

(Googler, please make sure the reason for overriding the CLA status is clearly documented in these comments.)

All commits in question are by @scolsen, who has signed CLAs and is reviewer on this thread.

Looks like we still have no luck here? Let me see if making another account does the trick.

Iiuc you started a review but never actually approved/closed it? It says there's one pending review. I'll try to dismiss it now.

Ah ok, we still need a review with your approval. Can you please restamp?


[SPARK-35447][SQL] Optimize skew join before coalescing shuffle partitions

What changes were proposed in this pull request?

This PR improves the interaction between partition coalescing and skew handling by moving the skew join rule ahead of the partition coalescing rule and making corresponding changes to the two rules:

Simplify OptimizeSkewedJoin as it doesn't need to handle CustomShuffleReaderExec anymore.
Update CoalesceShufflePartitions to support coalescing non-skewed partitions.

Why are the changes needed?

It's a bit hard to reason about skew join if partitions have been coalesced. A skewed partition needs to be much larger than other partitions and we need to look at the raw sizes before coalescing.
It also makes OptimizeSkewedJoin more robust, as we don't need to worry about a skewed partition being coalesced with a small partition and breaks skew join handling.
It also helps with https://github.com/apache/spark/pull/31653 , which needs to move OptimizeSkewedJoin to an earlier phase and run before CoalesceShufflePartitions.
Does this PR introduce any user-facing change?

No
How was this patch tested?

new UT and existing tests

cc @maryannxue @JkSelf @yaooqinn @viirya

Test build #138716 has started for PR 32594 at commit 7f40a92.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43238/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43238/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/43238/

Test build #138716 has finished for PR 32594 at commit 7f40a92.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/138716/

It's a bit hard to reason about skew join if partitions have been coalesced. A skewed partition needs to be much larger than other partitions and we need to look at the raw sizes before coalescing.

The raw size here is still the size after compression, not the uncompressed size?

The raw size here is still the size after compression, not the uncompressed size?

Yes, the raw size means size before coalescing.

Look fine to me

Test build #138878 has started for PR 32594 at commit 6a6ceda.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43399/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43399/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/43399/

Test build #138878 has finished for PR 32594 at commit 6a6ceda.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/138878/

Test build #138903 has started for PR 32594 at commit 19dfaa0.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43425/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43425/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/43425/

Test build #138903 has finished for PR 32594 at commit 19dfaa0.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/138903/

Test build #138919 has started for PR 32594 at commit fa0065b.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43440/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43440/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/43440/

Test build #138919 has finished for PR 32594 at commit fa0065b.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

thanks for the review, merging to master!

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/138919/


Failed to setup frame size

Required Info

Camera Model
D435i

Operating System & Version
Linux (Ubuntu 18.04 LTS)

Platform
NVIDIA Jetson Nano

Language
python

Segment
Robot

Issue Description
I have a problem with the camera. I am trying to set the frame size to 640x480 but it is not working:
import pyrealsense2 as rs
...
config = rs.config()
config.enable_stream(rs.stream.color,640,480,rs.format.bgr8,15)
I am trying to use this config function, but I am getting frames of 1280x720 instead.
Any idea how to fix that?

Hi @erkarim85  I cannot find any problem with the stream definition in the code snippet that you posted.  Would it be possible to post the full script, please?
Also, which librealsense version and camera firmware version are you using?

the pyrealsense2 version is 2.31.0
I am not sure how to get the firmware version
Here is the complete code:
`import pyrealsense2 as rs
import numpy as np
import cv2
import datetime
fps = 15
time = 20
x = 640
y = 480
pipeline = rs.pipeline()
config = rs.config()
config.enable_stream(rs.stream.color, x, y, rs.format.bgr8, fps)
pipeline.start()
fourcc = cv2.VideoWriter_fourcc(*'XVID')
out = cv2.VideoWriter('OneMinVid.avi',fourcc,fps,(x,y))
print(datetime.datetime.now())
try:
l = 0
for i in range(fps*time):
frame = pipeline.wait_for_frames()
Cframe = frame.get_color_frame()
if not Cframe:
continue
color_image = np.asanyarray(Cframe.get_data())
tmp = color_image[..., ::-1]
out.write(tmp)
if l == 0:
print(tmp.shape)
print(color_image.shape)
l = 1
    cv2.imshow('i',tmp)
    cv2.waitKey(1)

finally:
pipeline.stop()
print(datetime.datetime.now())
out.release()
`

The first thing that stands out to me is that your pipeline start instruction does not have a reference to config in the brackets, so the program would not know to apply your defined configuration to the pipeline and would use the default configuration for the camera model instead.  Please try changing the pipeline start instruction to the one below:
pipeline.start(config)

Awesome, thank you! This fixed it. I used it like you said and the video was in the right dimensions.
However, I tried to move forward to use depth and color, but I got an error, below is my code:
fps = 15
time = 10
x = 640
y = 480
#max0 = 0
#depth_array = np.zeros((y,x,fps*time))
pipeline = rs.pipeline()
config = rs.config()
config.enable_stream(rs.stream.color, x, y, rs.format.bgr8, fps)
config.enable_stream(rs.stream.depth, y, x, rs.format.z16, fps)
pipeline.start(config)
fourcc = cv2.VideoWriter_fourcc(*'XVID')
out = cv2.VideoWriter('color.avi',fourcc,fps,(x,y))
outd = cv2.VideoWriter('depth.avi',fourcc,fps,(x,y))
But I got an error in this line: pipeline.start(config)

Your depth config line is using y, x instead of x, y like the color definition does.  You are therefore requesting a resolution of 480x640, which the pipeline cannot fulfill because RealSense 400 Series cameras do not support that resolution.

I see. I just read it somewhere that the depth should be reversed. It is working fine now.
Thank you very much for your support.

You are very welcome - I'm pleased I could help!


Expose shard type property with the introduction of internal sharding

Enhancement Request
Internal sharding is a great addition for many bots, but it now introduces the possibility of client.shard being a thing without there being multiple processes.
In order to make switching between a development and production environment simple on a sharded bot I would frequently use
if(client.shard){ //Do some shard code (usually broadcast eval) }else{ //Do some normal code }
However this is no longer a valid option with internal shading.
Adding a property to the shard to determine if it was spawned via a manger or internal would open a lot of options

writing code in general for internal vs manager will be very different and shouldn't be done in such a way that they can be easily switched, as this would be bad for perf and such. IMO we should not do this

I agree having constant checks would indeed be a hit in performance. But the way I do this, it shouldn't be.
I have some operations that need to resolve guilds, update object properties that pose some challenge on sharded environments requiring the use of broadcast eval. To make my life easier being able to switch between my non-sharded dev bot, few shards beta-bot, and many shards prod-bot. I restructured the few functions that do call upton these methods.
On bot launch I create a object that is "mostly" globably accessible. I have a 2 .js files, one contains normal DJS functions, the other contains broadcastEval equivalents. Then based on the sharded status on the process, I mount one set or the other to said object which can then be called as needed without additional checks.
I guess this "Could" open up some avenues for poor code and performance, but aren't there already many many avenues for said issues.

Give person a gun, maybe he shoots himself, maybe he doesn't

i don't think you quite understand the structure of internal sharding. It just makes more ws connections. To the end user you can't tell its even sharding. The only way to tell will be client.ws.shards.size > 1


🐛 BUG: Cannot use node builtin polyfill "crypto"

What version of Wrangler are you using?
2.0.5
What operating system are you using?
macOS 12.4
Describe the Bug
In my worker’s code, I make use of Node’s built-in crypto package. Before upgrading to wrangler 2, it more or less worked (I had to use the crypto-browserify workaround). wrangler 2 refuses to build the code outright, even though tsc compiles it without a problem.
bash-3.2$ wrangler dev
 ⛅ wrangler 2.0.5
-------------------
Running custom build: npm run build

> <REDACTED>@<REDACTED> build
> tsc

⬣ Listening at http://localhost:8787
╭───────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╮
│ [b] open a browser, [d] open Devtools, [l] turn on local mode, [c] clear console, [x] to exit                                     │
╰───────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯
✘ [ERROR] Could not resolve "crypto"

    ../<REDACTED>.js:26:36:
      26 │ const Crypto = __importStar(require("crypto"));
         ╵                                     ~~~~~~~~

  The package "crypto" wasn't found on the file system but is built into node. Are you trying to
  bundle for node? You can use "platform: 'node'" to do that, which will remove this error.

✘ [ERROR] Build failed with 1 error:

  ../<REDACTED>.js:26:36: ERROR: Could not resolve "crypto"

If you think this is a bug then please create an issue at https://github.com/cloudflare/wrangler2/issues/new.

Does wrangler actually need to build the code for the browser? Can it be customized to build for Node instead?

Can you add node_compat = true to your wrangler.toml and try again?

Hi @threepointone! Thanks for the suggestion!
Setting node_compat to true makes it possible to build the project, but the crypto module specifically does not work as expected (it doesn’t work at all, throwing errors like Crypto.createHash is not a function).
With this option enabled, wrangler displays the following warning:
▲ [WARNING] Enabling node.js compatibility mode for built-ins and globals. This is experimental and has serious tradeoffs. Please see https://github.com/ionic-team/rollup-plugin-node-polyfills/ for more details.

…and the linked repo’s README says, “Crypto is not shimmed and and we just provide the commonjs one from browserify and it will likely not work, if you really want it please pass {crypto: true} as an option.”
So it seems there’s still a need for further customization options in wrangler config.

Oh hmm, ok we should just add the browserify one with {crypto: true}.
Some immediate options for you here -

enhance your custom build step to add the crypto polyfill yourself
replace usage of node's crypto with the web crypto api
Stick with wrangler 1 while we solve this on our end.

The module we use for this doesn't have options for enabling crypto right now. We want to replace that module with our own, and that would make enabling that much simpler. Overall, we plan on making node_compat better, but work on that might not start for a couple of weeks, we're in the middle of planning at the moment.
I'll leave this issue till we can fix your situation, happy to keep discussing options. Extremely sorry about this experience.

Thank you!

let's continue the discussion in https://github.com/cloudflare/wrangler2/issues/1345


[Toolkit bug]: Crashes when loading a scene with custom references

Contact Details (Optional)
No response
Describe the bug
Remix kept crashing when loading into a project until I renamed the capture, causing it to fail to find it and allowing me to load back in without crashing (I can load the same capture after opening the project and it works just fine).
kit_20240129_014758.log
How do you reproduce the bug?

Import a custom USD file into a scene via a custom reference.
Save the project.
Close Remix Toolkit.
Launch Remix Toolkit.
Load up the project
It crashes at this point

What is the expected behavior?
The toolkit should load the project without crashing.
Version
2024.1.1

Hi @hypsin0 not sure yet, but this may have more to do with the USD file. Does that still happen with a really simple USD file?
Are you able to share any of the files your using? capture or usd file?
Lastly, can you elaborate on step 1 a bit? Are you doing an asset replacement?
Thanks :)

USDs.zip
I am doing asset replacements. After reading the log, I decided to try loading a different project first whenever I restarted Remix Toolkit. That avoids the crash

Hello! Will you please test this on the latest release and let us know if the issue persists? Thank you!

I can confirm that the issue is still the same — if I try to open my Rome
Remix when the last map I opened was the world map, it crashes. If I first
open a different project, and then open the Rome Remix project (still
loading the world map), it works just fine.
On Mon, 6 May 2024, 23:21 NV-LL, @.***> wrote:

Hello! Will you please test this on the latest release and let us know if
the issue persists? Thank you!
—
Reply to this email directly, view it on GitHub
https://github.com/NVIDIAGameWorks/rtx-remix/issues/384#issuecomment-2097023307,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BFVPTALAV57XSHEMG54LEI3ZA765XAVCNFSM6AAAAABCOUNXWSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJXGAZDGMZQG4
.
You are receiving this because you were mentioned.Message ID:
@.***>


species-wise merge

merge should be done for each species but not implemented so currently.

This will make it easier to handle the data in the curate step as it should be done species-wise. It would be nice to store the merge outputs in species-wise directories like work_dir/merge/GENUS_SPECIES/. Could you work on this?

I think we had a conversation years ago on how amalgkit outputs should be structured. Do you still have the note I took? You can follow that.

Amalgkit update <IP_ADDRESS>:

amalgkit merge now organises output species-wise when metadata guided

https://github.com/kfuku52/amalgkit/commit/ff964bc39848f6c981242108e884764c4b54a3c1


Add multi-scale spectral loss

Content
This adds multi-scale spectral loss as described in [1] in paragraph 4.2.1. The implementation is compatible with the PITLossWrapper module with 'wo_src ' mode.
Also adds 2 unit tests for this loss function
TODO
Implement 'pairwise' and 'w_src' compatibility for PITLossWrapper module
References
[1] Jesse Engel and Lamtharn (Hanoi) Hantrakul and Chenjie Gu and
Adam Roberts
DDSP: Differentiable Digital Signal Processing
International Conference on Learning Representations
ICLR 2020, https://openreview.net/forum?id=B1x1ma4tDr

Last CI build is not successful. I suppose it is because of the new pytorch lightning version, I rebase master with the old PL version and see if the CI passes.

Actually, the build was done with 0.7.0 which was the latest version for 2hours, I think its should be fine with 0.7.1.


IOS notifications are removed automatically when the app goes in foreground

When I have some notifications of my app in the Notification Center of IOS, they are removed automatically when I open my app even if I don't tap on one of them.
This is incorrect behavior and this strange behavior only happens on IOS but not on Android.
Someone tells it causes because of using PushNotificationIOS.setApplicationIconBadgeNumber(0), but I don't do this in my code. It may run and be set automatically in the react-native-push-notification/ios package, IDK.
Someone fixed this problem for a flutter package by removing two lines of the code that he mentioned here.
Also, when I send 3-4 notifications to my real device through Firebase messaging panel, I see them in the Notification Center. All of them will be removed/disappear when I tap one of them and it is not the right behavior. Just that notification I tapped should be removed not all of them.

What I need and I think is correct and normal behavior:

When users touch the app icon, no notification should disappear from Notification Center.

When users touch a notification in the Notification Center, only that notification should be removed or disappear from there.

Versions:
"react-native": "^0.64.0"
"@react-native-community/push-notification-ios": "^1.10.1"
"react-native-push-notification": "^8.1.1"

do you call removeAllDeliveredNotifications somewhere ?

Make sure you provide a unique id to all the notifications. That should solve your problem.


Incompatibility with postfx

Hi, as I tried to use postfx : https://github.com/cansik/processing-postfx/  with peasycam  I got this bug :

As you can see, the original is displayed in the background, and the post fx render appears in the front, scaled down.
I tried to move the focus point of my peasycam object, but it doesn't fix the issue :/
a sketch which reproduce this bug :

import ch.bildspur.postfx.builder.*;
import ch.bildspur.postfx.pass.*;
import ch.bildspur.postfx.*;

import peasy.*;
import peasy.org.apache.commons.math.*;
import peasy.org.apache.commons.math.geometry.*;

PeasyCam cam;

PostFX fx;
void setup(){
  size(400,400,P3D);
  cam=new PeasyCam(this, 800);
  fx = new PostFX(this);
}
void draw(){
  background(100);
  pushMatrix();
translate(width/2,height/2);
rotateY(frameCount/17.0);
box(200);
    popMatrix();
 fx.render()
    .bloom(0.5, 20, 40)
    .compose();

}

regards

I posted the issue here too : https://github.com/cansik/processing-postfx/issues/28


Recursive require calls in node_modules aren't resolved when the @babel/env preset is used

I'm trying to package some code that includes a tree of modules inside node_modules, using rollup-plugin-node-resolve, rollup-plugin-babel, and rollup-plugin-commonjs. It works until I add the @babel/env preset, at which point require() calls sufficiently deep in the tree are left in the generated code.

Rollup Version: 1.20.3
Operating System (or Browser): OS X 10.13.6
Node Version: 10.16.3

How Do We Reproduce?

Clone https://github.com/sgravrock/rollup-babel-node-resolve-bug.
Run npm install and rollup -c.
Notice the presence of require() calls in dist/built.js.
Remove the @babel/env preset from rollup.config.js and re-run rollup -c.
Notice that the require() calls in dist/built.js have been replaced with the contents of the corresponding modules.

Alternately, if you prefer not to clone the repo, create rollup.config.js:
import resolve from 'rollup-plugin-node-resolve';
import babel from 'rollup-plugin-babel';
import commonjs from 'rollup-plugin-commonjs';

export default {
    input: 'index.js',
    output: {
        format: 'iife',
        name: 'foo',
        file: 'dist/built.js'
    },
    plugins: [
        resolve({
            preferBuiltins: false
        }),
        babel({
            "presets": [
                ["@babel/env"]
            ]
        }),
        commonjs()
    ],
};

and index.js:
const URL = require("whatwg-url").URL;

Expected Behavior
All require() calls are replaced with the contents of the resolved module.
Actual Behavior
Some require() calls are left in the generated code.

You should put a commonjs plugin before babel one. Otherwise, commonjs plugin might recognize some CJS files as modules (as they were already transformed by babel and some import statements could have been added to them).


dev: Add link to Dart package for JokeAPI

Added link to the Dart package with API wrapper for JokeAPI
Description
I've decided to develop API wrapper, so... I did it. Please, approve it or leave notes on how I can improve this API wrapper.
Related Issue
No related issues
Motivation and Context
This will solve problem of using JokeAPI in Dart and Flutter
How Has This Been Tested?
All existing endpoints wrappers are covered with tests by mocking HTTP client
Checklist

[x] I have read and followed the Contributing Guide
[x] I have read and accept the Code of Conduct (TLDR: just behave in a friendly manner)
[x] My code follows the general style of this project (explained here)
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation (if applicable)
[x] I ensured that the automated checks that ran on this PR have completed successfully
[ ] I added myself to the package.json file (optional but highly encouraged :)

Thanks a lot! I'll make sure to add it in the next update.


Errors should be more explicit. Closes #44.

With this branch:
$ python foo.py
Traceback (most recent call last):
  File "foo.py", line 13, in <module>
    response = client.search('San Francisco', term=u'ș'.encode('utf8'))
  File "/nail/home/kmitton/pg/other/yelp-python/yelp/endpoint/search.py", line 44, in search
    self.client._make_request(SEARCH_PATH, url_params)
  File "/nail/home/kmitton/pg/other/yelp-python/yelp/client.py", line 48, in _make_request
    return self._make_connection(signed_url)
  File "/nail/home/kmitton/pg/other/yelp-python/yelp/client.py", line 54, in _make_connection
    self._error_handler.raise_error(error)
  File "/nail/home/kmitton/pg/other/yelp-python/yelp/errors.py", line 107, in raise_error
    response
yelp.errors.InvalidParameter: {'msg': 'Bad Request', 'text': u'One or more parameters are invalid in request: term', 'code': 400, 'id': u'INVALID_PARAMETER'}

Compared to master:
...
yelp.errors.InvalidParameter:

tox green on 2.7 and 3.4 with my credentials in tests/json/credentials_secret.json for integration testing, and relatively-green without those creds.

Coverage remained the same at 100.0% when pulling e3ac014f02de53b37711e901d87ead89ad20b23e on u/mittonk/issue-44 into 4a8bea62cd337085f86dcc75d3f58f7f92a09234 on master.

merged in pull request!


MDM Section Is Empty

On this page, the section 'Editing an AppLocker policy by using Mobile Device Management (MDM)' is completely empty.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 9639e297-41c6-5e4f-6b64-45812976448b
Version Independent ID: 227e7fd0-b79c-9f93-754e-c3d26b02cd07
Content: Edit an AppLocker policy (Windows 10) - Windows security
Content Source: windows/security/threat-protection/windows-defender-application-control/applocker/edit-an-applocker-policy.md
Product: m365-security
Technology: mde
GitHub Login: @Dansimp
Microsoft Alias: dansimp

@e0i , issue is conformed , article must be update

Thanks @ivassallo19 for the feedback and thanks @MaratMussabekov for making the edit.


Do not send w3c capability as a workaround for browsestack schema validation

Ref #644

Merged to community branch, @williamdes can you try with dev-community dependency in your composer.json?

@OndraM works fine !

Released in 1.7.1.


Fixed for freezing issue when reloading at the chrome

Fixed for freezing issue when reloading at the chrome.
https://developers.google.com/web/updates/2017/09/autoplay-policy-changes

Thanks for that @Rybots!
Would it make more sense having this as an option? Perhaps passed to the constructor? I think for some situations you wouldn't necessarily want it to start muted - e.g when a user gesture starts playback


Wrong result

Hi,
Your project may be in very early stage. But I thought to add to comparison of various jsonpath js libraries. But I seen that it gave wrong output in case of first query.
I just wanted to to inform you.
var jp = require('@f5io/jsonpath');

var obj = {
    'a': {
        1: 'la',
        2: 'boo',
        'h': [
            {foo: [1,2,3]},
            {foo: [4,5,6]},
            {foo: 12, name: 'a'},
            {foo: 13.5, name: { 'h': 45}},
            {foo: 11.8, name: 'c'},
            true,
            123,
            [3,4,5]
        ]
    },
    'b': {
        1: 'la',
        2: 'boo',
        'h': [
            {foo: [1,2,3]},
            {foo: [4,5,6]},
            {foo: 12, name: 'a'},
            {foo: 13.5, name: { 'h': 45}},
            {foo: 11.8, name: 'c'},
            true,
            123,
            [3,4,5]
        ]
    },
    'c': {
        1: 'la',
        2: 'boo',
        'h': [
            {foo: [1,2,3]},
            {foo: [4,5,6]},
            {foo: 12, name: 'a'},
            {foo: 13.5, name: { 'h': 45}},
            {foo: 11.8, name: 'c'},
            true,
            123,
            [3,4,5]
        ]
    },
    'd': {
        1: 'la',
        2: 'boo',
        'h': [
            {foo: [1,2,3]},
            {foo: [4,5,6]},
            {foo: 12, name: 'a'},
            {foo: 13.5, name: { 'h': 45}},
            {foo: 11.8, name: 'c'},
            true,
            123,
            [3,4,5]
        ]
    }
}

var patterns = {
    pattern1: '$..h[*].foo',
    pattern2: '$.b[1,2]',
    pattern3: '$.b.2',
    pattern4: '$..h[?(@.foo>13)]'
}

console.log(jp(patterns.pattern1,obj)); //Incorrect
console.log(jp(patterns.pattern2,obj)); //correct
console.log(jp(patterns.pattern3,obj)); //correct
console.log(jp(patterns.pattern4,obj)); //correct

Hi @amit, thanks for taking the time to look into doing a comparison.
To be perfectly honest, this implementation is pretty loose to the JSONPath specification, so I wouldn't expect a lot of queries to give an exact match to other libraries, however, I am always looking to improve it.
Could you provide me with an actual response and an expected response, obviously I could do this myself, but it would be good to document it here too.
Again, thanks, I will make sure to look into it.

For the first query,
Expected
[ [ 1, 2, 3 ],
  [ 4, 5, 6 ],
  12,
  13.5,
  11.8,
  [ 1, 2, 3 ],
  [ 4, 5, 6 ],
  12,
  13.5,
  11.8,
  [ 1, 2, 3 ],
  [ 4, 5, 6 ],
  12,
  13.5,
  11.8,
  [ 1, 2, 3 ],
  [ 4, 5, 6 ],
  12,
  13.5,
  11.8 ]

Actual
[ [ 1, 2, 3 ],
  [ 4, 5, 6 ],
  12,
  13.5,
  11.8,
  undefined,
  undefined,
  undefined,
  undefined,
  [ 1, 2, 3 ],
  [ 4, 5, 6 ],
  12,
  13.5,
  11.8,
  undefined,
  undefined,
  undefined,
  undefined,
  [ 1, 2, 3 ],
  [ 4, 5, 6 ],
  12,
  13.5,
  11.8,
  undefined,
  undefined,
  undefined,
  undefined,
  [ 1, 2, 3 ],
  [ 4, 5, 6 ],
  12,
  13.5,
  11.8,
  undefined,
  undefined,
  undefined,
  undefined ]

However not completely wrong ;)
From performance wise it is best among all 5 jsonpath implementation. So I'll suggest if you can complete and test it against all the scenarios.
I'm excited to put it on changejs.

Ah it failed one more test. So I'm removing it from comparison table. But hoping for it's progress.

@amitguptagwl: I have a fix ready to go for the previous incorrect response. Is there anywhere I can see the other test specs which you are trying?

Try this code

var arraySize = 2333,
    resultCount = 150,
    itemCount = 50,
    groupCount = 145;

var json = {
    results: []
};

var i, j;

var bigArray = [];
for (i = 0; i < arraySize; i++) {
    bigArray[i] = 1;
}

var items = [];
for (i = 0; i < itemCount; i++) {
    items[i] = JSON.parse(JSON.stringify({a: {b: 0, c: 0}, s: {b: {c: bigArray}}}));
}

for (i = 0; i < resultCount; i++) {
    json.results[i] = {groups: [], v: {v: [1, 2, 3, 4, 5, 6, 7, 8]}};
    json.results[i].groups = [];
    for (j = 0; j < groupCount; j++) {
        json.results[i].groups[j] = {items: items, a: "121212"};
    }
}

var jp = require('@f5io/jsonpath');
console.log(jp('$.results[*].groups[*].items[42]',json));

Seems I have become a tester (have secured my future) ;)

thanks, will have a look at this hopefully this evening 👍

Plz give a star, if it helped you a bit work

Just so you know, I am looking at resolving the other issue. I am not sure it is my misunderstanding of the, admittedly loose, spec or the other implementations misinterpretation. I will continue looking


fix(bundler): ensure that there are no duplicate extension arguments when bundling on Windows, fixes #6103

This PR fixes #6103 .
Suppose we intend to enhance the installer by extending WIX fragments. In one of these fragments, we utilized the wix util/ui extension. However, the initial code would add any extension used in fragments as light arguments, which resulting in a bug.
This pull request replaced the previous fragment_extensions vector with a hashset in order to resolve the bug.
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Docs
[ ] New Binding issue #___
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change?

[ ] Yes, and the changes were approved in issue #___
[x] No

Checklist

[x] When resolving issues, they are referenced in the PR's title (e.g fix: remove a typo, closes #___, #___)
[ ] A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.
[ ] I have added a convincing reason for adding this feature, if necessary

Other information

Did the check fail because of the trailing comma in line 44?
If so, I can remove it

Ok to go

@biaocy looks like you've added a commit that is no signed, you will need to rebase your PR and sign all your commits

I apologize for the confusion. I tried to sign before the initial push, but I was not aware that it was necessary to include the line 'exec git commit --amend --no-edit -n -S' in order for the signing to take effect. As a result, my attempt at signing did not work.


Unable to display image on README.md file

The image link only displaying instead of image with following

Use below to display image:

stackoverflow Solution Link


Add HTML Generator & Fixes

This PR

Fix processing empty string in Crux
Fix marking "No answer extracted" as correct
Add HTML generator

Please run python state_of_limit/result_summary.py locally again asstate_of_limit/task_summary_mmlu-redux.json file is >100MB (may need to open git lfs or gitignore it).
To generate HTML, please run python state_of_limit/write_html_single_file.py

thank you!


ManageTransaction transaction does not rollback on ConcurrencyError during commit

Laravel Version: 6.20 to 8.0
PHP Version: 7.4
Database Driver & Version: PDO and Percona XtraDBCluster 5.7

Description:
I am getting the There is already an active transaction when using queue:work.
This happens after 20-30 min.
I was able to capture the error from the database it is:
WSREP detected deadlock/conflict and aborted the transaction. Try restarting the transaction

By looking at
src/Illuminate/Database/Concerns/ManagesTransactions.php@transaction
it seems that the exception on commit is not properly handled and the transaction is not rollback, is that by intention?
From what I see when we handle the transaction we are doing a rollback:
            // If we catch an exception we'll rollback this transaction and try again if we
            // are not out of attempts. If we are out of attempts we will just throw the
            // exception back out and let the developer handle an uncaught exceptions.
            catch (Exception $e) {
                $this->handleTransactionException(
                    $e, $currentAttempt, $attempts
                );

                continue;
            } catch (Throwable $e) {
                $this->rollBack();

                throw $e;
            }

but when we handle the exception during the commit we just continue.
            try {
                if ($this->transactions == 1) {
                    $this->getPdo()->commit();
                }

                $this->transactions = max(0, $this->transactions - 1);
            } catch (Exception $e) {
                $commitFailed = true;

                $this->handleCommitTransactionException(
                    $e, $currentAttempt, $attempts
                );
                continue;
            }

The end result is an endless loop where we want to work with a DB transaction that was started and never committed or rollback which fires the There is already an active transaction.
My proposal is to add
            try {
                if ($this->transactions == 1) {
                    $this->getPdo()->commit();
                }

                $this->transactions = max(0, $this->transactions - 1);
            } catch (Exception $e) {
                $commitFailed = true;

                $this->handleCommitTransactionException(
                    $e, $currentAttempt, $attempts
                );
                // Proposal
                $this->rollback();
                //
                continue;
            }

Steps To Reproduce:
This is a race condition, but I think it could be handled by the framework too.

@driesvints there were not change on Laravel 8 on how the exceptions are handled during commit.

I have the same issue too，Please tell me, the issue has sloved already in latest laravel version?


Cmake error while building for Android

Hello,
I'm getting this error while trying to build for Android.
It used to work, but I cannot pinpoint easily which dependency or Flutter version triggered the issue, but I've recently switched to Flutter  3.8.0-10.1.pre on the beta channel.
Launching lib/main.dart on Android SDK built for x86 in debug mode...
Running Gradle task 'assembleDebug'...

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':flutter_zxing:buildCMakeDebug[arm64-v8a]'.
> com.android.ide.common.process.ProcessException: ninja: Entering directory `/home/mauro/.pub-cache/hosted/pub.dev/flutter_zxing-1.1.1/android/.cxx/Debug/1l4a6h4w/arm64-v8a'
  [1/87] Building CXX object zxing/core/CMakeFiles/ZXing.dir/src/BinaryBitmap.cpp.o
  FAILED: zxing/core/CMakeFiles/ZXing.dir/src/BinaryBitmap.cpp.o
  /home/mauro/Android/Sdk/ndk/23.1.7779620/toolchains/llvm/prebuilt/linux-x86_64/bin/clang++ --target=aarch64-none-linux-android21 --gcc-toolchain=/home/mauro/Android/Sdk/ndk/23.1.7779620/toolchains/llvm/prebuilt/linux-x86_64 --sysroot=/home/mauro/Android/Sdk/ndk/23.1.7779620/toolchains/llvm/prebuilt/linux-x86_64/sysroot  -I/home/mauro/.pub-cache/hosted/pub.dev/flutter_zxing-1.1.1/src/zxing/core/src -Izxing/core -g -DANDROID -fdata-sections -ffunction-sections -funwind-tables -fstack-protector-strong -no-canonical-prefixes -D_FORTIFY_SOURCE=2 -Wformat -Werror=format-security   -fno-limit-debug-info  -fPIC -DZXING_BUILD_READERS -DZXING_BUILD_WRITERS -Wall -Wextra -Wno-missing-braces -Werror=undef -Werror=return-type -pthread -std=gnu++17 -MD -MT zxing/core/CMakeFiles/ZXing.dir/src/BinaryBitmap.cpp.o -MF zxing/core/CMakeFiles/ZXing.dir/src/BinaryBitmap.cpp.o.d -o zxing/core/CMakeFiles/ZXing.dir/src/BinaryBitmap.cpp.o -c /home/mauro/.pub-cache/hosted/pub.dev/flutter_zxing-1.1.1/src/zxing/core/src/BinaryBitmap.cpp
  In file included from /home/mauro/.pub-cache/hosted/pub.dev/flutter_zxing-1.1.1/src/zxing/core/src/BinaryBitmap.cpp:6:
  In file included from /home/mauro/.pub-cache/hosted/pub.dev/flutter_zxing-1.1.1/src/zxing/core/src/BinaryBitmap.h:13:
  /snap/flutter/current/usr/include/c++/9/memory:121:25: error: cast from pointer to smaller type 'uintptr_t' (aka 'unsigned int') loses information
    const auto __intptr = reinterpret_cast<uintptr_t>(__ptr);
                          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  1 error generated.
  [2/87] Building CXX object zxing/core/CMakeFiles/ZXing.dir/src/GlobalHistogramBinarizer.cpp.o
  FAILED: zxing/core/CMakeFiles/ZXing.dir/src/GlobalHistogramBinarizer.cpp.o
  /home/mauro/Android/Sdk/ndk/23.1.7779620/toolchains/llvm/prebuilt/linux-x86_64/bin/clang++ --target=aarch64-none-linux-android21 --gcc-toolchain=/home/mauro/Android/Sdk/ndk/23.1.7779620/toolchains/llvm/prebuilt/linux-x86_64 --sysroot=/home/mauro/Android/Sdk/ndk/23.1.7779620/toolchains/llvm/prebuilt/linux-x86_64/sysroot  -I/home/mauro/.pub-cache/hosted/pub.dev/flutter_zxing-1.1.1/src/zxing/core/src -Izxing/core -g -DANDROID -fdata-sections -ffunction-sections -funwind-tables -fstack-protector-strong -no-canonical-prefixes -D_FORTIFY_SOURCE=2 -Wformat -Werror=format-security   -fno-limit-debug-info  -fPIC -DZXING_BUILD_READERS -DZXING_BUILD_WRITERS -Wall -Wextra -Wno-missing-braces -Werror=undef -Werror=return-type -pthread -std=gnu++17 -MD -MT zxing/core/CMakeFiles/ZXing.dir/src/GlobalHistogramBinarizer.cpp.o -MF zxing/core/CMakeFiles/ZXing.dir/src/GlobalHistogramBinarizer.cpp.o.d -o zxing/core/CMakeFiles/ZXing.dir/src/GlobalHistogramBinarizer.cpp.o -c /home/mauro/.pub-cache/hosted/pub.dev/flutter_zxing-1.1.1/src/zxing/core/src/GlobalHistogramBinarizer.cpp
  In file included from /home/mauro/.pub-cache/hosted/pub.dev/flutter_zxing-1.1.1/src/zxing/core/src/GlobalHistogramBinarizer.cpp:7:
  In file included from /home/mauro/.pub-cache/hosted/pub.dev/flutter_zxing-1.1.1/src/zxing/core/src/GlobalHistogramBinarizer.h:9:
  In file included from /home/mauro/.pub-cache/hosted/pub.dev/flutter_zxing-1.1.1/src/zxing/core/src/BinaryBitmap.h:13:
  /snap/flutter/current/usr/include/c++/9/memory:121:25: error: cast from pointer to smaller type 'uintptr_t' (aka 'unsigned int') loses information
    const auto __intptr = reinterpret_cast<uintptr_t>(__ptr);
                          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  1 error generated.
  [3/87] Building CXX object zxing/core/CMakeFiles/ZXing.dir/src/HybridBinarizer.cpp.o
  FAILED: zxing/core/CMakeFiles/ZXing.dir/src/HybridBinarizer.cpp.o
  /home/mauro/Android/Sdk/ndk/23.1.7779620/toolchains/llvm/prebuilt/linux-x86_64/bin/clang++ --target=aarch64-none-linux-android21 --gcc-toolchain=/home/mauro/Android/Sdk/ndk/23.1.7779620/toolchains/llvm/prebuilt/linux-x86_64 --sysroot=/home/mauro/Android/Sdk/ndk/23.1.7779620/toolchains/llvm/prebuilt/linux-x86_64/sysroot  -I/home/mauro/.pub-cache/hosted/pub.dev/flutter_zxing-1.1.1/src/zxing/core/src -Izxing/core -g -DANDROID -fdata-sections -ffunction-sections -funwind-tables -fstack-protector-strong -no-canonical-prefixes -D_FORTIFY_SOURCE=2 -Wformat -Werror=format-security   -fno-limit-debug-info  -fPIC -DZXING_BUILD_READERS -DZXING_BUILD_WRITERS -Wall -Wextra -Wno-missing-braces -Werror=undef -Werror=return-type -pthread -std=gnu++17 -MD -MT zxing/core/CMakeFiles/ZXing.dir/src/HybridBinarizer.cpp.o -MF zxing/core/CMakeFiles/ZXing.dir/src/HybridBinarizer.cpp.o.d -o zxing/core/CMakeFiles/ZXing.dir/src/HybridBinarizer.cpp.o -c /home/mauro/.pub-cache/hosted/pub.dev/flutter_zxing-1.1.1/src/zxing/core/src/HybridBinarizer.cpp
  In file included from /home/mauro/.pub-cache/hosted/pub.dev/flutter_zxing-1.1.1/src/zxing/core/src/HybridBinarizer.cpp:7:
  In file included from /home/mauro/.pub-cache/hosted/pub.dev/flutter_zxing-1.1.1/src/zxing/core/src/HybridBinarizer.h:9:
  In file included from /home/mauro/.pub-cache/hosted/pub.dev/flutter_zxing-1.1.1/src/zxing/core/src/GlobalHistogramBinarizer.h:9:
  In file included from /home/mauro/.pub-cache/hosted/pub.dev/flutter_zxing-1.1.1/src/zxing/core/src/BinaryBitmap.h:13:
  /snap/flutter/current/usr/include/c++/9/memory:121:25: error: cast from pointer to smaller type 'uintptr_t' (aka 'unsigned int') loses information
    const auto __intptr = reinterpret_cast<uintptr_t>(__ptr);
                          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  1 error generated.
  [4/87] Building CXX object zxing/core/CMakeFiles/ZXing.dir/src/ReedSolomonDecoder.cpp.o
  [5/87] Building CXX object zxing/core/CMakeFiles/ZXing.dir/src/MultiFormatReader.cpp.o
  FAILED: zxing/core/CMakeFiles/ZXing.dir/src/MultiFormatReader.cpp.o
  /home/mauro/Android/Sdk/ndk/23.1.7779620/toolchains/llvm/prebuilt/linux-x86_64/bin/clang++ --target=aarch64-none-linux-android21 --gcc-toolchain=/home/mauro/Android/Sdk/ndk/23.1.7779620/toolchains/llvm/prebuilt/linux-x86_64 --sysroot=/home/mauro/Android/Sdk/ndk/23.1.7779620/toolchains/llvm/prebuilt/linux-x86_64/sysroot  -I/home/mauro/.pub-cache/hosted/pub.dev/flutter_zxing-1.1.1/src/zxing/core/src -Izxing/core -g -DANDROID -fdata-sections -ffunction-sections -funwind-tables -fstack-protector-strong -no-canonical-prefixes -D_FORTIFY_SOURCE=2 -Wformat -Werror=format-security   -fno-limit-debug-info  -fPIC -DZXING_BUILD_READERS -DZXING_BUILD_WRITERS -Wall -Wextra -Wno-missing-braces -Werror=undef -Werror=return-type -pthread -std=gnu++17 -MD -MT zxing/core/CMakeFiles/ZXing.dir/src/MultiFormatReader.cpp.o -MF zxing/core/CMakeFiles/ZXing.dir/src/MultiFormatReader.cpp.o.d -o zxing/core/CMakeFiles/ZXing.dir/src/MultiFormatReader.cpp.o -c /home/mauro/.pub-cache/hosted/pub.dev/flutter_zxing-1.1.1/src/zxing/core/src/MultiFormatReader.cpp
  In file included from /home/mauro/.pub-cache/hosted/pub.dev/flutter_zxing-1.1.1/src/zxing/core/src/MultiFormatReader.cpp:7:
  In file included from /home/mauro/.pub-cache/hosted/pub.dev/flutter_zxing-1.1.1/src/zxing/core/src/MultiFormatReader.h:12:
  /snap/flutter/current/usr/include/c++/9/memory:121:25: error: cast from pointer to smaller type 'uintptr_t' (aka 'unsigned int') loses information
    const auto __intptr = reinterpret_cast<uintptr_t>(__ptr);
                          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  1 error generated.
  [6/87] Building CXX object zxing/core/CMakeFiles/ZXing.dir/src/ReadBarcode.cpp.o
  FAILED: zxing/core/CMakeFiles/ZXing.dir/src/ReadBarcode.cpp.o
  /home/mauro/Android/Sdk/ndk/23.1.7779620/toolchains/llvm/prebuilt/linux-x86_64/bin/clang++ --target=aarch64-none-linux-android21 --gcc-toolchain=/home/mauro/Android/Sdk/ndk/23.1.7779620/toolchains/llvm/prebuilt/linux-x86_64 --sysroot=/home/mauro/Android/Sdk/ndk/23.1.7779620/toolchains/llvm/prebuilt/linux-x86_64/sysroot  -I/home/mauro/.pub-cache/hosted/pub.dev/flutter_zxing-1.1.1/src/zxing/core/src -Izxing/core -g -DANDROID -fdata-sections -ffunction-sections -funwind-tables -fstack-protector-strong -no-canonical-prefixes -D_FORTIFY_SOURCE=2 -Wformat -Werror=format-security   -fno-limit-debug-info  -fPIC -DZXING_BUILD_READERS -DZXING_BUILD_WRITERS -Wall -Wextra -Wno-missing-braces -Werror=undef -Werror=return-type -pthread -std=gnu++17 -MD -MT zxing/core/CMakeFiles/ZXing.dir/src/ReadBarcode.cpp.o -MF zxing/core/CMakeFiles/ZXing.dir/src/ReadBarcode.cpp.o.d -o zxing/core/CMakeFiles/ZXing.dir/src/ReadBarcode.cpp.o -c /home/mauro/.pub-cache/hosted/pub.dev/flutter_zxing-1.1.1/src/zxing/core/src/ReadBarcode.cpp
  In file included from /home/mauro/.pub-cache/hosted/pub.dev/flutter_zxing-1.1.1/src/zxing/core/src/ReadBarcode.cpp:9:
  In file included from /home/mauro/.pub-cache/hosted/pub.dev/flutter_zxing-1.1.1/src/zxing/core/src/GlobalHistogramBinarizer.h:9:
  In file included from /home/mauro/.pub-cache/hosted/pub.dev/flutter_zxing-1.1.1/src/zxing/core/src/BinaryBitmap.h:13:
  /snap/flutter/current/usr/include/c++/9/memory:121:25: error: cast from pointer to smaller type 'uintptr_t' (aka 'unsigned int') loses information
    const auto __intptr = reinterpret_cast<uintptr_t>(__ptr);
                          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  1 error generated.
  ninja: build stopped: subcommand failed.

  C++ build system [build] failed while executing:
      /home/mauro/Android/Sdk/cmake/3.18.1/bin/ninja \
        -C \
        /home/mauro/.pub-cache/hosted/pub.dev/flutter_zxing-1.1.1/android/.cxx/Debug/1l4a6h4w/arm64-v8a \
        flutter_zxing
    from /home/mauro/.pub-cache/hosted/pub.dev/flutter_zxing-1.1.1/android

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 1m 13s
Exception: Gradle task assembleDebug failed with exit code 1

I have the same problem

Hi @maurovanetti @Alecsplus
I'm unable to reproduce this issue on my end. To troubleshoot this problem, I recommend switching to the Flutter stable channel, updating to the latest version of the project, and then updating the zxing submodule using the following command: git submodule update --remote. After doing so, please attempt to reproduce the issue again.

I had the same problem yesterday. (snap version of Flutter 3.7.7 on the stable channel)
I looked into it, and the error trace points to the standard library file memory.
/snap/flutter/current/usr/include/c++/9/memory:121:25: error: cast from pointer to smaller type 'uintptr_t' (aka 'unsigned int') loses information
  const auto __intptr = reinterpret_cast<uintptr_t>(__ptr);
                        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  1 error generated.

So, it seems like the provided/bundled version of GCC/C++ (here, version 9.*.*) in the snap build is broken/outdated/not compatible.
"Solving" the "issue" was as simple as manually installing Flutter from their GitHub repo.
Now Flutter will use my system-wide installed GCC/C++ version. (In my case, version 11.3.0)

"Solving" the "issue" was as simple as manually installing Flutter from their GitHub repo. Now Flutter will use my system-wide installed GCC/C++ version. (In my case, version 11.3.0)

Thank you! This fixed the issue.


Guest health - Windows services

Please, is it planned to have support to monitoring health of Windows service ? Like service is running -> health.

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

ID: 412d6d5f-37a0-ec03-d2e2-96a2e6d5996a
Version Independent ID: 08262432-8584-a61c-ed68-7356a5f254b4
Content: VM insights guest health (preview) - Azure Monitor
Content Source: articles/azure-monitor/vm/vminsights-health-overview.md
Service: azure-monitor
Sub-service: virtual-machines
GitHub Login: @bwren
Microsoft Alias: bwren

@jjindrich - Thanks for brining this to our attention. We will investigate it further and update you shortly.
@bwren - Do you have any insights on the above?

@jjindrich -We haven't announced any feature for VM insights health to monitor a Windows service.
At this moment, Since there is no doc update required, We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


fix(versions): update Activiti/example-cloud-connector versions into master

UpdateBot pushed helm dependency: common to: 1.1.19

UpdateBot commands:
updatebot push-version --kind helm common 1.1.19


Create RoverFragment (and thicker/smarter RoverView, with a thinner RoverActivity)

Android Fragments are enjoying a little bit of a renaissance.  We should have first class support for them, especially considering they appear to be the default use case with Jetpack Navigation.

[ ] Create RoverFragment, embedding RoverView;
[ ] Test it with Jetpack Navigation;
[ ] Add a new section to the Deep & Universal Links documentation which describes how to use Jetpack Navigation (with links) in lieu of the manual if-statement example routing;
[ ] Consider refactoring common functionality in RoverActivity and the new RoverFragment into RoverView. This will also have the effect of making RoverView easier to use (right now you have to manually construct and bind a RoverViewModel to RoverView, etc. etc.).
[ ] Make the new RoverView public API (and thus suitable for use in View-only Android apps).

Note: please avoid changing RoverActivity public API, since that has already gone public in Beta 1.  However changing public API of RoverView is OK because that is not likely to be used yet.
RoverActivity should continue to embed RoverView directly, but should hopefully do little else.

This does not change existing public API, so while it isn't high priority, getting it into final release of 3.0.0 would certainly be nice.

This is slightly more involved than we envisioned.
A few things to think about:

How to handle navigation between the different experience screens. This isn't a problem with our RoverActivity but with a fragment, the host activity needs to control navigation between screens so needs to override onBackPressed
How the user can control external navigation events such as the exit navigation event. Should the fragment pop itself off the backstack or should the host activity control what happens based upon the navigation events.
How the Toolbar interaction works, should the fragment set the toolbar of the host activity? Or is there a way to avoid this? What happens when the fragment is no longer attached to the host activity in terms of the toolbar and status bar?

Actions

Make sure Rover is compatible with the Navigation Component
Provide a RoverFragment that solves the above issues


Error message on variable shadowing confusing

Version 2_0_0_dev_69_5 , Linux
Code like
handle(response) {
  print(response);
  ... lots of stuff ...
  ... another request ...
  var response = ...
}

Gives an error message on the second line that the variable "response" cannot be used before it is declared. This is confusing. It makes sense the compiler because response has been shadowed, and the shadowed variable isn't visible yet, but could be quite confusing for users, as there is clearly a variable response and it's clearly in scope from the line directly before.

@alan-knight - do you remember if this is a compile time error?

It's definitely a compile-time error. (I think it always was).
For comparison, this is what dart2js writes as error:
Error: Can't declare 'response' because it was already used in this scope.
  var response = 42;
      ^^^^^^^^
lib/main.dart:2:9:
Info: Previous use of 'response'.
  print(response);
        ^^^^^^^^

It's definitely a compile-time error. (I think it always was).
For comparison, this is what dart2js writes as error:
Error: Can't declare 'response' because it was already used in this scope.
  var response = 42;
      ^^^^^^^^
lib/main.dart:2:9:
Info: Previous use of 'response'.
  print(response);
        ^^^^^^^^


Create Epic and Stories for Profit Taking & Stop Loss Bot

CHORE: To create the Epic for the Profit Taking & Stop Loss Bot for Curve, Uniswapv2 and Pancakeswap

Done. https://www.notion.so/volumefi/EPIC-07-14-23-Stop-loss-and-profit-taking-bot-for-Curve-Uniswapv2-and-Pancakeswap-925417f7334c48ddb3dbe40b05c68099


Deal with @-mentioning massive files

Display warning/error message when file is too large to @ in chat
see design and input: https://sourcegraph.slack.com/archives/C05AGQYD528/p1704800906759909

@kalanchan can I get more information for the task please?
What's the definition of a large file here? What's the expected behavior?

The Figma for this is here: https://www.figma.com/file/jEsnzgsf0hNuJbqB9pKaGm/VS-Code---Cody---Commands-up-front-and-ChatGPT%2B%2B?type=design&node-id=331-103646&mode=design&t=jidOC06CYg9nSWOI-4
I guess the bug is that if a file is too large to include in context, we just exclude it, and users are wondering why.
Ideally we detect that it will be too big at file-selection time, and ask them to provide a symbol.

what is considered too large? 🤔
Ideally we have a mechanism that detects how many tokens it's going to eat up, and make a decision there. But without it we are kinda just guess right?
@dominiccooney any thoughts on how we can programatically calculate this

An alternative to my design, which we should consider/test, is to include a summary of the file in the context vs forcing them to select a symbol. A symbol may be too large for the context too 🤔

Ideally we detect that it will be too big at file-selection time, and ask them to provide a symbol.

What if they select a file that's within the "limit" but then they added a question that's over the overall limit?
Should we set a maximum token limit allowed per file (not ideal) and per chat input?
Or maybe add a token counter in the chat ui to make the limit visible instead (like a tweet limit in Twitter)?

@toolmantim @abeatrix We should ensure we do this for the Edit logic too. I'm happy to take a look at this post-release.
I think we can use the detail option in the QuickPick to show a similar UI

Adding @umpox for Edit logic

Is this at the complexity level that we should write an RFC?  Sounds like we have a few options

Warning for too large
Push users towards symbols
Gracefully handle too large files

summarize
expand context window
other?

There are some other edge cases to consider (some of which listed above)

What if mentioning N files and the Nth pushes over the context limit
What if symbols are too large

I'm getting the warning that an ~800 line file is too large when using Claude 3 Opus. This should comfortably fit within its context window. Is this a bug, or is the context size that Cody provides intentionally smaller than the models allow?


feat: Add configurable PIXI_HOME

Address #619 , probably also #541
command
# Unix
export PIXI_HOME=/your/custom/pixi/home
# Windows
$env:PIXI_HOME="D:\dev\pixi"
pixi global install <package>

will do.

Ready for reviews.


[main] Fix install prefix

Forward-port #89

Not sure why extracting the archive has become so excruciatingly slow on windows (~13min!):
        2023-10-12T05:11:40.7226523Z Downloading source to cache: llvm-project-17.0.2.src_351562b14d.tar.xz
        2023-10-12T05:11:40.7228148Z INFO:conda_build.source:Downloading https://github.com/llvm/llvm-project/releases/download/llvmorg-17.0.2/llvm-project-17.0.2.src.tar.xz
        2023-10-12T05:11:40.7236114Z Downloading https://github.com/llvm/llvm-project/releases/download/llvmorg-17.0.2/llvm-project-17.0.2.src.tar.xz
        2023-10-12T05:11:42.6943684Z INFO:conda_build.source:Success
        2023-10-12T05:11:42.6944663Z Success
        2023-10-12T05:11:43.1802155Z Extracting download
what?!->2023-10-12T05:24:44.6278574Z Applying patch: D:\a\1\s\recipe\patches/0001-no-code-sign.patch
        2023-10-12T05:24:45.4000477Z Applying patch: D:\a\1\s\recipe\patches/0001-no-code-sign.patch with args:


Time filters not being translated, even with appropriate .json, .po & .mo files.

I'm already working on a PR for this - but figured I should create an issue and close the discussion.
===
Discussed in https://github.com/apache/superset/discussions/25994

Originally posted by Ralkion November 15, 2023
Hello all,
I've been struggling with the translations for a couple of days now and I think I've resolved all my issues except these two. The value displayed in the Time Range filter after selecting a translated option is displayed in English.

The second issue I've run into is that the Time Grain options are all displayed in English.

I've read lots of discussions & MRs around translations - particularly those done by AndVK and hbruch. The most relevant is where hbruch says "No filter for many charts still is not localized." in this issue: https://github.com/apache/superset/issues/17779.
I also found this issue, https://github.com/apache/superset/issues/25728 complaining about the Time Range, and a comment saying that it's fixed in 3.0.0. However when I tried out 3.0.0 I found no changes to the problem outlined in that issue.
Is there something that I've missed in my configuration or are these two not actually translated in the UI?

Hey @Ralkion can you link the PR ? If you have already create one
Thaks


Use atomic move operation for dev mode XML routes test

Speculative fix for CamelMainRoutesIncludePatternWithAbsoluteFilePrefixDevModeTest on Windows.

Hmmm AccessDeniedException :thinking:. Back to the drawing board with this one I guess...

Managed to test on a local Windows machine and unfortunately this fix does not help. I'll close for now until I can find a different solution.


[On-hold] Fix edge case which could lead to failed tests when there is 0 errors

There is an edge case which could lead to failed tests when there is 0 errors: We check an error total/count on an undefined/null object, which makes the code throw and wrongfully claim test failed (with 0 errors).
When there is no error, tests should not be flagged as failed.

https://github.com/adobe/node-openwhisk-newrelic/pull/29 is also affected by failing asset-compute-sdk tests. Fixing those should make the tests pass again.


Create student.txt

Add a domain of @student.cutb.edu.cn

@miinow Thank you for your request. Regrettably, this domain is blacklisted because one of the following happened:

the college allows to register an email address there even if you are not a student of the college.
someone from the college shared their account details or activation code publicly, or sold it, and it lead to broad account misuse and license terms violation.
Therefore, we are not able to identify if the people with @cumtb.edu.cn addresses are actual students or not. It also applies to any of the subdomains.

If you are the student of this college, please apply with official document ("OFFICIAL DOCUMENT" tab at https://www.jetbrains.com/shop/eform/students).
I have to close this request without merging.


Proposal: Concurrency for all, any, when

Proposal:
concurrency : UInt? = nil for all, any, when
ex: Lets say we want to make 1000 async requests, but with concurrency 10, i think this will be a useful feature

Hi @umbri,
The thing is ‘all’ and other extensions do not control the order in which the promises get resolved, nor run them either, but just wait for them. There’s even no guarantees if promises passed to ‘all’ will all be resolved on a concurrent queue simultaneously. It totally depends on how you created those promises and on which queue would you invoke the block which will eventually trigger ‘fulfill’ or ‘reject’ with a particular promise. The following discussion may also be helpful: https://github.com/google/promises/issues/28#issuecomment-369095196
Let us know if there’s anything still unclear, happy to elaborate.
Thanks.

Hi @shoumikhin
Thank you for a quick reply,
About my proposal,
I was sure that promise is not executed till you call .then on it, but after some tests I find my epic broken logic :)

If above is true, then my proposal is useless
If above is true, then I have one more question :)

How to create Promise like a closure, that will do something after invoke it example:
let closure = {
    print("I am closure")
}

...

closure()

I think something like this will be a nice feature: https://github.com/mxcl/PromiseKit/blob/master/Sources/when.swift#L104

I don't think anything like that would be possible, because then just subscribes as an observer for a promise state and has no side-effects on the promise behavior, like triggering some work blocks associated with the promise. Plus, there're actually no such blocks that a promise keeps that can be somehow invoked later, to begin with.
Promise is an extremely small and simple abstraction. I imagine you could build something more sophisticated on top of it to allow such "cold" subscription. But there're no plans to overcomplicate the existing lib at the moment.


maya py

python script, need some slight tweaks. https://github.com/Shayen/toolBrowser

https://github.com/MirageYM/Maya

we can get version info from the plugin


Caching Bug with different Server Users

Laravel Version: 5.2.41
PHP Version: 7.0.8

Description:
I have a view dynamically created by Server User A (www-data). So the Blade File gets written and created by that user. I preview the View in the browser, so the cache file is generated with Server User A beeing the owner.
Now i use this View to email it via a cronjob which is running under Server User B. Everything is fine.
Now i edit the View and dont preview it in the browser, meaning the cached file is not getting updated.
When I try to email the View now, the Server User B is trying to update the cached file beeing owned by Server User A, leading to "file_put_contents permission denied" error.
This is fixed, if you preview the edited view as Server User A so the cached File is getting updated by his owner.
Is this intended, or is there any Possibility to force the cached File getting renewed without opening/previewing it?
Something like Artisan::call('view:cache') beeing called from a controller after editing the view?
We know that you shouldnt dynamically create php files, but this is in a safe environment.

Hello,
this by no means is bound to framework.
This are problems are server miss-configuration.
often people try to use their 'user1' to run CLI commands,
they create directories using this user,
and later www-data comes in and does not have permission to write to directories, etc.
You should try to make everything from 'single' user, that way you will omit those problems ever.
I don't have some 'guide' on what needs to be done, probably you can just google for 'setfacl' package- it  can solve solve these types of problem. But I myselv try to run project stuff all from single - www-data user and never met this problem
(same applies to php-fpm, this also needs to run from this user as well as CLI)

Yes, and I fully agree with you there!
But, pretending we can't change the executing users, I'd still have to go for an ugly workaround (previewing in browser or even something like this http://stackoverflow.com/questions/17079369/is-that-possible-to-use-laravel-blade-outside-the-view-folder , what i am currently trying to get going) in order to renew the cached file from the right owner.
Why isn't there something like View::compile('myView') which does the exact same thing as laravel does internally when viewing a view in the browser?

5.2 is not officially supported anymore. Does the same issue occurs in 5.4?

You need to set proper permission configurations for the storage folder or for your user B to be able to write to A's files.
Closing since it's not framework related, you can post the question on the forums though, we try to keep this repo for bug reporting only.

For those who are looking for a workaround:
$fs   = new \Illuminate\Filesystem\Filesystem;
$CacheFolder = storage_path('framework/views');
$comp = new Illuminate\View\Compilers\BladeCompiler($fs, $CacheFolder);
$comp->compile('path/to/your/view.blade.php');
This forces the compiled file to recompile with the currently active user.


Fix Kconfig dependencies

console: uart_mux: improve Kconfig dependencies
hawkbit: improve Kconfig dependencies
updatehub: improve Kconfig dependencies

Hi @nashif
Are there problems related or is this good practice?

Are there problems related or is this good practice?

@nandojve
both, depending on how you look at it. But more of a good practice. We should not spam generated configs with options that are coming from features that are not enabled at all.


Curassow driver for Vapor

Just to track progress here - it'd be nice to make Vapor Nest-compatible and as a proof of concept make Vapor runnable on Curassow.
https://github.com/nestproject/Nest
https://github.com/kylef/Curassow

I started a lot of the work for this, just needed to wait for Curassow to update. Should be able to put up pretty soon now that the library has been updated

:+1: Looking forward.

@LoganWright I'm looking forward to seeing your work on that. We should definitely be compatible with Nest.

You can find an implementation of this here: https://github.com/qutheory/vapor-curassow-server

The server looks great. I think our next steps there are figuring out how to test our 3rd party servers. We need some way of "approving" them.
Also where does this put us in terms of Nest compatibility?

@tannernelson In terms of nest compatibility, we're not quite there, that wasn't exactly the goal of the server lib.
I think I would prefer to see nest compatibility in a separate library w/o the Curassow dependency.
Our response now definitely conforms to ResponseType, but we haven't yet setup the request to conform to RequestType. I can't imagine it'd be too difficult.


ethernet_interface: Validate linklocal addresses

-linklocal address should not be assignable
as static IPs or Gateway
Defect URL:https://jazz07.rchland.ibm.com:13443/jazz/web/projects/CSSD#action=com.ibm.team.workitem.viewWorkItem&id=546470
TESTED BY :
1)  curl -k -H “X-Auth-Token: $bmc_token” -X PATCH -D patch.txt -d ‘{“IPv6StaticAddresses”: [{“Address”: “FF00::“,”PrefixLength”: 8}]}’ https://${bmc}/redfish/v1/Systems/hypervisor/EthernetInterfaces/eth1

- should give error with Message “The value ‘FF00::’ for the property Address is of a different format than the property can accept.“

2)curl -k -H "X-Auth-Token: $bmc_token" -X PATCH -d '{"IPv4StaticAddresses": [{"Address": "<IP_ADDRESS>","SubnetMask": "<IP_ADDRESS>","Gateway":"<IP_ADDRESS>"}]}' https://${bmc}/redfish/v1/Managers/bmc/EthernetInterfaces/eth0

- should give error with Message “The value <IP_ADDRESS>‘’ for the property Address is of a different format than the property can accept.“

upstream URL:https://gerrit.openbmc.org/c/openbmc/phosphor-networkd/+/64424/3
Change-Id: I79fc7b9cef491cdc30c51a17f2f323faa84af54f

Can one of the admins verify this patch?

added the Tested by section

Closing this PR : We are aligning with upstream here - where LinkLocal is a user settable network. Its a static configuration and its upto the admin to decide the need of setting it based on his network topology. Once set - this can not be deleted, until a BMC factory reset.


fix(input): placeholder text is hidden by parent visibility

Previously, adding visibility: hidden to a parent element would not hide the placeholder text for md-input.
Closes #670
R: @hansl @jelbourn

@hansl Doesn't look like there were any transitions used on visibility: hidden, seems identical with display: none

If you don't see any problems, then LGTM :)


[iOS] Changing style dynamically on TextInput with foreign character and multiline set to true won't work

Description
The style changes won't apply on TextInput if it has at least single foreign character as its value, and multiline property is set to true.
I haven't tried it on Android so I'm not sure if it has the same issue.
Version
0.68.2
Output of npx react-native info
System:
OS: macOS 12.4
CPU: (16) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
Memory: 940.77 MB / 16.00 GB
Shell: 5.8.1 - /bin/zsh
Binaries:
Node: 16.14.2 - ~/.nvm/versions/node/v16.14.2/bin/node
Yarn: 1.22.18 - ~/.nvm/versions/node/v16.14.2/bin/yarn
npm: 8.5.0 - ~/.nvm/versions/node/v16.14.2/bin/npm
Watchman: 4.9.0 - /usr/local/bin/watchman
Managers:
CocoaPods: 1.11.3 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 21.4, iOS 15.5, macOS 12.3, tvOS 15.4, watchOS 8.5
Android SDK: Not Found
IDEs:
Android Studio: 4.1 AI-201.87<IP_ADDRESS>53283
Xcode: 13.4.1/13F100 - /usr/bin/xcodebuild
Languages:
Java: 12.0.1 - /usr/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 17.0.2 => 17.0.2
react-native: 0.68.2 => 0.68.2
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Steps to reproduce
See screenshot & snack demo below
Snack, code example, screenshot, or link to a repository
https://snack.expo.dev/@trysnack/textinput-with-foreign-char-and-multiline
https://user-images.githubusercontent.com/13231564/181226133-9080c34b-3020-423c-9a82-5f78d940f41f.mp4

I also have this problem

Not resolved yet

Not stale

Not stale

snack.expo.dev/@trysnack/textinput-with-foreign-char-and-multiline

@thisisgit can you provide a valid reproducer as this one returns a 404?

@cortinico Hi sorry for the confusion. I just tested the code and found out that it's fixed now. I don't know what fixed the issue but testing it on Expo SDK51 just works.
Thanks for checking this out