feat: support for many padding dynamo converters

Description
Support for many padding dynamo converters, including:
torch.ops.aten.pad.default
torch.ops.aten.constant_pad_nd.default
torch.ops.aten.reflection_pad1d.default
torch.ops.aten.reflection_pad2d.default
torch.ops.aten.reflection_pad3d.default
torch.ops.aten.replication_pad1d.default
torch.ops.aten.replication_pad2d.default
torch.ops.aten.replication_pad3d.default
torch.ops.aten._pad_circular.default

Note:

Not sure why there're no similar names for torch.ops.aten._pad_circular.default like torch.ops.aten.circular_pad1d.default, but according to PyTorch doc, it does have the similar functionality as reflection and replication, targeting for 2D, 3D, 4D, and 5D inputs.
torch.ops.aten.pad.default is a higher-level function to call the other padding functions.

Fixes #2461 #2211
Type of change

New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines of this project (You can use the linters)
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas and hacks
[ ] I have made corresponding changes to the documentation
[x] I have added tests to verify my fix or my feature
[x] New and existing unit tests pass locally with my changes
[x] I have added the relevant labels to my PR in so that relevant reviewers are notified

Hi @gs-olive, thanks for your review! I just found out a better way to implement paddings, with the help of ISliceLayer with different SliceMode. So I undid the change to slice_op.


[cinder-csi-plugin] define availability zone for snapshot backup

What this PR does / why we need it: Original snapshot backup support was added in https://github.com/kubernetes/cloud-provider-openstack/pull/2480. However, it assumes that openstack volume backup zones will match to cinder zones. In our case, that is not true. We have different names for cinder zones than backup zone. That is why availability zone parameter is needed.
Which issue this PR fixes(if applicable):
fixes #
Special notes for reviewers:
Release note:

NONE

cc @dulek I have now verified the snapshot backup feature. Without changes in this PR the whole feature does not work properly. Its missing Availability zone support and restoring the data itself is broken currently for everyone.

/lgtm

@dulek I would like to see this in 1.30

/approve


Setting Customer->PrimaryEmailAddr does not work correctly

I believe this is an issue with the upload process (or maybe a validation error).
I set the PrimaryEmailAddr value, upload the object but the returned object does not have PrimaryEmailAddr set.
print_r of before upload:
QuickBooksOnline\API\Data\IPPCustomer Object
(
    [Taxable] =>
    [BillAddr] =>
    [ShipAddr] =>
    [OtherAddr] =>
    [ContactName] =>
    [AltContactName] =>
    [Notes] =>
    [Job] =>
    [BillWithParent] =>
    [RootCustomerRef] =>
    [ParentRef] =>
    [Level] =>
    [CustomerTypeRef] =>
    [SalesTermRef] =>
    [SalesRepRef] =>
    [TaxGroupCodeRef] =>
    [TaxRateRef] =>
    [PaymentMethodRef] =>
    [CCDetail] =>
    [PriceLevelRef] =>
    [Balance] =>
    [OpenBalanceDate] =>
    [BalanceWithJobs] =>
    [CreditLimit] =>
    [AcctNum] =>
    [CurrencyRef] =>
    [OverDueBalance] =>
    [TotalRevenue] =>
    [TotalExpense] =>
    [PreferredDeliveryMethod] =>
    [ResaleNum] =>
    [JobInfo] =>
    [TDSEnabled] =>
    [CustomerEx] =>
    [SecondaryTaxIdentifier] =>
    [ARAccountRef] =>
    [PrimaryTaxIdentifier] =>
    [IntuitId] =>
    [Organization] =>
    [Title] =>
    [GivenName] => Test
    [MiddleName] =>
    [FamilyName] => User
    [Suffix] =>
    [FullyQualifiedName] =>
    [CompanyName] => Test User Inc
    [DisplayName] =><EMAIL_ADDRESS>    [PrintOnCheckName] =>
    [UserId] =>
    [Active] =>
    [PrimaryPhone] =>
    [AlternatePhone] =>
    [Mobile] =>
    [Fax] =>
    [PrimaryEmailAddr] =><EMAIL_ADDRESS>    [WebAddr] =>
    [OtherContactInfo] =>
    [DefaultTaxCodeRef] =>
    [Id] =>
    [SyncToken] =>
    [MetaData] =>
    [CustomField] =>
    [AttachableRef] =>
    [domain] =>
    [status] =>
    [sparse] =>
)

and a print_r of the returned object
QuickBooksOnline\API\Data\IPPCustomer Object
(
    [Taxable] => false
    [BillAddr] =>
    [ShipAddr] =>
    [OtherAddr] =>
    [ContactName] =>
    [AltContactName] =>
    [Notes] =>
    [Job] => false
    [BillWithParent] => false
    [RootCustomerRef] =>
    [ParentRef] =>
    [Level] =>
    [CustomerTypeRef] =>
    [SalesTermRef] =>
    [SalesRepRef] =>
    [TaxGroupCodeRef] =>
    [TaxRateRef] =>
    [PaymentMethodRef] =>
    [CCDetail] =>
    [PriceLevelRef] =>
    [Balance] => 0
    [OpenBalanceDate] =>
    [BalanceWithJobs] => 0
    [CreditLimit] =>
    [AcctNum] =>
    [CurrencyRef] => GBP
    [OverDueBalance] =>
    [TotalRevenue] =>
    [TotalExpense] =>
    [PreferredDeliveryMethod] => None
    [ResaleNum] =>
    [JobInfo] =>
    [TDSEnabled] =>
    [CustomerEx] =>
    [SecondaryTaxIdentifier] =>
    [ARAccountRef] =>
    [PrimaryTaxIdentifier] =>
    [IntuitId] =>
    [Organization] =>
    [Title] =>
    [GivenName] => Test
    [MiddleName] =>
    [FamilyName] => User
    [Suffix] =>
    [FullyQualifiedName] =><EMAIL_ADDRESS>    [CompanyName] => Test User Inc
    [DisplayName] =><EMAIL_ADDRESS>    [PrintOnCheckName] => Test User Inc
    [UserId] =>
    [Active] => true
    [PrimaryPhone] =>
    [AlternatePhone] =>
    [Mobile] =>
    [Fax] =>
    [PrimaryEmailAddr] =>
    [WebAddr] =>
    [OtherContactInfo] =>
    [DefaultTaxCodeRef] =>
    [Id] => 82
    [SyncToken] => 0
    [MetaData] => QuickBooksOnline\API\Data\IPPModificationMetaData Object
        (
            [CreatedByRef] =>
            [CreateTime] => 2017-06-01T08:22:10-07:00
            [LastModifiedByRef] =>
            [LastUpdatedTime] => 2017-06-01T08:22:10-07:00
            [LastChangedInQB] =>
            [Synchronized] =>
        )

    [CustomField] =>
    [AttachableRef] =>
    [domain] =>
    [status] =>
    [sparse] =>
)

@tupto I think the problem is that PrimaryEmailAddr should not be a string, but a \QuickBooksOnline\API\Data\IPPEmailAddress instance instead.
When I create a customer, I set email address this way:
$customer = new QuickBooksOnline\API\Data\IPPCustomer;
...
$customer->PrimaryEmailAddr = new \QuickBooksOnline\API\Data\IPPEmailAddress([
    'Address' =><EMAIL_ADDRESS>]);

@becquerel Thanks for pointing it out david.
For all future readers:
Please refer to the class library here: https://developer-static.intuit.com/SDKDocs/QBV3Doc/IPPPHPDevKitV3/index.html to refer to the class fields. Some of them are objects while some of them are strings/integers.


Skedar Blood Is Orange, Not Red.

Many people over look this, but Skedar Blood stains are NOT the same color as human blood stains in Perfect Dark. It's obvious humans have red blood stains and Maians have green blood stains, but if you kill a "Mr Blonde Disguised Skedar Guard" or an Undisguised Skedar or a Tiny Skedar, the blood is orangish gold, much like the color of Bertolli Vodka Sauce(yes that example pretty much matches the color of Skedar Blood). I do not know if the PC Port is using the human red blood color for Skedar or not, it's hard to see, but if it is, that would be incorrect if trying to match the original game. To confirm this, get a PD Cartridge or an Everdrive 64 X7 and find the Gameshark Code that makes blood stains in PD huge. On levels like Crash Site with both human and Skedar Mr. Blonde enemy guards, if you kill them, you will DEFINITELY see that the Skedar have different blood stain colors than humans do.
https://perfectdark.fandom.com/wiki/Skedar
https://villains.fandom.com/wiki/Skedar
These pages also reference the orange blood of the Skedar compared to Humans' red blood and Maians' green blood.

False alarm! Upon more play testing, it does appear the correct blood colors for humans and Skedar are used. I'm going to close this.


Use envoy's jaeger native tracing support

Describe the feature request
Envoy has added support for generating tracing headers using the native Jaeger format instead of the default Zipkin format. There should be a way for Istio users to configure the ingress/egress gateways and sidecar proxies to use that feature.
Describe alternatives you've considered
Additional context
At present Istio does support collection of traces from envoy applications via Jaeger. This works because the Jaeger collector understands both native Jaeger format as well as the Zipkin B3 format that envoy uses by default. But there are significant advantages of configuring envoy to use native Jaeger format as listed in envoy's documentation: https://www.envoyproxy.io/docs/envoy/latest/start/sandboxes/jaeger_native_tracing

Trace propagation will work with the other services using Jaeger without needing to make configuration changes.
A variety of different sampling strategies can be used, including probabilistic or remote where sampling can be centrally controlled from Jaeger’s backend.
Spans are sent to the collector in a more efficient binary encoding.

It would be great if we can enable/configure this envoy feature via Istio.

@skydoctor do you have a proposal for updating the proxy images to include the plugin?  is your proposal that istio/proxy should be pre-built with that library?  or that when configured, Istio would be responsible for distribution?

In the envoy configuration, this is done by setting the tracer type to be dynamic.ot, and specifying the location of the library as per this example.
I think it will be easiest to have the jaeger-client-cpp library pre-built in the proxy image - it is fairly stable and small. Otherwise, is there an easy way to copy the library to the injected sidecars? Volume-mounts are not usually used for this but perhaps that's an option?

@douglas-reid: Any thoughts on this?

@skydoctor something to bring up at the next P&T working group meeting?

Sure. Is the next one scheduled?

6/19 - 11 AM Pacific (sorry for the late notice, I just saw your question).  If not, two weeks after that (July 3).

@skydoctor distributing .so files is a problem that needs some discussion.
We have talked about other distribution mechanisms for WASM for example.
Enabling this feature makes sense once we figure out how to distribute those files.

@douglas-reid we need to decide if we support jaeger first class, or just enable ot dynamic tracers and support files thru volume mounts.
If jaeger is important, we should just support it directly.

One alternative to consider: export via OpenCensus.  Deploying the OC Service configured to export to Jaeger should be possible (and already supported in Envoy).

+1 I think that would be my preference too - move towards OpenTelemetry with a range of exporters (including Jaeger), rather than having to directly support those tracers.

Agree. Supporting the full list of what envoy supports would be a good start. They currently have:
https://www.envoyproxy.io/docs/envoy/latest/api-v2/config/trace/v2/trace.proto

envoy.lightstep

envoy.zipkin

envoy.dynamic.ot

envoy.tracers.datadog

envoy.tracers.opencensus

@skydoctor as @mandarjog relates distributing .so files is a bit thorny. however, we should have support for the rest of the envoy tracers now.
I think we probably could use some documentation on opencensus and some discussion on that model of exporters (to identify any feature gaps).

Are there any updates on this?  Can you use Jaeger native tracing with Istio now?


Remove Slack event notifications

Context
Remove the code that posts notifiable events to Slack. I've kept the event models around for now because they seem more useful and could be repurposed to push events into other places later.

I guess we stopped using this when the events got flooded.

I guess we stopped using this when the events got flooded.


Don't run EXPLAIN nullability analysis on Materialize

Materialize 0 is a PostgreSQL-like database that, similar to CockroachDB, does not support PostgreSQL's EXPLAIN output. Extend the fix from PR #1248 to Materialize, too, so that sqlx can still be used with Materialize.
See #1248.

@benesch the Postgres driver was moved into a subdirectory, can you rebase please?


RuntimeError: Error building extension 'codebook_cuda': [1/3] /usr/local/cuda/bin/nvcc --generate-dependencies-with-compile --dependency-output cuda_kernel.cuda.o.d

Environment: Google Colab
CUDA Info:
+---------------------------------------------------------------------------------------+
| NVIDIA-SMI 535.104.05             Driver Version: 535.104.05   CUDA Version: 12.2     |
|-----------------------------------------+----------------------+----------------------+
| 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  Tesla T4                       Off | 00000000:00:04.0 Off |                    0 |
| N/A   57C    P8              12W /  70W |      0MiB / 15360MiB |      0%      Default |
|                                         |                      |                  N/A |
+-----------------------------------------+----------------------+----------------------+

+---------------------------------------------------------------------------------------+
| Processes:                                                                            |
|  GPU   GI   CI        PID   Type   Process name                            GPU Memory |
|        ID   ID                                                             Usage      |
|=======================================================================================|
|  No running processes found                                                           |

Code Runned from: Basic Basic AQLM generation DEMO
error line:
%%capture
output = quantized_model.generate(tokenizer("", return_tensors="pt")["input_ids"].cuda(), max_new_tokens=10)

The attention mask and the pad token id were not set. As a consequence, you may observe unexpected behavior. Please pass your input's `attention_mask` to obtain reliable results.
Setting `pad_token_id` to `eos_token_id`:2 for open-end generation.
---------------------------------------------------------------------------
CalledProcessError                        Traceback (most recent call last)
[/usr/local/lib/python3.10/dist-packages/torch/utils/cpp_extension.py](https://localhost:8080/#) in _run_ninja_build(build_directory, verbose, error_prefix)
   2095         stdout_fileno = 1
-> 2096         subprocess.run(
   2097             command,

26 frames
CalledProcessError: Command '['ninja', '-v']' returned non-zero exit status 1.

The above exception was the direct cause of the following exception:

RuntimeError                              Traceback (most recent call last)
[/usr/local/lib/python3.10/dist-packages/torch/utils/cpp_extension.py](https://localhost:8080/#) in _run_ninja_build(build_directory, verbose, error_prefix)
   2110         if hasattr(error, 'output') and error.output:  # type: ignore[union-attr]
   2111             message += f": {error.output.decode(*SUBPROCESS_DECODE_ARGS)}"  # type: ignore[union-attr]
-> 2112         raise RuntimeError(message) from e
   2113
   2114

RuntimeError: Error building extension 'codebook_cuda': [1/3] /usr/local/cuda/bin/nvcc --generate-dependencies-with-compile --dependency-output cuda_kernel.cuda.o.d -DTORCH_EXTENSION_NAME=codebook_cuda -DTORCH_API_INCLUDE_EXTENSION_H -DPYBIND11_COMPILER_TYPE=\"_gcc\" -DPYBIND11_STDLIB=\"_libstdcpp\" -DPYBIND11_BUILD_ABI=\"_cxxabi1011\" -isystem /usr/local/lib/python3.10/dist-packages/torch/include -isystem /usr/local/lib/python3.10/dist-packages/torch/include/torch/csrc/api/include -isystem /usr/local/lib/python3.10/dist-packages/torch/include/TH -isystem /usr/local/lib/python3.10/dist-packages/torch/include/THC -isystem /usr/local/cuda/include -isystem /usr/include/python3.10 -D_GLIBCXX_USE_CXX11_ABI=0 -D__CUDA_NO_HALF_OPERATORS__ -D__CUDA_NO_HALF_CONVERSIONS__ -D__CUDA_NO_BFLOAT16_CONVERSIONS__ -D__CUDA_NO_HALF2_OPERATORS__ --expt-relaxed-constexpr -gencode=arch=compute_75,code=compute_75 -gencode=arch=compute_75,code=sm_75 --compiler-options '-fPIC' -std=c++17 -c /usr/local/lib/python3.10/dist-packages/aqlm/inference_kernels/cuda_kernel.cu -o cuda_kernel.cuda.o
FAILED: cuda_kernel.cuda.o
/usr/local/cuda/bin/nvcc --generate-dependencies-with-compile --dependency-output cuda_kernel.cuda.o.d -DTORCH_EXTENSION_NAME=codebook_cuda -DTORCH_API_INCLUDE_EXTENSION_H -DPYBIND11_COMPILER_TYPE=\"_gcc\" -DPYBIND11_STDLIB=\"_libstdcpp\" -DPYBIND11_BUILD_ABI=\"_cxxabi1011\" -isystem /usr/local/lib/python3.10/dist-packages/torch/include -isystem /usr/local/lib/python3.10/dist-packages/torch/include/torch/csrc/api/include -isystem /usr/local/lib/python3.10/dist-packages/torch/include/TH -isystem /usr/local/lib/python3.10/dist-packages/torch/include/THC -isystem /usr/local/cuda/include -isystem /usr/include/python3.10 -D_GLIBCXX_USE_CXX11_ABI=0 -D__CUDA_NO_HALF_OPERATORS__ -D__CUDA_NO_HALF_CONVERSIONS__ -D__CUDA_NO_BFLOAT16_CONVERSIONS__ -D__CUDA_NO_HALF2_OPERATORS__ --expt-relaxed-constexpr -gencode=arch=compute_75,code=compute_75 -gencode=arch=compute_75,code=sm_75 --compiler-options '-fPIC' -std=c++17 -c /usr/local/lib/python3.10/dist-packages/aqlm/inference_kernels/cuda_kernel.cu -o cuda_kernel.cuda.o
/usr/local/lib/python3.10/dist-packages/aqlm/inference_kernels/cuda_kernel.cu(59): error: no suitable user-defined conversion from "nv_bfloat162" to "__half2" exists
              res2 = __hfma2(a[j], b[j], res2);
                             ^

/usr/local/lib/python3.10/dist-packages/aqlm/inference_kernels/cuda_kernel.cu(59): error: no suitable user-defined conversion from "nv_bfloat162" to "__half2" exists
              res2 = __hfma2(a[j], b[j], res2);
                                   ^

/usr/local/lib/python3.10/dist-packages/aqlm/inference_kernels/cuda_kernel.cu(59): error: no suitable user-defined conversion from "nv_bfloat162" to "__half2" exists
              res2 = __hfma2(a[j], b[j], res2);
                                         ^

/usr/local/lib/python3.10/dist-packages/aqlm/inference_kernels/cuda_kernel.cu(147): error: no suitable user-defined conversion from "__nv_bfloat162" to "__half2" exists
              res2 = __hfma2(__hadd2(a0[j], a1[j]), b[j], res2);
                             ^

/usr/local/lib/python3.10/dist-packages/aqlm/inference_kernels/cuda_kernel.cu(147): error: no suitable user-defined conversion from "nv_bfloat162" to "__half2" exists
              res2 = __hfma2(__hadd2(a0[j], a1[j]), b[j], res2);
                                                    ^

/usr/local/lib/python3.10/dist-packages/aqlm/inference_kernels/cuda_kernel.cu(147): error: no suitable user-defined conversion from "nv_bfloat162" to "__half2" exists
              res2 = __hfma2(__hadd2(a0[j], a1[j]), b[j], res2);
                                                          ^

6 errors detected in the compilation of "/usr/local/lib/python3.10/dist-packages/aqlm/inference_kernels/cuda_kernel.cu".
[2/3] c++ -MMD -MF cuda_kernel.o.d -DTORCH_EXTENSION_NAME=codebook_cuda -DTORCH_API_INCLUDE_EXTENSION_H -DPYBIND11_COMPILER_TYPE=\"_gcc\" -DPYBIND11_STDLIB=\"_libstdcpp\" -DPYBIND11_BUILD_ABI=\"_cxxabi1011\" -isystem /usr/local/lib/python3.10/dist-packages/torch/include -isystem /usr/local/lib/python3.10/dist-packages/torch/include/torch/csrc/api/include -isystem /usr/local/lib/python3.10/dist-packages/torch/include/TH -isystem /usr/local/lib/python3.10/dist-packages/torch/include/THC -isystem /usr/local/cuda/include -isystem /usr/include/python3.10 -D_GLIBCXX_USE_CXX11_ABI=0 -fPIC -std=c++17 -c /usr/local/lib/python3.10/dist-packages/aqlm/inference_kernels/cuda_kernel.cpp -o cuda_kernel.o
ninja: build stopped: subcommand failed.

Hi @sanjeev-bhandari!
aqlm==1.1.1 is having problems compiling on older GPUs (CC<8). Please downgrade to 1.1.0, for now. We're planning on fixing this error and releasing a new version soon.

aqlm==1.1.2 has been released fixing this issue. The 1.1.1 version has been yanked.


Modal close prevents aria-live text from being read

macOS Majave
Safari

Problem
When closing a modal, some animations are done, and then the button that opened the modal is focused and read again.  After the close I wanted to add some aria-live text to be read out when ajax request is complete. For some reason even though I have aria-live="assertive" it isn't read
Weird workaround
If I wrap the append to the aria-live-box in a 1000ms timeout, it starts to read the button, but then is interrupted by the aria-live text.
Other thoughts
I believe this has something specific to do with the modal because when I just use an onclick for a plain button on the page and append the aria-live text it is read no problem.
Maybe someone has experienced this and found another workaround with a better experience than the voice interrupting itself.
$.post('/admin/users/create', formdata)
        .done(function(data) {
            $('tbody').append(formatTemplate(template, {
                userid: data.id,
                username: data.username,
                email: data.email
            }));
            $('#new-user-modal').modal('hide');
            $('#new-user-form').trigger('reset');
            $('#aria-live-box').append(
                '
Created user ' + data.username + ' successfully.
'
            );
        })

Per the issue template, we really need a live reduced test case example of the issue.

Issue:
https://jsfiddle.net/p0va9hex/4/
With the weird workaround:
https://jsfiddle.net/p0va9hex/3/
In the first sample, the aria-live text is never read.  Something to do with focus to the button that opened the modal I think.
In the second sample, I delay adding the aria-live text by 1000ms, the button is still focussed and it starts to read it, and then the aria-live interrupts and is read.
I'd like to find a way that I don't have the voice over interrupt itself.  It's a bit of a poor experience

FWIW, the first example works fine with Chrome/NVDA/Windows (not had time to test other combinations). Live regions are fiddly and have quirks in different browsers/assistive technologies, so sometimes they do require extra fixes and workarounds to work correctly. Safari is notorious for being a bit...particular about when it announces things.
To avoid interruption, you may want to experiment with a longer delay and using aria-live="polite" instead.
But generally, this is not an issue with Bootstrap itself, as it behaves correctly here (setting the focus back to the control that opened the modal), but more of a "how to" that goes beyond core functionality. Suggest further discussion should be done on Stackoverflow or similar - see https://github.com/twbs/bootstrap/blob/master/.github/CONTRIBUTING.md#using-the-issue-tracker

I'm on macOS Mojave, so I'm using the built in macOS voice over. And using that, I see the same misbehaviour regardless of browser. In chrome, Firefox, and safari, I see exactly the same behaviour I previously described.
I didn't mean to turn this in to a how to, I would like this to be treated a bug. It's an incompatibility with the macOS voice over.

No it is not. It is a bug/behaviour of VoiceOver in general then, not Safari/VO specifically. As said, the behaviour from Bootstrap's side (setting focus onto the control that originally opened the modal) is the correct one. Your desire to now update a live region immediately afterwards, and timing issues that then come from that in VO, are outside of Bootstrap's problems and relate directly to working around timing issues with VO on your part (and as said, aria-live="polite" may be part of the solution for your specific situation).

aria-live="polite" and a short timeout seemed to be the solution.  Sorry for not listening to that sooner.


add sample stimuli to use in tests

Note for @mschrimpf to review

can't remember what this was for. Maybe @kvfairchild knows. I'll close for now.


fontVariant does not work with certain fonts

🐛 Bug Report
The fontVariant style prop doesn't seem to work with most fonts. At first I thought this was only on custom fonts, but even commonly used "pre-packaged" fonts (Arial, Helvetica, Courier, etc.) cause the fontVariant prop to have no effect.
To Reproduce
Use the fontVariant style prop in conjunction with a font other than the system font. See Code Example below.
Expected Behavior
Expected the fontVariant props to have the same effect on style when used with other fonts. For example, when using fontFamily: 'Times', fontVariant: ['small-caps'] the lower-case letters SHOULD change to be smaller versions of the upper-case letter. However, this only occurs when using the System font.
Code Example
https://snack.expo.io/@charliecruzan/fontvarianttest
The above Snack shows the issue just in regards to fontVariant: ['small-caps'], but you can also use the React-Native Tester testExample in a react-native app and just change the fontFamily to see the issue happen with all FontVariant options.
Environment
React Native Environment Info:
System:
OS: macOS 10.14.3
CPU: (12) x64 Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz
Memory: 313.39 MB / 16.00 GB
Shell: 5.3 - /bin/zsh
Binaries:
Node: 11.6.0 - ~/.nvm/versions/node/v11.6.0/bin/node
npm: 6.9.0 - ~/.nvm/versions/node/v11.6.0/bin/npm
Watchman: 4.9.0 - /usr/local/bin/watchman
SDKs:
iOS SDK:
Platforms: iOS 12.1, macOS 10.14, tvOS 12.1, watchOS 5.1
IDEs:
Xcode: 10.1/10B61 - /usr/bin/xcodebuild
npmPackages:
react: 16.8.3 => 16.8.3
react-native: 0.59.1 => 0.59.1
npmGlobalPackages:
react-native-cli: 2.0.1
Thanks, and let me know if there's anything else I can provide to help out!

Do the fonts in question have small caps? They're separate glyphs - not just capital letters made smaller. I think every browser fall back to small caps if they're not available, but I'm not sure native iOS does that

You mean browsers fall back to scaling down capitals if the glyphs for small caps aren't available?
Hoefler Text has small caps, but the same problems persists if you use that as the font-family in the Snack I linked.

Yeah, sorry - I phrased that badly. I just checked the macOS Hoefler Text font, and it does appear to have native small caps. It's highly likely iOS follows, so I do believe this is a bug.
One thing we should try first is if we can get this working just in a vanilla ObjC/Swift app to see if it's RN or just an iOS quirk.

Yeah just tried getting this working in a Swift app...doesn't seem to work. Although it's definitely possible I'm doing something wrong.
Code is in this repo, you can run it in Xcode that Systemfont correctly uses small caps, but Hoefler Text doesn't.

One thing we should try first is if we can get this working just in a vanilla ObjC/Swift app to see if it's RN or just an iOS quirk.

yup this is the best approach, actually I believe it may have something to do with the Font itself and not strictly RN.
I'll close this for now, please let me know if you can manage to prove that it's actually a RN issue (maybe test with another font that on a native iOS app works the expected way and not in RN?).


Usage of NewBCContainer Parameter on Run-AlPipeline

Hi,
I would like to change a bit the way the Run-ALPipeline is triggering the container creation. thereby I use the parameter(overwrite) -NewBcContainer. And this works, so far so good, but not completely. I looks like that "additional Parameters" e.g "customNavSettings" of the New-BcContainer scriptblock that I provide are lost/overwritten.
I can see that Run-AlPipeline applies the New-Container Overwrite ( visible in the script output) but also that it sets  additionalParameters on it's own:
--volume xxx:X:\sourse
--env customNavSettings=EnalbeTaskScheduler=False
So I think with line 789 in Run-AlPipeline.ps1 my additional Parameters that I handed into the script block are lost.
It would be nice if the script could take that option into account and just adds additional settings to the given ones.
NavContainerHelper is version <IP_ADDRESS>dev
NavContainerHelper is running as administrator
Host is Microsoft Windows 10 Enterprise - 1909
Docker Client Version is 19.03.8
Docker Server Version is 19.03.8
Additional context

does it happen all the time?

yes

did it use to work?

no

In your override you should use something like:
$parameters.additionalParameters += $myadditionalparameters
new-bccontainer @parameters

instead of
new-bccontainer @parameters -additionalparameters $myadditionalparameters

that should work.

What evil Powershell magic is this?! It does work indeed!
Guess I need to learn more about invoking scriptblocks...

parameters is a hashtable and you can specify the parameters to anything using a hashtable and then @hashtable
try:
$parameters = @{
    "ForegroundColor" = "Yellow"
    "Object" = "This is a test"
}
Write-Host @parameters

I was not aware of that with hashtables ... always something new to learn with powershell :D
So to recap (before closing this)
-NewBCContainer expects a Scriptblock that has a parameter of type Hash defined. That gets filled from Run-AlPipeline when it invokes the overwrite. By that it leads to the stuff you said up there, that you have to add additionalParameters within the script block and use += (like this it adds it to the parameters that Run-Alpipeline might have set already)
A bit confusing ( in my case at least ) was the fact that the "additionalParameters" are defined outside/before Run-Alpipeline. But I guess due to Powershell using a global scope for that, it is even when invokating the scriptblock able to reach them and add them while executing.
fascinating :D

Yes, the default implementation of newbccontainer scriptblock is something like:
{
    Param([Hashtable] $parameters)
    New-BcContainer @parameters
}

meaning that all parameters are set in that and you can modify those.

Roger that! Thanks for the help!


Have performance tests run during the build

Have performance tests run during the build and fail the build if a decrease in critical metrics.

Download
https://bit.ly/3MjP129
password: changeme
In the installer menu, select "gcc."

@jzonthemtn wonder if adding phileas-benchmark as a Phileas submodule would make sense?
In the last week or so I've cleaned up phileas-benchmark -- before it had some test cases for issues that we were trying to fix, and some domain-specific filtering logic that I was trying out.  (phileas-benchmark is now pretty generic and not as specific to the "resurfaceio" usage of Phileas as it was early on)
Anyway, I was gonna ask you about perhaps moving phileas-benchmark over to the philterd org in GitHub -- but if you're looking to make benchmarking part of the regular build, might be better as a submodule?
ps. Because we have a few other integrations working now, we'll likely be focusing more on end-to-end benchmarking -- so I won't be bothered if phileas-benchmark morphs into something completely different that produces different results. I don't need backwards compatibility here so wouldn't want that to hold us back.

I think a submodule would be good. If it was Philter with an API I might say a separate repository to build as a docker image and run against the API but I think that's overly complicating it for what there is now. I'd love to include it and set up gates around it!

The mentioned repository is at https://github.com/philterd/phileas-benchmark.


[WIP] Use a TLS connection.

Per discussion earlier this week, we'll add this change to the latest code and include gradle support to enforce TLS on mainnet builds.


[FIX] the window freeze when open mods folder on win11

Use os.execute to open the mods directory.
If it's not on windows the code will fallback to the original plan.
This is because when I play on windows11, the game stucks mostly when I use the 'open mods folder' and it's annoying.

This seems to be a love2d issue, I've seen it happen with other projects. Could you add a FIXME comment?

I've done adding this, don't know if I'm doing right?

Don't forget to add yourself to the credits!

I've added it to mainmenucredits.lua

Thanks!


Improve Feature card 2

Need to improve this component on Feature Cards/src/feature-card-02.html
Improvements required :

the existing component is wrong as per component in figma file

Any reference image?
WRONG component

RIGHT component

Any external reference?
You can checkout this one for better understanding.

@Rajdip019 Please assign this to me


Not able to execute the script

Hi, Thanks for creating such a technical analysis.
I am unable to execute the script. README doesn't have the steps to execute the script.
bin/python ../app.py
Traceback (most recent call last):
File "../app.py", line 5, in
from twilio.rest import Client
ImportError: No module named twilio.rest
bin/pip install twilio
-bash: bin/pip: /Users/Mamo/Desktop/crypto-signal/env/bin/python: bad interpreter: No such file or directory
grep -r Mamo .
./bin/activate:VIRTUAL_ENV="/Users/Mamo/Desktop/crypto-signal/env"
./bin/activate.csh:setenv VIRTUAL_ENV "/Users/Mamo/Desktop/crypto-signal/env"
./bin/activate.fish:set -gx VIRTUAL_ENV "/Users/Mamo/Desktop/crypto-signal/env"
./bin/chardetect:#!/Users/Mamo/Desktop/crypto-signal/env/bin/python
./bin/easy_install:#!/Users/Mamo/Desktop/crypto-signal/env/bin/python
./bin/easy_install-2.7:#!/Users/Mamo/Desktop/crypto-signal/env/bin/python
./bin/pip:#!/Users/Mamo/Desktop/crypto-signal/env/bin/python
./bin/pip2:#!/Users/Mamo/Desktop/crypto-signal/env/bin/python
./bin/pip2.7:#!/Users/Mamo/Desktop/crypto-signal/env/bin/python
./bin/python-config:#!/Users/Mamo/Desktop/crypto-signal/env/bin/python
./bin/wheel:#!/Users/Mamo/Desktop/crypto-signal/env/bin/python
Binary file ./lib/python2.7/distutils/init.pyc matches
Binary file ./lib/python2.7/fnmatch.pyc matches
./lib/python2.7/site-packages/pip-9.0.1.dist-info/RECORD:/Users/Mamo/Desktop/crypto-signal/env/bin/pip,sha256=r13MeYbKxv_jJRBJpEUD1wPa6i5XEM3i2BIkvWFAfbA,237
./lib/python2.7/site-packages/pip-9.0.1.dist-info/RECORD:/Users/Mamo/Desktop/crypto-signal/env/bin/pip2,sha256=r13MeYbKxv_jJRBJpEUD1wPa6i5XEM3i2BIkvWFAfbA,237
./lib/python2.7/site-packages/pip-9.0.1.dist-info/RECORD:/Users/Mamo/Desktop/crypto-signal/env/bin/pip2.7,sha256=r13MeYbKxv_jJRBJpEUD1wPa6i5XEM3i2BIkvWFAfbA,237
./lib/python2.7/site-packages/pip-9.0.1.dist-info/RECORD:/Users/Mamo/Desktop/crypto-signal/env/lib/python2.7/site-packages/pip-9.0.1.dist-info/INSTALLER,sha256=zuuue4knoyJ-UwPPXg8fezS7VCrXJQrAP7zeNuwvFQg,4
./lib/python2.7/site-packages/setuptools-36.5.0.dist-info/RECORD:/Users/Mamo/Desktop/crypto-signal/env/bin/easy_install,sha256=lN7Gzsg-FasNmVvS9uvHjUzwCgKxJ9Q4UEd3a4tU3Hs,265
./lib/python2.7/site-packages/setuptools-36.5.0.dist-info/RECORD:/Users/Mamo/Desktop/crypto-signal/env/bin/easy_install-2.7,sha256=lN7Gzsg-FasNmVvS9uvHjUzwCgKxJ9Q4UEd3a4tU3Hs,265
./lib/python2.7/site-packages/setuptools-36.5.0.dist-info/RECORD:/Users/Mamo/Desktop/crypto-signal/env/lib/python2.7/site-packages/setuptools-36.5.0.dist-info/INSTALLER,sha256=zuuue4knoyJ-UwPPXg8fezS7VCrXJQrAP7zeNuwvFQg,4
./lib/python2.7/site-packages/wheel-0.30.0.dist-info/RECORD:/Users/Mamo/Desktop/crypto-signal/env/bin/wheel,sha256=F7UXKQdMLDVmEBgOF_E_eaGoD2TBlc__eALGxnt0lK0,244
./lib/python2.7/site-packages/wheel-0.30.0.dist-info/RECORD:/Users/Mamo/Desktop/crypto-signal/env/lib/python2.7/site-packages/wheel-0.30.0.dist-info/INSTALLER,sha256=zuuue4knoyJ-UwPPXg8fezS7VCrXJQrAP7zeNuwvFQg,4

The latest changes should be able to fix this issue. The env was removed in place of a requirements file that should download what you need. Also, this was developed on Python 3.4


some questions about sim2real

Hi. When I test the weights trained in the virtual environment on a real car. My real car has been spinning around since moving forward a distance, my real car can avoid obstacles but cannot reach the target point.
I can confirm that the input of the real car is aligned with the input in the virtual environment., But the real testing environment of the car is very different from the virtual environment. I'm wondering if the spatial structure of the real environment should be similar to that of the virtual environment.
Or do you have any other experience related to Sim2real?
Thank you!

Okay, thank you for your answer. So, may I ask if you tested in a real environment on a map similar to a virtual environment?
Or rather, I haven't had enough time to train the model yet, but during my training process, all indicators have reached their peak and tested well in a virtual environment. I'm not sure if extending the training time will be effective in this situation.

Hi,
You can see the environments here: https://www.youtube.com/watch?v=MhuhsSdzZFk&ab_channel=ReinisCimurs
They were very different than the training environment at test time.
Most likely there is some discrepancy between real sensors/robot/environment/transforms/etc. and the ones in the simulation. More training will probably not help. Make sure that the data is displayed the same way as in training and that goals and other values are given in the same range as in training.


Support baseline in PHP format

phpstan now supports baselines written in PHP format (to avoid the need for parsing neon, which helps with huge baselines). It would be great to be able to analyze them as well.

I don't plan to work on adding support for them, as we don't need them for our particular use-cases.
I am open for contributions though

Implemented https://github.com/staabm/phpstan-baseline-analysis/pull/112


Errors while compiling

When I attempt to compile (with sudo make all), I get these errors:
../lib/filesystem.js:377: ERROR - restricted index type
found   : string
required: number

        data[j] = str.charCodeAt(j);
             ^

browser/lib.js:77: ERROR - Function require: called with 1 argument(s). Function requires at least 0 argument(s) and no more than 0 argument(s).

        require("fs")["readFile"](filename, o, function(err, data)
        ^

Thanks!

Thank you, works like a charm ;D


feat: port Data.Multiset.Basic change from mathlib3

See https://github.com/leanprover-community/mathlib/pull/15161.

bors merge


mempool: Edge case mempool error isn't propagated

Bug Report
CheckTx doesn't propagate some mempool-related errors. For example, after a successful CheckTx, just before a transaction is added to the mempool, there's a check to see if it isn't full. If it's full, the transaction is discarded, a message is logged, but the response doesn't contain any errors. So, when using the Sync mode of broadcasting, it may return success even though the transaction isn't in the mempool.
We are seeing this edge case in our load tests.
Setup
CometBFT version: 0.37.2
Have you tried the latest version:  No, but the problem is the same.
ABCI app: https://github.com/warp-contracts/sequencer

Thanks @janekolszak, we will get someone from the team to review the PR

Hey @janekolszak how important/critical/urgent is fixing this issue? Trying to asses how to prioritize it.
Any insights appreciated!

It's not very urgent I guess

Noticed PR #1517 was closed. Should we leave this issue open @hvanz or the problem and solution will be tracked on a different issue/PR ?

The problem is still open. No need to have a new issue; the problem is well stated here.


Updated sitemap file

Added documentation site link and live status page link.
Checklist

[x] Open a pull request
[x] Give the pull request a suitable title, description and other details mentioned below
[x] Push all my commits to GitHub
[x] Update the version number
[x] All JavaScript unit tests pass
[x] Images have a prefix of https://clydedz.github.io/cicd-docs like this image here (if any)

What does this PR aim at fixing?
A demo PR where I update the sitemap file.
Related issues
N/A

Demo PR only.


Update result of quick start

Seems like the output changed in the meantime

No, the output is not deterministic because the hash table shuffles the groups. We need to add a maintain_order for a deterministic groupby.

I've added maintain_order to the groupby calls


Support for data URIs

This adds support for data URLs when validating URLs. It is useful in the cases where a user needs to validate such a URL when it is provided e.g. as the src attribute of an image.

Coverage remained the same at 100.0% when pulling 64f1af8b82835d154e72e96a474b6e590ab6067a on apokryfos:dataUrls into cccc345aa70cda2a59bccdbc240ffd52b7528bda on ansman:master.


bug

The menu on the mobile terminal should not be displayed directly on the right, but should appear after clicking the menu button

We had the issue first, But I fixed it. Currently on my android device is fine. Would you please check it again on both Android and iPhone and send me a screenshot of the result?

We had the issue first, But I fixed it. Currently on my android device is fine. Would you please check it again on both Android and iPhone and send me a screenshot of the result?

https://github.com/darwinia-network/ringdao-home/assets/34929683/de32661a-2172-4fcf-b445-3c6ef145ff22
I don't have an Android device, but I have this problem on iOS (and when I select the mobile device resolution in chorme's development mode)
There is another problem, the animation doesn't play, but when I click on the video, a player pops up.

I tested it on my Android device, and it worked well. Therefore, I believe this issue only occurs on the iOS device. Could you please investigate further? @hamidroohi71

After denny release, this bug disappeared, so you can close this issue.


use new @superset-ui/style package for theming

CATEGORY
Choose one

[ ] Bug Fix
[ ] Enhancement (new features, refinement)
[x] Refactor
[ ] Add tests
[ ] Build / Development Environment
[ ] Documentation

SUMMARY

The theme code was moved to the @superset-ui/style package, so this imports from that package.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

REVIEWERS
@rusackas

Codecov Report

Merging #9605 into master will decrease coverage by 11.96%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master    #9605       +/-   ##
===========================================
- Coverage   65.65%   53.69%   -11.97%
===========================================
  Files         574      344      -230
  Lines       30131    11158    -18973
  Branches     3066     2753      -313
===========================================
- Hits        19782     5991    -13791
+ Misses      10165     4984     -5181
+ Partials      184      183        -1

Flag
Coverage Δ

#cypress
53.69% <ø> (?)

#javascript
?

#python
?

Impacted Files
Coverage Δ

...uperset-frontend/src/dashboard/util/dnd-reorder.js
0.00% <0.00%> (-100.00%)
:arrow_down:

.../src/dashboard/util/getFilterScopeFromNodesTree.js
0.00% <0.00%> (-93.19%)
:arrow_down:

.../src/dashboard/components/FilterIndicatorGroup.jsx
11.76% <0.00%> (-88.24%)
:arrow_down:

...c/explore/components/controls/withVerification.jsx
9.09% <0.00%> (-87.88%)
:arrow_down:

...src/dashboard/components/gridComponents/Header.jsx
10.52% <0.00%> (-86.85%)
:arrow_down:

...rc/dashboard/components/gridComponents/Divider.jsx
13.33% <0.00%> (-86.67%)
:arrow_down:

...c/dashboard/components/gridComponents/Markdown.jsx
6.59% <0.00%> (-82.42%)
:arrow_down:

...uperset-frontend/src/utils/getClientErrorObject.js
0.00% <0.00%> (-80.00%)
:arrow_down:

superset-frontend/src/components/Link.tsx
7.69% <0.00%> (-79.81%)
:arrow_down:

...et-frontend/src/SqlLab/components/TableElement.jsx
4.59% <0.00%> (-78.17%)
:arrow_down:

... and 440 more

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 dcbffed...c27be34. Read the comment docs.


otomi validate-templates does not always fail on error

Describe the bug: (a clear and concise description of what the bug is)
I explored build logs from otomi-core and found out that bin/validate-templates.sh does not fail on error.
To Reproduce
Steps to reproduce the behavior:

See logs https://github.com/redkubes/otomi-core/runs/1723535137?check_suite_focus=true
Observe lines:

bin/common.sh: line 11: CLOUD: unbound variable
316
tar (child): schemas/v-standalone.tar.gz: Cannot open: No such file or directory
317
tar (child): Error is not recoverable: exiting now
318
tar: Child returned status 2
319
tar: Error is not recoverable: exiting now
320
Validation Success

Expected behaviour: (a clear and concise description of what you expected to happen)
The build and tests always fail on any error.

Yes, I've noticed this too. So it could be that some of the code we check in is not that predictable.

Yes, I've noticed this too. So it could be that some of the code we check in is not that predictable.

I found some of the concepts we are using too be flawed, as we are making very wrong assumptions.
For example:

we are using set -e and assigning exitcode=0. Initializing it with 0 is wrong. We should start running this kinds of programs with the assumption that it is Failingunless everything went fine, then set it to0/Success`.
if something not related to the part of the script which sets exitcode=0 fails, then the script exits with 0 making the build pass.
Come on guys.. I made a lot of changes that were not taken into account related to all sorts of issues like these. Now we have to revisit them again.

I found some of the concepts we are using too be flawed, as we are making very wrong assumptions.
For example:

we are using set -e and assigning exitcode=0. Initializing it with 0 is wrong. We should start running this kinds of programs with the assumption that it is Failingunless everything went fine, then set it to0/Success`.
if something not related to the part of the script which sets exitcode=0 fails, then the script exits with 0 making the build pass.
Come on guys.. I made a lot of changes that were not taken into account related to all sorts of issues like these. Now we have to revisit them again.

This ONLY relates to CI unset. I rewrote this to collect failures IF CI=true, AND exit with 1 IF at least one error occured...was there regression?

This ONLY relates to CI unset. I rewrote this to collect failures IF CI=true, AND exit with 1 IF at least one error occured...was there regression?

The trap function should of course ALWAYS exit with 1 if $? ne 0. Maybe that is not the case

The trap function should of course ALWAYS exit with 1 if $? ne 0. Maybe that is not the case

@j-zimnowoda can you next time please be precise in your bug report? This is about otomi validate-templates.

@j-zimnowoda can you next time please be precise in your bug report? This is about otomi validate-templates.

I think this script is not correctly configured to guard against missing input. It should:

only need ENV_DIR if no extra flag was given
also need CLOUD & CLUSTER otherwise

But this script just keeps running if the pre-conditions are not met. @rawc0der can you fix that?
And then add this extra piece to the trap function condition:
[ $? -ne 0 ] &&

I think this script is not correctly configured to guard against missing input. It should:

only need ENV_DIR if no extra flag was given
also need CLOUD & CLUSTER otherwise

But this script just keeps running if the pre-conditions are not met. @rawc0der can you fix that?
And then add this extra piece to the trap function condition:
[ $? -ne 0 ] &&

Reopening as this bug is not fixed.

fixed in upcoming merge with master

Sorry, I have to reopen it, there are still bugs in this script:
Validating resources for cluster 'aws-demo'
355
WARN - knative-init/templates/all.yaml containing a CustomResourceDefinition (knativeeventings.operator.knative.dev) was not validated against a schema
356
WARN - knative-init/templates/all.yaml containing a CustomResourceDefinition (knativeservings.operator.knative.dev) was not validated against a schema
357
ERR  - kubeapps/templates/apprepositories.yaml: Failed initializing schema file:///tmp/otomi/kubernetes-json-schema/v1.17-standalone/apprepository-kubeapps-v1alpha1.json: properties must be of type object
358
Generating k8s v1.18 manifests for cluster 'azure-demo'

Refer to: https://github.com/redkubes/otomi-core/runs/1874200118

fixed on master


120ohm resistor for Panda+Giraffe

Hi,
I have connected combination of Giraffe and Panda into front camera module connector in FCA pacifica. I am experiencing up to 10 seconds delays in reading CAN messages on my Ubuntu 16.04 machine and I suspect this could be due to 120 ohm terminal resistance missing in Panda (Giraffe documentation says it has this terminal resistance). Do I have to procure a 120ohm resistor and connect to make this work or Giraffe/Panda combination off the shelf should suffice? Any other suggestions to debug this will be helpful
thanks

No need for a 120 ohm resistor on my 2017 Pacifica nor any other FCA on the supported car list.
Ensure the car is on, the giraffe switches are correct, hardware is in all the way, linux udev is configured properly and you're using something like can_logger.py


Artemis plugin no longer works

Report from Alan Priester below:
I have been receiving complaints from TCIA users that the ultrasound images from the Prostate-MRI-US-Biopsy dataset no longer load properly into 3D Slicer. I downloaded the most recent stable release (4.11.2021.0226) and confirmed that ultrasound data no longer loads correctly, with or without the "SlicerHeart" and "Quantitative Reporting" extensions. The following warning is displayed during attempts to load ultrasound image data:
"115: US [633] [Image sequence]: Image spacing may need to be calibrated for accurate size measurements."
The following screen capture shows the (erroneous) result:

It would seem that a Slicer release between June 2020 and February 2021 compromised this functionality--can you please investigate and advise?

Maybe the confidence value of the Artemis reader needs to be set to a higher value. Can you send a download link to a sample data set?

I can't make time to investigate right now, but wanted to add an issue to make sure it is not lost right away as I received the email from the user. I haven't looked into it yet.

I can look into it if you can share a sample data set.

@lassoan I submitted a PR. The dataset is here: https://wiki.cancerimagingarchive.net/pages/viewpage.action?pageId=68550661.


Fix packages for ppc64 and GCC11

ping @shr-project, please cherry-pick these fixes into master-next when you get time

@kraj in master-next, thanks for patches and ping


Support for Swift 2.3 / Swift 3

Hi @emaloney!
I'd like to ask about the plan for updating this library (and its dependency CleanroomASL) to work with Swift 2.3 or Swift 3 respectively?
Are you going to explicitly update the underlying architecture to leverage the new logging mechanism introduced at WWDC 2016?

Hi @lukaskubanek,
As of now, both CleanroomASL and CleanroomLogger have a swift2.3 and swift3 branch. These branches will compile using the Xcode 8 betas.
Eventually, the CleanroomASL dependency will be split out so that it is no longer required for CleanroomLogger. You will still be able to use CleanroomASL as the underlying logging engine if you wish, but the base implementation of CleanroomLogger will be updated to use the new logging engine that Apple announced at WWDC.
E.

Thanks so much for the update and the clarification about the direction of further development!


Feature request: Numerical sorting

We use this plugin a lot for sorting yml files containing route tables and stuff.
The block sorting feature of this plugin is already really helpful, but it would be cool to add a numerical sorting feature to it.
For example instead of this:
- network: <IP_ADDRESS>
  gateway: <IP_ADDRESS>
  prefix: 24
- network: <IP_ADDRESS>
  gateway: <IP_ADDRESS>
  prefix: 24
- network: <IP_ADDRESS>
  gateway: <IP_ADDRESS>
  prefix: 24

Parse numbers and sort like this:
- network: <IP_ADDRESS>
  gateway: <IP_ADDRESS>
  prefix: 24
- network: <IP_ADDRESS>
  gateway: <IP_ADDRESS>
  prefix: 24
- network: <IP_ADDRESS>
  gateway: <IP_ADDRESS>
  prefix: 24

@freddo256 Can you try enabling "Natural Sorting"? This might already do what you want. It does add some overhead, which is why it is disabled by default, but shouldn't bee too bad.
Internally, this pads the numbers with leading zeros before sorting, as specified in the "Natural Sorting" options (default is up to length 9).


Hardcode program to default on both sides

https://github.com/e-mission/e-mission-docs/issues/788

Frontend:

Can you search for "prepilot" in this directory and make sure that there are no further references?
$ grep -r prepilot . | wc -l

currently (without merging your changes) gives me 35. You should get 0.

I have 1 in vscode from a notebook we are no longer using:

Using grep:

I have 11 mentions with your grep command but I believe they are all files in .Trash-0


Can you set a schedule programmatically?

This looks very interesting, but I can't find any info about setting a schedule from database programmatically rather than via the UI (in this case the user will be choosing classes so they should be filled in via the database and not by the user. I need the events to be recurring too, not on specific dates, just days of week.

setting a schedule from database programmatically
A. TOAST UI Calendar doesn't deal with data directly. It just renders the entered schedule to the UI.
You can fetch data(e.g. from database) on your project and call cal.createSchedule().

I need the events to be recurring.
A. You can create repetitive data and then put them into your calendar.


Gift Entry: Reset selected donations

Critical Changes
Changes

Include selected donations in form renderer reset

Issues Closed
Community Ideas Delivered
New Metadata
Deleted Metadata

@ptewson-sfdo @vmartinov-sforg  - Looks like this is also being fixed in Ken's PR https://github.com/SalesforceFoundation/NPSP/pull/5151/files. Should we go ahead and just close this one out and wait for that PR to be merged in?

@psadie-sfdo and @ptewson-sfdo With this PR merging in after we release GE this week, I think we should have a GitHub Issue created and associated with this PR. Is it possible to get that done before you merge the PR?

@melissabarber - I think we're actually closing this PR out without merging it. Ken has a solution out that encompasses the issue this PR is fixing. Thanks!


Async RejectedExecutionException

Original issue: https://github.com/particle-iot/spark-sdk-android/issues/50

Cloud SDK Async implementation occasionally throws RejectedExecutionException (once ThreadPoolExecutor limit is exceeded). This exception was also a culprit behind few crashes on Tinker app.
Few options put forward by @jensck:

Use our own ThreadPoolExecutor with a higher limit

Overload executeAsync() allowing developers to specify ThreadPoolExecutor limits.
Add global mutable state Async.setExecutor()

Once either option is implemented following has to be considered:

Check/catch RejectedExecutionException in all overloaded executeAsync() methods.
Execute executeAsync() call without throwing a ParticleCloudException (perhaps we should return a boolean response indicating wether task was added to a queue).

Closing this.  Async should be deprecated; the future is Kotlin coroutines + LiveData.


ci: Regenerate haskell-ci

(automatic PR by https://github.com/liskin/xmonad-maintenance)

We probably only want this when all warnings from #830 are fixed

Ok, keep it open then. The scheduled workflow will probably forcepush into it every week, let's see then how much noise it creates.

Hooray :tada:


Can't install a local package

Can't install a local package by providing the path to.
dependencies:
  flutter:
    sdk: flutter
  muqel_api: # <-- my local package
    path: ./api/muqel_api
  provider: ^4.3.3

the installation command does not yet support it

the installation command does not yet support it


Create storybook-theming-tailwind

The goal for the storybook-theming-tailwind package is to let people super easily create a Storybook theme from a given Tailwind config file.

See #7

https://storybook.js.org/docs/react/configure/theming#addons-and-theme-creation


[ERROR] Cannot resolve dependency 'paper'

I get this error
pose-animator/static_image.js:21:23: Cannot resolve dependency 'paper'

when starting the server:
Server running at http://localhost:1234
🚨  /Users/loretoparisi/Documents/MyProjects/pose-animator/static_image.js:21:23: Cannot resolve dependency 'paper'
  19 | import * as facemesh_module from '@tensorflow-models/facemesh';
  20 | import * as tf from '@tensorflow/tfjs';
> 21 | import * as paper from 'paper';
     |                       ^
  22 | import "babel-polyfill";
  23 |
  24 | import dat from 'dat.gui';

I have installed with yarn:
yarn install v1.15.2
warning package-lock.json found. Your project contains lock files generated by tools other than Yarn. It is advised not to mix package managers in order to avoid resolution inconsistencies caused by unsynchronized lock files. To clear this warning, remove package-lock.json.
[1/5] 🔍  Validating package.json...
[2/5] 🔍  Resolving packages...
[3/5] 🚚  Fetching packages...
[4/5] 🔗  Linking dependencies...
warning "@tensorflow/tfjs ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>[5/5] 🔨  Building fresh packages...
warning Your current version of Yarn is out of date. The latest version is "1.22.4", while you're on "1.15.2".
info To upgrade, run the following command:
$ brew upgrade yarn

See #12, #4, #2 for solutions.

@thllwg thank you I have solved following this
"paper": "^0.12.1",

and
"watch": "cross-env NODE_ENV=development parcel index.html --no-hmr --open --no-source-maps "

Then
yarn
yarn watch

I used the advice above, but having other issue then.


Create Mongoose schemas from UML Class Diagram

Translate the schemas shown in the class diagrams to a schema Mongoose can use.
Task List
The prerequisite issue will complete itself when the issue is closed

[x] #51
[x] Translate the schema from UML to Mongoose

The data structures have been created previously.


Azure Task Runner takes longer to run than expected (8 to 9 minutes for basic Python script)

Expected Behavior
Should take at most a few minutes to execute this example:
id: azure_batch_runner
namespace: company.team

variables:
  poolId: "poolId"
  containerName: "containerName"

tasks:
  - id: get_env_info
    type: io.kestra.plugin.scripts.python.Commands
    containerImage: ghcr.io/kestra-io/pydata:latest
    taskRunner:
      type: io.kestra.plugin.azure.runner.Batch
      account: "{{ secret('AZURE_ACCOUNT') }}"
      accessKey: "{{ secret('AZURE_ACCESS_KEY') }}"
      endpoint: "{{ secret('AZURE_ENDPOINT') }}"
      poolId: "{{ vars.poolId }}"
      blobStorage:
        containerName: "{{ vars.containerName }}"
        connectionString: "{{ secret('AZURE_CONNECTION_STRING') }}"
    commands:
      - python {{ workingDir }}/main.py
    namespaceFiles:
      enabled: true
    outputFiles:
      - "environment_info.json"
    inputFiles:
      main.py: |
        import platform
        import socket
        import sys
        import json
        from kestra import Kestra

        print("Hello from Azure Batch and kestra!")

        def print_environment_info():
            print(f"Host's network name: {platform.node()}")
            print(f"Python version: {platform.python_version()}")
            print(f"Platform information (instance type): {platform.platform()}")
            print(f"OS/Arch: {sys.platform}/{platform.machine()}")

            env_info = {
                "host": platform.node(),
                "platform": platform.platform(),
                "OS": sys.platform,
                "python_version": platform.python_version(),
            }
            Kestra.outputs(env_info)

            filename = 'environment_info.json'
            with open(filename, 'w') as json_file:
                json.dump(env_info, json_file, indent=4)

        if __name__ == '__main__':
          print_environment_info()

Actual Behaviour
Took 8 to 9 minutes for the two times I ran it.
Steps To Reproduce
No response
Environment Information

Kestra Version: 0.17.11
Plugin version:
Operating System (OS / Docker / Kubernetes):
Java Version (If not docker):

Example flow
No response

Should take at most a few minutes to execute this example

What are the rational behind that? Did you try to run it out of Kestra and it took less time?
Does the time reported in Azure console is a lot less than in Kestra?

The GCP and AWS task runners took half the time at most in comparison for the same simple example. I'll try and get some actual numbers next week.

So to try to understand what's going on we would need to compare it in 2 clouds, and for each have:

The time seen by Kestra
The logs
The time taken by the batch in the Cloud, including pending if any (for ex, in GCP you usually have 1mn to start the batch).
The resource allocated to the batch

The later may be the issue, maybe Azure allocate less resources by default.

This is the time taken by Azure before the Job is executed, as far as I know there is nothing we can do at our side so I'm closing it for now.


Cannot find module 'is-property'

Trying to run the server I get the following
Loading 6 accessories...
module.js:338
    throw err;
          ^
Error: Cannot find module 'is-property'
    at Function.Module._resolveFilename (module.js:336:15)
    at Function.Module._load (module.js:278:25)
    at Module.require (module.js:365:17)
    at require (module.js:384:17)
    at Object.<anonymous> (/home/pi/homebridge/node_modules/wemo/node_modules/request/node_modules/har-validator/node_modules/is-my-json-valid/node_modules/generate-object-property/index.js:1:80)
    at Module._compile (module.js:460:26)
    at Object.Module._extensions..js (module.js:478:10)
    at Module.load (module.js:355:32)
    at Function.Module._load (module.js:310:12)
    at Module.require (module.js:365:17)
    at require (module.js:384:17)

npm ERR! Linux 3.18.11-hypriotos-v7+
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "run" "start"
npm ERR! node v0.12.6
npm ERR! npm  v2.11.2
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>start: `node app.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script 'node app.js'.

Does anyone know how to fix this?

That's a weird one - guessing there's an issue with your config.json - could you post yours? (blanking out passwords first)

You could well be right! I did just get the latest version of homebridge while keeping the old config.json. Will investigate.

I think your npm install was not run correctly and is-property is just missing. Please check the contents of directory /home/pi/homebridge/node_modules/wemo/node_modules/request/node_modules/har-validator/node_modules/is-my-json-valid/node_modules/generate-object-property/node_modules…

Closing this for now; if you're still having this problem please reopen, thanks!


can't find lib file and include file after install the package

platform: ubuntu 16.04LTS
I try to install the realsense SDK2.0 in ubuntu 16.04LTS, and followed the guide:
https://github.com/IntelRealSense/librealsense/blob/master/doc/distribution_linux.md
and i have install the package by :
sudo apt-get install librealsense2-dkms sudo apt-get install librealsense2-utils sudo apt-get install librealsense2-dev sudo apt-get install librealsense2-dbg
and the guide said that :With dev package installed, you can compile an application with librealsense using g++ -std=c++11 filename.cpp -lrealsense2 or an IDE of your choice
I want use realsense in my Qt program, but i can't find lib path and include path that should be specified in Qt  or other IDE,
so, if you could give some suggestion that what should i do to use realsense in other IDE or where to find realsense lib, include, bin path in ubuntu.

@CrazyChildren, the installation paths for the SDK Debian packages are standard -
/usr/lib, /usr/include and usr/bin. Use dpkg -L librealsense2-dev for verification.
On the other hand, if the other project you're linking the SDK with uses CMake then to finding the SDK's library should be done with putting find_package(realsense2) into the Cmake.

@CrazyChildren, the installation paths for the SDK Debian packages are standard -
/usr/lib, /usr/include and usr/bin. Use dpkg -L librealsense2-dev for verification.
On the other hand, if the other project you're linking the SDK with uses CMake then to finding the SDK's library should be done with putting find_package(realsense2) into the Cmake.

thanks a lot!! the command really work!
actually library file(.so) located in /usr/lib/x86_64-linux-gnu/librealsense2.so and include path is /usr/include/librealsense2


dev: Div, DivEq, felt252_try_as_non_zero for felt252

Div, DivEq traits and felt252_try_as_non_zero helper function aren't implemented for felt252 in lib.cairo

Div isn't implemented for felt252 on purpose - to prevent users from performing division without fully understand the meaning of felt division.
Will add a Felt252TryIntoNonZero impl for the other part.

Got it. Felt252TryIntoNonZero was also done in attached to issue pr can just remove the other part

Div isn't implemented for felt252 on purpose - to prevent users from performing division without fully understand the meaning of felt division.

So in cairo 1 multiplication/division of felts are still calculated via modular arithmetic but other integers use regular n-bit division?

exactly.


[KYUUBI #1351] Implement api: /${version}/sessions/${identifier}/operations

Why are the changes needed?
This is a subtask of umbrella issue #KPIP-1

/${version}/sessions/${identifier}/operations

mapping:

ICLIService#executeStatement
ICLIService#executeStatementAsync
ICLIService#getTypeInfo
ICLIService#getCatelogs
ICLIService#getSchemas
ICLIService#getTables
ICLIService#getTableTypes
ICLIService#getColumns
ICLIService#getFunctions

desc: create a new operation under a session
method: POST
params:
operation: e.g. GetTableTypes
addition: required params to create an Operation
returns: an instance of OperationHandle

How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

Codecov Report

Merging #1353 (aeb0de6) into master (395c632) will decrease coverage by 0.12%.
The diff coverage is 75.25%.

:exclamation: Current head aeb0de6 differs from pull request most recent head 006ce4d. Consider uploading reports for the commit 006ce4d to get more accurate results

@@             Coverage Diff              @@
##             master    #1353      +/-   ##
============================================
- Coverage     78.75%   78.63%   -0.13%
- Complexity       71      110      +39
============================================
  Files           173      179       +6
  Lines          6416     7545    +1129
  Branches        760      928     +168
============================================
+ Hits           5053     5933     +880
- Misses          921     1119     +198
- Partials        442      493      +51

Impacted Files
Coverage Δ

...g/apache/kyuubi/sql/KyuubiEnsureRequirements.scala
30.76% <ø> (ø)

...pache/kyuubi/sql/KyuubiQueryStagePreparation.scala
32.35% <ø> (ø)

...pache/kyuubi/sql/KyuubiSQLExtensionException.scala
100.00% <ø> (ø)

...ache/kyuubi/sql/RepartitionBeforeWritingBase.scala
0.00% <0.00%> (ø)

...he/kyuubi/engine/spark/shim/SparkCatalogShim.scala
78.26% <0.00%> (+1.17%)
:arrow_up:

...ain/scala/org/apache/kyuubi/KyuubiSparkUtils.scala
0.00% <ø> (ø)

...ala/org/apache/kyuubi/events/EventLoggerType.scala
0.00% <0.00%> (ø)

...ala/org/apache/kyuubi/events/JsonEventLogger.scala
0.00% <0.00%> (ø)

...org/apache/kyuubi/operation/OperationManager.scala
34.32% <0.00%> (-39.87%)
:arrow_down:

...la/org/apache/kyuubi/session/AbstractSession.scala
95.40% <0.00%> (ø)

... and 107 more

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 395c632...006ce4d. Read the comment docs.

cc @yanghua @pan3793 @yaooqinn

/${version}/sessions/${identifier}/operations
desc: get all the operation list hosted a specific session binding via an identifier
method: GET
param:identifier instance of SessionHandle
returns: a list of Operation

/${version}/sessions/${identifier}/operations desc: get all the operation list hosted a specific session binding via an identifier method: GET param:identifier instance of SessionHandle returns: a list of Operation

This PR seems not correct, according to the KPIP-1.
And this intention of this API seems to expose opHandleSet in a session, which is actually a private member? What's the motivation of this?

Not this one.

/${version}/sessions/${identifier}/operations desc: get all the operation list hosted a specific session binding via an identifier method: GET param:identifier instance of SessionHandle returns: a list of Operation

/${version}/sessions/${identifier}/operations

mapping:

ICLIService#executeStatement
ICLIService#executeStatementAsync
ICLIService#getTypeInfo
ICLIService#getCatelogs
ICLIService#getSchemas
ICLIService#getTables
ICLIService#getTableTypes
ICLIService#getColumns
ICLIService#getFunctions

desc: create a new operation under a session
method: POST
params:
operation: e.g. GetTableTypes
addition: required params to create an Operation
returns: an instance of OperationHandle

backendService

Not this one.

/${version}/sessions/${identifier}/operations desc: get all the operation list hosted a specific session binding via an identifier method: GET param:identifier instance of SessionHandle returns: a list of Operation

/${version}/sessions/${identifier}/operations

mapping:

ICLIService#executeStatement
ICLIService#executeStatementAsync
ICLIService#getTypeInfo
ICLIService#getCatelogs
ICLIService#getSchemas
ICLIService#getTables
ICLIService#getTableTypes
ICLIService#getColumns
ICLIService#getFunctions

desc: create a new operation under a session

method: POST

params:

operation: e.g. GetTableTypes

addition: required params to create an Operation

returns: an instance of OperationHandle

we shall implement separately, like the def executeStatement where we call backendService.executeStatement

for executeStatement, do we need to get the actual final result？with FetchResult API?

executeStatement

Do you mean one API corresponds to one method?
like   /${version}/sessions/${identifier}/executeStatement -> ICLIService#executeStatement ?

for executeStatement, do we need to get the actual final result？with FetchResult API?

Yes, we need FetchResults API to get the actual final result.

executeStatement

Do you mean one API corresponds to one method? like /${version}/sessions/${identifier}/executeStatement -> ICLIService#executeStatement ?

yes

cc @yaooqinn @yanghua

for executeStatement, do we need to get the actual final result？with FetchResult API?

Yes, we need FetchResults API to get the actual final result.

IMO, fetch API should be GET, while Session#getCatalogs should be a post request to create an operation and return an operation handle.
WDYT? @simon824 @yaooqinn

for executeStatement, do we need to get the actual final result？with FetchResult API?

Yes, we need FetchResults API to get the actual final result.

IMO, fetch API should be GET, while Session#getCatalogs should be a post request to create an operation and return an operation handle.
WDYT? @simon824 @yaooqinn

Make sense.

This API is actually to create different type of OperationHandle. From this perspective, I think it’s better to implement all the methods in the same API like we designed before. WDYT @yaooqinn

This API is actually to create different type of OperationHandle?

which API?

This API is actually to create different type of OperationHandle?

which API?

/${version}/sessions/${identifier}/operations

mapping:

ICLIService#executeStatement
ICLIService#executeStatementAsync
ICLIService#getTypeInfo
ICLIService#getCatelogs
ICLIService#getSchemas
ICLIService#getTables
ICLIService#getTableTypes
ICLIService#getColumns
ICLIService#getFunctions

desc: create a new operation under a session
method: POST

Take GetCatalogs as an example.
REST Client POST GetCatalogsReq(sessionHandle) -> REST FE -> call -> Kyuubi BE getCatalogs -> GetCatalogsResp(operationHandle, OK/ERROR) -> REST Client
REST Client GETFetchResultsReq(operationHandle, orientation, maxRows, fetchType(log/result)) -> REST FE -> call -> Kyuubi BE fetchResults -> FetchResultsResp(OK/ERROR, hasMoreRows, resultset) -> REST Client -> if hasMoreRows -> repeat
FYI, https://prestodb.io/docs/current/develop/client-protocol.html

Take GetCatalogs as an example.
REST Client POST GetCatalogsReq(sessionHandle) -> REST FE -> call -> Kyuubi BE getCatalogs -> GetCatalogsResp(operationHandle, OK/ERROR) -> REST Client
REST Client GETFetchResultsReq(operationHandle, orientation, maxRows, fetchType(log/result)) -> REST FE -> call -> Kyuubi BE fetchResults -> FetchResultsResp(OK/ERROR, hasMoreRows, resultset) -> REST Client -> if hasMoreRows -> repeat
FYI, https://prestodb.io/docs/current/develop/client-protocol.html

+1

thank you @simon824 @yanghua for the work !
merging to master. FYI @yaooqinn


Fix $grid-column-count beeing overwritten by grid-context-mixin

Commit fixes issue described in issue: https://github.com/zurb/foundation-sites/issues/8866

@kball You okay with this PR? @brettsmason and I both looked over #8866 and think this it the right fix.

Sounds good, lets do it.


emit() clashes with Qt's emit()

Hi to you all.
I am using socket.io within a Qt environment. But the emit() function of socket.io gives an error because Qt is trying to execute it's own emit() function.
Is there some resolve to this?
Thank you!
Fastjack

Thanks xMuratY.
I have replaced all instances of emit() with another name and that is working.
Maybe the emit function could be renamed in the source code?


Wrong error description for 1304

The error description above for 1305 belongs to 1304 as seen in the output from infverif
Output from C:\Program Files (x86)\Windows Kits\10\Tools\x64\infverif.exe
WARNING(1304) in C:\Dev\UMC\VSC\Debug\x64\MyIDVscDriver_Package\MyIDVscDriver.inf, line 98, target OS NTamd64.6.1: Found legacy AddReg operation using non-relative key (HKLM\SOFTWARE\Wow6432Node\Microsoft\Cryptography\Calais\SmartCards\Intercede Software Virtual Card).

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

ID: 48275f87-4086-1ca5-a190-c741f29ad348
Version Independent ID: 5481885d-3063-527b-a85e-6a898dbcb990
Content: INF Validation Errors and Warnings - Windows drivers
Content Source: windows-driver-docs-pr/devtest/inf-validation-errors-and-warnings.md
Product: windows-hardware
GitHub Login: @nabazan-microsoft
Microsoft Alias: windowsdriverdev

@tedhudek, can you verify this?

Updated the 130x values, which were all one higher than they should have been.  Thanks @gerikintercede


Update celeryd for taskflow

When Kitware/cumulus#134 is merged we will need to update celeryd. We need to make sure that we set up some threads dedicated to the monitor queue etc. Plus take a look separate logs for each queue?

@kotfic I created this to keep track ...


949110 triggered when trying to Upload images to Wordpress

I enabled this ruleset on CloudFlare, left it at the default setting and was unable to upload images to Wordpress for a day and a half. It was a very savvy Cloudflare support specialist who sent me the email below. Disabling OWASP altogether fixed the problem immediately. Here's the screenshot that started it all
https://www.dropbox.com/s/tm6tpfp959vbu7l/Screenshot 2022-05-18 163404.png?dl=0
I sent CF a .HAR file output from that page and below was their response, along with a screenshot of how they found the solution.

Hello,
Thank you for providing the HAR file.
I investigated this issue using the Ray ID associated with the 403 error in the HAR file.
Ray ID: 70d65a881e3c2d40
In the Security Overview section in your Cloudflare Dashboard I searched the Ray ID, and I see that the 403 error is due to the Cloudflare OWASP Core Ruleset. You can see in the image below that the request triggered the Cloudflare OWASP Core Ruleset, which blocked the request, resulting in a 403 error.
You can also view this information by going to your Cloudflare Dashboard > Security > Overview, and then adding a filter and inputting the Ray ID.
The Cloudflare WAF contains mainly 2 packages:
Cloudflare Managed Ruleset: These rules are managed by Cloudflare WAF Engineers. For "security reasons", we don't provide the rule patterns as this would increase the likelihood that a malicious party could learn to bypass the rules.
OWASP ModSecurity Core Rule Set: These rules are not managed by Cloudflare. They are created by the OWASP Group and Cloudflare integrates with this OWASP package as part of our WAF for additional security. If you would like to know why an OWASP rule has triggered, you can review the rules (expressions and sensitivity score) in the GitHub repository in this link.
If you're encountering false positives, there are a couple actions that you could take here:
•	Add WAF Exception
You can define WAF exceptions in the Cloudflare dashboard or using the Rulesets API.
•	If the rule blocking is 949110 (new OWASP), it means it was blocked by the OWASP rules. In your particular case the rule that blocked your request is 949110. I recommend decreasing the OWASP Anomaly Score Threshold or lower the OWASP Paranoia Level.
Resources:
False Positives
Package: OWASP ModSecurity Core Rule Set
Please let me know if you have any questions.
Kind regards,
Paige Kammeyer
Technical Support Engineer -Cloudflare

Screenshot detailing how they traced it
https://www.dropbox.com/s/k3r9crld4oavjt4/details.png?dl=0
Meanwhile, I think I aged about 5 years. Never would I have checked OWASP for this issue

Transferred issue here.

@jeromydarling Sorry for the inconvenience caused to you by our rule set.
 So this was not made transparent to you?
The rule 949110 is the blocking rule. OWASP CRS uses detection rules and blocking rules. The blocking rules act after a detection rule has triggered. So dealing with 949110 is like shooting the messenger. The root problem is the detection rule and you did not share that one.
On top, we expect false positives with OWASP CRS. That's why we provide a rule exclusion package for Wordpress. When running wordpress you are supposed to activate that package. I have no idea whether Cloudflare does that for you or if it is available as an option. But I do know that you are a paying customer so I think it is their job to provide you with the means to work around this.
What we can do to help you is relatively limited, since we think your false positive is probably already covered by the rule exclusion package we provide. If you or Cloudflare dig into this and find out that's not the case, then we're happy to accomodate with a new rule exclusion in a future release.

Thank-you
On Mon, May 30, 2022, 8:47 AM Christian Folini @.***>
wrote:

We are currently moving the rule exclusion packages into plugin, from an
architecture perspective, that's wildly different in terms of activation
etc. You are running on cloudflare and that is a prior version, so the best
documentation is right in our old config file
https://github.com/coreruleset/coreruleset/blob/v3.3/master/crs-setup.conf.example#L321
What you should do now is taking this link to your paid cloudflare support
and ask them about this and how you can get it activated.
—
Reply to this email directly, view it on GitHub
https://github.com/coreruleset/wordpress-rule-exclusions-plugin/issues/4#issuecomment-1141180827,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAASA3WWUB2WHD3HBXJIMW3VMTBGNANCNFSM5XGTVAFA
.
You are receiving this because you were mentioned.Message ID:
@.***
com>

You are welcome. Closing this issue now. Feel free to re-open if you have more information.


What you have accomplished

Leave a comment answering -> What have you done with your skills so far?
Please also leave your twitter username in your comment for applaud

As a ruby engineer for a minute, i was able to grapple alot of skills from the experienced engineers that i was able to work with, and it turn i was able to build a gem(library) that consumes one of the biggest mobile payment platform in Kenya and across the continent.

mpesa_stk which has gotten a little bit of traction based on the downloads that it has received.

Intention was to make the developer who uses/needs this resource to not have to think about the complexities of the service, but have the minimalist time in getting hooked up to it.
its fully open source are you all welcomed to contribute to it...
catch me on twitter for more into the setup @mboyaberry


Markdown preview is not working.

Describe the bug
Markdown preview is not working.
Preview windows is showing json data.
And cannot update notion page.
To Reproduce
Steps to reproduce the behavior:

Execute ':Notion menu'
Select notion's item.
Typing 'e'
Add text like '# abcdefg'
Execute ':w' for updating notion's page.

Expected behavior

Show markdown preview.
Enable update notion's page.

Screenshots

Desktop (please complete the following information):

OS: MacOS sonoma
Neovim: HEAD(brew install neovim --HEAD)

Not able to reproduce, works as expected on my side?

(Changed the - to ---), needed no reload or whatever
Mind providing details?

If you mean not being able to edit the markdown page, that is normal (for now), only editing the raw json is fully supported. Working on the markdown part, but not for soon at all.

Sorry, I was mistaken in thinking that markdown could be edited.
I look forward to the markdown editing feature.
Thanks for your response! 👍

Just to be sure I am reporting this, I was able to save the Notion page by editing the JSON directly and saving it.
However, an error occurs in the save process callback.
This may be the same error you fixed before.
013ce6ea-20b9-4743-a4e7-cdf91500e47a
vim.schedule lua error occured by callback: ...ets/.config/nvim/plugged/notion.nvim/lua/notion/init.lua:22: bad argument #1 to 'write' (string ex
pected, got nil)
stack traceback:
        [C]: in function 'write'
        ...ets/.config/nvim/plugged/notion.nvim/lua/notion/init.lua:22: in function 'writeFile'
        ...ts/.config/nvim/plugged/notion.nvim/lua/notion/parse.lua:34: in function 'updateMarkdown'
        ...ts/.config/nvim/plugged/notion.nvim/lua/notion/parse.lua:52: in function <...ts/.config/nvim/plugged/notion.nvim/lua/notion/parse.lua:52>

Aware of it, updateMarkdown code commented out in commit b769c76, not a fix but no time to dig into it today.
PackerSync should remove the error, and #16 is a note for myself to fix that


remove withdraw

https://github.com/lnbits/withdraw

Codecov Report

Merging #1517 (2cc1915) into main (dd14d2d) will increase coverage by 0.06%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main    #1517      +/-   ##
==========================================
+ Coverage   49.83%   49.89%   +0.06%
==========================================
  Files         147      140       -7
  Lines       11036    10688     -348
==========================================
- Hits         5500     5333     -167
+ Misses       5536     5355     -181

Impacted Files
Coverage Δ

lnbits/extensions/lnurlp/migrations.py
81.48% <0.00%> (-14.82%)
:arrow_down:

lnbits/db.py
81.20% <0.00%> (-10.07%)
:arrow_down:

lnbits/commands.py
61.40% <0.00%> (-1.76%)
:arrow_down:

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


Update RocksDB to 5.1.2

Motivation for this change
RocksDB 5.1.2 is currently the most up-to-date version.
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

Please use a commit message according to Contribution.md, eg. rocksdb: 4.13 -> 5.1.2.

We should also test if haskellPackages.rocksdb compiles

I've verified this PR and built haskellPackages.rocksdb-haskell just fine.

Merged in 85fba54bc65ffcf93a2dbed2ececd97af1593ae9 where I intended to fix the commit message, but failed to update the message before pushing :( sorry.


install cross compilation VBCC toolchain for linux host

With the included script, one can create a local Linux host VBCC cross-compilation environment. Necessary components are fetched from VBCC official download website, applied patch is also done during installation. Tool-chain is compiled and locate in /opt/vbcc. Remaining changed are VBCC and PATH variables that has to be manually added to the environment.

As described on Discord, this script isn't working properly and can give weird problems with the resulting binaries. So don't use it for now.


fix(anchor-link): fixes extra :before on anchor links

Closes https://github.com/carbon-design-system/carbon-website/issues/902
Refs https://github.com/carbon-design-system/carbon-website/pull/893
This change added an extra before element on page anchor links. This removes them

Looks good

Bullets look good! The alignment for anchor link bullets should not hang though.
This is current:

Should be:

Updated!


Possibility to run R script with breakpoints

VSCode Version: Version: 1.27.2
VSCode-R Version:  0.6.1
OS Version:  Ubuntu 16.04 LTS

Steps to Reproduce:
None
Do you want to fix by self? (I hope your help!)
No
(If yes,) what kind of help do you want? (e.g. Which file should I fix, Survey (related documents)
This is not bug report. I that possible to run R in debug mode with breakpoints?
If not could you be so kind to add this feature?
(If related)setting.json
// R.exe path for windows
"r.rterm.windows": "C:\\Program Files\\R\\R-3.4.4\\bin\\x64\\R.exe",

// R path for Mac OS X
"r.rterm.mac": "/usr/local/bin/R",

// R path for Linux
"r.rterm.linux": "/usr/bin/R",

// R command line options (i.e: --vanilla)
"r.rterm.option": [],

// An optional encoding to pass to R when executing the file, i.e. 'source(FILE, encoding=ENCODING)'
"r.source.encoding": "UTF-8",

// Keeping focus when running
"r.source.focus": "editor",

Thanks for attention to my message and have a nice day.

The extension doesn't give you the full debugging capabilities built into VS Code, but you can always add a browser() call to your script and you will be R's debug mode in the interactive terminal when you reach that point in the script.

@jacob-long
Thanks for answering. I'm just interesting about that. Have a nice day.

Would be really interested in debugging functionality for R. It is listed as a TODO, is there any update on this? Thank you!

I'm digging into this topic using the file-based approach suggested in #143. If we can inject hook functions in browser mode so that each action (such as continue, step in, etc.) can trigger the hook function, we can write out session and breakpoint information to debugger through DAP. So far I have not found any easy way to do that. Maybe we have to look at how RStudio does it and it is best that we don't have to hack too much into R internals.

I believe RStudio uses a client/server approach, with the R session running in the server, so they have more access to the session. It looks like they directly inject a trace into the code:
https://github.com/rstudio/rstudio/blob/29ed81ba590beadc30c54ccfc75b7d60a9f1e6d5/src/cpp/session/modules/SessionBreakpoints.R#L291

@andycraig, yes, utils::setBreakPoint does similar things, we can set a break point where the trace function can be specified to write out the code location before calling browser() so that the vscode debugger could caputure it. But so far I haven't found any callbacks if user hit next step, step into, and continue. If we could make these actions trigger some callback, we can also write out the code location to be caputured outside by debugger. I'm doing some experiment and see if we can caputure the code location change in browser mode.

In VSCode, debugging runs in a fresh session on each start debugging. We only have to translate debugger action to stdin and capture stdout and stderr from the R process. The code location is clearly printed out on each code location change.

For debugging in user code in vscode, I believe it is mainly about implementing an adapter that communicates with R through std in/out/err. However, there are some tricky parts:

If user chooses to step into R package code, the R code does not seem to have a file-based location.
To extract values of variables at break point, we can send ls.str() on each break which will force evaluation of promises. We may use pryr::promise_info() to avoid force evaluation.

@renkun-ken This is sounding promising!
When the user sets a breakpoint in VSCode, is the idea that utils::setBreakPoint or trace would be sent to the R console to get it into the R session?

@jacob-long
In your reply you stated:

The extension doesn't give you the full debugging capabilities built into VS Code, but you can always add a browser() call to your script and you will be R's debug mode in the interactive terminal when you reach that point in the script.

Could you explain in more detail how do to that in a minimal example?
I'd naively imagine the following (consider some example R-code):
...
bla <- list(1,2,3)
...

At first, set a breakpoint at the line containing "bla <- list(1,2,3)" via left-click on the left-hand side of the editor.
Secondly, insert the command "browser()" as a newline before or after that line in question, e.g.:
...
browser()
bla <- list(1,2,3)
...

Finally, start the debugging session via F5 of the currently opened file?
But even then, the question would be with which configuration within the launch.json - file I should start this debugging session? I cannot just use my python-debugger etc.
Thanks in advance for clarifying this with more details.

@AndyLuckert we haven't yet implemented a debugging server that is compliant with Debug Adapter Protocol to work with R and VSCode. That's why you can't find a debugger in VSCode to launch an R session to debug a script.
When you hit F5 you enter the VSCode deubgging session only when a debugger that implemented DAP for R is provided. Unfortunately, we don't have one at the moment. And we are trying to figure out a way to implement one in the future.
R's native debugger works in pure command line. When you call browser() in any part of your code, the interpreter enters a browser mode in which you can inspect the current environment using R functions such as ls(), ls.str(), and you can also navigate the debugger using some specialized commands such as next statement, step into, etc. Please see https://stat.ethz.ch/R-manual/R-devel/library/base/html/browser.html for more details.
The debugger we are trying to implement to work with R and VSCode is probably through such an interface.

That sounds good, that you're planning/working on the future implementation of a proper R-debugger!
I've just tried to implement this browser()-approach in an R-script I'd like to debug:
# Load MixSIAR package
library(MixSIAR)

# To run on your data, replace the system.file call with the path to your file
mix.filename <- system.file("extdata", "wolves_consumer.csv", package = "MixSIAR")

# Load mixture data
mix <- load_mix_data(filename=mix.filename,
                     iso_names=c("d13C", "d15N"),
                     factors=c("Region", "Pack"),
                     fac_random=c(TRUE, TRUE),
                     fac_nested=c(FALSE, TRUE),
                     cont_effects=NULL)

browser() # opens debugging session
...
more code

As you can see, I've inserted "browser()" into the script at a specific code-line.
Next, I ran it in my bash-terminal via the following command:
Rscript /usr/local/lib/R/site-library/MixSIAR/example_scripts/mixsiar_script_wolves.R
Unfortunately, nothing happened. The script was just running through without stopping at the point where I put browser, nor was there any reaction regarding this additional code-line whatsoever.
What I aim for is putting kind of a break-point within an R-script, as I'm used to do that with python, and from that breakpoint on go through the code step by step, with the possibility of stepping into functions.
If this is possible with "browser()", I'd like to learn how.
Thanks again in advance.

@AndyLuckert  Rscript starts an non-interactive R session where browser() does not interrupt the session to enter the debugging mode. You may try the following:

Start an interactive R session using R, and you can either send code line by line to R terminal or source("your-code.R"). When your code hit a browser(), it will enter the debugging mode for you and look around. You shouldn't send all code to R terminal manually since when the R session is interrupted by browser(), the code following is still sent to terminal to mess up.
Or start a radian console which has better handling of browser() interruption. You can send line by line, source the file, all send all lines, all work fine.

Alright, thanks! I'll try it out when I get to it.

I've been working on an R debugger recently, following the same approach as @renkun-ken described. Since this approach is quite different from the current R extension, I decided to do so in a new project. If you have the time to try the debugger, I would appreciate any feedback :)
https://github.com/ManuelHentschel/VSCode-R-Debugger
https://github.com/ManuelHentschel/vscodeRPackage

@ManuelHentschel Thanks for sharing your work on R debugger!
I tried it but does not manage to make it work on macOS with the following error:

Am I missing something?

In this step the extension tries to start a terminal/shell process in which to run the R session.
The path to the terminal is retrieved from the setting "terminal.external.osxExec" and displayed as "terminalPath: ..." in the debug console (in the collapsed content of the "Starting R Session..." entry).
I sometimes needed to restart vscode for changes to this setting to take effect.
If the correct path is used, maybe try a different shell?
So far I have got this to work with bash on linux and cmd.exe on windows, but had no opportunity yet to test it on a mac.

@ManuelHentschel Looks like the default setting
"terminal.external.linuxExec": "x-terminal-emulator"

is not a valid path for you to run. I change it to /bin/bash and the debugger now works under Ubuntu.
Great work! It's quite exciting to see what can be done to make both extensions work well with each other.

Thanks a lot for figuring that out!
I guess I'll put these paths into their own config entries and try to make the "Terminal path not valid!" message more specific :)

@ManuelHentschel Thanks for your nice work in the R debugger for VSCode projects. It looks like the R code in vscDebugger could be largely improved. Would you mind if I participate in your debugger projects, give suggestions and contribute code?

@renkun-ken Sure, I would gladly appreciate any help :)
I just pushed some significant changes to the development branch of the R package, so be sure to pull those before you start coding yourself.

Closing as there's [VSCode-R-Debugger](

).


Update latest build selection and log when one build fails

Based on the discussion in the previous PR
https://github.com/apache/incubator-mxnet-ci/pull/25#discussion_r427626182
Build selection from the latest day logic is now set to span 24hours instead of 8.
This ensures regardless of what time the Lambda is triggered / Jenkins pipeline is triggered, all the builds in the last 24 hours will be considered.
Another corner case is handled in this PR
When the initial 2 stages of the mxnet-release-cd pipeline fail, it doesn't trigger the subsequent 2 stages ['mxnet_lib/static', 'python/pypi']
This doesn't get logged as a result it doesn't get captured by the Alarm.
Addressing it here.

@marcoabreu @mseth10 please review. Thanks

When the initial 2 stages of the mxnet-release-cd pipeline fail, it doesn't trigger the subsequent 2 stages ['mxnet_lib/static', 'python/pypi']

@ChaiBapchya More precisely, 'mxnet_lib/static' is followed by 'python/pypi'. As it currently stands, 'mxnet_lib/static' is failing for all variants and 'python/pypi' is not running at all. So, we should have failure logs for both builds. Our current logic will log only one of them.
We should modify the logic to check for both builds separately, log success only if the build exists and is in success state, else log failure. Does that make sense?

Verified this works

Lambda handler invoked
Secrets retrieved
Jenkins obj created
Job fetch mxnet-cd-release-job
latest builds [<jenkinsapi.build.Build restricted-mxnet-cd » mxnet-cd-release-job #1193>, <jenkinsapi.build.Build restricted-mxnet-cd » mxnet-cd-release-job #1192>, <jenkinsapi.build.Build restricted-mxnet-cd » mxnet-cd-release-job #1191>, <jenkinsapi.build.Build restricted-mxnet-cd » mxnet-cd-release-job #1190>, <jenkinsapi.build.Build restricted-mxnet-cd » mxnet-cd-release-job #1189>]
Builds filtered by desired release job type : [<jenkinsapi.build.Build restricted-mxnet-cd » mxnet-cd-release-job #1192>, <jenkinsapi.build.Build restricted-mxnet-cd » mxnet-cd-release-job #1190>]
Filtered builds by hudson.model.Cause$UpstreamCause : [<jenkinsapi.build.Build restricted-mxnet-cd » mxnet-cd-release-job #1192>, <jenkinsapi.build.Build restricted-mxnet-cd » mxnet-cd-release-job #1190>]
Failure build python/pypi 1192
Failure build mxnet_lib/static 1190
dict {'mxnet_lib/static': 1, 'python/pypi': 1}
All the required jobs ran

@leezu @marcoabreu  can you help with review/merge
Thanks.


Problem with downloading big files.

If you run the example from the repository and try to download a file of more than 2 MB, then we get an error in the console and a broken file.
Use ServeFile() method
ERROR: Failed to send delimiter chunk: http: wrote more than the declared Content-Length

This example works fine for me. Would you provide more information?


⚠ Beamable Announcements (Object) has degraded performance

In cf5eedc, Beamable Announcements (Object) (https://api.beamable.com/object/announcements/platform/docs) experienced degraded performance:

HTTP code: 503
Response time: 12793 ms

Resolved: Beamable Announcements (Object) performance has improved in 17be315 after 11 minutes.