487 option to delete feedbacks

Description
Added possibility to delete feedback in the feedbacks list view
Related Issue
N/A
Type of Change

[ ] 📚 Examples / docs / tutorials / dependencies update
[ ] 🔧 Bug fix (non-breaking change which fixes an issue)
[x] 🥂 Improvement (non-breaking change which improves an existing feature)
[ ] 🚀 New feature (non-breaking change which adds functionality)
[ ] 💥 Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🔐 Security fix

Checklist

[x] I've read the CODE_OF_CONDUCT.md document.
[x] I've read the CONTRIBUTING.md guide.
[ ] I've updated the code style using make codestyle.
[ ] I've written tests for all new methods and classes that I created.
[ ] I've written the docstring in Google format for all the methods and classes that I used.

@andreybavt Took a look at vue-final-modal and it was pretty straightforward to use. We would require a bit of refactoring to migrate from previous modal to new but can be done progressively as the migration to the composition API.

Regarding the $dialog option, I quickly discussed it with Andrey earlier and there's 2 other solutions we talked about:

Vue 3 has provide/inject that would allow us to keep using it, but I couldn't find how to use it in Vue 2.7 so maybe explore if that's possible
Provide $dialog at the window level so it's available everywhere

I was going to explore this during the week but if you want to have a look that's also fine by me. Let me know !

Regarding the $dialog option, I quickly discussed it with Andrey earlier and there's 2 other solutions we talked about:

Vue 3 has provide/inject that would allow us to keep using it, but I couldn't find how to use it in Vue 2.7 so maybe explore if that's possible
Provide $dialog at the window level so it's available everywhere

I was going to explore this during the week but if you want to have a look that's also fine by me. Let me know !

If it's possible to inject the dialog reference it would be very int

@kevinmessiaen , did you have a chance to test Vue Final Modal's "dynamic modals"? Does it allow to await until the modal is closed or we'd have to define callbacks?

@andreybavt I used callbacks:
$vfm.show({
    component: ConfirmModal,
    bind: {
      title: 'Delete feedback',
      text: `Are you sure that you want to delete the feedback for feature '${feedback.featureName}' permanently?`,
      isWarning: true
    },
    on: {
      async confirm(close) {
        await api.deleteFeedback(feedback.id);
        await fetchFeedbacks();
        close();
      }
    }
  });

Regarding await it looks like it have to be implemented in the future: https://github.com/vue-final/vue-final-modal/issues/193
However we can take avantage of composition API and make our utility:
export function useModal(component) {
   return async (bind) => {
      return new promise((resolve) => {
        $vfm.show({
          component,
          bind,
          on: {
            result: (result: DialogResult<RESULT>) => {
  	resolve(result);
        },
        closed() {
  	resolve({ action: "close" });
        }
          }
        });
      }
   }
}

Great, I just noticed that you've added vfm to this PR already 👍


Sumbit to Redex maybe?

Looks like https://redex.github.io has grown to automatically search non-submitted packages:

but I'm guessing that since reason-elm isn't officially submitted it's greyed out. Also, I wonder why it's labelled as "neglected"?

This is a cool tool, I haven't heard of it before. It looks like it was marked as "neglected" because the package.json didn't fit the necessary requirements here: https://redex.github.io/publish.
I've updated package.json to include everything the specify, and submitted a PR adding bs-elm to their list of published packages here: https://github.com/redex/data/pull/5.
This PR will be closed once they accept.

bs-elm is on redex here: https://redex.github.io/package/bs-elm. Also, it's no longer labeled as "neglacted".
Thank you for pointing this out to me!

👍 👏 👏👏👏👏


Added changes to discover ConfigCenter and Monitor endpoint from service center

Coverage increased (+0.04%) to 62.576% when pulling 9fdc397cb8667abc2bdb344c79332112ff9623cb on savitaashture:endpointDiscovery into f8b2ddde51f31e248b32ac81094645a89c6d1e2a on ServiceComb:master.


NWC (Nostr Wallet Connect) suddenly does not work

https://github.com/MutinyWallet/mutiny-web/assets/63401712/b250d4cc-7000-4c73-a859-22341042ddbe
NWS is previously working in Mutiny Wallet version 0.6.6, But it stops working lately at 0.6.7.
I've tried the following:

Relogin
Recreated wallet connection
Using another client to use NWC

None of them working.
For log, This was emitted everytime:
a0c2 2024-04-21 12:21:31.999 TRACE [lightning_background_processor:693] Calling OnionMessageHandler's timer_tick_occurred
a0c2 2024-04-21 12:21:32.470 TRACE [mutiny_core::federation:380] Getting activity
a0c2 2024-04-21 12:21:32.546 TRACE [mutiny_core::federation:429] About to go through 0 operations
a0c2 2024-04-21 12:21:32.546 TRACE [mutiny_core::federation:451] Going through 0 pending invoices to extract status
a0c2 2024-04-21 12:21:34.094 TRACE [mutiny_core::federation:380] Getting activity
a0c2 2024-04-21 12:21:34.130 TRACE [mutiny_core::federation:429] About to go through 0 operations
a0c2 2024-04-21 12:21:34.130 TRACE [mutiny_core::federation:451] Going through 0 pending invoices to extract status

Can you submit your whole log file

Your log file only has like 5 minutes of activity? This isn't useful, did you strip it all out?
Make sure your NWC string is in amethyst correctly matches what your current wallet connection string is in mutiny. Amethyst has a habit of not persisting settings.

Good day.

Your log file only has like 5 minutes of activity? This isn't useful, did you strip it all out?

Yes. However i've checked the log before i started trimming out.

Make sure your NWC string is in amethyst correctly matches what your current wallet connection string is in mutiny. Amethyst has a habit of not persisting settings.

Yes. Already ensured that the configuration matches to the one that mutiny provided.
Not only i could not zap with Amethyst, but i could not do with Primal too.

Closing as this seems having to do with Amethyst itself being not persistent on configuration. Thanks for mentioning it. Strange that it just worked recently both on Amethyst and Primal.

https://github.com/vitorpamplona/amethyst/issues/580


Test Ubuntu 16.04 and 18.04

In this list https://docs.microsoft.com/en-us/azure/virtual-machines/linux/premium-storage#premium-storage-for-linux-vms we don't have current LTS Ubuntu images like 16.04 and 18.04
As a results, guide looks out-of-date for 4 years. If you will test/add mentioned images- it will be great.

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

ID: 5dec11a9-8b64-aff1-78ae-f739b5f26cf2
Version Independent ID: 163def32-3f42-df60-ea8b-e0a163a22fe2
Content: High-performance Premium Storage and Azure managed disks for Linux VMs
Content Source: articles/virtual-machines/linux/premium-storage.md
Service: virtual-machines-linux
GitHub Login: @ramankumarlive
Microsoft Alias: ramankum

Thanks for the feedback! It should work with newer versions also. We will check and update you shortly.

@SychevIgor thanks for the suggestion.
@ramankumarlive can you take a look?
CC @Karishma-Tiwari-MSFT

looping @siriusk-MSFT and @yuemlu for guidance

@ramankumarlive @siriusk-MSFT @yuemlu one more try


add option to force usage of fallback_auth_store

When running within a CI, it is sometimes easier not to rely on the native authentication toolchain, but to use a simple credentials file instead (see https://github.com/prefix-dev/rattler-build/issues/334). This PR adds an option to force the use of this fallback file and disable the use of the system toolchain.

I think this is "OK" for me. Not a super-fan of the option :)
I am wondering if we should rather create another authentication storage that can be supplied instead of the current one. The new one would rely only on the fallback storage and uses plain JSON as database, and then we would have a Rust trait for an authentication storage?

Yeah, that sounds a bit more elegant.

I still need to test this with rattler-build

Nice work! Looks good to me. If we can get rid of anyhow it's perfect :)

I adapted and tested the rattler-build PR with our real-world example. Works great! I also changed the file location of the .json file, as the format changed slightly and we now store valid JSON.

Would you prefer to return a Box<dyn Error + Send + Sync>instead of an anyhow Result? Since I wanted to leave the option for users to bring your own backend, you can't really type the error type.

right. let's try to ask @baszalmstra what he thinks is best.


Sik-Ho Tang | Review: Xception -- With Depthwise Separable Convolution, Better Than Inception-v3 (Image Classification).

Sik-Ho Tang. Review: Xception — With Depthwise Separable Convolution, Better Than Inception-v3 (Image Classification).

Overview
In this story, Xception [1] by Google, stands for Extreme version of Inception, is reviewed. With a modified depthwise separable convolution, it is even better than Inception-v3 [2] (also by Google, 1st Runner Up in ILSVRC 2015) for both ImageNet ILSVRC and JFT datasets. Though it is a 2017 CVPR paper which was just published last year, it’s already had more than 300 citations when I was writing this story.

Original Depthwise Separable Convolution

Original Depthwise Separable Convolution.
The original depthwise separable convolution is the depthwise convolution followed by a pointwise convolution.

Depthwise convolution is the channel-wise $n\times n$ spatial convolution. Suppose in the figure above, we have 5 channels, then we will have 5 $n\times n$ spatial convolution.
Pointwise convolution actually is the $1\times 1$ convolution to change the dimension.

Compared with conventional convolution, we do not need to perform convolution across all channels. That means the number of connections are fewer and the model is lighter.

Conventional Convolution vs. Depthwise Convolution.

The original Depthwise Separable Convolution differs from the Conventional Convolution, as the original Depthwise Separable Convolution has shared convolution filter for multiple feature maps. In the Conventional Convolution, one convolution filter only generates one final feature map!!!

But in the original Depthwise Separable Convolution, multiple combinations of intermediate feature maps are supported through the following $1\times 1$ convolution.

Modified Depthwise Separable Convolution in Xception

The Modified Depthwise Separable Convolution used as an Inception Module in Xception, so called “extreme” version of Inception module (n=3 here).

The modified depthwise separable convolution is the pointwise convolution followed by a depthwise convolution.

This modification is motivated by the inception module in Inception-v3 that 1×1 convolution is done first before any n×n spatial convolutions. Thus, it is a bit different from the original one. (n=3 here since 3×3 spatial convolutions are used in Inception-v3.)
Two minor differences:

The order of operations: As mentioned, the original depthwise separable convolutions as usually implemented (e.g. in TensorFlow) perform first channel-wise spatial convolution and then perform $1\times 1$ convolution whereas the modified depthwise separable convolution perform $1\times 1$ convolution first then channel-wise spatial convolution.

This is claimed to be unimportant because when it is used in stacked setting, there are only small differences appeared at the beginning and at the end of all the chained inception modules.

The Presence/Absence of Non-Linearity: In the original Inception Module, there is non-linearity after first operation. In Xception, the modified depthwise separable convolution, there is NO intermediate ReLU non-linearity.

The modified depthwise separable convolution with different activation units.
The modified depthwise separable convolution with different activation units are tested. As from the above figure, the Xception without any intermediate activation has the highest accuracy compared with the ones using either ELU or ReLU.

Overall Architecture

Overall Architecture of Xception (Entry Flow > Middle Flow > Exit Flow).
As in the figure above, SeparableConv is the modified depthwise separable convolution. We can see that SeparableConvs are treated as Inception Modules and placed throughout the whole deep learning architecture.

E.g. The number of filters after $1\times 1$ convolution is fixed, greatly reducing the number of trainable parameters!!!

And there are residual (or shortcut/skip) connections, originally proposed by ResNet [3], placed for all flows.

ImageNet: Validation Accuracy Against Gradient Descent Steps.
As seen in the architecture, there are residual connections. Here, it tests for Xception using non-residual version. From the above figure, we can see that the accuracy is much higher when using residual connections.

Thus, the residual connection is extremely important !!!

Comparison with State-of-the-art Results
2 datasets are tested. One is ILSVRC. One is JFT.
ImageNet — ILSVRC
ImageNet, is a dataset of over 15 millions labeled high-resolution images with around 22,000 categories.
ILSVRC uses a subset of ImageNet of around 1000 images in each of 1000 categories. In all, there are roughly 1.3 million training images, 50,000 validation images and 100,000 testing images.

ImageNet: Xception has the highest accuracy.
Xception outperforms VGGNet [4], ResNet [3], and Inception-v3 [2]. (If interested, please also visit my reviews about them, ads again, lol)
It is noted that, in terms of error rate, not accuracy, the relative improvement is not small!!!

ImageNet: Validation Accuracy Against Gradient Descent Steps.
Of course, from the above figure, Xception has better accuracy compared with Inception-v3 along the gradient descent steps.
But if we use non-residual version to compare with Inception-v3, Xception underperforms Inception-v3. Should it be better to have a residual version of Inception-v3 for fair comparison? Anyway,

Xception tells us that with both Depthwise Separable Convolution and Residual Connections, it really helps to improve the accuracy.

Model Size/Complexity.
Xception is claimed to have similar model size with Inception-v3.

JFT — FastEval14k
JFT is an internal Google dataset for large-scale image classification dataset, first introduced by Prof. Hinton et al., which comprises over 350 million high-resolution images annotated with labels from a set of 17,000 classes.
An auxiliary dataset, FastEval14k, is used. FastEval14k is a dataset of 14,000 images with dense annotations from about 6,000 classes (36.5 labels per image on average).
As multiple objects are appeared in one single image densely, mean accuracy prediction (mAP) is used for measurement.

FastEval14k: Xception has highest mAP@100.

FastEval14k: Validation Accuracy Against Gradient Descent Steps.
Again, Xception has higher mAP compared with Inception-v3.

References

[2017 CVPR] [Xception] Xception: Deep Learning with Depthwise Separable Convolutions.
[2016 CVPR] [Inception-v3] Rethinking the Inception Architecture for Computer Vision.
[2016 CVPR] [ResNet] Deep Residual Learning for Image Recognition.
[2015 ICLR] [VGGNet] Very Deep Convolutional Networks for Large-Scale Image Recognition.


handle image download error

Description

Added function for Download image in new tab

Fixes #5764
Type of change

Bugfix (non-breaking change which fixes an issue)

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ]  I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:red_circle: Total coverage has decreased
// Code coverage diff between base branch:release and head branch: bug/image-dwnload-cors

Status
File
% Stmts
% Branch
% Funcs
% Lines

:red_circle:
total
53.58 (-0.01)
35.32 (0)
32.12 (0)
54.13 (-0.02)

:red_circle:
app/client/src/components/designSystems/appsmith/ImageComponent.tsx
15.84 (-1.18)
15.63 (0)
8.33 (-0.37)
16.67 (-1.4)

@techbhavin Tested this on deploy preview and it's working fine as per the comment https://github.com/appsmithorg/appsmith/issues/5764#issuecomment-879940490


[TEST] IT for bug PDI-18786; verify that the

Internal.Entry.Current.Directory variable gets set correctly upon
import.

:white_check_mark: Build finished in 36m 32s
Build command:
mvn clean verify -B -e -Daudit -amd -pl engine

:ok_hand: All tests passed!
Tests run: 3680, Failures: 0, Skipped: 7    Test Results

:information_source:  This is an automatic message


Missing utils.py

Hi - nice work!
In dist_finetune you have from utils import *, but no utils.py file.
I assume the file is the one from the scBERT repo?
Thanks

Yes, that's the one. I just uploaded the file to this repo to make things easier. Thanks for catching this!


Support for dependency/prerequisite jobs similar to a Makefile

Hi, this looks great!
Full candor, I haven't actually tried mask yet, but I've gone through the Readme and the only hesitation I have before completely jumping on board is that I don't see support for dependency/prerequisite jobs (similar to Makefile prerequisites). Are there any tricks for deduplicating commands, or would it be a good future enhancement?

👋 Hey @MattyRad
If I understand correctly, it sounds like you want to define a mask command that automatically runs other mask commands as a prerequisite, before running its own logic. Is that right?
We don't have a single one-liner way to do that. But if your command is using a shell lang, then running other commands is very straightforward with the $MASK helper alias.
# run some prereqs
$MASK some_command
$MASK another_command
# rest of script here...

Yeah that makes sense, seems obvious is retrospect. Thanks!


String interpolation filter

Would you consider adding the following filter feature on the jinja?
https://github.com/mozilla/nunjucks/issues/899
thank you.

IMO such a feature is pretty nasty. "How to get you data leaked"-nasty. I can already see something calling this on a user-provided string with all the implications that come with it...

(on the risk of sounding totally mad) IMO if you are already exposing that data to the template it does not really matter what's leaked, From nunjucks perspective (not familiar with jinja) the data is contained by the context anyways any sensitive data does not even get to pass through the template.

Exposing e.g. a User model instance to a template that shows user information is perfectly normal. But you wouldn't want to expose e.g. user.password (even if that's just a hash) or user.is_hiv_positive ;)
Also, Jinja code can call methods on the objects in the context, so unless you use the sandboxed environment, your templates must be about as trustworthy as your Python code.
FYI: You could always add a function that renders a string as a template as a filter from your application code. No need to have something like this in the Jinja core.


No any dialogs popup for getting passphase, git push still keep asking for passphase

Dear all
I have successfully installed pshazz using scoop install pshazz
I try to use git push -u origin master to push to my repository, no dialogs shown up, so I manually using ssh-add to add Key into ssh-agent
ssh-add c:/Users/username/.ssh/xxx_id_rsa
# i have entered passphase at this prompt

ssh-add -l
# i got confirmation that Key has been added

The next git push still keep asking me for the passphase

Please guide, what did I do wrong
Thanks

#76  New ssh plugin should be able to handle key passphrase better.

Same here, just no popup when installing pshazz, and git keeps on asking for my passphrase. Any idea?
My setup is git-with-openssh and pshazz, on a W10 machine.


Add intent for schedule recommendations

even if the bot doesnt have the data to answer this question, it could give users a form to fill out so it could essentially act as the data collection mechanism.
or in the meantime i could hard code

form for collecting course sequence data:
https://goo.gl/forms/WRvti28ZoTgi0Frf2


dmidecode system information version field

Use Case
Some manufactures put their model information under "Version" others under "Product Name".  Having both will help include the expected information.  The "Family" field may also be useful
Describe the Solution You Would Like
Include the Version field from the dmi information
Describe Alternatives You've Considered
Weird local hacks
Additional Context
From a Dell system:
System Information
        Manufacturer: Dell Inc.
        Product Name: Precision 3660
        Version: Not Specified
        Serial Number: D7DJSAY
        UUID: 4c4caa44-00a7-3a10-804a-c4c0bb37c933
        Wake-up Type: Power Switch
        SKU Number: 0A9F
        Family: Precision

From a Lenovo system:
System Information
        Manufacturer: LENOVO
        Product Name: 30C6S3WT00
        Version: ThinkStation P330
        Serial Number: MA09AC3D
        UUID: be1aa8a2-a3da-e9b1-bfc5-00ddd190e433
        Wake-up Type: Power Switch
        SKU Number: LENOVO_MT_30C6_BU_Think_FM_ThinkStation P330
        Family: ThinkStation P330

From a Supermicro system:
System Information
        Manufacturer: Supermicro
        Product Name: X9DRW
        Version:<PHONE_NUMBER>
        Serial Number:<PHONE_NUMBER>
        UUID: 00000000-0000-0000-0000-0000000000
        Wake-up Type: Power Switch
        SKU Number: To be filled by O.E.M.
        Family: To be filled by O.E.M.

From a HP system:
System Information
        Manufacturer: HPE
        Product Name: ProLiant DL380 Gen10
        Version: Not Specified
        Serial Number: CD29A00ANK
        UUID: 3738aa38-3b30-5ac3-32a9-323dd04c4e4b
        Wake-up Type: Power Switch
        SKU Number: 868703-B21
        Family: ProLiant

From an Asus system:
System Information
        Manufacturer: ASUSTeK COMPUTER INC.
        Product Name: RS720A-E11-RS24U
        Version: 00
        Serial Number: NAS0SD003059
        UUID: 78d8a699-a4cd-270e-9ac2-0b421ddcabaaa
        Wake-up Type: Power Switch
        SKU Number: SKU
        Family: Server

From an Intel system:
System Information
        Manufacturer: Intel(R) Client Systems
        Product Name: NUC10i7FNH
        Version: K61081-302
        Serial Number: GEFN00000000
        UUID: 32102a1f-30ea-1a60-801a-1c690a6aaf53
        Wake-up Type: Power Switch
        SKU Number: BXNUC10i7FNH
        Family: FN

Tyan
        Manufacturer: TYAN
        Product Name: B5630G62FV10HR
        Version: empty
        Serial Number: empty
        UUID: 30560000-xxxx-4539-3442-49323030314b
        Wake-up Type: Power Switch
        SKU Number: empty
        Family: empty```

Facter could fairly easily include product version information. I think we just need to include product_version in https://github.com/puppetlabs/facter/blob/69b0978f47b04d14d037bb62f0fd39ddb684b7a8/lib/facter/resolvers/dmi.rb#L34


How are packages evaluated in the VSCode extension and LSP server?

I'm working through Practical Common Lisp and just got to Chap. 21 - Packages and Symbols. I came across unexpected package evaluation behavior and wanted to clarify if this is intended.
Suppose you're developing in a file test.lisp with the following code:
(defpackage :cake
  (:use :common-lisp))

(in-package :cake)
(defun hello () (format t "Hello from cake"))
(hello)

(defpackage :rice
  (:use :common-lisp))

(in-package :rice)
(defun hello () (format t "Hello from rice"))
(hello)

If I evaluate (using "Send to REPL") only the two definitions of hello, without evaluating the defpackage or in-package forms, both functions are evaluated in their respective packages. I'm verifying this by running the two invocations of (hello), also with "Send to REPL", and they print as if the entire file is being evaluated.
Can you clear up my confusion as to why this is happening? Apologies if this is extremely trivial, I just didn't expect evaluation of those forms to be aware of their context in the file.

I was reading through the extension source and saw that this tries to compile the file on every keystroke so that must be it.


HBASE-23147 Branches-1 shaded artifact for mapreduce integration miss…

…es MainClass

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m  0s
Docker mode activated.

-1 :x:
docker
10m 52s
Docker failed to build yetus/hbase:edccfe439a.

Subsystem
Report/Notes

GITHUB PR
https://github.com/apache/hbase/pull/3214

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3214/1/console

versions
git=2.17.1

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
8m 33s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-0 :warning:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ branch-1 Compile Tests _

+1 :green_heart:
mvninstall
10m 18s
branch-1 passed

+1 :green_heart:
compile
0m 17s
branch-1 passed with JDK Azul Systems, Inc.-1.8.0_262-b19

+1 :green_heart:
compile
0m 19s
branch-1 passed with JDK Azul Systems, Inc.-1.7.0_272-b10

+1 :green_heart:
shadedjars
3m 37s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 30s
branch-1 passed with JDK Azul Systems, Inc.-1.8.0_262-b19

+1 :green_heart:
javadoc
0m 13s
branch-1 passed with JDK Azul Systems, Inc.-1.7.0_272-b10

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 19s
the patch passed

+1 :green_heart:
compile
0m 16s
the patch passed with JDK Azul Systems, Inc.-1.8.0_262-b19

+1 :green_heart:
javac
0m 16s
the patch passed

+1 :green_heart:
compile
0m 20s
the patch passed with JDK Azul Systems, Inc.-1.7.0_272-b10

+1 :green_heart:
javac
0m 20s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

-1 :x:
xml
0m  0s
The patch has 1 ill-formed XML file(s).

+1 :green_heart:
shadedjars
3m 20s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
hadoopcheck
5m 25s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2.

+1 :green_heart:
javadoc
0m 11s
the patch passed with JDK Azul Systems, Inc.-1.8.0_262-b19

+1 :green_heart:
javadoc
0m 14s
the patch passed with JDK Azul Systems, Inc.-1.7.0_272-b10

_ Other Tests _

+1 :green_heart:
unit
0m 19s
hbase-shaded-server in the patch passed.

+1 :green_heart:
asflicense
0m 21s
The patch does not generate ASF License warnings.

40m 15s

Reason
Tests

XML
Parsing Error(s):

hbase-shaded/hbase-shaded-server/pom.xml

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3214/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3214

Optional Tests
dupname asflicense javac javadoc unit shadedjars hadoopcheck xml compile

uname
Linux feb4e69e16a5 4.15.0-163-generic #171-Ubuntu SMP Fri Nov 5 11:55:11 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-agent/workspace/Base-PreCommit-GitHub-PR_PR-3214/out/precommit/personality/provided.sh

git revision
branch-1 / a2406f3764

Default Java
Azul Systems, Inc.-1.7.0_272-b10

Multi-JDK versions
/usr/lib/jvm/zulu-8-amd64:Azul Systems, Inc.-1.8.0_262-b19 /usr/lib/jvm/zulu-7-amd64:Azul Systems, Inc.-1.7.0_272-b10

xml
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3214/1/artifact/out/xml.txt

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3214/1/testReport/

Max. process+thread count
64 (vs. ulimit of 10000)

modules
C: hbase-shaded/hbase-shaded-server U: hbase-shaded/hbase-shaded-server

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3214/1/console

versions
git=2.17.1 maven=3.6.0

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


CI: Don't test microvm

This disables running microvm in CI for now until https://github.com/hermitcore/libhermit-rs/issues/161#issuecomment-1006022190 is fixed.

I added this commit to https://github.com/hermitcore/libhermit-rs/pull/327, since we need both PRs for a passing CI.

Reopened hoping we can merge this being lucky with GitLab CI until https://github.com/hermitcore/libhermit-rs/pull/327 is solved properly.

Nevermind, I made a mistake in #327.

Not making the same mistake again, it seems like merging this should be possible though, if we are lucky.

@stlankes, are you okay with disabling microvm CI?

I would not remove the microVM tests. PR #335 is a workaround for this issue.
With KVM acceleration, the microVM is able to determine the frequency. By emulation, we pass the frequency with the Qemu flag -append "-freq XXX". However, the loader is currently not able to pass the command line to our kernel. I looked into the code. I don't see any failure in our implementation or in the crate multiboot. The latest Qemu version for Ubuntu could have an issue. We have to investigate it.

All right. :ok_hand:
Closing in favor of #335.


[SPARK-47680][SQL] Add variant_explode expression.

What changes were proposed in this pull request?
This PR adds a new VariantExplode expression. It separates a variant object/array into multiple rows containing its fields/elements. Its result schema is struct<pos int, key string, value variant>. pos is the position of the field/element in its parent object/array, and value is the field/element value. key is the field name when exploding a variant object, or is NULL when exploding a variant array. It ignores any input that is not a variant array/object, including SQL NULL, variant null, and any other variant values.
It is exposed as two SQL expressions, variant_explode and variant_explode_outer. The only difference is that whenever variant_explode produces zero output row for an input row, variant_explode_outer will produce one output row containing {NULL, NULL, NULL}.
Usage examples:
> SELECT variant_explode(parse_json('["hello", "world"]'));
 0	NULL	"hello"
 1	NULL	"world"
> SELECT variant_explode(parse_json('{"a": true, "b": 3.14}'));
 0	a	true
 1	b	3.14

Why are the changes needed?
This expression allows the user to process variant array and object more conveniently.
Does this PR introduce any user-facing change?
Yes.  A new SQL expression is added.
How was this patch tested?
Unit tests.
Was this patch authored or co-authored using generative AI tooling?
No.

thanks, merging to master!


There's no way to download zip anymore I can't download

So I'm trying to download the zip file and it's only giving me the option to view code and go to file and then I click on it and it's just the code

That's right, the code is the program you want to run (Code -> Download ZIP). Download the ZIP, extract it, and simply follow the instructions in the README.md file.

Run this command
$ git clone https://github.com/Sycnex/Windows10Debloater.git


Travis Config - Insecure API Key

@WesCossick Looking at the travis config, shouldn't the api key be hidden and used via environmental variable? This is opening up for insecure publishing.

@prageeth the value is marked with secure: xxxx and I think this is the encrypted value not the plain/text one so no direct exposure, see https://docs.travis-ci.com/user/environment-variables/#defining-encrypted-variables-in-travisyml

@atodorov Thanks for the explanation, it was my mistake. Yes, you're right and it makes sense. Thanks!


Feature Request: Add loading animation and failed message in the terminal.

Description
For some of the commands in the terminal the app relies on third party apis. While executing the command add a loading animation in the  terminal and show a failed message if the command fails to execute.
Screencast from 14-01-24 11:43:48 PM IST.webm
Checklist ✅

[X] I checked and didn't find similar issue
[X] I am participating in JWOC

Please assign it to me.

Please assign it to me.

Are you participating in JWOC?

@firtysh yes

Hey @firtysh, I would like to work on this issue under JWOC'24.

please Assign it to me


Add build options to UI

Update design for input page and toolbar
Add build options drop down to incorporate any exported options
Add rebuild button

relies on #1152

addresses issue #85. also overlaps with PR #90, which can probably be closed


build_choose_clause does not properly escape single quote

I've been receiving several QuickbooksException: Error parsing query recently when trying to query a list of customers. I tracked it down to the build_choose_clause and it looks like it's not properly escaping single quote characters as if I do something like Customer.choose[u"Someone's Company"], field=u'DisplayName', qb=client) then I get the following in response:
python2.7/site-packages/quickbooks/client.pyc in handle_exceptions(self, results)
    316                 raise SevereException(message, code, detail)
    317             else:
--> 318                 raise QuickbooksException(message, code, detail)
    319
    320     def create_object(self, qbbo, request_body, _file_path=None):

QuickbooksException: Error parsing query

I'm using python 2.7.12 and version 0.5.1 of the package. Also it looks like the error detail isn't getting include in the error output but that's a separate issue I think.
I tried passing the string directly to the build_choose_clause function and you can see the quote is not escaped:
choices = [u"Someone's Company"]
build_choose_clause(choices, u'DisplayName') # "DisplayName in ('Someone's Business')"

Fixed in version 0.5.3


This adds the ability to EXCLUDE from a trace

Goldrush was updated in 3.2.3 to 0.19, This extends the feature added by it for use in lager.

This is redundant with #389


Randomized names for ingress rules when not assigned

Description
We were previously running into a lot of collisions with subdomains within the environment since deploys would leave unassigned ingress rules with empty values for subdomain and path. This makes it very clunky to deploy since you need to explicitly assign subdomains to every ingress.
This PR uses a random name generator to assign randomized names to subdomains when there isn't one set explicitly. The generator also uses a tactic to ensure the same random subdomain is used for ingress rules on subsequent deploys which is really handy for remembering URLs:
Closes https://github.com/architect-team/arcctl/issues/136
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules
[x] I have checked my code and corrected any misspellings

I mean the code here is fine, but this feels odd to me. It would be odd to see a bunch of random domain names just popping up and them changing every run.

I mean the code here is fine, but this feels odd to me. It would be odd to see a bunch of random domain names just popping up and them changing every run.

They don’t change every run. The generator uses the node ID as a pinning value so that you’ll always get the same value for the same node ID.
But if that’s not enough do you have a better idea to solve the problem? My example component has 6 ingress rules across three components and assigning values for them all is annoying. Especially when I only care about the one web app that I’ll actually be opening manually (which can still be assigned with the “-i” flag).

No that sounds good. Though did realize 1 potential issue and what happens if you don't want any subdomain because you want it to be the root domain?

No that sounds good. Though did realize 1 potential issue and what happens if you don't want any subdomain because you want it to be the root domain?

We talked about this offline and I think we're going to assume that use-case is done using the @ symbol like many DNS providers. It'll be up to each arcctl provider to recognize this, but that should solve the problem.


请求提供一份简单样例

能否提供一个关于CNN和transformer的简单样例代码，并在其上使用UniPT，以便于可以通过学习这个样例来将该方法使用在其他领域上。十分感谢！

The definitions of UniPT on CNN and Transformer are in  adapter_for_cnn.py and adapter_for_transformer.py, respectively.
Please refer to two types of self.side_tuning_module in GPO/lib/encoders.py to activate them.
Don't forget to adjust related configurations including hidden_size, num_layer, and etc for your domains.
Besides, please follow the forward function in EncoderImageFull and EncoderText to implement the UniPT.
Some tips might be useful to you.

Set the learning rate of UniPT module as 10x of pretrained weights. (or 5x, or 20x)
Set the weight decay from 1e-4 to 1e-2. Smaller sizes of pretrain model, larger values of WD.
For CNN, the outputs of first one or two shallow layers can be abandoned. Generally, the memory usage is largely reduced, and performance is not sacrificed.
The structure of FFN layer in the confidence module can be adjusted (deeper or shallower) to get better results.
The orders of ReLU and Normlayer (LayerNorm or BatchNorm) are sometimes different according to some references.
Please set the minimum value of eps as 1e-6 in case of FP16 or Mixed FP16+32. It bothers me before！
Other issues ......

I'm really sorry that I've been so busy lately. Hence, I'll be out of contact for a while until the end of September.


Incorrect SRID in Value panel is shown for values without SRID

System information:

Operating system (distribution) and version win10
DBeaver version 21.2.4 EA

Describe the problem you're observing:

still reproducible

verified


Wrong transpose in src/align/detect_face.py

In src/align/detect_face.py, np.transpose() is used to transpose input images before feeding to PNet, RNet and ONet. For example, the following code does transpose before ONet:
tempimg[:,:,:,k] = imresample(tmp, (48, 48))
...
tempimg1 = np.transpose(tempimg, (3,1,0,2))
out = onet(tempimg1)

I think the correct one should be np.transpose(tempimg, (3,0,1,2)). The axis of tempimg is (height, width, color channel, batch). The expected axis of onet's input is (batch, height, width, channels), as documented by tf.nn.conv2d.
The wrong code still works because it's a square image, i.e. height = width, and the training and detection are both wrong.
It's written this way, is it because the model is directly converted from the Matlab project without training? Matlab's convention is width x height x color.

Yes. The author just loaded the parameters from the MATLAB version without training, so the tensorflow version must be a exact translation from the MATLAB version. As there is a lot of .T in the MATLAB version, so there are a lot of transpose() in the TF version. This is also what I ToCao a lot when I was implementing my Keras version.
For more reference, please go to this [issue](, this is because the tensorflow version of mtcnn is just a translation from the MATLAB version. )

That's what I thought. Thanks.

That's what I thought. Thanks.


[WFLY-13256] Upgrade BouncyCastle from 1.62 to 1.65

https://issues.redhat.com/browse/WFLY-13256
It's a known issue that this upgrade results in some odd behavior with Picketlink STS tests on JDK 13, where running certain tests before others causes those others to fail. As picketlink use is deprecated and our support for JDK 13 is meant for developer evaluation, myself, Darran, Alessio, Jim Ma and Ashley discussed this and I believe it's more important to get this in than to worry about the PicketLink + JDK 13 scenario.
Custom JDK 13 test run against this branch is at https://ci.wildfly.org/viewLog.html?buildId=205620&buildTypeId=WF_MasterLinuxJdk13

This looks like it produces some issues with the WS tests.


Markdown cell content not read out using VoiceOver  and new markdown renderer

Repro

With "notebook.experimental.useMarkdownRenderer": true
Open a notebook with rendered markdown cells
Enable VoiceOver
Try using arrow keys to navigate through the markdown cells

Bug
With the old markdown renderer this sort of worked (it seems to only read the first line). With the new renderer, it just reads: group for all markdown cells

Quick review of what I've tried so far to fix this:
Use aria attributes in the workbench dom
This involved setting  aria-label on the main cell node in the notebook dom.
This causes Voiceover to read out the markdown text, however I ran into a few problems/limitations:

The label needs to use the rendered markdown text, not the markdown markup.
This does not appear to support many of the screenreader navigation commends, such as navigating between paragraphs for long markdown content (not 100% sure I was testing this correctly and maybe there's another attribute we can use instead?)

Focus into the webview when a markdown cell is focused
This calls .focus on the rendered markdown cell inside the webview.
This also works but needs major work to polish:

focusing into the webview causes the cell to lose its active focus border
Once focused inside the webview, the tab order seems to be very odd (perhaps this has already been solved for output views?)
Voiceover will sometimes mention that you are inside a frame and offer frame navigation options. This is an implementation detail and should likely not be exposed to users

/cc @rebornix

For now, we've disabled the new markdown renderers when in screen reader mode. Will continue investigating proper support  accessibility for the new markdown renderer

This should have been fixed by 27858a0faf78e502acf3c8f620f86a671a55e473
Let me know if you are still seeing any issues


Throwing from disable() with invalid watcher

I've been running into timing issues causing Loop::disable() to throw due to a watcher being invalidated during the invocation of the watcher callback. Essentially, if the watcher callback calls Loop::disable() at the end of the callback after side-effects from the watcher callback have invoked Loop::cancel(), an exception is being thrown. While of course it is possible to use an additional flag to code around this problem, it seems unnecessary.
Some example code: https://github.com/amphp/socket/blob/amp_v2/lib/Server.php#L37-L51
Putting the call to Loop::cancel() in a defer watcher seems to solve the issue here. I tried the same in the Socket class within the same lib here. However, this does not appear to have solved the problem, as tests still occasionally result in throws due to invalid watchers.
I propose that InvalidWatcherException should not be thrown from disable() with cancelled watchers, as disabling a cancelled watcher does not change the expectation that further events will not be received on the watcher. enable() should continue throwing when a cancelled watcher is given.

I've observed this issue myself too. While not too hard to fix once reproduced, it's a ticking bomb…
 I only can find the obvious ones, but such scenarios where the cancel() gets triggered by a destructor through the cyclic garbage collector are not predictable and usefully testable.
If we don't change this as proposed, I'm looking forward to random crashes of long-running applications which are hard to debug (why did this happen???) ...

In the earlier amp implementations we just silently ignored disable() or cancel() calls for nonexistent (read: already cancelled) watchers ... The result is the same either way. I'm fine with this behavior, personally. Thoughts?


Generalize the tensor container to arbitrary depth

FEATURE
Currently, calling tf.dispose(obj) or tf.tidy(() => obj) results in walking the obj for depth of 1 and disposing of any tensors found.
With this change obj is walked to arbitrary depth with cycle-detection to defend against cyclical objects.
This makes tf.tidy() and tf.dispose() more useful in practice.
Also rename util.extractTensorsFromContainer to util.getTensorsFromContainer since we are not modifying (extracting from) the container.

For repository owners only:
Please remember to apply all applicable tags to your pull request.
Tags: FEATURE, BREAKING, BUG, PERF, DEV, DOC, SECURITY
For more info see: https://github.com/tensorflow/tfjs/blob/master/DEVELOPMENT.md

This change is

Review status: 0 of 5 files reviewed at latest revision, 1 unresolved discussion, some commit checks pending.

src/util.ts, line 459 at r1 (raw file):
Previously, nsthorat (Nikhil Thorat) wrote…
do you need |void?

yeah because TensorContainer is void|otherStuff and void is needed for tidy's that won't return anything.

Comments from Reviewable


Bump up the formidable package version.

Bump up the "formidable" package version to address a vulnerability issue
For more information about how to contribute to this repo, visit this page
Description
The currently used version of "formidable" package has a vulnerability bug which has been fixed in the latest version (3.2.4). Bumping up the version to the latest to address the vulnerability issue.
Steps to Reproduce Bug and Validate Solution

Only applicable if the work is to address a bug. Please remove this section if the work is for a feature or story
Provide details on the environment the bug is found, and detailed steps to recreate the bug.
This should be detailed enough for a team member to confirm that the bug no longer occurs

PR Checklist

Use the check-list below to ensure your branch is ready for PR. If the item is not applicable, leave it blank.

[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.
[x] My code follows the code style of this project.
[x] I ran the lint checks which produced no new errors nor warnings for my changes.
[x] I have checked to ensure there aren't other open Pull Requests for the same update/change.

Does this introduce a breaking change?

[ ] Yes
[x] No

If this introduces a breaking change, please describe the impact and migration path for existing applications below.

Testing

Instructions for testing and validation of your code so the reviewer can follow those steps and validate code works as expected

Any relevant logs or outputs

Use this section to provide either screenshots or output of logs as code snippets

Other information or known dependencies

Any other information or known dependencies that is important to this PR.
TODO that are to be done after this PR.

Closing in favor of #10451 , thanks Arik for working with me on this!


Add sync.yml files

This pull request will not function without a PAT added the the repository, but it serves to demo what such a workflow could look like, as requested in issue 7.
The actual syncing is all done by the BetaHuhn/repo-file-sync-action. In addition to including it as a step in the workflow, an additional file is also necessary to list what repositories and files should be kept in sync. The default location of this file is .github/sync.yml
I've included only SymbiFlow/prjxray-bram-patch for now, as that was the test repository decided upon, but it would be easy to add others and customize what we sync to them later on.
When the action runs, all of the target repositories listed in .github/sync.yml will be forked by the account listed under the FORK parameter, then after all of the files are synced on a new branch, a pull request is opened to the upstream repository listed in .github/sync.yml. This way, even with the PAT to the bot account, the action cannot modify the upstream repositories.

@nelsobe @acomodi @mithro

@xanjohns thanks for the detailed explanation. I think this looks good, and we can proceed with the generation of the PAT, just need access to the common-config-robot (I guess via email is fine)

I have added the GH_PAT secret, containing the PAT to access the common-config-bot user where the repositories will be forked and from where the PRs will be opened to the upstream repos. I'll merge this PR and verify that everything is working correctly.


list and button

https://github.com/pedroslopez/whatsapp-web.js/commit/b60029ed09fc44fdc7531da9fe8dc47823783102

+1

+1


[feedly] fix packages

Change(s):

Re-built the packages using this blog<EMAIL_ADDRESS>
Reason for Change(s):

The packages were built on Mac and wouldn't work on Linux

Hi @Mathieu4141,
Could you please provide us successful running screenshots of connector/invocation logs. Thanks

Hi @Mathieu4141, Could you please provide us successful running screenshots of connector/invocation logs. Thanks

The connector isn't the issue here, it's the process of packaging it
I had a working version of the code of the connector since the first PR I made back in September, but since then I failed to make a working packaged version of the solution
I was asked and provided your team during the last PR with credentials to use the solution, and that PR got merged
I didn't find any other way to test the packaged solution than having the PR merged and starting the offer publish process. This makes it really hard to spot errors, as I only encounter them once at a time. But maybe I'm missing something here...
Here are some logs of the working connector (but the packaged solution might still have issues I'm unaware of):


2 Streams never end - while each stream separately does

I have the following script:
gulp.task('htmlscripts', ['cleanup'], function() {
    var allStreams = [
    gulp.src(['*.html', '!index.html'])
        .pipe(useref())
        .pipe(cache('useref'))
        .pipe(gulp.dest(distFolder)),
    gulp.src('index.html')
        .pipe(useref())
        .pipe(cache('useref'))
        .pipe(gulpif('*.js',
            uglify()
        ))
        .pipe(gulpif('*.css',
            cleanCSS()
        ))
        .pipe(gulp.dest(distFolder))
    ];
    return merge.apply(this, allStreams);
});

If I comment out one of the two streams, the merge fires the end event - if I have both streams in the merge, all further dependencies are not started. I use merge-stream in other tasks where it works perfectly. I have no clue what's wrong here - maybe one of the pipes do not return properly?
PS: If you don't like the merge.apply syntax - don't worry, I tested the problem with merge(var1, var2) as well and it's the same.

Closing this because it's probably very stale. If it turns out this is still an issue that needs investigation, please re-open with a more minimal reproduction.

@geri777 did you ever resolve this? i'm having the same issue

I cannot exactly remember, but I think you should check the following:

remove single pipes and test if it works then. Sometimes single extensions do not respond with a stream/promise
Try to catch errors from the functions you are using. For example:

instead of...
uglify()

use
uglify().on('error', function(e){
	console.log("Uglify Error - maybe you are using ES6 Code?")
	console.log(e);
})

thanks @geri777  :) i ended up adding an .on('end', this.emit('unpipe')) to the individual pipes and it ended up working

@geri777 @anderoonies Please send a PR to fix it if possible. Thanks a lot!


March Post long running PR

Feature Request: BIP47 payment codes for private donations

What are PayNyms?
BIP 0047

New Bitcoin scaling whitepaper just dropped: "CoinPool: efficient off-chain payment pools for Bitcoin"


App: no shadows lighting on avatars

https://user-images.githubusercontent.com/6926057/141698302-c58ff029-693e-40a3-b4b5-07a3d7ca01f8.mp4
This was working before.

Looks like that's an old version of the avatar. Current VRM version in the drive works fine.

https://drive.google.com/file/d/1hz3n_JwwCl8YHprFmg21gaSokR6PZZCH/view?usp=sharing
Direct link if you need it.

Yup you're right, updated in master now. Thanks!


viewDidAppear and viewWillAppear not calling for parent controller

viewDidAppear and viewWillAppear not called on parent view controller after dismissing child view controller presented using SPStorkController.

Good day!
You set delegate?

Good day for you, yes I'm setting it before presenting the child view controller. didDismissStorkBySwipe is called, but viewDidAppear and viewWillAppear not called.

Please, create simple project and send here. I am tested it

Oh sorry, viewDidAppear and viewWillAppear not calling. It is true. But I do not know what is the reason.

Also viewDidDisappear and viewWillDisappear not calling, I hope you find a fix.

Did you find the solution?

@BrianTruong23 for now - no

Hope you can fix it. Waiting for you.

@fawzirifai are you sure that after the modal controller
Should these methods be called?

Good day, friends!
I am tested it bug, and now can describe more.
When you present modal controllers, the parent controller doesn't actually hide. It is simply blocked by another controller, but not hidden or closed.
That is why these methods are not called. They should not be called. You can do simple test:
override func viewDidAppear(_ animated: Bool) {
    super.viewDidAppear(animated)

    print("viewDidAppear")

    let controller = UIAlertController.init(title: "Title", message: nil, preferredStyle: .alert)
    controller.addAction(UIAlertAction.init(title: "Cancel", style: .cancel, handler: nil))
    self.present(controller, animated: true, completion: nil)
}

Print not call after dismissing.
SPStorkController is modal controller, and this methods shouldn't call.

So what you are saying is that we can not use view will appear or view did appear right?

What method should we use if we want to know when the view is appear?

conform to SPStorkController delegate and use this code in the parent view controller:
extension _: SPStorkControllerDelegate {
func didDismissStorkBySwipe() {
}

func didDismissStorkByTap() {
}

}

Sorry but it says that Use of undeclared type 'SPStorkControllerDelegate'

Did you import SPStorkController?

Yes

@BrianTruong23 methods viewWillAppear for parent controller. For modal controller, which use SPStorkTransitionDelegate all methods call correctly.

@BrianTruong23
I am tested now this:
extension HomeController: SPStorkControllerDelegate {}

All work correctly. Which version of pod you use?

I think I use the old version. I  am updating my pod

Is the new version 1.6.1 ?

@BrianTruong23 latest version is 1.6.1

I am sorry to bother again but somehow the dismiss stork by tap didn't trigger when the stork controller dismiss

@BrianTruong23 please, read Readme / Delegate section,

After implementing this transitionDelegate.storkDelegate = self, I get a weird bug Thread 1: signal SIGABRT

@BrianTruong23 if you think this bug in SPStorkController, you should create new issue. This is necessary for order with errors. Please write here only on the related topic.

I fix the error. Thanks a lot. But one more question, dismiss stork by tap is triggered when tap is dismissed. But it didn't trigger when we tap a button to dismiss in modal view controller. Do you have any suggest?


Unable to install extension 'github.vscode-pull-request-github' as it is not compatible with VS Code '1.69.2'

Extension version: 0.47.2022080211 & 0.49.2022080809
VSCode Version: 1.69.2
OS: Windows 10 Pro  21H2     build no: 19044.1826

Steps to Reproduce:

Download extension
Install from vsix

This is intentional. We now rely on changes in 1.70 for the extension to work.


Add example of saving / resuming DataLoader state with PyTorch Lightning

📚 Documentation
Right now, the README shows an example of pausing and resuming the StreamingDataLoader state within a simple for loop over the dataloader.
It's not clear to me how to adapt that example to work with PyTorch Lightning, where the training loop is abstracted away.
My first guess is to make a callback for the Trainer -- but it would be great to have a simple example of how to do this in PyTorch Lightning.

The README shows how you'd do it in plain torch yes. PyTorch Lightning already supports checkpointing out of the box, so you'd literally have to do nothing.
If your train_dataloader() methods return a StreamingDataLoader then its state will be saved into the checkpoints and when you resume from one via trainer.fit(..., ckpt_path=...) it loads the dataloader state back to continue.
Yes we can make it more explicit in docs etc, good idea!

Thank you for the clarification!

@awaelchli  I can work on this


Question about how gorm handles NULL

I see that when I have a pointer type on a struct, and I update it, gorm generates this SQL:
UPDATE "object" SET "nullable_column" = '<nil>'  WHERE ("id" = '5BdF4x')

The .Error property of the gorm call is nil, so nothing happens, and I think this updates.
I think somewhere this updates the column with NULL (if the column is nullable), but just wanted to confirm this feature or find this better documented somewhere.
Does gorm transform the above SQL to a different actual SQL that's sent to Postgres? Running the exact SQL doesn't work.

Hi @qimingweng
The above SQL is formatted by logger https://github.com/jinzhu/gorm/blob/master/logger.go, it is not the exactly SQL passed to Postgres, arguments are handled by database adaptor.

@jinzhu Is it possible to be pointed to the parts of the code handle this? I'd like to get a better understanding of the inner workings and contribute perhaps to the documentation


run is extremely slow after upgrade 3.5.1 from 3.5.0 with Window Os

Environment

Operating System: Windows_NT
Node Version:     v16.14.0
Nuxt Version:     3.5.1
Nitro Version:    2.4.1
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      -
Runtime Modules:  -
Build Modules:    -

Reproduction
none
Describe the bug
run is very slow after upgrading to v3.5.1 from v3.5.0.
I tried it with Window and Mac both and only with Window os gets slow after upgrading to nuxt v3.5.1.

Additional context
No response
Logs
No response

Would you try setting experimental.watcher to chokidar?

Would you try setting experimental.watcher to chokidar?

It works.

Great - next steps, would you be able to provide a reproduction for the original report? Is this reproducible with a new project?

Sorry, I don't really understand what you are saying provide a reproduction for the original report. Do you mean that make a new project with  setting experimental.watcher to chokidar or set experimental.watcher to chokidar for original project and see it works?
Just let you know @jongmin4943 is my one of colleages

My apologies for not being clearer. I was asking how the original issue (without the workaround I provided above) can be reproduced.

I had this problem after upgrading from 3.4.3 to 3.5.0

My apologies for not being clearer. I was asking how the original issue (without the workaround I provided above) can be reproduced.
I'm having the same issue on a completely new repo created with nuxi init. I am too on windows

Cheers

Noticed this myself, the quickest way to replicate it is to just setup a docus site
npx nuxi init -t themes/docus

Then update the package.json to the latest versions, including nuxt 3.5.1
Not sure if it's related
but I'm also noticing the search box under https://content.nuxtjs.org/ is broken as well

I had the same problem with all Nuxt3 projects under Windows. It took too long on each time when dev (Windows 11 - Intel 12900H):
ℹ Vite client warmed up in 4680ms
✔ Nitro built in 349075 ms
✔ Vite server hmr 95 files in 9440.

And it recompiles from time to time, on my Windows. It calls msbuild for compilation, which is not expected in minor updates.
The Node Runtime process consumes over 30% of the CPU and up to 4GB of memory.
There was a small speed increase after setup experimental: { watcher: 'chokidar' }, but still spent a lot of time on reloading.
ℹ Vite client warmed up in 2355ms
✔ Nitro built in 84662 ms
✔ Vite server hmr 95 files in 2027.28ms

Would you try setting experimental.watcher to chokidar?

The configuration doesn't work at all in my case.

I tried the empty Nuxt project and found that only 3.5.1 had problems.
But my own project still has the problem under 3.4.3
Therefore, it should be a problem of some upstream dependency
However, there are no exceptions that can be troubleshot through the diff file.

So is there any way to identify the problem?

Something also to bear in mind is that I was experencing some strange errors in addition to the above until I tried removing the node_modules and package lock file (pnpm-lock.yaml) to make sure all the dependencies were recalculated from scratch.
If you need more of a working example, this is a site I've been updating recently that ran into the problem with slowness untill I added in that chokidar option
https://github.com/Hecatron-Cad/LinuxCNC.Notes/tree/master/docs/docus

I can confirm the Edge Channel version (3.5.2) is too slow to build the project on Windows.

With the experimental.watcher: "chokidar" the build time has significantly improved.

It doesn't seems to be solved on 3.5.3 same starting time as previous versions on windows

@CharlesBT I don't know if it works for you.
I found that it watch the node_modules files in the subproject, so I ignored it and it worked.
So I hope this issue to be fixed.
export default defineNuxtConfig({
  ignore: [
    '**/*.stories.{js,ts,jsx,tsx}',
    '**/*.{spec,test}.{js,ts,jsx,tsx}',
    '**/*.d.ts',
    '.output',
    '**/-*.*',
    // watcher ignore docs/* to avoid preformance issue
    'docs',
  ],
}


Added support for integration via Carthage and/or Workspace

The way HCKalmanFilter is currently set up only provides for integration through CocoaPods or plain-old manual "moving source files out of version control and adding them to your project".
For a project using Carthage this is less than ideal, if not outright unusable.
HCKalmanFilter is too nice a project to leave it on the table for every other user. 🙃
Please consider adding support for Carthage.
Ps: There is one more step required to make things work with Carthage:

Create a tag for each version commit (starting from this current merge with 1.2.2).

@Hypercubesoft, any plans for merging this?

Hi Valeriyvan,
Sorry for late replay. We were overwhelmed with work so we missed your message. We are going to do this merge in next few days.
Best Regards,
Hypercybe

Any update on this, @Hypercubesoft?

@Hypercubesoft are you gonna merge this or should I just close the PR?


Small issues in documentation/API-design-guidelines.md in format and typos.

Problem description
Pls fix the documentation/API-design-guidelines.md in format and typos.
Small format adjustments would be appropriate in line:
216 - 218 : change "- - " to "--" or " -"
and typos in lines
70 - "comunicate"
120 - "analyse"
211 - "terminolgy"
217 - "mulitple"
288 - "senstive"
324 - "Unathourized"
405 - "recommendended"
412 - "Referer"
592 - "aAdded"
634 - "ellaborate" and "exisiing" and "errores"
709 - "returnss"
720 - "ascp" - i only know asc for sorting
1010 - "erros"
1050 - "desctiption"
1186 - "Anycase"
1324 - "explicited" (explicitly ?)
1335 - "accross" and "exemple"
1354 - "subscrition"
Thx
Expected behavior
format and typos are fixed.
THX

@dfischer-tech  Could you prepare PR introducing the needed corrections?


TDS css affects non-tds components

Description
I have a webpage where I want to use TDS components to follow telus style in part of the page, and in other parts of the page I need the flexibility to not use TDS components (such as tables for html).  I believe as a result of the reset css included in the project, it has removed the list styles, so the unordered list has no bullets, and ordered list has no numbers.
Also <h1> is the same size as <p>
Reproduction Steps

Create a component that takes raw html
Include import styles from<EMAIL_ADDRESS>anywhere in application
Notice list-item style is set to 'none'
Notice h1 - h6 styles have the same style as normal paragraphs

Reproduction Percentage (if applicable)

10/10 times

Meta

[ x] TDS version: v1.0.1
[ x] Willing to develop solution
[ ] Has workaround
[ x] High impact

Screenshots

Include any relevant screenshots

Description

Description of the problem

Recommendation
It would be great if TDS styles would be contained within TDS react components, and not applied to the entire page
Designs

Screenshots
Sketch file

Meta

[ ] Currently using TDS version: vX.Y.Z
[ ] Willing to develop solution
Has workaround? Yes or no
Impact: Low, medium, or high

Thanks for bringing this up!
Yes, at the moment the "tds.css" file contains a small amount of global styles and all the scoped component styles https://github.com/telusdigital/tds/blob/master/guide/getting-started/faq.md#what-global-styles-does-tds-set-on-the-page
We actually already have plans to improve this, which we will be working on in the next month. Our next big initiative is to separate the TDS components into multiple packages, 1 package per component. This will allow consumers to pick and choose which components and styles get included in their application. We will likely also introduce a package for the reset and the global styles, so that you have to choose to include them.

While you are waiting on that change, are you able to build components or styles to override the global styles from "tds.css"? This is what TDS components do, for example, the List components: https://tds.telus.com/#lists. Any class name you put on your components will have a higher specificity than the reset, allowing you to fully control the styles of your elements.

I will look into this, the problem I am anticipating is that most of my components actually don't need to be affected by TDS.  I will see if there's an easy way to do this tho.

@tsungting
The latest release of TDS splits all TDS components into separate packages. Each component package only contains scoped CSS classes that do not effect any other components on the page.
We have also created a new package called css-reset that includes all global CSS. This should allow you to include or not include the global CSS how you see fit in your project.
I believe this addresses your issue, so I'll be closing it. Feel free to reopen if you see any issue.


Add instructions for VoidLinux in README.md

This seems to be conflicting with the master branch due to PRs from other folks. I'd appreciate if you wouldn't mind rebasing; otherwise, I can fix it later.


Merge Patrick's Tabulator Branch

Don't forget to merge this branch prior to October meeting

See new Issue "Make New Branch Live"


[WIP] Implement EAP-NOOB (RFC9140) (IDFGH-7438)

This is an implementation of RFC9140 (EAP-NOOB)
The implementation is not completely standard compliant. The JSON library will remove any unnecessary spaces in objects (which the RFC forbids, the encoding has to be exactly the same for the HMAC calculation) and I'm not sure about UTF-8 support, since I use functions like strlen.
Since EAP-NOOB is a non-conventional EAP-Method in the meaning that it requires an out-of-band-message before the actual login is successful, I modified some of the logic in the esp_wpa stack.
This Pull Request is still work in progress, for now I would like to get feedback if the way I implemented this is ok or if I should adjust the architecture, especially regarding the public facing methods for OOB-Message generation/retrieval/offloading and initializing the EAP method.

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

For the JSON encoding compliance this is what the RFC says in section 3.3.2:

The inputs to the hash function for computing the fingerprint Hoob and to the HMAC for computing MACs, MACp, MACs2, and MACp2 are JSON arrays containing a fixed number (17) of elements. The array elements MUST be copied to the array verbatim from the sent and received in-band messages. When the element is a JSON object, its members MUST NOT be reordered or reencoded. White space MUST NOT be added anywhere in the JSON structure. Implementers should check that their JSON library copies the elements as UTF-8 strings, does not modify them in any way, and does not add white space to the HMAC input.

Thanks for your contribution.

Hi @Janfred , From architectural point of view, changes looks ok (some cleanup is required for public functions and code though). Let us know once you complete it, we will review this in depth.

@Janfred Thanks for contribution, any further updates for the PR? Thanks.

Unfortunately, this was part of a study program and my efforts in the IETF to specify a revised EAP protocol that uses CBOR instead of JSON did not receive feedback, so I kind of abandoned this.
If there is renewed interest with a use case, I would be happy to pick the ball up again.

Hi @Janfred, thanks for the update. We will close this request for now. Feel free to reopen if you start working on this again.


tests on new repo

I accidently deleted the testing repo, so I have regenerated the tests on new repo https://gitlab.com/packit-service/ogr-tests

https://github.com/packit-service/ogr/pull/186 was merged -- could you please rebase?

recheck


"play" paused wav file with same name string should continue playback

If a "play" command is received with the same filename as the file that is currently paused, playback should continue and not start over.

play: spielt file ab, wenn neue file dann ab beginn, wenn bereits begonnen dann weiter abspielen
pause: pausiert aktuelle file / jingle / freq
play empty: go to idle state


Qualify different role types when creating Env type Policies.

When creating Environment level Policies, you say you need Environment Admin role,  when there is a data verse, this role does not exists, its equivalent is most likely Sys Admin.   You should say so.

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

ID: dea88353-deff-d4d6-c1fc-ab36e1cf9d6d
Version Independent ID: 600c9506-f379-092b-b32a-2e64b4999bed
Content: Create a data loss prevention (DLP) policy - Power Platform
Content Source: power-platform/admin/create-dlp-policy.md
Service: power-platform
Sub-service: admin
GitHub Login: @jimholtz
Microsoft Alias: jimholtz

@binaryjam
Thanks for the suggestion. We'll incorporate this change soon.


fix bugs turned up on sierra testing

fix bug surfaced on sierra with reading cache files
improve error messages
fix so dsync will create target directory if it doesn't exist

@dsikich , can you think whether these fixes (especially the one for dsync) will trip up anything, or do we need more like them?

the second fix here is so that someone can do "dsync src dest", where the dest directory does not exist yet, it should now be created

@adammoody I can't think of anything that would trip up. I know dcp already creates the dest dir if it doesn't already exist. I don't think any other tool will need that though?


What is the default clock source for io_uring_prep_timeout

It's literally in the man page, if you don't set IORING_TIMEOUT_ABS then a relative clock is used.

It's literally in the man page, if you don't set IORING_TIMEOUT_ABS then a relative clock is used.

So it's MONOTONIC by default?

It's literally in the man page, if you don't set IORING_TIMEOUT_ABS then a relative clock is used.

So the default is MONOTONIC?

Right, monotonic and relative.


DID Key Upgrade

This PR:

[ ] Updates contexts since verification methods were removed from did core
[ ] Updates keypair classes to be more standard
[ ] Updates JOSE crypto interfaces

blocked pending resolution of @transmute/ed25519-signature-2018 bug.

PR addressing the suite issues: https://github.com/transmute-industries/verifiable-data/pull/64


Unable to get it running basic example, files missing

Hello,
I'm unable to get it running basic example as some files are missing.
Can you provide following files and example to use into an existing vue project.
import OutputDisplay from './OutputDisplay'
import FormText from './form-elements/FormText'
import FormSelect from './form-elements/FormSelect'
import FormCheckbox from './form-elements/FormCheckbox'
Thank you

Hello, @scramatte
The components used in the examples are only for example purposes. FormVueLatte does not provide form components as it is not a component library, but it enables you to use your own :)
If you need some base components I recommend you check out Vuetify or some similiar component library.

This is what I've done, I've followed you Blog post  and I make my component for bootstrap-vue. I was unable to get vulidate working so I've tried  your blog
https://www.vuemastery.com/blog/Validating-Schema-Driven-forms-with-Vuelidate/ and it works fine.

I'm glad it worked for you! Going to close the issue as it is not related to the lib


React/Redux Group Project

In this pull request my partner @shahnawaza75 and I have implemented the requirements for this project as follows:

sent HTTP request to the API endpoints provided and this only happens once the page is loaded.
set the rocket page as the default landing page.
data is fetched from the Space X API once the page loads and no additional data is fetched.
rendered the rocket and mission pages as per design.
when a user clicks the Reserve Rocket button, an action is dispatched with the ID of the selected rocket to update the store.
likewise, when the Join Mission button is clicked, an action is dispatched with the ID  of the selected mission to update the store.
rockets that have already been reserved have a Reserved badge with a Cancel Reservation button, also mission that has been joined shows an Active Member badge with a Leave Mission button.

Thank you for the feedback


Update Home page

/botio build

From: MushinBot (botio-academy.horizen.global)

Received
Command cmd_build from @tuanhvdev received. Current queue size: 0
Live output at: https://botio-academy.horizen.global/7910b5b3b5ff0d1/output.txt

From: MushinBot (botio-academy.horizen.global)

Success
Full output at https://botio-academy.horizen.global/7910b5b3b5ff0d1/output.txt
Total script time: 1.81 mins
You can download the generated site at: https://botio-academy.horizen.global/latest.tar.gz
Or view a preview at https://devweb-academy.horizen.global

/botio build

From: MushinBot (botio-academy.horizen.global)

Received
Command cmd_build from @tuanhvdev received. Current queue size: 0
Live output at: https://botio-academy.horizen.global/30de0b1a185b2ca/output.txt

From: MushinBot (botio-academy.horizen.global)

Success
Full output at https://botio-academy.horizen.global/30de0b1a185b2ca/output.txt
Total script time: 1.88 mins
You can download the generated site at: https://botio-academy.horizen.global/latest.tar.gz
Or view a preview at https://devweb-academy.horizen.global


Thanks! And a note!

You can close this, but I wanted to provide some additional context on the PRs submitted (#7 and #8).
The Python Software Foundation is currently doing Privacy and Data policy work, which necessitates creation of data maps for our various services. We run a lot of Django.
Our end goal is a DBML -> Fides Dataset based approach to minimize the amount of work that contributors need to do.
That necessitates verbatim table names, and the default support is just for prettier documentation. See PyPI and Python.org for example.

Hi @ewdurbin, I hope everything is going well with you.
I would like to know how the project is going and if there is anything else I can help with.


temporary release of correlator app to help send events into channels

This is needed for demos and discussions because it's difficult to hit channel endpoints from outside the cluster.
Suggested repo: projectriff/correlator

repo now available: https://github.com/projectriff/correlator
will make a few updates and then push to the projectriff docker hub org

first image pushed:
https://hub.docker.com/r/projectriff/correlator/tags/
looking forward to contributions


[Cosmos DB] Suppress output (resourceBody) for successful batch operation

Package Name: @azure/cosmos
Package Version: 3.9.1
Operating system: MacOS
[X] nodejs

version: 10.16.3

[ ] browser

name/version:

[ ] typescript

version:

Is the bug related to documentation in

[ ] README.md
[ ] source code documentation
[ ] SDK API docs on https://docs.microsoft.com

Describe the bug
Not really a bug but more of a clarification question. When I perform a batch/bulk operation (say Upsert), for all the documents that returned 200 status code, the document content is echoed back in resourceBody:
{
    statusCode: 200,
    requestCharge: 12.952380952380953,
    eTag: '"2f0079c7-0000-0700-0000-5f5b90960000"',
    resourceBody: {
        "FirstName": "Blah",
        "LastName": "Blah",
        "ZipCode": "22222"
    }
}

I am wondering if there's a parameter that I can pass to the operation which essentially does not echo back the resource body? I would very much like the response to look like:
{
    statusCode: 200,
    requestCharge: 12.952380952380953,
    eTag: '"2f0079c7-0000-0700-0000-5f5b90960000"',
}

Hey @gmantri, at the moment there is no way to suppress the resouceBody, but one option would be to remove it in some kind of call after the response, like
responseArray.map({ resourceBody, ...rest }) => rest)

I have a second round of work planned for Bulk and I can add this for consideration but there's no plan at the moment to implement something like this within the SDK.

Thanks @zfoster. That is certainly an option but it will not solve the purpose. Essentially I am looking at a server-side operation where I could instruct the API not to send the resource body. This is just to reduce the network traffic and costs considering we pay for egress traffic. To give you an example, please look at Prefer request header in Storage Table Insert Entity operation: https://docs.microsoft.com/en-us/rest/api/storageservices/insert-entity#request-headers.

Thanks for the reply @gmantri. Cosmos has support for Prefer in the request headers as well, I believe we support it in places in the SDK. I can look into if it already works with Bulk or what we need to do to get it working. Thanks again!

Thanks @zfoster! I'll close this issue now. Hoping to see the support for this in the SDK soon. I do have a clarification question about the bulk operations (basically trying to understand the behavior). I will create a separate issue for that.


HIVE-27412: Vectorization: VectorMapOperator does not reset VectorizedRowBatch after processing

This is related to this commit - https://github.com/apache/hive/commit/bc6c3cf66f096acf842c147a547a9088d0645c64
In the commit, vector is not reset when read type is VECTORIZED_INPUT_FILE_FORMAT
That also has the same issue of MapVector and ListVector growing infinitely.

@maswin there are several test failures due to the change, could you please check http://ci.hive.apache.org/blue/organizations/jenkins/hive-precommit/detail/PR-4389/1/tests
and update the PR accordingly, Thanks

@maswin there are several test failures due to the change, could you please check http://ci.hive.apache.org/blue/organizations/jenkins/hive-precommit/detail/PR-4389/1/tests
and update the PR accordingly, Thanks

Sorry, didn't realize vectors were reused. Fixed it.

@abstractdog can you please help review this?

@abstractdog - The same test (expanding_multivaluecolvector.q) when run in TestCliDriver will go through this code flow. Right now it is run in TestMiniLlapLocalCliDriver. But even then the test won't fail or timeout, because the file is not huge enough. This seems to a big memory leak, couple of ORC files internally we have runs forever in vectorized mode when they get over in few seconds in non vectorized mode. Also a lot of queries that run using 1GB memory in non vectorized mode needs 1.5-2GB memory.

@deniskuzZ - Maybe the issue is the reset is not happening in the reader and that is where the fix needs to be.
I did a reset similar to what happens in ParquetRecordReader and raised a PR - https://github.com/apache/hive/pull/4415
I tested the dataset that had issue with memory leak and it seemed to work with reset happening in ORCReader.

@deniskuzZ - Maybe the issue is the reset is not happening in the reader and that is where the fix needs to be.
I did a reset similar to what happens in ParquetRecordReader and raised a PR - #4415
I tested the dataset that had issue with memory leak and it seemed to work with reset happening in ORCReader.

great, should we then abandon the current PR and merge the #4415

great, @maswin should we then abandon the current PR and merge the #4415?

Sure. Closing this one.
Following issue fixed in this PR - https://github.com/apache/hive/pull/4415


Add Disk Space Check to Readiness Check

Is your feature request related to a problem?
The pan-os-upgrade-assurance package includes a check for free disk space.  This should be incorporated in the default readiness check options.
Describe the solution you'd like
Add 'free_disk_space' to the default readiness check.
Describe alternatives you've considered
Another option would be to add the target software version to the readiness check parameters.  But that would also require defining the options in the playbook where they're currently using the defaults.
Additional context
When there isn't enough free disk space, the upgrade can fail without providing an intuitive error message.  Ideally this check would be included to provide an opportunity to resolve the disk issues before starting the actual upgrade.

After looking, the disk space check is included, but only if the candidate version parameter is passed.  In the playbooks, that value is set to "panosnetworkoperationsupgradetargetfirewallversion".  But that value is only set in the batch incidents and not carried over to the device upgrade incidents.  The field set in the device upgrade incidents is "panosnetworkoperationsupgradetargetversion".  Should also include the default free_disk_space if not candidate version is included.


styles minification

Hi guys, is there any documentation related to styles minification auto prefix for class for production?

I don't there's any configuration to turn on these two features.

I have find the example I need, thanks for the great work.

@vincent178 what exactly have you found?

Hi @royeeshemesh , I believe I found the solution at that time, but I can't find the code any more, now I turn to use styled-components, I already use it for some time, it's pretty good.


Improve or re-design website

Is your feature request related to a problem? Please describe.

No
Describe the solution you'd like

The look and design of the current website can be improved. I would like to work upon that, either make changes to the current one or do a re-design. I wanted to get started doing that but i was not able to find the codebase for the website.
Describe alternatives you've considered

Additional context

Backporting

[ ] Needs backporting to older releases

Thanks for the offer! At the moment the docs are still entangled with the main Rancher docs, and the website itself is in a private repo and hosted on Rancher infra. We'll be renewing our focus on disentangling k3s from suse/rancher over the next few months.

Great, would love to contribute!

docs have moved.


Added Threat Modeling Tool Release Notes

Learn Build status updates of commit 61f23ac:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/security/develop/threat-modeling-tool-releases.md
:white_check_mark:Succeeded

articles/security/develop/threat-modeling-tool-releases-73209279.md
:white_check_mark:Succeeded

articles/security/develop/TOC.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@mgchaitanyababu @jegeib  - this content should be submitted for review via the private repo azure-docs-pr ... can you do that?

ideally you'd open a new PR in the private repo with these changes
however, if you need assistance moving the commits in this pr, we can help with that after this pr is 100% approved.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@Jak-MS, I was not able to find the repo "azure-docs-pr". Can you please point to the repository.
Thanks,
Chaitanya

Hi @mgchaitanyababu,
The private repo is here: https://github.com/MicrosoftDocs/azure-docs-pr.

Hi @Court72 ,
I am getting 404 error. Do I need any permissions to access the repo? Thanks a lot for offering to move the commits but I would prefer to set up the flow and not disturb you for every release. Can you please help us in getting access to the repo.
Thanks,
Chaitanya

Hi @mgchaitanyababu,
Yes, it's likely a permissions issue. You need to join the Microsoft Docs org in order to work in the private repo.
Here are some instructions: How to get to the private repo.

@Court72 , I have raised a PR to the private repo: https://github.com/MicrosoftDocs/azure-docs-pr/pull/217610. Please guide me with the next steps to get the docs published.
Thanks,
Chaitanya

@mgchaitanyababu
Great! Add a #sign-off comment to that PR (https://github.com/MicrosoftDocs/azure-docs-pr/pull/217610), and a PR reviewer will review it shortly.
Thanks.


_CSharpWriteProjectMetadataSources property/information

The current source generator provides the location of the ProjectPath for the aspire enabled project; please consider adding addition project information about the project such as namespace, assembly and build output location so that these details may be used/accessed by extension methods/helpers which may then injected/copy or mount folder onto a prepared container; like Amazon/Lambda/dotnet
example usage: builder.AddAmazonWebServiceLambda<Projects.Lambda>();
example usage: builder.AddAzureFunctions<Projects.Function>();
the about would result in the creation of a container with a volume mapping and the ENTRYPOINT/CMD defined to point to the project build output folder, assembly and method supplied.

Some of these things require full MSBuild evaluation to determine correctly, and as such likely shouldn't be hoisted onto the generated types as they can't be guaranteed to be correct. Instead, scenarios that require knowing the result of MSBuild operations (like project build or publish) should be retrieving those values via an MSBuild operation, e.g. this is what the Azure Developer CLI does today.

I'd love to be able to "inject a target" into a referenced project so that we could execute some MSBuild logic to hoist information we want into the AppHost project at build time.


Feature-SD-SH-001-Issue

Registration (deregistration) of a service when starting (stopping)

Identifier
Feature-SD-SH-001

Description

Priority
SH

Authorised users

Inputs

Outputs

Basic Service for service discovery implemented, not yet HTTPS secured though.


Sensor maps the NodeInventory to Node

Description
A detailed explanation of the changes in your PR.
Feel free to remove this section if it is overkill for your PR, and the title of your PR is sufficiently descriptive.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
TODO(replace-me)
Use this space to explain how you tested your PR, or, if you didn't test it, why
you did not do so. Valid reasons include, for example, "CI is sufficient",
"No testable changes". Feel free to attach JSON snippets, curl commands,
screenshots.
In addition to reviewing your code, reviewers must also review your testing
instructions and make sure they are sufficient.

Current dependencies on/for this PR:

master

PR #3800

PR #3755

PR #3757

PR #3892   👈

This comment was auto-generated by Graphite.

Closing in favor of https://github.com/stackrox/stackrox/pull/4484


⚠ Agora Aveiro has degraded performance

In eb2f75b, Agora Aveiro (https://agoraaveiro.org) experienced degraded performance:

HTTP code: 200
Response time: 11817 ms

Resolved: Agora Aveiro performance has improved in 443f77a after 8 minutes.


Added live example

This PR adds configuration to run and code on plot neural net in Gitpod, a free dev environment for GitHub we have been working on. The goal is to make contributions super easy by providing a single click to code experience. You can open gitpod workspaces on any github repository by prefixig the github URL with 'gitpod.io#/'. Depending on the URL it does the right thing. For instance, prefixing a pull request like this one will open it in code review mode.
For your project I added a docker with the needed dependencies and made the start script so that it runs test/unet.py and opens the resulting pdf. Also the python editor is opened so that people can play around with it.
I hope you find it useful, let me know we you have any questions.

It seems that your code does not load docker created for this repository out of the box. Would be nice if you add some guidelines or make it run out of the box.

How did you try it?
If you start a gitpod on this branch (or my fork) the Dockerfile will be picked up:
https://gitpod.io/#https://github.com/HarisIqbal88/PlotNeuralNet/pull/15


Test cat_years_to_hooman_years

Adding tests for cat_years_to_hooman_years. This is part of the first exercise of the testing tutorial from Pycon 2016.

Coverage increased (+1.1%) to 37.931% when pulling 8b4359cbc418b861e461cbdfd333d9be387a2003 on jlcoto:test-tutorial into d5fd52c818b07dcac4abe7eb852a216c80b6fde2 on keeppythonweird:master.


[Bug]: Android: Folders to ignore stops decks from opening

Describe the bug
If we use the "Convert folders to decks and subdecks option" and add anything in the "Folders to ignore" list, whenever we try to review all the flashcards, nothing happens.
To Reproduce

Activate "Convert FOlders to decks and subdecks"
Add some folders in "Folders to ignore"
Review flashcards
3a. From the bottom bar click "Review flashcards"
3b. From the command palette use "Spaced Repetition: Review flashcards from all notes"
Nothing appens

Expected behavior
With step 3a or 3b the decks should appear, but nothing happens
Screenshots
No response
OS of your device
Android
Obsidian version
v1.6.7
Plugin version
v1.12.25
Installer version
No response
Additional context
No response
Config file
No response

Hello, this Issue has been fixed and is available as a beta release at:
Beta release of outstanding PRs #1011
If you are able to try that, and let us know if that solves your issue that would be appreciated.
Cheers
Ronny

Can confirm, it has been fixed in the beta.

Thanks


To file the addition new dependency to support WAMR

Title: Add new dependency to support WAMR runtime build
Description:

Describe the issue.
Add WAMR - https://github.com/bytecodealliance/wasm-micro-runtime
[optional Relevant Links:]
Any extra documentation required to understand the issue.

Located at https://github.com/envoyproxy/envoy/pull/16057#discussion_r616297480

Finish the dependency.


Reference summary on approvals

Added a reference summary to help the user get information about the object they are approving without having to click view Reference so that they could filter requests out more easily with the table and have an idea of what they are looking at from the getgo.

@tharmaman .. can you add a toSummary method to UCJs which proxies to the toSummary of the sourceId or else this PR won't really help for any of the UCJ approvals.

@tharmaman .. can you add a toSummary method to UCJs which proxies to the toSummary of the sourceId or else this PR won't really help for any of the UCJ approvals.

Yeah so RE: np side, I had to manually call  .toSummary on the user/business where it's instantiated, proxying the toSummary of UCJ will only work if moving forward we always want ucj.toSummary to proxy only to the sourceId and not actually be a toSummary of the UCJ itself


Support disabling the creation of Request-Context header via configuration

Repro Steps

Create ASP.NET Core project and enable Application Insights.

Actual Behavior
Request-Context HTTP header gets set for every (except first?) request.
Expected Behavior
Ability to disable the creation of Request-Context header via configuration or other method.
Version Info
SDK Version : 2.2.1
.NET Version: 2.0.5
Workaround
Custom middleware that removes the header.
            app.Use((context, next) =>
            {
                context.Response.Headers.Remove("Request-Context");
                return next();
            });

Motivation
APIs for Industrial Internet and other scenarios where the bandwidth and client resources are limited and should be used sparingly.

https://github.com/Microsoft/ApplicationInsights-aspnetcore/wiki/Dependency-Tracking-and-Performance-Counter-Collection#configuring-telemetry-module-services - Please let me know if this wiki information is useful and can help you.

Sorry I misread the issue.
As of now there are no config options to achieve this. We are working on config story for aspnet core sdk, and will take this one as part of it.

This was accidently closed. Need to be addressed along with this:
https://github.com/Microsoft/ApplicationInsights-aspnetcore/issues/650

Fixed in #716, set ApplicationInsightsServiceOptions.RequestCollectionOptions.InjectResponseHeaders to false to disable response header injection.


feat: add a contract address partialord implementation

Feature Request
Add the ability to compare the numerical value of contract addresses to the corelib
e.g.
use core::integer::{u256_from_felt252, u128_wrapping_sub};
use core::traits::{Into};
use starknet::{ContractAddress, ContractAddressIntoFelt252};

// Allows comparing contract addresses as if they are integers
impl ContractAddressOrder of PartialOrd<ContractAddress> {
    #[inline(always)]
    fn le(lhs: ContractAddress, rhs: ContractAddress) -> bool {
        u256_from_felt252(lhs.into()) <= u256_from_felt252(rhs.into())
    }
    #[inline(always)]
    fn ge(lhs: ContractAddress, rhs: ContractAddress) -> bool {
        u256_from_felt252(lhs.into()) >= u256_from_felt252(rhs.into())
    }

    #[inline(always)]
    fn lt(lhs: ContractAddress, rhs: ContractAddress) -> bool {
        u256_from_felt252(lhs.into()) < u256_from_felt252(rhs.into())
    }
    #[inline(always)]
    fn gt(lhs: ContractAddress, rhs: ContractAddress) -> bool {
        u256_from_felt252(lhs.into()) > u256_from_felt252(rhs.into())
    }
}

Describe Preferred Solution
Add the above code to the core lib
Describe Alternatives
Create an intern function for comparing contract addresses
Related Code
We use this in Ekubo's core contracts, and most AMMs sort token addresses to determine a token0 and token1

@orizi I can PR this if you agree, we need it too


Atom installs its own tern

The setup section in the readme mentions that you'll have to install Tern and copy openui5.js to the plugins directory. However, atom-ternjs installs its own version of Tern. It is not necessary to install it seperately. The openui5.js should obsviously still be copied, but to a tern/plugins directory under the users' ~/.atom directory.

You are absolutely right.
What the readme is trying to say is that you need to install Tern, one way or another. The Atom specific section is simply suggesting a way how to easily get a Tern installation that integrates well with Atom.
I can see how the current description is confusing, though -- I will have another look at it over the next few days and try to find a better way to phrase it.

Thanks for looking into this Timo. You’re right, It put me on the wrong leg
as I am (was) not familiar with Tern. Do let me know if you’re open for a
contribution in this area. I don’t mind putting my copywriting skills to a
test ;)
Perhaps a bit off-topic: I also noticed that the webcrawler currently
breaks on a documentation page in which > and < characters weren’t encoded
properly. In order to fix it in the crawler, I’m afraid it would be
necessary to open the bonnet of cheerio. Instead, I have asked SAP to look
into the page and hope they’ll fix the invalid html-encoding on that page
instead: https://github.com/SAP/openui5/issues/875
Excellent initiative Timo! Love it!
Cheers,
Jan
On Thu, Feb 25, 2016 at 8:22 PM, Timo M. Staudinger <
<EMAIL_ADDRESS>wrote:

You are absolutely right.
What the readme is trying to say is that you need to install Tern, one way
or another. The Atom specific section is simply suggesting a way how to
easily get a Tern installation that integrates well with Atom.
I can see how the current description is confusing, though -- I will have
another look at it over the next few days and try to find a better way to
phrase it.
—
Reply to this email directly or view it on GitHub
https://github.com/TimoSta/tern-openui5/issues/5#issuecomment-188939219.

Feel free to contribute to this project -- I would love to make this a community effort! It's great to see that I'm not the only one using this plugin. :)
Regarding your second point, I haven't even noticed that one yet.
I opened issue #6 for it to keep it separate from this documentation issue.
Thanks!
Timo

@jpenninkhof I had a go at the docs, hope that clarifies things a bit.
I also described how to install the plugin in Atom directly via npm, getting rid of the copy-paste magic. :-)