[HAXapi] Add API call support to package.json directly via nodejs package

@heyMP Might know a good way to do this but basically I think we should be able to do the following:

from a project: yarn run api --cmd="addPage" --title="New page here" --body="
newcontent
"

If we did this for a few it would make some of the automation / docker stuff I know @heyMP wants to do easier.
Ideally we'd call the same function / file as the express app does when hitting the /system/api/createNewSite end point (yet to be made)

@whatsacomputertho this might be a good one that overlaps w/ your other projects / interests. Effectively, we need a way of saying "ok so we build url.com/api/apiCallRoute but now we want to mirror that on the CLI". This way you could say run a simulation of your football game entirely via commandline which would then be useful for building tests in a CI service to ensure it's running correctly, but also in our purposes that'd allow us to run something like hax addPage sitename "Test page name" --contents=./filepath.html and get a new page added to the site named sitename with contents taken from that file path

#699 related because it would then be able to leverage it

exists

@btopro I was just about to add this into my sprint backlog too ;)


Add an option for skipping static functions

close #455
cc @sloede

Thanks for this quick addition @Gnimuc! However, for me, static functions are only skipped if skip_static_functions = true in the generator.toml is before the [codegen] block other than suggested in the example file. Is that to be expected and if yes, could this be adjusted in the example generator.toml?

Thanks for this quick addition @Gnimuc! However, for me, static functions are only skipped if skip_static_functions = true in the generator.toml is before the [codegen] block other than suggested in the example file. Is that to be expected and if yes, could this be adjusted in the example generator.toml?

Oops! That's a miss.

Alright, thanks for the quick reply and the quick fix!


vulkan_samples.exe don't start

The windows just closes immediately. When I try to run thru MSVS debug, I am stepping into the "An app was not requested, can not continue". Adding "sample raytracing_basic" to the command line doesn't help

Hey @PeterSikachev if you are using the latest main branch the raytracing_basic has been renamed to ray_tracing_basic. See #735

Hey @PeterSikachev if you are using the latest main branch the raytracing_basic has been renamed to ray_tracing_basic. See #735

Thanks!
I am now crashing on VK_CHECK with VK_LAYER_KHRONOS_validation, but the release is working

There is a validation PR up as well see #774


New API generation script

Add a new script to generate the json that drives API docs. This does not remove the current method as transition to this new method will be gradual. This method primarily relies on the externalized deeplearnjs-docs component.

This change is

Review status: 0 of 2 files reviewed at latest revision, all discussions resolved, some commit checks failed.

scripts/build-api.ts, line 1 at r1 (raw file):

import * as path from 'path';

add a license

Comments from Reviewable


Implement different pipelines

Work to be performed with @akmue WP in 1000Farms.
Develop a unit for data organization and cleaning, and prepare a standard input data to be used in any of the possible outcomes in the pipeline.

Part of this work has been implemented here https://github.com/AgrDataSci/ClimMob-analysis/commit/df4de1e6c8d78a6a2ee7f323b27a6f9faf037cb8
Still a work in progress


Success when key error is None

fix https://github.com/dany74q/python-apache-ignite/issues/1

@dany74q Please rebuild https://pypi.python.org/pypi/ignite


Keeping this updated?

TailwindCSS 1.8.7 is out. This plugin is still on 1.7.3. I'd love to help keep this up to date but am not sure what is required. I'd love to get the conversation going on how we can possibly automate updating TailwindCSS to pass through this package?

I'm closing this because I support one can still manually update the dependencies as needed. Thanks again!


Fix modisco convert

The previous modisco convert PR is broken, my bad. This PR fixes the problems
@jmschrei @ivyraine

Let me know if this one works once you test it out.


Check for render props and throw errors if they exist

Parcels do not support passing down render props or react children. This is because parcels are framework agnostic, but render props are React elements. So if the parcel is not written in react, it will not be able to render React elements.
What i'm thinking is that we should check props either in the constructor or in componentDidUpdate (or both) to see if any props are valid react elements. If so, we should throw an error and tell people not to do that.

Could use react-is for the checking if we wanted to go down that route.

Closing since this is old.


Sync with latest effect

Hi Max,
I've taken the liberty to update this package to the latest effect. Including the dependencies (i've taken a look at effect-ts).
I've been using https://github.com/gcanti/parser-ts in the past, so i'm hoping to use this during the advent of code 2024 (i'm already two days behind...)
Hopefully this can be merged.
I'll be using it for the advent of code, and when i run into problems, I'll report it here, and will also try to fix it.
Regards,
Joep

Hey there @joepjoosten ! Thank you very much for all your work! This library definitely needs some love at some point in the future, but I just don't have the time to put into it at the moment, so I appreciate your work!
Once CI is passing I'll happily merge!

That great! I've fixed the remaining issue, Either generic parameter reversal, and it should now build correctly and all the tests are passing.

Also seems to be a linting failure in the recently introduced circular.js file.

@joepjoosten - CI is failing on the docgen step. For now, I would suggest we just remove the docs step from CI.

Thanks for your work @joepjoosten


TestCafe documentation

Hi,
I am a beginner in test automation with testcafe. Sometimes I run into some problems and I try to find some documentation to solve my issues but I find it very hard. Maybe I am just missing some point here but I feel that it is better to ask than not to know :-).
Example: I checked this testcafe page and there was this example on how to run some browser with some extra arguments, like writing "testcafe chrome test.js --new-window"
http://devexpress.github.io/testcafe/documentation/using-testcafe/programming-interface/runner.html#using-browser-aliases
runner.browsers({
path: 'C:\Program Files\Google\Chrome\Application\chrome.exe',
cmd: '--new-window'
});
Now my question is, assume this example was not there, how would I know this is possible to write in this way? I mean I will run into other issues and where can I read that this is possible to write? What other option can I write there?
Usually I run my tests both with "testcafe chrome" option from CMD but also via Node "node runner.js chrome". But also here knowing how to write some command in the runner.js file which I know who to execute via testcafe option is, for me, quite difficult. I spend a lot of time trying to figure out these question.
Would you please explain where I find the neccessary documentation in order to find out that info myself?

So I found following link which is quite good. I share it here in case other people have difficulties coming across this page.
https://github.com/DevExpress/testcafe/tree/master/docs/articles/documentation


Add tests for AdaptedReactiveMessageConsumer

No tests for acknowledgeAsynchronously nor acknowledgeScheduler . I opened https://github.com/apache/pulsar-client-reactive/issues/72 to track this.


Stable URL scheme for badges + links to launch binder

This is different from #13 since it focuses solely on what links people will use when clicking 'launch binder' links, rather than a HTTP API that code can also call. Requires different design and guarantees.
We also need to have a compatibility layer so current launch binder links continue to work.
Current URL structure
Only supported URL structure now is:
/repo/<username>/<repo>/<path-to-start>

While a great start, it has some extensibility problems:

Assumes it is using GitHub, and doesn't provide an easy way to use other providers.
Unclear how it can handle arbitrary git URLs (that aren't hosted in a 'provider' as such)
Not specified how you can provide other arbitrary run-time parameters in jupyterhubs that support it (such as memory limits, extra data to be provisioned, etc)

Whatever we do, we'll make sure these links continue to work for the foreseeable future. We should find and send PRs to people tho to change them.
Proposed new URL structure
/v2/repo/<git-clone-url>/ref/<ref>/?<runtime-params>

where:

git-clone-url is URL encoded URL pointing to a git repository (such as a github URL). We will interpret this pretty generously.
ref points to a commit hash, branch or tag. If it's a branch or tag it'll be redirected to a commit hash - permanently for a tag and temporarily for a branch.
runtime-params is query parameters used for all runtime parameters - including the path to launch, and in the future other parameters too (extra data to load, RAM requirements, etc). These will be formally defined too.

This separates the build parameters (git url + ref) from the runtime parameters. Also establishes that the canonical URL is the one with the commit hash, rather than branch or tag info.
Possible modifications:

Version this API too, with a /v2/ prefix Decided to actually do this.
Special case 'path' runtime parameter, include that in the path. Not sure how necessary that is.

Oops, just realized my comments were in the REST api issue and not this one...@minrk see this thread too :-)
for /v2/repo/<git-clone-url>/ref/<ref>/?<runtime-params>
would it be something like:
/v2/repo/https://github.com/binder-project/example-dockerfile-two.git/ref/master/?
It feels a little cluttered to me. Could we support /user/repo/branch and have people specify a non-github designation in the <runtime-params> section? I really like the cleanliness of /user/repo/<branch | commit | tag>.
so something like:
mybinder.org/v2/repo/choldgraf/ecogtools/master
and
mybinder.org/v2/repo/choldgraf/ecogtools/master?host=bitbucket
Another thought: what if we had prefixes that corresponded to the host, so:
mybinder.org/v2/gh/choldgraf/ecogtools/master
mybinder.org/v2/bb/choldgraf/ecogtools/master

I like the last one!
/v2/<provider>/<provider-specific-build-params>
so github would interpret that as username/repo, bitbucket can represent whatever it wants, and we can also support plain URLs that way. Definitely don't want to move non-github source in runtime-params - it's not a runtime param, and I want to keep build / runtime params separate. (I should also properly define somewhere what a runtime param is and what a build param is lol)

Cool - yeah I'm a fan of the last one as well. That way we can have whatever URL structure we want if we support something that has non-standard structure to it.

maybe we could have both a long-hand and short-hand name for each supported host. github/ and gh/, or bitbucket/ and bb/

We generally should have one 'canonical' form of a URL, and have all other
forms redirect to it. I'm also going to add a bit in the UI for actually
generating these links...
On Thu, May 18, 2017 at 10:32 AM, Chris Holdgraf<EMAIL_ADDRESS>wrote:

maybe we could have both a long-hand and short-hand name for each
supported host. github/ and gh/, or bitbucket/ and bb/
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/yuvipanda/builderhub/issues/15#issuecomment-302484667,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAB23ufsOXH9h02CKOX_OleJf3glQPI9ks5r7IC3gaJpZM4Nc4VC
.

--
Yuvi Panda T
http://yuvi.in/blog

From the user perspective the last proposal looks nice. Just my 1/inf cents ;)

👍 to including providers in the URL. It may or may not be useful to look at how nbviewer handles a similar concept.

e.g.:

https://nbviewer.jupyter.org/github/ipython/ipython/blob/4.0.x/examples/IPython%20Kernel/Index.ipynb
https://nbviewer.jupyter.org/url/jakevdp.github.com/downloads/notebooks/XKCD_plots.ipynb

I've currently implemented the following:
beta.mybinder.org/v2/gh///?filepath=
is the canonical URL. We've agreed on filepath as the name for the URL that'll be opened after launch. It'll do autodetect to try to open things in notebook or other view as necessary.
In addition, there's a redirect from
beta.mybinder.org/repo// to beta.mybinder.org/v2/gh///master. We haven't implemented filepath support yet but will do.

I think we should close this issue and open new ones for specific extensions of the URL structure (e.g. for bitbucket). WDYT?

We have this!


Compat with Tiered mod.

Can You add support for Tiered mod? Mod GitHub.
This mod gives tools, armor, and weapons random qualities when they enter your inventory, inspired by Terraria tools. All the qualities, as well as the tools that can get them, are completely configurable.

Seems like You would need to do this - "Mods need to add their tools to the respective tags inside the fabric namespace (fabric:pickaxes, fabric:chestplates, fabric:shovels, etc.)" - https://github.com/Draylar/tiered/issues/12

I'll see about adding support for this mod, I should probably have my tools in that namespace in the first place because it's likely other mods might do something similar.

I'll see about adding support for this mod, I should probably have my tools in that namespace in the first place because it's likely other mods might do something similar.

Yeah, so this feature has been added as of v1.2.3. Tiered hasn't been updated to 1.16.5 yet(at least I'm not sure if the current version works on it), but I did test it with the 1.15 version, and it all working I went ahead and added the tools into the correct tags in all supported versions of Survival Additions so if the mod is backported to 1.14.4 it should work.

Yeah, so this feature has been added as of v1.2.3. Tiered hasn't been updated to 1.16.5 yet(at least I'm not sure if the current version works on it), but I did test it with the 1.15 version, and it all working I went ahead and added the tools into the correct tags in all supported versions of Survival Additions so if the mod is backported to 1.14.4 it should work.


On Deployer's "global" nature

I'm starting to have a problem.
Deployer has a "global" nature, where you can set and get global variables, environment variables; we also have global functions, etc.
It simplifies things of course, but it also introduces numerous possibilities for errors.
This is the reason why we had to implement the ability to protect certain environment variables not so long ago: https://github.com/deployphp/deployer/pull/276
To further explain, and demonstrate my problem, consider the Symfony recipe.
Just in this recipe alone, there's a bunch of global variable declarations.
Since both the env and global variables declared here are "global", other tasks or recipes could conflict with them.
Look at this issue: https://github.com/deployphp/deployer/issues/311
Suppose I have a 3rd party recipe installed in my project, which already sets the "composer_options" global variable.
Now if this change gets implemented and merged in, my deployment workflow could be broken the next time I update Deployer to the newest version.
Having said all the above, I think there should be a discussion about how configuring the tasks could be solved a different way, which would address these issues.
To start off: I have an idea which could be implemented in the next (v4) version.
<?php

// in deployer.php
set_config('task-name', [
    'bin_dir' => '...',
    'composer_params' => '...',
]);

// in the recipe
task('task-name', function() {
    $config = get_config('task-name', [
        // default values
        'bin_dir' => '...',
        'var_dir' => '...',
        'composer_params' => '...',
    ]);
});

The idea here is similar to the global parameters, but now the variables are "scoped" under the task's name, so for example the "bin_dir" parameter could be used by any number of tasks, and they won't conflict with each other.
Just think about how simpler the 3rd party slack task could be.
With this solution the need for "global variables/parameters" would be gone too.

A somewhat relevant issue: https://github.com/deployphp/deployer/issues/292

Nice idea! But the main appointment is to keep config simple as possible.
Issues you linked are not about separating setting conflicts. Them are for add some more configs.
Also if you do not require 3rd party recipes with require, this does not lead you to errors.
The next features i suggest to implement in Deployer is next:

Immutable settings by default. With available to override, and parallel support.
Events with on('...', function ($event) {...}); based on The EventDispatcher Component.
DI Configuration based on Pimple.
So, this is my little roadmap :)
PS
Native SSH support (slow, but 100% featureful)

I see, that seems neat!
If you're going to use pimple, there's no need to implement my proposed solutions, since that can be achieved in a nicer way through container registration.
The introduction of the event dispatcher would be very nice too. :+1:
There would be no need to fully override a task just to do something a little bit different.
Are these plans for the 4.x release?
I'm just assuming, since these would be BC-breaking changes.

This all will be for v3.1. No BC-breaks.

Sounds great!
I can't wait. :)

If you want you can help with implementation of this things :)

Well first we have to plan exactly how the features are going to be implemented.
We could use the gitter channel for that.
And then if I feel capable I will gladly help. :)

Another example why the "global" way Deployer works could be harmful: https://github.com/deployphp/deployer/issues/316


Working branch ?

Hi !
Really like the project, did try the demo and it is great !
I'm trying to fork the master branch but I'm getting a bunch of error (VBWalkerUtils doesn't compile for instance, syntax for get & set throws errors).
Can you please point me to a working branch ?
Many thanks !

@george2giga we've stopped developing this project when the engine behind .NET reference source was open sourced: https://github.com/KirillOsenkov/SourceBrowser

@AmadeusW that's ok I've managed to get it working, you've done a fantastic job btw, thanks !

alright, thanks!


Feature/gulp instead of assetic

Coverage increased (+0.08%) to 88.25% when pulling ff51d71cb58ee6d5e8440cd928316ac34682f42b on feature/gulp-instead-of-assetic into ac8e4243f3e2d8258e69bf1ed9894c278cfc251d on master.

Coverage increased (+0.08%) to 88.25% when pulling ff51d71cb58ee6d5e8440cd928316ac34682f42b on feature/gulp-instead-of-assetic into ac8e4243f3e2d8258e69bf1ed9894c278cfc251d on master.

Coverage decreased (-0.02%) to 88.16% when pulling 31ec8c85cbb6c8f4eee8538284cf95293e71a33c on feature/gulp-instead-of-assetic into ac8e4243f3e2d8258e69bf1ed9894c278cfc251d on master.


How does this project be compatible with BPE tokenizers?

Hi, I have found this part in the doc:

It demonstrates the word "date" is not acceptable. Will sub workds like "uni" in BPE tokenizers are acceptable?

Thank you! Yes, the key doesn't have to be one token, not at all; a key constructed of several subword tokens will work just fine, this project is built around BPE tokenizers.


Production Trusted IPs does not allow enough IPs

Hiya,
We have a use case for more than the 20 or so IPs that Production Trusted IPs currently supports on our org.
This worked fine in Envkey1. Envkey2 seems to limit it for some reason. Is this a technical limitation or a UI one?
Also, can this be changed back to a freetext field please? It is laborious to enter them one at a time and it's easy to fat-finger a deletion and not know which one you've deleted, and then you have to hunt for it... just a pain all around :)

The description on the Org Firewall page says "add any number of IPV4/IPV6 ranges" so going to assume this is a UI bug. Try adding more than 20 or so CIDR ranges and you'll see what I mean, the UI stops you from entering any more.
Also copy/paste from envkey1 to envkey2 doesn't work, you'll again see what I mean if you try it.

You're right that for practical purposes there shouldn't be a limit. We do need a sanity-check upper limit but we can definitely make it higher (assuming that's what you're running into--I'll have to confirm). Could you give me a ballpark on how many you need to add?
We'll also see what can be done about adding back copy-paste support or just going back to free-form entry. Thanks for the feedback on that.

Copy-paste for these fields is now supported in 2.2.3. We couldn't reproduce the limit on IPs/CIDRs. They should be unlimited. Is it possible you were trying to add a duplicate value?
In any case, let me know if you're still seeing that and I'll re-open.

This works now, thanks.


fix for bug when reading Waterinfo #162

I think it depends on the station which of the two column names you need. For older measurement stations you might need ["WAARNEMINGDATUM", "WAARNEMINGTIJD"] whereas for newer stations you have to add the (MET/CET) to the "WAARNEMINGTIJD" column.
But you can supply the index columns in the function so you should be able to adapt your read accordingly.

I added some logic to deal with both "WAARNEMINGTIJD" and "WAARNEMINGTIJD MET/CET"


Undo button still clickable during undo animation

Steps to reproduce

ListView with a SimpleSwipeUndoAdapter (occurs in the sample app too if you're fast)
Swipe away an item to reveal the undo bar
Hit the undo button twice in rapid succession

Result

The onUndo() callback in the adapter is called twice
The undo swipe in animation is restarted (list item gets set back off screen and swipes in again)

Expected Result

onUndo() callback called only once, animation not resetted
the undo button should not be clickable during the animation

Also, it would be nice to have touch feedback on the undo button (apart from me wanting to have the layout like in the GMail app per default...), but that is probably a larger issue for another time, as it requires a totally different implementation...
Still great library, keep up the good work!


Fix master

Some of the recent changes broke master and we are no longer writings the files properly:
/opt/html # ls
bootstrap.min.css

/opt # ls diagnostics.json
diagnostics.json
/opt # ls initFile.txt
initFile.txt

Most likely, this was introduced in https://github.com/NebraLtd/hm-diag/pull/68 as this was a large rewrite.

I'm inclined to leave master broken in favor of https://github.com/NebraLtd/hm-diag/pull/81 as I've rolled back to an older version for testnet and mainnet.


enhancing: Improving railway configuration of karma gap backend

@RafaDSan can you add a description to this issue based on the feedbacks you received and want to implement?

[ ] Add the output of the task in this issue comment section
[ ] Low down the resource limits of the server
[ ] Create a documentation explaining each field of the database
[ ] Add README

Task output:
API endpoint on railway: https://trustful-karma-gap-backend-staging.up.railway.app/


Quantum metric bdg

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

31 of 31 (100.0%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
425

Relevant Lines:
425

💛  - Coveralls


Cache hash created based on file hash installs old versions

Hi,
we are using the mise action with mise.toml files that mostly contains only tools with their prefixes, like
terragrunt = "prefix:0"

Now it can happen that a new patch version of one tool is released but no changes to the .mise.toml file are necessary. The action installs the version of the cache based on the file hash but when I want to use the tool I get the error that the latest version, specified implicitly by the prefix notation, is not found
Output:  mise No version is set for shim: terragrunt
Set a global default version with one of the following:
mise use -g<EMAIL_ADDRESS>
Here, 0.67.3 is in the cache, but 0.67.4 was released. I tried upgrading using
  mise up
  mise install

  shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0}
  env:
    MISE_LOG_LEVEL: info
    MISE_TRUSTED_CONFIG_PATHS: /home/runner/_work/azr-projects-compositions/azr-projects-compositions
    MISE_YES: 1

mise All tools are up to date
mise all runtimes are installed

But that also does not upgrade to the latest version for some reason.
In my understanding it would be necessary to create the hash based on the tool versions actually used, not the file specifying them.
Any input on this is appreciated :)
BeneHa

@BeneHa, hi. Could you please tell how you solved this?

@BeneHa, hi. Could you please tell how you solved this?

We did not really solve it, we just delete the cache whenever this happens...

this should work when using the new lockfile setting now


Do all threads have to terminate before the main thread terminates?

We should continue the discussion #68 here.

I would vote for "it is the kernels responsibility to make sure that all threads terminate correctly and exit the VM". I haven't put much thought to it, but it seems much more straight forward. For example, the kernel may still have some work to do e.g. flushing buffer and may split that task among all cores. The current design forces the kernel to synchronize the cores and make sure that no exit code is provided until everything is done. Furthermore the hypervisor needs additional code to handle that case. But than again, I haven't thought to much on how hard it is to ensure that all cores exit the VM. 🤷♂

I would vote for "it is the kernels responsibility to make sure that all threads terminate correctly and exit the VM". I haven't put much thought to it, but it seems much more straight forward. For example, the kernel may still have some work to do e.g. flushing buffer and may split that task among all cores. The current design forces the kernel to synchronize the cores and make sure that no exit code is provided until everything is done. Furthermore the hypervisor needs additional code to handle that case. But than again, I haven't thought to much on how hard it is to ensure that all cores exit the VM. 🤷♂

I also think that the kernel should properly offline all cores when shutting down. If the kernel doesn't shut down all cores, then that should be treated as an error (as long as we don't support dynamic hotplugging).
My proposal would be for the following behavior:

If one thread exits with Err, then I think we can assume that the kernel crashed and can exit with an error, without waiting for any other threads.
If one thread exits normally, we should set up an alarm (or some other timer) to give the kernel some time to properly shut down all vCPUs.

If all other threads terminate before the alarm goes off, then the kernel exited successfully. The exit code needs some special attention here since each vCPU returns an option. I'd prefer if we explicitly check that a) only one vCPU returns an exit code, and b) it is the last vCPU that returns the exit code.
If at least one thread fails to terminate before the alarm handler gets invoked, then exit uhyve with an error code and a warning.

I also think that the kernel should properly offline all cores when shutting down. If the kernel doesn't shut down all cores, then that should be treated as an error (as long as we don't support dynamic hotplugging).
My proposal would be for the following behavior:

If one thread exits with Err, then I think we can assume that the kernel crashed and can exit with an error, without waiting for any other threads.
If one thread exits normally, we should set up an alarm (or some other timer) to give the kernel some time to properly shut down all vCPUs.

If all other threads terminate before the alarm goes off, then the kernel exited successfully. The exit code needs some special attention here since each vCPU returns an option. I'd prefer if we explicitly check that a) only one vCPU returns an exit code, and b) it is the last vCPU that returns the exit code.
If at least one thread fails to terminate before the alarm handler gets invoked, then exit uhyve with an error code and a warning.

* If one thread exits normally, we should set up an alarm (or some other timer) to give the kernel some time to properly shut down all vCPUs.

  * If all other threads terminate before the alarm goes off, then the kernel exited successfully. The exit code needs some special attention here since each vCPU returns an option. I'd prefer if we explicitly check that a) only one vCPU returns an exit code, and b) it is the last vCPU that returns the exit code.
  * If at least one thread fails to terminate before the alarm handler gets invoked, then exit uhyve with an error code and a warning.

I agree that this is probably the best solution, but I am wondering if that is worth the effort given that we need to maintain a testcase for it and find a reasonable value on how long a kernel is allowed keep some cores online. I would only implement such bug mitigation for a daemon. For a simple command line tool it seems fine to deadlock as I can still kill it easily.

* If one thread exits normally, we should set up an alarm (or some other timer) to give the kernel some time to properly shut down all vCPUs.

  * If all other threads terminate before the alarm goes off, then the kernel exited successfully. The exit code needs some special attention here since each vCPU returns an option. I'd prefer if we explicitly check that a) only one vCPU returns an exit code, and b) it is the last vCPU that returns the exit code.
  * If at least one thread fails to terminate before the alarm handler gets invoked, then exit uhyve with an error code and a warning.

I agree that this is probably the best solution, but I am wondering if that is worth the effort given that we need to maintain a testcase for it and find a reasonable value on how long a kernel is allowed keep some cores online. I would only implement such bug mitigation for a daemon. For a simple command line tool it seems fine to deadlock as I can still kill it easily.

For a simple command line tool it seems fine to deadlock as I can still kill it easily.

My main motivation is to improve the usability of uhyve for CI. If uhyve simply deadlocks, then the CI jobs will still be marked as "running" although they actually have already failed, which delays the feedback.

find a reasonable value on how long a kernel is allowed keep some cores online

For this purpose a timeout in the order of seconds would be perfectly fine, and I would be surprised if that wouldn't be enough for the kernel to shutdown the other threads regularly (but I may be wrong about that).

For a simple command line tool it seems fine to deadlock as I can still kill it easily.

My main motivation is to improve the usability of uhyve for CI. If uhyve simply deadlocks, then the CI jobs will still be marked as "running" although they actually have already failed, which delays the feedback.

find a reasonable value on how long a kernel is allowed keep some cores online

For this purpose a timeout in the order of seconds would be perfectly fine, and I would be surprised if that wouldn't be enough for the kernel to shutdown the other threads regularly (but I may be wrong about that).

For a simple command line tool it seems fine to deadlock as I can still kill it easily.

My main motivation is to improve the usability of uhyve for CI. If uhyve simply deadlocks, then the CI jobs will still be marked as "running" although they actually have already failed, which delays the feedback.

I see. In that case, would a timeout for the individual tests not make more sense? After all, there are other bugs that could cause a deadlock.

Timeouts for the tests have a larger overhead/safety margin since the time for the test itself also has to be considered.

After all, there are other bugs that could cause a deadlock.
If there is a deadlock in the kernel itself, then this, of course has to be caught with the existing test timeout.

Thinking about it again, I think the main issue I had, is that previously the kernel would not exit if one thread paniced in kernel mode (I think we addressed that and now exit after Err).
So I guess you are right and adding additional timeout handling for non Err return values is probably just adding unnecessary complexity.

Timeouts for the tests have a larger overhead/safety margin since the time for the test itself also has to be considered.

After all, there are other bugs that could cause a deadlock.
If there is a deadlock in the kernel itself, then this, of course has to be caught with the existing test timeout.

Thinking about it again, I think the main issue I had, is that previously the kernel would not exit if one thread paniced in kernel mode (I remember we addressed that and now exit after Err).
So I guess you are right and adding additional timeout handling for non Err return values is probably just adding unnecessary complexity.

I just noticed that I have not explicitly stated my motivation why I am skeptical of adding the timer: The hypervisors is security sensitive, which is why I am just worried that we are adding unnecessary code. If we can handle it in the CI than I think we should do it in the CI. That doesn't mean we should not add it to uhyve if it turns out to be necessary.


java.lang.IncompatibleClassChangeError thrown on collection.empty

When running with java8 and mvel 2.4.1.Final I get a very reproducible IncompatibleClassChangeError when evaluating a trivial expression to check if an arraylist is empty. See exact unit test below.
If I run with 2.4.0.Final it still works fine. I suspect the upgrade to ASM 7 (done in 2.4.1.Final) has something to do with this.
Please fix ;-)
public class MVELTest {
private static final int COUNT = 10000;

@Test
public void evaluateCollectionExpression() {
    Serializable compiled = MVEL.compileExpression("validationIssues.empty");
    Map<String, Object> properties = new HashMap<>();
    properties.put("validationIssues", new ArrayList<>());
    int trueCount = 0;
    for (int i = 0; i < COUNT; i++) {
        if (MVEL.executeExpression(compiled, properties, Boolean.class)) {
            trueCount++;
        }
    }
    System.out.println("Evaluated " + COUNT + " true -> " + trueCount);
}

}

Interestingly, if you replace .empty with .isEmpty() in the expression, it works.
Some logic in ASMAccessorOptimizer.getMethod() is missing from ASMAccessorOptimizer.getBeanProperty() - which is called for "empty".
The declaring class for the isEmpty method is java.lang.Collection, an interface, so it should be invoked using INVOKEINTERFACE rather than INVOKEVIRTUAL (around line 1119 in ASMAccessorOptimizer). Changing this fixes the issue.
The code seems to foresee this case by catching IllegalAccessException but this isn't thrown at this time.

@mariofusco could you approve PR 201 please?


Please validate the following proposal for the Release Note of 3.0.0

Release Note
The papiNet Central Work Group is pleased to announce the official release of the papiNet API 3.0.0! This release contains a completely new set of API endpoints focusing on the Warehouse Logistics use case. You should not see this 3.0.0 release as an evolution of 2.0.0, but as a sibling work introducing logistic concepts. We will later merge 3.0.0 with the previous releases, 2.0.0 remains for now fully relevant in the context of the Purchase Order use case.

Dear Maintainers, could you give an off-line approval for releasing 3.0.0 via this issue's comments...

approved

Validated and approved by me
Kind regards
Jens Nordenberg
Verksamhetsspecialist
T: 010-228 51 16
M: 070-559 04 49
E: @.***
@.***
Biometria ek för
Skepparplatsen 1, 852 34 Sundsvall
T: 010-228 50 00
W: www.biometria.sehttp://www.biometria.se/
Från: Patrice Krakow @.>
Skickat: den 18 december 2024 16:02
Till: papinet/papiNet-API @.>
Kopia: Jens Nordenberg @.>; Assign @.>
Ämne: Re: [papinet/papiNet-API] Please validate the following proposal for the Release Note of 3.0.0 (Issue #194)
<EMAIL_ADDRESS>liknar någon som tidigare har skickat e-post till dig, men kanske inte är den personen. Lär dig varför det kan vara en riskhttps://aka.ms/LearnAboutSenderIdentification
Information: Detta är ett mejl från en extern källa utanför Biometria. Iaktta varsamhet innan du klickar på länkar eller öppnar bifogade filer. Vid osäkerhet kontakta din lokala IT-avdelning.
Dear Maintainers, could you give an off-line approval for releasing 3.0.0 via this issue's comments...

Reply to this email directly, view it on GitHubhttps://github.com/papinet/papiNet-API/issues/194#issuecomment-2551552144, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APAUXMKWWLSNMAUMCAVIOW32GGE4VAVCNFSM6AAAAABSE5LBIWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNJRGU2TEMJUGQ.
You are receiving this because you were assigned.Message ID<EMAIL_ADDRESS>Vi skyddar dina personuppgifter.
Biometrias integritetspolicyhttps://www.biometria.se/om-biometria/om-oss/biometrias-integritetspolicy/

Approved

Approved


build<EMAIL_ADDRESS>

Publish version 4.4.0.
Expected to be released at 03.12.
note: The last commit must be build: publish@{version}.

Coverage: 100.0%. Remained the same when pulling e969cf16b591de7c38339bfdddc3919d153e19e7 on dev into 28fe66c6cd2356b9b849ff0cd771050b679c3d32 on main.

@noname0310 plz help to review


[MRG] Made API errors clear

Closes #16
General Checkups

[x] Have you checked that there aren't other open pull requests for the same issue/update/change?
[ ] If your code includes new features and not just bug fixes/copy editing: Did you include new tests?

Description

API errors raise exception based on their returning status codes.
Related Issue

Example

@madprime since we have vcrpy for the tests now, this PR is now obsolete I think. Shall I close this?

I'm not sure, the vcrpy tests just check how the module responds to different inputs and that it works the way we think the code should work. Your changes deal with how the api code handles errors during the runtime so it might still be useful to adapt these?

If it is so then I will refactor the code and resolve the conflicts soon so that this becomes mergeable. Thank you :slightly_smiling_face:

That sounds good! 👍

You now would also have to adapt the tests to check for your exceptions, see https://travis-ci.org/OpenHumans/open-humans-api/jobs/354398847 for the test log.

@mcescalante I have made the suggested changes. Please review. Thanks :D

Updated @gedankenstuecke

@mcescalante please review. Thanks :)


Allow adding additional disks to GCE instance

Created additional_disks option that attach additional disks to GCE instance.
https://github.com/fog/fog-google/pull/434

@criccomini @whynick1 LGTM, merging.
Sorry for the additional wait, past couple of weeks  have been very hectic due to some life events. I'll release an RC version shortly after some small fixes.

@criccomini @whynick1 Published the release candidate, you can test it out with vagrant plugin install vagrant-google --plugin-version 2.3.0.rc0 If you have a chance, please take a look.

Version 2.3.0.rc0 verified on our end! Thank you @Temikus

Awesome! Thanks for checking @whynick1.
Merging.

Released 2.3.0


providing data sources from excel file  crash visual studio while the Excel- file is opened

Model Builder Version: _16.0.1910.901
Visual Studio Version:  vs Enterprise 2019, v-16.3.7
Bug description
After you choose a scenario and arrived at data page  and then choose  excel file which is already opened behind  makes the Vs crashed.
Steps to Reproduce

Open model builder
select  price prediction scenario.
Open you data-set in excel
while your data-set file is opened in excel select  that opened file from model builder's  select file section

Expected Experience
The selected file top ten  rows previewed and  the user able to select label and feature columns
Or
Let the user there is an error and make sure to fix that to continue.
Actual Experience
Vs crashed

Validate on VS2019 VS2019 preview

Validated that an error message is shown


feat: add required option

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


\l__csl_language_code_map_prop is not initiated corrected

Describe the bug
\l__csl_language_code_map_prop is not initiated correctly, leading to an error with the latest expl3 release. See e.g. https://tex.stackexchange.com/questions/709871/failed-to-compile-latex-document-citation-style-language
Additional information

TeX distribution: TL'23
Package citation-style-language version: {2023-10-05} {0.4.5}
LaTeX engine: all

To Reproduce
\documentclass{article}
\usepackage{citation-style-language}

\begin{document}
\end{document}

Thanks for your report and fix!


[WIP] tests(userspace/libscap): move libscap tests into test folder and add engine specific tests

What type of PR is this?
/kind cleanup
/kind feature
Any specific area of the project related to this PR?
/area CI
/area tests
Does this PR require a change in the driver versions?
No
What this PR does / why we need it:
This PR moves all libscap tests into the test folder and adds some tests for engines modern_bpf, bpf, kmod. The PR is just a proposal, if you have other ideas or suggestions on how to do that I'm all ears
Which issue(s) this PR fixes:
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
NONE

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/milestone 0.11.0

@jasondellaluce: You must be a member of the falcosecurity/libs-maintainers GitHub team to set the milestone. If you believe you should be able to issue the /milestone command, please contact your maintainers of falcosecurity/libs and have them propose you as an additional delegate for this responsibility.

In response to this:

/milestone 0.11.0

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

/milestone 0.11.0

Glad we're back being friends, @poiana.

we still need to:

remove old libscap tests.
chose how to manage tests in CI, probably I would use the matrix just to build sinsp and have a separate job for running all scap and sinsp tests with all options BPF, KMOD, MODERN. GVISOR

There is a dependency with https://github.com/falcosecurity/libs/pull/816, that adds new tests

This PR adds the new test/libscap folder, unfortunately, I cannot remove the old directory libscap/test since it is heavily used in our CI jobs, and the switch would require massive changes in almost all the jobs. The issue is that new tests introduce engine-specific tests like kmod, bpf modern-bpf for this reason they try to inject the corresponding driver when running. This could be an issue since in some CI jobs we use QEMU and we are not able to inject BPF or the kmod :/ We need to find a solution to this, right now I've enabled these new tests in just one job test-scap-x86 but the final idea would be to replace the old libscap tests with the new ones also in CI
Moreover, if you need to add new libscap tests it would be great if you could add them directly under this new folder test/libscap

LGTM label has been added.  Git tree hash: 3ecee255790b5573a75a9612c4650caeac4a3ebf

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP, jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP,jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Branch restructuring fail to pick up branches inside loop

From DEBUGGRAPH=1 pytest numba_rvsdg/tests/test_mock_asm.py::test_mock_scfg_double_exchange_loop in #42
CFG:

loop and branch restructured

Missing branch regions inside loop region.

Now i realize that the restructuring functions are not by default recursive and fixed this in #42. This is no longer an issue.


win32:Evo-gen virus

I get this virus build that it seem one or more packages are infected. This is on window 10 64bit. It stop around
parking_lot...  /build_scipt ...
It strange that packages is infected or rogue dev. Not sure if this right to post this. Note this fresh build test.

@Lightnet Please report this here: https://github.com/Amanieu/parking_lot/issues

I see.


Add files via upload

CPP code for question no.11 in Array section

Added solution for question 11 in array section sum2.cpp


plus not working?

I saw that there was a recent fix, but it is still not working for me. The + character is not pulling in the current name of the layer. I have Sketch 49.1 and the latest RenameIt (3.7.2).
Thanks!

Sorry for that. I had to remove the plus because it was causing conflicts. If you use %* it should do the same thing.

Ah, got it. Thanks for the workaround.


Update OSX Run method

The OSX instructions are not working. Added a working one. The ./build.sh doesn't work with osx, need the osx specific one. And the tools folder in the repo causes Error: An item with the same key has already been added. even if I use osx specific script. I highly suggest add tools/ to .gitignore
Since this is a trivial build guidance change, no issue has been opened, if needed please comment.

@HustLion,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla2.dotnetfoundation.org.
It will cover your contributions to all .NET Foundation-managed open source projects.
Thanks,
.NET Foundation Pull Request Bot

@HustLion, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, .NET Foundation Pull Request Bot

@HustLion said...
The ./build.sh doesn't work with osx, need the osx specific one.

I am slightly confused by this.  Although we have separate URL's for OS x and Linux, the URL's actually resolve the to exact same file, which is this one:
https://raw.githubusercontent.com/cake-build/resources/master/build.sh
The bootstrapper also is responsible for downloading Cake, so I don't see any need to globally install Cake using homebrew.

@HustLion Did you have any comments about the review that I did above?  Were you in a position to update the PR?  Thanks for your help!

@gep13 Thanks for your review! Sorry that I didn't reply earlier. Since you pointed out the URL's actually resolve the to exact same file, I think the issue might be in my operations. Currently I'm not using Cake so I can't really confirm the source of the issue. Closing this now. If later I find out that the issue comes up again, I will provide more info.


DataCollector deadlock when using Test Impact Analysis (TIA) together with disabled CodeCoverage

Description
See title
Steps to reproduce

Have a multi core build machine
Run a UnitTest build with VSTest, MSTest 1.3.0-beta2, [assembly: Parallelize(Scope = ExecutionScope.ClassLevel)], Run only impacted tests and Run tests in parallel on multi-core machines and the following .runsettings

<RunSettings>
  <RunConfiguration>
    <TargetPlatform>x64</TargetPlatform>
    <TargetFrameworkVersion>Framework45</TargetFrameworkVersion>
  </RunConfiguration>

  <DataCollectionRunSettings>
    <DataCollectors>
      <!-- https://docs.microsoft.com/en-us/visualstudio/test/customizing-code-coverage-analysis -->
      <DataCollector enabled="false" friendlyName="Code Coverage" uri="datacollector://Microsoft/CodeCoverage/2.0"
                     assemblyQualifiedName="Microsoft.VisualStudio.Coverage.DynamicCoverageDataCollector, Microsoft.VisualStudio.TraceCollector, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a">
        <Configuration>
          <CodeCoverage>
            <ModulePaths>
              <Exclude>
                <ModulePath>.*FakeItEasy.*</ModulePath>
                <ModulePath>.*GongSolutions.WPF.DragDrop.*</ModulePath>
              </Exclude>
            </ModulePaths>
            <Sources>
              <Exclude>
                <Source>.*\.Designer\.cs</Source>
              </Exclude>
            </Sources>
            <Attributes>
              <Exclude>
                <Attribute>^System\.Diagnostics\.CodeAnalysis.ExcludeFromCodeCoverageAttribute$</Attribute>
              </Exclude>
            </Attributes>
          </CodeCoverage>
        </Configuration>
      </DataCollector>
    </DataCollectors>
  </DataCollectionRunSettings>
</RunSettings>

Expected behavior
The test process should exit and continue as usual.
Actual behavior
datacollector.exe causes a deadlock and tests will never finish
Note: If you manually process-kill the datacollector.exe, the build will continue and finish
Environment

multi core build machine
VS 15.6.6
VSTest 15.6.2 or 15.7.0

I dug deeper and found that the disabled CC Collector causes the issue
<DataCollector enabled="false" friendlyName="Code Coverage"

In further Analysis it shows that even without CC the TIA data collector deadlocks ... The circumstances vary, but the effect is always the same.
Kill datacollector.exe and proceed...

@Mertsch Sorry for the delay here. Can you take dump when datacollector.exe is hung and share it with us.

@acesiddhu Can you please tell me the CMD for that, I haven't taken a dump yet.

@Mertsch you can take dumps from task manager. Goto details tab, right click on datacollector process and click take dumps option there

@acesiddhu Here is the download, please tell me when you got it
https://freedommanufaktur-my.sharepoint.com/:u:/g/personal/martin_freedom-manufaktur_com/EVcuj4VGjcxNr0__bMdQbLsBysjOpDGpZ2a_cnCWGw_5jg?e=khRcGK

@Mertsch I got the dump files. will get back to you after investigations.

I believe I have a similar, if not identical problem.  Here are the callstacks from datacollector.exe
0:004> !CLRStack
OS Thread Id: 0xb44 (4)
Child SP               IP Call Site
000000f332aff0f8 00007ffbb572f0e4 [HelperMethodFrame_1OBJ: 000000f332aff0f8] System.Threading.WaitHandle.WaitOneNative(System.Runtime.InteropServices.SafeHandle, UInt32, Boolean, Boolean)
000000f332aff220 00007ffb9b4be98b System.Threading.WaitHandle.InternalWaitOne(System.Runtime.InteropServices.SafeHandle, Int64, Boolean, Boolean)
000000f332aff250 00007ffb9b4be95e System.Threading.WaitHandle.WaitOne(Int32, Boolean)
000000f332aff290 00007ffb3de69726 Microsoft.VisualStudio.Diagnostics.Logging.LoggerEngine.Stop(Boolean)
000000f332aff310 00007ffb3de66342 Microsoft.VisualStudio.TraceCollector.CommonDataCollector+LoggerWrapper.LoggedProcess_Exited(System.Object, Boolean)
000000f332aff360 00007ffb9b503423 System.Threading.ExecutionContext.RunInternal(System.Threading.ExecutionContext, System.Threading.ContextCallback, System.Object, Boolean)
000000f332aff430 00007ffb9b5032b4 System.Threading.ExecutionContext.Run(System.Threading.ExecutionContext, System.Threading.ContextCallback, System.Object, Boolean)
000000f332aff460 00007ffb9beb1425 System.Threading._ThreadPoolWaitOrTimerCallback.PerformWaitOrTimerCallback(System.Object, Boolean)
000000f332aff610 00007ffb9d406d93 [GCFrame: 000000f332aff610]
000000f332aff808 00007ffb9d406d93 [DebuggerU2MCatchHandlerFrame: 000000f332aff808]
0:004> ~5s
ntdll!NtWaitForMultipleObjects+0x14:
00007ffb`b572f0e4 c3              ret
0:005> !CLRStack
OS Thread Id: 0x51cc (5)
Child SP               IP Call Site
000000f332cfe408 00007ffbb572f0e4 [GCFrame: 000000f332cfe408]
000000f332cfe548 00007ffbb572f0e4 [GCFrame: 000000f332cfe548]
000000f332cfe588 00007ffbb572f0e4 [HelperMethodFrame_1OBJ: 000000f332cfe588] System.Threading.Monitor.Enter(System.Object)
000000f332cfe680 00007ffb3de5d9fd Microsoft.VisualStudio.TraceCollector.CommonDataCollector+LoggerWrapper.SendMessage(MessageCreator, Boolean)
000000f332cfe730 00007ffb3de4d7ce Microsoft.VisualStudio.TraceCollector.CommonDataCollector.DispatchMessage(MessageCreator)
000000f332cfe7a0 00007ffb3de64dd8 Microsoft.VisualStudio.TraceCollector.CommonDataCollector.OnTestCaseEnd(System.Object, Microsoft.VisualStudio.TestPlatform.ObjectModel.DataCollection.TestCaseEndEventArgs)
000000f332cfea98 00007ffb9d406d93 [DebuggerU2MCatchHandlerFrame: 000000f332cfea98]
000000f332cfedd8 00007ffb9d406d93 [HelperMethodFrame_PROTECTOBJ: 000000f332cfedd8] System.RuntimeMethodHandle.InvokeMethod(System.Object, System.Object[], System.Signature, Boolean)
000000f332cfef50 00007ffb9b513e60 System.Reflection.RuntimeMethodInfo.UnsafeInvokeInternal(System.Object, System.Object[], System.Object[])
000000f332cfefc0 00007ffb9b4e2519 System.Delegate.DynamicInvokeImpl(System.Object[])
000000f332cff010 00007ffb3de4b472 Microsoft.VisualStudio.TestPlatform.Utilities.MulticastDelegateUtilities.SafeInvoke(System.Delegate, System.Object, System.EventArgs, System.String)
000000f332cff070 00007ffb3de4aeea Microsoft.VisualStudio.TestPlatform.Common.DataCollector.TestPlatformDataCollectionEvents.RaiseEvent(Microsoft.VisualStudio.TestPlatform.ObjectModel.DataCollection.DataCollectionEventArgs)
000000f332cff0c0 00007ffb3de648db Microsoft.VisualStudio.TestPlatform.Common.DataCollector.DataCollectionManager.TestCaseEnded(Microsoft.VisualStudio.TestPlatform.ObjectModel.DataCollection.TestCaseEndEventArgs)
000000f332cff130 00007ffb3de5ee59 Microsoft.VisualStudio.TestPlatform.CommunicationUtilities.DataCollection.DataCollectionTestCaseEventHandler.ProcessRequests()
000000f332cff180 00007ffb3de4e57d Microsoft.VisualStudio.TestPlatform.CommunicationUtilities.DataCollection.DataCollectionRequestHandler.b__22_0()
000000f332cff1e0 00007ffb9b54ca96 System.Threading.Tasks.Task.Execute()
000000f332cff220 00007ffb9b503423 System.Threading.ExecutionContext.RunInternal(System.Threading.ExecutionContext, System.Threading.ContextCallback, System.Object, Boolean)
000000f332cff2f0 00007ffb9b5032b4 System.Threading.ExecutionContext.Run(System.Threading.ExecutionContext, System.Threading.ContextCallback, System.Object, Boolean)
000000f332cff320 00007ffb9b54cd35 System.Threading.Tasks.Task.ExecuteWithThreadLocal(System.Threading.Tasks.Task ByRef)
000000f332cff3d0 00007ffb9b54c463 System.Threading.Tasks.Task.ExecuteEntry(Boolean)
000000f332cff410 00007ffb9b5425f2 System.Threading.ThreadPoolWorkQueue.Dispatch()
000000f332cff8a8 00007ffb9d406d93 [DebuggerU2MCatchHandlerFrame: 000000f332cff8a8]

@mayerj the issue reported by @Mertsch is different than what you are seeing. if you are still seeing the issue please log a separate issue.
@Mertsch Is this consistent? the hang is actually while reading messages from socket.
Can you try with latest stable version of VS 15.9.0 once

@acesiddhu Yes it is consistent, I had to disable TIA on all my (10) pipelines and go with TestCategory to filter out long running tests.
Testing again is pretty annoying as I have to run TIA once and have it find tests the second time with appropriate checkins.

@Mertsch unfortunately I am not able to repro this on my side and I feel the dump which tells a hang in socket is actually not a hang but it is waiting for message to come from the parent process. I am saying this because the stack show this is happening while TestRunStarted event which is before the test run starts. From the issue you reported it seems that hang happened after test run complete.
if you are not able to try on build definition is it possible for you to try on your dev machine where VS is installed? Use the runsetting which you shared earlier.


generate_walks takes too long

When my number of nodes exceeds 100,000, it takes several days to complete generate_walks，it is too long! How to deal with large heterogeneous graphs？

Hi,
Sorry, this public version may not handle too large graphs. I'm planning to re-implement a C++/Cython version of the random walk procedure to speedup generating random walks.


Fix cli where env variables are not getting used

Description
When an end-user use env variables for client-id and client-secret for polaris cli, the value of  client-id and client-secret are not being used:
Type of change
Please delete options that are not relevant.

[ x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Verify client-id and client-secret are indeed being used:
./polaris --client-id xxxxx --client-secret xxxxx catalogs list

Then tried the same with env variables route:
export CLIENT_ID=xxxxx
export CLIENT_SECRET=xxxxx
./polaris catalogs list

Without the purposed code change, the catalogs list operation will failed due to the following line of code as it will try to load the client-id and client-secret from command line arguments regardless (https://github.com/MonkeyCanCode/polaris/blob/main/regtests/client/python/cli/polaris_cli.py):
token = PolarisCli._get_token(builder(), polaris_catalog_url, options.client_id, options.client_secret)

Thus generating following error:
Traceback (most recent call last):
  File "regtests/client/python/cli/polaris_cli.py", line 136, in <module>
    PolarisCli.execute()
  File "regtests/client/python/cli/polaris_cli.py", line 47, in execute
    client_builder = PolarisCli._get_client_builder(options)
  File "regtests/client/python/cli/polaris_cli.py", line 127, in _get_client_builder
    token = PolarisCli._get_token(builder(), polaris_catalog_url, options.client_id, options.client_secret)
  File "regtests/client/python/cli/polaris_cli.py", line 85, in _get_token
    raise Exception('Failed to get access token')
Exception: Failed to get access token

With the purposed fixed, it will then read from env variables and produce output similar to following:
{"type": "INTERNAL", "name": "my_catalog", "properties": {"default-base-location": "file:///tmp/1"}, "createTimestamp":<PHONE_NUMBER>684, "lastUpdateTimestamp":<PHONE_NUMBER>684, "entityVersion": 1, "storageConfigInfo": {"storageType": "FILE", "allowedLocations": ["file:///tmp/1"]}}

Checklist:
Please delete options that are not relevant.

[x ] I have performed a self-review of my code
[ x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ x] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[ ] If adding new functionality, I have discussed my implementation with the community using the linked GitHub issue
[x ] I have signed and submitted the ICLA and if needed, the CCLA. See Contributing for details.

Hey @MonkeyCanCode, can you try retriggering the tests?

Hey @MonkeyCanCode, can you try retriggering the tests?

Retriggering the test as sent an empty commit?

Thanks @MonkeyCanCode for the PR. Thanks @eric-maynard for the review.


TextFieldProps.autofocus does not take effect in PopupPropsMultiSelection.bottomSheets, but it does in PopupPropsMultiSelection.menu

Describe the bug
When I set TextFieldProps.autofocus to true when using PopupPropsMultiSelection.bottomSheets, it didn't get focus as I expected, but when I changed it to PopupPropsMultiSelection.menu, it took effect normally.
Expected behavior
I want it to work in PopupPropsMultiSelection.bottomSheets too
version: 6.0.1

I use focusNode, it is work

Finally, I used focusNode to solve this problem.


Update offline-google-analytics related content to workbox-google-analytics

Note: I thought it would be just changing link path, but I think most of this article is now out-of-date with release of workbox.
https://developers.google.com/web/updates/2016/07/offline-google-analytics
Probably add some warning on top "This is obsolete, use workbox [Link to workbox article]",  WDYT?

This was fixed by https://github.com/google/WebFundamentals/pull/5985


Fix linux live data reader

The module file name needs to be the full path for SOS to work and to match the core dump data reader.

No need for an official build yet.


Tool for generating schema type annotations

Our schema classes have generated getter methods for each of the fields:
class Pointer:
    source = so.SchemaField(
        so.InheritingObject,
        default=None, compcoef=None,
        inheritable=False
    )

    # this is auto generated by a meta class in objects.py
    def get_source(schema: s_schema.Schema) -> objects.InheritingObject: ...

This PR adds a tool that generates .pyi files in edb/schema/, which contain definitions for all of these generated getters.

We do have mypy extension that already picks-up these getters, even without .pyi definitions.
Regardless, this .pyi approach is useful because language servers (PyLance) cannot use mypy extension. Now, "go to definition", "show usages" and other code actions work.
There is a few drawbacks:

PyLance will not look at .pyi file when editing associated .py file. They suggest merging the two files.
In this example:subjexpr = constr.get_subjectexpr(ctx.env.schema)

... PyLance will understand that get_subjectexpr returns Expression, but it will infer type of subjexpr to be Any. What?

Because of these drawbacks, https://github.com/edgedb/edgedb/pull/7166 is still useful.

The verdict here is:

we want this, but working better. This would best be done with these .pyi files being actual
.py files in edb/schema/generated/ and the base classes adding the generated type-annotation classes as a mixin.
# edb/schema/pointers.py

class Pointer(..., PointerMixin):
    source = so.SchemaField(
      so.InheritingObject,
      default=None, compcoef=None,
      inheritable=False
    )

# edb/schema/generated/pointers.py
class PointerMixin:
    def get_source(schema: Schema) -> so.InheritingObject: ...

alternatively, we could stop creating these functions at run-time and just generate them in the mixin classes:
# edb/schema/pointers.py

class Pointer(..., PointerMixin):
    source = so.SchemaField(
      so.InheritingObject,
      default=None, compcoef=None,
      inheritable=False
    )

# edb/schema/generated/pointers.py
class PointerMixin:
    def get_source(self, schema: Schema) -> so.InheritingObject:
        self.get_field_value('source', schema)

@msullivan, can you have a look into why mypy is failing? A new realization is that that if I disable our plugin, the internal error disappears. I've also figured that if I disable StructTransformer, it disappears:
https://github.com/edgedb/edgedb/blob/schema-type-annotations/edb/tools/mypy/plugin.py#L73-L82

I've tested out performance impact of this change. Currently, there are a few commits on this PR, starting with the most basic setup and then optimizing from there:

git ref
compile_edgeql_script_webapp_get_movie
compared to master

master
74.8 ms += 2.1 ms
non-significant change

schema-mixins
93.4 ms += 3.4 ms
24.87% SLOWER

getter-specialization
84.3 ms += 2.9 ms
12.66% SLOWER

inlined-getters
79.7 ms += 3.0 ms
6.52% SLOWER

static-field
75.3 ms += 1.6 ms
non-significant change

The static-field approach does generate a lot of code, which could probably be cleaned-up a little without a major impact on performance.


Allow domain names for the gefyra up --endpoint parameter

Currently, you can only use IP addresses.
--endpoint IP:Port

That actually doesn't seem to be a problem.


mem: mlockall pages to prevent swap bypassing storage fault tolerance

Attempt to prevent swap from introducing disk errors into memory by locking all allocated pages before running the replica. On failure, hint to the operator that they should try to disable swap system-wide instead.

@kprotty LGTM, just needs spell checking fixed.


better css and padding and favicon on the markdown pages

Seems like the favicon is now working on the markdown pages...?


[SPARK-5478][UI][Minor] Add missing right parentheses

Test build #26310 has started for   PR 4267 at commit 9fe51cc.

This patch merges cleanly.

Doesn't even need a JIRA, but of course looks fine.

Test build #26310 has finished for   PR 4267 at commit 9fe51cc.

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

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


qr code api

create a simple api to create qr codes

moved to: https://github.com/Commandcracker/CC-packages


handle apostrophe

In sloppy mode, apostrophes ʼ should be converted to single quotes ', also maybe other types of space should be treated as regular spaces...

there are nbsp/#160 that should be preserved as spaces perhaps. Probably \r and \n could be converted to Wylie spaces ("_")


Engine linking and libcrypto interface policy

If any executable loads an engine, the executable already either has libcrypto.so loaded or is statically linked against libcrypto.a. Anyway it already has all libcrypto symbols present.
All in-tree engines are linked against libcrypto.so now. As a result, a diamond dependency is introduced. When such an engine is then loaded by an executable which is statically linked against libcrypto.a, ld will insist on loading libcripto.so, despite the executable already having all necessary symbols. When the executable is statically linked, shared object for libcrypto.so is usually not built, and ld won't find it.
Diamond dependency also allows libcrypto.so.3 to load engines built against libcrypto.so.1.1, and vice versa. When the engine is used the executable segfaults. I've faced this issue when experimenting with node.js-v17 which uses openssl-3, and gost-engine. Removing link-time dependency on libcrypto.so fixes diamond dependency problem for gost-engine. I will create a pull request with the proposed fix for in-tree engines.
The discussion with gost-engine maintainers showed that the problem has a second aspect. If libcrypto modifies any of its exported interfaces (structures and function signatures), the engine will still load, but may encounter hard-to-debug issues. So the second question is whether a policy on libcrypto exports can be enforced, so that changed to their signatures are never changed, only added or removed. That is if a(b, c) need to become a(b, c, d), then instead a(b, c) is removed and a2(b, c, d) is added.

@beldmit, @vt-alt

Engines must be linked with the same libcrypto as the calling application, for exactly the reasons you mention. Anything else is an undefined scenario.

Or yeah, not linking it at all with libcrypto and relying om the symbols to already be present works too... on platforms that support this, which not all do.

Engines must be linked with the same libcrypto as the calling application, for exactly the reasons you mention. Anything else is an undefined scenario.

Yep. And I propose a way to ensure this exact behavior on systems that have ld linker.

Marking as inactive, to be closed at the end of the 3.4 dev cycle, barring further input


Add ability to remove close button

Description
I don't update my EdgeFrfox frequently so I was stuck with v24.2.4. In this version and before, Linux support for the hide tabs toolbar tweak wasn't there. My Firefox window looked like this:

In v24.2.13, the next version, where Linux support was properly added for the hide tabs toolbar tweak, my Firefox window now looks like this:

It would be nice to have a feature to hide the close button and the drag space after it. I don't want to delete the entire Linux/GTK part of hide-tabs.css just to remove my close button.

Now the window controls can be hidden using the uc.tweak.hide-tabs-bar.no-window-controls tweak (either enabled with the existing tweak or on its own):


Libraries using @prismicio/types with different versions results in type errors

Versions

@prismicio/types: v0.1.15
node: v14.17.0

Reproduction
This is difficult to reproduce so I'll share a general proof.
enum A {
	foo = "foo",
}

enum B {
	foo = "foo",
}

const val: A.foo = B.foo;
// [tsserver 2322] [E] Type 'B' is not assignable to type 'A'.

Despite A.foo and B.foo having equivalent literal values, TypeScript treats them as distinct types.
In the types library, this scenario happens when dependent libraries resolve different @prismicio/types installations. More specifically, when the RichTextNodeType enum exists in more than one location, TypeScript sees them as incompatible.
The following node_modules structure produces this:

node_modules/@prismicio/types
node_modules/@prismicio/mock/node_modules/@prismicio/types

This occurs when packages resolve to different versions of @prismicio/types, thus requiring dependent packages to install their own version of @prismicio/types.
Steps to reproduce
I'm having trouble finding reproducible steps that cause this, but indeed it happens:

What is expected?
Type errors do not occur when multiple versions of @prismicio/types are installed. Type errors due to structural differences between versions are expected.
What is actually happening?
Multiple versions cause enum incompatibilities. See the above example errors.

Yeah, not using an enum is not ideal but I think using an object like above is the next best option. On the up side, it might take up less bytes than the generated enum output.
Are you good moving forward with this change?


upgrade to node 16

Issue
-

:warning: Break Change
No

Description
Updating all templates to ship with Node 16 image references instead of old Node 14.

@danielsuguimoto good catch! Have updated the PR accordingly.


Stuck on red screen.

After this package was brought inline with upstream, it no longer works for me. The screen turns completely red except for the cursor, which is still visible and usable. Typing does not bring up the indicator or allow me to unlock, I have to reboot using the power button to get back in. It doesn't seem to be related to any config option, as simply running swaylock from a terminal causes the issue. This is happening on both my personl and work systems, both running Arch Linux, one with intel+nvida graphics and one with AMD, so it doesn't seem to be related to graphics drivers. I have tried with both sway (and wlroots) stable and sway-git (wlroots-git), issue occurs in both scenarios.
Output of swaylock -d 2> log.txt:
2022-07-17 12:57:27 - [main.c:1834] Parsing CLI Args
2022-07-17 12:57:27 - [main.c:1881] Using ext-session-lock-v1
2022-07-17 12:57:27 - [main.c:688] output name is HDMI-A-1
2022-07-17 12:57:27 - [main.c:344] Surface for output HDMI-A-1 ready
2022-07-17 12:57:27 - [main.c:688] output name is eDP-1
2022-07-17 12:57:27 - [main.c:344] Surface for output eDP-1 ready
ext_session_lock_surface_v1@21: error 0: session lock surface has never been configured

That’s weird; I’m using it on Alpine Linux v3.16 with Sway 1.7 and don’t experience this problem.

That’s weird; I’m using it on Alpine Linux v3.16 with Sway 1.7 and don’t experience this problem.

Have you tried sway 1.8? (master branch)

I just updated to sway-git master today and ran into this problem. Seems like something must have changed within the past 2 - 3 weeks (last time I updated should be less than 1 month ago)

To confirm @PeterCxy comments: I just rolled back from -git versions of wlroots and sway and now red screen problems and also issue #5 are resolved for me.

Any update? I'm using the latest master sway and swaylock-effects of this repo. Still run into this issue, making it totally unusable.

@oxalica The only solution I've found is using the release version of sway and wlroots.

@oxalica The only solution I've found is using the release version of sway and wlroots.

Release version of sway has not impl ext-session-lock feature and thus hide this issue but not resolve it.
But the reason why I'm seeking for this fork is just because it claims to include the ext-session-lock impl https://github.com/jirutka/swaylock-effects/commit/1d1c75b6316d21933069a9d201f966d84099f6ca . Without it (like the previous fork https://github.com/mortie/swaylock-effects), keyboard input can bypass the lock when display is turned off, making the lock effectively useless. This would not be better than the original unmaintained fork to me.

No progress yet. :( I don't where exactly is the problem and since Sway 1.8 has not been released yet, thus it's not in stable Alpine, it doesn't bother me for now. I hope that, meanwhile, someone more skilled in Wayland development will help here. If not, I will try to fix it myself, but since I have tone of others things to do, it's unlikely to happen before a) it's causing problem on my system, or b) someone donate me to fix it.

I had this same problem of a dark screen, active pointer, and no way to unlock on the current jirutka master branch after I rebuilt my dwl with wlroots 0.16.0. Upstream swaylock and mortie/swaylock-effects do not produce this issue. I also did not have this issue when using wlroots 0.15.1.

Apparently using --screenshots option, the problem no longer occurs.

Please try out patches from #20, that should fix this issue.


Reduce re-renders on header back button

First PR related to Improve performance for React Navigation
Although the HeaderBackButton is a PureComponent - we are binding a new function on each re-render which still causes a rerender of HeaderBackButton

Thank you for the PR


Revert "Remove x-old-ref fields"

This reverts commit 4dd318810e9dcdcb7ea5be8239fec13700ffaf7a, which breaks documentation for responses where the response is an array of some schema.
@thomasleese doesn't remember why we originally made this change. It's possible we might need to fix the original problem in a different way, but I think we should roll this back as it limits what you can put in an OpenAPI spec.
Vanilla widdershins renders a table like this:

name
type
required
description

anonymous
[MyThing]
false
Description of my response

Followed by some child fields, which we intentionally strip out
(see https://github.com/alphagov/widdershins/pull/4)
Despite this table being mostly indecipherable nonsense, it does at
least have a working hyperlink to the actual MyThing schema
documentation.
The alphagov fork currently breaks this link, and it renders the type as
[object], thus completely destroying the illusion that there is any
meaningful information in this table.

Thinking about this some more, I wonder if we needed this in combination with #4 to actually get our intention to #4 to work - or perhaps this was a first attempt and then we decided to settle with #4 as a more complete solution.

The code in #4 is still working with this reverted, so hopefully it was the latter!


Clarified scalar operations

Added explicit mod r and moved the scalar multiplications before additions to avoid op-priority confusions.

LGTM.
Just a note. maybe we are encouraging people to the x*y + z operations and then mod r?? which I don't think is a good idea in practice (it should be ((x * y mod r) + z) mod r ??).
Another option would be to define that all operations between scalars happen in the multiplicative group mod r?? That would be the more "mathematically correct" description, although I’m uncertain if it would be the clearer one.
I'm definitely overthinking this though. I think we should move with merging this 😄

Just a note. maybe we are encouraging people to do the x*y + z operations and then mod r?? which I don't think is a good idea in practice (it should be ((x * y mod r) + z) mod r ??).

It's true, but it also becomes cumbersome. Your suggestion for a note somewhere that all calculations in that field is mod r. I simply wanted to avoid someone leaving a larger scalar and then trying to multiply that to the elliptic curve point; result would be validate but would be very inefficient. A simple mod r at the end indicates it's ok to apply the modulo at the end of the equation (confirming my interpretation of the spec).

Multiple reviews and approvals, merging.


Azure Monitor Queries: invalid date string format in MetricsClient

Fixes #40047

@microsoft-github-policy-service agree company="Microsoft"

Hi @wps0, thank you for your contribution. Could you please add a test showing this change?


Add license header plugin

Add Maven License Plugin to pom.xml
Add a step to check the license headers in continous-integration.yml.
Apply the license plugin to all Java sources.

We will take just the maven/ci bits and ignore rest of the files in the PR.

Hi.  I ended up cutting and pasting the plugin and ran it on my own branch to merge.  I didn't add to the CI, but added info in the readme what to do as the local compile will fail - must like the java-format plugin.
Thanks!
merged as bcb559a82ca15d02ca9078c8d10cb59d47276822


Convert from OBJ + MTL + textures to gLTF

As the title states, is there a way to use your library to convert from an Obj file along with an mtl file and textures to the gLTF format that conserves the tetxtures?

Hi!
Thx for your question.
This feature is not supported yet but is planned for a future release (https://github.com/Esri/pyprt/issues/8).
The only way at the moment would be to write your own CGA rule (RPK) to apply those materials and textures to the initial shape. To do so, you'll need CityEngine.

Thank you for the reply!


Can't unbox heterogeneous list

Code:
@njit
def test(nums):
	return

test([sum(2**i for i in range(64) if i%2==j) for j in range(2)])

Output:
C:\Users\e017200\AppData\Local\Programs\Python\Python38\lib\site-packages\numba\ir_utils.py:2030: NumbaPendingDeprecationWarning:
Encountered the use of a type that is scheduled for deprecation: type 'reflected list' found for argument 'nums' of function 'test'.

For more information visit http://numba.pydata.org/numba-doc/latest/reference/deprecation.html#deprecation-of-reflection-for-list-and-set-types

File "<stdin>", line 1:
<source missing, REPL/exec in use?>

  warnings.warn(NumbaPendingDeprecationWarning(msg, loc=loc))
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: can't unbox heterogeneous list: uint64 != int64

@k128 I am assuming this is now resolved, please feel free to re-open if needed.

TypingError: Failed in nopython mode pipeline (step: nopython frontend)
Unknown attribute 'choices' of type Module(<module 'random' from '/opt/conda/lib/python3.10/random.py'>)

TypingError: Failed in nopython mode pipeline (step: nopython frontend) Unknown attribute 'choices' of type Module(<module 'random' from '/opt/conda/lib/python3.10/random.py'>)

@said-ml  the problem you post has nothing to do with othis issue. Please be more mindful when posting in future and make sure you submit comments to the correct issue.
None the less to answer your question: choices doesn't appear to be implemented: https://numba.readthedocs.io/en/stable/reference/pysupported.html#random

@esc you're right ,  i will take advice in acount.


Connection pool will not honor maximum connections if all connections in use

It seems to me that the connection pool will not honor the maximum connections limit, because every time a connection is put back into the queue, the count is decremented.  So take the following scenario:

max_connections is 5
Queue is empty five calls in a row: _created_connections.current() is now 5
All five connections are put back into the pool: _created_connections.current() is now 0
All five connections are taken out of the queue: _created_connections.current() is now 0
Five new connections can be pulled from the pool.

    def get(self):
        try:
            return self.q.get_nowait()
        except Empty:
            if self._created_connections.current() < self.max_connections:
                conn = self.connection_class(**self.connection_kwargs).conn
                self._created_connections.incr()
                return conn
            raise

    def put(self, connection):
        self.q.put(connection)
        self._created_connections.decr()

I guess you are right. I only had in mind the process of creating connections until reaching the upper limit, did not think about this aspect. You are welcome to submit a pull request for this issue. Thanks for spotting this one!

I may do that - I fixed the bug on our repo, but also added an option to provide multiple connection arguments to do a simple round-robin against all of the cluster endpoints and to add some level of failure-recovery if one of the endpoints is down.  It needs test cases, and I'm not sure if you want all that functionality in there as well.

I'd rather not include the multiple connection feature, as I believe remodel should be a simple library and this will deviate a bit from my initial concept. Just as curiosity, why do you need extra connections for cluster endpoints? From my understanding, RethinkDB clusters work as expected by connecting to a single endpoint and running the queries against it.
Nonetheless, it would be great to fix the bug this issue is about, together with a handful of tests!

My thinking was twofold:

If our connection node dies, we want to try to fail-over to another node
Basic load-balancing as per here: https://github.com/rethinkdb/rethinkdb/issues/281#issuecomment-99697293

That said, there are a number of alternatives:

Actual fail-safe load balancer
Fetch addresses of other nodes and use them as a backup

I think for the purposes of the library, a load balancer probably would fit the bill philosophically.
I'll go back to my patch and make a few changes/tests.

I didn't invest that much effort in the connection pool, but I see some very valid points raised by you and in the issue you've linked to. If you put together a useful and working snippet, I will be more than happy to integrate it!

Cool.  I'll try to finish it up this week or early next!

I was able to do more investigation of this as well.  It seems like RethinkDB supports running a local instance in cluster proxying mode, which I think is a better solution than adding a round robin.  I still need to get to the patch.

Closing this one for now. Feel free to reopen the issue if you managed to find time for this!


Getting 500 error for api requests for some packages

For instance https://pub.dartlang.org/api/packages/analyzer. This happens when hitting it directly or just trying to do a pub upgrade.

Looks like this is fixed


feat: add ability to wrap execution of server actions

Adds the ability to wrap the execution of server actions by passing in a "wrapExecution" function when creating the safe action client where you can inject things like logging, tracing or performance monitoring. To do these things you also want the name of the action, which I've passed in via the utils object in this PR. This is backwards compatible without breaking changes.
Optimally I think this kind of functionality would best be implemented by changing the current middleware approach to one more similar to what playwright does with their fixtures but that would be a breaking change.
Written blind to get feedback before committing too much time, this has not yet been tested.
Example usage:
const performanceAction = createSafeActionClient({
  async wrapExecution(serverCode, parsedInput, ctx, actionName) {
    const t1 = performance.now();
    const res = await serverCode(parsedInput, ctx);
    const t2 = performance.now();

    // Prints 'Action "someAction" took 1000 milliseconds.'
    console.log(`Action "${actionName}" took ${t2 - t1} milliseconds.`);

    return res;
  },
});

const someAction = performanceAction(
  {
    name: z.string(),
  },
  async (input, ctx) => {
    await new Promise((resolve) => setTimeout(resolve, 1000));
  },
  {
    actionName: "someAction",
  }
);

Version 7 will include a much more powerful middleware system, kinda like the tRPC one. It should also cover all the use cases described in this PR. Please let me know what you think about it, thank you! Discussion here: #88


Training

I findthat the pre-training and fine-tuning time for this work is very long. Is there any way to speed up the training time?

You could try to use automatic mixed precision (amp) https://pytorch.org/blog/pytorch-1.6-released/


[ru] Localize Service into Russian

Description
This Issue is to localize Service into Russian.
Checklist before opening your PR (put x in the checkboxes) — note that you have to add this checklist to your PR and check all the boxes for it to be approved.

[] :warning: This PR uses the dev-ru branch as a source/target.
[] This PR does not contain plagiarism

don’t copy other people’s work unless you are quoting and contributing it to them.

[] I have signed off on all commits

signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

ETA: TBD
Useful links:
Как внести свой вклад
Руководство по стилю
Slack channel
Telegram channel

I want to work on this term . please assign me this issue ..

hey are you still working on this issue  do you need any help


World switcher does no longer respond in both game and start screen.

Type
Crash or hang
Operating System
Windows
OS Version
Windows 10
Bug description
Since yesterday I cannot switch worlds anymore. Help
Screenshots or videos
Difficult to screenshot as there just happens nothing
RuneLite version
<IP_ADDRESS>
Rev 216
2.6.8
Logs
No response

It works for me
I would delete both runelite and the plugin and download it all over again making sure to download the x64 bit edition versus the x32 bit edition

The issue tracker is not a support forum. Please join the Discord for questions or support.


Update README-zh.md

/lgtm
Didn't check if the full page was in sync.

@Yanping-io - can you look at the conflicting lines, and we can take another look.

/close
No change in this PR.


Patch to fix typo in help output

Below is patch code-typos.patch from Debian package <IP_ADDRESS>-10.  It fixes a minor typo in the help output.
KEN
# Description: Fix typos in the code.
# Author: Kenneth J. Pronovici<EMAIL_ADDRESS>Index: ncompress-<IP_ADDRESS>/compress42.c
===================================================================
--- ncompress-<IP_ADDRESS>.orig/compress42.c
+++ ncompress-<IP_ADDRESS>/compress42.c
@@ -877,7 +877,7 @@ Usage: %s [-dfvcVr] [-b maxbits] [file .
             If -f is not used, the user will be prompted if stdin is.\n\
             a tty, otherwise, the output file will not be overwritten.\n\
        -v   Write compression statistics.\n\
-       -V   Output vesion and compile options.\n\
+       -V   Output version and compile options.\n\
        -r   Recursive. If a filename is a directory, descend\n\
             into it and compress everything in it.\n");

can you please send these as pull requests instead of pasting them inline ?

Sorry, it wasn't entirely obvious the best way to provide patches.  Github doesn't allow attachments.  I haven't done a pull request before, but I'm sure I can figure it out.  I'll try to do it this weekend, otherwise it will be sometime next week.

This patch is not included in the pull request referenced above.  It looks like it's already been applied (not sure when).

see commit 148da0e0dcb7ff7833d4f9b2cd60cf0c8f6759b7


Guard against fetching local IP ranges

Anything that resolves to local IP ranges should not be allowed for actor fetching.

it would be good to have a development option to allow this, for testing federation for example

GitLab has implemented a logic similar to this, where by default they block domains resolving to local networks, but having a whitelist for both domains and IPs (domains - regardless of the actual target IP)
https://gitlab.com/gitlab-org/gitlab-foss/-/merge_requests/30350

httpx.Client is used in about a half dozen places, and HttpSignature in a half dozen more. (Honestly, I was expecting more.)
My suggestion is to make an httpx wrapper client that encapsulates all the fedi HTTP client extensions. So far, that's:

Signed requests (either when asked, or defaulting to the system actor)
Locking out IP ranges (I would suggest defaulting to every range mentioned in https://en.wikipedia.org/wiki/Reserved_IP_addresses )

Future additions might include:

HTTP/2 support
Async support
Tracing pass through
Tor Onion/IPFS/etc support

My plan was to instead do all of this in https://github.com/jointakahe/taktivitypub so it's reuseable

Oh. Did you want #679?

If you can make #679 roughly net-neutral in terms of code (i.e. remove the existing signing in the same PR) that's good to me. I don't know what the timeline would be for adding into the other thing.


Backwards compatibility for EditorsPick model move

This was previously referenced from the project template and it has now been moved to its own module:
https://github.com/torchbox/wagtail/blob/stable/1.0.x/wagtail/project_template/search/views.py#L24
Anyone who created their site on 1.0 and upgraded to 1.1 will get this error: ImportError: cannot import name 'EditorsPick'. It's an obvious fix, but annoying as everyone will get it.
Ideally, we should come up with a backwards-compatible fix which shows a DeprecationWarning and remove it completely in 1.3.

+1 - I didn't realise we were missing this. We definitely want this to go through the proper deprecation timeline.

What was the conclusion on this? Did we decide that it was better to let this fail in an obvious way, rather than put in hacks to accommodate references to a non-existent wagtailsearch.EditorsPick model?

Closing as discussed - there's no clean way to preserve backwards compatibility when moving a model between apps. The release notes now cover the changes that need to be made when upgrading.

link to the release notes for the lazy: http://docs.wagtail.io/en/v1.1/releases/1.1.html?highlight=editorspick#promoted-search-results-no-longer-in-wagtailsearch


"not well-formed" ExpatError.

I think there's something wrong with my reading list? I'm running osx 10.8.2
ExpatError: 'not well-formed (invalid token): line 21398, column 18'
function init in readinglistlib.py at line 17
xml = plistlib.readPlist(pipe)
function readPlist in plistlib.py at line 78
rootObject = p.parse(pathOrFile)
function parse in plistlib.py at line 406
parser.ParseFile(fileobj)
copy output
Program exited with code #1 after 0.50 seconds.

I got this error as well. The workaround is to open Bookmarks.plist in an editor and go to the location in the "invalid token" error line -- in your case "line 21398, column 18". Whatever text is there, delete it, then run the script again. Repeat until it stops erroring.


Refactor classes to class directive

Currently we're calculating all classes at once using a helper function. While this offers a great dev experience, it might not be the best in terms of performance. Hence we should refactor our classes to use the class directive class:myclass={myclass} and the shorthand class:myclass where possible.

@KilDesu Since you've worked on this, I'm closing here. :)

#46

46

#46


Verify dependency arrays for usages of useCallback

Discussion in https://github.com/stevekuznetsov/avalanche-forecast/pull/14#discussion_r1047927359

I've fixed all of these!


i18n(ko-KR): update link-buttons.mdx

Description

update link-buttons.mdx
#2372

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

Locale
File
Note

ko
components/link-buttons.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.


Irrelevant to Noto just some issues im running into programming android apps, close if needed

I don't know if your familair with android studio but i have some problems im having and i cannot find anywhere how to fix them

Gradle never stops syncing it has been hours and still syncing
Android project list also never loads
No modules show up when trying to add a configuration to run the programs

Can you post your build.gradle file?

Opening my project list with Project works fine but with Android it loops on loading

Are you connected to the Internet? Maybe, it's trying to download some stuff.

Im on mobile data currently, i dont see why it needs 2+ hours to install but i wouldnt know im still new

Are you connected to a VPN?

no, just normal 4G

Have you tried connecting to a local Wi-Fi network?

theres no wifi networks around here, on a mountain for 5 days. although i'am connected to another phone's hotspot

I don't know what the problem could be, to be honest, but my guess is that it needs to download some stuff.
I'd recommend trying again when you have access to a local Wi-Fi network.
Sorry

Could be downloading alot of files cause of first time use, my biggest problem currently is probably that i cant find a module in configuration editer, I have no way of running/debugging or exporting as an APK

I think it needs to finish syncing first, then you'd be able to create a configuration. Could you try disconnecting from the network and sync again?

It stopped syncing but im running into this now

I tried resyncing but same error

Try this: File → Invalidate Caches → Invalidate and Restart

ill try some other fixes but for now im still experiencing these bugs, might be fixed when im home installing it on my computer with ethernet but i dont know what to do currently

I'd say leave it until you're back home.

ill watch those tutorials you mentioned but yeah ill do it at home

Okay then


⚠ SceneTime has degraded performance

In ac21d4d, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1007 ms

Resolved: SceneTime performance has improved in 60ba703 after 6 minutes.