[2D Feature Proposal] A more simple way to detect collisions between Area2D nodes

Another 2D Feature proposal to make Godot 3 even more friendly for beginners.
Today I read a comment from a great youtube creator (HeartBeast) of Gamemaker 2 and Godot 3 content. It's about what features that he wish were implemented into Godot 3. Gonna create one post per feature request (there are three).
The feature proposal is "a more simple way to detect collisions between Area2D nodes".
(I'm honestly not sure what this would look like but something that would be easier for beginners).
His channel is https://www.youtube.com/user/uheartbeast/videos
Thank you.

"a more simple way to detect collisions between Area2D nodes"

This is not really a suggestion, it's a vague statement with no actionable content. What's "hard" about Area2D now? You connect the area_entered signal.
As a teacher, I can tell you "easier for beginners" is a very debatable topic. What one person finds easy, another has trouble grasping. What does "beginner" mean? New to Godot? New to game dev? New to programming entirely? These are all wildly different states of mind.
I would recommend closing this unless there's a specific suggestion to discuss.

I'm not sure why you are opening issues in HeartBeast's behalf (did he ask you to?), but this description is not helpful at all.

"a more simple way to detect collisions between Area2D nodes"

I'm sorry, but that could mean anything. I find it quite simple already. Are you having trouble with the current way? In which step the workflows fail for you? Did you try something and it didn't work? Do you have a concrete suggestion?
I advise you to read the Godot's FAQ, especially the linked section about ideas.

More simple than a signal callback and the use of get_overlapping_areas? (remember Areas do not collide)
Also an engine should not point to be beginner friendly, beginners are in that situation for a couple of weeks only then stop being in that group after learning how things work, the only thing useful in that short timespan (for any tool) are documentation and tutorials.

All the repect to any youtuber that do content to godot. You can tell to him that there is a system for wishes in the patreon (not necesary to contribute, it's open) and everyone can proposal all kind of things (specific or not).


Lra 611 lsa ride share car change gas datatype from float to int

db migrate to change car field from float to int

To make things right let's edit the car's _form.html.erb.
Right now the field for gas is 'form.text_field :gas' and it's not right, it should be an integer.
And I would add a validation into the Car model, that the number should be between 1 and 100.


Can't view/list clusters in Cluster Management

there are 4 clusters available, but aren't being listed in dashboard on master-head (ebad1df)

note: can still create a cluster, but after revisiting the clusters page, no clusters are listed.

I'll fix it, but what kind of cluster do you have that has no spec.kubernetesVersion?

there are other issues for 2.6 that are preventing clusters from provisioning all of the way.  I did provision 1 through ember that deployed successfully, and local cluster is always there as well.

on master-head (53f17be) I am now able to see the list of clusters again

I was able to reproduce this by doing this on master-head (1e1ed67):
* deploy a cluster with a bad configuration (typo in yaml)
  * cluster list still displays
* edit cluster yaml that is in bad state
* cluster list shows empty again

tested on master-head (e756276):

clusters are all listed properly on the cluster management page


How can I configure the wifi band to 5 GHz or 2.4 GHz?

I was trying to use Powershell script to set up windows 10 mobile hotspot (which is the function in "Settings" -> "Network & Internet" -> "Mobile hotspot"). I was able to turn it on through the following script:
[Windows.System.UserProfile.LockScreen,Windows.System.UserProfile,ContentType=WindowsRuntime] | Out-Null

# Define functions. Not important to this question
Add-Type -AssemblyName System.Runtime.WindowsRuntime
$asTaskGeneric = ([System.WindowsRuntimeSystemExtensions].GetMethods() | ? { $_.Name -eq 'AsTask' -and $_.GetParameters().Count -eq 1 -and $_.GetParameters()[0].ParameterType.Name -eq 'IAsyncOperation`1' })[0]
Function Await($WinRtTask, $ResultType) {
    $asTask = $asTaskGeneric.MakeGenericMethod($ResultType)
    $netTask = $asTask.Invoke($null, @($WinRtTask))
    $netTask.Wait(-1) | Out-Null
    $netTask.Result
}
Function AwaitAction($WinRtAction) {
    $asTask = ([System.WindowsRuntimeSystemExtensions].GetMethods() | ? { $_.Name -eq 'AsTask' -and $_.GetParameters().Count -eq 1 -and !$_.IsGenericMethod })[0]
    $netTask = $asTask.Invoke($null, @($WinRtAction))
    $netTask.Wait(-1) | Out-Null
}

# Create tethering manager
$connectionProfile = [Windows.Networking.Connectivity.NetworkInformation,Windows.Networking.Connectivity,ContentType=WindowsRuntime]::GetInternetConnectionProfile()
$tetheringManager = [Windows.Networking.NetworkOperators.NetworkOperatorTetheringManager,Windows.Networking.NetworkOperators,ContentType=WindowsRuntime]::CreateFromConnectionProfile($connectionProfile)

# Create configuration
$configuration = new-object Windows.Networking.NetworkOperators.NetworkOperatorTetheringAccessPointConfiguration
$configuration.Ssid = "test"
$configuration.Passphrase = "12345678"

# ===================== My question is here =====================
[enum]::GetValues([Windows.Networking.NetworkOperators.TetheringWiFiBand])
$configuration | Get-Member
# ===============================================================

# Check whether Mobile Hotspot is enabled
$tetheringManager.TetheringOperationalState

# Set Hotspot configuration
AwaitAction ($tetheringManager.ConfigureAccessPointAsync($configuration))

# Start Mobile Hotspot
Await ($tetheringManager.StartTetheringAsync()) ([Windows.Networking.NetworkOperators.NetworkOperatorTetheringOperationResult])

but I could only set the SSID and the Passphrase of the network but not the network band which we can choose between '5 GHz', '2.4 GHz' or 'Any available' in the setting gui.
I saw on this post (https://blogs.windows.com/windowsdeveloper/2019/09/10/windows-10-sdk-preview-build-18975-available-now/#DwOj8B0wPu5zd9hK.97) that it seems there is an enum 'TetheringWiFiBand' to set that starting from Windows 10 SDK, version 1903, build 18362 (it's exactly my windows version). However, as you can see, in the middle of my script, when I tried to access this enum, I got an error:
Unable to find type [Windows.Networking.NetworkOperators.TetheringWiFiBand].
Also, there is no such an enum when I print the member of Windows.Networking.NetworkOperators.NetworkOperatorTetheringAccessPointConfiguration
Does anyone have an idea of how to set the wifi band of the mobile hotspot based on this method? Thanks.

Thanks for reaching out with your feedback. I'll work on updating the documentation with this info, but in the meantime you'll find quicker support by consulting our developer support resources. For community assistance with development questions I recommend that you ask on our forums or StackOverflow. Otherwise, please see the developer support page for your support options. If you get assistance through these other resources and believe that information would help us improve the docs, please come back and let us know.


Terminal

Type: Bug
im unable to write any cmd in terminal please check and resolve it while my windows cmd working properly but here in visual code its not working
VS Code version: Code 1.78.2 (b3e4e68a0bc097f0ae7907b217c1119af9e03435, 2023-05-10T14:39:26.248Z)
OS version: Windows_NT x64 6.2.9200
Modes:
Sandboxed: Yes

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-2720QM CPU @ 2.20GHz (8 x 2195)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: unavailable_offraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: unavailable_offvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
3.90GB (0.34GB free)

Process Argv
--crash-reporter-id b3353354-5d86-4f8a-821a-91a358cb3b94

Screen Reader
no

VM
67%

Extensions (20)

Extension
Author (truncated)
Version

composer-php-vscode
DEV
1.34.13120

phptools-vscode
DEV
1.34.13120

profiler-php-vscode
DEV
1.34.13120

vscode-html-css
ecm
1.13.1

vscode-test-explorer
hbe
2.21.1

fontawesome-autocomplete
Jan
1.3.1

chatgpt-helper
kir
4.1.0

search-on-things
Mas
0.0.1

test-adapter-converter
ms-
0.1.7

java
red
1.18.0

LiveServer
rit
5.7.9

intellicode-api-usage-examples
Vis
0.2.7

vscodeintellicode
Vis
1.2.30

vscode-gradle
vsc
3.12.7

vscode-java-debug
vsc
0.50.0

vscode-java-dependency
vsc
0.22.0

vscode-java-pack
vsc
0.25.11

vscode-java-test
vsc
0.38.2

vscode-maven
vsc
0.41.0

php-debug
xde
1.32.1

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vstes627cf:30244335
vslsvsres303:30308271
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscorecescf:30445987
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593cf:30376535
pythonvs932:30410667
cppdebug:30492333
vsclangdf:30486550
c4g48928:30535728
dsvsc012cf:30540253
pynewext54:30695312
azure-dev_surveyone:30548225
2e4cg342:30602488
pyind779:30671433
f6dab269:30613381
vscrpc:30673769
pythonsymbol12:30671437
2i9eh265:30646982
showlangstatbar:30737416
vsctsb:30748421
pythonms35:30701012
pythonfmttext:30731395
fixshowwlkth:30730052
hidesbindicator:30730055
pythongtdpath:30739704

/confirmPlease

Thanks for creating this issue! To help narrow it down, please take the time to follow below steps. Thanks a lot and happy coding 🙏
Is the issue caused by an Extension?
Stop VS Code and from the command line (NOT the integrated terminal in Code), execute: code --disable-extensions and try your steps again to see if it reproduces. If it still reproduces, jump to the next chapter.
If you see it is an issue with the extension, please run the command Start Extension Bisect and follow the instructions to find the extension that is causing this issue.

Please report the issue to the extension causing this.
Is the issue caused by VS Code Core?
If the issue is not caused by an extension, it is maybe caused by your configuration. Try to run code --disable-extensions --user-data-dir <directory> where <directory> is an empty folder. This will ensure Code is starting with a fresh data directory, e.g. no specific settings and without any extensions running.
If it still reproduces, it was maybe fixed already in our nightly release. Please download from https://code.visualstudio.com/insiders/ and see if you can reproduce the issue.
Finally, to help us narrow down what change caused the issue, please run the following command to find the build. It will start previous releases of VS Code insiders to see where the regression was introduced:

npx --yes vscode-bisect@latest
follow the instructions until you found the offending build
report back the commit range in this issue

Thanks!


expand on the ivyDeps documentation

I don't really understand it enough yet from the examples. I am trying to get it working with bintray so I can use JME3, for example. The Mill docs end up saying to go read the Ammonite docs, which is a link to some stuff that doesn't syntactically look like what was in build.sc, so it pretty much looks like a dead-end to a new user afaict.

If anybody else cares, here's what ended up (apparently) working for me for using jmonkeyengine with mill:
 import mill._, scalalib._
 import coursier.maven.MavenRepository

 object x extends ScalaModule {
   def scalaVersion = "2.12.4"
   def repositories = super.repositories ++ Seq(
     MavenRepository("http://dl.bintray.com/jmonkeyengine/org.jmonkeyengine")
   )
   def ivyDeps = Agg(
     ivy"org.jmonkeyengine:jme3-core:3.2.1-stable",
     ivy"org.jmonkeyengine:jme3-desktop:3.2.1-stable",
     ivy"org.jmonkeyengine:jme3-lwjgl3:3.2.1-stable",
   )
 }

Questions and discussion belong on gitter


Bug in handling envname PFPATH

I was doing some test today and encountered a seg fault. It happens at this line:
https://github.com/wangyinz/mspass/blob/c6b354ebc70e8e133a6886f0f33ff34b8e68cb42/cxx/src/lib/utility/AntelopePf.cc#L365
when s is a null pointer. This happens when PFPATH is not defined in the env, and the if branch up there seems want to handle that situation.
I also wonder if this is a relic from a previous version, as I thought the only env variable we are dependent on should be the MSPASS_HOME.

As I mentioned in our phone call earlier today I had found that bug and fixed it, but the fix is on the add_deconvolution branch.


[Event Request] Codeunit 14 "Gen. Jnl.-Show Entries" - Show custom page

Dear BaseApp team,
I'd like to kindly ask you to review the following request.
In codeunit 14 (Gen. Jnl.-Show Entries) when page "Customer Ledger Entries" is about to be shown to the user we'd like to display a different custom page:

The desired event signature is as follows:

Please consider adding such a modification if it looks appropriate.
Thanks.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Sass & HAML

Would it be possible to just set the template language in an initializer? I use the hamlit gem (not haml-rails or something) but now komponent generates .erb files.
And for sass/scss, could it be possible to import one index file with all sass component partials? Since variables can't be used globally when each file is imported by the javascript.

Hello @meneerprins ,
For the first point:
Rails.application.config.generators.template_engine = :haml

For the second point, I need to investigate and discuss with @spone about the need and find a solution to make it possible without list all files into one globally.

A possible solution is to create a file _variables.scss in frontend folder and import it in each sass/scss files you need variables.
// frontend/_colors.scss

$color: yellow;

// frontend/components/button/button.scss

@import '../../_colors;

.button {
  background-color: $color;
}

Hi @florentferry thanks for the tip, didn't know about that setting for HAML.
For the scss, that just doesn't look very practical, especially when you have a large project with a lot of components. In addition to that, variables are used in almost every file in my experience. Some sort of autogenerated _components.scss partial (sorted alphabetically) that you can manually include would be a nice addition imo.

Hi @meneerprins, good idea! We'll do some tests and keep you informed.

Solution 1:

Create a new file application.scss in frontend/packs

@import '../colors';
@import '../components/index';

Require it in application.js in frontend/packs

import './application.scss';
import 'components';

Create a new file index.scss in frontend/components and require components stylesheets into it.

@import './button/buttton';

Remove import './button.scss'; from frontend/components/button/button.js

So now, each components stylesheets have access to variables globally defined in colors file.

Hi @florentferry and @Spone,
I've ran into the same issue as @meneerprins. Your solution solves the usage of variables in components, but the price is that all imports done from the css file in the component break.
Say the button uses the following css:
.button {
    background-image: url("./button-background.png");
}

where the image is located at ./button/button-background.png.
With the proposed solution the loader will try to resolve ./button-background instead of ./button/button-background.png which shouldn't be the case as the image is part of the component.

 I use the first method I mentioned, importing in each component.

Hi @Joralf
I don't have this issue, where all imports break. Can you please create a gist with code to reproduce the issue?

Thanks for the quick reaction guys!
@Spone, here's the gist: https://gist.github.com/Joralf/41923d3a22245c8dbd5e790b9e58aeee
This throws an error with:
ERROR in ./frontend/packs/entry.scss
Module build failed: ModuleNotFoundError: Module not found: Error: Can't resolve './test.jpg' in '/Users/joralf/Sites/myapp/frontend/packs'

Which makes sense, because the entry point of the scss file is now packs/entry.scss which means any url resolving is done from that path. You can "solve" this by changing the line to  background-image: url('../components/test/test.jpg');, which means the css in the component needs to be aware of the path and is no longer "standalone". That would be a huge dealbreaker for me.
Another potential issue with the proposed solution is that all css is now accessible by any component in the tree. To prove this I've created a second gist:
https://gist.github.com/Joralf/17981dc7950ace3c207e1fe58413fb6a
When using the default Komponent way, css defined in the component folder is only accessible by the component itself (as it's loaded via the .js file in the component). With the proposed solution this is no longer the case.

Thanks @Joralf for taking the time to prepare the gists.
I agree with you that the most important here is to have "plug-and-play" components with proper encapsulation and no dependency to its path.
I feel that using a single entry point for each component (the .js file) is more true to the Webpack way. But it comes at the expense of not having access to variables without explicitely importing them.
Maybe we could keep it the current way, but include the variables CSS file to the Komponent install generator, and then import it in the default CSS template used by the component generator. This way, you wouldn't have to add the import manually for each component.
What do you think?

Hi @Spone and @florentferry ,
That sounds like a good solution. So it's a similar approach to https://github.com/komposable/komponent/issues/82 except that the css file you want to include on top will be dynamic?

Not sure what you mean about #82

Ah sorry. Maybe wrong comparison. But as far as I understand you want to make sure that:
@import '../../src/stylesheets/variables';
Will be added automatically to the top of every css file when you generate it via rails generate component xxx

Yes exactly. But I think that means we have to enforce a convention for naming this file.
For instance I use ../../base/variables.scss but you use ../../src/stylesheets/variables
Also, I think we want to @import with a path based on Komponent's root, so we don't need the ../../ in front of the path.
Would that work for you?

Yeah, I'm fine with forcing the name/location for the file. And that import statement would make it a lot easier!

Ok, I created a issue for that: #88 and I'm closing this one.


So there won't be any ynd support for sollumz?

It is not planned at the moment.


Lab demo fails with hostname mismatch

Jupyter notebooks will fail to communicate with egeria servers which are not on localhost with the following error:
Checking OMAG Server Platform availability...
Exception: HTTPSConnectionPool(host='lab-core', port=9443): Max retries exceeded with url: /open-metadata/platform-services/users/garygeeke/server-platform/origin (Caused by SSLError(CertificateError("hostname 'lab-core' doesn't match 'localhost'")))
    Core Platform is down - start it before proceeding
Exception: HTTPSConnectionPool(host='lab-datalake', port=9443): Max retries exceeded with url: /open-metadata/platform-services/users/garygeeke/server-platform/origin (Caused by SSLError(CertificateError("hostname 'lab-datalake' doesn't match 'localhost'")))
    Data Lake Platform is down - start it before proceeding
Exception: HTTPSConnectionPool(host='lab-dev', port=9443): Max retries exceeded with url: /open-metadata/platform-services/users/garygeeke/server-platform/origin (Caused by SSLError(CertificateError("hostname 'lab-dev' doesn't match 'localhost'")))
    Dev Platform is down - start it before proceeding
Done.

This occurs despite the correct truststore being in place.
This is because the certificate we (the Jupyter server) receive should be unique to the host, and contain the DNS hostname (in this case of the egeria platforms).
To fix this requires that we dynamically generate certs as part of the deployment - or require each/every egeria platform to have a unique cert.
There is no longer seemingly any way in python 3.10 using the requests module to turn off just the hostname verification.
Until we have revisited the whole area of cert deployment, we will need to change the notebooks to explicitly disable certificate checking

The 'best' approach is probably to switch to using a custom context for requests.
However this really requires a little refactoring to establish the context at the beginning of each notebook, and also to check that the persistent connections/context don't cause other issues.
Due to this in the short term I am making an explicit change to set 'verify=False' on all requests module calls. I'd hoped to avoid this, but at this time it seems the pragmatic approach until we can properly resolve deployment of certificates & potentially cleaner structuring of the python library code


Updates needed for osx

https://github.com/mongodb-js/mongodb-prebuilt/issues/59
It seems like this and mongodb-prebuilt need some TLC. Could someone with permission make the various updates to deps and also urls?

Maybe @LoneRifle


ClangFormat integration

In this PR following things has been added/modified:

Rework of TheIDE's menu bar. Right now there is a separated "Reformat" menu that are available in "Edit" and "Assist" menus. ALT+SHIFT+R is the shortcut for invoking reformat command.
The possibility to reformat XML/JSON outside code editor has been removed. It is duplicated functionality we currently have. For more details please see Ide::FormatJSON_XML_File method deletion (FormatCode.cpp).
Full integration with clang-format binary.
New File dialog now allows to create .clang-format configuration file
Default style available in the repository root directory basing on the style present in U++ source files

@mirek-fidler please review if this change is going into the good direction. The code responsible for calling clang-format executable will be implemented later. I would like to avoid situation when we will do not agree on this initial changes.

@mirek-fidler not merging this PR is a big waste from my point of view. I lost significant amount of my free time and we agree that I will be implementing clang-format integration into TheIDE. So, I was shocked when I saw https://github.com/ultimatepp/ultimatepp/commit/2e77d71f1212215efb4eba5f209873a00abdd6ce. You had my implementation and you didn't added any comments here to make it better.

I have actually started with your implementation. Current code is based on
it. I just squashed it to merge. Still fixing small issues.
On Wed, Feb 1, 2023 at 9:40 PM Zbigniew Rębacz @.***>
wrote:

@mirek-fidler https://github.com/mirek-fidler not merging this PR is a
big waste from my point of view. I lost significant amount of my free time
and we agree that I will be implementing clang-format integration into
TheIDE. So, I was shocked when I saw 2e77d71
https://github.com/ultimatepp/ultimatepp/commit/2e77d71f1212215efb4eba5f209873a00abdd6ce.
You had my implementation and you didn't added any comments here to make it
better.
—
Reply to this email directly, view it on GitHub
https://github.com/ultimatepp/ultimatepp/pull/122#issuecomment-1412692998,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AARH23YPXM4SDXKJC4NE4N3WVLC2VANCNFSM6AAAAAATITZK3M
.
You are receiving this because you were mentioned.Message ID:
@.***>


IPN Callback suddenly failed, probably cert installation. Maybe switch to REST API?

In some Store Instances that all run the latest version of CStore and the PayPal Standards Gateway, the IPN delivery started to fail one after the other. Not the same day, but within the same timeframe.
I did some investigation and it seems to be a certificate problem, I am currently working on that.
While doing the research I found, that the IPN method is deprecated and developers are asked to use the REST API (https://paypal.github.io/PayPal-PHP-SDK/) instead as the IPN method will be dropped somewhen in the future.
So if someone is up for that, it would likely be the future path.

The IPN method has always seemed flakey to me. Failures across the years for no reason.
I've reworked the checkout somewhat recently, meaning it's now easier to implement things like Paypal's 'checkout' scripts, and do a proper upgrade for Stripe elements.
I've started on the Paypal one, which I think could then end up being a replacement for Paypal Standard. But I may also be able to swap out the callback handling at the same time, since it'll likely be the same code.

Do you have an estimation for that? I'm ripping my hair dealing with the IPN/cert stuff. It works well and goes through when the IPN is triggered manually from the history but will do nothing when called programmatically as it should. So I'm torn between either debugging this occult behavior or focus on building something better. I don"t like to put much energy in something which will be useless in the forseeable future. Thank you!

No solid estimation I'm afraid, it's just something I've been working on between projects.
It was just something I had pencilled in for the next few weeks.

So you're not talking about a Payment Gateway but some completely new checkout process that handles integration more easy? I'm currently thinking about implementing the REST API into a new iteration of the Paypal Standards Gateway, which seems to be just about handling the validation Class differently. I'm actually in a hurry because ALL my clients on that server don't receive IPNs for now.

Not a completely new checkout, just a reworking of how/when the payment gateways get loaded in (they get loaded in the step, rather than on the page load). It won't impact existing gateways or the checkout page overall, it's really just a javascript change. And there's no difference to the way callbacks are coded up.
(it's a bit off topic, but I've done this also as part of better handling order attributes)
I'd love to be able to jump in and help here, I'm just under some time pressures this week myself.
I'd suggest looking at updating the existing Paypal Standards payment method with the REST API at this point, as you've suggested. I'll help with whatever time I do have, just message if you need someone to bounce something off of.

I finally solved the issue by changing which TLS ciphers are used in the webserver config. PayPal just changed what they accepted without giving notice. So for whoever runs into a similar issue - look that direction. For now, that Issue is closed but I believe implementing the REST API approach is the way to go.

Great to hear you've solved this. I had a feeling it was something to do with with something server related, but I totally agree that updating things so it uses REST is appropriate.


Confusion with arrays

Given a type like this:
input PostInput {
  urls: [String]!
}

I'm getting this:
export interface PostInput {
  urls: String[] | String;
}

But I guess it should be
export interface PostInput {
  urls: (String | undefined)[]
}

This error is being caused at this line
I'll fork this project and try to fix this error and submit a PR!
Thanks for all!!!

This diff solved my issue.
I'm not sure if we should use (type | undefined)[] or (type | null)[] but this change solved my issue.
I'll await this conversation to evolve and I can submit a PR (I won't mind if you go and fix it, I don't care about getting a contributor flair because I 2 lines).
For the time being I can keep using my fork 👍

Hi @degroote22, I run into the same issue as you, the problem is still actual. if your fix is working then submit a PR :) as for null vs undefined I prefer null because if we look into [GraphQL spec](url
https://facebook.github.io/graphql/draft/#sec-Value-Completion) we can see

If result is null (or another internal value similar to null such as undefined or NaN), return null.

Thank you for reporting.
In the last few months, since the transition of many libraries under The Guild's leadership, We've reviewed and released many improvements and versions to graphql-cli, graphql-config and graphql-import.
We've reviewed graphql-binding, had many meetings with current users and engaged the community also through the roadmap issue.
What we've found is that the new GraphQL Mesh library is covering not only all the current capabilities of GraphQL Binding, but also the future ideas that were introduced in the original GraphQL Binding blog post and haven't come to life yet.
And the best thing - GraphQL Mesh gives you all those capabilities, even if your source is not a GraphQL service at all!
it can be GraphQL, OpenAPI/Swagger, gRPC, SQL or any other source!
And of course you can even merge all those sources into a single SDK.
Just like GraphQL Binding, you get a fully typed SDK (thanks to the protocols SDKs and the GraphQL Code Generator), but from any source, and that SDK can run anywhere, as a connector or as a full blown gateway.
And you can share your own "Mesh Modules" (which you would probably call "your own binding") and our community already created many of those!
Also, we decided to simply expose regular GraphQL, so you can choose how to consume it using all the awesome fluent client SDKs out there.
If you think that we've missed anything from GraphQL Binding that is not supported in a better way in GraphQL Mesh, please let us know!

In the context of that particular issue - GraphQL Mesh uses GraphQL Code Generator under the hood and I believe it should generate the right typings but if not, please feel free to open a new issue on the GraphQL Mesh repo.
We're looking forward for your feedback of how we can make your experience even better!


How to catch error in stage.loadfile?

Using our app which includes NGL, we let users type in pdb codes and sometimes these are just plain wrong / don't exist - on these occasions we get errors in the console like below, as NetworkStreamer throws an error:
ngl_verbose.js:50028 GET http://mmtf.rcsb.org/v1.0/full/xyz1 404 (Not Found)
_read @ ngl_verbose.js:50028
(anonymous) @ ngl_verbose.js:49686
read @ ngl_verbose.js:49684
parse @ ngl_verbose.js:88301
load @ ngl_verbose.js:50129
autoLoad @ ngl_verbose.js:50360
loadFile @ ngl_verbose.js:76325
etc

With this being asynchronous I can't wrap the .loadfile in a try/catch to get the error, and the promise catch clause isn't reached and .then isn't either
var promise = stage.loadFile (uri, params)
           .catch (function (reason) {
                console.log ("FAILED", reason);
            })
            .then (function (structureComp) {
                console.log ("whoooo");

So how do I get some sort of indication back from loadfile that things have failed?

Right, the errors were not re-thrown, now they are https://github.com/arose/ngl/commit/4ececc67c57dea8a6d44297afad219da3fb35386. Can you try the ts2 branch?

Thanks Alex, I'll give it a go tomorrow (uk time)

Thanks, using the ts2 branch I can catch file errors now, thanks, but 2 caveats (I guess since it's labelled as 2.0.0 a fair bit is changing from the version I was using)

I needed to change one of the lines in package.json so it worked on windows, the mkdir command looked like it was expecting the mac version:
"ts": "tsc && cp src/shader/.vert src/shader/.frag build/js/src/shader/ && mkdir "build/js/src/shader/chunk" && cp src/shader/chunk/*.glsl build/js/src/shader/chunk/ && cp package.json build/js/",

I suppose a pure npm solution using one of the various makedir modules could be attempted

The second is probably more serious for me since I haven't solved it. Basically all my DistanceRepresentations I use for crosslinks have turned into stripes, and the browser is toiling overtime to calculate them (50s and 2GB peak memory for quite a small protein and ~150 links) - it looks like every DistanceRepresentation is being segmented into tiny sub-objects - getFixedLengthDataDash and Float32Array are showing up in my profile as using this 50s hit.

Is there a way to just use the old cylinder-style representations to represent my cross-links or tell the dashing to turn off for the distance reps?

should be fixed by 7560902f398c61c0f0be06b86a3dd3000772857a, please try

Added a parameter useCylinder to the distance representation<PHONE_NUMBER>bff81e6d465274ca3eff1137f3254d), please try. Could you please make an example with a number of links on codepen? I would like to include it in the example scripts.

Had to switch to using 'copyfiles' as 'copy' kept chucking npm errors on my windows machine, but it was a straight swap - changed copy to copyfiles in the ts task.

Superb! That worked straight off. I'll look at adding a codepen, just need to make it so I'm not adding all my code as well, just set up a protein with some preset links

Thanks a lot, Martin

switched to the copyfiles package (see 565a5688559ca66cd43c388f5eb551a50c492f0d), hope it works out of the box for you now

Alex, have a look at this, see what you think --> https://codepen.io/martingraham/pen/RjqXmQ

This is great! Do you want to make a PR? I.e. put it into examples/scripts/interactive/crosslink.js. Otherwise I can copy your code myself.


  Jenkins is currently complaining it cannot finr the win32ole gem.  I understand this gem will not exist on a non-windows system but it looks like puppet-rspec is trying to do some sort of monkey patching by looking at the stacktrace.  Is this a bug or something that can or cannot be implemeneted to work?
12) role::windowsworkstation should compile into a catalogue without dependency cycles should compile into a catalogue without dependency cycles
     Failure/Error: it { is_expected.to compile }

     LoadError:
       cannot load such file -- win32ole
     # ./.bundle/ruby/2.4.0/gems/rspec-puppet-2.6.8/lib/rspec-puppet/monkey_patches.rb:275:in `require'
     # ./.bundle/ruby/2.4.0/gems/rspec-puppet-2.6.8/lib/rspec-puppet/monkey_patches.rb:275:in `require'
     # ./.bundle/ruby/2.4.0/gems/facter-2.5.1/lib/facter/util/wmi.rb:18:in `connect'
     # ./.bundle/ruby/2.4.0/gems/facter-2.5.1/lib/facter/util/wmi.rb:31:in `execquery'

That's a duplicate of #599: platform dependent facts being loaded, when they should be mocked.

Going to verify and close this if this is indeed the same issue which is what it sounds like.  Thanks!

closing since this is a duplicate

Duplicate of #599


MNT: New release and publish action

Handle both RC and stable releases in one action

this should publish a draft github release when a new tag includes the rc suffix

@knaaptime Let's merge?


Update describe_instance and add create snapshot

For the describe_instance function, wecreated variables for the list objects returned by GCP in order to make this data more useable. We also added a new action (function) to create a vm snapshot.

Added manual instance_data iteration to describe_instance function to make the data returned from this action more useable in FortiSOAR. Also added a new action (function) called disk_snapshot. This creates a disk snapshot of the VM boot disk.


Update Readme

Refers to proposed README update here https://github.com/twitter/finatra-activator-thrift-seed/issues/1
Adds Quick Start, Running and License sections to README.md
I realise quick start wasn't really discussed , I just thought it might be nice to let people see a quick client implementation, I also went ahead and made a client branch with said implementation to make sure it works.

All updates can be viewed in commit number 3 https://github.com/twitter/finatra-activator-thrift-seed/pull/2/commits/210538f82cf929be9817b69d035ddef6193962ef

@divanvisagie thanks!


feat(challenge): adminRespondToAuthChallenge, adminCreateUser SUPPRESS

Hey again,
I've added adminRespondToAuthChallenge, SUPPRESS for messageAction and legacy name ADMIN_NO_SRP_AUTH .

Looks great, thanks for the contribution. I'll merge ASAP.


Handle failure to activate extension and notify user

Handle errors in extension activation
Notify user with error message and information to diagnose this.

#3535 turned out to be a similar issue where ENOENT was being thrown.

That's a different issue, as the extension did activate (i.e. it didn't crash).
This issue is to track cases where extension falls over completely.


*: run clippy for integration tests

What is changed and how it works?
What's Changed:
Run clippy for integration tests and change script to also run it in CI
Related changes
Check List
Tests

No code

Release note
None

/merge


fix escaping if quote is present in field

Escape the field with double quotes if it contains a double quote in the value.
This is specified in the RFC as follows:

6 .  Fields containing line breaks (CRLF), double quotes, and commas
should be enclosed in double-quotes.

I also added a fix to the escape mechanism to use the user-specified field delimiter instead of hard-coded comma


Failure: install freetype

My OS is Win10, and I'm using VS 2017 community. I tried to use vcpkg to install freetype, but error is as follows.
D:\software\MeshLab>vcpkg install freetype
The following packages will be built and installed:
freetype[core]:x86-windows
Starting package 1/1: freetype:x86-windows
Building package freetype[core]:x86-windows...
-- CURRENT_INSTALLED_DIR=D:/software/vcpkg/vcpkg/installed/x86-windows
-- DOWNLOADS=D:/software/vcpkg/vcpkg/downloads
-- CURRENT_PACKAGES_DIR=D:/software/vcpkg/vcpkg/packages/freetype_x86-windows
-- CURRENT_BUILDTREES_DIR=D:/software/vcpkg/vcpkg/buildtrees/freetype
-- CURRENT_PORT_DIR=D:/software/vcpkg/vcpkg/ports/freetype
-- Downloading http://download.savannah.gnu.org/releases/freetype/freetype-2.8.1.tar.bz2...
-- Downloading http://download.savannah.gnu.org/releases/freetype/freetype-2.8.1.tar.bz2... Failed. Status: 7;"Couldn't connect to server"
CMake Error at scripts/cmake/vcpkg_download_distfile.cmake:155 (message):

  Failed to download file.
  If you use a proxy, please set the HTTPS_PROXY and HTTP_PROXY environment
  variables to "https://user:password@your-proxy-ip-address:port/".
  Otherwise, please submit an issue at https://github.com/Microsoft/vcpkg/issues
Call Stack (most recent call first):
ports/freetype/portfile.cmake:4 (vcpkg_download_distfile)
scripts/ports.cmake:72 (include)

Error: Building package freetype:x86-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with .\vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: freetype:x86-windows
Vcpkg version: 0.0.113-nohash

Additionally, attach any relevant sections from the log files above.

I notice that there is an issue about the problem. And I've tried "Project->C++->Command Line" to add " /source-charset: utf-8 " But it doesn't work.

judging by the log problem with the proxy

For me that link redirects to a mirror and the mirror is down. https://mirror.freedif.org
Status of mirrors: http://download.savannah.gnu.org/mirmon/savannah/

@Modori1998 Thanks for your feedback! I installed freetype successfully. Please open a new issue if this is still a problem for you.


Design: Where to put timestamps in envelope?

Some signing schemes, particularly around artifacts and metadata, rely on timestamp authorities to attest to when a signature was produced. (JARs and Authenticode are examples of this in the wild today).
In this case, the keyID and signature are no longer enough to actually verify that a signature is correct. We have to also check that the signature was produced during the validity window for that key. I think it might make sense to add a field for this somewhere in the envelope.

I don't think it needs to be in the signature envelope itself. It could be part of the signed content, but looking forward to objections here.

I don't think putting it in the signed content itself would work - what we really need is proof that the signature was created during the validity window of the key, not the signed content.
So we have something like:
Bob signs Payload A with Key A, getting Signature A.
Bob hashes this signature itself and asks the Trusted Timestamp Authority to "timestamp" that.
The Timestamp Authority generates a Payload B containing its own system clock and this payload digest, and signs that.
This way we have independent, non-repudiatable proof that the signature existed before a specific time (typically the key expiry)

Yes, this makes sense.

Dan, could you write a brief concrete proposal on how that would work in practice? It sounds like this wouldn't be just a timestamp but rather a single field containing both timestamp and signature. Are there any standards or conventions for how this is conveyed? Do different timestamp authorities use different ones?

There's a standard that contains both the timestamp and signature together defined in rfc3161. There are alternates such as https://github.com/cyborch/tsa-json, that also do the same thing.
I think the simplest would be add a new, optional field to the "signature" type to contain this, and treat it as an opaque byte string.

Do you also need a type to indicate how to interpret it?
And either way, this does not affect the signature itself (i.e. the PAE), correct?

We don't have a type on the sig field, so I don't think we would need a type here either: https://github.com/secure-systems-lab/signing-spec/blob/master/envelope.proto#L29
I'd be fine either way. And correct - this does not affect the PAE layer.

So basically we'd say that keyid is a hint for both sig and timestamp? That seems OK to start with.

So basically we'd say that keyid is a hint for both sig and timestamp? That seems OK to start with.

Mark, this is not clear to me. Could you elaborate?

I'd be fine either way. And correct - this does not affect the PAE layer.

Dan, do you mean sticking the timeserver-signed H(timestamp|signature) alongside the original signature?

My idea would basically be this modification, applied here: https://github.com/secure-systems-lab/signing-spec/blob/master/envelope.proto#L26
message Signature {
  // Signature itself. (In JSON, this is encoded as base64.)
  // REQUIRED.
  bytes sig = 1;

  // *Unauthenticated* hint identifying which public key was used.
  // OPTIONAL.
  string keyid = 2;

  // *Unauthenticated* timestamp attesting to when the sig itself occurred.
  // OPTIONAL.
  string timestamp = 3;
}

My idea would basically be this modification, applied here: https://github.com/secure-systems-lab/signing-spec/blob/master/envelope.proto#L26

Good: this is what I had in mind also. I don't think the keyid could be multiplexed for this, but i might have misunderstood Mark's proposal.

Yeah, I'm not sure I understand that part either.

To confirm, this is not blocking #37 because the timestamp does not appear in the PAE, correct?
Sorry for the confusion around keyid. The consumer needs to know how to interpret the timestamp field, namely what format and authority (public key) to use. Is the suggestion that this is tied to the public key used for sig (and hinted by keyid)? For example, if key X signed sig, then the consumer would know that the timestamp field is expected to be in the RFC 3161 format and signed by key Y?
A few other questions:

Does there need to be a separate "timestamp" field from the "timestamp signature"? Or is the timestamp itself embedded in the blob?
Is there a use case for more than one timestamp, such as in the case of having more than one authority?

To confirm, this is not blocking #37 because the timestamp does not appear in the PAE, correct?

I don't think this blocks the changes in #37 except for maybe the V1 part.

Whoops, wrong button.

Sorry for the confusion around keyid. The consumer needs to know how to interpret the timestamp field, namely what format and authority (public key) to use. Is the suggestion that this is tied to the public key used for sig (and hinted by keyid)? For example, if key X signed sig, then the consumer would know that the timestamp field is expected to be in the RFC 3161 format and signed by key Y?

Good questions. I don't think these crucial information (perhaps even part of the signed metadata) should implicitly tied to the keyid. Dan, WDYT?

In my understanding, we can put TimeStampToken as defined in RFC3161#2.4.2, which is encoded in ASN.1 BER, in the timestamp field.
As defined in RFC3161, we can extract the generalized timestamp from the TSTInfo. The real issues here are how we verify the time stamp token. Precisely, how do we find the public key / certificates to verify?
One is to set CertReq to true in the time stamping request to the TSA server, and the partial cert chain is embedded in the TimeStampToken. In this case, the consumer follows the PKI. Otherwise, information equivalent to timestamp.keyid is required unless the consumer is using few private timestamp servers. Apparently, the timestamp.keyid cannot be hinted by keyid since the signer and the TSA cannot be the same party.

@shizhMSFT that all matches my understanding.
There are a few other timestamp formats we're investigating too. Cc @asraa can give the latest updates there.

So it sounds like the consensus is to add a single timestamp field with semantics equivalent to sig: the producer and consumer agree on the details out of band?
If so, I'll send out a PR to add such a field.

I'm in no huge particular rush on this one, so if we want to leave this open to bake a little more that's fine with me.

Sounds good. It might also be good to start with an implementation before changing the spec.

So it sounds like the consensus is to add a single timestamp field with semantics equivalent to sig: the producer and consumer agree on the details out of band? And we can always add more fields later if needed, after trying this in the real world.

Yeah. Seems like it. Agreed that TimeStampToken for an RFC 3161 would match this. Other signature formats we're exploring based on signed notes would include the same information as this, including at least a reference to a certificate chain to use for verification.
Even then you would need a root CA to provide out of band to verify the chain against, so yes, I think you will need to agree on at least one thing to trust out of band.

We extended DSSE to support timestamps.  The timestamp is a counter signature to the signature on the data.
Here is our implementation:   https://github.com/testifysec/go-witness/blob/main/dsse/dsse.go#L56
We would like to upstream this


Request(Social): Add profilecolor command.

Aliases: pfc, mycolor.
The profile color command will be used to allow further customization of profiles and leveling cards. This color will also carry over to almost all embeds a user interacts with.

resolved in https://github.com/FoxxieBot/Foxxie/commit/27b31fd7df3720bec73cbaacf3e32a63a992115b


Package request: wlock

wlock: Itsy-bitsy sessionlocker
wlock is a itsy-bitsy sessionlocker for Wayland compositors that support the ext-session-lock-v1 protocol; an effective port of slock to Wayland, merging the alternate color patch to give a sense of feedback.
Metadata

homepage URL: https://codeberg.org/sewn/wlock
source URL: https://codeberg.org/sewn/wlock
license: gpl3+
platforms: linux

Add a :+1: reaction to issues you find important.

PR: #347893


fix spelling and edit validation messages

spotted a small typo in a validation message when evaluating this project for use and wanted to suggest making the validation messages more similar to each other.

Thank you! Sorry it took some time to merge :)


OCPBUGS-44013: add a controller for surfacing CNS registration issues before upgrades

We want to add a controller that would reveal CNS migration issues prior to upgrading to 4.14 where CSI migration is enabled for vSphere. This should help users troubleshoot the issue with specific volumes in time which is useful as observed in previously opened bugs (e.g.: https://issues.redhat.com/browse/OCPBUGS-43393)
This controller intentionally does not use CnsVSphereVolumeMigration CR, since the only reasons for using it are:

not break the driver after enabling CSI migration (upgrading) which should not be a concern because:

the only component reading and relying on the CR is controller publish, in this case the CR presence is taken care of during volume creation for new volumes and by syncer that actively migrates all volumes. The only real use of this CR in the driver appears to be mapping volumeIDs to volumePath and setting VM delete protection: https://github.com/kubernetes-sigs/vsphere-csi-driver/blob/da66c0af7c274836e2946dafae01a68bcdeba525/pkg/csi/service/vanilla/controller.go#L2109
the migration itself (manager.CreateVolume) should be also safe to call if the volume is already present: https://github.com/kubernetes-sigs/vsphere-csi-driver/blob/20a615225e6d542dc2bb08cac7b515f37a1af865/pkg/common/cns-lib/volume/manager.go#L487

skip already migrated volumes during reconcile to improve efficiency

Alternatively, we could be using the CR but there's some downsides too:

we would have to add quite a bit of code (~600L) that manages the CR creation/updates/cleanup and is almost a copy-paste from the driver
we risk creating the CR wrong creating potential issues when driver picks them up after enabling CSI migration

cc @openshift/storage

/assign @gnufied  for review

/jira refresh

@gnufied 4.13 tests are posted here - on fresh installation every new volume will be already registered by the syncer since CSI migration is enabled. I also tested 4.12 to verify the behavior with CSI migration disabled to see if the CNS registration works well with the controller.


[SPARK-14795][SQL] Remove the use of Hive's variable substitution

What changes were proposed in this pull request?
This patch builds on #12556 and completely removes the use of Hive's variable substitution.
How was this patch tested?
Covered by existing tests.

Test build #56490 has started for PR 12561 at commit c878aef.

https://github.com/apache/spark/pull/12561/commits/c878aef1ea7bb61e29961253a2e3510954b57348 looks good

Test build #56490 has finished for PR 12561 at commit c878aef.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class HiveSqlParser(conf: SQLConf) extends AbstractSqlParser

Merged build finished. Test FAILed.

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

Test build #56504 has started for PR 12561 at commit 737ea41.

Test build #56507 has started for PR 12561 at commit 8864f2a.

Test build #2841 has started for PR 12561 at commit 8864f2a.

Test build #2841 has finished for PR 12561 at commit 8864f2a.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #56507 has finished for PR 12561 at commit 8864f2a.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #56504 has finished for PR 12561 at commit 737ea41.

This patch fails Spark unit tests.
This patch does not merge cleanly.
This patch adds no public classes.

Build finished. Test FAILed.

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

Test build #56555 has started for PR 12561 at commit f74b381.

LGTM

Merging in master.

Test build #56555 has finished for PR 12561 at commit f74b381.

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

Merged build finished. Test PASSed.

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


Firmware make erro during "running dsdl compiler"

Hello guys:
As the topic and picture below, I need your help! I just clone the code yestoday.

Can you retry after a full clean? In my experience building on windows frequently has intermittent problems like this. I recommend you use Ubuntu in Virtualbox or VMWare.
make distclean
make px4fmu-v2_default

@gjhave Any update?

@dagar  Yes, I tried building in Ubuntu with a VMWare. It works, Thank you for your support！


Revert a80cbb2dcaa4d526287e04929d25f1da1bf61ed9

Reverts amplab/tachyon#1914
After this PR, vagrant deploys client jar to $HADOOP_CLASSPATH on starting the EC2 instances. However, it also makes start-yarn.sh fail to work. Revert this PR for now.

I'm personally fine with this, since then on deployments, we can actually exercise the scenarios where the client jar is not deployed everywhere (and use libjars option).

LGTM


DB: Adds updateFromV43 patch that adds unique index to storage_pools_config and networks_config table

Prevents duplicate config rows for the same node and key being inserted.
Uses IFNULL(node_id, -1)  to ensure unique index is still enforced for non-node specific config keys where node_id is NULL, and uses -1 as the pseudo value in order to avoid issues where the node ID is genuinely 0 which @stgraber advises is potentially possible on DBs from older installations.
I tried creating a new table with an updated UNIQUE constraint, but sqlite doesn't support the IFNULL expression in the CREATE TABLE statement.
Also updates previous duplicate row detection patch to use IFNULL(node_id, -1) too.
Fixes #8260
Signed-off-by: Thomas Parrott<EMAIL_ADDRESS>

Testsuite passed


chore: add launch task to start webpack and z-wave with F5

With this PR, one can just press F5 to start both the dev script (if not running yet) and the dev:server script for debugging, instead of having to start both manually via the CLI.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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 23.456%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3818

Relevant Lines:
17195

💛  - Coveralls


Cpr timeout fix

Different implementation of: https://github.com/jonathanslenders/python-prompt-toolkit/pull/688

Also seems to fix the issue.


[Service Connector] az webapp/spring/containerapp connection create mysql: Deprecate mysql single server connection command

Related command

az webapp connection create mysql
az spring connection create mysql
az containerapp connection create mysql

Description

Deprecate the above commands as Mysql Single Server is on the retirement path. https://learn.microsoft.com/en-us/azure/mysql/single-server/single-server-overview
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

Service Connector

Hi @kairu-ms, the pr is ready for review, can you help review it & complete it?


Shortest distance between nodes of BST in CPP

💥 Proposal
GSSoC'21
Given a Binary Search Tree, Find the shortest distance between given two nodes in CPP
Have you read the Contributing Guidelines on Pull Requests?
Yes

/assign


Correction for raw affiliation Department of Chemistry, Umeå University, SE-90187 Umeå, Sweden; Université de Rennes, Ecole Nationale Supérieure de Chimie de Rennes, CNRS, ISCR-UMR 6226, F-35000 Rennes, France

Correction needed for raw affiliation Department of Chemistry, Umeå University, SE-90187 Umeå, Sweden; Université de Rennes, Ecole Nationale Supérieure de Chimie de Rennes, CNRS, ISCR-UMR 6226, F-35000 Rennes, France
raw_affiliation_name: Department of Chemistry, Umeå University, SE-90187 Umeå, Sweden; Université de Rennes, Ecole Nationale Supérieure de Chimie de Rennes, CNRS, ISCR-UMR 6226, F-35000 Rennes, France
new_rors: 05kb8h459;01h0ffh48;015m7wh34;02feahw73;00adwkx90
previous_rors: 05kb8h459;01h0ffh48;015m7wh34;02feahw73
works_examples: W4387363061
contact: 2bd8667493d95949d0266456b566ced7:e4f1a9ed63f77cf170c3491160ed6feb @ univ-rennes.fr

This issue was accepted and ingested by the OpenAlex team on 2024-10-10. The new affiliations should be visible within the next 7 days.


Printout for Auto Greed / Need Should Wait until First Person Joins Group

Super low priority as the message will fire when they enter an instance too. We don't want it have to fire it every time someone joins... yeah meh, very low priority.

May just need small delay built in. Not a huge priority though.


soroban-rpc: Add ledger bumps preflight hack until core provides a way to fix it

it looks like most of the automated testing are failing with this change..

it looks like most of the automated testing are failing with this change..

They were already failing in the base branch (the CLI hasn't been brought up to date yet, causing all the Rust checks to fail)

@tsachiherman PTAL


If OLED off when unchecking Sleep when Inactive, it stays off

Describe the bug
A clear and concise description of what the bug is.
I just recently upgraded to firmware 1.0.0 and application <IP_ADDRESS>. If you uncheck "sleep when inactive" but the display is already asleep, the oled stays off, and volume controls do not work.
As a workaround, if I want the oled to stay always-on, I need to check "sleep when active," move the dial to wake it up, and then un-check it while the display is on. It then works as expected.
Desktop (please complete the following information):

OS: Windows 10 Pro x64

Good catch, thank you!


LDAP - Configurable strategy to identify users

NIFI-3020:

Introducing a user identity strategy for identifying users.
Fixing issue with the referral strategy error message.
Adding code to shutdown the application when the authorizer or login identity provider are not initialized successfully.

Reviewing..

Good call. I've updated the Admin Guide in a second commit. Thanks @ijokarumawak!

I am +1 now. If @ijokarumawak has no issues let's merge

@mcgilman Thanks for updating the docs! Exception handling at shutdown looks good. +1
Would you squash the commits to get it merged?

@mcgilman @ijokarumawak squashing and merging


Optimize Content::has, Introspector::query_label, and MetaElem

See the title.
It allows has to not allocate which while a small optimization on large documents and templates should really add up.
It also optimizes Introspector::query_label to be reference based, this does several things:

Removes cloning for RefElem that are actually CiteElem
Removes one clone in RefElem when it's an actual reference
Removes all cloning in FootnoteElem::declaration_location

Also makes MetaElem be Unlabellable to make it smaller. I'd go as far as making it a custom impl without location, span, etc. too

The difference these small changes makes is actually surprising, when editing references (i.e bibliography & headings) it actually provides a surprising 11% decrease in incremental compile time, so a bigger win than one might expect, likely due to the reduce amount of cloning. It also provides a small bump in cold compile of about 8% on masterproef which also surprises me, but we take the gains we get.

Thank you!


[Feature Support]The Flowise setup does not support OpenAI's newly released O1 series models. When attempting to configure these models, the following errors are encountered:

Describe the bug
The Flowise setup does not support OpenAI's newly released O1 series models. When attempting to configure these models, the following errors are encountered:

400 Unsupported value: 'messages[0].role' does not support 'system' with this model.
400 Unsupported value: 'stream' does not support true with this model. Only the default (false) value is supported.
400 Unsupported value: 'temperature' does not support 0.9 with this model. Only the default (1) value is supported.

To Reproduce
Steps to reproduce the behavior:

Go to the Flowise interface or API integration.
Configure the OpenAI O1 series model.
See the 400 error messages as described above.

Expected behavior
The O1 series models should support configurations for system roles, enabling message flow customization, and allow more flexibility with the stream and temperature parameters, similar to other OpenAI models.

https://openai.com/index/introducing-openai-o1-preview/


Filter qos parameters overrides in params tutorial

I'm not sure if it's better to add use the filter option or to update the output (which might change again in the future).

The ros2 param dump output is also not okay, and ros2 param load shows the following output in the turtlesim node:
Set parameter background_b successful
Set parameter background_g successful
Set parameter background_r successful
Set parameter qos_overrides./parameter_events.publisher.depth failed: parameter 'qos_overrides./parameter_events.publisher.depth' cannot be set because it is read-only
Set parameter qos_overrides./parameter_events.publisher.durability failed: parameter 'qos_overrides./parameter_events.publisher.durability' cannot be set because it is read-only
Set parameter qos_overrides./parameter_events.publisher.history failed: parameter 'qos_overrides./parameter_events.publisher.history' cannot be set because it is read-only
Set parameter qos_overrides./parameter_events.publisher.reliability failed: parameter 'qos_overrides./parameter_events.publisher.reliability' cannot be set because it is read-only
Set parameter use_sim_time successful

which isn't too bad, but not ideal.

(edit) It would be nice to have a --no-read-only argument in dump ros2 param dump --no-read-only.
Which is useful if you want to store the file to later use it with ros2 param load.

I'm honestly not sure what we want to do about read-only parameters.  On one hand, the fact that they are read-only means that there is no point in dumping them; we can never reload those values.  On the other hand, it might be nice for third-party tools to take a look at a parameter dump file and see all of the parameters that are set.
But we can discuss that elsewhere, to keep this PR focused on the filtering.

I'm not sure if it's better to add use the filter option or to update the output (which might change again in the future).

Yeah, I think in a beginner tutorial, it is better to not use advanced options.  The immediate question is "why?", and we can't really explain it very easily to a beginner without going into QoS and other topics.
So my opinion on this would be to update the output to match what the user will see.  I also don't see it as a big problem that it might change in the future; we have separate branches per distribution for exactly that reason, and we do this testing to find problems like this.

Which is useful if you want to store the file to later use it with ros2 param load.

It's worth noting that the same file with initial values for read-only parameters can be loaded via the command line (using --ros-args --params-file path/to/params.yaml).

It's worth noting that the same file with initial values for read-only parameters can be loaded via the command line (using --ros-args --params-file path/to/params.yaml).

That's a good point.  So yeah, I guess we want to continue dumping them.

Yeah, I think in a beginner tutorial, it is better to not use advanced options. The immediate question is "why?", and we can't really explain it very easily to a beginner without going into QoS and other topics.

See https://github.com/ros2/ros2_documentation/pull/1334/commits/0ff4d6a6a849fe2570d08f930bf19437a0fdb2d8.

It's worth noting that the same file with initial values for read-only parameters can be loaded via the command line (using --ros-args --params-file path/to/params.yaml).

Yes, that's true.
I added some minimal notes explaining the difference between ros2 param load and --params-file and how that affects loading read-only parameters.

Unrelated comment:

That's a good point. So yeah, I guess we want to continue dumping them.

I agree that we want to continue bumping read only parameters by default, but a --no-read-only option would be handy for when you want to use the file with ros2 param load.


[New Port Request] TUIO 1.1 and 2.0

Library name:
TUIO 1.1 and TUIO 2.0
Library description:
TUIO is an open framework that defines a common protocol and API for tangible multitouch surfaces. The TUIO protocol allows the transmission of an abstract description of interactive surfaces, including touch events and tangible object states. This protocol encodes control data from a tracker application (e.g. based on computer vision) and sends it to any client application that is capable of decoding the protocol.
Source repository URL:
TUIO 1.1:  https://github.com/mkalten/TUIO11_CPP
TUIO 2.0: https://github.com/mkalten/TUIO20_CPP
Project homepage (if different from the source repository):
https://www.tuio.org/
Anything else that is useful to know when adding (such as optional features the library may have that should be included):
TUIO 1.1 and 2.0 are distinct and incompatible protocols and should be available as two separate ports.
TUIO11_CPP contains sample programs SimpleSimulator, TuioDemo, and TuioDump which are useful tools for testing servers and clients and would be great to have available as a 'tools' feature.
TUIO20_CPP contains demo programs Tuio2Demo, Tuio2Dump, Tuio2Simulator  and Tuio2Test which are useful tools for testing servers and clients and would be great to have available as a 'tools' feature.
Both of these repositories have been unchanged for > 4 years.

If you wish to add this port in the future, you can reopen this issue, but we're closing it for now.


VisualScript callable nodes method_cache update fix

Issue referenced:  #37173
Bug:- VisualScript callable nodes were not getting updated when the properties
of their corresponding visualscript function nodes were changed.
Reason:- method_cache of the callable nodes was not getting updated when their corresponding function node was updated.
Results after the fix:-

I recommend against putting the code inside the _ports_changed function.
It will hurt the performance of the editor in the long run if we started to add all checks inside weird frequently executed functions.
And also make the code more harder to debug down the line.
I myself am clearing up a bunch of the old code from such functions.
For modifying script due to changes to nodes it's better to keep everything inside the Script and Nodes themselves.
It is both better for performance and complexity management.

Test


Is background color possible?

I've been attempting to change the background using a spritemap, however it's ignored in
var spriteMap = new THREE.TextureLoader().load('textures/white.png');
var spriteMaterial = new THREE.SpriteMaterial({
	map: spriteMap,
	color: 0xffffff
});
var mat = new THREE.Sprite(spriteMaterial);
for (j = 0; j < 100; j++) {
	let sprite = new THREE.TextSprite({
		textSize: 1,
		redrawInterval: 250,
		texture: {
			text: wrapText(titles[j], 30),
			fontFamily: 'Arial, Helvetica, sans-serif',
		},
		mat
	});

I assume I'm doing this entirely wrong, is it possible to add background colors in some way?

Try to set material to not transparent.

The object returns transparent:false when I console log it and it's opacity is also set to 1. I've edited the above code to use your example so that it's more readable, I'm sure I'm missing something simple.

You are using it wrong. Look inside readme.
https://github.com/SeregPie/THREE.TextSprite#members


Does this work also with Sublime Text 3?

It's only tested with Sublime Text 2. I have no idea if it works with Sublime Text 3.


Refactor runtime.go to move component lifecycle managment into the runtime processor

PR moves the component lifecycle management to the relevant runtime processor. The intention of this PR is to further abstract each component type so that we can achieve graceful shutdown and hot reloading. This refactor should also help to improve the maintainability and test-ability of runtime.
Closing components are now done concurrently. Daprd will return exit 0 as soon as all components, servers, topics etc. have closed, and will fatal if the graceful timeout is exceeded during shutdown.

@JoshVanL does this PR contain behavioral changes as part of the refactor, like changes to shutdown order / component close sequence? (I assume this is all to support hot-reloading) - If so, it'd be helpful for reviewers to call these out on the description

/ok-to-test

/ok-to-perf

/test-sdk-all

/test-version-skew


Update SFWater logo to link to Adopt-a-Drain page

Fixes #257

Coverage remained the same at 98.491% when pulling b6766fb9b9bace843832d7ba5e0b58dd086ebf6f on update-sfwater-link into<PHONE_NUMBER>0ccbd9d494c1c8fa1fdedb69838c5a on production.


missing dataset title label, when plotting an expression or user variable

Reported by andrew wittenberg on 29 Jul 2016 21:47 UTC
Reported to me by Will Cooke at GFDL.
In the following example, why is the bottom-right plot missing its DATASET TITLE label?  The dataset title should be fully discoverable, since there is no inter-dataset conflict within the variable definition.
 	NOAA/PMEL TMAP
 	FERRET v7.01 (beta/optim)
 	Linux 2.6.32-642.1.1.el6.x86_64 64-bit - 07/20/16
 	29-Jul-16 17:30

yes? use coads_climatology
yes? save/clob/file="sst.nc"/l=1/title="DATASET TITLE" sst
 LISTing to file sst.nc
yes? use sst.nc
yes? can view; set view ul; shade sst
yes? set view ur; shade sst*2
yes? set view ll; shade sst[set view lr; shade sst[d=2](d=2]
yes?)*2

Output is attached as plot1.gif.
The following is even weirder:
yes? let v = sst[can view; set view ul; shade v
yes? let/d=2 v = sst[d=2](d=2]*2;)*2; set view ur; shade v
yes? let v = sst[d=2]*2; set view ll; shade v

Output attached as plot2.gif.  Why does the bottom-left plot have a title, when the top-left one doesn't, even though we issued the same command with the same variable definition?
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/2460

Modified by karl.smith on 29 Jul 2016 22:01 UTC

Comment by ansley.b.manke on 29 Jul 2016 22:09 UTC
There is a concept of "the dataset of a variable", including file varibles, and for user-defined variables when there's a single dataset identified which the variable depends upon. There's an obvious inconsistency or two here.

Comment by ansley.b.manke on 13 Oct 2016 18:19 UTC
For the second part of the example,
yes? let v = sst[can view; set view ul; shade v
yes? let/d=2 v = sst[d=2](d=2]*2;)*2; set view ur; shade v
yes? let v = sst[set view ll; shade v

the upper-right and lower-left panels are both plotting the LET/D=2 variable. Once the LET/D=2 variable has been defined it takes precedence over the unattached LET variable of the same name.  Change the definition of the LET/D= variable,
yes? let v = sst[d=2](d=2]*2;)*2; can view; set view ul; shade v
yes? let/d=2 v = sst[set view ur; shade v
yes? let v = sst[d=2](d=2]*4;)*2; set view ll; shade v

Now it's clear from the plot labels that the last "shade v" drawing the lower-left plot is showing us sst[and it has the same dataset labels as the upper-right one. So that part of the script is showing consistent behavior.
The question is why the plot of sstd=22 does not get a dataset label, but sst2 and sst[both do.  As Ferret is getting ready to load the plot, it's evaluating the context, so it can go read data and do any operations on it before sending the loaded data to be plotted. One of the elements of the context is "the data set", from which Ferret's getting data.
In general terms, as it's working along to load the data, when Ferret sees sstd=2 it sts flags to say "we have the  dataset info given".  sst*2  gets marked as "need to find the dataset", and then a bit later, the datset is determined to be dataset 2, and that information is saved.  The problem is that the flags used to determine the context work perfectly to get the data, but the information is not always kept on the context stack to the end of the process where it can reliably be used to make plot labels.
I have not been able to find a consistent way to preserve the information that doesn't break the way info is stored in the stack as it loads expressions into memory.  I think that the linked-list management of user-variables could be used to handle this.
Leaving the ticket open.

Attachment from andrew.wittenberg on 29 Jul 2016 21:47 UTC
REPLACE THIS TEXT WITH UPLOADED FILE ./attachments/TRAC_2460_GIT_1732/plot1.gif

Attachment from andrew.wittenberg on 29 Jul 2016 21:47 UTC
REPLACE THIS TEXT WITH UPLOADED FILE ./attachments/TRAC_2460_GIT_1732/plot2.gif


Don’t set SWIFT_BUILD_SCRIPT_ENVIRONMENT when building swift-format

SWIFT_BUILD_SCRIPT_ENVIRONMENT should only be set when building swift-syntax. All other projects should be building swift-syntax without any swift-syntax-specific environment variables set.
Also remove SWIFT_SYNTAX_PARSER_LIB_SEARCH_PATH, which isn’t needed anymore since we re-wrote the parser in Swift and we no longer have the C++ parser library.

@swift-ci Please test


How to run the examples?

Sorry for the dumb question, but I really need your help on this.
Thank you in advance.

Sorry. I managed to figure it out. I just couldn't imagine how a simple javascript sample project can become so complicated. :(

How were you able to do it?

For anyone who can't figure it out.

Install yarn in your OS.
Go to the root directory of this project
Type 'yarn install' in CLI
Type 'yarn run' in CLI
Then the CLI will output the examples that can be run. (Or instead of step 4. you can do yarn run simple-cli for example)


Questions regarding POST /bookings/{bookingId}/fulfillments

Questions regarding POST /bookings/{bookingId}/fulfillments:

with asynchronous fulfillment, will all providers return 202 Accepted to a POST /bookings/{bookingId}/fulfillments?
what is the status of the fulfillment(s) after this? CONFIRMED? FULFILLED? Either?

Observations so far
When a retailer invokes distributor OSDM, either gets 200 and Fulfillments FULFILLED, or 202 and Fulfillments will be later available in the Booking in FULFILLED state.
When distributor performs the POST fulfillment, it may directly create Fulfillmens in CONFIRMED state to indicate it was paid/sold, assign the IDs, and have them FULFILLED once the FulfillmentDocuments are ready.
Retailer can verify in the booking if all Fulfillments were asynchronously provided by checking all booked offer parts for status = FULFILLED. Status CONFIRMED means it was paid/sold but Fulfillment is not ready yet.
PATCH Fulfillments is used exceptionally in cases when providers use Fulfillments as proof of delivery and cancel of booking is no longer possible.

It is not relevant for fare providers as they do not create fulfillments. -Clemens

Models - Fulfillment State Model: remove the call of patch fulfillments, as it is used for activation only. Instead, label it as a "time event".
openapi - update docs of patch fulfillment to indicate it is used for activation process.


Hambuger pattern

in the ControlInfoData.json about the Hambuger pattern the link is "Title": "Hamburger pattern", "Uri": "https://github.com/Microsoft/Windows-universal-samples" }. I do not think that explains anything about the Hamburger pattern, if there is a link about this, is interesting to update to the right link

This will be fixed in our next refresh. Thanks for reporting it.


Dont Update Up

update's our detection coverage mitre map
adds a log4shell correlation search
updates spec to include the Risk datamodel

I am going to refactor the search to use SES observable fields. add WIP flag do not merge :smile:


centos7-run: mysql public key not installed

building the public/centos7-run.dockerfile currently fails https://github.com/smanders/buildpro/blob/22.01/public/centos7-run.dockerfile#L45-L54
warning: /var/cache/yum/x86_64/7/mysql80-community/packages/mysql-community-client-plugins-8.0.28-1.el7.x86_64.rpm: Header V4 RSA/SHA256 Signature, key ID 3a79bd29: NOKEY
Public key for mysql-community-client-plugins-8.0.28-1.el7.x86_64.rpm is not installed
Public key for mysql-connector-odbc-8.0.28-1.el7.x86_64.rpm is not installed
--------------------------------------------------------------------------------
Total                                               49 MB/s |  68 MB  00:01
Retrieving key from file:///etc/pki/rpm-gpg/RPM-GPG-KEY-mysql
Importing GPG key 0x5072E1F5:
 Userid     : "MySQL Release Engineering<EMAIL_ADDRESS> Fingerprint: a4a9 4068 76fc bd3c 4567 70c8 8c71 8d3b 5072 e1f5
 Package    : mysql80-community-release-el7-3.noarch (@/mysql80-community-release-el7-3.noarch)
 From       : /etc/pki/rpm-gpg/RPM-GPG-KEY-mysql

Public key for mysql-community-client-8.0.28-1.el7.x86_64.rpm is not installed

 Failing package is: mysql-community-client-8.0.28-1.el7.x86_64
 GPG Keys are configured as: file:///etc/pki/rpm-gpg/RPM-GPG-KEY-mysql

The command '/bin/bash -c yum -y update   && yum clean all   && yum -y install --setopt=tsflags=nodocs      https://repo.mysql.com//mysql80-community-release-el7-3.noarch.rpm   && yum -y install --enablerepo=mysql80-community --setopt=tsflags=nodocs      mysql-community-client      mysql-connector-odbc   && yum clean all' returned a non-zero code: 1

similar issue seen in VIVID development https://isrhub.usurf.usu.edu/ajgraham/VIVID/commit/e84b7cc3fff1ad693a57063ca14a9e0ac998ff13
https://codingbee.net/uncategorized/yum-error-public-key-for-rpm-is-not-installed

completed with commits to dev branch referenced above


Include new Video Model

Jason has trained a new model for Videos. Can we get the new Model as well?
Maybe you can also throw some light on the conversion process from the pytorch pth file to your model file and why you have chosen to put some of the file content into code.

DeOldify.NET it does not support video processing, so there are no models for video in the list. The conversion of pth to custom files takes place in several stages:

spectral_form and weight_form are removed from the model in PyTorch (in fact, they are not just removed, but the weights and biases of convolutional layers are calculated).
All weights from pth are sequentially stored in a binary file without separators. At the same time, C# code is generated to read weights from this file. Since the data in the binary file is not structured, the weights must be read in the same sequence in which they were written. The generated C# code allows you to get a dictionary with named weights (like state_dict in PyTorch).


[Feature] Request: wxListCtrl [sf#4]

Reported by shadowlore on 2003-03-13 10:18 UTC
I use the wxListCtrl control quite a bit, and was
disappointed when looking at duplicating a Boa built
wxPython app in wxGlade. Any idea when wxListCtrl will
be available?

Commented by agriggio on 2003-03-13 10:37 UTC
Logged In: YES
user_id=375945
I borrowed Guido's time machine and added it on CVS a couple
of days ago ;-)
(Anyway, the support is at the moment very basic) If you try
it, let me know what you think.
Thanks,
Alberto


TypeError: wer() got an unexpected keyword argument 'standardize'

Traceback (most recent call last):
File "main.py", line 222, in
main(0, args)
File "main.py", line 121, in main
model.fit(dataset_train,
File "EfficientConformer-master/models/model.py", line 344, in fit
raise e
File "EfficientConformer-master/models/model.py", line 303, in fit
wer, truths, preds, val_loss = self.evaluate(dataset, val_steps, verbose_val, eval_loss=True)
File "EfficientConformer-master/models/model.py", line 425, in evaluate
batch_wer = jiwer.wer(outputs_true, outputs_pred, standardize=True)
TypeError: wer() got an unexpected keyword argument 'standardize'

jiwer.wer(outputs_true, outputs_pred, standardize=True)

I met the same issue. Just change jiwer version to 2.2.1.
pip install jiwer==2.2.1

Hi, the standardize keyword was removed in newer versions. The repo was not updated yet so the current requirements.txt file do not specify the version to install. This is why you probably downloaded a newer version.


⚠ T:Riza Roblox Proxy has degraded performance

In 855549d, T:Riza Roblox Proxy (https://rprxy.triza.dev) experienced degraded performance:

HTTP code: 200
Response time: 1993 ms

Resolved: T:Riza Roblox Proxy performance has improved in 54cdc08.


Use observer from zen observable package

Use observer from zen observable package to compile link with latest version of apollo-link

Codecov Report

Merging #16 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #16   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines          25     25
  Branches        3      3
=====================================
  Hits           25     25

Impacted Files
Coverage Δ

src/QueueLink.ts
100% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Thanks @wtrocki!

Oh weird, you used double quotes for the import and it failed lint when I ran it locally. I wonder how it passed on Travis CI.


AWS API Gateway, Lambda & DynamoDB

Would be interesting to have another server mode that could run this as on AWS Lambda behind API Gateway, loading its YAML files from another datasource such as DynamoDB or S3 (could adding yaml files/datasources be pluggable perhaps rather than filesystem?)
https://zappa.gun.io/ does something similar with Flask, though it creates all the API Gateway configuration - something which Terraform and Apex can also do, but in a lighter weight deployment mechanism (ie without virtualenv) and with more control.
API Gateway of course now has Swagger import also, but this has to be done upfront with creation of API Gateway infrastructure so less dynamic in nature than what Connexion could potentially offer (plus ability to run APIs locally or on non-serverless based VMs/containers as it currently does)
Any thoughts on the feasibility / interest on this appreciated

managed to do something similar to zappa and invoke out-of-the-box connexion running as lambda behind API Gateway using apex
https://4wf2zjem76.execute-api.us-east-1.amazonaws.com/dev/greeting/bob
would need to do all the middleware mapping as zappa however (might just be easier to user zappa to call connexion :-) )

@rog555 That sounds like an interesting setup. But I cannot imagine how Connexion could change to help on that. It sounds more like a separated tool that would deploy Connexion apps in AWS Lambda and as you said seems like Apex did the job. I will give a try to this use case to get more familiar with this idea.

@rog555 it was pretty straight forward to deploy the connexion example project with zappa https://vgd8lgzajc.execute-api.us-east-1.amazonaws.com/dev/pets
I will close this issue since it has nothing to do with Connexion.


Clean up food chain - continued

re #132
Continuing to normalize the strings and in so doing making the Heredoc
the One Pattern here.  It is a little bit of overkill maybe for the
first 2 or three tests, but soon makes sense to do this, May as well get
them all in the same pattern.

I deliberately did not use the heredoc for the individual verses, since I don't particularly like using heredocs in the example solution (which tends to be what happens because the easiest thing to do is to copy/paste from the tests). The place where it makes sense to stop copy/pasting is the multiple verses strung together.
I may just be over-thinking things. What do you think?

I definitely had the same feeling, that the test can direct the solution.  For what I want to do was to make an array and test it against the array and clean up the tests, as they are not DRY... and since this is not a  normal project, it would be appropriate to do so.  Still, it is something that can bias the solution.
The solution to that, then, becomes having an external source for the song lines, and forming the expected result from that.  And using the library as the actual result.
The test becomes more cryptic, but the reporting from the test remains helpful, hopefully not biasing the solution.

I'd prefer it if the tests don't get clever. Ideally, I'd only want one test--the last one. The rest are really just training wheels.
What if we only have three tests:

a place-holder test that says: "you know how this works, go ahead and TDD it yourself"
the final test that shows where we're headed.
the book-keeping test (VERSION).

Details in the commit message.  Let me know what you think.
Are we thinking that the self-drive TDD will eventually also be shared to be nitpicked?  Or a self-evaluate kind of thing?
Hoping you find the song text being in the external file is not any more clever (perhaps even less clever) than using the `" character to escape the new line being used to continue a line of code, in the other versions of the test.

One other thing, I did hear you about not lilking to use the heredoc solution in the example.rb.  It may be less 'clever' than using the " \ <newline>" syntax to continue the line.

I usually find HEREDOCs to be completely fine. I've avoided them with beginners because it seems to confuse and distract them. Probably by this point it won't be confusing.
I think the example looks good. I also think it's nice to read from song.txt. It's not clever, and it keeps the test file easier to read.

Are we thinking that the self-drive TDD will eventually also be shared to be nitpicked? Or a self-evaluate kind of thing?

My thought is just self-evaluate.
I don't think we need more than the single test that reads from song.txt. At this point they've done enough problems that we can say "write whatever tests will help you solve this".

Oh. Hm. I wonder if it's failing because the song.txt doesn't get copied when make test puts it in a temp dir. Investigating.

Try adding this right after line 23 of the Makefile (@echo "running tests for: $(ASSIGNMENT)").
@cp -r $(ASSIGNMENT)/ $(OUTDIR)

You can test it locally with make test-assignment ASSIGNMENT=food-chain


feat: add nversion support to azurekeyvault provider certificates and keys

Description
What this PR does / why we need it:
Adds a configuration option to the Azure Key Vault provider of KMP called versionHistoryLimit that allows the user to define the number of versions of a certificate and or key to store in the Ratify KMP cache for artifact verification. Needed to fully implement the refresh interval feature proposed in design doc for KMP periodic retrieval.
Which issue(s) this PR fixes (optional, using fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when the PR gets merged):
Fixes #1751
Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

Note: Might be breaking, requires workload identities to have certificate & key List permission on AKV.

[x] This change requires a documentation update

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

[x] Confirm that multiple certificates/keys are correctly stored in the KMP.
[x] Ensure that the refresh process updates the KMP with the latest versions
[x] Verify that the latest version is selected when versionHistoryLimit is not specified.
[x] Prevent runtime errors by handling cases where versionHistoryLimit exceeds the total available versions.
[x] The refresh process updates the total versions whenever new certificate or key versions are created, even if the specified limit exceeds the number of existing certificates.

apiVersion: config.ratify.deislabs.io/v1beta1
kind: KeyManagementProvider
metadata:
  name: akv
spec:
  type: azurekeyvault
  refreshInterval: 1m
  parameters:
    vaultURI: https://ratifyakvs5r.vault.azure.net/
    certificates:
      - name: ratify
        versionHistoryLimit: 2
    keys:
      - name: test
        versionHistoryLimit: 2
    tenantID:
    clientID:

## KMP Status showing multiple versions stored
 properties:
    Certificates:
    - Enabled: "true"
      LastRefreshed: "2024-11-27T14:16:56Z"
      Name: ratify
      Version: 230b4bef7a2d435e80235f04310320df
    - Enabled: "true"
      LastRefreshed: "2024-11-27T14:16:56Z"
      Name: ratify
      Version: 67ae6208331e4ab8a03229a10bce9a6e
    Keys:
    - Enabled: "true"
      LastRefreshed: "2024-11-27T17:10:13Z"
      Name: test
      Version: 4f74798850e242c3914545961cdf595e
    - Enabled: "true"
      LastRefreshed: "2024-11-27T17:10:13Z"
      Name: test
      Version: 7df0412c291f46fc9112dc0df2df6a22

Note: Image showing latest versions of the certificate are stored in KMP

Note: Image showing latest versions of the key are stored in KMP

Checklist:

[x] Does the affected code have corresponding tests?
[ ] Are the changes documented, not just with inline documentation, but also with conceptual documentation such as an overview of a new feature, or task-based documentation like a tutorial? Consider if this change should be announced on your project blog.
[ ] Does this introduce breaking changes that would require an announcement or bumping the major version?
[ ] Do all new files have appropriate license header?

Post Merge Requirements

[ ] MAINTAINERS: manually trigger the "Publish Package" workflow after merging any PR that indicates Helm Chart Change

Discussion: Should the user be able to define multiple specific version of a certificate?
spec:
  type: azurekeyvault
  refreshInterval: 1m
  parameters:
    vaultURI: https://ratifyakvs5r.vault.azure.net/
    certificates:
      - name: ratify
        versions:

Confirming the expected behavior for the following examples:
Versions: a,b,c
spec:
 type: azurekeyvault
 refreshInterval: 1m
 parameters:
   vaultURI: https://ratifyakvs5r.vault.azure.net/
   certificates:
     - name: ratify
       version: "b"
       versionHistoryLimit: 1

Should return: a,b (specified version + nversion)
spec:
 type: azurekeyvault
 refreshInterval: 1m
 parameters:
   vaultURI: https://ratifyakvs5r.vault.azure.net/
   certificates:
     - name: ratify
       versionHistoryLimit: 2

Also returns: a,b

@susanshi @binbin-li @akashsinghal the PR is ready for a final round of review. :)

Just a side note: I completed the refactoring work in a separate branch. If everyone is good with the changes in this PR, I'll submit the refactoring as a different PR for review. The functionality doesn't change at all. I simply added additional methods and functions to improve readability and unit testing.

Hi @binbin-li , this is approved, but lets hold off merge till after we have cut the release.


Update TensorFlowLiteC.framework download link in README

The file at https://github.com/tensorflow/flutter-tflite/releases/download/v0.5.0/TensorFlowLiteC.framework.zip doest exist

Hi @Theunodb - can you link to the README that you're referencing? I don't see the URL in question to fix. Thanks!

Hi Paul, i see it was removed in the latest readme. It was under the (Important) Initial setup : Add dynamic libraries to your app section in the original readme:
iOS
Download TensorFlowLiteC.framework. For building a custom version of tensorflow, follow instructions in wiki.
Place the TensorFlowLiteC.framework in the pub-cache folder of this package.
Pub-Cache folder location: (ref)
~/.pub-cache/hosted/pub.dartlang.org/tflite_flutter-/ios/ (Linux/ Mac)
%LOCALAPPDATA%\Pub\Cache\hosted\pub.dartlang.org\tflite_flutter-\ios\ (Windows)

Ah got it - iOS instructions were going to be written by someone that updated the sample, but I'm not sure when/if that'll happen :) I'll run through this after IO to see if I can get it fixed. Thanks!

Alright link in the readme should be fixed. Reopen this if it doesn't work out for you :)


how to use hystrix-javanica by cglib

not spring project, use cglib aop, I just want to use javanica by my own code, but all the test example is on the spring. can you give me some tips ? thank you.

It uses aspect j aop specifically. You can read about configuring it without spring here http://www.eclipse.org/aspectj/doc/next/devguide/ltw-configuration.html

I'm confused about that, why spring project can use jdk or cglib proxy, thank you all the same


docs(state-query/dozer): add live query and metadata discovery 🚗

Can we break this into table metadata and live query?
Live query on Dozer is not quite ready (we are missing a notification from Dozer when a record gets deleted).

Can we break this into table metadata and live query? Live query on Dozer is not quite ready (we are missing a notification from Dozer when a record gets deleted).

Yes, see https://github.com/latticexyz/mud/pull/3186 .

Live queries don't really work (due to the lack of a delete), so no need for this yet.


Scroll event triggered twice on touch screen scroll

When listening for scroll events on the window and scrolling on a mobile device with a touch screen the scroll event will sometimes end and then start again and quickly finish. I think it might have something to do with the way the mobile browser resizes and/or handles overflow scroll on the window.
This is the code I'm using to listen for the scroll events:
  const bind = useGesture({
    onScroll     : () => {
      // ...code
    },
    onScrollStart: () => {
       // ...code
    },
    onScrollEnd  : () => {
       // ...code
    },
  }, { domTarget: window })

  useEffect(bind, [bind])

Steps to reproduce:
Clone this repo: https://github.com/rynoV/calumsieppert.me-v2
Yarn install and run:
yarn develop-mobile
(You might need to install the gatsby cli, I'm not sure)
Open ip address given by the command on a mobile device. (I was using an iphone 6 with safari and chrome)
Watch how the scroll events are triggered.

Hey, thanks for reporting the issue, but I'm not sure I can reproduce. Can you please clarify and possibly set a simple example on codesandbox (ideally not using console.log to make iPhone debugging easier)?
Just to clarify, depending on the version you're using, Safari iOS doesn't handle scroll events the same way. Also, the onScrollEnd event is debounced (there's no native scrollend event) with a timeout of 100ms. If for some reason the scroll bounces and doesn't emit events for over 100ms and then starts again, you'll indeed have a end scroll firing, and the start and then quickly finishes as you describe.

Thanks for the quick response! Here is a codesandbox where you should be able to reproduce the behaviour I described.

Hey @rynoV I changed the default timeout from 100ms to 120ms and released 5.2.2 but I'm not sure it helps... Let me try this once more when I get home.

Seems to have fixed it, thanks a lot!


Added Decimal formatting

The implementation is quite naive; we will improve it overtime. Getting this particular overload to be fast/non-allocating is not lower priority. I filed #1765 to track this.
cc: @ahsonkhan, @joshfree, @shiftylogic

The boilerplate code looks fine to me.  Is this the plan to share some common code and reuse tests with the existing UTF16 string routines?

The boilerplate code looks fine to me. Is this the plan to share some common code and reuse tests with the existing UTF16 string routines?

If the routines get larger, then we should think about reuse. For now (given the tests are 4 lines), trying to reuse would make the code hard to read, IMO.


JBEHAVE-1215

Add support for Russian locale.
Add i18n/keywords_ru.properties with Russian keywords map

Additional I append keywords_ru_sbt.properties for ru_sbt locale (Russian keywords map of SBT)

Update test