sap.m.UploadCollection add "random" request headers

OpenUI5 version: 1.44.9
Browser/version (+device/version):
Chrome 48.0.2564.116 (64-bit)
Any other tested browsers/devices(OK/FAIL):
Safari OK
URL (minimal example if possible):
User/password (if required and possible - do not post any confidential information here):
I have written a web app for my company that incorporates sap.m.UploadCollection.
The target of the upload is a system at a different URL.
When I try to start the upload it fails on Chrome with this reason...
"Request header field requestId1496864333304 is not allowed by Access-Control-Allow-Headers in preflight response"
UploadCollection is adding the "requestId1496864333304" and "fileNameRequestId1496864333304" as request headers
But since the numbers at the end are "random" my server cannot respond with an appropriate "Access-Control-Request-Headers" reposes to the pre-flight request from Chrome.
There for the upload fails.
What is the expected result?
The file uploads
What happens instead?
Error message from Chrome
These headers either should not be sent or I should be able to control if they are sent.
If the headers are added by the browser itself then I'll have to find a workaround.

Hello @DeanDavis ,
Thank you for sharing this finding. I've created a internal incident<PHONE_NUMBER>. You can check its status on GitHub.
Regards,
Diana

Hello @DeanDavis,
I'm Oli from the responsible develeopment team. The intension of these parameters is only internally by the  control itself. We need the parameters to handle progress indication, aborting / cancleing of request and error handling.
To solve your backend issue, you can add the following header parameter via the uploadcollection control.
https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Access-Control-Allow-Headers
The control creates there request headers as constants. You can access this via UploaCollectionInstance._headerParamConst property.
I will provide a fix for this as well, as a quick fix you can try out the above idea. Feedback is welcome :-)
Best regards,
Oli..

Thanks Oli,
I understand the need so I have implemented CORS on my backend to reflect the headers you are sending as allowable.
I can now upload files.
But I have another issue and this may not the the correct forum to ask this question, if not please directly to a place where I can ask about this issue.
I cannot get FileUploader to display the progress of the upload.
The items aggregation is bound to my model and I'm using "instant upload" to add a file.
In the example for FileUploader, right when the upload starts a line is added to the list and you can see the progress of the upload.
This doesn't happen for me.
I can see the file upload using my developer tools but an entry is only added to the list when I receive the "uploadComplete" event and I update the model with the new entry.
My backend is IIS8.5 and the file upload is being handled by a Node.js application.
Any suggestions?

Hey @DeanDavis,
nice to hear that you can now upload your files :-)
The UploadCollection exposes the change event in case a file is selected for upload. Within the event handler you should register to, you need to invalidate the UploadCollection by calling: UCinstance.invalidate();
This is not intendet to be needed but because of backwards compatibility we cannot change this behavior. If you invalidate the UploadCollection control you should be able to see the progress indicater.
If not, feel free to come back :-)
Best regards,
Oli..

Brilliant, that worked. Thanks.
I would have never figured that on my own and spent a day and a half trying to figure out how to get it to work.
Is there a proper user forum with experts like yourself where a question like mine, when I run into a roadblock, is better asked? I'd don't like filing a report in GitHub unless I think it is a bug or something that doesn't make sense.
And when I see expected behavior working in the example program but not working in my program I tend to believe it is my issue and wouldn't usually file a bug report just to get tech support.
Thanks again very relived to get this working.

@DeanDavis,
nice to hear again that it is working now :-)
I forwared you last question about a kind of forum because I'm not aware of something, but maybe there is a place.
I will post a comment again when I get an answer.
Mayn greetz,
Oli..

Thanks.
One last comment then. I see a lot of English text built into this control.
How can I localize it to different languages?

See the tutorial for translation :-)
https://openui5.hana.ondemand.com/#docs/guide/df86bfbeab0645e5b764ffa488ed57dc.html

Hoi again :-)
There are 2 places where you can discuss and ask in communities.

slack.com/openui5
https://stackoverflow.com/questions/tagged/openui5

Doei,
Oli..

Hi,
I have the exact same issue as @DeanDavis.
I am trying to upload a file through the sap.m.UploadCollection to an external system running on another system.
I've sniffed the traffic with Fiddler and I see that the "Access-Control-Request-Header" is set with the following values:

authorization
content-type
filerequestid1500xxxx
requestid1500xxxx

Honestly, I didn't understand the solution mentioned in one of the prior posts.
As far as I understood the solution, @DeanDavis passed these headers from the backend to the client?
As I have no influence on the external I cannot adapt the behaviour of the CORS implementation on the backend.
Is there any solution, how I can upload through the sap.m.UploadCollection to that external system?


Sortable not working with Array of Objects

I have here a CodeSanbox  to show the issue I have, One Grid is with an Array of Objects which have id & title but handleDragEnd does not work here for some reason and the items jump on drag end to the starting position back.
But when I use an Array with just numbers it works fine and handleDragEnd moves the Array correctly.
What am I missing here and doing wrong? Thanks

Inside your setItems() function call inside of the handleDragEnd() handler, you are not correctly getting the index of the matching items.
Try the following instead:
// const oldIndex = items.indexOf(active.id);
const oldIndex = items.findIndex(item => item.id === active.id);
//const newIndex = items.indexOf(over.id);
const newIndex = items.findIndex(item => item.id === over.id);

Thanks you, it works now :) And Thank you for this library, hope the Examples will improve I will try to help where I can.
The next problem now is in my case the performance is very bad and the drag experience laggy, What I have in my app is an array of arrays each array is a page (Swiper) and each swiper page has it's own DndContext & SortableContext, I use useMemo where I can but somehow it feels like it just takes the lib too long to compare the id's, is there any way I can improve this?
const { data } = useQuery([queryKey, { id }], getData);
// chunk the big array to get array of arrays for pagination
const pages = useMemo(() => chunk(data, perPage), [data]);

return (
 <Swiper>
          {pages?.map((page) => (
            <SwiperSlide
              key={JSON.stringify(page)}
              className={`grid auto-rows-[max-content] unset-slide-styles`}
            >
              <SortableStateContainer page={page}> // in this wrapper I set the items because it's paginated
                {(
                  items: any[],
                  handleDragEnd: ((event: DragEndEvent) => void) | undefined
                ) => (
                  <DndContext
                    collisionDetection={closestCenter}
                    onDragEnd={handleDragEnd}
                  >
                    <SortableContext items={page.map((i) => i.id)}>
                      {items?.map((item: any) => (
                        <SortableItem
                          key={item.id}
                          id={item.id}
                          title={item.title}
                          url={item.url}
                          context={context}
                        />
                      ))}
                    </SortableContext>
                  </DndContext>
                )}
              </SortableStateContainer>
            </SwiperSlide>
          ))}
        </Swiper>
)
```

@ivanjeremic Use Virtualization for long lists, like https://virtuoso.dev/

@ivanjeremic Use Virtualization for long lists, like https://virtuoso.dev/

I feel like this is not the issue because dnd-kit has nothing really expensive to do, I do the chunking upfront and the biggest array a single SortableContext gets is around 40 items which is not really a big thing. Hmm

This can happen if your component is expensive to re-render. Try creating a separate presentational component that you are connecting to useSortable and memoize it using React.memo to avoid un-necessary re-renders.
Read more about this pattern here:
https://docs.dndkit.com/api-documentation/draggable/drag-overlay#presentational-components

This can happen if your component is expensive to re-render. Try creating a separate presentational component that you are connecting to useSortable and memoize it using React.memo to avoid un-necessary re-renders.
Read more about this pattern here: https://docs.dndkit.com/api-documentation/draggable/drag-overlay#presentational-components

Browser devtools where the issue, when devtools are open then it becomes laggy. Thanks


CASSGO-28 ignore error retry type

Closes https://github.com/apache/cassandra-gocql-driver/issues/1808

Can you add a test to ensure that the issue is actually fixed? And to prevent regressions.

This branch has no conflicts with the base branch when rebasing

Hello! OK I will try to get to it on this weekend

Can you add a test to ensure that the issue is actually fixed? And to prevent regressions.

Added a test. Wanted to let observer know that error was ignored but observer is called before retry policy is checked.
there is one more problem - error will not be ignored if we have no more attempts. Added a fix&test for this too

We need another committer +1 to merge this.

You mean anyone who've made a commit or apache employee ?

We need another committer +1 to merge this.

You mean anyone who've made a commit or apache employee ?

Committers are people who have write access to the codebase, a PR needs two +1s from committers to get merged ideally and I'm a committer so we need one more.

LGTM - just needs conflicts to be resolved.

just needs conflicts to be resolved

Resolved conflicts with rebase

Please also add Jackson as a reviewer to the commit message and I'll merge this afterwards


all the unused "use" have been removed

Found and removed all unused "use" as  the second step of last pull request (from both src and tests folders).
Now there should be no more unused "use".

Merged in https://github.com/mongodb/mongo-php-library/commit/942af4edc26daf66d48af0689167322bb68cf110 with the commit amended to include the necessary use statements. Thanks!


How to correctly speech recognise from an audio file?

I got the example that transcribes my voice direct from the microphone to work, but now I want to transcribe a file.
I tried
using (var recognizer = new SpeechRecognizer(config, AudioConfig.FromWavFileInput(@"/Full/Path/To/File.wav")))
{
    recognizer.Recognizing += (s, e) =>
    {
        Console.WriteLine($"RECOGNIZING: Text={e.Result.Text}");
    };

    recognizer.Recognized += (s, e) =>
    {
        if (e.Result.Reason == ResultReason.RecognizedSpeech)
        {
            Console.WriteLine(e.Result.Text);
        }
        else if (e.Result.Reason == ResultReason.NoMatch)
        {
            Console.WriteLine($"NOMATCH: Speech could not be recognized.");
        }
        else if (e.Result.Reason == ResultReason.Canceled)
        {
            var cancellation = CancellationDetails.FromResult(e.Result);
            Console.WriteLine($"CANCELED: Reason={cancellation.Reason}");

            if (cancellation.Reason == CancellationReason.Error)
            {
                Console.WriteLine($"CANCELED: ErrorCode={cancellation.ErrorCode}");
                Console.WriteLine($"CANCELED: ErrorDetails={cancellation.ErrorDetails}");
                Console.WriteLine($"CANCELED: Did you update the subscription info?");
            }
        }
    };

    recognizer.Canceled += (s, e) =>
    {
        Console.WriteLine($"CANCELED: Reason={e.Reason}");

        if (e.Reason == CancellationReason.Error)
        {
            Console.WriteLine($"CANCELED: ErrorCode={e.ErrorCode}");
            Console.WriteLine($"CANCELED: ErrorDetails={e.ErrorDetails}");
            Console.WriteLine($"CANCELED: Did you update the subscription info?");
        }
    };

    recognizer.SessionStarted += (s, e) =>
    {
        Console.WriteLine("\nSession started event.");
    };

    recognizer.SessionStopped += (s, e) =>
    {
        Console.WriteLine("\nSession stopped event.");
        Console.WriteLine("\nStop recognition.");
    };

    await recognizer.StartContinuousRecognitionAsync();
}

But I never get the file's transcript, the console prints only "Session started" and then exits.
How can I transcribe an audio file with Microsoft.CognitiveServices.Speech?

Please take a look here -- https://github.com/Azure-Samples/cognitive-services-speech-sdk/blob/d79a4fa67017591fb0b6ab31e70ad9bf2a0d7d2f/samples/csharp/sharedcontent/console/speech_recognition_samples.cs#L171-L243
StartContinuousRecognitionAsync() merely starts to recognize, and then you're removing the recognizer.
You'll need to wait for your results, then eventually to StopContinousRecognitionAsync() and/or dispose.
Hope this helps, Mark

I will close the issue with the current resolution, let us know if you need more information.


Cannot read property 'bool' of undefined

React version: 16.6.3
prop-types version: 15.6.2
Uncaught TypeError: Cannot read property 'bool' of undefined
    at eval (VM66354 PulseLoader.js:37)
    at Object../node_modules/halogen/PulseLoader.js (VM65913 bundle.js:3810)
    at __webpack_require__ (VM65913 bundle.js:725)
    at fn (VM65913 bundle.js:102)
    at eval (VM66353 Loading.js:10)
    at Module../src/components/Loading/Loading.js (VM65913 bundle.js:5746)
    at __webpack_require__ (VM65913 bundle.js:725)
    at fn (VM65913 bundle.js:102)
    at eval (VM66348 Handler.js:11)
    at Module../src/containers/Extractors/Handler/Handler.js (VM65913 bundle.js:5919)

The same problem

okay as the repo is inactive for a long long time lemme fork it and upgrade the same fixing some of the issues


Authenticate requests to spine-dev database

This PR is a part of https://github.com/SpineEventEngine/web/pull/43.
The spine-dev Firebase database rules have changed and now it is required to authenticate all write requests to its REST API.
For the authorization, we use service account file spine-dev.json, which is already present in the project.

@armiol PTAL.


fixed multi connect and  namespace

fixed multi socket and  namespace

Coverage decreased (-0.2%) to 93.844% when pulling b3ee761ddcec5b859594421fbc138e9b3bf2078a on ckken:master into 51dd66e9cd577353d06a42f6070ef01348a44106 on 75team:master.


How to estimate the required video memory?

when i adjust processor.image_processor.size  from {'height': 960, 'width': 720} to higher,  GPU said CUDA out of memory..
my Video Card 8GB video, i wanna know, if i wanna use {'height': 2560, 'width': 1920}, how much VRAM will be used? does 16GB ok ?

More input size means more GPU, keep in mind that batch size and other parameters affect the same so as per your requirement 16 GB would be enough i guess what happened when you tried using 16GB?


Added support for receiving non-string messages

CloudAgent currently assumes uses the BrokeredMessage.GetBody<_>() function to get messages from Azure Service Bus queues. This function uses a DataContractSerializer to construct the message body. https://msdn.microsoft.com/en-us/library/azure/hh144211.aspx . However in some cases (e.g. receiving service bus messages that are sent from a python client), the DataContractSerializer doesn't work and a different XmlObjectSerializer is needed. I exposed that option and temporarily called it a "WireSerializer" to distinguish it from ISerializer objects that convert a string to whatever object the user wants. A better name should probably be used.

Nice! I am actually hoping to add some more configuration options etc. at some point anyway regarding serialization.


Feature: Automatically reload page when content script changes

Summary
Currently when I edit content.js each time I need to reload the browser in order for the content code to run again.
I'd like the plugin to reload the page for me so I don't have to press CTRL + R 100000 times during my workday.
Is your feature request related to a bug/issue?
No.
Alternatives
Pressing CTRL + R 100000 times :)
Additional Context
N/A

You can use Extensions Reloader Google Chrome extension.. that I'll save your lot's of time

The page itself should be reloaded in addition to extension. Extension is being reloaded right now so that's good.


[bug] A promise was rejected with a non-error

When using Promise based order creation, and if it gives error, Node JS will print a warning in console "a promise was rejected with a non-error".
This is because the object returned in Promise.reject is not instance of Error. The object is a simple JavaScript Object as below

This issue has been fixed and merged in master, therefore i am closing this issue.
@adarshmadrecha Please let us know if you need any help


CDS-91035-Store Terraform Plan files on Harness Delegate temporarily

https://harness.atlassian.net/browse/CDS-91035

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://661faff5365e33abd8cc3533--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://661fb1325fc4e616df5039c3--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://661fb1e703674422bf20797a--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://661fc8f6365e33c312cbb731--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://661fcad3af8e4a29f0581bed--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://661fccf3fb5859367742464e--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://661fcd6e365e33c7c4cbb699--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://661fce5d8ce25d2dac562275--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66200c31b489d61e0a534080--harness-developer.netlify.app


Support for cfg attributes in host functions definitions

As per title...
If I'm not terribly wrong, before this PR was not possible to define a host function gated by some #[cfg(feature = ...)].  A similar technique has been used by frame.

Usage scenario: we require to early integrate and experiment with an host function that is still in "beta" phase as this one:
https://github.com/davxy/substrate/blob/5d2e2bbf17bb7f375d6d24bfe696be2c6e54aa2a/primitives/io/src/lib.rs#L1146-L1159

I'm not a rust macro wizard 🧙 so please carefully check

It would be nice to have the example of usage in: primitives/runtime-interface/README.md.

Please add it to the crate docs as well (or only the crate docs), but never the README exclusively. Most of these readmes were generated from the crate docs.


Your Readme is incorrect

I believe you are specifying the wrong param for initializes the object.  One should do:
ip = ItunesParser.new(file: "/route/to/the/file.xml")
not
ip = ItunesParser.new(xml_dir: "/route/to/the/file.xml")

true, I'll fix it

Fixed in 51c0e0b9


adding option for lzma compression

Adds option to further compress the client with lzma.
examples of final client size:

File
GOOS
Size

lzma
linux
2.65 MB

lzma_win.exe
windows
3.43 MB

uncompressed
linux
8.94 MB

uncompressed_win.exe
windows
16.12 MB

upx
linux
3.49 MB

upx_win.exe
windows
5.65 MB

Howdy!
Thanks again for the fantastic commit.
My only feedback is could you make this as one statement:
upxArgs := []string{"-qq", "-f", f.FilePath}
if config.Lzma {
  upxArgs  = append([]string{"--lzma"}, upxArgs...)
}

output, err := exec.Command("upx", upxArgs...).CombinedOutput()
if err != nil {
   return "", errors.New("unable to run upx: " + err.Error() + ": " + string(output))
}

I'd do it myself but Im working on something else today!

Thank you!


Self-signed certificates generated with a single openssl req command fail to verify (error 20 at 0 depth lookup: unable to get local issuer certificate)

Summary
When generating self-signed root CA or issued certificates, the openssl verify command fails if the certificate is generated with a single openssl req ... command, but not if using a openssl req ... + openssl x509 ... commands to generate the certificate.
Versions

Working:

OpenSSL 1.1.1, Ubuntu 18.04, openssl req
OpenSSL 1.1.1, Ubuntu 18.04, openssl req + openssl x509
OpenSSL 1.1.1f, Ubuntu 20.04, openssl req + openssl x509

Not working:

OpenSSL 1.1.1f, Ubuntu 20.04, openssl req

Installation Steps
# Ubuntu 18.04
docker run -it ubuntu:18.04 /bin/bash
apt-get update
apt-get install -y curl

# Ubuntu 20.04
docker run -it ubuntu:20.04 /bin/bash
apt-get update
apt-get install -y curl

Reproduction Steps (Not Working)
# Not working on OpenSSL 1.1.1f

# Create a ".rnd" file to avoid warnings
openssl rand -writerand ~/.rnd

# Create a working directory
mkdir -p /tmp/cert
cd /tmp/cert

# Create the root CA private key and certificate
openssl req \
    -new \
    -x509 \
    -nodes \
    -sha256 \
    -newkey rsa:4096 \
    -keyout root.key \
    -out root.cer \
    -days 3650 \
    -subj '/C=CA/ST=State/L=City/OU=POC/OU=Certificate Authorities/CN=POC Root CA' \
    -addext "basicConstraints = CA:TRUE" \
    -addext "subjectKeyIdentifier = hash" \
    -addext "authorityKeyIdentifier = keyid:always, issuer:always" \
    -addext "subjectAltName = DNS:POC Root CA" \
    -addext "keyUsage = digitalSignature, nonRepudiation, keyEncipherment, dataEncipherment, keyAgreement, keyCertSign, cRLSign, encipherOnly, decipherOnly"

# Verify the new root CA certificate
openssl verify -CAfile root.cer root.cer

Reproduction Steps (Working)
# Working on OpenSSL 1.1.1f

# Create a working directory
mkdir -p /tmp/cert2
cd /tmp/cert2

# Create the root CA CSR and private key
openssl req \
    -new \
    -nodes \
    -sha256 \
    -newkey rsa:4096 \
    -keyout root.key \
    -out root.csr \
    -subj "/C=CA/ST=State/L=City/OU=POC/OU=Certificate Authorities/CN=POC Root CA"

# Create the root CA .conf file
cat > /tmp/cert2/root_openssl.conf << EOF
[ v3_attributes ]
basicConstraints     = CA:TRUE
subjectKeyIdentifier = hash
keyUsage             = digitalSignature, nonRepudiation, keyEncipherment, dataEncipherment, keyAgreement, keyCertSign, cRLSign, encipherOnly, decipherOnly
EOF

# Create the root CA certificate
openssl x509 \
    -req \
    -sha256 \
    -signkey root.key \
    -in root.csr \
    -out root.cer \
    -days 3650 \
    -set_serial `date +%Y%m%d%H%M%S%N` \
    -extfile /tmp/cert2/root_openssl.conf \
    -extensions v3_attributes

# Verify the new root CA certificate
openssl verify -CAfile root.cer root.cer

More Information
A more advanced test can be found here:

https://stackoverflow.com/questions/66484609/openssl-verify-error-20-at-0-depth-lookup-unable-to-get-local-issuer-certific

But in the end, just verifying the root.cer is sufficient to show the error.
Any ideas why there is a difference between the two methods for generating self-signed certificates?
Thanks in advance!

I tried your failing command and looking at the resulting certificate the extensions seem quite screwed up with various duplicated extensions:
openssl x509 -in root.cer -noout -text
....
        X509v3 extensions:
            X509v3 Subject Key Identifier:
                36:BC:F6:04:6A:94:46:B9:D9:2F:DA:26:54:7C:83:EA:8F:00:AD:5B
            X509v3 Authority Key Identifier:
                keyid:36:BC:F6:04:6A:94:46:B9:D9:2F:DA:26:54:7C:83:EA:8F:00:AD:5B

            X509v3 Basic Constraints: critical
                CA:TRUE
            X509v3 Basic Constraints:
                CA:TRUE
            X509v3 Subject Key Identifier:
                36:BC:F6:04:6A:94:46:B9:D9:2F:DA:26:54:7C:83:EA:8F:00:AD:5B
            X509v3 Authority Key Identifier:
                keyid:36:BC:F6:04:6A:94:46:B9:D9:2F:DA:26:54:7C:83:EA:8F:00:AD:5B
                DirName:/C=CA/ST=State/L=City/OU=POC/OU=Certificate Authorities/CN=POC Root CA
                serial:0A:0E:D4:AD:4D:BF:4E:AD:AF:C1:59:50:B3:ED:BF:64:B9:B1:34:8A

            X509v3 Subject Alternative Name:
                DNS:POC Root CA
            X509v3 Key Usage:
                Digital Signature, Non Repudiation, Key Encipherment, Data Encipherment, Key Agreement, Certificate Sign, CRL Sign, Encipher Only, Decipher Only
....

Hmm, interesting. That does work on my end as well. That does point out a flaw in the original reproduction where it was not a like-for-like comparison.
However, if we line up the two extension attributes, there is still a failure with the single command:
# Create a ".rnd" file to avoid warnings
openssl rand -writerand ~/.rnd

# Create a working directory
mkdir -p /tmp/cert_error
cd /tmp/cert_error

openssl req \
    -new \
    -x509 \
    -nodes \
    -sha256 \
    -newkey rsa:4096 \
    -keyout root.key \
    -out root.cer \
    -days 3650 \
    -subj '/C=CA/ST=State/L=City/OU=POC/OU=Certificate Authorities/CN=POC Root CA' \
    -addext "basicConstraints = CA:TRUE" \
    -addext "subjectKeyIdentifier = hash" \
    -addext "authorityKeyIdentifier = keyid:always, issuer:always" \
    -addext "subjectAltName = DNS:POC Root CA" \
    -addext "keyUsage = digitalSignature, nonRepudiation, keyEncipherment, dataEncipherment, keyAgreement, keyCertSign, cRLSign, encipherOnly, decipherOnly"

# Verify the new root CA certificate
openssl verify -CAfile root.cer root.cer

# Create a ".rnd" file to avoid warnings
openssl rand -writerand ~/.rnd

# Create a working directory
mkdir -p /tmp/cert_working
cd /tmp/cert_working

openssl req \
    -new \
    -nodes \
    -sha256 \
    -newkey rsa:4096 \
    -keyout root.key \
    -out root.csr \
    -subj "/C=CA/ST=State/L=City/OU=POC/OU=Certificate Authorities/CN=POC Root CA"

# Create the root CA .conf file
cat > /tmp/cert/root_openssl.conf << EOF
[ v3_attributes ]
basicConstraints       = CA:TRUE
subjectKeyIdentifier   = hash
authorityKeyIdentifier = keyid:always, issuer:always
subjectAltName         = DNS:POC Root CA
keyUsage               = digitalSignature, nonRepudiation, keyEncipherment, dataEncipherment, keyAgreement, keyCertSign, cRLSign, encipherOnly, decipherOnly
EOF

# Create the root CA certificate
openssl x509 \
    -req \
    -sha256 \
    -signkey root.key \
    -in root.csr \
    -out root.cer \
    -days 3650 \
    -set_serial `date +%Y%m%d%H%M%S%N` \
    -extfile root_openssl.conf \
    -extensions v3_attributes

# Verify the new root CA certificate
openssl verify -CAfile root.cer root.cer

Given your working example though, does that suggest including -addext "basicConstraints = CA:TRUE" and -addext "subjectKeyIdentifier = hash" in openssl req ... leads to duplicate attributes with one method, but not with the other?
Thanks!

I've stumbled across this problem by generating a certificate with req, trying to add from the command-line the extension
-addext "basicConstraints = critical, CA:true, pathlen:2"```

however, the /etc/ssl/openssl.cnf default file was containing

[ req ]
x509_extensions = v3_ca # The extensions to add to the self signed cert
...
[ v3_ca ]
basicConstraints = critical,CA:true```
I'm seeing at least 2 problems (possible bugs)

I'd expect an explicit command-line parameter/option/extension to override the same option/extension from the config file
openssl verify reports a quite misleading error message (unable to get local issuer certificate), rather than pointing to the actual problem (two CA:true extensions in the certificate, at least as dumped by openssl x509 -text)

System:
$ grep Ubuntu /etc/os-release
NAME="Ubuntu"
PRETTY_NAME="Ubuntu 20.10"
$ openssl version
OpenSSL 1.1.1f  31 Mar 2020```

Thanks.

Please open new issues for the enhancements in regards to the error reporting or other enhancements. The original issue was answered. Closing.


Segfault in master grpc_distribtests csharp_linux_x64_alpine_dotnetcli

The prod:grpc/core/master/linux/grpc_distribtests job has been consistently failing since Jan 14th, dashboard.
+ dotnet exec bin/Debug/netcoreapp2.1/DistribTestDotNet.dll
test/distrib/csharp/run_distrib_test_dotnetcli.sh: line 54:   259 Segmentation fault      (core dumped) dotnet exec bin/Debug/netcoreapp2.1/DistribTestDotNet.dll
+ FAILED=true
+ '[' '' '!=' '' ']'
+ docker rm -f build_and_run_docker_9b106c8b-7a41-4ef3-8a72-99911dd10f5b
build_and_run_docker_9b106c8b-7a41-4ef3-8a72-99911dd10f5b
+ '[' true '!=' '' ']'
+ exit 1

2021-01-15 09:47:48,693 FAILED: distribtest.csharp_linux_x64_alpine_dotnetcli [ret=1, pid=14462, time=52.0sec]

Tracked internally as b/177904713

Tracked internally as b/177904713

I looked at the diff between the last working build and the first broken build and found this PR:
https://github.com/grpc/grpc/pull/25140  I'll need to see whether it could have caused the problem.

I looked at the diff between the last working build and the first broken build and found this PR:
https://github.com/grpc/grpc/pull/25140  I'll need to see whether it could have caused the problem.

I tracked the problem down to https://github.com/grpc/grpc/pull/25140 and created a revert PR: https://github.com/grpc/grpc/pull/25212. I'll comment more on the internal bug.

I tracked the problem down to https://github.com/grpc/grpc/pull/25140 and created a revert PR: https://github.com/grpc/grpc/pull/25212. I'll comment more on the internal bug.

This is fixed by the revert.

This is fixed by the revert.


Overall improvements related to demo

Codecov Report
Attention: Patch coverage is 93.75000% with 1 line in your changes missing coverage. Please review.

Project coverage is 85.59%. Comparing base (1df316b) to head (9016dd9).

Files with missing lines
Patch %
Lines

src/utils/readNwrite.js
83.33%
1 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##           master     #323       +/-   ##
===========================================
+ Coverage   65.74%   85.59%   +19.84%
===========================================
  Files         121      121
  Lines        3661     3665        +4
  Branches      711      713        +2
===========================================
+ Hits         2407     3137      +730
+ Misses       1186      498      -688
+ Partials       68       30       -38

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


Support for Stripe Payment Intents

Hi,
Great little component here, saving my bacon right now, any thoughts about supporting Stripe PaymentIntents?
https://stripe.com/docs/payments/payment-intents/quickstart
https://stripe.com/docs/stripe-js/reference#stripe-handle-card-payment
Seems you'd need support for the stripe.handleCardPayment() function.
Thanks

take a look here if you get a chance, I'm not entirely sure I have nailed it, and wont have a chance to test it right now: https://github.com/CromonMS/vue-stripe-elements/tree/adding-new-functions

This is a very important feature as it would takes effect starting septembre in all europe !
@sedrickcz when could we have this please ^^ !!

Please see pull request at https://github.com/fromAtoB/vue-stripe-elements/pull/74
Limited testing but working very well with the 2 main possible flows within the Payments Intent API

Just checking to see if there has been any update in integrating Payment Intents?
Laravel Cashier v10 no longer accepts a Stripe Token to create a card source.

You also need to add handleCardAction

With v.0.3.0 we are using Payment Intent API


demo cloudwebRTC server is down

The online demo cloudwebRTC server seems to be down when I try to connect it through the Flutter app or any web browser
link: https://demo.cloudwebrtc.com:8086/

The online demo cloudwebRTC server seems to be down when I try to connect it through the Flutter app or any web browser
link: https://demo.cloudwebrtc.com:8086/

this demo sometimes runs and sometimes down, so keep test it everyday.


LICENSE file

In order to share the PKGBUILD on the AUR for ArchLinux, a LICENSE file or license information is missing. It seems to be a project under the MIT license, but it would be nice to retrieve that information so that we can put it in a file in /usr/share/licenses/${pkgname}

Hi @l-alfred It is added into master now.


TV weights equal or less than zero

This PR resolves issue #26 and implements a new .from_numpy_map method on Map. The latter lets one instantiate a map object from a numpy array in one step instead of two.
Happy to have suggestions of how to name the function to make it clear that the numpy array should be a 3-d array of voxels in real space...

fix bug where tv methods were not propagating (unused) uncertainties from the input to output


AudioSession is switched from active to inactive a few times

Describe the bug
With the latest SDK <IP_ADDRESS> and CallKit I'm noticing that AVAudioSession is set active then inactive and active back (I set symbolic breakpoint). There is Red Microphone appears at the status bar after a few seconds after an app joined a channel.
As I see this project doesn't have CallKit example anymore. I have the old one AgoraRTCWithCallKit and if I replace SDK to the latest one I'm getting this issue there too.
To Reproduce
Replace SDK for the project AgoraRTCWithCallKit
Run the project AgoraRTCWithCallKit. Click Outgoing Call.
You notice the Red Mic icon at the status bar. The delegate method didAudioRouteChanged are called 2-3 times instead of one time. If you set a breakpoint on the method -[AVAudioSession setActive:withOptions:error:] you will see it's called a few times (instead on one time with previous SDK)
Expected behavior
AVAudioSession should be set active once
Screenshots

Smartphone (please complete the following information):

Device: iPhone 11
OS: iOS 13.5

i will check tomorrow


Support for wallet connect

Does Tangem support wallet connect at this particular time?
I've seen on twitter that it was in beta.

Tangem app supports wallet connect.
This repo is the card SDK and provides API for working with Tangem card.


Ensure that component prop 'context' really contains a React context …

…before using it
after switching to react-redux 6.0.0, we've had a lot of errors stating
Warning: React.createElement: type is invalid -- expected a string (for built-in components) or a class/function (for composite components) but got: undefined. You likely forgot to export your component from the file it's defined in, or you might have mixed up default and named imports.
all over in our app. After digging deeper, we've discovered that we use a lot of (connected) components already using a property named context which conflicts with your new connectAdvanced code. So maybe you can improve the check on this.props.context a little bit to ensure it is used only if it really contains a valid React context.

Can you add a test?

ok, I'm going to add tests within the next few days

improved checks & added test

maybe it would even be better to check for this.props.context.$$typeof === REACT_CONTEXT_TYPE, but react doesn't seem to export its type symbols...

We do have react-is being imported already - might be a function in there you can use?

we could check for this.props.context.$$typeof === ContextConsumer
with export const ContextConsumer = REACT_CONTEXT_TYPE; being exported by 'react-is'
this works and all tests would succeed.
But a dedicated method like isContext doesn't seem to be available from 'react-is'.

great, thanks!
When we decide to have type checks on members of this.props.context, we could even omit the check on this.props.context.provider as only .consumer is used after the checks.

Yeah, I removed that. It was also complaining about isContextProvider not being exported from react-is. Weird.

hm, it doesn't seem to know isContextConsumer either...!?

And now isContextConsumer. wut?

weird, when running the tests locally, isContextConsumer is imported correctly.
however, the check must be done this way, otherwise other test cases fail:
const ContextToUse =           this.props.context &&           this.props.context.Consumer &&           isContextConsumer(<this.props.context.Consumer />)             ? this.props.context             : Context
seems to be a litte overhead, so maybe it would be better to simply check for
this.props.context.$$typeof === ContextConsumer as I've stated above.
I'm beginning to think that react-is' ContextConsumer export should rather be named ContextToBeConsumedByContextConsumer though....

When do you plan on cutting a new release with this change?

@timdorr ?


Fixes #11: apoc.load* don't actually support tar and tar.gz

Fixes #11
Tests modified with "true" tar and tgz (instead of zip renamed to tar / tgz / tar.gz)
Added ArchiveType factory which recognizes the file from the extension

Looks good :) Just need to fix up the unused imports as it seems to be failing the spotless check


Fix linter rule violations so exceptions can be removed

Violations need to be fixed and/or exclusions granted to enable these rules on CI.

[ ] parameter missing help
[x] command missing help
[ ] list command with --ids
[x] missing command group help

Originally transferred this to azdev, but really this should stay here.

add to S165.


Upgrade local provision to v2.0.0

related to #2274  and #2232.
update the tag from v1.0.1 to `v2.0.0'.
migrate the configmap to the new format
cc @mirwan / @hswong3i

@ant31 working on the fixup and upgrade now ;-)

@ant31 and @mirwan update to 2.0.0 done with https://github.com/kubernetes-incubator/kubespray/pull/2344, please kindly review :-)

@ant31 https://github.com/kubernetes-incubator/kubespray/pull/2344 get merged ;-)

#2427 also get merged, too


\boldsymbol malfunction for local mathjax font in firefox

Local font is installed on Windows from MathJax/fonts/HTML-CSS/TeX/otf.
If \boldsymbol is used, then all normal font is displayed same as \boldsymbol in firefox.

The \boldsymbol and normal font displays correctly for web font, stix local font, IE, or chrome.

Thanks.

@daegunlee could you let us know if you still see this?


Allow ignoring private inner classes

Even though private inner classes are runtime public, many projects consider them internal API's. It would be great if MIMA could recognize and automatically exclude these classes.

@pwendell I absolutely agree it would be useful to have in MiMa a Scala working mode. In fact, I've captured this idea in ticket #34. I'm closing this ticket as a duplicate.


Add HUYA to adopters.

Add HUYA to adopters

Ⅰ. Describe what this PR does
Ⅱ. Does this pull request fix one issue?

Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Thanks, @BobLiu20


Support external schemas

See https://github.com/kogosoftwarellc/express-openapi/issues/17

Changes Unknown when pulling d7588692e143bcb881132828f43382b9f8197b1f on MugeSo:support-external-schemas into ** on kogosoftwarellc:master**.

Thanks!  Published as v0.3.0


Supporting port-share

With openvpn, you can run a web server on the same port as openvpn.
I wrote a small extension to support this feature.

diff --git a/manifests/server.pp b/manifests/server.pp
index 24a7048..940284d 100644
--- a/manifests/server.pp
+++ b/manifests/server.pp
@@ -74,6 +74,10 @@
Integer.  The port the openvpn server service is running on
Default: 1194

+# [portshare]
+#   String.  The address and port to which non openvpn request shall be forwared, e.g. <IP_ADDRESS> 8443
+#   Default: undef
+#
[proto]
String.  What IP protocol is being used.
Default: tcp
@@ -376,6 +380,7 @@ define openvpn::server(
$local                     = $::ipaddress_eth0,
$logfile                   = false,
$port                      = '1194',

$portshare                 = undef,
$proto                     = 'tcp',
$status_version            = '',
$status_log                = "/var/log/openvpn/${name}-status.log",
diff --git a/templates/server.erb b/templates/server.erb
index 715570d..5769606 100644
--- a/templates/server.erb
+++ b/templates/server.erb
@@ -31,6 +31,9 @@ proto <%= @proto %>
nobind
<% else -%>
port <%= @port %>
+<% if @portshare -%>
+port-share <%= @portshare %>
+<% end -%>
<% end -%>
<% if @real_tls_server -%>
tls-server

patch.txt

Thank you for your work. I opened a PR for your patch: #185.


Add Weave Content

https://github.com/rancher/docs/issues/1008

Can you rebase off of the staging branch @galal-hussein

LGTM


Enable setting custom wallet service on setup

Enable setting custom VWS service on setup.
Right now when creating a wallet you can only adjust the wallet service afterwards.
Make it possible to do it on the setup views.

Will be default for v2.0.0 as it will support multiple wallets and thus there's no need for the current setup guide.


Regression in 4.8 where string union type widens to string

Bug Report
🔎 Search Terms
string union widening, string union regression, union widening
🕗 Version & Regression Information
This regression occurred in 4.8.0, and can be reproduced with the 4.8.0-beta, and nightly (4.9.0-dev.20220904) tag in TS Playground. This works with 4.7.4.

This changed between versions 4.7.4 and 4.8.0-beta

I feel like this might be related to the Improved inference from binding patterns change
⏯ Playground Link
Playground link with relevant code
To reproduce, hover over the "isWorking" variable to see that the type definition includes a union of the strings. Switch to 4.8.0-beta, and note that the type definition is now just string.
💻 Code
interface Guard<T> {
    (val: unknown): val is T;
}

type ObjectGuard<T> = {
    [key in keyof T]: Guard<T[key]>;
};

function isObject(val: unknown): val is Record<string, unknown> {
    return val !== undefined && val !== null && typeof val === 'object' && !Array.isArray(val);
}

function isObjectOfShape<T>(
    value: unknown,
    shape: ObjectGuard<T>,
): value is T {
    if (!isObject(value)) {
        return false;
    }
    let validShape = true;

    for (const key in shape) {
        const guard = shape[key];
        if (guard && !guard(value[key])) {
            return false;
        }
    }

    return validShape;
}

function createObjectGuard<T>(guard: ObjectGuard<T>) {
    return (val: unknown): val is T => isObjectOfShape(val, guard);
}

function asLiteral<T extends (string | boolean | number)[]>(...literals: T) {
    return (val: unknown): val is T[number] => {
        return literals.includes(val as T[number]);
    };
}

// See type of `isWorking` - should include the type key as a union of strings
const isWorking = createObjectGuard({
//    ^?
    type: asLiteral('these', 'should', 'be', 'a', 'union'),
});

🙁 Actual behavior
String union type is widened to string.
🙂 Expected behavior
String union type remains a union, rather than widening

Probably the same cause:
Not only unions. Here is a simpler example:
https://www.typescriptlang.org/play?ts=4.8.2#code/GYVwdgxgLglg9mABAZQKZgCYCEA2cBGAFBgIZQkBci4A1mHAO5gA0i6EcGMYA5lbfSYBKRAG8AsAChEMxDGCJC7Ttx6IAhAF5N1TKmDdUGRADITbSCt4btiAOQgowABx2RE6bK9QAFgCdGRDBUBkQAUT8Av0I7ZS5eNwBuKS8AXykU2QB6LMQAWTgQAGdURB84ADdUP0QoOEQS0oxUKFRoI1qATwAHUsyZHMQAdR90aiLVRAAVZEQKgBYAOmdFgCY5KDkisVTEAB9dZoNgjH7EQZGx4smZuaWAdkX5ja3EACJHFzf9w-1DU88MjiqmSklSoKAA
function SendBlob(data: unknown, encoding: unknown) {
    if (encoding !== undefined && encoding !== 'utf8') {
        throw new Error('encoding');
    }

    // Mouse hover to see detected type
    // When using TS v4.8.2 it is {} | undefined
    // When using TS v4.7.4 it is "utf8" | undefined
    encoding;
};

@typescript-bot agrees with your guess about the cause, which is strange and definitely unintentional since there is no binding pattern in the repro 🤔

I think the milestone should be adjusted because that ship has sailed...  :)

Thanks. I forgot to update this issue, but we decided that the fix looked a little risky to ship in a patch when weighed against the small number of reports of this issue we’ve gotten.


Use the time 0f the executeing methond  as the default lock time

Is your feature request related to a problem? Please describe.
now use lockAtLeastForString and lockAtMostForString to controle the lock thie ,which I'think is useless to actural work
Describe the solution you'd like
Use the time 0f the methond execution which annotatied @ShedlockSchedule
Describe alternatives you've considered
Use the methond execution time as the lock  time
Additional context
Add any other context or screenshots about the feature request here.

Sorry, I really do not understand.


Update MSIX package names' spec

Update MSIX package names' spec to match actual 1.0 behavior (missed updating spec when we updated code for 1.0-stable)

/azp run


Peer deployment guide

Doc updates for instructions for deploying a
production peer
Change-Id: I08b5e3ac0394a2fa089a88dd11c28acd6f8b0a57
Signed-off-by: joe-alewine<EMAIL_ADDRESS>Type of change

Documentation update

Description
The initial push of this material will include updates to the deployment overview and to add a doc about planning for a peer and a checklist of variables listed in core.yaml. A doc that walks users through the process of deploying the peer itself will be added to this PR when it's ready.

@Mergifyio backport release-1.4


Update pt_PT translation

Thanks again.  Apologies, I have more i18n extractions to commit as I keep finding more.  Sorry for the multiple passes required.  Really appreciate your help on this!

Hi.
Not a problem.
However, I can only do it tomorrow it is late in my country.
I ve to work tomorrow, so I will only by able to do it in the end of the
day.
Best regards
Antonio Casqueiro
A segunda, 13/06/2022, 21:48, Guy Davis @.***> escreveu:

Thanks again. Apologies, I have more i18n extractions to commit as I keep
finding more. Sorry for the multiple passes required. Really appreciate
your help on this!
—
Reply to this email directly, view it on GitHub
https://github.com/guydavis/machinaris/pull/693#issuecomment-1154424678,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABHYTFGN5RXM6SMLDOM7UJ3VO6NCHANCNFSM5YUJZ4UA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Modify support page to point to form

Purpose
Point users to support form, rather than email, so we can capture more information about their request up front and in a standardized way.
Summary of Changes
email address --> form link
Side Effects
n/a
Feature Flags
n/a
QA Notes
n/a
Ticket
Reviewer Checklist

[ ] meets requirements
[ ] easy to understand
[ ] DRY
[ ] testable and includes test(s)
[ ] changes described in CHANGELOG.md

Pull Request Test Coverage Report for Build 2759

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 70.825%

Totals

Change from base Build 2745:
0.0%

Covered Lines:
5751

Relevant Lines:
7319

💛  - Coveralls


Abusive IPs targeting SIT server

Hi all, I know this isn't the correct repo for the server but issues are not supported on the others. These malicious IPs were previously not targeting our server prior to running the Docker image for SIT.
I was hoping I could get some insight as to why this is happening, or advice on how to prevent this. Looks like many of the IPs are linked to port scanning, hacking, ddos attacks.

Password Spraying Activity Targeting Various VPN Appliances, Firewalls, and Other Public Web-Based Applications

IP Abuse Reports for <IP_ADDRESS>

Source IP
Date
Time
Message

<IP_ADDRESS>
04/29
9:25am
We've blocked a known malicious IP from Hong Kong from accessing this device.

<IP_ADDRESS>
04/28
9:03am
We've blocked a known malicious IP from China from accessing this device.

<IP_ADDRESS>
04/28
5:39am
We've blocked a known malicious IP from Russian Federation from accessing this device.

<IP_ADDRESS>
04/26
4:36am
We've blocked a known malicious IP from United States from accessing this device.

<IP_ADDRESS>
04/26
3:55am
We've blocked a known malicious IP from United States from accessing this device.

<IP_ADDRESS>
04/25
2:01pm
We've blocked a known malicious IP from Russian Federation from accessing this device.

this is risk inherent with opening inbound ports. this is exactly why. just because you are hosting the server in a container does not affect the number of attackers trying to brute force their way in.
if anything, i might guess attackers have adjusted their methods to account for the sheer number of firewalls that are now forwarding 6969-6972 over the weekend.

😨

Thanks @apfaffman - I'm going to look into getting this in a VPS for now. May be a good idea for a maintainer to recommend something like that in docs


Question about loss function of direction

In direction loss function, “torch.sign” will result in gradients being almost zero, which means that while it won't throw an error, it essentially has no effect in practice. If there are any mistakes, please point them out.

Thanks for your comments. Yes, you are correct that the sign function is not differentiable and this is an error. One way to circumvent this is to use the tanh function. We plan to update the repo once we've done some systematic tests but so far it looks like this improves performance slightly.
More discussion here: https://github.com/snap-stanford/GEARS/issues/36


docs: update flex-layout story

added image snapshots,
fixed links,
improved sample

Thanks for the PR. Could you squash and follow the guidelines (same as Angular)? Thanks. This seems like a docs change so something akin to docs: update flex-layout story.

@hansl - done.


DCOS-14171: Clean up usage and styling of DeleteRowButton

This PR cleans up the styling of the DeleteRowButton component. It removes all horizontal padding on the delete row button, uses column-auto instead of column-2, and increases the column width of all sibling FormGroups to ensure that the delete button takes up only as much space as is necessary. I also moved all classes to the FormGroup element so that we don't have to remember which classes to use in the future.
This adds two new props to the FormGroup component:
isDeleteButton — when true will apply the correct column and flush classes
applyLabelOffset — when true will apply the correct class, adds padding to the top of the form group so that it aligns vertically with its siblings who have labels
Before (note the real estate used by the button's column and its alignment with other delete buttons):

After:

Checklist

[x] Did you add a JIRA issue in a commit message or as part of the branch name?
[ ] Did you add new unit tests?
[x] Did you add new integration tests?
[ ] If this is a regression, did you write a test to catch this in the future?

Can one of the admins verify this patch?


ResourcePolicy without rptype is allowed but seems useless

Describe the bug
I was alerted on Slack by @MW3000 that there can be resourcepolicy records with null rptype.  We have several thousand of these in our main repository.  I have so far found no code that recognizes these policies -- they seem to have no meaning.  The only code I've seen for null rptype seems to be guarding against NPE from .equals tests that should simply be reversed to avoid the NPE.  (There is also one useless test before applying a method which does not terminate abruptly on null input.)
To Reproduce
I have not determined the source of these curious policies.  Observations or hypotheses would be welcome.
Expected behavior
If a policy with no type has meaning, that meaning should be documented and implemented.
If a policy with no type is invalid, we should not be able to store one and we should not find any in the database.

Adding a NOT NULL column constraint (after fixing up the NULL values) should quickly show us where these are being created.


dotnet test has exit code 0 despite BadImageFormatException

From @bender2k14 on Monday, October 7, 2019 7:01:10 PM
Steps to reproduce and Environment data
When using the pool with VM image label windows-2019, the relevant part of the dotnet --info output is
RID: win10-x64

Therefore, it is no surprise that executing
dotnet test Tests\Tests.fsproj --runtime win-x86

doesn't work since the RIDs are incompatible.  Specifically, the relevant part of the logs when "Enable system diagnostics" is checked is
Test run for D:\a\1\s\src\Tests\bin\Debug\netcoreapp3.0\win-x86\Tests.dll(.NETCoreApp,Version=v3.0)
Microsoft (R) Test Execution Command Line Tool Version 16.3.0
Copyright (c) Microsoft Corporation.  All rights reserved.

Starting test execution, please wait...

A total of 1 test files matched the specified pattern.
[xUnit.net 00:00:02.06] Exception discovering tests from Tests: System.BadImageFormatException: Could not load file or assembly 'Tests, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'. An attempt was made to load a program with an incorrect format.
File name: 'Tests, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'
   at // ... stack track omitted

No test is available in D:\a\1\s\src\Tests\bin\Debug\netcoreapp3.0\win-x86\Tests.dll. Make sure that test discoverer & executors are registered and platform & framework version settings are appropriate and try again.
Results File: D:\a\_temp\VssAdministrator_fv-az604_2019-10-07_18_48_39.trx

##[debug]Exit code 0 received from tool 'C:\Program Files\dotnet\dotnet.exe'

Expected  behavior
I expected dotnet test to return a nonzero exit code since it encountered a BadImageFormatException.
Actual behavior
dotnet test to returned an exit code of 0 despite encountering a BadImageFormatException.
Copied from original issue: dotnet/cli#12842

@bender2k14 This is the current behaviour, such a change in behavior would be a breaking change for them (specially case when using wildcard or giving multiple dlls)

We are also hitting this issue. In my project, test discovery failed for a different reason that what @bender2k14 reported, but we did not detect it for months because vstest returns exit code 0 when no tests are discovered. Our CI pipeline reported "successful" results even though no tests were running at all, and so we did not detect regressions in our project which could have been caught by the tests.
@singhsarab will there be an opportunity in the future where this breaking change could be made? Or can the behavior be made user-configurable?

@mcooley, does your CI pipeline use VSTest?  If so, then https://github.com/microsoft/azure-pipelines-tasks/issues/11328 might (eventually) help you.


Step5

Ajax is not working properly yet. commit 5778451 has it updating the detail page every OTHER time  I hit the share button.
Commit  065a7d8 has alot of adjustments, and they are showing on the page, but I have it just displaying the the new entry, instead of actually updating that first entry. So if you edit the entry multiple times you will see what I mean. But it is editing and updating the database as you can see from the main page.
I really want to talk to you about this on Monday just because it is boggling my mind as to why its doing this.

I'm curious about the reported behavior.  I'd like to see it in action and help you debug what's going on.  Ajax interactions can be a bit mind-bending at first.


[DOC] Update documentation for cluster settings to include leader/checker timeout setting

What do you want to do?

[x] Request a change to existing documentation
[ ] Add new documentation
[ ] Report a technical problem with the documentation
[ ] Other

Tell us about your request. Provide a summary of the request and all versions that are affected.
We recently made leader/checker timeout setting dynamic. The document is to be updated to reflect the changes.
https://github.com/opensearch-project/OpenSearch/issues/10528
What other resources are available? Provide links to related issues, POCs, steps for testing, etc.

Hi @niyatiagg, is this a feature going in to 2.12?

Hi @niyatiagg, is this a feature going in to 2.12?

Yes, it is.

Thanks! Will you be the one submitting a doc PR? I'll assign @Naarcha-AWS as the technical writer collaborator.

Thanks! Will you (@niyatiagg) be the one submitting a doc PR? I'll assign @Naarcha-AWS as the technical writer collaborator.

Yes, I can work on it. Is there a timeline for 2.12?

@niyatiagg As mentioned by @hdhalter the absolute deadline is Feb 06 but we would appreciate if you can raise a draft PR as soon as possible to hep with review.

@niyatiagg As mentioned by @hdhalter the absolute deadline is Feb 06 but we would appreciate if you can raise a draft PR as soon as possible to help with review.

Sure! Will get it done today.


How do you resolve the problem of squatting?

Suppose as an early adopter I manage to get google.eth for cheap. Supposing I don't want to give it up, what recourse would a company such as Google have to take the name? Its not clear to be based on the wiki here.

This is described in EIP 162. Please reopen if you have further questions about it.


Fix additional_identifier_chars with _

Also Fix omitting second binaryOp if precedence is zero (or not set), since it breaks after reading the binaryOp from the string, but then ignoring it.

Whoops, thanks!


Need fix after amphp/sql update

After commit e257237b61da7cb25a59a95299e232e9b536d93c (Improve naming) at amphp/sql package need to fix Amp\Postgres\Connection class.
Fatal error: Class Amp\Postgres\PgSqlConnection contains 2 abstract methods and must therefore be declared abstract or implement the remaining methods (Amp\Sql\Link::beginTransaction, Amp\Sql\TransientResource::getLastUsedAt) in vendor/amphp/postgres/src/PgSqlConnection.php on line 14

Yes, there have been a few renames. Would you like to provide a PR?

Ok, I can to provide PR, but later...maybe tommorow at leisure time)


Wrapped Icons break InputField layout?

import Icon from 'react-native-vector-icons/FontAwesome';
class Icon2 extends Component {
  render() {
    return (
      <Icon {...this.props} />
    );
  }
}
<InputField iconLeft={<Icon2 />} />

Seems to break layout in input fields – the actual input field / placeholder will disappear.

I added the case in the example.. I don't see any problem.... please let me know.


test(iroh-cli): make cli resumption tests not flaky

Description
Takes a different approach to tests by doing the setup for them all first
before running the tests, to prevent windows from whining about a locked
blobs.db used by the provider. This still requires an ephemeral iroh to obtain
the blobs.db but does not require much re-starting, making it less prone to
weird shutdown issues.
Breaking Changes
n/a
Notes & open questions
Note that there is sleep as a synchronization point because otherwise the
folder for the tests will not have the correct contents.
Change checklist

[x] Self-review.
[ ] Documentation updates following the style guide, if relevant.
[x] Tests if relevant.
[ ] All breaking changes documented.

We were duplicating a bit of work here, I was going through exactly the same motions yesterday :upside_down_face:
The last issue I've hit was port binding not working when the iroh is rapidly stopped & started again.

|Error: iroh is already running on port 4919

I think in rare cases port binding can be racy. Have you hit this as well?


Use .. to identify autoscoped constructors

Pull Request Description
Fixes #9275.
Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:

[x] The documentation has been updated, if necessary.
[x] All code follows the
Scala,
Java,
style guides.
All code has been tested:

[x] Unit tests have been written where possible.

Not to start language wars but...

The syntax discussion is the right place to express such opinions.

@jdunkerley, you wanted me to test:
from Standard.Base import all

main =
    a = 5
    a..7

which yields Method .. of type Integer could not be found. error.


problem trying to 'amplify init' to create prod env

I'm trying to intialise a new environment with

amplify init

for prod (as shown in the Environments and Teams sections: [https://aws-amplify.github.io/docs/cli-toolchain/quickstart])
Do you want to use an AWS profile? Yes
Please choose the profile you want to use amplify-my-profile
You project is associated with multiple Amplify Service Apps in the region eu-west-1
TypeError: options.Push is not a function
at SelectFromExistingAppId (/Users/mb/.nvm/versions/node/v12.10.0/lib/node_modules/@aws-amplify/cli/node_modules/amplify-provider-awscloudformation/lib/amplify-service-manager.js:310:11)
init failed
I've gone in and console.logged out the value for: appIdsInTheSameLocalProjectAndRegion
[ 'd2meqvq92i0efd', 'd2meqvq92i0efd' ]
this value is the AmplifyAppId value for my dev and stage environements, which I managed to create fine.
options is set to [ 'd2meqvq92i0efd', 'd2meqvq92i0efd' ]
options.Push(NONE) is the line where it's failing...
I'm trying to do all this while on my master branch (which will be prod)
What am I doing wrong, please help.

very odd, I've edited Push to be push and have managed to get a bit further along

What is the version of your locally installed Amplify CLI?
Execute amplify -v to find out.

hi, editing Push to be push allowed the init process to complete. not sure how push got replaced with Push.


Add an icon for Twitch

As requested by https://github.com/calleluks/Tofu/issues/64

Thanks @bartvdbraak, I really appreciate it!


Fix fold in the case the update map is not empty

The issue with the dangling hash is that the fold in https://github.com/mirage/irmin/pull/1525, visits values by reading their children from disk, but newly added values are not on disk, hence the error.
We can fix this by visiting the updatemap separately (in commit https://github.com/mirage/irmin/commit/40ccecebeb33df3b7f9bba20fc430d0aaf3f14f0). This works on the backport (see https://github.com/mirage/irmin/pull/1531) but does not work with the current upstream, where to_value is going to raise an error if there are elements in the updatemap. Another possibility is to just revert to to_map. I don’t see a significant difference between the two in my benchmarks in terms of neither memory and run time for the migration, so the quick fix is maybe the way to go?

What happens for entries both in the value and in the updatemap? Are we folding over them twice?

What happens for entries both in the value and in the updatemap? Are we folding over them twice?

I added a test to ensure that we only visit the updated entry in this case.
Just to clarify, there are two possible solutions to the bug introduced in  #1525:

one solution is to visit the updatemap separately during the fold (commit 40ccece) - but this does not work straight away with the current master;
a second solution is to use to_map when there is a non empty updatemap (commit 5cbac7a)- this works with both master and the backport and so I think we should go with this

The fix I went for the fold over values with an updatemap, is to use the updatemap to update the bindings of a node. It is rather straightforward.
I have tried converting to portable value during the fold, but the issue is that once a node is portable we can no longer get nodes out of its bindings to traverse them. So we have to keep both the updatemap and the original value to be able to fold over all children.
I also tried your solution @samoht to catch dangling hash errors and use the updatemap to resolve them  - its a bit messier, we have to propagate the updatemap everywhere (but it is here if you want to have a look https://github.com/icristescu/irmin/commit/6e404356c94b7dca82fd31e06898f82cb0edf7f8);
and anyway, I don’t see how to make it work with portable nodes, since we cannot apply the modifications in the updatemap to the nodes and hence we never get a dangling hash in this use case.

LGTM

As discussed offline, this PR now reverts to using to_map in the fold to guarantee that elements are visited in the lexicographical order of their keys. Also added a test to check this (taken from https://gitlab.com/tezos/tezos/-/blob/master/src/lib_context/test/test_context.ml#L241).

LGTM


[MM-27447] Remove usage of jQuery from post components

Ticket Link
https://github.com/mattermost/mattermost-server/issues/15160
Release Note
NONE

I've updated failing test.
Problem was of this kind https://github.com/cypress-io/cypress/issues/5858
Cypress was typing in wrong field:

Thanks @jgilliam17 @saturninoabril for all suggestions.
I've made fixes.
When test case in Zephyr is ready, I'll add missing test key.

@devinbinnie Can you please help merge? Thanks 🙂


check_fn For S3KeySensorAsync Is Not Compatible With check_fn Of Its Super Class S3KeySensor

Describe the bug
If you have specified check_fn for S3KeySensorAsync, its super class S3KeySensor will also call that check_fn like this:
https://github.com/astronomer/astronomer-providers/blob/96f75535dcc3749dade524d73de060ff17f11ea9/astronomer/providers/amazon/aws/sensors/s3.py#L88-L94
self.poke calls the method poke of super class S3KeySensor: https://github.com/apache/airflow/blob/62a534dbc7fa8ddb4c249ade85c558b64d1630dd/airflow/providers/amazon/aws/sensors/s3.py#L132-L136
def poke(self, context: Context):
    if isinstance(self.bucket_key, str):
        return self._check_key(self.bucket_key)
    else:
        return all(self._check_key(key) for key in self.bucket_key)

This poke calls _check_key method of the super class S3KeySensor. The problem occurs inside this _check_key method if you have given check_fn: https://github.com/apache/airflow/blob/62a534dbc7fa8ddb4c249ade85c558b64d1630dd/airflow/providers/amazon/aws/sensors/s3.py#L127-L128
...
    ...
        if self.check_fn is not None:
            return self.check_fn(files)
    ....
....

The example value of files it gives is [{'Size': 100}, {'Size': 200}, {'Size': 300}]. They have mentioned this in the docstring: https://github.com/apache/airflow/blob/62a534dbc7fa8ddb4c249ade85c558b64d1630dd/airflow/providers/amazon/aws/sensors/s3.py#L105C9-L111C12
        """
        Set variable `files` which contains a list of dict which contains only the size
        If needed we might want to add other attributes later
        Format: [{
            'Size': int
        }]
        """

But the S3KeyTrigger.run method returns complete metadata:
https://github.com/astronomer/astronomer-providers/blob/96f75535dcc3749dade524d73de060ff17f11ea9/astronomer/providers/amazon/aws/triggers/s3.py#L79-L83
The example value of s3_objects here is:
[
    {
        'Key': 'test.csv',
        'LastModified': DateTime(2023, 6, 25, 10, 10, 10, 855000, tzinfo=Timezone('UTC')),
        'ETag': '"333e09..."',
        'Size': 300,
        'StorageClass': 'STANDARD',
        'Owner': {
            'DisplayName': 'S3',
            'ID': '02d61...'
        }
    }
]

If you are writing check_fn based on this format, it will throw KeyError for missing keys in the dictionary whenever S3KeySensor._check_key will pass the value to your check_fn. S3KeySensor.poke always runs before S3KeySensorAsync._defer is called:
https://github.com/astronomer/astronomer-providers/blob/96f75535dcc3749dade524d73de060ff17f11ea9/astronomer/providers/amazon/aws/sensors/s3.py#L95-L96
This change was made recently in #1037 : https://github.com/astronomer/astronomer-providers/commit/1bc969a3f00a6c140c04b7a45bb7d78b8a3bd0e9 to implement "poke before defer" functionality.
Solution
I think it is better to fix the output in upstream as the Size attribute of the bucket_key is not enough to validate the files. A user will likely expect name of the file to validate if the proper naming convention was followed which is how I encountered the KeyError. If we upstream it, we may have to upstream use_regex functionality as well otherwise S3KeySensor (with wildcard_match to False)  will disagree with S3KeySensorAsync (with use_regex to True).
Workaround
Put this condition in your check_fn to disregard the value provided by S3KeySensor._check_key:
    if values:
        _value, *args = values
        if not _value.get('Key'):
            return False

It checks if the first item in the list has the key name Key. If it does not, return False to defer it. Now follow up calls to your check_fn will have the value provided by S3KeyTrigger.run method.

At the moment, I think restricting the way we call chec_fn in the repo is the best option but if you feel we should add more attributes other than size then we need to make some changes in the Airflow repo and request you to create a separate feature request of it


[WIP] Fix #6121

Opening this PR to fix #6121 and iterate on it. The problem is that forall always returns a new Dispatcher, which immediately compiles the specialization. Prior to #5709 being merged, the cache of compilations was used and any previously-compiled kernel could be returned by specialize. The fix in this branch introduces a cache of specialized Dispatcher objects to work around the issue.

Buildfarm ID: numba_smoketest_cuda_92 for 86241da

numba_smoketest_cuda_92 passed, will try 4f17d85

Buildfarm ID: numba_smoketest_cuda_93 for 4f17d85

numba_smoketest_cuda_93 passed.

@stuartarchibald As far as I can tell from testing cuDF and other libraries, the issue appears to be resolved and I can't see that this introduces any new issues.

@stuartarchibald As far as I can tell from testing cuDF and other libraries, the issue appears to be resolved and I can't see that this introduces any new issues.

Thanks for confirming. This will make 0.51.0.


Expose ability to set python version via client

What is the expected behavior?
Expose ability to set python version via client.
Something like
serverless.run(program, config=Configuration(python=Configuration.PY_39))

closing as done by #1077


virtctl.sh: Call appropriate binary by architecture

In particular, call the Darwin executable if run in MacOS.
Signed-off-by: Ben Warren<EMAIL_ADDRESS>
What this PR does / why we need it:
Some make targets call the virtctl.sh wrapper script, which really only works with Linux.  This fix makes the script work with MacOS as well.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
None
Release note:

NONE

Hello contributor, thanks for submitting a PR for this project!
I am the bot who triggers "standard-CI" builds for this project.
As a security measure, I will not run automated tests on PRs that are not from white-listed contributors.
In order to allow automated tests to run, please ask one of the project maintainers to review the code and then do one of the following:

Type ci test please on this PR to trigger automated tests for it.
Type ci add to whitelist on this PR to trigger automated tests for it and also add you to the contributor white-list so that your future PRs will be tested automatically.
If you are planning to contribute to more than one project, maybe it's better to ask them to add you to the project organization, so you'll be able to run tests for all the organization's projects.

@rmohr @cynepco3hahue : This is currently failing an OpenShift CI test that I have no way to replicate.    I assume it's related to #1282, but I really don't know anything about the test.

@benwatcisco We still have number of flaky tests, the PR looks good.


roles/nv_gpu/tasks/install.yml: give more delay for the GPU Operator CRD to be applied

This should solve this error on the CI:
fatal: [localhost]: FAILED! => {
    "attempts": 5,
    "cmd": [
        "oc",
        "apply",
        "-f",
        "roles/nv_gpu/files/042_customresources.yaml"
    ],
    "msg": "non-zero return code",
    "stderr": "error: unable to recognize \"roles/nv_gpu/files/042_customresources.yaml\": no matches for kind \"ClusterPolicy\" in version \"nvidia.com/v1\"",
}

TASK [nv_gpu : Failed when creating the clusterpolicy CR for the GPU Operator] ***

@ArangoGutierrez thanks, pushed to all release branches

@ArangoGutierrez thanks, pushed to all release branches


[Feature] Metadata system for mods

Currently there is no metadata system at all for mods. Some sort of metadata system would be useful.
This could allow users to specify stuff like the name of the main dll/script a mod loads something like "main_file_name": "custom_mod_name" Then instead of trying to load a file named main.dll/lua it would then try to load the file with the name the user specified.
This would also allow for a dependency system to utilize the metadata system
{
 "dependencies": [
  "ModNameOne",
  "ModNameTwo",
 ]
}

The dependency system should allow for version specifying or just the name and nothing more
Specifying a version of the mod is also a valid thing "version": [1, 0, 0]
Allowing users to specify custom fields to read from could also be added.
A full example of a metadata file could be something like
{
    "name": "MyAwsomeMod",
    "version": [1, 0, 0],
    "dependencies": {
        "OtherAwesomeMod": {
            "min": [1, 0, 0],
            "max": [2, 0, 0],
            "name": "OtherAwesomeMod"
        },
        "AnotherAwesomeMod": {
            "min": [1, 0, 0],
            "max": [0, 0, 0],
            "name": "AnotherAwesomeMod"
        }
    },
    "ue4ss_version": {
        "min": [2, 6, 0],
        "max": [0, 0, 0] // Allows any version above 2.6.0
    },
    "description": "This is my awesome mod!",
    "author": "UserName",
    "main_file_name": "MyCustomModMain"
}

This seems good but I think it's a good idea to put anything that only has one line like description,author, and main_file_name at the top of the file and anything that can have multiple lines at the bottom in any official examples.
That way you don't have to look through a sea of arrays and objects to find a single-line setting.

For C++ mods I'd like the metadata to be a struct in the base mod class.  That way the mod authors only need to edit it in one place, and glaze can generate a json for modmanagers to use.  Lua mod authors will unfortunately have to generate the metadata file themselves until we have a proper manager

For C++ mods I'd like the metadata to be a struct in the base mod class. That way the mod authors only need to edit it in one place, and glaze can generate a json for modmanagers to use. Lua mod authors will unfortunately have to generate the metadata file themselves until we have a proper manager

I was thinking we would get rid of any metadata in the C++ mod class but I guess your way works as well.
We'd have to parse the metadata at runtime and re-set the metadata in the C++ mod class for each mod so that manual changes to the metadata json file actually have an effect.

Yeah, I was debating about that in my head, but I don't think end users should ever be editing the metadata manually (this also works as a bit of an anti-mod-theft mechanism).  My idea was to only reset the metadata if the mod's versioning was higher than the versioning in the metadata file.  So if mod is ver 0.2.0 and metadata is 0.1.0 it will replace it (potentially making a metadata.bak with the old one), but if both are 0.2.0 it will take the edits.

The more I think about it, the more I think there should only be one "true" metadata, and the file shouldn't really be edited by end users at all.  But, my idea above would still allow edits in an emergency where the mod author put a max version that isn't actually necessary or something.

My idea for handling it wouldn't have that issue.

My idea for handling it wouldn't have that issue.

Then I don't understand your idea.

I also don't think it's common to include "max" versions at all.

Why bother with the min/max fields if we could just include semver there?

Version checks also don't have to be blocks. They can warn and prompt to enable anyway or disable.

imo if a version check fails it should block the mod from loading, as having an incorrect version will lead to unexpected behavior

I disagree, at least for ue4ss version.

why?

Because it's perfectly possible for a mod to work for several future versions of ue4ss

what about min version check? and it's not guaranteed for it to work with newer versions because stuff might and will change, which could break the mod

power users can edit the mod metadata in the case they want to see if it breaks

End users shouldn't have to go in and edit a bunch of mod files after updating ue4ss

end users shouldn't be launching mods that do not advertise compatibility for a ue4ss version, you are not talking about powerusers in this case, and if we allow loading incompatible mods, this will increase bugreport count from users loading mods on incompatible versions and potentially crashing, if we are talking about powerusers, they can edit the metadata

I think at least if the version is to low it should throw an error where as if its to low it just throws an error. If its to high it throws a warning

but in both cases we could get a situation where an api was changed and will break the mod

Then they post the log and we see any warnings for out of date mods.

why should we by default allow users to crash their game and then waste both the user's and our time with figuring out the bug report?

Reference for semver: https://devhints.io/semver

You left that part out of your earlier statements.  But including a max that way would make it untested on all versions after when it was created and we wouldn't even get a warning that a mod was out of date so it's way worse for your argument that it would lead to unnecessary bug reports.

we should preferrably follow semver at least slightly, and not introduce breaking changes in at least patch releases, then it could be ~2.6.0, which would allow loading on any 2.6.x but not 2.7.x. still, if anyone wants to load an incompatible mod, they should go and change the meta, adding functionality that allows doing this completely defeats the point of having ANY version checks

going and changing the meta is more of a pain than enabling a config option/etc., which makes it less likely that someone will actually do it to try and "fix" someone's mod

I'm saying there is no point in having ANY maximum UE4SS version checks.

then after we make a minor update with api changes mods just break and not notify the user they are incompatible, they just break and crash

how would they warn if there is no max version check?

Is this now something worth working on considering it seems to have been discussed?

this can be done with pre defines. mods and setting has to put in certain values to get right sources and actor., when update the define only change inside ues, all user has to do is to add the variable. wich is the same.


Return-typed functions run even without return keyword

in the below sample function test() has string return type but missed to return anything. This is compiling and running without any issue even when assigning it to a string param;
function test()(string str){
system:println("Test Return Type");
}
//inside main function
test();
//OR
string teststr = test();

Fixed in v0.89


[BUG] Issue when runnning multipe pools (Bun)

Duplicate issue

[X] I've searched for any related issues and avoided creating a duplicate issue.

Description
I am getting an error when i try to run a FixedThreadPool with more than 1
Working
// a fixed worker_threads pool -> 1
const pooly = new FixedThreadPool(1, new URL('_worker.js', import.meta.url), {
    errorHandler: (e) => console.error(e),
    onlineHandler: () => console.info('worker is online')
})

Not Working
// a fixed worker_threads pool -> 2
const pooly = new FixedThreadPool(2, new URL('_worker.js', import.meta.url), {
    errorHandler: (e) => console.error(e),
    onlineHandler: () => console.info('worker is online')
})

Error message - relates to use of transformers.js - but only happens when i trigger multiple poolls

[!Note]
This worked perfectly before using node - I am porting to bun, issue could be transformers.js related but could also be something triggered here?

@xenova - I have an issue here, using your new branch - running it on its own seems fine, but running it in poolifer seems to cause an issue

Reference: https://github.com/poolifier/poolifier-web-worker/commit/746bbd71a3243634aa313eec3e0a6de1a747cf5d


Update dependabot frequence

Summary
It is too noisy to update go.mod daily. move it to weekly
Related issue(s)
Fixes #
Tests

[ ] Unit/function tests have been added and incorporated into make unit-tests.
[ ] Integration tests have been added and incorporated into make integration-test.
[ ] E2E tests have been added and incorporated into make e2e-test-all.
[ ] List other manual tests you have done.

/lgtm


Tab/window titles looking strange

I think that has to do with some of my configurations. Need to look at sections of my code that handle tab/window titles.

No longer an issue. Closing.


CI: Update to JRuby <IP_ADDRESS>

fix double free or corruption (out) error in JRuby

Adding sleeps will not help, so merged only 9b8eed5 into master. Thanks!


Fix warning in cups_pre_install.yml

Received warning as follows:
TASK [HP41.cups : Add OpenPrinting APT Key] *****************************************************************************************************
 [WARNING]: when statements should not include jinja2 templating delimiters such as {{ }} or {% %}. Found: {{cups_ricoh_openprinting_ppds}}

changed: [localhost]

TASK [HP41.cups : Add OpenPrinting Package repo] ************************************************************************************************
 [WARNING]: when statements should not include jinja2 templating delimiters such as {{ }} or {% %}. Found: {{cups_ricoh_openprinting_ppds}}

Fixed with this change.

Thanks for the fix!


Take off only works once

Hello,
Whenever I try to take off more than once it doesn't work.
`import asyncio
from abc import ABC
import logger
from drone.drone_action import DroneAction
from drone.drone_state import DroneState
class TakeOffAction(DroneAction, ABC):
log = logger.init_logger(name, "info")
async def action(self, drone_manager):
    self.log.info(f'Starting take_off action, state={drone_manager.drone_info.state}, args={self.args} '
                  f'info={drone_manager.drone_info.to_string()}')

    if not drone_manager.drone_info.in_air:

        if not drone_manager.drone_info.armed:
            arm_action = drone_manager.actions['arm']
            arm_action.supply(arm_action, args=None)
            await arm_action.action(arm_action, drone_manager)

        drone_manager.drone_info.state = DroneState.TAKING_OFF

        await drone_manager.drone.action.takeoff()

        await asyncio.sleep(5)

        drone_manager.drone_info.taken_off = True

        self.log.info(f'Finished take_off action, state={drone_manager.drone_info.state}, args={self.args} '
                      f'info={drone_manager.drone_info.to_string()}')

`
This is the code for taking off. But landing does work multiple times when I just type

commander takeoff

In the console and then call this function again.
This is the land code
`    log = logger.init_logger(name, "info")
async def action(self, drone_manager):
    self.log.info(f'Starting land_action action, state={drone_manager.drone_info.state}, args={self.args} '
                  f'info={drone_manager.drone_info.to_string()}')

    if drone_manager.drone_info.in_air:

        if not drone_manager.drone_info.armed:
            arm_action = drone_manager.actions['arm']
            arm_action.supply(arm_action, args=None)
            await arm_action.action(arm_action, drone_manager)

        drone_manager.drone_info.state = DroneState.LANDING

        await drone_manager.drone.action.land()

        await asyncio.sleep(5)
        drone_manager.drone_info.taken_off = False

        self.log.info(f'Finished land_action action, state={drone_manager.drone_info.state}, args={self.args} '
                      f'info={drone_manager.drone_info.to_string()}')`

And this is the code for the arm action
`class ArmAction(DroneAction, ABC):
log = logger.init_logger(name, "info")
async def action(self, drone_manager):
    self.log.info(f'Starting arm action, state={drone_manager.drone_info.state}, args={self.args} '
                  f'info={drone_manager.drone_info.to_string()}')

    if not drone_manager.drone_info.armed:
        drone_manager.drone_info.state = DroneState.ARMING
        await drone_manager.drone.action.arm()
        drone_manager.drone_info.armed = True

    self.log.info(f'Finished arm action, state={drone_manager.drone_info.state}, args={self.args} '
                  f'info={drone_manager.drone_info.to_string()}')

`
Everything gets called correctly and I also get no exceptions.

Figured it out, when landing the drone also disarms

hello,  only find u commentd recently . want to ask some question
i tried to run the land and take off codes on my windows through my usb serial
but i found the only example

Serial:` serial:///path/to/serial/dev[:baudrate]

what's its like on windows?i tried the follow but it dosent work,

await drone.connect(system_address="serial:COM13:115200")

can you help me?

hello, only find u commentd recently . want to ask some question
i tried to run the land and take off codes on my windows through my usb serial
but i found the only example

Serial:` serial:///path/to/serial/dev[:baudrate]

what's its like on windows?i tried the follow but it dosent work,

await drone.connect(system_address="serial:COM13:115200")

can you help me?

Hello, I'm not sure how it would work with the serial but I have found this:
https://github.com/mavlink/MAVSDK-Python/issues/100
and this
https://discuss.px4.io/t/serial-instead-of-udp-for-dronecore-examples/5459/3
The people there are using

serial:///dev/ttyACM0

and

serial:///dev/ttyUSB0:57600

So from what I think yours needs to be
serial:///dev/ttyUSB13:115200
or
serial:///dev/ttyCOM13:115200

@LiangR2017: It seems to me that your issue is about connecting over a serial link and has nothing to do with the original issue here. Is that correct? If yes, would you mind opening a new issue with your question? :blush:


Wnfs.publish returns empty error message

Err (Just Wnfs, "")

Not entirely sure this comes from publish.

Noticed this, too. I think it's because of this line
https://github.com/fission-suite/webnative-elm/blob/7ae3bfb89f509ce5d9e47b5c0afab855408ee68c/src/Wnfs.elm#L126

Noticed this, too. I think it's because of this line
https://github.com/fission-suite/webnative-elm/blob/7ae3bfb89f509ce5d9e47b5c0afab855408ee68c/src/Wnfs.elm#L126

So usually you'd provide a tag yourself for other WNFS functions like ls, rm, etc.
But publish just sets the tag to an empty string, so if your tag parser doesn't expect that, it'll error.

So usually you'd provide a tag yourself for other WNFS functions like ls, rm, etc.
But publish just sets the tag to an empty string, so if your tag parser doesn't expect that, it'll error.

So usually you'd provide a tag yourself for other WNFS functions like ls, rm, etc.
But publish just sets the tag to an empty string, so if your tag parser doesn't expect that, it'll error.

Oohhh, good catch, thanks! 👏

So usually you'd provide a tag yourself for other WNFS functions like ls, rm, etc.
But publish just sets the tag to an empty string, so if your tag parser doesn't expect that, it'll error.

Oohhh, good catch, thanks! 👏

Fixed in v3, also added an Error type so we can tell where the error message is exactly coming from.

Fixed in v3, also added an Error type so we can tell where the error message is exactly coming from.


ModuleNotFoundError: No module named 'tracks'. I pulled the code from Github but I keep getting the error in the "tracks " app that there is no module named "tracks". The folder doesn't even come with a manage.py file. Please any help will be appreciated.

ModuleNotFoundError: No module named 'tracks'. I pulled the code from Github but I keep getting the error in the "tracks " app that there is no module named "tracks". The folder doesn't even come with a manage.py file. Please any help will be appreciated.

Tracks is not a Django project - dj4e-samples is one Django project and tracks is an application.   There is only one manage.py file.

So

Tracks is not a Django project - dj4e-samples is one Django project and tracks is an application. There is only one manage.py file.

So please how do I use that manage.py for the tracks application? In your tutorial, you have a manage.py file in your tracks folder.

Where do I have a manage.py in a tracks folder?   That would mean that tracks is a project and not an application.  If you could point to where you are seeing this, it would help.

Hi Chuck. I was a bit disturbed too. In the long form video under the freeCodeCamp playlist (18 hours version) you use a manage.py inside the tracks folder, but it is not present on the course material here. I post the link with the timestamp at around 5.05 here. There is also a folder named tracksmodel in the clip which is also missing in the repo. Just before going through this part in the video you present the model of the track and mention, that we built this in the lecture, but this was not the case and I follow the course on python4everywhere since the beginning. For me it is no problem, I can get around and I enjoy your teaching style and the course. But it might help some other students that are irritated when watching the freeCodeCamp version. Have a good time and thanks for the course.
https://youtu.be/o0XbHvKxw7Y?si=jR9DIgTzEopPHEMB&t=18292


Two questions

What's the key improvement of TK compared to tensorrt?
Will TK provide some easy-to-use interfaces such as python wrapper?

new architectures, i guess?
you can bind up to pytorch but we're not really planning to write a JIT from python in the near future, so if you want that, Triton is a better bet.


Fix #55 by moving react & react-dom dependencies one level up

This is a workaround for Next's https://github.com/vercel/next.js/issues/9022#issuecomment-559622803

@NOtherDev this is great! Thanks for your help, I am merging this right now.