Restructure Gulp Tasks

Make the naming of the gulp tasks and their dependencies more convenient.

gulp task production should use the minified version of the assets
gulp task production will be used for the prototype view and therefore it should be ensured that the server always is on (e.g. with forever)
there could be an additional gulp task witch gives a temporary public url for testing purposes (e.g. ngrok)

It would also be more convenient, if gulp tasks would be split up into separate task files which then get aggregated. Something like unic did, I'm currently working on this in another project myself (updating it in a generated project).

I already began with the implementation of this task. No worry – the tasks will be splitted up in separate task files ;-)  Although our approach is a little bit different than the approach mentioned above.

Useless task production was removed.
Typically, a production server needs more control.

Sorry, didn't mean to reopen. Just wanted to say that there are a few servers actually using the task.. Might run into the "update"-issue we've discussed on Thursday..?


Add indra2vec month 10 report

@bgyori I can't push to  https://github.com/bgyori/emmaa so I just branched off of your branch and am PR'ing back into your branch. Please let me know if this is too long or too short and I can adjust it accordingly. Also, should I be a bit more reference-happy?

I was thinking that you could just push to your fork's branch on top of my branch without PR-ing but it's fine like this too!


CustomOverlay does not exist in MobileBarcodeScanner

Hi,
I've updated to version <IP_ADDRESS> but I still cannot find CustomOverlay property in MobileBarcodeScanner.
What I'm missing?

And this was broken in v2.4.1 too?

It was missing in the v2.41 too


chore(package): update got to version 10.0.3

:tada: This PR is included in version 3.0.5 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Setting language or other parameters

Is there any way to select language of the source file to transcribe and set some parameters if available? Now its accepting file language as English as default which is not the use case for my case.

Hi there 👋 you can do this by clicking the settings (gear) icon, then enabling "Multilingual" support! In the main version, you can do this after loading an audio file.


Refactor the Secured booster documentation

Based on the outcome of the SSO booster planning discussion and the related email thread
Proposed changes:

Consider renaming the mission "Secured" -> "Secured HTTP API" (for accuracy).
Docs should focus on describing CLI-based interaction with the secured HTTP-API (using curl)
if possible, include a procedure on requesting the bearer token using  the booster's web UI
(alternatively, the web UI could contain cmdline instrucutions on obtaining the token , or point to the relevant procedure described in docs)
Both worflows (cmd and WebUI) should work with the same Keycloak keycloak configuration, which should be provided out of the box with no additional config change required.
Document the constraints of the booster vs. real world applications (Such as the security downgrade of the Keycloak realm from bearer-only to public, which while necessary for the booster to work out of the box, could present a problem when faced with real-world application)
Drop the java library from the SSO module and replace with curl commands
Drop the integration tests and move them to each booster implementation
Reduce the SSO module to just the SSO setup (i.e. remove the java code) and keep the git sub module approch

DISCLAIMER: The list above is not final, decisions are still being made about the next iteration of the Secured booster, but we can keep track of it here.
TODO: Link to relevant JIRA tickets here...

Docs should focus on describing CLI-based interaction with the secured HTTP-API (using curl)

We actually already have this :-)

We started to capture the required changes in a google doc, before consolidating it and submitting a PR: https://docs.google.com/document/d/10AR1-QW1J4UGphEXcy4c-bWulxH1Lo06XcjqkVCfyE4/edit#

Oh snap, you're right, @Ladicek :) Thanks for catching that. I've removed the item.

closing this as the tasks are now covered by individual issues (see description)


Change of behaviour in link parsing - a breaking change?

I've been using email links like the following in my raw markdown
[info@website.com](<mailto:info@website.com?subject=Turf Availability Enquiry>)

these worked up till commit 3b46d7151f3eb09744d78529e060e0769bee80ac where behaviour has changed.  I now have to encode the subject part, encoding the spaces in the subject, prior to this commit the library correctly parsed and output a correctly formed mailto link.
I don't mind encoding where I have to, but ultimately this raw markdown will be entered and maintained by end users whom I don't expect to understand or know about encoding.

This matches the specification (see try.commonmark.org, though you might need to refresh the page after loading).
This was changed in CommonMark specification 0.24, the release notes mention two related discussions:

Disallow spaces in link destinations, even inside pointy braces. Discussion at http://talk.commonmark.org/t/779 and http://talk.commonmark.org/t/1287/12

Thank you, I thought that might be the case, the spec took a bit to decipher and wasn't sure if I read it right.
If I were to undo that commit so as to allow spaces in <> links, do you know if there would be any implications to the library correctly parsing other markdown?  Or even better how could I go about implementing (an extension?), spaces in query parameters for <> links?

I would probably advise against that since that would go against the goal of CommonMark which means that the same markup can be used everywhere. By the way, the spaces in the links are supported only in a few parsers: http://johnmacfarlane.net/babelmark2/?normalize=1&text=[foo](<asd+asd>)


'Virtual network is not supported in location 'southcentralus' for container group'

[Enter feedback here]
South Central US is not supported for vnet integration of ACI, please modify the details on the link
https://docs.microsoft.com/en-us/azure/container-instances/container-instances-vnet#regions-and-resource-availability

Location
OS
CPU
Memory (GB)

North Europe, Southeast Asia, Central US, East US, East US 2, South Central US, West US 2
Linux
4
16

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

ID: 81e00caf-dd95-441b-d2d5-0b3c979cb11b
Version Independent ID: 83ed8e59-7ae8-332e-9ce0-7d80e8a973f8
Content: Deploy container instances into an Azure virtual network
Content Source: articles/container-instances/container-instances-vnet.md
Service: container-instances
GitHub Login: @dlepow
Microsoft Alias: danlep

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

@dlepow We are seeing this error: Virtual network is not supported in location 'southcentralus' for container group 'test'. Please use one of the locations 'EastUS2EUAP,CentralUSEUAP,WestUS,WestCentralUS,NorthEurope,WestEurope,EastUS,AustraliaEast,JapanEast,NorthEurope,SoutheastAsia,EastUS2,WestUS2,CentralUS'.
Can you please confirm and update the document as needed.
@JuneZhao Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

@JuneZhao - Thanks for pointing this out. I repro the issue as well, so will update the article.
cc:/ @dkkapur - FYI

Fixed in https://github.com/MicrosoftDocs/azure-docs-pr/pull/81766. Update will publish later today.
#please-close


Shows audio and slides only if data is in correct format

Checklist

[x] I have read the Contribution & Best practices Guide.
[x] My branch is up-to-date with the Upstream development branch.

Short description of what this resolves:
Shows audio and slides only if data is in the correct format.
Changes proposed in this pull request:

Adds check statements for data format.

Fixes #1929
Server Link: https://opev-gen.herokuapp.com/
Preview Link: https://agbilotia1998.github.io/OWA/tracks.html#4323

@geekyd @pradeepgangwar @srv-twry please review.


Table view as dashboard filter - don't highlight item or clear filter

Make sure these boxes are checked before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

Superset version
0.28.1
Expected results
Table view can be used as dashboard-wide filter if the checkbox "Emit Filter Events" is set.
Clicking an item once should filter the whole dashboard and highlight the selected items.
Clicking again should then clear that filter and un-highlight the item.
Actual results
The filter is applied but the item is not highlighted.
Clicking a second time does not clear the filter.

same issue

That's fixed in master

I deployed superset using docker
https://superset.incubator.apache.org/installation.html#start-with-docker
so version is master
It seems that

Clicking a second time does not clear the filter

still doesn't fixed.
Actually,
after first time clickiing - row is selected, highlighted and other charts (on the same datasource) are refreshed (see requests to server)
after second clicking - row is deselected, but other charts aren't refreshed (no requests at chrome dev tools)

It seems to be，highlight item is ok，but clear filter is not

Someone wrote in to<EMAIL_ADDRESS>regarding this still being an issue. We should investigate further if it's still an issue in master.


plotter to plot()

use HEdtree method to dispatch on plot() generic? If we make our own inherited Node class should be easy to do...

I'd prefer that indeed, and would see as a benefit to introducing a specific class


Release v0.3

Codecov Report
Attention: Patch coverage is 85.63734% with 80 lines in your changes missing coverage. Please review.

Files with missing lines
Patch %
Lines

src/web.rs
81.38%
67 Missing :warning:

src/cli.rs
95.03%
7 Missing :warning:

src/bin/git-events-runner.rs
0.00%
3 Missing :warning:

src/bin/gitrepo-cloner.rs
0.00%
3 Missing :warning:

Files with missing lines
Coverage Δ

src/cache.rs
66.77% <100.00%> (+0.79%)
:arrow_up:

src/config.rs
92.57% <100.00%> (+0.07%)
:arrow_up:

src/controller.rs
46.15% <100.00%> (+0.54%)
:arrow_up:

src/lib.rs
100.00% <100.00%> (ø)

src/resources/action.rs
99.50% <100.00%> (+<0.01%)
:arrow_up:

src/resources/trigger.rs
79.62% <100.00%> (+0.15%)
:arrow_up:

src/bin/git-events-runner.rs
0.00% <0.00%> (ø)

src/bin/gitrepo-cloner.rs
0.00% <0.00%> (ø)

src/cli.rs
84.65% <95.03%> (+41.80%)
:arrow_up:

src/web.rs
82.03% <81.38%> (+0.70%)
:arrow_up:


remove minergate and minergate-cli

per discussion in https://github.com/caskroom/homebrew-cask/pull/45666

Thank you.


front matter params not correctly linked

Sorry if I appear newbie... I am new to this framework.
If I try to access url params from partials like below:

{{param}} will resolve to an empty array eventually. Like this:

I looked into the DefaultController which comes with using you guys' F4A CLI tool.

Looks like the way params is initialized to be an array is the culprit?
After modifying so params init to be a {} it sorta works.

Is my workaround valid or is there another way to access url params?

@robotaoyuan This workaround is fine, but you can also create your own controller specific to that view and extend the default controller to access the params that way by using the $stateParams scope variable.
---
name: algo
controller: MyAlgoController
---
params: {{params}}

MyAlgoController.$inject = ['$scope', '$stateParams', '$state'];

function MyAlgoController($scope, $stateParams, $state) {
    $scope.params = $stateParams;
}


NodeGroup reverses the rendering of elements within a Each

Reported on community.
Fuse Version
fuse version 1.3.0
OS
Mac OS High Sierra
What do I expect?
I expect to display items according to the array order.
What happens instead?
items are displayed in reverse order.
What is the cause?
The cause is the use of the NodeGroup that for some reason reverses the order at the time of the rendering within a loop Each.
Reproduce the issue
<App>
   <JavaScript>
   var Observable= require("FuseJS/Observable");
   var items = Observable(1,2,3,4,5,6,7,8,9,10);

   module.exports ={items:items}
   </JavaScript>

   <StackPanel>
   	<Each Items="{items}">
   		<NodeGroup>
   			<Text>Element:</Text>
   			<Text>{}</Text>
   		</NodeGroup>
   	</Each>
   </StackPanel>
</App>


Added Tim Sort

Added Tim Sort in Python, C++ and Java.

@shhossain Hey, Can you label this PR as Hacktoberfest accepted.

@SreeHaran added.


upgrade typespec to 0.58.0

upgrade typespec to 0.58.0

Regen PR in autorest.csharp : https://github.com/Azure/autorest.csharp/pull/4921


411 error returned when calling _api/contextinfo

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

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

Additional environment details
Chrome version : Version 92.0.4515.107 (Official Build) (64-bit)
SPFX - 1.6.0
Describe the bug / error
We have a form built using SPFX solution. On save or submit action sharepoint rest api is invoked.
When user clicks on save or submit the form get stuck and does not perform any action. On the browser console log we could see 411 error returned (content length required) on the rest api _api/ContextInfo request(screenshot attached).
We tried in chrome, edge and mozilla browser all shows the same result. In some blogs, there were solutions to add content-length:0 in the header. But this value is been automatically been added by all the major browsers if there is no content in post request.
We tried clearing browser cache and clearing cookies. All these resolution failed. Any suggestion on how to resolve this issue ?
Steps to reproduce
The issue happened to a particular user and he did a recent windows update in win 10 machine. Post that when he tries to submit a form. the browser does not respond and in the console log he gets 411 error issue

Expected behavior
411 error on the _api/ContextInfo rest api request.

Who is making the call to ContextInfo?


CHS Patchouli Guide Translation

@Determancer @mcBegins2Snow @TUsama I wonder if you can help me review this ^^

It can be merged, everything is finished.


Can't turn off Keyboard Shortcut Manager in Settings V2

Keyboard manager is not turning off, but this seems to apply to everything in the settings

For KBM this is being fixed in PR #2314

@saahmedm I renamed the Issue.

As @arjunbalgovind mentioned, this should be fixed on this PR #2314

@saahmedm, as discussed offline this issue ONLY applies to KBM and PowerLauncher. I will re-assign it to @arjunbalgovind and @traies  since he is closing it in his pr

Should be resolved as of #2314


https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Using_promises

Request type

[ ] Please close this issue, I accidentally submitted it without adding any details
[ ] New documentation
[x] Correction or update

Details
Testing promises, I think, I've found a small mistake in this piece of documentation:

Unlike "old-style", passed-in callbacks, a promise comes with some guarantees:
Callbacks will never be called before the completion of the current run of the JavaScript event loop.

I've made a little script to test this behavior:
var m

console.log("test: init")

m = (new Promise(function(resolve, reject) {
  console.log("test: inside 'Promise' callback")
  resolve("O")
})).then(function(result) {
  console.log("test: inside 'then' callback")
   return result + "K"
})

setTimeout(function() {
	console.log(">> another event-loop cycle: m =", m)

}, 0);

console.log("test: m =", m)

This is the console log in firefox:
 test: init
 test: inside 'Promise' callback
 test: m = Promise { <state>: "pending" }
 undefined
 test: inside 'then' callback
 >>> another event-loop cycle: m = Promise { <state>: "fulfilled", <value>: "OK" }
```

From this log seem that Promise-callback have been called in current event loop cycle.

I'd like to take on this one 🤚
First hunch is it's because there's a different queue for setTimeout and promises, but I will confirm after rewatching Jake's In the loop talk

Sure thing. I've assigned you!

Hey @fabio-sassi
The reason why in your code you see the console log from the .then() callback is because it's a microtask
Whereas the setTimeout is queuing it into the task queues
But the microtask will run almost asap whenever possible, I'd definitely recommend watching this talk (you can start from the timecode I specified in the link) to better understand the concept: https://youtu.be/cCOL7MC4Pl0?t=1444

@chrisdavidmills do you think we should close the issue or would you like to have some update for the docs?

@jadjoubran thanks for looking into this Jad. I think it'd be nice to offer a bit more explanation of this, as there is a lot of misunderstanding here.

Sure thing, I just added a section on Task queues vs microtasks with a refactored example from @fabio-sassi (Thank you!) and linked to the MDN Tasks vs microtasks page.
https://wiki.developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Using_promises#Task_queues_vs_microtask
It would be wise to get a quick technical review on what I authored in case I made a mistake.

@jadjoubran looks good, thanks! @fabio-sassi , what do you think?

Thank you so much @jadjoubran to have a look over this problem.
The video about microtask give a deeper and better comprehension
about the concept behind Promises.
I think my example code was a bit confused (I could avoid the
setTimeout).
I write it again in a more clear way:
console.log("sync - start")

var m = (new Promise(function(resolve, reject) {
  console.log("Promise callback")
  resolve()
})).then(function() {
  console.log("Then callback")
})

console.log("sync - end")

This is the log:
sync - start
Promise callback
sync - end
undefined
Then callback

In the documentation, subject of this issue, we can read:

Callbacks will never be called before the completion of the current run of the JavaScript event loop.

but the promise-callback is called in a sync way, so code execution don't follow this rule!
This is similar to the Jake Archibald example at time 26:54 but in this
case 'Hey!' is printed before 'Yo!'
Also if promise-callback is a microtask, it should not be executed after console.log("sync - end")?
The then-callback instead called at the end of main execution as we expected.

Hey @fabio-sassi, sure thing happy that it's clearer now!
I think it's because the constructor of the promise is not a callback, which is why it's being called immediately. So I'd rewrite your code as following:
console.log("sync - start")

let m = (new Promise(function(resolve, reject) {
  console.log("Creating promise")
  resolve()
})).then(function() {
  console.log("Then callback")
})

console.log("sync - end")

The promise constructor (the function that you pass to new Promise()) needs to run immediately so that it returns an instance of promise so that the .then() could would queue the callback in the future.
Does that make sense?

@jadjoubran, you are right!
I looked it up online and I found that argument of Promise is a called "during the process of constructing the promiseObj" here:
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Promise/Promise

new Promise(executor)
executor
A function to be executed by the constructor, during the process of constructing the promiseObj.  The executor is custom code that ties an outcome to a promise.

My confusion now is about the semantic, because if executor can be considered a callback as defined in wikipedia:
https://en.wikipedia.org/wiki/Callback_(computer_programming)
then this sentence is wrong:

Callbacks will never be called before the completion of the current run of the JavaScript event loop.

because executor callback is called in the current event loop but if callback is defined as call-after:
https://stackoverflow.com/questions/824234/what-is-a-callback-function/7549753#7549753
the the sentence is correct.

Hey @fabio-sassi
The executor is not a callback.. I think you might be confusing that because the wikipedia page mentions executable code? that is different than executor
So far from the tests above, the sentence Callbacks will never be called before the completion of the current run of the JavaScript event loop. seems to be correct
Does that make sense?

Hey @jadjoubran
From wikipedia:

In computer programming, a callback, also known as a "call-after"[1] function, is any executable code that is passed as an argument to other code; that other code is expected to call back (execute) the argument at a given time. This execution may be immediate as in a synchronous callback, or it might happen at a later time as in an asynchronous callback.

Executor is a function passed as argument of other code (the promise constructor) and "This execution may be immediate as in a synchronous callback" as in this case ... then why isn't a callback?
From the point of view of code workflow, executor is the same of PrintTwoNumbers's argument, in wikipedia page C example:

/* The calling function takes a single callback as a parameter. */
void PrintTwoNumbers(int (*numberSource)(void)) {
    int val1 = numberSource();
    int val2 = numberSource();
    printf("%d and %d\n", val1, val2);
}

[...]

int main(void) {
    [...]
    PrintTwoNumbers(&rand);
    [...]
}

I know that callback used to be thought always as "asynchronous" but definition (at least in wikipedia) talk also of syncronous-callback.
From my point of view to avoid this semantic trap, the sentence: Callbacks will never be called before the completion of the current run of the JavaScript event loop. should be enriched with a note that exclude Promise argument from the "Callbacks" group.
Something like:
Callbacks will never be called before the completion of the current run of the JavaScript event loop (note: Promise argument is called in a sync way and must be excluded from ...

Hey @fabio-sassi
This is outside my experience but as far as I know, the function you pass as a constructor is not a callback. You can think of it as the constructor definition, so I don't see it as an exception

@chrisdavidmills is it possible to get a second opinion on this one to see whether further changes need to be made? Thanks!

I think we should close this one now.


Seeing a panic on sign-and-notarize

Hi there,
What happened:
quill sign-and-notarize dist/dot_0.0.1_darwin_arm64/dot
Caught panic:

interface conversion: tea.Model is frame.Frame, not *frame.Frame

Restoring terminal...

goroutine 11 [running]:
runtime/debug.Stack()
        /opt/hostedtoolcache/go/1.23.2/x64/src/runtime/debug/stack.go:26 +0x64
runtime/debug.PrintStack()
        /opt/hostedtoolcache/go/1.23.2/x64/src/runtime/debug/stack.go:18 +0x1c
github.com/charmbracelet/bubbletea.(*Program).recoverFromPanic(0x14000562500)
        /home/runner/go/pkg/mod/github.com/charmbracelet/bubbletea@v1.2.1/tea.go:705 +0x84
panic({0x1032bffa0?, 0x14000612090?})
        /opt/hostedtoolcache/go/1.23.2/x64/src/runtime/panic.go:785 +0x124
github.com/anchore/quill/cmd/quill/internal/ui.(*UI).Update(0x140005e65a0, {0x1033500c0, 0x1400013ab80})
        /home/runner/work/quill/quill/cmd/quill/internal/ui/ui.go:173 +0x7e4
github.com/charmbracelet/bubbletea.(*Program).eventLoop(0x14000562500, {0x1033f9ff8?, 0x140005e65a0?}, 0x14000042380)
        /home/runner/go/pkg/mod/github.com/charmbracelet/bubbletea@v1.2.1/tea.go:461 +0x6a8
github.com/charmbracelet/bubbletea.(*Program).Run(0x14000562500)
        /home/runner/go/pkg/mod/github.com/charmbracelet/bubbletea@v1.2.1/tea.go:595 +0x8cc
github.com/anchore/quill/cmd/quill/internal/ui.(*UI).Setup.func1()
        /home/runner/work/quill/quill/cmd/quill/internal/ui/ui.go:62 +0x54
created by github.com/anchore/quill/cmd/quill/internal/ui.(*UI).Setup in goroutine 1
        /home/runner/work/quill/quill/cmd/quill/internal/ui/ui.go:60 +0x1d4
notarization failed: binary is not signed thus will not pass notarization

What you expected to happen:
DIdn't expect the panic
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Not sure, it's my first time trying this tool :)
Environment:

Output of version command: go version go1.23.0 darwin/arm64
OS (e.g: cat /etc/os-release or similar): OSX

Hi @matthewmueller which quill version is this?

Most likely latest. I just ran the install script and tried to sign and
notorize
On Sat, Dec 28, 2024 at 12:39 AM Keith Zantow @.***>
wrote:

Hi @matthewmueller https://github.com/matthewmueller which quill
version is this?
—
Reply to this email directly, view it on GitHub
https://github.com/anchore/quill/issues/529#issuecomment-2563901785, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AABJSO7KCSEC4OBZ35G6SFL2HWGFRAVCNFSM6AAAAABUG5J5BCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNRTHEYDCNZYGU
.
You are receiving this because you were mentioned.Message ID:
@.***>


[FIX-BUG][Master]fix bug-6277 submit-time error in parallel tasks.

close #6277
submit-time error in parallel tasks.

Codecov Report

Merging #6278 (4d88466) into dev (69a153c) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##                dev    #6278      +/-   ##
============================================
- Coverage     38.72%   38.71%   -0.01%
+ Complexity     3213     3212       -1
============================================
  Files           637      637
  Lines         25321    25322       +1
  Branches       2752     2752
============================================
- Hits           9805     9804       -1
- Misses        14638    14639       +1
- Partials        878      879       +1

Impacted Files
Coverage Δ

...lphinscheduler/service/process/ProcessService.java
33.84% <0.00%> (-0.04%)
:arrow_down:

...er/master/dispatch/host/assign/RandomSelector.java
77.77% <0.00%> (-5.56%)
:arrow_down:

Continue to review full report at Codecov.

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


updateOnContentResize

I presume this parameter is for fluid layout that change in size.  When the browser is resized there is all this empty space at the bottom of the div(the div with the scrollbar plugin). it's not working for me on my end in chrome or ie haven't tested it in firefox but there is alot of bugs in this plugin man.

I'm in trouble with the Horizontal Scrolling.... on IE the updateOnContentResize don't update, and make my scroll............................................................................................ a lot of width.
$('#mySelector').mCustomScrollbar({
                axis: "x",
                theme: "light-3",
                advanced: {
                    autoExpandHorizontalScroll: true,
                },
                scrollbarPosition: "outside"
});

I'm not proud, but I fix it putting 1px of width, after scroll initialise.
$('#selector').css('width', $('#selector').width() + 1);


STAR fails to create index

This might not be directly related to HTSinfer itself, but when running HTSinfer on my PC with the samples SRR19067720 and SRR12414066, the process terminates and I get the error message 'Failed to create STAR index'. Both samples are from the same organism Pongo abelii (even though the size of the samples are very different, 6.5 GB vs 166 MB), which I suppose is not a frequent model organism, but it also happened with a human sample too, when I used the --records set to default.
It's probably a memory issue, because the termination happens after throwing an instance of std::bad_alloc; also, it doesn't seem to happen when running HTSinfer on the cluster.
I'm just brining this up because it could potentially arise when other end-users run it on their own setup (and not on a cluster).
Maybe we could just add a minimum memory recommendation in the readme?

I'm a bit confused. Aren't we building the same index, regardless of the source organism and the sample? 🤔

There's actually three samples with the same organism, SRR10393299, SRR12414066 and SRR19067720. When I run them on my PC, there's the 'Failed to create STAR index' error, so I ran them on the cluster, and this time the index is created and the error is at the alignment step:
[2023-04-04 17:23:19 DEBUG] Processing SAM file: '/scratch/balajt0000/slurm-job.25928198/tmp_htsinfer/tmp_FZ031/alignments/file_1/Aligned.out.sam'
[W::sam_hdr_create] Duplicated sequence "RPS2|YGL123W|YGL123W_mRNA|scerevisiae|4932" in file "/scratch/balajt0000/slurm-job.25928198/tmp_htsinfer/tmp_FZ031/alignments/file_1/Aligned.out.sam"
[W::sam_hdr_create] Duplicated sequence "RPS3|YNL178W|YNL178W_mRNA|scerevisiae|4932" in file "/scratch/balajt0000/slurm-job.25928198/tmp_htsinfer/tmp_FZ031/alignments/file_1/Aligned.out.sam"
[W::sam_hdr_create] Duplicated sequence "RPS5|YJR123W|YJR123W_mRNA|scerevisiae|4932" in file "/scratch/balajt0000/slurm-job.25928198/tmp_htsinfer/tmp_FZ031/alignments/file_1/Aligned.out.sam"
[E::sam_hrecs_update_hashes] Duplicate entry "RPS2|YGL123W|YGL123W_mRNA|scerevisiae|4932" in sam header
[E::sam_parse1] failed to parse header
[W::sam_read1_sam] Parse error at line 10463
[2023-04-04 17:23:19 ERROR] FileProblem: Failed to open SAM file: '/scratch/balajt0000/slurm-job.25928198/tmp_htsinfer/tmp_FZ031/alignments/file_1/Aligned.out.sam'

I checked and the organism pabelii is in the data/transcript.fasta.gz so the organism is not the issue.

I'm a bit confused. Aren't we building the same index, regardless of the source organism and the sample? 🤔

As it turns out, there is actually a filtering step based on the inferred organism, so the size of reference and SA pre-indexing string size for STAR changes every run. If no organism is getting inferred, STAR uses Size of reference: 9290472 and SA pre-indexing string size: 10 for the indexing step, and that's what causes the memory issue, as the number of references is too large. In this case, --genomeChrBinNbits should be reduced to reduce RAM consumption. The scaling for this can be calculated as min(18,log2[max(GenomeLength/NumberOfReferences,ReadLength)]).
I'll add this to the create_star_index function in htsinfer/get_read_orientation.py.

So just to summarize:

Added --genomeChrBinNbits to get_read_orientation.py to circumvent the memory issue which arises when no organism filtering happens, as well as optimizing the memory usage of STAR indexing.
As it turns out, running STAR with --genomeChrBinNbits set to different values, it also generates slightly different genome indices, especially the generated index/Genome, the genome reference sequences fasta files. I'm not yet entirely sure why this happens, but it results in different and fewer alignments in the next step, which caused the test files paired_isf, paired_isr and paired_iu to not get their orientation inferred.
By changing the default of --read-orientation-min-mapped-reads to 18, which sets the minimum number of mapped reads for deeming the read orientation result to be reliable, we can avoid this issue, as there's 18 mapped reads in the test samples with the calculated --genomeChrBinNbits setting.

I know this is not a perfect solution, but I think when running it on normal samples, 18 vs 20 min mapped reads don't really make a difference, but I'll run it again on our test samples of 770, to see if the inferred orientation changes.


Storage CLI page could use some example of credential being used

When executing an example for listing the contents of an existing blob container,
az storage blob list \
    --container-name mystoragecontainer \
    --output table

the CLI returns an error explaining how to credential the access:
> Missing credentials to access storage service. The following variations are accepted:
(1) account name and key (--account-name and --account-key options or
set AZURE_STORAGE_ACCOUNT and AZURE_STORAGE_KEY environment variables)
(2) account name and SAS token (--sas-token option used with either the --account-name
option or AZURE_STORAGE_ACCOUNT environment variable)
(3) account name (--account-name option or AZURE_STORAGE_ACCOUNT environment variable;
this will make calls to query for a storage account key using login credentials)
(4) connection string (--connection-string option or
set AZURE_STORAGE_CONNECTION_STRING environment variable); some shells will require
quoting to preserve literal character interpretation.
It would be nice if this page could discuss an example of selecting the storage account and credentialing it, using one of these options.

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

ID: 4482b415-9411-f74c-1721-e4a9eda472b3
Version Independent ID: b9f5613e-bafe-b0ab-ae84-70b4ed789b3b
Content: Azure Quickstart - Create a blob in object storage using Azure CLI
Content Source: articles/storage/blobs/storage-quickstart-blobs-cli.md
Service: storage
GitHub Login: @tamram
Microsoft Alias: tamram

Hi @andreasmarkmann - Thank you for your feedback! We will review and update as appropriate.

@andreasmarkmann Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

azcopy
--source /home/aasif/testFiles
--destination  https://az300filestorage1.blob.core.windows.net/customerupload001
--dest-key 6kEgNhV5zlZ+VbPoa0Ze4hdUp8dAFOL8aN6wtRthYvlfk6W3L7mjo2y7FrUnlYoed/6P97Ze+ejmudXKZdu8sw==
--include "helloWorld.txt"
Tried above command from the Cloud shell but its not working. Following error is generated.
Use "azcopy [command] --help" for more information about a command.
unknown flag: --source

@andreasmarkmann - I have updated the article to describe how to authorize with Azure AD credentials. I have also updated the examples to use the --auth-mode parameter to specify that Azure CLI should use the user's Azure AD credentials to call the operation.
You can also use the --auth-mode parameter to specify that the CLI uses the account access key instead.
#please-close

@tamram Hi this is not working for RBAC Credentials, like mentioned here: https://github.com/Azure/cli#configure-azure-credentials-as-github-secret
Should I open another issue or do you reopen this?

@DanielHabenicht - Could you clarify for me exactly what is not working? Thanks!

Yep,
I am trying to create a Storage and upload files from it for every PR.

I create a Service Principal with this guide: https://github.com/Azure/cli#configure-azure-credentials-as-github-secret
I create a storage account with: az group deployment create --resource-group covapp --template-file .github/workflows/az-storage-template.json --parameters storageAccounts_name=${{ env.STORAGE_NAME }} --subscription ${{ SECRETS.AZURE_SUBSCRIPTION_ID }} (template)
I am trying to update the storage with az storage blob service-properties update --account-name ${{ env.STORAGE_NAME }} --static-website  --index-document index.html --subscription ${{ SECRETS.AZURE_SUBSCRIPTION_ID }} but that fails with:

ERROR:
Missing credentials to access storage service. The following variations are accepted:
    (1) account name and key (--account-name and --account-key options or
        set AZURE_STORAGE_ACCOUNT and AZURE_STORAGE_KEY environment variables)
    (2) account name and SAS token (--sas-token option used with either the --account-name
        option or AZURE_STORAGE_ACCOUNT environment variable)
    (3) account name (--account-name option or AZURE_STORAGE_ACCOUNT environment variable;
        this will make calls to query for a storage account key using login credentials)
    (4) connection string (--connection-string option or
        set AZURE_STORAGE_CONNECTION_STRING environment variable); some shells will require
        quoting to preserve literal character interpretation.

You can see the build here: https://github.com/CovOpen/CovQuestions/pull/12/checks


Undefined $factory notice fix

When a plugin manager uses a static class method as a factory (i.e. FactoryClass::factoryMethod), as of release 2.7.10, it causes an undefined variable notice to be thrown due to an uncovered else case. This pull request provides a test case to demonstrate the problem as well as a fix. This problem does not appear to exist in the 3.x code, so it is presumably a side effect of backporting to 2.7. However, for projects depending on earlier versions of the service manager, it can be a significant problem and should be corrected -- that is why I am targeting this PR against the release-2.7 branch.

Thanks @demiankatz! Released in 2.7.11 :-)


Is anyone else using this project?

We have a rather big code base depending on this project. Is there somebody else using it, and interested in trying to maintaining it (for example to implement nullsafety)?

Since this project has been archived i'm not sure it will be updated :disappointed:  (but i'm not on the dart team so it's more a guess maybe better to wait for response from them)
What you can do is migrating to shelf_router_generator the api is somewhat the same and with some middleware you should be able to almost replicate what is done in this package.

I'm extracting some functionality out in an open source the code is here as of now https://github.com/Kleak/shelf_helpers

Nice. Will this also generate the client libraries?

short answer: Not with this package it's just some middleware put together to help with CORS, serialize/deserialize for now.
Here are some way of doing it now :
You can look at openapi from this yaml you can automatically generate client lib.
The last part would be to generated all the boilerplate backend code from this yaml file.
I don't know any package that can do that as of now :disappointed:


Documentation updates.

Fix warnings.
Add documentation for some TSMimeHdrField functions.
Add notes on HSTS example in header_rewrite.

I would approve this if it were 3 or 4 commits.

Allright, let me set that up.

OK, split in to 4 commits :+1:

[approve ci autest]

@SolidWallOfCode Thanks. Can you also update this PR's title?

Done.


ng new [PROJECT NAME], eslint is not added by default. (Angular CLI: 12.2.4)

Which @angular/* package(s) are relevant/releated to the feature request?
No response
Description
Angular CLI: 12.2.4
ng new [PROJECT NAME]
After this command is executed, eslint is not added by default
Proposed solution
Add eslint by default
Alternatives considered
eslint

Closing as this is indeed by design

Duplicate of https://github.com/angular/angular-cli/issues/20769


Update to latest jest version 23.6.0

Fixes: https://github.com/facebook/immutable-js/issues/1620

Any update on this?

Looks like you forgot to include the yarn lock file which is causing travis to fail, I'll try to merge manually

Merged in 9ea87744446cbbf17fb19ceb1aaa656eae5f4f93

Thanks for merging, but looks like the commit is not linked to my account.

Unfortunately that happens in a manual merge which I had to do because this PR wasn’t passing tests

This is a known issue and have been discussed before (example https://github.com/python/miss-islington/issues/16)
I've created an issue at #1633 to discuss how it can be fixed.


ConstructionStage type as string instead of integer

Or both!
You may use the attribute [Parseable("1", "from_this_stage_on", "From this stage on", etc)] then in grasshopper component use the EnumParser class.


Bump minio to v8.0.0 to enable use of Node v20

minio v7.1.3 only works with Node up to v18. Using the latest v8.0.0 enables building grist-core with Node v20.

Hi!
Thanks for your contribution, looking forward to seeing this PR ready! :)

minio v7.1.3 only works with Node up to v18

Are you sure? I know previous releases of Grist included minio-js in version 7.0.X which was not compatible with node 20, but now with minio v7.1.3,  I am able to build grist using node in version 20.13.1. And the package.json file of the library seems to allow that.
What error do you get?

@fflorent Thanks for pointing it out - I seem to be wrong. I was working on Grist Desktop (formerly grist-electron) and I vaguely remember that yarn install would fail because of the minio dependency when using Node v20. I double checked and could not reproduce the error now, so minio v7.1.3 is good.
But anyway bumping the dependency should be helpful to keep us up to date. I'll still try to finalize this PR.

(I looked through the code review checklist and found no issues)


fix(TFD-4946): only read the first batch of documents while fetching a sample from Elasticsearch

What is the current behavior? (You can also link to an open issue here)
See https://jira.talendforge.org/browse/TFD-4902 and https://jira.talendforge.org/browse/TFD-4946.
Fetching a sample from Elasticsearch will cause TCOMP to read all documents from the index and to keep them all in memory to build the initial PCollection. The sample is then extracted from this potentially giant PCollection.
When the index is big enough not to fit into memory, fetching a sample will cause the TCOMP instance to crash.
What is the new behavior?
Unfortunately, what is described on https://www.elastic.co/guide/en/elasticsearch/reference/2.4/query-dsl-limit-query.html does not seem to apply even on Elasticsearch 2.4. It seems not to be only deprecated but rather no longer supported.
The only reliable way to limit the number of documents read from each shard is to specify the terminate_after parameter on the GET /_search Elasticsearch endpoint.
As far as I can see, Apache Beam 2.3 does not expose anything on the ElasticsearchIO allowing to provide such parameters on the search request.
This PR will only work by compiling my own fork of Apache Beam 2.3 from this commit:
https://github.com/pradasouvanlasy/beam/commit/efbb47d70cfb7d23c1a9d03064d341248ef1aa51
I created this PR mainly for discussion purposes since it implies code that I don't own.
Please check if the PR fulfills these requirements

[ ] The commit message follows Talend standard
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features) ?
[ ] The code coverage on new code >75%
[ ] The new code does not introduce new technical issues (sonar / eslint)

What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build / CI related changes
[ ] Other... Please describe:

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:

Pull request test result: failure :x:

Waiting to merge to avoid impact on the Beam 2.5 story -- we'll see how to address this after (either patch Beam 2.5.0 as well, or try the alternative technique proposed by the apache team.)

Can be closed after PR#1346 which applies the same technique to Beam 2.5.0-tlnd


Fixed interfaces icons on Windows and MacOS

This PR fixes the issues with icons on Windows and MacOS.
According with the documentation

Note: By default, only X11 will support themed icons. In order to use themed icons on Mac and Windows, you will have to bundle a compliant theme in one of your themeSearchPaths() and set the appropriate themeName().

Related with this PR https://github.com/ros-visualization/rqt_publisher/pull/18
Signed-off-by: ahcorde<EMAIL_ADDRESS>

Closing this PR in favour of this other PR https://github.com/ros2/ros2/pull/928


i18n(de): update guides/deploy/flightcontrol.mdx

Description (required)

update guides/deploy/flightcontrol.mdx

Related issues & labels (optional)

#2912
#5845
#8167
#9240
Suggested label: i18n

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File
Note

de/guides/deploy/flightcontrol.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


ci(semantic-versioning): introduce semantic version workflow

name: Pull Request Template
about: 'template for PRs that ensures basic information is provided'
title: '(feat|fix|chore|doc): description of introduced change'
Description
Thanks for opening this contribution!
What does this PR introduce? Does it fix a bug? Does it add a new feature? Is it enhancing documentation?
Fixes # (issuenumber from PR destination repo)
Pre-review checks
Please ensure to do as many of the following checks as possible, before asking for committer review:

[ ] DEPENDENCIES are up to date. Dash license tool. Committers can open IP issues for restricted libs.
[ ] Copyright and license header are present on all affected files

Additional information

To request a review, check out who's involved to see a list of contributors and committers
Use the mailing list if you are unsure, who to contact, or if want to engage in a general discussion
Check out our guide on how to contribute
Check out our Release Guidelines
Check out the Eclipse Foundation contribution guidelines

@SebastianBezold We will add some additions here in the future for automatism, but for now this is fine.

Hi @DominikPinsel,
I'll merge it, but please add the copyright header to the workflow file in an upcoming PR


Multiple deposits

Users can do multiple deposits.
That mean 2times 10 items

Yea this is not a issue lol.

@TheAnthonyNL  is it possible to stop this?

Implement a feature to count items deposit from currentPot and if it exceeds 10 decline all trades from that user ... but this is useless anyway (in my opinion).


Feature - disables Azure Logic Apps from configuration file

Creates new PowerShell module Arcus.Scripting.LogicApps that includes (for now) a single to disable Azure Logic Apps based on a provided configuration file that controls how and in what order these Logic Apps should be scheduled for disabling.
Includes integration tests with real configuration files and a (preview) feature docs page to describe the functionality.
Closes #20

All done now? @mbraekman


JsonRpcBatch request/response and composer

Status
Type
⚠ Core Change
Issue

In Progress
Feature
Yes (enhancement)
#213

Background
"Requests can be sent in batches by sending an array of JSON-RPC request objects as the data for a single POST."
From: https://docs.solana.com/developing/clients/jsonrpc-api
Solution
This is a working implementation that moves us towards solving #213
Created JsonRpcBatchRequest, JsonRpcBatchResponse and JsonRpcBatchResponseItem for serialization and deserialization of batches of requests and responses in an abstract way.
Created SolanaRpcBatchComposer to create the batch of requests and coordinate the concrete data type of the requests and responses.
TokenWallet is a good first candidate for migration onto batched requests as we can combine the GetBalance and GetTokenAccountsByOwner requests into a single request.
The performance improvement on large request batches looks promising.
Example usage:
// compose a new batch of requests
var batch = new SolanaRpcBatchComposer();
batch.GetBalance("9we6kjtbcZ2vy3GSLLsZTEhbAqXPTRvEyoxa8wxSqKp5");
batch.GetTokenAccountsByOwner("9we6kjtbcZ2vy3GSLLsZTEhbAqXPTRvEyoxa8wxSqKp5", null, TokenProgram.ProgramIdKey);
batch.GetConfirmedSignaturesForAddress2("9we6kjtbcZ2vy3GSLLsZTEhbAqXPTRvEyoxa8wxSqKp5", 200, null, null);
batch.GetConfirmedSignaturesForAddress2("88ocFjrLgHEMQRMwozC7NnDBQUsq2UoQaqREFZoDEex", 200, null, null);
batch.GetConfirmedSignaturesForAddress2("4NSREK36nAr32vooa3L9z8tu6JWj5rY3k4KnsqTgynvm", 200, null, null);

// execute batch
var response = batch.Execute(client);

Next Up
The above sample shows the handling of the response in batch after the execute method.
There are opportunities to design more elogant ways to compose a batch combining callbacks or using the async/task pattern for each method, e.g.
// supply a callback
batch.GetConfirmedSignaturesForAddress2("9we6kjtbcZ2vy3GSLLsZTEhbAqXPTRvEyoxa8wxSqKp5", 200, null, null,
      x => {
          // use x
      });

// async/task model
var result = batch.GetConfirmedSignaturesForAddress2Async("9we6kjtbcZ2vy3GSLLsZTEhbAqXPTRvEyoxa8wxSqKp5", 200, null, null);

The crudely named RpcBatchReqRespItem (working title) which handles the association of each request and its return type could be extended to handle callbacks or tasks.
IMO there are at least a couple of different directions we could go in to best cater for different skill levels of the developers interacting with the library. It might be better to offer more complex callback/async wrappers on top of a simple batch handling kernal such as this composer.
As of now, the SolanaRpcBatchComposer supports a small PoC subset of the full RPC client methods, once the approach is proven, the remaining methods can be added.

Added callbacks for batch methods. e.g.
// compose a new batch of requests
var batch = new SolanaRpcBatchComposer();
batch.GetBalance("9we6kjtbcZ2vy3GSLLsZTEhbAqXPTRvEyoxa8wxSqKp5",
    callback: x => found_lamports = x.Value);
batch.GetTokenAccountsByOwner("9we6kjtbcZ2vy3GSLLsZTEhbAqXPTRvEyoxa8wxSqKp5", null, TokenProgram.ProgramIdKey,
    callback: x => found_balance = x.Value[0].Account.Data.Parsed.Info.TokenAmount.AmountDecimal);
batch.GetConfirmedSignaturesForAddress2("9we6kjtbcZ2vy3GSLLsZTEhbAqXPTRvEyoxa8wxSqKp5", 200,
    callback: x => sig_count += x.Count);
batch.GetConfirmedSignaturesForAddress2("88ocFjrLgHEMQRMwozC7NnDBQUsq2UoQaqREFZoDEex", 200,
    callback: x => sig_count += x.Count);
batch.GetConfirmedSignaturesForAddress2("4NSREK36nAr32vooa3L9z8tu6JWj5rY3k4KnsqTgynvm", 200,
    callback: x => sig_count += x.Count);

There are many different execution and concurrency options to be considered as well as callback semantics using tasks/asyncs in creating a general multi-purpose library. This callback arrangement was the simplest solution to achieve my goals working with the batch composer.
For simplicity and thread-safety, callbacks are invoked sequentially by SolanaRpcBatchComposer.ProcessBatchResponse method. This is decoupled and invocable separately from the Execute method for testability.

I have to say I was surprised at how quickly you came up with a solution to this. I thought we'd have to change the client and probably end up with breaking changes.
Regarding the two possible solutions, I think the callback one is cleaner and less error prone. There is no chance for the user to invoke the Result before the Execute call. As for exceptions, it's a matter of intercepting them and adding the relevant info to the relevant RequestResult.
Again, thank you and good job :)

Looking to bring this work into the main branch soon.
Been working on usability of this and just about to commit some auto-execute functionality to simply the processing of workloads.
Found that when using the code in workloops there is a common usage pattern emerging to need to check batch sizes and execute when batch size is full and then to check for partial batches after the workloop.
Came up with this tidier call pattern instead:
        // setup batch to auto-execute batches of 200 requests
        var batch = new SolanaRpcBatchWithCallbacks();
        batch.AutoExecute(BatchAutoExecuteMode.ExecuteWithFatalFailure, client, 200);

        // fetch all transactions
        var txs = new List<TransactionMetaSlotInfo>();
        foreach (var sig in sigs)
            batch.GetTransaction(sig, callback: (tx, ex) => txs.Add(tx));
        batch.Flush();

When using Blazor wasm while executing a batch: Cannot wait on monitors on this runtime. Changing Execute and ExecuteWithFatalFailure to use await rather than .Result for SendBatchRequestAsync fixes it.

What's the status of this PR? Is it ready for review?

Howdy - totally missed that comment, sorry about that.
Not heard of Blazor wasm before...
That's good feedback from @avuorine, I'll see how I can rework this to fit with await rather than .Result and reach out and ask them to check - that's another dev I know of using the batch stuff 🙂
I'm pretty happy with how it works for what I've been using it for, also pleased with the simplied calling syntax since the introduction of auto-exexcute.

Paging @avuorine ...
Please could you take a look at the following update and report back on whether this is works for your Blazor wasm use case?
I've added ExecuteAsync and ExecuteWithFatalFailureAsync await friendly versions of the Execute methods on the SolanaRpcBatchComposer that avoid using .Result - hopefully you should be able to switch to these and you're good to go. The previous .Result syncronous methods remain but reuse the new async methods.
Would also welcome your views on the different styles of using batches - did you prefer SolanaRpcBatchWithAsyncs or SolanaRpcBatchWithCallbacks style? Have you tried the auto-execute mode?

Paging @avuorine ...
Please could you take a look at the following update and report back on whether this is works for your Blazor wasm use case?
I've added ExecuteAsync and ExecuteWithFatalFailureAsync await friendly versions of the Execute methods on the SolanaRpcBatchComposer that avoid using .Result - hopefully you should be able to switch to these and you're good to go. The previous .Result syncronous methods remain but reuse the new async methods.

I tested the Async methods and they work on Blazor wasm.

Would also welcome your views on the different styles of using batches - did you prefer SolanaRpcBatchWithAsyncs or SolanaRpcBatchWithCallbacks style? Have you tried the auto-execute mode?

I tried both but ended up going with SolanaRpcBatchWithCallbacks. I've been using AutoExecute so would be helpful to have an Async version for that as well.
Thank you!

Thanks for the reply.
I've found I prefer the callback pattern over the asyncs too.
Thinking of holding that one back in the finished branch but did not want to leave you abandoned.
There is a deliberate design simplification to make the batch flush execution block the calling thread due to the internal storage of the batch composer and reusing that for the next batch. In typing this through with here, I think there would be performance benefits to be had in taking a non-blocking approach and overlapping the calling code (adding to the batch) and the execution of batches (flushing the queued requests, executing and invoking the callbacks). My concern is could be quite possible for a caller loop to generate many batches very quickly that are all executing concurrently (e.g. batches of many slow getProgramAccounts)
I'm wondering whether it could be solved with another auto-execute option, something like a concurrent-batch-limit that would guard against a runaway scenerario like describe above by blocking when the limit is reached.
Does the current blocking AutoExecute behavior function ok in your Blazor project?
Please could you share a little more about your use case so I could think through a good solution?
Open to suggestions of course :-)
Also - is there any chance that you share a trivial hello world Blazor web-asm project that includes some Solnet activity?
I've never used this before and it would be great to have a test framework we could use to evaluate whether the batch code works and will help us to ship things that work for your use case first time :-)

Thanks for the reply. I've found I prefer the callback pattern over the asyncs too. Thinking of holding that one back in the finished branch but did not want to leave you abandoned.
There is a deliberate design simplification to make the batch flush execution block the calling thread due to the internal storage of the batch composer and reusing that for the next batch. In typing this through with you here, I think there would be performance benefits to be had in taking a non-blocking approach and overlapping the calling code (adding to the batch) and the execution of batches (flushing the queued requests, executing and invoking the callbacks). My concern is quite possible for a caller loop to runaway generating many batches faster than they execute concurrently (e.g. batches of many slow getProgramAccounts)
I'm wondering whether it could be solved with another auto-execute parameter, something like a concurrent-batch-limit that would guard against a runaway scenerario like above by blocking when the limit is reached.
Does the current blocking AutoExecute behavior function ok as it is now in your Blazor project? Please could you share a little more about your use case so I could think through a good solution? Open to suggestions of course :-)

When using AutoExecute, Flush calls the non-async versions of Execute so it fails with the same error that I posted previously.  I changed it to use the async methods instead, and it seemed to work alright on the wasm runtime after some quick testing.

Also - is there any chance that you share a trivial hello world Blazor web-asm project that includes some Solnet activity? I've never used this before and it would be great to have a test framework we could use to evaluate whether the batch code works and will help us to ship things that work for your use case first time :-)

I'll try to put something together in a few days so it can be tested further.
Thanks.

@liquizard been looking at this at it looks fine. Just gonna nitpick about some minor stuff. Beyond this, are we leaving both mechanisms? It seems a bit of overkill for future code maintenance, so I think it would be smart to keep just one of them and nix the other.


login e2e tests are incorrect / dangerous

login e2e tests currently attempt to delete all projects. This might not work and is quite dangerous if the tests are run by an unsuspecting user on a cluster with elevated rights. Moreover, it might be difficult to ensure that the state is appropriate (i.e. no projects) when the tests are launched so end to end testing of the different login scenarios might be difficult/impossible.

@mohammedzee1000 Please look to this issue with priority

I am already working on alterering it in fix PR,  will push it out


Laravel 10.x Compatibility

This is an automated pull request from Shift to update your package code and dependencies to be compatible with Laravel 9.x.
Before merging, you need to:

Checkout the l10-compatibility branch
Review all comments for additional changes
Thoroughly test your package

If you do find an issue, please report it by commenting on this PR to help improve future automation.

:warning: Shift detected GitHub Actions which run jobs using a version matrix. Shift attempted to update your configuration for Laravel 10. However, you should review these changes to ensure the desired combination of versions are built for your package.

:alembic: Using this package? If you would like to help test these changes or believe them to be compatible, you may update your project to reference this branch.
To do so, temporarily add Shift's fork to the repositories property of your composer.json:
{
    "repositories": [
        {
            "type": "vcs",
            "url": "https://github.com/laravel-shift/laravel-paddle.git"
        }
    ]
}

Then update your dependency constraint to reference this branch:
{
    "require": {
        "protonemedia/laravel-paddle": "dev-l10-compatibility",
    }
}

Finally, run: composer update


Add Dragon Essence as Liquid

Register the Liquid on the FluidRegistry. Make some kind of similar effects of the Cauldron,

Meh, no. Most people don't want.


"User did not grant permissions" if I click "Yes" too quickly on Windows?

I'm running it in a slow Windows 10 VM and I'm getting "User did not grant permissions" if I click "Yes" too quickly. Anyone ever got that? How can I debug what the problem is?
If I wait 5-10 seconds it seems to work most of the time. But if I click right away it never works. I don't have access to a real Windows system to test it for now.

Thanks @dave-dm, I'm sorry for the delayed response.
One way to debug would be to add console log statements to sudo-prompt for each exec() method called. It may be that you are getting another error back from Windows, e.g. maybe an EPERM because of antivirus locking and scanning the file after it's just created, but then sudo-prompt returns the stock "User did not grant permissions" (it's actually pretty difficult for us to figure out whether the user authorized the prompt).
If it is actually an antivirus issue, you could try disable your antivirus while you test?
If you could record a quick screencast that would also help me track this down.
And please also make sure to use the included test.js file rather than your own project code so we have something simple.


jestPuppeteer.debug support in Jest-circus

💬 Questions and Help
I wonder if there any plans to support jest-circus. While jest-circus works kinda fine out of the box, I noticed that jestPuppeteer.debug() does not really work as expected.
I have this CustomEnvironemnt class that should stop the execution on test failure:
const PuppeteerEnvironment = require( 'jest-environment-puppeteer' );

class CustomEnvironemnt extends PuppeteerEnvironment {
	async handleTestEvent( event ) {
		if ( event.name === 'test_fn_failure' ) {
			await this.global.jestPuppeteer.debug();
		}
	}
}

It does trigger the chrome's debugger, but unfortunately, after resuming the debugger - browser just closes itself.

Hello @brbrr, yeah jestPuppeteer.debug() is kind of hacky and we should remove it. The only problem here is that we don't have any correct way to set the timeout of the fly to actually "pause" a test in Jest.

make sense. What would you recommend as an alternative? I wonder if we can use some sort of DEBUG env variable to setup jest timeouts, and run the debugger on test failure or on-demand (similar to jestPuppeteer.debug())

Hello yes maybe it would be a good solution but no very user-friendly.

Have you tried to add adding freak timeout as test argument?(I'm not using custom environment, just add jestPuppeteer.debug() to each test while test development)
It is the only way I can use debug
Just like this:

jest.retryTimes(3)

describe('Test', () => {
    test('Test_1', async () => {
        await jestPuppeteer.resetBrowser()
        await loginPage.login()
        await jestPuppeteer.debug() //paused in browser, and after resuming in browser stays paused in test runner
        await loginPageAssert.isLoggedIn()
    }, 999999) //timeout

})

This question is asked all over the place. The timeout https://github.com/smooth-code/jest-puppeteer/blob/master/packages/jest-environment-puppeteer/src/PuppeteerEnvironment.js#L42 is probably ignored due to the same reason as https://github.com/facebook/jest/issues/9359. You can use --testtimeoutnumber to test: https://jestjs.io/docs/en/cli#--testtimeoutnumber


CDATA stripped from fields

CDATA blocks are stripped on import. Export values are exposed without the CDATA encapsulation.

This is a cosmetic issue, not a bug.  The important thing is that content is preserved.  How escaping is done is an XML serialization detail, and not worth getting fancy and complicated to force one way or the other.  Closing as "wontfix".


Enable custom ordering of tasks on travis

Travis appears to schedule tasks in roughly the order they appear in the matrix - it would be nice if we could change the ordering of the things in the env section so that we can start the slower tasks first.
One way this could be done is by allowing users to specify an ordering by task name and/or package name?

This is effectively closed now, you can order stages and the tasks within those are ordered as well.


Switch to esbuild?

I got a basic build working with esbuild via:

npm i esbuild
Add the following build script as build.mjs

import esbuild from 'esbuild';
const { build } = esbuild;

build({
    entryPoints: ['lib/index.ts'],
    outdir: "dist",
    sourcemap: "linked",
}).catch(() => process.exit(1));

run node build.mjs

This is advantageous because esbuild has 30x faster build times:
$ time npm run build > /dev/null

real	0m1.854s
user	0m2.890s
sys	    0m0.111s

$ time node build.mjs

real	0m0.048s
user	0m0.032s
sys	    0m0.016s

Sure. Never heard of esbuild before, and I wasn't bothered by a few seconds to build, but webpack clearly has issues in doing this in pyodide-worker-runner may be interesting. Just check that dist is the same before and after, and make a PR.

I tried following these steps in pyodide-worker-runner. Issues I noticed:

No .d.ts file
No minification
Need to handle uses of webpack raw-loader and worker-loader.

I'm sure these are all fixable, just pointing them out.

Right the word "basic" is doing some work here, unfortunately I mean with some loss of functionality. Hopefully it's possible to get the functionality you need back but I think esbuild is not a fully general replacement for webpack.


Event Grid - Having PLE enabled Azure Functions as Subscriber

Hi experts,
PLE enabled Azure Functions is not able to get the events from Event Grid - Can you please assist.
If PLE is not enabled on Azure Functions it works fine.
Thanks!
Chaisen
[Enter feedback here]

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

ID: 2c2e9bf7-f822-cfcf-0abf-ce7362a08a07
Version Independent ID: 9b09e66c-ba2a-520b-3ab9-4f39a862b2ff
Content: Quickstart: Send custom events to Azure Function - Event Grid - Azure Event Grid
Content Source: articles/event-grid/custom-event-to-function.md
Service: event-grid
GitHub Login: @spelluru
Microsoft Alias: spelluru

@chaisen2006 You cannot use private endpoint for egress traffic i.e. from event grid to the endpoint as the Network resource provider feature, was not designed for this scenario. That said, there are alternative options you could use: Deliver events using private link service - Azure Event Grid.
Please let me know if you have any additional questions.

@chaisen2006 Just following up here. Hope my previous comment clears things up.

@chaisen2006 Hope the above helps and clears things up. We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Support for koa2 / Untraced mysql query

I've noticed that this library supports koa (btw, for some reason readme didn't mention it). My app is running on koa v2, so, i've patched this library a little: https://github.com/1602/cloud-trace-nodejs/commit/b809244b72c64a63c8a13587cff1dc180d3b2d8a
Not sure if I could submit PR adding koa v2 support.
And another question, possibly unrelated one. In app i'm tracing I use mysql and knex odm, and I see no traces of db calls, connected to root span registered by koa middleware, in debug log i see
DEBUG:@google/cloud-trace: Untraced mysql query: { sql: ...

Any suggestions on how to make it working would be very welcome. Thanks!

The readme does not mention koa because it is not a framework we officially support and keep up to date. We welcome pull requests and are happy to accept changes to support koa v2. I see that your koa patch uses async functions which I assume requires transpilation. We do not currently have a transpilation workflow for this project so adding tests for the koa v2 patch may require some more machinery.
The problems with mysql tracing are likely caused by interference from the generic-pool module which is used by knex instead of the builtin connection pooling in the mysql module. If this is the case, we will need to instrument the generic-pool module to make sure mysql requests can be attached to the appropriate koa spans. We can take a look at knex and verify that this is actually the issue you are observing.

re: koa2 support, thanks it is clear, I will rewrite to pure promises and submit PR
re: mysql, sounds like this is it, i can not confirm, because it seems like it is not possible to not use generic pool in knex. it would be very nice if you help to explore generic-pool issue, i'm totally lost in cls stuff.

I've opened #327 to track the enhancement request for preserving context across knex. Closing this issue.


Mouse cursor jumping in certain games.

Hi, I have an issue where the mouse cursor is jumping in certain games when moving it to the left. GPU is an Nvidia GTX 1060 with the proprietary driver.
Games I noticed it in and how the jump is happening exactly:

War Thunder, jumping while playing nomally, causing the camera to randomly rotate or flick to the right, basically unplayable.
Deep Rock Galactic, jumping while tabbed out, basically unable to use the mouse on other applications while the game is running.
Minecraft, jumping when opening menus or the inventory, causing the mouse to be in the bottom-right quarter of the screen initially.

JWM Config: https://pastebin.com/raw/rYECuDJi

Turns out it was a problem with my mouse settings for some reason.


Added 10.12 support

kinda, and changed the clear color to a better shade of blue, objectively.

best pull request NA


Error in flattening logic for nested model names

Environment

OS Version: Ubuntu Bionic
Binaries: SDFormat 9.5.1 , Gazebo 11.5.1

Description
In the test case provided in https://github.com/j-rivero/sdf_bug, Gazebo11 is complaining with:
[Err] [Joint.cc:297] EXCEPTION: Couldn't Find Child Link[sub_door::door]

when using an SDF similar to:
<?xml version="1.0" ?>
<sdf version="1.5">
  <model name='hinged_door_ticket'>
      <static>0</static>
      <link name='frame'>
      </link>
      <model name='sub_door'>
	<static>0</static>
	<link name='door'>
	  <pose frame=''>-0 0 2.04 3.14159 -0 1.58</pose>
	</link>
      </model>
      <joint name='hinge' type='revolute'>
        <parent>frame</parent>
        <child>sub_door::door</child>
      </joint>
    </model>
</sdf>

The great @azeey first impression was: "It looks like the breaking change is that we added a prefix to nested models when flattening (https://github.com/osrf/sdformat/pull/399). Running gz sdf -p frame.world  shows that the joint is looking for sub_door::door. In Gazebo7, there is a model called sub_door. In Gazebo11, that model is now hinged_door_frame::sub_door. For some reason, the flattening logic is not handling that properly

Expected behavior: the name got resolved correctly
Actual behavior: failing to resolve the name, error message is emitted

Steps to reproduce

Download the example repository in the description
Execute GAZEBO_MODEL_PATH=${GAZEBO_MODEL_PATH}:${PWD} gazebo --verbose frame.world

Can this be closed now that #597 is merged?


WIP: stats: attempt to consolidate warnings and errors

Reference issue
Closes gh-13462
What does this implement/fix?
This attempts to resolve gh-13462, which reported that there are too many warnings in the scipy.stats public API, and that we need a better way to organize them.
Things got messy because I tried to put all the warnings and errors into the _result_classes file and ended up with circular imports that I found difficult to resolve. I think the better way to go for now is to move warnings and errors into their own file.
Let's see if I got the docs to build correctly.

@tupui any idea how I'm getting this error from this PR? Can't even begin to build the docs.
make: Entering directory '/home/circleci/repo/doc'
mkdir -p build/html build/doctrees
LANG=C "/home/circleci/.pyenv/versions/3.8.13/bin/python" -msphinx -WT --keep-going -t scipyorg  -b html -d build/doctrees "-j2" source build/html-scipyorg
Running Sphinx v4.5.0

Traceback (most recent call last):
  File "/home/circleci/.pyenv/versions/3.8.13/lib/python3.8/site-packages/sphinx/config.py", line 331, in eval_config_file
    code = compile(f.read(), filename.encode(fs_encoding), 'exec')
  File "/home/circleci/repo/doc/source/conf.py", line 194
    "version_match": version,
                   ^
SyntaxError: invalid syntax

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/circleci/.pyenv/versions/3.8.13/lib/python3.8/site-packages/sphinx/cmd/build.py", line 272, in build_main
    app = Sphinx(args.sourcedir, args.confdir, args.outputdir,
  File "/home/circleci/.pyenv/versions/3.8.13/lib/python3.8/site-packages/sphinx/application.py", line 202, in __init__
    self.config = Config.read(self.confdir, confoverrides or {}, self.tags)
  File "/home/circleci/.pyenv/versions/3.8.13/lib/python3.8/site-packages/sphinx/config.py", line 165, in read
    namespace = eval_config_file(filename, tags)
  File "/home/circleci/.pyenv/versions/3.8.13/lib/python3.8/site-packages/sphinx/config.py", line 335, in eval_config_file
    raise ConfigError(msg % err) from err
sphinx.errors.ConfigError: There is a syntax error in your configuration file: invalid syntax (conf.py, line 194)

Configuration error:
There is a syntax error in your configuration file: invalid syntax (conf.py, line 194)

make: *** [Makefile:118: html-scipyorg] Error 2
make: Leaving directory '/home/circleci/repo/doc'
Traceback (most recent call last):
  File "dev.py", line 650, in <module>
    main(argv=sys.argv[1:])
  File "dev.py", line 264, in main
    subprocess.run(cmd, check=True)
  File "/home/circleci/.pyenv/versions/3.8.13/lib/python3.8/subprocess.py", line 516, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['make', '-Cdoc', 'PYTHON="/home/circleci/.pyenv/versions/3.8.13/bin/python"', 'html-scipyorg', 'SPHINXOPTS="-j2"']' returned non-zero exit status 2.

Exited with code exit status 1
CircleCI received exit code 1

@tupui any idea how I'm getting this error from this PR? Can't even begin to build the docs.

Ark yes sorry I missed a comma in my last edit to be pushed! Will send a PR now.

@mdhaber should be fixed now

I'm going around in circles here. I don't think I want to work on this "for fun".

@rgommers Is this on the right track to address the warnings and errors part of gh-13462? If so, I'll send a message to the mailing list about the three public warning classes (DegenerateDataWarning, ConstantInputWarning, NearConstantInputWarningFitError) and one public error class (FitError).

This seems quite reasonable to me based on a quick look. Thanks @mdhaber

Message to mailing list sent 4/8; let's give it (at least) a week for feedback.

@rgommers there were no complaints from the mailing list. Would you be willing to review in enough detail to consider merging before 1.9.0 branches?

@rgommers I thought I'd check in again, since I don't think others would be very interested to review.


Fix MRTK asset menu layout / remove duplicate menu

This change consolidates duplicate asset menus and places all profile options into the profile menu.
Before:

After

/azp run mrtk_pr

Should the MixedRealityInputRecordingProfile go under profiles as well?

Should the MixedRealityInputRecordingProfile go under profiles as well?

I didnt notice that one... sure, i can add it

/azp run mrtk_pr


Fix loadBalancerConfig

Closes #186

:+1:

:+1:

Coverage remained the same at 11.21% when pulling c1b3ce802857226d4bf1c791781a9caa1180f265 on fix-loadbalancer-config into a8e1c00e65e253d15d0db078c558c14788606a7a on master.

:+1:


Bug fix PUT operation for products in a shopping cart

As a USER
I need to be able to issue a put request for products in a shopping cart without error
So that I can change products in my shopping cart
Assumptions:

Current implementation is faulty, returns error 500

Acceptance Criteria:
Given a shopping cart with products in it
When I issue PUT /shopcarts/sid/products/sku with a properly formatted payload
Then the product should update with a HTTP 200 response code

@WesPainter Shouldn't update success return HTTP 200 not 201?

You're right, fixed


File upload

Description
I want to upload a file to the connexion server running with aiohttp. I am experiencing exactly the same problem as described in https://github.com/zalando/connexion/issues/975
Uploading a file seems to me as a very basic functionality, so I am surprised that this still fails. Is there any workaround or other way to upload a file?
Just to replicate my very special case:
 /upload:
    put:
      description: "Upload a dataset."
      operationId: main.upload_dataset
      requestBody:
        content:
          multipart/form-data:
            schema:
              type: object
              properties:
                file:
                  type: string
                  format: binary

I have tried to send files via python requests and via postman:

Python request:

import requests
url = "http://<IP_ADDRESS>:5000/api/v1/upload"
files = {'file': open("dataset.jsonl", 'rb')}
resp = requests.request("PUT", url=url, headers=None, data=None, files=files)

Postman:
URL: localhost:5000/api/v1/upload
Body: KEY is the word "file" and VALUE is a reference to the file dataset.jsonl
Method: PUT

Expected behaviour
Connexion should process the file and provide it via
connexion.request.files['file'] to the FLASK function that is called.
Actual behaviour
Connexion fails before even executing the function code. In the module
connexion/decorators/validation.py, lines 159 to 162 are:
elif self.consumes[0] in FORM_CONTENT_TYPES:
    data = dict(request.form.items()) or (request.body if len(request.body) > 0 else {})
    data.update(dict.fromkeys(request.files, ''))  # validator expects string..
    logger.debug('%s validating schema...', request.url)

In debug mode, I can see that data is of type bytes because it took the value of request.body which means that if fails in the next step where it treats data as a dictionary.
Steps to reproduce
Create a minimal endpoint with this configuration:
 /upload:
    put:
      description: "Upload a dataset."
      operationId: main.upload_dataset
      requestBody:
        content:
          multipart/form-data:
            schema:
              type: object
              properties:
                file:
                  type: string
                  format: binary

Send any file to the endpoint, for example as I described above for python requests.
Additional info:
Output of the commands:

python --version
Python 3.7.5

pip show connexion | grep "^Version\:"
Version: 2.7.0

Yesterday, I had the exact same issue as #975 but with Axios and React. Connexion was returning:
data.update(dict.fromkeys(request.files, ''))  # validator expects string..
while testing file uploads.
In my case, the issue were the headers. I just followed the suggestion at https://stackoverflow.com/a/47630754/2489998 to correctly include the 'multipart/form-data' part and that fixed the issue.
Maybe this info can help you.

Closing as duplicate of #975.


Security: "poisoning" objcts with additional properties slips through ReflectedClass#matchesValue

matchesValue does not report, when objects contain properties which are not specified. Contrary the typescript compiler reports this as an error. I see this as security relevent in my case as my RPC framework restfuncs wants to use this checking for all input from the evil client.  So an attacker could sneak in some properties that will get stored in the database, get evaluated by third party libs etc.. Could you have a look at it ?
Reproduce it with the following jest code:
    class User {
        id : number;
    }

    const reflectedClass = reflect(User);

    expect(
        reflectedClass.matchesValue({id: 1, poisonedProperty: true})
    ).toBeFalsy()

Agreed. I think we may want to be able to do both kinds of checks. matchesValue has too many positional parameters, so I think we'll change it to an options object and add something like allowExtraProperties. That'll be a new minor version for compat. Not sure which mode should be the default yet.

Cool, thx !
I'd feature the secure version as default. Also that would be the expected behavior in a typescript sense.

It's not quite that simple in Typescript.
Nonetheless, we might go for allowExtraProperties=false as the default just to avoid the footgun.

Ah lol, you're right. I was confused about that, see that issue.

Also an (optional) automatic cleaning of extra properties would be very handy in my RPC case. Cause you never know what extra properties might get attached by other client frameworks and we don't want to bother the user with an error then.
So if you're implementing the checks anyway...

Oops this wasn't closed when 0.9.0 shipped. You must now opt in to allowing extra properties as discussed.


Use a real CHANGELOG?

It's nice that you're using GitHub's releases feature, but nothing beats actually including a CHANGLOG so that you can easily look over the changes from the terminal.

Do you feel that the level of detail noted in release notes is sufficient, or do you think that it should be more granular or in different format?

I think the level of detail in the release notes is totally cool :+1:
Once RxSwift is big enough it might warrant something more like CocoaPods/AFNetworkings

I've added changelog as of 1.9.1
I'm going to close this issue. Let me know if you feel that we need to modify it :)

I'm happy


os.startfile is windows only

When you try to run this on Linux, you get the error:
Default Audio Player for mp3 files is not set up, like vlc or something.
This happens because to open the audio file, Stephanie uses the module os.startfile which is Windows only in speaker.py
To work cross-platform, this needs to be changed to something else.

Duplicate of #1 but with more info

Main discussion on #1


feat: testing for repeat ranges

Issue
Closes #654.
Checklist

[ ] I have updated CHANGELOG.md
[ ] I have run all the tests locally and no test failure was reported
[x] I have run the linter, formatter and spellchecker
[x] I did not do unrelated and/or undiscussed refactorings

Stumble across the exit code 37 in the action phase (which is "not enough Toncoins") no matter how big I make the treasure to be. @Gusarich, could you please check?

Stumble across the exit code 37 in the action phase (which is "not enough Toncoins") no matter how big I make the treasure to be

@novusnota There is a maximal gas limit for processing one message,  it should 0.4 TON atm: https://github.com/tact-lang/tact-docs/pull/298#issuecomment-2207412040. I haven't checked if Sandbox respects this limit, but it should be configurable (although we actually shouldn't increase it)


[docs][traits] improve section on mpi

We should be more specific on what exactly ghost points mean?

Closed with #152


Fix code coverage

Codecov Report
Base: 67.48% // Head: 64.69% // Decreases project coverage by -2.80% :warning:

Coverage data is based on head (260468f) compared to base (86c1719).
Patch has no changes to coverable lines.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop      #42      +/-   ##
===========================================
- Coverage    67.48%   64.69%   -2.80%
===========================================
  Files           52       52
  Lines          895      895
  Branches         0       73      +73
===========================================
- Hits           604      579      -25
+ Misses         291      287       -4
- Partials         0       29      +29

Impacted Files
Coverage Δ

...lDieFromWorking/Extensions/AgentStateExtensions.cs
66.66% <0.00%> (-33.34%)
:arrow_down:

...ods.BugFixes/Patches/ArmorCreaturePatchOnNotice.cs
38.09% <0.00%> (-14.29%)
:arrow_down:

...fts/Patches/UseSkillPatchFinishWorkSuccessfully.cs
50.00% <0.00%> (-12.50%)
:arrow_down:

...s/Patches/CreatureEquipmentMakeInfoPatchGetProb.cs
50.00% <0.00%> (-9.10%)
:arrow_down:

...ions/CreatureEvaluators/HappyTeddyBearEvaluator.cs
90.90% <0.00%> (-9.10%)
:arrow_down:

...n/Patches/AgentInfoWindowPatchEnforcementWindow.cs
69.23% <0.00%> (-7.70%)
:arrow_down:

...BugFixes/Extensions/CustomizingWindowExtensions.cs
53.57% <0.00%> (-7.15%)
:arrow_down:

...llDieFromWorking/Extensions/AgentSlotExtensions.cs
75.00% <0.00%> (-6.25%)
:arrow_down:

...eFromWorking/Extensions/CommandWindowExtensions.cs
75.51% <0.00%> (-6.13%)
:arrow_down:

...omization/Patches/CustomizingWindowPatchConfirm.cs
64.70% <0.00%> (-5.89%)
:arrow_down:

... and 10 more

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

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


Fix TSC issue in typescript sample

External Contributions
This project is not yet set up to accept pull requests from external contributors.
If you have a pull request that you believe should be accepted, please contact
the Developer Relations team<EMAIL_ADDRESS>with details
and we'll evaluate if we can setup a CLA to allow for the contribution.
For Internal Contributors
Description Of Changes
Add a brief description of the changes that you have involved in this PR
Follow conventional commit format: https://www.conventionalcommits.org/en/v1.0.0/
<TICKET_ID>

@msell Thanks for the support, will try to add this it to UIKit
Can you share a description of the changes, please

Closing in favor of the linked PR


Plugin not working

I've followed the instructions correctly, logged in to both discord and the developer platform with the same account. I'm just getting a yellow caution triangle when I hit the volume mixer key.

Hello, please visit our discord server for support: https://discord.com/invite/294BQE6Xdp #danols-plugin-support


parser: remove dependency on tidb/util/hack package

What problem does this PR solve?

Fix https://github.com/pingcap/tidb/pull/9230
What is changed and how it works?
remove tidb/util/hack.String
Check List
Tests

No code

PTAL @jackysp @lysu


left-right flip

It seems like left and right are flipped in your 3D representation, e.g. the following code https://github.com/una-dinosauria/human-motion-prediction/blob/c9a2774fe59bb78c6ec6d04e15f2cb7243c4348c/src/viz.py#L25
seems to conflict with the human 3.6m matlab code
switch part
        case 'rootpos'
          joints = 1;
        case 'rootrot'
          joints = 1;
        case 'leftarm'
          joints = 18:24;% p/p2/a fine
        case 'rightarm'
          joints = 26:32;% p/p2/a fine
        case 'head'
          joints = 14:16;% p/p2/a fine
        case 'rightleg'
          joints = 2:6;% p/p2/a fine
        case 'leftleg'
          joints = 7:11;% p/p2/a fine
        case 'upperbody'
          joints = [14:32];% p/p2/a fine
        case 'arms'
          joints = [16:32];% p/p2/a fine
        case 'legs'
          joints = 1:11;% p/p2/a fine
        case 'body'
          joints = [1 2 3 4 7 8 9 13 14 15 16 18 19 20 26 27 28];% p/p2/a fine
        otherwise
          error('Unknown');
      end

.
The plot below shows the original 3D ground truth location ('Ground Truth') from human3.6m, the particular frame of the video and the 3D plot in expmap format, transformed to 3D joint coordinates using your forward kinematics function, provided with your repository ('ExpMaps').

Is there an intention behind the flip?

Hi @jutanke,
Nice catch. There seem to be two flips going on here:

I am just trying to find this figure today! So nice of you, it is very useful for my project.

It seems like left and right are flipped in your 3D representation, e.g. the following code
human-motion-prediction/src/viz.py
Line 25 in c9a2774
self.LR  = np.array([1,1,1,0,0,0,0, 0, 0, 0, 0, 0, 0, 1, 1, 1], dtype=bool)
seems to conflict with the human 3.6m matlab code
switch part
        case 'rootpos'
          joints = 1;
        case 'rootrot'
          joints = 1;
        case 'leftarm'
          joints = 18:24;% p/p2/a fine
        case 'rightarm'
          joints = 26:32;% p/p2/a fine
        case 'head'
          joints = 14:16;% p/p2/a fine
        case 'rightleg'
          joints = 2:6;% p/p2/a fine
        case 'leftleg'
          joints = 7:11;% p/p2/a fine
        case 'upperbody'
          joints = [14:32];% p/p2/a fine
        case 'arms'
          joints = [16:32];% p/p2/a fine
        case 'legs'
          joints = 1:11;% p/p2/a fine
        case 'body'
          joints = [1 2 3 4 7 8 9 13 14 15 16 18 19 20 26 27 28];% p/p2/a fine
        otherwise
          error('Unknown');
      end

.
The plot below shows the original 3D ground truth location ('Ground Truth') from human3.6m, the particular frame of the video and the 3D plot in expmap format, transformed to 3D joint coordinates using your forward kinematics function, provided with your repository ('ExpMaps').

Is there an intention behind the flip?

Are you sure about the label of joints number in the figure? From the code to visualize data, I see that :+1:
parent = np.array([0, 1, 2, 3, 4, 5, 1, 7, 8, 9,10, 1,12,13,14,15,13,
17,18,19,20,21,20,23,13,25,26,27,28,29,28,31])-1
Because 1 has appearance three times, 13 two times so I think the hip is 1 and chest is 13, or hip is 0 and chest is 12. It cannot be hip mark 0 and chesht 13, it doesn't match this code

Hi,
I just take the exp map data taken from here, as explained here and then parse the n x 99 dim into 3D joint coordinates using forward kinematics as done here:
https://github.com/una-dinosauria/human-motion-prediction/blob/c9a2774fe59bb78c6ec6d04e15f2cb7243c4348c/src/forward_kinematics.py#L174-L178
. This yields n x 32 x 3 as 3D joint coordinates which you can then simply plot. The number is the respective index. I am not super familiar with the code base yet but the code you posted
parent = np.array([0, 1, 2, 3, 4, 5, 1, 7, 8, 9,10, 1,12,13,14,15,13,17,18,19,20,21,20,23,13,25,26,27,28,29,28,31])-1

seems to be taken from matlab as there is a -1 at the end.
Joint 0 and joint 11 (when starting with 0) seem to both be the root joint based at the centre of the hip in the ground-truth data as well as in the expmap data.
Cheers

Update
The 3D representation extracted from the exponential maps via forward kinematics seems to have some other interesting properties.
I align the ground-truth pose (poses from the original dataset and not converted from the exp maps) by first translating the whole pose so that the hip centre is at (0, 0, 0), then rotate around the z-axis so that the person faces forward in the positive y-direction ("facing" means: the 2d normal in the xy-plance from the vector [left hip - right hip]), then rotate around the y-axis to ensure that the left and right hip both touch the x-axis.
In the pictures below the orange/green person is from the original ground truth points while the blue/red person is the pose extracted by forward kinematics from the exponential maps. It seems that the poses generated by the forward kinematics lean backwards quite significantly. I was first thinking that there might be some constant rotation difference but that is also not the case: in the last plot I plot the the radians along the x-axis needed to match the two poses and it is definitively not constant.
@una-dinosauria would you think that there are issues with the forward kinematics or are there other things at play that I have not taken into account?

Hi @jutanke,
Thanks for your great work tracking down this issue, and sorry for the delay in responding; I was very busy with a deadline.
I suspect this is probably my fault. I try to be pretty careful when I port code to make these transformations, but it seems like this time I really did mess something up.
Another user noticed errors in my forward_kinematics code -- https://github.com/una-dinosauria/human-motion-prediction/issues/23#issuecomment-365824784 -- which is likely the source of error.
You may want to look into that function and the corresponding matlab reference code and that may fix the issue. I'll be super grateful if you can submit a fix as well.
Cheers,


Does this look better?

The Worker file is in line 2005, the original code for unit conversion:
 $bytes_format = function ($bytes) {
  if ($bytes > 1024 * 1024 * 1024 * 1024) {
    return round($bytes / (1024 * 1024 * 1024 * 1024), 1) . "TB";
  }
  if ($bytes > 1024 * 1024 * 1024) {
    return round($bytes / (1024 * 1024 * 1024), 1) . "GB";
  }
  if ($bytes > 1024 * 1024) {
    return round($bytes / (1024 * 1024), 1) . "MB";
  }
  if ($bytes > 1024) {
    return round($bytes / (1024), 1) . "KB";
  }
   return $bytes . "B";
};
More appropriate code:
$bytes_format = function ($bytes) {
  $dataUnits = ['B', 'KB', 'MB', 'GB', 'TB', 'PB'];
  $index = 0;
  while (($bytes = $bytes / 1024) > 1024) {
   if ($index === 0) {
     $index = 1;
    }
    ++$index;
  }
 return round($bytes, 1) . "{$dataUnits[$index]}";
};

I think each has its own advantages. Your code looks compact but not easy to understand.

If you add more data units, your code needs to copy one line and keep accumulating the number of lines of code.
I have added more data units here, just the array ['B','KB','MB','GB','TB','PB','EB', and more];

TB is enough for the size of data transmitted by a single connection.


Change Background and foreground colors

Hi,
How can I change the Foreground and background color in this lib.

The background uses the form background, foreground is determined by the PDF.,

@1stmetro is right, closing


Modify the Nightshade LDAP configuration to work with Dundee LDAP server

I have compared the LDAP configuration between the Learning server (already works Dundee LDAP server) and Nightshade. I have modified the playbook accordingly.

These changes have been integrated into the private repo. This repo changed drastically in the meantime and thus the PR has conflicts. Closing.


Make StringBuilderExtensions public?

Would it be possible to make StringBuilderExtensions public? Especially for provider writers the methods there are useful and reimplementing/copying these doesn't make much sense.

@cincuranet The problem is that we don't want to be in the business of shipping (and supporting) extension methods over basic types that we don't own. So for simple helpers like this we opted to make them internal. I believe other provider writers have copied the ones they use or source linked to the file in some way. @ErikEJ @roji may have more details.

Fair enough. I suppose, given it's closed now, it wouldn't make sense to discuss creating a separate NuGet package from these (i.e. Microsoft.EntityFrameworkCore.[Shared|Extensions]) and marking it as "internal" (with the same reasoning behind as internal namespaces), right? ;)

@cincuranet We've discussed that in the past as well and concluded that we don't want to do that for basically the same reasons, but that if somebody in the community wanted to create and maintain such a package, then we would consider using it and potentially even contributing to it. (I say "consider" because taking a new dependency is something that can impact more than just EF, so we have to be careful.)

OK. I'll keep that in mind. Maybe later we can extract some base layer for providers.


Generate and document password hashes

This commit does three main things:

Add a script to generate password hashes (./scripts/generate_password.py)
Improve the strength of the password hashes by doing 10,000 iterations of the hash function
Update the documentation with guidance on using the script

Looks great, thanks Chris! :+1:


[Registry Mobile] Remove sections from mobile version

Data explorer section with Venn diagrams
Distribution by SDG tiles
"About the Digital Public Good Standard" Button

The "About the Digital Public Good Standard" button is wordpress content, can be hidden from wp admin or through the wp theme.

Acceptance Criteria / User Acceptance Test

visit the https://digitalpublicgoods.net/registry/ url on a mobile device or emulated mobile width in browser
the data section with number and Venn diagramm should NOT be visible
the tiles diagram should NOT be visible


Added display for hg bookmark

@dawikur @caiogondim anything else needed for this to be good to go?

I do not use hg, so don't know if change makes sense. Taking only script for consideration: I think it looks fine.

Rebased everything on top of master branch.

Up on this one. Is it fine?

Rebased everything on top of master

@dawikur @caiogondim Is it ok or should I change/update something to get it merged?


E117: Unknown function: strptime

Steps to reproduce

Restart gVim 9.1
:echo strptime("%Y %b %d %X", "1997 Apr 27 11:49:23")

This is the first example in :help strptime()
Result: E117: Unknown function: strptime
Expected behaviour
echo message: 862156163
Version of Vim
9.1.0309
Environment

Windows Pro 22H2
gvim_9.1.0309_x64_signed.exe

Logs and stack traces
No response

https://github.com/vim/vim/issues/6573

I noticed that lervag has already solved this issue in wiki.vim. Could this solution be integrated into Vim?
https://github.com/lervag/wiki.vim/issues/254

duplicate of #6573
If your underlying C runtime does not provide strptime() function there is nothing that Vim can do against it. The solution from the mentioned vim plugin is simply testing whether strptime() exists. That is basically what everybody should do, before using strptime()

The solution from the mentioned vim plugin is simply testing whether strptime() exists.

Not only testing for the existence of strptime() but also recreating its functionality.
https://github.com/lervag/wiki.vim/blob/master/autoload/wiki/date/strptime.vim

Yes, that's largely what a plugin can do. I don't plan to add strptime() or strftime() alternative inside Vims code. That would be way out of an editors business. It's a best effort approach currently and that's why you should always use exists() before using those functions.