Initiating a drag event when nothing is selected causes a crash

Hold the cursor pressed over an element in the list
Press Esc while still holding, to deselect the element
Drag the cursor around
Crash

It happens because of the static function 'filter_descendant_xids' at
res://src/utils/Utils.gd line: 113
the 'new_xids' array is empty
but at line: 113
you are assigning 'last_acepted' to a value don't exist:  Out of bound exception
i honestly didn't explore the project code further
but a temporary quick fix :
static func filter_descendant_xids(xids: Array[PackedInt32Array]) -> Array[PackedInt32Array]:
	if xids.is_empty():
		return xids
	var new_xids: Array[PackedInt32Array] = xids.duplicate()
	new_xids.sort_custom(Utils.compare_xids_r)
	# Linear scan to filter out the descendants.
	var last_accepted := new_xids[0]
	var i := 1
	while i < new_xids.size():
		var xid := new_xids[i]
		if Utils.is_xid_parent_or_self(last_accepted, xid):
			new_xids.remove_at(i)
		else:
			last_accepted = new_xids[i]
			i += 1
	return new_xids


RangeStart/RangeEnd Time Zone

[Believe when you enter the RangeStart/RangeEnd date/time value, it got to be in UTC timezone. Would be good to mention that.]

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

ID: 703aa6c4-1701-ea81-5fe6-c0afd163021e
Version Independent ID: 13601081-10c5-3947-f19b-daa8c44a5382
Content: Configure incremental refresh and real-time data for Power BI datasets - Power BI
Content Source: powerbi-docs/connect-data/incremental-refresh-configure.md
Service: powerbi
Sub-service: pbi-data-sources
GitHub Login: @Minewiskan
Microsoft Alias: owend

Thanks for your comment, @bharath47. This issue has been assigned to the doc author for review.

@bharath47 - Did you run into a scenario where specifying a non-UTC time for the parameters caused an issue? For example, Scheduled Refresh is UTC, but parameters specified were non-UTC. Can you describe why you think this is important to mention.
Thanks

@Minewiskan - I'm working in NZT time zone (and the same configured in my locale settings); When i first tried this feature, i set the RangeStart date as 1/04/2022 12:00:00 pm (intention is to use 1-Apr-2022), but noticed, the records changed prior to Apr were pulled in. Later i changed it to 4/01/2022. Please let me know if my understanding is incorrect, will recheck.

@bharath47 - NZT is UTC+12 or 13. Is the date/time format in the date column of your data in dd/mm/yyyy or mm/dd/yyyy format?


fix: also publish to version tag if on master, not just to master

Overview
A switch condition was being hit to deploy with master as the tag if we were on master, but it was NOT also deploying to the version when i think it should.
This has left the docker composes unupdated after the last release, when they should have been updated
WARNING: I HAVE NOT RAN THIS SCRIPT CHANGE

https://github.com/AztecProtocol/aztec-packages/pull/10095 solved by


add: Better handling of environment values in docker-compose usage

Added better handling of environment variables in docker-compose.yaml files especially when there are different values for local testing instances.
I also added some light in-file documentation for the usage of environment values in the .env files.
Note: Please also read my in-file comments about : usage in environment keys.

Nice bit of documentation, and I like the idea of placing a .env file for configuration.
Let's review and merge on the next stream


TST, CI: GitHub Actions MacOS Failures

GitHub Actions MacOS CI jobs have been failing consistently. A sample log is available here: https://github.com/scipy/scipy/pull/13094/checks?check_run_id=1436142836
SciPy itself is built successfully, but runtime access to gfortran dynamic libraries (via openblas lib) is compromised, causing a cascade of failures during the testsuite that look like this:
E   ImportError: dlopen(/Users/runner/work/scipy/scipy/build/testenv/lib/python3.8/site-packages/scipy/spatial/qhull.cpython-38-darwin.so, 2): Library not loaded: /libgfortran.3.dylib
E     Referenced from: /Users/runner/work/scipy/scipy/openblas/lib/libopenblasp-r0.3.9.dylib
E     Reason: image not found
_ ERROR collecting build/testenv/lib/python3.8/site-packages/scipy/cluster/tests/test_hierarchy.py _
ImportError while importing test module '/Users/runner/work/scipy/scipy/build/testenv/lib/python3.8/site-packages/scipy/cluster/tests/test_hierarchy.py'.

There are quite a few shims in that CI code that deal with dynamic library behavior--I'll try to check this today as it will be important for CI to be consistently green with 1.6.0 branching imminent. If someone else is already close to a solution please chime in to avoid duplicated effort.

It looks like the major culprit is qhull not being able to load/find the correct libgfortran library.


Generic type errors in the v2 SupabaseClient

Bug report
Describe the bug
TS2314 errors after updating the supabase-js module to v2
A clear and concise description of what the bug is.
my Angular v14 project is not starting because I receive several 'TS2314: Generic type' errors coming from the SupabaseClient class
To Reproduce
I am not able to reproduce the problem on StackBlitz
I just updated a working supabase-js v1 project to v2 and started receiving these errors when trying to start the project locally
Expected behavior
My project would start and I can continue working on it
Screenshots
I added 3 screenshots: The errors, the Angular service in which I initialize the client and the package.json
System information

OS: [e.g. Windows]
Version of supabase-js: 2.0.1
Version of Node.js: 16.16.0

Hey there, can you update the lib and verify if it works now?

I confirm that the problem was solved by the update. Thank you for fixing this in such a short time.


SPFx 1.18 scaffolded with --use-heft does not compile because of missing webpack SCSS loader

Target SharePoint environment
SharePoint Online
What SharePoint development model, framework, SDK or API is this about?
💥 SharePoint Framework
Developer environment
Windows
What browser(s) / client(s) have you tested

[ ] 💥 Internet Explorer
[ ] 💥 Microsoft Edge
[ ] 💥 Google Chrome
[ ] 💥 FireFox
[ ] 💥 Safari
[ ] mobile (iOS/iPadOS)
[ ] mobile (Android)
[X] not applicable
[ ] other (enter in the "Additional environment details" area below)

Additional environment details

SPFx 1.18 rc1
Node.js v16.20.2
pnpm 7.32.5

Describe the bug / error
New SPFx v1.18 project scaffolded with --use-heft most likely is missing some dev dependencies or is using incomplete webpack configuration, or there is some problem with scss file itself. If you try to build project with OOB web part, process fails while trying to parse SCSS file:
PS C:\Solutions\Experiments\spfx-1.18-rc1-patched> pnpm build

><EMAIL_ADDRESS>build C:\Solutions\Experiments\spfx-1.18-rc1-patched
> heft test --clean

 ---- build started ----
[build:clean] Deleted 2 files and 5 folders
[build:sass] Generating sass typings...
[build:sass] Generated sass typings
[build:typescript] Using TypeScript version 4.7.4
[build:lint] Using ESLint version 8.48.0
[build:typescript] Copied 10 folders or files and linked 0 files
[build:webpack] Using Webpack version 4.44.2
[build:webpack] Running Webpack compilation
[build:webpack] Error: ./lib/webparts/helloWorld/components/HelloWorld.module.scss 1:0
[build:webpack] Module parse failed: Unexpected character '@' (1:0)
[build:webpack] You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
[build:webpack] > @import<EMAIL_ADDRESS>[build:webpack] |
[build:webpack] | .helloWorld {
 ---- build encountered an error (32.642s) ----
-------------------- Failed (32.646s) --------------------
Encountered 1 error
  [build:webpack] ./lib/webparts/helloWorld/components/HelloWorld.module.scss 1:0
Module parse failed: Unexpected character '@' (1:0)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
> @import<EMAIL_ADDRESS>|
| .helloWorld {
 ELIFECYCLE  Command failed with exit code 1.

This may or may not be related to this peer dependency issue:

Steps to reproduce

Create new project with --use-heft

pnpm --package yo@latest --package @microsoft/generator-sharepoint@next dlx yo @microsoft/sharepoint@next --skip-install --package-manager pnpm --use-heft

Follow series of question to build your SPFx project with simple WebPart and React
Install dependencies

pnpm install

Build project

pnpm run build

Note: currently build does not work without some additional patch-work (see: #9073)
Expected behavior
It should be possible to build new project without any additional steps and error messages.

@ltdu please, see the comment in https://github.com/SharePoint/sp-dev-docs/issues/9073


Reference to self is undefined

https://github.com/irimitenkan/picam2ctrl/blob/6c37a5268a3c769db729cfed3f52cedc51687b73/picamclient.py#L181
you are referencing self here, but you pass in cfg to checkPanTiltActive()

I'm gong to go a different route...


PartFamily By Name node retrieves null

PartFamily By Name node retrieves null ! I checked many times and I’m sure the family name is correct. According to the description of the node, this works only with Part PIPE Family in the Part List. It seems like it doesn’t work with Part Structure Family.
Civil 3d is updated, civil 3d toolkit is 1.1.24
check the thread here
https://forum.dynamobim.com/t/partfamily-by-name-node-retrieves-null/63538

fixed in 1.1.27


Loading the SHH config file is slow

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.88.1
OS Version: MacOS 14.12

Steps to Reproduce:

Open the Remote Explorer tab

The Remote Explorer tab is expected to read the SSH config files and display available hosts. It takes significant time to display when the first time open it.

Does this also happen when you disable your other extensions?

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Not able to pass down properties to external template

Hello, what i'm trying to do is pass down properties to HTML that's defined via template tag:
HomeComponent.vue
<template id="home-comp">

{{message}}

		my home message

</template>

Then in my app view i register the component and add a message
App.vue
<template>

		<home-component message="testing a message"></home-component>

</template>

Results in the following error:
"Property or method "message" is not defined on the instance but referenced during render. Make sure to declare reactive data properties in the data option."
It seems pretty straight forward and not sure if this is a bug or if I'm doing something wrong. I checked the forum as well as did some Googling around and can't really find an answer.
Thanks!

I found out that it does work with straight up javascript though
HomeComponent.js
'use strict';

import Vue from 'vue';

export default Vue.component('home-component', {
	template: '
{{message}}
',
	props: ['message']
})

You don't need to give the template an id in vue files... They are already in the same file.

<template>

{{message}}

		my home message

</template>

Still results with the same error message.

You also need to export your component instead of registering it globally... Please read the docs for vue-loader.

This worked, thanks for the tips and your hard work.


Fixing the shortcomings of pausing and Overlunky interoperability

I do have some OL updates planned, like

function to disable the unpause on level start, leaving you with pause=2 on the first frame of the level automatically
moving speedhack from UI only to API
fixing the custom PRE_UPDATE pause flag so it will actually only advance one update on frame advance
fixing the custom PRE_UPDATE pause flag so it can be used along normal pause flags to force them on pre update
some kind of structure in shared memory, where OL scripts and even PL mods can

check if OL is present
get and set some select OL options
disable some OL features, possibly to reimplement them better in Lua
reserve some keyboard keys for the script, ignoring them in OL without having to toggle the whole keyboard controls option

However, the best thing you can actually do, with or without these updates, is implement all of the required OL features in the mod, so it is not dependent on OL. I spruced up my old mod to test these features on PL some more and you can almost use it as is if you want to. It allows for (automatic) first frame pausing and even pre-first frame pausing, frame advancing cutscenes, skipping fades or disabling slowhack during fades.
https://gist.github.com/Dregu/bd3c68931d8591d2f4baa2a487569159
It will however clash with the default overlunky keys for the time being.
I didn't actually even test this with TASW and I'm not sure TASW is even compatible with first frame movement atm.

Yep I actually implemented hotkey suppressing from PL side in the mentioned PR already, with an updated example how to do that.

That's excellent, thank you for working on this. Besides the GUI clutter, the limited frame advance is one of the biggest pain points I have with TASW and I'd love to fix that.

Most of those went in yesterday, but next I'm adding a hook that lets you pause and frame advance menus too. You might want to wait for that.

I'm done messing with TAS/camera/pause related UI and API for now. With the amount of useful tools in OL now, requiring OL seems even more logical, but maybe implement the bare minimum (frame advance) in mod when OL is not present.

I looked over all the new pause changes and I want to clarify a couple things before I start working with the new system.

Should I actually try to integrate with OL's pause system, or should I recreate the PRE_GAME_LOOP freeze pauses myself in TASW? I'm inclined to do the latter, and just suppress OL's pause hotkeys if needed. It looks like I could technically import "overlunky/pause" to use OL's pauses, but that's undocumented API behavior and you mentioned wanting to move that code to C++. Did I miss some other OL call that can get/set freeze pauses? PRE_GAME_LOOP freeze pauses seem to be the best ones for TASing anyways, so I'd rather not try to support the other OL pause types if I don't have to.
I want to make sure I understand what set_start_level_paused is doing internally. It looks like it skips the game code that would have unpaused the game engine during the update where the fade-in ends, and leaves the game in a fade pause. Assuming I give TASW its own pause controls, I think I have a couple options. I could force this setting to turn off and do a freeze pause on the last fade-in frame to get roughly the same effect. Alternately, if start_level_paused is active, then I could clear the fade flag at the end of the update and swap it out for a TASW freeze pause at the start of the next frame. Is this setting shared between OL and PL? For that matter, are all of these write_mem_recoverable changes always shared between OL and PL?

Ugh, I was supposed to implement all the features of the pause script's exports system in the bucket so you could command it from playlunky too, but I guess I missed setting the actual paused state, only added pause_type. Fixed some of that on WHIP now.
But yeah doing the pause system yourself and disabling the OL keys would probably still be the simplest way to make sure you only have to support one pause type and do it well. The camera update also makes it possible to only do one PRE_GAME_LOOP and adding all the conditions (during fading, menu, transition, start of level etc) there, without touching any state.pause flags at all. The bucket also only updates the overlunky options on next render loop, which might actually be too slow for what you're doing in 60Hz land.

Yeah the loading/fade unpause line in game code is simply removed, so theoretically nothing should have a chance to update in the level. Swapping the state pause to a freeze on one of the callbacks should also be seamless. Then again the !pause_loading option does only the "roughly the same effect".
The status of dirty/original recoverable memory is not shared between the tools yet, but it should really move to the bucket now, so one tool can enable a hack and other can disable it. It's very confusing for the tools atm. If one has already edited the memory before the other was even launched, it has no idea what the original code was.
TL;DR: Maybe just do it all yourself in PRE_GAME_LOOP if you want it done right, disabling all OL pause features. I'll even give you an option to make it impossible to use them.

I've released TASW 1.1.0, which stops using fade pauses and upgrades to the new pause API. That should take care of all the pause interoperability issues with Overlunky. Thanks for all your work on the script API side of things.


Retry On Error

Occasionally the render process fails, and it would be desirable to allow the render to immediately try again under certain conditions, such as aerender exiting with a non-zero exit code.
This PR adds the ability for the user to supply a custom onShouldRetry function to trigger a retry as required.

I like the idea! @erlendp do you want to add more work here?

Any movement on this as it would be a very useful thing to have

Only change I can see that would be needed, is that we limit the number of retries a render can have when erroring out.
Worry could be that we get stuck in a render loop when it's dealing with an issue where an render is erroring out everytime

@inlife On further review I propose the following changes

Right now onShouldRetry is set as event, not as an option flag. This should be set as a flag option "--retry-on-error" (defaulted to false)
A second flag "--number-of-retries" should be added as an Number (default to 1) which sets the number of times a render should be retried before erroring out (to avoid infinite loop)

As it looks like OP has abandoned the change, I'll try and find time to do this.


Cannot read property 'Event' of undefined at LeafletDrawDirective.ngOnInit

I am using ngx-leaflet-draw and ngx-leaflet. The map is coming on the div. Also, the Drawing option is coming. But the draw lines (Also, circle, rectangle, line etc) is not persisting on the screen.

On the console, we are getting an error.

This is coming from below part of the code in the file leaflet-draw.directive.ts.
		// Register the main handler for events coming from the draw plugin
		this.leafletDirective.getMap().on(L.Draw.Event.CREATED, (e: any) => {
			const layer = (e as L.DrawEvents.Created).layer;
			this.featureGroup.addLayer(layer);
		});
	}

Please help.

Can you give some details about your setup? angular-cli? What versions of Leaflet and Leaflet.draw?

@Koushik-Sarkar  can you let us know of which versions of leaflet and leaflet draw you are using ? thank you

This sounds like a problem with versions of libraries and/or versions of typings. Particularly if it compiles, but then fails at runtime, it sounds like it might be an issue with typings.

Haven't heard back, gonna close this. See above comments if you're having this issue.


mnist.mnist() doesn't exist?

the mnist example imports the python mnist library and calls the mnist() member, which doesn't seem to exist:
~/repos/mlx-examples
venv > python3 -V                                                                             Tue Dec  5 23:36:09 2023
Python 3.11.6

~/repos/mlx-examples
venv > pip install mnist                                                                      Tue Dec  5 23:36:15 2023
Collecting mnist
  Using cached mnist-0.2.2-py2.py3-none-any.whl (3.5 kB)
Collecting numpy (from mnist)
  Using cached numpy-1.26.2-cp311-cp311-macosx_11_0_arm64.whl.metadata (115 kB)
Using cached numpy-1.26.2-cp311-cp311-macosx_11_0_arm64.whl (14.0 MB)
Installing collected packages: numpy, mnist
Successfully installed mnist-0.2.2 numpy-1.26.2

~/repos/mlx-examples
venv > python -c "import mnist; mnist.mnist()"                                                Tue Dec  5 23:36:33 2023
Traceback (most recent call last):
  File "<string>", line 1, in <module>
AttributeError: module 'mnist' has no attribute 'mnist'

nor in the source
error is located here
am i using this wrong/supposed to be importing mnist from another source? replacing the mnist.mnist() call with calls to mnist.train_images/train_labels/test_images/test_labels() seems to work fine

I think you refer to the documentation here: https://ml-explore.github.io/mlx/build/html/examples/mlp.html
and its example code
# Load the data
import mnist
train_images, train_labels, test_images, test_labels = map(
    mx.array, mnist.mnist()
)

this is a bit misleading indeed. At the very bottom is a link to the full example

The full example is available in the MLX GitHub repo.

which has a wrong link. A bit unfortunate. The real link is here https://github.com/ml-explore/mlx-examples/tree/main/mnist.
There you can see that there is a file mnist.py. This file has indeed the mnist function and this file is also the one that is being imported when you do import mnist in its sibling files e.g. main.py. mnist.py implements its own mnist file loader, so there is actually no need to install the package mnist. I think renaming the file in the examples would avoid this unfortunate mistake in the future.

The doc link should be fixed, sorry about that.
The mnist package is a file in the mnist/ directory. If you run python main.py from inside mnist/ it should work.


Working example?

Hi Tgardner,
Do you have a working example for WearOS 2.0 (with a standalone watch app) that sends and displays text on the watch?
Kind Regards,
Bence

Still looking for a working example... do you still maintain this project? 😊


Do we really need SDK version 26?

Looks like I can't build my app on Android without SDK version 26 (Oreo) using this repo. Is it really needed to be that high? According to a recent blog post I read, only 0.5% of Android users have Oreo or above. And since IAPs have been out since... idk a long time, it doesn't seem like we would need v26.
Nevertheless, I am pretty much a noob when it comes to Android, so maybe someone could answer this question better than I can guess at and make the fix (if plausible).
Not sure if it just needs a number tweak in android/build.gradle or not...
android {
  compileSdkVersion 26
  buildToolsVersion "26.0.2"
  ...
}

@KjellConnelly
Hi KjellConnelly~ You can change your buildToolsVersion as your taste. You can set it to sdk 23 when you prefer it.


Allow "aliases" to certain tags or attributes

With Web Components and frameworks like Angular it not uncommon to create elements or tags that behave similar to existing elements or attributes. It would be nice if we could configure this. E.g. I'd like to use img-req-src but I have <img>'s which use ng-src instead of src.
Instead of setting { 'img-req-src': true } a config like this would be nice:
{
  'img-req-src': { // using an object implies `true` - JSCS behaves like this, too
    aliases: [ 'ng-src', 'data-ng-src' ]
  }
}

For this specific case you can change the filter property of the img-req-src rule to include those aliases. Making that more easily configurable is tricky. I'll have to think about it more.

Thanks for your answer. Could you give me an example to filter?  Couldn't found something in the docs.

It's not a configuration parameter: you have to change the line filter: ['img'] in lib/rules/img-req-src.js. I'm still trying to work out a way to allow the user to do this without modifying the source.

Any update on this?
I also need
aliases: [ 'ng-src', 'data-ng-src' ]


Need to cite this repo in a paper. Create a DOI on Zenodo?

Hi!
I used your MXResnet implementation in a CVPR Competition track and wanted to cite your repo.
Could you create a DOI of this repo so I can cite it?
The sooner the better, I've got a deadline in a day. 😅
I realize that I should have asked for this much earlier, but I hadn't planned to use your code until yesterday.
Thanks!

Hi @akashpalrecha - sure, thanks for letting me know.  I'll make it in the next 2 hours here and post.

Here you go:
or html:
Hope that helps and please let me know if you cite it (and thanks for the push to get setup on Zenodo)

Sorry for responding so late. The deadline for the paper got extended so we put things aside for a while. We'll definitely cite it in the paper (deadine is 9th April, 2020, 11:59 PM UTC)
Thanks a lot for creating the DOI.


Update to latest Yargs to resolve vulnerable transient dependencies

GitHub reports a CVE (WS-2018-0236) in the following dependency chain:
<EMAIL_ADDRESS>><EMAIL_ADDRESS>> yargs@^8 > os-locale@^2 > mem@^1
All of these are patch pinned, and mem@1 is not patched, however,
both yargs@8 and mem@1 are WAY behind: yargs@13 and mem@4 are available.
I suggest mjml-cli updates to yargs@^13 to resolve this vulnerability
for everyone depending on mjml.

Hey thanks for this PR !
Did you check that nothing break here, it seems like a big jump ?

Short answer: no.
Normally one would have tests for that, but I don't see any in this package. I'm not an active user of this project, I just happen to depend on mjml, which itself has a dependency on its cli.

Yeah the test coverage on MJML is .... really low...  🙃
There's a list of command on the main readme : https://github.com/mjmlio/mjml#command-line-interface
We only use yargs to handle most of input options so it shouldn't be that hard to check if everything is ok

As part of #1700 I tested a few of the comments and it looks like mjml's use of yargs is not effected by the breaking changes.


Update Spring Cloud Azure Support File

Update Spring Boot and Spring Cloud versions for the compatibility test.
If you merged this PR, please update Spring Cloud Azure Timeline.
This PR is created by GitHub Actions: https://github.com/Azure/spring-cloud-azure-tools/actions/runs/3448812445

@Netyyyy can we update the github issue title to "Update Spring Boot and Spring Cloud versions for the compatibility test."
I think it's better than "Update Spring Cloud Azure Support File"
What's your opinion?


add def has_clientserver? & def vim from vmail.rb

This is a pull request in response to issue https://github.com/danchoi/vmail/issues/209.
After applying the solution suggested by @PedroLopes,
updating now works for me on mac osx with the following specs:
VIM - Vi IMproved 8.0 (2016 Sep 12, compiled Jul 26 2017 19:10:24)
vmail 2.9.8 (from gem)
It may be preferable to use the habpy version of vmail (which I believe arose from issue https://github.com/danchoi/vmail/issues/199), as @PedroLopes suggested, but I think the solution in this pull request is the simplest and cleanest fix.

I apologize for the delay. I didn't have osx to test this with, so it just kept getting delayed. But the change seems trivial, so we can merge it. Thanks for your patch and your workaround for clientserver. I will be linking it to the Readme.


Upgrade go dependencies

Upgrade all dependencies in go.mod
golang.org/x/net v0.17.0 also fixes CVE-2023-39325

Need to re-run tests a few times to confirm gocron is stable, see https://github.com/go-gitea/gitea/pull/27511

gocron still causing the issue:
--- FAIL: TestAddTaskToScheduler (0.00s)
    tasks_test.go:39:
        	Error Trace:	/home/runner/work/gitea/gitea/services/cron/tasks_test.go:39
        	Error:      	Not equal:
        	            	expected: "30 5 4 * * *"
        	            	actual  : "5 4 * * *"

        	            	Diff:
        	            	--- Expected
        	            	+++ Actual
        	            	@@ -1 +1 @@
        	            	-30 5 4 * * *
        	            	+5 4 * * *
        	Test:       	TestAddTaskToScheduler
FAIL

I was unable to create a backport for 1.21. @harryzcy, please send one manually. :tea:
go run ./contrib/backport 27599
...  // fix git conflicts if any
go run ./contrib/backport --continue


Umbrella term: Compute Instance?

The documentation says that any compute instance can talk to service endpoints. It's obvious that VM can act as service endpoint client. And I guess App Service/Functions as well
However what exactly does the umbrella term compute instance contain?

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

ID: 1eb0407d-f4a5-6059-488d-fd082915c675
Version Independent ID: 7c2fb429-743c-aea0-f1d5-de6dc2e62e8e
Content: Azure virtual network service endpoints
Content Source: articles/virtual-network/virtual-network-service-endpoints-overview.md
Service: virtual-network
GitHub Login: @sumeetmittal
Microsoft Alias: sumi

Thanks for the feedback! We are currently investigating and will update you shortly.

Service Endpoint is a subnet level property. All traffic from subnet will take the optimum route. Compute instance could be any compute deployed in the subnet – VM, VMSS, AKS, Container Instances, Batch, App Service, Dedicated Host and many more. Hope this helps!

We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Add deprecation notice?

Seems like it doesn't support fp-ts@2 and I suggest to inform about it in the readme.
Not to complain, but I'm new to fp-ts and hardcore FP in general and I've spent quite a bit of time trying to understand what I'm doing wrong with logging-ts only to finally realise that it uses fp-ts version 1 instead of version 2.

(btw this package is not deprecated, I just had no time to upgrade yet)

Yeah, I guess deprecation wasn't the right word, more like 'outdated'. Btw looking forward to become more familiar with fp-ts, it looks awesome :) Thanks for the amazing work.


Weird parallax / scaling behavior

Hi there! I was so excited to learn about the ability to put Deck and Mapbox in the same canvas context for a number of reasons. After implementing the approach in the docs this post, i found that it seems the mapbox layer is "zoomed" in more than my deckgl layer...

In addition, I've noticed a few other quirks:

Transitions (FlyTo) dont seem to work
Opacity on scatterplot layers

If anyone has any direction to point me in, I'd really appreciate it...thank you!

The implementation of my components:
import { StaticMap } from 'react-map-gl';
import DeckGL from<EMAIL_ADDRESS>...
    const map = mapRef.current.getMap();
    const deck = deckRef.current;
    map.addLayer(new MapboxLayer({ id: 'deck-stuff', deck }));
...
      <DeckGL
        initialViewState={mapViewport}
        controller={true}
        onViewStateChange={handleViewStateChange}
        layers={getLayers()}
        useDevicePixels={false}
        ref={(ref) => { deckRef.current = ref && ref.deck; }}
        onWebGLInitialized={handleWebGLInitialized}
      >
        {webGLContext && (
          <StaticMap
            ref={(ref) => { mapRef.current = ref; }}
            mapboxApiAccessToken={mapboxApiAccessToken}
            mapStyle={mapStyle}
            preventStyleDiffing
            onLoad={handleMapLoaded}
            reuseMaps
            preserveDrawingBuffer
            gl={webGLContext.gl}
          />
        )}
      </DeckGL>

and here's what my map viewport object looks like:
Formatted JSON Data
{
   "width":1440,
   "height":694,
   "latitude":40.00929265067086,
   "longitude":-105.12338640756488,
   "zoom":9.692802919210216,
   "bearing":0,
   "pitch":0,
   "altitude":1.5,
   "maxZoom":14,
   "minZoom":0,
   "maxPitch":60,
   "minPitch":0,
   "position":[
      0,
      0,
      0
   ]
}

@kaseymccurdy What are the versions of your mapbox-gl and deck.gl?

<EMAIL_ADDRESS>"^7.3.6",
<EMAIL_ADDRESS>"^7.3.6",
<EMAIL_ADDRESS>"^7.3.6",
<EMAIL_ADDRESS>"7.3.6",
<EMAIL_ADDRESS>"^7.3.6",
    "react-map-gl": "5.0.10",
    "mapbox-gl": "1.2.1",

I can't see any obvious issue with your code other than you are using react ref incorrectly (you should set ref={mapRef} instead of callback function if you choose to use React.createRef()). Mapbox's addLayer sometimes silently fails so it's hard to pin down the problem.
I put together a code sandbox example: https://codesandbox.io/s/deckgl-mapbox-layers-interleve-651q1

<EMAIL_ADDRESS>"^7.3.6",
<EMAIL_ADDRESS>"^7.3.6",
<EMAIL_ADDRESS>"^7.3.6",
<EMAIL_ADDRESS>"7.3.6",
<EMAIL_ADDRESS>"^7.3.6",
    "react-map-gl": "5.0.10",
    "mapbox-gl": "1.2.1",

嗨，您好！出于多种原因，我很高兴了解将Deck和Mapbox置于同一画布上下文中的能力。在实现本文的文档中的方法之后，我发现似乎mapbox层比我的deckgl层“放大了”。

此外，我还注意到了其他一些怪癖：

转场（FlyTo）似乎不起作用
散点图图层上的不透明度

如果有人有任何指示要指向我的方向，我将非常感谢...谢谢！
我的组件的实现：
import { StaticMap } from 'react-map-gl';
import DeckGL from<EMAIL_ADDRESS>...
    const map = mapRef.current.getMap();
    const deck = deckRef.current;
    map.addLayer(new MapboxLayer({ id: 'deck-stuff', deck }));
...
      <DeckGL
        initialViewState={mapViewport}
        controller={true}
        onViewStateChange={handleViewStateChange}
        layers={getLayers()}
        useDevicePixels={false}
        ref={(ref) => { deckRef.current = ref && ref.deck; }}
        onWebGLInitialized={handleWebGLInitialized}
      >
        {webGLContext && (
          <StaticMap
            ref={(ref) => { mapRef.current = ref; }}
            mapboxApiAccessToken={mapboxApiAccessToken}
            mapStyle={mapStyle}
            preventStyleDiffing
            onLoad={handleMapLoaded}
            reuseMaps
            preserveDrawingBuffer
            gl={webGLContext.gl}
          />
        )}
      </DeckGL>

这是我的地图视口对象的样子：
Formatted JSON Data
{
   "width":1440,
   "height":694,
   "latitude":40.00929265067086,
   "longitude":-105.12338640756488,
   "zoom":9.692802919210216,
   "bearing":0,
   "pitch":0,
   "altitude":1.5,
   "maxZoom":14,
   "minZoom":0,
   "maxPitch":60,
   "minPitch":0,
   "position":[
      0,
      0,
      0
   ]
}

Brother, have you solved the problem？

for right now...i just have it in two webGL contexts :( im going to try some things when i get availability and will report back...

@Sitong2014 @kaseymccurdy It would help us diagnose the issue if you can put together a code sandbox that reproduces this issue.
If you see the same issue on the link in my previous comment, please report your OS and devicePixelRatio.


[Bug]: Any user can remove all Coolify accounts from the Admin View of its personal team

Error Message and Logs
Admin View from any team seems to be global to all users, so any user from its personal team can remove any other user from coolify database
The best option here is to display admin view only for the Root team
Steps to Reproduce

Connect as Root
Invite new user to Coolify
New user joins, his own team is created
From user account, move to Teams > Admin View
User can remove Root from database

Example Repository URL
No response
Coolify Version
v4.0.0-beta.379
Are you using Coolify Cloud?
No (self-hosted)
Operating System and Version (self-hosted)
Ubuntu 22.04
Additional Information
No response

See my comment on discord: https://discord.com/channels/459365938081431553/1311626147947352076/1320767917545226322


height prop doesn't do anything

I installed the module and included the component in my React Native app.
function Schedule() {
  return (
    <View style={{ height: '100%' }}>
      <Calendar events={[]} height={100} />
    </View>
  );
}

The height prop doesn't do anything. The result is the same whether I set it to 50, 80, 100 or 120.
I'm on React Native 0.63.4 and OS is Android 11.

You should specify absolute height of your calendar as a number. Please
calculate actual height using Dimension.
On Wed, Jan 5, 2022, 6:58 PM David Heisnam @.***> wrote:

I installed the module and included the component in my React Native app.
function Schedule() {
return (
<View style={{ height: '100%' }}>

);
}
The height prop doesn't do anything. The result is the same whether I set
it to 50, 80, 100 or 120.
I'm on React Native 0.63.4 and OS is Android 11.
—
Reply to this email directly, view it on GitHub
https://github.com/acro5piano/react-native-big-calendar/issues/605, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACRZCBYAITE34EHZDYA26HDUUQI57ANCNFSM5LJP2R2Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Provide the results as Iterable of generators instead of lists

When returning Iterables by the DICOMwebClient methods that are created by a Generator instead of lists, it would be possible to process the results of each method on the fly (maybe in another thread or process).

There are iter_ versions for the retrieve_ methods. eg, iter_series and retrieve_series.
The iter_ methods should stream the results back if the server supports it.

Thanks a lot. I somehow overlooked your answer, but that is what I was looking for. Thanks again and I am closing this issue.


defects4j.org SSL certificate expired

0.467 curl: (60) SSL certificate problem: certificate has expired
It seems like defects4j.org's SSL certificate expired.
Issued On	Tuesday, January 31, 2023 at 6:00:00 PM
Expires On	Thursday, February 1, 2024 at 5:59:59 PM

The zip file is hosted there.
https://defects4j.org/downloads/defects4j-repos.zip

Thanks for reporting! Will get this resolved asap.

This should be resolved pretty soon.
As a workaround, add the following option to wget or curl, which ignores the expired certificate.
OSX:
Add --no-check-certificate on line 41 in get_repos.sh
Other OS:
Add --insecure on line 43 in get_repos.sh
I'll update this issue as soon as the newly issued certificate is in place.
Sorry about the trouble.

The new certificate is in place. The workaround is no longer needed.


develop

Coverage remained the same at 92.109% when pulling f7467f40454147075f5b9d18455673d57ab7c8e6 on develop into 1b5b3e22139654cf4fdf0ed84753b7cb2651b6e3 on master.


auction: credit VCB when ending an auction

Describe your changes

[x] hotfix for #4388
[x] reproduction for #4388
[x] migration for #4388

Checklist before requesting a review

[x] If this code contains consensus-breaking changes, I have added the "consensus-breaking" label. Otherwise, I declare my belief that there are not consensus-breaking changes, for the following reason:

This is consensus-breaking and state-breaking. Migration WIP.

I have tested this in two ways:

a local devnet running v0.75.0 with a collection of auctions running every block
an application test that reproduce the bug (and attests that the fix works).

Additionally, I ran the migration on a fork of the running testnet to observe if any deviations were recorded, but without resuming. I didn't see any deviations, that is to say the tallied auction VCB and the one onchain were equal.

This is generating conflicts to every other U1 migrations and has received a lot of offline attention so merging this optimistically, but async reviews are welcome.


Fix table format

"Find Previous" row isn't well-formed.

Awesome, thanks!

@nwinkler It was nothing! :)


request_views is using an old view key format for coltests

request_views is still using the format 't.props.{}{}|||' but this should now be changed to 't.props.{}{}|:||'.

Resolved by #449.


Bug: import project not working under Linux

Description:
wxUiEditor 1.2.0, Ubuntu Linux 22.04. When I try to import a project, the program closes without any warnings or messages. I tried running it from the terminal, and apparently it is still running, just not showing anything. No new error messages there either.

Is this any project that you have tried, or a specific project or type of project?

Hi,
Any attempt to import fails, whether at the initial window or after opening
the design window and trying to import something. In the first case the
program closes, in the latter it freezes for a while and then closes. I
tried invoking it from the terminal, and aside from a bunch of GTK errors
it shows on starting up, nothing new appears when trying to import.
Is there anything I can do to try and see if I get more information?
On Fri, Feb 2, 2024 at 10:07 AM KeyWorksRW @.***> wrote:

Is this any project that you have tried, or a specific project or type of
project?
—
Reply to this email directly, view it on GitHub
https://github.com/KeyWorksRW/wxUiEditor/issues/1393#issuecomment-1923767376,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A64PJVHV4QVIP4V64SIH6D3YRTQJVAVCNFSM6AAAAABCTGYWOWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRTG43DOMZXGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Kenneth Rochel de Camargo, Jr. MD PhD
Professor Titular - Full Professor
Instituto de Medicina Social - Social Medicine Institute
Universidade do Estado do Rio de Janeiro - Rio de Janeiro State U. (Brazil)
Editor Emeritus - American Journal of Public Health
e-mail: @.***

That's all I need, thanks. I'll take a look at it later today and see if I can spot the problem.

So far, I can't reproduce the problem. I've tried on Fedora 39 and Ubuntu 20.04 and I am able to import a wxFormBuilder 4 project file in both Debug and Release builds without a problem.
Would it be possible for you to attach the project you are trying to import? I'd like to see if there is something specific to that project that is causing the problem.

Hi, that is the problem, I can't even attempt to import anything. As I hit
the import button, it goes off. I don't even get to choose a project or
file to import.
I have forgot to add, I have wxWidgets version 3.05 installed, don't know
if that makes any difference.
Thanks for your attention,
ken
On Fri, Feb 2, 2024 at 3:36 PM KeyWorksRW @.***> wrote:

So far, I can't reproduce the problem. I've tried on Fedora 39 and Ubuntu
20.04 and I am able to import a wxFormBuilder 4 project file in both Debug
and Release builds without a problem.
Would it be possible for you to attach the project you are trying to
import? I'd like to see if there is something specific to that project that
is causing the problem.
—
Reply to this email directly, view it on GitHub
https://github.com/KeyWorksRW/wxUiEditor/issues/1393#issuecomment-1924467104,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A64PJVB43DGQWNFPHZKXM7DYRUW4NAVCNFSM6AAAAABCTGYWOWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRUGQ3DOMJQGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Kenneth Rochel de Camargo, Jr. MD PhD
Professor Titular - Full Professor
Instituto de Medicina Social - Social Medicine Institute
Universidade do Estado do Rio de Janeiro - Rio de Janeiro State U. (Brazil)
Editor Emeritus - American Journal of Public Health
e-mail: @.***

I tried running it on a virtual machine, and it didn't show the same
problem. I don't have the wxWidgets dev libraries installed on the virtual
machine, maybe the problem is around that?
On Fri, Feb 2, 2024 at 3:39 PM Kenneth Camargo Jr. @.***>
wrote:

Hi, that is the problem, I can't even attempt to import anything. As I hit
the import button, it goes off. I don't even get to choose a project or
file to import.
I have forgot to add, I have wxWidgets version 3.05 installed, don't know
if that makes any difference.
Thanks for your attention,
ken
On Fri, Feb 2, 2024 at 3:36 PM KeyWorksRW @.***>
wrote:

So far, I can't reproduce the problem. I've tried on Fedora 39 and Ubuntu
20.04 and I am able to import a wxFormBuilder 4 project file in both Debug
and Release builds without a problem.
Would it be possible for you to attach the project you are trying to
import? I'd like to see if there is something specific to that project that
is causing the problem.
—
Reply to this email directly, view it on GitHub
https://github.com/KeyWorksRW/wxUiEditor/issues/1393#issuecomment-1924467104,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A64PJVB43DGQWNFPHZKXM7DYRUW4NAVCNFSM6AAAAABCTGYWOWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRUGQ3DOMJQGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Kenneth Rochel de Camargo, Jr. MD PhD
Professor Titular - Full Professor
Instituto de Medicina Social - Social Medicine Institute
Universidade do Estado do Rio de Janeiro - Rio de Janeiro State U. (Brazil)
Editor Emeritus - American Journal of Public Health
e-mail: @.***

--
Kenneth Rochel de Camargo, Jr. MD PhD
Professor Titular - Full Professor
Instituto de Medicina Social - Social Medicine Institute
Universidade do Estado do Rio de Janeiro - Rio de Janeiro State U. (Brazil)
Editor Emeritus - American Journal of Public Health
e-mail: @.***

I've uninstalled and installed it again, now everything works. I must have
scared off some gremlin or other.
Sorry for the disturbance.
On Fri, Feb 2, 2024 at 4:05 PM Kenneth Camargo Jr. @.***>
wrote:

I tried running it on a virtual machine, and it didn't show the same
problem. I don't have the wxWidgets dev libraries installed on the virtual
machine, maybe the problem is around that?
On Fri, Feb 2, 2024 at 3:39 PM Kenneth Camargo Jr. @.***>
wrote:

Hi, that is the problem, I can't even attempt to import anything. As I
hit the import button, it goes off. I don't even get to choose a project or
file to import.
I have forgot to add, I have wxWidgets version 3.05 installed, don't know
if that makes any difference.
Thanks for your attention,
ken
On Fri, Feb 2, 2024 at 3:36 PM KeyWorksRW @.***>
wrote:

So far, I can't reproduce the problem. I've tried on Fedora 39 and
Ubuntu 20.04 and I am able to import a wxFormBuilder 4 project file in both
Debug and Release builds without a problem.
Would it be possible for you to attach the project you are trying to
import? I'd like to see if there is something specific to that project that
is causing the problem.
—
Reply to this email directly, view it on GitHub
https://github.com/KeyWorksRW/wxUiEditor/issues/1393#issuecomment-1924467104,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A64PJVB43DGQWNFPHZKXM7DYRUW4NAVCNFSM6AAAAABCTGYWOWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRUGQ3DOMJQGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Kenneth Rochel de Camargo, Jr. MD PhD
Professor Titular - Full Professor
Instituto de Medicina Social - Social Medicine Institute
Universidade do Estado do Rio de Janeiro - Rio de Janeiro State U.
(Brazil)
Editor Emeritus - American Journal of Public Health
e-mail: @.***

--
Kenneth Rochel de Camargo, Jr. MD PhD
Professor Titular - Full Professor
Instituto de Medicina Social - Social Medicine Institute
Universidade do Estado do Rio de Janeiro - Rio de Janeiro State U. (Brazil)
Editor Emeritus - American Journal of Public Health
e-mail: @.***

--
Kenneth Rochel de Camargo, Jr. MD PhD
Professor Titular - Full Professor
Instituto de Medicina Social - Social Medicine Institute
Universidade do Estado do Rio de Janeiro - Rio de Janeiro State U. (Brazil)
Editor Emeritus - American Journal of Public Health
e-mail: @.***

Glad to know the problem resolved itself.
Just as an FYI, wxUiEditor is built with a static version of wxWidgets, so it makes no difference what version you have. However, be aware that the C++ code it generates may require wxWidgets 3.1. As long as you don't try to use anything in wxWidgets that wasn't added after your version was released, you should be fine. However, no conditionals are added for code that requires 3.1, so if you do  use something that requires 3.1, you won't know until you try to compile or link.

Closing since there isn't anything I can do if installation fails.


Node._unhandled_input() doesn't catch events that are not handled by GUI controls

Godot version:
3.1.1
Issue description:
It is my understanding (though I may be mistaken) that Node._unhandled_input should catch any event that isn't handled by the GUI. Instead, it appears to never catch events that occur over a GUI control, whether they were handled or not.
Steps to reproduce:

Add a node to an empty scene, such as a Sprite.
Add a control to the scene, such as a Panel, and place it partly over the sprite.
Set the control's mouse_input property to MOUSE_FILTER_PASS so that all unhandled mouse events get propagated.
Override Node._unhandled_input in Sprite.
Notice that none of the mouse events over the panel (move or click) are caught by it.

So... what entity consumed the event, exactly?
Minimal reproduction project:
unhandled_input.zip
Click on the sprite to see a message in the console. Click on the panel over the sprite and notice that no message gets displayed.

It should work if you set the mouse mode to MOUSE_FILTER_IGNORE. Though I don't know what would be the difference with MOUSE_FILTER_PASS then... maybe it catches it by default and it makes sense only with some controls?

@Zylann You're right--it does produce the expected result with MOUSE_FILTER_IGNORE on the control, which I should've mentioned in the OP. Thanks for pointing it out.
However, I feel like things should behave the exact same with MOUSE_FILTER_PASS unless an event is marked as handled by consuming it through Control.accept_event() (within _gui_input). That would seem more logical and consistent, unless I'm missing something.

I also encountered the same problem - mouse -> pass blocks event to go further to _unhandled_input.
"Pass" should mean - process and allow go to unhandled input.
"Ignore" should mean - don't process and allow to go to unhandled input.

Just mentioning that, after some pain trying to figure this out, I found clearly stated in the documentation for MouseFilter.MOUSE_FILTER_PASS that

Even if no control handled it at all, the event will still be handled automatically, so unhandled input will not be fired.

In fact, I suspect that a wise developer had a good reason for this design and I'm just starting to appreciate it.

As stated above, this is not a bug.


custom VNF placement

It is a good feature request to have custom VNF placement:

you can add your own version instead of the default  'RoundRobinDcPlacementWithSAPs' algorithm.
you can specify the datacenter at VNF instantiation via the rest api :
curl -X PUT http://<IP_ADDRESS>:5001/restapi/compute/dc1/server -H 'Content-Type: application/json' -d '{"image":"sonatanfv/sonata-iperf3-vnf"
or via the son-emu-cli:
 son-emu-cli compute start -d dc1 -n server -i sonatanfv/sonata-iperf3-vnf
There is also the possibility to add custom paths to the chaining function, although I never tested this:
https://github.com/sonata-nfv/son-emu/blob/master/src/emuvim/dcemulator/net.py#L537

Can we specify this path through the cli/rest api?

I think we can specify it through the networking REST API, or not?
Just for clarification, if you now do:
son-emu-cli compute start -d dc1 -n client -i sonatanfv/sonata-empty-vnf
son-emu-cli compute start -d dc2 -n server -i sonatanfv/sonata-empty-vnf

son-emu-cli network add -b -src client:client-eth0 -dst server:server-eth0

It will automatically use the shortest path between the dc1 and dc2 switch, right?

yes, default is shortest path calculation

the networking REST api/cli  should support this path specification as an extra argument

yes, I agree this would be a cool extension 👍


[WIP] Support Transpose in SQL

What changes were proposed in this pull request?
Support

logical operator "Transpose"
"ResolveTranspose" analyzer rule

Logical plan before analysis:
'Transpose
+- SerializeFromObject [staticinvoke(class org.apache.spark.unsafe.types.UTF8String, StringType, fromString, knownnotnull(assertnotnull(input[0, org.apache.spark.sql.test.SQLTestData$CourseSales, true])).course, true, false, true) AS course#4, knownnotnull(assertnotnull(input[0, org.apache.spark.sql.test.SQLTestData$CourseSales, true])).year AS year#5, knownnotnull(assertnotnull(input[0, org.apache.spark.sql.test.SQLTestData$CourseSales, true])).earnings AS earnings#6]
   +- ExternalRDD [obj#3]

Logical plan after analysis:
Project [key#16, __pivot_first(DISTINCT value) AS `first(DISTINCT value)`#26[0] AS dotNET#27, __pivot_first(DISTINCT value) AS `first(DISTINCT value)`#26[1] AS Java#28]
+- Aggregate [key#16], [key#16, pivotfirst(course#4, first(DISTINCT value)#20, dotNET, Java, 0, 0) AS __pivot_first(DISTINCT value) AS `first(DISTINCT value)`#26]
   +- !Aggregate [key#16, course#4], [key#16, course#4, first(distinct value#14, true) AS first(DISTINCT value)#20]
      +- Expand [[course#4, year, cast(year#5 as double)], [course#4, earnings, earnings#6]], [course#4, key#18, value#19]
         +- SerializeFromObject [staticinvoke(class org.apache.spark.unsafe.types.UTF8String, StringType, fromString, knownnotnull(assertnotnull(input[0, org.apache.spark.sql.test.SQLTestData$CourseSales, true])).course, true, false, true) AS course#4, knownnotnull(assertnotnull(input[0, org.apache.spark.sql.test.SQLTestData$CourseSales, true])).year AS year#5, knownnotnull(assertnotnull(input[0, org.apache.spark.sql.test.SQLTestData$CourseSales, true])).earnings AS earnings#6]
            +- ExternalRDD [obj#3]

Logical plan before analysis:
'Transpose [dotNET, Java], IntegerType
+- SerializeFromObject [staticinvoke(class org.apache.spark.unsafe.types.UTF8String, StringType, fromString, knownnotnull(assertnotnull(input[0, org.apache.spark.sql.test.SQLTestData$CourseSales, true])).course, true, false, true) AS course#4, knownnotnull(assertnotnull(input[0, org.apache.spark.sql.test.SQLTestData$CourseSales, true])).year AS year#5, knownnotnull(assertnotnull(input[0, org.apache.spark.sql.test.SQLTestData$CourseSales, true])).earnings AS earnings#6]
   +- ExternalRDD [obj#3]

Logical plan after analysis:
Project [key#16, __pivot_first(DISTINCT value) AS `first(DISTINCT value)`#26[0] AS dotNET#27, __pivot_first(DISTINCT value) AS `first(DISTINCT value)`#26[1] AS Java#28]
+- Aggregate [key#16], [key#16, pivotfirst(course#4, first(DISTINCT value)#20, dotNET, Java, 0, 0) AS __pivot_first(DISTINCT value) AS `first(DISTINCT value)`#26]
   +- !Aggregate [key#16, course#4], [key#16, course#4, first(distinct value#14, true) AS first(DISTINCT value)#20]
      +- Expand [[course#4, year, cast(year#5 as double)], [course#4, earnings, earnings#6]], [course#4, key#18, value#19]
         +- SerializeFromObject [staticinvoke(class org.apache.spark.unsafe.types.UTF8String, StringType, fromString, knownnotnull(assertnotnull(input[0, org.apache.spark.sql.test.SQLTestData$CourseSales, true])).course, true, false, true) AS course#4, knownnotnull(assertnotnull(input[0, org.apache.spark.sql.test.SQLTestData$CourseSales, true])).year AS year#5, knownnotnull(assertnotnull(input[0, org.apache.spark.sql.test.SQLTestData$CourseSales, true])).earnings AS earnings#6]
            +- ExternalRDD [obj#3]

Test output
Logical plans
Logical plan before analysis:
'Transpose [dotNET, Java]
+- SerializeFromObject [staticinvoke(class org.apache.spark.unsafe.types.UTF8String, StringType, fromString, knownnotnull(assertnotnull(input[0, org.apache.spark.sql.test.SQLTestData$CourseSales, true])).course, true, false, true) AS course#4, knownnotnull(assertnotnull(input[0, org.apache.spark.sql.test.SQLTestData$CourseSales, true])).year AS year#5, knownnotnull(assertnotnull(input[0, org.apache.spark.sql.test.SQLTestData$CourseSales, true])).earnings AS earnings#6]
   +- ExternalRDD [obj#3]

Logical plan after analysis:
Project [key#16, __pivot_first(DISTINCT value) AS `first(DISTINCT value)`#24[0] AS dotNET#25, __pivot_first(DISTINCT value) AS `first(DISTINCT value)`#24[1] AS Java#26]
+- Aggregate [key#16], [key#16, pivotfirst(course#4, first(DISTINCT value)#18, dotNET, Java, 0, 0) AS __pivot_first(DISTINCT value) AS `first(DISTINCT value)`#24]
   +- Aggregate [key#16, course#4], [key#16, course#4, first(distinct value#17, true) AS first(DISTINCT value)#18]
      +- Expand [[course#4, year, cast(year#5 as double)], [course#4, earnings, earnings#6]], [course#4, key#16, value#17]
         +- SerializeFromObject [staticinvoke(class org.apache.spark.unsafe.types.UTF8String, StringType, fromString, knownnotnull(assertnotnull(input[0, org.apache.spark.sql.test.SQLTestData$CourseSales, true])).course, true, false, true) AS course#4, knownnotnull(assertnotnull(input[0, org.apache.spark.sql.test.SQLTestData$CourseSales, true])).year AS year#5, knownnotnull(assertnotnull(input[0, org.apache.spark.sql.test.SQLTestData$CourseSales, true])).earnings AS earnings#6]
            +- ExternalRDD [obj#3]

DataFrame API
+------+----+--------+
|course|year|earnings|
+------+----+--------+
|dotNET|2013| 48000.0|
|  Java|2013| 30000.0|
+------+----+--------+
df:
()
+--------+-------+-------+
|     key| dotNET|   Java|
+--------+-------+-------+
|earnings|48000.0|30000.0|
|    year| 2013.0| 2013.0|
+--------+-------+-------+
transposed df:
()


Display all variants at stop position in variant list

Per user email and related discussion in this slack thread.
The frontend performs some logic to filter the list of variants recieved down to only variants in specific regions. This PR changes the Lodash function used in the stop index of that filtering from sortedIndexBy to sortedLastIndexBy so that lists of variants that are filtered include all variants that occur at the last position, instead of just the first one.
The example in the email now correctly shows all 3 variants when the region is specified as the single location

Includes unrelated change to the regional genomic constraint track, previously tiny regions that did not have genomic constraint data would use the same message as regions that are too large. Now small regions without data show a message that correctly describes why nothing is shown.

Tip for the future: it is ...

Aha, that makes total sense - I'll use your suggestions when making PRs in the future!


TINRelief issue

The current OCL constraint for TINRelief says that the lodXMultiSurface properties inherited from core:AbstractThematicSurface shall not contain a GM_MultiSurface as defined in the Core module, but only a GM_TriangulatedSurface (or subtypes thereof). But imho lodXMultiSurface cannot be restricted this way because GM_MultiSurface is not a supertype of GM_TriangulatedSurface. So I think the OCL constraint is invalid.
There are different ways to solve this:

The OCL constraint could be rewritten such that the GM_MultiSurface of each lodXMultiSurface property may only have exactly one surfaceMember of type GM_TriangulatedSurface. As a result, the triangulated surface representing the terrain would always have to be embraced by a multi-surface, which is not really nice in my opinion. And the encoding would be different compared to CityGML 2.0.
A new association is added from TINRelief to GM_TriangulatedSurface, and the use of the inherited lodXMultiSurface properties is prohibited by OCL constraints.

The second option would be consistent with how the geometry is currently defined for the other relief components like BreaklineRelief and RasterRelief. Another benefit would be that the representation of TINRelief remains identical to CityGML 2.0.
But I see another issue here: Although all relief components are derived from core:AbstractThematicSurface, they all prohibit the use of the inherited geometry properties and define their own additional ones (only MassPointRelief is a minor exception). So maybe core:AbstractThematicSurface is not the proper supertype?
That's why a third solution might be preferable:
We could introduce a new class core:AbstractSpaceBoudary in the core module, which takes the place of core:AbstractThematicSurface in the associations with core:AbstractSpace. The class core:AbstractSpaceBoudary should not define geometry properties. core:AbstractThematicSurface could then be defined as subtype of this new class. And also AbstractReliefComponent could be changed to be a subtype of core:AbstractSpaceBoudary and not of core:AbstractThematicSurface anymore.
Benefits:

AbstractReliefComponent would not inherit useless geometry properties anymore that have to be prohibited by OCL constraints.
The geometry of relief components can still be defined for the classes TINRelief, MassPointRelief, BreaklineRelief and RasterRelief as in the current Relief module.
Thus, TINRelief would also remain identical to CityGML 2.0.

Based on the discussion in the web conference on 8 November 2019 the following things were updated in the latest commit:

The class AbstractSpaceBoundary is introduced in the Core module as superclass of AbstractThematicSurface. The "bounds-boundary" association relates now AbstractSpace with AbstractSpaceBoundary.
The classes ReliefFeature and AbstractReliefComponent in the Relief module are now defined as subclasses of AbstractSpaceBoundary.
The class TINRelief is again associated with GM_TriangulatedSurface as in CityGML 2.0.
The class MassPointRelief is again associated with GM_MultiPoint as in CityGML 2.0. In addition, it is associated with the class AbstractPointCloud.


Provide a way to add Post-Notification handler

If there is a Post-Notification handler, MediatR can act as an in-process message service which can guarantee at lease once message delievery.
In specific, let's consider the following way of publishing and consuming an event (notification). First I save notification N0 in DB, after that I publish N0 using MediatR, and my handlers handles N0 (maybe one after another or concurrently), commiting some transactions to the database and then return (to MediatR codes). When all handlers return without throwing exceptions, MediatR calls post processor. The post processor could update the DB to set N0.HandledByAllHandlers=true.

Forget it. I can simply add code after calling mediator.publish()


Add a test wich proves there is a issue with copySheet method

Thanks for your pull request.  I have fix this issue and updated go test in another commit.

Nice work, thank you!

When is the release?

New version has been released now.


814 add list of gpodder synchronized podcasts

Description
Administrators can now see all subscribed podcasts from all devices in the settings menu and also add them to the webview.

Linked Issues
Additional context

hi @SamTV12345,
sorry mate, I am a little bit confused. I see the column "Manage GPodder podcasts" in my settings but it's empty.
I thought if I remove all subscriptions from podfetch and then do a full sync on my antenna pod I am going to be able to see all my subscriptions in this column in setting. Isn't it true?
Best Regards,


Implement error handling/retrying when downloading models

###################################
# Whispering Tiger is starting... #
###################################
Websocket: Server started.
Initializing medium NLLB-200 model.
Downloading medium NLLB-200 model...
Websocket: Client connected.
  1% (47431680 of<PHONE_NUMBER>) |                                                  | Elapsed Time: 0:00:06 ETA:   0:05:43Websocket: Client connected.
Websocket: Client connected.
 17% (528465920 of<PHONE_NUMBER>) |#######                                  | Elapsed Time: 0:03:16 ETA:  14 days, 2:27:20<urlopen error retrieval incomplete: got only 528465069 out of<PHONE_NUMBER> bytes>
Whisper AI Ready. You can now say something!

Had this happen 3-4 times now. I would imagine Whispering is not ready for use because the model failed to download. Currently I'm downloading it within .cache/nllb200/ myself. Probably should have done that the first time the download failed. As of writing this issue, the download froze for around a minute and resumed again while downloading the model via my browser multiple times. Is there a limit on the storage provider?
Does the zip file just contain the model's checkpoint? If so can I just use the one that is directly provided by Meta? If that's the case, I'm not sure why there's a need to store the models on your server aside from the compression, which will just get uncompressed anyways.

Sorry to hear that you have issues downloading the model.
Unfortunately since it is using the transformer implementation from huggingface, it will not work with just the checkpoints provided by meta.
It should also work by downloading the files from https://huggingface.co/models?search=facebook/nllb and put them to the correct location.
Since huggingface only allows downloading if you have an account on huggingface and have set up a token, i thought it will be too difficult for most people to get it running, so i decided to host the files myself.
There should be no limit except a speed limit by default at about 10 MByte/s (=80 Mbit/s).
I will see if i can add some retry mechanism (or even better, that it just continues where it stopped) or an automatic switch to the US server. (i haven't uploaded it to the US Server yet because i have no automatic switching)
or if its too bad maybe i will host it somewhere else.
Can i ask if you are more closely located to the US or to the EU?
And are you using a WLAN connection or are you connecting via LAN-Cable?

US, connected directly via ethernet
I'll keep note in the future in case there are new models that I should use the ones from huggingface thank you.

@Infinitay
can you test if downloading from the US server is more stable?
NLLB-200 medium size:
https://usc1.contabostorage.com/8fcf133c506f4e688c7ab9ad537b5c18:ai-models/NLLB-200%2Fmedium.zip

can you test if downloading from the US server is more stable?

Worked with no apparent issue.

btw. the NLLB-200 model is not required to run the application. Its just the text translation model for further translation of the whisper results.

I understand but Argos is horrible for Korean. I was resorting to transcribing and manually copy-pasting the transcriptions into Papago's translator. NLLB-200 seems to be doing very well from what little tests I did.

thank you for your help.
Next version will have retries and "fallback" to the US Server.
So i hope it is okay that i close this.
Feel free to open a new one if you find anything else.


Test snapshot packages for v0.10.1

Prior to the v0.10.1 release, the snapshot packages and corresponding Docker images need to be tested.
NPM packages:
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
Docker images:
api3/airnode-deployer-dev:snapshot-v0.10.1
api3/airnode-client-dev:snapshot-v0.10.1
api3/airnode-admin-dev:snapshot-v0.10.1

v0.10.1 testing
tl;dr - everything works, but minor note: there is a warning about the AWS JavaScript SDK version (see end).
A checked box indicates success.
Test that npm packages admin, deployer, and validator, which have a --help CLI argument, are compiled correctly using npx

[x] admin

[x] generate-mnemonic

[x] deployer (tested more below)
[x] validator

[x] Succeeds with --config "packages/airnode-validator/test/fixtures/config.valid.json" --secrets "packages/airnode-validator/test/fixtures/secrets.valid.env" (on the latest commit of the v0.10 tag)

The following use the coingecko-signed-data integration in airnode-examples on Ethereum sepolia, but modified to also test the regular HTTP gateway.
Run Airnode client Docker image locally:

[x] Testnet request
[x] HTTP gateway request
[x] Signed HTTP gateway request
[x] Withdrawal request

Test AWS deployment with deployer Docker image:

Requests

[x] Testnet request
[x] HTTP gateway request
[x] Signed HTTP gateway request
[x] Withdrawal request

Deployer commands (with Docker image)

[x] list
[x] info
[x] remove

Test GCP deployment with deployer Docker image:

Requests

[x] Testnet request
[x] HTTP gateway request
[x] Signed HTTP gateway request
[x] Withdrawal request

Deployer commands (with Docker image)

[x] list
[x] info
[x] remove-with-receipt (differs from AWS to achieve additional command coverage)

Failures
None, but the AWS JavaScript SDK update notification is present in the deployer commands and Airnode logs.
This has been fixed in #1659 for v0.11.
2023-04-05 23:51:39 Please migrate your code to use AWS SDK for JavaScript (v3).
2023-04-05 23:51:39 For more information, check the migration guide at https://a.co/7PzMCcy
2023-04-05 23:51:39 (Use `node --trace-warnings ...` to show where the warning was created)
2023-04-05 23:52:02 (node:24) NOTE: We are formalizing our plans to enter AWS SDK for JavaScript (v2) into maintenance mode in 2023.


Typo on scipy minimization tutorial

On this page https://docs.scipy.org/doc/scipy/reference/tutorial/optimize.html#tutorial-sqlsp and many others where it is reproduced, there is a typo.
For the part that says:

Sequential Least SQuares Programming (SLSQP) Algorithm

The code says
ineq_cons = {'type': 'ineq',
             'fun' : lambda x: np.array([1 - x[0] - 2*x[1],
                                         1 - x[0]**2 - x[1],
                                         1 - x[0]**2 + x[1]]),
             'jac' : lambda x: np.array([[1.0, 2.0],
                                         [-2*x[0], -1.0],
                                         [-2*x[0], 1.0]])}

The first Jacobian is wrong, though. It forgot negative signs I believe. It should be
ineq_cons = {'type': 'ineq',
             'fun' : lambda x: np.array([1 - x[0] - 2*x[1],
                                         1 - x[0]**2 - x[1],
                                         1 - x[0]**2 + x[1]]),
             'jac' : lambda x: np.array([[-1.0, -2.0],
                                         [-2*x[0], -1.0],
                                         [-2*x[0], 1.0]])}

I don't know where to submit typo request, so I am posting it here.

It is seems to be right your comment. The jacobian has a partial derivative of a function.
That means the partial derivative of 1 -x[0] -2*x[1] is [-1, -2].
Should I take the this issue ?
I will commit and PR.

#9128


Modify API Gateway steps to include the new Request body passthrough security option

When creating a body mapping template, APIGW now asks if you want to allow incoming only if the content-type maps to one listed in your mapping templates. Need to include these steps in the instructions.

included in v3.0.0


Support Request: ESP-ME3 with no remaining time under ioBroker

Describe Your Problem
I am using the RainBird integration for ioBroker (ioBroker.rainbird) with my ESP-ME3. I understand that the ioBroker integration is based on this library. All function work fine, except the indication of the remaining time during activation by the programs. I assume this is related to the ESP-ME3 type. Could we try to troubleshoot this...!? Thanks...
Relevant log output
No response
Config for RainBird
No response
Screenshots
No response
Device & Model
ESP-ME3
Node.js Version
n.a.
Rainbird Version
n.a.
Operating System
ioBroker raspberry pi

Should already be supported:
https://github.com/donavanbecker/homebridge-rainbird/issues/396


What is the strategy of carbon-aggregator to deal with delayed data

Hi,
I'm using carbon-aggregator to receive my metrics with some basic rules.
sum.<env>.all.min.metricsReceived (10) = sum <env>.*.min.metricsReceived
If the data is calculated, but there is an , this one overrides the calculated value.
At this moment, sum.test.all.min.metricsReceived:
[10.0,<PHONE_NUMBER>]
After ten minutes,send
test.host1.min.metricsReceived 2<PHONE_NUMBER>
after that sum.test.all.min.metricsReceived change to:
[2.0,<PHONE_NUMBER>]
What is the strategy of carbon-aggregator to deal with delayed data? Is there any setup to avoid this?
Thanks in advance

It sounds like the bucket had already been processed during that 10-minute period and the value of 10 flushed, so when the new reading was received it was treated as a new bucket.  The aggregator always uses the current time, delayed data is not supported.

@DanCech
This situation does exist, and the problem seems to can be fixed, and whether there is a plan to fix it

Hello @nfw999,
It's a know limitation of aggregator, unfortunately.
We are not happy with the current state of aggregator, indeed, but I'm afraid it's not possible to easily fix that. Aggregator should be totally reworked to support delayed and we have no such plans currently.


[Authoring] maxLines is not well handled for a TextBlock whose height is stretch

Target Application
Microsoft Teams
Application Operating System
Other
Schema Version
1.3
Problem Description
When the height of a TextBlock is stretch, the TextBlock will show more than maxLines lines.
Expected Outcome
The rendered TextBlock should never exceeds maxLines lines, regardless of its height.

Actual Outcome
In the web version of Microsoft Teams, the card is rendered as:

In the Online Designer, the card is rendered as:

In both applications, the TextBlock shows the third line, which contradicts "maxLines": 2.
Card JSON
{
    "$schema": "http://adaptivecards.io/schemas/adaptive-card.json",
    "type": "AdaptiveCard",
    "version": "1.3",
    "body": [
        {
            "type": "ColumnSet",
            "columns": [
                {
                    "type": "Column",
                    "items": [
                        {
                            "type": "Image",
                            "url": "https://adaptivecards.io/content/cats/1.png",
                            "width": "60px",
                            "height": "60px"
                        }
                    ],
                    "width": "auto"
                },
                {
                    "type": "Column",
                    "items": [
                        {
                            "type": "TextBlock",
                            "text": "The height of this textblock is stretch. The maxLines of this textblock is 2. But its content is too long for 2 lines. You can see that the third line is also shown.",
                            "height": "stretch",
                            "maxLines": 2,
                            "wrap": true
                        }
                    ],
                    "width": "stretch"
                }
            ]
        }
    ]
}

Repro Steps
Paste the above Card JSON into the Online Designer.

Addressing this issue is not currently on our roadmap, so closing. Please feel free to continue the conversation in this issue for future consideration.


NVM crashes app on Nano S Plus

Any attempt to use NVM on the Nano S Plus results in app panic, even if data accessed just for reading.
The deeper investigation shows that panic happens in the method below (nvm.rs):
    fn which(&self) -> AtomicStorageElem {
        if self.storage_a.is_valid() {
            StorageA
        } else if self.storage_b.is_valid() {
            StorageB
        } else {
            panic!("invalidated atomic storage");
        }
    }

The same code works perfectly fine on Nano S.
Any ideas how to fix that?

Is this on device or in speculos ?
There is a limitation on speculos that prevents it from mapping the whole nvram_data section properly, so it remains uninitialized.

Any updates?

Could you point to a repo that reproduces the issue?

The .data section does not contain the NVM storage, this number is related to something else. Usually apps are "forbidden" to include a .data section because it would not be initialized by the os when starting.

Thanks for the links.
You should use the NVMData wrapper in nanos_sdk, because it handles correct memory access for all devices. It works on Nano S because there's no need for the rwpi relocation model, but on Nano X and S+ this is another story
The following works correctly on device:
#[link_section = ".nvm_data"]
static mut SETTINGS: NVMData<AtomicStorage<u32>> = NVMData::new(AtomicStorage::new(&3));

(We should update documentation in nvm.rs to reflect that some examples will not work unless going through NVMData @kingofpayne for information)

Thanks. this change solved the issue.


onmouseenter for tabs Components

Feature request type
Enhance component
Component name
Tabs
Is your feature request related to a problem?
Ineed onmouse enter event for Tabs to show panel on mouse enter
Describe the solution you'd like
i need it to create ribbon menu
Have you seen this feature anywhere else?
No response
Describe alternatives you've considered
No response
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

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

Cant you do it with blazor's @onmouseenter?

Yes i use this but its not fired when i enter this with mouse

<MudTabPanel OnClick="Toggle1" @onmouseover=@(()=>{Console.WriteLine("test");}) @onmouseout=@(()=>Hover1(false<EMAIL_ADDRESS>Text="API" BadgeData='"!"'  BadgeColor="Color.Error">
@if (Tab1)
{

test

}

You can use onmouseenter but it has a little trick. Little different from onmouseover, look at this:
<MudTabPanel @onmouseenter="MyEvent" ... />

private void MyEvent(EventArgs arg)
{
   //Do something
}

Mouse enter uses event args, not mouse event args


Provide decent error message for wrapped errors

Add an error: Box<dyn std::error::Error> member to the Error enum here. Then include the original error message (and possibly its backtrace, if any) in the Debug and Display impl for the matter-rs Error enum.
Just like the backtrace member, it should be enabled only when both std and backtrace features are enabled, as it would be bringing non-trivial cost to the unhappy path.
Yet very useful for proper diagnostics of errors originating outside of matter-rs itself.

Just like the backtrace member, it should be enabled only when both std and backtrace features are enabled, as it would be bringing non-trivial cost to the unhappy path.

Does it mean this feature will not work on branch no_std? That is, running the no_std version of onoff_light will not have a nice error report.

It will work on branch no_std. "std" is enabled on that branch by default as well.

It will work on branch no_std. "std" is enabled on that branch by default as well.

Oh, I think I am confused. So the no_std branch is not actually no std(It does use the std library conditionally)
Given no_std branch has the std feature enabled by default, does it mean to be really no_std, it is necessary to turn off the std feature manually?

Yeah. I don't see a big deal here. This is a standard pattern for a bunch of no_std-compatible crates.

@ivmarkov what are your thoughts on an API like this to resolve this issue?
pub struct Error {
    code: ErrorCode,
    #[cfg(all(feature = "std", feature = "backtrace"))]
    backtrace: std::backtrace::Backtrace,
    #[cfg(feature = "std")]
    inner: Option<Box<dyn std::error::Error>>,
}

impl Error {
    pub fn new(code: ErrorCode) -> Self {
        Self {
            code,
            #[cfg(all(feature = "std", feature = "backtrace"))]
            backtrace: std::backtrace::Backtrace::capture(),
            #[cfg(feature = "std")]
            inner: None,
        }
    }

    #[cfg(feature = "std")]
    pub fn new_with_details(code: ErrorCode, detailed_err: Box<dyn std::error::Error>) -> Self {
        Self {
            code,
            #[cfg(all(feature = "std", feature = "backtrace"))]
            backtrace: std::backtrace::Backtrace::capture(),
            #[cfg(feature = "std")]
            inner: Some(detailed_err),
        }
    }

    #[cfg(feature = "std")]
    pub fn details(&self) -> Option<&dyn std::error::Error> {
        self.inner.as_ref().map(|err| err.as_ref())
    }
} // impl Error

#[cfg(feature = "std")]
impl From<std::io::Error> for Error {
    fn from(e: std::io::Error) -> Self {
        // Keep things simple for now
        Self::new_with_details(ErrorCode::StdIoError, Box::new(e))
    }
}

With the above, I get a log like this:
[2023-08-17T23:57:44Z WARN  on_off_light] mDNS exited: Os { code: 98, kind: AddrInUse, message: "Address already in use" }

Generated by:
res = mdns_runner => {
    log::warn!("mDNS exited: {:?}", res.unwrap_err().details().unwrap());
},

With the above, I get a log like this:
[2023-08-17T23:57:44Z WARN  on_off_light] mDNS exited: Os { code: 98, kind: AddrInUse, message: "Address already in use" }

Generated by:
res = mdns_runner => {
    log::warn!("mDNS exited: {:?}", res.unwrap_err().details().unwrap());
},

Would work, however I would also suggest:
#[cfg(all(feature = "std", feature = "backtrace"))]
inner: Option<Box<dyn std::error::Error>>,

I.e. the Error member should be protected with the backtrace feature in addition to std as its collection is non-trivial and needs allocations.
Additionally:

The fmt::Display implementation of Error should be extended - when std and backtrace are enabled - to display the error message from the dyn std::error::Error so that you don't have to reach out to res.unwrap_err().details() every time
The fmt::Debug implementation of Error should also be extended to capture the backtrace - if it exists - from the dyn std::error::Error - if possible. Not sure if this is possible with Rust stable as of yet and if the std::error::Error captures a backtrace


(BOLT-879) Add note to docs about installing kerberos

What this changes Adds a note to the docs that the user needs to install the kerberos gem on their own
Why The license for kerberos is incompatible with other components we distribute, so we need to remove it from our packaging and users who need it need to install it on their own.

It looks like the logic around the gem not being installed is already there, and we've already removed it from our packaging. Let me know if there's any other work we need to do that I'm missing.

Waiting for CLA signature by @lucywyman
@lucywyman - We require a Contributor License Agreement (CLA) for people who contribute to Puppet, but we have an easy click-through license with instructions, which is available at https://cla.puppet.com/
Note: if your contribution is trivial and you think it may be exempt from the CLA, please post a short reply to this comment with details. http://docs.puppet.com/community/trivial_patch_exemption.html


Request: Proper Documentation

It's not clear precisely what this mod does. Under which circumstances do vanilla stacks overflow? Which stack in particular? And why is there an emphasis on the fact that this is for servers? I would expect this to also be the case for client servers? This should probably be spelled out because it's not even clear to me whether any crashes can occur or why this is necessary.

This is mostly for servers preventing players having stacks bigger than the max stack size. This could be caused by glitches etc..). It was a specifically requested by a user on reddit.
Hopefully this clears this up. Feel free to close the issue or ask more questions.

Here is the original post
https://www.reddit.com/r/fabricmc/comments/ncxegt/looking_for_a_server_moddatapack_that_sets_the/?utm_medium=android_app&utm_source=share


[Enhancement] Time Frames and Search Scopes

To make reading from the CSV report the application compiles easier, it would be nice to specify a number of time frames depending on the configured duration.  The frame starts with the first commit on that day and ends with the latest commit which still fits the given duration.  If the duration exceeds, a new frame starts.  Single commits would become frames on their own.  The application should support up to five or six time frames to be called with the --frame <INTEGER> option, optional shorthand:  -F, being zero by default.  In case one frame is empty but configured, no string will be written to the report for that CSV field.
The second thing is a search scope.  When a time pattern with a fixed format is given using the --scope <TIMESTAMP> option, optional shorthand:  -S, only commits which match this time pattern will be revealed for analysis.  Example:
commit-analyzer --git --scope 202205.. # Read from the local Git history, only take care for May 2022.

What do you think about these suggestions, @wert007?

Hmm, what would the time frame feature be for?
Hmm, sounds interesting, but I think specifying the date format could be a little bit tricky. Like should this be fixed, to always specify a year and a month? Or are other specifications possible? If it is the former then I think scope is to generic, since it basically is just a monthly filter.

The time frames are for the Excel report; there is a Rust crate to write to Excel sheets and I intended the time frames to be the input information. And yes, a filter by year and month would totally suffice; I just thought a more generic processing would be nice, as well.

I still do not quite get, how the time frames are useful for that. 😅
Might be, but that would also turn into a lot of overengineering. I think it would be too hard doing it right, so a simple filter option should suffice.

I thought this crate might work: https://github.com/outersky/simple_excel_writer.
But unfortunately, it is Apache 2.0 and thus higher than commit-analyzer. There were two other crates, as well, but I need to look for their licenses at first; they are excel_writer and basic_xlsx_writer.

Not being a lawyer and stuff, but using it as a library in this code base should be fine.

Licenses need to be compatible with each other.  MIT is upward compatible with Apache 2.0 such that any MIT stuff can be used in Apache 2.0 projects without any restrictions.
But Apache 2.0 is not downward compatible with MIT such that any Apache 2.0 stuff cannot be used in MIT projects.  It would be a license infringement.
To take care about this problem, many Rust projects are dual-licensed MIT and Apache 2.0, including Rust itself.  Dual-licensed projects are known to OR their licenses.
To solve this problem, you could add a further license to your project -- namely Apache 2.0 -- or we employ another library therefore.  The latter option is the easier one.

Okay, I researched a little bit, the reason everything in rust is often dual licensed is, because the primary license is Apache 2.0, but add MIT to be GPL compatible. Adding Apache 2.0 crates in your Cargo.toml is totally fine. source

Thank you for your research.  Could you please summarise the ideas of the source?  Honestly, I have no idea how read those reddit threads since there is no clear relationship between the single messages...

All of those licenses are "permissive"/"non-protective" licenses (naming depends on whether you want to highlight their pros or cons) and they are very much free to use as dependencies. Your choice of license for your own crate is not restricted by them. [1]

[1] ...unless you are releasing your crate into the public domain, in which case you cannot "embed" them (using them is fine).

Basically this answer. So embedding the code by cloning the git repository would be problematic (then again I think, that would be doable), but just adding a dependency to the toml file is totally fine.

Since there are multiple implementations to choose from, we should take a look at all of them since their APIs can be expected to differ.  This might be an important reason to search for alternatives anyway.  We should choose the crate with the most comfortable API for this use case.

This seems to be the crate with the most features and some recent activity:  https://github.com/informationsea/xlsxwriter-rs (Apache 2.0).
Do you know any other crates with MIT license, recent changes in the repository, access to the repository at all and functionality to read from as well as write to Excel files?

This seems to be the crate with the most features and some recent activity: https://github.com/informationsea/xlsxwriter-rs (Apache 2.0).

I mean, there is no need to have the most features, is there?

Do you know any other crates with MIT license, recent changes in the repository, access to the repository at all and functionality to read from as well as write to Excel files?

No, I have never tried writing anything to excel files in rust, nor have I read from people trying to do it.

Since there are multiple implementations to choose from, we should take a look at all of them since their APIs can be expected to differ. This might be an important reason to search for alternatives anyway. We should choose the crate with the most comfortable API for this use case.

Hmm, maybe you should define, how this program should write to excel files. How it should interact with excel files and what formatting it should do. And how configurable it should be.


Doc: Clear type errors and warnings in the rescript playground where posible

fixes #600

@cristianoc 👍🏿


Pagination support?

Would be nice to have pagination support in gh-got so consumers don't have to implement it themselves

Agreed. I've needed that before too, but never had the time to implement it here. A pull request would be welcome, but I honestly doubt I'll get to it myself...

github-parse-link might help for that as it can parse the pagination header from GitHub.

@SamVerschueren as next, last, first, and prev are well defined relation types according to section 6.2.2 of RFC 5988, could we build pagination support on top of those Link references into got itself?

@destroyerofbuilds How would that look like if it would be part of got itself? Maybe exposed as an async iterator of some sort?

From my perspective as a user, an async interator accessed via for-await-of, looks ideal.
As a consumer, my code would be:
(async function () {
  for await (const response of gotIterator(`https://api.github.com/orgs/microsoft/repos`, {json: true})) {
    console.log(response.body.map(repository => repository.name));
  }
})();

The async generator was also quite easy to implement:
'use strict';

const got = require(`got`);
const parse = require(`github-parse-link`);

async function* gotIterator(url, options) {
  let next = url;
  do {
    const response = await got(next, options);
    next = parse(response.headers.link).next;
    yield response;
  } while (next);
}

A few thoughts on the approach:

I had to wrap got in an async generator. Is there a way to reuse the same entry point?
Async iterators are only available starting with Node 10.3 (without a flag).

Fetching only the first result is very verbose in my opinion:
const repositories = gotIterator(`https://api.github.com/orgs/microsoft/repos`, {json: true})[Symbol.asyncIterator]();
repositories.next().then(x => x.value).then(response => console.log(response.body.map(repository => repository.name)));

The semantics of got's stream interface wouldn't allow us to stream a series of responses based on fetching all next resources, nor would it be flexible as async iterators, since you would end up fetching everything, regardless of whether you needed it.
Perhaps, as an alternative, you could use the existing got promise interface, with an option of passing in a previous request, to assist got in augmenting the next request:
let request;
do {
  request = got(`https://api.github.com/orgs/microsoft/repos`, {previous: request});
}  while (request.hasNext);

That looks great!

I had to wrap got in an async generator. Is there a way to reuse the same entry point?

I don't really know. I'm not that familiar with async generators and iterators yet.

Async iterators are only available starting with Node 10.3 (without a flag).

That's fine. We'll just detect if Symbol.asyncIterator exists and only support it then. We'll also document that it requires Node.js 10.

Fetching only the first result is very verbose in my opinion:

Why would you use the iterator interface for that? You could just use normal Got.

Perhaps, as an alternative, you could use the existing got promise interface, with an option of passing in a previous request, to assist got in augmenting the next request:

Not clear if you mean this as an alternative to async iterator support or in addition to?

That's fine. We'll just detect if Symbol.asyncIterator exists and only support it then. We'll also document that it requires Node.js 10.

:+1:

Why would you use the iterator interface for that? You could just use normal Got.

I simply wanted to avoid the cognitive overhead of remembering two functions and their use cases (one for the first page, another for fetching all pages).
However, you are correct. I could use the original got interface for fetching the first page.

Not clear if you mean this as an alternative to async iterator support or in addition to?

As an alternative. To allow using the same function for fetching the first page, and paginating, and to allow support for older Node releases.

This now has pagination support, inherited from Got: https://github.com/sindresorhus/got#pagination

just a heads up that GitHub's REST API endpoints that paginate are not consistent. Most respond with an array, but some respond with an object and a key holding the results array. Not even the key is always the same 🤷♂
See https://github.com/octokit/plugin-paginate-rest.js/blob/e7c5443dc4126f9826be9d069c5396b55ef98fc9/src/normalize-paginated-list-response.ts#L20-L47


FAI-335: Plugging data dictionary to state management (PMML editor PoC)

See https://issues.redhat.com/browse/FAI-335
@manstis I plugged data dictionary fields add, batch add, edit, delete, order and partial range constraints handling (only one interval at the moment).
Following PR's will cover full range and enums constraints and validation.

@kelvah Do you mind pushing an empty commit to this PR?  We fixed our CI :: Full downsteram check and a new commit will trigger it with the fixed script.
git commit --amend --no-edit && git push -f [remote] [branch]

sure, no problem. pushed.


re-support ruby 2.3

Hi. I'm a maintainer of fluentd. fluend began to use async-http in this PR
I'd like protocol-http to re-support ruby 2.3 which is the version fluentd supports yet.
I know that MRI teams had ended the support of ruby 2.3. however, ruby 2.3 is still used by many people, and we have to support this version.
If this gem re-support ruby 2.3, it's very helpful for fluentd.
What do you think of this?  @ioquatix

I am okay with this, but do we need more than one PR to make it possible?

Could you release new version protocol-http and async-http which uses the new version of protocol-http when you have time.

Sure.

We also need to check protocol-http1 and protocol-http2 for compatibility.

these gems seem not to be needed to update. or is there anything I missed?
https://github.com/socketry/protocol-http1/blob/95d318bb1b8a74da377e40d1c94f1fa98a9e5a1d/protocol-http1.gemspec#L21
https://github.com/socketry/protocol-http2/blob/6f4259832da603167d1fe25ec575b7bf6bf543e5/protocol-http2.gemspec#L22

Need to add 2.3 back to travis
e.g. https://github.com/socketry/protocol-http2/blob/master/.travis.yml

Thank you for your help. I sent patches!
https://github.com/socketry/protocol-http2/pull/2
https://github.com/socketry/protocol-http1/pull/1


Updating to use the AWS 2.0 SDK

Problem Statement
The AWS SDK was significantly out of date.  Also, the withokta script made it difficult to use when not actually sending an aws --profile command.
Solution
Updated to the Java 2 SDK and removed the obfuscation around profile.

@DavidTanner Can you please resolve conflicts in this PR?

@mraible Sorry, I missed your message.

@mraible @aaronpk @rdegges or @robert-chiniquy I'd really like to get this merged so we can start using it.  I've been using it locally for months.

@DavidTanner great PR!
Looks good!

lots of package name changes
fluent method's drop the with
region is no longer a simple string
minor regex change to add -gov (plus a test ❤)

Longer-term question:
It's been a while since I've looked at this project, so forgive me for asking.
Is this tool still needed now that the  v2 version of the AWS CLI supports SSO directly
IIRC, The AWS CLI v2 uses an OAuth Device Grant (e.g. similar to login /w Netflix using a code on your TV), which will pop a browser if possible.  Then AWS does the SAML dance with Okta.
Either way, getting this in and released would help folks in the short term 😄

@bdemers It looks like SSO isn't available in GovCloud yet which is why we still need it.  Is there anything else I need to add to this PR to get it merged?

@DavidTanner Thanks for the note about SSO in GovCloud!
I'll chat with the other maintainers about cutting a release early next week.
Thanks again!!

@DavidTanner @bdemers while they may have fixed things for those not using a profile, it broke things when actually using a profile.
#374 #375 #376
Can we either roll this back, or fix the other issues?  This is unusable in the current state if you are using a profile.

@dangeReis The changes I made are breaking.  I updated the Readme accordingly.  You should start using the OKTA_PROFILE to set the profile name.

@DavidTanner which Readme did you update?  I'm not seeing anything relevant here:
https://github.com/oktadev/okta-aws-cli-assume-role/blob/master/Readme.MD
Specifically, commands like this don't currently work properly:
okta-aws test sts get-caller-identity

I see the issue.  Would the preferred functionality be then to force the okta profile to match the aws one?  At my company one OKTA_PROFILE gives us access to multiple aws profiles.  I can put up a PR to fix the scripts.

At my company, once you authenticate with OKTA, it locks you into an AWS role.  If you want a different role/account, you need to use a different okta profile or clear our some session files.  Not quite sure how you are using this.
We currently associate the okta profile with the appropriate role/account that we want and just reauthenticate when the timeout expires.  This allows us to use multiple profiles at the same time if we needed to.

~/.aws/config
[profile company-dev]
region = SOME_REGION
role_arn = SOME_ARN
source_profile = SOME_OKTA_PROFILE

BASH_PROFILE="SOME_OKTA_PROFILE" AWS_PROFILE="company-dev" AWS_SDK_LOAD_CONFIG="1" withOkta aws ...

~/.aws/credentials
[SOME_OKTA_PROFILE]
...


Update azure-multiapi-storage package to take 'futures' fix

Closes https://github.com/Azure/azure-cli/issues/3988

Thanks for the review.
Would like @troydai to take a look also.


[WR] show street segment errors for a specific locality

Pivotal Card
Fixes a bug where street segment errors for a specific locality were not showing.
@tie-rack, is this inline with the way you set these up?

Yep. That's the pattern.
LGTM.