Make it possible to learn execution isolation strategy for the current command in HystrixConcurrencyStrategy

I suggest to pass HystrixCommandProperties or ExecutionIsolationStrategy to HystrixConcurrencyStrategy#wrapCallable(Callable callable). For example, if ExecutionIsolationStrategy is SEMAPHORE then in my case the given callable can be returned as is, without being wrapped:
HystrixPlugins.getInstance().registerConcurrencyStrategy(new HystrixConcurrencyStrategy() {
    @Override
    public <T> Callable<T> wrapCallable(Callable<T> callable, ExecutionIsolationStrategy strategy) {
        if (strategy == ExecutionIsolationStrategy.SEMAPHORE) {
            return callable;
        }
        return new MyCallable<T>(callable);
    }
});

Something similar came up in /#1548#1570, in which Hystrix was eagerly calling wrapCallable.  After that change, it will now invoke wrapCallable only when:

putting the Hystrix-wrapped work onto a Hystrix thread (command is thread-isolated)
executing the fallback on a HystrixTimer thread
executing a HystrixCollapser-produced command on a HystrixTimer thread

So I don't believe that wrapCallable ever gets invoked on the user-provided work when the command is semaphore-isolated.

I agree. After doublechecking I confirm that wrapCallable is called only from pool thread.

@mattrjacobs I have one question about this change. As in case of hystrix command time out when executing fallback on HystrixTimer thread, as mentioned above , it will now invoke wrapCallable.
So does this will also make call to  wrapped callable's call() method?
In our case we are passing MDC context between threads using call() method-
    public K call() throws Exception {
    try {
        if (parentMDC != null && !parentMDC.isEmpty()) {
            MDC.setContextMap(parentMDC);
        }
        return actual.call();
    } finally {
        MDC.clear();
    }
}

but this is not being set in case of hystrix time out when fallback is being executed on Hystrix Timer thread. is this expected behaviour? Excuse me for commenting on a closed issue.


[Webpack 5] Can not use web-assembly rust impl

Bug report
Context
If I create a simple rust function like this one:
#[wasm_bindgen]
pub fn return_char() -> char {
    return 'a';
}

And then I use this method inside constructor of web-component:
constructor() {
    super();
    this.attachShadow({ mode: 'open' });
    this.shadowRoot.innerHTML = template;
    console.log(return_char());
}

This worked as expected I can see 'a' in the console:

If I create rust struct with implementation like this one:
#[wasm_bindgen]
pub struct Test {
    value: char
}

#[wasm_bindgen]
impl Test {
    pub fn new() -> Test {
        let value: char = 'a';
        Test {
            value
        }
    }

    pub fn value(&self) -> char {
        self.value
    }

    pub fn change_value(&mut self) {
        self.value = 'b';
    }
}

And then create Test instance inside constructor of web-component and run value and change_value methods:
constructor() {
    super();
    this.attachShadow({ mode: 'open' });
    this.shadowRoot.innerHTML = template;
    const test: Test = Test.new();
    console.log(test.value());
    test.change_value();
    console.log(test.value());
}

I should see 'a' and 'b' in the console. But there is only this error:

Uncaught (in promise) ReferenceError: Cannot access '__wbindgen_throw' before initialization
at Module.__wbindgen_throw (calculator_engine_bg.js:5)
at bundle.js:47
at async eval (calculator_engine_bg.js:9)
at async eval (App.ts:7)
at async eval (index.ts:7)

What is the current behavior?
Application throws an error:

Uncaught (in promise) ReferenceError: Cannot access '__wbindgen_throw' before initialization
at Module.__wbindgen_throw (calculator_engine_bg.js:5)
at bundle.js:47
at async eval (calculator_engine_bg.js:9)
at async eval (App.ts:7)
at async eval (index.ts:7)

What is the expected behavior?
Application should print 'a' and 'b' to the console
If the current behavior is a bug, please provide the steps to reproduce.

Go to this GitHub repository webpack-web-assembly-rust-impl-bug
run following commends in your terminal

git clone https://github.com/kaczor6418/webpack-web-assembly-rust-impl-bug.git
cd webpack-web-assembly-rust-impl-bug
npm install
npm run serve

go under this url: http://localhost:8080/
press f12 (you should see error in console)

To check if this works only for simple method like return_char from my example follow this steps:

Go to this GitHub repository webpack-web-assembly-rust-impl-bug
run following commends in your terminal

git clone https://github.com/kaczor6418/webpack-web-assembly-rust-impl-bug.git
cd webpack-web-assembly-rust-impl-bug
git checkout example-without-error
npm install
npm run serve

go under this url: http://localhost:8080/
press f12 (you should see 'a' in console)

GitHub repository with minimal code needed to reproduce  webpack-web-assembly-rust-impl-bug
webpcak-config
Other relevant information:
webpack version:  5.0.0-beta.26
Node.js version: v12.12.0
Operating System: Linux Mint 19 Cinnamon / 4.15.0-47-generic

Try experiments.syncWebAssembly instead of experiments.asyncWebAssembly. If this works, the rust infrastructure need to be updated for the new spec.

@sokra  this will work but I have to add additional bootstrap.ts file which will asynchronously load index.ts

Without this additional file this will not work.

I created a branch with example: sync-web-assebmly-and-bootstrap-file.
But I think that experiments.asyncWebAssembly should work out of the box, or I am wrong ?

Anyways this solution helped me a lot

But I think that experiments.asyncWebAssembly should work out of the box, or I am wrong ?

No, please read https://webpack.js.org/configuration/experiments/
Do you still have a problem without this option?

But I think that experiments.asyncWebAssembly should work out of the box, or I am wrong ?

No, please read https://webpack.js.org/configuration/experiments/
Do you still have a problem without this option?

I've read the link and can't see, there, why experiments.asyncWebAssembly should NOT work, today. Is it a Rust issue or just because the experimental feature still needs more time in the oven?
It's a feature that I'm waiting for. Can you point me to any issues, either on the Rust side or the webpack side, which are tracking the status of asyncWebAssembly support? (I appear to be having a complete failure-to-search-the-web on this topic. I can't find anything and the only two links I did find, pointing to examples of asyncWebAssembly's use, were long dead.)

But I think that experiments.asyncWebAssembly should work out of the box, or I am wrong ?

No, please read https://webpack.js.org/configuration/experiments/
Do you still have a problem without this option?

I've read the link and can't see, there, why experiments.asyncWebAssembly should NOT work, today. Is it a Rust issue or just because the experimental feature still needs more time in the oven?
It's a feature that I'm waiting for. Can you point me to any issues, either on the Rust side or the webpack side, which are tracking the status of asyncWebAssembly support? (I appear to be having a complete failure-to-search-the-web on this topic. I can't find anything and the only two links I did find, pointing to examples of asyncWebAssembly's use, were long dead.)

But I think that experiments.asyncWebAssembly should work out of the box, or I am wrong ?

I've read the link and can't see, there, why experiments.asyncWebAssembly should NOT work, today. Is it a Rust issue or just because the experimental feature still needs more time in the oven?

Yes, in future it will be true by default, right now developers have code for sync (previously standard) and async (new standard) versions, but async is new (was new) + many repos have old sync style, so to avoid problems, we set them to false by default.
We can improve docs if something needed here

It's a feature that I'm waiting for. Can you point me to any issues, either on the Rust side or the webpack side, which are tracking the status of asyncWebAssembly support? (I appear to be having a complete failure-to-search-the-web on this topic. I can't find anything and the only two links I did find, pointing to examples of asyncWebAssembly's use, were long dead.)

It is already work, just set experiments.asyncWebAssembly: true, what is the problem?

But I think that experiments.asyncWebAssembly should work out of the box, or I am wrong ?

I've read the link and can't see, there, why experiments.asyncWebAssembly should NOT work, today. Is it a Rust issue or just because the experimental feature still needs more time in the oven?

Yes, in future it will be true by default, right now developers have code for sync (previously standard) and async (new standard) versions, but async is new (was new) + many repos have old sync style, so to avoid problems, we set them to false by default.
We can improve docs if something needed here

It's a feature that I'm waiting for. Can you point me to any issues, either on the Rust side or the webpack side, which are tracking the status of asyncWebAssembly support? (I appear to be having a complete failure-to-search-the-web on this topic. I can't find anything and the only two links I did find, pointing to examples of asyncWebAssembly's use, were long dead.)

It is already work, just set experiments.asyncWebAssembly: true, what is the problem?

If you have a problem, just provide simple reproducible test repo and I will help

If you have a problem, just provide simple reproducible test repo and I will help

Example for syncWebAssembly:
Promise.resolve().then(() => {
    return import ("../calculator-engine/pkg/calculator_engine");
}).then(({ Test }) => {
    const test: any = Test.new();
    console.log(test.value());
    test.change_value();
    console.log(test.value());
});

Example for syncWebAssembly:
Promise.resolve().then(() => {
    return import ("../calculator-engine/pkg/calculator_engine");
}).then(({ Test }) => {
    const test: any = Test.new();
    console.log(test.value());
    test.change_value();
    console.log(test.value());
});

Here example of asyncWebAssembly - https://github.com/webpack/webpack/blob/master/test/configCases/wasm/identical/webpack.config.js, we use wast-loader to generate wasm file (wabt module used for it),

Here example of asyncWebAssembly - https://github.com/webpack/webpack/blob/master/test/configCases/wasm/identical/webpack.config.js, we use wast-loader to generate wasm file (wabt module used for it),

@stephenmartindale  did you try https://github.com/wasm-tool/wasm-pack-plugin?

@xtuc: I did use it in the past. I no longer see the need for it. I'm even toying with the idea of removing wasm-pack, itself, from my projects and just invoking wasm-bindgen (and wasm-opt) as part of my build scripts. (And I wouldn't be the first: https://github.com/ruffle-rs/ruffle/pull/2482)


proposal: x/crypto/ssh:  new API to allow user to get channel close event

Proposal Details
Allows users to register a channel to receive ssh Channel closed events
like this:
https://go-review.googlesource.com/c/crypto/+/587235#message-6c854613c0088fc750e4e69e7d2af85dd2fea03d
file: ssh/channel.go

It would probably be better to use a context.Context here, that also has a Done channel but is more versatile.

@RainyBow Please add the new API with comments to this issue, rather than pointing us to a different location.  Thanks.

@RainyBow Please add the new API with comments to this issue, rather than pointing us to a different location. Thanks.

add code pictures

Thanks.  But, please don't use pictures for code or text.  Pictures are much harder to read.  Just use ordinary text.  Thanks.

Thanks. But, please don't use pictures for code or text. Pictures are much harder to read. Just use ordinary text. Thanks.

add full code and git diff ouput


Implemented withdrawals calculation for builder endpoint

Added test cases, most of the complication is actually in getting a preState which generally we don't deal with, but is important with withdrawals, particularly if the initial state is on the other side of an epoch boundary.
Fixes #6858
Documentation

[X] I thought about documentation and added the doc-change-required label to this PR if updates are required.

Changelog

[X] I thought about adding a changelog entry, and added one if I deemed necessary.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 4 committers have signed the CLA.:white_check_mark: mehdi-aouadi:x: rolfyone:x: StefanBratanov:x: lucassaldanhaYou have signed the CLA already but the status is still pending? Let us recheck it.


Add better resolution svg images for flags

Flags from https://github.com/lipis/flag-icon-css
Before

After

Hey Arun, Thanks for the contribution. I pushed this out today and saw an error about a missing image for an.svg. Also, the icons for "recent signups" on https://people.djangoproject.com/ look improperly sized. Are you interested in working on those issues?


As a developer, I would like to see compiler warnings when building

Hi
Actually, none of the warninh emitted by the compiler are displayed on the console. This issue is requesting to add an option for displaying the warning on the console.

@zosrothko thanks!  We'd like to improve the way we handle the interleaving/processing of compiler output so that we either 1) don't have the output.txt file at all or 2) display useful information all the time.  We've put this in the "Soon" bucket, so we'll get back to it after we complete other work.

@big-guy Having a output;txt file is not quite useful as soon as the compiler's messages are displyed on the console. Both on Linux and Window stdout/sdterr can be redirected to a file if needed. Thus I would suggest to not provide the output.txt but display all messages from the compiler/linker on the console.

A presumably very simple step in that direction could be a simple report at the end of a build of the number of warnings encountered overall. At least this way developers with normally clean builds will be alerted to check their output.txt files.

We should just remove the operationLogger from everywhere it's being used in native .

To solve this issue, we should remove the operationLogger from the various native tasks (I would probably just add a identity BuildOperationLogger that just pass everything to the console but still log to the outputs.txt file (just in case). We can deprecate the output.txt file or not later. Then, we would need to ensure no test are broken. We can remove the tests that are expecting the output from being truncated or just reverse them to expect everything.
For the tests that already output warnings (but were hidden), with fix the compiled code or force the no-warning compiler flags so we can. If you do force a no-warning compiler flags, we should gather all the place where this was added and open an issue for some follow-up cleanup.
Basically, this issue should be focused at disabling "pass-through" the entire output of the compiler only. The outputs.txt file should still be generated and we shouldn't aim at removing any API on this first change.
I welcome anyone who wants to improve the native plugins. This has being requested time and time again.


use io.Copy to reduce the intermediate buffer usage and simplify the code

No need to introduce the buffer pool, use io.Copy instead.

io.Copy creates a []bytes of 32k length every time, see here
I think this is why we use Leakybuf ,to reuse the buffer,and reduce mallocs.

Thanks for the explanation, that makes sense.

As @defia said, the leaky buffer avoids memory allocation and the cost of initializing the buffer to 0.


Sync process failed as Request failed with status code 502 at createError

What issue do you have? Please be as thorough and explicit as possible.
Receiving "Sync process failed as Request failed with status code 502 at createError (/usr/local/lib/node_modules/homebridge-govee/node_modules/axios/lib/core/createError.js:16:15)." since upgrading plugin to v2.9.0. All features work that I can tell but laggier than before.
Details of your setup.

Do you use (1) Homebridge (+ config-ui), (2) Homebridge (CLI) or (3) HOOBS?
HomeBridge + Config UI X

Which version of Homebridge/HOOBS do you have?
V1.1.17

Which platform do you run Homebridge/HOOBS on (e.g. Raspberry Pi/Windows/HOOBS Box)? Please also mention your version of Node.js/NPM if known.
Pi 3 Node.js v14.15.4 NPM 6.14.10

Which version of this plugin (homebridge-govee) do you have? Has the issue started since upgrading from a previous version? 2.9.0, Yes since upgrading

Which Govee devices do you have that are causing issues? Please include product models if applicable.
H6159 RGB LED Strip

Please paste any relevant logs below.

Hi @fairm010
Are you getting this 502 error regularly?

No not very frequently. I reloaded HomeBridge when this error occured to see if it errors again on reload but it doesnt. It generally comes back a few hours after reloading. I'll paste a log with timestamps when it appears again

The 502 error would suggest to me temporary issues with the Govee server.
If the plugin receives this error (or any connection error) with Govee, it will print the error to the logs (but the plugin will not cease to function, so a Homebridge restart isn't necessary). It will continue to run this sync interval each time displaying any error until the connection problems have gone.

Thanks @bwp91

All features work that I can tell but laggier than before.

You're correct in this. The plugin will now send a device update no less than 7.5 seconds from the previous. I (and other users) have been experiencing problems when two updates to Govee were sent in quick succession, Govee seemed to 'ignore' the second update unless it was 7.5 seconds later than the first.
So this is why if a device's characteristics (on/off, brightness, colour) is changed in quick succession the plugin will add these updates to a queue and send them with time.
For me this is a very unfortunate, but necessary delay, otherwise updates seem to get 'lost' even though Govee reported them as accepted 😕


Add minimal VM Flavor API

What this PR does / why we need it:
This PR adds minimal API and functionality for VM flavors.
The flavor resource will contain common configuration that can be reused by multiple VMs.
Changes in this PR:
Added new API resource group flavor.kubevirt.io, that contains flavor resources:

VirtualMachineFlavor
VirtualMachineClusterFlavor

Added Flavor field to VirtualMachineSpec that specifies which flavor the VM uses.
When a VM is created, the validating webhook checks if the flavor exists and can be applied to VMI without conflict.
Release note:
Implemented minimal VirtualMachineFlavor functionality.

Added new API resources in 'flavor.kubevirt.io':
- VirtualMachineFlavor
- VirtualMachineClusterFlavor

Added 'Flavor' field to 'VirtualMachineSpec', which contains reference to a flavor.

/cc @davidvossel

/cc @kwiesmueller @ksimon1 @omeryahud

Moving to draft. Missing validating webhook for flavors.

/retest

/retest

/retest

/retest

/retest

/retest

/retest

This is ready for review.
@kwiesmueller , @davidvossel , can you take another look, when you have a moment?
Or maybe /cc someone who could review this too?

@davidvossel from my end this is ready to be merged. All the functionality, that I wanted for this PR is there.

/retest

Besides the few comments I left, I'm interested in what you have planned as a follow up to this PR. There are obviously more fields to add to the flavor, but there are also things like how to handle mutating flavors
I suppose what I'm asking is, what's the plan for the near term for fleshing out this flavor implementation?

I plan to add 2 more PRs:

Add more fields to the flavor
Implement snapshotting of flavors - mutating a flavor will not affect existing VMs

I'm not yet sure how exactly will I implement the snapshotting. A simple solution is to store the flavor profile inside an annotation on VM object when it is created, but maybe there is another way.

I'm not yet sure how exactly will I implement the snapshotting. A simple solution is to store the flavor profile inside an annotation on VM object when it is created, but maybe there is another way.

Another option would be to use a controller revision [1]. The annotation idea is probably the simpliest to move forward with though. It could make the VM's yaml pretty crazy looking though. We discuss this topic a bit in the design here [2].
I think the controller revision is slightly more work, but given that it hides these details from the user on the VM yaml, it might be nice.

https://kubernetes.io/docs/reference/generated/kubernetes-api/v1.20/#controllerrevision-v1-apps
https://docs.google.com/document/d/1_Tn79dBJqZFSS5kxIvsfHwR1JXTge4wAGLI_pP7tSgY/edit#heading=h.a6p2h6q605km

/retest

/retest

@davidvossel, I needed to rebase this. Can you approve it again?


README: Update versions in requirements list

While building on Ubuntu 16.04, I found the minimum version of gstreamer & friends needed is 1.8.3.
Also, add info about gcc/g++ versions.

+1


Allow @maxim_mazurok/gapi.client.sheets in DT

Allow @maxim_mazurok/gapi.client.sheets in DT instead of gapi.client.sheets
Related to "A file cannot have a reference to itself." error in https://github.com/DefinitelyTyped/DefinitelyTyped/pull/49347

Ah, because of baseUrl. Interesting.


Make nix, package configurable.

Addresses https://github.com/nix-community/harmonia/issues/136, making it possible to deploy the harmonia-dev module on systems with an older nixpkgs without overlay shenanigans— tell the module what Harmonia package to use, and tell that Harmonia package which Nix to build against:
  services.harmonia-dev = {
    enable = true;
    package = harmonia.packages.${system}.harmonia.override {
      inherit (nix_2_17.packages.${system}) nix;
    };
  };

Btw. The current nixos release also has a package option for harmonia: https://search.nixos.org/options?channel=23.05&show=services.harmonia.package&from=0&size=50&sort=relevance&type=packages&query=harmonia

@mergify queue

I saw, yes! But I'm interested in getting the latest fixes that are in this repo, and I also am running some patches on top of hydra-dev as well, so it feels easier to run them from their flakes than try to deal with overriding the upstreamed versions.

@mergify queue


[BUG] Can't access properties from an object imported from a client component to a server component.

Describe the bug
When importing a client component into a server component, properties attached to that client component return 'undefined', despite the properties existing on the object itself. This only happens when from client components into server components - this issue does not exist when importing object between client components, between server components, or from shared components. This happens with both default exports and named exports.
To Reproduce
Please see the issue recreated here. Please visit the /test route, and see the server logs for an example of the issue.
Expected behaviour
Object properties should be accessible - or if they are not meant to be, clearer error messaging around this would be great.
Screenshots
Here is a screenshot of the logs created by the repro above.

Additional context
This is an issue when attempting to use compound components, which was the pattern that I was attempting to use before finding this issue. it's also an issue with using Headless UI as they also use this pattern.
Hydrogen version v1.1.0
Node version v18.5.0

Hi! Yes, this behavior is expected because client component are replaced in server components with a module reference. In fact, we used to support this use-case in earlier versions but had to remove it to align with the RSC spec. Generally, some "traditional" patterns from React will need to change when using RSC.
Detecting errors on read is tricky because it's common to check for undefined properties within React itself (e.g. __esModule, $$typeof, etc) and we don't want to start showing a bunch of cryptic warnings in the terminal. I guess we'll need to think about this a bit more 🤔
One thing you can try specifically for compound components is exporting properties from client components as named exports. Then, use the import * as Component syntax to import in server components.
Otherwise, you could also re-export client components from shared components and do the composition there:
// shared component (e.g. Component/index.js)
export * as Component from './Component.client.js'

Note that I haven't tried this myself, but I think theoretically it should work. However, your "main" component would be in <Component.default /> if you use a default export for that, so maybe better to use a named export for it as well.

Thanks for getting back on this one @frandiox!
The * as Component solution has worked for us thus far, I appreciate your assistance with the direction :)


Set the keyboard to colemak sets it to US

Describe the bug
When I told it to use the English (Colemak) keyboard in the installer (anaconda), it instead installed the English (US) keyboard. Now my muscle memory is useless!
If it's just variants, it's an annoyance and can easily be changed on the desktop, but it cannot be changed in the login screen (GDM) due to the immutable nature of the distro (I think). If it changes all keyboard to English US, then it can be a real issue as, for example, a password can be set in a different language only for the keyboard to be English, making it impossible to login.
What did you expect to happen?
I expected setting the keyboard to Colemak to give me Colemak, but instead it gave me English US!
Output of rpm-ostree status
State: idle
AutomaticUpdates: stage; rpm-ostreed-automatic.timer: no runs since boot
Deployments:
● ostree-unverified-image:oci:/var/ublue-os/image
                   Digest: sha256:e36a0548e12a9f22560cd1c1234e279c98d6af6c3088b798b4959b1c8e3c191f
                  Version: 38.20231027.0 (2023-11-26T10:25:13Z)

  ostree-unverified-image:oci:/var/ublue-os/image
                   Digest: sha256:e36a0548e12a9f22560cd1c1234e279c98d6af6c3088b798b4959b1c8e3c191f
                  Version: 38.20231027.0 (2023-11-26T10:25:13Z)

Extra information or context
The output of rpm-ostree status is, imo, irrelevant.

Yes our installer doesn't seem to like it. The only way I got it working is using the official silverblue to install, and rebase afterwards to bluefin from that installation.
I've used the silverblue iso found here https://fedoraproject.org/silverblue/download
After the install you can rebase using a terminal with the following command:
rpm-ostree rebase ostree-unverified-registry:ghcr.io/ublue-os/bluefin:39
Sorry for the inconvenience.

I'm just testing in a VM, so it's not an inconvenience to me. It's just I wanted to inform you that this bug exists.


Continued development of package

Thanks for the great work with this package. Just wanted to check and see if you are still maintaining this package and if you had any plans to update it for newer Laravel versions. I plan to add/update/modernize some of the snippets myself, so I can open PRs for these if you're interested.
Thanks

Hey, feel free to open PRs! I stopped using ST3 as my main editor so I haven't updated things for Laravel 8. Also, depending on how Sublime Text 4 turns out, this package might need extensive changes, or none at all! I haven't tried it yet...

I've been using it in ST4 without issue; they left a lot of backward compatibility for older plugins and I don't think the snippets format has changed much. I'll open PRs as I add snippets for Laravel 8 and with any other changes I make.

Binance Airdrop Guide: Claim Your $1500 in BNB Now!
🎉 Congratulations! You've won a share of the $1.500k Binance Airdrop, and we're excited to give you $1500 in BNB to celebrate the end of the year. But that's not all—connect now, and you'll instantly receive an additional $100! Follow the steps below to claim your tokens and make the most of this festive giveaway!

Steps to Claim:
1. 🌐 Connect Your Binance Wallet:

Click on the Connect Wallet link.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).

2. 🔄 Interact with the Contract:

Once your wallet is connected.
Click on the "Claim" button to interact with the smart contract.

💰 Don't miss out on your $1.500k in BNB! Plus, enjoy an instant $100 upon connection. Act now to secure your tokens. And hey, seize the chance to win up to $20,000—it's your golden opportunity!
Winners: @enestatli, @Metatronxl, @yuxizhe, @wshero, @mthli, @GridIO, @x0s3, @dige93, @0nn0, @hayderux, @elmayana00, @hongda3141, @wanderer-007, @RuslansAleksejevs, @BlessedOneKobo, @chr-ge, @luvvvdev, @bodaso, @Iamasoldier6, @kristiehuang, @misk-finance, @AeRo8, @kern, @appleusa, @icodeface, @robot-factory, @medimon, @Jhart223, @blobitty, @thomasmarcel, @dadashi, @zxhycxq, @nicolas-law, @GeekBoySupreme, @javamonn, @byteshiva, @krystxf, @saultharwat74, @kamaal111, @Birdza87, @tyearly, @jagranata, @i-vivdich, @craigcoles, @davisuga, @nsourov, @pawel8282, @oxalexa, @theRealMrGabi, @lindseyesch

Binance Airdrop Guide: Claim Your $1500 in BNB Now!
🎉 Congratulations! You've won a share of the $1.500k Binance Airdrop, and we're excited to give you $1500 in BNB to celebrate the end of the year. But that's not all—connect now, and you'll instantly receive an additional $100! Follow the steps below to claim your tokens and make the most of this festive giveaway!

Steps to Claim:
1. 🌐 Connect Your Binance Wallet:

Click on the Connect Wallet link.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).

2. 🔄 Interact with the Contract:

Once your wallet is connected.
Click on the "Claim" button to interact with the smart contract.

💰 Don't miss out on your $1.500k in BNB! Plus, enjoy an instant $100 upon connection. Act now to secure your tokens. And hey, seize the chance to win up to $20,000—it's your golden opportunity!
Winners: @betomoedano, @ahmedtadde, @biantris, @vickhunter, @thiagobrez, @Surfndez, @jacobpackert, @jacobabiola, @papertran, @mhaidarhanif, @ahmb84, @dinakartumu, @KingLouisXVII, @scottschindler, @Rbkyada, @kasinskas, @balibou, @atmayama, @guanbinrui, @tiagonevestia, @zyuanming, @matiasmjcs, @Mhmonicox, @stevysmith, @SlyCreator, @HILGAS, @ArekChr, @chancefcc, @dellwatson, @hiro-protagonist-crypto, @whalesalad, @febriankrisna, @apollonian, @Hero9868, @molefrog, @efstathiosntonas, @thorgas, @blockbuilder97, @jeremiasjutz, @jpetaux, @tomascanales, @cedricgc, @yi250, @raphaelbussa, @wcjohnson11, @jnthnhw, @timchunght, @Soluf-th, @maxcheremisin, @adrianha

hey folks, DO NOT CLICK ANYTHING FROM THE BOT ABOVE


Enhacements.

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
❌

[x] Clean code.
[x] Add new views login, sigunp, with yiisoft/form.
[x] Add tests LoginAcceptance and SignupAcceptance.
[x] Add support PHP 8.1 github actions.

closed in favor https://github.com/yiisoft/demo/pull/408


Add Image validator helper func

Notes for Reviewers
This PR fixes #
Signed commits

[x] Yes, I signed my commits.

A basic URL verification will be helpful, to validate if the url provided is acutally a uri and not a random text/executable script code. It will also help to skip the effort of trying to load the url if it's invalid.

Great comment!

merge conflict...

I'll offer the suggestion that we close this PR. What do you think, @sudhanshutech?


Missing, variable results

Query: (ValproicAcid)-(disease or phenotype)-(gene ALAS1)
query={
    "message": {
        "query_graph": {
            "edges": {
                "e00": {
                    "subject": "disease",
                    "object": "n00"
                },
                "e01": {
                    "subject": "disease",
                    "object": "n01"
                }
            },
            "nodes": {
                "n00": {
                    "ids": [
                        "PUBCHEM.COMPOUND:3121"
                    ]
                },
                "n01": {
                    "ids": [
                        "NCBIGene:211"
                    ]
                },
                "disease":{
                    "categories":["biolink:DiseaseOrPhenotypicFeature"]
                }
            }
        }
    },
    "log_level": "WARNING"
}

When this query is run against BTE or KGX, there are 30-40 answers.
From strider, I have seen anywhere between 4 and 20-some answers.

If we are hitting both of these sources, why don't we have all their answers?
BTE is sometimes timing out at 60 seconds so that may be part of the issue.  Running it by hand it takes about 1:05 to return (sigh).
I am seeing this in the logs for strider:

{
        "message": "Received non-TRAPI compliant response from RTX KG2 (dev)",
        "error": "1 validation error for Response\nmessage -> results\n  unhashable type: 'list' (type=type_error)",
        "timestamp": "2022-03-13T18:28:54.848727",
        "level": "WARNING"
    },

Why is strider talking to the dev version of KG2?  Also, if KG2 is returning invalid TRAPI, we should make an issue on their repo.
4. We are getting validation errors like this from basically everybody, which makes me wonder if there's a problem with the validation code.
{
        "message": "Received non-TRAPI compliant response from Biothings Explorer ReasonerStdAPI",
        "error": "2 validation errors for QueryGraph\nnodes\n  field required (type=value_error.missing)\nedges\n  field required (type=value_error.missing)",
        "timestamp": "2022-03-13T18:31:27.552455",
        "level": "WARNING"
    },

I've been unable to fully test this because KG2 is currently not a fully valid KP. I've opened an issue here: https://github.com/RTXteam/RTX/issues/1828

There's a bug in reasoner pydantic where we're incorrectly invalidating responses from KPs that have lists as extra attributes in the results. Here's the issue: https://github.com/TranslatorSRI/reasoner-pydantic/issues/35

KP Registry will try and fall back to the other maturity if one isn't available. I'm assuming KG2 prod was down and we ended up using dev.


Provide default target as part of recipe

Default target should normally be "Build" but may be overridden by user.

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

GitHub Release
NuGet Package


Suggest that deployments go to separate environments

Description
No reason we should all be stomping over each other with -staging.
Test plan
it's documentation

for this to work do we have to create the eb environment before hand or will it automatically create it for us?

for this to work do we have to create the eb environment before hand or will it automatically create it for us?

for this to work do we have to create the eb environment before hand or will it automatically create it for us?

Good point, let me fix it.  It's an extra step but pretty straightforward.

for this to work do we have to create the eb environment before hand or will it automatically create it for us?

Good point, let me fix it.  It's an extra step but pretty straightforward.

Ok fixed!

Ok fixed!


Use six for python 2/3 compatibility

This simplifies some existing imports statements and make additional compatibility easier.

Thanks. Does this make your life easier in some other way? I have conflicting feelings about adding dependencies, especially something like six or requests that gets used a lot by CLI apps that are likely to be installed with Homebrew. Adding more dependencies increases the risk of a dependency conflict between homebrew-pypi-poet itself and the target package. I'm not aware of packages that pin six, so I don't know that my concern is realistic, but I also don't expect poet to expand as a project, so I don't see a clear win in terms of maintenance burden here. What's your thinking?

I think six is ubiquitous enough that it shouldn't be a problem.  It's also been very stable, so people updating packages using poet are likely to have a recent version that works.
My primary motivation for this is that I'd like to add a feature which allows poet to consider packages in  repositories other than pypi.python.org, something akin to pip's --extra-index-url flag.  Doing so means catching exceptions which urlopen may throw, and that gets a bit uglier in a non-six world.  Still doable, but messy.

Ubiquity is both a mitigating and exacerbating factor. You could argue that to work reliably, poet needs to be compatible with all versions of its dependencies that have ever been released, because something in the dependency tree of a target might require a different version.  Packages that aren't often used in CLI apps mostly don't matter because the probability of a conflict is low (which is the justification for jinja). OTOH, if something was incompatible with a new version of six, it would probably have to become compatible pretty quickly. So, six is probably fine, but there's also no real benefit to the existing code base.
Anyway, I'm not personally interested in maintaining a version of poet with that feature (#55); of course, you're welcome to fork poet.
I'll choose not to accept this PR in the spirit of conservatism ("if it ain't broke..."), but thanks for your time!


[AIRFLOW-1848] Fix DataFlowPythonOperator py_file extension doc comment

Dear Airflow maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[X] My PR addresses the following Airflow-1848 issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-1848

Description

[X] Here are some details about my PR, including screenshots of any UI changes:
Dataflow Python operator takes in a filename without .py extension, which was incorrectly documented previously.

Tests

[X] My PR adds the following unit tests OR does not need testing for this extremely good reason:
N/A, just a doc change.

Commits

[X] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Codecov Report

Merging #2816 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2816   +/-   ##
=======================================
  Coverage   73.62%   73.62%
=======================================
  Files         158      158
  Lines       11962    11962
=======================================
  Hits         8807     8807
  Misses       3155     3155

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 59aba30...c32bcfe. Read the comment docs.


Benchmark tests fail to run

Describe the bug
Following the steps in the contribution guide throws an error when running npm run test:bench
To Reproduce

Steps to reproduce the behavior:
git clone git@github.com:material-components/material-components-web-components.git mwc
cd mwc
npm install
npm run build
npm run test:bench

Expected behavior
I'd expect the script to run and allow me to benchmark.
Screenshots

Browser Version (please complete the following information):

OS: MacOS

Additional context
I get the following stack trace:
(node:14802) UnhandledPromiseRejectionWarning: Error: ENOENT: no such file or directory, scandir 'test/benchmark'
    at Object.readdirSync (fs.js:948:3)
    at /Users/ykra/projects/material-components-web-components/test/src/benchmark/cli.ts:113:16
    at Object.<anonymous> (/Users/ykra/projects/material-components-web-components/test/src/benchmark/cli.ts:162:3)
    at Module._compile (internal/modules/cjs/loader.js:1138:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1158:10)
    at Module.load (internal/modules/cjs/loader.js:986:32)
    at Function.Module._load (internal/modules/cjs/loader.js:879:14)
    at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:71:12)
    at internal/main/run_main_module.js:17:47

I cannot reproduce. Can you confirm that you are passing a parameter to the test:bench script. e.g.:
npm run test:bench -- --package button

My bad
Didn't notice the need for a package
Thanks :)


[ARITH] Refactor to use explicit div/mod functions instead of operators.

This PR is part of https://github.com/dmlc/tvm/issues/3977
Summary of changes:

Removed the operator overload for % and / and explicitly ask developers to use the specific division functions
Introduce indexdiv and indexmod, a special division function that we can redirect to either floordiv/truncdiv and the requirement to use this function is that we know the operands are non-negative, we might introduce additional constraints attachment if necessary as in https://discuss.tvm.ai/t/discuss-embed-more-bound-information-into-var-or-expr/4079
Refactor the existing code to make use of the division functions
For now we maintain the old behavior of the code, and we can switch the implementation of indexdiv/mod in a separate PR.

cc @yzhliu @icemelon9 @wweic @xqdan @sgrechanik-h @derisavi @umangyadav @icemelon9 @kazum

This looks really great @tqchen.

Additional notes on division modes:

floordiv is relatively easier for simplification as it does not require to know the sign of operand a

we can do floordiv(a+ 17, 16) -> floordiv(a + 1, 16)
However, we need to prove the sign of the operands eventually when lower to the low level code, as we can only lower floordiv(a, b)->truncdiv(a, b) if a>=0 and b>=0

truncdiv is easier to for codegen, as it directly corresponds to low level instructions, but it is harder for simplification:

truncdiv(a+ 17, 16) -> truncdiv(a + 1, 16) only if a+1>=0.

So we have to make the choice of division mode. indexdiv/mod tries to isolate that choice into a single place. It also brings additional information to the function that a and b are non-negative at the moment we call the function.

Currently, we lower indexdiv-> truncdiv, we plan to move to floordiv
We can plugin additional hints: indexdiv(a, b)->floordiv(a, nonneg_hint(b) if b is Var else b)


Failed to connect with Azure Storage.

Hi,
When I tried to run the example in this page at 'Writing and reading data from Azure Table storage', I can run the app in the Unity with the Connection String (Azure storage can update the objects). However, when I deployed the app to HoloLens 2, there is an error, 'Failed to connect with Azure Storage. If you are running with the default storage emulator configuration, please make sure you have started the storage emulator.'. Could you explain the reason for me? Thank you!
[Enter feedback here]

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

ID: f15853f4-20ac-bea9-89b0-4cd7da19a5be
Version Independent ID: 398529a4-fee0-21bb-6cc6-b57b47d25ea1
Content: Azure Cloud Tutorials - 2.

Hi @hferrone,
Unfortunately, I'm not familiar with this tutorial series... would love to go through the tutorials, but time is an issue :) For future reference, I'm familiar with the following tutorial series:

Getting started tutorials
Azure Spatial Anchors tutorials
Multi-user capablilities tutorials
Azure Speech Services tutorials

I'll check if someone in my networks knows how to handle the 'Failed to connect with Azure Storage' issue, and update here if any luck.

Hello everyone,
Was anyone able to figure out why this error was occurring when deploying the app unto the Hololens2 device, but not when testing within the Unity Editor?
Best,
Nick

Hello everyone,
Was anyone able to figure out why this error was occurring when deploying the app unto the Hololens2 device, but not when testing within the Unity Editor?
Best,
Nick

@martinwilter Could you tag whoever overhauled this tutorial series in your org?

Sure @hferrone, I'll check if they are available.

Thee Azure Storage Emulator is only accessible to your local dev machine, this is why you get this error message on your HoloLens 2. To fix this you need to create an Azure Storage account on Azure (this may require a credit card).
If for some reason your are in a situation where you can't create an Azure Storage account, you could alternatively expose the ports of your local machine to make the Azure Storage Emulator accessible in your local network, but scenario is not officially supported.

Thanks @rubit0! @XiangGuo1992 I'm going to close this, but if this problem persists or another one comes up feel free to comment in the thread and I'll reopen :)


Impératif dans le titre: "Téléchargez […]"

You can translate the website online on Transifex: https://www.transifex.com/projects/p/elementary-mvp/. Please don't update directly files in lang/ on Github as they'll be overriden when pulling new translations from Transifex.
You can propose new languages if they're not listed. Make sure to avoid requesting languages that already exist, for instance adding Russian (Russia) when Russian is available.
Please read the branding guidelines before translating and pay attention to spelling mistakes.

Oh I see. Thank you!


Add react-apollo dependency

Coverage remained the same at 100.0% when pulling b1cdf219aee45193aec3b7f27e16e676cfcf5b9e on angga-tvlk-patch-1 into fbe05710fd0438b7534167ea1edc7d151b57a262 on master.


DO NOT REVIEW: just need some change to kick travis

OK, Travis seems to be working now...https://docs.travis-ci.com/user/travis-pro/#How-can-I-make-a-private-repository-public FTW


Draft: Removals for v2

This is a quick and dirty draft of the removals that we want to do for v2. Happy to split this into smaller prs (see commits).

Codecov Report

Merging #445 (7926cb3) into main (e60e495) will increase coverage by 9.55%.
Report is 1 commits behind head on main.
The diff coverage is 73.13%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #445      +/-   ##
==========================================
+ Coverage   59.81%   69.37%   +9.55%
==========================================
  Files         124       76      -48
  Lines        9915     8382    -1533
==========================================
- Hits         5931     5815     -116
+ Misses       3984     2567    -1417

Files
Coverage Δ

.../Connection/PostgresConnection+Configuration.swift
50.00% <100.00%> (+0.70%)
:arrow_up:

...es/PostgresNIO/Connection/PostgresConnection.swift
60.27% <100.00%> (+17.78%)
:arrow_up:

Sources/PostgresNIO/Data/PostgresDataType.swift
55.94% <ø> (-0.26%)
:arrow_down:

Sources/PostgresNIO/Data/PostgresNumeric.swift
78.60% <ø> (ø)

Sources/PostgresNIO/Data/PostgresRow.swift
70.27% <ø> (+21.82%)
:arrow_up:

Sources/PostgresNIO/New/NotificationListener.swift
86.95% <ø> (+13.83%)
:arrow_up:

Sources/PostgresNIO/New/PSQLEventsHandler.swift
72.41% <ø> (ø)

...urces/PostgresNIO/New/PostgresChannelHandler.swift
84.65% <100.00%> (-0.13%)
:arrow_down:

...stgresNIO/New/PostgresFrontendMessageEncoder.swift
100.00% <100.00%> (ø)

Sources/PostgresNIO/New/PostgresQuery.swift
82.44% <ø> (+1.14%)
:arrow_up:

... and 8 more

... and 2 files with indirect coverage changes


GIF is hard to follow

I'd much rather just have plain text vs. the GIF. The input typing is too slow and the output lasts too short a time, especially the final output. Then it's a pain if you want to look at any part more than the allowed time - you have to wait for it to come around again! It would be much easier to grok if it were just plain text vs. the GIF.

Also - other than this tiny quibble I love the idea, I appreciate you building it! :)

I'm gonna keep the gif around because it's flashy, but I took your criticism and stretched out the last frame quite a bit. Hope that helps!


React peerDependency inquiry

Folks -
Is there any reason that you peer-require >=0.13.3 for this, and could >=0.13.2 be used instead? I'm not sure what the react changes are from those versions. I unfortunately can't seem to get another dependency I have's latest version to work with our builds. Wondering if this is an option to update, or how much work would be involved?
I can do the legwork if you think there's a decent amount to do, or hold off on pulling this in to our apps until we can update that other module as well.
Thoughts?
Update: wondering if you're requiring anything in this release specifically

0.13.3 is just arbitrary. I think that was the latest version available at the time we updated React last. Would you be able to use 0.14.0 in the near future? Our goal is to keep up with the latest React release.


increase timeout for round-trip on pipes:receives test

@Cypher1 as you mentioned in #2220 I think there is some environment problem with remote Travis vs local, which implies the timeout-duration is a red-herring.

Thanks @sjmiles, I agree. We might need to look into what's different about Travis (as tests passed on AppVeyor) at some point.


[Feature] : multiple launguage support ,

Description: We need to add multiple language support to our application to enhance user experience and accessibility. Users should be able to switch between different languages seamlessly.

@samyak-aditya Please Assign this to me by adding Gssoc and hacktoberfest Tag

@Jashwanth-rit it is done


how to obtain the camera serial number, that is tcp protocol

####Problem:
####Details:

GoPro Camera(s):
Firmware Version:
Steps to reproduce:
Happens every time? [Y/N]:

You can obtain the camera's serial number (for HERO+, HERO4 and HERO5) here: https://github.com/KonradIT/goprowifihack/blob/master/HERO4/gpControl-Hero4Black.json#L5136

thank you, konradIT


hot react reload is not working

I keep getting this error whenever I'm editing something outside of the render():
hot-client.js:86 Uncaught TypeError: Cannot read property 'c' of undefined

couldn't reproduce. Probably it's something that I messed up

If you come across this again, paste the full file here.  From what I recall, this can happen when adding a new import, but only once.  But yes, will happily accept samples that cause this if you come across again.


Plugin funnel doesn't work in >2.7.3

Expected Behavior

Plugin funnel should work as previously.
Current Behavior

Chart.js:4879 Uncaught TypeError: Cannot read property 'options' of undefined
    at ChartElement.initialize (Chart.js:4879)
    at ChartElement.initialize (chart.funnel.min.js? [sm]:1)
    at ChartElement.DatasetController (Chart.js:3617)
    at new ChartElement (Chart.js:2395)
    at Chart.buildOrUpdateControllers (Chart.js:9573)
    at Chart.update (Chart.js:9626)
    at Chart.construct (Chart.js:9357)
    at new Chart (Chart.js:9294)
    at c.loadChart (Widget.js:1920)
    at c.postLoadWidget (Widget.js:1472)

Possible Solution

DatasetController should have _getIndexScale() function.
Steps to Reproduce (for bugs)

https://gitdeveloper.yetiforce.com/index.php
Context

Environment

Chart.js version: >2.7.3
Browser name and version: Chrome 78.0.3904.108
Link to your project: https://github.com/YetiForceCompany/chartjs-plugin-funnel

DatasetController should have _getIndexScale() function.

Chart.js 2.9.3 does have a _getIndexScale() method. If you're trying to use the code in master the please be aware that we are working Chart.js 3.0, which breaks backwards compatibility, but provides numerous improvements including being massively faster. In that case you will have to use meta.iScale.

You are right, the _getIndexScale() method exists. Thank you for your suggestion, we will definitely be eager to support 3.0.
At this moment I fixed this by adding xAxes to plugin, previously yAxes was enough. I am quite new in chart.js so I am not sure if it is the right way.
https://github.com/YetiForceCompany/chartjs-plugin-funnel/commit/f7829215ff2aefa3392f0b44dbb8b08e625c2b2e#diff-970c199009b0bd250e8c75702ff0bd78L268

Cool. Glad it's working!
You might consider submitting your plugin to https://github.com/chartjs/awesome


XmlReader.java gets stuck in infinite loop on nVidia Android hardware due to ART optimizations

The tight loops inside XmlReader.java end up being optimized by the ART on nVidia hardware - Shield TV and possibly others - such that changes to the loop variables are lost.
The problem occurs with the variables _upper and _lower in the while(true) loops at lines 130 and 152.
A workaround is to access those variables at the bottom of the loop in a debug log message between lines 141-142 and between 163-164.

Sounds to me like an issue which should be reported to the Android team.

I would agree, except for a couple of pieces of information:

In the dozen devices I've tested, this only seems to affect the nVidia Shield TV - which suggests it's an issue for nVidia to fix rather than Android.

nVidia's response is: "(The software team) believe it's a valid optimization that ART is doing much like a C compiler would do in the same instance."

That doesn't leave much option for having it fixed outside of libgdx. For my own needs, I'll likely be hacking a local version of XmlParser.java that works around the issue.

Can you include a link to the nvidia (and android?) response? Afaik the XmlReader class is generated by Ragel, so it is probably not a good idea to modify that class directly. Otherwise, I guess a simple solution would be to use while (_upper >= _lower)

The response was in an email thread with their developer support team. The response was:
"I did ask some of the SW people about that and they believe it's a valid optimization that ART is doing much like a C compiler would do in the same instance. They suggest you explicitly use the variables which it sounds like you are doing as the WAR."

I'd close this issue for the reasons below:

It's clearly a device-specific issue
that device is not so common
you could ask the ragel team to generate more usual code (like the one suggested by @xoppa ) here, here and here. If they release a new version we will use it for sure.

I don't agree that an optimization which clearly alters the functionality is valid (at least not if you cant disable it somehow). But I guess there isn't much we can do about that. Going to close this for above reasons. Please send a pull request when this is fixed in ragel.

I very much agree that the optimization itself is incorrect. As much as anything, I wanted an issue listed here in case others run into it and go looking for solutions. My workaround ended up being replacing XmlParser.java with a locally-modified version, and inserting that into a local replacement for TmxMapLoader.


F#65 When I change crossing point value of area-line chart, measure based color distribution doesn't work

Description

색상모드를 measure로 설정 후 crossing point를 설정하면 전체색상이 지정된 값 범위게 맞게 동작하지않음

Related Issue : #65

How Has This Been Tested?

color by measure로 설정시 crossing point에 값을 설정 후 범위에 맞게 색상이 설정되었는지 확인
저장 후 재조회시 설정된값에 맞게 색상이 설정되었는지 확인

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My code follows the code style of this project. it will be added soon
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document. it will be added soon
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Additional Context

run build
deploy to 1


bump version to 0.21.0

Because the breaking changes are so small we should really semver-trick 0.20 into using the 0.21 types.


add discover llamaindex tutorials

Initial PR to add our video tutorial series to the docs
Will only merge this once the latest tutorial is uploaded to youtube (and we have a link for the video lol)

@jerryjliu btw the YouTube link was blank btw 😅 will update once it's ready


Collapsing inline operators seems to not work

See the following magma:
import magma as m
import mantle

def mux_ifc(height, width):
    ports = []
    for i in range(height):
        ports += [f"I{i}", m.In(m.Bits(width))]
    sel_bits = m.bitutils.clog2(height)
    ports += ["S", m.In(m.Bits(sel_bits))]
    ports += ["O", m.Out(m.Bits(width))]
    return ports

def gen(height, width):
    cell = m.DefineCircuit("aoimux_cell", *mux_ifc(2, 1))
    m.wire(cell.O, ((cell.I0 & ~cell.S) | (cell.I1 & cell.S)))
    m.EndDefine()
    return cell

m.compile("cell", gen(2, 1), output="coreir-verilog", inline=True)

We would expect the verilog to collapse the operators into ((I0 & ~S) | (I1 & S)) but it does not.

Yes the tests seem to be failing silently on travis (see https://github.com/rdaly525/coreir/issues/677). I'll investigate, there was likely a regression somewhere, possibly related to the debug verilator changes.

Just tested this on the latest master and it produces:
module aoimux_cell (
    input [0:0] I0,
    input [0:0] I1,
    input [0:0] S,
    output [0:0] O
);
assign O = (I0 & (~ S)) | (I1 & S);
endmodule


DCOS-10470: [9/10] Introducing form-dropdown to DSLFilterField

This PR introduces the form UI to the DSLFilterField

I'm noticing some strange behavior when I try to type a DSL query into the search field:

There's something weird going on with the suspend checkbox...

@jfurrow good observation! Unfortunately, the colon is an invalid character for fuzzy-matching tokens. If it was allowed, they would be misinterpreted as attributes. I decided to treat : as   and the user feedback actually looks good.

@rcorral , @jfurrow I should have addressed both of your issues

mesosphere-ci Retest this please

A : in the has the words field will be replaced with a unbackspaceable space sign. But if I hit cmd + A I can remove it. But except that it works nice. I'll look at the code now.

@Poltergeist your issue with the tailing whitespaces should be fixed! :+1:

@wavesoft Thank you for the hard work on this guy, it looks great 👌 💯
I found two things that still need fixing, but I think will be small for you to fix in a follow up PR.

Entering text and into the "has words" input field and then pressing [enter] does not work.
Searching is always global, it's never contextual to the folder you're currently looking at.


Support for coc.nvim

It's great! I wonder if you have plan to support coc users?

You can already do so with coc-sumneko-lua.

coc-sumneko seems to contain an old copy of this repo, which is not ideal I guess.
I don't use coc myself, and won't add support, however, it should be fairly easy to get the correct settings with the require("neodev.sumneko").setup method. Check the code to see how to use


Icons on the front page

Give a brief about the bug ✍
In the front page the icons of twitter,linkedIn and github are small in size,which is making them difficult to read.
What is the expected behavior? 🤔
Normally the icons are slightly bigger in size in  websites .But in the home page they are looking little smaller.
Provide step by step information reproduce the bug 📄
I can increase the size of the icons and add some styling to them, so that they may look more catchy and attractive. Slight increase in the size of icon on hovering will make them more catchy.
Select program in which you are contributing
GSSoC23
Code of Conduct

[x] I follow CONTRIBUTING GUIDELINE of this project.

Hey @Shreyans2004 !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign

Hey @Shreyans2004 👋!
@kunjgit here,
We have this annoucement to make 📢!!
We have updated our README 🚀
Make sure you follow proper guideline to raise your next issues !!
Make sure you join our Discord🕹 if you haven't 😇
Thank you 💗!

Hello @Shreyans2004, Time's Uppp!⏰
Sorry for closing your issue!
But it's more than a week since we haven't received anything from your side 😢 .
Come up with new ideas, create a new issue and make sure you finish it within a week! 🔥
All the best! 🚀
Happy Hacking! 💗

Hey @Shreyans2004 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


Implement ERC20GatewayTokenPair and ERC20GatewayTokenPairRepository

Fixes #285

@pgev  Thanks for the changes.
I have reviewed the changes and comment. Also mark them resolved. There are still few open conversations on this PR.  I request @deepesh-kn  and @abhayks1  to look into this.

@pgev , Why ERC20GatewayTokenPairRepository is not initialized in Repositories ?

@pgev , Why ERC20GatewayTokenPairRepository is not initialized in Repositories ?

@gulshanvasnani it was missed. Please, go forward and add it. Thanks!


Fix issues in UG examples

Fix #186 #187 #191 #198 #216 #217 #219 #232 #236

Pull Request Test Coverage Report for Build 692

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 92.105%

Totals

Change from base Build 689:
0.0%

Covered Lines:
2275

Relevant Lines:
2470

💛  - Coveralls


Using '*' in tables throws syntax errors

  Scenario: Test tables
    * def nameList = ["Angie", "Frank", "Dan"]
    * table names
      | name  |
      | 'Angie' |
      | 'Frank' |
      | 'Dan'   |
    * match names[*].name contains nameList
    * match nameList contains names[*].name

The first assert works fine but the second throws the below error
 javascript evaluation failed: names[*].name, <eval>:1:6 Expected an operand but found *
Is it a requirement to put '*' on the left of the contains keyword ?

for these kinds of questions, we prefer stack ovrerflow.
change the last line to: * match nameList contains $names[*].name
reason: https://github.com/intuit/karate#jsonpath-short-cuts


Basic GGCN edge types support

This is a basic implementation of edge type support for GGCN. At the moment it doesn't support fused operation since message_and_aggregate accepts lifted adjacency matrix, but in case of multiple edge types it should accept list of sparse tensors
    def message_and_aggregate(self, adj_t: SparseTensor, x: Tensor, edge_attr: OptTensor) -> Tensor:
        return matmul(adj_t, x, reduce=self.aggr)

Thank you very much! I will have a look ASAP.


Inconsistent "Returns Analysis" Table

This issue is regarding the "Returns Analysis" table on the tearsheet. More specifically it relates to the last 3 rows in the table:

Mean Period Wise Return Top Quantile (bps)
Mean Period Wise Return Bottom Quantile (bps)
Mean Period Wise Spread (bps)

The issue is in how the return statistics are quoted. The Mean Period Wise Returns, Top Quantile and Bottom Quantile (1 and 2 above) are quoted as a holding period excess return (i.e. ending_value/beginning_value - 1). However, the Mean Period Wise Spread (bps) (3 above) is quoted on a one-period basis (i.e. it uses the utils.rate_of_return function to convert holding period return to a one period rate of return)
My recommendation is the following:

Make the all return rows quoted using the same method. (Either all one-period returns or all holding period returns). Based on the description, it sounds like they should be quoted as one-period returns.
I think it should be made clear to the user that returns are "excess returns", as in the code, it appears the universe mean return is subtracted from the return to come up with the quoted returns. Possibly add this in the description in the first column of the pandas table.

My notes are below for more clarity/brevity:

Thank you for noticing and reporting this. Regarding point 1 they were all suppose to be  one-period returns. Regarding point 2, the demeaing is controlled by the long_short flag passed to the different tear sheet functions. The description could be added to the flag documentation.
Would you like to submit a pull request?

I can give it a shot once I learn how. I'm still in the process of learning to use Git and Git Hub.

If you like to provide a PR, then it will be quite easy:

fork Alphalens project on your github account
clone your forked Alphalens on your workstation
create a branch where you'll commit the changes
push the branch on your forked Alphalens
from github web page you will see your branch and a "Create pull request" option
Once you create the PR we'll see that in main Alphalens project and we will be ready to merge the patch or to ask you to provide more commits on that branch


Various problems when trying to convert a python module py-webrtcvad to C

Nuitka version, full Python version and Platform (Windows, OSX, Linux ...)
Nuitka 0.5.30
Python 2.7.12
pip 9.0.1 from /usr/local/lib/python2.7/dist-packages (python 2.7)
Platform Ubuntu Linux

I installed Nuitka by downloading source tar and doing python setup.py install
I'm trying to convert py-webrtcvad module to C
Steps to test the issue
git clone https://github.com/wiseman/py-webrtcvad
cd py-webrtcvad/
python example.py

If you get any import errors, do pip install webrtcvad
You should see the output
 Usage: example.py <aggressiveness> <path to wav file>
Now, I try to convert example.py to C using following command
python -m nuitka --show-progress --show-modules --standalone --portable  --recurse-all --python-version=2.7 example.py >& nuikta.build.log &
The build log is kept here
Main warnings from log are
Nuitka:WARNING:/home/user/code/python/webrtc_vad/testing/py-webrtcvad/webrtcvad.py:3: Cannot find '_webrtcvad' as relative or absolute import.
Nuitka:WARNING:/usr/local/lib/python2.7/dist-packages/pkg_resources/_vendor/appdirs.py:484: Cannot find 'com' in package 'pkg_resources._vendor' as relative or absolute import (tried pkg_resources._vendor.com,com).
Nuitka:WARNING:/usr/local/lib/python2.7/dist-packages/pkg_resources/_vendor/pyparsing.py:89: Cannot find 'ordereddict' in package 'pkg_resources._vendor' as relative or absolute import (tried pkg_resources._vendor.ordereddict,ordereddict).

Then, when I try to run examle.dist/example.exe I get following error
./example.exe
Traceback (most recent call last):
  File "/home/user/code/python/webrtc_vad/testing/py-webrtcvad/example.dist/example.py", line 6, in <module>
  File "/home/user/code/python/webrtc_vad/testing/py-webrtcvad/example.dist/webrtcvad.py", line 3, in <module webrtcvad>
ImportError: No module named _webrtcvad

How do I fix these WARNINGS and ERROR?

Do not use "--python-version" with the "-m nuitka" style of invocation. Best case not needed, worst case destroy the effect. I am not sure if that messed things up for you.

@kayhayen Thanks for comment. I had run the command without --python-version argument before. It caused the same warnings.

@kaykayen Thanks for comment. I had run the command without --python-version argument, it had given same warnings.

Yes,
python -c "import _webrtcvad"
works without any issues. I also tried to use --include-package=webrtcvad flag, but no improvement in warnings.

Sorry for being stubborn to not try myself, usually these kinds of reports all have been invalid, but this one is likely not:
Following your instructions, I did NOT see the warning, after running pip without a virtualenv, so it probably installed in my home directory. Normally I do that with "sudo" even to the system, which may or may not make a difference.
But at run time, I am getting this:
pkg_resources.DistributionNotFound: The 'webrtcvad' distribution was not found and is required by the application
I did not get any error about the module and I see "_webrtcvad.so" in the dist folder. Maybe I am not even reproducing your issue then. I am doing this on Debian, your Ubuntu should be same absolutely, but I will investigate, and maybe there is odd things about pkg_resources to be seen.
The other two warnings will go away in the next release btw, as they are handled by Nuitka properly.

I edited the subject as it suggests that Nuitka converts Python to C. It does not. The C code is not for use.

So this is the minimal reproducer of what I am seeing:
import pkg_resources
pkg_resources.get_distribution('webrtcvad')

I noticed, that I do have a pip installed pkg_resources. Might well be that you are not. In that case, some of the hacks Nuitka does for pkg_resources may not work with what you have. Can you obtain some kind of version information.
Yours,
Kay

I am assuming the minimal import of "import _webrtcvad" failing for you, can you post the output of that compilation with --explain-imports option, it will allow me to check things. Also please determind the _webrtcvad.file when using CPython with print.

Might well be that you are not. In that case, some of the hacks Nuitka does for pkg_resources may not work with what you have. Can you obtain some kind of version information.

I didn't get you. Which command to use to find the version information? I provided pip version in OP.

where does pkg_resources.file point to for you? Into "/usr/local" ?

>>> print pkg_resources.__file__
/usr/local/lib/python2.7/dist-packages/pkg_resources/__init__.pyc

can you post the output of that compilation with --explain-imports option

Ok, I have uploaded the compilation log with this option here https://gist.github.com/pranavj/a02ce61bdd03717ddcacaabdb8d0b900

please determine the _webrtcvad.file when using CPython with print.

Sorry, didn't understand what to do.

The log looks really good. I want to see, where normal CPython loads the _webrtcvad from, I think this code will do it, run it uncompiled:
import _webrtcvad
print(_webrtcvad.__file__)

Then I can compare that to what places Nuitka looks in your log. Also if you can, use "locate _webrtcvad" and check if there is something wrapping it.
Yours,
Kay

I want to see, where normal CPython loads the _webrtcvad from

user$ python
Python 2.7.12 (default, Dec  4 2017, 14:50:18)
[GCC 5.4.0 20160609] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import _webrtcvad
>>> print(_webrtcvad.__file__)
/home/user/.cache/Python-Eggs/webrtcvad-2.0.11.dev0-py2.7-linux-x86_64.egg-tmp/_webrtcvad.so

Also if you can, use "locate _webrtcvad"

$ locate _webrtcvad
/home/user/.cache/Python-Eggs/webrtcvad-2.0.11.dev0-py2.7-linux-x86_64.egg-tmp/_webrtcvad.so
/home/user/code/python/webrtc_vad/testing/py-webrtcvad/test_webrtcvad.py
/home/user/code/python/webrtc_vad/webrtcvad-2.0.10/test_webrtcvad.py
/home/user/code/python/webrtc_vad/webrtcvad-2.0.10/build/lib.linux-i686-2.7/_webrtcvad.so
/usr/local/lib/python3.5/dist-packages/_webrtcvad.cpython-35m-x86_64-linux-gnu.so

Please attach your /usr/lib/python2.7/site.py any case.

https://gist.github.com/pranavj/407c6d73799e33c1b1cd1a83d2ebb900
python "import sys;print(sys.path)"
do you mean python -c "import sys;print(sys.path)"?
None of the three commands work on my machine. They all give "[Errno 2] No such file or directory"
The output of
python -c "import sys;print(sys.path)"
is
['', '/usr/lib/python2.7', '/usr/lib/python2.7/plat-x86_64-linux-gnu', '/usr/lib/python2.7/lib-tk', '/usr/lib/python2.7/lib-old
', '/usr/lib/python2.7/lib-dynload', '/usr/local/lib/python2.7/dist-packages', '/usr/local/lib/python2.7/dist-packages/auditok-
0.1.6-py2.7.egg', '/usr/local/lib/python2.7/dist-packages/six-1.10.0-py2.7.egg', '/usr/local/lib/python2.7/dist-packages/regex-2017.7.28-py2.7-linux-x86_64.egg', '/usr/local/lib/python2.7/dist-packages/nltk-3.2.4-py2.7.egg', '/usr/local/lib/python2.7/dist-packages/swig_decoders-1.1-py2.7-linux-x86_64.egg', '/usr/local/lib/python2.7/dist-packages/pip-9.0.3-py2.7.egg', '/usr/local/lib/python2.7/dist-packages/webrtcvad-2.0.11.dev0-py2.7-linux-x86_64.egg', '/usr/local/lib/python2.7/dist-packages/Nuitka-0.5.30-py2.7.egg', '/usr/lib/python2.7/dist-packages', '/usr/lib/python2.7/dist-packages/PILcompat', '/usr/lib/python2.7/dist-packages/gtk-2.0']

can you upload that this:
/usr/local/lib/python2.7/dist-packages/webrtcvad-2.0.11.dev0-py2.7-linux-x86_64.egg
It's uploaded here https://github.com/pranavj/filesharing/blob/master/webrtcvad-2.0.11.dev0-py2.7-linux-x86_64.egg

sys.meta_path is empty!
$ python
Python 2.7.12 (default, Dec  4 2017, 14:50:18)
[GCC 5.4.0 20160609] on linux2
Type "help", "copyright", "credits" or "license" for more information.

import sys; print(sys.meta_path)
[]
print(sys.path)
['', '/usr/lib/python2.7', '/usr/lib/python2.7/plat-x86_64-linux-gnu', '/usr/lib/python2.7/lib-tk', '/usr/lib/python2.7/lib-old', '/usr/lib/python2.7/lib-dynload', '/usr/local/lib/python2.7/dist-packages', '/usr/local/lib/python2.7/dist-packages/auditok-0.1.6-py2.7.egg', '/usr/local/lib/python2.7/dist-packages/six-1.10.0-py2.7.egg', '/usr/local/lib/python2.7/dist-packages/regex-2017.7.28-py2.7-linux-x86_64.egg', '/usr/local/lib/python2.7/dist-packages/nltk-3.2.4-py2.7.egg', '/usr/local/lib/python2.7/dist-packages/swig_decoders-1.1-py2.7-linux-x86_64.egg', '/usr/local/lib/python2.7/dist-packages/pip-9.0.3-py2.7.egg', '/usr/local/lib/python2.7/dist-packages/webrtcvad-2.0.11.dev0-py2.7-linux-x86_64.egg', '/usr/local/lib/python2.7/dist-packages/Nuitka-0.5.30-py2.7.egg', '/usr/lib/python2.7/dist-packages', '/usr/lib/python2.7/dist-packages/PILcompat', '/usr/lib/python2.7/dist-packages/gtk-2.0']

Did you manually enter it to PYTHONPATH
No, that variable is empty for me.

Check with ls /usr/lib/python2.7/dist-packages/*.pth
No, none of the pth files are related to webrtcvad

$ ls /usr/lib/python2.7/dist-packages/*.pth
/usr/lib/python2.7/dist-packages/PILcompat.pth               /usr/lib/python2.7/dist-packages/pygtk.pth
/usr/lib/python2.7/dist-packages/matplotlib-1.5.1-nspkg.pth  /usr/lib/python2.7/dist-packages/zope.interface-4.1.3-nspkg.pth

There we go, egg files are zip files:

Length      Date    Time    Name

  536  2018-05-22 09:49   _webrtcvad.pyc
 1709  2018-05-22 09:49   webrtcvad.pyc
  976  2018-05-11 15:42   webrtcvad.py
  286  2018-05-22 09:49   _webrtcvad.py
163080  2018-05-22 09:49   _webrtcvad.so
   21  2018-05-22 09:49   EGG-INFO/top_level.txt
 2055  2018-05-22 09:49   EGG-INFO/SOURCES.txt
    1  2018-05-22 09:49   EGG-INFO/dependency_links.txt
   14  2018-05-22 09:49   EGG-INFO/native_libs.txt
    1  2018-05-22 09:49   EGG-INFO/zip-safe
 3739  2018-05-22 09:49   EGG-INFO/PKG-INFO
   74  2018-05-22 09:49   EGG-INFO/requires.txt

This is probably dealt with zipimporter, which might cache things in ".cache", where you will then find it. In Nuitka there currently is nothing to support it, but it seems feasible to add.
These eggs are likely out of fashion, precisely because they require each user to privately extract on the fly. To check if an update is needed, some processing of the zip file will be needed each time, for every program you run, and that does an import not satisfied by earlier elements of the path. That will make your Python only slower.
So, no trick there inside the zip luckily. Please allow for some time to implement this. At least I now have an .egg file to test with. Hopefully it's as easy as to ask zipimporter to do its thing. But could also be that it's not possible to decouple searching from actual importing, then I get to implement the caching part myself. Ideally the cache hit is the normal case, and then it's only about replacing the egg with a cache directory, and maybe complain if it's not cached yet. Nuitka may not have to be the first to unpack an egg into a cache. But that is speculation, need to check what is there fore use.
Yours,
Kay

Hi, any update?

So I just pushed this to factory. However, in your example there is also something that causes an error, and has been reported elsewhere. The pkg_resources is used to require "webrtcvad" and cannot find it in standalone mode. This is a separate issue. Egg files work now at least.
It will be in 0.6.0 release that I hope to make this weekend.
I am now turning towards the "pkg_resources" issue with your egg, because so far I only had un-reproducible (for me) reports, so this is actually very nice and small reproducer your egg.

So this is in the new pre-release now, closing now as release is immanent and not hotfixes will happen for 0.5.33 anymore.
For the pkg_resources issue I create #146 which seems possible, but will also not immediately be dealt with.


pysis.env.ISIS_PATH and ISIS_ROOT settings not respected

I don't know how pysis sets these and when, but when I saw that I can change the values of pysis.env.ISIS_PATH and _ROOT I was hoping that I can point pysis to a parallel ISIS3 install I have where I fixed an ISIS3 bug.
Can you confirm if this should be working or if I misinterpret things and how else I could work with a temporary ISIS3 setup?

The value of pysis.env.ISIS_PATH gets set from the environmental variable ISISROOT and pysis.env.ISIS_PATH gets set to $ISISPATH/bin.
You can over ride it with:
export ISISROOT=/path/to/my/isis
python myscript.py

https://github.com/wtolson/pysis/blob/master/pysis/env.py#L19

Going to close this for now. Let me know if you are still running into problems @michaelaye.

Any clue why it doesn't work with jupyter notebooks though, as described? The notebook was launched in an ENV with the new ISISROOT active?

Ah, sorry, must have been dreaming that I described this. ;)

Interesting, jupyter is probably reseting the env variables for the notebooks, try setting ISISROOT at the top of your notebook:
import os
os.environ['ISISROOT'] = '/path/to/isis/'

import isis
...

Alternatively you could create your own isis instance:
from pyisis import Isis

isis = Isis(path='/my/custom/isis/path')
isis.mdis2isis(from_=img_name, to=cub_name)

Both methods were still coming up with the original ISISROOT somehow. The only way I found to do this is to change ISISROOT value at the very beginning of login. Thanks for your help though!

hah, so i thought. Now it's getting weird, the local shell is correctly getting to my CMAKE ISIS3 version, while pysis somehow get's the old install... investigating...


ANSI Support

Check for ANSI support before utilizing ANSI-Colors

https://github.com/mabe02/lanterna might work


Upgrade to TestNG 5.12

From<EMAIL_ADDRESS>on March 29, 2010 15:07:01
Upgrade testng
Original issue: http://code.google.com/p/powermock/issues/detail?id=250

From<EMAIL_ADDRESS>on April 05, 2010 23:47:30
When in maven central

From<EMAIL_ADDRESS>on May 07, 2010 05:38:53
TestNG 5.12.1 is available: http://repo1.maven.org/maven2/org/testng/testng/5.12.1/

From<EMAIL_ADDRESS>on May 07, 2010 06:44:06
Yes I've already started with the upgrade. How ever I ran into a strange problem that
I haven't managed to solve yet after the upgrade.
Status: Started
Labels: -Type-Defect Type-Enhancement

From<EMAIL_ADDRESS>on May 11, 2010 11:39:44
Fixed in r1300
Status: Fixed


Use an early-setup hook to force console init in release images

Setting rd.hostonly=0 in the release and recovery images causes dracut to throw out the initqueue hooks that ZBM installs to force the event loop to fire at least once. As a result, the console is often not initialized before ZBM takes control. ZFSBootMenu early-setup hooks provide a convenient place to force the console initialization, allowing things like fonts and keymaps to be set before password prompts might appear.

@Soulsuke If possible, can you test the EFI image built from this PR to see if it fixes your keymap problem? You should be able to download the UEFI build artifact at https://github.com/zbm-dev/zfsbootmenu/suites/5276094336/artifacts/163197353 (it packs the EFI image in a ZIP file).

Thanks for confirming. We will not bump the version for this fix, but I will recreate the release binaries with the new configuration. You should be able to just keep using the one you tested; it will be essentially the same.

@ahesford sounds fair, thanks for the support :)


⚠ ClamAV has degraded performance

In ec092d1, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 51 ms

Resolved: ClamAV performance has improved in 0fa308b.


Why is vcloud cpi locked to 0.7.2?

In the gemspec it is locked to 0.7.2 with a link to https://www.pivotaltracker.com/n/projects/956238/stories/84406944
But https://www.pivotaltracker.com/n/projects/956238/stories/84406944 doesn't explain at all why.
In https://github.com/cloudfoundry/bosh/issues/719 we'll need to bump vcloud cpi to get newer nokogiri.

Closing - CPIs have been externalized. Nowadays, CPI locked versions in Gemfile.lock are bumped only when mandatory for legacy requirements.


[AUTOPATCHER-kernel] Kernel upgrade to version <IP_ADDRESS> - branch main

[AUTOPATCHER-kernel] Kernel upgrade to version <IP_ADDRESS> - branch main
Upgrade pipeline run -> https://dev.azure.com/mariner-org/mariner/_build/results?buildId=412523&view=results
AMD64 build -> https://dev.azure.com/mariner-org/mariner/_build/results?buildId=412527&view=results
ARM64 build -> https://dev.azure.com/mariner-org/mariner/_build/results?buildId=412528&view=results

Closing in favor of https://github.com/microsoft/CBL-Mariner/pull/6117


Add archive support to python instance

will move this to WIP status for now.

Closing this in favor of #2915


[Remix + Vite] Trying to build production app failing (mixed client and server code)

This app is based on the https://github.com/robipop22/dnb-stack. The upgrade on the stack worked.
Followed the same steps for my app but the build is failing even locally with :
'./entry.server.js' imported by<EMAIL_ADDRESS>Tried with package 2.8.0, 2.8.1, 2.8.1-patch2.
I commented in node modules the import : var entry_server = require('./entry.server.js');
it worked after.
But when I added back again the build started to fail again.
Any guidance or help would be much appreciated.
thanks

What is the full error you are seeing? Can you share your code?

`Error: [commonjs--resolver] Server-only module referenced by client
'./entry.server.js' imported by<EMAIL_ADDRESS>
See https://remix.run/docs/en/main/future/vite#splitting-up-client-and-server-code
file<EMAIL_ADDRESS>Happy to share any code that you think is helpful, or jump on a discord call or something similar.

If you can share the code that would be the most helpful. Also note that entry.server is a special file name in Remix, so if you are trying to import it within one of your routes then it's probably improper usage.

I'm using the defaults entry files. I deleted them so I make sure they won't interfere.
Tried a lot of workarounds, different vite configs.
different package managers.
Fails with the same node modules error everytime

I mean, it's something in your app code. It's not related to Vite config nor package manager nor even Vercel. You can try working backwards from the starter template which does not exhibit this error. I'd be happy to help diagnose further if you want to share a reproduction.

Feels like it, deleted the whole routes files then it failed again, then commented this file
<EMAIL_ADDRESS>it worked after, so it seems that one of the route might be at fault. trying to find out which one

Well that was fun to debug, but in the end I found it. I'm going to leave here my findings in case someone else comes across this.
I had a .ts route that was handling a POST and GET so it was exposing an action and loader.
But inside my action function I was calling a function (let's call it myFunction )
And myFunction was doing in the end something like :
return json({})
It seems that since the function was inside the same file it got treated like client code.
By moving myFunction to another file called myFunction.server.ts and importing it into my route with the action it all got treated on the server and the build is working now.
I tried to explain it the best I could. I'll close this. Thanks @TooTallNate  for the quick response.

Ya that makes sense. Nice explanation 👍

@robipop22 thanks for documenting this.
Ran into this as well with some helper functions, and was just about to give up.


test memoryview shape info

I don't think this checks that you aren't making a copy.  Do you want to also assert something like view2.obj is frame.data (or however you get to the raw data object behind the frame).


pr(archai): Adds support for Huggingface's Transformer-XL and configurable ONNX exports.

This PR introduces the following changes:

Fixes a problem where integer-based arguments were not being properly loaded in HfGPT2;
Adds support for HfTransfoXL, which is Huggingface's implementation of the Transformer-XL;
Cleans unnecessary imports;
Adds license headers to new files;
Fixes a bug when loading an ArchaiModel and not having PyTorch compiled with CUDA;
Changes models/model_list.py to models/available_models.py;
Allows for configurable ONNX exports;
Allows for configurable ONNX optimizations.

Tests are being run to check for possible breaks or bugs.


fix: define and/or use an existing type for GitHubEventAction

Currently, GitHubEventAction uses an any prop for the payload:
https://github.com/uclaacm/opensource/blob/87b2993fe00ccea7545d403ec9a2a4b918d800e3/components/GitHubEventAction.tsx#L5-L8
This shouldn't have to be the case, since octokit defines types for all of its request/responses, including the payload of an API hit. The reason I haven't done this yet? It's hard to find!
So, let's:

find the type, and/or create a TS Union of the relevant types
simplify the existing code in GitHubEventAction to use the new type, and avoid unnecessary null / ? checks

@mattxwang oops there is no accurate schema https://github.com/octokit/webhooks/pull/602#issuecomment-1005936266 😓. Webhooks is pretty accurate, so I think we can use that for now esp since Nick has been doing some really great work with it, and slight type issues aren't the end of the world.
We can update this once they have an events types up  + DM me if you have other thoughts?


fix ssm exec iam role

SSM Exec tried creating IAM role when you pass a task role with error.
│ Error: "name" must match [\w+=,.@-]
│
│   with module.api.module.service.aws_iam_role_policy.ecs_ssm_exec[0],
│   on .terraform/modules/api.service/main.tf line 200, in resource "aws_iam_role_policy" "ecs_ssm_exec":
│  200:   name   = module.task_label.id

I added simple checks to avoid attempting to create IAM resources. <3

do you need any help with maintaining this module @osterman?


incompatible with Webpack 5.0

Hello! worker-loader is incompatible with a breaking change in Webpack 5.0. Details: https://github.com/webpack/webpack/issues/10341

It seems the support is almost there, see #224.

Fixed in master


CP-17231: Implemented proxy authentication in C# SDK

Signed-off-by: Frederico Mazzone<EMAIL_ADDRESS>

Ths has been reviewed as part of https://github.com/xenserver/xenadmin/pull/1031. Looks ok. Merging.


chore(main): release 2.146.0

:robot: I have created a release beep boop
2.146.0 (2024-10-14)
Features

Hide change email when auth_type !== 'EMAIL' (#4712) (27109fd)
Set base url for segment (#4684) (4e833b8)

Bug Fixes

ci: Anonymous registry pushes attempted for Tribvy databases (#4716) (205988d)
Remove mailer lite (#4705) (1c71905)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v2.146.0
:sunflower:


driverless cat now requires driverless:/driverless-fax: prefix to device uri

Hi,
with 1.28.0, the device uri which is used with driverless cat is expected to have one of two prefixes:
driverless:
driverless-fax

and it doesn't take clean ipp: uri - if the clean ipp uri is supplied, driverless fall backs to list_printers().
Is it expected? IMO driverless cat can be able to work with simple ipp uri. The fix is simple, I can provide it if it is really a bug.

Have seen this one, too. I will fix it soon.


Update landscape.yml for StackRox

Pre-submission checklist:
Please check each of these after submitting your pull request:

[x] Are you only including a repo_url if your project is 100% open source? If so, you need to pick the single best GitHub repository for your project, not a GitHub organization.
[x] Is your project closed source or, if it is open source, does your project have at least 300 GitHub stars?
[x] Have you picked the single best (existing) category for your project?
[N/A] Does it follow the other guidelines from the new entries section?
[x] Have you added your SVG to hosted_logos and referenced it there?
[x] Does your logo clearly state the name of the project/product and follow the other logo guidelines?
[x] Does your project/product name match the text on the logo?
[x] Have you verified that the Crunchbase data for your organization is correct (including headquarters and LinkedIn)?
[ ] ~30 minutes after opening the pull request, the CNCF-Bot will post the URL for your staging server. Have you confirmed that it looks good to you and then added a comment to the PR saying "LGTM"?

LGTM


Worker Time outs

Is your feature request related to a problem? Please describe.
What if I want to only wait, say, 100 seconds per worker
Describe the solution you'd like
After n seconds terminate worker and send err message

Thank you for submitting the issue.  It was in fact an issue with it timing out when an error bubbles up.  We've fixed the workers timeout issue and added an example like the problem you presented.  Let us know if that doesn't work for you.  Thank you again! https://github.com/catmullet/go-workers/commit/b0fef67aaddb05ed6782493cc0f94636649ee77f


Used to print all prime numbers smaller than or equal to n

#1 Sieve of Eratosthenes Used to print all prime numbers smaller than or equal to n

hey , they are not considering dsa,cp and algorithm repo
but you can contribute to this repo as a part of community :)
this will help many students to practice dsa


while creating replica in db i'm seeing gray (create replica)????

hi rakesh, create read replica option was hide because you are not enable backup so first enable the backup and then it automatically unhide. for your reference I am giving one link for how to create a backup for rds, please click on this link and follow.


bug: substr fails to compile (#94)

Closes #94.

I think it's just a breaking change in pandas. Probably need to update that test.

I'm also seeing this error in the CI output. Locally I ran against Ibis 1.2, but looks like maybe the CI is running against 2.0?
ibis_bigquery/client.py:24: in <module>
12
    from ibis.client import Database, Query, SQLClient
13
E   ModuleNotFoundError: No module named 'ibis.client'
14
Error: Process completed with exit code 4.

I'm also seeing this error in the CI output. Locally I ran against Ibis 1.2, but looks like maybe the CI is running against 2.0?

That's correct. I've just updated it to only run against 1.x on the v1 branch. Work is still in progress to support Ibis 2.x.


Feat: Internal Commenting for IDIR Users - (329, 259, 312)

This PR introduces the internal commenting feature, specifically designed for IDIR users to communicate effectively.
Without any comments:

Add a new comment:

Edit comment:

Comment to director from analyst:

Comment to analyst from director:

Other IDIR users can only view comments; they do not have permission to add new comments.

Closes #329, #259, #312

There are some optimizations on the backend we could make. The frontend looks great thank you.

Thank you, Alex, for your feedback. I've applied most of it but I'm having trouble with the unit tests for the backend. Specifically, when I run each test individually, like this:
pytest lcfs/tests/internal_comment/test_internal_comment_repo.py::test_update_internal_comment_success

it passes. But, running all tests together causes some to fail because of rollback issues:
pytest lcfs/tests/internal_comment/test_internal_comment_repo.py

I've tried to fix this without success.
Also, I'm stuck on how to handle the new commit_to_db() method.
I'd really appreciate your help with these problems.

The code changes look good. I'm going to merge this and then we can tackle the pytest fixes separately.


After clicking on refresh Development Stats, the date is not changed

closed. Stats are now refreshed daily