Trailing Whitespace causes format (-F) to fail the first time

I have a file here that when I run ktlint -F on it, it fails with messages like:
ktlint:
   [ktlint] /windows/Users/jimne/Desktop/OtherStuff/tensorflow_java/tensorflow-core-kotlin/tensorflow-core-kotlin-api/src/gen/annotations/org/tensorflow/op/kotlin/IoOps.kt:1588:33: Trailing space(s) (no-trailing-spaces)

If you run ktlint without the -F afterwards, you'll see the reported no-trailing-spaces again.
However, if you run with -F a second time, they will be fixed without issue.
(There were warning messages from JDK encapsulation, but they were derailing the issue)

Seems it is related to this Kotlin issue: https://youtrack.jetbrains.com/issue/KT-44624

That's not actually causing the crash though, it's just warnings.

This is raising an error with Java 16.

@Moisan you are right, this warning actually transformed to an error with Java 16:
How come that you assume its related to homebrew?
ktlint -F
Exception in thread "main" java.util.concurrent.ExecutionException: java.lang.ExceptionInInitializerError
	at java.base/java.util.concurrent.FutureTask.report(FutureTask.java:122)
	at java.base/java.util.concurrent.FutureTask.get(FutureTask.java:191)
	at com.pinterest.ktlint.KtlintCommandLine.parallel(Main.kt:575)
	at com.pinterest.ktlint.KtlintCommandLine.parallel$default(Main.kt:547)
	at com.pinterest.ktlint.KtlintCommandLine.lintFiles(Main.kt:298)
	at com.pinterest.ktlint.KtlintCommandLine.run(Main.kt:262)
	at com.pinterest.ktlint.Main.main(Main.kt:70)
Caused by: java.lang.ExceptionInInitializerError
	at org.jetbrains.kotlin.com.intellij.pom.java.LanguageLevel.<clinit>(LanguageLevel.java:25)
	at org.jetbrains.kotlin.com.intellij.core.CoreLanguageLevelProjectExtension.<init>(CoreLanguageLevelProjectExtension.java:26)
	at org.jetbrains.kotlin.com.intellij.core.JavaCoreProjectEnvironment.<init>(JavaCoreProjectEnvironment.java:42)
	at org.jetbrains.kotlin.cli.jvm.compiler.KotlinCoreProjectEnvironment.<init>(KotlinCoreProjectEnvironment.kt:26)
	at org.jetbrains.kotlin.cli.jvm.compiler.KotlinCoreEnvironment$ProjectEnvironment.<init>(KotlinCoreEnvironment.kt:121)
	at org.jetbrains.kotlin.cli.jvm.compiler.KotlinCoreEnvironment$Companion.createForProduction(KotlinCoreEnvironment.kt:425)
	at com.pinterest.ktlint.core.internal.KotlinPsiFileFactory.initializePsiFileFactory(KotlinPsiFileFactory.kt:74)
	at com.pinterest.ktlint.core.internal.KotlinPsiFileFactory.acquirePsiFileFactory$ktlint_core(KotlinPsiFileFactory.kt:39)
	at com.pinterest.ktlint.core.KtLint.format(KtLint.kt:378)
	at com.pinterest.ktlint.internal.FileUtilsKt.formatFile(FileUtils.kt:182)
	at com.pinterest.ktlint.KtlintCommandLine.process(Main.kt:366)
	at com.pinterest.ktlint.KtlintCommandLine.access$process(Main.kt:89)
	at com.pinterest.ktlint.KtlintCommandLine$lintFiles$3$1.call(Main.kt:289)
	at com.pinterest.ktlint.KtlintCommandLine$lintFiles$3$1.call(Main.kt:288)
	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1130)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:630)
	at java.base/java.lang.Thread.run(Thread.java:831)
Caused by: java.lang.reflect.InaccessibleObjectException: Unable to make protected void java.util.ResourceBundle.setParent(java.util.ResourceBundle) accessible: module java.base does not "opens java.util" to unnamed module @76e041cf
	at java.base/java.lang.reflect.AccessibleObject.checkCanSetAccessible(AccessibleObject.java:357)
	at java.base/java.lang.reflect.AccessibleObject.checkCanSetAccessible(AccessibleObject.java:297)
	at java.base/java.lang.reflect.Method.checkCanSetAccessible(Method.java:199)
	at java.base/java.lang.reflect.Method.setAccessible(Method.java:193)
	at org.jetbrains.kotlin.com.intellij.util.ReflectionUtil.makeAccessible(ReflectionUtil.java:252)
	at org.jetbrains.kotlin.com.intellij.util.ReflectionUtil.getDeclaredMethod(ReflectionUtil.java:269)
	at org.jetbrains.kotlin.com.intellij.DynamicBundle.<clinit>(DynamicBundle.java:22)
	... 18 more

This is a known issue caused by JDK 16 encapsulating it's internals and modules more strongly, it should be fixed with 1.5 (well, once ktlint uses 1.5).  It's entirely separate from the issue I originally raised though (at least, I'm 99% sure it is).

The problem is caused in the snippet below:
    @JvmName("decodeRawReified")
    public inline fun <reified T : TType> decodeRaw(bytes: Operand<TString>, littleEndian: Boolean?
            = null): DecodeRaw<T> = decodeRaw<T>(bytes, T::class.java, littleEndian)

The first run of ktlint --format --verbose --relative --experimental results in output:
src/main/kotlin/Issue1116.kt:3:11: Trailing space(s) (no-trailing-spaces)

and the result files (of course without the added comment) becomes:
@JvmName("decodeRawReified")
public inline fun <reified T : TType> decodeRaw(
    bytes: Operand<TString>,
    littleEndian: Boolean? =    // <-- Note that here are still some trailing spaces
        null
): DecodeRaw<T> = decodeRaw<T>(bytes, T::class.java, littleEndian)

The second run of ktlint --format --verbose --relative --experimental results in output:
@JvmName("decodeRawReified")
public inline fun <reified T : TType> decodeRaw(
    bytes: Operand<TString>,
    littleEndian: Boolean? = // <-- Note that the trailing spaces are now gone
        null
): DecodeRaw<T> = decodeRaw<T>(bytes, T::class.java, littleEndian)

When running ktlint --verbose --relative --experimental on the first example of this comment, the output is as follows:
src/main/kotlin/Issue1116.kt:2:49: Parameter should be on a separate line (unless all parameters can fit a single line) (parameter-list-wrapping)
src/main/kotlin/Issue1116.kt:2:49: Missing newline after "(" (indent)
src/main/kotlin/Issue1116.kt:2:73: Missing newline after "," (indent)
src/main/kotlin/Issue1116.kt:2:74: Parameter should be on a separate line (unless all parameters can fit a single line) (parameter-list-wrapping)
src/main/kotlin/Issue1116.kt:3:1: Unexpected indentation (6) (should be 4) (indent)
src/main/kotlin/Issue1116.kt:3:7: Line break before assignment is not allowed (no-line-break-before-assignment)
src/main/kotlin/Issue1116.kt:3:12: Missing newline before ")" (indent)
src/main/kotlin/Issue1116.kt:3:13: Missing newline before ")" (parameter-list-wrapping)

it is noteworthy that the trailing spaces are not reported above. But we can see that at line 3, several other rules report a problem. I expect that the parameter-list-wrapping rule is responsible for formatting the second code sample in this comment.
Running ktlint --format --verbose --relative --experimental --disabled_rules=parameter-list-wrapping on the first example of this comment again, the output is as follows:
@JvmName("decodeRawReified")
public inline fun <reified T : TType> decodeRaw(
    bytes: Operand<TString>,
    littleEndian: Boolean? = // No trailing spaces found at the *first run*
        null
): DecodeRaw<T> = decodeRaw<T>(bytes, T::class.java, littleEndian)

So the problem should be solved in the parameter-list-wrapping. When inserting a newline, it should be more careful not to insert trailing spaces as well.

This problem i(at least with the snippet from my analysis of December 2021) is not reproducable on versions 0.45.x and 0.46.x.


Curseforge has an issue with your choice of licensing

I am unable to use your mod in a public modpack for curseforge.
If this is intended, perfect. Close the ticket.
https://gyazo.com/12cf53c74ad9efa98c8143451f1887eb

Interesting.  I would like my license to be very open, and allow use in modpacks, without explicit permission.  It's currently using the license provided by the fabric mod example project, which is CC0.  I think this is an odd choice, since a mod or more of a software product than a work of art?  Not sure, I'm not super versed on my licenses.
Do you have a suggestion on the license?  My first instinct is GPLv3, but I'm not entirely certain what constraints need to be met for the curseforge mods.

Ah, so I never explicitly declared this in curseforge, so it defaulted to ARR.  I've changed the curseforge page to be GLPv3.  Let me know if that works now.  Thanks for using my mod in a modpack :)

I think I've resolved the licensing issue on curseforge, so I'll close this ticket.  Feel free to reopen if the problem still persists.


Show object diff when chai deepEqual fails - feature request

When an assertion using deep.equals fails, it would be nice to get a detailed output that shows
piece by piece what is different between the expected and actual values. Something like:

Sure:
https://github.com/chaijs/deep-eql
karma-mocha-reporter has this feature implemented:
https://github.com/litixsoft/karma-mocha-reporter/blob/master/index.js#L373
so it may be good inspiration point.

Would you be willing to put a PR up? There is a little bit of work since we enforce TDD. Thanks for the help, I haven't had as much time to contribute recently. I'm still surprised with the amount of people using this since Webpack is the "future".

Closing as this is old and only one person has commented. Will re-open if someone else requests it or if they have a PR ready to go


Getting permissions problems in creating dirs at the end of the docker file

Describe the bug
Permission denied creating directories
pimush    | cp: cannot create regular file '/mush/game/./CONNLOG.md': Permission denied
pimush    | cp: cannot create regular file '/mush/game/./README': Permission denied
pimush    | cp: cannot create regular file '/mush/game/./access.README': Permission denied
pimush    | cp: cannot create regular file '/mush/game/./alias.cnf': Permission denied
pimush    | cp: cannot create regular file '/mush/game/./aliascnf.dst': Permission denied
pimush    | cp: cannot create directory '/mush/game/./data': Permission denied
pimush    | cp: cannot create regular file '/mush/game/./getdate.README': Permission denied
pimush    | cp: cannot create regular file '/mush/game/./getdate.template': Permission denied
pimush    | cp: cannot create symbolic link '/mush/game/./info_slave': Permission denied
pimush    | cp: cannot create directory '/mush/game/./log': Permission denied
pimush    | cp: cannot create regular file '/mush/game/./mush.cnf': Permission denied
pimush    | cp: cannot create regular file '/mush/game/./mushcnf.dst': Permission denied
pimush    | cp: cannot create regular file '/mush/game/./names.cnf': Permission denied
pimush    | cp: cannot create regular file '/mush/game/./namescnf.dst': Permission denied
pimush    | cp: cannot create symbolic link '/mush/game/./netmush': Permission denied
pimush    | cp: cannot create regular file '/mush/game/./restart': Permission denied
pimush    | cp: cannot create regular file '/mush/game/./restart.dst': Permission denied
pimush    | cp: cannot create regular file '/mush/game/./restrict.cnf': Permission denied
pimush    | cp: cannot create regular file '/mush/game/./restrictcnf.dst': Permission denied
pimush    | cp: cannot create directory '/mush/game/./save': Permission denied
pimush    | cp: cannot create symbolic link '/mush/game/./ssl_slave': Permission denied
pimush    | cp: cannot create directory '/mush/game/./txt': Permission denied
pimush    | cp: preserving times for '/mush/game/.': Operation not permitted
pimush exited with code 1

To Reproduce...
docker-compose.yaml
services:
  pimush:
    build: .
    container_name: pimush
    ports:
      - 4201:4201
    volumes:
      - ./game:/mush/game

Steps to reproduce the behavior.
docker-compose up
Expected behavior
Launches docker container successfully

This sounds like the same problem that was reported in #2. Can you check there and see if that helps?

Also, I don’t think you can specify relative paths for volumes, unless this has changed in recent versions of Docker, so you might need to use something like $PWD/game:/mush/game

The volume paths is working ok. It seems that the issue is that the git clone and related commands are being run as su, so when the mush user tries to create files and folders it raises the permission denied error.
Also (and maybe worth another ticket) the git checkout line is not needed any more since there was a merge recently that changes the pcre2 url to the correct one.

Thanks for identifying the permissions issue. That's something I can fix.
The PCRE change doesn't need a separate ticket (unless you mean a pull request, in which case, feel free to submit one 😀).

I'm not much of a git hero. I've created a local branch, made my change and am trying to push my change but am getting 403. Do I need to be added to the project as a contributor?
Or is a PR something different?

You'll need to "fork" the repository first. Check out this guide and see if it helps. https://docs.github.com/en/get-started/quickstart/contributing-to-projects

A more detailed overview of contributing is available here, but the link in my last post should be enough to get you started. 🙂 https://opensource.guide/how-to-contribute/#how-to-submit-a-contribution

That was very helpful. I have learned something which is always nice. I have submitted the PR as suggested / requested.
:-)

Thanks! I'll try to merge it soon. I'm waiting for Docker Hub to resolve an issue with my account. 😄


Feature | PEO-837 | Add the ability to generate xlsx outputter

https://garage.atlassian.net/browse/PEO-837
Description
Replace this text with a summary of the changes in your merge request
Before submitting the merge request make sure the following are checked

[x] Followed the style guidelines of this project
[x] Performed a self-review of own code
[x] Wrote the tests that prove that fix is effective/that feature works

@za4emnik also, you need to update option class with the correspondent info


Add/link, remove/unlink in bulk with WeakMaps

Or arrays of arrays as well. This feature needs to be thought out carefully.

Won't do. The interface will be kept simple.


Some styling and grammar fixes (eu)

Hi there! I bring some fixes to the Basque translation. Kind regards.

@a-mento Thank you!


No results when filtering polling stations places should show an error message that makes it clear you've got filters applied

Should it say how many are in the unfiltered set?

Likewise for GPS search


Upadte README: login is no longer supported

Hi!
The section method reference contains login as ConanMultiPackager method, however it was moved to AuthManager.
We have two options, or re-add the login method again, or describe how to login using AuthManager.
/cc @spiderkeys
Regards!

@lasote Do you think we should introduce the login method again?
I think yes, it's better for users, where it does not require internal elements from CPT, like accessing self.auth_manager from ConanMultiPackager

If someone requests it, yes, it can be propagated to the main class.

Yes, @spiderkeys wishes the old login method. He needs to login on private remotes to be able to download, it sounds legitimate.

ok


Bump nginx version to 1.6.2

Also,

Update cardboard to 2.1.0.
Update tests to new contain_class rspec syntax.
Update rvm version to 2.1.5.

/cc @mfischer-zd @sandlerr

What do I have to do with this? :)

:+1:

:+1: please also make a pr to the boxen repo

@grosser https://github.com/boxen/puppet-nginx/pull/41


release output device when not playing

grab output device only when active and doing playback
allows to coexist with other players like RuneAudio

Great work. Can confirm that it's working flawless until now.

I can confirm it works flawlessly with the "card" tag changed to "device" for my softvol as well

sadly at the end of a playback queue no event is triggered. So there's also the device not released.
I've opened an issue #20.


/bin/sh: /etc/bash_completion.d/helm.sh: No such file or directory

BUG REPORT:
Environment:

Cloud provider or hardware configuration: baremetal

OS: Container Linux by CoreOS

Output of ansible run:
TASK [kubernetes-apps/helm : Helm | Set up bash completion] ********************
Thursday 09 November 2017  23:07:52 +0000 (0:00:01.283)       0:12:36.098 *****
fatal: [k8s-node0.xx]: FAILED! => {"changed": true, "cmd": "umask 022 && /opt/bin/helm completion bas
h >/etc/bash_completion.d/helm.sh", "delta": "0:00:00.004624", "end": "2017-11-09 23:07:52.838300", "failed": true, "rc": 1, "start":
"2017-11-09 23:07:52.833676", "stderr": "/bin/sh: /etc/bash_completion.d/helm.sh: No such file or directory", "stderr_lines": ["/bin/sh: /etc/bash_completion.d/helm.sh: No such file or directory"], "stdout": "", "stdout_lines": []}

The condition is
  when: (helm_container is defined and helm_container.changed) or (helm_task_result is defined and helm_task_result.changed) and not ansible_os_family in ["CoreOS", "Container Linux by CoreOS"]

Your ansible_os_family set to container linux by coreos should prevent this from running.


What type of project file is Kernel in this Solution?

Hi, I can see its a new logo which kernel have in this solution, can anyone explain what type of project it is? Do microsoft provide any template for this? I have never seen this with C# and .Net.

Its shared proj file https://learn.microsoft.com/en-us/xamarin/cross-platform/app-fundamentals/shared-projects?tabs=windows


BibEdit: import data with crossref

I try to import data from crossref  but always i've got  this error "This is not a correct DOI, please correct it "
Note : by entring this URL in the Browser http://www.crossref.org/openurl/?pid=USERNAME&noredirect=tru&id=doi:10.1021/acsnano.5b02466
i've got an XML record
i'm using Invenio v1.2.1

Dear @chokribr what exactly are you typing as string in BibEdit? Note that the field accept only a plain doi such as: "10.1021/acsnano.5b02466", without further prefix. If e.g. I type "doi:10.1021/acsnano.5b02466" then I obtain exactly your same error.

@kaplun @ludmilamarian

@kaplun @ludmilamarian @jirikuncar i Think that the problem with password i'm using crossref openurl without password by testing the existance ":" + CFG_CROSSREF_PASSWORD variable and adding ":"  i've got the record https://github.com/inveniosoftware/invenio/blob/legacy/modules/miscutil/lib/crossrefutils.py#L69

@chokribr you mean without password it works? Could then be a problem of encoding the URL of the request BibEdit does to crossref...

@chokribr can you check if the patch in #3516 solves your issue?

@kaplun the problem is with ":"
#3516 http://www.doi.org/factsheets/DOIProxy.html

Good point

@kaplun i try this https://github.com/chokribr/invenio/commit/45f78bb6de207712a2ecdaae0f0505bddfa69145
and it work

@chokribr I think we should merge our two solutions, because both proper encoding of the request and support for accounts without password are both needed.

many thanks sam @kaplun congrat  it's a pleasure to find solution with you


--Login Page Added.

--Code used from tutorial, link is in Login.tsx

This is what it looks like:


[feature request] Add 'params' prop to the 'A' component

Describe the bug
Not a bug, it's a feature request.
I think that it would be nice to add a params prop to the A component, for example:

I'm proposing this for 2 reasons:

It seems like a nicer and more maintainable API when working with an object containing all paths. For example:

This would be a nice combination with named routes proposed here https://github.com/solidjs/solid-router/issues/61
This is the API I'm thinking about:

When working with named routes there must be some way of providing params.
An alternative API could be

Your Example Website or App

Steps to Reproduce the Bug or Issue

Expected behavior

Screenshots or Videos
No response
Platform

Additional context
No response

Interesting take. I was working on named routes implementation these days and was looking for a interpolable API.
The alternative API is exactly what I came up as well. href could be responsible for determining the URL based on path string or RouteQuery object.
the object is a "find a match" description. It could precise name, path and/or params, making it very flexible, as well as it allows to easily pass a link reference without API split.
So I am +1 on thi solution :)

What I like about having a separate params prop is that it would be really easy to switch between a simple route, for example

to

and vice versa.
It's also a bit cleaner imo.
But it doesn't matter to me that much.
As for the buildUrl function, I think that it might be a good idea to add this function regardless.
And we already have a function to detect whether a path is active or not, useMatch, unless I'm misunderstanding you.
Also in other issues I proposed adding an options object as the second argument to useMatch, so maybe we could have it take params as well:
useMatch('/users/:id', { params: { id: userId }})

Yes, that's exactly what we need. The same way, we could have buildUrl working: buildUrl('users/:id', { parsams: { id: 2 } })
Having this API would even allow to build named routes on top.
Im wondering if it would be possible to use those functions without a need for context.

Why not buildUrl('users/:id', { id: 2 })?
Is there anything else that you can see going there other than params?
I mean making it an options object is more future proof but I don't really see what options we could potentially add.

Good question. Yes, the route can be described as well by QueryParams which are separate from RouteParams. There might be also a need for other route's descriptors, like history state.
ideally I would love to see it as a self contained object that has all the information to identify a route. Similarly like you send an API request to a search endpoint, or do DB query.
That way you will be able to specify wide or exact, route you need.
Putting the path into this object would be also nice, as that would ease the named routes integration as well.

That's a good point.
Btw apparently tanstack router also has a params prop on its <Link> component.


Is there any zoomAndPan method?

Describe the bug
Is there any zoomAndPan method? Because focal option not working and zoom after pan looking glitchy.
Your environment
panzoom: 4.1.0
Mozilla Firefox 75 (64 Bit)
Expected behavior
When i use panzoom.zoom(6, {focal: {x: 0, y:0}}), zoom center instead of my point.
Steps to reproduce
this template

Thanks for opening an issue. I'm not sure what would be causing the focal option to be glitchy. The template you linked looks like the default template. Perhaps you just needed to click clone or save a snapshot after you made changes?
Another option is to use both the pan and zoom methods.
Can reopen with an updated test case.

Sorry, I don't understand how to use jsbin. So I wrote the example here.
I am trying to zoom in to the point I want. I add focal to the zoom function, but it does not zoom to the point I want.
When the "contain" setting is "outside", the "zoom" and "pan" (with animate) methods zoom into the wrong spot when called consecutively.
The "pan" method looks glitchy when delayed by "setTimeOut".


fix: Tagline url opening inconsistency

What
Change how tagline url is opened. I presume we should use LaunchUrlHelper to open urls, like it's done on other places in the app.
Screenshot

Fixes bug(s)
Fixes #4667

Codecov Report

Merging #4683 (36fc9d7) into develop (6820165) will increase coverage by 0.00%.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           develop   #4683   +/-   ##
=======================================
  Coverage     9.91%   9.91%
=======================================
  Files          310     310
  Lines        15795   15794    -1
=======================================
  Hits          1566    1566
+ Misses       14229   14228    -1

Files
Coverage Δ

...mooth_app/lib/widgets/smooth_product_carousel.dart
1.37% <0.00%> (+<0.01%)
:arrow_up:

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


improve layout / design

Layout/Design of the whole App or just certain components?

Layout/Design of the whole App or just certain components?

just meeting info page


Fix for issue #24 - xDscWebService: PSDSCComplianceServer returns HTTP Error 401.2

Added check for IIS Windows authentication in Test-TargetResource and added it if needed in Set-TargetResource. Coded to work in 2008 R2 and newer but only tested in 2012 R2.

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

Closes #24


How do I add shadows to a window when its undecorated attribute was set to true?

add a simple text in the middle of a box that fillMaxSize

add a simple text in the middle of a box that fillMaxSize

@Adiudiu please provide more context:

What is the expected result?
What is the actual result?
Your environment: kotlin version, compose version, jdk version, operating system name and its version?
Add relevant screenshots & reproduction sample if possible.

Maybe I can add some more details since I'm facing the same issue.
Expected Result
My use case is that I would like to build a Spotlight/Alfred/Raycast style input window. These are all undecorated but have an (OS made) DropShadow on MacOS. This behaviour would be the expected result.
Actual Result
I can build an undecorated window with transparent background that behaves in a similar way but it doesn't have a shadow which is a bit unusual on MacOS.

Environment
I am working on Kotlin 1.9.0, compose version 1.4.3, jdk version <IP_ADDRESS>, MacOS Ventura 13.5
Additional Info
It is actually not trivial to produce this behaviour with AppKit on MacOS. It can be achieved by leaving decorations on but then making the titlebar transparent, making the space of the titlebar space usable by the app and hiding the close, minimize and maximize buttons. With that, it looks something like this:

It would be great to have some more access to the detailed window behaviour in each operating system but I am not sure how and if this is achievable.

@Raphael-N I provided a solution for native shadows and animations for Windows:

https://github.com/JetBrains/compose-multiplatform/issues/3388#issuecomment-1672732504

Maybe it will be handy for you as well.

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


[CELEBORN-1754][CIP-14] Add exceptions and checking utils to cppClient

What changes were proposed in this pull request?
This PR adds exceptions and checking utils code to CppClient.
Besides, the ctest framework is added to CppClient for UTs.
Why are the changes needed?
To provide exception utils and UT frmework to CppClient.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Compilation and UTs.

@FMX Could you kindly help review this PR? Thanks a lot.
Besides the exceptions and checking logic, the UT framework is supported now, so the uts are added.
Also note that the prebuild dev image is updated to holylow/celeborn-cpp-dev:0.2 to add gtest module dependency.
Any suggestion is welcome.

Copy that. I'll review this PR within this week.

@HolyLow I found out that I can run the UTs successfully.
If I run the UTs in Clion, I got following error.
1: Test command: /tmp/cpp/cmake-build-debug/celeborn/utils/tests/celeborn_utils_test
1: Working Directory: /tmp/cpp/cmake-build-debug/celeborn/utils/tests
1: Test timeout computed to be: 1500
Exception: Illegal

If I run the UTs in Docker container, I got following errors.
root@82f7ea7a535c:/celeborn/cpp/build# ctest
Test project /celeborn/cpp/build
    Start 1: celeborn_utils_test
1/1 Test #1: celeborn_utils_test ..............***Exception: Illegal  0.02 sec
0% tests passed, 1 tests failed out of 1

Total Test time (real) =   0.04 sec

The following tests FAILED:
	  1 - celeborn_utils_test (ILLEGAL)
Errors while running CTest
Output from these tests are in: /celeborn/cpp/build/Testing/Temporary/LastTest.log
Use "--rerun-failed --output-on-failure" to re-run the failed cases verbosely.
root@82f7ea7a535c:/celeborn/cpp/build# cat /celeborn/cpp/build/Testing/Temporary/LastTest.log
Start testing: Dec 04 02:55 UTC
----------------------------------------------------------
1/1 Testing: celeborn_utils_test
1/1 Test: celeborn_utils_test
Command: "/celeborn/cpp/build/celeborn/utils/tests/celeborn_utils_test"
Directory: /celeborn/cpp/build/celeborn/utils/tests
"celeborn_utils_test" start time: Dec 04 02:55 UTC
Output:
----------------------------------------------------------
<end of output>
Test time =   0.02 sec
----------------------------------------------------------
Test Failed.
"celeborn_utils_test" end time: Dec 04 02:55 UTC
"celeborn_utils_test" time elapsed: 00:00:00
----------------------------------------------------------

End testing: Dec 04 02:55 UTC

Looks like there is something wrong with the UT, I'll merge this PR after it is fixed.

After some investigation, I found the errors that occurred were related to the difference between the arm64 and the x86 platforms. I'll merge this PR. Docker saved the day.


Add placement group control to the pool resource.

This allows pg_num to be controlled idempotently as part of the pool resource.  It will set pg_num and pgp_num if the pg_num property changes on the pool resource.

Really nice :)


Do not put these scripts in any folder.

https://github.com/Kiteretsu77/FAST_Anime_VSR/tree/main/Installation_script
@Kiteretsu77
Why can user @HikariDawn777 make changes, but without Verified "This commit was created on GitHub.com and signed with GitHub’s verified signature." and without a pull request?

This is my second account. I push with a mix.

This is my second account. I push with a mix.


Adopt FoundationEssentials instead of Foundation where available

Use FoundationEssentials if available.

Checklist

[x] I've run tests to see all new and existing tests pass (macOS and linux)
[x] I've followed the code style of the rest of the project
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary

If you've made changes to gyb files

[ ] I've run .script/generate_boilerplate_files_with_gyb and included updated generated files in a commit of this pull request

bash .script/generate_boilerplate_files_with_gyb
bash: .script/generate_boilerplate_files_with_gyb: No such file or directory

Motivation:
Would be great that packages that depend on swift-crypto can be built without linking full Foundation.
Modifications:
This patch simply replaces import Foundation with a conditional import FoundationEssentials.
In some files this also required to explicitly import the C standard lib.
Maybe we could follow the same approach as in https://github.com/swiftlang/swift-tools-support-core/blob/main/Sources/TSCLibc/libc.swift and add an internal module that abstracts this import dance in a single place.
Result:
After this change the package should only depend on FoundationEssentials.

Thanks for this! This is a worthwhile change but we have to be careful with merging it, as it requires a semver major due to Swift’s “leaky” imports.

Thanks for this! This is a worthwhile change but we have to be careful with merging it, as it requires a semver major due to Swift’s “leaky” imports.

hm, not sure I follow. Are you saying, import Foundation makes the public Foundation API available to clients? Isn't this what @_exported import is for?

It doesn’t make the full API available, but it makes some things available. An example of one such problem is available at https://forums.swift.org/t/pitch-fixing-member-import-visibility/71432.
The Swift team consider this a bug, and I agree, but our semver policies take the language as it is, not as it is intended to be. So this will require us to burn a major.
The good news is that a) this will be a “cheap” major, as most codebases won’t be affected and so they can use a both-major range, and b) we have to do one anyway. So we’ll definitely make this change, I just wanted to clarify that it won’t be immediately merged.


[BEAM-10956] Fixes incorrect API usage in PipelineInstrumentTest causing flakiness

Change-Id: Ied7a07ce16ebb1e155692666335a6dae498a01ea
Test was using the mark_pcollection_computed method incorrectly.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Verified fix with for i in {1..100}; do python pipeline_instrument_test.py & done

thanks Sam! (LGTM fwiw)


Update Event

How to update event attributes (color, title...) and render in the calendar?

item in your example  is an array, are you sure that will work !!

Hi, you can use eventsModel attribute to pass events to calendar.
<ng-fullcalendar
    [options]="calendarOptions"
    [eventsModel]="calendarEvents"
    (clickButton)="onClickButton($event)"></ng-fullcalendar>


Documentation showing old version

Which package(s) are showing incorrect data?

https://swiftpackageindex.com/pointfreeco/swift-composable-architecture

Please explain what data is incorrect and what you expected to see
I noticed that TCA's docs nicely include the 0.x and 1.x series! Hope this is intentional, as it's a nice way of keeping old docs around :)
One thing I noticed, though, is that the 0.x docs are pointing to 0.57.0, but the latest tag for 0.x is 0.59.0.

Hi @stephencelis , that is indeed intentional! We don't delete generated docs and even though we only show the latest set per major release in the picker, they're still there to be used as a link target.
I've just looked through our storage bucket and compared the folders to the tags. Here's what we have (from 0.36.0 onwards, which is when we started generating) and what's missing:
0.36.0
0.37.0
0.38.0
0.38.1
0.38.2
0.38.3
0.39.0
0.39.1
0.4.0
0.40.0
0.40.1
0.40.2
0.41.0 <-- missing
0.41.1 <-- missing
0.41.2
0.42.0
0.43.0
0.44.0
0.44.1
0.45.0
0.46.0
0.47.0
0.47.1
0.47.2
0.48.0
0.48.1
0.49.0
0.49.1
0.49.2
0.50.0
0.50.1
0.50.2
0.50.3
0.51.0
0.52.0
0.53.0
0.53.1
0.53.2
0.54.0
0.54.1
0.55.0
0.55.1
0.56.0
0.57.0
0.58.0 <-- missing
0.58.1 <-- missing
0.58.2 <-- missing
0.59.0 <-- missing
1.0.0
1.1.0
1.2.0

It's hard to tell in hindsight why these weren't generated - or uploaded - correctly. There are a number of things that could be at play here 🤔
I should be able to re-process these for you. Is it just 0.59.0 you're interested in as the latest 0.x?
Once I get it to process it's probably no big deal to generate them all but I'm asking in case there are issues to which lengths I need to go to massage this for all or just the one ;)

@finestructure I think they might be missing because 0.57.0 was the last release before 1.0.0. 0.58.0 through 0.59.0 have been maintenance releases to better help folks migrate to 1.0.0. Would love if you could kick off the latest 0.59.0! I wonder if more needs to be done to support the use case of continual updates to older major versions?

Right, that's exactly what happened here. When we're looking for missing builds, we only check the "latest" triple: the latest default branch, release, and pre-release versions.
Since at the time you tagged 0.58.0 there was already a "latest" release in 1.0.0, we didn't trigger builds for 0.58.0.
We restrict ourselves to the "latest" triple to keep a lid on how many versions we need to look at per package, as there are some with hundreds of tags. We couldn't want to try and build all of them :)
Ideally, we'd have something like "latest release per major version", at least for doc building purposes. I'll need to have think about that, it's not going to be a trivial thing to change 🤔

I noticed that TCA's docs nicely include the 0.x and 1.x series! Hope this is intentional, as it's a nice way of keeping old docs around :)

One of those features we knew would take a while for people to notice :)

Just to keep you updated, I'm actively looking into rebuilding the docs for these versions but it's a bit more complicated than I was hoping.

Reopening until I've actually re-generated the docs on prod

I've regenerated all the missing doc sets and they're now available via the documentation link.
Unfortunately, we don't have a process in place to do this automatically for older major release tracks. So if you we to release a 0.59.1 in the future, I'm afraid you're going to have to ping me to trigger a rebuild. It's a simple command an easy enough to do on a case by case basis, it's just not something we can easily make available or automate at the moment.


Satellite map doesn't plot anything

Using latitude and longitude data I am able to plot using Scatter or Atlas Map, however the type Satellite Map doesn't plot anything at all. I am using the version 2.8.0 on a MacOS 10.12.6.

To use Satellite Maps in the chart editor, Falcon requires Mapbox access tokens.
We haven't implemented yet any means to provide these tokens to Falcon.

@n-riesco we should disable those trace-types in the embedded editor then until this is supported.

@nicolaskruchten Is there an easy way to do that?

https://github.com/plotly/react-chart-editor/blob/master/dev/App.js#L44

@nicolaskruchten Falcon is using <PlotlyEditor> at the moment.

@n-riesco thanks for letting me know. I was thinking I missed something.

On further research, the idea was that if no mapbox token is provided to the editor, those trace-type options should automatically hide...

Perhaps this is just because we're not using the latest version of the editor here. We should definitely upgrade the editor in the next week :)

Resolved by bumping react-chart-editor to v0.21.1.


Lau 1045 loop through root cases

JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/LAU-1045
Change description
Looping through root cases and build family
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x] No

not necessary


Setup Slack Integration

Instructions: https://www.gitbook.com/blog/features/slack

The docs repository is already part of the Slack activity feed. I think we can close this but feel free to reopen if you think we should still set this up.


Emoticon-picker plugin

Hello!
My team needs an emoticon-picker so we are going to create a plugin.

First of all, I'm surprised it does not exist yet. Do you confirm no such plugin exists?
I think it's a usual use case so we wonder if we should create a new project or if you would accept a pull request for a new official plugin in this repository. What do you think?
We know the version 3 is in progress and will change the way plugins are developed. But I haven't found any information about the release date of the version 3 : do you confirm that version 3 will not be available before February so we should develop this plugin for version 2 if we want it to be in production at the end of February?

@Niolak Nice to hear you are interested in making the Editor's plugin! We discussed this, but it seems difficult for version 3 to be released within the schedule you mentioned. And there is a plugin management issue, so it would be better to create a new project and manage it as you said.

if we should create a new project ...

The example below is a plugin project that someone made a while ago, and I think you can develop it like this.

https://www.npmjs.com/package/tui-font-size-picker

If you make it as a personal project, I think you can fork it from the current version 2.

@Niolak Nice to hear you are interested in making the Editor's plugin! We discussed this, but it seems difficult for version 3 to be released within the schedule you mentioned. And there is a plugin management issue, so it would be better to create a new project and manage it as you said.

if we should create a new project ...

The example below is a plugin project that someone made a while ago, and I think you can develop it like this.

https://www.npmjs.com/package/tui-font-size-picker

If you make it as a personal project, I think you can fork it from the current version 2.

Thank you for the answer! We'll do that and I'll share the plugin here after then.

Thank you for the answer! We'll do that and I'll share the plugin here after then.

I recently came across tui.editor for one of my projects. I've created a very basic emoji plugin. Might help somebody in the future.
import { Picker } from 'emoji-picker-element';

export default function editorEmojiPlugin(editor, options = {}) {
  const toolbarItems = editor.options.toolbarItems;
  let emojiIndex = toolbarItems.indexOf('emoji');

  let emojiTriggerChar = options.emojChar ? options.emojChar : ':';

  if (!editor.isViewer() && emojiIndex > -1) {
    const toolbar = editor.getUI().getToolbar();
    editor.eventManager.addEventType('emojiButtonClicked');

    const button = document.createElement('button');
    button.className = 'custom-button';
    button.innerHTML =
      '<svg xmlns="http://www.w3.org/2000/svg" width="16" height="16" viewBox="0 0 24 24"><path d="M12 2c5.514 0 10 4.486 10 10s-4.486 10-10 10-10-4.486-10-10 4.486-10 10-10zm0-2c-6.627 0-12 5.373-12 12s5.373 12 12 12 12-5.373 12-12-5.373-12-12-12zm5.507 13.941c-1.512 1.195-3.174 1.931-5.506 1.931-2.334 0-3.996-.736-5.508-1.931l-.493.493c1.127 1.72 3.2 3.566 6.001 3.566 2.8 0 4.872-1.846 5.999-3.566l-.493-.493zm-9.007-5.941c-.828 0-1.5.671-1.5 1.5s.672 1.5 1.5 1.5 1.5-.671 1.5-1.5-.672-1.5-1.5-1.5zm7 0c-.828 0-1.5.671-1.5 1.5s.672 1.5 1.5 1.5 1.5-.671 1.5-1.5-.672-1.5-1.5-1.5z"/></svg>';

    const { index = emojiIndex } = options;
    toolbar.insertItem(index, {
      type: 'button',
      options: {
        name: 'emoji',
        className: 'emoji',
        event: 'emojiButtonClicked',
        tooltip: 'Emoji',
        el: button
      }
    });

    const wrap = document.createElement('div');
    wrap.className = 'popup-emojis';

    const picker = new Picker();
    picker.addEventListener('emoji-click', (event) => {
      editor.insertText(`${event.detail.emoji.unicode}`);
    });
    picker.style = `width: 100%; height: auto;`;
    wrap.appendChild(picker);

    const popup = editor.getUI().createPopup({
      header: false,
      title: null,
      content: wrap,
      className: 'tui-editor-popup',
      target: editor.getUI().getToolbar().el
    });

    editor.eventManager.listen('focus', () => {
      popup.hide();
    });

    editor.eventManager.listen('emojiButtonClicked', () => {
      if (popup.isShow()) {
        popup.hide();

        return;
      }

      editor.eventManager.emit('closeAllPopup');
      popup.show();
    });

    editor.eventManager.listen('closeAllPopup', () => {
      popup.hide();
    });

    editor.eventManager.listen('removeEditor', () => {
      popup.remove();
    });
  }
}

We have fallen behind in our project and the emoticon-picker has been deprioritised.
Thank you mananjadhav for this code which will also be useful when we put emoticons back in our roadmap!


Fce 395 ex webrtc

I implemented most of the comment so far. I would like to implement those two suggestions in the next PR:

we should implement the same mechanism like in https://github.com/fishjam-dev/membrane_rtc_engine/pull/408 to avoid :track_metadata_update RC
I think we should (if possible) subscribe only on tracks that were accepted in negotiationswe should implement the same mechanism like in https://github.com/fishjam-dev/membrane_rtc_engine/pull/408 to avoid :track_metadata_update RC

I would like also to address somehow the issue with pipe formatting. In general, I think we should follow this guide. It's annoying to find wrong formating cases by hand, so we could use credo for this. But there are also some cases where we don't want to follow the convention, mainly when using Membrane spec pipes. What do you think?

I implemented most of the comment so far. I would like to implement those two suggestions in the next PR:

we should implement the same mechanism like in #408 to avoid :track_metadata_update RC
I think we should (if possible) subscribe only on tracks that were accepted in negotiationswe should implement the same mechanism like in #408 to avoid :track_metadata_update RC

I would like also to address somehow the issue with pipe formatting. In general, I think we should follow this guide. It's annoying to find wrong formating cases by hand, so we could use credo for this. But there are also some cases where we don't want to follow the convention, mainly when using Membrane spec pipes. What do you think?

same as above, just please add TODO comments


SR-IOV: increase the verification timeout if SR-IOV is present

Certain drivers like i40e take a long time to modify the VFs in the
kernel. Nmstate is timing out on verification because of that. In order
to fix this, nmstate is incresing the verification time if SR-IOV is
present on the desired state.
Signed-off-by: Fernando Fernandez Mancera<EMAIL_ADDRESS>

oVirt has confirmed the fix. Merging.

The CI is failed on AttributeError: 'NoneType' object has no attribute 'props'


Can't use WithArguments and FromNewComponentOnNewPrefab simultaneously

Hi,
I try to bind MemoryPool like this:
Container.BindMemoryPool<BeamObstacle, BeamObstacle.Pool>()
.WithId(beamObstacleData.Id)
.WithArguments(beamObstacleData.HitType)
.FromNewComponentOnNewPrefab(beamObstacleData.Prefab);
And the problem is that i can't use method WithArguments when i use FromNewComponentOnNewPrefab. A have error:
error CS1061: Type Zenject.ConditionCopyNonLazyBinder' does not contain a definition for FromNewComponentOnNewPrefab' and no extension method `FromNewComponentOnNewPrefab'
Without usage of WithArguments it works fine. Tell me please is it bug or feature?

Yeah, you can't use WithArguments with pools or factories currently.  I'll take a look at adding support for this in the next version.  In the meantime, maybe what you could do is have the HitType be an argument to BeamOstacle.Pool, and then create a new class that contains all the different memory pools and passes in the right hit type in the .Spawn()?

Thank you! Yes, now i have class ObstacleSpawner that contains all the different memory pools and HitType is public field of pool:
public class ObstacleSpawner
{
public ObstacleSpawner(
[Inject(Id = ZenjectIds.FireBeamPool)] BeamObstacle.Pool fireBeamObstaclePool,
[Inject(Id = ZenjectIds.FrostBeamPool)] BeamObstacle.Pool frostBeamObstaclePool)
{
fireBeamObstaclePool.HitType = PlayerHitHandler.HitType.Fire;
frostBeamObstaclePool.HitType = PlayerHitHandler.HitType.Frost;
_beamObstaclePools.Add(fireBeamObstaclePool);
_beamObstaclePools.Add(frostBeamObstaclePool);
}
}
Maybe later i will inject differrent HitTypes in ObstacleSpawner too if it is more complicated.

I also added the WithFactoryArguments so you can specify arguments for the factory/pool now in the binding.  You can also do stuff like FromIFactory(x => x.To<FooFactory>().WithArguments(...)) as well


Remove uint implementation

What was wrong?
We unnecessarily define and implement traits for U256 and U512 types.
How was it fixed?
Use U256 and U512 types from ethereum-types.
To-Do

[x] Add entry to the release notes (may forgo for trivial changes)
[x] Clean up commit history

@jacobkaufmann  Could you please rebase this on master?


Provide an Intent to the browser to focus the TWA.

Hey,
This is the long awaited counterpart to https://chromium-review.googlesource.com/c/chromium/src/+/1849682

Closed in favour of https://github.com/GoogleChrome/android-browser-helper/pull/53


Device Orientation

Hello! Is there a way to know the orientation of the device?

DeviceMotion plugin

Thanks!


Standardise files with files in sous-chefs/repo-management

Signed-off-by: Dan Webb<EMAIL_ADDRESS>

Released as: 5.1.1


Add the opt_table_font() method

This adds the opt_table_font() method, which can be used for the easy addition of fonts (or a font stack) to the table. Here's an example where a font stack was used:
from great_tables import GT, exibble

gt_ex = (
    GT(exibble.head(5), rowname_col="row", groupname_col="group")
    .tab_header("THE HEADING", "(a subtitle)")
    .tab_stubhead("THE STUBHEAD")
    .tab_source_note("THE SOURCE NOTE")
    .opt_table_font(stack="industrial")
)

gt_ex

Here's another example where a single font name is prepended to the list of default fonts:
from great_tables import GT, exibble

gt_ex = (
    GT(exibble.head(5), rowname_col="row", groupname_col="group")
    .tab_header("THE HEADING", "(a subtitle)")
    .tab_stubhead("THE STUBHEAD")
    .tab_source_note("THE SOURCE NOTE")
    .opt_table_font(font="Comic Sans MS")
)

gt_ex

Fixes: https://github.com/posit-dev/great-tables/issues/175

Sounds good. PR #260 is ready for a re-review.

I've incorporated your suggestions in two commits. If CI passes I'll merge this PR. Thanks!


Update Set-DnsServerSetting.md

This PR remove the spurious class snippet from the Inputs section

Docs Build status updates of commit 19862f5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/dnsserver/Set-DnsServerSetting.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Hello @X-Guardian
First of all, thanks for your contributions.
Let me ask you, why are you formatting the INPUTS section as source code? Also, afaik INPUTS section only specifies the allowed pipelined input that a cmdlet accepts, see some examples
https://docs.microsoft.com/en-us/powershell/module/skype/set-csuserserver?view=skype-ps#inputs
https://docs.microsoft.com/en-us/powershell/module/skype/set-cswatchernodeconfiguration?view=skype-ps#inputs


script run on linux and Mac

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

apidoc  new done,
todo release

done


timerender: Use 12 hours to switch to display yesterday.

This allows us to not directly jump to showing 2days ago from showing
timestamps in hours when now is around midnight.

As came up in the chat.zulip.org discussion, this precise change probably isn't what we want; I think we can live with what's in master or do something more complex, and in either case we can close this PR.  Thanks @amanagr for working on this!


hpx won't build on POWER9 with clang 8

Expected Behavior
Normal compilation
Actual Behavior
compile error (could be just a clang 8.0.1 problem, not POWER9 related):
/home/users/khuck/src/phylanx/tools/buildbot/src/hpx/libs/serialization/include/hpx/serialization/traits/brace_initializable_traits.hpp:48:59: error:
      no member named 'index_sequence' in namespace 'std'
        inline constexpr auto is_brace_constructible(std::index_sequence<I...>,
                                                     ~~~~~^

Steps to Reproduce the Problem
... Please be as specific as possible while describing how to reproduce your problem.

cmake -DCMAKE_CXX_COMPILER=/packages/llvm/8.0.1/bin/clang++ -DCMAKE_C_COMPILER=/packages/llvm/8.0.1/bin/clang -DCMAKE_BUILD_TYPE=Release -DBOOST_ROOT=/home/users/khuck/src/phylanx/tools/buildbot/build-cyclops-ppc64le-Linux-clang/boost-1.65.1 -DHPX_WITH_MALLOC=system -DHWLOC_ROOT= -DCMAKE_INSTALL_PREFIX=. -DHPX_WITH_THREAD_IDLE_RATES=ON -DHPX_WITH_PARCELPORT_MPI=ON -DHPX_WITH_PARCEL_COALESCING=OFF -DHPX_WITH_TOOLS=OFF -DHPX_WITH_TESTS=ON -DHPX_WITH_EXAMPLES=OFF -DHPX_WITH_THREAD_MANAGER_IDLE_BACKOFF=OFF -DHPX_WITH_APEX=TRUE -DAPEX_WITH_ACTIVEHARMONY=TRUE -DACTIVEHARMONY_ROOT=/usr/local/packages/activeharmony/4.6.0-ppc64le -DAPEX_WITH_OTF2=TRUE -DOTF2_ROOT=/usr/local/packages/otf2/2.0-ppc64le -DAPEX_WITH_PAPI=TRUE -DPAPI_ROOT=/usr/local/packages/papi/5.5.1 -DHPX_WITH_APEX_NO_UPDATE=FALSE -DHPX_WITH_APEX_TAG=develop -DHWLOC_ROOT=/usr/local/packages/hwloc/1.8 -DHPX_WITH_RDTSCP=OFF -DHPX_WITH_DEPRECATION_WARNINGS=Off -DHPX_WITH_MAX_CPU_COUNT=160 /home/users/khuck/src/phylanx/tools/buildbot/src/hpx
make -j40

Specifications
... Please describe your environment

HPX Version:  current master
Platform (compiler, OS):  POWER9, Clang 8.0.1

This looks like a missing header file #include <utility> in that file. @khuck: could you try adding this here, please: https://github.com/STEllAR-GROUP/hpx/blob/master/libs/serialization/include/hpx/serialization/traits/brace_initializable_traits.hpp#L18?

That seems to have worked!  Thanks.


Update changelog

Description
This might fix #335
@Yoshanuikabundi I don't have any specific asks but I wanted to get your eyes on this, and I'm also all ears if you have other opinions on how changelogs should be maintained. My general feeling is something that is imperfect but exists is better than me bikeshedding something that never sees the light of day, so this isn't the cleanest work.
Checklist

[ ] Add tests
[ ] Lint
[ ] Update docstrings

I think this is great! I would suggest adding a "next release" section to the top of the file. New changes can be added to that section as PRs are merged, and then when a release is prepared it can be renamed and a new "next release" section added.

Could use some TLC at some point, but not bad now: https://openff-interchange--336.org.readthedocs.build/en/336/releasehistory.html


Right click on a filesystem entry contains glitch

Godot version
Godot 4 beta 1 Mono
System information
Ubuntu 22.04
Issue description
When you right click on a folder and/or file, it pops up a context menu which is perfectly normal!
Normally if you were to release the right click, you'll need to click on an item in the context menu again so that you can pick an option on the menu.
But Godot 4 Beta 1 apparently saids release the right click, and we will simulate a left click of whereever your mouse is on the context menu
Steps to reproduce
Just right click and watch as it autoclicks an item for you
Bonus: try right click and hold it and move it to an desired location on the context menu and it'll click it for you when you release the mouse
Minimal reproduction project
No response

Oh wait nevermind its my mouse thats glitchy


Loosen bounds check on CompositeByteBuf's maxNumComponents

Motivation:
In versions of Netty prior to 4.1.31.Final, a CompositeByteBuf could be
created with any size (including potentially nonsensical negative
values). This behavior changed in e7737b993, which introduced a bounds
check to only allow for a component size greater than one. This broke
some existing use cases that attempted to create a byte buf with a
single component.
Modifications:
Lower the bounds check on numComponents to include the single component
case, but still throw an exception for anything less than one.
Add unit tests for the case of numComponents being less than, equal to,
and greater than this lower bound.
Result:
Return to the behavior of 4.1.30.Final, allowing one component, but
still include an explicit check against a lower bound.
Note that while creating a CompositeByteBuf with a single component is
in some ways a contradiction of the term "composite", this patch caters
for existing uses while excluding the clearly nonsensical case of asking
for a CompositeByteBuf with zero or fewer components.
Fixes #8613.

cc: @njhill

@netty-bot test this please

@nicktrav @njhill has a good point here... maybe do this :)

Move the tests 👍 .

@netty-bot test this please

@nicktrav thanks a lot buddy!


Fixed an exception while using Postgres on 3.1.2

Got error due to Postgres requiring a call to CAST(...)
SQLSTATE[XX000]: Internal error: 7 ERROR: failed to find conversion function from unknown to text").

Hi @gplanchat,
The circumstances when the fix is needed is quite hard to understand for me.
Can you share the full query or the query builder, where the fix was required?


add new rss recipe link to community guides

Description (required)
Adding a new recipe for an RSS feed image fix where using relative imports in markdown files was not pointing to the correct build image.
Discord: billy_le

Related issues & labels (optional)

Related to #7570,
Suggested label: https://github.com/withastro/docs/labels/community resource, https://github.com/withastro/docs/labels/recipe

First-time contributor to Astro Docs? Yes

Hello! Thank you for opening your first PR to Astro’s Docs! 🎉
Here’s what will happen next:

Our GitHub bots will run to check your changes.
If they spot any broken links you will see some error messages on this PR.
Don’t hesitate to ask any questions if you’re not sure what these mean!

In a few minutes, you’ll be able to see a preview of your changes on Vercel 🥳

One or more of our maintainers will take a look and may ask you to make changes.
We try to be responsive, but don’t worry if this takes a few days.

Thanks @billy-le, can I just ask you to remove the translations and keep only English? We have a system in-place for our contributors to jump in and translate to other languages, this way we ensure native speakers are the ones adding content in other languages. 🙌

Yes :) Sure thing. Sorry bout that.


Add tests for @yamada-ui/avatar

related: #74
Description
Currently, no tests in @yamada-ui/avatar. In the future, I feel the need for testing when operating @yamada-ui/avatar.
Problem Statement/Justification
Testing is essential to ensure quality when operating a package.

I'll take it!!


❓ Add support for Haskell-style lambda when expressions?

In my own Elm code, it's incredibly common to write something like:
foo : Thing -> ...
foo thing =
    case thing of
        A -> ...
        B -> ...

Here, thing is the only argument and it's being immediately pattern matched on, usually never used again in the body of the function.
Haskell has a handy extension called LambdaCase, OCaml has function expressions to achieve the same thing. I think it'd be nice to also support these "anonymous" pattern matches.

Proposal: Support the wildcard pattern after when to indicate a lambda when.
when _
  is [x, y] => ...
  is [x] => ...

Should be equivalent to
fun a =>
  when a is
    is [x, y] => ...
    is [x] => ...

Superseded by #57


remove protobuf from brew install docs

remove protobuf from brew install documentation since #596 got merged

did some mistake, will open a new PR which includes the changes of this one 🤦🏻♂


Search and add specific keyword to title

Hello!
Sorry to bother, I'm trying to search for an specific keyword and put that into the tab title. I've tried the current method, which uses {selector} to inject it to the title with no success.
Is this actually possible? If so, I'd like to use it in 2 different scenarios:
A. Keyword never changes on the specified URL, but it's inside a huge wall of text.
B. Keyword changes inside the URL to a random fixed value (only 5 random fixed values)
Thank you!

Hello @Survap,
Please paste the HTML source code and give the rule you have tried.
Feel free to reopen the issue if needed.
Best


"Change effect" result is incorrect after pressing green flag or stop

Expected behavior
The green flag and stop button both clear the audio effects to 0. If you do these things, for example:
click [set pitch effect to 100]
press green flag
click [change pitch effect by 10]
click [play sound meow]
The meow sound should play with pitch effect set to 10.
Actual behavior
The meow sound plays with pitch effect set to 110. Apparently, the green flag and stop buttons are not completely clearing the sound effect values.
Not that the problem appears only after using "change effect." If you press the green flag and then play the sound, it correctly has the pitch effect set to 0. So the problem is with the stored value used by "change effect" to update it.
Steps to reproduce
See above.
Operating system and browser
Mac OS 10.11.6 Chrome 57

This bug is happening because we're not resetting the "custom state" storing the sound effect values on green flag and stop events. Specifically, in rendered target, we're clearing effects in the audio engine (where their state used to be stored), but not in Scratch3_sound's custom state.
Currently, we don't have a way to use green flag and stop events to reset custom state (AFAIK).
Graphic effects probably pre-date the custom state mechanism. They're stored directly inside of the rendered target, and all updates to the graphic effects happen there, including clearing on green flag and stop.
So, my question now is, should we:

Move the sound effects state into rendered target, so we can handle them the same way as graphic effects, and correctly reset on green flag and stop.

or

Keep the sound effects state in custom state in Scratch3_sound, and create a way for rendered target to directly reset its custom state. This would probably require a bunch of duplicated code, or an abstraction layer to help (as I suggest in https://github.com/LLK/scratch-vm/issues/822).

Or is there some other way?
Would love your thoughts @cwillisf @rschamp


Nuxt Compatibilty 4 Glob error

Hi,
Hope you well.
Been getting this on all Nuxt 4 builds. Any clues?
vite-pwa/vite-plugin-pwa#15 149.6 PWA v0.21.0
vite-pwa/vite-plugin-pwa#15 149.6 mode      generateSW
vite-pwa/vite-plugin-pwa#15 149.6 precache  3 entries (0.21 KiB)
vite-pwa/vite-plugin-pwa#15 149.6 files generated
vite-pwa/vite-plugin-pwa#15 149.6   ../.output/public/sw.js
vite-pwa/vite-plugin-pwa#15 149.6   ../.output/public/workbox-4723e66c.js
vite-pwa/vite-plugin-pwa#15 149.6 warnings
vite-pwa/vite-plugin-pwa#15 149.6   One of the glob patterns doesn't match any files. Please remove or fix the following: {
vite-pwa/vite-plugin-pwa#15 149.6   "globDirectory": "/app/.output/public",
vite-pwa/vite-plugin-pwa#15 149.6   "globPattern": "**/_payload.json",
vite-pwa/vite-plugin-pwa#15 149.6   "globIgnores": [
vite-pwa/vite-plugin-pwa#15 149.6     "**/node_modules/**/*",
vite-pwa/vite-plugin-pwa#15 149.6     "sw.js",
vite-pwa/vite-plugin-pwa#15 149.6     "workbox-*.js"
vite-pwa/vite-plugin-pwa#15 149.6   ]
vite-pwa/vite-plugin-pwa#15 149.6 }

Just manifest with icons and short name and name and description.

Maybe we need to check v4 compat mode and use róotDor instead srcDir, will check it tmr.
Can you check it using create pwa https://vite-pwa-org.netlify.app/guide/scaffolding.html ? Select vue then Nuxt 4 and follow the wizard

Hey @userquin yip! that is the issue! Build fine when using the nuxt 4 compat template!

@userquin any temp workaround. What can I set the srcdir as?

Check https://github.com/vite-pwa/nuxt/blob/main/src/utils/config.ts

You need to change both, workbox.globDirectory  and outDir: previous module will not configure workbox.globDirectory.

If you're using SSR (ssr: true or not configured or running nuxt generate ), check also if the sw.js and workbox-**.js present in .output/server folder.

If you're using SSR (ssr: true or not configured or running nuxt generate ), check also if the sw.js and workbox-**.js present in .output/server folder.

am using ssr, not present in .output/server only in public

Add this in your nuxt config file:
import { createResolver } from '@nuxtjs/kit'`

const { resolve } = createResolver(import.meta.url)

const outDir = resolve('./.output/public')

that wont work nuxtjs kit is not available in nuxt config if not module

or
import { fileURLToPath } from 'node:url'
import { resolve } from 'node:path'

const outDir = fileURLToPath(new URL('./.output/public', import.meta.url))

Use __dirname instead of import.meta.url, you can also add "type": "module", to your package.json

Nope, sorry - none of this works. of course type is set to module - but nuxt kit is only availble as far as i understand in an actula nuxt module context?
@danielroe any simple workaround for the moment? I see you did the nuxt 4 compat version for the scaffold cli

If repo created using create-pwa works, why your repository is not working? Did you follow the instructions in nuxt docs https://nuxt.com/docs/getting-started/upgrade#new-directory-structure (check Migration Steps a few line below)?

Hey. Of course have quite a few projects running on v4 compact for about 3 months already!
I think that the specific nuxt module is not compatible with v4 yet. Even though when scaffolding it is since it isn't using the same module?

🙏 provide a minimal reproduction with your issue, I will check what's happening...

seems like this was a cache issue was using experimntal build cache on swapover from Nuxt 3 to 4 - which for some reason even after setting to false was persisting in production - this is not a bug as far as i can tell.


[WIFI-13097] Fixed issue with Navbar Overlap

Fixed Overlapping issue with GW UI Navbar overlap on Device Details page.
JIRA URL

@BourqueCharles thanks, sorry about that! It should be there now.

@BourqueCharles thanks, sorry about that! It should be there now.

No problem! It's all fixed now, will merge your PR. Good catch and great fix!


Row Level Security

Hi,
   Has anyone implemented RLS with forms login, currently we have Power BI on premise, and we don't use Analysis Server to get the data, is there anyway can implement RLS (ex: a user login to the report server, it will only show certain data by passing the parameter - not filtered)

Thanks,
Ryan

This is currently only possible with live analisys services connections (not embedded data), in that scenario you can use CustomData={{userid}} in the connection string, why can in turn be used to filter the data. For embedded models you need pbidesktop RLS which is still not supported. If you havent posted this yet to ideas.powerbi.com please do so to get more visibility and help prioritize that feature.


request for resnet pretrained on kinetics400

Hi, thx for the great work and open source repository!
Due to the poor computing resource, I can not train the resnet18 and resnet50 on kinetics400. Could you provide the pretrained model?

Hi @dreamedrainbow
Please find the pretrained models in modelzoo.

@xvjiarui sorry, I am confused about the model in model zoo. Is this model a kinetics-pretrained model or a davis-train-saved model for fine-grained correspondence?

Hi @dreamedrainbow
We didn't fine-tune on DAVIS for fine-grained correspondence. We directly use the kinetics pretrained model to inference on DAVIS VOS.

@xvjiarui thx, my problem solved.


page for showing data for leetcode

Data from redux store store/leetcode needs to be shown in a separate page when user clicks on leetcode from the drawer

Hello @sifatul bhaiya,
can I look into this issue?


gift command

suggested by someone in the gen:unite server; a command similar to !trade where you can give someone a card, but they don't have to give you anything in return. basically just smthn like !gift (id) (user) and it would just give that card to the target user

This one could be done pretty easy, I'd just copy paste the code from the trade command, change the syntax and remove the lines where it prompts for confirmation.
Should I make it prompt to confirm from the sender, receiver, both, or none?

i think sender confirmation would be enough, that way it can be a surprise(????? pings still happen but idk. nice gestures)

I can make it work like this:
User: !gift 1234 @Example (or user ID)
Bot: @User, would you like to send card Example Card (1234) to Example? Reply 'yes' or 'no'.
User: yes
Bot: @Example, @User has gifted you Example Card (1234)!

I'll try and implement this later tonight.

Added to beta bot. Pushing change to main bot around midnight tonight, pushing code to repo now.

Pushed to repo. See commit 2824aa1.


Additional type page

Problem
The additional type integration needs a standalone page to list the related sources. In the last release assets (Audio release) there is no page for this case.
Description

Desktop
Mobile

The page proposed above is based on No results and Timeout pages, and the content is center-aligned in 6 columns. The header has disabled the filter button as some sources do not allow specific filters. If we agree with the idea, I need to add the button to the Design Library and create a ticket for its integration.
I am not fully sure about the h1 text. The description below is from the meta-search sources list visible under the load more button.
Additional context

Figma mockups
Related ticket: #739

@panchovm should this ticket have needs design tag on it?

Thank you, Francisco!


Dynamic documentation for the Layouts

I recommend that whatever you do the keyboard remains mostly self-documenting

Actually it might be a good idea to add some kind of auto-generated documentation depending on the layout used, by simply parsing the XMLs and showing movements for each possible output. But I'm not sure that's very easy to implement, specially the UI/UX side...
Sorry we finda deviated a lot from the original subject, but I feel like this is still a constructive discution to make this app better :)
Originally posted by @bonswouar in https://github.com/flide/8VIM/issues/253#issuecomment-1079660540
This is a brilliant Idea... a layout reading program that converts the movement into a picture sort of thing and creates the documentation...
any ui design ideas?

This may not be the best approach, but I have been imagining a layout customization page that you get to from the settings menu. Typically, all of the options here open as modals, but I think this needs its own page. This would be best if custom configurations could be saved (so that you could switch back and forth from the defaults), and imported/exported so you can share with friends or change devices.
But let me get back to the main idea, the layout customization page. it should:

list all of the characters that have configured gestures
provide options to edit or preview the gestures for each listed character
provide an option to add special characters from a character map
A gesture preview should:
show a demo keyboard with an animated touch trail based on whatever has been set
The input edit should:
ask you to input what you want for the selected character
ask for confirmation by showing the demo output
alert you if your new input has already been used elsewhere, and ask for confirmation (could leave this out, but it would be nice)

And a final note, I could see this having all of the characters including the ones on the primary 8vim board. If any of those are changed, the characters displayed on the keyboard itself could change positions based on your inputs. But I am going a bit far here. To keep it simpler initially, the gesture configuration options could exclude characters that are displayed on the main 8vim keyboard, unless the user wanted to add additional alternate configurations for them for some reason.
@bonswouar @flide let me know if you think this is not what you had in mind

Though I suppose even without being customizable, just having a character library would be useful. Still, I think you should be required to open up the preview from a character listing (mostly to save screen space). I would love to hear perspectives from a designer too!

Any designer around? :smile:
I personally don't have much idea about the UI
The basic idea would be to have some sort of list (but what kind of list? It has to be quite long, and mostly 1 char per item, that's not something usual, I have no good example in mind) and an image (static with an example trace, or animated?). And for easier navigation I think both should be on the same screen / popup
And @MegaKeegMan proposition to integrate customizable gestures is great but is certainly much more work, maybe that could be done afterwards?

Would you guys agree that this would be a good place to show the gestures? This are is relatively unused.


Fix vocab name search

@markquintontulloch not sure why this PR is failing on creating allele / references.

@markquintontulloch I see the reason for my error...


is it legal

I have question, if this recapacha for ios legal?
As Apple terms and google terms ?

I'm no legal expert, but I don't know of any guidelines or terms of usage for both platforms that this framework may be breaking. I believe the biggest concern would be regarding executing arbitrary code, which ReCaptcha does not. For all effects, it's simply a WKWebView displaying web content.
I have publicized apps and I know of third party ones that are currently in the AppStore running this framework. So if your concern is regarding the app not being approved upon review, rest assured ReCaptcha won't be the reason for why it fails.


Notifications service doesn't like Kubernetes ConfigMaps

White working on https://github.com/owncloud/ocis-charts/pull/330, I ran into this:
kubectl -n ocis logs -f deployment/notifications
{"level":"error","service":"notifications","error":"read /etc/ocis/notifications/templates/html/img/..data: is a directory","event":"ShareCreated","time":"2023-06-20T15:09:57.6929667Z","line":"github.com/owncloud/ocis/v2/services/notifications/pkg/service/shares.go:59","message":"could not get render the email"}

Looks like the current implementation doesn't like Kubernetes ConfigMaps, because the template path looks like this when mounting these files as configmap:
kubectl exec -it -n ocis pods/notifications-76bbd45d77-znbf2 -- sh
~ $ cd /etc/ocis/notifications/
/etc/ocis/notifications $ tree -a
.
└── templates
    ├── html
    │   ├── ..2023_06_20_15_09_09.176225182
    │   │   └── email.html.tmpl
    │   ├── ..data -> ..2023_06_20_15_09_09.176225182
    │   ├── email.html.tmpl -> ..data/email.html.tmpl
    │   └── img
    │       ├── ..2023_06_20_15_09_09.1371932421
    │       │   ├── logo-mail.gif
    │       │   ├── logo-mail.jpeg
    │       │   └── logo-mail.png
    │       ├── ..data -> ..2023_06_20_15_09_09.1371932421
    │       ├── logo-mail.gif -> ..data/logo-mail.gif
    │       ├── logo-mail.jpeg -> ..data/logo-mail.jpeg
    │       └── logo-mail.png -> ..data/logo-mail.png
    └── text
        ├── ..2023_06_20_15_09_09.4107791090
        │   └── email.text.tmpl
        ├── ..data -> ..2023_06_20_15_09_09.4107791090
        └── email.text.tmpl -> ..data/email.text.tmpl

10 directories, 10 files

Originally posted by @wkloucek in https://github.com/owncloud/ocis-charts/issues/330#issuecomment-1599014706

@kobergj @mmattel Now we have the default path like {NOTIFICATIONS_EMAIL_TEMPLATE_PATH}/templates/...
I can change the path to {NOTIFICATIONS_EMAIL_TEMPLATE_PATH}/email/templates/... Is it ok to change the path and readme on this release step?

look like I misunderstood, It doesn't matter which  path will be under the NOTIFICATIONS_EMAIL_TEMPLATE_PATH, the Kubernetes ConfigMaps anyway add the extra.


Package request: firebase-admin-python

Project description
The Firebase Admin Python SDK enables server-side (backend) Python developers to integrate Firebase into their services and applications.
Metadata

homepage URL: https://firebase.google.com/docs/admin/setup
source URL: https://github.com/firebase/firebase-admin-python
license: Apache Software License (Apache License 2.0)
platforms: Linux, macOS, Windows

I believe I can try to package this library, can someone assign me?


Testing for filters.py

Testing for filters.py
Hello Everyone!
I hope you are all doing well!
Description
I wrote tests for filters.py. I also added a fixture that generates per_id of job_listings, hopefully the code and comments are clear enough on that one. I tried to isolate all testing units as much as I could from each other.
List any developers that will be affected or those who you had merge conflicts with.
Context of change
Please add options that are relevant and mark any boxes that apply.

[ ] Software (software that runs on the PC)
[ ] Library (library that runs on the PC)
[x] Tool (tool that assists coding development)
[ ] Other

Type of change
Please mark any boxes that apply.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
Provide instructions so we can reproduce.
Please also list any relevant details for your test configuration.

[x] Tested test_filters.py in isolation.
[x] Tested the entire pytest suite as a whole.

Checklist:
Please mark any boxes that have been completed.

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

Your code is very clean and simplistic... I like 👍


Added two Stream tests that might come handy when making Stream truly lazy

(both tests are currently ignored)

Codecov Report

Merging #2227 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #2227      +/-   ##
============================================
- Coverage     97.16%   97.15%   -0.01%
- Complexity     5282     5283       +1
============================================
  Files            94       94
  Lines         12141    12177      +36
  Branches       1581     1595      +14
============================================
+ Hits          11797    11831      +34
- Misses          191      193       +2
  Partials        153      153

Impacted Files
Coverage Δ
Complexity Δ

...src/main/java/io/vavr/collection/RedBlackTree.java
93.39% <0%> (-0.57%)
23% <0%> (ø)

...c/main/java/io/vavr/collection/JavaConverters.java
90.05% <0%> (-0.53%)
1% <0%> (ø)

vavr/src/main/java/io/vavr/Value.java
93.72% <0%> (-0.21%)
141% <0%> (ø)

...rc-gen/main/java/io/vavr/collection/ArrayType.java
100% <0%> (ø)
26% <0%> (ø)
:arrow_down:

...rc/main/java/io/vavr/collection/BitMappedTrie.java
100% <0%> (ø)
72% <0%> (ø)
:arrow_down:

vavr/src/main/java/io/vavr/collection/Array.java
99.53% <0%> (ø)
242% <0%> (ø)
:arrow_down:

vavr/src/main/java/io/vavr/collection/Vector.java
99.69% <0%> (ø)
212% <0%> (ø)
:arrow_down:

...n/java/io/vavr/collection/HashArrayMappedTrie.java
98.5% <0%> (ø)
1% <0%> (ø)
:arrow_down:

...rc/main/java/io/vavr/collection/PriorityQueue.java
98.67% <0%> (ø)
108% <0%> (ø)
:arrow_down:

vavr/src/main/java/io/vavr/control/Try.java
95.7% <0%> (+0.02%)
98% <0%> (ø)
:arrow_down:

... and 2 more

Continue to review full report at Codecov.

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


SSV2 Model

This is great work, I noticed that the Top1 in the SSV2 dataset of the paper is 75.2%, do you have any plans to publish the model weights with the best results?

Thank you. I have uploaded our best models. The remaining models will be uploaded gradually.


flutter driver _flutter.screenshot: (-32000) Could not capture image screenshot

Sometimes an error occurs when taking a screenshot of a screen in test driver flutter
E/flutter (20595): [ERROR:flutter/shell/common/rasterizer.cc(717)] Last layer tree was null when screenshotting.
_flutter.screenshot: (-32000) Could not capture image screenshot
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.0.1, on Microsoft Windows [Version 10.0.19044.1706], locale en-US)
[√] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
[√] Chrome - develop for the web
[X] Visual Studio - develop for Windows
    X Visual Studio not installed; this is necessary for Windows development.
      Download at https://visualstudio.microsoft.com/downloads/.
      Please install the "Desktop development with C++" workload, including all of its default components
[√] Android Studio (version 2021.1)
[√] Connected device (4 available)
[√] HTTP Host Availability

! Doctor found issues in 1 category.

flutter doctor
code screenshot
`extension on FlutterDriver {
  Future<void>takeScreenshot(String screenName)async{
    final File file =  File('screenshot/$screenName.png');
    if(file.existsSync()){
      await file.delete(recursive: true);
    }
       await file.create(recursive: true);
      final png=await screenshot();
      file.writeAsBytesSync(png);
      stderr.writeln("screen shot pass");

  }
}`

؟؟؟

@Nasrallahelfarra
Can you provide complete minimal reproducible code sample that shows this error ?
Can you also try using integration_test package which is now preferred over flutter_driver and see if you still get same error using it ?
https://docs.flutter.dev/testing/integration-tests
https://github.com/flutter/flutter/tree/main/packages/integration_test#usage
https://github.com/flutter/flutter/tree/main/packages/integration_test#screenshots


feat: export according to the TrainingSetSelectionStrategy

for #174
This change adds a flag for the export_labels endpoint to export elements according to the elements that the system uses for training a model. For example, if the TrainingSetSelectionStrategy adds weakly labeled elements, these weakly labeled elements will also be exported.

Codecov Report

Merging #242 (48ff32f) into main (a85c42d) will decrease coverage by 0.10%.
The diff coverage is 60.00%.

@@            Coverage Diff             @@
##             main     #242      +/-   ##
==========================================
- Coverage   82.43%   82.33%   -0.11%
==========================================
  Files          56       56
  Lines        3683     3690       +7
==========================================
+ Hits         3036     3038       +2
- Misses        647      652       +5

Impacted Files
Coverage Δ

label_sleuth/app.py
61.05% <0.00%> (-0.24%)
:arrow_down:

label_sleuth/data_access/data_access_api.py
96.29% <ø> (ø)

label_sleuth/orchestrator/orchestrator_api.py
68.73% <72.72%> (-0.37%)
:arrow_down:

label_sleuth/orchestrator/test_orchestrator_api.py
100.00% <100.00%> (ø)

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


when run "pcap.OpenOffline()", if file is chinese path,will give  Illegal byte sequence

when run "pcap.OpenOffline()", if the file contains  chinese or path contains chinese ,will give " Illegal byte sequence"

when run "pcap.OpenOffline()", if the file contains chinese or path contains chinese ,will give " Illegal byte sequence"

你现在解决了吗？


New authn filter to support Beta JWT and mTLS policy

Describe the feature request
Current authn filter accept the full authentication policy, doing request rejects for JWT/origin authentication. Later on with beta policy, we want to

Construct source.principal from x509, if available.
Still maintain the same check to only authenticate the same and/or local trust domain.
Converting attributes from JWT filter results, if available. All go into request.auth.xxx (i.e no binding logic).
If more than one JWT is given at run time, use only the first one (sorting criteria is not matter, as long as it is consistent).
Not reject if JWT is missing.

We are considering adding new C++ compiled filter in istio/proxy to have cleaner separate with today's authn filter, but please let us know if there's any WASM bit needs to be taken into account. @PiotrSikora
[X ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Additional context

not applicable, obsolote. see the closing pr comment.


OpenHands development mode crashes OrbStack

Describe the bug
I want to use OpenHands to develop software with AI help (https://github.com/All-Hands-AI/OpenHands) (SWE Bench best performer).
Running OpenHands Docker image fails (an error is thrown from the software).
Running OpenHands in development mode crashes OrbStack.
To Reproduce
Just follow https://github.com/All-Hands-AI/OpenHands/blob/main/Development.md
I have used Micromamba. MacOS 15.1.
Expected behavior
OpenHands should work as when using Docker Desktop.
Diagnostic report (REQUIRED)
OrbStack info:
Version: 1.8.2
Commit: 44cf8269710cd29802bfee176a7cec65b55cc822 (v1.8.2)
System info:
macOS: 15.1.1 (24B91)
CPU: amd64, 16 cores
CPU model: Intel(R) Core(TM) i9-9980HK CPU @ 2.40GHz
Model: MacBookPro16,4
Memory: 32 GiB
Full report: https://orbstack.dev/_admin/diag/orbstack-diagreport_2024-12-03T11-38-38.487280Z.zip

Screenshots and additional context (optional)
Logs from the application:
utils.py:217 - Please upgrade to Docker Desktop 4.29.0 or later to use host network mode on macOS. See https://github.com/docker/roadmap/issues/238#issuecomment-2044688144 for more information.

utils.py:217 -

It is just a warning. Are there any logs after this line to pinpoint which line leads to crashing?


add Yahoo!Japan as an adopter

Signed-off-by: Yu Kitazume<EMAIL_ADDRESS>
What type of PR is this?
/kind documentation

If contributing rules or changes to rules, please make sure to also uncomment one of the following line:

Any specific area of the project related to this PR?
What this PR does / why we need it:
Yahoo! Japan uses Falco OSS for AI platform run on Kubernetes. Akira Oiwa from Yahoo! Japan, introduces how to use Falco on the AI platform. I'm filing this PR on behalf of him.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Add Yahoo! Japan as a Falco adapter.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign leogr after the PR has been reviewed.
You can assign the PR to them by writing /assign @leogr in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/assign @leogr

cc @danpopSD

LGTM label has been added.  Git tree hash: 7954cde92e6362e26236d801f0282ca330a3f553

/lgtm
/approve

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: danpopSD
To complete the pull request process, please ask for approval from leodido after the PR has been reviewed.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: danpopSD, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Lift save/up hangs at creating SQLite db

The command prisma2 lift save and prisma2 lift up hangs at Creating database .....

This behavior exists in the following versions I tested:
<EMAIL_ADDRESS>binary version: b72fd0a786adc77ba6c61d0a9637ff0371e433b0
<EMAIL_ADDRESS>binary version: b8d90fea39d266b128b4d748db5aca8505bb1026
It still works with the version<EMAIL_ADDRESS>binary version: 377e2db71bae5da19d0090558c8819ff57dfdfc6
Env
Mac 10.14.6

I am unable to reproduce this. Can you please share your datamodel with us?

Of course, it appears in this project https://github.com/fivethree-team/nestjs-prisma-starter/tree/prisma2
datasource db {
  provider = "sqlite"
  url      = "file:../db/dev.db"
}

generator photon {
  provider = "photonjs"
}

model User {
  id        String   @default(cuid()) @id @unique
  createdAt DateTime @default(now())
  updatedAt DateTime @updatedAt
  email     String   @unique
  password  String
  firstname String?
  lastname  String?
  posts     Post[]
  role      Role
}

model Post {
  id        String   @default(cuid()) @id @unique
  createdAt DateTime @default(now())
  updatedAt DateTime @updatedAt
  published Boolean
  title     String
  content   String?
  author    User?
}

enum Role {
  ADMIN
  USER
}

I can confirm this bug.
I think it is related to https://github.com/prisma/prisma2/issues/936#issuecomment-558578657 so for the time being don't use a relative path.

Thanks for looking into the issue

Thanks a lot for reporting 🙏
This issue is fixed in the latest alpha version of prisma2.
You can try it out with npm i -g prisma2@alpha.
In case it’s not fixed for you - please let us know and we’ll reopen this issue!

I tested it with prisma2 version  2.0.0-alpha.453 and hangs at creating the db at path file:../db/dev.db

I can confirm this is still an issue.

(Note: only happens now when folder db doesn't exists)

This is a problem in the create database RPC of the migration engine.


fix wait_for_snapshot_purge

Signed-off-by: Mohamed Eldafrawi<EMAIL_ADDRESS>

LGTM.

Changes already merged in #256


Update Git-ReadMe.md

The current format only suggests installation instructions for windows but there should links for mac installs as well.  Additionally, many of the links in the Windows install are need to be updated.  I just installed the mac version of the bithub client, which is very graphical, however, they have tools to use mac's terminal.

Chris, great suggestion.  Could you put together a markdown file describing what you had to do to set up git on your Mac and add it to the pull request?

I'll start a Wiki until I figure the rest out.  I haven't figured out where to store it besides my own web server to get an address and I am reluctant to use my webspace to host anything for the class.

@verwirrtwombat I would suggest just adding a new mark up file maybe "Git-Mac-Install.md" to the repository and then you can push it as part of this pull request then you don't have to worry about where to host it.


Markers vs Decorations: Markers do not track editing changes

monaco-editor version: 0.13.1
Browser: Edge & Chrome (presumably all)
OS: Windows 10
Steps or JS usage snippet reproducing the issue:

add markers via monaco.editor.setModelMarkers
edit the document inserting or removing lines above the marker or characters before the marker
visually, the squiggly underlines remain with the text they were applied to, including expanding and contracting if typing in the middle of the marker, per TrackedRangeStickiness.NeverGrowsWhenTypingAtEdges
Monaco.editor.getModelMarkers({}) returns the markers with the original, no longer correct, start/end-Line/Column values.

So - it appears the decorations used to display the markers are tracking the model, but the markers themselves do not?  This seems very counter-intuitive.  If this is by design, do we lose anything just using Decorations directly instead of Markers?
Thanks,
-Blake

To fully explain the differences between markers and decorations, I think it is best to use VS Code as an example.
Markers can exist without having a file opened
e.g. in this screenshot there are no models instantiated, yet the markers are still stored

Markers get bound to a model when the URIs match
When the file is opened, the markers are converted into decorations and bound to the model. But, markers are never modified by the decorations, only the owner of the markers gets to set / modify them.

I tried markers with different playground samples.
It works as expected with https://microsoft.github.io/monaco-editor/playground.html#interacting-with-the-editor-line-and-inline-decorations
Marker moves to line 3 when I hit enter in first line.
Using https://microsoft.github.io/monaco-editor/playground.html#extending-language-services-configure-json-defaults when I hit enter marker moves to line 3 but few monents later it returns to line 2 but the marked content is not there anymore.
I used this code:

monaco.editor.setModelMarkers(editor.getModel(), null, [
    {
            startLineNumber: 2,
            startColumn: 1,
            endLineNumber: 1,
            endColumn: 8,
            message: 'message',
            severity: 4
        }
]);

How can I make it follow line feeds when I use JSON schema validation as well?


Fix for one of the tests

This PR fixes "test_repo_gc"

Thanks for finding this!
You'll still receive attribution for this fix :slightly_smiling_face:

Also feel free to open a new PR if you have any other fixes (CI-related or not) that you think could be useful!


Improve billion laughts attack

billion laughts attack can be done for in-memory data, not only files, and might happen on any language containing anchors and references, like YAML.
Added spanish translation as well.

Merged. Thanks for the contribution. :+1: 😊


please is there a demo on how to use it outside NAT ?

I'm trying to use the solution to get RTSP camera to webrtc to get displayed in the browser but each time don't work properly ,would you please create or maybe refer me to a demo ,I've seen the NAT hole punching mechanism but no clue

All config examples in the docs:
https://github.com/AlexxIT/go2rtc#module-webrtc


Populate Team assessment forms due on 27th with the last submission

Reminder to populate the team assessment forms. Ref Jira : https://davidryan316.atlassian.net/browse/CC-19?atlOrigin=eyJpIjoiZjI3YjQ3MzdkNDY5NGFlNjk3MGM5OTFkZjliYWNjMmUiLCJwIjoiaiJ9

Done


Feature/collateral factory #94

Description
Creates a collateral pool factory that streamlines deployment, adds more security and reduces number of needed transactions for a user to deploy a contract (down to 2 vs 3) fixes #94
Also fixes some style issues as well as the deprecate keccak calls without using abi.encodePacked
Checklist

[x] Linter status: 100% pass
[x] Changes don't break existing behavior
[x] Tests coverage hasn't decreased

Affected core subsystem(s)
protocol, docs will need to be updated as well as dApp
Testing
tests updated and passing with new migration script.
Refers/Fixes
fixes #94

@eswarasai - once this is reviewed, I will work to create the first release of the new @marketprotocol/marketprotocol npm package

I am wondering why Travis complains on those two lines not covered compared to master https://coveralls.io/builds/17315005/source?filename=contracts%2FMarketContract.sol#L261 , it would be good to double check why the error is not emitted anymore during cancelOrder.

@cryptomental that is a good question, will look into it. thank you for the review!

@cryptomental - I think this is an issue with coverage.  I updated the tests in https://github.com/MARKETProtocol/MARKETProtocol/pull/102/commits/004b583e672ee0ae5210593887481ab0adeee1cb to be sure that the specific error code was generated and magically coverage change.
I am going to merge.
@captnseagraves sorry for throwing you down a wild goose chase on this one!


Bundle and minify CSS

Fixes #336.
Bundled them up into one file (goes from 13.5KB for 6 files to 12KB for 1), but does not minify. I tried minifying manually using requireJS, but it only removed 1KB (I followed http://requirejs.org/docs/optimization.html#onecss) so I'm not going to worry about getting it hooked up.
Another side effect was that the embed fig wasn't getting the correct CSS and now it is since they are all bundled.
Currently:

This PR:

So the embedded version now has margins? Seems like a lot to get picky about, but if we're serious about encouraging WSCs to use those embedded versions, we probably don't want the margins.

@aappling-usgs I think it's because of what we are choosing to embed. Proabably shouldn't be the whole map div, but should just be the svg. Let me see if I can quick fix it

Ok, so the div class .main-svg is actually a part of vizlab, so I just changed it around so that the embed div does not get that class.
Now embed looks like this:


Fix Attribute error, add new attribute 'file'

Fix the attribute error thrown while fetching a LanguageInfo object ( #2 )

Add new file attribute

Tests

[x] Tests are passing after this modification.

Sample tests
print(client.get_language_info(ext="py").name)

Output
Python

thank you!


Java version of Bagels

This is a Java port of Bagels that I did last year, with a few minor recent tweaks.
The main class just handles user input/output, tracks number of games played, gives instructions, and so on. This stays true to the look and feel of the original. All of the game state, logic, and guess evaluation is encapsulated in a separate class, and could (in theory) be used with a completely different front-end class.

Thanks!! 🤗


Install self-signed dev-certs on Ubuntu.2204.Amd64* helix agents using https://github.com/BorisWilhelms/create-dotnet-devcert/

Can we install self-signed dev-certs on linux using https://github.com/BorisWilhelms/create-dotnet-devcert/ for Ubuntu.2204.Amd64* helix agents?
Ubuntu based distributions: ./scripts/ubuntu-create-dotnet-devcert
This would allow us to remove https://github.com/dotnet/aspire/blob/faf26a8700c79ec435c2348d566caf4c1a4ebc13/tests/helix/send-to-helix-inner.proj#L100 .
cc @joperezr

Release Note Category

[ ] Feature changes/additions
[ ] Bug fixes
[ ] Internal Infrastructure Improvements

Release Note Description

@radical is this still something on your radar?