add ui5.table tests for latest ui5 version

Table column headers in newer ui5 versions don't have the tooltip. We don't use tooltip value in the selectors in ui5.table, so ui5.table.sortColumnAscending and ui5.table.sortColumnDescending work fine with newer ui5 versions.
Current tests use ui5 version 1.99.0, and have column header selectors with tooltip value.
Since newer ui5 versions don't have tooltips for column headers, added tests for new ui5, with proper selectors.

Fix for sorting is in PR - https://github.com/SAP/wdio-qmate-service/pull/100
Don't require these additional tests for ui5 > 1.99, since they don't add anything.


Provide an alternative to DynamoDb's Object Persistence Model attributes similarly to EF Core's Fluent API

Expected Behavior

Using DynamoDb's Object Persistence Model, I would like to configure my entities similarly to how I would do it with EF Core's Fluent API. For example (taken from here):
protected override void OnModelCreating(ModelBuilder modelBuilder)
{
    //Write Fluent API configurations here

    //Property Configurations
    modelBuilder.Entity<Student>()
            .Property(s => s.StudentId)
            .HasColumnName("Id")
            .HasDefaultValue(0)
            .IsRequired();
}

Current Behavior

On its current state, the Object Persistence Model only allows you to specify that configuration via attributes. This implies the project containing your entities should also contain a reference to AWSSDK.DynamoDBv2.
With a fluent API (or equivalent), the same configuration could be easily added on top of the entities from another project, and passed when the DynamoDBContext are created.
Context

I am trying to achieve a typical onion architecture with three layers: a business layer, an infrastructure layer and a presentation layer. The lack of such API is forcing me to find a workaround, such as:

Combine the business and infrastructure layers into a single one (not ideal)
Add a reference to AWSSDK.DynamoDBv2 to the business layer and define the attributes directly there (a bit better solution, but still not ideal)
Create intermediate entity classes inside the infrastructure layer which contain those attributes instead, and use AutoMapper to convert between the initial / annotated entities (this adds a lot of boilerplate)

The lack of fluent entity builder is not an issue in the AWS SDK for .NET, but the presence of such would be a new feature. The convenience of not using attributes is that you can separate your business or service layer from your data layer as well as you can dynamically use different DynamoDB tables/indexes in the different environments(dev, stage, test, production). Desiring that behavior I created my own simple fluent API for access DynamoDB. You can see some examples here. It's available as a nuget package. It's using is very similar to Entity Framework Core's using.

Your library seems very promising and in the exact direction I wanted to go. I will definitely try it!

Thanks for that @msotiroff.  We haven't done a full review of your library, but it does look useful upon first glance.


[R000592] Rep. Todd Rokita

Rep. Todd Rokita:
http://rokita.house.gov
Likely contact form URL to start with:
http://rokita.house.gov/contact-me/email-me

YAML file lost. I am working on it.


Manageorder issue fix

JIRA link (if applicable)
Change description
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x] No

no longer needed. closing


reduce max calls to $.browser from 3 to 1

http://jsperf.com/quicky

That is micro-optimisation at its best.

…though Firefox seems to be throwing up surprising results!?

5.01 million (3 calls) vs 5.2 million (1 call).
I agree this is pretty insignificant performance-wise.  But if the work is already done, and it is a performance gain without obfuscation, seems like a good idea.


Feat: Add playfair cipher

Description of Change

This pull request adds the implementation of the Playfair Cipher algorithm. The Playfair cipher is a digraph substitution cipher, which encrypts pairs of letters from the plaintext using a 5x5 grid. It provides both encryption and decryption functions, handling edge cases such as repeated letters and handling of the 'j' character by replacing it with 'i'.
Key Features:

Implementation of the encryption and decryption methods.
A utility to clean input strings by removing non-alphabet characters and converting the text to lowercase.
Generation of the Playfair 5x5 grid based on a provided key.
Supports formatted digraphs and insertion of 'x' for repeated characters and padding.

Checklist

[x] Added description of change
[x] Added file name matches File name guidelines
[x] Added tests and example, test must pass
[x] Added documentation so that the program is self-explanatory and educational - Doxygen guidelines
[x] Relevant documentation/comments is changed or added
[x] PR title follows semantic commit guidelines
[x] Search previous suggestions before making a new one, as yours may be a duplicate.
[x] I acknowledge that all my contributions will be made under the project's license.
Notes:
This pull request includes a detailed file header using Doxygen-style comments, an example of usage with both encryption and decryption of sample text, and test cases to ensure the correctness of the algorithm.

are there any additional requirments for this PR?

Have put in all the necessary changes


Use repository deploy key to push new tags in cd-workflow.yml

Currently the RELEASE pipeline is not triggered when a workflow file pushes a new versioned release. It turns out this limitation is provided by design: An action in a workflow run can’t trigger a new workflow run. (See: https://github.com/orgs/community/discussions/27028)
A fix to this problem is to use your Personal Access Token (PAT) or use Deploy Keys. Deploy Keys are much safer / access controlled as they're repository scoped: You can launch projects from a repository on GitHub.com to your server by using a deploy key, which is an SSH key that grants access to a single repository.
Therefore, this ticket is to create a Deploy Key to the propactive project to use for pushing new tags which should then resolve the RELEASE pipeline trigger issue.  You would then use it as follow:

Add the private key as a Secret in your repository secrets (i.e. DEPLOY_KEY)
Authenticate Git operations using the Deploy Key

- uses: actions/checkout@v2
  with:
    ssh-key: "${{ secrets.DEPLOY_KEY }}"

Now, any Git operation performed by your Workflow – e.g: a Push – will trigger Workflows, because GitHub does not restrict Deploy Keys. For further details please read:
https://docs.github.com/en/developers/overview/managing-deploy-keys#deploy-keys

CHANGES

Created/Added RELEASE_DEPLOY_KEY to secrets


Update README.md

What does this PR do?
Updates README to :

include link for documentation site
replace certain words with more descriptive words .

Test Plan
Not applicable .
Related PRs and Issues
Fixes #68
Have you read the Contributing Guidelines on issues?
Yes

Sure. I don't feel like some of these are necessary, but also doesn't hurt :)

yeah, it is just the line Pink Design is built with Astro seemed kinda ambiguous to me,since pink design is design system so I decided to propose update 😅 .


Strange padding above keyboard on animated transition - new Compose release

Hello, it seems that the new compose updates caused a bug on animated transition - see https://issuetracker.google.com/issues/270656245
Here is a sample project to reproduce:
https://github.com/polivmi1/Bug270656245
To reproduce you have to:

use native: false view
click into edit text won't cause it and you first need to switch page at least once and then click into edit text.

When I use non-animation, it will not cause the bug:
val navController = rememberNavController()
val navHostEngine = rememberNavHostEngine()
So it seems like the issue might be from https://google.github.io/accompanist/navigation-animation/ . When using native compose navigation with accompanist animation, it doesn't seem to cause the issue.

https://github.com/polivmi1/Bug270656245/blob/master/app/src/Screenshot 2023-03-02 at 21.52.10.png this is the problem - see the padding above keyboard. Everything points to this library and new compose update because separately the other parts work.

Hi!
I cannot reproduce with the sent project.
Also, the two (with and without this lib) are not equivalent. You are not passing the default navigation animations into AnimatedNavHost.
The equivalent code in the native side is:
                                AnimatedNavHost(
                                    navController = navController,
                                    startDestination = "one",
                                    contentAlignment = Alignment.TopCenter,
                                    enterTransition = { fadeIn(animationSpec = tween(700)) },
                                    exitTransition = { fadeOut(animationSpec = tween(700)) }
                                ) {
                                    composable("one") { One(navController) }
                                    composable("two") { Two(navController) }
                                }

Notice also that defaultAnimationsForNestedNavGraph doesn't do anything here because you have no nested nav graphs. NavGraphs.root is the root, so it is not nested in any other graph :)

tryingToReproduce.webm
As you can see, I have native false and I navigate and then tap on the Text fields, everything is working here 🤔

@raamcosta on the video you shared it is also visible, but smaller (maybe a different emulator or phone - I am using API 33). You can see that if you scroll, the text fields are disappearing before they reach the keyboard area.
If I remove navHostContentAlignment = Alignment.TopCenter,
it will work correctly. (The defaultAnimationsForNestedNavGraph is there because I have it in different project where our UI tests discovered this and was trying to reduce that to minimum reproducible sample app)

Thanks for adding the native version. I was testing the transitions, but the problem was:
AnimatedNavHost(                                     navController = navController,                                     startDestination = "one",                                     contentAlignment = Alignment.TopCenter,                                 ) {                                     composable("one") { One(navController) }                                     composable("two") { Two(navController) }                                 }
So this seems like an accompanist bug. I will report there.

I will close this one then. Glad I could help 🙂

@raamcosta ok, but it is visible in your video. See the space above the keyboard. It is just a little smaller than on my screenshot.

Seems an accompanist issue.

Il reopen with dependency tag. That’s probably better 👍

We're not depending on accompanist on latest versions, hopefully it's working now!


docs(changelog): copy edits

prefer talking in terms of apps in third person rather than in terms of what "you" are using and "your" apps.
remove "Removed" section from pending unreleased changelog, since we're not using that section to document removing things. :smile:
wrap at 80 characters

Contributor License Agreement adherence:

[x] This Contribution is under the terms of Individual Contributor License Agreements (and also Corporate Contributor License Agreements to the extent applicable) appearing in the Apereo CLA roster.

Certainly writing in the third person (and especially with passive voice) can obfuscate rather than clarify.
I don't think I object to "you" generally. I objected to this usage specifically.
I don't think anthropomorphizing apps helps. So reading

If you are already using <frame-page>, you should get

as "you the app", well, that's goofy.
And reading it as "you the developer", well, you aren't using a directive, the app that you play some role in maintaining uses it, and it should get...
So could write this as something like

If the app you are maintaining is already using <frame-page>, it should get

I wouldn't object to that on "you" grounds, but I'd still prefer

<frame-page> usages should be unaffected.

for brevity.
Gotta tailor the documentation to the audience. This is BREAKING CHANGE documentation in a CHANGELOG. There's a premium on brevity, directness, and clarity on just what is and is not broken with the change. That's what I'm going for in that copy edit.


[Improvement-14464][Task] Remove the useless taskResultString

Purpose of the pull request

close: #14464

Brief change log
Remove the useless taskResultString
Verify this pull request

Codecov Report

Merging #14465 (90c1774) into dev (76b1eef) will decrease coverage by 0.04%.
The diff coverage is 0.00%.

:exclamation: Current head 90c1774 differs from pull request most recent head 16ad833. Consider uploading reports for the commit 16ad833 to get more accurate results

@@             Coverage Diff              @@
##                dev   #14465      +/-   ##
============================================
- Coverage     38.55%   38.52%   -0.04%
+ Complexity     4561     4552       -9
============================================
  Files          1237     1237
  Lines         43462    43421      -41
  Branches       4812     4782      -30
============================================
- Hits          16757    16727      -30
+ Misses        24847    24843       -4
+ Partials       1858     1851       -7

Impacted Files
Coverage Δ

...erver/master/runner/StreamTaskExecuteRunnable.java
0.00% <0.00%> (ø)

...er/runner/execute/TaskExecutionContextFactory.java
0.00% <0.00%> (ø)

...duler/plugin/task/api/AbstractCommandExecutor.java
0.00% <ø> (ø)

... and 41 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Add static database of questions and answers

Add questions and answers of multiple topic for the quiz.
Add this in json format.

The feature has been added.


Xcode 9.0.1 doesn't appear in xcversion list

Hi,
Xcode 9.0.1 was released but I don't get it when running xcversion list, what's the normal procedure when new Xcode versions are released? Any way I can help to update the list?

Have you first run xcversion update?

@timsutton I had no idea I had to do that. It works now, thanks!


#10099 ability to choose Index type via Index UI creation added

Please turn Other into Default and make it the default option


File::move now returns new file instance for relocated file. Fixes #1782

Change the return type of move() method in comment block


Add zita-dpl1, a digital peak limiter.

This package  cannot be merged yet: we need written permission to distribute it from the author.
I have sent him a mail and will report back here as soon as I have an answer.

I haven't heard from the author yet, but apparently he has not given any distro permission yet.
I'll close the issue... :(


feat: Save a data snapshot as a CSV file

Asana ticket: Ability to save measurement snapshots across a range of parameter values
Capture prediction result stats for the current state of a delay alert algorithm across a range of parameters defined by the algorithm itself.

@ErinLMoore I stumbled my way into this approach after a couple of false starts. I think it’s fairly reasonable in the end, but definitely let me know if you can think of better ways to approach any of this.

@ErinLMoore I stumbled my way into this approach after a couple of false starts. I think it’s fairly reasonable in the end, but definitely let me know if you can think of better ways to approach any of this.

I think I like the approach for now, I think I'll have to see what it's like with a second algorithm, whether it's easier to keep it like this or to give each algorithm their own csv functions (like maybe a csv behavior they implement or something). I think they're gonna be similar enough where this is fine, though.

@ErinLMoore Yep, I agree 100%. Let's see what it feels like with one or two more examples and if it turns out to be awkward I'm by no means wedded to it.


using db_and_auth template, can not register user - 401 connection refused

steps:

aqueduct create -t db_and_auth db_and_auth
create db 'dart_test' with 'dart':'dart' userid:password
aqueduct db generate
aqueduct db upgrade --connect postgres://dart:dart@localhost:5432/dart_test
aqueduct auth add-client --id com.local.test --secret mysecret  --connect postgres://dart:dart@localhost:5432/dart_test
curl -X POST -v http://localhost:8888/register -H 'Content-Type: application/json' -d '{"username":"bob", "password":"password"}'

curl log
curl -X POST -v http://localhost:8888/register -H 'Content-Type: application/json' -d '{"username":"bob", "password":"password"}'
Note: Unnecessary use of -X or --request, POST is already inferred.
*   Trying ::1...
* TCP_NODELAY set
* Connection failed
* connect to ::1 port 8888 failed: Connection refused
*   Trying <IP_ADDRESS>...
* TCP_NODELAY set
* Connected to localhost (<IP_ADDRESS>) port 8888 (#0)
> POST /register HTTP/1.1
> Host: localhost:8888
> User-Agent: curl/7.54.0
> Accept: */*
> Content-Type: application/json
> Content-Length: 41
>
* upload completely sent off: 41 out of 41 bytes
< HTTP/1.1 401 Unauthorized
< x-frame-options: SAMEORIGIN
< x-xss-protection: 1; mode=block
< x-content-type-options: nosniff
< content-length: 0
< server: aqueduct/1
<
* Connection #0 to host localhost left intact

Result:

user is not created
_authclient is created

Log:
[INFO] aqueduct: Server aqueduct/1 started.
[INFO] aqueduct: Server aqueduct/2 started.
[INFO] aqueduct: Server aqueduct/3 started.
[INFO] aqueduct: Server aqueduct/4 started.
Application started on port: 8888.
Use Ctrl-C (SIGINT) to stop running the application.
[INFO] aqueduct: POST /register 7ms 401

Attempt to fix
change channel.dart:
    /* Create an account */
    router
        .route("/register")
        //    .link(() => Authorizer.basic(authServer))   ////////////comment out this line
        .link(() => RegisterController(context, authServer));

Result User gets created
Then tried curl -v POST http://localhost:8888/auth/token -H 'Authorization: Basic Y29tLmhlcm9lcy50dXRvcmlhbDo=' -H 'Content-Type: application/x-www-formbob&password=password&grant_type=password'
Result
curl -v POST http://localhost:8888/auth/token -H 'Authorization: Basic Y29tLmhlcm9lcy50dXRvcmlhbDo=' -H 'Content-Type: application/x-www-form-urlencoded' -d 'username=bob&password=password&grant_type=password'
* Rebuilt URL to: POST/
* Could not resolve host: POST
* Closing connection 0
curl: (6) Could not resolve host: POST
*   Trying ::1...
* TCP_NODELAY set
* Connection failed
* connect to ::1 port 8888 failed: Connection refused
*   Trying <IP_ADDRESS>...
* TCP_NODELAY set
* Connected to localhost (<IP_ADDRESS>) port 8888 (#1)
> POST /auth/token HTTP/1.1
> Host: localhost:8888
> User-Agent: curl/7.54.0
> Accept: */*
> Authorization: Basic Y29tLmhlcm9lcy50dXRvcmlhbDo=
> Content-Type: application/x-www-form-urlencoded
> Content-Length: 50
>
* upload completely sent off: 50 out of 50 bytes
< HTTP/1.1 400 Bad Request
< content-type: application/json; charset=utf-8
< x-frame-options: SAMEORIGIN
< x-xss-protection: 1; mode=block
< x-content-type-options: nosniff
< content-length: 26
< server: aqueduct/1
<
* Connection #1 to host localhost left intact

im a newbie, did it have an example about scope in this tutorial ? ty


Installer assumes disk is called /dev/sda

Tectonic Version
1.4.7
Environment
libvirt
Expected Behavior
Installation of CoreOS to disk
Actual Behavior
After PXE booting the virtual machine successfully the progress stopped because the installer couldn't find /dev/sda.
Reproduction Steps
PXE boot libvirt machines using bootcfg.
Other Information
After replacing /dev/sda with /dev/vda in the bootcfg files in /var/lib/bootcfg, restarting bootcfg (not sure if that was necessary) and PXE booting again the installation proceeded as expected.

Hey @abh, thanks for filing this issue. Will look into this and get back to you shortly with some questions from the team.

Hey @abh, thanks for filing this issue! So today, we hardcode using the /dev/sda hard-drive when installing Tectonic on bare metal. Enabling configurability around this is something we're going to look into as we build out functionality for installing on virtual machines. I'll make a note to return to this issue when we further scope out that functionality!

@abh For now, substituting /dev/sda in the /var/lib/bootcfg/ignition/install-reboot.json file is the right way to go. This does not require bootcfg to be restarted.

I am having the same issue on xenserver, the installer is trying to install to /dev/sda where it should be /dev/xvda and silently fails. Journalctl on the machine gave me the hint. Trying the workaround now...

@tnsasse did the workaround work for you?

In the new terraform-based installer, it is best to just set the disk as a variable. You do not need to modify matchbox state. https://github.com/coreos/tectonic-installer/blob/master/platforms/metal/cl/coreos-install.yaml.tmpl#L24

@mfburnett changing the disk names in the templates worked for me.

@dghubble Thank you!

I'll point out the Matchbox Kubernetes example supports this customization. https://github.com/coreos/matchbox/tree/master/examples/terraform/bootkube-install#optional It may be valuable for Tectonic to support the same.

If you are running heterogeneous systems, e.g. a combo of metal and VM systems, then the install_disk variable needs to be on a per-node basis.
We've hacked up our installer to handle per-node configuration options as some need special serial console handle, amongst other things.
Here is part of the special core-metal.tfvars that we use:
core-metal_map = {
    "core-minion01" = {
        name = "core-minion01"
        int_vlan111_mac = "xx:xx...."
        int_vlan111_address = "x.x.x.x/x"
        is_minion = "true"
        is_virt = "true"
        install_disk = "/dev/vda"
        power_handling = "snapshot"
        sc_primary = "false"
        want_swap = "1G"
    }
...
}

Some keys like is_minion (or is_master)  are used in {{ if ... }}...{{end}} template blocks as we have tried to DRY our templates.  Other keys like power_handling is used during development so we can take snapshots between installation steps so we can just focus on the stage we are tweaking on subsequent runs.
We will use the above map as a lookup when setting the metadata:
resource "matchbox_group" "install-core-metal" {
    count = "${length(var.core-metal_map)}"
    ...
    metadata {
        ...
        install_disk = "${lookup(var.core-metal_map[element(keys(var.core-metal_map), count.index)], "install_disk", var.install_disk)}"
        ...
    }
}

Unfortunately, per-node becomes invasive pretty quick due to the nature of how terraform/HCL works as the existing resource "matchbox_group" "coreos_install" is pretty much a one-size fits all implementation.
If the tectonic-installer developers are open to a PR on this, I can put in the effort with reworking "coreos_install" to be per-node capable, with built-in defaults for unset keys.


Gradle buildscripts of Flutter plugins interfere with app project

I'm trying to modernize the Gradle buildscripts of the Android part of my Flutter app, but I'm encountering problems.
Use case
I want to get rid of the allprojects in my android/build.gradle, because it's an anti-pattern.
I don't want my app project to add repositories to all the other subprojects (i.e. Flutter plugins). And vice versa: I also don't want buildscripts of other Flutter plugins to add repositories to my app project (or generally, modify it in any way.)
A way to enforce this is to set RepositoriesMode.FAIL_ON_PROJECT_REPOS in the dependencyResolutionManagement block (see android/settings.gradle below). This is also known as Central repositories declaration. There are also other RepositoriesModes available.
Another (very experimental) way to block plugin projects from modifying the app project is Project Isolation.

In the experiment, the configuration model of Gradle projects are "isolated" from each other. This means that build logic, such as build scripts or plugins, applied to a project cannot directly access the configuration of another project.

Example (part 1)
I created a new blank Flutter app with flutter create example --platforms android --empty.
I proceed delete the following lines from android/build.gradle:
-allprojects {
-    repositories {
-        google()
-        mavenCentral()
-    }
-}

rootProject.buildDir = "../build"
subprojects {
    project.buildDir = "${rootProject.buildDir}/${project.name}"
subprojects {
  project.evaluationDependsOn(":app")
}

tasks.register("clean", Delete) {
    delete rootProject.buildDir
}

At this point, I expect my Android build to fail, becuase it won't be able to
resolve its dependencies. And so it does:

Failing build
$ flutter build apk --debug

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':app:checkDebugAarMetadata'.
> Could not resolve all files for configuration ':app:debugRuntimeClasspath'.
   > Could not find org.jetbrains.kotlin:kotlin-stdlib-jdk7:1.7.10.
     Searched in the following locations:
       - https://storage.googleapis.com/download.flutter.io/org/jetbrains/kotlin/kotlin-stdlib-jdk7/1.7.10/kotlin-stdlib-jdk7-1.7.10.pom
     If the artifact you are trying to retrieve can be found in the repository but without metadata in 'Maven POM' format, you need to adjust the 'metadataSources { ... }' of the repository declaration.
     Required by:
         project :app
   > Could not find androidx.lifecycle:lifecycle-common:2.2.0.
     Searched in the following locations:
       - https://storage.googleapis.com/download.flutter.io/androidx/lifecycle/lifecycle-common/2.2.0/lifecycle-common-2.2.0.pom
     If the artifact you are trying to retrieve can be found in the repository but without metadata in 'Maven POM' format, you need to adjust the 'metadataSources { ... }' of the repository declaration.
     Required by:
         project :app > io.flutter:flutter_embedding_debug:1.0.0-45f6e009110df4f34ec2cf99f63cf73b71b7a420
   > Could not find androidx.lifecycle:lifecycle-common-java8:2.2.0.
     Searched in the following locations:
       - https://storage.googleapis.com/download.flutter.io/androidx/lifecycle/lifecycle-common-java8/2.2.0/lifecycle-common-java8-2.2.0.pom
     If the artifact you are trying to retrieve can be found in the repository but without metadata in 'Maven POM' format, you need to adjust the 'metadataSources { ... }' of the repository declaration.
     Required by:
         project :app > io.flutter:flutter_embedding_debug:1.0.0-45f6e009110df4f34ec2cf99f63cf73b71b7a420
   > Could not find androidx.lifecycle:lifecycle-runtime:2.2.0.
     Searched in the following locations:
       - https://storage.googleapis.com/download.flutter.io/androidx/lifecycle/lifecycle-runtime/2.2.0/lifecycle-runtime-2.2.0.pom
     If the artifact you are trying to retrieve can be found in the repository but without metadata in 'Maven POM' format, you need to adjust the 'metadataSources { ... }' of the repository declaration.
     Required by:
         project :app > io.flutter:flutter_embedding_debug:1.0.0-45f6e009110df4f34ec2cf99f63cf73b71b7a420
   > Could not find androidx.fragment:fragment:1.1.0.
     Searched in the following locations:
       - https://storage.googleapis.com/download.flutter.io/androidx/fragment/fragment/1.1.0/fragment-1.1.0.pom
     If the artifact you are trying to retrieve can be found in the repository but without metadata in 'Maven POM' format, you need to adjust the 'metadataSources { ... }' of the repository declaration.
     Required by:
         project :app > io.flutter:flutter_embedding_debug:1.0.0-45f6e009110df4f34ec2cf99f63cf73b71b7a420
   > Could not find androidx.annotation:annotation:1.1.0.
     Searched in the following locations:
       - https://storage.googleapis.com/download.flutter.io/androidx/annotation/annotation/1.1.0/annotation-1.1.0.pom
     If the artifact you are trying to retrieve can be found in the repository but without metadata in 'Maven POM' format, you need to adjust the 'metadataSources { ... }' of the repository declaration.
     Required by:
         project :app > io.flutter:flutter_embedding_debug:1.0.0-45f6e009110df4f34ec2cf99f63cf73b71b7a420
   > Could not find androidx.tracing:tracing:1.0.0.
     Searched in the following locations:
       - https://storage.googleapis.com/download.flutter.io/androidx/tracing/tracing/1.0.0/tracing-1.0.0.pom
     If the artifact you are trying to retrieve can be found in the repository but without metadata in 'Maven POM' format, you need to adjust the 'metadataSources { ... }' of the repository declaration.
     Required by:
         project :app > io.flutter:flutter_embedding_debug:1.0.0-45f6e009110df4f34ec2cf99f63cf73b71b7a420
   > Could not find androidx.core:core:1.6.0.
     Searched in the following locations:
       - https://storage.googleapis.com/download.flutter.io/androidx/core/core/1.6.0/core-1.6.0.pom
     If the artifact you are trying to retrieve can be found in the repository but without metadata in 'Maven POM' format, you need to adjust the 'metadataSources { ... }' of the repository declaration.
     Required by:
         project :app > io.flutter:flutter_embedding_debug:1.0.0-45f6e009110df4f34ec2cf99f63cf73b71b7a420
   > Could not find androidx.window:window-java:1.0.0-beta04.
     Searched in the following locations:
       - https://storage.googleapis.com/download.flutter.io/androidx/window/window-java/1.0.0-beta04/window-java-1.0.0-beta04.pom
     If the artifact you are trying to retrieve can be found in the repository but without metadata in 'Maven POM' format, you need to adjust the 'metadataSources { ... }' of the repository declaration.
     Required by:
         project :app > io.flutter:flutter_embedding_debug:1.0.0-45f6e009110df4f34ec2cf99f63cf73b71b7a420

* 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 3s
Running Gradle task 'assembleDebug'...                              3.6s
Gradle task assembleDebug failed with exit code 1

Now I'm adding a Flutter plugin – let's say a popular and widely-used package:location:
# pubspec.yaml

dependencies:
  location: ^5.0.0

and I run the build again:
$ flutter build apk --debug

Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Running Gradle task 'assembleDebug'...                             16.6s
✓  Built build/app/outputs/flutter-apk/app-debug.apk.

We've just witnessed how a plugin project location modified the repositories of the app project. These lines in the plugin's buildscript were responsible for that:
https://github.com/Lyokone/flutterlocation/blob/a623e2d76c9b1d62df85e941f4718295782a4f45/packages/location/android/build.gradle#L17-L22
This is a simple example, but the larger the app gets, the more dependencies it amasses, the build becomes more and more intertwined with the buildscripts of the plugins.
Example (part 2)
I continued the experiment – I tried to set RepositoriesMode.FAIL_ON_PROJECT_REPOS to find what else silently adds repositories to my app project.
android/settings.gradle
pluginManagement {
    repositories {
        google()
        mavenCentral()
        gradlePluginPortal()
    }
}

dependencyResolutionManagement {
    repositoriesMode.set(RepositoriesMode.FAIL_ON_PROJECT_REPOS)
    repositories {
        google()
        mavenCentral()
        gradlePluginPortal()
    }
}

include ':app'
// rest of file ommitted

I run flutter build apk --debug again, and it failed because of the Flutter Gradle Plugin (aka flutter.groovy) itself modifying the repositories of my app:
https://github.com/flutter/flutter/blob/c94e6766fd4ed376293bf026ee241ae75fd358e1/packages/flutter_tools/gradle/src/main/groovy/flutter.groovy#L72-L88

Failed build logs
$ flutter build apk --debug

FAILURE: Build failed with an exception.

* Where:
Script '/Users/bartek/fvm/versions/3.10.5/packages/flutter_tools/gradle/flutter.gradle' line: 182

* What went wrong:
A problem occurred evaluating script.
> Failed to apply plugin class 'FlutterPlugin'.
   > Build was configured to prefer settings repositories over project repositories but repository 'maven' was added by plugin class 'FlutterPlugin'

* 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 503ms
Running Gradle task 'assembleDebug'...                             987ms
Gradle task assembleDebug failed with exit code 1

The problem
The problem is that basically, the whole ecosystem of Flutter plugins (for Android) has Gradle buildscripts that use a common anti-pattern. For more info about this, see also #124167.
Summary
I don't see a general solution to this problem. Best we can do is educate plugin authors to fix their Gradle buildscripts to not use allprojects, and instead to migrate to dependencyResolutionManagement. A migration tool could be provided, but since Gradle buildscripts are just executable code, it might be hard to cover every edge case.
This is not a huge problem for me (except for the Gradle-perfectionist part). I'm creating this issue to hint at the situation and make whoever might be concerned aware of it.
More interesting reading:

"Herding Elephants" by Square
How Flutter apps are compiled with Gradle for Android
#29328

I played around with this now and am sharing what I found.
First, let's estabilish some ground understanding:

A modern Gradle replacement for "configuring repositories {} for all projects participating in a build using allprojects" is the dependencyResolutionManagement {} API. To visualize:
Old way:
allprojects {
    repositories {
        google()
        mavenCentral()
    }
}

New way that doesn't have the problem described in the OP
dependencyResolutionManagement {
    repositories {
        google()
        mavenCentral()
    }
}

This is code inside FlutterPlugin is problematic for us:
https://github.com/flutter/flutter/blob/16821886a399e34e7c064dc14aaa0b5aff8713c2/packages/flutter_tools/gradle/src/main/groovy/flutter.groovy#L241-L252
We could remove it from here and move it app_plugin_loader.groovy instead:
class FlutterAppPluginLoaderPlugin implements Plugin<Settings> {
    @Override
    void apply(Settings settings) {
        def flutterProjectRoot = settings.settingsDir.parentFile

+        settings.dependencyResolutionManagement {
+            repositories {
+                google()
+                mavenCentral()
+                maven {
+                    url("https://storage.googleapis.com/download.flutter.io")
+                }
+            }
+        }

this would fix the problem with FlutterPlugin itself "interfering" with the app project.
Of course, this doesn't fix the Flutter plugin ecosystem on pub.dev that has allprojects {} in their Android buildscripts.

Code actually adds dependencies (which reside in repositories):
https://github.com/flutter/flutter/blob/c94e6766fd4ed376293bf026ee241ae75fd358e1/packages/flutter_tools/gradle/src/main/groovy/flutter.groovy#L327-L360

Custom repositories should also be supported using the dependencyResolutionManagement block, which doesn't seem possible using that, nor does it seem seem possible with other approach of having migrated away from using the buildscript block in build.gradle.
Tried this:
dependencyResolutionManagement {
    repositoriesMode = RepositoriesMode.PREFER_PROJECT
    repositories {
        google()
        mavenCentral()
        maven {
            url = uri("https://our-custom-maven-repo-url.com")
            credentials {
                username = <username>
                password = <password>
            }
        }
    }
}

But the dependencies are not picked up, possibly because the repos are hardcoded somewhere.

Of course, this doesn't fix the Flutter plugin ecosystem on pub.dev that has allprojects {} in their Android buildscripts.

Easy PR to make though. The community already had to go through adding namespaces to everything; often twice (due to not being backwards compatible the first time).


Allow * when specifying multiple input files

When listing input files as
cleancss public/css/* public/app.css

it gives the error:
WARNING: Invalid character(s) '/*;@import public/app.css;' at 1:18. Ignoring.

Different variations of forward and backward slash, as well as removing the * (hoping it would peek into the folder), resulted in failure as well.
It would be handy if I didn't have to list each file separately, by allowing a wildcard *. I would like to merge the css files from imported libraries with my own css to reduce the number of http calls the browser has to make. Unfortunately, the calls to this tool become quite lengthy due to the number of libraries.
Version: 4.0.0
OS: Windows

Yup, we could have it in CLI. 👍

@Cezille07 you can do it now in clean-css 4.1, thanks for the idea!

Awesome, thanks! :D


Compiler doesn't allow --debug and opaque types  && Failure to update document

I am trying to edit a Mark.Parsed
First try
https://ellie-app.com/5GhRZRzHxD3a1
In one instance, if I define a Msg that holds a Mark.Edit.Id the code fails to compile with some internal error with the --debug flag.
elm: Map.!: given key is not an element in the map
CallStack (from HasCallStack):
 error, called at ./Data/Map/Internal.hs:610:17 in containers-<IP_ADDRESS>-K2TDqgYtGUcKxAY1UqVZ3R:Data.Map.Internal

Without the --debug flag the updating fails with I tried to edit your document but couldn't find any blocks with the Mark.Edit.Id that you provided
Second Try
https://ellie-app.com/5Gj5wNd3cMCa1 - click the text and see the Logs tab
If I define the Msg to hold a String instead of a Mark.Edit.Id and convert to and from it, I still get the same update fail where the id is not found, as above.

I believe at least one mistake of mine was that I was applying a style edit on something that was not a styled text block. I managed to apply the edit after a few more tries.
If that is true then the error message could be a bit more truthful. It was not the ID that was wrong but the type of block.

Good call on having a better error message here.  I've implemented some more editing error messages beyond Id Not Found: https://github.com/mdgriffith/elm-markup/commit/b7a9aaddaa33d99f1f83378f58f1f09ab99fd848
A patch release should be out soon.  I'm going to close this issue, but thanks a lot for reporting it!


Add lightning talk on Foundation by Myo

Looks good @mkyaw thanks for doing this! Saved me work. You're now an open source contributor.

You actually forgot to put the link for your video in the YT player link, I missed it too. It's fixed now though.

You rock, @mkyaw! :) :+1:


Fix possible deadlock in delete_expired_series().

Signed-off-by: Harkishen Singh<EMAIL_ADDRESS>Fixes: #660

@cevian I am getting invalid transaction in CI.
Is this because of some invalid condition in nested procedures?

@cevian can you please see why are upgrade tests failing?

bors r+

bors r+


SQForm reset enables submit

I am using SQForm having 2 required fields, Reset and Submit buttons. When I fill one of the required fields and then click Reset button, it enables the Submit button even though the other required field is not entered yet. Also isValid from useSQFormContext turns true.
Code: https://codesandbox.io/s/adoring-lewin-8zz4g?file=/src/SQFormReset.js
Loom: https://www.loom.com/share/e46e650ed2e84cdb9883e30509870efc

I'm unable to replicate this particular issue in storybook. Here's the branch I made https://github.com/SelectQuoteLabs/SQForm/tree/SQFormIsValidExample


WIP: Make spinner preloaded.

In making sense of https://github.com/godotengine/godot/issues/72273 and other I noticed there is nowhere preloaded.
This PR is not OK as I changed the string into a preload.
This PR could fix the spinner resource, but needs testing.
I also notices lots of loads instead of preloads .. is there a reason for using load instead of preload?

This PR was just a search for Godot error log issues and one was mine removing *.import files.
I do not know how to test for this particular preload

@onready var ICON_SPINNER = preload("res://addons/gdUnit4/src/ui/assets/spinner.tres")

thus now you know.
Feel free to close.


pip install needs MS Visual C++ 14.0

Description
I'm trying to pip install pycsw into an anaconda python 3.5 environment, and it hangs on /pyproj-<IP_ADDRESS>.tar.gz with message " distutils.errors.DistutilsPlatformError: Microsoft Visual C++ 14.0 is required. Get it with "Microsoft Visual C++ Build Tools": http://landinghub.visualstudio.com/visual-cpp-build-tools"
I've gone to that web site, and done the install; int installs v15.0 (studio 2017), did some googling and found https://www.microsoft.com/en-us/download/details.aspx?id=48159 that install v14., added C:\Program Files (x86)\MSBuild\14.0\Bin to my path for the env, still getting same error on the pip install.
Environment
Windows 10, Anaconda environment with python 3.5; trying 'pip install pycsw'

operating system:
Python version:
pycsw version:
source/distribution

[ ] git clone
[ ] DebianGIS/UbuntuGIS
[ ] PyPI
[ ] zip/tar.gz
[ ] other (please specify):

web server

[ ] Apache/mod_wsgi
[ ] CGI
[ ] other (please specify):

easy_install hangs here:
Running Shapely-1.5.17\setup.py -q bdist_egg --dist-dir C:\Users\STEPHE~1\AppData\Local\Temp\easy_install-_wre1qhi\Shapely-1.5.17\egg-dist-tmp-5q5to1cy
error: [WinError 126] The specified module could not be found

SOLUTION:
conda install -c conda-forge pycsw works in the py2.7 environment and in the py3.5 environment.


Conflicting rules: no-reversed-links and no-bare-urls

This:
(Incorrect link syntax)[https://www.example.com/]

Gets corrected to this:
(Incorrect link syntax)[<https://www.example.com/]>

I think bare-urls should not attempt to fix it in this case.

The rules detect and report independently. When attempting to fix both at once, the library will not apply overlapping edits and so the first fix it sees “wins”.
That said, I may be able to detect this scenario and maybe not report it as a bare URL to avoid this scenario? I’m not sure I love that option; I will think about this some more.


Added Neumorphic Carousel

Fixes Issue🛠

Closes #438
Description👨💻
Added a Neumorphic Carousel in the compenents section
Type of change📄

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

How this has been tested✅
Tested on my local machine, works and looks as expected.
Checklist✅

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added demonstration in the form of GIF/video file
[x] I am an Open Source Contributor

Screenshots/GIF📷

Merged ✨ @SoumyadevSaha


feat: generalize all app sync integration hooks

Closes #117
Previous, much of the integration logic for AppSync were in the app sync logic. This change moves all of the per integration (ex: app sync calling lambda) into the specific integrations (ex: out of app sync file and into the Function file).
Changes:

Change the vtl integration into appSyncVtl as the integrations are specific to AppSync and not other VTL implementations (Api Gateway)
Change the vtl/appSyncVtl integration from a single function to 3 methods.

request - the main request vtl template
result - a post request processor that can transform the results from the service call (used by step functions now)
dataSource - retrieve a CDK data source for the integration

Use CDK's tryFindChild to resolve hierarchical singleton constructs instead of a separate map.
Add a static kind to each of the integration types so that they may be found by the compiler. Previously, only the top level objects (Function, StepFunction, Table) had kind. Now each of the integrations will have a kind (ex: StepFunction.describeExecution)
Deleted things
Compile: Only add functionless import when necessary.
Improve the creation of localstack tests, reduce boilerplate
fix: format function names for app sync
feat: Added getField to use Functionless AppSync Resolver with CodeFirst

Note: a larger change is planned with #119 to refactor app sync, however this change adds some more capability without changing the interface.
[ ] Update Docs

Code First App Sync Resolver
const commentResolver = new functionless.AppSyncResolver(...);
const getPostResolver = new functionless.AppSyncResolver(...);

/*
  type Query {
    getPost(postId: string!): Post
  }

 type Post {
  postId: ID!
  title: String!
  comments(nextToken: String, limit: Int): CommentPage
 }

 type CommentPage {
  nextToken: String
  comments: [Comment]!
 }

 type Comment {
  postId: ID!
  commentId: ID!
  commentText: String!
  createdTime: String!
 }
 */

const post = api2.addType(
  new appsync.ObjectType("Post", {
    definition: {
      postId: appsync.GraphqlType.id({
        isRequired: true,
      }),
      title: appsync.GraphqlType.string({
        isRequired: true,
      }),
    },
  })
);

const comment = api2.addType(
  new appsync.ObjectType("Comment", {
    definition: {
      postId: appsync.GraphqlType.id({
        isRequired: true,
      }),
      commentId: appsync.GraphqlType.id({
        isRequired: true,
      }),
      commentText: appsync.GraphqlType.string({
        isRequired: true,
      }),
      createdTime: appsync.GraphqlType.string({
        isRequired: true,
      }),
    },
  })
);

const commentPage = api2.addType(
  new appsync.ObjectType("CommentPage", {
    definition: {
      nextToken: appsync.GraphqlType.string(),
      comments: appsync.GraphqlType.intermediate({
        intermediateType: comment,
        isRequiredList: true,
      }),
    },
  })
);

post.addField({
  fieldName: "comments",
  field: commentResolver.getField(api2, commentPage.attribute(), {
    args: {
      nextToken: appsync.GraphqlType.string(),
      limit: appsync.GraphqlType.int(),
    },
  }),
});

api2.addQuery(
  "getPost",
  getPostResolver.getField(api2, post.attribute(), {
    args: {
      postId: appsync.GraphqlType.string({ isRequired: true }),
    },
  })
);

Why is this merging into local stack instead of main?

Is this a feat or chore?

Why is this merging into local stack instead of main?

@sam-goodwin. This change is sitting on the localstack testing change (#109), it may not need to be, #103 already depends on #109 and it can only have one parent.
Need to look at why the localstack testing branch is failing build.

Is this a feat or chore?

I considered it a feat because now we can extend app sync integrations without changing app sync. Will update the name to reflect.

Is this a breaking change?

Is this a breaking change?

Nope, all backwards compatible. Add docs for the new feature, will have them out in 5.

Is this a breaking change?

Nope, all backwards compatible. Adding docs for the new feature, will have them out in 5.

Change the vtl integration into appSyncVtl as the integrations are specific to AppSync and not other VTL implementations (Api Gateway)
Change the vtl/appSyncVtl integration from a single function to 3 methods.

Is this a breaking change?

Only breaking for internal Functionless implementation, consumers would not be impacted. Someone may have a nasty merge if they were deep into a larger feature (like I am doing right now with #103 )

Is the interface for integrations public or private? I think private. In practice, I doubt this breaks anyone. Just trying to be principled.

Is the interface for integrations public or private? I think private. In practice, I doubt this breaks anyone. Just trying to be principled.

Totally. Right now I was considering integration to be internal. I was going to make an issue where we'd make integrations public, documented, and with examples. At this point, I'd agree, this is breaking for integrators.

Oh I meant to say public. They are public lol. Which is why I think this is a breaking change.

Well it wasn't really possible to implement one before this change since AppSync had a bunch of internal logic. Do you think anyone is using it? I guess I am OK with marking this as break if you want.
Either way, this it the effort I think is left to be officially "publicly supported": #121

Happy with that. Let's move forward.

We can consider it private for now and therefore not breaking.


reduce frequency of dash wind streaming requests

App name
dash-wind-streaming
Description of bug
At this time this application is making requests very frequently, we could probably improve performance by reducing the frequency of requests while still showing a good example of live streaming
Other notes
Suggested by devops team: https://plotly.slack.com/archives/CHLNH6ZTL/p1559755002009900
cc @shammamah to assign out

@shammamah will this automatically be done in R-land? @Scott-Clone can you make sure you track this change please ? :)

Ideally for demo purposes this should happen frequently for a minute or two and then slow down in case the page is left open :)

Ok will do!


Update current 'standard' simulation model database Staging-CTA-Simulation-Model

I would like to replace the current standard simulation model database Staging-CTA-Simulation-Model by a fresh copy of the prod5/prod6 model versions as saved in the model_parameters gitlab.
``Staging-CTA-Simulation-Modelis a copy of the originalCTA-Simulation-Model` DB with some additions (and also some wrong uploads during the development of the model parameter schema). This leads to error when testing the model parameters more strictly (as it is implemented in PR #895 ).
I have already a playground DB uploaded (called sandbox_20240422) and my plan is the fix a new default one (with a better name) in the next days.
Don't expect any impact on the development work of everyone else, is we always first submit to the model parameters gitlab before updating the database.

Change of plan: I don't want to put the time in and prepare a new database simply because the name sandbox_20240422 is not nice. Could we please use that one for now? I am pretty sure we find mistakes and find a better name for the next database.
What you need to do: set in your .env file
SIMTOOLS_DB_SIMULATION_MODEL='sandbox_20240422'

Please do this after #895 has been reviewed, corrected, and merged.


forced az login even when authenticated

Description
when using the ./helpers/GenerateResourcesAndImage.ps1, even when already authenticated, a login is forced if an identity is already provided.
https://github.com/actions/runner-images/blob/main/helpers/GenerateResourcesAndImage.ps1#L243
        # Login to Azure subscription
        if ([string]::IsNullOrEmpty($AzureClientId)) {
            Write-Verbose "No AzureClientId was provided, will use interactive login."
            az login --output none
        }
        else {
            Write-Verbose "AzureClientId was provided, will use service principal login."
            az login --service-principal --username $AzureClientId --password=$AzureClientSecret --tenant $AzureTenantId --output none
        }
        az account set --subscription $SubscriptionId
        if ($LastExitCode -ne 0) {
            throw "Failed to login to Azure subscription '$SubscriptionId'."
        }

Platforms affected

[ ] Azure DevOps
[ ] GitHub Actions - Standard Runners
[ ] GitHub Actions - Larger Runners

Runner images affected

[ ] Ubuntu 20.04
[ ] Ubuntu 22.04
[ ] Ubuntu 24.04
[ ] macOS 12
[ ] macOS 13
[ ] macOS 13 Arm64
[ ] macOS 14
[ ] macOS 14 Arm64
[ ] Windows Server 2019
[ ] Windows Server 2022

Image version and build link
this issue doesn't effect runners, only runner provisioners
Is it regression?
no
Expected behavior
when running the image create script, if already logged in, the current profile should be used
Actual behavior
when logged in, az login should be skipped
Repro steps
first login az login
Then
Import-Module ./runner-images/helpers/GenerateResourcesAndImage.ps1
GenerateResourcesAndImage -SubscriptionId $SUBSCRIPTION_ID -ResourceGroupName $RESOURCE_GROUP_NAME -ReuseResourceGroup -ImageGenerationRepositoryRoot ./runner-images -ImageType $IMAGETYPE -AzureLocation $REGION -Tag @{source='https://github.com/actions/runner-images'}

and an auth prompt will be shown
suggested workaround (i've already tested this locally and with my pipelines and it works):
        # Login to Azure subscription
        try {
            $azAccount = az account show -o none
            Write-Warning "Already logged in..."
        }
        catch {
            if ([string]::IsNullOrEmpty($env:AzureClientId)) {
                Write-Verbose "No AzureClientId was provided, will use interactive login."
                az login --output none
            }
            else {
                Write-Verbose "AzureClientId was provided, will use service principal login."
                az login --service-principal --username $AzureClientId --password=$AzureClientSecret --tenant $AzureTenantId --output none
            }
        }

@Xander-Rudolph Thank you for bringing this issue to us. We are looking into this issue and will update you on this issue after investigating.


chore: Add Verify Github action and ensure it passes

What type of PR is this?
/kind cleanup

What this PR does / why we need it:
Adds a Github action to run make verify and updates the codebase to pass it.
Which issue(s) this PR fixes:

Requirements:

[x] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
@sunpa93 I needed to revert the image pull policy changes in the v0/v1 charts in order for verify to pass without also updating the package files. We'll pick up the updates when upstream master is integrated since you've made the necessary changes to the recent v1 charts there.
@landreasyan I removed the scheduler extender YAML files from the root of the deploy directory since these files are only used to install the latest v1 driver. Their presence causes verify-yaml.sh to fail.
@nearora-msft FYI- I added a Github action to execute make verify.
Release note:
none

Pull Request Test Coverage Report for Build 916145506

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

Totals

Change from base Build 907421885:
0.0%

Covered Lines:
1171

Relevant Lines:
3155

💛  - Coveralls


Release to PyPi

Hi @tvogels ,
I was wondering if you could bump the version and release the new version of pyexr to pypi? Thanks!

Hi @BrianPugh, this should work now. Somehow the version on Pypi was already 0.3.6 while the one on github was still 0.3.5. Changed it to 0.3.7 and uploaded the current code.

Thank you so much!

Hey Thijs, a new bump and PyPI release would be great, as writing support in the current 0.3.7 release is a bit broken. Thanks in advance!

Done. Thanks for the message.

@tvogels hey tvogels! can you release the current master? Thanks!

Done, thanks @BrianPugh


plz add opencl support, thx!

VSCode Version:1.4.0
OS Version:win 10

Steps to Reproduce:

plz add opencl support, thx!

@hemmingway Can you add some more specifics? Pointer to the OpenCL language spec? Is it about https://en.wikipedia.org/wiki/OpenCL? What would support for OpenCL mean?

@hemmingway Need more information here. I'll close the issue for now, but will gladly reopen once I undertand the request


:openssl_verify_mode=>"peer" can cause OpenSSL::SSL::SSLError: SSL_connect returned=1 errno=0 state=SSLv3 read server certificate B: certificate verify failed

If i set :openssl_verify_mode=>"peer" as part of Mail.defaults do delivery_method :smtp, would fail to send emails in the two environments I have access too unless I pass in a certificate file to verify the cert from.
I've got code that throws a RuntimeError if OpenSSL::SSL::VERIFY_PEER is being set, but there isn't ca_file or ca_path being passed in, or being provided by default.
I'm working on the test(s), and updated documentation and examples.
--
In Ubuntu 12.04, running 1.9.3 from https://www.brightbox.com/docs/ruby/ubuntu/
irb(main):015:0> Net::SMTP.default_ssl_context.ca_path
=> nil
irb(main):016:0> Net::SMTP.default_ssl_context.ca_file
=> nil
irb(main):017:0> RUBY_VERSION
=> "1.9.3"

With OSX 10.11.6, with rvm running MRI ruby 2.2.3, and openssl installed via homebrew.
Loading development environment (Rails 4.2.4)
[1] pry(main)> Mail.defaults do
[1] pry(main)*   delivery_method :smtp, {
[1] pry(main)*     address: 'smtp.sendgrid.net',
[1] pry(main)*     port: 587,
[1] pry(main)*     domain: ENV.fetch('SMTP_DOMAIN'),
[1] pry(main)*     user_name: ENV.fetch('SMTP_USERNAME'),
[1] pry(main)*     password: ENV.fetch('SMTP_PASSWORD'),
[1] pry(main)*     authentication: 'plain',
[1] pry(main)*     enable_starttls_auto: true,
[1] pry(main)*     openssl_verify_mode: 'peer'
[1] pry(main)*   }
[1] pry(main)*
[1] pry(main)* end
=> #<Mail::SMTP:0x007fa43602b598
 @settings=
  {:address=>"smtp.sendgrid.net",
   :port=>587,
   :domain=>"...",
   :user_name=>"...",
   :password=>"...",
   :authentication=>"plain",
   :enable_starttls_auto=>true,
   :openssl_verify_mode=>"peer",
   :ssl=>nil,
   :tls=>nil}>
[2] pry(main)> Mail.deliver do
[2] pry(main)*   from<EMAIL_ADDRESS>[2] pry(main)*   to<EMAIL_ADDRESS>[2] pry(main)*   subject  'testing sendgrid'
[2] pry(main)*   body     'testing sendgrid'
[2] pry(main)* end
OpenSSL::SSL::SSLError: SSL_connect returned=1 errno=0 state=SSLv3 read server certificate B: certificate verify failed
from /Users/christopher.evans/.rvm/rubies/ruby-2.2.3/lib/ruby/2.2.0/net/smtp.rb:586:in `connect'
[3] pry(main)> Net::SMTP.default_ssl_context.ca_path
=> nil
[4] pry(main)> Net::SMTP.default_ssl_context.ca_file
=> nil
[5] pry(main)> RUBY_VERSION
=> "2.2.3"

If I specify '/usr/local/etc/openssl/cert.pem' as the ca_file, it works.
Loading development environment (Rails 4.2.4)
[1] pry(main)> Mail.defaults do
[1] pry(main)*   delivery_method :smtp, {
[1] pry(main)*     address: 'smtp.sendgrid.net',
[1] pry(main)*     port: 587,
[1] pry(main)*     domain: ENV.fetch('SMTP_DOMAIN'),
[1] pry(main)*     user_name: ENV.fetch('SMTP_USERNAME'),
[1] pry(main)*     password: ENV.fetch('SMTP_PASSWORD'),
[1] pry(main)*     authentication: 'plain',
[1] pry(main)*     enable_starttls_auto: true,
[1] pry(main)*     openssl_verify_mode: 'peer',
[1] pry(main)*     ca_file: '/usr/local/etc/openssl/cert.pem'
[1] pry(main)*   }
[1] pry(main)*
[1] pry(main)* end
=> #<Mail::SMTP:0x007f90a1e450a0
 @settings=
  {:address=>"smtp.sendgrid.net",
   :port=>587,
   :domain=>"...",
   :user_name=>"...",
   :password=>"...",
   :authentication=>"plain",
   :enable_starttls_auto=>true,
   :openssl_verify_mode=>"peer",
   :ssl=>nil,
   :tls=>nil,
   :ca_file=>"/usr/local/etc/openssl/cert.pem"}>
[2] pry(main)> Mail.deliver do
[2] pry(main)*   from<EMAIL_ADDRESS>[2] pry(main)*   to<EMAIL_ADDRESS>[2] pry(main)*   subject  'testing sendgrid'
[2] pry(main)*   body     'testing sendgrid'
[2] pry(main)* end
=> #<Mail::Message:70129600622660, Multipart: false, Headers: <Date: Fri, 27 Jan 2017 11:07:02 -0800>, <From<EMAIL_ADDRESS><To<EMAIL_ADDRESS><Message-ID<EMAIL_ADDRESS><Subject: testing sendgrid>, <Mime-Version: 1.0>, <Content-Type: text/plain>, <Content-Transfer-Encoding: 7bit>>

This is a known trouble spot with Ruby + OpenSSL on macOS.
For RVM, check this out: https://rvm.io/support/fixing-broken-ssl-certificates
rvm osx-ssl-certs update all

Is this also a known issue with rvm and ubuntu, because where it failed in our qa environment.
I also did some testing with the Docker ruby image based on alpine linux.
https://gist.github.com/becomingwisest/983d2e71770df889579d1b6544fc6f19
Is there a known/supported platform that allows openssl_verify_mode: 'peer' but doesn't require ca_file, or ca_path to be set?

Is there a known/supported platform that allows openssl_verify_mode: 'peer' but doesn't require ca_file, or ca_path to be set?

That's up to your OpenSSL config. If the default CA path (or file) includes current root certificates, it'll always work out of the box. This is true for most distributions.
If the certs are outdated, you'll need to update them. Use your distro's tooling to update your CA certs. On Ubuntu, that's update-ca-certificates: http://manpages.ubuntu.com/manpages/trusty/man8/update-ca-certificates.8.html
On an Alpine container, you need to install ca-certificates first in your Dockerfile, e.g. apk --no-cache add ca-certificates && update-ca-certificates
In any case! This is far out of scope for the Ruby Mail library 😊

It doesn't seem to use openssl default though.
This doesn't work:
require 'mail'

Mail.defaults do
  delivery_method :smtp, {
    address: 'smtp.fastmail.com',
    port: 465,
    user_name: '<REDACTED>',
    password: '<REDACTED>',
    authentication: 'plain',
    tls: true,
    openssl_verify_mode: 'peer'
  }

end
Mail.deliver do
  from     '<REDACTED>'
  to       '<REDACTED>'
  subject  'testing fastmail'
  body     'testing fastmail'
end

# raises /opt/ruby24/lib/ruby/2.4.0/net/protocol.rb:44:in `connect_nonblock': SSL_connect returned=1 errno=0 state=error: certificate verify failed (OpenSSL::SSL::SSLError)

But this does:
require 'net/https'
require 'uri'

url = URI.parse "https://myconan.net/"
# to make sure it's actually VERIFY_PEER, uncomment the following line to raise exception instead.
#url = URI.parse "https://<IP_ADDRESS>/"

http = Net::HTTP.new(url.host, url.port)
http.use_ssl = true
http.verify_mode = OpenSSL::SSL::VERIFY_PEER
request = Net::HTTP::Get.new(url.path)
response = http.request(request)

puts response.code # 200

According to this, set_params must be called for ca_* parameters to be filled in with defaults.


Route keedrop.de to /de/ pages by default and keedrop.com to base site (english)

Wouldn't this just make things more complicated? What's the benefit here?

so its entirely based on browser language?

Only when navigating to the root path. Afterwards it's still possible to explicitly select another language.


Disable Travis bots, and Appveyor Release bots.

Kokoro bots currently cover all Travis bots:
macos-clang-debug
macos-clang-release
ubuntu-clang-debug
ubuntu-clang-release
ubuntu-gcc-debug
ubuntu-gcc-release
ndk-build
android
check-format
Kokoro bots also cover all Windows Release builds:
windows-VS2013-release
windows-VS2015-release
windows-VS2017-release
Due to a compiler issue on the Kokoro Windows VM, we currently do not
run the Windows Debug build on Kokoro.
Therefore, I am disabling all Travis jobs and the appveyor Release jobs.

Rebased and pushed into master as 6e550f4181


Test online version of app with new R package version

As data structure has been changed (read in of default data from /data/SensorVSample.RData does need loading of package SMCcalibration) the online shiny app has to be tested.

Hi Johanness,
Yes at the moment the app (downloaded form this repo) is not working and yields the following error. You think is better that I try to fix it or you want to do it by yourself?
Warning: Error in rep: invalid 'times' argument
50: reactiveValues
49: server [C:\Users\GGenova\AppData\Local\Temp\Rtmp0uO9J1\shinyapp35d46dd41959\SMCcalibration-MultiSelect\R/server.R#4]
Error in rep(x = TRUE, times = nrow(data)) : invalid 'times' argument

Hey Giulio,
best option for me would be if you could fix it - if you already know how to and as it is a EURAC repository where the app is published?!
I did change some data reading to the app, to be consistent with the R package structure.
At the moment the R package does install, which was no more possible with you branch.
Best option would be a R package that does install, so one can run the shiny app offline easily, and on the other hand a product that is available online. If both is not possible in the MASTER code trunk, we will have to do a branch for the online version.

For me after commit JBrenn@4070fd1a7f694c2f4a441891343f9a262b46b5bf the following test is working well:
shiny::runGitHub('JBrenn/SMCcalibration',  subdir="R",  launch.browser = TRUE)

@GiulioGenova could you also try once more. Does this mean the software should also work as an online application?

when I try to install the package I get the following error:
Downloading GitHub repo JBrenn/SMCcalibration@master
from URL https://api.github.com/repos/JBrenn/SMCcalibration/zipball/master
Installing SMCcalibration
Installation failed: Invalid comparison operator in dependency: leaflet (>=2.0.2), leaflet.extras (>=1.0.0), robustbase (0.93-3), tidyr (>=0.8.1)

Yes the code:
shiny::runGitHub('JBrenn/SMCcalibration',  subdir="R",  launch.browser = TRUE)

Is still not working. I get this:
> shiny::runGitHub('JBrenn/SMCcalibration',  subdir="R",  launch.browser = TRUE)
Downloading https://github.com/JBrenn/SMCcalibration/archive/master.tar.gz

Listening on http://<IP_ADDRESS>:7551
Warning: Error in rep: invalid 'times' argument
  50: reactiveValues
  49: server [C:\Users\GGenova\AppData\Local\Temp\RtmpOqRBqC\shinyapp2bdc3f52578e\SMCcalibration-master\R/server.R#91]
Error in rep(TRUE, nrow(data)) : invalid 'times' argument

seems like is not finding "the data" file or it is null. may be for you is working because you have loaded the package?
to put the app online I actually only need to make the app run in RStudio (no matter how) and then I deploy It on the shinyapps.ui servers

Ok got this.
My workflow at the moment is the following:
library(SMCcalibration)
data("SensorVSample")
shiny::runGitHub('JBrenn/SMCcalibration',  subdir="R",  launch.browser = TRUE)

This is working. And if its also working for the shinyapps.ui server COOL :)
So we can deploy to there the next days and close this issue after.

@JBrenn great, we are almost there.
I managed to install the package. I needed R 3.5 and had to install manually some libraries ("jsonlite","rcurl")
I deployed the app online but it crashes. I thing it's because it can't find the SMCcalibration package.
One solution could be to place in the ui.R or in server.R the following lines:
if (!require("devtools")) install.packages("devtools")
if (!require("SMCcalibration")) install_github("JBrenn/SMCcalibration")

So that the package can be installed on shinyapps.io servers

see Email today:
I did restructure the package a bit. Added an inst/shiny folder, moved ui.R and server.R there, wrote some wrapper to stat app locally and in shinyapps.io. Shiny app can be run now locally by
devtools::install_gituhub(SMCcalibration)
runShinyapp()

I also successfully deployed the app to shinyapps.io, see https://jgbr.shinyapps.io/shiny/
devtools::install_gituhub(SMCcalibration)
deployApp(‘inst/shiny’)

Giulio could you try once more for the alpenv shinyapps.io account?

Great, it works now!
So the problem was just finding the data file, right?
if (!require("SMCcalibration")) install_github("JBrenn/SMCcalibration")

was not needed as rsconnect looks for the needed packages already
nice!

Perfect! Yes, lib SMCcalibration and example data set are loaded before server function...
library(SMCcalibration)
data("SensorVSample")


The Promise constructor cannot handle void returns on resolve

Configuration Check
My compilation target is es5 and my lib is dom, es5, es2015.promise and es2015.iterable.
Missing / Incorrect Definition
The value for the resolve callback in the Promise constructor is required:
new <T>(executor: (resolve: (value: T | PromiseLike<T>) => void, reject: (reason?: any) => void) => void): Promise<T>;
Sample Code
Using void as the generic (return) type for a Promise results in an error:

Workaround
The current workaround is to return undefined:

But undefined is not the same as void: https://www.typescriptlang.org/docs/handbook/2/functions.html#void

What TypeScript version are you using? It works fine with 4.3.5:
https://codesandbox.io/s/typescript-playground-export-forked-ej4hy?file=/index.ts

We were running v4.3.2 and updated to v4.3.5 but no joy.

hi @Sc0tTyXL
{
  "compilerOptions": {
    "target": "es5",
    "lib": ["dom", "es5", "es2015.promise", "es2015.iterable"]
  }
}

even you are using es2015.promise and es2015.iterable but its runtime compiler wont use promises without polyfil.
use polyfil to execute promise base execution at run time in es2015
npm i core-js
//AND
import 'core-js/es6/promise'

use promise like we do in es6 and passing nothing in resolve would be interpreted as **void**

const myPromise: Promise<void> = new Promise<void>((resolve, reject) => {
  setTimeout(() => {
    console.log("Async operation completed");
    resolve();  // No value returned --void--
  }, 1000);
});


how to set the umi length?

Hi, as is explained in the document, the default the umi length is 6bp, however for my data, there is a 12bp umi at the beginning of each read, how to change the umi length as I can't find any parameter from the connor help or from the source code?

Thanks, I'll try it out.


fix-up typo

What this PR does / why we need it:
/kind  cleanup fix-up typo
Special notes for your reviewer:
If applicable:

[ ] this PR contains documentation
[ ] this PR contains unit tests
[ ] this PR has been tested for backwards compatibility

/retest


Make Dafny 4.2 the default, without a prompt?

We recently introduced my class of 120+ students to Dafny with an in-class Dafny coding challenge.  In years past, students often ran into various issues trying to get Dafny and .NET installed.  To try to forestall that this year, I used GitHub's Classroom to setup a GitHub Codespace with all of the relevant Dafny starter files and with the Dafny extension pre-installed in VS Code.  For many students this worked very smoothly.  Some students, however, either didn't read our directions or they missed clicking on the right option when the Dafny extension prompts to upgrade to Dafny 4.2.  This left them in a state where they were running an old version of Dafny that produced strange errors or failed to run at all on the starter files.
Are we nearing the point where the extension can default to the latest Dafny version?  If not, if anyone has tips for creating a Codespace with the plugin configured to the latest version, they would be much appreciated!

This was happening in a fresh "Codespace", which is basically a fresh container spun up when a student first connects to it.  Hence, there's no prior version of Dafny installed.  VS Code is pre-installed, and in the configuration file for the Codespace (as shown below), you can specify extensions by ID, which get installed when you first boot.
As soon as the Codespace launches, it shows the (old) prompt install Dafny 4.2.0:

If the student fails to click the gray "Accept 4.2", they get old Dafny-style errors, such as:

If it's not happening on the Dafny side, then perhaps there's a stale cache of VS Code extensions out in the wild somewhere?
{
  "image": "mcr.microsoft.com/devcontainers/dotnet:6.0",
  "features": {
  },
  "customizations": {
		// Configure properties specific to VS Code.
		"vscode": {
			// Set *default* container-specific settings.json values on container create.
			"settings": {},
			"extensions": ["dafny-lang.ide-vscode"]
		}
   }
}


refactor: revert migrate to 17

😅

The pipeline is passing now. If you feel like this MR is still required you can reopen it and add a comment as to the motivation


Remove any use of ansible_user_id and become_user

I am not sure what the right way is to target specific user in the tasks... but my current solution does not feel right either even if it works.

I think i could set default variable like some_custom_user: '{{ ansible_user_id }}' as fallback.
For those, who want to use custom username, it then is possible to override it.


Verify that Ctrl-C/Command-C doesn't copy text from write-only text fields (password fields)

We should verify that if a text field is obscured, it doesn't allow copying of the contents of the field.

Related (duplicate?) #89282


It is unclear how to rebuild the database from scratch

Describe the bug
While it is said

Extract the Pettifor and Glawe scales from Glawe et al. 2018
The data was extracted from the paper using Tabula and stored as csv

it seems that the data needs some postprocessing and is not all the data contained in the db.

Hey @KOLANICH, thanks for reaching out.
Could you rephrase your issue as either a question or provide more context around what outcome are you trying to achieve?

Clarified the text a bit.

Thanks, I think I understand a bit better now.
I quite curious on what are the security restrictions not allowing you to read from a sqlitedb.
Unfortunately, there is no way of generating the database from original data sources since the current state is a result of ~9 years of development and data curation.
You can however dump all the tables (either as SQL or CSV) from the elements.db and recreate the sqlite db yourself on a secure machine from plain text files generate by the dump. Would that be an option for you?

You can however dump all the tables

I can, but it would mean interfacing the database file, which is a security risk. So I prefer them to be shipped pre-dumped within either this repo, or another repo dedicated for the dataset. Maybe even elements.sqlite file should be gitignored and people should be instructed to generate it themselves. Or, as I have said, a separate package doing everything needed during it being built.
Also I think I'd prefer if the data was in TSV files, not in SQL files doing the queries, because SQL files contain SQL queries that can do arbitrary things to the database file, but TSV files are limited, and I suspect that TSV files are safe to use that way (though it may be unsafe to open them in Excel/LO Writer). sqlite3 CLI utility can import TSV files, so the pack to recreate the files can look like a small and clear shell script and a set of TSV files.

I can, but it would mean interfacing the database file, which is a security risk.

If you believe the current db is compromised I would advise not to use at all. The package is MIT licensed and comes with no warranties. But I'm guessing you are also not using any other open source packages since by the same logic they (or their dependencies or the dependencies of their dependencies ... ) might be vulnerable. Unless you're going vetting all the code personally.
If not, I recommend
sqlite3 elements.db .dump > elements.sql

Unless you're going vetting all the code personally.

Sometimes I do that. How do you think I had discovered that mendeleev uses SQLite ;) Also I use static analysers like bandit, that detect potentially dangerous API calls (unfortunately, not all of them).


Support for GML Simple Features Profile

Does this library support the GML Simple Features Profile, as can be seen on Wikipedia?
https://en.wikipedia.org/wiki/Geography_Markup_Language#GML_Simple_Features_Profile
Thank you!

Simple Feature Profile is a profile of OGC GML Standard. This library implementation is close to some subset of this standard SF-0, and mostly correspond to wiki information.


[ray.so] doesn't get a correct theme in the url

Extension
https://www.raycast.com/garrett/ray-so
Raycast Version
1.88.4
macOS Version
15.1.1
Description
When submitting a request to ray.so theme is not honored. I'm selecting a theme, say "Forest"

The URL (and thus ray so theme) is selected as Candy

Steps To Reproduce

Create a new snippet with a theme "Forest"
Observe theme not match

Current Behaviour
Theme not passed in the url
Expected Behaviour
Theme is passed in the url

Works for me

Works for me

Here's a video. Am I doing anything wrong?
https://github.com/user-attachments/assets/5f377e33-bd66-493f-ba85-74eba1d20b26

Oh I see the issue, if you keep the default theme it's not properly passed to the ray.so in url

I've opened PR that should fix it https://github.com/raycast/extensions/pull/16081, thanks for reporting 🫡

Thanks Dominik!


Check for empty password on login

If a login is attempted with an empty password, immediately fail and show a specified error message, instead of going through a whole login attempt.

:100:
So excited that you did this! Gets me at least once a day


Remove warning about default branch from readme

We now support test coverage for pull requests.
@codeclimate/review @dblandin

LGTM


These kinds of suggestions should check adjacent lines to see if maybe the code was written to fit the  format of previous or following code. Examples include e.g.
case foo of
  a -> return $ Just a
  _ -> return $ Nothing

or
instance QueryResults Foo where
  convertResults fields values | length fields == 3 =
    Foo{..}
      where
        !fooId                      = convert (fields !! 0) (values !! 0)
        !fooField                   = convert (fields !! 1) (values !! 1)
        !fooAnotherField            = convert (fields !! 2) (values !! 2)
  convertResults fs vs = convertError fs vs 3

I have thought about this a few times. I think it would be great. But I also suspect it's very difficult. These "looks like" relationships that are obvious to a human are super difficult for a computer. Instead, I invested in improving the warning suppression mechanisms, so people can ignore the bit that annoys them. Let's use that instead.
I still welcome someone figuring out a smart idea in this area, if anyone is up for it, but I suspect we need a research paper first.


Enhancement ideas

Just sharing some I think would be impactful-

tooltips for hotkeys

ability to change hotkey bindings (for ex. i run into issue on windows using Ctrl-R to regenerate the response closes out the extension window and refreshes the page would be nice to change this binding the user settings)

setting to keep the extension window open so it doesn't auto-close (helpful when having longer interactions

option to move the extension window over to a sidebar on left or right margin. this will be very useful for maintaining full visibility of the page and being able to interact with model on the sidebar.

option to save chat history so you can pick up where you left off if you close the browser window and re-open at a later time.

Whatcha think about these?

I think these are all great. Quick comments on each item:

tooltips for hotkeys

I'll add this back.

ability to change hotkey bindings (for ex. i run into issue on windows using Ctrl-R to regenerate the response closes out the extension window and refreshes the page would be nice to change this binding the user settings)

Will definitely consider it.

setting to keep the extension window open so it doesn't auto-close (helpful when having longer interactions
option to move the extension window over to a sidebar on left or right margin. this will be very useful for maintaining full visibility of the page and being able to interact with model on the sidebar.

I've considered this before and it will definitely require a larger refactor of the "frontend" of the extension.

option to save chat history so you can pick up where you left off if you close the browser window and re-open at a later time.

This functionality is available now. Sorry, the UI is unintuitive. The "down arrow" icon button is the "save chat" button. The "counter-clockwise arrow" icon button opens the chat history. I'll update the UI to make this more clear.
Would you be open to working on any of these? 😄


Makes refactor rename tests into mock tests and fixes some of them.

General test improvements.
Updates interface of PythonSuggestedImportAction.
Updates VS 2015 build to target .NET 4.6.

:+1:


Ensure CI builds proper Mac/aarch64 binaries

https://github.com/usefulness/webp-imageio/pull/88 started to build all supported binaries on CI.
0.2.1 used binary from https://github.com/gotson/webp-imageio/pull/1
the next release will ship with a binary build "locally" via Github Actions
This issue will serve as a place as a notepad, where I can write down all macos&ARM related updates.

I managed to run tests on macOS 13.2 (aarch64) https://scans.gradle.com/s/sxnyqobt3egde which surprisingly completed with a success 😮 Which seems to suggest, the binaries build using updated gotson's image do actually work - link

It seems like everything work as expected. I was able to read and write webp images on M1 mac (via Bitrise) 🎉


Back-End - Charging Station in error notification form end-user

An End-User should be able to push the information that a charging station is in error (charging scenario is failing)
A button on the mobile application should be build in a different issue.

Related issue: https://github.com/LucasBrazi06/ev-server/issues/1650

Related issue: #1650

Not related, 1650 is to change a roaming status ;-).

More related to LucasBrazi06/ev-mobile#31 and Epic LucasBrazi06/ev-dashboard#973

Related issue: #1650

Not related, 1650 is to change a roaming status ;-).

OCPP 1.6 define a list of charging station statuses. Not need to reinvent the wheel for end user issue report to tenant administrators to link it to one of already predefined statuses at the time of the erro.

OCPP 1.6 define a list of charging station statuses. Not need to reinvent the wheel for end user issue report to tenant administrators to link it to one of already predefined statuses at the time of the error.

The user just wants to report an issue with the charging station and not necessarily a status error ;-)

OCPP 1.6 define a list of charging station statuses. Not need to reinvent the wheel for end user issue report to tenant administrators to link it to one of already predefined statuses at the time of the error.

The user just wants to report an issue with the charging station and not necessarily a status error ;-)

You add automatically the status at report creation: for example if the status is 'Faulted' since 2 days, the report can be discarded because he tries to use an out of order station.

What about if the charging station is simply not usable. Physical damage, connector not accessible, vandalize. Such cases will not be necessarily detected/handled by the OCPP status. However, the charging station will actually not be usable any longer.

What about if the charging station is simply not usable. Physical damage, connector not accessible, vandalize. Such cases will not be necessarily detected/handled by the OCPP status. However, the charging station will actually not be usable any longer.

You still add automatically all the targeted charging station information snapshot (including the status) to the report. The report granularity is connector -> charging station.

To be tested with the UI


168x168x3 JPEG-type artefacts

168x168x3 RGB observations seem to have poor-quality JPEG-type artefacts. Is this on purpose, or is it possible to get higher quality 168x168x3 RGB observations? See the marked areas in the figure (scaled 600% without interpolation).

Hi @ndrwmlnk
After some testing during the development process we decided to go with using jpg compression in Obstacle Tower. The reason for this is that we saw ~50% increase in simulation step throughput with it vs using uncompressed png. The thinking was that since most image datasets use jpg compression, these artifacts should be something a CV pipeline (or CNN) can deal with.
Are these artifacts causing any issues with your training process?


Command 'todoist.sync' not found

Hi, I just installed this extension a few minutes ago, and added my API token. Whenever I press the sync button in the sidebar I get the error in the title

The same thing happened for me on MacOS Catalina, I also just installed the addon and get the same error after putting my API token.

@aadilayub @ChrArendt This should be fixed in the latest version v0.4.3

@aadilayub @ChrArendt This should be fixed in the latest version v0.4.3

It's working now for me, thank you :)


Restrict application to single instance

only 1 instance supported at the moment because

context menu works on fixed path and uses fixed port on localhost
config files (name)
...

How?

there is a feature in launch4j that prevents multiple instances (only works if started using the .exe and not jar)
use some Java mechanism (socket, ...)

We should still be able to run multiple instances during development!

done (for windows, i.e. launch4j prevents multiple instances. User could launch jar itself multiple times manually)


Support custom request and notification handlers

This PR allows users to specify custom handler functions instead of the receive-* multi-methods.
The main reason for that is to enable middleware, e.g. to log/time/trace requests, execute them on different threads, etc. Using the :request-handler and :notification-handler options, one could create Ring-style middleware:
(defn wrap-vthread [handler]
  (fn [method context params]
    (p/vthread (handler method context params))))

;; ...

(defmulti receive-request (fn [method _ _] method))

(def server (server/chan-server {:request-handler (wrap-vthread #'receive-request)}))

Avoiding global state in the multi-methods also allows having multiple LSP implementations on the classpath. Not a common use case, but we found ourselves in the situation where we had both clojure-lsp (to use the linter) and our custom LSP on the dev classpath.
Some tests could now be simplified, using :request-handler instead of with-redefs.

@mainej -- Done. Will you please review these? You are better with words than I am :)

Added a middleware example to the README and added :response-executor to chan-server's doc-string. Should be ready to merge now.


gcsfuse 0.18.2

Merged in https://github.com/Homebrew/homebrew-fuse/commit/f2669b988886bdc2a26ba2c21c866f931af0d829. Thanks once more @jacobsa! 😺


Inconsequential things

'Nuf said.

actually, gee it's nice developing

Add GitHub co-pilot and the pair is quite amazing.

Add GitHub co-pilot

It keeps on nagging me to start a trial. I should give it a go when I have

car
ocpp charger
time 😂

car
ocpp charger
time 😂

and

ten bucks a month for stuff I swore I would never use...

And yet, I was fixing to craft a check for specifying multiple same API keys for the integration as yeah, nah.  And that's quite clever to suss out what I was doing, saving me from typing it, even if I changed the message...
But ten bucks a month to save my fingers some work? Umm... Whatever. I did it anyway.

Yeah, I'll give it a go.
I was comically disappointed when I was first trying to figure out python (see https://github.com/jasonacox/Powerwall-Dashboard/tree/main/weather/contrib and https://github.com/jasonacox/Powerwall-Dashboard/tree/main/tools/ecowitt-weather-history for my hilarious early efforts) when ChatGPT would come up with absolutely completely wrong suggestions for how to parse json using python.
It would come up with stuff that looked good but was just wrong.  And, at that stage, I had no idea on how to set up any sort of dev or debug client, so was flying blind.
Since then, I've had the chance to review some T-SQL produced by co-pilot (probably six months ago), and it runs, but it is still comically inefficient.  Now, in this case, I'm perhaps judging too harshly, as I think in sets and can run optimisers in my heads and construct complex queries so that optimisers make the right decisions without thinking, and there's still a lot of art to that, because you still need to understand cardinality, keys, selectivity, and stuff like that, and language models can't do that.
That's my beef with AI writing code: syntactically it can be right, but it can't always write performant code, and that matters, still.  It matters for UX, it matters for the web, it matters for phones, and it matters for cars, and it matters for banking, and airlines, and medical and industrial, and pretty much everything except startups trying to get their round A away.
But nah, yeah, I'll give it a go.  I might learn something 🤣


add title attribute and text to describe spotify card for accessibility

Issue
#31
Changes proposed
iframe has a title attribute describing the content of it.
Screenshots
Note to reviewers`

thanks @choir241  enjoy your @holopin
https://holopin.io/collect/cm297v5hw18430cl1rosd4p5s


Refactor backends

Move behaviour of IParameter ToSqlString into dbqf.Sql library (as well as SqlString class itself). Inherit every existing concrete IParameter (e.g. LikeParameter -> SqlLikeParameter), and implement the ToSqlString (possibly with another ISqlParameter interface too, which defines this).
All the ParameterBuilder logic should probably move into the dbqf.Sql library, leaving interfaces behind in the core (maybe?).  Or copies, depends on whether having ParameterBuilders that make vanilla IParameters is even worth it? (i.e. we'll always need a backend of some kind, so the implementetions will usually end up there).
Even the behaviour of Subject.SourceSQL and .Sql(string) should probably move into dbqf.Sql as they have no bearing on an ORM implementation.  Check references on these properties to see how often they are actually referenced (if only in the SQL and Standalone projects then we're all good).
Make an ISqlGenerator interface based on the dbqf.Sql SqlGenerator, and another for the ISqlListGenerator.
Make a dbqf.MsAccess version, which references core and dbqf.Sql as we'll be wanting to use the IParameter types defined in dbqf.Sql (maybe).  Create concrete implementations of SqlGenerator with MsAccess-specific SQL structure.
For EF, make a dbqf.EF project, with Configuration, Subject and Field types that bind to property info (or something) that defines mapping within EF.  Either use the vanilla IParameter implementation in core (if FieldPath is enough), or inherit and provide further functionality for building LINQ expression trees for each IParameter (more likely).  Rather than have an SqlGenerator class, we'll need to determine what kind of class is required to take the LINQ expression trees from the parameters and combine them together and execute via EF.  Remembering it is possible to run direct SQL through EF so this is an option.
Standalone should have a factory for generating all required objects for querying and retrieving results.  Instead of ISqlResults, it should probably be more like BackendFactory which has methods for getting an ISqlGenerator, ISqlListGenerator, and the concrete DbConnection/DbCommand objects.  Or, we just encapsulate all the logic and don't even expose generators or connections, just have an IDbService which only has:
DataTable GetResults(FieldPath[], IParameter)
List<string> GetList(IField)

Then we can have a factory class that returns an IDbService based on string key for the list of connections in a configuration; "SqlClient", "SQLite", and "MsAccess".  In fact the connection object in the configuration should really just serialise the concrete type (e.g. rather than object Connection, have an SqlClientConnection which has ConnectionString, an SQLiteConnection which has Filename, etc.).

Implement this: http://weblogs.asp.net/cazzu/62141

Closed in 2c011050c53a8128b01877979f2ebc2713f408f6


The Feat "Astral reach" doesnt seem to work as intended

I was trying to make work a Sentinel+Astral Reach build on my Discordance monk (Great Sub-Class BTW) and the part of the feat that is supposed to trigger an opportunity attack at 10 feet doesnt do anything?
Maybe I am actually wrong? and dont understand how it is supposed to work?

Please add screenshots and your game save.

Location.zip
This is my save, A screenshot would prob not help at all cause you have to see how the enemies get into range for it to "Trigger"
Its not a campaign save, because I was testing the whole interaction on a Test Character and Test Campaign before doing it in my actual playthrough.

Only with Unarmed. working as intended.

Oh it doesnt apply when you have a monk weapon in your hand, I thought it would just use an "unarmed strike" for the reaction instead of the weapon. Didnt know it would simply not work when a monk weapon was in my hand. Thank you anyway!


Change ModestConstructorQuery behavior to skip copy constructors

I feel like an improvement similar to the one made in #644 should also be applied to ModestConstructorQuery - skip constructors that contain parameters of the requested type in order to avoid circular references.

Hi @desmondgc,
Thanks for registering the issue for us. In order to better understand the context of the issue, could please show a real world samples of the situation when such API is present?
While it's possible to skip those methods, we should make such decisions very carefully. Current AutoFixture's approach is to activate objects when everything is trivial. When there some complications with a design of a particular class, we usually don't attempt to handle all the possible cases out-of-the-box and require user to customize the fixture.
Let's see what are the samples and whether they cover some common real-world scenario.

We want to hide the default public constructor in many of our domain entities in order to force users to call a specific constructor which may include guards and other construction logic. Most of these entities include a copy constructor which also happens to be the "most modest" constructor. In unit tests, we generally want to use AutoFixture default behavior to call the most modest constructor (excluding the copy constructor).
public class Payment
{
    private Payment()
    {
    }

    public Payment(Payment payment)
    {
        if (payment == null)
        {
            throw new ArgumentNullException(nameof(payment), $"Cannot copy a null {nameof(Payment)}.");
        }

        PaymentUid = payment.PaymentUid;
        CurrencyInitials = payment.CurrencyInitials;
        CurrencyAmount = payment.CurrencyAmount;
        PaymentAmount = payment.PaymentAmount;
        // ...
    }

    public Payment(string currencyInitials, decimal currencyAmount)
    {
        PaymentUid = Guid.NewGuid();
        CurrencyInitials = currencyInitials;
        CurrencyAmount = currencyAmount;
        PaymentAmount = 0;
        // ...
    }

    public Payment(string currencyInitials, decimal currencyAmount, decimal paymentAmount)
    {
        PaymentUid = Guid.NewGuid();
        CurrencyInitials = currencyInitials;
        CurrencyAmount = currencyAmount;
        PaymentAmount = paymentAmount;
        // ...
    }

    private Guid _paymentUid;
    public Guid PaymentUid
    {
        get
        {
            return _paymentUid;
        }

        set
        {
            if (value == Guid.Empty)
            {
                throw new ArgumentException("Guid.Empty is not a valid identifier.");
            }
            else
            {
                _paymentUid = value;
            }
        }
    }

    public string CurrencyInitials { get; set; }
    public decimal CurrencyAmount { get; set; }
    public decimal PaymentAmount { get; set; }
    // ...
}

With the current behavior, this causes an ObjectCreationException: AutoFixture was unable to create an instance of type Ploeh.AutoFixture.Kernel.SeededRequest because the traversed object graph contains a circular reference.

Thanks for that sample! As far as I got it, your wish is to skip all the constructors that takes only single argument and it's of the current class type.
Meaning that we are not going to skip constructors that take more than one argument:
public class Payment
{
    public Payment(Payment oldPayment, string currency) { }
}

@desmondgc Do you agree on that?
If that is so, I'm basically fine with such a feature request. It looks logical to me and such constructors indeed don't look as good candidates for constructions.
However, what concerns me more, is that currently we have the following queries:

ModestConstructorQuery
GreedyConstructorQuery
ArrayFavoringConstructorQuery
EnumerableFavoringConstructorQuery
ListFavoringConstructorQuery

If we are going to ignore copy constructors, should we skip them for all the queries? Otherwise, it will be inconsistent that constructor might still be still picked if other query type is used. Even if those queries "favor" other constructor overloads, they still could end up with a "copy-constructor" if better candidates are missing.
@moodmosaic Could you please share also your thoughts. For me it looks like an improvement for our heuristic and I can't imagine scenario when this change causes issues.

@zvirja Yes, in my case all of our copy constructors take only a single argument of the current type.
I feel like this request is most important with respect to the ModestConstructorQuery since a single-parameter copy constructor will almost always be chosen by this strategy, but is much less likely for GreedyConstructorQuery. I see that the Array/Enumerable/List-favoring queries fall back to the modest strategy so it might be applicable there, too.

@desmondgc Thanks for the confirmation. Let's also see what @moodmosaic thinks about that and later decide what to do.

@adamchester Your vote will be also appreciated, given that you were involved into #644 😉

@zvirja, @desmondgc, I'm not sure I follow; which constructor you'd want AF to call in your example?
If the current constructor strategies aren't enough, you can always override them and even supply your own custom ones...

which constructor you'd want AF to call in your example?

Given that you have the following type:
class Payment
{
  public Payment(Payment payment) { }

  public Payment(string currencyInitials, decimal currencyAmount) { }

  public Payment(string currencyInitials, decimal currencyAmount, decimal paymentAmount) { }
}

AutoFixture by default should pick the  public Payment(string currencyInitials, decimal currencyAmount) { } constructor instead of the public Payment(Payment payment) { } as it currently does. Otherwise, it leads to a recursion and fixture simply fails.

you can always override them and even supply your own custom ones...

Yep, you can. However, the issue is about improvements for the AutoFixture constructor selection heuristic. If AutoFixture is always unable to construct instances using the "copy constructors", I don't see any value to pick up those constructors and surely fail. Given that we already applied the same enhancement to the factory methods in #644, it looks that we are fine to care about that.
I like to have a robust behavior by default and if somebody has a tricky scenario, he could always customize fixture to pick up the copy constructor instead.

But you can force AutoFixture to use the greediest constructors.

Yep. There is a way to work around the issue and configure fixture in a way that it works. There is no issue with that.
The suggestion is to change the default behavior to make it robust as current strategy will never create specimens - exactly what we did in #644.

Sounds good 👍 Sorry that it took my so long to figure out what needs to be done 😮

Converted that to a feature 😉

I opened a pull request for this issue 😄