Fix git diff command in repo linter

Addresses https://github.com/sindresorhus/awesome/pull/1962#issuecomment-830787984
The problem was that you need to add a -- between the branch and the file path.
https://git-scm.com/docs/git-diff
Maybe I removed that part by accident when refactoring the code.
Anyway, this is working now.
Sorry for the hiccup @sindresorhus

Still seems to fail: https://github.com/sindresorhus/awesome/pull/2013/checks?check_run_id=2717401725


Trying to open PNG: Error: Input buffer contains unsupported image format

Version
npm view sharp dist-tags.latest:  0.26.0
What are the steps to reproduce?
      sharp('/Users/s710/test.png')
      .toBuffer()
      .then((res) => {
         setImmediate(() => cb(null, { success: true, binaryResult: res }));
      })
      .catch((err) => setImmediate(() => {
         log.error(self, 'Failed to process image (' + err + ')');
         cb(500);
      }));

Prints: Failed to process image (Error: Input file contains unsupported image format)
What is the expected behaviour?
Should return a buffer containing the PNG image.
It does work fine if I change the first line to sharp('/Users/s710/test.jpg') (that is, open a JPG file instead of a PNG file).
Sysinfo
$ npx envinfo --binaries --system
npx: Installierte 1 in 2.048s

  System:
    OS: macOS 10.15.6
    CPU: (4) x64 Intel(R) Core(TM) i5-7287U CPU @ 3.30GHz
    Memory: 192.05 MB / 16.00 GB
    Shell: 3.2.57 - /bin/bash
  Binaries:
    Node: 12.16.1 - /usr/local/bin/node
    npm: 6.13.4 - /usr/local/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman

What is the output of running node -p "require('sharp').format.png"? Please can you provide the PNG file that fails in this manner?


New package: BetaML v0.1.0

Registering package: BetaML
Repository: https://github.com/sylvaticus/BetaML.jl
Created by: @sylvaticus
Version: v0.1.0
Commit: ccf09ecbc7e0abf4aaa1bc81982f63391f0f8e33
Reviewed by: @sylvaticus
Reference: https://github.com/sylvaticus/BetaML.jl/issues/1#issuecomment-640678287
Release notes:

First registered release of BetaML, the Beta Machine Learning toolkit
(this pull request replace the one made using the old name: https://github.com/JuliaRegistries/General/pull/15790 )

[noblock]
This package has already undergone the 3-day waiting period.


setTimeout ignores additional arguments when run inside Angular class constructor

Hi,
Short version: setTimeout fails inside angular2 class cosntructor, but works fine outside of its context.
Plunker example: https://plnkr.co/gaBThueKGEfYXsG3g4yp?p=preview It contains function "run" which starts coutdown. Once inside angular2 code, once outside. The expected behavior is for countdown to hit zero both times.
ps. Tested at newest chrome/firefox on ubuntu and windows system.
ps. I've found two issues about setTimeout arguments: https://github.com/angular/angular/pull/344 and https://github.com/angular/angular/issues/6088. But I'm not sure if they are related.
ps. I've been redirected here from: https://github.com/angular/angular/issues/7165

Fixed in v0.6.0


WIP: Improved ci

Improving build of jsonic.
@rjrodger : i'm considering replacing jasmine-node by lab.
Do you agree??

$ touch @rjrodger  :) :microscope:

@AdrieanKhisbe  lab - yes please!


remove unneeded GLES headers

I wasn't careful enough while writing #81 and I missed these two headers.
Since GLES was also removed, they are not needed anymore.
Also update, README.md while at it.

Thanks ! and sorry for the extra noise.


fix(auth): changed text colour of button initially to black and on hover we change to white

Before:-
Text colour on button initially used to be white so it was invisible to us.
After:-
Changed the initial text colour to black as it should be visible.
Now on hover we have grey background for button so as we made button colour as black it won't look good.
So on hover made button text as white
Closing issues
closes #2744
Self Check before Merge
Please check all items below before review.

[x] Corresponding issues are created/updated or not needed
[x] Docs are updated/provided or not needed
[x] Examples are updated/provided or not needed
[x] TypeScript definitions are updated/provided or not needed
[x] Tests are updated/provided or not needed
[x] Changesets are provided or not needed

Hey @s-vamshi ,
I have reviewed but the issue persists. Can you check again?

Oh, my mistake @s-vamshi . The original of this project is the storefront example of Next.js. Can we do it like in this example?
https://demo.vercel.store/

Oh, my mistake @s-vamshi . The original of this project is the storefront example of Next.js. Can we do it like in this example? https://demo.vercel.store/

Sure. Will start working on this.

@omeraplak can you please check now?


[SPARK-33722][SQL] Handle DELETE in ReplaceNullWithFalseInPredicate

What changes were proposed in this pull request?

This PR adds DeleteFromTable to supported plans in ReplaceNullWithFalseInPredicate.
Why are the changes needed?

This change allows Spark to optimize delete conditions like we optimize filters.
Does this PR introduce any user-facing change?

No.
How was this patch tested?

This PR extends the existing test cases to also cover DeleteFromTable.

cc @dongjoon-hyun @holdenk @viirya @sunchao @dbtsai @cloud-fan @HyukjinKwon

Test build #132502 has started for PR 30688 at commit 61ca405.

LGTM. Pending tests.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37104/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37104/

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

Test build #132502 has finished for PR 30688 at commit 61ca405.

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

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


This breakpoint will not currently be hit.

I sometimes have to rebuild the entire bits in order break points be available, meanwhile I am editing the same and only .cs file.

We'd need more repro steps to understand what is causing this, but it is unlikely to be an issue with the WinUI product.


How to use CloudFlare DoH resolver on Android?

(Apologies in advance if this isn't the right place to post this, but it's the most relevant place I found.)
I'm using OkHTTP 4.9.0 to do DNS lookups on your DoH resolver. Things work perfectly on the JVM 8+ (tested on macOS, Linux and Windows), but it fails with the following exception on Android (tested on Android 5 and 8):
    Caused by: javax.net.ssl.SSLHandshakeException: java.security.cert.CertPathValidatorException: Trust anchor for certification path not found.
        at com.android.org.conscrypt.ConscryptFileDescriptorSocket.startHandshake(ConscryptFileDescriptorSocket.java:219)
        at okhttp3.internal.connection.RealConnection.connectTls(RealConnection.kt:379)
        at okhttp3.internal.connection.RealConnection.establishProtocol(RealConnection.kt:337)
        at okhttp3.internal.connection.RealConnection.connect(RealConnection.kt:209)
        at okhttp3.internal.connection.ExchangeFinder.findConnection(ExchangeFinder.kt:226)
        at okhttp3.internal.connection.ExchangeFinder.findHealthyConnection(ExchangeFinder.kt:106)
        at okhttp3.internal.connection.ExchangeFinder.find(ExchangeFinder.kt:74)
        at okhttp3.internal.connection.RealCall.initExchange$okhttp(RealCall.kt:255)
        at okhttp3.internal.connection.ConnectInterceptor.intercept(ConnectInterceptor.kt:32)
        at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.kt:109)
        at okhttp3.internal.cache.CacheInterceptor.intercept(CacheInterceptor.kt:95)
        at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.kt:109)
        at okhttp3.internal.http.BridgeInterceptor.intercept(BridgeInterceptor.kt:83)
        at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.kt:109)
    	at okhttp3.internal.http.RetryAndFollowUpInterceptor.intercept

Based on that traceback, I'd say the problem is that the issuer of your certificate isn't on the Conscrypt trust store. I can't imagine things would be different with a different HTTP client, given that the problem is in Conscrypt.
I can work around this issue by using Google's DoH resolver.
Here's the DoH client I'm using in case you want to test this, but you should be able to replicate this with a simple HTTP request to cloudflare-dns.com: https://github.com/relaycorp/doh-jvm
So, how are other people using this resolver on Android?

👋 @gnarea  I'm sorry, we only take care of developers.cloudflare.com documentation in this repo! I recommend reaching out to community.cloudflare.com for additional assistance. Thanks!

👋 @gnarea  I'm sorry, we only take care of developers.cloudflare.com documentation in this repo! I recommend reaching out to community.cloudflare.com for additional assistance. Thanks!

Thanks for the pointer! I've just re-posted it to: https://community.cloudflare.com/t/how-to-use-the-doh-resolver-at-cloudflare-dns-com-on-android/239118

Thanks for the pointer! I've just re-posted it to: https://community.cloudflare.com/t/how-to-use-the-doh-resolver-at-cloudflare-dns-com-on-android/239118


Media elements video container is 1 px to wide

Hi Rafael,
I just tested the new video widget in wp 4.8 (nightly bleed) and i discovered that the video container has inline css which is actually 1px to wide. Tested with different video's. All the same issue. See image below of video widget in footer.

Tested with 20-seventeen and other themes. All the same issue.
In the admin area it is ok. But in the front of the website its not and the 1px is added.  See image of admin area.

Any suggestion?

On mobile the issue shifts to the left of the video

If i then manually adjust the width by 1px it is ok again.

at a certain resize point there are two black borders.

If i then adjust it by 1px then its ok again.

This seems to be something with the theme itself. Does this happen on the version WP has by default of MEJS, or did you upgrade manually to the 4.1.1 version?

Hi
Its happening in any default wordpress theme. I downloaded wp 4.8 from the nightly bleed.  Tested also commercial themes and it happens the same.
https://wordpress.org/plugins/wordpress-beta-tester/

looking forward to your solution :-)

Note: I have to add that I dont know that wp 4.8 is using your latest version of media elements. But above results are what i get. Maybe you can shine your light on this. Thank you for doing so.

OK I'll check this and keep you posted

Hi Rafael
Thank you for the reply.  I have posted this just also on wp core as the more i think about it the more i believe that it might be related to wp? https://core.trac.wordpress.org/ticket/40788#ticket
You might see them reply also.
Thank you for keeping me posted in case you find something.

@BackuPs OK. So I performed everything you said and upgraded to 4.1.1 version (since the nightly build uses 2.22.0) and I can say that it does have the 1px issue on both; however, I'm not sure why that calculation is done that way. What I can tell you is that if you set the following style
.mejs-container {
    background: transparent;
}

you mitigate that issue. I saw that when you resize the window, without the style I just mentioned, sometimes disappear, sometimes comes back. So you may wanna set this just in case. Let me know your thoughts. I have to update the documentation with this change as well.

@BackuPs Update: please download the master branch and update MEJS on your wp-includes/js/mediaelement folder. I found out that a calculation was done with parseInt and should have been done with parseFloat

@rafa8626
I can cinfirm that replacing the mediaelement-and-player.min.js file in the wp-includes/je/mediaelement folder with MEJS 4.1.1 fixes the issue.
Now the question is how do we get the WP Core  Team to adapt this fix.
Thank you for resolving this.

@rafa8626
Actually patch v222 wp uses is also easy i guess. Just add this at line  3038  in the mediaelement-and-player.js
if (t.isVideo) {parentWidth=parentWidth-1;};
So you get this

I dont know if it is a good solution but it works also.

@BackuPs Yeah I wouldn't do something like that; however, can you test the following: in the lines above, there is the definition of newHeight; you will notice that during the definition there's a parseInt. Can you change it to parseFloat and see what happens, please? I'm more in favor of using the latest version, since I'm just guessing here. But I'm glad this has been solved on the latest version

Please star both https://github.com/mediaelement/mediaelement and https://github.com/mediaelement/mediaelement-plugins to continue improving the package and giving it more popularity; also tell your friends/coworkers/etc. that have a GitHub account to do the same to support the project, if you don't mind. It would be greatly appreciated it.

The moment wp loads mejs v 222 and the theme loads mejs v 4.1.1 there are weird issues. The video in the sidebar suddenly is having wrong width and height. So one has to unload the wp scripts if newer versions are loaded. Its better that wp starts following latest build of mejs. But they are always so slow to update their 3rd party scripts....

Thanks I appreciate it. The more visibility we gave the player and it's plugins, the more chances for improvement for both. If you wanna comment your final thoughts on WordPress it's fine. Closing this issue now.

Hi Rafael
I am sorry but I have to reopen this... as replacing the script in the wp js mediaelements folder with the one from version 3.1.1 does result in wp not loading the elements scripts anymore and revert to default video layout.

So that solution does not work.
The problem we have is that we either need to fix the js file wp is using or get wp to use the latest media elements files as when i have two versions of the script loaded (one in the theme using the new mejs and wp using the v 221) at the same time in one page the video in the sidebar (the one wp is loading by the wigdet) doubles in size.

There is no way to unload the wp-mediaelements and have them load the latest one. The filter only allows unloading it but not enqueueing a new version which would fix the issue. At least i haven found one.
This causes me to keep on using the older version in the theme and revert all my changes. But then the one pix issue is back.
So far wp did not respond at all to the ticket created.
Any thoughts on this?

I think it all comes down to this....https://core.trac.wordpress.org/ticket/39686
They just marked the 1px thread to a duplicate. Which is not... Apparently they dont understand the issue reported.

Yeah just provide an explanation on the ticket about this. The WP files you need to replace are in the documentation related to install Wordpress. It's a zip file with all of them


feature/detailed output python - draft

Hope it is okey, but just to give you an idea how I would like this feature.

Python test now takes detailed_test_output: bool as a parameter.
For this to work, the output from the python-macro needs to be a dataframe(pandas or spark). If this is not the case, it will throw an error.
The outputed dataframe will be saved to a separate table.
Instead of just "fail_count" in the edr-report, a sql-query will have an sql-query for that table.

This is just rough draft, but works. Things that could be nice to add is possibility to chose which columns that should be saved, and maybe a limit to avoid saving hundreds of rows.

https://github.com/elementary-data/elementary/issues/701

Closing this, I just got what Elon said in the feature-request. Will come a new MR soon.


Bintray jcenter -> Maven Central synchronization broken

Bintray jcenter contains the newest version available (2.22.6-alpha - lower minor number but released after 2.24.2) - https://bintray.com/netflixoss/maven/conductor/2.22.6-alpha
Latest version avaiable on Maven central version is 2.24.0 (2.24.2 is missing) - https://search.maven.org/search?q=g:com.netflix.conductor
Could this be fixed? Thanks!

@apanicker-nflx any news on this one? Thanks!

@apanicker-nflx  Can you help fix this problem as we need the latest versions for our deployments? Thanks!

Apologies for the delays, we will get back with an update on this soon.

FYI it seems synchro work for alpha releases, but not the standard ones.

@Valarin @mdepak We are still trying to figure out why maven publishing is broken.
As a temporary relief, you can use v2.24.5-alpha.2 which is available in maven and is the same as v2.24.5

Also, v2.24.5 is available on maven - https://mvnrepository.com/artifact/com.netflix.conductor/conductor-core/2.24.5

@apanicker-nflx Thank you for publishing the latest version in maven.


TMPE president edition mod not working/showing.

Whenever i subscribe to the mod and turn it on, i go into city skylines, but the mod does not show at the bottom toolbar.
I want to make a road a right turn only.
Screenshots?

Notes or questions?
Can you please give a fix.

Did you enable the mod on the Mods tab in the Main menu??

All my current mods dont for som reason show up in the content manager, so when io load in my world, i havce to hope that they worked

Did you enable the mod on the Mods tab in the Main menu??
All my current mods dont for som reason show up in the content manager, so when io load in my world, i havce to hope that they worked The only place i see my mods is in the steam workshop at the bottom of the content manager page on the categories tab

It's not the mod issue in any way. The reason is quite obvious. If there are no mods downloaded then they won't work, so make sure that Steam downloaded things you subscribe.
Try changing the download region city in the Steam client Downloads settings, and see if it started downloading workshop content. It's clearly problem with Steam, not the first time,

can you give me step by step instructions to fix

hello?

This is not a support chat.
For chat experience please join discord at https://discord.gg/faKUnST

can you give me step by step instructions to fix. Oh yeah, when i download a mod i see it download at the bottom of the screen, but when i subscribe to this mod, i dont see it download at the bottom of the screen. how do i get the mod to download

I won't post step by step instructions because it's not the problem with the mod in any way. There's nothing I can do.
As I said, what you experience is classic Steam client workshop synchronization issue.
The problem is pretty common when Steam servers are experiencing increased load, usually during Steam sales or something broke on your side e.g.: some configuration file got corrupted. I don't know how to fix it, search game forums or try CS discord server, they might know how to help you.


NOJIRA Corrected minor typo - rel-4-3-patches

Checklist

[x] the individual contributor license agreement is signed
[x] commit message follows commit guidelines
[ ] tests are included
[ ] documentation is changed or added
[ ] message properties have been updated with new phrases
[ ] view conforms with WCAG 2.0 AA

Description of change

Corrected minor typo in PortletDefinitionRegistryImpl

@jonathanmtran does this also need to be corrected in master?

Yep. PR coming soon


Smart pipette in upgrade planner does not function

Description
What happens:
If I use pipette (q) on an item in the upgrade planner, either nothing happens or it pipettes the tile below the window.

What I expected:
The entity-prototype is put onto my cursor as if I had done the pipette on something in a blueprint.

Reproduction
No response

This is because the game is not providing selected_prototype in the upgrade planner. This is not something I can fix on the mod side.


Showing Chinese as default language when the device default language is unsupported

Hi,
I found an issue is that, when the phone's language selected Indonesia, it showing Chinese instead of English, please check out the image attached below.
Could anyone please help on this issue? Thanks in advance!

Thanks and Regards,
Bevis.

Same problem here.
The only solution I found was to "force" English language in the android native code
https://stackoverflow.com/a/9173571

Check new version: 2.0
Docs: https://baronha.github.io/react-native-multiple-image-picker/

@baronha I still have the issue on my side even if I force the language with the language option.


Server action not found when using App Route with i18n

Description
When I have i18n routing setup with server actions, it is giving me a 404 error. It seems to be an issue with the locales dynamic routes? I am also using unstable_setRequestLocale in my server component. I have tried with and without static generation and neither work.
Server action also works locally but not when deployed

Verifications

[X] I've verified that the problem I'm experiencing isn't covered in the docs.
[X] I've searched for similar, existing issues on GitHub and Stack Overflow.
[X] I've compared my app to a working example to look for differences.

Mandatory reproduction URL
https://github.com/0xHoneyJar/minimal-reprod
Reproduction description
Steps to reproduce:

Open reproduction
Deploy to vercel
Click server action button in top left corner

Expected behaviour
The server actions should be found and operate normally.

I'm getting a 200 in your deployed example:

Did you fix something after posting this?
If a 404 is triggered, it's typically caused by the middleware not matching a given request (see "Unable to find next-intl locale because the middleware didn't run on this request.").

I'll move this to a discussion since it seems to be a usage question and not a bug report, esp. since you mention that it works locally.


s390x: implement OPENSSL_rdtsc as STCKF

STCK has an artificial delay to ensure uniqueness which can result in a performance penalty if used heavily concurrently.
Artificially delaying a high precision clock is a counter-intuitive thing to do for a timer function like rdtsc. Uniqueness is only required for nonces (which are better implemented using a counter anyway) and its effect on clock based entropy harvesting is not clear.
On the other hand, STCK has been observed to slow down applications which use it concurrently. libcrypto seems to use it in a safe way, but better replace it by STCKF if available.

Merged. Thanks!


ISPN-10151 Implement REST counter operations

https://issues.jboss.org/browse/ISPN-10151

@tristantarrant Comments need addressing

@galderz @pruivo @ryanemerson Addressed all issues.

integrated! thanks @tristantarrant

@tristantarrant is this needed in 9.4.x?

No
On Tue, May 21, 2019 at 10:36 AM Pedro Ruivo<EMAIL_ADDRESS>wrote:

@tristantarrant https://github.com/tristantarrant is this needed in
9.4.x?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/infinispan/infinispan/pull/6867?email_source=notifications&email_token=AAC72HUNXRRWFS6TUPNBNP3PWOX7PA5CNFSM4HJBZH22YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODV3FXBI#issuecomment-494295941,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAC72HRIUIXXQVPAQTM53ITPWOX7PANCNFSM4HJBZH2Q
.


Add snippets for the derive and allow attributes

The added snippets will expand to the following:
derive -> #[derive(trait)]
allow -> #[allow(lint)]
allow! -> #![allow(lint)]

Thanks. I also added snippets for deny and warn attributes.


Error when verifying arduino code

Have the Dmx Simple, FastLED and Adafruit_LSM303 libraries installed and I can see them from the Sketch menu of Arduino 1.0.6 but when I verify the code I got this error.
Arduino: 1.0.6 (Mac OS X), Board: "Arduino Uno"
Arduino.app/Contents/Resources/Java/hardware/arduino/cores/arduino Arduino.app/Contents/Resources/Java/hardware/arduino/variants/standard -
Arduino/libraries/DmxSimple -I/Users/macanthropus/Documents/Arduino/libraries/FastLED-3.1.0 -Arduino.app/Contents/Resources/Java/libraries/Wire -
Arduino/libraries/Adafruit_LSM303-master /var/folders/s3/lqlyl00127dc8wtfdk_ygtl00000gn/T/build7414988327060524094.tmp/sketch_jan08a.cpp -o /var/folders/s3/lqlyl00127dc8wtfdk_ygtl00000gn/T/build7414988327060524094.tmp/sketch_jan08a.cpp.o
In file included from sketch_jan08a.ino:2:
Arduino/libraries/FastLED-3.1.0/FastLED.h:12:2: warning: #warning FastLED version 3.001.000 (Not really a warning, just telling you here.)
sketch_jan08a:427: error: 'BLEND' was not declared in this scope
sketch_jan08a.ino: In function 'void ChangePalettePeriodically()':
sketch_jan08a:460: error: 'BLEND' was not declared in this scope
sketch_jan08a:464: error: 'BLEND' was not declared in this scope
sketch_jan08a:472: error: 'BLEND' was not declared in this scope
sketch_jan08a:476: error: 'BLEND' was not declared in this scope
sketch_jan08a:480: error: 'BLEND' was not declared in this scope
sketch_jan08a:488: error: 'BLEND' was not declared in this scope
sketch_jan08a:492: error: 'BLEND' was not declared in this scope
sketch_jan08a:496: error: 'BLEND' was not declared in this scope
sketch_jan08a:504: error: 'BLEND' was not declared in this scope

Try changing BLEND to LINEARBLEND.  It got renamed a while back: https://github.com/FastLED/FastLED/issues/131#issuecomment-101862204
If that works, feel free to submit a pull request.

Thanks. Now I got this message
/Users/macanthropus/Documents/Arduino/libraries/DmxSimple/DmxSimple.cpp:11:20: error: wiring.h: No such file or directory
/Users/macanthropus/Documents/Arduino/libraries/DmxSimple/DmxSimple.cpp: In function 'void dmxBegin()':
/Users/macanthropus/Documents/Arduino/libraries/DmxSimple/DmxSimple.cpp:57: error: 'digitalPinToPort' was not declared in this scope
/Users/macanthropus/Documents/Arduino/libraries/DmxSimple/DmxSimple.cpp:57: error: 'portOutputRegister' was not declared in this scope
/Users/macanthropus/Documents/Arduino/libraries/DmxSimple/DmxSimple.cpp:58: error: 'digitalPinToBitMask' was not declared in this scope
/Users/macanthropus/Documents/Arduino/libraries/DmxSimple/DmxSimple.cpp:61: error: 'OUTPUT' was not declared in this scope
/Users/macanthropus/Documents/Arduino/libraries/DmxSimple/DmxSimple.cpp:61: error: 'pinMode' was not declared in this scope
/Users/macanthropus/Documents/Arduino/libraries/DmxSimple/DmxSimple.cpp: In function 'void dmxWrite(int, uint8_t)':
/Users/macanthropus/Documents/Arduino/libraries/DmxSimple/DmxSimple.cpp:183: error: 'max' was not declared in this scope
/Users/macanthropus/Documents/Arduino/libraries/DmxSimple/DmxSimple.cpp: In function 'void dmxMaxChannel(int)':
/Users/macanthropus/Documents/Arduino/libraries/DmxSimple/DmxSimple.cpp:194: error: 'min' was not declared in this scope

Ah, try this version of the DmxSimple library instead: https://github.com/PaulStoffregen/DmxSimple

It compiled! :)

Cool!  I'll get the readme updated to point to the proper library.


Better error messages for unsupported syntax

I've gone through all of the TypeScript features that are not currently supported by the compiler and ensured that a more sensible error message than Unsupported syntax node X is shown to the user. In some cases I've also added a suggestion of alternate syntax the user could use.

Hi @thomasdenney, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

@pelikhan Now calling lf(). I wasn't able to switch to pxt.log as it caused Travis to fail (see build report for https://github.com/Microsoft/pxt/pull/165/commits/a643338d47fcfdff8c1a9e1ddfe1570f5da93639)

@pelikhan Made the changes to the calls, but the localisation scanner won't pick them up because it only scans webapp/src by the looks of things.


replace the python covidcast client in google-symptoms

similar to #1931 and #1972 , replace the usage of the old covidcast client in the google-symptoms indicator.  it is only really used in one place: https://github.com/cmu-delphi/covidcast-indicators/blob/1526dc940f61e5a1ab8ff2ce795f35c965c2e48e/google_symptoms/delphi_google_symptoms/run.py#L69-L70
its a pretty basic usage, and covidcast.metadata() doesnt do a whole lot more than Epidata.covidcast_meta()

blocked by #1972


Bug: can't use Terraform variable on dev_db_url property

Hello,
I have the following message Warning: dev_db_url is unset on terraform plan when i use variable on dev_db_url property on atlas_schema ressource (variable on url property work fine).
Example :
resource "atlas_schema" "nifi" {
  hcl = file("${path.module}/db_atlas_schema.hcl")
  url = local.main_instance_url
  dev_db_url = "postgres://pgadmin:${data.vault_kv_secret_v2.secret_pgadmin_user.data.secret}@${var.instance_address}:5432/atlas?sslmode=disable)"
}

hey @noiretr-cg
thanks for reaching out. What version of the provider are you using?

Hello @noiretr-cg, I think the warning is falsy. Just ignore it for the next version, thank you. 🙏

Hi,
0.31 for the Terraform provider and 0.8.0 for Altas

Ok, thanks for your support :+1:


Typst export breaks DOI links with text

Description
The MyST link
This is [a link to a DOI](https://doi.org/10.5281/zenodo.10081003)

loses the inline text following changes in #880
Proposed solution
Preserve link-like behavior for DOI links with non-empty labels.
Additional notes
The generated citation also seems invalid, I suspect because a URL is not a valid label. Perhaps we need to escape it?
Here's a DOI #cite(<https://doi.org/10.5281/zenodo.10081003>, form: "prose")

@fwkoch any ideas? :)

Fixed by https://github.com/executablebooks/mystmd/pull/999


edit channel permissions

Codecov Report

Merging #102 (cdb6e91) into public-channels (d8ea746) will increase coverage by 0.52%.
The diff coverage is 97.43%.

@@                 Coverage Diff                 @@
##           public-channels     #102      +/-   ##
===================================================
+ Coverage            82.97%   83.50%   +0.52%
===================================================
  Files                   80       81       +1
  Lines                 3136     3164      +28
  Branches               513      517       +4
===================================================
+ Hits                  2602     2642      +40
+ Misses                 422      410      -12
  Partials               112      112

Impacted Files
Coverage Δ

app/helpers/channels.py
78.84% <88.88%> (+0.84%)
:arrow_up:

app/helpers/permissions.py
89.37% <100.00%> (+0.13%)
:arrow_up:

app/routers/channels.py
97.77% <100.00%> (+0.21%)
:arrow_up:

app/schemas/permissions.py
100.00% <100.00%> (ø)

app/services/channels.py
82.60% <100.00%> (+1.40%)
:arrow_up:

app/services/messages.py
85.24% <0.00%> (+1.63%)
:arrow_up:

app/helpers/queue_utils.py
62.16% <0.00%> (+12.16%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d8ea746...cdb6e91. Read the comment docs.


Docs.MS CI Updates for Azure.AI.TextAnalytics

CI Update for release of Azure.AI.TextAnalytics

Docs Build status updates of commit 801f630:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

bundlepackages/azure-dotnet-preview.csv
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Some improvements to our logs

Motivated as part of going through the logs with my production hat on.

bors r+


takeScreenshot does not return image path?

As far as I can see takeScreenshot does not return path of image?
It's return type of Promise<void>, also some of code examples are still using it like it returns path, but it doesn't.

@noomorph

@noomorph

Oh. 😸 We only recently started supporting external typings, and since it was a community product earlier, there still might be discrepancies between the real Detox API and the type declarations.
Anyway, thanks to your reports we'll align them gradually.
Good that you've raised this issue. 👍

Oh. 😸 We only recently started supporting external typings, and since it was a community product earlier, there still might be discrepancies between the real Detox API and the type declarations.
Anyway, thanks to your reports we'll align them gradually.
Good that you've raised this issue. 👍


PR unmerged

Expected Behavior
1.1.0 contains latest PR with fixed direction computed value
Actual Behavior
1.1.0 is same as 1.0.4
Environment

OS: Write here
NPM version: Write here
vue.js version: Write here
vue-parallax version: Write here

Should be fixed now.


Json to Dart generate model failed

Version
5.1.6
Platforms
dart, Android
Device Model
Oppo A12
flutter info
[√] Flutter (Channel stable, 3.22.1, on Microsoft Windows [Version 10.0.22631.3447], locale en-IN)
    • Flutter version 3.22.1 on channel stable at D:\SDK\flutter_windows_3.22.1-stable\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision a14f74ff3a (13 days ago), 2024-05-22 11:08:21 -0500
    • Engine revision 55eae6864b
    • Dart version 3.4.1
    • DevTools version 2.34.3

[√] Windows Version (Installed version of Windows is version 10 or higher)

[!] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at C:\Users\panch\AppData\Local\Android\Sdk
    X cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    X Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/windows#android-setup for more details.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[X] Visual Studio - develop Windows apps
    X Visual Studio not installed; this is necessary to develop Windows apps.
      Download at https://visualstudio.microsoft.com/downloads/.
      Please install the "Desktop development with C++" workload, including all of its default components

[√] Android Studio (version 2022.3)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-10027231)

[√] VS Code (version 1.89.1)
    • VS Code at C:\Users\panch\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.90.0

[√] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.22631.3447]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 125.0.6422.78
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 125.0.2535.79

[√] Network resources
    • All expected network resources are available.

! Doctor found issues in 2 categories.

How to reproduce?
I found bug when i clicked on Build > FlutterBeanFactory

Logs
No response
Example code (optional)
No response
Contact
<EMAIL_ADDRESS>

If "No classes that inherit JsonConvert were found" is displayed, delete the ". Idea "directory under the project and click" invalidate Caches"in your (Andorid Studio or IDEA) button to restart the IDE


Some areas within pipestat do not disambiguate between sample-level and project-level

          Some areas within pipestat do not disambiguate between sample-level and project-level

Originally posted by @donaldcampbelljr in https://github.com/pepkit/pipestat/issues/37#issuecomment-1564357457

For reference, some outstanding TODO comments to investigate:

Disambiguation could use some improvement via refactoring. Generally, it is a boolean. However, pipestat is also using that value to set a string as either 'Sample' or 'Project' during the pipestat data object construction. When creating these strings, it may be better to always refer to them as level instead of project level and only use project_level when dealing with booleans to avoid confusion.

This should be fixed with 22e418d


Update Copyright

Update copyright to say the class, project & group names.

I moved this issue under this ticket - https://github.com/gwengibbes/com6338-module10-assignment10-gibbes-longo-avery/pull/59
The corresponding branch would no longer work for me in VSC.


StreamRecorded with latest CODAL contains pulses at regular intervals

With the latest CODAL StreamRecorded  raw data has peaks every ~23ms.
Bottom is a build with the latest CODAL, top is using the StreamRecorder hex from:

https://github.com/microbit-foundation/dev-docs/issues/169

Old CODAL wav file: old-build.wav.zip

Old CODAL raw data:<PHONE_NUMBER>624e45005160190000003b000000009796990b.txt

Current CODAL wave file: latest-codal.wav.zip

Current CODAL raw data:<PHONE_NUMBER>0528202803be62c2d82852000000006e052820.txt

#include "MicroBit.h"
#include "SerialStreamer.h"

MicroBit uBit;

int main() {
    uBit.init();

    SerialStreamer *streamer = new SerialStreamer(uBit.audio.processor->output, SERIAL_STREAM_MODE_BINARY);
    uBit.audio.requestActivation();
    uBit.audio.activateMic();

    while (true) {
        uBit.sleep(1000);
    }
}

MICROBIT.hex.zip

PR:

https://github.com/lancaster-university/microbit-v2-samples/pull/72


Identify components that have logs flowing into customer log system

In order to get a better understanding of our logging system, we need to document what components feed data into it
Background

Presidential Memorandum M-21-31.
Notes from discussion with assurance team about M-21-31

Acceptance Criteria

[ ] Documentation or a diagram exists created to show what components feed data into our customer logs system

Security considerations
Diagramming our logging system makes it more intelligible to operators and easier to maintain. Both outcomes ultimately improve our ability to assess and to remediate security risks in the system.
Implementation sketch

[ ] Research/identify system components that feed logs into our customer logs system
[ ] Create documentation or a diagram of what components feed logs into our customer logs system

This is a diagram of how customer data flows through our system: https://diagrams.fr.cloud.gov/10-4.1-customer-data-flow.html


broken efcore-docs-sqlserver-save-changes-and-computed-columns link

here https://github.com/dotnet/efcore/blob/main/src/EFCore.SqlServer/Properties/SqlServerStrings.Designer.cs#L215 there is a link to  https://aka.ms/efcore-docs-sqlserver-save-changes-and-computed-columns which is broken

Fixed.


Native system libraries should be opt-in instead of configured by default

Expected Behavior
As an example, it is possible to build native project without using Windows SDK.

Current Behavior
The system library and the toolchains are tightly couple together which makes it almost impossible to opt-out as expressed in this forum post.

Context
It is completly valid to build application without using the Windows SDK. It also make kernel development more complicated where the Windows DDK should be use instead.

Steps to Reproduce (for bugs)
N/A

Your Environment

See this forum post: https://discuss.gradle.org/t/native-compilation-with-msvc-and-arm/20092

c compiler need to be flexible rather than binding to gcc native toolchain, Solaris compiler CC, cc and etc shall be able to work in gradle as well.
https://discuss.gradle.org/t/use-custom-toolchain-in-gradle-no-gcc-or-clang/21139


Update for Exchange 2016 CU 15 ActivationPreference

Hello. I have upgraded to Exchange 2016 CU 15 and the Activation Preference is no longer showing.
The line in the script that gets the Activation Preference i is not getting the preference.
$pref = ($database | Select-Object -ExpandProperty ActivationPreference | Where-Object {$_.Key -ieq $mailboxserver}).Value
If I run the script using an Exchange 2010 shell is works but not when it is Exchange 2016. Can someone help me fix the script so I get the preference again.

I am new to GitHub, I am trying to get help in fixing an issue I see in the Test-ExchangeServerHealth script related to the activation preference not showing in the report after an upgrade to Exchange 2016 CU 15.


DisableSharingForNonOwnersStatus Change

Learn Build status updates of commit 645a8e1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-online/Get-SPOSite.md
:white_check_mark:Succeeded
View (sharepoint-ps)

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

@JohanFreelancer9

@dariomws @tseward @get-itips

Learn Build status updates of commit 692a213:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-online/Get-SPOSite.md
:white_check_mark:Succeeded
View (sharepoint-ps)

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

@tiburd Please merge this PR. Thanks

Learn Build status updates of commit 1a3c3b8:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-online/Get-SPOSite.md
:white_check_mark:Succeeded
View (sharepoint-ps)

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


Bug when calling get on only entry in LruObject

Hey folks,
first of all thanks for building and maintaining this library ❤

I stumbled across this issue when investigating failures coming from fastify-rate-limit. I created an issue there because thats where I noticed the impact first.
However based on my current understanding there is an issue in the LruObject implementation - more specifically in the get implementation. Here is a simple test case that I wrote:
  describe('get', () => {
    it('does not overwrite cache.first', async () => {
      cache = new LruObject(5, 500)

      const key = '<IP_ADDRESS>'
      const value = 100

      cache.set(key, value)
      expect(cache.first).not.toBeNull()

      cache.get(key)
      expect(cache.first).not.toBeNull()
    })
  })

 FAIL  test/LruObject.spec.js > LruObject > get > does not overwrite cache.first
AssertionError: expected null not to be null
 ❯ test/LruObject.spec.js:77:31
     75|
     76|       cache.get(key)
     77|       expect(cache.first).not.toBeNull()
       |                               ^
     78|     })
     79|

Can you confirm that this is indeed not intended?

this is especially problematic once the cache limit is reached as the evict() call results in a failure
  describe('get', () => {
    it('does not cause TypeError when reaching the cache limit', async () => {
      const maxCacheSize = 3
      cache = new LruObject(maxCacheSize, 500)

      const key = '<IP_ADDRESS>'
      const value = 100

      cache.set(key, value)
      cache.get(key)

      for (let i = 0; i < maxCacheSize; i++) {
        cache.set(i, i)
      }
    })
  })

 FAIL  test/LruObject.spec.js > LruObject > get > does not cause TypeError when reaching the cache limit
TypeError: Cannot read properties of null (reading 'key')
 ❯ LruObject.evict src/LruObject.js:85:30
     83|       const item = this.first
     84|
     85|       delete this.items[item.key]
       |                              ^
     86|
     87|       if (--this.size === 0) {
 ❯ LruObject.set src/LruObject.js:149:12
 ❯ test/LruObject.spec.js:91:15


Stack stop requested

@bbihari plz review

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/2543/


Slice lengths are broken

--   ::= import "core.ic"
io   ::= import "io.ic"

foo := "abc"
io.Print(foo.length, "\n")

bar := builtin.slice(foo.data, 2)
io.Print(bar.length, "\n")

Output:
3
32062544

I'm not sure if this is an issue with the SliceLengthInstruction or the builtin.slice() function, or something else entirely.

Replacing the numeric literal 2 with n and declaring n: u64 = 2 above works around the issue, but n :: u64 = 2 or n ::= 2 do not. Also explicitly passing 2 as u64 works. This makes me think it's something happening with the implicit cast from integer to u64.


Strange behaviour of NavigationAgent3D around NavigationObstacle3D

Godot version
4.0.beta3
System information
macOS Monterey 12.6
Issue description
When navigation agent should go around moving obstacles it almost seems it pushes into obstacle until it is no longer there (i tested it by disabling collisions on them but visually agent behaved the same). I would expect agent to try to "get around" dynamic obstacle rather than wait for it to clear.
Steps to reproduce
Option 1:
Have agent go around 2 obstacles that are moving up and down (not sure direction has effect) each being down at different time. Their movement is done using animation player animating position attribute of parent node like so:

You will notice agent trying to get "through" obstacle rather than go around it.
Option 2:
Open repro project, start it and you can order red ball (agent) to go around level by lef mouse clicking on floor (clicking on purple walls and ramps will not react, only floor does).
Minimal reproduction project
Navmesh.zip

Avoidance != Pathfinding
Avoidance does not change the pathfinding only the velocity. Your path insists that the agent moves through the obstacles und enforces this every single _physics_frame update.
3D avoidance shapes are radius / spheres. Your box / rect shaped obstacles only have this shape for visuals or physics collision but for avoidance calculation they are in fact very large spheres that only leave tiny gaps for the agent to move through when they are "up".
When the agent is close to the obstacle and the obstacle sphere moves down it "squeezes" the agent. The correct 3D avoidance would be to move through the floor but it obviously cant so it gives up with the suggested avoidance "safe_velocity" and moves on. If you have "ignore_y" enabled the safe_velocity will ignore the y-axis, so the agent basically stands still and waits for better times or until the obstacles moves its radius out of the overlap with the agent.

I understand all this (i was author of c++ navigation integration for one of AAA titles released not long ago). What seems to be big problem for me is that avoidance sphere cannot be seen. Maybe drawing them at least as part of navmesh debug rendering, ideally even in editor would help a lot.
Btw so avoidance is using navmesh itself for avoiding obstacles (seems it is not the case) but then again, i dont see the sphere so it is har to debug what system see internally 🤔
Great work btw!

Maybe drawing them at least as part of navmesh debug rendering, ideally even in editor would help a lot.

Good idea <3

Btw i tried to add spheres roughly size that would encapsulate whole obstacle. There is visible enough space for agent to fit through when obstacle is up but it tries to push itself into it instead of going around using navmesh. Hope it helps with debugging.

Yes, it is difficult to find good avoidance property values without any debug visuals. Debug visuals for the editor are on the "to-do" list. Even with good property values and setup it is difficult with the current avoidance to get a good result.
The RVO2-3D library currently integrated in Godot is intended for air, water or space agents and has issues with flat surface 2D avoidance in general as soon as things get more crowded or something pushes from above or below. It also has no support for hard-surface obstacles to build e.g. navmesh constrains. All current NavigationObstacles are just a (soft) avoidance radius created by a fake agent with some properties set at values that it can push most other agents but not all.
These are issues that are outside of what a user can control or change without digging into the source code. As shown in the linked rvo rework proposal there are a few things currently worked on that should improve the avoidance situation for Godot 4.

I see, thank you for great details!😊
Will be looking forward for further improvements then 👍

The new avoidance is more detailed without much jitter if in close contact with an obstacle even when following the contour of moving avoidance obstacles very closely which should also help this issue here. There is no collision in this animation here, this is only avoidance movement.

Looks nice! Will test it as soon as it is available in beta build 👍


Out-GridView filtering doesn't allow entering strings with embedded spaces

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
Entering filtering strings such as b c in Out-GridView filtering boxes doesn't work: the space character is "swallowed."
Workaround: enter bc, backspace between the b and the c and enter the space character.
This issue is present in 7.4.2 and 7.5.0-preview.3.
This works fine in Windows PowerShell 5.1.
To repro:

run @('a','b c') | ogv
type b <space> c into the 'filter box'

The correct behavior is:

The incorrect behavior of PowerShell 7 is:

Expected behavior
The filtering box contains "b c"

Actual behavior
The filtering box contains "bc"

Error details
No response
Environment data
Name                           Value
----                           -----
PSVersion                      7.4.2
PSEdition                      Core
GitCommitId                    7.4.2
OS                             Microsoft Windows 10.0.22621
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
No response

Curious bug; fortunately, Out-ConsoleGridView (the cross-platform, TUI counterpart to Out-GridView) is not affected.
The workaround for now is: use the left arrow to position the cursor between the b and the c, which then allows you to insert a space.

Added to the Cmdlet Working Group to review.


Added Calendar Component to Opportunities Page

This is just for adding the Calendar Component, that @mkerwoodchamp recently added to the project files, to the Opportunities page so we can have an idea of appearance when working in the project.

For an idea of how this PR looks in the current project, look at the comment PR #40 with the screenshot of the local project.


Invalid argument: No OpKernel was registered to support Op 'PyFunc' used by {{node PyFunc}}with these attrs: [Tin=[DT_FLOAT], Tout=[DT_INT32], token="pyfunc_0"] Registered devices: [CPU, GPU] Registered kernels:     	 [[PyFunc]]

Click to expand!
Issue Type
Support
Source
source
Tensorflow Version
1.14
Custom Code
Yes
OS Platform and Distribution
Linux Ubuntu 18.04
Mobile device
No response
Python version
3.6
Bazel version
0.26.0
GCC/Compiler version
7.5
CUDA/cuDNN version
10.0
GPU model and memory
1050ti 4GB
Current Behaviour?
2022-05-24 17:04:39.195755: I tensorflow/stream_executor/platform/default/dso_loader.cc:42] Successfully opened dynamic library libcuda.so.1
2022-05-24 17:04:39.220670: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:1005] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2022-05-24 17:04:39.220885: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1640] Found device 0 with properties:
name: NVIDIA GeForce GTX 1050 Ti major: 6 minor: 1 memoryClockRate(GHz): 1.62
pciBusID: 0000:01:00.0
2022-05-24 17:04:39.270904: I tensorflow/stream_executor/platform/default/dso_loader.cc:42] Successfully opened dynamic library libcudart.so.10.0
2022-05-24 17:04:39.500409: I tensorflow/stream_executor/platform/default/dso_loader.cc:42] Successfully opened dynamic library libcublas.so.10.0
2022-05-24 17:04:39.611684: I tensorflow/stream_executor/platform/default/dso_loader.cc:42] Successfully opened dynamic library libcufft.so.10.0
2022-05-24 17:04:39.662292: I tensorflow/stream_executor/platform/default/dso_loader.cc:42] Successfully opened dynamic library libcurand.so.10.0
2022-05-24 17:04:39.919741: I tensorflow/stream_executor/platform/default/dso_loader.cc:42] Successfully opened dynamic library libcusolver.so.10.0
2022-05-24 17:04:40.068422: I tensorflow/stream_executor/platform/default/dso_loader.cc:42] Successfully opened dynamic library libcusparse.so.10.0
2022-05-24 17:04:40.553116: I tensorflow/stream_executor/platform/default/dso_loader.cc:42] Successfully opened dynamic library libcudnn.so.7
2022-05-24 17:04:40.553460: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:1005] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2022-05-24 17:04:40.554410: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:1005] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2022-05-24 17:04:40.555188: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1763] Adding visible gpu devices: 0
2022-05-24 17:04:40.591205: I tensorflow/stream_executor/platform/default/dso_loader.cc:42] Successfully opened dynamic library libcudart.so.10.0
2022-05-24 17:04:40.705261: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1181] Device interconnect StreamExecutor with strength 1 edge matrix:
2022-05-24 17:04:40.705335: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1187]      0
2022-05-24 17:04:40.705356: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1200] 0:   N
2022-05-24 17:04:40.716567: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:1005] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2022-05-24 17:04:40.717333: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:1005] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2022-05-24 17:04:40.717987: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:1005] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2022-05-24 17:04:40.718531: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1326] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:0 with 2732 MB memory) -> physical GPU (device: 0, name: NVIDIA GeForce GTX 1050 Ti, pci bus id: 0000:01:00.0, compute capability: 6.1)
2022-05-24 17:04:40.779337: E /home/zhj/lidarDetect/pointnet_cpp/fpointnet.cpp:106] Invalid argument: No OpKernel was registered to support Op 'PyFunc' used by {{node PyFunc}}with these attrs: [Tin=[DT_FLOAT], Tout=[DT_INT32], token="pyfunc_0"]
Registered devices: [CPU, GPU]
Registered kernels:
  <no registered kernels>

	 [[PyFunc]]
------------------------------------------------------------------------------------------------------------------------------------------------------------------------
I use freeze_graph to generate pb from cpkt which include tf.py_func.Then I use LoadGraph to load the pb model to infernece by c++.And it make the above bug.

Standalone code to reproduce the issue
############################################
The following code is the function which using tf.py_func.
##############################################
def tf_gather_object_pc(point_cloud, mask, npoints=512):
    ''' Gather object point clouds according to predicted masks.
    Input:
        point_cloud: TF tensor in shape (B,N,C)
        mask: TF tensor in shape (B,N) of 0 (not pick) or 1 (pick)
        npoints: int scalar, maximum number of points to keep (default: 512)
    Output:
        object_pc: TF tensor in shape (B,npoint,C)
        indices: TF int tensor in shape (B,npoint,2)
    '''
    def mask_to_indices(mask):
        indices = np.zeros((mask.shape[0], npoints, 2), dtype=np.int32)
        for i in range(mask.shape[0]):
            pos_indices = np.where(mask[i,:]>0.5)[0]
            # skip cases when pos_indices is empty
            if len(pos_indices) > 0:
                if len(pos_indices) > npoints:
                    choice = np.random.choice(len(pos_indices),
                        npoints, replace=False)
                else:
                    choice = np.random.choice(len(pos_indices),
                        npoints-len(pos_indices), replace=True)
                    choice = np.concatenate((np.arange(len(pos_indices)), choice))
                np.random.shuffle(choice)
                indices[i,:,1] = pos_indices[choice]
            indices[i,:,0] = i
        return indices

    indices = tf.py_func(mask_to_indices, [mask], tf.int32)
    object_pc = tf.gather_nd(point_cloud, indices)
    return object_pc, indices

Relevant log output
node name: "PyFunc"
op: "PyFunc"
input: "Squeeze_1"
attr {
  key: "Tin"
  value {
    list {
      type: DT_FLOAT
    }
  }
}
attr {
  key: "Tout"
  value {
    list {
      type: DT_INT32
    }
  }
}
attr {
  key: "token"
  value {
    s: "pyfunc_0"
  }
}

Hi @ZHUANG-JLU! Thank you for raising the issue.
TF v1.14 is an older version of TF which is not actively supported. We recommend you to kindly upgrade to the latest TF versions(2.4 or later) and let us know if the issue still persists in newer versions. For more details please refer to the migration document. Thank you!

The PyFunc kernel corresponds to a tf.py_func in your model, which requires a Python interpreter to execute.
So, it seems the model you're using requires the Python interpreter, which is not linked into the Java API and hence you see the error.
You'd want to figure out where in your model tf.py_func is being used replace the computation expressed as a Python function in your model with something that uses TensorFlow operations.

The PyFunc kernel corresponds to a tf.py_func in your model, which requires a Python interpreter to execute.
So, it seems the model you're using requires the Python interpreter, which is not linked into the Java API and hence you see the error.
You'd want to figure out where in your model tf.py_func is being used replace the computation expressed as a Python function in your model with something that uses TensorFlow operations.

Thanks,I see.Another question is what is py_func.cc used for？Doesn't the REGISTER_OP("PyFunc") in py_func.cc correspond to tf.py_func?

e to t

Yeah,I will.

Hi @ZHUANG-JLU! Thank you for raising the issue. TF v1.14 is an older version of TF which is not actively supported. We recommend you to kindly upgrade to the latest TF versions(2.4 or later) and let us know if the issue still persists in newer versions. For more details please refer to the migration document. Thank you!

Hi @ZHUANG-JLU! Thank you for raising the issue. TF v1.14 is an older version of TF which is not actively supported. We recommend you to kindly upgrade to the latest TF versions(2.4 or later) and let us know if the issue still persists in newer versions. For more details please refer to the migration document. Thank you!

The PyFunc kernel corresponds to a tf.py_func in your model, which requires a Python interpreter to execute.
So, it seems the model you're using requires the Python interpreter, which is not linked into the Java API and hence you see the error.
You'd want to figure out where in your model tf.py_func is being used replace the computation expressed as a Python function in your model with something that uses TensorFlow operations.

@ZHUANG-JLU Could you please refer to the comment above and let us know?
Thank you!

@ZHUANG-JLU Could you please refer to the comment above and let us know? Thank you!

Ｉ think it will still persists.This error  is because I use the tf.py_func which is not included on the ops_op_lib from c++.And I Recoded and re-registered an op that implements the same functionality to tensorflow ops_op_lib by C++.And it is done.

@ZHUANG-JLU Thank you for the update!
In order to expedite the trouble-shooting process, please provide a code snippet in TF v2.x to reproduce the issue reported here. Thank you!


Repository tags are not being created with registry version 2.1.1

I am not sure if there are any breaking changes in 2.1.1 from 2.0.1.
I just updated the registry version in common-compose.yml to 2.1.1. I don't see repository tags in Portus UI, every thing else being same.
This is the version of Portus https://github.com/SUSE/Portus/commit/f84bc786f4419ca829b8799458510b6b478e57c6 I am using.

Sorry for the delay, can you provide us more details about your setup? How are you running portus?

I just run compose-setup.sh to get the portus setup Up on an EC2 with Ubuntu.
Btw, I reverted back to registry 2.0.1 because of https://github.com/SUSE/Portus/issues/300 .
May be this fixed the issue with tags, Tags are being created now.

In the meantime we also fixed the issue with the latest version of distribution.
Closing the issue, but feel free to reopen if you have other troubles.


To clarify, the Texture2D appears to be correct, but within Unity's 'Game' view the window becomes black. :)

Was the version that you used v1.0.1 or v1.0.0?

Tested now with v1.0.1 and can still reproduce the issue.
Here a video demonstrating the issue:
https://gyazo.com/d0e399ccaab525560d4133e0f029e05f

I was able to reproduce the bug and will fix it as soon as possible.


jacoco_test failing with openj9 jdk11 and 14

While adding the jacoco test suite to the openj9 external group, I noticed that there was one test consistently failing with openj9 jdk11 and 14 on all platforms (note the test passes with hotspot).
Here is the error message produced by https://ci.adoptopenjdk.net/view/Test_grinder/job/Grinder/4940/
Failed tests:   startup_should_not_create_duplicate_class_definition(org.jacoco.core.runtime.InjectedClassRuntimeTest)
and
10:56:10 [[1;34mINFO[m] JaCoCo :: Test :: Core ............................. [1;31mFAILURE[m [  3.992 s]
The failing test can be found at https://github.com/jacoco/jacoco/blob/master/org.jacoco.core.test/src/org/jacoco/core/runtime/InjectedClassRuntimeTest.java#L50-L61

FYI @DanHeidinga @pshipton - passes on jdk8 j9

The test is skipped on JDK < 9:
https://github.com/jacoco/jacoco/blob/a20dc090fc5a0531da5b65ed454426b122374a56/org.jacoco.core.test/src/org/jacoco/core/runtime/InjectedClassRuntimeTest.java#L35
I'd start from the code that calls MethodHandles.Lookup.defineClass:
https://github.com/jacoco/jacoco/blob/a20dc090fc5a0531da5b65ed454426b122374a56/org.jacoco.core/src/org/jacoco/core/runtime/InjectedClassRuntime.java#L52-L58

@tajila fyi

@hangshao0 Can you please take a look

https://github.com/eclipse/openj9/issues/11243 is resolved. The latest OpenJ9 build should be able to pass test org.jacoco.core.runtime.InjectedClassRuntimeTest now.

Thanks @hangshao0 . @nikolamilijevic1 could you please re-enable the test? Thanks.

The test is re-enabled. PR https://github.com/AdoptOpenJDK/openjdk-tests/pull/2105


fixes for WSL2 path problems: use a docker volume instead of a bind m…

…ount because WSL2 has some path problems with /mnt/c vs. /c and neither of those really seems to work right, so an external volume mounted at /data fixes the file permissions and path problems.
the caveat is that the files tarsnap accesses must be 'docker cp' into and out of the volume.
and yet, docker secrets work normally!
another bug: tarsnap prints many errors of the form   Can't translate pathname x to UTF-8 . I set LANG=C to try to fix it, and the files get copied, but it doesn't suppress the errors due to UTF characters

.


Create SetServerHeader http layer

Sets the server http header to the given value.
Also should include a constructor to build a rama server header,
which can be used by people (and ourselves) to advertise the web service is driven by rama.
Replaces the odd constructor now defined ad-hoc in SetResponseHeaderLayer.

Change of plans, instead better to provide a TypedHeader for the server header and allow those response/request header setter layers to also use typed headers.

Closed in https://github.com/plabayo/rama/commit/ad80331d2ec90268b4cd02dc4d0e808eadc47b5b.


Shut down the server when rebuilding

It would be nice if, when the Rollup is started in watch mode, the server would stop when the bundle is being rebuilt. That way if you refresh your browser before it's done, you won't be loading the outdated bundle.
It looks like ongenerate could have been used to be notified when the bundle is being rebuilt, but from my tests there is some delay between the file being changed and ongenerate firing. I am not sure if this delay is due to the the detection being slow or something else.

Proposed fix:
diff -ur rollup-plugin-serve.old/dist/index.cjs.js rollup-plugin-serve/dist/index.cjs.js
--- rollup-plugin-serve.old/dist/index.cjs.js	2018-12-12 15:38:12.000000000 -0500
+++ rollup-plugin-serve/dist/index.cjs.js	2018-12-12 15:39:52.000000000 -0500
@@ -9,6 +9,8 @@
 var mime = _interopDefault(require('mime'));
 var opener = _interopDefault(require('opener'));

+var server;
+
 function serve (options) {
   if ( options === void 0 ) options = { contentBase: '' };

@@ -70,7 +72,10 @@
   };

   // If HTTPS options are available, create an HTTPS server
-  var server;
+  if (server) {
+    // release previous server instance if rollup is reloading configuration on watch
+    server.close()
+  }
   if (options.https) {
     server = https.createServer(options.https, requestListener).listen(options.port);
   } else {

Let me know if you are ok with this change, I will submit PR.

I didn't know how to fix this, but #27 has an elegant solution 👍


Feedback on EPNS Front end SDK usage issues faced in the Hackathon

Hello Frens,
(Not sure what would have been the right forum to ask this, so thought github would be the best place!)
First of all thanks a ton for trying out EPNS in your project!!
I am from the EPNS team and was trying to follow up on your feedback on https://devpost.com/software/graphytime

Would love to hear the pain points and the positives on the front end SDK while you were trying to use it, so that we can improve on that! and make it easier for people who follow.
Cheers!!
Thanks Again!

wrong repo!!


#289: Move to Rust ResourceIndex

This PR doesn't solve the task. Probably, we should take it slower and first just move storages into arklib-android.


Fix #1007 -- small improvements to training request workflow

Looks good to me - please merge when you're ready.

Looks good to me - please merge when you're ready.


Add more smoke tests and abstract more code

This PR fixes #

Coverage decreased (-0.1%) to 34.214% when pulling 34ffd6d8a82c3702c485f859d3bf29fe13bff593 on moresmoketests into 75e3e4c8c9010bd41e8d7c05a109afaa39a3e959 on master.


Notifications by Web Push

We could send notifications to users through Web Push. Probably add a button in the dashboard to enable it. User confirms. We get URL to post notifications to. For each matching module, send the notification.
Web Push suggested by @koenbollen at the Go Amsterdam meetup yesterday.

A reason for me to suggest this is so I don't have to give my email address. Would be nice if you could create an account without sharing your email address, e.g. using passkeys. Then it makes more sense to use Web Push notifications.

I had a quick look at web push, but don't yet know if it's possible to create an "anonymous" subscription. Because why would you even need an account to get notified? From the gopherwatch-side, we could just have association of module path with a web push subscription to notify.
Passkeys could be a useful thing for all accounts.
Though I wonder if web push is really that great for users. I like the email because it is async. I can read it a bit later. The notifications aren't urgent. If they are like on a phone, I would probably often not read them...


DACAP Throughput

Hi.
I was running test_uwdacap.tcl file with two nodes having positions as follows:
node 0: (0,0)
node 1:  (0,1500)
The position of node 1 changes as follows:
node 1: (0,3000)
node 1: (0, 4500)
node 1: (0, 6000)
node 1: (0,7500)
the tcl file has been tested changing the position of the node (1500 meter). From (0,1500) to (0,4500) there is a decrease in the Mean throughput. But from (0, 4500) to (0, 6000) the throughput increases. Again it decrease from (0,6000) to (0, 7500). The output seems strange. Is there any specific reason in DACAP to get this type of output?

I solved it.


1.12 update supported releases up to 1.12.2

Part of https://github.com/dapr/docs/issues/3871

Thanks for adding Python 1.12. I was actually about to update this but found it's already up to date :)


How can i get draft's url?

Hello everyone :)
I want to upload an asset to the final draft generated, there is any way to set an output with the draft url?
what i have:
    - name: Create Release
      id: create_release
      uses<EMAIL_ADDRESS>      env:
        GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
      with:
        tag_name: ${{ github.ref }}
        release_name: Release ${{ github.ref }}
        draft: true
        prerelease: true
    - name: Upload Release Asset
      id: upload-release-asset
      uses<EMAIL_ADDRESS>      env:
        GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
      with:
        upload_url: ${{ steps.create_release.outputs.upload_url }} # This pulls from the CREATE RELEASE step above, referencing it's ID to get its outputs object, which include a `upload_url`. See this blog post for more info: https://jasonet.co/posts/new-features-of-github-actions/#passing-data-to-future-steps
        asset_path: ./XX_Build.zip
        asset_name: XX_Build.zip
        asset_content_type: application/zip

what i want:
    - uses<EMAIL_ADDRESS>      env:
        GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
    - name: Upload Release Asset
      id: upload-release-asset
      uses<EMAIL_ADDRESS>      env:
        GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
      with:
        upload_url: ${{ steps.create_release.outputs.upload_url }} # This pulls from the CREATE RELEASE step above, referencing it's ID to get its outputs object, which include a `upload_url`. See this blog post for more info: https://jasonet.co/posts/new-features-of-github-actions/#passing-data-to-future-steps
        asset_path: ./XX_Build.zip
        asset_name: XX_Build.zip
        asset_content_type: application/zip

This functionality is currently not supported by Release Drafter but should be trivial to add by logging the release draft url with some special formatting: https://help.github.com/en/actions/automating-your-workflow-with-github-actions/development-tools-for-github-actions#set-an-output-parameter-set-output.

Is there any way this could use the step output that way another step could use the release that is created? It would be really valuable for many workflows to be able to use that data.
One example is that GitHub does not trigger an event for draft releases so I can not listen for it and then trigger an API call to update the target branch which if I could get the release ID from the output of this I could then make that feature work.

Yup, we'll be exposing it as a step output. I'll get to this soon, seeing there are many people requesting this feature :)

This has been released in v5.4.0 :)
Be aware that actions/upload-release-asset doesn't overwrite release assets, so repeatedly running the workflow will result in failure. See https://github.com/release-drafter/release-drafter/pull/337#issuecomment-568505018 and https://github.com/release-drafter/release-drafter/pull/337#issuecomment-568634352 for more details. There's an open issue and PR that tackle this: https://github.com/actions/upload-release-asset/issues/13.


Fig tutorial - no flask dist found

Hi,
I'm trying to get the Flask tutorial up and running, but I get the follow error:
➜  sudo fig up
Recreating figtest_redis_1...
Creating figtest_web_1...
Building web...
 ---> cdc18ed4ed6f
Step 1 : ADD . /code
 ---> Using cache
 ---> af5b6bfbc4c1
Step 2 : WORKDIR /code
 ---> Using cache
 ---> cf32151738e6
Step 3 : RUN pip install -r requirements.txt
 ---> Running in 37a472e3cf93
Downloading/unpacking flask (from -r requirements.txt (line 1))
  Cannot fetch index base URL https://pypi.python.org/simple/
  Could not find any downloads that satisfy the requirement flask (from -r requirements.txt (line 1))
Cleaning up...
No distributions at all found for flask (from -r requirements.txt (line 1))
Storing debug log for failure in /.pip/pip.log
Service 'web' failed to build: The command [/bin/sh -c pip install -r requirements.txt] returned a non-zero code: 1

+1
I have the same issue when trying to run the setup for Django from the official site.
Step 5 : RUN pip install -r requirements.txt
---> Running in 322224c370a6
Downloading/unpacking Django (from -r requirements.txt (line 1))
Cannot fetch index base URL https://pypi.python.org/simple/
Could not find any downloads that satisfy the requirement Django (from -r requirements.txt (line 1))
Cleaning up...
No distributions at all found for Django (from -r requirements.txt (line 1))
Storing debug log for failure in /root/.pip/pip.log
Service 'web' failed to build: The command [/bin/sh -c pip install -r requirements.txt] returned a non-zero code: 1

I had to restart boot2docker for whatever reason.

Closing, no response in almost a year


Listing node GPU card takes longer than it should.

Description
I deleted the deployment that used the GPU card and waited more than an hour for the card to be listed again, but it never did.
I made sure no one was using the card and waited long enough, even though that I had previously used the same node card and this issue had never occurred to me.
Logs/Alerts

the playground marks the GPU card as used based on a call to the node zos.gpu.list if the contract's field in the result is not 0. And the contract wasn't 0 in after deleting the contract

3.8.2 has thsi fix https://github.com/threefoldtech/rmb-rs/pull/144

verified now once you delete a gpu machine you can deploy another gpu machine at the same instant
Screencast from 08-01-2023 04:21:54 PM.webm


[Refactor Request] Codeunit 5802 "Inventory Posting To G/L"

We are migrating our code from BC140 C/AL to BC160/170.
We have a routine to create fixed asset based on item no. from purchase order.
The function uses codeunit 22 and from there codeunit 5802 to post the G/L entry.
In old version we made a change in codeunit 5802, but this not allowed anymore.
Still we need to do the posting through codeunit 5802 to get the same initiation of variable to post general journal.
If not the next entry no. is being mixed up.
Can you move the call of GenJnlPostLine.RunWithCheck(GenJnlLine) to it's own global function ?

Below follows an example of how it can be solved:

I hope this will be possible to solve.
Thank you ! :)

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Content missing

Steps to reproduce the problem:

./mitmdump -b <IP_ADDRESS> -p 8000 -s "mitm.py" -U http://<IP_ADDRESS>:8001
chrome --proxy-server=http://<IP_ADDRESS>:8000
open video page: http://v.pptv.com/show/IxoEg8c0LmzPTbU.html?&rcc_src=B3&spm=v_show_web.<IP_ADDRESS>.<IP_ADDRESS>.0.0.0
Note: In "mitm.py", there is no special operation for "response()".

From the output of mitmdump, there are some tips:
GET http://<IP_ADDRESS>/0/11343872/0/2b9f90009f74b959a772d5b65173aab4.mp4?fpp.ver=<IP_ADDRESS>&key=cbcfea3666fecc72c526e905281ec847&k=e9cb6c7327d7556a30ae542141209680-a8f6-1537374975%26bppcataid%3D9&type=web.fpp&vvid=b84822b8-e010-b294-3811-f1d4b9ff5a03    << 206 Partial Content [content missing] 887ms
or
<IP_ADDRESS>:56546: GET http://<IP_ADDRESS>/0/11343872/0/2b9f90009f74b959a772d5b65173aab4.mp4?fpp.ver=<IP_ADDRESS>&key=cbcfea3666fecc72c526e905281ec847&k=e9cb6c7327d7556a30ae542141209680-a8f6-1537374975%26bppcataid%3D9&type=web.fpp&vvid=b84822b8-e010-b294-3811-f1d4b9ff5a03               << 206 Partial Content 8.66m <IP_ADDRESS>:56546: Error in HTTP connection: TcpDisconnect('[Errno 32] Broken pipe',)
This will lead to the request status to "canceled" in chrome.
Note: This video link is breakpoint continuingly.
Any other comments? What have you tried so far?

For other links, no problem.
Without upstream proxy, no problem.
I have also tried mitmproxy 4.0.1, and added the options, like "--stream" and "--body-limit-size", but have the same issue.
I have read the series issues in https://github.com/mitmproxy/mitmproxy/issues/1764, and tried the method in https://github.com/mitmproxy/mitmproxy/issues/856 to set response.stream=True, but also failed.

System information
Ubuntu 16.04

mitmdump version: 2.0.2 and 4.0.1

Is this a bug or something need to set? Thanks.

Resolved it by setting
flow.request.stream = True
and setting the option stream_large_bodies in command line:
mitmdump --listen-host <IP_ADDRESS> --listen-port 8000 --set stream_large_bodies=30

this worked for me:
mitmproxy --set stream_large_bodies=10m


BERT-ML.NET NER

Hi !
Thank you for the project. It was a great help.  Is it possible to create examples for the NER model?

I tried but there is something wrong with the accuracy when converting to onnx, the model works fine in python in pytorch but after convering it has very bad accuracy so i am stuck at the moment so maybe i have to find the motivation and time but i will try and make it work 😄 .

Hello,
We have an exciting opportunity for you! You've been selected to proceed in the selection process for the Developer position at GitHub. Congratulations on your achievement!
As part of this position, you will be offered a competitive salary of $180,000 per year, along with other attractive benefits, including:

Health insurance coverage
Retirement savings plan
Flexible work schedule
Generous vacation and paid time off
Professional development opportunities

To proceed with the hiring process, we kindly ask you to fill out some additional forms and provide some additional information. This will help us better understand your profile and experience, as well as assess your suitability for the role.
Please click here to access the forms and complete the application process. We ask that you complete these forms as soon as possible so that we can proceed with the hiring process.
Important: You have 24 hours to complete the application process.
If you have any questions or need further information, please don't hesitate to contact us.
Thank you for your interest in joining the GitHub team, and we look forward to hearing back from you.
Best regards,
GitHub Recruitment Team
thebluetropics, @MrRobotIsHere, @DevilIris, @untitled01ipynb, @willowdong, @AlexandreBrown, @jeffrom, @herbyme, @kaayce, @mathiasesn, @jbrnjfr, @mikefong, @shenoynikhil, @susumutomita, @Raidenyamato2, @matsuyoshi30, @cod3rboy, @0-hero, @cs50victor, @VictorBusque


Support for OpenShift Container Platform 4.2

Now that OpenShift Container Platform 4.2 is released, are there any timeline for when support for the new version will be added to the OpenShift Container Platform Deployment Template for Azure?

@janegilring The infrastructure components are provisioned as part of the OCP 4.x installer so there is no need for an ARM template.

I see, thanks. We will go ahead and try out the new version and see what the deployed infrastructure looks like, It looks like load balancers is not deployed as part of the installation, but I might be wrong as I didn't look into the details yet.

It does indeed deploy a load balancer.  Actually 2.  One for master api and one for application nodes.


Sync output of parallel make tasks in CI

Checklist

[ ] Affected Issues have been mentioned in the Closing issues section
[ ] Documentation has been written/updated.
[x] Changelog entry has been written

The parallel tasks in CI (-jN) all write to STDOUT at once and the output is garbled. This change syncs the output to avoid the interspersed messages from parallel make tasks.
Changelog Entry
Improvement - Sync output of parallel CI tasks
Closing issues
n/a

This is definitely much easier to read, but unfortunately this biffers the logs while they are happening and only paste them after they are done.
And many times to debug I need realtime logs to see check which test is maybe failing to login into the worker node and check stuff on the CLI.
If make is buffering the logs this won't be possible anymore :(
Is there maybe a possibility that we run multiple sh parallel steps of Jenkins which each runs just a single make test? I guess the question is then how do we explain Jenkins it should only run two tests parallel and not all of them? But I'm sure @smlx you will find a clever way :)

hmm. okay so I've done a bit of research. make's sync-output functionality uses tmpfile(), which uses open(..., O_TMPFILE). This means that the output buffer never actually gets a pathname so we can't tail it.
I think that re-implementing make's parallel functionality in the Jenkinsfile will get messy..
maybe the best way to do this is to have a variable in the Makefile that can be set to control sync-output functionality. By default output can be synced, and you can flip the variable to disable syncing when debugging. What do you think of that solution @Schnitzel?

ugh, yea looks like Jenkins doesn't really have a way to do this, see also https://issues.jenkins-ci.org/browse/JENKINS-44085
I guess your suggestion @smlx with a variable would be the easiest, can you add this to this PR?


Update to latest Penn State brand guidelines

Strategic Communications has asked our IT Comms to provide us with the latest logo and such. The guidelines are here: http://visualidentity.psu.edu/standard/brand-architecture-tier-3-institutes-and-centers
I have the favicon and new logo.

Repurposing this ticket for Sufia 7 and ScholarSphere 3 work, since SS Sufia 6 in production is up-to-date.

Will be fixed by #283


Change log level for failure to acquire lock to warn

I don't think there's a good reason to increase the log level. The user gets notified about a failed locking attempt by a Timeout exception and can then choose how to handle it (logging the exception including the traceback, trying again, ...). Changing the log level would also pollute the logs of some applications which rely heavily on filelocks. So better leave it up to the user if the Timeout exception should be logged or not.


Update README.md

use the right filename

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


Export grammar-related generators from CLI

For the Langium playground I need to import the Textmate generator.
Therefore I am exporting a couple of generators from the CLI.
My current alternative is to copy the code from this repo, which is actually not that nice.
Speak up if you want a different export strategy. Currently I am exporting each generator in its own export alias.

@Lotes
In general I think it is good to export useful components to users. One thing to keep in mind, when exporting code we make it a public API, hence need to care that we do not break it in the future and we should document it.
If you just need the textmate generator, maybe we should first only export just this generator, WDYT?

@dhuebner Thanks for the argument. That is a very valid point. Then I will only export the Textmate generator. But even there I am unsure: Shall I export it under "/textmate" or better under something more general like "/syntax-highlighting"...? Maybe start with something specific and deprecate when the time has come and it becomes more general?

I agree with @dhuebner here. Exporting just what we need makes sense for now. And as for the name, textmate seems fine, but we should document that new endpoint to make it clear what it provide (later on, before the next release).

@montymxb As far as I can see there is no next change log file or something similar. So I will document the endpoint in the main description of this PR.
@msujew I have done the split. Nice catch. Please have another look.

I need to wait for #1437 since I want to increment the Langium CLI version and trigger Langium generator for the examples.


⚠ Samsung has degraded performance

In 75ac8bd, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2544 ms

Resolved: Samsung performance has improved in a42099e.


Fix #1890, Improve resource ID branch coverage

Describe the contribution
Fix #1890 - pushes resource ID coverage to 100/100
Note it was just an out-of-order test where Id was made invalid before the section that needed a valid Id to get full coverage, so swapped the tests.
Testing performed
Build/run/report coverage for unit tests
Expected behavior changes
None
System(s) tested on

Hardware: Intel i5/Docker
OS: Ubuntu 18.04
Versions: Bundle main + this commit

Additional context
None
Third party code
None
Contributor Info - All information REQUIRED for consideration of pull request
Jacob Hageman - NASA/GSFC

CCB:2021-09-01 APPROVED


License

Since this code depends on Ultralytics, which uses the AGPL license, how does it come under the MIT license?

Since this code depends on Ultralytics, which uses the AGPL license, how does it come under the MIT license?

I do not use original code but just treat Ultralytics as a package like numpy.


Fix cache calls in visible_posts tests to ensure accurate results

I think this isn't working properly because of the allow being on Rails.cache but I'm not sure I just know I was getting false all the time when I was testing blocking caches.

Maybe we could use a memory_store, and clear it between tests, like in https://stackoverflow.com/questions/13309121/is-rails-cache-purged-between-tests?

Work uses fakeredis iirc, we can look into that?

Superseded by #1537


Add support for query parameters in the bash template and ensure that all query parameters are URL-encoded.

Codecov Report

Merging #545 into master will increase coverage by 0.28%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master     #545      +/-   ##
============================================
+ Coverage     90.59%   90.88%   +0.28%
- Complexity      239      290      +51
============================================
  Files            14       14
  Lines           670      691      +21
============================================
+ Hits            607      628      +21
  Misses           63       63

Impacted Files
Coverage Δ
Complexity Δ

src/Postman/CollectionWriter.php
93.18% <100%> (+0.68%)
19 <0> (+10)
:arrow_up:

...c/Tools/ResponseStrategies/ResponseTagStrategy.php
100% <0%> (ø)
11% <0%> (+2%)
:arrow_up:

.../Tools/ResponseStrategies/ResponseFileStrategy.php
100% <0%> (ø)
13% <0%> (+2%)
:arrow_up:

src/Commands/GenerateDocumentation.php
94.07% <0%> (+0.13%)
44% <0%> (+10%)
:arrow_up:

src/Tools/Generator.php
95.69% <0%> (+0.27%)
86% <0%> (+25%)
:arrow_up:

...ols/ResponseStrategies/TransformerTagsStrategy.php
82% <0%> (+0.36%)
26% <0%> (+2%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 98d0456...6bc4b53. Read the comment docs.

Good work, thanks!