Add gosec to the static tests

Describe what this PR does
gosec is a security scanning tool that can be used similar to go vet for standard security concerns. This PR addresses the reported issues and adds gosec to the standard tests.

@humblec would you like to include this enhancement?

@humblec would you like to include this enhancement?

@nixpanic Why not? lets try to get it in :) . Can you please check why the CI fails?

The failure in CI looks unrelated:
Aug 30 11:15:58.366: INFO: stderr: ""
Aug 30 11:15:58.366: INFO: stdout: "cephblockpool.ceph.rook.io/replicapool created\n"
Aug 30 11:15:58.375: INFO: ExecWithOptions {Command:[/bin/sh -c ceph fsid] Namespace:rook-ceph PodName:rook-ceph-tools-76c7d559b6-7cfds ContainerName:rook-ceph-tools Stdin:<nil> CaptureStdout:true CaptureStderr:true PreserveWhitespace:true}
Aug 30 11:15:58.375: INFO: >>> kubeConfig: /home/travis/.kube/config
Aug 30 11:15:59.351: INFO: stdErr occurred
%!(EXTRA string=unable to get monitor info from DNS SRV with service name: ceph-mon
[errno 2] error connecting to the cluster
)[AfterEach] RBD

This is the same error that @Madhu-1 hit in #580.

now we have  one more issue in CI
Total download size: 72 M
Downloading packages:
Delta RPMs disabled because /usr/bin/applydeltarpm not installed.
http://download.ceph.com/rpm-nautilus/el7/x86_64/librados2-14.2.3-0.el7.x86_64.rpm: [Errno 14] HTTP Error 404 - Not Found
Trying other mirror.
To address this issue please refer to the below wiki article
https://wiki.centos.org/yum-errors
If above article doesn't help to resolve this issue please use https://bugs.centos.org/.
Error downloading packages:
  2:librados2-14.2.3-0.el7.x86_64: [Errno 256] No more mirrors to try.
The command '/bin/sh -c rm -f /etc/yum.repos.d/ceph-iscsi.repo && yum -y update && yum clean all' returned a non-zero code: 1
Makefile:58: recipe for target 'image-cephcsi' failed
make: *** [image-cephcsi] Error 1


Improve/restore intended parallax example layout

Picture must appear behind the semi-transparent header. The header initially appears under the image on iOS until the device is rotated for the first time.
Content overlaps the header when scrolling. It must stay behind the header.
Hide the UI toolbar (optional)

Fixed in 4870e4d7760e4f32589d555a19607115a7e89550.


[WIP] XCAF-765 build: update to angular 6

Tested this myself locally, looks really good

Checked this locally and it looks good to me, some numbers I saw:
Before:
ionic-split-pane.umd.js = 207kb (no sourcemaps embedded)
ionic-split-pane.umd.min.js = 63kb
After:
ionic-split-pane.umd.js = 592kb (sourcemaps embedded as well as external one?)
ionic-split-pane.umd.min.js = 55kb


Adding Collection View instead of reviews image

I really like the idea of having 'Cells' in the popup view instead of the static image, but when i add the collection view as a subview of the popup view, I lose the ability to scroll and tap on the collection view cells. Is there a way I can have the pan gesture on the popup view as well as being able to interact with the collection view?
Heres how I add the collection view: (not subclassed)
private lazy var collectionView: UICollectionView = {
        let layout = UICollectionViewFlowLayout()
        layout.itemSize = CGSize(width: 60, height: 30)
        let view = UICollectionView(frame: (CGRect(x: 0, y: 0, width: self.popupView.frame.size.width - 10, height: self.popupView.frame.height)), collectionViewLayout:layout)
        view.backgroundColor = CustomColors.blue
        view.isUserInteractionEnabled = true
        view.register(CollectionViewCell.self, forCellWithReuseIdentifier: "CollectionViewCell")
        return view
    }()

    //MARK: - VC Lifecycle
    override func viewDidLoad() {
        super.viewDidLoad()
        collectionView.delegate = self
        collectionView.dataSource = self
        layout()
        popupView.addGestureRecognizer(panRecognizer)
    }

Any updates?


ttnn.hypot_bw low PCC

Describe the bug
ttnn.hypot_bw fails with low PCC when input_tensor_a and/or input_tensor_b has bfloat8_b dtype
To Reproduce
Steps to reproduce the behavior:
Sweep test for hypot_bw is located in 'tests/sweep_framework/sweeps/eltwise/binary_backward/hypot_bw/hypot_bw.py'

Go to 'tests/sweep_framework/sweeps/eltwise/binary_backward/hypot_bw/hypot_bw.py'
Generate new parameter vectors and run the sweep test

python3 tests/sweep_framework/sweeps_parameter_generator.py --elastic cloud --module-name eltwise.binary_backward.hypot_bw.hypot_bw
python3 tests/sweep_framework/sweeps_runner.py --elastic cloud --module-name eltwise.binary_backward.hypot_bw.hypot_bw --suite-name xfail

See the error. Results can be found on elastic cloud as explained here: https://github.com/tenstorrent/tt-metal/tree/main/tests/sweep_framework

ttnn.hypot_bw uses ttnn.reciprocal in its implementation which has this issue #14672
This is causing the PCC drop when using bfloat8_b

Hi @amalbasaTT ,
When using bfloat8_b dtype, could you try with non-zero inputs to see if these cases pass ?
You can find more information on this here https://github.com/tenstorrent/tt-metal/issues/14672
When involving bfloat8_b and ttnn.reciprocal related ops we need separate implementations for zero- and non-zero inputs.
Handling these at kernel level will make the op slower, so we'll discuss what the best approach will be - whether to do it at composite level. For now could we test on non-zero inputs to identify any other issues other than this.

@KalaivaniMCW ,
Of course, no problem. Do you want a unit test or a sweep test for each of ttnn.reciprocal related ops? Also is there a list of ttnn.reciprocal related ops somehwere?

For now, the below issues seems to be related.
If these op sweeps pass for non-zero inputs, we can rule out these to be related to bfloat8_b behaviour.
#13776
#13937
#13975

ttnn.hypot passes in all cases when all inputs (input_tensor_a and input_tensor_b) are with non-zero elements and with bfloat8_b datatype
ttnn.rdiv_bw (issue) passes in 4.167% cases when all inputs (grad_tensor, input_tensor_a and factor) are with non-zero elements and when grad_tensor and input_tensor_a have bfloat8_b datatype

Do you want torch inputs to exclude only 0.0, or do you also want to exclude some range of very numbers which are close to 0 (like -0.01 to 0.01)?

I have tested rdiv_bw, hypot_bw and div_bw with inputs outside of range [-1, 1]. Before running the ops, i have converted inputs to ttnn tensor with dtype bfloat8_b and back to torch before running ops with them, so that i could ensure that all the inputs are non-zero after any eventual cut-off when converting them to ttnn tensor with bfloat8_b. Issue i found is that when input_tensor has bfloat8_b and "certain" shape, it produces an all-zero tensor (or tensors for binary ops). You can check details in issues i found bellow, i have provided a unit test and a sweep test for each op:
https://github.com/tenstorrent/tt-metal/issues/16305
https://github.com/tenstorrent/tt-metal/issues/16304
https://github.com/tenstorrent/tt-metal/issues/16306


Still active when holding Item

Hello, although it has been several months, I've been messing around with this. I have noticed an issue when holding an object, the Grabbity seems to be focusing still on different objects when one is already being held. Throwing an object is very annoying as one tends to fly at you at the same time, I have a video demonstrating.
https://www.youtube.com/watch?v=HzU-JMMDiFE
The fixing of this issue would be great, if you need my messy ass project file to look further I would be glad to provide that as well.

Hi @Saladope.
I'm not currently maintaining this project, but if you want to fix it yourself, I'm happy to accept pull requests.
The code in question is in the selector class
https://github.com/provencher/XR-Interaction/blob/master/Assets/XR_Interaction/Scripts/Grabbity/GrabbitySelector.cs
private void Selection() is where you want to look.
Basically, you'll need to add a check on whether the gripping hand is already holding an object. My code sits alongside the XR interaction toolkit, which is why this is allowed to happen. If the hand in question is gripping, you'll have to block the selection function from continuing to run.
Wish you luck with your project!


DDP Spawn can only test a child process error

** To reproduce
Steps to reproduce the behavior:
Produces a traceback in DDP spawn (cpu only), where workers crash (still trains fine)
from composer.trainer import TrainerHparams, Trainer

hparams = TrainerHparams.create('composer/yamls/models/classify_mnist_cpu.yaml')
hparams.set_datadir("~/datasets")
trainer = Trainer.create_from_hparams(hparams)

trainer.fit()

/home/avery/mosaic/composer/composer/utils/ddp.py:20: UserWarning: DDPDefaultValueWarning: RANK env var not set and process group not initialized; returning 0 for global rank.
  warnings.warn(f"DDPDefaultValueWarning: {env_var} env var not set"
Epoch 1:   3%|▎         | 1/29 [00:00<00:21,  1.32it/s]                                                                                /home/avery/mosaic/composer/composer/utils/ddp.py:20: UserWarning: DDPDefaultValueWarning: WORLD_SIZE env var not set and process group not initialized; returning 1 for world size.
  warnings.warn(f"DDPDefaultValueWarning: {env_var} env var not set"
Epoch 1:   3%|▎         | 1/29 [00:00<00:21,  1.32it/s, loss/train=2.3191]                                                                   Exception ignored in: <function _MultiProcessingDataLoaderIter.__del__ at 0x7ff33771e5e0>
Traceback (most recent call last):
  File "/home/avery/.local/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 1328, in __del__
    self._shutdown_workers()
  File "/home/avery/.local/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 1320, in _shutdown_workers
    if w.is_alive():
  File "/usr/lib/python3.8/multiprocessing/process.py", line 160, in is_alive
    assert self._parent_pid == os.getpid(), 'can only test a child process'
AssertionError: can only test a child process
Exception ignored in: <function _MultiProcessingDataLoaderIter.__del__ at 0x7ff33771e5e0>
Traceback (most recent call last):
  File "/home/avery/.local/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 1328, in __del__
    self._shutdown_workers()
  File "/home/avery/.local/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 1320, in _shutdown_workers
    if w.is_alive():
  File "/usr/lib/python3.8/multiprocessing/process.py", line 160, in is_alive
    assert self._parent_pid == os.getpid(), 'can only test a child process'
AssertionError: can only test a child process

Is this in an interactive shell or jupyter?

Closing as we haven't been seeing this...please re-open if it comes back.

Closing as we haven't been seeing this...please re-open if it comes back.


feat(ExchangeClient): Add market open and close methods

Description
This PR introduces market_open and market_close methods to the ExchangeClient, aligning the Rust SDK's functionality with the existing Hyperliquid Python SDK.
Key Changes

Added market_open method for placing market buy/sell orders
Added market_close method for closing positions with market orders
Implemented price slippage calculation with respect to asset-specific decimal requirements
Added helper functions for rounding to significant figures and decimal places

Implementation Details

Mirrors the functionality of the Python SDK's market_open and market_close methods
Uses the existing InfoClient to fetch asset metadata and current mid-prices
Simulates market orders using IOC (Immediate-or-Cancel) limit orders
Respects Hyperliquid's tick size and lot size rules for each asset

Usage Example
let market_open_params = MarketOrderParams {
    asset: "ETH",
    is_buy: true,
    sz: 0.01,
    px: None,
    slippage: Some(0.01),
    cloid: None,
    wallet: None,
};

let response = exchange_client.market_open(market_open_params).await?;

Testing

Manually tested against Hyperliquid testnet
Verified behavior matches Python SDK output for equivalent inputs

Related Issues
Closes #44

Uodated debug format @lmlmt :)

Uodated debug format @lmlmt :)

Thanks for detailing everything so extensively!


[UWP] Using the navigation buttons on a mouse while the cursor is over a toucheffect control will throw a null reference exception

On UWP, you can have navigation buttons on your mouse.
This means that while you press them, it'll trigger the pressed events, which will navigate to a different page but also attempt to do things like animate the background color of the affected control.
However, as the app has since navigated to a different page, the control it is trying to act upon no longer exists and a null reference exception is thrown at
TouchVisualManager.cs line 195
 new Animation{
                {0, 1,  new Animation(v => sender.Control.BackgroundColor = new Color(v, sender.Control.BackgroundColor.G, sender.Control.BackgroundColor.B, sender.Control.BackgroundColor.A), sender.Control.BackgroundColor.R, color.R) },
                {0, 1,  new Animation(v => sender.Control.BackgroundColor = new Color(sender.Control.BackgroundColor.R, v, sender.Control.BackgroundColor.B, sender.Control.BackgroundColor.A), sender.Control.BackgroundColor.G, color.G) },
                {0, 1,  new Animation(v => sender.Control.BackgroundColor = new Color(sender.Control.BackgroundColor.R, sender.Control.BackgroundColor.G, v, sender.Control.BackgroundColor.A), sender.Control.BackgroundColor.B, color.B) },
                {0, 1,  new Animation(v => sender.Control.BackgroundColor = new Color(sender.Control.BackgroundColor.R, sender.Control.BackgroundColor.G, sender.Control.BackgroundColor.B, v), sender.Control.BackgroundColor.A, color.A) },
            }.Commit(sender.Control, ChangeBackgroundColorAnimationName, 16, (uint)duration, easing, (d, b) => animationCompletionSource.SetResult(true));

sender.Control is null so it'll fail.

@MitchBomcanhao your are best QA haha)

@MitchBomcanhao https://github.com/AndreiMisiukevich/TouchEffect/blob/master/TouchEffect/TouchVisualManager.cs#L343
Adding
if(sender.Control == null)
{
return;
}
Should solve this issue.
Would you like to make extra PR (test it before please)

It still fails on the same spot.
goes up to:

then fails

@MitchBomcanhao got it, thanks

Need to refactor code a little

@MitchBomcanhao does it work?)

Yes, first impression is that it is working fine ;)


Add travis build status image to README

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

The build failure isn't related to this change. Merging.


[e2e]: generic push main default slsa3

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/3301904717
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.push.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.push.main.default.slsa3.yml
Trigger: push
Branch: main
Date: Sat Oct 22 04:29:00 UTC 2022

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/3305779614
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.push.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.push.main.default.slsa3.yml
Trigger: push
Branch: main
Date: Sun Oct 23 04:42:15 UTC 2022
Tests are passing now. Closing this issue.


Set requestedHostname in nodesyncer

related issue: https://github.com/rancher/rancher/issues/11532

LGTM.


#791: make ISwaggerProvider singleton

I see you have a CI, let me just test my theory ...
If this doesn't break anything, I'll try to add a unit test for my case.

Your theory is correct - however failure when using transient lifetime is a symptom rather than a cause. See #629 for more details


TableViewController not being added to child view controllers

See: https://developer.apple.com/library/archive/featuredarticles/ViewControllerPGforiPhoneOS/ImplementingaContainerViewController.html

Yes, Currently I don't add TableViewController as child controller I only add tableView as a subview.
....


Setup Cosmos DB and serverless functions to connect to it

Status: Joey is currently researching this!
Resources
Azure Cosmos DB Serverless
TODO

[ ] Research how to implement
[ ] Create a diagram

We didn't get to this but the idea was to use Cosmos DB to manage the state of the IoT network.


Extended tty definition to support for debug.js

debug.js is a package used by many libraries out there and it depends on tty library. Following is a stub implementation of the only tty call needed by debug.js.

Thank you very much!


Feature request: Argmax function should return an array if multiple entries with max value exist

Correct me if I am wrong, but currently it is quite a hassle to find a way in a network that should work in hybrid mode (so no asnumpy()) to get ALL indices where entries with maximum values exist.
However, there are quite a lot of functions that use list of indices, e.g. take(). But there are not many functions that will return list of indices, so functions like take() are often harder to use than it should be.
So, feature request is to make argmax return a list of indices, like the argmax function in numpy or pytorch. This shouldn't slow down argmax as it doesnt change the time complexity.
#6705 is basically the same problem. Maybe it would be possible to implement this request with #11061

@mxnet-label-bot add [feature]


esp_netif_destroy is not working (IDFGH-4335)

If a user didn't sent correct SSID and PW in first attempt, he resends the credentials and "wifi_init_sta(void)" is executed again.
But the second time core panics. Reason most probably is that the first instance of "esp_netif" is not destroyed.
I know "esp_netif_init()" should be fired only once and if firing second time, it should be destroyed first. But question is how to destroy :?: :?:
Bellow is my code which is not working
esp_netif_t *my_esp_netif;
'
'
'
'
'
if(my_esp_netif){
        esp_netif_destroy(my_esp_netif);
      }

   my_esp_netif= esp_netif_init();
   ESP_ERROR_CHECK(esp_event_loop_create_default()); //     This is the line at which core panics.
    esp_netif_create_default_wifi_sta();
    wifi_init_config_t cfg = WIFI_INIT_CONFIG_DEFAULT();
    ESP_ERROR_CHECK(esp_wifi_init(&cfg));

Hi @Zeni241
Yes, correct, the API esp_netif_init() should be called only once, to initialize the TCPIP stack.
Then, the interface itself is created as a new instance using esp_netif_new().
This instance, a pointer to esp_netif_t object, could be then passed to the API  esp_netif_destroy().
Please check the ESP-NETIF programmer's manual for more details.
In case of these Wifi examples, another API is used to setup and create a default wifi network interfaces:
esp_netif_create_default_wifi_sta();

which also returns a netif pointer (on success), too, but some examples just ignore it: https://github.com/espressif/esp-idf/blob/178b122/examples/wifi/getting_started/station/main/station_example_main.c#L73
so you can update that code to
    esp_netif_init();
     ....
    esp_netif_t *my_esp_netif = esp_netif_create_default_wifi_sta();
     ....
    esp_netif_destroy(my_esp_netif);

to correctly destoy the created netif

@david-cermak thanks a lot for your reply.
 But if I call void wifi_init_sta(void) second time, the core panic.

@Zeni241 Please check the WiFi library documentation, as well:
https://docs.espressif.com/projects/esp-idf/en/latest/esp32/api-reference/network/esp_wifi.html#api-reference
The same principle applies, please deinit the wifi before calling the init again with esp_wifi_deinit()

@david-cermak thanks again. I will go through it thoroughly as soon as get the time. For the time being I have improvised my code as follows and its working fine.
int wifiCnt=0;
'
'
'
'
'
void wifi_init_sta(void)
{
f(wifiCnt==0){
    ESP_ERROR_CHECK(esp_netif_init());
    ESP_ERROR_CHECK(esp_event_loop_create_default());
    esp_netif_create_default_wifi_sta();
    wifiCnt=1;
}

    wifi_init_config_t cfg = WIFI_INIT_CONFIG_DEFAULT();
    ESP_ERROR_CHECK(esp_wifi_init(&cfg));
    ESP_ERROR_CHECK(esp_event_handler_register(WIFI_EVENT, ESP_EVENT_ANY_ID, &wifi_event_handler, NULL));
    ESP_ERROR_CHECK(esp_event_handler_register(IP_EVENT, IP_EVENT_STA_GOT_IP, &wifi_event_handler, NULL));
'
'
'
'
}

Please let me know if you see any pitfall in above code.

@Zeni241 There's no problem in initializing certain module only once and reusing it. Moreover it's generally a preferable approach  in embedded environments rather than creating and deleting the objects.
Could you please close the issue, since the esp_netif_destroy() apparently works as expected. Also please use the ESP32-Forum for general questions like this one.


RFC: Add parts_mut() -> &mut Parts to allow mutate parts directly

Hello, I plan to propose an RFC to add parts_mut() -> &mut Parts to enable direct mutation of Parts. We already have body_mut, headers_mut, etc., but we lack a function that allows us to simultaneously update headers, methods, and queries.
This change will will allos us to:
- let (mut parts, body) = req.into_parts();
- do_my_work(&mut parts);
- let req = Request::from_parts(parts, body);
+ do_my_work(req.parts_mut())

This was brought up before: https://github.com/hyperium/http/issues/372#issuecomment-563548370 and the comments afterwards.


Connection reject by server

Hi,
I've an error when I connect to my OPC Server.
Error: The connection has been rejected by server, Please check that client certificate is trusted by server. Err = (connect ETIMEDOUT xxx.xxx.xxx.xxx:4840)     at Object.start (/snapshot/node-opcua-logger/src/opcua.js:0:0)
I use anonymous auth.
I try with UA Expert Client and it's ok.
What's the certificate?
Regards

I'm having a similar issue with anonymous auth and I'm getting back an [FATAL] main - Error: Invalid server Nonce which should not be there as that check has to be skipped in case of anonymous auth.
All other OPC-UA clients are working fine.

Even with anonymous auth sometimes the server requires the certificate to be accepted. With UA server are you using?

Hello,
I'm using a server from a Schneider Electric PLC. UA expert and a lot of other clients are able to connect flawlessly but the only one that is reporting the invalid nonce is this one, which we like and need so badly ;)
Anyway I also opened a ticket to the node-opcua library https://github.com/node-opcua/node-opcua/issues/904#issuecomment-720960288
Thanks!

Hi,
I am also facing similar issue when I try to run the code. Need help. But when I tried running influx-opcua-logger_v2.0.0-alpha_win-x64.exe, its working fine and I am getting the response.
[2020-11-04T12:06:36.561] [INFO] main - Starting Influx OPCUA Logger v2.0.0-alpha, brought to you by FACTRY (www.factry.io)
[2020-11-04T12:06:36.562] [INFO] main - Initialising influxClient
[2020-11-04T12:06:36.571] [INFO] influx - localhost:8086 responded in 6ms running 1.7.6
[2020-11-04T12:06:36.572] [INFO] main - Initialising buffer
[2020-11-04T12:06:36.574] [INFO] main - Connecting OPCUA
[2020-11-04T12:06:36.604] [FATAL] main - Error: The connection has been rejected by server,
Please check that client certificate is trusted by server.
Err = (Internal Error)
at Object.start (D:\SPIA\opcua\node-opcua-logger-2.0.0-alpha\src\opcua.js:60:11)
[2020-11-04T12:06:36.605] [INFO] buffer - Persisting membuffer.

Hello,
I'm using a server from a Schneider Electric PLC. UA expert and a lot of other clients are able to connect flawlessly but the only one that is reporting the invalid nonce is this one, which we like and need so badly ;)
Anyway I also opened a ticket to the node-opcua library node-opcua/node-opcua#904 (comment)
Thanks!

As regards my request, this has been fixed by  node-opcua/node-opcua#904 (comment).
Thanks

Hello,
I'm using a server from a Schneider Electric PLC. UA expert and a lot of other clients are able to connect flawlessly but the only one that is reporting the invalid nonce is this one, which we like and need so badly ;)
Anyway I also opened a ticket to the node-opcua library node-opcua/node-opcua#904 (comment)
Thanks!

As regards my request, this has been fixed by node-opcua/node-opcua#904 (comment).
Thanks

Thanks for the help. The code is working fine after I upgraded the opcua client.


Apply cleanup formatting to descriptions

See: https://www.wikidata.org/wiki/User_talk:ProteinBoxBot#Descriptions

Looking for descriptions starting with "a ..", I came across a few generated by this bot (sample correction). For future runs, maybe the initial "a", underscores in text and final dots can be removed directly.
--- Jura 14:16, 7 September 2017 (UTC)

https://www.wikidata.org/wiki/Help:Description

See test edits:
https://www.wikidata.org/w/index.php?title=Q18553225&diff=prev&oldid=571519288
https://www.wikidata.org/w/index.php?title=Q4347774&diff=prev&oldid=571519291
Not sure what to do with multi-line descriptions (like: https://www.wikidata.org/w/index.php?title=Q4690047&oldid=571518916), so they are being ignored.


Fix race condition that resets environment setting to default (#6083)

Pull request opened by the merge tool on behalf of #6083

Merged into branches iso6 in 376373f6649abf823d87d2840441ec860d0db969


VSExtension: integrate into cmake build system.

The build failure is probably due to a configuration error on the build agent machine. The logs complain about not finding VsSdk.

It properly builds now.


Proposal: break out of infinite loops using a hit counter

Is your feature request related to a problem? Please describe.
Stryker might be mutating your code to be an infinite loop. There is no way around that, because of something called the Halting problem. For example:
let goOn = true;
let i = 0;
while(goOn) {
  if(activeMutant(1)) {
    i--;   // 👈 mutant that creates an infinite loop
  }
  else {
    i++;
  }
  if(i >10){
    goOn = false;
  }
}

Currently, StrykerJS's way to handle infinite loops is to kill the worker process and start a new one. This works, but it can be very expensive, especially when webpack or other JIT build steps are involved. Take an Angular project for example. Having a new test runner worker process take a minute to start is quite 'normal' for larger projects (besides the fact that a new browser is started as well).
Describe the solution you'd like
When StrykerJS is instrumenting the code, we could add a hit counter that counts the number of times that mutant is executed. If an absolute limit is reached, we can set a "hitCountLimitReached" flag and throw an error. Then in the test runner, we can check for the presence of the flag and report the mutant as a timeout result.
Pseudo code:
function activeMutant(id) {
  if (id === global.__stryker__.activeMutant) {
    global.__stryker__.hitCount++;
    if (global.__stryker__.hitCount >= 1000) {
      global.__stryker__.hitCountLimitReached = true;
      throw new Error('Mutant hit count reached');
    }
    return true;
  }
  return false;
}

This works sometimes, but has some disadvantages:

It is difficult to determine a universal "hard limit". For some tasks (image processing, performance tests, etc) a hard limit of 1000 would be far too low. I think therefore this limit should be configurable: --mutantHitLimit?  Setting this value to 0 could mean turning the hit limit off.
The error might not be handled properly by the test. It might result in an unhandled exception or unhandled promise rejection. We might want to think about adding handlers for those, or investigate if each test runner properly handles that already.
The error might be caught by your code. This can have unexpected consequences

If the error is caught inside the loop, the loop won't be broken and remain infinite.
If the error is caught outside of the loop, where a different error was expected to be handled. This probably will result in a failing test (which should be converted to a timeout by the test runner), but still: unexpected behavior.

Test runners will get more complicated (although we could move some of this code to @stryker-mutator/utils):

decide when there is a timeout based on the global "hitCountLimitReached" flag.
Reset both the hitCounter and the hitCountLimitReached variables. If we forget to reset, false negatives will occur.

As you can see, it can only ever be a best-effort approach and Stryker will always need its current behavior to handle infinite loops in addition.
Given all these disadvantages, we might want to first default it to be turned off, until we're happy with the results and turn it on by default in a major release.
Even so, there are still timeouts that are unhandled by this implementation, especially with regards to using callbacks in your test logic. For example:
// foo.js
function maybeEmitFooEvent(a, b) {
  if (activeMutant(1) ? a > b : a < b) {
     emitter.emit('foo');
  }
}

// foo.spec.js
it('should raise event', (done) => {
  emitter.on('foo', done);
  maybeEmitFooEvent(1, 2);
});

Describe alternatives you've considered
There is no real alternative that I can think of, other than what we're doing now.
Additional context
Karma has difficulties with infinite loops: https://github.com/stryker-mutator/stryker-js/issues/2989#issuecomment-885461482

@Garethp What do you think of this? I know we had multiple discussions about infinite loops in the past.

I've been meaning to implement this in pitest for years. Rather than a fixed hard limit I'd always planned to record the number of hits each test made to the method in its unmutated state, then error out if the mutate probe hit the code 'x' times more than that for the current test.

Ohhh I like that approach! More work to do for the test runner plugin (set the limit), but that is a relatively small addition. It might remove the need for a configuration option altogether, or can you still think of a reason to disable the feature?
@hcoles why didn't you implement the feature yet? Is byte code mutation limiting you in some way?

Not a byte code issue, just quite a lot of work given how the code has evolved. I'd need a solid week or two to sit down and look at it, which I've never had.

I think it's not a bad idea and calculating the number of hits per test is probably a way safer way to do things IMO. I'm not sure that making the number of hits user configurable is all that valuable since it requires the user to know what a good amount is. I think the concerns about where the error is caught is valid, maybe it would be better for it to send a signal up to it's thread parent indicating an infinite loop so the thread can be killed and marked that way?
Personally I'm still a fan of the heartbeat idea we talked about as a long term solution to this

I'm not sure that making the number of hits user configurable is all that valuable since it requires the user to know what a good amount is.

We could still allow users to configure a factor. For example, a factor of 2 would mean twice as many times as in the dry run. However, I agree that we shouldn't want to make that configurable, since it should not be needed. We can make the internal factor a high number to lean on the safe side (for example, a factor of 100).'

I think the concerns about where the error is caught are valid, maybe it would be better for it to send a signal up to its thread parent indicating an infinite loop so the thread can be killed and marked that way?

Hmm, interesting. However, tests might run inside the browser (karma), or inside a VM (jest). So this implementation should be test-runner dependent.

maybe it would be better for it to send a signal up to its thread parent indicating an infinite loop so the thread can be killed and marked that way?

What do you mean by "thread parent" and "the thread can be killed"? JS is single-threaded. Test runner processes generally run tests in their thread, except for karma, which runs the tests inside the browser (also single-threaded). The only way to recover from this, without breaking out of the loop, would be to kill the test runner worker and start a new process. With the sending of the "signal" as you put it, we could kill and restart sooner than we normally would, because we wouldn't have to wait for the timeout timer to expire, but it would still be expensive for JIT heavy projects.
Therefore, I prefer the "throw Error" approach. It can be baked into the instrumenter, so less work for the test runner. A mutant run would look like this in the test runner:
class FooTestRunner {
  async runMutant(options) {
    // Prepare hit counter stuff
    global.__stryker__.hitCount = 0;
    global.__stryker__.hitCountLimitReached = false;
    global.__stryker__.hitCountLimit = options.hitCountLimit;

     // Switch active mutant
    global.__stryker__.activeMutant = options.activeMutant.id;

    // Do the actual test run
    const result = await underlyingTestRunner.run();

    // See if it was a timeout
    if (global.__stryker__.hitCountLimitReached) {
      return { state: 'timeout' };
    } else {
      return result
    }
  }
}

The nice thing is that the test-runner could choose to ignore the hit counter stuff entirely. If a test runner doesn't implement it, we would still be able to handle timeouts as we now do.

That looks really good. When we were profiling stryker, we have already noticed that restarting processes is quite expensive so it might really help in performance. I believe it might also help with stability, since restarting childProcesses could theoretically mess something in runs :)

This feature just got released for Stryker 5.3 (karma runner only) please give it a try


Option to set the step/stride in reduce_axis

@Huyuwei and I found that it may be useful to support step/stride, the distance between two adjacent values, in reduce axis (for dilation). It could help skip step number of elements for the reduce axis, instead of branching.

Would this be equivalent to (stride =2)
tvm.sum(A[k*2], k=tvm.reduce_axis((0, n/2))

Yes. It is equivalent to modify the compute in this way. Thank you!


chore: update v4 core

Related Issue
Which issue does this pull request resolve?
See title
Description of changes
See title

@saucepoint do you mind help merging? thanks


Python for scripting of Windows

Thanks for putting this useful resource together.
For beginners, I think they could benefit from some important information around PowerShell, and the exposure of APIs via Managed DLLs.
For example, many of the system configuration options in the Windows platform are only available via managed DLLs, using a PowerShell CmdLet or using the native management tools.
If the users were coming at this from a *nix background they might assume that they can automate the OS using flat-file configuration and shell commands, which they can't.
If someone wanted to setup a web application, they would use the IIS cmdlets, not edit the IIS XML files by hand (I hope!) https://docs.microsoft.com/en-us/powershell/module/iisadministration/?view=win10-ps
The examples in the page are great, but perhaps a caveat a the top about would save people a lot of time in future.
Thanks,

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

ID: 5009df67-94f4-00b3-3ffe-67801622f152
Version Independent ID: 35d84635-3e48-afe0-d2ae-79202497d64a
Content: Scripting and automation with Python on Windows - Windows apps
Content Source: hub/python/get-started/python-for-scripting.md
Product: windows
Technology: hub
GitHub Login: @mattwojo
Microsoft Alias: mattwoj

Thanks Tony!  Appreciate the feedback!
Do you think it would be more helpful if we set the context at the beginning of the article that we're not really doing special Windows-specific stuff... rather just pointing out some of the helpful libraries in Python that can automate tasks across platforms... like searching your file system, accessing the internet, parsing file types, etc... but from a Windows-centered approach?
Maybe with a link at the end to https://docs.python.org/library/ctypes.html and https://docs.python.org/library/winreg.html for more Windows-specific operations?
...and to https://pypi.org/project/winrt/ for Windows Runtime API calls?
Is that the sort of caveat that you mean?

Yes, this is exactly what I mean. thanks.

Took me a while to get to it (sorry--got focused on a new project), but finally got a note about this is in: https://github.com/MicrosoftDocs/windows-uwp-pr/pull/342
Thanks again for your feedback Tony! Closing this out.


Some props are not working

Hi,
borderWidth
buttonInnerColor
buttonOuterColor
buttonOuterSize
those props are not working.

Yes, these options are not working.I was struggling with this.
Please provide some updates.

Same here.Can anyone set props wrapStyle?

since last year and no body update on this?

Same here.Can anyone set props wrapStyle?

you can use the "style" props instead


How can we get the FLOPs of one model? and How can we test model's inferring speed?

Dear friends, thanks for the nice codebase. Are there scripts for the above questions? thanks

To test inference speed, you can use benchmark.py. For now, we do not support calculating flops since usually, the input of models varies a lot. We will plan to calculate flops in the future.

Thanks so much.


add rule (vue-caution): Implicit parent-child communication

https://vuejs.org/style-guide/rules-use-with-caution.html#implicit-parent-child-communication

For this rule would be enough to check for props mutation (prop inside v-model) and check no $parent, getCurrentInstance are present in the file? 🤔

Yes, it will cover most of the problematic parts.

Hi @rrd108 I created a PR for this issue #67


add 'object' type to definitions.queryParameterSubSchema.properties.type.enum

the reason for this is to expose the mongodb readonly query api (see working screenshot, with hack applied to swagger-ui client)

this will also bring it more inline with http://json-schema.org/draft-04/schema#/properties/enum
        "simpleTypes": {
            "enum": [ "array", "boolean", "integer", "null", "number", "object", "string" ]

I promise you I'm not trying to be mean but there are a number of problems with this pull request.  Below is the list of issues I have, here purely for learning purposes and not to be a jerk:

Swagger does not adhere to JSON Schema exactly so the chosen list of primitives in swagger-tools were driven by Swagger:

Swagger 2.0 JSON Schema
Swagger 1.2 JSON Schema

You modified generated resources instead of the sources themselves (All files in the browser directory are automatically generated during the build process.  If you really wanted to change the primitives, you'd need to change lib/specs.js and that change would result in failed tests which would also need to be changed.)
You didn't run tests (I'm sure of this because we rebuild the browser files during this phase)
I'm not sure how swagger-tools is in the picture here because swagger-ui uses swagger-js, not swagger-tools

swagger-tools' primitives are driven by the Swagger specification so I don't think it makes sense to change them.  I'm also not finding how changing this in any way impacts swagger-ui or would fix an issue in swagger-ui since they are unrelated.  I would love to help you solve this problem but I'm not sure it's in this project.  I've done a ton of work recently for swagger-js that properly supports the display of arrays, inline models and references so the problem you're trying to solve may already be fixed.

ok, i added a pull request to swagger-ui https://github.com/swagger-api/swagger-ui/pull/1077 that fixes the frontend.
the problem is that the schema used (attached below), will fail swagger editor validation without this patch. i'm using swagger-tools currently to validate my schema, and i think 'object' type should be allowed

i might resubmit this pull request later on applied to the specs and with the tests fixed
{
    "basePath": "/api/v0.1",
    "definitions": {
        "ContentDraft": {
            "allOf": [{
                "$ref": "#/definitions/JsonApiResource"
            }],
            "properties": {
                "content": {
                    "type": "string"
                },
                "summary": {
                    "type": "string"
                },
                "title": {
                    "type": "string"
                }
            }
        },
        "JsonApiError": {
            "properties": {
                "code": {
                    "type": "string"
                },
                "detail": {
                    "type": "string"
                },
                "href": {
                    "type": "string"
                },
                "id": {
                    "type": "string"
                },
                "links": {
                    "items": {
                        "type": "string"
                    },
                    "type": "array"
                },
                "paths": {
                    "items": {
                        "type": "string"
                    },
                    "type": "array"
                },
                "status": {
                    "type": "integer"
                },
                "title": {
                    "type": "string"
                }
            }
        },
        "JsonApiLinks": {
            "properties": {
                "related": {
                    "type": "string"
                },
                "self": {
                    "type": "string"
                }
            }
        },
        "JsonApiMeta": {
            "properties": {}
        },
        "JsonApiResource": {
            "properties": {
                "_timeCreated": {
                    "format": "date-time",
                    "type": "string"
                },
                "_timeModified": {
                    "format": "date-time",
                    "type": "string"
                },
                "aliasOf": {
                    "type": "string"
                },
                "aliasType": {
                    "type": "string"
                },
                "id": {
                    "type": "string"
                },
                "parentId": {
                    "type": "string"
                },
                "subtype": {
                    "type": "string"
                },
                "type": {
                    "type": "string"
                }
            }
        },
        "JsonApiResponseData": {
            "properties": {
                "data": {
                    "items": {
                        "$ref": "#/definitions/JsonApiResource"
                    },
                    "type": "array"
                },
                "included": {
                    "items": {
                        "$ref": "#/definitions/JsonApiResource"
                    },
                    "type": "array"
                },
                "links": {
                    "$ref": "#/definitions/JsonApiLinks"
                },
                "meta": {
                    "$ref": "#/definitions/JsonApiMeta"
                }
            }
        },
        "JsonApiResponseDataContentDraft": {
            "allOf": [{
                "$ref": "#/definitions/JsonApiResponseData"
            }],
            "properties": {
                "data": {
                    "items": {
                        "$ref": "#/definitions/ContentDraft"
                    },
                    "type": "array"
                }
            }
        },
        "JsonApiResponseError": {
            "properties": {
                "errors": {
                    "items": {
                        "$ref": "#/definitions/JsonApiError"
                    },
                    "type": "array"
                }
            }
        },
        "Object": {
            "properties": {}
        }
    },
    "info": {
        "description": "demo of cms2 swagger-api",
        "title": "cms2 api",
        "version": "0.1"
    },
    "paths": {
        "/ContentDraft/create": {
            "post": {
                "operationId": "create",
                "parameters": [{
                    "description": "ContentDraft object",
                    "in": "body",
                    "name": "body",
                    "required": true,
                    "schema": {
                        "$ref": "#/definitions/ContentDraft"
                    }
                }],
                "responses": {
                    "200": {
                        "description": "200 ok - http://jsonapi.org/format/#document-structure-top-level",
                        "schema": {
                            "$ref": "#/definitions/JsonApiResponseDataContentDraft"
                        }
                    },
                    "default": {
                        "description": "internal server error - http://jsonapi.org/format/#errors",
                        "schema": {
                            "$ref": "#/definitions/JsonApiResponseError"
                        }
                    }
                },
                "summary": "create ContentDraft object",
                "tags": ["ContentDraft"]
            }
        },
        "/ContentDraft/createOrReplace": {
            "put": {
                "operationId": "createOrReplace",
                "parameters": [{
                    "description": "ContentDraft object",
                    "in": "body",
                    "name": "body",
                    "required": true,
                    "schema": {
                        "$ref": "#/definitions/ContentDraft"
                    }
                }],
                "responses": {
                    "200": {
                        "description": "200 ok - http://jsonapi.org/format/#document-structure-top-level",
                        "schema": {
                            "$ref": "#/definitions/JsonApiResponseDataContentDraft"
                        }
                    },
                    "default": {
                        "description": "internal server error - http://jsonapi.org/format/#errors",
                        "schema": {
                            "$ref": "#/definitions/JsonApiResponseError"
                        }
                    }
                },
                "summary": "create or replace ContentDraft object",
                "tags": ["ContentDraft"]
            }
        },
        "/ContentDraft/createOrUpdate": {
            "patch": {
                "operationId": "createOrUpdate",
                "parameters": [{
                    "description": "ContentDraft object",
                    "in": "body",
                    "name": "body",
                    "required": true,
                    "schema": {
                        "$ref": "#/definitions/ContentDraft"
                    }
                }],
                "responses": {
                    "200": {
                        "description": "200 ok - http://jsonapi.org/format/#document-structure-top-level",
                        "schema": {
                            "$ref": "#/definitions/JsonApiResponseDataContentDraft"
                        }
                    },
                    "default": {
                        "description": "internal server error - http://jsonapi.org/format/#errors",
                        "schema": {
                            "$ref": "#/definitions/JsonApiResponseError"
                        }
                    }
                },
                "summary": "create or update ContentDraft object",
                "tags": ["ContentDraft"]
            }
        },
        "/ContentDraft/deleteById/{id}": {
            "delete": {
                "operationId": "deleteById",
                "parameters": [{
                    "description": "ContentDraft id",
                    "in": "path",
                    "name": "id",
                    "required": true,
                    "type": "string"
                }],
                "responses": {
                    "200": {
                        "description": "ok - http://jsonapi.org/format/#document-structure-top-level",
                        "schema": {
                            "$ref": "#/definitions/JsonApiResponseData"
                        }
                    },
                    "default": {
                        "description": "internal server error - http://jsonapi.org/format/#errors",
                        "schema": {
                            "$ref": "#/definitions/JsonApiResponseError"
                        }
                    }
                },
                "summary": "delete ContentDraft object by id",
                "tags": ["ContentDraft"]
            }
        },
        "/ContentDraft/getById/{id}": {
            "get": {
                "operationId": "getById",
                "parameters": [{
                    "description": "ContentDraft id",
                    "in": "path",
                    "name": "id",
                    "required": true,
                    "type": "string"
                }],
                "responses": {
                    "200": {
                        "description": "200 ok - http://jsonapi.org/format/#document-structure-top-level",
                        "schema": {
                            "$ref": "#/definitions/JsonApiResponseDataContentDraft"
                        }
                    },
                    "default": {
                        "description": "internal server error - http://jsonapi.org/format/#errors",
                        "schema": {
                            "$ref": "#/definitions/JsonApiResponseError"
                        }
                    }
                },
                "summary": "get ContentDraft object by id",
                "tags": ["ContentDraft"]
            }
        },
        "/ContentDraft/getByQuery": {
            "get": {
                "operationId": "getByQuery",
                "parameters": [{
                    "default": "{\"_id\":\"foo\"}",
                    "description": "mongodb query param",
                    "in": "query",
                    "name": "query",
                    "required": true,
                    "type": "object"
                }, {
                    "default": "{}",
                    "description": "mongodb projection param",
                    "in": "query",
                    "name": "projection",
                    "type": "object"
                }, {
                    "default": 1,
                    "description": "mongodb cursor limit param",
                    "in": "query",
                    "name": "limit",
                    "required": true,
                    "type": "integer"
                }, {
                    "default": "{\"_timeModified\":-1}",
                    "description": "mongodb cursor sort param",
                    "in": "query",
                    "name": "sort",
                    "type": "object"
                }],
                "responses": {
                    "200": {
                        "description": "200 ok - http://jsonapi.org/format/#document-structure-top-level",
                        "schema": {
                            "$ref": "#/definitions/JsonApiResponseDataContentDraft"
                        }
                    },
                    "default": {
                        "description": "internal server error - http://jsonapi.org/format/#errors",
                        "schema": {
                            "$ref": "#/definitions/JsonApiResponseError"
                        }
                    }
                },
                "summary": "get ContentDraft object by query",
                "tags": ["ContentDraft"]
            }
        }
    },
    "swagger": "2.0",
    "tags": [{
        "description": "draft content",
        "name": "ContentDraft"
    }]
}

not sure which frontend workaround u mentioned, but this is the only current that i know of:
-            '/{{_modelName}}/getByQuery': {
-                get: {
-                    _requestHandler: local.cms2.serverMiddlewareCrudDefault,
-                    parameters: [{
-                        description: 'mongodb query',
-                        default: '{"_id":"foo"}',
-                        in: 'query',
-                        name: 'query',
-                        required: true,
-                        schema: { $ref: '#/definitions/Object' }
-                    }, {
-                        description: 'mongodb cursor limit',
-                        default: 1,
-                        in: 'query',
-                        name: 'limit',
-                        required: true,
-                        type: 'integer'
-                    }, {
-                        description: 'mongodb cursor sort',
-                        default: '{"timeModified":-1}',
-                        in: 'query',
-                        name: 'sort',
-                        required: true,
-                        schema: { $ref: '#/definitions/Object' }
-                    }],

and it will fail the validation as well, since 'schema: $ref' seems to be forbidden in query params

The first issue I see is the double curly braces in your path.  I'm not sure any tool out there would handle that properly.  Next, do you have an Object definition?  #/definitions/Object is not the same as {schema: {type: 'object'}}.  I just tried a schema with a $ref in a parameter and it works fine.
Also, you're using swagger-editor and not swagger-ui.  swagger-editor does use swagger-tools.

hmm ... i've tried what u said, and they both seem to fail swagger-tools validation (attached screen-captures)

sorry, i wasn't very clear in my pull request description,
but what i'm trying to do is make SwaggerTools.specs.v2.validate() pass for the "type": "object" case in parameters. (see screen-capture of  local.swagger_tools.v2.validate passing in nodejs)
local.swagger_tools.v2.validate({paths:{'/':{'get':{parameters:[{name:'foo',in:'query',type:'object'}],responses:{200:{}}}}},info:{title:'',version:''},swagger:'2.0'}, debugPrint)

SwaggerTools.specs.v2.validate({paths:{'/':{'get':{parameters:[{name:'foo',in:'query',type:'object'}],responses:{200:{}}}}},info:{title:'',version:''},swagger:'2.0'}, console.log.bind(console))

Screen Shot 2015-03-26 at 23.22.12

You can only use the schema attribute when in is body.  Here is the pertinent information in the Swagger 2.0 Specification: https://github.com/swagger-api/swagger-spec/blob/master/versions/2.0.md#parameterObject


Add Swiss German locale mapping

Since babel supports swissgerman and nswissgerman for some time now, could you please map these two languages to the locale de_CH?

This is done in the DEV branch now and will be in the next release. I should apologise since I live in Switzerland ...


sbt 0.13.15. fix warnings

@xuwei-k thanks! don't worry about CI (it requires private creds). I'll run it locally and merge


Make html template optional but still one of the text or html is mandatory.

Related to feature request https://github.com/niftylettuce/node-email-templates/issues/127

Coverage increased (+0.02%) to 95.79% when pulling 628de6d477312cb36f09379a5f60df1b738d7e7f on alexandrusavin:v2.0 into 4ed28d72011a21b9893738f9730fa94b1f4acd3f on niftylettuce:v2.0.

Thank you!


wip: retrieve logged level for content and display in about

Draft PR for initial interface and criticism.  At a minimum, I want to clean up the About.tsx file further before this merges this way (and it looks like I may need to touch up the unit tests for the ActorUpdate synths as well)

I updated the way to access level to make it a getter on ActorResources -- while I was digging into this, found that my initial implementation besides being a pain to use also wound up crawling the actor updates a bunch (every time the ActorResources constructor got called) to retrieve the level property.  Getter seems to work as expected since the actorUpdate with the actor's level is going to be the first packet we receive for any actor in a given pull.


default_packages default value of false appears to be ignored

I have copied the usage snippet into my package project. When the action is run, then the SublimeText/syntax-test-action@v2 fails.
To me it looks like the default value for the input is not used:

Fetching default packages (ref: , tests: false)
/tmp/tmp.3jzkggdKKn ~/work/purebasic-language-for-sublime-text/purebasic-language-for-sublime-text
--2021-11-03 11:58:13--  https://github.com/sublimehq/Packages/archive/.tar.gz

At a quick glance I see no error in the action.yml or the shell script. Or am I or is the snippet missing some additional information on how to get started?

I checked a few other repositories that I found via github search and they don't seem to specify a ref for the default packages either (1, 2). Later I realized that in your snippet, it rather seems as if an empty string was used as the ref instead of false as a boolean, so I fixed the template to not include that by default. Thanks for reporting and sorry for not replying earlier.


A major update to SharePoint rate limiting

As part of the continuous improvement to the SharePoint API and developer experience, we are publishing the rate limiting for SharePoint APIs and bringing more clarity in how SharePoint throttling works.
Category

[x] Content fix
[ ] New article

Related issues

fixes: N/A
partially: N/A
mentioned in: N/A

What's in this Pull Request?
A major update to SharePoint throttling. Clarify the different types of throttling. Publish the rate limiting number. And introduce new capabilities.

Docs Build status updates of commit ecf1598:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/general-development/how-to-avoid-getting-throttled-or-blocked-in-sharepoint-online.md
:warning:Warning
View
Details

docs/general-development/how-to-avoid-getting-throttled-or-blocked-in-sharepoint-online.md

Line 2, Column 1: [Warning: ms-prod-and-service - See documentation] Only one of the following attributes can exist: 'ms.prod', 'ms.service'. Use ms.prod for on-premise products, or ms.service for cloud services.
Line 15, Column 3: [Warning: bookmark-not-found] Cannot find bookmark '#How-to-handle-throttling' in 'general-development/how-to-avoid-getting-throttled-or-blocked-in-sharepoint-online.md', did you mean '#how-to-handle-throttling'?
Line 17, Column 3: [Warning: bookmark-not-found] Cannot find bookmark '#Scenario-specific-limits' in 'general-development/how-to-avoid-getting-throttled-or-blocked-in-sharepoint-online.md', did you mean '#scenario-specific-limits'?
Line 52, Column 145: [Warning: hard-coded-locale - See documentation] Link 'https://docs.microsoft.com/en-us/office365/servicedescriptions/sharepoint-online-service-description/sharepoint-online-limits#limits-by-plan' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 79, Column 4: [Warning: hard-coded-locale - See documentation] Link 'https://docs.microsoft.com/en-us/graph/json-batching' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 52, Column 145: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/en-us/office365/servicedescriptions/sharepoint-online-service-description/sharepoint-online-limits#limits-by-plan' will be broken in isolated environments. Replace with a relative link.
Line 79, Column 4: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/en-us/graph/json-batching' will be broken in isolated environments. Replace with a relative link.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit ce9277e:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/general-development/how-to-avoid-getting-throttled-or-blocked-in-sharepoint-online.md
:warning:Warning
View
Details

docs/general-development/how-to-avoid-getting-throttled-or-blocked-in-sharepoint-online.md

Line 2, Column 1: [Warning: ms-prod-missing - See documentation] Missing attribute: 'ms.prod'. If you specify 'ms.technology', you must also specify 'ms.prod'.
Line 52, Column 145: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/office365/servicedescriptions/sharepoint-online-service-description/sharepoint-online-limits#limits-by-plan' will be broken in isolated environments. Replace with a relative link.
Line 79, Column 4: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/graph/json-batching' will be broken in isolated environments. Replace with a relative link.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 0f4566f:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/general-development/how-to-avoid-getting-throttled-or-blocked-in-sharepoint-online.md
:warning:Warning
View
Details

docs/general-development/how-to-avoid-getting-throttled-or-blocked-in-sharepoint-online.md

Line 2, Column 1: [Warning: ms-prod-and-service - See documentation] Only one of the following attributes can exist: 'ms.prod', 'ms.service'. Use ms.prod for on-premise products, or ms.service for cloud services.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Any reason why the sample code was removed?


[bug] There are some errors when column's name is a keyword in rust.

When I use sea-orm-cli to generate models, there are some errors when columns' name is a keyword in rust.
error:
error: expected identifier, found keyword 'type'

Hey @bestgopher, welcome!
It was discussed on #217.

Close as duplicate


Visualized graph has wrong conditional edges

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangGraph/LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangGraph/LangChain rather than my code.
[X] I am sure this is better as an issue rather than a GitHub discussion, since this is a LangGraph bug and not a design question.

Example Code
from langgraph.graph import END, StateGraph
from langgraph.checkpoint.memory import MemorySaver
from langchain_core.runnables import RunnablePassthrough
from typing import TypedDict

class State(TypedDict):
    current_step: int
    total_number_of_steps: int

g = StateGraph(State)
g.set_entry_point("entry")

g.add_node("entry", RunnablePassthrough())
g.add_edge("entry", "A")

g.add_node("A", RunnablePassthrough())
g.add_edge("A", "B")

g.add_node("B", RunnablePassthrough())
g.add_edge("B", "reduce_middle_step")

g.add_node("reduce_middle_step", RunnablePassthrough())
g.add_conditional_edges(
    "reduce_middle_step",
    lambda state: (
        END if state["current_step"] >= state["total_number_of_steps"] else "A"
    ),
)

main_graph = g.compile(checkpointer=MemorySaver(), interrupt_after=["B"])

with open("./main_graph.png", "wb") as f:
    f.write(main_graph.get_graph().draw_mermaid_png())

Error Message and Stack Trace (if applicable)
The graph diagram has unspecified connections

Description
Run the code and check the image generated.
It has conditional edges from "recude_middle_step" to "entry", "A", and "B" where I only connected to "A".

System Info
not required.

You can refer to the reply in #987.

You can refer to the reply of this issue:

add_conditional_edges does not work without optional argument path_map #987.

Or check the description in the docstring.

Thank you!!

+1, thank you @gbaian10 ! we would also love to invite you to join LangChain Community on Slack if you're interested / haven't done it yet! https://www.langchain.com/join-community

@vbarda Thank you, I have successfully joined. I never knew there was such a great place. 😳


Only build master as a branch on appveyor

An attempt to not have the annoying 2 appveyor builds on cargo PRs, one of which never starts? This might block both of them though, let's find out

r? @matklad
(rust_highfive has picked a reviewer for you, use r? to override)

I'd assume you want to build auto-cargo too, since otherwise r+ won't test appveyor at all, presumably...

mmmm nope this didin't do what i hoped


Added proper  definition for NRF51_MICROBIT_B

@0xc0170

Thanks !
Tested locally, working, dont need to run CI for this changeset


Not working starting from Filament v2.15.29

Hello,
I noticed some slowness in panel after an update of filament to v2.15.33 than usual, than I found that the plugin is not working at all.
After some inspection, I confirmed that is not working starting from the Filament v2.15.29 (with Laravel 9.26.1/PHP8.1).
I didn't dig deeper, I'm not sure, but I remember that the PR #3719 regarding support for Turbo Navigation got my attention that may cause similar issues.

Is this still valid with the latest version? I accidentally removed the JS file with v0.3.4.
For me, everything is working fine.

Sorry, I didn't noticed that the plugin were update to v0.3.4, that's why I suspected the Filament update.
Now it is OK, I confirm that the plugin is working correctly.


Supersized the minion-y thing.

i think the game looks better with a larger inion. changed the scaleX and scaleY properties in sprite_animation.js

Hi @siddhanth0412, thanks for the contribution!  We're not actively development this project any longer, but we'd be happy to put a link in the README pointing to your fork of hero-engine, if you wish to continue development.


Version 1.0.0 doesn't work on 1.16.5 because it was compiled with Java Runtime Version .55

[main/ERROR] [ne.mi.fm.ja.FMLModContainer/LOADING]: Failed to load class com.envyful.advanced.holograms.forge.ForgeHolograms
[02.03 20:59:33] [Server] java.lang.UnsupportedClassVersionErrorcom/envyful/advanced/holograms/forge/ForgeHolograms has been compiled by a more recent version of the Java Runtime (class file version 55.0), this version of the Java Runtime only recognizes class file versions up to 52.0

Yep.

You need to use Java 11

1.16.5, Vanilla, and thus Forge and most mods offically only support JAVA 8, therefor a lot of modpacks break with the newer version of Java, I'm sure Vahelisa 3 won't even start with Java 11, But I can try.


[MRG] Add multiple-gpu documentation

This pull request resolves #531.
The pull request is ready to be merged.
The details of the pull request:
Add documentation for multiple-gpu training

Coverage remained the same at 93.702% when pulling 19e0aa62039e1c0184f3e087a9c889c652c3f734 on multi-gpu-doc into 9e4d96a131a6818ce6ef0abffebc096908204abe on master.


Harness for Go crypto/x509

See #9.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


2+ stations maps bugs

Description
maps with more than 1 station, has the following error:
Having unique roles on stations that do not exist on other stations has a chance to break the logic of role distribution between players
none of the players may get this unique role and they will all stay in the lobby or become greytide, although the role will remain free.
Reproduction

create a map with multiple stations.
Add a unique role to one of the stations that does not exist on the other station
Open 4 clients and connect to server
trying to get that unique roles all the clients (QM, CMO, CE and HOP for example)
???
some clients will remain in the lobby, although vacant roles for them exist at some stations

You can have multiple stations per map?
God forbid nukies have to choose which one to nuke


SRC-1469 HPA template for Elasticsearch

This will be faced in https://github.com/empathyco/helm-charts/pull/66


Local build instructions missing

We followed the instructions to build the frontend from here but we fail to build the backend.
What do we need to execute to have the backend running locally?

Hi @moritz-deissner-turbinekreuzberg.
You need Java 11+ and Maven to build and run the backend locally. The most straightforward approach which does not involve any IDE, such as Intellij, is to build the project with Maven and then execute one of the jars produced from the deploy-modules.
Build with Maven:
mvn clean compile validate package

Run Jar
java -jar deploy/deploy-jetty/deploy-jetty-jar/target/

However, many IDEs, such as Intellij, supports running applications directly. I normally just choose to create an application and choose the main class of the Jetty application:
com.castlemock.deploy.jetty.jar.config.JarJettyApplication

You can easily create an application configuration in Intellij by doing the following steps:
Run -> Edit Configurations -> + -> Application

Module: deploy-jetty-jar
Main class: com.castlemock.deploy.jetty.jar.config.JarJettyApplication

I can improve the CONTRIBUTION section and make it more clear.
I hope this helps. For more information, please see our contribution file: https://github.com/castlemock/castlemock/blob/master/CONTRIBUTING.md
Best regards,
Karl Dahlgren

Hi @moritz-deissner-turbinekreuzberg.
You need Java 11+ and Maven to build and run the backend locally. The most straightforward approach which does not involve any IDE, such as Intellij, is to build the project with Maven and then execute one of the jars produced from the deploy-modules.
Build with Maven:
mvn clean compile validate package

Run Jar
java -jar deploy/deploy-jetty/deploy-jetty-jar/target/

However, many IDEs, such as Intellij, supports running applications directly. I normally just choose to create an application and choose the main class of the Jetty application:
com.castlemock.deploy.jetty.jar.config.JarJettyApplication

You can easily create an application configuration in Intellij by doing the following steps:
Run -> Edit Configurations -> + -> Application

Module: deploy-jetty-jar
Main class: com.castlemock.deploy.jetty.jar.config.JarJettyApplication

I can improve the CONTRIBUTION section and make it more clear.
I hope this helps. For more information, please see our contribution file: https://github.com/castlemock/castlemock/blob/master/CONTRIBUTING.md
Best regards,
Karl Dahlgren

Thanks.
FYI: we are trying to fix https://github.com/castlemock/castlemock/issues/100

Thanks.
FYI: we are trying to fix https://github.com/castlemock/castlemock/issues/100

it would be great if you could just add your answer to the CONTRIBUTING section

it would be great if you could just add your answer to the CONTRIBUTING section


Update java-jwt to 3.12.1

Updates com.auth0:java-jwt from 3.10.3 to 3.12.1.
GitHub Release Notes - Changelog - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.auth0", artifactId = "java-jwt" } ]

labels: library-update, semver-minor

Codecov Report

Merging #182 (5e3ee0f) into master (e31fc88) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #182   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files          24      24
  Lines         234     234
  Branches        9       9
======================================
  Misses        234     234

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 e31fc88...5e3ee0f. Read the comment docs.

Codecov Report

Merging #182 (5e3ee0f) into master (e31fc88) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #182   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files          24      24
  Lines         234     234
  Branches        9       9
======================================
  Misses        234     234

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 e31fc88...5e3ee0f. Read the comment docs.

Superseded by #189.


[BUG] module is conflicting with MIDI

Module Version: v0.6.11
Describe the bug
bonus from any attribute on damage is reset
To Reproduce
Steps to reproduce the behavior:

have the MIDI and CV modules active.

The MIDI module with full Automation.
fvtt-midi-qol-settings.zip

attacks have their damage bonus by attributes reset to zero.

Expected behavior
as the module should operate only the field of vision it shouldn't interfere with the generation of attack cards in the chat, but it's happening.

I changed the MIDI play order (as a priority) and the CV (without priority) in libwrapper and it got it right again.
but I still don't know if it causes other problems.

there seems to be a synchronization problem between the updates of the actor between the two modules, I didn't understand anything, but Tim seems to have found the solution on his side with Midi it should be solved with the next Midi release


.kokoro: enable Build Cop Bot

Instructions here: https://github.com/googleapis/repo-automation-bots/tree/master/packages/buildcop
The bot will automatically open and close issues for failing tests. You can file issues about it here: https://github.com/googleapis/repo-automation-bots/issues

I installed the bot on the repo (first step of the instructions). We still need to call the buildcop binary to publish the test logs.

Enabled and seems to be filing issues. Marking as fixed. Thank you, @kurtisvg!


Parallel Compilation with no engine

In certain cases dev wish to precompile shaders on a different canvas, outside of the context of a babylon engine.
This can accelerate future shader compilations as the browser itself caches shaders (tested on both chrome and safari).
So a pipeline can be created where:

Pipeline is generated first, without any Babylon engine
babylon engine is created
Pipeline is (re)created with the same shader, resulting in a much faster shader compilation.

If step 1 can be done in parallel to step 2, this can speed up the loading process (for example if the package using in step 2 is large enough to be slow to download).
However, so far there was no support for parallel precompilation as this information comes from the engine. as there is no engine, there is no flag set.
This PR makes the following changes:

If parallel compilation can be enabled, it is enabled, while maintaining the current behavior in thinEngine
WebGL preprocessor is not included per default and not async loaded, as it is the default one and is probably the one mostly used. This saves one async import
A new flag was added when generating the pipeline - if waitForIsReady is set to true, the promise will resolve when the pipeline is ready,a s opposed to right after the shader was compiled.

Reasoning behind using setInterval - this code is only used for shader precompilation by external packages (and not by babylon internals). to avoid addding unneeded code to this package, I decided to use a standard browser feature.
Though this PG will only work locally (due to non-exported internal function) this si the PG i have used to test the functionality - #8Z0MKW#104 . this is for future reference, mainly for me.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/15898/merge
Test environment:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15898/merge/index.html
To test a playground add it to the URL, for example:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15898/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/15898/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/15898/merge
https://gui.babylonjs.com/?snapshot=refs/pull/15898/merge
https://nme.babylonjs.com/?snapshot=refs/pull/15898/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/15898/merge#BCU1XR#0

Visualization tests for WebGPU (Experimental)
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15898/merge/testResults/webgpuplaywright/index.html

WebGL2 visualization test reporter:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15898/merge/testResults/webgl2playwright/index.html

_executeWhenRenderingStateIsCompiled

This is executed when the function _finalizePipelineContext is executed, but until this PR it was only executed in the thinEngine and not when there was no engine, because it is only executed in the _isRenderingStateCompiled and only when it returns true. So everything is needed here. I can show you the entire process if you want.

Approved virtually, @RaananW will merge it when all good.

@sebavan - committed a simplified scheduling code, let me know what you think.


Add visualize methods

It's actually quite nice to be able to see where the cyclic dependency happens rather than simply not allowing cyclic graph....!


Extract sha digest of Docker image:tag

This commit adds two functions in dockerfile.py that will extract the
sha256 digest of an image given its name and tag using the Docker SDK
for Python. All changes are in tern/analyze/docker/dockerfile.py.

find_image_digest() takes an image name and tag and outputs the
associated tag@sha:digest value from Docker. There are two digests
that associate to an image -- a) the transport digest that shows
when you run 'docker pull  on your local command line and
b) The digest that you see if you were to navigate the dockerhub
webui repository and look for the image. find_image_digest() will
return the digest from option (a).

expand_from_images() will take a dockerfile object (created using
get_dockerfile_obj() in dockerfile.py) and update the structure
dictionary and parent_images list with the full tag@sha:digest
value. It uses find_image_digest() to get the associated digest
shas. This will be used later to "pin" a base image in a
dockerfile. This works for multistage builds as well.

Works towards #507
Signed-off-by: Rose Judge<EMAIL_ADDRESS>

@nishakm I was going to open another PR  adding test coverage for these but agree that this could be a good first issue. I will open it.


angular.js:13424 TypeError: d.scope.init is not a function

Angular 1.5.3
Getting this error when the first toast is called, after we have to click the toast to disable it, the it works fine.
angular.js:13424 TypeError: d.scope.init is not a function
    at angular-toastr.min.js:1
    at angular.js:15757
    at m.$eval (angular.js:17025)
    at m.$digest (angular.js:16841)
    at angular.js:17064
    at e (angular.js:5824)
    at angular.js:6100

var CMWebApp = angular.module('CMWebApp', ['ngRoute', 'ngAnimate', 'ngCookies', 'toastr','ngMaterial']);

CMWebApp.config(function(toastrConfig) {
  angular.extend(toastrConfig, {
    autoDismiss: true,
    maxOpened: 0,
    newestOnTop: true,
    positionClass: 'toast-bottom-right',
    preventDuplicates: false,
    preventOpenDuplicates: true,
    allowHtml: true,
    closeButton: false,
    closeHtml: '<button>×</button>',
    extendedTimeOut: 1000,
    iconClasses: {
      error: 'toast-error',
      info: 'toast-info',
      success: 'toast-success',
      warning: 'toast-warning'
    },
    messageClass: 'toast-message',
    onHidden: null,
    onShown: null,
    onTap: null,
    progressBar: true,
    tapToDismiss: true,
    templates: {
      toast: appUrl + '/views/directives/toast/toast.html',
      progressbar: appUrl + '/views/directives/progressbar/progressbar.html'
    },
    timeOut: 3000,
    titleClass: 'toast-title',
    toastClass: 'toast'
  });
});

CMWebApp.config(function($routeProvider, $sceDelegateProvider, $locationProvider) {
  $sceDelegateProvider.resourceUrlWhitelist([
    'self',
    appUrl + '/**'
  ]);

  $routeProvider

    .when('/', {
    title: 'Homepage',
    templateUrl: appUrl + '/views/home.html',
    controller: 'mainCtrl'
  })

  $locationProvider.html5Mode({
    enabled: true,
  });

});

CMWebApp.controller('mainCtrl', function($scope, $rootScope, $sce, $route, $routeParams, $location, $http, $window, toastr) {

  $scope.$route = $route;
  $scope.$location = $location;
  $scope.$routeParams = $routeParams;
  $scope.domain = domain;
  $scope.home = webUrl;
  $scope.assets = appUrl;
  $scope.trustAsHtml = $sce.trustAsHtml;

  $scope.goTo = function(url) {
    $window.open(url, '_blank');
   toastr("Button clicked");
  }

  console.log("Page is loaded");

});

Please help!

You need to cache (in templateCache) those templates or you will get than error. Are those templates custom your the default ones?

Those are the default ones, how do i cache the templates?
Thanks!

Use the .tpls.js file that contains those. Also, you don't need to add the default values again in the config, just use it to override the ones you need.

Can you tell me how to do that? I am new to angularjs.
I mean where to put the files, where and what to edit in the configuration?
That will be really helpful.
Thanks!

Removing template directive, changing .js to tpl.js?
It worked, thanks :+1:

B/W What about $templateCache you told me?

That is an angular service that caches the template for you, so you don't need to request them to the server.  Here is the docs: https://docs.angularjs.org/api/ng/service/$templateCache

I saw that but i am asking how to use that in my condition of the app?

Sorry, this goes really outside the purpose of this issue tracker.

Ok thanks :)


[Snyk] Security upgrade requests from 2.25.0 to 2.32.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `pip` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

requirements.txt

Vulnerabilities that will be fixed
By pinning:

Severity
Priority Score (*)
Issue
Breaking Change
Exploit Maturity

566/1000  Why? Recently disclosed, Has a fix available, CVSS 5.6
Always-Incorrect Control Flow Implementation SNYK-PYTHON-REQUESTS-6928867
No
No Known Exploit

(*) Note that the real score may have changed since the PR was raised.
Some vulnerabilities couldn't be fully fixed and so Snyk will still find them when the project is tested again. This may be because the vulnerability existed within more than one direct dependency, but not all of the affected dependencies could be upgraded.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Note: This is a default PR template raised by Snyk. Find out more about how you can customise Snyk PRs in our documentation.
Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

Qovery can create a Preview Environment for this PR.
To trigger its creation, please post a comment with one of the following command.

Command
Blueprint environment

/qovery  preview 86d23237-413e-4d36-8107-dd4eefb5c091
blueprint

/qovery  preview aa879f73-0195-4e3e-ad3b-64ac8ebf3967
matang-env

/qovery  preview afb4eb55-9a99-4985-8ed2-f3661c8dbc91
new-env

/qovery  preview {all|UUID1,UUID2,...}
To preview multiple environments

This comment has been generated from Qovery AI :robot:.
Below, a word from its wisdom :

Write comments that add new information that doesn’t exist in the code


CNV-32929: Show user-created cluster instancetypes in Catalog > InstanceTypes

📝 Description
This PR fixes a bug wherein only VirtualMachineInstancetypes created by the user were displayed on the Catalog -> InstanceTypes tab page in the "User provided" section. This bug was fixed by displaying VirtualMachineClusterInstancetypes in that section.
Jira: https://issues.redhat.com/browse/CNV-32929
🎥 Screenshots
Before
default namespace (namespaced ITs exist)

openshift namespace (no namespaced ITs exist)

After
default namespace (namespaced ITs exist)

openshift namespace (no namespaced ITs exist)

what is this PR fixing? is there any logical change or only a naming change?

/lgtm

/retest


Gsoc proposal

Hello,
I intend to apply to Google Summer of Code by using the idea Writing high-performance, multithreaded kernels for image processing and my  proposal, will use CUDArt.jl to improve the parallel image processing performance.
I'm writing here to ask for some feedback on what should I know first about CUDArt.jl to start working on my project or if you guys could help me on pointing how I can improve the idea.
This is  my project proposal, feel free to have a look and comment on the dock or here in the thread.
Thanks.
Naelson Douglas

Hi @NaelsonDouglas. Thanks for your interest! I've taken a glance at your proposal. The main thing would be to flesh it out with some more specifics. Which algorithms will you tackle? How would you leverage CUDArt? Are the kernels written in julia, C/C++, or some combination? Can you break the tasks down into steps? What kind of milestones do you envision?

Thanks @timholy,
I'm taking note of your observations and editing the doc based on them. Since the deadline is  almost now, I'm going to write in the doc first and then I can answer here.
I appreciate your attention.
Naelson Douglas


Surprising name resolution in typeclass arguments in variable statements

Prerequisites
Please put an X between the brackets as you perform the following steps:

[x] Check that your issue is not already filed:
https://github.com/leanprover/lean4/issues
[x] Reduce the issue to a minimal, self-contained, reproducible test case.
Avoid dependencies to Mathlib or Batteries.
[x] Test your test case against the latest nightly release, for example on
https://live.lean-lang.org/#project=lean-nightly
(You can also use the settings there to switch to “Lean nightly”)

Description
Typeclass arguments in variable statements get annotated with errors that occurred during their elaboration at statements that are arbitrarily far from them, causing confusing non-local errors.
Steps to Reproduce
Consider the following code:
inductive Foo : Type | foo

class Bar (α : Type) where bar : α

instance : Bar Foo where bar := .foo

class Baz {α : Type} [Bar α] (a : α) : Prop where

variable (k : Foo) [Baz k]

inductive Quux.Fu : Type

-- Possibly a lot of code

inductive Quux.Foo : Type

and
inductive Foo : Type | foo

class Bar (α : Type) where bar : α

instance : Bar Foo where bar := .foo

structure Baz {α : Type} [Bar α] (a : α) : Prop where

variable (k : Foo) (l : Baz k)

inductive Quux.Fu : Type

-- Possibly a lot of code

inductive Quux.Foo : Type

Expected behavior: Neither of the two code blocks error, or at least error in the same way.
Actual behavior: The first code block throws
failed to synthesize instance
  Bar Foo

on [Baz k]. The second one does not.
Versions
4.8.0
Impact
Add :+1: to issues you consider important. If others are impacted by this issue, please ask them to add :+1: to it.

This is an accidental name capture bug. While elaborating inductive Quux.Foo, (k : Foo) is interpreted as (k : Quux.Foo). We can use pp.fullNames true to expose the issue:
 error: failed to synthesize
  Bar Quux.Foo

Here is an even nastier version where no error is reported, but def a is bogus
inductive Foo : Type | foo
class Bar (α : Type) where bar : α
instance : Bar Foo where bar := .foo
structure Baz {α : Type} [Bar α] (a : α) : Prop where
variable (k : Foo) (l : Baz k)
inductive Quux.Fu : Type
inductive Quux.Foo : Type
open Quux
def a := l
#print a
-- def a : sorryAx Prop true → sorryAx Prop true := fun l => l

maybe the part of the code relevant to variable elaboration is about to change significantly in a way that will make this bug disappear?

@YaelDillies Correct. This is our current plan.

Unfortunately #4814 did not change variables sufficiently enough to fix this issue. Actually, I believe this is a duplicate of #1876. As documented there, we unfortunately do not know of a good way to fix this issue.


How would I go about saving a snapshot of a directory?

What's your question?

I am testing a CLI application that operates on files. I am using pytest and pytest-datadir like this:
tests/test_entire_app.py
import subprocess
def test_scenario_one(datadir, snapshot):
    d = datadir / "test_scenario_one"
    p = subprocess.run(["myapp", d], capture_output=True, text=True)
    assert <somehow_load_directory_to_compare(d)> == snapshot # somehow snapshot should save that directory

What I'm doing right now is manually with open(f): data1 = f.read() for each expected output test file. Then all those go in a dict object { "app.ts": appts, "app.js": appjs, ... } which gets mashed into a huge .ambr file. I'd prefer a regular  real directory structure if possible.
I want it to result in something like this:
.
└── tests/
    ├── __snapshots__/
    │   └── test_entire_app/
    │       └── test_scenario_one/
    │           ├── app.ts
    │           ├── app.js
    │           ├── utils.ts
    │           └── utils.js
    ├── test_entire_app/
    │   └── test_scenario_one/
    │       ├── app.ts
    │       └── utils.ts
    └── test_entire_app.py

...but I'm not familiar with how to go about actually doing that. I would appreciate some guidance about how to do this. I see that there's some examples of how to use the syrupy.extensions.single_file which seems cool, but I don't know how to go about making an extension that creates a directory with multiple files.

The "detect unused snapshot" files logic assumes a snapshot lives in one file I believe, but everything else should be possible at least with a custom extension.
I can take a look in a couple days and see if I can put together an example for you.


Add findomain subdomain scanner

Description
Add findomain tool in the list of subdomain scanning tools
Motivation
Find domain scans domains from multiple sources and has the capability to add domains into the database which can be later used for monitoring, for now, I have only added single domain search using findomain.

Looks awesome, thanks for the integration. Merging and triggering Docker's auto-build.


Typo "Azure storage Emulator"→"Azure Storage Emulator"

https://docs.microsoft.com/en-us/azure/cosmos-db/table-storage-how-to-use-php

@hyoshioka0128 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

@SnehaGunda: I'm sorry - only the author of this article, @sakash279, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


Corona-Warn app in foreign app sores

I have Chinese friends working in Germany since many years.
They cannot use the Corona-Warn app on their iPhones because it is not available in the Chinese App store. They cannot switch to the German app store because they have an Apple Music subscription.
Why not make the App available internationally?

Your Question

Website URL Path:
Line / Paragraph:
Question:

@wolfgang35435
Please take a look at this FAQ Entry: https://www.coronawarn.app/en/faq/#international

Hello @wolfgang35435 ,
As @Ein-Tim said, FAQ entry https://www.coronawarn.app/en/faq/#international should answer your question. Particularly this line:

After a detailed examination of the situation it was determined that for legal reasons any publication of the Corona-Warn-App in App Stores outside of Germany is currently only possible after a case-by-case assessment.

Countries are added one by one after resolving and clarifying legal issues. You can also request support for a specific country in the wishlist repository.
I hope that answers your question.
Regards
CH

Corona-Warn-App Open Source Team

@wolfgang35435 We'll close this issue because your question was answered. Please notify us if the information we provided was not sufficient.
Regards,
CH

Corona-Warn-App Open Source Team


Do we need input splits as new S3 objects?

Since Amazon S3 supports the HTTP Range header in the GET Object API [1], do we really need to split the input files? For Mappers, the framework could just invoke Lambda functions asynchronously with an input file and a byte range to process.
[1] https://docs.aws.amazon.com/AmazonS3/latest/API/RESTObjectGET.html

Input splits are already implemented this way. 🙂An input split is just a S3 object handle (path), with a range of bytes -- there's no copy done to create new S3 objects from splits.
https://github.com/bcongdon/corral/blob/master/split.go#L15


Release of version <IP_ADDRESS> of ComputerManagementDsc

Releasing version <IP_ADDRESS>

This change is

Codecov Report

Merging #208 into master will decrease coverage by <1%.
The diff coverage is 87%.

@@          Coverage Diff           @@
##           master   #208    +/-   ##
======================================
- Coverage      90%    90%    -1%
======================================
  Files           7      8     +1
  Lines         735    844   +109
======================================
+ Hits          667    762    +95
- Misses         68     82    +14


Network unavailability handling

Let user know something went wrong and provide helpful action
Now it just looks 'broken'

unstable internet connection
node is down
coingecko is down
mdw is down

anything else?
Suggested by @janmichek.
Example of a problem we want to solve here: https://github.com/aeternity/aescan/issues/127

The data failed to fetch component which was implemented just for Oracle Events had to be removed, because was colliding with loaders.
There needs to be more adjustments to distinguish to work with loaders. Ideally for the whole app, not just one entity.
https://github.com/aeternity/aescan/pull/419/commits/4487bf6df0e3819f03b8f11f82097b4a97ba3acd

The page should not stop working if the count is not working.
As happened in here
https://github.com/aeternity/aescan/pull/815


Service Account template is referring to .Release.Name whilst image-pull-secret is using .Release.Namespace

https://github.com/aquasecurity/aqua-helm/blob/a8cc2531ea0ea9d9e23e24aa559ff2fb6902c281/server/templates/image-pull-secret.yaml#L6
I'm having trouble deploying aqua server using helm. After populating the values.yml file with the correct values for imageCredentials, my pods cannot pull the images from the private registries as they getting a 401 Unauthorized. After a bit of digging it seems like the service account cannot find the image-pull-secret created by helm due to the following:
The Service account template is using {{ .Release.Name }}-registry-secret when referring to the registry secret, however, the image-pull-secret template is using {{ .Release.Namespace }}-registry-secret as its name. Both are set when the .Values.imageCredentials boolean value is set to true, and causes the service account to not find the secret.
Perhaps I'm missing something here? Otherwise, it seems like a bug.
Looking forward to understanding the issue.

It is a bug, fixed in #256.
The registry secret name should be with the name aqua (as namespace recommended).

Hi, I've pulled the 6.0.1 Server helm chart, it is still showing this bug. I thought it was fixed in 6.0.1?