Cannot restrict EntraCP to only return Groups

Discussed in https://github.com/Yvand/EntraCP/discussions/215

Originally posted by NaveenKanukuntla November 28, 2023
Here is my setup

EntraID as the directory lookup.
Utilizing EntraCP to fill claims for user on SharePoint 2016.
I already use Windows /Claims NTLM for user lookup from active directory.
Most of my security groups have been created in Azure and not in On-Prem AD.

Due to this, I would like to use EntraCP to lookup Security Groups in EntraID
Is it possible to limit my people picker results returned from EntraCP to only resolve/return role claims ?

@NaveenKanukuntla please see my feedback in https://github.com/Yvand/EntraCP/discussions/215


[Bug] Notepads and Windows Explorer freeze upon attempting to enter compact overlay mode

Describe the bug
Attempting to enter Compact Overlay mode via menu or by pressing F12 resizes window but freezes it and the entire desktop (eventually leading it to crash)
To Reproduce

Open an existing .txt file
Attempt to enter Compact Overlay

Expected behavior
It would enter Compact Overlay mode without Windows Explorer freezing
Screenshots
N/A
Desktop (please complete the following information):

OS

Version

Additional context
Just installed it

It seems like a system bug, does it persists after rebooting?


Allow any application to access the imported key without warning

Checklist

[ ] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I see several green ci/circleci builds in the "All checks have passed" section of my PR (connect CircleCI to GitHub if not)
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.
[ ] I've added or updated relevant unit tests.

Motivation and Context

I use fastlane match to handle certs on github actions but don't (yet) use fastlane to build.
For reasons beyond my understanding, codesign hangs (probably with a warning message on some virtual screen) unless I specify the -A parameter.

[x] Todo: this needs to be an opt in flag. And the docs need to be updated. I am happy to improve what's needed.

Credits: https://github.com/Apple-Actions/import-codesign-certs

Description

Testing Steps

For reasons even further beyond my understanding it started to work suddenly now even without -A specified. I'll close this and reopen in case stars align badly again ...


ADD: WiFi Shield support

Driver for openbci wifi shield!
Let's do it!

It'd be my pleasure to look into that... should I have a wifi shield to get my hands on :stuck_out_tongue:
Is the only difference the way to acquire raw data (i.e. through wifi instead of BLE)? Apart from that did the protocole change?

@jfrey-xx i think the first thing i will so is just build the device discovery part, add a function to connect, and send commands to the board, and stream data over in JSON with nano_volts. I don't really want to spend all the time on the raw mode hahah if you want to, you can take the code i write for connecting to it and such and write the code to have the board send raw. Faster sample rates are possible with raw for sure! but sample rates (500Hz daisy) and (800Hz) ganglion should be possible with JSON. Going for MVP here. Going to write these types of drivers in as many languages as i can :)

yea i think you need a wifi shield!

Is the only difference the way to acquire raw data (i.e. through wifi instead of BLE)? Apart from that did the protocole change?

Yes! In raw mode, the wifi shield will send the classic 33byte with 0xA0 and 0xCn where n is 0-F. The ganglion over raw mode with wifi will send the same thing, just have four empty channels, but will have full 24bit resolution instead of 18/19 bit delta compression.

Same raw format and a JSON version? Neat ;)

Yea the raw format is so you can pipeline to existing parsers. The JSON sends:
{
  "chunk": [
    {
      "timestamp": ...,
      "data": [
         ....
      ]
  ]
}

Time is in micro seconds and data is in nano volts

@jfrey-xx hey i need some help/advice
I am trying to open a tcp server and have it run all from the init function of OpenBCIWifi() but the thing is the server requires an asyncore.loop() which is blocking. So how do i start the server and keep going with the init function?
Thanks!

It has been a long time since I fought with TCP directly, quickly I'd say you could use a timeout parameter (if effective) to avoid anything blocking, or just use a dedicated (sub)class using Thread for opening socket and sending/receiving data. ...but you must have thought of this, no? I might be missing the point.

Well i got an MVP working, it's printing JSON data to the console. I need some help making it route nicely like yours does but it's a rough demo and more importantly you can check out how cool and easy it is!!!! AH i love it haha

Thread for opening socket and sending/receiving data

Probably a good way to do this. This is so easy in javascript land haha


Fix binary installation for i386 architectures.


User-store configurations doesn't include by-default configurations

By default user store configurations are missing in the document page[1] with the new unique user id user store managers.
Ex: If we switch the default user store manager to the JDBC user store manager based on the [1], an error will be thrown from the server startup. This is because the default user store manager type needs to be suffixed with unique_id term.
[1]  https://is.docs.wso2.com/en/next/setup/configuring-a-jdbc-user-store/

Created doc issue https://github.com/wso2/docs-is/issues/1144


deprecated function

Hi, I've found a function that was removed in Bootstrap v3, andSelf, this can be replaced with addBack with no other changes.
https://api.jquery.com/andself/

Thanks for the change!

My pleasure. Thanks for the plugin.


APNSwift and Vapor 3

Is there a release or branch that still supports Vapor 3? I am not ready to migrate to Vapor 4 yet but would still like to implement push notifications using APNSwift. I attempted to use the following code below in my Package.swift file:
package(url: "https://github.com/kylebrowning/APNSwift.git", .exact("1.3.0"))
Unfortunately this throws multiple errors (along with v1.7) about requiring certain versions of Vapor and other dependent packages.

@kschins oof. there's probably an alpha of vapor/apns.
Can you try using this commit, https://github.com/vapor/apns/commit/5a9c3b8084788d3a34da5a72e9c1a4435468b963

Actually that won't work.

I think you'll have to make your own provider, but not sure which version of APNSwift you will have to use.

I appreciate the quick reply. I can try using a third party service like Pusher or Amazon SNS for now until I migrate to Vapor 4.

AS much as Id hate to throw other packages out there, you could try this one, https://github.com/MihaelIsaev/FCM

Thank you, I will take a look!


Leave checkstyle violation to each module's target directory

Because of output log size policy for Travis CI, we can't print violation report to the console. Instead of that, we can just leave violation to the file.
This would help to track and fix checkstyle violations.

Btw I don't like xml representation but when I set output format to plain, checkstyle plugin complains.
Output format is 'plain', checkstyle:check requires format to be 'xml'

+1
The check goal doesn't currently support "plain" output, the documentation is wrong. http://svn.apache.org/viewvc/maven/plugins/tags/maven-checkstyle-plugin-2.17/src/main/java/org/apache/maven/plugin/checkstyle/CheckstyleViolationCheckMojo.java?view=markup#l556
It's because the plugin writes the checkstyle results to disk, then parses them back up again to figure out if there were any rule violations so it can fail the build. The parsing code only understands XML.

+1
Although I think this is already happening.  As a part of https://github.com/apache/storm/pull/2113 I added in an xslt script to make reading the XML format better (like plain) and a python script to help search through it to help do things like look at violations only for files I touched with this patch.


VISUAL_CONFIG for Epoch20_LXRT.pth

Hello, I am trying to extract the cross-modal representations from Epoch20_LXRT.pth. I am using entry.py. What should be the args for l_layers, x_layers and r_layers for this model? I tried various values and it says some weights are missing in either loaded or model.
Best regards.

It is because the pre-trained model has more weights in predicting the masked values, which will not be loaded in fine-tuning / extraction.
If you load BERT weights for other tasks, you would get the same message.

Ok, I understand. But then what should I input as the layer values? 9,5,5?


Does this suppose to be a generic tool for other django projects?

I see https://github.com/vintasoftware/cdrf.co is heavily based on this, but changed a lot, is there a easy way to do this?

@iamsk – The current version is heavily tied to django.views.generic and related mixins (they're defined in settings).  I've been working on a static generation tool that's a lot less tied to that particular section.  You can see the work on the static-pages-requiem if you're interested, however it's not complete yet.

@ghickman Thanks for your reply, I will read the code.


GEODE-6639: Use AtomicLong for processingMessageStartTime

When reviewing, focus on the changes in https://github.com/apache/geode/pull/3450/commits/5ea16f374fc41f2cc2bf5fe89dd94d19a0e08d99. The other commit is cleanup before fixing.


HBASE-25738 Backport HBASE-24305 to branch-2.2

Signed-off-by: Duo Zhang<EMAIL_ADDRESS>Signed-off-by: stack<EMAIL_ADDRESS>

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

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

_ Prechecks _

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

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

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

+1 :green_heart:
test4tests
0m  0s
The patch appears to include 14 new or modified test files.

_ branch-2.2 Compile Tests _

+0 :ok:
mvndep
0m 16s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
3m 27s
branch-2.2 passed

+1 :green_heart:
compile
1m 45s
branch-2.2 passed

+1 :green_heart:
checkstyle
2m 42s
branch-2.2 passed

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

+1 :green_heart:
javadoc
1m 21s
branch-2.2 passed

+0 :ok:
spotbugs
3m 29s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
5m 32s
branch-2.2 passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 14s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
3m  8s
the patch passed

+1 :green_heart:
compile
1m 43s
the patch passed

+1 :green_heart:
javac
1m 43s
the patch passed

+1 :green_heart:
checkstyle
0m 27s
hbase-common: The patch generated 0 new + 0 unchanged - 25 fixed = 0 total (was 25)

+1 :green_heart:
checkstyle
0m 40s
The patch passed checkstyle in hbase-client

+1 :green_heart:
checkstyle
1m 34s
hbase-server: The patch generated 0 new + 279 unchanged - 3 fixed = 279 total (was 282)

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

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

+1 :green_heart:
hadoopcheck
28m 17s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 2.10.0 or 3.1.2 3.2.1.

+1 :green_heart:
javadoc
1m 18s
the patch passed

+1 :green_heart:
findbugs
5m 54s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
3m  2s
hbase-common in the patch passed.

+1 :green_heart:
unit
3m 39s
hbase-client in the patch passed.

+1 :green_heart:
unit
113m 52s
hbase-server in the patch passed.

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

200m 22s

Subsystem
Report/Notes

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

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

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux a75788124e43 4.15.0-136-generic #140-Ubuntu SMP Thu Jan 28 05:20:47 UTC 2021 x86_64 GNU/Linux

Build tool
maven

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

git revision
branch-2.2 / a227a4aa37

Default Java
Oracle Corporation-1.8.0_282-b08

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

Max. process+thread count
2244 (vs. ulimit of 12500)

modules
C: hbase-common hbase-client hbase-server U: .

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

versions
git=2.20.1 maven=3.6.3 findbugs=3.1.11

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

This message was automatically generated.

Jenkins is actually green. Let me kick off the build again to have it also reflected here.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

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

_ Prechecks _

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

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

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

+1 :green_heart:
test4tests
0m  0s
The patch appears to include 14 new or modified test files.

_ branch-2.2 Compile Tests _

+0 :ok:
mvndep
0m 17s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
3m 13s
branch-2.2 passed

+1 :green_heart:
compile
1m 45s
branch-2.2 passed

+1 :green_heart:
checkstyle
2m 36s
branch-2.2 passed

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

+1 :green_heart:
javadoc
1m 25s
branch-2.2 passed

+0 :ok:
spotbugs
3m 21s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
5m 22s
branch-2.2 passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 16s
Maven dependency ordering for patch

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

+1 :green_heart:
compile
1m 42s
the patch passed

+1 :green_heart:
javac
1m 42s
the patch passed

+1 :green_heart:
checkstyle
0m 26s
hbase-common: The patch generated 0 new + 0 unchanged - 25 fixed = 0 total (was 25)

+1 :green_heart:
checkstyle
0m 39s
The patch passed checkstyle in hbase-client

+1 :green_heart:
checkstyle
1m 25s
hbase-server: The patch generated 0 new + 279 unchanged - 3 fixed = 279 total (was 282)

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

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

+1 :green_heart:
hadoopcheck
25m 51s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 2.10.0 or 3.1.2 3.2.1.

+1 :green_heart:
javadoc
1m 23s
the patch passed

+1 :green_heart:
findbugs
5m 46s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
3m  9s
hbase-common in the patch passed.

+1 :green_heart:
unit
3m 52s
hbase-client in the patch passed.

+1 :green_heart:
unit
93m 47s
hbase-server in the patch passed.

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

175m  3s

Subsystem
Report/Notes

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

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

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux 846d9e380260 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 GNU/Linux

Build tool
maven

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

git revision
branch-2.2 / 3abea8e47f

Default Java
Oracle Corporation-1.8.0_282-b08

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

Max. process+thread count
3016 (vs. ulimit of 12500)

modules
C: hbase-common hbase-client hbase-server U: .

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

versions
git=2.20.1 maven=3.6.3 findbugs=3.1.11

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

This message was automatically generated.


Add undefined constant

To represent the NaN value whose name we are not allowed to pronounce, and have to call undefined, thereby causing confusion for JS coders we try to lure to Ceylon (the scaring of whom potentially being a good thing).
I'd also suggest renaming Float.undefined to Float.notANumber but I suppose it's too late?
Anyways, I sort of need that value to test #700, and it feels weird having to evaluate 0/0 just to get it.
@tombentley suggested adding it as a value constructor to Float, but then infinity is a toplevel, though we could make it point to Float.infinity for backwards compat, and add it there too for extra cleanliness.

imo we should:

add value constructors Float.infinity and Float.nan (I find Float.notANumber just unnecessarily difficult to type)
deprecate top level infinity since it's been replaced by Float.infinity.
retain the attribute Float.undefined as the test for NaN

I suppose we could deprecate Float.undefined and add Float.isNan, to make the naming more consistent.

I've implemented this for JVM, but the metamodel tests fail: We need to merge the ctor-model branches before looking into that.

I'm totally against this and it seems completely ill-conceived to me. By definition, the undefined value has no well defined equality or identity so representing it as a constant value is just going to cause people to shoot themselves in the foot in all sorts of ways. Furthermore, I can't think of any usecase for wanting to pass the undefined value to a function. (Unlike ±infinity, for which there could be  meaningful return value.)
I don't know where this idea is coming from, but it seems that you need to go back and rethink your usecase because I can't see how it could possible be well-motivated.
May I close this issue please?

And totally huge -1 on "nan" too. What kind of psychopath thinks "not a number" is a reasonable way to describe something?
As anyone who has ever studied high school mathematics knows, we say

zero divided by zero is undefined

i.e.
(0/0).undefined

What kind of psychopath thinks "not a number" is a reasonable way to describe something?

Every programmer, not because it makes sense (like most words), because that's the convention.

As anyone who has ever studied high school mathematics knows, we say "zero divided by zero is undefined"

Well, this is elitism. I'm not saying it's wrong at all. I'm saying the fact that it is known to math people doesn't mean it's known to most programmers. Not only that but I suggest this is widely dependent on language. I've never heard of that sort of definition in French. In fact I don't even recall being taught that in school, but if it were, it'd probably be defined as "you can't divide zero by zero".
You can't expect most programmers to be well versed in math terminology. Certainly not more so than computer science terminology, where that thing is definitely called NaN, for better or worse.
Again, I'm not saying we have to drop the math terminology, but at least make it programmer-friendly with an alias that they can discover. I've never heard of Ceylon being targeted at math people first so far, and I'm not sure that's something we should start doing.

I didn't say anything about mathematicians. I'm using the terminology taught in high school class.

There's a use case for these as constants when testing functions that take
floats, at least.
I don't really care about naming so long as people can find these thing by
the other name in things like the apidoc search. And ideally autocomplete.
The fact is that every other programming language calls this NaN, so that
is what people will look for and it's perverse if we don't help them find
it by that name. I don't care how we do that. Use an alias or add a synonym
annotation or whatever. But right now this stuff is not vet discoverable at
all.
On 29 Aug 2015 23:25, "Gavin King"<EMAIL_ADDRESS>wrote:

I didn't say anything about mathematicians. I'm using the terminology
taught in high school class.
—
Reply to this email directly or view it on GitHub
https://github.com/ceylon/ceylon.language/issues/730#issuecomment-136057555
.

What are you guys even talking about? Hover over the identifier Float and tell me what you see.
Sent from my iPhone

On 30 Aug 2015, at 19:16, Tom Bentley<EMAIL_ADDRESS>wrote:
There's a use case for these as constants when testing functions that take
floats, at least.
I don't really care about naming so long as people can find these thing by
the other name in things like the apidoc search. And ideally autocomplete.
The fact is that every other programming language calls this NaN, so that
is what people will look for and it's perverse if we don't help them find
it by that name. I don't care how we do that. Use an alias or add a synonym
annotation or whatever. But right now this stuff is not vet discoverable at
all.
On 29 Aug 2015 23:25, "Gavin King"<EMAIL_ADDRESS>wrote:

I didn't say anything about mathematicians. I'm using the terminology
taught in high school class.
—
Reply to this email directly or view it on GitHub
https://github.com/ceylon/ceylon.language/issues/730#issuecomment-136057555
.

—
Reply to this email directly or view it on GitHub.

That is not a replacement for autocomplete. Users look for a method, then a
top-level and then are quite likely to give up. Users often don't read the
documentation.
On 30 Aug 2015 11:03, "Gavin King"<EMAIL_ADDRESS>wrote:

What are you guys even talking about? Hover over the identifier Float
and tell me what you see.
Sent from my iPhone

On 30 Aug 2015, at 19:16, Tom Bentley<EMAIL_ADDRESS>wrote:
There's a use case for these as constants when testing functions that
take
floats, at least.
I don't really care about naming so long as people can find these thing
by
the other name in things like the apidoc search. And ideally
autocomplete.
The fact is that every other programming language calls this NaN, so that
is what people will look for and it's perverse if we don't help them find
it by that name. I don't care how we do that. Use an alias or add a
synonym
annotation or whatever. But right now this stuff is not vet discoverable
at
all.
On 29 Aug 2015 23:25, "Gavin King"<EMAIL_ADDRESS>wrote:

I didn't say anything about mathematicians. I'm using the terminology
taught in high school class.
—
Reply to this email directly or view it on GitHub
<
https://github.com/ceylon/ceylon.language/issues/730#issuecomment-136057555

.

—
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHub
https://github.com/ceylon/ceylon.language/issues/730#issuecomment-136123233
.

Remember, this all started due to confusion over a (poorly written but correct as per junit) Java unit test of mine that looked like:
assertEquals(calcResultingInNaN(), Double.NaN)

@gavinking's ex post facto prediction nails it regarding the confusion:

Having a constant led to the false implication of equality
The name 'nan' sounds like an unavoidable marker value or error that might be equatable, not some more fundamental and meaningful mathematical concept that is neither equatable nor a value.

This unit test would have been written in a more readable fashion in Ceylon.

Users who don't read documentation will google "ceylon nan" which will take them directly to a discussion #40 where we went through this design process. And where you @tombentley explained why there should be no constant value NaN.
P.S. Perhaps we should discuss whether infinity could be a value constructor of Float i.e. Float.infinity, but I don't see a strong reason to do that at this point.

Aren't there multiple kinds of NaNs?

There are, but Java conflates them all as a single one
On 31 Aug 2015 20:36, "Ross Tate"<EMAIL_ADDRESS>wrote:

Aren't there multiple kinds of NaNs?
—
Reply to this email directly or view it on GitHub
https://github.com/ceylon/ceylon.language/issues/730#issuecomment-136477998
.

I'm using the terminology taught in high school math class.

Again, this may be true for you, but not for everyone. This is not called undefined in French, but forbidden.
Now, if we have to get a little more pedantic, we haven't implemented any undefined behaviour: we have implemented the IEEE spec which calls these values Not a Number, not undefined. In fact it's precisely the opposite: it's very well defined how these values work and interact.

Remember, this all started due to confusion over a (poorly written but correct as per junit) Java unit test of mine that looked like:

FTR, here's the current translated test:
    value nan = 0.0/0.0;
    check(nan.wholePart.undefined);

Sometimes, people have to work with IEEE, which we implement, whether we like it or not. And I strongly suspect that anybody implementing serious things in math using floats have to deal with that standard's properties. Besides the naming of undefined that deviates from the standard and every other programming language I ever worked with, I don't see why we must make it hard to access those special values. Surely the people who care about them know how to get them because they know how they work, so it's not like we can hide them either (we do provide a test for that one), we're just pissing off people who have work to do with them.

I don't think there should be a "constant" undefined. Nor do I think we should rename the attribute. Closing.

Sigh. Can I add some docs to Float to explain howto get to one then?

Well, in retrospect, perhaps at least part of this issue can be fixed using a doc alias like proposed in #734, so that people will find notANumber with completion.

perhaps at least part of this issue can be fixed using a doc alias like proposed

Precisely.

Can I add some docs to Float to explain howto get to one then?

You mean like this:

In addition, the following special values exist:

[[infinity]] and -infinity, and
[[undefined values|undefined]], denoted [NaN][] by the
IEEE standard.

As required by the IEEE standard no undefined value is
equal to any other value, nor even to itself. Thus, the
definition of [[equals]] for Float violates the general
contract defined by [[Object.equals]].

Well, it does not explain that if you need one you have to do 0/0, that's what I'm proposing to add.

OK, fine. But it's 0.0/0 you have to write.

Ah yes, another reason to document it ;)

It actually is already documented if you look at Float.undefined.

OK, true, that's good enough.

Fixed.


Docker Volume not updating automatically

Everytime I deploy a new .yml file I start getting a 404 not found even though its in the same directory and the same volume. Is there maybe some issue with the image that doesn't let me do this and have to restart the service everytime I change the yml file?
Thanks.

Hey @Gabb1995,
not sure but I guess I had the same problem. Everytime I changed something in my YAML file, I needed to restart the ReDoc container to see the changes in the UI. According to https://tachingchen.com/blog/container-file-mount-dose-not-update-with-changes-from-host/ it's due to the behavior of how the mounting works.
So mounting the complete directory into a sub-directory and thus changing the SPEC_URL works perfectly for me.
For example: instead of using this:
docker run -ti --rm -p 80:80 -v $(pwd)/project/docs/v1.yaml:/usr/share/nginx/html/v1.yaml -e SPEC_URL=v1.yaml redocly/redoc
use this:
docker run -ti --rm -p 80:80 -v $(pwd)/project/docs/:/usr/share/nginx/html/yaml/ -e SPEC_URL=yaml/v1.yaml redocly/redoc.
Hope this helps.
Best
Michael

@mtrlitza thanks for your answer!
Closing this issue!
@Gabb1995 feel free to reopen if you face further issues!

Thanks this worked! :)


Not able to access cluster over P2S VPN but can access from Peered VNET

[Enter feedback here]
Deployed Private cluster and able to access from VNET and peered VNET. But when trying to access the Cluster from Azure P2S VPN then it is not working. If it is not supported, then what is the workaround?

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

ID: 0b68f2c4-bb6c-11a2-6c61-8af4057a2438
Version Independent ID: e3498bed-1447-6841-8353-9f1b5d3dc8df
Content: Create a private Azure Kubernetes Service cluster - Azure Kubernetes Service
Content Source: articles/aks/private-clusters.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

@vikaskumawat80
Thanks for your feedback! We will investigate and update as appropriate.

@vikaskumawat80 As per the document, you should be able to connect to Private AKS cluster using VPN. Please  check on configuring VPN here.
As mentioned you are able to access from VNET & peered VNET, just make sure that VPN is able to access the subnet block assigned to AKS. You can try connecting to a virtual machine which is launched in same subnet of AKS via VPN before trying to connect to AKS.

@vikaskumawat80 Any update on the issue?

@vikaskumawat80
Hope the provided information is helpful.
We will now close this issue. If there are further questions regarding this, please tag me in a comment. I will reopen it and we will continue the discussion.

We tried Azure VPN is not working. If I try to access from the VNET, it works but not on over VPN.


Webhook validation failing for blank subnet cidr

What happened:
webhook validations failing for templates/cluster-template-arm-free-tier.yaml. Since the subnet CIDR's are set the validation fails.
What you expected to happen:
The blank CIDRs would still pass validation, but would be automatically populated by the provider
How to reproduce it (as minimally and precisely as possible):
Try to create a workload cluster using the templates/cluster-template-arm-free-tier.yaml template.
Anything else we need to know?:
Environment:

CAPOCI version: 0.3.0
Cluster-API version (use clusterctl version):
Kubernetes version (use kubectl version):
Docker version (use docker info):
OS (e.g. from /etc/os-release):

Name also is not mandatory for subnets.


Damageable component with tests

Will init the component with full health passed in and subtract damage
from this initial number. This interface is meant to limit errors with
incorrectly setting health and keep things simple.

Fixes #61


Recommendation - Add Aquarium Tutorial and Figma Toolkit links to Main MRTK3 page

Recommendations:

Add a link to to checkout the MRTK3 "Aquarium" Tutorial. Github
You can link to the following link for More information: Microsoft Blog

Add a link to the Figma Toolkit for MRTK3 Preview Figma

Add the following video:
TEC73 - Deploy Everywhere with MRTK3 and OpenXR Youtube

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

ID: cf4a1898-af6b-be1b-4fa3-8c1269f855a9
Version Independent ID: cf4a1898-af6b-be1b-4fa3-8c1269f855a9
Content: Mixed Reality Toolkit 3 Developer Documentation - MRTK3
Content Source: mrtk-unity/mrtk3-overview/index.md
Product: mixed-reality
GitHub Login: @MaxWang-MS
Microsoft Alias: wangmax

Hi @Victor-x1, thanks for the suggestion! I have added the links to our tutorial and Figma toolkit. I decided to not add a link to the YouTube video as we already have links to Microsoft Mixed Reality Dev Days 2022 talks on the page, which include one session that is very similar.


ARROW-12972: [CI] Fix centos-8 cmake error

Install libarchive to fix cmake error on centos-8.
cmake: undefined symbol: archive_write_add_filter_zstd

@kou , anyway to trigger related crossbow build?

@github-actions crossbow submit centos-8-*

@github-actions crossbow submit centos-8-*

The centos-8-arm64 failure is another problem.

@github-actions crossbow submit centos-8-*

I don't understand why x86 doesn't have this issue. Per my test with centos:8 image, both x86 and Arm have this cmake undefined symbol error.

I don't understand why x86 doesn't have this issue.

This issue is "cmake: undefined symbol: archive_write_add_filter_zstd" not pygit2 error https://travis-ci.com/github/ursacomputing/crossbow/builds/228149814#L5782 , right?
This issue happens on x86 without this fix: https://github.com/ursacomputing/crossbow/runs/2760533808?check_suite_focus=true#step:11:775

So libarchive is necessary in both places. Will add back.

Oh, sorry.

No worry. It's trivial. :)

@github-actions crossbow submit centos-8-*


Prompts unreadable on light themes

Presumably thanks to the text colour being force-set to white, the prompts are unreadable in a terminal using a light theme:
Konsole/Yakuake with Linux colours theme for reference

…and with with some default light themes

LXTerminal with Solarized Light

Some terminals export a variable called COLORFGBG (e.g. iTerm2, rxvt), other xterm compatible terminals rely on an escape sequence:
$ echo $COLORFGBG
15;0

$ printf '\033]11;?\007'
11;rgb:3e/3e/3e

One could probably wrap this up in some detection code and then change the output color based on this.

Thanks for opening the issue @qtlis and thanks for the tip @citizen428!
Will look into it as soon as I have some spare time. The fix will most likely have to be implemented in inquire, since that where the prompts are handled.

@tmattio If I have time — which wasn't really the case these past few months as you have noticed — I'll take a look at inquire and see what I can come up with.

@citizen428 Sure, let me know if I can help with anything if you end up looking into it 🙂

This should be fixed by #111, which upgraded to the latest version of inquire. The prompts are now bold only and use the default color of the terminal.
I'll cut a release of Spin soon, so the next version will be usable on light terminals 🙂


LGETKF: add function to tune horizontal localization for each observer

Description
This PR adds a mechanism to customize horizontal localization (e.g.,  Gaspari-Cohn) for obs in LGETKF.

add a directory /jedi/interfaces/geos_atmosphere/observations/localization
each yaml in that directory is used to update localization for the corresponding observer

Dependencies
Impact

A quick question, did we confirm having localization in observation YAMLs break the Variational applications?

A quick question, did we confirm having localization in observation YAMLs break the Variational applications?
@Dooruk, the added directory jedi/interfaces/geos_atmosphere/observations/localization is used by local ensemble da only.   And other modifications are in

M       src/swell/tasks/run_jedi_local_ensemble_da_executable.py
M       src/swell/test/suite_tests/localensembleda-tier1.yaml

This PR does not affect variational runs.

I don't want to hold back this any longer, thanks @metdyn for working on this.
@mranst just FYI the 3dvar suite doesn't work on SLES15 in the gmao_ci context so tier1 test will break, but 3dvar suite works locally on my end with defaults. I'm a bit puzzled. I will merge this PR and look into the issue, I suspect something to do with the node allocations.


Saving training results to file, loading trained network from file

Hi,
Is there any way to do something like that:
use Phpml\Classification\KNearestNeighbors;

$samples = [[1, 3], [1, 4], [2, 4], [3, 1], [4, 1], [4, 2]];
$labels = ['a', 'a', 'a', 'b', 'b', 'b'];

$classifier = new KNearestNeighbors();
$classifier->train($samples, $labels);
$classifier->save('trained_network.txt');

$classifier2 = new KNearestNeighbors();
$classifier2->load('trained_network.txt');
$classifier2->predict([3, 2]);

Any advice how to save and load trained network?
Have a nice day! :)

Actually there is ModelManager class, pleas read: http://php-ml.readthedocs.io/en/latest/machine-learning/model-manager/persistency/


Build failing on arm in void linux

Describe the bug
After updating to 0.20.0, arm (6 and 7) builds are now failing for the spotify-player package
To Reproduce
These builds are automated in void-linux github actions. To reproduce simply create a PR updating
the version to 0.20.0
https://github.com/void-linux/void-packages/actions/runs/11687808367/job/32546762512
https://github.com/void-linux/void-packages/actions/runs/11631015422/job/32391178874
Expected behaviour
The build to succeed
Log and backtrace
We don't get to this point, since the build is failing. Build output in the 2 jobs above
Please refer to this URL for more details.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment

OS: void linux
Application version: 0.20.0
Application features: if applicable, provide cargo features used to build spotify_player

Additional context
Add any other context about the problem here.

Might be related to https://github.com/aome510/spotify-player/issues/596. Not sure how the build process works for void-linux but can you try to build with the latest master?

Make sure LTO is disabled  for the build- however you do that in void linux.

Should be fixed with latest master


Is it possible to maintain context using the "quick response" method?

I'm not sure if this is possible, but an option to maintain memory/context when using the 'quick response' method would allow quick contextual follow-ups without having to get into the chatgpt application.  I like to use chatgpt as a terminal side-kick and this would at least be valuable for my usecase.

I really like the idea! I don't have much time to implement it currently, but I would welcome and help in any PR.
It should be fairly easy to do it, we just need to make sure to reuse the same session when using chatgpt from cli, and add some command like new when we want to start a new conversation

I'm not a fan of this idea, I really think conversation management should be handled at the level of the shell program.
That being said, you could certainly create a small wrapper around the Python module to accomplish this goal yourself.

I've considered this further, and I'm pretty sure there's not a clean way to implement it.
The main problem is accurately determining which conversation to associate a one-shot prompt to.
Even if we provided a config setting to enable it, the only sane setting would be 'most recent conversation', and without being inside the interface to have visibility on the conversation, it seems way too error prone (e.g. you forgot you started another conversation, then it uses that one, and now you have a message in the wrong conversation).
The only other option I can think of is passing a conversation ID with the prompt, but at that point, it really seems more efficient to just open the shell and navigate to the conversation.


Build C++ projects with source dependencies

This should have a sample in native-samples

@eljobe could you take a look at this?

Fails on Windows due to a long file path (>300 characters):
https://builds.gradle.org/viewLog.html?buildId=8644536&buildTypeId=Gradle_Check_Quick_Java7_Oracle_Windows_languageNative
:(

I'm not sure what we should do about this.
The paths on CI are way over the limit.  This is ~300 characters (I think the C:\ don't count):

C:\tcagent1\work\6031ced305d8bea4\subprojects\language-native\build\tmp\test files\CppDependenciesIntegrationTest\can_depend_on_C___l..._from_VCS\cla6j.gradle\vcsWorkingDirs\3j106tklsik3q9jsnzaf5obc2\114d2253a10f763e1138e11c31e17b3e600f456f\log\build\obj\main\debug\bf0q611b80li3hv48z24l97tm\logger.obj

To get to just under 260, we could flatten the hierarchy again.  Instead of vcsWorkingDirs/<repo>/<commit>, we just do vcsWorkingDirs/<repo+commit>.  We can save a few characters by changing vcsWorkingDirs to just vcs.
We could move the working directory to the GRADLE_USER_HOME.  That's likely to be on a shorter path, but it may have other problems (slower/mounted over NFS, need to worry about concurrent builds).
Alternatively, we could keep the repo id and shorten vcsWorkingDirs and substitute the commit id to some internal mapping, e.g., vcs/<repo>/<mapped commit> where  would be ~single characters.  I think I'm partial to this idea.  WDYT @eljobe ?

This is more a problem with the C++ compile task (or the tool chain really). I would fix that instead of using an artificial location for the vcs working dir.

The path that we're choosing right now only leaves 22 characters left for the build in our test, so even if we put the obj files directly into build, we wouldn't fit.
Or to put it another way, 90/260 characters are due to the way we choose the VCS working directory.

This doesn't have anything to do with source dependencies. I can manually check out the same source into a directory whose path has the same length and run into the same problem. And this will keep happening to me even if we change what we happen to use for the working directory for source dependencies.
Instead of changing where we check things out to, we should fix the C++ plugin so that it works for projects that are deeply nested on the file system. For example, we can invoke cl.exe in the project directory and use relative paths to point to things.

I took a stab on my branch to use relative paths in more places: https://github.com/gradle/gradle/pull/3144
It's not complete.  Lots of unit tests fail since I need to pass around a working directory everywhere and there seems to be something special going on with the PCH integration tests, but it shows a few problems:

If the path to the working directory is over 260 characters, the compiler won't even start (it'll complain the path is too long).
If the working directory + path to a file is over 260 characters, the compiler complains the file doesn't exist.

Here's some of the failures:
https://builds.gradle.org/viewLog.html?buildId=8750150&
To confirm this, I tried running the compiler by hand from a directory just under the path limit with a file that was under the path limit (m.cpp) and one just over it (main.cpp).  You can see that the m.cpp compiles (but fails to link for other reasons) and main.cpp is reported not found.

I think making the paths relative (and the working directories for all the tools consistent) would be a good idea, but I'm going to create a new issue for that.
To just get the test coverage working again, I shortened the path by reducing the length of the test name and switching over to the non-Git VCS backend.  See https://github.com/gradle/gradle/commit/e8b69d267dde8e3f6533e850e675e18885a2e95c

Relative path issue is #218


Upgrade github actions runner helm code and chart to v2

Removed all old v1 code, and using v2 code
Changelog
Instead of a single Actions Controller, it is replaced with a scale set controller and scale set.
Scale set requires Scale set controller to be deployed and working first before its own deployment.
New version only requires the Github Auth creds at the Scale Set level, controller level no longer requires it.
Webhook server is removed and autoscaling handled by listener in Scale Set deployment.

@uchinda-sph can help to take a look?


Re-enable ruby cck

It was disabled recently because it won't pass until cucumber-ruby is updated to use the latest messages. Let's wait to do this until all the open messages issues/PRs have landed on master and we've made a new monorepo release.

It'll have to wait for the next release of cucumber-ruby as this uses the released gems.

Bump @vincent-psarga

Bump @vincent-psarga

We should move compatibility-kit/ruby into the cucumber-ruby repo and run there. A failure in the ruby cck is a failure of cucumber-ruby, not the monorepo.
We can use the same approach as in Cucumber-JVM to copy over the golden messages:
https://github.com/cucumber/cucumber-jvm/blob/fa9085425282d1eacfc42eed4eee9e1b004ea9f4/Makefile#L21-L30

Reopening this again since it fails the build for some reason. We need to move this out of the monorepo and into cucumber-ruby.

Done above


unmerge master with the branch

Accidently merged the master and the branch. If it's important, we can reverse that.

@kulvait , this is what I mean. the  '/data/gan_phase/data_spider.tif' is not in the main repo anymore. Perhaps It was removed due to the branching?


[libmupdf] Update to 1.19.0-rc2

I'm trying to update libmupdf to 1.19.0-rc2 on linux but I'm having issues during compilation:
...
/libmupdf/src/aed442f7f0-251a14e671.clean/source/fitz/memento.c
/work/vcpkg/buildtrees/libmupdf/src/aed442f7f0-251a14e671.clean/source/fitz/memento.c:45:10: fatal error: memento.h: No such file or directory
   45 | #include "memento.h"
      |          ^~~~~~~~~~~
compilation terminated.
...

The memento.h file is in the include/ directory which should be included with https://github.com/microsoft/vcpkg/blob/master/ports/libmupdf/CMakeLists.txt#L41 so I'm really not sure how to fix that. Any ideas?
Basically I just changed the REF value in the portfile:
diff --git a/ports/libmupdf/portfile.cmake b/ports/libmupdf/portfile.cmake
index 7eb51680f..98f20cd34 100644
--- a/ports/libmupdf/portfile.cmake
+++ b/ports/libmupdf/portfile.cmake
@@ -5,8 +5,8 @@ vcpkg_check_linkage(ONLY_STATIC_LIBRARY)
 vcpkg_from_github(
     OUT_SOURCE_PATH SOURCE_PATH
     REPO ArtifexSoftware/mupdf
-    REF af0e25cec567868a04eaacf6410c395712fe4b90 #1.18.1-so-3.11.14
-    SHA512 3dc6b75964d93af84921ee30a5b14e0ab84d16afa31f97a0fbf62e2006ace62f9c0366d1c3872cd678dab71eb23a422daeca0eb0b5db58e434f27657bbf9b5bc
+    REF 61b63d734a7b9df618f6b45dda2466aed442f7f0 # 1.19.0-rc2
+    SHA512 16661c012e18ac72b24c46caf5c02515c29a05e0a8dcf95076eff3a1f2e87c225245037480ed37068858fe6e04ff4a404f69877599b208ab9265d054ec117820
     HEAD_REF master
 )

diff --git a/ports/libmupdf/vcpkg.json b/ports/libmupdf/vcpkg.json
index f0342c005..6b16b9a9d 100644
--- a/ports/libmupdf/vcpkg.json
+++ b/ports/libmupdf/vcpkg.json
@@ -1,6 +1,6 @@
 {
   "name": "libmupdf",
-  "version-string": "1.18.1-so-3.11.14",
+  "version-string": "1.19.0-rc2",
   "description": "a lightweight PDF, XPS, and E-book library",
   "homepage": "https://github.com/ArtifexSoftware/mupdf",
   "supports": "!osx",

For reference, the tarball from their website compiles fine on this machine. I attached the full log of the error.
install-x64-linux-dbg-out.log
Thanks!

Hi @simon987
Thanks for posting this issue.
This issue is caused by the upstream commit https://github.com/ArtifexSoftware/mupdf/commit/87b55c20ad61b8c6755801097d6aec7b3bee3beb#diff-29f0c2b0981540541a1267690d4b872a0e65173f7542d4c6053f5da870716834.
To enable it work on vcpkg, we should define MEMENTO_MUPDF_HACKS. Since upstream has added it in Makerules and libmupdf.vcxproj.
Could you please try to add it to CMakeLists.txt to rebuild this?
if(WIN32)
  target_compile_definitions(libmupdf PRIVATE -DSHARE_JPEG -DFZ_ENABLE_JS=0 -DFZ_ENABLE_ICC=0 -DMEMENTO_MUPDF_HACKS)
else()
  target_compile_definitions(libmupdf PRIVATE -DSHARE_JPEG -DFZ_ENABLE_JS=0 -DFZ_ENABLE_ICC=0 -DHAVE_PTHREAD=1 -DMEMENTO_MUPDF_HACKS)
endif()

https://github.com/microsoft/vcpkg/blob/9d892ff9270f2d2fe3b8e490471f0a7dfa78981e/ports/libmupdf/CMakeLists.txt#L34-L38

Hi,
With your proposed changes I'm getting this error now:
/work/vcpkg/buildtrees/libmupdf/src/aed442f7f0-251a14e671.clean/source/fitz/output-docx.c:24:10: fatal error: extract.h: No such file or directory
   24 | #include "extract.h"
      |          ^~~~~~~~~~~
compilation terminated.

I think it's looking for this dependency: https://github.com/ArtifexSoftware/extract
I have no clue what output-docx.c does. I assume it's part of the mutool binary? (which is not compiled in vcpkg anyway) If I exclude it like this:
diff --git a/ports/libmupdf/CMakeLists.txt b/ports/libmupdf/CMakeLists.txt
index 6800bea9a..ec267d5e4 100644
--- a/ports/libmupdf/CMakeLists.txt
+++ b/ports/libmupdf/CMakeLists.txt
@@ -28,13 +28,14 @@ find_library(JBIG2DEC_LIB NAMES jbig2decd jbig2dec)
 file(GLOB_RECURSE SOURCES "source/*.c" "generated/*.c")
 list(FILTER SOURCES EXCLUDE REGEX "source/tools/[a-z]*\\.c$")
 list(FILTER SOURCES EXCLUDE REGEX "source/tests/.*.c$")
+list(FILTER SOURCES EXCLUDE REGEX "source/fitz/output-docx.c")

 add_library(libmupdf ${SOURCES})

 if(WIN32)
-  target_compile_definitions(libmupdf PRIVATE -DSHARE_JPEG -DFZ_ENABLE_JS=0 -DFZ_ENABLE_ICC=0)
+  target_compile_definitions(libmupdf PRIVATE -DSHARE_JPEG -DFZ_ENABLE_JS=0 -DFZ_ENABLE_ICC=0 -DMEMENTO_MUPDF_HACKS)^M
 else()
-  target_compile_definitions(libmupdf PRIVATE -DSHARE_JPEG -DFZ_ENABLE_JS=0 -DFZ_ENABLE_ICC=0 -DHAVE_PTHREAD=1)
+  target_compile_definitions(libmupdf PRIVATE -DSHARE_JPEG -DFZ_ENABLE_JS=0 -DFZ_ENABLE_ICC=0 -DHAVE_PTHREAD=1 -DMEMENTO_MUPDF_HACKS)^M
 endif()
 target_include_directories(libmupdf
   PUBLIC

it compiles without errors, however, when I try to compile my app I get linker errors:
/usr/bin/ld: /work/vcpkg/installed/x64-linux/lib/liblibmupdf.a(noto.c.o): in function `fz_lookup_builtin_font':
noto.c:(.text+0x493): undefined reference to `_binary_CharisSIL_cff_size'
/usr/bin/ld: noto.c:(.text+0x49f): undefined reference to `_binary_CharisSIL_cff'
/usr/bin/ld: noto.c:(.text+0x601): undefined reference to `_binary_NotoSerif_Regular_otf_size'
/usr/bin/ld: noto.c:(.text+0x60d): undefined reference to `_binary_NotoSerif_Regular_otf'
/usr/bin/ld: noto.c:(.text+0x61a): undefined reference to `_binary_CharisSIL_Bold_cff_size'
/usr/bin/ld: noto.c:(.text+0x626): undefined reference to `_binary_CharisSIL_Bold_cff'
/usr/bin/ld: noto.c:(.text+0x638): undefined reference to `_binary_CharisSIL_Italic_cff_size'
/usr/bin/ld: noto.c:(.text+0x644): undefined reference to `_binary_CharisSIL_Italic_cff'
/usr/bin/ld: noto.c:(.text+0x64c): undefined reference to `_binary_CharisSIL_BoldItalic_cff_size'
/usr/bin/ld: noto.c:(.text+0x658): undefined reference to `_binary_CharisSIL_BoldItalic_cff'
...

I think that the make generate step does not work properly; when I try to run it by myself I get this error:
simon987@pc:/work/vcpkg/buildtrees/libmupdf/src/aed442f7f0-251a14e671.clean$ make generate
make: *** No rule to make target 'thirdparty/extract/src/template.docx', needed by 'thirdparty/extract/src/docx_template.c'.  Stop.

It looks like it's trying to generate things for the extract third-party dependency (which vcpkg does not have) here:
https://github.com/ArtifexSoftware/mupdf/blob/master/Makethird#L270
Should I attempt to comment out this part with a patch? I'm not really sure how to do that

Generally speaking, we don't suggest to use thirdparty. I think we can comment this out. You can try to rebuild this.
Please also provide your application if you still encountered problem.

@NancyLi1013 I removed the extract third-party dependency parts from Makethird and had to use FZ_ENABLE_DOCX_OUTPUT to make it compile & link. Could you review the PR I just created?


skip over master brave window when querying brave/browser-laptop#8811

One of the issues with that is -1 already indicates WINDOW_ID_NONE. Normally I would agree, but in this instance it's probably better to filter on something more definitely unique to the main Brave window. (Perhaps, it should have a special key set on its object.) That said, -1 will (should) still work if we decide to change it.

@kevinlawler I'm not 100% .url.indexOf('chrome://') will match only the Brave frame. We could also check that the title matches /^brave$/i to be certain.
{
    "active": false,
    "audible": false,
    "autoDiscardable": true,
    "discarded": false,
    "height": 1050,
    "highlighted": false,
    "id": 1,
    "incognito": true,
    "index": -1,
    "mutedInfo": {
        "muted": false,
        "reason": "user"
    },
    "pinned": false,
    "selected": false,
    "status": "complete",
    "title": "Brave",
    "url": "chrome://brave/C:/Users/jjdsa/AppData/Local/brave/app-0.15.303/resources/app.asar/app/extensions/brave/index.html",
    "width": 1922,
    "windowId": -1
}

OK, I'll match against -1 then unless there are objections.
Matching against title=="Brave" I think gives us the same problem before: false positives are improbable but a possible bug nonetheless.

I wasn't suggesting a single check, but rather "chrome://" and "Brave" 🙂 Either one alone could yield false positives indeed. We'll, "chrome://" may not—I'm not sure.

definitely not save to check for indexOf('chrome://'), but indexOf('chrome://brave') would be safe

Thanks, @bridiver. @kevinlawler, how about tab.url.startsWith('chrome://brave/')?

sorry I've let this one sit forever. Is this safe to merge for the next build? Doesn't require any browser-laptop changes?

@bridiver No changes to browser-laptop.

ok, I'll merge and look at removing the TabHelper instead when I have time

Resolves https://github.com/brave/browser-laptop/issues/8822.
@bridiver, do you know when this should appear in preview builds? Still repros for me in 0.19.0 (https://github.com/brave/browser-laptop/commit/c5968c5ea2a8eaab1834d638aa4ead5af5d517e2).

It should be in 4.1 builds for muon

On Jun 18, 2017, at 1:00 PM, Jonathan Sampson<EMAIL_ADDRESS>wrote:
Resolves brave/browser-laptop#8822.
@bridiver, do you know when this should appear in preview builds? Still repros for me in 0.19.0 (brave/browser-laptop@c5968c5).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


fa-user-circle missing

fa-user-circle and fa-user-circle-o seem to be missing.

+1

Fixed by https://github.com/encharm/Font-Awesome-SVG-PNG/pull/54


Wait for authorization policies should use count

If an authorization policy is being skipped/is not needed, the wait block for that authorization policy should also be skipped. The counts for these wait blocks should match the resource block creating the authorization policy.

:tada: This issue has been resolved in version 1.20.2 :tada:
The release is available on:

GitHub release
v1.20.2

Your semantic-release bot :package::rocket:


New version of system.js v0.19.29 broke loading app in IE 11

https://gyazo.com/4bf2cfa58b0bd04696b61019c8d47e5b

This doesn't seem like an Aurelia issue.


build options added

change the inlining options and npm run build instead of production

Thanks for the PR! Can you direct this to master for docs updates? Thanks!

Yes @rafibomb, I placed the code block inlined to save space, but tell me if its better to have liked before
https://github.com/zurb/foundation-emails/pull/467

Thanks for the updated PR!


analyze the code on the CI

analyze the code on the CI

cc @lambdabaa

There are also tons of errors coming out of dartanalyzer for me locally after landing the initial revision. Is there initial project configuration needed beyond pub get?

Great, thanks! My only question is about pub run test. Does that assume that your entrypoint is test/all.dart? It does seem to run all of the tests correctly.

package:test uses the analyzer (I believe w/o resolution) to scan files in test/ and determine which are tests. Having an test/all.dart which delegates to all other tests isn't necessary (but I don't believe can hurt).

@devoncarew There are also tons of errors coming out of dartanalyzer for me locally after landing the initial revision. Is there initial project configuration needed beyond pub get? Do we need an analysis options file?

I see one failure in the CI on this PR:
https://travis-ci.org/dart-lang/tflite_native/builds/560757540#L507
Which I didn't address, as I think your in-progress PR will fix it.
We don't need an analysis options file locally unless we want to tighten things down to be more restrictive. I'd suggest we look into that after this lands, if desired.

there are also tons of errors coming out of dartanalyzer for me locally

It shouldn't need any more config than pub get. What are the errors?

Also, I don't know how well dart:ffi will analyze...

And, I think you'll need to merge this PR.

It shouldn't need any more config than pub get. What are the errors?

Interestingly they aren't showing up on CI, so we can land. I'll also go ahead and add you as a collaborator. Thanks!


Re-export Coordinate

Would you consider re-exporting Coordinate? The way things stand now, users of this crate also need to explicitly import geo.
(Edit: in which case it's down to the user to explicitly co-ordinate the versions of Coordinate between geo, and geohash: using geohash with geo 0.8.0 doesn't work.)

potentially relevant rust-geo pull request: https://github.com/georust/rust-geo/pull/201

It is relevant but I'm not sure it's a solution: I think it would still require manual version co-ordination.

done in https://github.com/georust/rust-geohash/pull/19


Refactor project

Refactor project structure and naming convention after this schema

rest.v1 -> service
RestController -> RestService
Animal -> AnimalEntity

@lilliCao @hohwille

For the service you also dropped the versioning from the package. There is an ongoing discussion to create a new service by "forking" the app and then having both versions (e.g. v1 and v2) as separate apps, containers and processes. In that case it would be a nice simplification (at least for dev maybe not for deployment and SCM). Also in devon4j we so far only prepared the structure and design for service versioning but tried to keep the default as simple as possible. So the package without any version is the latest version. Only when you need to start changing an API incompatible (and have consumers that can not be changed at the same time), we copy the existing code to a v«x» package and change the "latest" API afterwards. Therefore IMHO this change is fine as well.

Packages for versioning is one side of the problem, but we need to keep in mind also the 'path' and openapi schema.
When we have the latest and no package versioning, we need to keep it without change and a make new version with a new path otherwise, it will break contract to the consumer and API cannot be classified as stable.
We can still have no version in package and version in path `/v1/* but this could be confusing for developers. For the API-First approach, developers do not have a control over the versioning of the API. In few services, new version v2 was already planned and designed before the version v1 was in production.
Making a new version with separated microservice will work for simple microservice and where you do not need to support multiple versions of the API for long time period. With this approach, we need to copy also the business logic and support multiple source bases. It is not possible, but we should have written down the downside of this approach.
Andrej


dhcp6c handling, multiple LAN tracking, advanced dhcp6 fix.

This commit corrects an issue when using advanced dhcp6c settings in that it now correctly configures the LAN interface(s) names. It also allows multiple LAN interfaces  to be used.
A complete break out of the configuration creation and script creation has been done, this is because to correctly allow for a change to a LAN interfaces that may affect tracking, the script creation MUST take place in the interface_configure routine  as a change to LAN tracking must be caught and the configs adjusted before calls to interfaces_dhcp6_configure.
New routines have been added that launch or reset dhcp6c plus kill and find & reset. These are all there to try and mitigate the chances of multiple dhcp6c clients. The kill_dhcp6c_client_process takes advantage of feature added to dhcp6c last year. Using the USR1 signal will force dhcp6c to exit cleanly but with no release. This is important, as a lot of the issues with dhcp6c are caused by timing. On a normal TERM  kill, dhcp6c will try to release any assigned values by signalling the server and waiting for a response, this takes time. Thus if we are taking down the interface or making changes we want an immediate exit and as a new REQUEST will be sent we are not interested in releasing, thus dhcp6c exits quickly.
RTSOLD has been changed to a one shot. I am aware of a previous issue but the clean-up here around dhcp6c should mean that there is no requirement for it to be resident.

Removed unused lock file deletion reference.

@marjohn56 can we split out the smallest required atomic part and reiterate until we have all we need? I see some code that seems to be borrowed elsewhere and in part seems to revert the code back to where it was in terms of code style... both of which make me doubt we are going forward with the whole of this patch :/

It needs it all as a whole to work. OK, let's break this down into parts.
There are new functions that handle the starting, killing and process finding of dhcp6c. Those a pretty straight forward, the reason for them is to help mitigate multiple dhcp6c clients. Using them takes advantage of the extra bits that exist in dhcp6c that opnsense is not currently using,  See line 2524, it explains it. The Key is the use of SIGUSR1. There is also a function reset_dhclient_process(), This uses the additional function I have added to dhcp6c to re-read the conf on sigup, thus if the process is running we do not kill it, we just send a sigup and let it carry on, no chance of a new dhcp6c process.
The next major breakout is the function interface_dhcpv6_configure, all I have done there is break out all of the configuration functions from that call and there is a very good reason for this. Whenever LAN config changes, i.e. tracking, then that change MUST be reflected in an update to dhcp6c_*.conf, but we do not always want to call a interface_dhcpv6_configure, especially not at that point, the call to re-write the config is in interface_configure. I have made a comment at around line 2747 that explains it.
Now, we also need to take into account that there may be an advanced dhcp6c needed, this is also handled in the function configure_dhcp6c_conf()  which calls the config routines needed, whether they are  basic or advanced_config functions, it also writes the RTSOLD script as needed.
Now, I know you have this thing about rtsold needing to be running constantly, and I have removed the -1 parameter to leave it resident, but the issue there is that if there is an RS before dhcp6c solicit and know for a fact that Sky UK will then ignore the dhcp6c request, hence scripts either have dhcp6c launching RTSOLD or RTSOLD launching DHCP6C.
I am happy if you wish to break this all down into separate commits, but they do form a whole process.
Where do you see code regression?

Slept on it...
OK.. First thing we need to do before any of this commit is to get the dhcp6c changes integrated and sort out dhclient vlan priority. Once those are done I will re-work  this commit so it's done in separate stages. The dhcp6c changes I have already committed, dhclient appears to use raw sockets hence it;s refusal to play nicely.

Ignore this PR, breaking down into smaller chunks and will then delete

@marjohn56 Thanks, much appreciated. We still have the dhclient patch on this branch.... https://github.com/opnsense/src/commits/dhclient_77
I'm not opposed to this, but it needs testing and maybe fixing as I've tried to break it down and simplify as much as possible.

Not sure what you mean with 'I'm not opposed to this, but it needs testing and maybe fixing'. Do you mean dhclient?

yes, patching dhclient to enable vlan because set-prio doesn't seem to work. it's not a straightforward way, but we can always reiterate and find a better solution later when we have a working base implementation. :)

OK, I'll take a gander...

Also continued in #2258


Configure the Celery service to have a backend

The Celery service needs to have a backend service to communicate task statuses and results. Set up a backend service for the Celery service and have the Celery service utilize it.

This is to be closed by #222.

#222 is complete and merged. Thus, this is done.


Add heroku upload button.

I think you should add a Heroku Easy Setup Button for easy installation.
What do you think @tobiaslins ?

So i made #8 that adds this! Remember to change variables in app.json and in README.md otherwise it ain't going to be pretty!

Thanks for the merge! :D


Deprecate and remove source fetching

Adds code bloat and a better solution is to have the Sentry server manage releases.

Do you mean sourcemap fetching that enables readable exception stack trace? Very useful feature for us, otherwise a compiled JS stack trace is pretty useless.

@j15e we mean "linesOfContext" as it's not very useful and causes problems with packet size (the server takes care of this)

Specifically, this is related to fetching the source from the client, not the server. :)

Oh ok great! Thanks for the quick reply.

I believe #585 fixed this

@statianzo – Closing, and thanks.


Option to return 'in ....' or '... ago'

Hello,
first let me thanks for a very helpful project!
Now to the issue/feature request. I would like to use this in template, showing the last_changed attribute in local language. In English, this would follow by calling relative_time_plus() and appending 'ago'. In the Czech language, this is not possible to do due to declensions. As an example:
relative_time_plus() may output '5 minut' (5 minutes) but the correct string is 'před 5 minutami' (past). A future time would read 'za 5 minut'. For 1 minute, we would have '1 minuta', but 'před 1 minutou' (past) and 'za 1 minutu' (future)...
Is there a way to implement this?

You're explaining exactly why I didn't implement this. I know how this would work for Dutch and English, as I speak those two languages. But I also know it's relatively simple in those two languages.
For Czech (and I guess for more Slavic languages) this would mean every translated time fraction should have 4 more translations (singular past, singular future, plural past, plural future). For other languages it might be that the suffix/prefix like (in or ago might change based on the time fraction). And some languages might use a suffix, where others use a prefix. So besides the 4 additional translations for Czech, You would also need suffix_past_singular, suffix_past_plural, suffix_future_singular, suffix_future_plural, prefix_past_singular, etc, etc.
Then I would need help from all of you to add those for all languages.
So it can be done, but it's a lot of work to support this for all languages. Maybe I can have a look in the near future, but I'm a bit busy now.


Error running Example

$ python examples/example.py
Traceback (most recent call last):
  File "examples/example.py", line 1, in <module>
    from sdf import *
  File "/Users/jimmygunawan/sdf/sdf/__init__.py", line 1, in <module>
    from . import d2, d3, ease
  File "/Users/jimmygunawan/sdf/sdf/d2.py", line 5, in <module>
    from . import dn, d3, ease
  File "/Users/jimmygunawan/sdf/sdf/dn.py", line 7
    def union(a, *bs, k=None):
                      ^
SyntaxError: invalid syntax

Hmm ok python3 examples/example.py seems to work

For the records, I faced the same problem but just running python3 examples/example.py wasn't enough.
I needed to run pip3 install -e . (instead of pip install -e .).


[BUG]: _videoRef$current.setNativeProps is not a function (it is undefined)

Version
6.1.2
What platforms are you having the problem on?
No response
Architecture
Old architecture
What happened?
A bug occurred!
To dynamically change the token, I use the following code:
videoRef.current?.setNativeProps({
      source: {
        uri: uri,
        type: 'm3u8',
        headers: {
          Authorization: `Bearer ${token}`,
        },
      },
    });

The above code works well in version 5.2.1.
Reproduction
repository link
Reproduction
Step to reproduce this bug are:
open the app
run the video player

setNativeProps is deprecated, you should save source in state - if you want to update source via ref you will need to create your own component with forwardRef that will handle update of source state

but it reset the video duration

setNativeProps is deprecated, you should save source in state - if you want to update source via ref you will need to create your own component with forwardRef that will handle update of source state

@KrzysztofMoch After making these changes, the progress value goes back by 15 seconds with each token change.

Hello @amitmehtacode, I am a bit surprised, because header is not parsed in source 🤔
There is only a field "requestHeaders" in source which is not documented :/

Hello @amitmehtacode, I am a bit surprised, because header is not parsed in source 🤔 There is only a field "requestHeaders" in source which is not documented :/

But If I remove the headers:{}, then the video is not playable.

Hello @amitmehtacode, I am a bit surprised, because header is not parsed in source 🤔 There is only a field "requestHeaders" in source which is not documented :/

I also put requestHeaders but still the video is not playable 🙁

@freeboub Any update for the same. Can you also share the docs


OpenSDS CSI pods in CrashLoopBackOff state

Is this a BUG REPORT or FEATURE REQUEST?:

Uncomment only one, leave it on its own line:

/kind bug

/kind feature

What happened:
Tried to install OpenSDS with Kubernetes CSI on machine named hs-opensds-test, following this doc: https://github.com/opensds/opensds/wiki/OpenSDS-Integration-with-Kubernetes-CSI.
After OpenSDS with CSI installation all CSI pods showed CrashLoopBackOff state, checked the pod logs got this error:
Invalid value: "hs-opensds-test,iqn.1993-08.org.debian:01:9a1f1265941c,nqn.ini.29E8D714-84DA-423B-ED25-5B33877FB6BB.hs-opensds-test,<IP_ADDRESS>": must be 128 characters or less,}
What you expected to happen:
All the CSI pods should run irrespective of cluster name
How to reproduce it (as minimally and precisely as possible):
Try to setup OpenSDS with Kubernetes CSI on Kubernetes cluster of name greater than 14 characters
Anything else we need to know?:
Environment:

Hotpot(release/branch) version: v0.6.0 Capri
OS (e.g. from /etc/os-release): Ubuntu 16.04.6
Kernel (e.g. uname -a): Linux hs-opensds-test
Install tools:
Others:

Current status:

Code modifications in progress based on host management changes

Pending Items:

Code completion
E2E Test

Expected closure date: Nov 21


Default name from base64 is 'aaaaa'

Super minor detail, but surely this isn't intentional? A better default could be "Player" or something

It gave me a good laugh though :)

not intentional... LOL
yeah i’ll get this fixed


Ensure Ids are included in render timeline if needed

If timeline is being applied on the frontend (i.e., TileAdmin.enableFrontendScheduleScripts is true), the element Ids need to be included.
This was the case for scripts stored in the display style's JSON properties, but not for those stored on RenderTimeline elements - they were always unconditionally omitted.

@mergifyio backport release/4.0.x


Rough draft for Kazari JS plugin for code decoration

This PR introduces "Kazari", a Scala.JS-powered plugin that will allow developers to add Scala Exercises-like abilities to their documentations. Basically it will traverse the DOM of any site which includes the generated JS script, decorate any Scala code block by adding an "evaluate" button (which will send requests to our Scala remote evaluator) and also allow live code edition and evaluation.
Right now the plugin just decorates the code with a button to allow calls to the remote evaluator, sending basic requests to the remote evaluator. Refactoring of the network code is also needed, as now it's using JQuery simple Ajax calls. Plans to integrate the remote evaluator client are in motion!
This PR is partly dependant on scala-exercises/evaluator#31, which allows the Scala evaluator to receive CORS calls (like the ones produced by the plugin).
Could you please review @raulraja @dialelo @rafaparadela? Thanks!!

@raulraja @dialelo: @juanpedromoreno and I have added support to use the evaluator-client to establish connection with the remote evaluator. Could you please review?


Remove the PostCode::setService() and PostCode::getService() method

Related to #107

This repository has been closed and moved to laminas/laminas-i18n; a new issue has been opened at https://github.com/laminas/laminas-i18n/issues/5.


Draggable: reset internal property _moved on drag end (instead of start)

Prevents race conditions when canvas process events earlier than map.
Fix #7007.
Note: Tests seem passed, but I have not reviewed all possible code paths of Draggable.

Note: Tests seem passed

But it means only that we have no test for this case.
Unfortunately this change brakes proper click detection:
	var mymap = L.map('mapid', {tap:false}).setView([51.505, -0.09], 13);
	var marker = L.marker([51.5, -0.09],{draggable:true}).addTo(mymap);
	mymap.on('click', console.log);

After marker drag click MUST NOT occur .


Add ability to Link OU to entities (i.e. measurables)

Needed for linking process model to owning units.

Fixed in an earlier relase 1.2x ?


compileOnly files('C:\Android\flutter\bin\cache\artifacts\engine\android-x64\flutter.jar') breaks build on non-windows environments

https://github.com/fluttercommunity/flutter_wear_plugin/blob/87ef7bb7fd43b39bd26bd4d3f9c44f843498e46f/android/build.gradle#L49
This line breaks builds on non-Windows environments.

Bump on getting this merge - currently using regex to fix on my pipeline ;)

When would this fix be released? @mjohnsullivan

up

@johnson-yeap just releasing it now. Sorry chaps. Left that line in by mistake.

Thanks @sdstolworthy and everyone. Published fix as 1.1.0 https://pub.dev/packages/wear/versions/1.1.0


Replay Mirana/Slark juke from opposite (or all-vision) perspective

We might want to add a audio dialog: "Where did she go? Let's see it from both sides now." and then do it, to explain why it's happening twice.

Done in #292 by showing Mirana through fog


Docker version of Clipbucket

Hi , just want to check do you have plan to push an working image of CB to dockerhub so that it is more convenient to use?

Sorry, I'm no longer maintaining this role. I don't think the core developers invest enough into supporting installs. I've switched to MediaGoblin, which supports Docker out of the box.


fix / adding DS changes to skillable

TODO: review merge conflicts

Cancelling PR - will resolve changes locally first.


User ID mismatch issue

It appears that the user IDs are not aligning correctly, leading to inconsistencies in the data. This pull request aims to address this issue by ensuring that user IDs are synchronized across the system.

@Rizayev i think its better to hide users selector if the checkbox is checked, can you update it?

@Rizayev Thank you for your contribution. I appreciate it.


Cloud Gaming Link Disappeared

The link to view all cloud gaming suddenly disappeared. It was playing fine earlier today and then the link disappeared. I'm not sure what happened.

Sometimes the button disappears if there are issues with Xbox Live. More details in #300


Unexpected behaviour with invalid first line of request

Consider the HTTP request GET /path?param=This is invalid HTTP/1.1 which is missing escaping of whitespace.
Now run the http server hello world example of node (6.7.0) and send the request to the server either with telnet or curl.
I would expect the server to respond with a 400 Bad Request, but the code just waits and does nothing.
Is this behaviour intentional? HTTP servers in other languages (at least golang and python) respond with 400 Bad Request

I can't really reproduce it. Would you care to provide a test case?

To make it clearer, it doesn't send 400 indeed, but this behavior is configurable. Default one is to close connection.

I should have included a test case in the first case. I cannot reproduce it either at this time..
The next time I encounter this behaviour I'll make a test case !


Add a temporary workaround to layernorm ONNX export

Seems like ORT is performing template-matching for LN and incorrectly concludes that it doesn't have a kernel for FP32 LN. The work-around adds the addition of fake_zero which is meant to prevent the template matching while keeping the graph virtually unchanged. This also requires do_constant_folding=False in torch.onnx.export.

/te-ci

@ptrendx I pushed fixes for the linter errors. I think that's all that failed

/te-ci

/te-ci

/te-ci

Will this fix the spurious CI failures we've been seeing with layernorm?
Yup

@timmoon10 I'm sorry, but I have made some changes to the code after your review!
The changes simplify the previous changes and add verbose explanation of the tests.
The purpose is to make the tests more robust.

/te-ci


feat: Nightly build publish

TODO: Need better text for release
How does it impact release page?
Shall we publish docker nightly also?


[5.8] Add deleted_at date cast for soft deleting models

This PR utilises a trait initializer to automatically add deleted_at to the $dates array of any model that uses the SoftDeletes trait. Previously this has to be added manually to each model - unlike the created_at and updated_at that is handled "under the hood".
In essence...
class Post extends Eloquent
{
    use SoftDeletes;

    // no need for this anymore
    // protected $dates = ['deleted_at'];
}

Notes:

An existing$casts entry for deleted_at will take precedence (test exists).
A duplicate $dates entry for deleted_at will have no impact as the array is not a key => value pair, but a value array. Also when getDates is called it already runs array_unique to remove any duplicate entries (test exists).
An existing mutator getDeletedAtAttribute will take precedence (test exists).

Breaking:
This will be a breaking change if you are not expecting the value of deleted_at to be cast, i.e. you never added it to the $dates array as outlined in the docs. But as mentioned earlier, a mutator will take precedence if for whatever reason you don't want the automatic casting.

Is there any path for opting out of the cast?

Just added a new test to confirm the current behaviour can be maintained by adding a cast.
This is the easiest opt-out and the one I see being in the upgrade guide if this PR was to be accepted. A boolean toggle, as used for created_at and updated_at, seemed like overkill as the cast is a one-liner.
protected $casts = [
    'deleted_at' => 'string', // retrain current functionality
];

It is also possible to opt-out with an attribute mutator - but obviously that is more than a one liner.
If you think a boolean toggle would be better for consistency I am happy to add that instead.

I actually think I was mistaken and the cast will cover all cases as a opt-out. If you prefer a boolean flag approach I can add the following to this PR...
trait SoftDeletes
{
    protected $castDeletedAtToDateInstance = true;

    public function initializeSoftDeletes()
    {
        if ($this->castDeletedAtToDateInstance) {
            $this->dates[] = $this->getDeletedAtColumn();
        }
    }

    // ...
}

And then the dev can opt-out with...
class Post extends Eloquent
{
    use SoftDeletes;

    protected $castDeletedAtToDateInstance = false;
}

I think the boolean flag is the better approach after sitting on this for the day so I've implemented that, but in a seperate commit so you can roll it back if you prefer the previous approach.

Not that I have a better alternative but the current property name looks super confusing and long..

Eh, I think I prefer the string cast to the property.

Just reverted and removed the boolean flag. However Travis is said to have failed - but as I'm looking at it right now it actually hasn't run the tests. Not sure if you can whack it with a big spanner to get it to run again?


Not working with aws-cli / aws-amplify

Describe the bug
After installing vite and adding aws-amplify I get following error message:
[plugin:vite:imports] Failed to resolve import "process". Does the file exist?
/Users/XXXXX/node_modules/@aws-sdk/util-user-agent-node/dist/es/index.js:1:22
1  |  import process from "process";
   |                       ^
2  |  export function defaultUserAgent(packageName, packageVersion) {
3  |      var engine = process.platform + "/" + process.version;
Error: Failed to resolve import "process". Does the file exist?
    at TransformContext.error (C:\Users\XXXXX\node_modules\vite\dist\node\chunks\dep-262f13eb.js:59552:50)
    at TransformContext.transform (C:\XXXXX\node_modules\vite\dist\node\chunks\dep-262f13eb.js:52957:30)
    at async Object.transform (C:\Users\XXXXX\node_modules\vite\dist\node\chunks\dep-262f13eb.js:59742:30)
    at async transformRequest (C:\Users\XXXXX\node_modules\vite\dist\node\chunks\dep-262f13eb.js:65977:29)
    at async C:\Users\XXXXX\node_modules\vite\dist\node\chunks\dep-262f13eb.js:66068:32

Obviously the AWS Package can not resolve nodeBuiltins.
I tried to exclude the aws packages from dependency optimization and also tried to add them to the allowNodeBuiltins option in various combinations e.g. as follows:
// vite.config.js
optimizeDeps: {
    allowNodeBuiltins: [
      'aws-amplify', '@aws-cli'
    ]
 },

Is it more like a problem of the aws-cli package? Or are there any known workarounds?
Reproduction
Download https://github.com/sem4phor/aws-cli-bug
Run npm run dev
See error in console
Modify vite.config to allowNodeBuiltins and nothing happens
System Info

required vite version: 2.0.0beta5
required Operating System: Windows 10
required Node version: 15.2
Optional:

npm version 6.14.9

This is a wont fix, because aws-amplify is legit trying to import Node-only builtins when it's actually intended to be a client library. Unlike webpack 4, Vite does not shim Node built-ins. Note this means aws-amplify will break in webpack 5 as well because it no longer shims Node built-ins.
aws-amplify should stop relying on Node built-ins if it actually is meant to be used in the browser.

This is a wont fix, because aws-amplify is legit trying to import Node-only builtins when it's actually intended to be a client library. Unlike webpack 4, Vite does not shim Node built-ins. Note this means aws-amplify will break in webpack 5 as well because it no longer shims Node built-ins.
aws-amplify should stop relying on Node built-ins if it actually is meant to be used in the browser.

aws-amplify depends on aws-sdk, which upgrade itself to differentiate the node and js runtime by using the browser field. However, its browser field setting is as follows:
"browser": {
     "./runtimeConfig": "./runtimeConfig.browser" // it means if a file A encounters a relative path import, find the browser file using the path relative to file A, not the root.
}

Currently this relative path could confuse vite resolver.


typescript declaration files

when styling a declaration file, after a declared function, the syntax highlighting is gone, it may have something to do with it not containing a body

Yeah, I bet this is because we expect a {.  But I'd think it could be fixed.
Can you confirm it's still an issue and perhaps provider a jsfiddle example?
You can use my template:
https://jsfiddle.net/ajoshguy/nagkqytv/18/

Pretty sure this is still an issue but if you could provide the raw text instead of a snap that'd be super useful for us writing a test case.

Heres a link
<EMAIL_ADDRESS>And another
<EMAIL_ADDRESS>


[WIP] feat: add prop to render custom node inside minimap

This is WIP because my local env is broken and I can't test anything 😅

add nodeComponent prop to <MiniMap /> for user-defined minimap nodes.
expose MiniMapNodeProps.
add example flow to vite-app/examples.

This looks very good :) Thanks!


Add a test for the VCK5000

Summary of Changes

Add a python test for running resnet50 on VCK5000
Add a --hw filter for Pytest to optionally run only tests using some particular device

Motivation
We need a test for the VCK5000 to expand the test coverage to more boards.
Implementation
Implementation is similar to that of the tests on U250 FPGA boards: pre-process the data and construct a request -> load the resnet50 model -> send the request to the server -> get the response and post-process.
The addition here is to allow Pytest to set a custom environment per test which can be configured by the test. For now, the default environment for U250 is unchanged. It should be updated to match the latest released device (currently V70).
Notes
This PR requires existing users to redownload the test assets (resnet50 models for U250 and VCK5000 boards) or this test will give an error about missing assets.
It is an updated version of #206

Build successful!


[Bug] [SeaTunnel Client] Bug  client never stop

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
after job finished , client never stop but  displaying statistical information continuously.
SeaTunnel Version
2.3.1
SeaTunnel Config
hazelcast:
  cluster-name: seatunnel
  network:
    rest-api:
      enabled: true
      endpoint-groups:
        CLUSTER_WRITE:
          enabled: true
        DATA:
          enabled: true
    join:
      tcp-ip:
        enabled: true
        member-list:
          - node07
          - node08
          - node09
          - node10
    port:
      auto-increment: false
      port: 5801
  properties:
    hazelcast.invocation.max.retry.count: 20
    hazelcast.tcp.join.port.try.count: 30
    hazelcast.logging.type: log4j2
    hazelcast.operation.generic.thread.count: 100

seatunnel:
  engine:
    backup-count: 1
    queue-type: blockingqueue
    print-execution-info-interval: 60
    print-job-metrics-info-interval: 60
    slot-service:
      dynamic-slot: true
    checkpoint:
      interval: 10000
      timeout: 60000
      max-concurrent: 5
      tolerable-failure: 2
      storage:
        type: hdfs
        max-retained: 3
        plugin-config:
          namespace: /tmp/seatunnel/checkpoint_snapshot
          storage.type: hdfs
          fs.defaultFS: hdfs://node07:9000

Running Command
$SEATUNNEL_HOME/bin/seatunnel.sh --config $SEATUNNEL_HOME/config/mysql_test

root     11887 11844  0 15:51 pts/0    00:00:20 java -Dhazelcast.client.config=/root/install/apache-seatunnel-incubating-2.3.1/config/hazelcast-client.yaml -Dseatunnel.config=/root/install/apache-seatunnel-incubating-2.3.1/config/seatunnel.yaml -Dhazelcast.config=/root/install/apache-seatunnel-incubating-2.3.1/config/hazelcast.yaml -Dlog4j2.configurationFile=/root/install/apache-seatunnel-incubating-2.3.1/config/log4j2_client.properties -Dseatunnel.logs.path=/root/install/apache-seatunnel-incubating-2.3.1/logs -Dseatunnel.logs.file_name=seatunnel-starter-client -Xms128m -Xmx512m -XX:+PrintGCDetails -XX:+PrintGCDateStamps -XX:+PrintTenuringDistribution -XX:+PrintGCApplicationStoppedTime -Xloggc:/tmp/seatunnel-6227436161010426923 -XX:+UseGCLogFileRotation -XX:NumberOfGCLogFiles=32 -XX:GCLogFileSize=64m -XX:+HeapDumpOnOutOfMemoryError -XX:HeapDumpPath=/data/seatunnel/ -cp /root/install/apache-seatunnel-incubating-2.3.1/lib/*:/root/install/apache-seatunnel-incubating-2.3.1/starter/seatunnel-starter.jar org.apache.seatunnel.core.starter.seatunnel.SeaTunnelClient --config /root/install/apache-seatunnel-incubating-2.3.1/config/mysql_test

Error Exception
client nerver stop ,only display the following content:

2023-04-19 18:27:28,184 INFO  org.apache.seatunnel.engine.client.job.JobMetricsRunner -
***********************************************
           Job Progress Information
***********************************************
Job Id                    :<PHONE_NUMBER>00162050
Read Count So Far         :              859634
Write Count So Far        :              859634
Average Read Count        :                 0/s
Average Write Count       :                 0/s
Last Statistic Time       : 2023-04-19 18:26:28
Current Statistic Time    : 2023-04-19 18:27:28
***********************************************

Flink or Spark Version
No response
Java or Scala Version
openjdk version "1.8.0_362"
OpenJDK Runtime Environment (build 1.8.0_362-b08)
OpenJDK 64-Bit Server VM (build 25.362-b08, mixed mode)
Screenshots
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Is this a streaming job?

I forgot to upload my config, it's a batch job.
env {
execution.parallelism = 1
job.mode = "BATCH"
}
source {
OssFile {
path = "/test/20230315/"
bucket =
access_key =
access_secret =
endpoint =
file_format_type = "csv"
delimiter = "#"
schema {
fields {
ng_date = string
data = string
ip = string
channel = string
}
}
}
}
transform {
}
sink {
Jdbc {
url =
driver = "com.mysql.cj.jdbc.Driver"
user =
password =
query = "insert into test_table(ng_date,data) values(?,?)"
batch_size = 10000
batch_interval_ms = 500
}
}

@liugddx


Dockerd v1.12.6 memory leak

heap.txt
RancherOS Version: (ros os version)
v1.0.0
Where are you running RancherOS? (docker-machine, AWS, GCE, baremetal, etc.)
baremetal (vsphere 6 VM)
docker info
Containers: 120
Running: 98
Paused: 0
Stopped: 22
Images: 176
Server Version: 17.03.1-ce
Storage Driver: overlay2
Backing Filesystem: extfs
Supports d_type: true
Native Overlay Diff: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
Volume: local
Network: bridge host macvlan null overlay
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Init Binary: docker-init
containerd version: N/A (expected: 4ab9917febca54791c5f071a9d1f404867857fcc)
runc version: N/A (expected: 54296cf40ad8143b62dbcaa1d90e520a2136ddfe)
init version: N/A (expected: 949e6facb77383876aeff8a6944dde66b3089574)
Security Options:
seccomp
Profile: default
Kernel Version: 4.9.21-rancher
Operating System: RancherOS v1.0.0
OSType: linux
Architecture: x86_64
CPUs: 10
Total Memory: 19.61 GiB
Name: server01
ID: BJAD:JZTL:WXKZ:6SJH:AMOL:4457:NKS7:QOJL:MF3W:K4BY:FYEB:I2QM
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): true
File Descriptors: 10655
Goroutines: 30527
System Time: 2017-06-13T00:20:07.899030681Z
EventsListeners: 10032
Registry: https://index.docker.io/v1/
Experimental: false
Insecure Registries:
<IP_ADDRESS>/8
Registry Mirrors:
https://docker-mirror.docker.company.com/
Live Restore Enabled: false
Using centos console
After a 3-4 days dockerd consumes a large amount of memory
top - 00:22:07 up 6 days, 21:38,  0 users,  load average: 220.88, 241.41, 257.26
Tasks: 472 total,   5 running, 467 sleeping,   0 stopped,   0 zombie
%Cpu(s):  1.2 us, 29.5 sy,  0.0 ni,  0.2 id, 68.7 wa,  0.0 hi,  0.5 si,  0.0 st
KiB Mem : 20562184 total,   166300 free, 17294900 used,  3100984 buff/cache
KiB Swap:  4194300 total,        0 free,  4194300 used.   224816 avail Mem
PID USER      PR  NI    VIRT    RES    SHR S  %CPU %MEM     TIME+ COMMAND
1530 root      20   0  9.983g 3.101g      0 S  14.0 15.8 499:07.33 dockerd
7864 105       20   0 8567252 2.055g      0 S   2.0 10.5   8:01.87 java
11968 1000      20   0 11.940g 1.852g      0 S   6.0  9.4  23:01.34 java
17103 1000      20   0 8133656 1.280g      0 S   4.8  6.5  20:14.58 java
1141 root      20   0 2336364 1.061g      0 D   5.0  5.4 879:55.07 node
7520 1000      20   0 9394176 920316      0 S   2.8  4.5 276:17.32 java
21130 1000      20   0 7534088 804656      0 S   1.3  3.9 391:34.42 java
15927 105       20   0 9351724 499208      0 S   3.5  2.4   2783:43 java
Docker debug
go tool pprof --inuse_space /usr/bin/dockerd heap.txt
Entering interactive mode (type "help" for commands)
(pprof) top5
2519.65MB of 2638.57MB total (95.49%)
Dropped 1584 nodes (cum <= 13.19MB)
Showing top 5 nodes out of 59 (cum >= 31.01MB)
flat  flat%   sum%        cum   cum%
2217.84MB 84.05% 84.05%  2217.84MB 84.05%  bytes.makeSlice
157.41MB  5.97% 90.02%   157.41MB  5.97%  github.com/docker/docker/pkg/pubsub.(*Publisher).SubscribeTopic
66.21MB  2.51% 92.53%    66.21MB  2.51%  net/http.newBufioWriterSize
47.18MB  1.79% 94.32%    47.18MB  1.79%  net/http.newBufioReader
31.01MB  1.18% 95.49%    31.01MB  1.18%  runtime.malg
go tool pprof --alloc_space /usr/bin/dockerd heap.txt
Entering interactive mode (type "help" for commands)
(pprof) top5
364.97GB of 942.35GB total (38.73%)
Dropped 1481 nodes (cum <= 4.71GB)
Showing top 5 nodes out of 162 (cum >= 49.04GB)
flat  flat%   sum%        cum   cum%
122.48GB 13.00% 13.00%   123.09GB 13.06%  io/ioutil.readAll
89.57GB  9.50% 22.50%    89.57GB  9.50%  github.com/docker/docker/pkg/pubsub.(*Publisher).SubscribeTopic
52.93GB  5.62% 28.12%    66.16GB  7.02%  github.com/docker/docker/daemon/events.(*Filter).matchEvent
50.95GB  5.41% 33.53%    50.95GB  5.41%  reflect.unsafe_New
49.04GB  5.20% 38.73%    49.04GB  5.20%  reflect.unsafe_NewArray
I have been watching the following thread, seems contained to Docker 1.12.x

Thank you for the writeup and report!
I'll have to look into it after 1.0.3 is done.
have you tried Docker's current LTS release v17.03? they're more likely to make a new release of that.

Same problem here with Docker 17.05.


My initial testing of a gui implementation

This is still very buggy and only a proof of concept.  You can launch tables but don't rely on the wheel image to be the correct table.  The correct table will be shown in the cli if run from a shell.

Looks like we need to update the build setup for this one:
https://github.com/bevyengine/bevy/blob/latest/docs/linux_dependencies.md#ubuntu

I didn't realize.  I must have already have met the dependencies.
------ Original Message ------
Received: Fri, 18 Oct 2024 09:16:17 AM EDT
From: Francis De Brabandere @.>
To: francisdb/vpxtool @.>Cc: tdb30 @.>,
Author @.>
Subject: Re: [francisdb/vpxtool] My initial testing of a gui implementation
(PR #383)

Looks like we need to update the build setup for this one:

https://github.com/bevyengine/bevy/blob/latest/docs/linux_dependencies.md#ubuntu

--
Reply to this email directly or view it on GitHub:
https://github.com/francisdb/vpxtool/pull/383#issuecomment-2422455154
You are receiving this because you authored the thread.
Message ID: @.***>

I see the branch is still there at https://github.com/tdb30/vpxtool/tree/gui-wip, mind re-creating the PR?
If you allow me to modify the pr I can fix the build.


autoplay=true/false > change browsertab, responsive behaviour

hi there...
there is a problem with autoplay in all slider versions in firefox, chrome and safari browsers but not in ie11
if you have responsive settings set for autoplay like:
$('.slider').slick({
    slidesToShow: 3,
    slidesToScroll: 1,
    autoplay: true,
    autoplaySpeed: 500,
    adaptiveHeight: false,
    responsive: [
        {
            breakpoint: 1024,
            settings: {
                slidesToShow: 2,
                autoplay: false,
                adaptiveHeight: true
            }
        },
        {
            breakpoint: 768,
            settings: {
                slidesToShow: 1,
                autoplay: false,
                adaptiveHeight: true
            }
        }
    ]
});

so if the browser window is <= 1024px autoplay starts working again if you change the tab to another page you have opened and go back to the tab with the slider and also with iOS 8/9 you only have to touch into the window to make it start autoplay again. Maybe it has something to do with the browserevent $(window).blur or $(window).focus wich is fired when you focus a tab again?

I have just tried that by creating this jsfiddle
Steps I took to reproduce the bug

Resize the window so that the slider match <= 1024px
Changed the tab
Returned back to the slider
Nothing happened. I tried to go beyond 1024 so that autoplay can start then go back again but nothing happened. I was testing using Chrome. Can you please specify a scenario where that issue happen? Or perhapse post a jsfiddle or URL to where you encountered that issue?

Hi guys, just to contribute to document this bug, I'm having the issue in a current project with a slider with auto play settings, I was able to confirm this behavior on Chrome, Firefox.

The problem is here, there is no check if the slider is paused to set auto play again
   Slick.prototype.visibility = function() {

        var _ = this;

        if (document[_.hidden]) {
            _.paused = true;
            _.autoPlayClear();
        } else {
            if (_.options.autoplay === true) {
                _.paused = false;
                _.autoPlay();
            }
        }

    };

Maybe the simpler solution would be avoid to pause when the screen change

should be fixed in the revamp just applied to master , please hang tight for a release :)


Please update the feature flags for 3.13.0

Is your feature request related to a problem? Please describe.
We updated to rabbitmq 3.13 and the documentation looked like there are no new feature flags to enable.
Turns out there are, but they are missing from the documentation.
Describe the solution you'd like
The following flags need to be added afaics:

detailed_queues_endpoint
khepri_db
message_containers
stream_update_config_command

Also usually it is recommended to enable all of them after the upgrade, but khepri_db seems different according to the 3.13 release announcement. That should probably highlighted in the docs as well.
Describe alternatives you've considered
No response
Additional context
No response

@bodograumann as the docs now say,
rabbitmqctl enable_feature_flag all

will only enable stable feature flags, and khepri_db is currently an experimental one.

@mohag you are welcome to contribute one. This is open source software after all.


fix(server): Use correct description for Http Header Pascal Case rule. (Fixes #906)

Fixes issue #906

Codecov Report

Merging #905 into master will not change coverage.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master     #905   +/-   ##
=========================================
  Coverage     84.56%   84.56%
  Complexity      908      908
=========================================
  Files           175      175
  Lines          2747     2747
  Branches        438      438
=========================================
  Hits           2323     2323
  Misses          203      203
  Partials        221      221

Impacted Files
Coverage Δ
Complexity Δ

...do/zally/rule/zalando/PascalCaseHttpHeadersRule.kt
100% <100%> (ø)
3 <0> (ø)
:arrow_down:

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 23114f9...3137316. Read the comment docs.


Oluttamo isn't working properly in https-pages

We can't load iframe via http if current site is using https and ratebeer.com doesn't support https at the moment.
This is #wontfix until for example this is implemented: https://github.com/kehittamo/oluttamo-api/issues/2 or ratebeer implements https support.

Actually it was possible to use https in ratebeer.com, we just need to use https://www.ratebeer.com/beer/CUSTOMPREFIX/BEERID
Fixed: https://github.com/kehittamo/oluttamo-browser-extension/commit/a00599a826a8bc0ff73d12c1cd156690fb7a6d08