ci: build multiarch docker images

Build multiarch (amd64 and arm64) images based on setup-buildx-action and setup-qemu-action.
See: https://docs.docker.com/build/ci/github-actions/multi-platform
All base images are compatible with the arm64 target architecture:

https://hub.docker.com/_/ubuntu/tags
https://hub.docker.com/_/debian/tags
https://hub.docker.com/r/bitnami/minideb/tags

Thanks!


Add link to slides for "Thinking In Ecto"

Thanks for putting this together! 😃

Thanks Darin, great talk!

Thank you ☺


tests/bert_for_sequence_classification.py reports "This is an invalid model"

Hi, I installed torch-ort as the instruction. 'python -m torch_ort.configure' does not report any error. However, when I run the verification, it reports the following errors:
======== Epoch 1 / 4 with batch size 32 ========
Warning: Unsupported operator ATenOp. No schema registered for this operator.
Warning: Unsupported operator ATenOp. No schema registered for this operator.
Warning: Unsupported operator ATenOp. No schema registered for this operator.
Warning: Unsupported operator ATenOp. No schema registered for this operator.
Warning: Unsupported operator ATenOp. No schema registered for this operator.
Warning: Unsupported operator ATenOp. No schema registered for this operator.
Warning: Unsupported operator SoftmaxCrossEntropyLossInternal. No schema registered for this operator.
2021-11-16 23:56:39.446639117 [W:onnxruntime:Default, graph.cc:2538 InitFunctionBodyForNode] Function body initialization failed for node 'Softmax_131_Grad/SoftmaxGrad_0' optype SoftmaxGrad. Error message /onnxruntime_src/onnxruntime/core/graph/function.cc:749 onnxruntime::FunctionImpl::FunctionImpl(onnxruntime::Graph&, const NodeIndex&, const onnx::FunctionProto&, const std::unordered_map<std::basic_string<char>, const onnx::FunctionProto*>&, std::vector<std::unique_ptr<onnxruntime::Function> >&, const onnxruntime::logging::Logger&, bool) status.IsOK() was false. Resolve subgraph failed:This is an invalid model. Error in Node:0x557a5b74e130 : Node (0x557a5b74e130) has input size 2 not in range [min=1, max=1].
. Execution will fail if ORT does not have a specialized kernel for this op
2021-11-16 23:56:39.464632288 [W:onnxruntime:, graph.cc:2538 InitFunctionBodyForNode] Function body initialization failed for node 'Softmax_131_Grad/SoftmaxGrad_0' optype SoftmaxGrad. Error message /onnxruntime_src/onnxruntime/core/graph/function.cc:749 onnxruntime::FunctionImpl::FunctionImpl(onnxruntime::Graph&, const NodeIndex&, const onnx::FunctionProto&, const std::unordered_map<std::basic_string<char>, const onnx::FunctionProto*>&, std::vector<std::unique_ptr<onnxruntime::Function> >&, const onnxruntime::logging::Logger&, bool) status.IsOK() was false. Resolve subgraph failed:This is an invalid model. Error in Node:0x557a60d5e230 : Node (0x557a60d5e230) has input size 2 not in range [min=1, max=1].
. Execution will fail if ORT does not have a specialized kernel for this op
Inconsistency detected by ld.so: dl-version.c: 205: _dl_check_map_versions: Assertion `needed != NULL' failed!

Testbed:
V100-32GB
CUDA-10.2
torch==1.9.0
torch-ort==1.9.0
onnxruntime-training==1.9.0

Could you share  your model file with us? Looks like an op is not supported in your model.

Hi, @wschin , the model file is the test in the torch-ort repo.
python bert_for_sequence_classification.py

This model is runnable on my side. _dl_check_map_versions: Assertion needed != NULL' failed!` probably means you need to reinstall all related packages.

Hi @wschin , it works after I re-installed the dependencies. Thank you!


adds a completed user profile section and fixes newsletter styles

@xuelink I added a user completed section, I commented it out on the home page for now. Inside of the component inside the loop, I added some comments to better understand what is going on.

It looks ok thank you, i will attach more commits with this PR directly ok

No problem, you can play around with the colors,  I'm not sold on the light theme for it but its up to you

@jakebogan01 Hey how are you ? I hope everything is going well recently with you.
If you will have time nowadays, can you please focus a little bit more about the design itself here ? Not circular profile pictures, if it may be look like as the app has, gonna be awesome.
i did here with angular with these component by using ionic framework in the base
This is a single user-item
HTML
https://github.com/langx/langx/blob/78491133782f493e8e85bd8da1a45b44afe5aa77/src/app/components/user-list/user-list.component.html
SCSS
https://github.com/langx/langx/blob/main/src/app/components/user-list/user-list.component.scss
This is a list of users
https://github.com/langx/langx/blob/78491133782f493e8e85bd8da1a45b44afe5aa77/src/app/components/home/target-language/target-language.component.html#L6-L35

@Honzoraptor31415 What do you think about this PR ?
Do you want to join here ?

@Honzoraptor31415 What do you think about this PR ? Do you want to join here ?

yeah, I'm looking at the commits now,
What exactely is going on here?

@Honzoraptor31415 What do you think about this PR ? Do you want to join here ?

yeah, I'm looking at the commits now, What exactely is going on here?

Basically, if you do similar design here with the app then i will implement API to show realtime data in our homepage.
It should be responsive as well.


Cannot seem to use a local windows iso file

Hi i have a local windows iso that i would like to use but i cannot get it working, alpine linux booted as normal so i read your readme and change option to windows and place a boot.iso to my /storage drive but it cannot seem to  boot.

I would have to check again actually in the same container i managed to get Endeavour Os Linux working,
could you please tell how exactly my docker compose should look if use a windows iso file?
I cannot specify at - BOOT: it seems.
Also on a side note vnc works but web vnc does not for now, i tried a combination of ports a few tiems but no luck.
version: "3"
services:
qemu:
container_name: qemu
image: qemux/qemu-docker:latest
environment:
RAM_SIZE: "4G"
CPU_CORES: "4"
DISK_SIZE: "128G"
DISPLAY: "web"
BOOT_MODE: "uefi"
BOOT: "https://mirror.albony.xyz/endeavouros/iso/Endeavouros-Galileo-11-2023.iso"
volumes:
- /var/qemu:/storage
devices:
- /dev/kvm
cap_add:
- NET_ADMIN
ports:
- 2222:22
- 5900:5900
- 8006:8006
stop_grace_period: 2m
restart: unless-stopped

I can try the windows booting, also the web vnc site does not work at  8006, i keep getting site cannot be reached on the browser
Here are the logs:
❯ Starting QEMU for Docker v3.15...
❯ For support visit https://github.com/qemus/qemu-docker
❯ Installing web-based VNC client...
❯ Booting image using QEMU emulator version 8.1.2 ...
3h3h3hBdsDxe: loading Boot0004 "endeavouros" from HD(1,GPT,BCCB9164-8004-4FFA-8215-99161AF9B685,0x1000,0x1F4000)/\EFI\endeavouros\grubx64.efi
BdsDxe: starting Boot0004 "endeavouros" from HD(1,GPT,BCCB9164-8004-4FFA-8215-99161AF9B685,0x1000,0x1F4000)/\EFI\endeavouros\grubx64.efi

And here is my updated stack:

version: "3"
services:
  qemu:
    container_name: qemu
    image: qemux/qemu-docker:latest
    environment:
      RAM_SIZE: "4G"
      CPU_CORES: "4"
      DISK_SIZE: "128G"
      DISPLAY: "web"
      BOOT_MODE: "uefi"
      BOOT: "https://mirror.albony.xyz/endeavouros/iso/Endeavouros-Galileo-11-2023.iso"
    volumes:
      - /var/qemu:/storage
    devices:
      - /dev/kvm
    cap_add:
      - NET_ADMIN
    ports:
      - 8006:8006
    stop_grace_period: 2m
    restart: unless-stopped

Are you sure you are using the correct IP to connect to 8006?
Because in the Readme i wrote http://localhost:8006/ but ofcourse this assumes that you run the container on the same machine as where you run the webbrowser. So maybe instead of localhost you must use the IP of the docker host?

Sorry for the late reply. I am running all my containers in this format only http://<IP_ADDRESS>:8006. All my other containers including my nextcloud and kasm instances seem to be running fine, i even checked if any of the ports overlap with the container i run, but i really am not able to view through web vnc.
I tried toggling add blocker off, changed to a chromium browser, used a different machine and even tried running it directly off my main machine. Vnc works without any issues but  web vnc is simply not working for me, you can try running my compose and check.

You can rename an file to boot.iso and place it in the \storage folder and it will override any configured URL in the BOOT variable.
So what you want is already possible (with a single file). Not with multiple files yet..

I did try that with a local iso but unfortunately i get this error.
This keeps going on and on in logs
❯ ERROR: Failed to download http://www.example.com/image.iso, reason: 8
❯ Starting QEMU for Docker v4.03...
❯ For support visit https://github.com/qemus/qemu-docker
❯ Downloading image.iso as boot image...
❯ ERROR: Failed to download http://www.example.com/image.iso, reason: 8
❯ Starting QEMU for Docker v4.03...
❯ For support visit https://github.com/qemus/qemu-docker
❯ Downloading image.iso as boot image...
❯ ERROR: Failed to download http://www.example.com/image.iso, reason: 8
❯ Starting QEMU for Docker v4.03...
❯ For support visit https://github.com/qemus/qemu-docker
❯ Downloading image.iso as boot image...
❯ ERROR: Failed to download http://www.example.com/image.iso, reason: 8
❯ Starting QEMU for Docker v4.03...
❯ For support visit https://github.com/qemus/qemu-docker
❯ Downloading image.iso as boot image...
❯ ERROR: Failed to download http://www.example.com/image.iso, reason: 8
❯ Starting QEMU for Docker v4.03...
❯ For support visit https://github.com/qemus/qemu-docker
❯ Downloading image.iso as boot image...
❯ ERROR: Failed to download http://www.example.com/image.iso, reason: 8

I even update my compose and tried both where my already installed EOS is and a fresh container:
version: "3"
services:
  qemu:
    container_name: qemu
    image: qemux/qemu-docker:latest
    environment:
      RAM_SIZE: "4G"
      CPU_CORES: "4"
      DISK_SIZE: "128G"
      DISPLAY: "web"
      BOOT_MODE: "uefi"
    volumes:
      - /var/qemu:/storage
    devices:
      - /dev/kvm
    cap_add:
      - NET_ADMIN
    ports:
      - 8006:8006
    stop_grace_period: 2m
    restart: unless-stopped

Tried both with and without a download link.

Okay, my fault, sorry!
I said to you boot.iso but the correct name is boot.img. I will create a new version that accepts both names to avoid this mistake in the future.

Please do tell if you finalize a release that accepts the .iso format. And thank you for this project.


Support Filtering Items in STASH Units

Just found this plugin and it seemed like it would be a nice feature. There's likely a more efficient implementation, but hopefully this works well enough

Thanks for the quick response! Sorry for the poor code formatting, I missed a few spots when cleaning it up. I'll fix up the inconsistencies now

Much thanks for contributing to this project!
I have merged this pull request into a new feature branch, applied some changes, and added it through #29 .
There were some edits I made before merging. In case you are interested, below I have provided a small summary.

Instead of using a new data structure to cache STASH items to filter, I have modified the overlay class to infer their statuses using the existing data mappings. Moreover, these mappings are pre-processed in ./data/EmoteClueAssociations.java and allow me to traverse the emoteClue data set.


Add #unstable? method for cask versions.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew tests with your changes locally?

To be used in automerge. Also, remove tests for slashes_to_* and no_slashes, since slashes shouldn't be used in versions.

Cheers @reitermarkus!


[ENGDOCS-849] Add Linux registry.json

Proposed changes
Add info to create registry.json in Linux.
https://deploy-preview-15405--docsdocker.netlify.app/docker-hub/configure-sign-in/
Related issues (optional)
ENGDOCS-849

Closing as duplicate.


Fix routing after scaling bounding box

Me and Aditi are gonna be working on this one

Running it on Comp seems to route successfully but when it's run on ADC1 it breaks at "AcsMgr::computeAcs Start Access Points Generation"
DrGridAstar::run        Route net net020, Sym: 1, SelfSym: 0, StrictDRC: 1
DrGridAstar::run        Route net net020, Sym: 0, SelfSym: 0, StrictDRC: 1
DrGridAstar::run        Route net net028, Sym: 1, SelfSym: 0, StrictDRC: 1
DrGridAstar::run        Route net net028, Sym: 0, SelfSym: 0, StrictDRC: 1
DrGridAstar::run        Route net SUM_N, Sym: 1, SelfSym: 0, StrictDRC: 1
DrGridAstar::run        ERROR: Route net SUM_N failed!
DrGridAstar::run        Route net SUM_N, Sym: 1, SelfSym: 0, StrictDRC: 0
DrGridAstar::run        ERROR: Route net SUM_N failed!
python3: /tmp/pip-req-build-rsalgwvt/src/dr/drGridRoute.cpp:90: bool ANAROUTE::DrGridRoute::runNRR(auto:5&, bool, ANAROUTE::Int_t) [with auto:3 = __gnu_pbds::priority_queue<ANAROUTE::Net*, ANAROUTE::DrGridRoute::SignalNetCmp, __gnu_pbds::pairing_heap_tag, std::allocator<char> >; ANAROUTE::Int_t = int]: Assertion `bSuccess' failed.```

Also the placer output looks a bit weird when it's ran on ADC1. It doesn't really look symmetrical and it looks like it tries wiring? I checked the layers of the "wires" too and they're the M1 M2 layers which I'm pretty sure are used for wiring. Maybe it's also just because it tries doing routing for the subcircuits before wiring the entire circuit together.

Last thing is that I tried opening Comp in KLayout and measuring some distances and they seemed to be kind of weird. I used bounding boxes of (-9, -9) and (9, 9) so the expected distances should be around 18 but I was measuring distances of 17

My guess is that it isn't really a problem and that it's not that the distances between the edges of the circuits have to be 18 but it's just that the distances between the center of the devices are 18.

I looked into it a bit more and the wiring seems to be expected for ADC1. Subckt route files are still generated normally so I think MAGICAL isn't failing on all the routing and probably successfully generates routing for each individual subckt before placing all of the subckts together, but fails routing all the subckts together which is why we see routing in the place file


training script

Hello author, I use zoomingslowmo framework training script has a gradient error problem, I would like to ask if you can provide the overall framework file？

We have trained this code multiple times and used it for other related tasks, but did not find the problem you mentioned. Perhaps you can provide the detailed error report.

/home/iv/miniconda3/envs/stvsr/lib/python3.8/site-packages/torch/optim/lr_scheduler.py:131: UserWarning: Detected call of lr_scheduler.step() before optimizer.step(). In PyTorch 1.1.0 and later, you should call them in the opposite order: optimizer.step() before lr_scheduler.step().  Failure to do this will result in PyTorch skipping the first value of the learning rate schedule. See more details at https://pytorch.org/docs/stable/optim.html#how-to-adjust-learning-rate
warnings.warn("Detected call of lr_scheduler.step() before optimizer.step(). "
/home/iv/miniconda3/envs/stvsr/lib/python3.8/site-packages/torch/functional.py:568: UserWarning: torch.meshgrid: in an upcoming release, it will be required to pass the indexing argument. (Triggered internally at  /opt/conda/conda-bld/pytorch_1646755903507/work/aten/src/ATen/native/TensorShape.cpp:2228.)
return _VF.meshgrid(tensors, **kwargs)  # type: ignore[attr-defined]
/home/iv/miniconda3/envs/stvsr/lib/python3.8/site-packages/torch/nn/functional.py:1944: UserWarning: nn.functional.sigmoid is deprecated. Use torch.sigmoid instead.
warnings.warn("nn.functional.sigmoid is deprecated. Use torch.sigmoid instead.")
Traceback (most recent call last):
File "train.py", line 196, in
main()
File "train.py", line 162, in main
model.optimize_parameters(current_step)
File "/mnt/Temp_new/QXT/OFR-BRN-master/src/models/VideoSR_base_model.py", line 103, in optimize_parameters
l_pix.backward()
File "/home/iv/miniconda3/envs/stvsr/lib/python3.8/site-packages/torch/_tensor.py", line 363, in backward
torch.autograd.backward(self, gradient, retain_graph, create_graph, inputs=inputs)
File "/home/iv/miniconda3/envs/stvsr/lib/python3.8/site-packages/torch/autograd/init.py", line 173, in backward
Variable._execution_engine.run_backward(  # Calls into the C++ engine to run the backward pass
RuntimeError: one of the variables needed for gradient computation has been modified by an inplace operation: [torch.cuda.FloatTensor [16, 2, 32, 32]], which is output 0 of AsStridedBackward0, is at version 3; expected version 2 instead. Hint: enable anomaly detection to find the operation that failed to compute its gradient, with torch.autograd.set_detect_anomaly(True)

你好作者,我使用您链接里的方法,返回的错误是/home/iv/miniconda3/envs/stvsr/lib/python3.8/site-packages/torch/autograd/init.py:173: UserWarning: Error detected in CudnnGridSamplerBackward0. Traceback of forward call that caused the error:
File "train.py", line 179, in
main()
File "train.py", line 147, in main
trainer.train_one_sample(train_data, current_step)
File "/mnt/Temp_new/QXT/RSTT-main/tools/Trainer.py", line 72, in train_one_sample
self.outputs = self.model(self.inputs)
File "/home/iv/miniconda3/envs/stvsr/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1110, in _call_impl
return forward_call(*input, **kwargs)
File "/home/iv/miniconda3/envs/stvsr/lib/python3.8/site-packages/torch/nn/parallel/data_parallel.py", line 166, in forward
return self.module(*inputs[0], **kwargs[0])
File "/home/iv/miniconda3/envs/stvsr/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1110, in _call_impl
return forward_call(*input, **kwargs)
File "/mnt/Temp_new/QXT/RSTT-main/models/OFR_BRN.py", line 83, in forward
flows_forward, flows_backward, flow_out_forward_half, flow_out_backward_half = self.compute_biflow(lrs)
File "/mnt/Temp_new/QXT/RSTT-main/models/OFR_BRN.py", line 73, in compute_biflow
flows_forward[:, i, :, :, :] = flow_warp(flows_forward[:, i, :, :, :], flow_out_forward_half[:, i, :, :, :].permute(0, 2, 3, 1))
File "/mnt/Temp_new/QXT/RSTT-main/models/general_module.py", line 199, in flow_warp
output = F.grid_sample(
File "/home/iv/miniconda3/envs/stvsr/lib/python3.8/site-packages/torch/nn/functional.py", line 4201, in grid_sample
return torch.grid_sampler(input, grid, mode_enum, padding_mode_enum, align_corners)
(Triggered internally at  /opt/conda/conda-bld/pytorch_1646755903507/work/torch/csrc/autograd/python_anomaly_mode.cpp:104.)
Variable._execution_engine.run_backward(  # Calls into the C++ engine to run the backward pass
Traceback (most recent call last):
File "train.py", line 179, in
main()
File "train.py", line 147, in main
trainer.train_one_sample(train_data, current_step)
File "/mnt/Temp_new/QXT/RSTT-main/tools/Trainer.py", line 74, in train_one_sample
loss.backward()
File "/home/iv/miniconda3/envs/stvsr/lib/python3.8/site-packages/torch/_tensor.py", line 363, in backward
torch.autograd.backward(self, gradient, retain_graph, create_graph, inputs=inputs)
File "/home/iv/miniconda3/envs/stvsr/lib/python3.8/site-packages/torch/autograd/init.py", line 173, in backward
Variable._execution_engine.run_backward(  # Calls into the C++ engine to run the backward pass
RuntimeError: one of the variables needed for gradient computation has been modified by an inplace operation: [torch.cuda.FloatTensor [5, 2, 64, 64]], which is output 0 of AsStridedBackward0, is at version 3; expected version 2 instead. Hint: the backtrace further above shows the operation that failed to compute its gradient. The variable in question was changed in there or anywhere later. Good luck!

Could you please send me your complete training program file? I am very interested in your work, and your work is very good! Thank you very much.


Frontend Team - Lisiting Page Top-Bar

Add a top bar to the listing page. As shown in the design

On it!

Problems
The Logo is not available on the navbar. It gives a 404 error and the alignment of the items is not according to the wireframe

The colors are not used as required. Find the colors from the Figma wireframes
TODO

href should be changed from "assets/images..." to "./assets/images..."

Elements in the navbar must be aligned properly

Color for the search icon button should be changed as per the Figma wireframe

On it!

Problem

The top-bar is not responsive. The Alignment breaks on mobile screens.
![Annotation 2020-08-25 153656](https://user-images.githubusercontent.com/66865722/91162013-22924080-e6e9-11ea-956b-51bfe9cc318f.png)
Please refer to this design while making the top bar.

On It!


⚠ Turnos 24x7 has degraded performance

In 9ba80da, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 1288 ms

Resolved: Turnos 24x7 performance has improved in d829151.


Proposal: Subnet Scarcity Phase 2 detailed design

Reason for Change:

empty NC creation
pool scaling math

Issue Fixed:

Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests

Notes:

I'm loving these docs. Everyone else, if you're not reviewing these with the markdown previews turned on, you're missing out on some great diagrams :)


Error when running the web package: axios_1.default.post is not a function

Hello guys,
I got an error while trying to I am trying to run the web application with the following steps.:

yarn && yarn build
cd packages/web
yarn start
Press Sign In in the website and sign twice with MetaMask.
The error came: Uncaught runtime errors: axios_1.default.post is not a function

The error happens in the file: /packages/lib/offchainResolver-api/src/index.ts at:
    const { status } = await axios.post(url, data);
It seems something related to the build configuration. As when logging axios_1 in the browser:

{default: '/static/media/axios.cae3b842dc3a1d186a27.cjs'}

So, I am wondering how you run the website from packages/web. And I am wondering, what I am missing?
node -v: v16.16.0
npm -version: 9.3.0
OS: Linux Mint
Thanks,

Hi Muhammad,
sorry for the late reply. We are releasing the version of the components at the moment and will address this issue as well.


Update course-definition.yml

"send" doesn't take you to the specific section because it is "sending".

Not sure how I missed this - thanks @redjonzaci!


feat: res\synapse\workspace firewall child resource added

Description
Fixes #1189
fixes #2372
Pipeline Reference

Pipeline

Type of Change

[ ] Update to CI Environment or utilities (Non-module affecting changes)
[x] Azure Verified Module updates:

[x] Bugfix containing backwards-compatible bug fixes, and I have NOT bumped the MAJOR or MINOR version in version.json:

[x] Someone has opened a bug report issue, and I have included "Closes #{bug_report_issue_number}" in the PR description.
[ ] The bug was found by the module author, and no one has opened an issue to report it yet.

[ ] Feature update backwards compatible feature updates, and I have bumped the MINOR version in version.json.
[ ] Breaking changes and I have bumped the MAJOR version in version.json.
[ ] Update to documentation

Checklist

[x] I'm sure there are no other open Pull Requests for the same update/change
[x] I have run Set-AVMModule locally to generate the supporting module files.
[x] My corresponding pipelines / checks run clean and green without any errors or warnings

Hey @elanzel,
Is this PR something that is still being worked on, or should we close it? :)

Hey @elanzel, Is this PR something that is still being worked on, or should we close it? :)

Hi @AlexanderSehr I need to apply the changes suggested by Kris

Hi @krbar , i applied the changes and re-tested the code. Can you merge?

🌊


Create Undo system

Have recreated the Undo system from the old code. Works for:

Props
DynShapes
Wheel (but it currently doesn't remove the extra GameObject on the parent)
Weld
Redo functionality is a TODO.

A couple of other small (mostly related) changes:

Weld tool secondary (Unweld all) moved to Reload
Added SteamId, GroupId, and User Name to 'Player' class

Good to go @Nebual . Merry Christmas :)


Minor js bug - Status on patient dashboard now editable

Thanks for creating an issue! Please fill out this form so we can be sure to have all the information we need, and to minimize back and forth.

What are we trying to do?

This should be a display field, and looks like one, but appears editable now -

To recreate, open a patient record and edit the status in the patient dashboard. (Note that it doesn't actually save db state!)

What feature or behavior is this required for?

keeping uneditable data from appearing editable

How could we solve this issue? (Not knowing is okay!)

change the input element

Anything else?

I'll grab this


feat: support artifacts search

https://github.com/cncf-tags/wg-artifacts/pull/34
An experiment to make this possible

What type of PR is this?

Which issue does this PR fix:
What does this PR do / Why do we need it:
If an issue # is not available please add repro steps and logs showing the issue:
Testing done on this change:

Automation added to e2e:

Will this break upgrades or downgrades?
Does this PR introduce any user-facing change?:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 92.31%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2209      +/-   ##
==========================================
- Coverage   92.86%   92.31%   -0.56%
==========================================
  Files         168      165       -3
  Lines       22289    28583    +6294
==========================================
+ Hits        20698    26385    +5687
- Misses        992     1623     +631
+ Partials      599      575      -24

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


질문 폼 생성 - 변경된 아이콘 적용

🤔 해결하려는 문제가 무엇인가요?
close #227
🎉 변경 사항

사용되는 아이콘 추가 및 적용

🌄 스크린샷

Codecov Report
Patch coverage has no change and project coverage change: +0.04 :tada:

Comparison is base (352012d) 84.80% compared to head (b4c7e6c) 84.84%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #246      +/-   ##
==========================================
+ Coverage   84.80%   84.84%   +0.04%
==========================================
  Files          42       42
  Lines         329      330       +1
  Branches       57       57
==========================================
+ Hits          279      280       +1
  Misses         50       50

see 1 file with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Remove wp embeds in AMP

Fixes #190

:tada: This PR is included in version 0.12.9 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


ImportError: No module named 'pigar.extractor'

$ pip install git+https://github.com/Damnever/pigar.git@master --upgrade
...
$ pigar
Traceback (most recent call last):
  File "/home/louis/.virtualenvs/pypimonitor/bin/pigar", line 9, in <module>
    load_entry_point('pigar==0.6.8', 'console_scripts', 'pigar')()
  File "/home/louis/.virtualenvs/pypimonitor/lib/python3.5/site-packages/pkg_resources/__init__.py", line 568, in load_entry_point
    return get_distribution(dist).load_entry_point(group, name)
  File "/home/louis/.virtualenvs/pypimonitor/lib/python3.5/site-packages/pkg_resources/__init__.py", line 2720, in load_entry_point
    return ep.load()
  File "/home/louis/.virtualenvs/pypimonitor/lib/python3.5/site-packages/pkg_resources/__init__.py", line 2380, in load
    return self.resolve()
  File "/home/louis/.virtualenvs/pypimonitor/lib/python3.5/site-packages/pkg_resources/__init__.py", line 2386, in resolve
    module = __import__(self.module_name, fromlist=['__name__'], level=0)
  File "/home/louis/.virtualenvs/pypimonitor/lib/python3.5/site-packages/pigar/__main__.py", line 12, in <module>
    from .pypi import update_db, check_latest_version, search_names
  File "/home/louis/.virtualenvs/pypimonitor/lib/python3.5/site-packages/pigar/pypi.py", line 19, in <module>
    from .extractor import Extractor
ImportError: No module named 'pigar.extractor'
$ python --version
Python 3.5.1+

I can provide more information on request.
Regards

OK, fixed.

Fixed. Thanks.


BVTCK-193 Exclude the Maven wrapper directory from the license check

https://hibernate.atlassian.net/browse/BVTCK-193

IMPORTANT: this PR is to be applied to the 2.0 branch.

Applied to 2.0 and also cherry-picked to master.


[Wallet] Add error related analytics

Description
Add error related analytics to make verification issues easier to debug
Tested
Just adding analytics, have not tested locally
Related issues

Fixes #4131

Send events already added in Nam's PR here: https://github.com/celo-org/celo-monorepo/pull/4165


Add angular.$q.prototype.resolve definition to 1.5

Per https://code.angularjs.org/1.5.9/docs/api/ng/service/$q, resolve is an alias of when so I just copied the definition here. For some reason, those typedefs do not include several optional parameters but I'll keep this change light just in case there's good reasons to have it like that.

@papadopa Looks like this requires fixing too many things internally so I'm not sure if it can land :( Maybe you can try migrating to Angular 1.6?

Forgot to reply here. I pointed @papadopa to a related thread internally. Closing this.


add image to stickerview?

Hey is it possible to add an image/imageview to the sticker imageview?

@Jarish have you added image to sticker imageview ? i also want to do same task.


Mod doesn't install right

When I install it through thunderstorm it doesn't work the only way to do it is to install it manually

Can you point to the specific thunderstore package you are referring to, this patcher is used for multiple games.

valheim


update deps in tox

Coverage remained the same at ?% when pulling ee7709daf14c153f12c7bd5807ceed2c87fbc4ae on auvipy-patch-1 into 867ef130e25a6adfec74946140851f7659f263b3 on master.


Resize filesystem at first boot

The first boot should resize the filesystem.
See https://github.com/hypriot/image-builder-nvidia-shieldtv/pull/3

This is fixed by #12.


Suggestions on pd.merge

Issue Report Checklist

[ ] Searched the issues page for similar reports
[ ] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[ ] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
[ ] Could not reproduce inside jupyter qtconsole (if console-related)
[ ] Tried basic troubleshooting (if a bug/error)

[ ] Restarted Spyder
[ ] Reset preferences with spyder --reset
[ ] Reinstalled the latest version of Anaconda
[ ] Tried the other applicable steps from the Troubleshooting Guide

[ ] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
In data manipulation, it's a common step that you need to link two database together. But very oftenly, we will met a problem: you need to merge two database replying on more than one individual symbols. For example, you can link the firms with patent on CUSIP symbols, or just firm's name, but it's better to use them both.
Demand: I have Dataframes "Firm" and "Patent", I want the "Merged" DataFrame.
Firm=pd.DataFrame(np.array([('00001',np.nan,'3000B'),('00002','Banana','2000B')]),columns=['CUSIP','name','market_value'])
Patent=pd.DataFrame(np.array([('00001','Apple','2'),(np.nan,'Banana','3')]),columns=['CUSIP','name','patent_number'])
Merged=pd.DataFrame(np.array([('00001','Apple','3000B','2'),('00002','Banana','2000B','3')]),columns=['CUSIP','name','market_value','patent_number'])
I think an "UPDATE" function is suitable. Here is how it describe: update missing values of same-named variables in master with values from using. If not the same-named, you can just normally merged.
For now, I need to write code like below:
Merged=pd.merge(Firm,Patent[['CUSIP','patent_number']],on=['CUSIP'],how='left',indicator='_merge')
Merged_left=Merged[Merged['_merge']=='left_only']
Merged=Merged[Merged['_merge']=='both']
Merged_left=pd.merge(Merged_left.drop('patent_number',axis=1),Patent.drop('CUSIP',axis=1),how='left',on='name')
Merged=Merged.append(Merged_left)
And of course, I still miss the "Apple" firm name.
What steps reproduce the problem?

What is the expected output? What do you see instead?
Paste Traceback/Error Below (if applicable)

PASTE TRACEBACK HERE

Versions

Spyder version:
Python version:
Qt version:
PyQt version:
Operating System name/version:

Dependencies

PASTE DEPENDENCIES HERE

Hi @mIgLLL,
It seems you don't have any error with Spyder, rather a problem with pandas. For this, please create an issue in the pandas issue tracker because we are unable to fix it here.
Thanks for reporting!


Suggested way to implement location still triggers errors from the toolkit

I recently found the toolkit for validating ARM templates and was running my templates through this tool. One of the errors I encountered was the "Location should not be hardcoded". I checked my template and I use a parameter 'location', that has a defaultValue '[resourceGroup().location]', which the error-message states "must not have a defaultValue property". I was looking for a decent solution, as I like this as a fallback/default, but maybe there was a beter solution. But then I found this page, encouraging me to do what I did, setting the defaultValue to the location of the resourceGroup, eventhought the tool says this is not the way to go, so I'm confused and unsure if the documentation is incorrect, or if this is a bug in the toolkit.
Thanks for the toolkit btw, I's really nice to run templates through to make sure they're all up to the same standard.

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

ID: 09b0964f-2681-1293-5370-7e2cb03a2935
Version Independent ID: 751c977d-2ab5-c304-155c-8caeac141645
Content: Test cases for test toolkit - Azure Resource Manager
Content Source: articles/azure-resource-manager/templates/test-cases.md
Service: azure-resource-manager
Sub-service: templates
GitHub Login: @tfitzmac
Microsoft Alias: tomfitz

@SeppeDev Thanks for your comment. We will review the issue and get back to you shortly.

@SeppeDev - the approach of using a parameter with a default value of resourceGroup().location is definitely the recommended way to set location. I have run into the scenario you describe a couple of times. For me, there were a few times when most of the resources were using the location parameter but one resource in the template was hard-coded. The other possibility is that the test toolkit is evaluating another template in the folder. You might want to double check the name of the file that contains the warning.

Aha, yeah that'll be the problem, the name of my file is not one approved as '$isMainTemplate'. So it'll probably validate my template as a linked template, which are not allowed to have a defaultValue for the location-variable. So changing the fileName to e.g. 'mainTemplate.json' should solve the error popping up, right?

@SeppeDev - I tested a scenario similar to your scenario. Yes, you're right. Changing the name to mainTemplate.json made the warning go away.

I added some clarification to that section about linked and nested templates not using a default value for location. It should get published tomorrow.
#please-close


[Facebook] Fails to download

youtube-dl -v https://www.facebook.com/FetishDollEmily/videos/832207340241477

[debug] System config: []
[debug] User config: [u'--download-archive', u'~/.ytdlarchive', u'--retries', u'100', u'--no-overwrites', u'--call-home', u'--continue', u'--write-info-json', u'--write-description', u'--write-thumbnail', u'--write-annotations', u'--all-subs', u'--sub-format', u'srt', u'--convert-subs', u'srt', u'--write-sub', u'--add-metadata', u'-f', u'bestvideo+bestaudio/best', u'--merge-output-format', u'mkv', u'--embed-subs', u'--prefer-ffmpeg']
[debug] Command-line args: [u'-v', u'https://www.facebook.com/FetishDollEmily/videos/832207340241477']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2016.02.13
[debug] Python version 2.7.10 - Linux-4.2.0-27-generic-x86_64-with-Ubuntu-15.10-wily
[debug] exe versions: ffmpeg 2.7.6-0ubuntu<IP_ADDRESS>, ffprobe 2.7.6-0ubuntu<IP_ADDRESS>, rtmpdump 2.4
[debug] Proxy map: {}
[debug] Public IP address: <IP_ADDRESS>
[facebook]<PHONE_NUMBER>41477: Downloading webpage
ERROR: An extractor error has occurred. (caused by KeyError(u'instances',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 315, in extract
    return self._real_extract(url)
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/extractor/facebook.py", line 189, in _real_extract
    for item in server_js_data['instances']:
KeyError: u'instances'
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/YoutubeDL.py", line 666, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 321, in extract
    raise ExtractorError('An extractor error has occurred.', cause=e)
ExtractorError: An extractor error has occurred. (caused by KeyError(u'instances',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Yeah --ignore-config gives me the same result.

Duplicate of #8572. This is already fixed in but not included in the latest release yet.


Add support for multiple outbound foreign keys constraints

This adds support for tables with multiples outbound foreign keys constraints. This is a modification of pull request #36 who allowed only one foreign key constraint.
The original problem in issue #34 was that lhm tries to create a table with foreign keys constraint having the same name than the original table, producing an error. #36 tried to fix it by modifying the last number at the end of the name. table_ibfk_1 would then become table_ibfk_2.
The problem was that the new constraints names were the same when the table had more than one outbound foreign keys, producing the same error than before. Another problem with that solution is that the names produced are not consistent when you undo or redo a migration. The original names are lost forever, so undo a migration won't give you the same structure than before. The new name depends on the original name, so redo a migration may not give the same structure either.
My solution appends a suffix "_lhmn" to the constraint name. If the suffix is already there, we remove it in the new name. If you apply two change_table, the constraint names will then be exactly as before. This allows migrations undo and redo to work as they should. Besides, the uniqueness of constraints names is no longer a problem, making lhm support as many outbound foreign key constraints as necessary.

What is needed to get this PR merged? Can we help on getting it merged?

Bump, I'd also like to use this, is anything I can do to help get this merged?

Yes, this is a problem, and we need to fix it.
I like the solution you proposed, however I am having a bit trouble follow the code, Can you squash all commits in one and rebase this from latest master.
After that I can give a last look and merge it in.
thanks

I've squashed the commits and rebased the branch.

@arthurnn how does it look? We've been using @pomier's fork in production at PriceMatch for a while now and haven't run into any issues.
Thanks!

:+1:

Now that we only support ActiveRecord, would be cool to use the FK method they provide https://github.com/rails/rails/blob/4-2-stable/activerecord/lib/active_record/connection_adapters/abstract_mysql_adapter.rb#L538-L548 , Only problem is that is only on rails 4.2+ . So maybe we should copy/paste that logic, if we are running on rails 4.2- . WDYT?

I think we should focus on getting this out instead of continuously commenting / rebasing / refactoring once every six months. It's tested, and it's been working live for months now. It won't be too late to refactor it later to include the relevant rails 4.2 logic if that really stands out as a priority.

@arthurnn @hannestyden I would be most thankful if you could merge this ? Thanks guys.

:+1: anything remaining work here?

Just ran into this problem today. Thanks for putting in the work to resolve this!

Any update on this? Would love to see this merged.

:+1: Also running into this problem. Thanks for looking into the fix

Just got bit by this one too... :+1:

It's been almost a year since this PR was made. Can we get a maintainer to chime in on what it would take to get this merged in (if at all)?

I've rebased this PR from latest master.
The tests are now randomly failing the same way it currently happens on master branch. Apart from that, everything is ready to be merged any time now.

@arthurnn not sure if you are the right guy, but maybe you can point to the right person. What needs to be done to get this merged? It's the only thing preventing me from using this, and right now i have to resort to downtime migrations :disappointed:

@trobrock you can always use @pomier's fork directly in your gemfile:
gem 'lhm', git: 'https://github.com/pomier/lhm', branch: 'outbound-foreign-keys'

It's been rebased on this repo recently which means there are little drawbacks to using this fork IMO.

Kind of frustrating that project maintainers aren't moving at all on this PR. This is incredible work and would really improve the gem.

I wish this PR was merged.  It would solve the problem I have this morning.

Guys, any updates with this merge?

@arthurnn are there still changes required before this can be merged, or are you looking for a totally different direction? We'd like to avoid using a fork if at all possible.

Hey Guys, Can we get this merged and avoid fork?

@arthurnn We would love to get this in - and are willing to help with this if any changes are needed and help is desired. Can any maintainer let us know whether this is a good direction for the solution, and/or what we can do to help?

Any chance of getting this merged? It'd really help my project.

door stuck

It's 2020!!!

It's 2021!!! Knock-Knock...

It's 2021!!! Knock-Knock...

FWIW, the Shopify fork is more active these days.

FWIW, the Shopify fork is more active these days.


Add supervisor(like systemd stuff) to auto restart Alluxio processes … [skip ci]

…in case of crashes for Databricks Docker container
Contributes to #6459
Add supervisor for Alluxio. Supports Alluxio processes automatically starting when encountering crashes.
Docker image by default does not support systemd, it's heavy to install and configure. Usually, docker uses supervisor instead of systemd.
Signed-off-by: Chong Gao<EMAIL_ADDRESS>

This is also a sync from GitLab code to GitHub code.
After this the codes of GitLab and GitHub are identical.

build


Implement the rpc handler method for RegisterAccount

Description

The suggestions in this discussion have been pulled into this PR.
closes: #2055

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[x] Targeted PR against correct branch (see CONTRIBUTING.md)
[x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards.
[ ] Wrote unit and integration tests
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added relevant godoc comments.
[ ] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Review Codecov Report in the comment section below once CI passes

Codecov Report

Merging #2110 (76df541) into main (915d974) will increase coverage by 0.04%.
The diff coverage is 82.60%.

@@            Coverage Diff             @@
##             main    #2110      +/-   ##
==========================================
+ Coverage   80.03%   80.07%   +0.04%
==========================================
  Files         170      170
  Lines       11813    11828      +15
==========================================
+ Hits         9454     9471      +17
+ Misses       1941     1939       -2
  Partials      418      418

Impacted Files
Coverage Δ

...7-interchain-accounts/controller/keeper/account.go
82.35% <71.42%> (+4.57%)
:arrow_up:

...nterchain-accounts/controller/keeper/msg_server.go
100.00% <100.00%> (+100.00%)
:arrow_up:


Fix audiobook playback

Fix audiobook playback

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (0e295f2) to head (af411ed).
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #346   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            4         4
  Lines          508       508
  Branches        20        20
=========================================
  Hits           508       508

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


pytorch distributed multi-gpu training

Hi, thanks for the great project!
I need to train in a multi-gpu setting and the pytorch init step requires something like
import torch.multiprocessing as mp

def main_worker():
   ....

mp.spawn(main_worker, nprocs=4, args=(4, myargs))

I'm wondering what changes do I need to make for adapting sacred. Is the below correct?
import torch.multiprocessing as mp

@ex.main
def main_worker():
   ....

if __name__ == '__main__':
        mp.spawn(ex.run_commandline(), nprocs=4, args=(4, myargs))

I assume you meant mp.spawn(ex.run_commandline, nprocs=4, args=(4, myargs)) without the parenthesis after run_commandline. If you keep the parens the experiment is executed and mp.spawn gets executed afterwards which is certainly not what you want.
Without the parenthesis, ex.run_commandline is executed in every process, so every process creates its own run, i.e., you end up with multiple entries in your experiment database. You (probably) want to call ex.run_commandline once and inside your main function you want to call mp.spawn to initialize distributed training:
import torch.multiprocessing as mp

def main_worker(...):
    ...

@ex.automain
def main():
    mp.spawn(main_worker, ...)

And if this works as desired actually depends on what you are doing in there. I think that metrics reporting and similar stuff doesn't work within the spawned processes (because it copies the observers, but only the main observer will ever save metrics/artifacts...). As long as you use sacred only for config handling and output capturing, this variant should be fine. Note, however, that the main worker cannot be a captured function because the main worker has to be a pickleable function.

Hi @thequilo , thank you for your reply. This seems to be what I'm looking for. In single GPU setting, In the function main_worker, I'm also collecting some metrics and data with ex like this:
def main_worker(...):
    ex.add_artifact(...)
    ex.log_scalar(...)
    ...

You said "main worker cannot be a captured function because the main worker has to be a pickleable function." That does not affect this purpose, right? And only the main process will do the loggings, not the spawed processes?

You said "main worker cannot be a captured function because the main worker has to be a pickleable function." That does not affect this purpose, right?

I'm a little bit confused at this point. When using mp.apply_async, ex is available in the global scope as expected. But when I use mp.spawn, it seems that the run is not initialized correctly in the spawned processes. I don't know why, but it seems that ex.current_run is for some reason not accessible in the spawned processes when using torch.multiprocessing.spawn.

And only the main process will do the loggings, not the spawed processes?

This is partially true. Only the main process will emit heartbeat events to the observers to perform the logging of scalars and the experiment state (scalars are also collected in all sub-processes but never passed to observers). As far as I know (and I might be wrong), only add_artifact behaves differently. It directly saves the artifact and writes to the run info when the artifact is added. This will also happen in the sub-processes and (temporarily) overwrite the run info with the sub-process local run info. This means that you can end up with saved artifacts that are not listed in the run info and a (temporarily, until the next heartbeat event) corrupted run info.
Also, in mp.spawn, the main process doesn't execute main_worker, it only manages the creation and termination of the sub-processes. So I am starting to think that accessing the experiment in main_worker beyond configuration management is currently not possible.
Depending on your use-case, e.g., if you just train a model on multiple GPUs, it might be worth using DataParallel instead of multiprocessing.

@thequilo
Thanks you! it seems using pytorch DistributedDataParallel(DDP) with sacred is not that straightforward as I imagined. The reason I used DDP rather than DataParallel is that DataParallel will introduce imbalanced usage of GPUs (such as discussed here) and is slower than DDP. I will need to dig around and see what is the best solution.

@Guptajakala
Hi, I encounter the same issue recently. The official docs say that the calculations on different GPUs are independent if we use PyTorch DistributedDataParallel. This means all the intermediate states are not shared between GPUs like accuracy and other metrics. Thus, we have to collect states in the main worker and record them to sacred. I don't know whether my understanding is correct. Do you have any progress?

No news? Fix, Workaround, anything?
I'd love to use sacred but this makes it impossible to log scalars and stuff. This is unfortunate.

Anyone know any solution or nice alternatives? I love sacred but it still cannot work well with distributed training, which is becoming more and more frequent nowadays.

If anyone would prepare a PR with a solution, I would be happy to accept it. But I currently don't need distributed training and don't have the resources (both time and hardware) to implement and test a solution for this.


Add tags support behind feature flag

Tags are stored as Newtype Structs with the special name _TagStruct. The tags
are represented as a struct tuple consisting of the tag identifier and a value.
Example:
let tag = Value::Tag(123, Box::new(Value::Text("some value".to_string())));
let tag_encoded = to_vec(&tag).unwrap();

This implementation is heavily based on the msgpack-rust library, when support
for extensions were added 1. They have the same problem as CBOR has, that
such an extension/tagging type is not directly supported by Serde, hence a
workaround is needed. For me it makes sense to use a similarly working workaround
across different libraries.
This commit also contains an example on how to use custom types as tags. It can
be run via:
cargo run --example tags --features tags

This is an alternate proposal to https://github.com/pyfisch/cbor/pull/129.

I really thought this is a good idea. But after digging into all the discussions about tag support in Serde, I also come to the conclusion that this should be properly solved within Serde so that there's interoperability between different formats.
I would want to revive the approach take at https://github.com/serde-rs/serde/pull/408. @pyfisch do you still have your tagged_values branch of cbor lying around somewhere (no worries if not, it shouldn't be too hard to implement)?

Hi @vmx, integrating with serde is a great idea. The tagged_values branch is still available on GitHub isn't it? https://github.com/pyfisch/serde/tree/tagged_values
I'd probably do some things differently than in this old branch. For example I'd use a TaggedVisitor as suggested in the discussion.

I will send a PR for the serialization part to the serde repo soon.

@pyfisch Wow, that's great to hear. I so far have ported the old version to master and I started playing around with it. I thought about creating a small example with a tag that is used in both, CBOR and msgpack, which is then (correctly) serialized/deserialized.
So please let me know if the new version of the patch for Serde is up. I really want to see this happen and I also have time to spend on this if there's a chance to get this upstream :)

See also #157

@pyfisch I haven't done that much, I pushed whatever I currently have though:

Serde: https://github.com/vmx/serde/tree/wip-revive-tagged-values
CBOR: https://github.com/vmx/cbor/tree/wip-revive-tagged-values

I kind of stopped when I saw that you're working on it again. But I'm happy to help, when you tell me what I should work on (my Serde knowledge is still pretty basic, though).

Tags are on master, so I guess we can close this.


Update lab4.md

Add steps on how to install inference process ti the ingest pipeline

@dadoonet of course I will.
I am working on testing the solution on my deployment ;)

@dadoonet I addressed all your comments, I let you have another review then I merge ;)

@dadoonet I already did. But I kept it in both step0.md & lab4-soluce.md
Do you prefer to keep it only in step0.md?

@alsayasneh My bad...
I meant lab0.md...


Create headers_DL_unsolicited.yml

Description
Add coverage for inbound messages using DLs to relay credential phishing.  This provides a generic method of detecting the technique in this blog
Associated samples

Sample 1
Sample 2

Associated hunts

Hunt 1

/update-test-rules


Do not hang when Internet connection is down

On Chrome, if my connection is cut, clicking on the extension icon hangs forever.
I'd like it not to hang & to show some status instead if possible.

When you write "the extension icon hangs forever", the FIP icon is presented with a yellow or red small square and when you click on the icon, nothing happens, no popup is shown?

Hm you're right it seems to be working sometime. Other times it really hangs forever but I can't reproduce that right now...

Ah, I don't know if I'm right, I was asking it as a question 😄 I wonder what does "hang" mean for you (a click with no action, playback never resumes when network is on, icon state does not change even though you're back online, etc.).
Let me know 🙂


SDK 1.10

SDK 1.10 has been released as part of the ETS2 / ATS 1.35 updates, which results in some major API changes (primarily multiple trailers, special transport, World of Trucks, etc).  Any chance on getting an updated SDK built?
(I'm currently trying to stumble through the code myself, but so far it's a bit over my head 😉 )

I am working on an update over here https://github.com/truckingvs/ets2-sdk-plugin. I have seemingly managed to update the telemetry plugin, but now got stuck a little bit while updating the client side class library.

For an repository with a few bigger changes, but already implemented SDK 1.10 https://github.com/RenCloud/scs-sdk-plugin. Note: Most gameplay events should work, but job declined seems not to be fired by the game at the moment.
It also have missing values of this repository added.

@RenCloud that looks very promising, I'll have to give it a try 😃


[SPARK-9955][SQL] correct error message for aggregate

We should skip unresolved LogicalPlans for PullOutNondeterministic, as calling output on unresolved LogicalPlan will produce confusing error message.

Merged build triggered.

Merged build started.

cc @marmbrus @rxin

Test build #40880 has started for   PR 8203 at commit 7593080.

Test build #40880 has finished for   PR 8203 at commit 7593080.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
 implicit class StringToColumn(val sc: StringContext)

Merged build finished. Test PASSed.

Merged build triggered.

Merged build started.

Test build #40951 has started for   PR 8203 at commit 1c67ca7.

Test build #40951 has finished for   PR 8203 at commit 1c67ca7.

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

Merged build finished. Test PASSed.

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

Thanks!  Merging to master and branch-1.5


oxlint should support lint categories in config file

By default only "recommended" rules are enabled.
I understand I can configure which rules run via the CLI of correctness vs suspicious, etc. as documented here: https://oxc.rs/docs/guide/usage/linter/cli.html#allowing-denying-multiple-lints
I'd like to be able to configure that in the .oxlintrc.json file so that the vscode plugin will respect these settings, as well as enforce consistency across developers on projects without needing to put all the details in the package.json scripts

@NateRadebaugh according to https://github.com/oxc-project/oxc/pull/6088 will be released soon

Thanks, closing


binfmt: ignore empty lines (#492)

This Pull Request closes #492.
According to https://freedesktop.org/software/systemd/man/binfmt.d.html#Configuration Format:

Empty lines and lines beginning with ";" and "#" are ignored.
Note that this means you may not use those symbols as the delimiter in binary format rules.

thx bud


Topic docker cmd

What?
Support for automating the launch and attachment of local NeoLoad controller and load generator containers adequate to run the current test-settings.
Why?
Tests need infrastructure to run, plain and simple. It is not always the case that these resources are already available when someone wants to simply run a quick test of their neoload project via NCLI. It should be quick and easy in a PoC as well as on a day-to-day basis to spin up infrastructure, run the test, and spin it down in an ad-hoc manner.
With the NCLI, the classic NLG is not assumed to be installed, even still that manual configuration of agents and attachment is a desirable part of the workflow described above. Why not do this for them by using Docker (if available)? Both on the desktop and in CI environments where Docker (socket) is available, this dramatically simplifies the process of "bringing your own infra to the table".
How?
Using the Docker API for Python, add a command "neoload docker" with lifecycle subcommands to handle attachment, detachment, preparation (settings only for chain command with "run" to auto setup/teardown), and forget docker settings activities. If docker isn't installed or working properly (ps command doesn't work), that's not our fault and fail any docker-related commands using this quick check.
'prepare' --> Simply translates the # of LGs, zone, and profile connection info into a spec remembered as metadata in the user profile. This comes in handy when wanting a "run" call to pre-run the docker attachment process and then the docker detachment process after the test is done.
'attach' --> Takes test-settings and options, remembers them in profile metadata, and actually automates Docker (whatever socket is attached, local or remote) to spin up one controller and X# of load generators (from test-settings) containers, providing them the proper NLW url, token, zone, and local mapping to each other. These containers will always be able to talk to each other directly.
'detach' --> stops any containers associated with the prior command 'attach'; the --all option also includes any containers (possibly from duplicate calls) to be decommissioned as well...a sort of "clean up everything" command, regardless of which came from the most recent attachment.
'forget' --> stops current containers (if running) and removes profile metadata so that subsequent calls to 'run' do not try to auto-start old docker infra specs.
Testing
Originally, I had integration-style tests in the tests folder, however we need a live system to test these. It cannot all be done with simple monkeypatch mocks. Though this works in Travis because the build node has a docker socket attached, I don't know if there's a better way. Currently we can keep these tests to requiring a live environment and not to be executed with the default [unit] test suite. If we make changes to this functionality, we should run the integration tests manually/locally, and maybe compensate later on with a nightly/weekly integration suite run somewhere else. Integration tests ARE required though for this functionality.
Supportability
We can explicitly call out that this is not the preferred production way to handle infra at large scale, but rather use dynamic infra or existing static zones. Since we do not support the customers local docker deployment or the Docker API libraries, it should be considered as "use for local development and at-your-own-risk". Support should not be troubleshooting the world of Docker on Windows and all it's faults. If the user can run a command line such as "docker ps" and get back non-error results, our docker command should work for experimental activities (such as sanity checks and CI when no infra is available).
Additional Notes
With docker and the neoload-cli installed, usage should be as simple as adding this to the TL;DR usage example:
neoload login $NLW_TOKEN
test-settings --zone $NLW_ZONE_DYNAMIC --lgs 5 --scenario sanityScenario createorpatch NewTest1
project --path tests/neoload_projects/example_1 upload NewTest1
docker prepare
run
Before executing the normal 'run' logistics, the run.py should see the prepared metadata, spin up the resources, then run the test, then finally spin them down. If the docker resources are already running (such as in an explicit call to 'attach'), the run command does nothing extra, neither spin up nor spin down. If prepared and the containers can't spin up, the CLI should fail there and not try to launch the test.
Each call to commands that might invalidate the current zone, such as login/logout/test-settings, should re-run the prepare and/or attach commands after switching test-settings. If attachment fails in this context, it should be a warning, not a hard error, requiring the user to re-run the appropriate command explicitly.
If long-running docker containers are preferred, these should be separate commands:
neoload login $NLW_TOKEN
test-settings --zone $NLW_ZONE_DYNAMIC --lgs 5 --scenario sanityScenario createorpatch NewTest1
docker attach
neoload project --path tests/neoload_projects/example_1 upload NewTest1
run
neoload docker detach
Both models are useful from the desktop when test suites are being written/tested/sanity-checked.
The attach/detach provides CI systems such as Jenkins explicit control over provably disposing of containers (in a finally or always block) if parts of the test fail but containers are still running.

For the docker command, this pull request is abandonned and replaced by the PR #124


rename HOST_IP env variables from the helm

The HOST_IP env variable in kuberenets is actually pointing to the internal minio url. So this env variable name should be rename to more proper name.

This should go to kuberentes repository. Closing.


feat: support custom migration name for db pull

What kind of change does this PR introduce?
feature
closes https://github.com/supabase/cli/issues/1612
What is the new behavior?
$ supabase db pull my_schema
Connecting to remote database...
Setting up initial schema....
Creating custom roles supabase/roles.sql...
Applying migration<PHONE_NUMBER>2533_test.sql...
Schema written to supabase/migrations/20231025092542_my_schema.sql
Update remote migration history table? [Y/n] n
Finished supabase db pull.

Additional context
Add any other context or screenshots.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

7 of 12 (58.33%)  changed or added relevant lines in 4 files are covered.
5 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.02%) to 58.594%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

cmd/db.go
0
5
0.0%

Files with Coverage Reduction
New Missed Lines
%

internal/gen/keys/keys.go
5
12.31%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
5669

Relevant Lines:
9675

💛  - Coveralls


Stop adding several copies of the same styles?

When you use react-toolbox, for some reason you get many copies of the same styles. For example, if you open the Web Inspector on react-toolbox.com, you can see 10 copies of the ::-webkit-scrollbar rule set:

Is this preventable?

Wow that's weird. The style is defined as a global in commons. I'll check in it during the week and see what happens. Thanks for reporting @alltom

This was happening because we were importing commons.scss in globals.scss inside the docs subproject. After that, globals was required in multiple files in order to import some general variables. Now it's fixed by moving commons scss import to the main App component.

Sweet, thanks. :)


Add completion handlers for operator hub

/kind feature
Which functionality do you think we should add?
We should add support for completion handlers for operator hub based services
Why is this needed?
Service completion handlers are being removed as part of #4906 we should add them back, implementing them for operator hub services

/close
Not relevant atm.


Kulve Taroth Gamma used in results, even though the event armor option is disabled

result: https://gyazo.com/7a72af0fb3a21b07a3ad68518b2160ea
option for KT gamma is disabled: https://cdn.discordapp.com/attachments/319524075976261643/542044207997517844/unknown.png

I can't reproduce this bug. Did you change the option to disable it after searching? Because it doesn't automatically update the search results when you change the options (maybe it should?)
If you click Advanced Search, does any KT gamma armor appear in the lists? Does searching again fix it?

No, I did not touch the option. I believe the check box is disabled by default, yet the gamma armor is enabled, if that makes sense.
If I download a fresh version and never touch the KT gamma checkbox (which is disabled by default afaik) the program still uses the KT gamma pieces. Once I toggle the checkbox on and back off it stops using them.
For me this only happened with the KT gamma set, but other event sets could also be affected by this bug.
I hope this helped

DLC armor should be enabled by default. On a fresh download of v0.60b, this works correctly for me (the checkbox is checked, the armor is used). Can you please double-check?
To be more specific, the steps I followed are:

Open fresh app (no settings.cfg file)
Check DLC options (all of them are already enabled), make no changes.
Select Weakness Exploit (a skill that appears on KT gamma waist piece).
Open Advanced Search, KT gamma waist is there.

Is there something I'm missing? Can you repro the bug from these steps?

Okay, i figured out some steps to reproduce this.

Download fresh app
(Starting a search with all KT Gamma armor skills and no decos yields some results using KT gamma parts)
Disable KT Gamma set
(Starting a search with all KT Gamma armor skills and no decos yields no results)
Disable KT alpha+beta sets
Reenable KT alpha+beta sets
Starting a search with all KT Gamma armor skills and no decos yields results using KT gamma parts, even though the KT Gamma set is still disabled

Ah, yeah that does the trick. Toggling the alpha/beta armor also affects the gamma armor. This also happens with Lunastra armor. Have fixed, thanks for the help!


Cloudflare refactory with Terraform

Change content and motivation
Setting Cloudflare domain with Terraform module instead of Ansible. It is very easy to forget to run the playbook when destroying.
GitHub cross-links

Fixes: fixes #35

Docs:  kubenow/docs#12

@andersla CI failed probably due to timeout in OpenStack... trying to restart...

Thanks @andersla! Pinging @jonalv and @StaffanArvidsson for test... that's a long wanted feature 🙂


Minor cleanups: return err, unused args...

Fixed some code warnings

rename server to s to not be the same as the import
remove unused arguments
return error for RegisterName

Codecov Report

Merging #123 (a4deb8a) into main (ad15293) will decrease coverage by 0.80%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##             main     #123      +/-   ##
==========================================
- Coverage   63.30%   62.50%   -0.81%
==========================================
  Files           3        3
  Lines         278      280       +2
==========================================
- Hits          176      175       -1
- Misses         85       87       +2
- Partials       17       18       +1

Flag
Coverage Δ

unittests
62.50% <33.33%> (-0.81%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

server/rpcserver.go
23.40% <20.00%> (-3.27%)
:arrow_down:

server/service.go
69.72% <100.00%> (ø)

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 ad15293...a4deb8a. Read the comment docs.

Thanks! I hope you're aware that the current work is in the rest branch - PR #119. Expecting to finish the rest API and merge that on Monday.


utils/mkdir-p: check if dir exists also after mkdir failed

with "make install -j8" it happens very often that two or more make
instances are creating the same directory in parallel. As a result one
instace creates the directory and second mkdir fails because the
directory exists already (but it did not while testing for it earlier).
Signed-off-by: Sebastian Andrzej Siewior<EMAIL_ADDRESS>

+1

+1 I'll merge now.

commit 70a56b9  Thanks!


Configure docker to collect core dumps from the server

Is your feature request related to a problem? Please describe.
Currently if the server crash no core dump is created for debugging
Describe the solution you'd like
Bootstrap need to

create a local folder for core dump collection for each container
Configure each container to write core dumps

More information on core - https://le.qun.ch/en/blog/core-dump-file-in-docker/


training result delayed by 1 step when running with tensorflow-gpu

Reproduced with tensorflow-gpu 1.13.2 and 1.15.0:
python3 -m kungfu.tensorflow.v1.examples
[<IP_ADDRESS>.10000::stdout] step 0, result: 1.000000
[<IP_ADDRESS>.10000::stdout] unexpected result: 1.000000, want: 0.800000
[<IP_ADDRESS>.10000::stdout] step 1, result: 0.800000
[<IP_ADDRESS>.10000::stdout] unexpected result: 0.800000, want: 0.640000
[<IP_ADDRESS>.10000::stdout] step 2, result: 0.640000
[<IP_ADDRESS>.10000::stdout] unexpected result: 0.640000, want: 0.512000
[<IP_ADDRESS>.10000::stdout] step 3, result: 0.512000
[<IP_ADDRESS>.10000::stdout] unexpected result: 0.512000, want: 0.409600
[<IP_ADDRESS>.10000::stdout] step 4, result: 0.409600
[<IP_ADDRESS>.10000::stdout] unexpected result: 0.409600, want: 0.327680

This seems to be a tensorflow problem.
When we run
v, _ = sess.run([x, train_step])

with tensorflow-cpu, v is evaluated as the value of x after train_step,
with tensorflow-gpu, v is evaluated as the value of x before train_step.


Remove "implements Function" from Kernel

This code:
class D implements Function {
  // ...
}

is equivalent to
class D {
// ...
}

in Dart 2.
However, in Kernel the "implements Function" is preserved even though "D" is actually not a subtype of "Function". This is very confusing for transformations which expect that the "supertypes" field of a "Class" will contain actual supertypes.

Note that there is one exception to this rule, which is _Closure in dart:core, which actually does implement Function. It would be nice if we could also avoid special handling of _Closure in transformations as well -- perhaps this could be controlled by the Kernel Target.

/cc @johnniwinther


How to register custom storage provider?

Hello,
I am trying to convert an Identity 2 custom storage provider to work with Identity 3 (without EF).
I'm having difficulty understanding how to register ASP5 middleware for Identity without EF.
At the moment I have done the following:
services.AddIdentity<ApplicationUser, IdentityRole>()
                .AddRoleStore<MySQL.RoleStore<IdentityRole>>()
                .AddUserStore<MySQL.UserStore<IdentityUser>>()
                .AddDefaultTokenProviders();

I'm not sure how I can also include the DI for my Database to be registered also?
I just want to ensure I am registering the middle-ware correctly (hopefully that will solve my problem).
I cannot find any documentation on this and registering EF seems to be done differently.
When clicking 'Register' or the default MVC6 template I am receiving an error of:
"InvalidOperationException: Unable to resolve service for type 'Bondii.Identity.MySQL.MySQLDatabase' while attempting to activate 'Bondii.Identity.MySQL.UserStore`1[Bondii.Identity.MySQL.IdentityUser]'."
My GitHub repository to see my code is:
https://github.com/simonpbond/bondii.identity.mysql
Original storage provider tutorial for Identity 2 - I am trying to upgrade:
http://www.asp.net/identity/overview/extensibility/implementing-a-custom-mysql-aspnet-identity-storage-provider

You can create your own extension methods to chain after the AddIdentity() call, e.g.
            services.AddIdentity<ApplicationUser, IdentityRole>()
                .AddSimonsMySqlStores()
                .AddDefaultTokenProviders();

It is really up to you whether the method will register all the services needed or whether you want to have registered separately.
For the default service registration we do in the templates we do have a separate registration for the EF services and also for the specific DbContext that then Identity uses, e.g.:
             services.AddEntityFramework()
                 .AddSqlServer()
                 .AddDbContext<ApplicationDbContext>(options =>
                     options.UseSqlServer(Configuration["Data:DefaultConnection:ConnectionString"]));

That makes sense in this case because EF and the ApplicationDbContext are usable outside of Identity.


Updates for ActivitiesFragment

Update the UI for Activities to align with iOS.
NOTE:  Before this will work properly, it will need an update to BridgeDataProvider.  However BridgeDataProvider needs some of the model tweaks included here so it is a bit circular.

Looks slick.  My main concern is on customization of the ViewHolders for the headers and normal cells.  If we could push that somehow to a listener and have it overridable in the Fragment, I think that would be a win.

I could make it so you can extend or override the entire TaskAdapter.  Do you think that would satisfy your goal for customizable UI?  The view holders and adapter are pretty tightly coupled so changing one may require a change in the other.  I could externalize the TaskAdapter class and allow you to inject your own.

@dephillipsmichael - I incorporated your feedback.  I don't have permission to merge though so please do so when you can.


[Bug] WASM bindings not compatible with React-Native app

Bug description
I have just tried to use the wasm bindings in a react-native project. To get access to node modules like fs which are required in the bindings I setup my project with rn-nodeify  which is a work-around to use node modules in an react-native app. But I stumbled on an error that prevents a react-native app from using the bindings.
React native is incompatible with the bindings because the bindings have a dynamic require in the identity_wasm.js file at line 10:
//identity_wasm.js line 10
const { TextDecoder, TextEncoder, inspect } = require(String.raw`util`);

This is not supported by metro (The javscript bundler for RN) which makes the bindings incompatible with react native.
Versions
@iota/identity-wasm 0.3.1
Steps To reproduce the bug
I have setup an example app that throws the error by startup. To reproduce the bug you will need to have the dev environment for react native setup.
git clone https://github.com/JonasHiltl/DigitalIdentityNodeified.git
cd ./DigitalIdentityNodeified
npm install
npx react-native start
npx react-native run-android

Expected behaviour
It would be great to achieve compatibility with react native which would allow us to build apps to interact directly with the identity library without having to call a node.js backend.
Errors
Error: Dynamic require defined at line 10; not supported by Metro

I just did some more research and this is more of a react-native problem and it's inability to access node modules. I'm able to require the util package after installing it manually via npm. But there are also other issues like that TextDecoder and TextEncoder are not included in this package. So I used  fastestsmallesttextencoderdecoder to make them accessible globally via a polyfill, which worked.
An issue I couldn't find a work-around for is that the BigUint64Array is not available in react-native, only in Browser and Node.js. It is used in the bindings which probably makes it impossible to use the bindings in a React-native project.

Thanks for the report!

the BigUint64Array is not available in react-native

What error are you getting? I haven't developed with React Native before, but according to the docs, the JS runtime for React Native is JavaScriptCore, which implements the ECMAScript standard and thus should also support BigUint64Array, right :thinking:?

I get the error ReferenceError: Can't find variable: BigUint64Array and the line below is the conflicting one:
//identity_wasm.js line 290
const uint64CvtShim = new BigUint64Array(u32CvtShim.buffer);

Turns out we had this problem sometime ago internally as well. The 0.3.1 version you're using is our main branch, which got its last update in May. The problem has since been fixed in the dev branch. You can use the dev tag on npm to use the newer version.

When using version 0.3.6 I now get the error  ReferenceError: Can't find variable: BigInt64Array, because in the new version  BigUint64Array got replaced with BigInt64Array.

Sorry for the troubles. Looks like we reintroduced the problem recently. We'll get started on a fix right away.

Version 0.3.7 is on npm now, and no longer requires any BigInt* or BigUint* imports, which should hopefully fix your issue.


Add examples to puppet-jira

@mkrakowitzer this commit uses an email that's not in your github profile


Client Activity Widget

Are the Active and Inactive parts suppose to be linked to the client page or is it by designed not to do anything?

At the moment they're designed to not to anything but inform you.


3352 fixes rounding error with the FMV on the storage locations page

Resolves #3352
Description
The full market value on the storage locations page shows a decimal but rounds all the values to the nearest dollar.
I looked in the view and the controller but the value is being rounded in the model.
The view gets the FMV from a method in the model that was returning an integer.
The inventory_total_value_in_dollars method in the storage_location model returns an integer.
Changed it to return a float and added a test to confirm

Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
There are already multiple tests for this method, I added one to check that the cents are being returned.
All other tests pass.

@cotabas About to merge your work into main.

@cotabas Thanks for this one too - just released!

Hey @cotabas -- thought you might like to see this feedback from a user:  "PS YAYYYYYY you and everyone else that did the coding so that the storage areas now show the actual cents of total inventory value. Small thing, but it is great to see our inventory value in HE match our QuickBooks inventory value to the penny. 😊 Thanks!"


Undefined variable error

I am installing for the first time but it gives this error. How can I solve it?
npm create svelte@latest .
npm i
npx svelte-add@latest bootstrap
npm i
Error: Undefined variable.
╷
20 │   @each $color, $value in $theme-colors-rgb {
│                           ^^^^^^^^^^^^^^^^^
╵
node_modules/bootstrap/scss/_root.scss 20:27  @import

Potentially caused by https://github.com/twbs/bootstrap/issues/36785#issuecomment-1189969247

I couldn't do it. Can you help me? What do I need to do? @babichjacob

Hey @turkic-dev. You pinged in Bootstrap repo. I managed to reproduce your issue here.
IDK how this ecosystem works but I can see there is a src/variables.scss in which you can add @import 'bootstrap/scss/maps'; which is missing (see Lean Sass imports).
Seems to work when I add it and relaunch npm run dev -- --open.
Hope it can help.

It's working! Thank you so much man you saved me ^^ @julien-deramond

Thank you so much @julien-deramond for the extra guidance on this issue! The lean Sass imports link is especially helpful since I didn't know this was documented and came up with a list of imports (and what order to do them in) on my own and ended up missing some — that's been fixed in https://github.com/svelte-add/bootstrap/commit/cf107b1c17f8c5921e51af4fb2ddbae3f1c54360.
Anyway, as it concerns this issue, thank you for the report @turkic-dev! This has been fixed in https://github.com/svelte-add/bootstrap/commit/06cd29437f44781beedb578ad81a0c05d81fb725 and published to npm in https://github.com/svelte-add/svelte-add/runs/7475220861


Check for extra arguments in WebsocketMessage constructor

newWebsocketMessage should report failure if len(closeCode) > 1, similar to how newResponse handles len(opts.rtt) > 1.
New test should be added - TestWebsocketMessage_CloseCode, which checks:

if closeCode is not passed to ctor, message Code() is zero
if closeCode is passed, message Code is the same
if multiple closeCodes are passes, ctor reports failure

hi @gavv, PR is ready for this one.

BTW I don't see any PR

I just pushed it. @gavv

Hi @gavv , can you see it now ?

Yep, thanks!


⚠ SceneTime has degraded performance

In 9105d05, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1461 ms

Resolved: SceneTime performance has improved in ab6191c after 6 minutes.


New Compatibility mod - power-armor-mk2 recipe error onload

This issue happens on load with the compatibility mod installed for me.
Failed to load mods: Error while loading recipe prototype "power-armor-mk2" (recipe): IngredientPrototype must be a dictionary and contain "type" property.
Shorthand format using an array is no longer supported in property tree at ROOT.recipe.power-armor-mk2.ingredients[0]
Modifications: Base mod › Space Age › 5Dim's mod - New Equipment › AAI Industry › 5Dim's mod - New Compatibility

it should work now


perf: ⚡ perf parse-value

my computer:
current:
running 6 tests
test citm_catalog_json_large_ast   ... bench:  69,412,450 ns/iter (+/- 25,243,601)
test citm_catalog_json_large_value ... bench:  88,551,560 ns/iter (+/- 38,142,516)
test package_json_ast              ... bench:      88,635 ns/iter (+/- 13,077)
test package_json_value            ... bench:     103,713 ns/iter (+/- 7,369)
test tsconfig_json_ast             ... bench:      32,696 ns/iter (+/- 14,447)
test tsconfig_json_value           ... bench:      32,818 ns/iter (+/- 1,638)

test result: ok. 0 passed; 0 failed; 0 ignored; 6 measured; 0 filtered out; finished in 62.98s

before:
running 6 tests
test citm_catalog_json_large_ast   ... bench:  68,363,600 ns/iter (+/- 15,877,140)
test citm_catalog_json_large_value ... bench:  96,260,580 ns/iter (+/- 22,216,507)
test package_json_ast              ... bench:     103,069 ns/iter (+/- 50,363)
test package_json_value            ... bench:     117,762 ns/iter (+/- 38,949)
test tsconfig_json_ast             ... bench:      28,376 ns/iter (+/- 2,011)
test tsconfig_json_value           ... bench:      29,111 ns/iter (+/- 1,894)

test result: ok. 0 passed; 0 failed; 0 ignored; 6 measured; 0 filtered out; finished in 67.75s

cc @dsherret， could you help for review?

Hey @IWANABETHATGUY, thanks for this. It is a little bit faster, but I would prefer to keep the public API only have std types. We could wrap it, but it seems that fxhash does not prevent against DDOS which I could see being a feature people would want in some use cases.
I previously took a look at what serde_json does here and they use a BTreeMap. My guess would be that it will be faster for parsing, but I would guess lookups will be slower depending on the size of the collection. I think in most cases people will be parsing small files so it would probably be better overall. You may want to try switching this to std::collections::BTreeMap.

using BTreeMap
test citm_catalog_json_large_ast   ... bench:  78,636,360 ns/iter (+/- 13,782,568)
test citm_catalog_json_large_value ... bench:  98,731,780 ns/iter (+/- 12,531,601)
test package_json_ast              ... bench:      99,940 ns/iter (+/- 50,865)
test package_json_value            ... bench:     139,453 ns/iter (+/- 52,373)
test tsconfig_json_ast             ... bench:      37,636 ns/iter (+/- 6,581)
test tsconfig_json_value           ... bench:      38,800 ns/iter (+/- 9,324)

std::HashMap
test citm_catalog_json_large_ast   ... bench:  80,079,050 ns/iter (+/- 15,920,799)
test citm_catalog_json_large_value ... bench: 107,082,760 ns/iter (+/- 15,669,294)
test package_json_ast              ... bench:      94,383 ns/iter (+/- 20,151)
test package_json_value            ... bench:     131,700 ns/iter (+/- 26,319)
test tsconfig_json_ast             ... bench:      31,697 ns/iter (+/- 7,448)
test tsconfig_json_value           ... bench:      32,712 ns/iter (+/- 9,258)

In fact, BTreeMap is slower than std::HashMap

Thanks for investing! That is interesting. I wonder why they used it.

but I would prefer to keep the public API only have std types. We could wrap it.

fxhash is pretty small, it would not introduce to much binary size, also I found that we already use serde_json as dependency.

but it seems that fxhash does not prevent against DDOS which I could see being a feature people would want in some use cases.

I am not sure who would use the user uploaded json as direct input and then do some logic, this sounds pretty weird, using json with comments as directly input sounds more weird.
I previously took a look at what serde_json does here and they use a BTreeMap. My guess would be that it will be faster for parsing, but I would guess lookups will be slower depending on the size of the collection. I think in most cases people will be parsing small files so it would probably be better overall. You may want to try switching this to std::collections::BTreeMap.

Thanks for investing! That is interesting. I wonder why they used it.

I am pretty sure , HashMap is faster than BTreeMap in most scenario, HashMap lookup just use nearly O(1), BTreeMap need O(h)
https://github.com/Rufflewind/bench-maps, here is the benchmark.
I guess they want the json entry by order?


version 1.8.0 generates infinite loop with retry configured to 0

aurelia-fetch version 1.8.0
the following code does not work and it keeps generating requests:
  config
    .rejectErrorResponses()
    .withRetry({
      strategy: 0,
      maxRetries: 0
    }));

@artmasa Haha, cool discovery 😆
Would be nice if you could PR to fix this 👍

Has this every been fixed?


ByteDance/AnimateDiff-Lightning text2vid support

Builds on https://github.com/livepeer/ai-worker/pull/51 to add the ByteDance/AnimateDiff-Lightning model.
Works, but models seems very VRAM hungry. Also getting minutes of inference time vs <!0 seconds of their demo, so something needs to be finetuned.

Sloppy code, but the motion LoRas are pretty neat

Note: this model seems to fall apart when generating longer videos. About 16 frames seems good. There is also a regular AnimateDiff, which should be able to hook into any (SD based) text2img model (there's a separate one for SDXL based text2img). It's much slower, but I've seen some really nice results coming out of that.


customRowAction Values

When switching to Board View on Microsoft List there doesn't seem to be a way to disable the default click behavior on the cards. Ideally I would like to override the default click behavior and replace that with an explicit click of a custom button added using JSON formatting. It doesn't appear there is a way to disable the default click behavior, the closest is to set the customRowAction value to "setValue" which has other unintended consequences such as a message being displayed indicating "Updated List Item". I am suggesting an option for disabling the click in the specification enum below:
"enum": [
"defaultClick",
"disableClick",
"executeFlow",
"share",
"delete",
"editProps",
"openContextMenu",
"setValue"
]

Hello @exwhyz
if you want to suggest new features and improvements, there is a feedback portal with a section dedicated to Microsoft Lists app: https://feedbackportal.microsoft.com/feedback/forum/ab3ad59e-6dd1-ec11-a7b5-0022481f35a4


BAU rollback passport upgrade

WHAT
Passport made some changes to sessions in 0.6.0 that are currently breaking the toolbox GitHub SSO integration

I have manually overridden the Snyk security check for this PR:


Argo CD Create/Delete Endpoints and Scaffolder Action

Feature Suggestion
Argo CD Create/Delete options for use with create plugin
Is this something desired be the community? If the contribution will be accepted, we can discuss open sourcing what we have already built. We would be moving from read-only to read/write permissions, however that can be controlled at the Argo CD level.

Possible Implementation
American Airlines has create and delete endpoints built into the backend plugin that is open sourced.
We use the delete endpoint in a custom catalog component.
For the create endpoint, we incorporated it into a scaffolder action and the above mentioned custom catalog component.
We are proposing to open source the create and delete endpoints on the backend as well as the scaffolder action for Argo CD create.

Context
Allowing Argo CD projects and applications to be created from the scaffolding create plugin.
Allowing deletion of projects and applications.
Right now due to code differences we manually pull-in contributed upstream code to our local and we'd prefer to contribute to open source.

Hi @karlhaworth Thank you so much for proposing this!
I think this sounds very interesting and useful and from my point of view we would be very happy to accept it.
As you say, the only concern would be around security so as long as that can be controlled through Argo itself or configuration (and the plugin plays nicely when the permissions are not granted for some actions) it should be fine.


Fixed-window rolling file pattern is not documented as being a relative path

I'm ok with this being closed as "wontfix" if I'm misreading something, but the documentation should probably mention that the pattern for a fixed-window rolling file is a path
I recently ran into an issue where our log file was /var/log/myservice.log, but our rolled logs were in /app/myservice.0.log because we thought pattern: myservice.{}.log was a pattern against the log file, not a path relative to the working directory
To be clear, the only thing that needs changing is the docs for log4rs::append::rolling_file::policy::compound::roll::fixed_window::FixedWindowRollerDeserializer

Hi @codehearts,
Thanks for opening this issue, I can see how that would be confusing. Clarifying the path that the roller pattern is relative to seems like a great idea. In fact it might be interesting to also add the note to the FixedWindowRollerBuilder as well.
Would you be willing to add those two things to the docs? Happy to help if you have any questions along the way!

It’s not a difficult fix, I should be able to get a PR up tomorrow! I’ll look around and see if any of the other docs could use some polish, it might also be nice to have a central location which shows all possible config options

Awesome! Lmk if you have any questions.

@codehearts If you have any better ideas for organizing the examples, please let me know!

I’m thinking the main docs can mention what functionality is available, and then link to the Deserializer docs for more info. I had a lot of trouble finding where some features in my config were documented because the search doesn’t seem to find results in the body text


[BUG] x-adminlte-input Undefined variable $errors

I have issue with x-adminlte-input, it returns error https://share.cleanshot.com/D1lzbQ73

@Globax what version of this package are you using? Can you check in what line of this package that error is generated?

@dfsmania
Package version is 3.8.5
Error comes from https://github.com/jeroennoten/Laravel-AdminLTE/blob/master/resources/views/components/form/input.blade.php line 5
There is $errors variable which is not defined anywhere.

That's strange, because that change is new and not included on a release still, however you may be using dev-master on your composer.json file. How is defined the route that redirects to that view, it needs to be using the web middleware, otherwise the errors bag from Laravel won't be available and so the error...

My composer.json https://share.cleanshot.com/ZGzg0CHw, i added package with composer require jeroennoten/laravel-adminlte by this instructions https://github.com/jeroennoten/Laravel-AdminLTE/wiki/Installation
And yes, issue was with web middleware, i use PHP 8 attributes (spatie/laravel-route-attributes) to define resources and i was defined only resource.
I added web middleware to controller and now it works.
Thank you for your quick help :)

I will keep this open, just to add a safety check to cover the issue when not using the web middleware.


Proxy Support?

Is it possible for you to add proxy support? As Digital Ocean IP range is banned.

Guzzle has the option to use a proxy, so that shouldn't be too hard. I might do a quick PR for this after work :smile:

See #83

Resolved, thanks @DrDelay


ModuleNotFoundError: No module named 'google.generativeai'

Description of the bug:
Using the following code:
import os
import google.generativeai as genai
I tried every method there was.
I even did:

pip install google-generativeai
pip install -U google-generativeai
pip install google-ai-generativelanguage

Please help me with this
Actual vs expected behavior:
Working in colab, but giving the error ; "ModuleNotFoundError: No module named 'google.generativeai'"
Any other information you'd like to share?
import os
import google.generativeai as genai
GOOGLE_API_KEY="---"
genai.configure(api_key=os.environ[GOOGLE_API_KEY])
prompt = input("You: ")
Generate a response
model = genai.GenerativeModel("gemini-1.5-flash")
response = model.generate_content(prompt)
print(response.text)

Hi @DronePower69
It worked fine when I tested it in Google Colab. You can check out the gist here. Please try running the code from the provided link.
Thanks

google is a namespace package. They have some strange behaviors.

Have you tried uninstalling the package?
Have you tried creating a fresh virtualenv?
Are you sure your pip is installing into the same python environment that you're running?

Considering you have a version of Python 3.8 or below, you need to use a version of Python 3.9 or higher. I had a problem with the 3.8 version. I've tested with Python 3.10.15, it works flawlessly.

@sukruburakcetin, yes, we don't support Python < 3.9. Please upgrade your python, I bet that will fix it.


Thanks @ksatrasala! 👏 Please let others know once it is packaged.

Thanks @ksatrasala! 👏 I am sure others would be very interested to get this update asap.


Revert to using cache action

This reverts back to using actions/cache and upgrades it to v3.
As mentioned in #5, there were some Python errors using the action as it is implemented on main.

Hmm. This will mean the return the Node warnings, yes?

Not in my experience since actions/cache@v3 uses node v16.

Great! I'm generally fine with this change if it works for everybody, though I have to confess I am curious why you're encountering the bugs. This upgraded take has worked for me.

I'm sorry it took me so long to merge this. I lost the thread.


why using radian instead of degree by default in service named MoveCartesian

Such inconsistent development specifications can easily cause confusion and are not friendly to the developer's experience.
In ROS2, is it more intuitive to use degree instead? Or is there another reason?
example command
# x: -297.7
# y: 193
# z: 5002
# r: 3.14 (radian)
# p: 0.1 (radian)
# y: 0.2 (radian)
ros2 service call /xarm/set_position xarm_msgs/srv/MoveCartesian "{pose: [-297.7, 193, 500.2, 3.14, 0.1, 0.2], speed: 50, acc: 500, mvtime: 0}

in UFACTORY Studio
x, y, z postions and r, p, y degrees.

in xARM Python SDK
the function named set_position() only need x, y, z.

Hi @celaraze, since ROS is mainly for developers, it will be convenient to use radian data for trigonometric and other calculations like rotation matrix, Euler angle to Quaternions etc. Most of the available math library functions require radian expression as input, this is the major reason.

Hi @celaraze, Ufactory Studio is mainly for display, jogging and simple programming, so it will be more important to be intuitive. ROS is targeted for developers, it will be more convenient to use radian data for trigonometric and other calculations like rotation matrix, Euler angles to Quaternions etc. Most of the available math library functions require radian expression as input, this is the major reason.
Sorry for the inconvenience, you can do a unit transfer or use Python API get_position(is_radian=True) to get the robot position in radian.

Thanks, I understand.


No type hints

Need to add type hints. It will make code more self-documented also it will make usage the library more simple and surprise-free.

Great idea! Added type hints for all functions