ModuleNotFoundError: No module named 'torch'

Hello,
I try to use your library for my Bachelor Thesis
When I run  sudo python3 setup.py develop in my conda environment this pops up:
(env_pytorch) dan@apollo:~/Documents/OpenPCDet$
Traceback (most recent call last):
  File "setup.py", line 5, in <module>
    from torch.utils.cpp_extension import BuildExtension, CUDAExtension
ModuleNotFoundError: No module named 'torch'

I have torch installed though and I can import the above modules
(env_pytorch) dan@apollo:~/Documents/OpenPCDet$ python
Python 3.6.12 |Anaconda, Inc.| (default, Sep  8 2020, 23:10:56)
[GCC 7.3.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import torch
>>> import torchvision
>>> from torch.utils.cpp_extension import BuildExtension, CUDAExtension
>>> exit()
(env_pytorch) dan@apollo:~/Documents/OpenPCDet$ which python
/home/dan/anaconda3/envs/env_pytorch/bin/python

I have installed all requirements:
(env_pytorch) dan@apollo:~/Documents/OpenPCDet$ pip3 install -r requirements.txt
Requirement already satisfied: numpy in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from -r requirements.txt (line 1)) (1.19.4)
Requirement already satisfied: torch>=1.1 in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from -r requirements.txt (line 2)) (1.7.1)
Requirement already satisfied: numba in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from -r requirements.txt (line 3)) (0.52.0)
Requirement already satisfied: tensorboardX in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from -r requirements.txt (line 4)) (2.1)
Requirement already satisfied: easydict in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from -r requirements.txt (line 5)) (1.9)
Requirement already satisfied: pyyaml in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from -r requirements.txt (line 6)) (5.3.1)
Requirement already satisfied: scikit-image in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from -r requirements.txt (line 7)) (0.17.2)
Requirement already satisfied: tqdm in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from -r requirements.txt (line 8)) (4.54.1)
Requirement already satisfied: numpy in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from -r requirements.txt (line 1)) (1.19.4)
Requirement already satisfied: llvmlite<0.36,>=0.35.0 in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from numba->-r requirements.txt (line 3)) (0.35.0)
Requirement already satisfied: setuptools in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from numba->-r requirements.txt (line 3)) (51.0.0.post20201207)
Requirement already satisfied: tifffile>=2019.7.26 in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from scikit-image->-r requirements.txt (line 7)) (2020.9.3)
Requirement already satisfied: matplotlib!=3.0.0,>=2.0.0 in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from scikit-image->-r requirements.txt (line 7)) (3.3.3)
Requirement already satisfied: scipy>=1.0.1 in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from scikit-image->-r requirements.txt (line 7)) (1.5.4)
Requirement already satisfied: PyWavelets>=1.1.1 in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from scikit-image->-r requirements.txt (line 7)) (1.1.1)
Requirement already satisfied: pillow!=7.1.0,!=7.1.1,>=4.3.0 in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from scikit-image->-r requirements.txt (line 7)) (8.0.1)
Requirement already satisfied: networkx>=2.0 in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from scikit-image->-r requirements.txt (line 7)) (2.5)
Requirement already satisfied: imageio>=2.3.0 in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from scikit-image->-r requirements.txt (line 7)) (2.9.0)
Requirement already satisfied: numpy in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from -r requirements.txt (line 1)) (1.19.4)
Requirement already satisfied: six in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from tensorboardX->-r requirements.txt (line 4)) (1.15.0)
Requirement already satisfied: protobuf>=3.8.0 in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from tensorboardX->-r requirements.txt (line 4)) (3.14.0)
Requirement already satisfied: numpy in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from -r requirements.txt (line 1)) (1.19.4)
Requirement already satisfied: typing-extensions in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from torch>=1.1->-r requirements.txt (line 2)) (<IP_ADDRESS>)
Requirement already satisfied: dataclasses in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from torch>=1.1->-r requirements.txt (line 2)) (0.8)
Requirement already satisfied: numpy in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from -r requirements.txt (line 1)) (1.19.4)
Requirement already satisfied: pillow!=7.1.0,!=7.1.1,>=4.3.0 in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from scikit-image->-r requirements.txt (line 7)) (8.0.1)
Requirement already satisfied: numpy in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from -r requirements.txt (line 1)) (1.19.4)
Requirement already satisfied: pillow!=7.1.0,!=7.1.1,>=4.3.0 in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from scikit-image->-r requirements.txt (line 7)) (8.0.1)
Requirement already satisfied: numpy in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from -r requirements.txt (line 1)) (1.19.4)
Requirement already satisfied: cycler>=0.10 in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from matplotlib!=3.0.0,>=2.0.0->scikit-image->-r requirements.txt (line 7)) (0.10.0)
Requirement already satisfied: kiwisolver>=1.0.1 in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from matplotlib!=3.0.0,>=2.0.0->scikit-image->-r requirements.txt (line 7)) (1.3.1)
Requirement already satisfied: python-dateutil>=2.1 in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from matplotlib!=3.0.0,>=2.0.0->scikit-image->-r requirements.txt (line 7)) (2.8.1)
Requirement already satisfied: pyparsing!=2.0.4,!=2.1.2,!=2.1.6,>=2.0.3 in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from matplotlib!=3.0.0,>=2.0.0->scikit-image->-r requirements.txt (line 7)) (2.4.7)
Requirement already satisfied: six in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from tensorboardX->-r requirements.txt (line 4)) (1.15.0)
Requirement already satisfied: decorator>=4.3.0 in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from networkx>=2.0->scikit-image->-r requirements.txt (line 7)) (4.4.2)
Requirement already satisfied: six in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from tensorboardX->-r requirements.txt (line 4)) (1.15.0)
Requirement already satisfied: six in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from tensorboardX->-r requirements.txt (line 4)) (1.15.0)
Requirement already satisfied: numpy in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from -r requirements.txt (line 1)) (1.19.4)
Requirement already satisfied: numpy in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from -r requirements.txt (line 1)) (1.19.4)
Requirement already satisfied: numpy in /home/dan/anaconda3/envs/env_pytorch/lib/python3.6/site-packages (from -r requirements.txt (line 1)) (1.19.4)

I have a RTX 2070 and CUDA installed
(env_pytorch) dan@apollo:~/Documents/OpenPCDet$ nvidia-smi
Fri Dec 11 11:49:25 2020
+-----------------------------------------------------------------------------+
| NVIDIA-SMI 450.80.02    Driver Version: 450.80.02    CUDA Version: 11.0     |
|-------------------------------+----------------------+----------------------+
| GPU  Name        Persistence-M| Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp  Perf  Pwr:Usage/Cap|         Memory-Usage | GPU-Util  Compute M. |
|                               |                      |               MIG M. |
|===============================+======================+======================|
|   0  GeForce RTX 2070    Off  | 00000000:01:00.0  On |                  N/A |
|  0%   34C    P8     5W / 175W |    180MiB /  7981MiB |      3%      Default |
|                               |                      |                  N/A |
+-------------------------------+----------------------+----------------------+

+-----------------------------------------------------------------------------+
| Processes:                                                                  |
|  GPU   GI   CI        PID   Type   Process name                  GPU Memory |
|        ID   ID                                                   Usage      |
|=============================================================================|
|    0   N/A  N/A      1107      G   /usr/lib/xorg/Xorg                 64MiB |
|    0   N/A  N/A      1495      G   /usr/bin/gnome-shell               86MiB |
|    0   N/A  N/A      1996      G   ...token=4170610791772182387       23MiB |
|    0   N/A  N/A      2725      G   /usr/lib/firefox/firefox            2MiB |
+-----------------------------------------------------------------------------+

Can you please help me?
Thank you for your time.

Try to use python setup.py develop rather than python3

env: Ubuntu 18.04 + Python3.6 + PyTorch 1.5 + cuda10.2 + spconv1.2.1
run demo.py
OSError: libcuhash.so: cannot open shared object file: No such file or directory

I had the same problem that seems to be caused by the sudo in sudo python3 setup.py develop.
Miss out sudo and use python3 setup.py develop --user worked for me.

I had the same problem that seems to be caused by the sudo in sudo python3 setup.py develop.
Miss out sudo and use python3 setup.py develop --user worked for me.

Thank you so much. It's working for me too.


support ember-network fetch mocking

This might not be common but I still believe an essential feature. Mirage/pretender should support fetch() requests.
I have one component that doest fetch request to a model endpoint however the endpoint is dynamic sometimes it is the same as the person model endpoint
I have this:
// in mirage/config.js
this.get('/people', ({people}, request) => {
    const params = request.queryParams;
    if (params.query) {
      return peopleSearch(people.all());
    }

    return people.all();
});

I have this call in a components:
// in components/foo.js

export default Component.extend({
    searchPath: computed('attribute', function() {
        const entityType = this.get('entityType');

        if (entityType === 'person') {
            return '/people';
        } else {
            return '/application';
        }
    },
    keyUp() {
        let query = $(event.target).val();

        if (query.length > 0) {
            // IMPORTANT PART:
            fetch(config.apiHost + `${this.get('searchPath')}?query=${query}`)
            // I do more with it
        }
    }
});

this fetch makes a request to the same endpoint as Ember 'person' model adapter. It would be great if we can make mirage support fetch calls. Currently I have to mock this with something like: https://github.com/wheresrhys/fetch-mock and duplicate the ember mirage code/logic.
P.S I will always use fetch instead of $.get() because that is what fastboot supports.

I think this ticket is related with how pretender could do it https://github.com/pretenderjs/pretender/issues/60

After some quick reading it looks like @stefanpenner recommends ember-fetch, could somebody verify this?
If so I think that should be the solution, ember-fetch polyfills fetch but never falls back to default (as of today), so Pretender should always work.

Yes, this is indeed the case, I think ember-network will get replaced by ember-fetch. ember-network also has caveats regarding its then/catch promise handling.

I have a similar use case here, it's a dashboard component that uses ember-fetch to load data. Is there any way to help with this or should we wait for pretender to be updated?

so i'm working on an app where we've

removed jQuery as a dependency
migrated to ember-fetch as a replacement for $.ajax / XMLHTTPRequest
2a) Configured fastboot to use node-fetch
configured Ember Data to use adapter-fetch to retrieve data using fetch

the last piece of the puzzle is getting mirage to work, and it seems like pretender would need to support fetch for this to work?

ember-fetch is currently always a polyfill and actually using XHR under the hood. Eventually we would like to fix that, but for that to work we need a "FakeFetch" implementation for pretender.

@rwjblue to rephrase: so ember-fetch currently never uses native fetch, and always uses XHR. To get mirage to work, we need pretender to use the the same "FakeFetch"™ that ember-fetch uses?

Partially correct, ember-fetch never uses native fetch and instead uses XmlHttpRequest which mirage mocks (via pretender).
Mirage should already work with this setup...

Feature requests are being moved to Canny.
View this feature request on Canny

@0xadada I've done pretty much what you did, but still not able to make it work with mirage.
Im using ember-cli-fastboot-testing and looks like is always trying yo fetch using node-fetch, so pretender is intercepting my requests.
Any clues of what I'm doing wrong?

@williamweckl I think it'd be easier to help if you opened a new issue with some more info/a reproduction

@samselikoff Actually I'm a little lost. Just want to test my ember app with fastboot but I'm not able to mock any requests to my API. Maybe it's not a mirage issue, I just hope that someone already did that and can show me the path to do it.
As you already saw, there is an issue at ember-cli-fastboot-testing.


[stable/falco] Add Amazon SNS integration

This allows Falco to publish alerts to a SNS topic
Signed-off-by: Néstor Salceda<EMAIL_ADDRESS>
What this PR does / why we need it:
This PR enables integration between Sysdig Falco and Amazon SNS. This allows to Falco to send its alerts to an Amazon SNS topic.
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)
- fixes #
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

/assign @bencer

/lgtm
/approve


Add the ability to add additionnal properties to queries

For the second example, I test that the attribute is not present, that's why only one VCR cassette changes.


Add local mode support

Some context: https://support.neurosity.co/hc/en-us/community/posts/360055931631-First-thoughts-on-Notion-DK1

Hey everyone,
I'm excited to announce that we just shipped Local Mode. Next time your OS updates, you'll be able to use this feature.
With local mode, device metrics like brainwaves, calm, focus, etc will stream via your local WiFi network and not the default cloud server.
Local Mode is disabled by default, to enable it via the Developer Console, you can go to Setting -> Application Settings:

You will see the "Local Mode" icon to the left.
To enable it while using the API:
await notion.enableLocalMode(true);

To disable it:
await notion.enableLocalMode(false);

Keep in mind:

Activity Logging will not work while this setting is enabled.
Your Notion must be connected to the same WiFi network as this device to establish communication.
For encryption to work, you must be connected to a private network.
Some WiFi networks may block IP ranges that might include your Notion's IP address.
An internet connection is still needed to authenticate, get device status and add metric subscriptions.
This setting is not global and needs to be set for every Notion app you wish to affect.

Documentation: https://docs.neurosity.co/docs/reference/classes/notion#enablelocalmode
Add good use case @JesseRWeigel pointed out is to connect your Notion to your phone's hotspot and enable Local Mode to use the device while driving.
Special thanks to @noopkat and @matthieu for pushing us to build this feature.


Feature/mcp (version 1)  "Greetings, programs!"

This has the java-side of MCP ready to use, and with plenty of implementations to be immediately useful:

Deep prune is a recursive folder removal tool, also useful for stopping sling jobs, purging workflow, etc.
Asset ingestor (aka Anteater V2.0) is a fully-functional high-speed asset ingestion tool
Folder relocator allows you to move or rename very large folders
Asset reporter allows you to get a detailed report on how much space binaries for assets are using

But not only does this pull request contain these tools, but the reusable framework (MCP) which makes building these tools incredibly easy.

Coverage decreased (-1.9%) to 23.718% when pulling d9d942236348ec12b093a6b90ebb806d8be071c6 on badvision:feature/MCP-V1 into 34342b0888baa38dde609a6d6e4ae48339812d96 on Adobe-Consulting-Services:master.

So high level comments...

The API surface here is gigantic and I'm not sure all of it is necessary. I'd ask that you take a second look at each exported class and make sure it is absolutely necessary. It is generally a bad sign when you have @Component classes in an exported package.
Coverage is going in the wrong direction.
The FrozenAsset thing is really problematic. I'll think a little about a different solution.
Missing ProviderType annotations (I think), although this goes along with the first item

It is an API for creating new solutions so most (if not all) of the exports are intentional.  I think that some of the model beans can be hidden (such as the archived ones) since I already export the interface.  What is the recommended package naming convention for that? ...model.impl?

Coverage decreased (-1.9%) to 23.718% when pulling 96176c081eb1b67643fbe1c4d56f8ad75be70970 on badvision:feature/MCP-V1 into 34342b0888baa38dde609a6d6e4ae48339812d96 on Adobe-Consulting-Services:master.

Coverage decreased (-1.8%) to 23.843% when pulling 8cebc962809ff26f014769596e128250624d01bf on badvision:feature/MCP-V1 into 34342b0888baa38dde609a6d6e4ae48339812d96 on Adobe-Consulting-Services:master.

Yeah, I understand it is going to have a large API. I'm just questioning how large. Keep in mind that it is always easier (trivial in fact) to take something which isn't API and make it API but the reverse is never true.
I think model.impl is fine.

Coverage decreased (-1.7%) to 23.889% when pulling d25a7286363bba48ee9f60eae9727fdc3191f8ef on badvision:feature/MCP-V1 into 34342b0888baa38dde609a6d6e4ae48339812d96 on Adobe-Consulting-Services:master.

Coverage decreased (-1.3%) to 24.325% when pulling ca740421f796016fc65c8ed854dea9af895a27fc on badvision:feature/MCP-V1 into 34342b0888baa38dde609a6d6e4ae48339812d96 on Adobe-Consulting-Services:master.

Coverage decreased (-0.5%) to 25.056% when pulling ef27df00b72311d10787edc1c559ab84ff5fcb74 on badvision:feature/MCP-V1 into 34342b0888baa38dde609a6d6e4ae48339812d96 on Adobe-Consulting-Services:master.

Coverage decreased (-0.3%) to 25.278% when pulling 9deda30c63f038694f495ec6483b2f99765b0248 on badvision:feature/MCP-V1 into 34342b0888baa38dde609a6d6e4ae48339812d96 on Adobe-Consulting-Services:master.

Coverage increased (+0.1%) to 25.701% when pulling 01061f853470d08ed1e27fb85163d2b265d687bf on badvision:feature/MCP-V1 into 34342b0888baa38dde609a6d6e4ae48339812d96 on Adobe-Consulting-Services:master.

Ok I have coveralls happy now, @justinedelson and @davidjgonzalez  --- any other showstoppers?

Coverage increased (+0.9%) to 26.471% when pulling 9a5df2b8269e456a8e2490db9680edcfa02a2e71 on badvision:feature/MCP-V1 into 34342b0888baa38dde609a6d6e4ae48339812d96 on Adobe-Consulting-Services:master.

Coverage increased (+1.1%) to 26.717% when pulling bcfaef2e70ab269d60de3014c2ff855b39895c9f on badvision:feature/MCP-V1 into 34342b0888baa38dde609a6d6e4ae48339812d96 on Adobe-Consulting-Services:master.

I somehow missed the front-end code in my initial review. Sorry about that. I see a lot of coral classes used. Are you sure that all of these are "public" classes. Per https://docs.adobe.com/docs/en/aem/6-3/develop/ref/coral-ui/coralui3/styles.html, only public classes are considered API. For example, I don't see coral-table-inner on https://docs.adobe.com/docs/en/aem/6-3/develop/ref/coral-ui/coralui3/styles.html#coral-Table. Just want to make sure we aren't opening ourselves up to a compatibility issue later.

Coverage increased (+1.1%) to 26.714% when pulling 6d63980d976b8a928f6de005ae56b005bc172e4f on badvision:feature/MCP-V1 into 34342b0888baa38dde609a6d6e4ae48339812d96 on Adobe-Consulting-Services:master.


no initialization of modules in referenced console applications

Create a solution with two Console Application projects:
ConsoleApplication1\Program.fs:
module SharedValue
let hello = "Hello" |> id

ConsoleApplication2\Program.fs (references ConsoleApplication1)
match SharedValue.hello with
| null -> printfn "SharedValue.hello is null!"
| _    -> printfn "all good"

Expected output: all good
Actual output: SharedValue.hello is null!
needless to say, getting a null value from an F# project is quite surprising.
Changing the type of ConsoleApplication1 to a Library fixes this.

i think is the same problem as fsharp/fsharp#326 , the console project doesn't initialize hello because you dont declare/execute the main function.
see @latkin 's comment for detailed explaination and workaround

Yrs, an explicit Main is required, and this is currently considered by design, for better or worse.
Adding a warning on referencing such a component may well be feasible.

What's best?

warning (error?) if a console app doesnt have a main
warning if a referenced fsharp assembly (discriminate on .exe extension?) doesn't have a main

#2 - emit an attribute or other bit when no main on .exe, and detect that when the component is referenced

#2 - emit an attribute or other bit when no main on .exe, and detect that when the component is referenced


Fix for #90 - Add stats to management api

Also added tests.

Thank you for working on this. I was traveling so it took me a few days to get back to you. I just saw your questions on issue #90 as well.
There is a lot of copy/paste on the API part. I do not think it is ideal, but I also do not see a better abstraction yet.

@hjr3 Alright! Very true, the idea between the workers and managers. I will try to re-implement something and get back.


Map Display Issue: Municipal Filtering Errors

map filtering - identify error causing a filter on munis

Have determined this is related to issues with simplified layer of data. Have temporarily switched to single full data layer, speed is slower, but data is accurate. When new simplified layer is finalized will look to reimplement and test that this issue does not re-appear.


⚠ Slack has degraded performance

In 4f5e6d0, Slack (https://status.slack.com/api/v2.0.0/current) experienced degraded performance:

HTTP code: 200
Response time: 42 ms

Resolved: Slack performance has improved in dbe6948.


05_02_Pipeline_lightweight missing import for kfp components

Step:
add_op = comp.func_to_container_op(add, base_image='python:3.6.8')
Fails with unknown identifier for comp.
Can be fixed by adding import to the function definition step:
#Define a Python function
import kfp
from kfp import components

Ah.. I have similar issues yesterday and I probably forget to import the libraries. Thanks. I will file a PR  to address this issue


Additional stories to choose from

I added four more stories with gender text placeholders - Alice in Wonderland, Caught Out, The Phantom Avenger, and Conan - Queen of the Black Coast.

Resolved via https://github.com/markfullmer/gendered_text/commit/307590de96a49de5af85fcaa0073324100db1158


Mark TestECSSuite/TestCPU e2e test as flaky

What does this PR do?
Mark TestECSSuite/TestCPU/metric___container.cpu.usage{^ecs_container_name:stress-ng$} as flaky.
Motivation
For an unknown reason, since recently, sometimes, stress-ng is consuming significantly less CPU than expected:

Additional Notes
Investigation notebook
Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


DialogV2 should have aria-modal="true"

Description
The DialogV2 component enables a focus trap and background overlay by default. This makes it a modal dialog by definition:

A section of content is "modal" means navigation is limited to the area itself and the background (the ancestors and siblings of the modal) is hidden. Setting aria-modal="true" on dialog and alertdialog role containers indicates the presence of a "modal" element to users of assistive technology

By default, items with role="dialog" have aria-modal="false", indicating to screen readers that navigation outside the dialog is possible. But it is not, so this is misleading and can get screen readers into a state where interaction appears possible but is not.
Explicitly setting the aria-modal="true" prop on the Dialog component does not work as a workaround because that prop is not forwarded to the underlying element.
Steps to reproduce
Render a Dialog component from @primer/react/drafts
Version
v35.17.0
Browser
Chrome

Wondering if we might want to close this issue in favor of https://github.com/github/primer/issues/1551 which will include using the PVC Dialog as a reference for markup and accessibility features.

Thanks for filing @iansan5653, and thanks for the link @langermank!
The PVC Dialog component has been approved by the accessibility team and we have plans to replicate the improved markup in React as part of https://github.com/github/primer/issues/1551. Feel free to follow progress along on that issue, or reach out if you're interested in contributing the change.

Would a PR be accepted for a quick fix to resolve this for now? This is a pretty significant accessibility problem for which there is no good workaround - it seems to me like it would probably be worth shipping the one-line fix for now even if this component would be replaced by that related work eventually.

Yes, we would be happy to review a PR contribution assuming it's a quick fix! Let me know if you want me to reopen this for you to track against.


Saving As of 500 Mb files results in a 0 byte size file being written

SQL Operations Studio Version:
Version 0.33.5
Commit 9ca6200018fc206d67a47229f991901a8a453781
Date 2017-12-15T12:00:00.000Z
VS Code 1.26.1
Shell 2.0.8
Renderer 61.0.3163.100
Node 8.9.3
V8: 6.1.534.41
Architecture x64

Steps to Reproduce:

Open a 500 mb tsql file in editor.
This gives a warning that the file is too big and turns of intellisense and other editing features on file. Do not force enable those features.

Make minor edits to this file and save this file. This succeeds.

Make some more edits and perform a Save As operation to a new file name. This ostensibly succeeds.

Go to the filesystem and check the size of the new file that was written.

Expected: A 500 mb file gets written to the new location.
Actual: A 0 byte size file gets written.

There have been many fixes in handling large files that we picked up from upstream.  Please open an issue against vscode if this is still occurring and can be repro'ed upstream.  Thanks!


http://stackoverflow.com/questions/8461832/explicit-qualification-in-…

…declaration

For those of you who are confused, the fix this PR is doing is as explained here: http://stackoverflow.com/a/30769937/139041
Meaning, Bad:
namespace MathUtils
{

MathUtils::mBuildHull2D(..)
{
   ..
}

} // namespace MathUtils

Good:
namespace MathUtils
{

mBuildHull2D(..)
{
   ..
}

} // namespace MathUtils

Seems to work.


pint 0.10, python 3.8, year 2020

Description
pint 0.10, python 3.8, year 2020
Changelog description
Already updated
Status

[x] Code base linted
[x] Ready to go

CI-wise, I think this is ready to merge. There's not anything preventing a conda env with qcel 0.12 (last release) and py38, so I don't see that a version bump is needed immediately or that the existing c-f package needs rebuilding, though the pint constraint should be added to the recipe at next edit.


Update README.md

Codecov Report

Merging #6 (c162c5f) into main (3c8a0b4) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #6   +/-   ##
=======================================
  Coverage   31.12%   31.12%
=======================================
  Files           3        3
  Lines         151      151
  Branches       17       17
=======================================
  Hits           47       47
  Misses         87       87
  Partials       17       17

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 3c8a0b4...c162c5f. Read the comment docs.


Migrated asp.net core best practice to performance article & moved code to samples

Fixes #14605
Internal Review URL
What was done:
Migrated: https://github.com/davidfowl/AspNetCoreDiagnosticScenarios/blob/master/AspNetCoreGuidance.m@
to : performance-best-practices.
Light edit and moved all code snippets into a samples project (3.0).
Project does not compile yet.  @Rick-Anderson will take over for next steps.

Fixing the code snippet links and topic links....

@Rick-Anderson, should be in proper shape now for what you plan to do with this on the next step.  Let me know if you need anything or I didn't do this how you wanted.  Thanks.

@wadepickett  I moved your commits to #14734

This was moved to branch directly off of aspnet/AspNetCore.Docs instead of my fork since Rick is also going to work on it.


[question] Some models being rendered extremely large

Hey @ThomasGorisse  thanks for maintaining the scene form SDK this library has been a life saver.
I just wanted to know if there is a known issue about some models being rendered in extremely huge sizes and some being rendered perfectly well.
https://sketchfab.com/3d-models/gladiador-874fc7e11d674f51a4dc2082d29c9b9a - This renders Perfectly and the size can be adjusted
https://poly.google.com/view/75WQH5E29tF  - This renders in extreme size I haven't been able to see the complete model and for some reason it makes the model extremely difficult to resize
https://glb-packer.glitch.me/ - I am using this to convert the models from GLTF to GLB
And again thanks a lot for the library.

Hi @akshatbahety,
Actually, the problem come with your plane that has an enormous size (181 meters) has you can see in the transform panel:

Comparing to the gladiator asset which has a "normal" height of 2.03 meters

You should definitely consider using Blender to reduce it size.
Just change the desired size on the same panel and export it again to glb.

Thanks a lot


Temporary fix for Android concerning  _handleStartShouldSetPanResponder

Until the PR https://github.com/facebook/react-native/pull/3426 is merged, we need to always return "true" in the method _handleStartShouldSetPanResponder, as discussed in https://github.com/jeanregisser/react-native-slider/issues/5#issuecomment-150227431.

Awesome @PhilippKrone!
I'm submitting 0.3.2 now to NPM.


SIMD printing with modcc.

Merges master branch.
Config/build changes:

Split arbor modcc vectorization option (now ARB_VECTORIZE) and target-architecture optimization (now ARB_ARCH).
For arbor and arbormech targets, and in particular not the modcc target, use ARB_ARCH to generate corresponding target-appropriate binaries, including, for example, appropriate SIMD support.
Extend CompilerOptions.cmake to map as best as able between the various target architecture names (we use the gcc names) and the correct option to pass to the compiler based on the compiler and platform.
Add work-around for misidentification by CMake of XL C as Clang.
As a temporary work-around, include arbormech library twice on link line to resolve circular arbor–arbormech dependencies.

SIMD class changes:

Incorporate SIMD module printing into modcc cprinter.
Add simd_cast operation between SIMD value types of the same width, and with std::array. (Note: this was tested and used in an early development version of the code, but not in this version. It was still a lacuna in the original SIMD wrappers, so it has been left in.)
Restructure SIMD gather/scatter API to use a simd::indirect expression, which encapsulates a pointer and SIMD offset.
Add simd::index_constraint scoped enum to describe knowledge of contention in indirect indices, so that we can branch on this to the appropriate implementation.
Add SIMD concrete implementation routines reduce_add for horizontal reduction and element0 for access to first lane scalar value.
Add SIMD value method sum() that exposes implementation reduce_add.
Add SIMD concrete implementation routine compound_indexed_add that provides the implementation for indirect(p, simd_indices) += simd_value construction.
Fix SIMD implbase bug where some static methods were using the implbase fall-back functions instead of the derived class specialized implementations.
Move SIMD mathematical functions into friend routines of simd_impl in order to resolve implicit conversions from scalars in mixed SIMD-scalar operations.
Use a templated tag class to dispatch on SIMD concrete implementation types, to avoid problems with incomplete types in method signatures.
Remove old SIMD intrinsics.

FVM class changes:

Add signed fvm_index_type (alias for int) for the representation of indices (such as CV index, ion instance index etc.) in fvm layouts and mechanisms. This was chosen as an alternative to making unsigned versions of all our SIMD implementation classes.
Make mechanism width_ a pointer into data_ for uniform access in generated modules.

Modcc/module changes:

Modify SIMD printing options for modcc: -s <simd-abi> specifies the vectorization target: none, avx, avx2, avx512, default, or native. -S <N> allows a specific data width to be prescribed.
Fix problem in test_ca.mod with uninitialized ion current.
Extend cprinter to incorporate SIMD-specific output.
Add infrastructure support to allow future pre-computation of SIMD index conflict cases for (hopefully) faster scatters and updates.
Remove old SIMD printing code.

Unit test changes:

Extend repertoire of generic sequence equality/near equality testing support in common.hpp.
Add warning suppression for icc for the malloc instrumentation code.
SIMD unit tests for indirect expressions, compound indirect add, reduction.
Make some exact tests into floating point 'near' tests when comparing computed areas and lengths in swc and fvm layout tests, to account for compiler (e.g. icc) performing semantically inequivalent floating point operation reordering or fusion at -O3.

TODO:

SIMD documentation needs to be updated to describe new API and functionality.
Unit tests need to be supplied for new modcc cprinter.
Padded index implementation for density channels needs to be re-examined, as currently we cannot take advantage of a faster index constraint than none. (I'll detail this in a new ticket.)

Superb, thank you for the update!

We can defer much of the low level documentation, however the installation guide should be changed to:

reflect the new vectorization options.
reflect changes to cross-compilation workflow.

ARB_ARCH should be inherited by the unit tests so that they test the architecure for which the library was compiled.

Also, add mechanisms/generated to .gitignore.

Regarding generated mechanisms, they really need to be put in the build directory. (Currently, maintaining multiple build directories requires manually cleaning out the mechanisms/generated directory, and it's also just good practice.)
Shall I move them in there in this PR?


Feature: invalid/rejected objects by repository

As a user operating a repository, I want to be able to monitor the number (or existence) of invalid&rejected objects per repository, so that I can monitor the health of my repository without false positives.
rpki-client gives overall statistics for which you can expose using a container but this is noisy for alerting since it is per trust anchor.
Ideally I would like stats per repository or the ability to limit the repositories being considered for validation.
rpki-client stats including the lacnic + RIPE tal:
# HELP rpki_objects Number of objects by type
# TYPE rpki_objects gauge
rpki_objects{type="roas"} 16356.0
rpki_objects{type="failedroas"} 0.0
rpki_objects{type="invalidroas"} 0.0
rpki_objects{type="certificates"} 11873.0
rpki_objects{type="failcertificates"} 0.0
rpki_objects{type="invalidcertificates"} 0.0
rpki_objects{type="manifests"} 11873.0
rpki_objects{type="failedmanifests"} 3.0
rpki_objects{type="stalemanifests"} 0.0
rpki_objects{type="crls"} 11870.0
rpki_objects{type="repositories"} 13.0
rpki_objects{type="vrps"} 88325.0
rpki_objects{type="uniquevrps"} 88325.0

Thanks for the suggestion! It is definitely a useful addition to the metrics. We’ll try and add it to the next release.

We are planning to substantially rewrite the logic for updating publication points for 0.9, so I think it makes sense to implement per-publication point metrics as an integral part of that work rather than now implementing something and then throwing it away.

I have a bit more experience with the wrapper for rpki-client I created a while back. It now reports detailed errors which allows me to alert on errors at the publication point level instead of per trust anchor. This allows me to alert per publication point.
For reference, besides the metrics I mentioned in the initial issue, I have the following metrics:
# HELP rpkiclient_warnings Warnings from rpki-client
# TYPE rpkiclient_warnings gauge
rpkiclient_warnings{hostname="rpkica.mckay.com",type="expired_manifest"} 1.0
rpkiclient_warnings{hostname="rpkica.mckay.com",type="missing_file"} 1.0
rpkiclient_warnings{hostname="ca.rg.net",type="missing_file"} 0.0
rpkiclient_warnings{hostname="rpki.ripe.net",type="bad_message_digest"} 0.0
rpkiclient_warnings{hostname="ca.rg.net",type="expired_manifest"} 0.0
# HELP rpkiclient_pulling Last time pulling from this repository was started (referenced).
# TYPE rpkiclient_pulling gauge
rpkiclient_pulling{uri="rpki1.terratransit.de/repo"} 1.613574761312486e+09
...
# HELP rpkiclient_pulled Last time repo was pulled (before process ended due to timeout).
# TYPE rpkiclient_pulled gauge
rpkiclient_pulled{uri="rpki1.terratransit.de/repo"} 1.6135747613125722e+09
...

An example instance of the whole set of metrics is available here

Do you think it is enough to just provide a number of, say, rejected repositories and leave it to a user to go hunt for details in the log? This would keep the amount of metrics from exploding.
I think I would rather add some sort of structured log later on. Given that a log happens in parallel, this may actually be a better approach anyway.


.IPA for the newest release

The TestFlight link is down for the newest release, could you upload an IPA for the newest version

Non-developer accounts lack the necessary NFC permissions to self-sign and install IPA files. Additionally, the TestFlight link appears to be functioning correctly at this time.


Residue rename

Fixes #351

Residue -> ConstMonty
DynResidue -> Monty
BoxedResidue -> BoxedMonty
residue_params -> params
params.r -> params.one

Please don't open other PRs before this one is merged, it's a pain to rebase.

Your description doesn't quite reflect the names, which are MontyForm instead of just Monty.
For the params names, I'd prefer MontyParams instead of MontyFormParams which seems needlessly long.
Otherwise yes, we should get this rename done ASAP.

Renamed


Correct value to set CFG_INT_TRACE_LEVEL_MIN

Is your question about how to consume a C++ library?
Yes
Looking at cpp_client_telemetry\lib\pal\DebugTrace.hpp:
 enum LogLevel {
        Error = 1,
        Warning = 2,
        Info = 3,
        Detail = 4
    };

    namespace detail {
        extern LogLevel g_logLevel;
        extern void log(LogLevel level, char const* component, char const* fmt, ...);
    } // namespace detail

and
#define MATSDK_LOG_ENABLED_(level_)   (PAL::detail::g_logLevel >= (level_))

If I want to log Error, Warning and Info, which value should I use for CFG_INT_TRACE_LEVEL_MIN? Shouldn't it be <= and not the other way around?

If I want to log Error, Warning and Info, which value should I use for CFG_INT_TRACE_LEVEL_MIN

CFG_INT_TRACE_LEVEL_MIN should be 3 to log for Error, Warning, and Info. <= is the correct operator to achieve that.

Thanks. That makes sense


adding runtime configuration structure

adding pre sandbox hooks
fixes #5

Test coverage increased. Runtime increased from 10.5% to 21.0%
--- b	2018-03-20 20:56:43.874972243 -0400
+++ o	2018-03-20 20:55:09.185916365 -0400
@@ -1,6 +1,7 @@
-ok  	github.com/automationbroker/bundle-lib/apb	0.063s	coverage: 4.1% of statements
-ok  	github.com/automationbroker/bundle-lib/clients	0.044s	coverage: 20.8% of statements
+ok  	github.com/automationbroker/bundle-lib/apb	0.049s	coverage: 4.1% of statements
+ok  	github.com/automationbroker/bundle-lib/clients	0.040s	coverage: 20.8% of statements
 ?   	github.com/automationbroker/bundle-lib/crd	[no test files]
-ok  	github.com/automationbroker/bundle-lib/registries	0.034s	coverage: 74.7% of statements
-ok  	github.com/automationbroker/bundle-lib/registries/adapters	0.082s	coverage: 6.0% of statements
-ok  	github.com/automationbroker/bundle-lib/runtime	0.038s	coverage: 10.5% of statements
+ok  	github.com/automationbroker/bundle-lib/registries	0.048s	coverage: 74.7% of statements
+ok  	github.com/automationbroker/bundle-lib/registries/adapters	0.090s	coverage: 6.0% of statements
+ok  	github.com/automationbroker/bundle-lib/runtime	0.039s	coverage: 21.0% of statements
+?   	github.com/automationbroker/bundle-lib/runtime/mocks	[no test files]


Example Code

It would be great to have some example code to show how to use this interesting library.  Trying to tease it out from the test cases has proven to be unsuccessful so far.

Hello, @pjakobsen! I'm having the same problem here: I'm implementing the Parquet rows plugin and needed to read the parquet-python's source code to know how to use it -- it's a difficult, non-pythonic way. So I've created a little helper function which can also help you:
from collections import namedtuple
import parquet

OPTIONS = namedtuple('Options', ['col', 'format'])(col=None, format='custom')

def import_data(filename):
    data, field_names = parquet.dump(filename, OPTIONS, lambda *args: args)
    length = len(data[field_names[0]])
    return [{field_name: data[field_name][index] for field_name in field_names}
            for index in range(length)]

The function is pretty straighforward to use, for example, this code:
parquet_rows = import_data('test-data/nation.dict.parquet')
for row in parquet_rows:
    print row

Will generate the following output (each row is a Python dict):
{'region_key': 0, 'nation_key': 0, 'name': 'ALGERIA', 'comment_col': ' haggle. carefully final deposits detect slyly agai'}
{'region_key': 1, 'nation_key': 1, 'name': 'ARGENTINA', 'comment_col': 'al foxes promise slyly according to the regular accounts. bold requests alon'}
(... more 24 rows ...)

@pjakobsen, you can now use my library rows to read and convert parquet files! :) More information on this blog post.

I took a try for a more pythonic API in #11. PTAL!


Incorporate changes requiring cookies

Update README accordingly
Add .gitignore stub for node.js
Update sample.js with new usage

This is close. The prepare code that sets up the cookie happens asynchronously, so that doesn't setup the request right away. The cookie eventually gets there, but there is a chance that the first few requests will not have the cookie. I think that the prepare function needs to take a callback or return a promise or something to ensure that any requests to cleverbot contain the appropriate cookie.


WordPress.DB.PreparedSQL.InterpolatedNotPrepared flagging $this->table_name but not $wpdb->table_name

There is no way we should use a placeholder for a tablename.
Yet WPDB flags this approach as false and suggests using placeholders (which is potentially not only wrong but even more unsafe).
$existing = $wpdb->get_var( $wpdb->prepare( "SELECT COUNT(*) FROM {$this->table_name} WHERE bla = %s", $bla ) ); => Fails sniff.
This way forces to rewrite the database table name hardcoded potentially countless times instead of setting it to the object.
I can't believe this is how we have to do it.
It seems to have $wpdb whitelisted, however you still would have to hardcode the table_name every time you do a get_var or get_row query.

There is no way we should use a placeholder for a tablename.

Your information is outdated. You can use the %i placeholder for tablenames since WP 6.2. Also see: https://make.wordpress.org/core/2022/10/08/escaping-table-and-field-names-with-wpdbprepare-in-wordpress-6-1/
I suggest closing this ticket.

Ok, thanks.
This is very new to me. Table and Column names can not be replaced by parameters in PDO, afaik, and that is what I used most in terms of DB Interactions (of course, outside WP)
And as until now this was not an issue, nor a requirement in WP, I did not even think (nor now) about a change like 6.2.
It is also almost not findable when googling for it, all that pops up is the WP DOC that mentions nothing about tablename requiring or being available for preparation... Sorry it ended up here.

I guess I will write for "only" 67% of the WP Users then. Not really a big issue in this particular project.
However this remains a problem if one wants to support older versions.
Hardcoding would be the only valid solution to also support users on older than 6.2?
(I do not want to go into version checks. Either it works or not. If this is the only thing I can do, I will just not support older WP versions, I think, in future)

Because there are 33% of all WordPress websites (currently) not on 6.2
That is a third of over 800 million websites if we believe stats.
That is the reason why "backwards compatibility" is even a thing, right?
Me personally I try to keep the things up to date, unfortunately that is not always the case/possible/the real world.

Hi @smileBeda, I'm the one who added %i to wpdb, and I'm looking to make further changes (e.g. supporting WHERE id IN (%...d), and specifying the first argument to wpdb::prepare() as a literal-string)... I appreciate where your coming from, especially with backwards compatibility... if you wanted to chat about any of these points, feel free to contact me off-list (allows for more of a conversation, then we can feed back any key points); the intention is to (very) slowly move towards an environment where it's not possible to have injection vulnerabilities (yes, I know, sounds unlikely, but there is a way).

Sorry the delay here - happy to talk @craigfrancis
I assume with if you wanted to chat about any of these points, feel free to contact me off-list you meant slack, so I have pinged you there.


Use the width property of ColumnDef to size columns instead of jQuery

@caguthrie your recent upgrade from table to DIV has made a lot of behaviors possible
to give ourselves more performant table do you think it is a good idea to leverage the width property of ColumnDef instead of assigning a dynamic CSS, which requires manual jQuery action?
The way it would work

we allow users to specify exactly how width each column ought to be via size on ColumnDef
if no size is specified it is set to w% where w = 1 / visibleColumns.length
turn overflow on the cells to hidden, to ensure alignment across rows

Theoretical justification
A column's width is a state that dictates how wide each column is. We already have this width property in GigaGrid.
This view fits into the prop/state driven view of React ... the render() function of the main component under this proposal would still be a pure function of its props/state and we can render everything in 1 cycle - thus improving performance
If we ever need to reason about how wide columns ought to be, we avoid having to deal with direct DOM manipulation in our logic
How do we see hidden content
It is my belief that if a user wants to see overflowing content in a cell, they will intuitively want to drag a handle
At which point, a drag action on cell borders is an action to change its width ... so we dispatch that action, and the entire app re-renders ... all very 'React-isc' without resorting to complicated lifecycle methods and direct DOM manipulation
please let me know what you think!

@caguthrie I've begun working on this ... in addition to creating type definitions (found a easy way) ... this however, will screw up some scrolling code you've put in ... and may need your advice
concurrent with this update will be the ability to resize columns using a drag handle in the headers

memo ... WIP

giga-grid-right-data-container: get rid of float: left
giga-grid-right-data-container && giga-grid-left-headers-container: add display: inline-block
giga-grid-body-view-port: add white-space: nowrap; max-width: 100%; overflow-x: hidden
move infinite scrolling div.placeholder to be the direct child of .giga-grid-body-viewport instead as a row


Run unit tests on PRs and pushes to d2iq/* branches

We'll use d2iq/main as our fork's main branch, and the d2iq/ prefix for any release branches. This workflow will build and run unit tests on any PRs or pushes to these branches.

This change is

I accidentally chose vmware:main as the base branch. I meant to choose mesosphere:main. :facepalm:
My apologies!


[Web] flutter beta needs more implementations related to copy/paste

My issue is about [Web]
This is a heads up that the beta flutter version now requires more methods for EditorState.
When compiling 2.0.7 or 2.0.9 of quill, I get compilation errors like:
Error: The non-abstract class 'RawEditorState' is missing implementations for these members:
 - TextSelectionDelegate.copySelection
 - TextSelectionDelegate.cutSelection
 - TextSelectionDelegate.pasteText
 - TextSelectionDelegate.selectAll
class RawEditorState extends EditorState

% flutter --version
Flutter 2.6.0-5.2.pre • channel beta • https://github.com/flutter/flutter.git
Framework • revision 400608f101 (5 weeks ago) • 2021-09-15 15:50:26 -0700
Engine • revision 1d521d89d8
Tools • Dart 2.15.0 (build 2.15.0-82.2.beta)```

I helped myself by adding this to the RawEditorState, but that's really not a stable solution.
@override
  noSuchMethod(Invocation msg) => "got ${msg.memberName} "
      "with arguments ${msg.positionalArguments}";

I understand that you are not going to fix this right away, but I thought filing that for later would come handy for you.

Same issue, would be great to get this fixed ASAP

Please Submit a PR

I'm not sure how to interpret that. Implement a PR to override noSuchMethod hack ? That seems ill-advised.
A proper fix ? Unfortunately, I'm not understanding the code base enough (or flutter) to have any confidence, Sorry. I really wanted to give the developers a heads up in the hope that it will be useful to know about this future problem.


Review for memory leaks/performance

Possible idea proposed by Adriaan:
Calculate the inverse distance weighting only once. Then, when a new grid needs to be calculated, you can quickly interpolate.
There is however a downside: this will not work when one of the radars has no data, because then you only use the other radars in the inverse distance weighting.

im running a localhost version and it crashes/leaks memory all the time.
any tips for performance? i'm running same numbers (stations, birds, just different longlats for testing)


docs: Add note on on local spl-stake-program deployment

I lost several hours trying to figure this out. It may help someone else to have this info blurb be in the docs.

Thanks for your contribution! Is the normal line to pull the program from devnet with -c not working?
I know the program is out of date on devnet, but it should be much easier for people running the script to just use what's already deployed rather than force them to build the program.

Agreed
It's interesting -- it was not working for me. I tried to run setup-stake-pool.sh and it would consistently give an error that the program is missing.
Looking at the documentation of -c:
    -c, --clone <ADDRESS>...
            Copy an account from the cluster referenced by the --url argument the genesis configuration. If the ledger
            already exists then this parameter is silently ignored

I think it's possible when running locally that there is a ledger present already and this parameter is ignored. I can confirm.

I think it's possible when running locally that there is a ledger present already and this parameter is ignored. I can confirm.

That's likely the situation you're in. Maybe we can change the script to use --url mainnet-beta so it's always copying the most up-to-date program. What do you think?


Custom data - register known CAD model - how can I further optimize alignment?

Hi there!
Thank you so much for this amazing work!
I'm trying to register known CAD models of anatomy to real world counterparts (patients) based solely on the partially occluded and noisy reconstructed point cloud data I get from the ToF camera of the HoloLens 2 mixed reality headset(using no RGB - only D). I'm completely new to Deep Learning, and I was wondering how I can best leverage the fact that I have the mesh of the 3D model I'm trying to register beforehand.  Ideally I want to create a pipeline that lets one upload a 3D model  and do a bunch of processing before to create the best possible network to register the CAD model.
Before doing any retraining, is there anything I can change in my config file to optimize the pretrained network for this use case of aligning smaller sized objects like the CT scan of a knee? I used the indoor config in my test and only changed dgcnn_k to 4 to avoid an error.(see What I have done so far ) Do you have advise what else I could optimize/ change? Unfortunately out of the box the network occasionally registers the CAD model upside down.
Would it be a good idea to train this network based solely on the one CAD model that I'm trying to register? Should I collect a lot of noise data that one would encounter in my use case and then train a network to be able to generate realistic noise that can be used during this custom training step?
Thank you so much for your consideration and sharing your work!
What I have done so far (for any complete beginners wondering how to use this for custom data):

SetUP before demo.py

on Ubuntu 20.04.3 LTS with the right devtools installed git, ninja etc
git clone https://github.com/overlappredator/OverlapPredator.git
conda activate py3.8.5 (environment with all requirements installed)
cd OverlapPredator
cd cpp_wrappers
sh compile_wrappers.sh
cd ..
sh scripts/download_data_weight.sh
python scripts/demo.py configs/test/indoor.yaml

demo.py works like a charm!.

Using  custom data:

I want to align 3D model (source.ply) of my arm with a sensor reading (Target.ply) of it
files can be found here: https://drive.google.com/drive/folders/1sbIwYgY4YODjrLvshC6-q_010YuCv9DU?usp=sharing
download and place into the asset folder
comment out lines  47 & 48 in scripts/demo.py
uncomment lines 51 -56 in scripts/demo.py like so:

def __getitem__(self,item):
       # get pointcloud
       # src_pcd = torch.load(self.src_path).astype(np.float32)
       # tgt_pcd = torch.load(self.tgt_path).astype(np.float32)

       src_pcd = o3d.io.read_point_cloud(self.src_path)
       tgt_pcd = o3d.io.read_point_cloud(self.tgt_path)
       src_pcd = src_pcd.voxel_down_sample(0.025)
       tgt_pcd = tgt_pcd.voxel_down_sample(0.025)
       src_pcd = np.array(src_pcd.points).astype(np.float32)
       tgt_pcd = np.array(tgt_pcd.points).astype(np.float32)

       src_feats=np.ones_like(src_pcd[:,:1]).astype(np.float32)
       tgt_feats=np.ones_like(tgt_pcd[:,:1]).astype(np.float32)

       # fake the ground truth information
       rot = np.eye(3).astype(np.float32)
       trans = np.ones((3,1)).astype(np.float32)
       correspondences = torch.ones(1,2).long()

       return src_pcd,tgt_pcd,src_feats,tgt_feats,rot,trans, correspondences, src_pcd, tgt_pcd, torch.ones(1)

copy indoor.yaml and rename to testconfig.py
change line 73 & 74 to point to source and target point cloud like so:

demo:
  src_pcd: assets/Source.ply
  tgt_pcd: assets/Target.ply
  n_points: 1000

run python scripts/demo.py configs/test/testconfig.yaml
cause "RuntimeError: invalid argument 5: k not in range for dimension at /pytorch/aten/src/THC/generic/THCTensorTopK.cu:26"
try setting dgcnn_k from 10 to  a lower number (4 worked for me) in testconfig.yaml like so:

overlap_attention_module:
  gnn_feats_dim: 256
  dgcnn_k: 4
  num_head: 4
  nets: ['self','cross','self']

Results:

hi,
Thanks for your interest and encouraging words.
To answer your question:

Due to the symmetry in your data, "registering objects upside down" is quite expected. If you check those deep models registering ModelNet40 datasets, quite often, they restrict the test samples to have rotation angles within 45 degrees such that they can directly avoid ambiguity caused by symmetry. In large scale data, we rely on larger context to disambiguate them, I guess this would be challenging in your case.

Without some hacks, KPConv backbone is sensitive to hyper-parameter changes, so not much you can do with the pre-trained models. However, you can check D3Feat paper to see how they hack the backbone to achieve generalisation, though only to some extent.

I think finetuning the pretrained model on your collected data is  a good idea.

Let me know if your need more assistance. I was once asked about such application scenarios and I am looking forward to see how it works.
best,
Shengyu


Add CA categories page

Add a new category index page
Link to the various ways knowing the category is useful
Other miscellaneous code analysis improvements
Fixes #22457

Preview link.

Thanks Manish!


Add synonym/decomposing support for specific words

See https://github.com/quarkusio/search.quarkus.io/issues/40#issuecomment-1804109979:

Customize analyzers so that "dev services" is considered a synonym of "devservices". This is important because we do appear to use "devservices", without a space, in the :topic: metadata.

These filters may be relevant (from most likely to help to least likely):

https://www.elastic.co/guide/en/elasticsearch/reference/current/analysis-hyp-decomp-tokenfilter.html
https://www.elastic.co/guide/en/elasticsearch/reference/current/analysis-dict-decomp-tokenfilter.html
https://www.elastic.co/guide/en/elasticsearch/reference/current/analysis-common-grams-tokenfilter.html
https://www.elastic.co/guide/en/elasticsearch/reference/current/analysis-word-delimiter-graph-tokenfilter.html (would only work for e.g. DevServices, not devservices)

Other "synonyms" may be needed, see comments on this issue.

vertx should be a synonym for vert.x
See https://quarkusio.zulipchat.com/#narrow/stream/187038-dev/topic/.E2.9C.94.20How.20exactly.20does.20filtering.20work.20on.20the.20Guides.20page.3F

RestEASY should be handled particularly too: Rest easy, resteasy, restEasy should all be considered the same thing.


[ADXT-512] Dual shipping disabled by default

What does this PR do?

[!NOTE]
I'll bump the test infra definition version after this gets merged.

Dual shipping will be disabled by default following this PR, removed WithoutDualShipping.
These tests still have dual shipping:

containers
orchestrator

See https://github.com/DataDog/test-infra-definitions/pull/1283 for definition.
Motivation
Describe how you validated your changes

Manually tested kind vms (standard way / containers tests) and verified that there is DD_ADDITIONAL_URL in the kubectl config.
Possible Drawbacks / Trade-offs
Additional Notes

/merge


ImportError: cannot import name 'test' from 'test'

I am getting following error -  ImportError: cannot import name 'test' from 'test'
I am using python v3.11

use import test
instead of from test import test

make sure test file your main python file are in floder / directory

use import test instead of from test import test

what is the test file? are we supposed to create it ourselves?

Follow the instructions in the readme file for the spoofing feature.

use import test instead of from test import test

what is the test file? are we supposed to create it ourselves?

Follow the instruction of sir @computervisioneng , its on the spoofing file links given on the README


Fixed user not found error on username change

Pull Request Test Coverage Report for Build 974608829

1 of 2 (50.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.004%) to 95.656%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

accounts/views.py
1
2
50.0%

Files with Coverage Reduction
New Missed Lines
%

accounts/views.py
1
81.82%

Totals

Change from base Build 974365917:
0.004%

Covered Lines:
1123

Relevant Lines:
1174

💛  - Coveralls


Expecting property name enclosed in double quotes

I'm writing an app for a client that needs to write data to his Google Drive. I'm following the Python tutorial here. However, I'm getting an error.
flow = google_auth_oauthlib.flow.Flow.from_client_secrets_file(
    "client_id.json",
    scopes=["https://www.googleapis.com/auth/drive.file"]
)

Unhandled exception in thread started by <function check_errors.<locals>.wrapper at 0x7f780bd04378>
web_1  | Traceback (most recent call last):
web_1  |   File "/virtualenv/lib/python3.5/site-packages/django/utils/autoreload.py", line 228, in wrapper
web_1  |     fn(*args, **kwargs)
web_1  |   File "/virtualenv/lib/python3.5/site-packages/django/core/management/commands/runserver.py", line 124, in inner_run
web_1  |     self.check(display_num_errors=True)
web_1  |   File "/virtualenv/lib/python3.5/site-packages/django/core/management/base.py", line 359, in check
web_1  |     include_deployment_checks=include_deployment_checks,
web_1  |   File "/virtualenv/lib/python3.5/site-packages/django/core/management/base.py", line 346, in _run_checks
web_1  |     return checks.run_checks(**kwargs)
web_1  |   File "/virtualenv/lib/python3.5/site-packages/django/core/checks/registry.py", line 81, in run_checks
web_1  |     new_errors = check(app_configs=app_configs)
web_1  |   File "/virtualenv/lib/python3.5/site-packages/django/core/checks/urls.py", line 16, in check_url_config
web_1  |     return check_resolver(resolver)
web_1  |   File "/virtualenv/lib/python3.5/site-packages/django/core/checks/urls.py", line 26, in check_resolver
web_1  |     return check_method()
web_1  |   File "/virtualenv/lib/python3.5/site-packages/django/urls/resolvers.py", line 256, in check
web_1  |     for pattern in self.url_patterns:
web_1  |   File "/virtualenv/lib/python3.5/site-packages/django/utils/functional.py", line 35, in __get__
web_1  |     res = instance.__dict__[self.name] = self.func(instance)
web_1  |   File "/virtualenv/lib/python3.5/site-packages/django/urls/resolvers.py", line 407, in url_patterns
web_1  |     patterns = getattr(self.urlconf_module, "urlpatterns", self.urlconf_module)
web_1  |   File "/virtualenv/lib/python3.5/site-packages/django/utils/functional.py", line 35, in __get__
web_1  |     res = instance.__dict__[self.name] = self.func(instance)
web_1  |   File "/virtualenv/lib/python3.5/site-packages/django/urls/resolvers.py", line 400, in urlconf_module
web_1  |     return import_module(self.urlconf_name)
web_1  |   File "/virtualenv/lib/python3.5/importlib/__init__.py", line 126, in import_module
web_1  |     return _bootstrap._gcd_import(name[level:], package, level)
web_1  |   File "<frozen importlib._bootstrap>", line 986, in _gcd_import
web_1  |   File "<frozen importlib._bootstrap>", line 969, in _find_and_load
web_1  |   File "<frozen importlib._bootstrap>", line 958, in _find_and_load_unlocked
web_1  |   File "<frozen importlib._bootstrap>", line 673, in _load_unlocked
web_1  |   File "<frozen importlib._bootstrap_external>", line 665, in exec_module
web_1  |   File "<frozen importlib._bootstrap>", line 222, in _call_with_frames_removed
web_1  |   File "/app/urls.py", line 11, in <module>
web_1  |     ] + aldryn_addons.urls.patterns() + i18n_patterns(
web_1  |   File "/virtualenv/lib/python3.5/site-packages/django/conf/urls/__init__.py", line 50, in include
web_1  |     urlconf_module = import_module(urlconf_module)
web_1  |   File "/virtualenv/lib/python3.5/importlib/__init__.py", line 126, in import_module
web_1  |     return _bootstrap._gcd_import(name[level:], package, level)
web_1  |   File "<frozen importlib._bootstrap>", line 986, in _gcd_import
web_1  |   File "<frozen importlib._bootstrap>", line 969, in _find_and_load
web_1  |   File "<frozen importlib._bootstrap>", line 958, in _find_and_load_unlocked
web_1  |   File "<frozen importlib._bootstrap>", line 673, in _load_unlocked
web_1  |   File "<frozen importlib._bootstrap_external>", line 665, in exec_module
web_1  |   File "<frozen importlib._bootstrap>", line 222, in _call_with_frames_removed
web_1  |   File "/app/authenticate/urls.py", line 3, in <module>
web_1  |     from . import views
web_1  |   File "/app/authenticate/views.py", line 106, in <module>
web_1  |     scopes=["https://www.googleapis.com/auth/drive.metadata.readonly"]
web_1  |   File "/virtualenv/lib/python3.5/site-packages/google_auth_oauthlib/flow.py", line 172, in from_client_secrets_file
web_1  |     client_config = json.load(json_file)
web_1  |   File "/usr/local/lib/python3.5/json/__init__.py", line 268, in load
web_1  |     parse_constant=parse_constant, object_pairs_hook=object_pairs_hook, **kw)
web_1  |   File "/usr/local/lib/python3.5/json/__init__.py", line 319, in loads
web_1  |     return _default_decoder.decode(s)
web_1  |   File "/usr/local/lib/python3.5/json/decoder.py", line 339, in decode
web_1  |     obj, end = self.raw_decode(s, idx=_w(s, 0).end())
web_1  |   File "/usr/local/lib/python3.5/json/decoder.py", line 355, in raw_decode
web_1  |     obj, end = self.scan_once(s, idx)
web_1  | json.decoder.JSONDecodeError: Expecting property name enclosed in double quotes: line 12 column 3 (char 534)

I re-downloaded the json file from the Google Dev console, and it seems to be working.


typing_extensions is only needed for python < 3.8

🚀 The feature
specify the dep typing_extensions is only needed for python < 3.8
Motivation, pitch
specify the dep typing_extensions is only needed for python < 3.8
Alternatives
No response
Additional context
According to https://github.com/pytorch/vision/search?q=typing_extensions, torchvision only uses typing_extensions.Literal, which is available since python 3.8, see also this.
we only need to change setup.py and replace typing_extensions with something like typing_extensions; python_version<"3.8".
ref issue https://github.com/archlinuxcn/repo/issues/2762

In case the syntax is unknown, this conditional dependency is a environment marker specified in PEP508.

Within our code we need to do something like
if sys.version_info >= (3, 8):
    from typing_extensions import Literal
else:
    from typing import Literal

Within our code we need to do something like
if sys.version_info >= (3, 8):
    from typing_extensions import Literal
else:
    from typing import Literal

Any chance we can avoid that? This is going to be a headache.
I'm not sure I completely understand the original issue here. We test torchvision on 3.7-3.10 and we're not hitting any of these related issues.
Also the original post suggests

typing_extensions is only needed for python < 3.8

But the condition is the other-way around?

if sys.version_info >= (3, 8):
from typing_extensions import Literal

there is also no harm from using them from the backport package. This is why we also don't see issues in CI

Thanks Philip, that's were I'm confused: if there is no harm in using the backport, what is the actual cause of the original issue? Perhaps @hubutui can help me understand :) ?

Hmm, it seems there is no need to fix this issue. I think I could simply update the dep when I create pkg for ArchLinux.

You dropped support for python 3.7 in the nightly build and removed typing_extension as a dependency. So this issue can be closed?


[Infra] Set up Github Actions

Is your feature request related to a problem?
Currently we don't have proper Github Actions set up for query insights plugin.
What solution would you like?

MVP: We should have automated actions to build, test and release the plugin.
Github actions for tagging
Github actions for backport

This is done by the above PRs.


medium article is for members only

kindly make this article open
https://medium.com/@matthewdorrian/save-money-and-improve-performance-with-the-lambda-power-tuner-ui-bad594176008

This is public. medium may restrict to 3 articles a day but thats not in my control


React icon shows up in slack preview instead of our favicon

It's like, a tiny tiny issue but kind of annoying.  We should use our beautiful favicon instead!

When sending the URL in Slack it also shows the name of the app as "Web site created using create-react-app".
It would be great to change it to "CfA Brigade Network Index" or something similar.
Unfortunately I don't know how to make the change, sorry.

This is fixed by #73?


Do you have articles to explain how to use this mvp framework it？

Hi, benoitletondor,
Do you have articles to explain how to use this mvp framework it or some documentation？
thanks.

Hi there,
How can I help you using it? What's not clear for you?

I carefully studied u2020 this project,after that，i noticed your framework relative is more interaction than the traditional framework MVP, and M,V,P three are decoupled, does this interaction is to reduce the burden of P, I do not know whether i understand it rightly?

I'm sorry but I don't get your question :/


fix(test): Make resetEnv() properly reset settings

I got test failures in #7323 when I started using the --debug flag in one of my tests commands.
It turns out that because the 'settings' variable is a pointer, it cannot be used to store the original envSettings and then restore them.
Instead, this commit creates an entirely new envSettings, after having set the environment variables, which may affect it.
Signed-off-by: Marc Khouzam<EMAIL_ADDRESS>

Thanks @mattfarina


build(deps): revert verify/deploy change

This pull request reverts 9e5a1df32d7978bcc6b7e362c9fa6c337c9d32e2, which was an experiment as part of #2009.
Cc: @cmarchand

Now that this change has been merged, I see a list of new snapshot artifacts at
https://oss.sonatype.org/content/repositories/snapshots/io/xspec/xspec/3.1-SNAPSHOT/
I downloaded the enduser-files.zip artifacts and checked that the MD5 and SHA1 values in xspec-3.1-20240928.104704-71-enduser-files.zip.md5 and xspec-3.1-20240928.104704-71-enduser-files.zip.sha1, respectively, are the same as the ones I get from the following commands.
certutil -hashfile xspec-3.1-20240928.104704-71-enduser-files.zip  MD5
certutil -hashfile xspec-3.1-20240928.104704-71-enduser-files.zip  SHA1

It looks like the signing is working as expected with the Maven version and GPG plugin version we're now using based on #2009.


Mark Final members as class variables for mypy

Change Summary
Mark Final members as class variables for mypy so that mypy does not mistake them as fields.
The main change is in line 758 where we also check if a node is a Final. I moved this check a bit behind so that type inference for Finals without type annotations is still performed when necessary.
A side note: I thought about why Final nodes are considered model fields as per the current implementation, but I cannot really figure out what kind of cases this handles. I played with a few different scenarios and mypy could catch type mismatch with changes in this PR, so I suppose this PR won't break anything, but I'm not completely sure if I missed out anything.
Related issue number
Fixes #10474.
Checklist

[ ] The pull request title is a good summary of the changes - it will be used in the changelog
[x] Unit tests for the changes exist
[x] Tests pass on CI
[x] Documentation reflects the changes where applicable
[x] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

please review

Thanks, handled in https://github.com/pydantic/pydantic/pull/11121.


Documenting Class<? extends SOMETHING> breaks the plugin

Hey, thanks for picking up the slack. This seems to be the only maintained maven plugin for generating openapi docs by now!
I tried it with my project and it errors with:
Execution documentation of goal io.github.kbuntrock:openapi-maven-plugin:0.0.12:documentation failed:
Type ? super T (+WildcardTypeImpl is not supported yet

I debugged the code with mvnDebug and learned that the following code in my project causes the problem:
public abstract class CriteriaWithDateType extends BaseCriteria {

	private final Class<? extends CriteriaDateType> dateTypeClass;

	public CriteriaWithDateType(Class<? extends CriteriaDateType> dateTypeClass) {
		this.dateTypeClass = dateTypeClass;
	}
// --- SNIP ---
}

Class is inspected i.e., exploreDataObject  is called with a DataObject where its javaClass is set to java.lang.Class. The result is quite interesting, but I argue that everything under java.lang.* should not be inspected in general. It descends through Class until it hits the following code:
package java.lang.ref;

public abstract class Reference<T> {
    private T referent;
    volatile ReferenceQueue<? super T> queue;

where the inspection dies as ? super T is not supported yet.

Oh god @JonasCir, so sorry for ghosting your message, I did not saw it. I will investigate it asap!

This should be fixed, will post a PR on your project to test it.

Hi @kbuntrock ,
sorry for getting back to you so late!
I am not longer involved with the other project, so I didn't follow up on details.
The project uses io.swagger.core.v3.swagger-maven-plugin which appears to work now.
I really appreciate you work! Thank you for resolving this issue :)

No worries at all @JonasCir. This bug report and the link to the Sormas project helped me a lot to improve this plugin.
Don't hesitate if you find some more or if you have some evolution ideas! ;)
Wish you a great day.


Managed Accounts

Doesn't seem to work with managed accounts ie, Plex Home users.

Yes, I noticed this as well. I think it works for shared users, interestingly, but not managed home users.

Yup, can confirm shared users work great.

I have a hunch the reason it's not working for managed users is because managed users have their own Plex token, separate from the main user. The token is a bit more difficult to find, but you can figure it out by viewing an item from the managed user's web interface, and searching the page source for "token."
I'm currently trying to figure out a way to run a separate Plex-Auto-Languages docker container and feeding it the managed user's token in the yaml file. This is clearly not an ideal situation, but might work until/if a fix is provided by the developer.

Currently only shared users are supported, I'm working on adding support for managed users as well but for some reason it cannot be done in the same way.

Yeah the managed users are set up a bit differently/more complicated than regular shared users.

Thank you for the fast response. It should be fixed in v1.0.10.

Also, I just tested with a managed user, and I don't believe it's working there either. Not sure if the fix will also fix that? But I'll test as soon as it is deployed.

That fixed it, thanks!

Works flawlessly, thanks!


Nil parameter unexpectedly coerced as empty string

What Ruby, Rails and RSpec versions are you using?
Ruby version: 2.5.1
Rails version: 5.2.1
Rspec version: 3.8

Observed behaviour
When a query parameter is passed as nil, the query is passed to Rails and coerced as an empty string.
Expected behaviour
Expected that params: { query: nil } be passed without a query parameter, and tests looking for empty strings (e.g. /users?query= ) must explicitly test with an empty string in those cases. This is especially useful for using lazily evaluated let syntax to add/remove parameters without having to rewrite them.
Can you provide an example app?
Here: https://github.com/etgrieco/rspec-rails-nil-params-example. Test expected to fail is passing.
UsersController
  def index
    escaped_query = CGI.escape(params[:query]) # CGI.escape(nil) should yield TypeError
    render(json: { query: escaped_query })
  end

UsersController Spec
  before { get :index, params: params }
  subject { JSON.parse(response.body) }

  let(:query) { "this is a query" }
  let(:params) { { query: query } }

  context 'with a query' do
    it { should eql("query" => "this+is+a+query") }
  end

  context 'without a query' do
    let(:query) { nil }

    it { should eql("query" => '') } # This spec is unexpectedly passing
  end

Hello, this is how Rails behaves, you can't ever have a nil parameter passing through, http://myapp/path/to/controller?query= would come through as { "query" => "" } if you want nil as a value you need to omit the key e.g. http://myapp/path/to/controller which should make params { }

@JonRowe Going to the route http://myapp/path/to/controller?query will indeed produce nil params. You can have a nil parameter pass through:
Nil param
Started GET "/users?query"
Processing by UsersController#index as HTML
  Parameters: {"query"=>nil}
Completed 500 Internal Server Error in 0ms (ActiveRecord: 0.0ms)

TypeError (no implicit conversion of nil into String):

Empty string
Started GET "/users?query=" for <IP_ADDRESS> at 2018-09-05 11:58:02 -0400
Processing by UsersController#index as HTML
  Parameters: {"query"=>""}
Completed 200 OK in 0ms (Views: 0.2ms | ActiveRecord: 0.0ms)

I believe a controller test with params: { "query" => nil } should behave the same way as if the parameters were actually {"query"=>nil}, as shown above.

Thats because GET "/users?query" is invalid, query here isn't a param, its just junk.

You can't recreate this in Rails test mode. Everything is expected to be a string.

(This is not RSpec's doing, it's all Rails, we pass things through as you give them to us)

@JonRowe Thanks for the clarification, not sure whether was happening in Rails or RSpec -- definitely unexpected though when Rails, within the controller, clearly can differentiate between nil and string params when going to the route -- so if this change were to happen it would need to happen on the Rails testing end I guess?

You're missing the point, nil is not a param, it's just an invalid string, valid url params will always have = and thus will always be strings, if you raise this with the Rails team I suspect they will fix it so /users?query returns a params of {} which is the technically correct versions.

For the record and anyone else who finds this issue. You can prevent this situation like this:
before { get :index, params: params, as: :json }

Seen here https://stackoverflow.com/a/44704018/1407371


Add  Export-DbaSysDbUserObject to exported functions

Type of Change

[ ] Bug fix (non-breaking change, fixes # )
[x] New feature (non-breaking change, adds functionality, fixes #6561 )
[ ] Breaking change (effects multiple commands or functionality, fixes # )
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] If new file reference added for test, has is been added to github.com/sqlcollaborative/appveyor-lab ?
[ ] Nunit test is included
[ ] Documentation
[ ] Build system

Purpose

Make the  Export-DbaSysDbUserObject command available.

Thank you! I had some weird conflicts and ended up adding it manually so will close 👍


Need to accept empty data

Errors occurred when I tried to use empty data object to initialize a chart instance.
  const chart = new frappe.Chart('.panel .chart', {
    data: {},
    type: 'line',
    height: 320,
    colors: ['#0294FF'],
  })

This caused an Uncaught TypeError:
frappe-charts.min.iife.js:1 Uncaught TypeError: Cannot read property 'map' of undefined
    at Ct (frappe-charts.min.iife.js:1)
    at i.value (frappe-charts.min.iife.js:1)
    at i.e (frappe-charts.min.iife.js:1)
    at new i (frappe-charts.min.iife.js:1)
    at Tt (frappe-charts.min.iife.js:1)
    at new t (frappe-charts.min.iife.js:1)
    at HTMLDocument.<anonymous> (stats-overview.js:20)
    at l (jquery.js:2)
    at Object.fireWith [as resolveWith] (jquery.js:2)
    at Function.ready (jquery.js:2)

Seems frappe tried to call map function of datasets or labels. But I only need to create an empty chart at that time, and update data on future user events.
I tried to use this as init data
{
  labels: [],
  datasets: [],
}

and got a lot of SVG errors which blocked the future data updatings.
Error: <g> attribute transform: Trailing garbage, "translate(0, undefined)".

It's weird that
{
  labels: [''],
  datasets: [{
    values: [0],
  }],
}

also cause some errors such as
frappe-charts.min.iife.js:1 Error: <path> attribute d: Unexpected end of attribute. Expected number, "M".

and
Error: <circle> attribute cy: Expected length, "undefined".

but without blocking future updatings.
Seems it only works fine with two or more data points.
User-Agent:
Mozilla/5.0 (Macintosh; Intel Mac OS X 10_14_2) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/73.0.3683.86 Safari/537.36

I experienced the same issue. In a typical asynchronous component this is a quite mandatory flow.

The when creating charts it is necessary to provide the labels at least. With the labels in place the Charts will automatically fill in the value 0 as default for the  missing values.
So a config like the following will work without any errors.
data: {
		labels: ['Monday, 'Tuesday', 'Wednesday', 'Thursday', 'Friday'],
		datasets: [],
	}

I hope this answers your question.

The when creating charts it is necessary to provide the labels at least. With the labels in place the Charts will automatically fill in the value 0 as default for the missing values.
So a config like the following will work without any errors.
data: {
	labels: ['Monday, 'Tuesday', 'Wednesday', 'Thursday', 'Friday'],
	datasets: [],
}

I hope this answers your question.

Thanks a lot for your help. But actually for me, it's not a perfect solution 😂. I don't know any of the labels until the server responds them via an xhr. Better if frappe could draw an empty chart without any data or labels provided at initialization. Now I manually set two data points as a walkaround, it seems a little bit strange showing two unrelated data points on every refresh before real data loaded.

The when creating charts it is necessary to provide the labels at least. With the labels in place the Charts will automatically fill in the value 0 as default for the missing values.
So a config like the following will work without any errors.
data: {
	labels: ['Monday, 'Tuesday', 'Wednesday', 'Thursday', 'Friday'],
	datasets: [],
}

I hope this answers your question.

Thanks a lot for your help. But actually for me, it's not a perfect solution 😂. I don't know any of the labels until the server responds them via an xhr (for example last 30 days as labels). Better if frappe could draw an empty chart without any data or labels provided at initialization. Now I manually set two labels as a walkaround, it seems a little bit strange showing two unrelated zero data points on every refresh before real data loaded.

I had the same issue, but a workaround will be to use empty string values in an array as far as you have a fixed length for your data that should work...
Example: label: ['',''] where prosterity lenght is 2

When all values o dataset is zero, i'm having SVG error. Any tip to avoid it?
{
  "labels": ["A", "B", "C"],
  "datasets": [{ "values": [0, 0, 0] }]
}

Error: <path> attribute d: Expected number, "M157 55\n\t\tLNaN NaN\n\t\tA 55 5…".

i.setAttribute(n,a))

Hi, I wanted to point out that the library still throws an error when only a single data is provided, as blueabysm pointed out in the original post:
It's weird that

{
  labels: [''],
  datasets: [{
    values: [0],
  }],
}
also cause some errors such as

frappe-charts.min.iife.js:1 Error: <path> attribute d: Unexpected end of attribute. Expected number, "M".
and

Error: <circle> attribute cy: Expected length, "undefined".
but without blocking future updatings.

Seems it only works fine with two or more data points.

Hi, I wanted to point out that the library still throws an error when only a single data is provided, as blueabysm pointed out in the original post:
It's weird that

{
  labels: [''],
  datasets: [{
    values: [0],
  }],
}
also cause some errors such as

frappe-charts.min.iife.js:1 Error: <path> attribute d: Unexpected end of attribute. Expected number, "M".
and

Error: <circle> attribute cy: Expected length, "undefined".
but without blocking future updatings.

Seems it only works fine with two or more data points.

I am too experiencing the same issue. When there is only one dataset is present, it throws console errors like
 attribute d: Unexpected end of attribute. Expected number, "M"

Facing the same issue. The charts don't handle a single label/value properly. Any solutions yet?

Error still exist... Kindly ask autors to resolve this problem...


Add support multiple notebooks in a single nblink file

This PR adds the ability to include multiple notebooks in a single .nblink file. The semantics to achieve this are as follows:
   [
        {
            "path": "relative/path/to/notebook"
        },
        {
            "path": "relative/path/to/notebook_2"
        }
    ]

So, in short, instead of supplying a single JSON object, instead a list of JSON objects are provided in the .nblink file.
closes #12

Just wanted to note here, for now ive created a new forked project called nbsphinx-multilink that adds the functionality provided in this PR. @vidartf you are still the "author" of the package, I have listed myself in the maintainer role and will publish the forked version on pypi for now. @vidartf is fully credited with the packages authorship.


[SPARK-14581] [SQL] push predicatese through more logical plans

What changes were proposed in this pull request?
Right now, filter push down only works with Project, Aggregate, Generate and Join, they can't be pushed through many other plans.
This PR added support for Union, Intersect, Except and all unary plans.
How was this patch tested?
Added tests.

cc @marmbrus @cloud-fan

Test build #55662 has started for PR 12342 at commit 2329623.

Test build #55663 has started for PR 12342 at commit 8fdf31b.

Test build #55662 has finished for PR 12342 at commit 2329623.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #55663 has finished for PR 12342 at commit 8fdf31b.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

My general thoughts about filter push down: If the filter's condition is non-deterministic, we shouldn't push it down through some operators that will change the number or order of the input rows, e.g. set operations, others should be ok, e.g. the following optimization is allowed:
// From:
df.select('a, 'b).filter('c > rand(42))
// To:
df.filter('c > rand(42)).select('a, 'b)

If the underlying operator contains non-deterministic expression, filter push down is not allowed. Currently only 4 operator may contain non-deterministic expression: Project, Filter, Aggregate, Window

@cloud-fan Most of the predicates are determistic, so I'd like to not push down non-determistic predicate aggresively in this PR.

Test build #55685 has started for PR 12342 at commit 3c11e4c.

Test build #55686 has started for PR 12342 at commit 099b668.

LGTM

Test build #55688 has started for PR 12342 at commit 21c8482.

Test build #55685 has finished for PR 12342 at commit 3c11e4c.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #55686 has finished for PR 12342 at commit 099b668.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

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

Merged build finished. Test FAILed.

Test build #55688 has finished for PR 12342 at commit 21c8482.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

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

Merged build finished. Test FAILed.

Test build #55697 has started for PR 12342 at commit 388dbc0.

Test build #55697 has finished for PR 12342 at commit 388dbc0.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

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

Merged build finished. Test FAILed.

Test build #55731 has started for PR 12342 at commit 71a8317.

Test build #55731 has finished for PR 12342 at commit 71a8317.

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/55731/
Test PASSed.

Merging this into master, thanks!


Islandora manual installation hands on

i'm trying to run the Drupal installer with Drush:
when i run the code       "cd /opt/drupal/web"
i get       "bash: cd: /opt/drupal/web: No such file or directory"

Hi @Frazer-Nyambe, Can you tell me what step you are on so I can follow along with your process?

Installing composer, Drush, and Drupal.
Subtitle: Run the Drupal installer with Drush.
Just the first code: "cd /opt/drupal/web"
On Wed, Apr 27, 2022, 15:07 Jared Whiklo @.***> wrote:

Hi @Frazer-Nyambe https://github.com/Frazer-Nyambe, Can you tell me
what step you are on so I can follow along with your process?
—
Reply to this email directly, view it on GitHub
https://github.com/Islandora/documentation/issues/2098#issuecomment-1110978316,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AV33MNB723D4DILPJ4AI6ZLVHE3ZFANCNFSM5UORQUJA
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
_
_
CONFIDENTIAL NOTICE: This message (and its attachments) may contain
confidential information. Any unauthorized use or disclosure is prohibited.
If you have received this communication in error, please notify us
immediately by responding to this email and then delete it from your
system.

I believe that directory should have been created by the final command in the block
# Start by giving Drupal somewhere to live. The Drupal project is installed to
# an existing, empty folder.
sudo mkdir /opt/drupal
sudo chown www-data:www-data /opt/drupal
sudo chmod 775 /opt/drupal
# Change the ownership of default Apache directory so Composer can access it
sudo chown -R www-data:www-data /var/www/
# Clone drupal-project and build it in our newly-created folder.
git clone https://github.com/drupal-composer/drupal-project.git
cd drupal-project
# Expect this to take a little while, as this is grabbing the entire
# requirements set for Drupal.
sudo -u www-data composer create-project drupal-composer/drupal-project:9.x-dev /opt/drupal --no-interaction

Located at Drush 10 and Drupal 9

The path $ opt/drupal... Exists
But $ opt/drupal/web... Doesn't exist
On Wed, Apr 27, 2022, 15:34 Jared Whiklo @.***> wrote:

I believe that directory should have been created by the final command in
the block
Start by giving Drupal somewhere to live. The Drupal project is installed to
an existing, empty folder.
sudo mkdir /opt/drupal
sudo chown www-data:www-data /opt/drupal
sudo chmod 775 /opt/drupal
Change the ownership of default Apache directory so Composer can access it
sudo chown -R www-data:www-data /var/www/
Clone drupal-project and build it in our newly-created folder.
git clone https://github.com/drupal-composer/drupal-project.git
cd drupal-project
Expect this to take a little while, as this is grabbing the entire
requirements set for Drupal.
sudo -u www-data composer create-project drupal-composer/drupal-project:9.x-dev /opt/drupal --no-interaction
Located at Drush 10 and Drupal 9
https://islandora.github.io/documentation/installation/manual/installing_composer_drush_and_drupal/#drush-10-and-drupal-9
—
Reply to this email directly, view it on GitHub
https://github.com/Islandora/documentation/issues/2098#issuecomment-1111011059,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AV33MNHMKMWDUL7IMDIXOFDVHE675ANCNFSM5UORQUJA
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
_
_
CONFIDENTIAL NOTICE: This message (and its attachments) may contain
confidential information. Any unauthorized use or disclosure is prohibited.
If you have received this communication in error, please notify us
immediately by responding to this email and then delete it from your
system.

You might want to go back and try that command again then.
You can see where the drupal-composer/drupal-project tries to install the files here.
If you see any errors during that process it might explain what is happening.

Thanks, I'll check it out.
On Wed, Apr 27, 2022, 17:08 Jared Whiklo @.***> wrote:

You might want to go back and try that command again then.
You can see where the drupal-composer/drupal-project tries to install the
files here
https://github.com/drupal-composer/drupal-project/blob/9.x/composer.json#L72-L76
.
If you see any errors during that process it might explain what is
happening.
—
Reply to this email directly, view it on GitHub
https://github.com/Islandora/documentation/issues/2098#issuecomment-1111119832,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AV33MNHSRLJTQWIZPMPEW2TVHFKABANCNFSM5UORQUJA
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
_
_
CONFIDENTIAL NOTICE: This message (and its attachments) may contain
confidential information. Any unauthorized use or disclosure is prohibited.
If you have received this communication in error, please notify us
immediately by responding to this email and then delete it from your
system.

This issue still remains unresolved, so i decided to restart the whole manual installation process but i'm still getting errors.

Can you save the output of your terminal session and post it so we can see the whole process?

I meant the whole history. Most terminal tools have an option for saving the terminal session into a text file that you can upload. This will show all the commands and output leading up to this point (well, Windows will have an upper limit on the number of lines it will have saved).


mgmt aks, support FIPS-enable OS for agent pool machines

Description
issue: https://github.com/Azure/azure-sdk-for-java/issues/33870
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-resourcemanager-containerservice


internal/cloud/gcp/compute: Add SEV_SNP_CAPABLE Guest OS Feature

See: https://github.com/coreos/coreos-assembler/pull/3547
See: https://cloud.google.com/blog/products/identity-security/rsa-snp-vm-more-confidential
See: https://issues.redhat.com/browse/COS-2343

This pull request includes:

[ ] adequate testing for the new functionality or fixed issue
[ ] adequate documentation informing people about the change such as

[ ] submit a PR for the guides repository if this PR changed any behavior described there: https://www.osbuild.org/guides/

Hi @travier.
Thanks a lot for the PR. Do you know if all RHEL-9 versions support SNP feature or if it is available only since some specific minor version of RHEL-9?

Do you know if all RHEL-9 versions support SNP feature or if it is available only since some specific minor version of RHEL-9?

Unfortunately I don't know. I've asked that to the folks that should know 🙂.

From another chat, this is RHEL 9.2 only (potentially 9.1 but with lesser support): https://bugzilla.redhat.com/show_bug.cgi?id=2081424

The installer test on RHEL 8.9 has been causing issues all around lately.  I did a rebase to help it along.


Validate k8s 1.11 for 2.0

The 1.11 release cycle begins on Tuesday, April 3rd, 2018, and ends
on release day, Tuesday, June 26th.

https://github.com/kubernetes/kubernetes/blob/master/CHANGELOG-1.11.md

This would be super useful, as k8s 1.11 introduces a feature for detecting name resolution service used.
Previous versions would fail to get kubedns running on Ubuntu 18.04 Bionic as they cannot detect systemd-resolved... There's a workaround for that, but still it'd be nice to have it running OOB :)

As coreDNS is promoted to GA, we add it as an alternative dns provider, and consider making it a default provider after testing.
Also need to revise all other system addons images and see if upgrade is needed for them.

rancher/hyperkube:v1.11.0-rancher1 is available

@alena1108 if coreDNS really becomes the default for Rancher, will it automatically replace kubedns with a simple upgrade of Rancher?

See https://github.com/rancher/rke/issues/757 to validate network plugins in 1.11.

Closing in favor of https://github.com/rancher/rancher/issues/14613