JVM 17 & Kotlin 1.6.x Support

The changes made to the final version of 1.20.0 have reverted fixes that addressed the invalid jvm target error experienced in this ticket. https://github.com/detekt/detekt/issues/4287
* What went wrong:
Execution failed for task ':match-api:detekt'.
> Invalid value passed to --jvm-target

I'm currently unable to upgrade to resolve the current XML External Entity (XXE) Injection vulnerability. Version 1.20.0-RC2 continues to have no issues with JVM 17 and Kotlin 1.6.x, but does not have a resolution for the vulnerability.

The changes made to the final version of 1.20.0 have reverted fixes that addressed the invalid jvm target error experienced in this ticket. #4287

Could you link to the revert you're talking about?

@cortinico this is the ticket he's referring to that should have fixed it, but it is no longer working. #4277

Is there a workaround, or do we need to revert to 1.19 for now?

How are you folks passing the --jvm-target? Via CLI or via Gradle? Which value are you specifying?
Could you provide a reproducer of any form to help us understand what's going on?

In my case I'm using the gradle plugin. I am not passing that argument explicitly at all.

In my case I'm using the gradle plugin. I am not passing that argument explicitly at all.

I'm not able to reproduce. Tested with:
$ java -version
openjdk version "17.0.2" 2022-01-18 LTS
OpenJDK Runtime Environment Zulu17.32+13-CA (build 17.0.2+8-LTS)
OpenJDK 64-Bit Server VM Zulu17.32+13-CA (build 17.0.2+8-LTS, mixed mode, sharing)

Running ./gradlew detetkMain without specifying --jvm-target leads to a successful build.

I use gradle toolchains configuration in my build. Maybe that's causing a problem?

Seems like the problem might be this declaration in my Gradle build:
tasks.withType<org.jetbrains.kotlin.gradle.tasks.KotlinCompile>().configureEach {
  kotlinOptions {
    jvmTarget = JavaVersion.VERSION_11.toString()
  }
}

It also doesn't seem to matter which version is actually specified -- detekt bails in any case.

jvmTarget = JavaVersion.VERSION_11.toString()

That should work. Could you try with just "11" or with JvmTarget.JVM_11?
Also @3flex do you have some context here as you introduced the JvmTargetConverter?

That should work. Could you try with just "11" or with JvmTarget.JVM_11?

The output of JavaVersion.VERSION_11.toString() is already "11" but I tried it anyway, and as expected, the same error results.

Thanks for the hint @cortinico on JvmTargetConverter... I added a debugger to DetektInvoker. Assuming I'm looking at the right place, I see that my build does not pass a --jvm-target option at all.
Looking at the code at https://github.com/detekt/detekt/blob/main/detekt-cli/src/main/kotlin/io/gitlab/arturbosch/detekt/cli/ArgumentConverters.kt#L54-L57, it looks like this code assumes that --jvm-target is a required argument.
I now see also in the README at https://github.com/detekt/detekt#with-gradle it does actually specify that the jvm target needs to be set explicitly for Detekt -- this appears to be new in 1.20 and was not called out in the migration guide at all.
I'm wondering if the plugin can simply use the JVM target from the Kotlin compiler options rather than having to inject this value separately.

Thanks for looking into this @rocketraman
Yes, it's a required argument but it has a default specified, if you don't specify any:
https://github.com/detekt/detekt/blob/16bbc16da0b9438d6f4fcf7a861cac9875817be7/detekt-cli/src/main/kotlin/io/gitlab/arturbosch/detekt/cli/CliArgs.kt#L189-L195

I assumed when I wrote my previous message that adding the explicit jvmTarget configuration for the Gradle plugin would fix the problem, but nope.
Debugging further, I think I see the problem... the CompilerSpec generated by the CLI invoker tooling is setting the jvmTarget value to "JVM_11", which is the toString value of the JvmTarget enum. It's this "JVM_" value that is causing the problem.

The failure seems to be at this line:
val filesToAnalyze = measure(Phase.Parsing) { parsingStrategy.invoke(settings) }

in the Lifecycle interface.

More specifically, EnvironmentFacade.environment calls createCompilerConfiguration, which a parameter: compilerSpec.parseJvmTarget().
This JvmTarget.fromString(jvmTarget) on the toString output of JvmTarget is invalid i.e.
JvmTarget.fromString(JvmTarget.JVM_11.toString) == null

I submitted a PR https://github.com/detekt/detekt/pull/4800 to hopefully fix this. I haven't tested this TBH but it does compile.

I'd like @3flex to take a look here and eventually revert https://github.com/detekt/detekt/pull/4694 if needed.

I've added some new tests in #4801. Strings are mapped from Gradle plugin and parsed correctly by the CLI.
I'm not sure what I'm missing here, so a full reproducer will be helpful, but I will continue investigating in the meantime.

What could be the problem is if JvmTarget.JVM_11.toString() is called from Gradle, and an older version of Kotlin is on Gradle's classpath, it will return "JVM_11", because Gradle embeds an earlier version of Kotlin which doesn't have the new toString() override on JvmTarget that detekt CLI relies on.
It should work if you just set jvmTarget = "11" in the build file, or jvmTarget = JavaVersion.JVM_11.toString(), and don't use JvmTarget.JVM_11.toString().

What could be the problem is if JvmTarget.JVM_11.toString() is called from Gradle, and an older version of Kotlin is on Gradle's classpath, it will return "JVM_11", because Gradle embeds an earlier version of Kotlin which doesn't have the new toString() override on JvmTarget that detekt CLI relies on.

I was on Gradle 7.3.3. I updated to 7.4.2 to check if that was the issue. Same problem with both versions. There should be no change in the version of Kotlin on the classpath -- I have no local customizations in this regard. Gradle uses whatever it uses, my build uses 1.6.10.

It should work if you just set jvmTarget = "11" in the build file, or jvmTarget = JavaVersion.JVM_11.toString(), and don't use JvmTarget.JVM_11.toString() in build files.

No this does not work. In fact, I've never had JvmTarget.JVM_11.toString() in any of my build files. The call to JvmTarget.JVM_11.toString() is happening later in Detekt inside Spec.kt.
I'm not sure why all of these conversions from JvmTarget to and from Strings are going on. If you look at PR https://github.com/detekt/detekt/pull/4800 I submitted earlier, eventually you need to pass a JvmTarget, so why not just remove the multiple back-and-forths between JvmTarget and Strings and just keep the parsed JvmTarget value all the way through?

Please provide a reproducer and we can hopefully narrow down exactly what's going on.

A build scan may also help.

The reproducer is nothing special. Its I believe the most basic possible setup: https://github.com/rocketraman/test-detekt-4786.
./gradlew --version

------------------------------------------------------------
Gradle 7.4.2
------------------------------------------------------------

Build time:   2022-03-31 15:25:29 UTC
Revision:     540473b8118064efcc264694cbcaa4b677f61041

Kotlin:       1.5.31
Groovy:       3.0.9
Ant:          Apache Ant(TM) version 1.10.11 compiled on July 10 2021
JVM:          17.0.3 (Oracle Corporation 17.0.3+8-LTS-111)
OS:           Linux 5.16.20-200.fc35.x86_64 amd64

$ ./gradlew detekt
> Task :detekt FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':detekt'.
> Invalid value passed to --jvm-target

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 696ms
1 actionable task: 1 executed

Here is the build scan: https://scans.gradle.com/s/3okubnbhzpsb4

If you saw an earlier email from me due to a comment on this issue, please ignore it -- I thought I had created a reproducer but it was a silly mistake on my part. So far I'm not having any luck with the reproducer.
I'll continue trying to reproduce for a bit, but I'm not sure how much longer I can spend on this. I'm happy to jump on a meeting to debug this together.

I finally reproduced it... here is the repro repository: https://github.com/rocketraman/test-detekt-4786.
Looks like this particular block in my gradle build is the source of the problem: https://github.com/rocketraman/test-detekt-4786/blob/master/build.gradle.kts#L13-L21.
I normally do this in my builds to ensure that my chosen version of Kotlin is used everywhere in my build.

It looks like that block is pulling in the 1.6.10 version of kotlin-compiler-embeddable whereas detekt is expecting the 1.6.20 version.
Build scan with invalid target:
https://scans.gradle.com/s/xhnopmleir2au/dependencies?dependencies=kotlin-compiler-embeddable&expandAll
Build scan with working detekt:
https://scans.gradle.com/s/ctcsi2vahsa7o/dependencies?dependencies=kotlin-compiler-embeddable&expandAll
Take a look at the detekt dependencies for kotlin-compiler-embeddable.

It looks like that block is pulling in the 1.6.10 version of kotlin-compiler-embeddable whereas detekt is expecting the 1.6.20 version.

That's exactly what's happening - detekt uses the "detekt" configuration to manage the Gradle plugin's dependencies, which would normally be pulling in detekt-cli 1.20.0 and kotlin-compiler-embeddable 1.6.20 (or 1.6.21, can't recall which we shipped with).
When you override all configurations you're also overriding the "detekt" configuration, overriding the expected kotlin-compiler-embeddable version.
You can do this instead: configurations.matching { it.name != "detekt" }.all {
I've tested on your repo and it fixes the issue.

@Nitewriter I'm conscious that you might have a different root cause - or is this enough info to resolve your issue as well?

When you override all configurations you're also overriding the "detekt" configuration, overriding the expected kotlin-compiler-embeddable version.

Is it correct for detekt to be tied so strongly to a particular kotlin version? And even if yes, then should detekt fail in such an esoteric way if that constraint is not met?

Is it correct for detekt to be tied so strongly to a particular kotlin version?

Yes. detekt utilises internal compiler code for its analysis. There's no public API as such so we don't have much choice here. Some updates to Kotlin are breaking for detekt so maintaining compatibility with multiple Kotlin versions isn't realistic.

should detekt fail in such an esoteric way if that constraint is not met?

I think that's debateable. In your case it's a build configuration issue, and we can't account for everything that someone might do in their own build files when they inadvertently override the behaviour of the detekt Gradle plugin. We can't lock the specific Kotlin compiler version in the Gradle plugin either, because users can choose different detekt-cli versions and apply to the "detekt" configuration. A warning might be feasible, PR welcome.

I think that's debateable. In your case it's a build configuration issue, and we can't account for everything that someone might do in their own build files when they inadvertently override the behaviour of the detekt Gradle plugin.

There is no need to account for "everything that someone might do". There should be a need to account for fundamental constraints of detekt being violated and report on that accordingly.
If this statement is true (it may be, but not really in this case -- the error condition in question was not anything fundamental with the compiler here, but rather the code relying on a specific version's behavior related to an ancillary function like JvmTarget.toString() which could be handled in a different way that would work across versions i.e. avoid converting from String -> JvmTarget -> String -> JvmTarget which is brittle by nature):

Yes. detekt utilises internal compiler code for its analysis. There's no public API as such so we don't have much choice here. Some updates to Kotlin are breaking for detekt so maintaining compatibility with multiple Kotlin versions isn't realistic.

Then Detekt should do everything possible to ensure that this constraint is actually met, and if it is not (regardless of what esoteric condition caused the constraint failure), then it should warn with a message like:
"Detekt requires Kotlin compiler version 1.6.20 for analysis. Detekt is currently running with Kotlin compiler version x.x.x. Check the Detekt configuration."
instead of a message like:
"Invalid --jvm-target value"

A warning might be feasible, PR welcome.

I've spent way more time on this issue already than I can justify -- might as well spend a bit more :-) Maybe I'll get to this. Pointers/guidance welcome.

BTW, other libraries that depend on specific compiler versions, like Compose, have just such logic in them. Compose will fail in the presence of the wrong Kotlin version, unless a specific setting is used to override that check.
Furthermore, IIRC Compose uses the version of Kotlin in the project and requires that to match the version expected by Compose -- it doesn't try to bring in its own version and use that, regardless of the version selected by the user. It seems the approach of selecting a compiler version which may not match the user's selected version is bound to cause other problems.

I'm using JDK11, detekt 1.20.0 and kotlin 1.6.10 and all works perfect so this is a bit more complex. I mean, detekt right now works on other versions of kotlin too.

See https://github.com/detekt/detekt/issues/4786#issuecomment-1116722049
We have the root cause and a fix for that issue, but I'm not sure if the original reporter has a different issue.

You can do this instead: configurations.matching { it.name != "detekt" }.all {

@3flex
I'm new to detekt so would it be possible to create an share a pull request on the repodruced repo where to add this change ?
Thanks

@itd-pal I think that would be better raised in a separate support request, thanks! Try https://github.com/detekt/detekt/discussions

I opened #4813 @3flex

@itd-pal I think that would be better raised in a separate support request, thanks! Try detekt/detekt/discussions

Should we close this issue then?

FYI
For me it fixed the problem by updating either
detekt to 1.20.0-RC2 with kotlin 1.6.10
or updating kotlin to 1.6.20 and detekt 1.20.0


ComponentRegistrationException:

From @Tratcher on Monday, October 9, 2017 12:37:07 PM
From @codetian on October 6, 2017 2:41
I use castle  Ioc  in my application  ,like code example
but An error occurred while starting the application.
ComponentRegistrationException: Component Microsoft.Extensions.Options.OptionsManager`1 could not be registered. There is already a component with that name. Did you want to modify the existing component instead? If not, make sure you specify a unique name.
Castle.MicroKernel.SubSystems.Naming.DefaultNamingSubSystem.Register(IHandler handler)

ComponentRegistrationException: Component Microsoft.Extensions.Options.OptionsManager`1 could not be registered. There is already a component with that name. Did you want to modify the existing component instead? If not, make sure you specify a unique name.
Castle.MicroKernel.SubSystems.Naming.DefaultNamingSubSystem.Register(IHandler handler)
Castle.MicroKernel.DefaultKernel.AddCustomComponent(ComponentModel model)
Castle.MicroKernel.Registration.ComponentRegistration.Castle.MicroKernel.Registration.IRegistration.Register(IKernelInternal kernel)
Castle.MicroKernel.DefaultKernel.Register(IRegistration[] registrations)
Castle.Windsor.WindsorContainer.Register(IRegistration[] registrations)
BlueBlog.Utility.WindsorRegistration.Register(IWindsorContainer container, IEnumerable<ServiceDescriptor> descriptors) in WindsorRegistration.cs
+
                    container.Register(
BlueBlog.Utility.WindsorRegistration.Populate(IocContainer container, IEnumerable<ServiceDescriptor> descriptors) in WindsorRegistration.cs
+
            Register(container.Container, descriptors);
BlueBlog.Startup.ConfigureServices(IServiceCollection services) in Startup.cs
+
            container.Populate(services);
System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
Microsoft.AspNetCore.Hosting.ConventionBasedStartup.ConfigureServices(IServiceCollection services)
Microsoft.AspNetCore.Hosting.Internal.WebHost.EnsureApplicationServices()
Microsoft.AspNetCore.Hosting.Internal.WebHost.BuildApplication()

Copied from original issue: aspnet/Home#2227
Copied from original issue: aspnet/DependencyInjection#591

We periodically close 'discussion' issues that have not been updated in a long period of time.
We apologize if this causes any inconvenience. We ask that if you are still encountering an issue, please log a new issue with updated information and we will investigate.


Limitations

Please include limitations inside readme.md file so the newcomers will not stay frustrated when decide to use these features
Like

Zooming
Dragging
Brushing
e.t.c

I guess this could be useful. The example does mention the fact that it doesn't support anything like animations or anything that mutates the DOM. The mixin tool that was contributed only adds support for some animations too.
I guess it would be nice if the limitations were clearer though.

Yes, I agree, when I started using react-faux-dom, I had impression that I could do everything, which d3 has done including dragging & zooming & brushing


SportScanner

Hi just stumbled upon your post and since you look like you know all this stuff have you tried the SportScanner bundle as a Plex agent no one has worked on it for years so thought it would help you with your project and I'll get it working for me as well lol.
https://github.com/mmmmmtasty/SportScanner
Thanks

I'll take a look, but at this time I don't plan on converting this into a full-fledged plugin.
However, the use of The Sports DB API is interesting and will be worth further investigation.


Flip up and down vote emojis

@PlasmaPower I summon you 😉

Is everybody downvoting this PR actually trying to upvote it :thinking:

@PlasmaPower I'm hoping not 😛

BTW you really shouldn't up vote this. It's a bad, bad idea.

👍

😢 it's for the best.


Revert "revert" to test toolchain change

Revert "Revert "Fix deprecated launch and launchPath API on Process""
Revert "Revert "Fix deprecated readDataToEndOfFile API""
Revert "Revert "Bump Swift version (#4)" (#11)"

Bug/issue #, if applicable:
Summary
Revert #11 and #12 to test the change in the toolchain build.
@franklinsch
Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[x] Ran the ./bin/test script and it succeeded

Tracking testing this alongside Swift here: https://github.com/apple/swift/pull/40031

Could you please run test on this repo so that we can merge it after the test on swift success? @franklinsch

Now that the smoke tests pass, I also triggered toolchain builds in that PR.
@swift-ci please test

Now that the smoke tests pass, I also triggered toolchain builds in that PR.
@swift-ci please test

Seems like both smoke tests and toolchain builds are passed. Could you review the change and merge it?

@franklinsch Could we run test again and merge it?

@swift-ci Please test


Channels > 1-12-0 don't refer to the right rubocop version

The 1-12-1 channel refers to rubocop 1.12.0, and the rest of the channels after that refer to rubocop 1.10.0. This makes it so we can't use the newer rubocop versions.
Also, rubocop 1.18.3 is out but doesn't have a channel yet.

Looks like there are open PRs for all of these. I should have checked that first.

PRs got merged.


Exclude LICENSE.md

Fixes https://github.com/Particular/Particular.CodeRules/pull/21

Should also exclude src/NServiceBus.snk ?

@bording @andreasohlund done, please approve + merge


inconsistent semantics for how to "access" a user

In a user link, (e.g. https://matrix.to/#/@uhoreg:matrix.org), matrix.to lists a link for Riot such as https://riot.im/app/#/user/@uhoreg:matrix.org, which will show the user's profile, whereas for the other clients it says to type "/invite @uhoreg:matrix.org", which will invite the user to the room.
Since different clients may support different actions for users, and clients may support multiple possible actions, so it may be good to list multiple ways of interacting with the user and a description of what it does, such as:

Name
...
Access @uhoreg:matrix.org
.

Riot
...
View profile
https://riot.im/app/#/user/@uhoreg:matrix.org

.
...
Start chat
(something once vector-im/riot-web#4034 is fixed)

.
...
Invite to a room
Type /invite @uhoreg:matrix.org

WeeChat
...
Invite to a room
Type /invite @uhoreg:matrix.org

etc...

It might also be useful to be able to add a parameter to the matrix.to link to specify a certain action.  For example, something like https://matrix.to/#/@uhoreg:matrix.org?action=chat would mean that someone clicking on the link should start a chat with @uhoreg:matrix.org.  (cf. XEP-0147)

vector-im/riot-web#4034 was fixed some time ago. Probably it would make sense to make action=chat the default.


Add a parameter to substitute -a (Amount) with a relative target

When rebalancing, an absolute target to rebalance isn't always appropriate, especially when using a range of --pto.
Is it possible to add a percentage target, eg --per=0.7, to enable a range of targets to be filled up and the routine not stopping, until a percentage of 70% inbound is reached?
It's relevant for a larger node-runner, who has like 10 or more channels with < 10% outbound.
Ideally the rebalancer is running, until it hits a desired target with all channels market as target, with desired outbound at X %

I think there should be two parameters then, one that sets the target as capacity * per / 100 and calculates the amount for every attempt as capacity * per / 100 - local_balance, and another to make it run until there are no source or target channels. Shouldn't be hard to implement.

I think this issue is now resolved with both #12 and #18 merged.


Merge master branch 2619197 to collab authn

Merge master https://github.com/istio/istio/commit/26191979591b5f575c36f3658a0ac4a18d79c13e to collab-authn branch.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: lei-tang
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:
Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@lei-tang: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
1c36cfa90edb5aa7da9fcbbc256767311c532d4e
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


PHP 8  \GDImage object Support

duplicate of https://github.com/Intervention/image/pull/1059

duplicate of https://github.com/Intervention/image/pull/1059

@mfn That PR will still not work on PHP8, compare the changes and approach.

@mfn That PR will still not work on PHP8, compare the changes and approach.

Is the other one incomplete? Both PRs touch different lines… would it need both? 🤷♀

Is the other one incomplete? Both PRs touch different lines… would it need both? 🤷♀

@mfn  \Intervention\Image\Gd\Decoder uses the AbstractDecoder abstract Class

@mfn  \Intervention\Image\Gd\Decoder uses the AbstractDecoder abstract Class

I merged https://github.com/Intervention/image/pull/1059, which adressed the same issue.

#1059 fixed AbstractDecoder, but did not fix Gd/Commands/ResetCommand. See #1107


after v0.2.24,  FirstOrCreate method will set id to 0

GORM Playground Link
https://github.com/go-gorm/playground/pull/48
Description
mysql FirstOrCreate method will set id to 0. & id appears 2 times in the query condition

Fixed, thank you for your report.


chore: refactor code smells

I have done a few code refactors to reduce code smells and increase readability. This is part of my University assignment.

 Decompose conditional: Commit  f434744

Moved HTTP request filter condition to a separate method in StaplerInstrumentationServletFilter class.

Rename variable : Commit 476447c

eol and c renamed to endOfLine and character respectively for better readability in ConsoleNotes class.

Extract Method: Commit  35bffde

onNewHead method in GraphListenerAdapterToPipelineListener was performing two distinct functionalities. Separated them out in processPreviousNodes and processCurrentNode
I have not created an issue as it was a refactoring change. Please let me know if an issue is required, I will create it.
Testing done

### Submitter checklist
- [x] Make sure you are opening from a **topic/feature/bugfix branch** (right side) and not your main branch!
- [x] Ensure that the pull request title represents the desired changelog entry
- [x] Please describe what you did
- [ ] Link to relevant issues in GitHub or Jira
- [ ] Link to relevant pull requests, esp. upstream and downstream changes
- [ ] Ensure you have provided tests - that demonstrates feature works or fixes the issue

There is no small contribution, Thank you for your contribution


Influx 1.0.2 fill(0) behavior is different than 0.9.6

Directions
Bug report
We use  Kapacitor to build health check alert.
Each host will send out heartbeat every 10s. The alert script will monitor a time window. And any host has no point in a window, the alert will be raised for that host.
So base on that, the alert query is something like
 `SELECT sum(\"value\") as value FROM \"collectd\".\"default\".xxxxxxxxxx where time >= 'xxxxxxx' AND time < 'xxxxxxxx' GROUP BY host fill(0)`

Expected behavior: [What you expected to happen]
in 0.9.6 the query result is :
{     "results": [         {             "series": [                 {                     "name": "xxxxxxx",                     "tags": {                         "host": "host1"                     },                     "columns": [                         "time",                         "value"                     ],                     "values": [                         [                             "timexxxxxxxxxxxxx",                             24                         ]                     ]                 },                 {                     "name": "xxxxxxx",                     "tags": {                         "host": "host2"                     },                     "columns": [                         "time",                         "value"                     ],                     "values": [                         [                             "timexxxxxxxxxxx",                             0                         ]                     ]                 }             ]         }     ] }
Actual behavior: [What actually happened]
but in 1.0.2:
{     "results": [         {             "series": [                 {                     "name": "xx",                     "tags": {                         "host": "host1"                     },                     "columns": [                         "time",                         "value"                     ],                     "values": [                         [                             "timexxxxxxxx",                             24                         ]                     ]                 }             ]         }     ] }
As you can see, "host2" is not in result and alert will not be triggered for "host2".

@earthnut Do you have a way that we could reproduce this?

@jsternberg Any ideas?

@jwilder this looks like it's expected and changed as part of the new query engine in 0.11. It will not fill in intervals where there is not at least one series in that interval, otherwise, we would have a lot of difficulty with high cardinality data returning null values for way beyond them being gone.
We should possibly think of opening a discovery issue for this though as it's a common point of confusion. During all of the times I've explained it, I haven't heard a reasonable solution that works, but it's definitely something worth exploring.

Is there any workaround so far?


Can wheelView supports images?or how to set custom view layout item

Can wheelView supports images?or how to set custom view layout item?Thanks a lot!

@devishell Emm.. Base Library - Wheelview implements functions by canvas.drawText，So for now,  it has not yet supports. But you can download the project , and modify the Wheelview.Class source code ( onDraw() method ) to suport image ( or your custom view ). Hope this will help you.


Create pdf from pdf-blob

Is it possible to create a pdf document from a stored pdf-blob in my database? Something like:
function saveDatabaseBlob (blob) {
    var doc = new jsPDF("portrait", "mm", "a4", "compressPdf");
    doc.saveAs(blob, 'ZZZ.pdf');
}

When I create a pdf file like this:
var doc = new jsPDF();
doc.text(20, 20, 'This is the default font.');
doc.setFont("courier");
doc.setFontType("normal");
doc.text(20, 30, 'This is courier normal.');
var pdfString = doc.output('datauristring');
// pdfString will be saved to my Database

Is it possible to create a new PDF from pdfString? If yes, how?

Thank you very much for response.  on the client I do have response as shown and now what I need is to render it pdf and  display the pdf.
Do you have any suggestion or example, how to do it ?
Thank you,
jagruti

What you need is the opposite project – PDF.js by Mozilla. That will allow
you to display the PDF.
On Fri, May 22, 2015 at 1:14 PM, jagrutidave<EMAIL_ADDRESS>wrote:

[image: encodedbytearray]
https://cloud.githubusercontent.com/assets/8224810/7770216/02095180-005a-11e5-92e4-cd7e6ea9fcc0.PNG
Thank you very much for response. on the client I do have response as
shown and now what I need is to render it pdf and display the pdf.
Do you have any suggestion or example, how to do it ?
Thank you,
jagruti
—
Reply to this email directly or view it on GitHub
https://github.com/MrRio/jsPDF/issues/502#issuecomment-104645982.

--
James Hall
Director
Parallax
+44<PHONE_NUMBER>
http://parall.ax/
Registered office: The Old Brewery, High Court, Leeds, LS2 7ES
Registered in England no. 07430032
VAT No. 101 3405 84

Thank you very much for response. on the client I do have response as shown and now what I need is to render it pdf and display the pdf.
Do you have any suggestion or example, how to do it ?
Thank you,
jagruti

Duplicate of #131


Clarify the status of blank lines before and after blocks

In a way, this is a followup to rakudo/rakudo#5442.
The current RakuAST implementation treats empty lines before and after blocks as insignificant delimiters, no exceptions made. Up to my understanding, this is to follow guidelines in this synopsis. This approach results in LTA behavior with regard to the blank lines when AST of an existing document is deparsed.
My understanding is that anything within any RakuDoc block is part of the document where formatting may play role to the document writer. Blank lines are no exception as, for example:
=begin rakudoc

=begin item
I1
=end item
=begin item
I2
=end item

=begin item
Not sure yet if this one is to be included.
=end item
=end rakudoc

Visual separation can be insignificant for the output, but it is helpful for the writer themselves.
I think the statement about blank lines must only apply to the outers of a rakudoc. I.e. it must play role for something like:
=item Argument 1 is...

=item Argument 2 is ...

method foo($arg1, $arg2) {...}

In the above example both item texts would have exactly one trailing newline.

@vrurg In RakuDoc the situation is not the same as RakuAST.
A block is defined as terminating with a blank line or a new block. So,
=item this is the first item, short on one line
=item this is the second item, no need for a blank line above since on one line
=item this is the third item,
but the text extends to another line

=item this is the fourth item, but it needs to be distinguished from the previous one by a
blank line because the third item ran over lines.

This text is now a para block and is not an item

BTW, yes, your example touches another nuance.
Is the blank line part of the abbreviated block above it? I mean, when rendered it is obviously will be ignored. But how do I know, while traversing RakuAST, if the block had that trailing blank line(s) or not? Basically, it winds down to how we treat the following:
=item I1
=item I2

=item I3

Para

Would that be:
=begin item
I1
=end item
=begin item
I2
=end item
=begin item
I3
=end item

Or, as I'd expect it:
=begin item
I1
=end item
=begin item
I2

=end item
=begin item
I3
=end item

Para

Though it's still questionable where the blank line before the paragraph belongs to.
Apart of that, my example is about =begin. What about the spacing before =begin and after =end? My point is that it must be part of the surrounding text. E.g. the blanks between an =end and the following =begin become a paragraph containing these blanks as-is.
The overall point is to let DEPARSE produce the same output as the originally parsed document. Exactly for the reason that it is a document where content matters.

@vrurg @lizmat @thoughtstream  Some of this discussion is beyond me at the moment in that it relates to the interaction of RakuAST and RakuDoc.
Some comments (and attach 'I think' to all of them)

it is very important to distinguish between a 'block' in the ambient (that is in a Raku program) and a 'block' in RakuDoc. By doing so, the rules for a 'block' may differ.
In RakuDoc, an explicit blank line (or lines) may terminate a block, but is not a part of the content of the block.

In this respect the first & terminating blank line is equivalent to a =end xxx line. The terminator is not included in the content

In order to ensure that blank lines can be included as content, there are special verbatim blocks, such as code and output that when delimited with =begin / =end preserve whitespace both vertical and horizontal.
In the example given above

=item I2

=item I3

would only have I2 as the content of the block, so it would be equivalent to
=begin item
I2
=end item

If you want to have explicit vertical whitespace in the item 2, you would need
=begin item
=begin code
I2

=end code
=end item

From the issue statement here and the other one cited above, the ambiguity may arise when RakuDoc blocks are mixed together with ambient Raku. If this is the case, then a solution may be to require all RakuDoc statements to be enclosed within a =begin rakudoc / =end rakudoc terminators (other than perhaps =comment) and for whatever is within them to be considered a Raku block with the leading and trailing blank lines as defined by RakuAST.

it relates to the interaction of RakuAST and RakuDoc.

I'd say it relates to any RakuDoc parsing engine, not RakuDoc alone.

it is very important to distinguish between a 'block' in the ambient (that is in a Raku program) and a 'block' in RakuDoc. By doing so, the rules for a 'block' may differ.

That's part of my expectations.

In order to ensure that blank lines can be included as content, there are special verbatim blocks, such as code and output that when delimited with =begin / =end preserve whitespace both vertical and horizontal.

I didn't consider this aspect previously, BTW. To me plain =item I1\n\n\n would render into just I1 implicitly. The question of preserving the newlines remains, but I'll be back to it later. For the moment, I don't like the idea of using code or output because these are likely to be formatted, with font and background differring from the surrounding text.
Perhaps it would make more sense to have :verbatim with para:
=begin para :verbatim
I1

=end para

In RakuDoc, an explicit blank line (or lines) may terminate a block, but is not a part of the content of the block.

This is also apparent. The question is what representation specific cases would have within a parser. Not that RakuDoc spec must regulate it, but concrete wording would let avoid discrepancies in treating the original document.
Back to the examples.
=item I1

=item I2

When it comes to internal representation, would it be:

item
"\n\n"
item

Or:

item
item

Though my preferences still goes to

item<I1\n>
item

The point in doing so is that it'd be up to a renderer to decide how to treat the trailing newlines. Most would ignore them (except if the idea about :verbatim makes it through).
BTW, there are also cases of:
=item the last

Text goes here

But I think it would still has to be item<the last\n\n because the blank here is more of a terminator than part of the text.
When it comes to =begin, then there is very little is to be discussed about the separating newlines:
=begin item
I1
=end item

=begin item
I2
=end item

This is:

item
"\n\n"
item

Then, again, a rendered may or may not pay respect to the blanks. But to whatever tool these blanks may matter – they wouldn't be lost after parsing.
And getting a little bit back, as to what to do about:
=begin item
I1

=end item

I'd expect this also to be considered as item<I1\n\n>. Then, again, a renderer is likely to ignore these, but any other tool would know about them and use them for whatever purpose it has.
To wind down a bit:
We must remember that aside of the renderers there are other tools to come. For them elements of the original document we consider irrelevant for the representation may have different meaning for processing the doc in the context of the task they're solving. Apparently, they could develop their own parser in order to have these bits available. But why'd they need it for if there is RakuAST around? By taking into account that it is based on the spec, all we need is to make the spec clear about such edge cases.

@vrurg The issue you are concerned about seems to be the way RakuAST pre-processes the RakuDoc source, and the "entropy" (more later, because it is important for another reason).
RakuDoc does not care about whitespace as content, unless directed to using a Verbatim block or S< ... > markup, which preserves horizontal white-space inline.
You suggested that the specification be amended. In the section on 'Block syntax' suppose we add text at the end of the section as follows:
Where an extended or abbreviated block forms uses a blank line as the block terminator, the blank line is not included as part of the content of the block. All subsequent whitespace, whether horizontal or vertical (spaces, tabs, new lines), is ignored. Note that horizontal and vertical whitespace can be included inside delimited blocks, as described below.

Is that acceptible?
Verbatim blocks. Using a :verbatim metadata option on a paragraph is not a good idea. The problem is that RakuAST would need to look for metadata and obey them. An aspect of the design of RakuDoc was to clearly distinguish between the responsibility of the compiler and the renderer. The compiler handles blocks, the renderer handles metadata options.
Paragraphs by definition squash all whitespace.
A code (verbatim) block explicitly allows for white space to be preserved.
It would be better for a code or verbatim block to be given a metadata option to remove styling. In fact, we have got :syntax-highlighting to turn on syntax highlighting in combination with :lang. So not turning on syntax-highlighting will render a code block with no other marking.
Entropy
Currently RakuAST discards all comments and vertical whitespace, delivering only the expected contents. Indeed the renderer is required to handle the horizontal whitespace itself.
Thus RakuAST increases entropy.
The problem is that when a document is DEPARSED, the discarded portions are needed. For example,
my $a = '1'; # this is a comment

yields a RakuAST structure without the comment.
In order to create a syntax highlighter that works directly from DEPARSE, the comments need to exist somewhere.
I wonder whether that if we want extra horizontal whitespace, this is included in the same way comments are.

Is that acceptible?

Looks good to me. Hopefully, the part about "inside delimited blocks" covers the =begin rakudoc block too. It should.

Verbatim blocks. Using a :verbatim metadata option on a paragraph is not a good idea. The problem is that RakuAST would need to look for metadata and obey them.

This is not what I meant. If memory serves me well (and it doesn't because I needed to google it out first ;) ), my interpretation of verbatim vs. code or pre comes from LaTex.
The problem of interpreting verbatim as code is the way it would be rendered. It is not always that where one wants 1-to-1 treatment for a block of text they also want it to be in a fixed font with likely different background. Currently there is nothing in between "we don't care about text structure" and "it is a code block".
A use for such blocks would be, for example, inserting some lengthy poetry into text where line breaks and spaces between stanzas do matter. Apparently, one can use explicit line break code (which we don't have), but it'd be tedious. Instead:
=begin para :verbatim
How doth the little crocodile
Improve his shining tail,
And pour the waters of the Nile
On every golden scale!

How cheerfully he seems to grin,
How neatly spreads his claws,
And welcomes little fishes in,
With gently smiling jaws!
=end para

BTW, V<>, as documented, seemingly does almost that.

@vrurg There are always going to be special needs. For this reason, RakuDoc v2 expressly allows for Custom blocks. It is then for the custom block to handle the 'contents' (all the text between =begin and =end, including whitespace, horizontal and vertical).

closing as resolved


Add MODULE_INIT_COMPRESSED_FILE definition

Opening this alongside https://github.com/rust-lang/libc/pull/3782 to backport the const into libc-0.2 so that we can use it in nix

We are slightly changing the workflow here - PRs should target main, then they will get cherry-picked to stable if applicable. So I am going to close this and stable-nominate https://github.com/rust-lang/libc/pull/3782 (which I will review once I get the chance).


doc fix

mainly updates schema fast field doc
various lint elsewhere
closes #1232

Codecov Report

Merging #1391 (33063fb) into main (83d0c13) will decrease coverage by 0.00%.
The diff coverage is 76.00%.

:exclamation: Current head 33063fb differs from pull request most recent head 5c8cf6b. Consider uploading reports for the commit 5c8cf6b to get more accurate results

@@            Coverage Diff             @@
##             main    #1391      +/-   ##
==========================================
- Coverage   94.29%   94.29%   -0.01%
==========================================
  Files         236      236
  Lines       43418    43433      +15
==========================================
+ Hits        40942    40953      +11
- Misses       2476     2480       +4

Impacted Files
Coverage Δ

query-grammar/src/occur.rs
100.00% <ø> (ø)

src/core/index.rs
92.60% <ø> (ø)

src/core/index_meta.rs
94.36% <ø> (ø)

src/core/segment_reader.rs
91.16% <ø> (ø)

src/docset.rs
67.50% <ø> (ø)

src/fieldnorm/mod.rs
100.00% <ø> (ø)

src/fieldnorm/reader.rs
100.00% <ø> (ø)

src/indexer/merge_policy.rs
88.88% <ø> (ø)

src/positions/mod.rs
100.00% <ø> (ø)

src/positions/reader.rs
100.00% <ø> (ø)

... and 20 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 83d0c13...5c8cf6b. Read the comment docs.

Can you run make fmt? The CI is catching some formatting errors.


Linked camera calibration in eeprom

We noticed that in the ENSENSO_INSTALL folder (e.g. `/opt/ensenso') are files to seem to link to certain memory addresses in the eeprom of the camera. It seems that without these files, the link between rgb and depth camera is not loaded correctly when launching the ros driver.
Am I correct in thinking that on startup the SDK loads the link calibration from the camera eeprom using the address in this file?
Does this mean that when launching the driver from another PC (with a cleanly installed SDK) with the same json file for the same camera, this link calibration is not loaded, since the address to the eeprom of the camera is different?
If so, this means that part of the camera configuration is 'hidden' in the file (as in: it's not enough to have the data is the eeprom and the json to complete reconstruct the setup). What would be the right workflow to fix this? Basically run the calibration routine on the new system again? Or is there some way to save/load this file somewhere and use it on a different installation?

The files in /opt/ensenso/cameras/ are used to emulate the EEPROM for devices which don't have a real EEPROM. You can simply copy those files to a different PC when you want to move the camera between different PCs. The file name includes the serial number of the device.
For devices that have an EEPROM all information is stored in the EEPROM, as expected.
This affects mostly IDS GigE Vision cameras with firmware < 2.4. Later firmware versions do support the real EEPROM and the Ensenso SDK also implements it since version 3.2 (see the upgrade notes). If you want to use this already, you have to explicitly delete the EEPROM file and upgrade the firmware manually, since we still ship an older one.
The upcoming version 3.3 of the SDK will ship a newer firmware for IDS GigE Vision cameras and therefore support the EEPROM out of the box.

Ah, that makes sense!
Good to hear this workaround is no longer needed with the new firmware and SDK.
We'll work around it for now and upgrade when SDK v3.3 is released.


Add vscode debug configuration

Added two launch configurations:

Launch Chrome
Launch Firefox

Each will open the browser with the extension loaded ready for development.
Also added 3 vscode tasks:

npm: dev:chrome: Start webpack in development mode for chrome and watch for changes. (used as preLaunchTask)
npm: dev:firefox: Start webpack in development mode for firefox and watch for changes. (used as preLaunchTask)
stop node: Used to stop the other task. (used as postDebugTask)

First resolve issue #108, then this pr can be closed.

This wraps up this pr
https://github.com/SimonStnn/pop-a-loon/assets/113850648/38c3e544-3ccd-47f8-88ff-9eb485eef3bd


Can we make a “Class Level Relationship” in Laravel, as opposed to an “Object Level Relationship”?

Consider the following scenario:
There are couple of entities in my Laravel application like the following:

Post
Page
Image
Video

All the above entities can have CustomFieldValues, which is another entity. The structure of the custom_field_values table is as follows:

ID
entity_id
custom_field_definition_id
value
[Timestamp fields]

All the CustomFieldValues belong to a single CustomFieldDefinition entity. Its table custom_field_definitions looks like following:

ID
parent_entity_name
definition_name
[Timestamp fields]

Following are some sample data from the custom_field_definitions table:

ID
parent_entity_name
definition_name

1
Post
AuthorTwitterUrl

2
Page
SeoTitle

3
Image
OriginalSourceUrl

4
Video
MpaaRating

As you can see, CustomFieldDefinitions are definitions of extra data, that we can store about each type of entity.
Following are some sampel data from the custom_field_values table:

ID
entity_id
custom_field_definition_id
value

1
1
1
https://twitter.com/StackOverflow

2
1
2
My Page's SEO Title

3
1
3
http://example.com/image.jpg

4
1
4
G – General Audiences

A little description about the data contained in the custom_field_values table:

CustomFieldValue:1: The value for CustomFieldDefinition:1 and its entity 1 (Post:1, in this case, because CustomFieldDefinition:1 is related to Post.) is "https://twitter.com/StackOverflow".
CustomFieldValue:2: The value for CustomFieldDefinition:2 and its entity 1 (Page:1, in this case, because CustomFieldDefinition:2 is related to Page.) is "My Page's SEO Title".
CustomFieldValue:3: The value for CustomFieldDefinition:3 and its entity 1 (Image:1, in this case, because CustomFieldDefinition:3 is related to Image.) is "http://example.com/image.jpg".
CustomFieldValue:4: The value for CustomFieldDefinition:4 and its entity 1 (Video:1, in this case, because CustomFieldDefinition:4 is related to Video.) is "G – General Audiences".

custom_field_values table's entity_id can refer to any entity class, therefore it is not a foreign key in the DB level. Only in combination with custom_field_definition_id we can find to which entity it actually refers to.

Now, all is well and good, until I need to add a relationship called customFieldDefinitions to any of the entities (Say Post.).
class Post extends Model {
 public function customFieldDefinitions(){
  $this -> hasMany ('CustomFieldDefinition');
 }
}

The above does not work, because the datapoint that indicates the CustomFieldDefinition's relationship is not a foreign key field in the custom_field_definitions table, named post_id. We have to somehow build the relationship based on the fact that some records in the custom_field_definitions table has "Post" as the value of the field parent_entity_name.
CustomFieldDefinition::where('parent_entity_name', '=', 'Post');

The above snippet fetches the CustomFieldDefinitions that are related to the Post, however, it is not possible to do something like the following with the relationship:
class Post extends Model {
 public function customFieldDefinitions(){
  $this
   -> hasMany ('CustomFieldDefinition')
   -> where ('parent_entity_name', '=', 'Post')
  ;
 }
}

The where constraint works. But Laravel also injects the ID of the current Post object into the set of constraints.
So, what I want to do is, not consider the current object's ID at all, and build a "Class Level Relationship", and not an "Object Level Relationship".
Is this possible under Laravel?

Heya, thanks for submitting this.
This seems like a feature request or an improvement. It's best to post these at the laravel/ideas repository to get support for your idea. After that you may send a PR to the framework. Please only use this issue tracker to report bugs and issues with the framework.
Thanks!

It looks like you are trying to implement Entity Attribute Value model (EAV).

@driesvints

https://github.com/laravel/framework/issues/29186#issuecomment-511804198

I wasn't aware that this is not doable in Laravel right now.
I made an issue in laravel/ideas: https://github.com/laravel/ideas/issues/1720

This is indeed possible with the current state of the code.
Please see following answer in Stack Overflow from Matei Mihai: https://stackoverflow.com/a/57059279/445438


Upcoming Reminders

As a user, I should be able to get a list of upcoming reminders.

This Issue works on getting all due reminders that are upcoming for a particular day.
Testing link
UpcomingReminders
This issue references PR #346


[Crash] Server crash when using shell constructor

What were you doing when the crash occured?
The crash occurs when right-clicking the shell constructor. It happens regardless of gamemode; I tried in both survival and creative to be sure it wasn't an issue with the damage dealing. Every time I try it, the server crashes. It does not cause the clientside instance of Minecraft to crash, but I haven't tested in singleplayer.
Did the crash occur client-side or server-side?
 Also worth noting is that none of the devices seem to register as being powered, according to WTHIT's popup; this includes constructors and storage.

Yup, I know :D It's a duplicate of #46. Sadly, haven't time to fix it yet, because I plan on rewriting huge portions of this mod.

Are there any workarounds to get it to work, be it with commands, creative mode, or anything else? We're really only using the mod for those of us on the server with DID so each alter can have their own origins, inventory, etc., so losing the survival aspect of things isn't a big deal if there's some sort of workaround we can use.

Wow, this mod has become unexpectedly popular amongst the DID folks! I guess I really need to revive it ASAP if it helps people :)
So, here's the issue: the mod worked perfectly fine until Fabric changed how its @Environment annotation is processed. The last time I tested the mod, it was with Fabric Loader 0.14.9. If you're able to downgrade to that version, the mod should start working again. You could also try 0.15.0, as it's not too far off from 0.14.9 and also includes MixinExtras, which many modern mods rely on.
Unfortunately, if downgrading isn't an option, there's no workaround. This issue affects the very core component of this mod, making it impossible to resolve without an update.

I mean it's a shockingly perfect way to manage stuff without having to get separate accounts for every alter! It's definitely helpful for us, it makes it so the inventory swapping and changing origins/classes just happens automatically in one step rather than having to unload all of your gear manually and figure out a way people can swap origins without giving everyone access to all commands. As for as reviving it ASAP, we're definitely rooting for you, but definitely don't push yourself too hard, either.
Downgrading would, unfortunately, mean downgrading all the other mods on the server, which would be an annoyance for all the other players of the server, so for now we'll probably just do things the old fashioned way until the mod is updated. Until then we'll eagerly await an update!


Using npm3

I see that npm3 defaults to false in the top level default.nix. How can I actually use npm3 ? At first I thought I just need to set it to true but that is not the case I guess - i have to provide a tgz ? I am a bit confused about this ;-)

So I used to have it use npm3 by default, but unfortunately some upstream dependency of npm3 is currently failing to fetch because of problems with release-tag ranges in semver-range (by far this is the number 1 thing I need to fix with this project right now...). To get a working copy of npm3, first run
nixfromnpm -o /some/path -p npm

(or -p npm%3 if you want to be explicit)
This will (probably, unless the situation is fixed by now) fail, at which point you can follow the steps in the README to fix the problem (i.e. manually npming the failed dependency, seeing what version gets fetched, fetching that with nixfromnpm, and updating the broken dependency). Once this is done, you can change the default of npm3 to true in the top-level default.nix file, and everything should be ok.
I've already done this in the nix-node-packages repo, but if you want to have it work with a fresh package set, you'll have to do it yourself, until I fix the semver-range issue.

Has anything happened on this front since?

@gilligan
Sorry I haven't really taken a look at this. I just generated npm3 and the error in the requests dependency (which requires a version of form-data that semver-range doesn't know how to satisfy) remains. Needless to say, semver-range has not yet been fixed, because I haven't spent time on it and still don't have a clear enough grasp of how semver handles version ranges that have release tags.
So you can fix the broken upstream dependencies as described in the README. However, there's also a problem with some circular dependencies (es6-iterator, es6-symbol, d, and es5-ext all depend on each other). There are also instructions on how to fix this particular issue in the README. So long story short:

generate an npm3 expression by running the command I put in my above comment, then fix the broken form-data dependency in the requests library.
fix the 4 circularly-dependent packages I listed above.

This is a bit of a pain but you only need to do it once. After that point you will be able to use npm3 to build your node packages.
There are two other options which are worth considering:

Use nix-node-packages for your node packages. This already has a working version of npm3 including the above fixes, as well as many other packages, including those which had to be modified from what nixfromnpm generated. You might save yourself some work by using a fork of that repo.
Just don't use npm3, if you're not sure you need it.

Of course, at some point I hope to fix the semver-range issue, and the circular packages issue, but I think we both can tell that that's probably going to take longer than the above course of action. :wink:

OK with the latest changes this should be able to be closed. I successfully generated a nix expression for the latest npm, and it built. 👻 👍

Great! :+1:


Worked on recursive hasher script

Codecov Report
Base: 57.08% // Head: 59.13% // Increases project coverage by +2.05% :tada:

Coverage data is based on head (8a66168) compared to base (8204b81).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #65      +/-   ##
==========================================
+ Coverage   57.08%   59.13%   +2.05%
==========================================
  Files          12       12
  Lines         692      668      -24
==========================================
  Hits          395      395
+ Misses        297      273      -24

Impacted Files
Coverage Δ

dfimagetools/recursive_hasher.py
0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[BUG] Text not visible in animation duration input

Description
In the animator, in duration input field, the value isn't visible.
This is a quite easy bug to solve as we just have to change the input value color to white.
Screenshots

It's great having you contribute to this project by creating an issue Make sure you are assigned to this before you work on it and you read the Contributing.md file, Thank you! Welcome to the community :nerd_face:

Would you like to work on it?

Would you like to work on it?

Yea, sure!

Hey @Dun-sin, instead of just changing the text color of input value, shouldn't I style the whole input field? Currently, we're using the default input field, which is, of course, not looking good!
For now, I'll only style the input field in the animation section, and if you like it, you can apply it everywhere else!

Sure, go for it

:tada: This issue has been resolved in version 2.1.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Remove sudo from build.sh

Somewhat controversial, but in general scripts shouldn't have sudo embedded, if necessary the user should run the whole script under sudo. See for example discussions in:
https://askubuntu.com/questions/425754/how-do-i-run-a-sudo-command-inside-a-script
https://unix.stackexchange.com/questions/20284/should-i-be-using-sudo-in-scripts-that-i-write
with conflicting answers. However, removing the sudo from the script here would allow removing dependency on sudo from Dockerfile (I believe) since in Docker the install is done as root, while a user installing elsewhere is free to use sudo for the whole script.

@timomrs Thanks for your patch!

Thanks for the research, thorough explanation with links, and finally the patch :)

@an-tao @rbugajewski No problem, thank you for quick merges so I didn't even have to change my project to refer to my fork in the meantime!


Splatfacto: use z-depth for supervision.

I am working on using gaussian splat models with data directly from 3D software - such as blender, and there i am able to generate my own .ply pointclouds and all necessary passes in order to start training. The idea here is to be able to use gsplat as an alternative way to render out 3d models and display them easily on web.
I have managed to get really great results using my own pointclouds together with image-renders and masks. So the only thing missing now to get rid of any 'noisy' splats is to use the z-depth pass as training data as well.
(little demo of the results so far)
https://github.com/user-attachments/assets/5f18e4d0-7491-40b9-8157-8b58c53ce670

I will ofc help out with delivering the files needed in order to test with.

@michael-vedeler this should be very much doable. You can check out my project dn-splatter where I add depth (z-depth or euclidean depth) supervision as well as some other things to the base splatfacto method. Repo here


reset everything and restart person models

Pull Request Test Coverage Report for Build<PHONE_NUMBER>0
Details

91 of 93 (97.85%)  changed or added relevant lines in 15 files are covered.
7 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.7%) to 95.349%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

server/core/validators.py
0
1
0.0%

server/media/image/views.py
0
1
0.0%

Files with Coverage Reduction
New Missed Lines
%

server/core/validators.py
7
0.0%

Totals

Change from base Build<PHONE_NUMBER>0:
0.7%

Covered Lines:
1866

Relevant Lines:
1929

💛  - Coveralls


Octave - Easy: Factorial Calculation

Pull Request Type
What kind of change does this PR introduce?
[ ] Solved an issue/task
added factorial.m
Description: An Octave script defining a function to calculate the factorial of a given non-negative integer. The script includes an example usage to calculate the factorial of 5.

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your pull request has been merged. The files have been reset for the next contributor.
What's next?
If you're looking for more ways to contribute, I invite you to check out our forkcommitmerge.io website repository. The repository contain real issues that you can help to resolve or create some of your own if you see any bugs or want to add new features to the site, etc. You can also check out the Influences section in the README to find more projects similar to this one.
Also please leave a star to this project if you feel it helped you, i would really appreciate it.
I look forward to seeing your contributions!


[SPARK-19465][SQL] Added options for custom boolean values in CSV

What changes were proposed in this pull request?
It adds trueValue and falseValue options for customising the values used for Boolean types in CSV files. The default behaviour remains unchanged but the two new options allow for more flexibility. As it is currently the case, the values are matched ignoring their case.
How was this patch tested?
Extended existing unit tests for custom Boolean values and added a new test for asserting the IllegalArgumentException thrown by Scala's StringLike.toBoolean function.

Can one of the admins verify this patch?

Hm, do we really need this? I think we could simply work around this by case & when, if or other expressions with a projection after loading. Or using csv(ds: Dataset[String]) API after preprocessing in the source dataset.

That would remain a workaround though. The uniVocity parser for boolean supports this as well: https://github.com/uniVocity/univocity-parsers/blob/master/src/main/java/com/univocity/parsers/conversions/BooleanConversion.java. It's also been raised as a ticket against the original spark-csv so there seems to be at least some interest. Personally, I think this can help a lot when integrating Spark with other systems (i.e. some DB exports where modeling booleans as flags is common) similar to having ways of specifying the format of a date or null value.

Could you give some systems that have such a feature? Thanks!

Oracle doesn't have boolean so it's usually modelled as char(1) with Y/N, Sybase doesn't have boolean either but bit which is 1/0. PosgreSQL supports a range of values (https://www.postgresql.org/docs/10/static/datatype-boolean.html). In regards to CSV parsers I found the same feature in SuperCSV and uniVocity (linked above). I think it's a very useful and common feature similar to custom date formats or null strings, but as mentioned before, not something that can't be worked around.

Can one of the admins verify this patch?

This can be easily worked around, no?

This can be easily worked around, no?

It can be worked around, losing the boolean type by converting to string in the process though. With any workaround it'll be better to get eventually fixed/enhanced, especially as it was already raised against the original CSV datasource years ago. I did comment on this before and still think it's a valid improvement given this already exists for null, nan, inf and date values and the other reasons mentioned above.

I still don't think need this since the workaround is easy. If other committers find it worth, I won't object.
If there are no interests fro this PR afterwards, I would just close this.

I agree, just map the string field to a boolean field, translating as you like. There are hundreds of cases like this and they should just be handled by a one-liner transformation rather than build it into the CSV support.


ARM API to retrieve key breaks when enabling slots

Enabling slots sets AzureWebJobsSecretStorageType=Blob, which changes how secrets are managed. But the ARM API goes through Kudu, which does not honor this and returns an unusable key.
The main scenario that this breaks in Logic Apps, since they use that API. So effectively, slots cannot be enabled when using Logic Apps.
One option that was discussed is to make the ARM API go through the runtime instead of Kudu. The cons is that it won't work when the app is stopped, though that is I think a lesser evil.

Note: if an app gets enabled for slots and breaks as a result, the workaround is to go to the App Settings and remove AzureWebjobSecretStorageType=blob.

@lindydonna - this basically means that Logic Apps + Slots is broken. We need to probably update UI, docs, etc. to help prevent people tripping up on this bug.

@davidebbo suggests we also allow you to disable slots via the UI (it is currently 1 way)

Resolved by https://github.com/Azure/azure-functions-host/issues/3994.


Multi-graph scenes?

Is it currently possible to create scenes with multiple graphs at once (i.e. for demonstrating connections between f(x) and f'(x) vertically)? It seems like that would be helpful if not already possible. I can draw multiple FunctionGraphs but not have multiple axes and planes.

You can just change self.graph_origin and run self.setup_axes again

Hey @Li357, have you found a solution? I'm struggling on your very same problem!

How do you do this without using a GraphScene?

For future reference for anyone else who sees this, you can just shift and scale the graphs. Ex: my_axes.scale(0.4).shift(UP * 2)


Unable to query default group name from API

I can't seem to find a way to query the default group name from API. I know the default name is default but if the server renames this to another name then I have no way of knowing what the group is.

The server is unable to rename it unless they hardcode the change - which I don't think the API accounts for. If they try, LP creates a new default group.

Ah yes, I see the wiki states display name here
https://github.com/lucko/LuckPerms/wiki/Default-Groups#keep-using-the-default-group-but-just-modify-the-display-name
So this is a non-issue.
Thanks!

No worries!


Parameter check failed at: ../builder/Network.cpp::addConcatenation::162, condition: (inputs[j]) != nullptr

Begin parsing model...
ERROR: Parameter check failed at: ../builder/Network.cpp::addConcatenation::162, condition: (inputs[j]) != nullptr
Segmentation fault (core dumped)

I am not sure what is the issue about the parsing.
Can you tell me what the model and prototxt you used? It seems that your input caffemodel does not match.


Does not handle comments in code snippets

As the title describes, if there are comments in a code snippet, it treats them as headers incorrect when converting to html (markdown.markdown(text))
...
# this is a comment
...

It will treat # this is a comment as a header as opposed to just a comment

Have you enabled the fenced_code extension? As a reminder, the original Syntax rules do not include any provision for fenced code blocks. Therefore, they are not enabled by default.

Thanks, that solves this for me.


computer crashes when opening and running second interactive pane on different file

Environment data

VS Code version: jan 2021
OS and version: windows 10
Python version : 3.8

Expected behaviour
open file run cells in interactive pane
open second file and run cells in interactive pane
Actual behaviour
on running cells in second file while first pane is still open
second pane opens but then computer crashes with blue screen

I found if I open an empty interactive pane first, I can open multiple files after that without crashing.


vcpkg is downloading the incorrect version of a package

vcpkg is installing an old version of the "crow" package. According to vcpkg.io the current version is crow | Version: 0.3.2 however the vcpkg command keeps installing 0.1-1. I've tried clearing the cache and download directories however this does not help.

OS: Windows 10

Steps to reproduce the behavior:

./vcpkg install crow

vcpkg should be installing version listed on vcpkg.io, however it is pulling a much older version from somewhere.

git pull in your vcpkg folder

Yes I can confirm this fixed the error.

Good to know.
PS E:\vcpkg\clean\vcpkg> ./vcpkg install crow:x64-windows
Computing installation plan...
The following packages will be built and installed:
    crow[core]:x64-windows -> 0.3.2#1
Detecting compiler hash for triplet x64-windows...


New Logo Proposal

This is the logo proposal I submitted for your application, sir. Tell me how you think sir. Thank you.
https://steemit.com/utopian-io/@iqbalridha/logo-design-proposal-for-signature-spoofing-checker-app

It’s here: https://github.com/Lanchon/sigspoof-checker

Thank you sir, I have checked it was wrong, I will replace it


Navigation arrows not shown

Describe the bug
Left and Right arrows icon shown on some android devices but refuse to show on other devices, its shows "n" and "ni" instead of "←" and "➝"
Expected behavior
It should show  "←" and "➝" correctly
Steps to reproduce OR link to code

<ContentPage.Content>

<xcViews:CalendarView  />

Xamarin Forms or .NET MAUI (If related to UI)
XF
Additional context (Optional)
Device Info (Optional)
Device Model:
Android Version:
IOS Version:
Windows Version:

Well which Android devices doesn't it show the arrows?
I would suspect older ones because the arrows are just Unicode characters.
I don't really know if there is anything I can do about that.

Android version 10

Do the arrows show up when you put "←" and "→" in a Label outside the Calendar?


Non-modifiable Selected Items in Dropdown

Subject
Non-modifiable Selected Items in Dropdown
Description
Dear @AbdullahChauhan,
I have created a dropdown using the multiSelectSearchRequest() factory method and encountered an issue. The problem is that I am unable to modify the selected items programmatically.
It is a common use case to allow users to clear all the selected items through an icon button or perform other modifications to the selection. However, with the current implementation, it seems that the selected items cannot be modified once they are set.
Steps to Reproduce

Create a dropdown using the multiSelectSearchRequest() factory method.
Select one or more items in the dropdown.
Attempt to modify the selected items programmatically (e.g., clear the selection).

Expected Behaviour

I expect to be able to modify the selected items programmatically, such as clearing the selection or updating it based on certain conditions.

Actual Behaviour

The selected items cannot be modified programmatically. Any attempts to clear or update the selection have no effect.

Reproduction Code

Supporting Material
The solution to this problem is pretty straight-forward 😊. You simply have to add a property for selected item(s) in your constructors and factory methods.
Please let me know if you need any further information or clarification. Any assistance or guidance on how to resolve this issue would be greatly appreciated. 😇
Thank you for your time and consideration. 🙏
Best regards,
Parsa Zeinali

Hi @ParsaZa79
Thanks very much for the details you've provided above. Controllers support are now available in release 3.1.0. You can check. Thanks again 😊


Incorrect text

the Multiple groups and multiple licenses section describes 2 licensing groups: e3 base services and e3 extended services which are not visible on the screen shot. please correct the text or the screenshot as appropriate

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

ID: e7e4fdb9-a3d4-e97b-de30-9904868b87a5
Version Independent ID: fbade46e-55b3-3c50-9cff-6ec88ee5ee6e
Content: Group-based licensing additional scenarios - Azure AD
Content Source: articles/active-directory/users-groups-roles/licensing-group-advanced.md
Service: active-directory
Sub-service: users-groups-roles
GitHub Login: @curtand
Microsoft Alias: curtand

@DeanGross Thanks for the feedback !  I have assigned this issue to content author to investigate and update the document as appropriate.

Hi @DeanGross
This is fixed in PR https://github.com/MicrosoftDocs/azure-docs-pr/pull/157422. the sacreen =was out of date and not needed to make the point, so I removed it.
#please-close


fix: bug with operator check on optional session and external id

fix a bug of a bug, originally tried to fix this and used the wrong operator.

:tada: This PR is included in version 3.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Test store dependency binding

This makes it so that the test context is set when constructing the TestStore and in the modify closures of send/receive, and provides a trailing closure initializer for TestStore for customizing dependencies.
This idea came from @crayment in https://github.com/pointfreeco/swift-composable-architecture/discussions/1613.

Love it, thanks!!

I just realized receive should probably be updated as well to have the test stores dependencies for updateExpectingResult. Totally overlooked that in my first discussion.

@crayment That should be the case by virtue of the fact that we updated the private function expectedStateShouldMatch, and both send and receive make use of it. Are you seeing otherwise?
We probably should update the test to get coverage on this fact though.

I haven't verified on this branch yet. Still using my helpers. If you give me a bit I can verify, and maybe even PR an update to the test 😄

I did just verify locally with a test and it seems OK. But yes please feel free to PR a test to get coverage.

Done: #1644
Feel free to just close if you like your test updates better though!


Error: Couldn’t communicate with a helper application.

Hello I'm facing [Error: Couldn’t communicate with a helper application.] when I fetch from RNIap.getProducts
Please help with the solution

I have the same error...

@manthanpl93 I've solved this issue. Please re-check the SKUs which you provide for getProducts() function. I had SKU with null value. I hope it will help you.


Added ability to set the keystore\truststore type

#436 Added ability to set the keystore\truststore type using either properties or setters
AMQP-626

@habelson Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

Just a note, I didn't know what version to use for the @since markings on the new methods, so I left them as "?" thinking someone who knew better could update them.
Additional note, I made sure to default the values to the previous hardcoded values so that no existing code is broken.

You can find the version in the JIRA.
So, it is 1.6.2 for you here.
Thank you for contribution!
I'll take a look today, but would like to see some test-cases as well on the matter.
Just in case to cover as much code as possible.

@habelson Thank you for signing the Contributor License Agreement!

Understood, I will work on some test cases a bit later today

M-m-m. Did you make some reformatting in that class?
Did you change our tabs indents to whitespaces?

@artembilan Not that I'm aware of. the only lines I changed were the ones I added. They passed the style checks locally. Possible eclipse made changes on its own?

Possible eclipse made changes on its own?

Looks like. Or your Git client.
Try to figure out that issue and revert to tabs, please.

Great! I see you have fixed an issue with formatting.
Now you even can add your name to the @author list.
But some test is an obligation :smile:

@artembilan I think it should be good to go now

@habelson , looks like not yet.
See changes here on GH.
I think you have a problem with line separator. Or anything else, because we have a changes problem even in the ssl.properties.
Check, please, and let us know what's going on with your settings.

I still do not know what is wrong with my formatting settings. I imported the xml file into eclipse as per instructions. The only way I was able to correct the issue was to revert back to an old version of the file and retype the changes.
My first thought, when it happened the first time, is that maybe my git client did something weird when I stashed my changes to move it to a branch. But I did not do any stashing when I made the test changes and it still happened. So it is still somewhat of a mystery. It seems to only occur to files I've touched, as other files in the project diff fine with what is on github.

Merged as https://github.com/spring-projects/spring-amqp/commit/208775dd2e9cb0405d2aa6e6331ff19365ce38c2
Thank you for the contribution.
Looking forward for more!


DeveloperGuide.adoc updates for CS2101

Coverage remained the same at 90.663% when pulling 32f9090832cdc9be64210cdcb8e2ebebf843f9b6 on iMarbles:docs into 6da5a8588f329c84b084dfae028db37f32bc4006 on CS2103-AY1819S1-W16-2:master.


[WIP] Bugfix/ get true PATH variable

This PR should address #2251 and covers a little though not a big change in itself.

It seems running shellEnv.sync() has been failing silently as import('shell-env') returns an object like { default: Function } so calling sync on it fails but the way the try catch was structured led to a silent failure - ~I've switched the call to a simple require as shell-env is a tiny library and not sure we gain much by async importing it (guesstimation) also the behaviour between, webpack and typescript of async importing seems to be in flux a little so puts this import at risk.~ I've imported this as shellSynv.default.sync which is possibly brittle or maybe what is now the standard way that module.exports map to dynamic imports 🤷♂.

secondly I use a similar pattern as https://github.com/sindresorhus/fix-path to correct the path inconsistency on mac, not sure if this is definitely mac only but it is definitely an issue on mac - so on mac we set the path var to shell envs path
NOTE: since we had the heisenbug due to process constantly being reassigned the way that this works is as a one off call in the constructor of the shellEnvFetcher - this means that the packaged version of Oni has access to the users actual path variable.

This fixes issues I noticed for example trying to run a language server that depends of environment variables like the PATH location of a users goroot or cargo/bin for go and rust respectively.
Loose Ends

A question I have is that we seem to dynamically pass in the path but the variable seems like something that can be static unless we are accounting for the edge case where a user modifies the additionPaths var in their config, as even in terminal if I update my PATH i would have to restart the shell.

One thing that would be helpful here is to create a test case to document this behavior - it seems like we've had issues with this for some time. So having a test case can help us make sure we keep this behavior working, since respecting the path is important.
We already have the capability to pass in custom stuff with the env option in the test settings:
https://github.com/onivim/oni/blob/42840530d89f498f5bd55046573eaa448dfc16d5/test/common/runInProcTest.ts#L40
So what I'm thinking would be useful is a test case where we override PATH, and then run echo $PATH in neovim, and verify the output is as expected. This would be great to help make sure the functionality stays safe, regardless of the fix we take (and as a side benefit, it might help clarify the exact cases where we run into problems - so we can evaluate where in the pipe the fix needs to be)

A question I have is that we seem to dynamically pass in the path but the variable seems like something that can be static unless we are accounting for the edge case where a user modifies the additionPaths var in their config, as even in terminal if I update my PATH i would have to restart the shell.

Yes, most applications don't expect / handle the environment variable to change. They get a set of environment variables when the process starts, and use those. But ya, we don't expect Oni to have that full context (which is why we need to ask the shell for the path), so then we can pass the full-shell path to the process when it starts.
Oni itself shouldn't care about the PATH so much, since it's the language servers / neovim / etc that care about it - so we should make sure to send them the proper, expected path, but it shouldn't be necessary to change Oni's process.env.PATH at runtime - sending the PATH to the spawned processes should have the same effect.

@bryphe I've removed the changes as requested (leaving in the fix to the import which is definitely broken), regarding the test if I were to override the path in the test wouldn't that impact neovim's startup i.e. not sure if any particular variables are required that my overwriting might remove, also if any variables are added to the path maybe by neovim on startup the test could end up being flaky (I'm vaguely aware of neovim setting some env vars on startup)


word error update and small cleanup

Updated WER calculation

Is the current implementation not incorrect?

How so?

1). Reference is your x, Hypothesis/Prediction is your y
2). The for loop is unnecessary and it causes your denominator to be wrong.
Ex:
ref = "The dog is under the table"
hypothesis = "The dog is the fable"
Total words here is 16 with the current implementation should be 6 so WER should be .33 but the output is .5

Code is a simplified version from here: https://github.com/mlcommons/inference/blob/890af6b844cdc3d2e3053f51a40df20041b3852e/speech_recognition/rnnt/pytorch/metrics.py#L40

Ah okay, I think how it's being used in model_eval threw me off. Seems redundant.

Ah I'm dumb the input is not what I thought it would be. Anyway, if I have time, I think its usage in model eval can be simplified.  Thanks.


Is this connector capable of writing to BigQuery?

The doc seemly implies that the connector can only be used for reading from BigQuery, cannot be used to write to BigQuery, hope we clarify it in the doc. It will help users make correct choice between this connector and the other one.

https://github.com/GoogleCloudPlatform/spark-bigquery-connector/pull/54


#35 issue: rearranged  buttons and added hover effect

Summary

This Pull request Rearrange the Try a live demo and  Install caMicroscope  and some hover effect on this buttons.
Before:

After
https://github.com/camicroscope/camicroscope.github.io/assets/95683323/7fc81896-88bd-4208-ae84-603256d51faa

Fixes #35


Upgrade to Android X

Getting
Cannot drop Jetifier due to following external dependencies:

com.optimizely.ab:android-sdk:3.6.0

with the plugin described here: https://adambennett.dev/2020/08/disabling-jetifier/

Optimizely is also the only SDK in our app not migrated to androidX yet. Would love to see the SDK migrated so that we can drop Jetifier.

This is still an issue in the latest 3.7.0 SDK:
Cannot drop Jetifier due to following external dependencies:

* com.optimizely.ab:android-sdk:3.7.0

Would love to understand if there is a timeline here, suffering the same issue!

Only Lib preventing us to drop Jetifier too

Any update on this?

Any update on when a release will include this? I see the change was merged about 2 weeks ago - I understand there may be additional changes you wish to release simultaneously, but it would be great to push a release so consumers can migrate off Jetifier

Any update on when a release will include this? I see the change was merged about 2 weeks ago - I understand there may be additional changes you wish to release simultaneously, but it would be great to push a release so consumers can migrate off Jetifier

@kevinzetterstrom should be going out in the next couple of weeks.

@kevinzetterstrom should be going out in the next couple of weeks.

@kevinzetterstrom 3.9.0 has been released which includes update to androidX.  I'm closing this issue.


Multiple mappings with same URL, different query parameters

I am using version 2.9.2 and am unable to get Springfox/Swagger UI to generate correct API documentation for each of my endpoints.
My endpoints have the same URL but different query parameters, like the following:
@GetMapping(value = "/profiles", params = { "serverId" })
public List<String> getProfiles(@RequestParam("serverId") String serverId)

@GetMapping(value = "/profiles", params = { "serverId", "profileName" })
public List<String> getProfilesByName(@RequestParam("serverId") String serverId, @RequestParam("profileName") String profileName)

@GetMapping(value = "/profiles", params = { "serverId", "profileType" })
public List<String> getProfilesByType(@RequestParam("serverId") String serverId, @RequestParam("profileType") String profileType)

With this, the Swagger UI page shows only one of the endpoints. Do I need to configure something special?

Thanks for the response. You mention a lower version of swagger-ui is required for URL templating: was support for it removed? Or broken with some other upgrade in newer version? I'd like to try and help out on this if possible--just need a nudge in the right direction. Thanks!

@l3ender there was experimental support in the old swagger-ui. There hasn't been an opportunity to port that to the new version of swagger-ui. It still works minus a few bugfixes in swagger-ui with swagger 2.0. It will not work with OpenApi Spec 2.0
It would be amazing if you could take a look at that if you're familiar with react.


[qt5-3d] Support Visual Studio 2022

Describe the pull request
The error shows related to the qt5-3d module when I'm compiling COLMAP 3.6 by vcpkg on Windows 10 with Visual Studio 2022 installed. After tracing the logs, I realized that only assimp-vc143-mt.lib available in VCPKG_ROOT\installed\x64-windows\lib

What does your PR fix?
N/A

Which triplets are supported/not supported? Have you updated the CI baseline?
No

Does your PR follow the maintainer guide?

Yes

Host Environment
OS: Windows 10
Compiler: Visual Studio 2022
To Reproduce the error
Steps to reproduce the behaviour:
vcpkg install qt5-3d:x64-windows

@Neumann-A Does the qt upstream support Visual Studio 2022 now?

@JackBoosY  I'm so sorry about the versioning; this is my first time here. Thank you!

Does the qt upstream support Visual Studio 2022 now?

no but that is not the question here.
Does assimp install pc files which could be used instead of hardcoding the library names?

Does the qt upstream support Visual Studio 2022 now?

no but that is not the question here.
Does assimp install pc files which could be used instead of hardcoding the library names?

Yes assimp has.

LGTM, thanks for this PR!


Filling PIXI.Text with fill:Number does not work

Referring to this example from the sources:
var text = new PIXI.Text('This is a pixi text',{font : '24px Arial', fill : 0xff1010, align : 'center'});

which renders black text.

Sorry, duplicate of https://github.com/GoodBoyDigital/pixi.js/issues/1745


Organization Cleanup

So we are going to convert a lot of our old repositories to private or remove it maybe.

[ ] #91

closing this


add environment file for binderized examples

[x] Closes #3563
[ ] Tests added
[ ] Passes black . && mypy . && flake8
[ ] Fully documented, including whats-new.rst for all changes and api.rst for new API

yay thanks joe!


Include name of test in log when test times out

It can be difficult to tell which test has timed out by looking at the logs since the log is hardcoded as test timed out after.... This PR includes the test name in the log to make it more explicit what timed out.

This looks great, thanks!


WiX: package up DocC conditionally

Add support to package up DocC in the windows distribution.  This is the last large component that was missing.  At this point, the toolchain distribution is generally speaking complete with:

Preprocessor (clang)
Assembler (IAS)
C/C++/Swift Compiler (clang/clang++/swiftc)
Linker (lld)
Binary Utilities (LLVM)
Debugger (LLDB)
REPL
Package Manager (SwiftPM)
Runtime Inspection Tools (swift-inspect)
LSP (sourcekit-lsp)
Code Formatter (swift-format)
Documentation Tools (DocC)
Additional IDE integration Tools (lldb-vscode)
Redistributables (MSMs)
Runtime (host-only)
SDKs (AMD64, ARM64, X86)

CC: @barnson - I'm extending the harvesting a fair amount - not sure if there is a better way to include a web application

Tested at https://ci-external.swift.org/job/swift-PR-build-toolchain-windows/1027/artifact/build/artifacts/


[requires.io] dependency update on master branch

Coverage remained the same at 98.187% when pulling ac2427d60800152962f938ad2a0627e9fe5f0312 on requires-io-master into fc0deaa803578afdc440f66c58019d406036df4b on master.

Coverage remained the same at 98.187% when pulling 27b18c5d27cc1f9ad2d41c10d481a1dd9fceb389 on requires-io-master into fc0deaa803578afdc440f66c58019d406036df4b on master.

Coverage remained the same at 98.187% when pulling 31b115418f9373b1d91f8e61d0a7169c7b523049 on requires-io-master into fc0deaa803578afdc440f66c58019d406036df4b on master.

Coverage remained the same at 98.187% when pulling 31b115418f9373b1d91f8e61d0a7169c7b523049 on requires-io-master into fc0deaa803578afdc440f66c58019d406036df4b on master.

Coverage remained the same at 98.187% when pulling 39afbc817d09baec4dc93ebd642c728f571a6a05 on requires-io-master into fc0deaa803578afdc440f66c58019d406036df4b on master.

Coverage remained the same at 98.187% when pulling 43d241fd30dabbf1fdbd3655c2ea1c38a5169f8e on requires-io-master into fc0deaa803578afdc440f66c58019d406036df4b on master.

Coverage remained the same at 98.187% when pulling ddd96ea2975464760bd3635c518161f3cd46d1f1 on requires-io-master into fc0deaa803578afdc440f66c58019d406036df4b on master.

Coverage remained the same at 98.187% when pulling 1a4f4753dda1ec79695b138413da3dbf060b4bef on requires-io-master into fc0deaa803578afdc440f66c58019d406036df4b on master.

Coverage remained the same at 98.187% when pulling e11acdf4daca429d79d9afe80133e77a8824f1d7 on requires-io-master into fc0deaa803578afdc440f66c58019d406036df4b on master.

Coverage remained the same at 98.187% when pulling c604c38794bb20f2edcaf8bb239e800fe2b042b1 on requires-io-master into fc0deaa803578afdc440f66c58019d406036df4b on master.


DAT Header Mismatch

I get 2 issues when using this program.
The first is when saving a configuration:
"An error has occurred: Failed to save region settings. Reasons:"
When attempting to do an audit I get an error about "DAT header mismatch". Could someone explain the meaning of this error? Does the DAT or archive need to be named in a particular way?

Thanks for bringing this up. Apparently there was an obvious syntax error that broke saving region settings. I don't know how I didn't catch it earlier, but I just pushed a new version 1.01 with a fix.
As for the second issue: The DAT file name only matters for automatic detection within the optional "Input No-Intro DAT Directory"; you can also supply a file manually on a per-system basis.
Aside from that, the DAT file should be Parent/Clone (see below), and the header name within the file needs to match the system name. Something like:
<datafile>
	<header>
		<name>Sega - Mega Drive - Genesis (Parent-Clone)</name>
		[other things go here]
	</header>
<datafile>

If and only if the  field starts with the correct system name (in this case, "Sega - Mega Drive - Genesis"), then it's considered a match. Parent-Clone DAT files obtained from No Intro's DAT-o-MATIC should work without needing to rename or modify them (as of v1.01), but let me know if there's a discrepancy.
DAT-o-MATIC -> Download -> Daily -> P/C XML -> Prepare


Web Music Player

WebMasterLog Repository (Proposing new issue)
:red_circle: Project Title : Web music player
:red_circle: Aim : spotify clone
:red_circle: Tech Stack Used : HTML, CSS, core JS
:red_circle: Category :  Vanilla-JS-Projects
Screenshots 📷

:white_check_mark: To be Mentioned while taking the issue :

Full name :  Anshul Soni
GitHub Profile Link : anshulsoni4
What is your participant role? : GSSOC

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

@anshulsoni4 Assigned to you. Follow the contributing guidelines carefully.


Prey (2017) not detected

Ludusavi version
v0.21.0
Operating system
Windows
Installation method
Standalone
Description
Ludusavi doesn't seem to detect Prey (2017) for some reason. I have vague recollection of seeing it on the list in earlier version of the app (didn't do the backup at that time) but I'm not 100% positive so it could be my imagination. Manifest for the game seems to be ok from cursory glance.
Tried CLI with debugging:
ludusavi.exe backup --preview "Prey (2017)"
log result (cropped)
[2023-08-23T19:39:48.304Z] DEBUG [ludusavi::cli] Invocation: Backup { preview: true, path: None, force: false, merge: false, no_merge: false, update: false, try_update: false, wine_prefix: None, api: false, sort: None, format: None, compression: None, compression_level: None, full_limit: None, differential_limit: None, cloud_sync: false, no_cloud_sync: false, games: ["Prey (2017)"] }
[2023-08-23T19:39:48.886Z] DEBUG [ludusavi::scan::launchers::generic] ranking installations for Steam: G:/SteamGames
[2023-08-23T19:39:48.886Z] DEBUG [ludusavi::scan::launchers::generic] actual install folders: 3DMark | AER | Akane | atomzombiesmasher | Blood | Brigador | Control | Core Keeper | Crush Your Enemies | DeathComing | Desktop Dungeons | Dorfromantik | Dungeon and Puzzles | Fates of Ort | Hades | Halo The Master Chief Collection | Hero's Hour | Heroes of Hammerwatch | HyperLightDrifter | LegendofDungeon | Loop Hero | Lust for Darkness | Mainframe Defenders Test | Moonlighter | MudRunner | Ori and the Will of the Wisps | Pixplode | Plants Vs Zombies | PortalRTX | Prey | Return 2 Games | Revenge of the Titans | Roadwarden | Spyro Reignited Trilogy | Stellaris | Subnautica | Supraland | Tyranny | Waves | Yes, Your Grace | Z
[2023-08-23T19:39:48.886Z] DEBUG [ludusavi::scan::launchers::generic] [Prey (2017)] selecting subdir with score<PHONE_NUMBER>854775807: Prey
[2023-08-23T19:39:48.886Z] DEBUG [ludusavi::scan::launchers::generic] ranking installations for Steam: C:/Program Files (x86)\Steam
[2023-08-23T19:39:48.887Z] DEBUG [ludusavi::scan::launchers::generic] actual install folders: Steam Controller Configs | Steamworks Shared
[2023-08-23T19:39:48.887Z] INFO [ludusavi::cli] beginning backup with 1 steps
[2023-08-23T19:39:48.892Z] INFO [ludusavi::cli] completed backup

So the app seem to correctly identify game folder but then does nothing:
Overall:
  Games: 0
  Size: 0 B

Logs
ludusavi_rCURRENT.log

Based on the PCGamingWiki article, the save data should be under %USERPROFILE%\Saved Games\Arkane Studios\Prey (or Prey_Epic or Prey_MS), not in the game install folder. Could you check if you have that folder and what's in it?

Ok I think I found the issue. I think there is inconsistency how special folders (documents, saved games) are detected. I moved Saved Games folder using move option from folder properties (it can be used for special folders like documents, saves etc). to H:\Home\Documents\Saved Games. Documents folder was also moved using this method. In explorer both show normally in in my %USERPROFILE%, which is why I didn't notice it initially, but using command line both are missing so I assume explorer is doing some magic behind the scenes to show it in original location.
But, there is other game in that folder that is detected: "Hades"
So Hades saves location from pcgaming wiki: "%USERPROFILE%\Documents\Saved Games\Hades\" and it is detected
while Prey saves location is %USERPROFILE%\Saved Games\Arkane Studios\Prey and is not.
Hope that helps

Gotcha, that does help.
Ludusavi uses the Windows API to get the correct %USERPROFILE% and %USERPROFILE%\Documents folders (which is why Hades works even though the documents folder was moved), but not for %USERPROFILE%\Saved Games, which I didn't realize was a special folder. I can look into adding some handling for that.

Yes, I think Saved Games special folder is relatively recent addition. Many games, like Hades, just use  %USERPROFILE%\Documents and then append Saved Games to the path, which is completely different folder in default installation. It's just happens that I merged both, making it bit more confusing (to debug).

Is this still an open issue?
Ludusavi seems to properly detect it on my Linux machine (using proton -> using Windows version of Prey).

Yeah, this is still an issue, but it should only on Windows itself. Wine prefixes should use the standard name/location, but on Windows, you can move the folder and query the new location from the OS.


Update composer install to use programmatic install

The current composer install in the Dockerfile has hard coded the hash. This changes every time composer updates.
Instead, I've updated to make it use the programmatic install which downloads the hash on the fly.
https://getcomposer.org/doc/faqs/how-to-install-composer-programmatically.md

Good works @zorfling! Thanks!


[BUG]: Admin | Sync Template v2 (Flows) runs never ending....

Describe the bug
Admin | Sync Template v2 (Flows) runs never ending....some runs is 6d old...and it's still running
I put a screen shot where the flow "jam"
Component (please tell us which flow or app you are experiencing issues with):

Solution Core
Version 1.70.3
Flow: Admin | Sync Template v2 (Flows)

To Reproduce
Steps to reproduce the behavior:

Start the sync

Expected behavior
My flows data uptodate and the flows was ending
Screenshots

And the action who seems to "block"

Additional context
I do a copy of the flow, and I got the same issue.

Hello. This is a known issue and fixed in 70_6. You will need to upgrade and then please ensure you see the following in the flow to ensure upgrade worked correctly
Duplicate of #497