Execution failed for task ':transformClassesWithJarMergingForDebug' in Ionic Beta 11 for Android

From @dalezak on September 8, 2016 19:43
Since upgrading to Beta 10 or Beta 11, I can no longer build the app for Android 23, getting the error:
* What went wrong:
Execution failed for task ':transformClassesWithJarMergingForDebug'.
> com.android.build.api.transform.TransformException: java.util.zip.ZipException: duplicate entry: com/google/android/gms/analytics/CampaignTrackingReceiver.class

I've tried to ionic platform rm android and then ionic platform add android, but still no luck.
ionic platform add android
Adding android project...
Creating Cordova project for the Android platform:
Path: platforms/android
Package: xxx.yyy.zzz
Name: app
Activity: MainActivity
Android target: android-23
Android project created with<EMAIL_ADDRESS>Installing "cordova-fabric-plugin" for android
Installing "cordova-plugin-app-version" for android
Installing "cordova-plugin-console" for android
Installing "cordova-plugin-device" for android
Installing "cordova-plugin-email" for android
Installing "cordova-plugin-google-analytics" for android
Installing "cordova-plugin-googleplayservices" for android
Installing "cordova-plugin-inappbrowser" for android
Installing "cordova-plugin-ios-longpress-fix" for android
Installing "cordova-plugin-ios-no-export-compliance" for android
Installing "cordova-plugin-networkactivityindicator" for android
Installing "cordova-plugin-spinner-dialog" for android
Installing "cordova-plugin-splashscreen" for android
Installing "cordova-plugin-statusbar" for android
Installing "cordova-plugin-x-socialsharing" for android
Installing "ionic-plugin-deeplinks" for android
Installing "ionic-plugin-keyboard" for android
Running command: .../010_add_platform_class.js
add to body class: platform-android

Any idea what is causing this problem?
Copied from original issue: driftyco/ionic#8018

From @dalezak on September 8, 2016 21:19
Hmm, looks like the issue might have been related to the Cordova plugin cordova-plugin-googleplayservices.
Simply removing the plugin from config.xml wasn't enough, I also needed to remove it using cordova plugin rm cordova-plugin-googleplayservices.
And then re-created the Android project using ionic platform rm android and ionic platform add android, and afterwards was able to compile the app using ionic build android.


Update docs for fetching token using container

Sorry, another pull request for documentation update, applying the changes proposed by @owenvoke (Thanks for the improvements)

Moved the image to alpine to reduce image size
Change CMD to only require a single command

Thanks for simplifying this!


Test plan in multiconfig bug and some UI review fixes

Hi @RanjanarMS, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor. If you're full-time or an intern, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

Approved


samples: hid-mouse: protect report buffer

Use the message queue to pass the new report from the input callback, and use a semaphore to protect the report buffer until it is transferred to the host.
Resolves: #71299

@fabiobaltieri please review


Hybridisation using firedrake.SCPC

Pulling out the reimplementation of hybridizationPC using SCPC from #152
This implementation will work with the complex-proxy module as well as with >2 fields, whereas HybridizationPC is restricted to plain HDiv x L2 and only 2 fields.
Required:

[x] Use ufl.replace to break the form rather than relying on form_mass and form_function
[x] Allow the HDiv space to be at any index
[x] Combine the complex-proxy implementation with the real implementation
[ ] Test for checking that the PC solves the real LSWE in one iteration
[ ] Test for checking that the PC solves the complex-proxy LSWE in one iteration

Additions (probably for later PRs as needed):

[ ] Deal with boundary conditions
[ ] Deal with extruded meshes with tensor product elements
[ ] Deal with nullspace
[ ] Add a view method


Failure running on a read-only filesystem

Hello @walkor
Attempting to run on a read only filesystem fails as follows.
fopen(/home/dev1/proj1/vendor/workerman/workerman/../_home_dev1_proj1_test.php.pid.lock): Failed to open stream: Read-only file system in file /home/dev1/proj1/vendor/workerman/workerman/Worker.php on line 640

Workerman[/home/dev1/proj1/test.php] start in DEBUG mode

file_put_contents(/home/dev1/proj1/vendor/workerman/workerman/../workerman.log): Failed to open stream: Read-only file system in file /home/dev1/proj1/vendor/workerman/workerman/Worker.php on line 2156

file_put_contents(/home/dev1/proj1/vendor/workerman/workerman/../_home_dev1_proj1_test.php.pid): Failed to open stream: Read-only file system in file /home/dev1/proj1/vendor/workerman/workerman/Worker.php on line 1324

PHP Fatal error:  Uncaught Exception: can not save pid to /home/dev1/proj1/vendor/workerman/workerman/../_home_dev1_proj1_test.php.pid in /home/dev1/proj1/vendor/workerman/workerman/Worker.php:1325

Stack trace:
#0 /home/dev1/proj1/vendor/workerman/workerman/Worker.php(556): Workerman\Worker::saveMasterPid()
#1 /home/dev1/proj1/test.php(31): Workerman\Worker::runAll()
#2 {main}
  thrown in /home/dev1/proj1/vendor/workerman/workerman/Worker.php on line 1325

Is there a way to turn off dependency on pid/log file usage?
I know I can configure pidFile/logFile to some other writes me space, but can I just turn this off completely to be able to operate in a read only environment?

can I just turn this off completely to be able to operate in a read only environment?

Sorry, It not support turn off


Footer misaligned depending on page height

iOS 15.0.2 a717aa79555c2ab282360f555cf4a29851cf3130

Image 1
Image 2

I can't reproduce this bug, even the hidding of Safari's address bar


Generic.CodeAnalysis.JumbledIncrementer: prevent false positives / check with higher accuracy

The following issues were found during the review of PR #385.
Describe the bug
While the sniff works well for the most common case of a variable being incremented/decremented in the third expression in the condition of a for loop, it does not take other possible operations involving variables into account, making it highly inaccurate for all but the most common case.
Code sample
for ($i = 0; $i < 20; $i++) {
    for ($j = 0; $j < 20; $j = $i + 10) { // $i is not being written to, so is not being jumbled here.
    }
}

$j = 0;
for ($i = 40; $i > 20; $i -= $j) { // $j is not being written to here ...
    for (; $j < 20; $j++) { // ... so $j is not being jumbled here.
    }
}

To reproduce
Steps to reproduce the behavior:

Create a file called test.php with the code sample above...
Run phpcs test.php --standard=Generic --sniffs=Generic.CodeAnalysis.JumbledIncrementer
See error message displayed

2 | WARNING | Loop incrementor ($i) jumbling with inner loop (Generic.CodeAnalysis.JumbledIncrementer.Found)
8 | WARNING | Loop incrementer ($j) jumbling with inner loop (Generic.CodeAnalysis.JumbledIncrementer.Found)

Expected behavior
No warnings to be thrown when a variable seen in the third expression is not being written to.
Additional context
Here are some additional code samples for inspiration as each of these will also result in a false positive:
for ($i = 0; $i < 20; $i++) {
    for ($j = 0; $j < 20; $j[$i]++) {
    }
}

for ($i['key'] = 0; $i['key'] < 20; $i['key']++) {
    for ($i['otherkey'] = 0; $i['otherkey'] < 20; $i['otherkey']++) {
    }
}

for (; $this->propA < 20 && $this->propB > 0; $this->propA++, --$this->propB) {
    for (; $this->propC < 20; $this->propC++) {
    }
}

$var = 50;
for ($i = 0; $i < 20; $i = functionCall($var)) {
    for (; $var < 20; --$var) {
    }
}

$i = 0
for ($varA = 0, $varB = 0; $i < 20; $varA++, $varB++)) {
    for (; $i < 20; $i = $varA + $varB)) {
    }
}

Please confirm:

[x] I have searched the issue list and am not opening a duplicate issue.
[x] I confirm that this bug is a bug in PHP_CodeSniffer and not in one of the external standards.
[x] I have verified the issue still exists in the master branch of PHP_CodeSniffer.

Adding one more false positive from #385:
$label = 'label';

for ($i = 1; $i <= 10; $i++, print $label) {
    for ($j = 0; $j < 5; $j++, print $label);
}


Emit table of return sorts

Part of: https://github.com/runtimeverification/llvm-backend/pull/905
In #905, we are implementing a Python binding for the backend's function evaluator: given a function label and list of argument Patterns, construct runtime terms for the arguments, evaluate the function with the given label, and return the result as an AST pattern.
To safely reify the runtime term produced by the function call to an AST pattern, we need to know its sort (so that the machinery in #907, #908 can be used correctly). In some places in the bindings, we have to require that callers provide a sort when reifying terms back to patterns. However, when calling a function, the label of the function determines precisely the correct sort to use.
This PR emits a new table of global data into compiled interpreters that maps tags to declared return sorts, along with a function that abstracts away indexing into this table. This change is similar to (but simpler than) an existing table of argument sorts for each symbol that we already emit.
Testing is handled by binding the new function to Python.

LGTM! One question, though, as we're emitting more code, how does this modification affect the performance of the uses of our generated runtime lib?

I'd be very surprised if this caused any kind of meaningful performance change; the new code-generation steps is one new O(symbols) among a set of similar steps and the data getting emitted is only a single pointer for each symbol (the sort names are already present in the binary).


Image Tag does not work if the file name contains '()'

Example:
![image](sometext(sometext)sometext.png)

[link](sometext(sometext)sometext.png)

Convert to html image:

sometext.png)

Convert to html link:

link

Thanks for the report. It looks like you found a bug.
Sigh. Another one of the differences between links and images. Sometimes I think we should just use the same partial regex for both, but then there is some behavior of links we don't want to replicate elsewhere (for example see #590).

If/when we ever get 3 out the door, we can probably make that change and remove spaces from the original links.


Provide option to add terminal in explorer

It would be great if I can just right click on a folder (its background) and choose "Open in Terminal". Which simply opens the Windows Terminal with the path. Keep it simple. Just one context menu entry with the icon and text. No complex menu or additional options. I could add my own shell entry in regedit. But this should not the solution. The idea is to provide this option in the Terminal settings (add and remove; maybe just in user scope which requires no admin). Another problem is that the new Windows Terminal folder path (apps) has a dynamic folder (path) name. The app version is included. This makes it hard to add a custom shell entry. Even when using just the "wt" command instead of the path. I get an error (0x80070005). Anyway, it would be great if you provide an official solution.

Workaround: My current reg file:
Windows Registry Editor Version 5.00

[HKEY_CURRENT_USER\Software\Classes\Directory\Background\shell\psterminal]
@="Terminal"
"icon"="C:\\Windows\\System32\\cmd.exe"

[HKEY_CURRENT_USER\Software\Classes\Directory\Background\shell\psterminal\command]
@="wt -d ."

Icon is still set to cmd because the path to this Windows Terminal contains the version in name like:
C:\Program Files\WindowsApps\Microsoft.WindowsTerminal_1.15.2712.0_x64__ect

This should work today; we offer a right-click option on all folders, selected items that are folders, and the desktop background.
I suspect you're seeing an issue where it's just not working properly?
What version of Windows are you running? What other shell extensions do you have installed?
(I'm gonna move this discussion over to /dup #14185)

Ok, it seems that the explorer needs to be restarted (e.g. by killing process in task manager and run again or just reboot PC). I had weird behavior. Two instances of explorer was in task manager. In some paths it worked in other not. Anyway, sorry. A PC reboot (explorer restart) is the solution for everything. The IT Crowd didn't lie. ^^

Ha! Glad it helped!


Maximum size

Is it possible to make a survey show as big as possible on mobile browser screen?
I mean, I can play with JS/CSS, detect mobile phones and try to do this myself, but maybe something could be done with parameters? It looks really small on mobile, because it takes the entire screen width and the questions become very small and hardly seen somewhere in the corner. This is how it looks on my mobile Chrome.

It looks like font size is too small. Ordinary on devices survey looks like on the screenshots in the https://github.com/surveyjs/surveyjs/issues/864 issue. I have no ideas why it looks so. We need a link (example code) to reproduce the issue on our side.

I never changed the fonts, I mean I use Bootstrap according to your guide.
Here is the sample link:
http://survey.clinicrowd.xyz/survey.php?id=10&lang=en&nickname=clinicrowd
Thanks in advance!

You need to add meta viewport tag to the head of your site:
<meta name="viewport" content="width=device-width, initial-scale=1">

You can find more info in various articles like this - https://www.w3schools.com/css/css_rwd_viewport.asp

Great, thanks!


Stock Price Checker Proxy should not be tracking changes to cache/db

I remixed the FCC Stock Picker API referenced in this forum post and notice the disk space has been maxed out.  Glitch only allows about 200 MB of disk storage and that includes the .git folder too.  Glitch automatically commits every single change made to the repo which would include the cache/db.  I suggest adding a .gitignore file to the project and ignoring the cache/db so it does not create a commit every time it gets updated.
The actual Glitch project in question is located here
I would like to fix this issue, but I do not see a repo under freeCodeCamp for the proxy.  It appears the project was created by @Em-Ant but I could not find a stock-checker-proxy repo in his GitHub account or the freeCodeCamp account.  We should really have this under the FCC umbrella anyway.
NOTE:  A quick fix is to have someone who has access to the Glitch project to try and run git prune or git gc.

@RandellDawson i can do it . let me try...

Thank you for catching this @RandellDawson, and @Em-Ant for jumping in to work on this.
Please let me know if there's anything I can do to help out or if I can add anyone to the fCC Glitch team for future access.

@scissorsneedfoodtoo This entire API should be in a GitHub repo owned by freeCodeCamp shouldn't it?  Then changes can be made to the repo and then pushed to Glitch.

@RandellDawson, at the moment it only exists on Glitch. I'll create a repo for it so future changes will be easier.

i added a simple cache cleaning (only invalid items) on app restart. Unless there are other problems, i think we can close this issue

Awesome! Thank you for the quick fix @Em-Ant. I'll close this and create a repo like @RandellDawson suggested.


Update playback example

Looks like this example fell behind in some miniaudio 0.9 updates. Tweaked it and seems to work well on MacOS.
Thanks for the very useful Go package!

Merged, thanks!


Import aiohue inside functions again

Description:
Check config does not install dependencies. If people run check config on CI, there are no packages installed. I have a fix for it in #23500 but it needs more discussion to see if that is the direction we want to go.
Related issue (if applicable): fixes #23488 & related to #23500

Hi,
there are also other PRs that moved imports to the top: #23321
#23330 & #23339 (Can be reverted by #23521
BR

Also #23472. Should we revert all or is this just for 0.92?

We should only need to revert changes that made it into 0.92.x.  Once we merge the Config validate PR, we should be set to resume imports at the top... right?  :)

Decided we should move forward with having reqs installed for validation, so that constants can be used inside validation too. Closing this.


ci(dependabot): add dependabot settings for alpha branch

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Async programming with blocking operations

Hi,
I am trying to figure out how to get some blocking tasks performed in parallel, the example is here:
https://gist.github.com/yoanisgil/a28d57abca781fd84af1
Roughly the script will launch 5 asynchronous tasks (asyncMultiply) each of which will perform a random sleep. When I run the script with:
time docker run -v $(PWD):/usr/src/myapp  -ti --rm yoanisgil/php56-cli-composer php test.php

it takes about ~ 16 secs, which indicates that tasks are performed synchronously (each task performs a random sleep between 2 - 4 seconds).
Given the deferred/asynchronous nature of promises and that I'm using Amp\immediately I would have expected about ~ 4 seconds of execution.
For sure I most be doing something wrong. Any ideas what that might be?
Bests,
Yoanis.

A roughly equivalent implementation in Python3 of the same script could be found here:
https://gist.github.com/yoanisgil/85ce168fa7792c484a1f
and as expected it takes ~ 4 secs to complete. However, I do specify the  maximum number of available works for computation tasks.

Hi Yoanis,
your problem are the blocking operations. As the event loop is single threaded, you'll not gain any improvements when using blocking operations. Fortunately, there's a non-blocking waiting mechanism: yield new Pause($ms).
<?php

require __DIR__ . '/vendor/autoload.php';

use Amp\Pause;
use function Amp\run;
use function Amp\all;
use function Amp\stop;

function asyncMultiply($x, $y) {
    // Create a new promisor
    $deferred = new Amp\Deferred;
    // Resolve the async result one second from now
    Amp\immediately(function() use ($deferred, $x, $y) {
        $sleep = rand(2, 4);
        echo "Sleep $sleep seconds \n";
        yield new Pause(1000 * $sleep);
        $deferred->succeed($x * $y);
    });
    echo "new promise\n";
    return $deferred->promise();
}

run(function() {
    $promises = [];
    for ($i = 0; $i < 5; $i++) {
        $promises[] = asyncMultiply(6, 7);
    }

    $results = (yield all($promises));
    var_dump($results);
    stop();
});

So the question is how to turn that blocking operation into a non blocking one. Or is that outside of the scope of the project? Say I have a function MyBlockingFunction which I'd like to run asynchronously and for which there is no equivalent operation within the amp framework. Is it still possible to do that? Like I notice you have several projects which are using the amphp framework to provide asynchronous implementations of DNS, MySQL,HTTP clients (which by default are blocking). Maybe I am missing something but I though that an event loop and promises should be pretty much what is required to run concurrent tasks.

For your use case you need https://github.com/amphp/thread which does the async stuff based on the pthreads php-extension.
This will work for blocking operations but writing code non-blocking will be the superior approach if feasible in your context.

Like I notice you have several projects which are using the amphp framework to provide asynchronous implementations of DNS, MySQL,HTTP clients

those dont do blocking IO but non blocking. see e.g. http://php.net/stream_set_blocking

That's the difference between asynchronous concurrency and parallel concurrency. There's amphp/thread to execute those blocking operations in worker threads, but it's not yet up to date and doesn't work with any other than the NativeReactor, so it won't work if you have uv or libevent enabled. amphp/amp provides asynchronous concurrency.
Joe Watkins has a good blog post about "sync vs. async vs. parallel": http://blog.krakjoe.ninja/2015/07/the-universe-is-not-aware.html
If you do blocking operations, the task scheduler can't run other things while it's waiting for completion, because you never return control to the scheduler.

@rdlowrey We really need some good documentation with images and examples to explain what's going on in such a case, because a lot of people will have exactly this question.

@kelunik @staabm Thanks for the information (and yes I noticed there was amp/thread right after I posted my last comment :(). For sure some images and examples will be of a lot of help. Let me know if I can be of any help.

Sure, you can (almost) always help. Did you already see our docs repository?
Would be nice if you could provide some feedback e.g. regarding these points:

How easy is it to get your first "Hello World" running?
Could you move on from there? What do we need to provide after a "Hello World"?
What's important that should be on the main page?
etc.

@kelunik I am trying to clone the docs repository and I am getting this:
git clone git@github.com:amphp/amphp.github.io amphp/docs
Cloning into 'amphp/docs'...
The authenticity of host 'github.com (<IP_ADDRESS>)' can't be established.
RSA key fingerprint is 16:27:ac:a5:76:28:2d:36:63:1b:56:4d:eb:df:a6:48.
Are you sure you want to continue connecting (yes/no)? yes
Warning: Permanently added 'github.com,<IP_ADDRESS>' (RSA) to the list of known hosts.
Permission denied (publickey).
fatal: The remote end hung up unexpectedly

I do not recall any "Hello World" application, I just started reading some code and documentation here and put together a use case scenario. I would be more than glad to contribute but the clone thing kinda blocks me from going further ahead.

Looks like you do not use public keys to authenticate against GitHub. I'd recommend setting that up, there's an article in GitHub's help section. Otherwise you can still clone using HTTPS instead, probably the cloning method you're used to.

@kelunik  I took at look at the home page and I think it would be great if a somewhat more elaborated example is used (not elaborated enough to confuse users ;)). I also read that article about sync vs. async vs. parallel and it was really clarifying, so maybe a reference to amp/thread might help.
Honestly, I was just rushing to get a working example with amp/dns and I went straight ahead and set it up ;). It was just after a quick debate about async programming with one of my colleagues that I thought about this async sleep test and that's when things got a bit confusing. But it's all clear now ;).
On a somehow related subject I was wondering if amp/dns supports DNS resolution of A,TXT, MX, CNAME, etc records?Will I incur in a performance penalty if I launch Amp/run to perform up to 6 DNS queries within the context of an HTTP request?
Right now I am using https://netdns2.com/ since it's feature complete from a DNS point of view but it's 100% sync. I don't think that will scale too well for thousands or even hundred of simultaneous (needs to test though ;)), hence my question(s).
Bests,
Yoanis.

We're really just at the beginning of setting up our docs, so any feedback is really valuable, thanks. Regarding DNS, I think it's currently not supported, but @bwoebi wanted to add support for it. Once it's in, you can fire of all 6 requests at once and wait for all 6 then, so they're basically parallel then, because most of the time you'll be waiting for the other party to respond.

Currently amp/dns will only resolve A, AAA and CNAME records. There are plans for a more robust feature set but it just takes time :)

That's not that far from my current use case where I need A, CNAME, TXT and MX records. Let me know if there is anything I can do to help.

@yoanisgil amphp/dns supports those other records now, anything else open in this issue or can it be closed?

I think it can be closed. Can you send me a link so I can take a look at
what was done?
Thanks,
Yoanis
Le jeudi 10 septembre 2015, Niklas Keller<EMAIL_ADDRESS>a
écrit :

@yoanisgil https://github.com/yoanisgil amphp/dns supports those other
records now, anything else open in this issue or can it be closed?
—
Reply to this email directly or view it on GitHub
https://github.com/amphp/amp/issues/34#issuecomment-139354602.

@yoanisgil https://github.com/amphp/dns/tree/master/lib ;-)


CASMTRIAGE-4018 use rootfs instead of the deprecated filesystem.squashfs

Signed-off-by: Jacob Salmela<EMAIL_ADDRESS>Summary and Scope
filesystem.squashfs has been renamed to rootfs so this test needs to be updated to align with that change.
Issues and Related PRs

Resolves CASMTRIAGE-4018

Testing
Tested on:

#surtur

Test description:
Installed RPMs.  Ran both csi pit validate --livecd-preflight and /opt/cray/tests/install/livecd/automated/livecd-preflight-checks.  Validated that the test no longer failed.
Risks and Mitigations
Low.
Pull Request Checklist

[ ] Version number(s) incremented, if applicable
[ ] Copyrights updated
[ ] License file intact
[ ] Target branch correct
[ ] CHANGELOG.md updated
[ ] Testing is appropriate and complete, if applicable
[ ] HPC Product Announcement prepared, if applicable

/backport main


Keyboard doesn't close on back screen if country selected after searching from dialog fragment (fine in bottom sheet)

Subject of the issue
Describe your issue here.
Steps to reproduce
Tell us how to reproduce this issue. Please provide a working demo.
Expected behaviour
Tell us what should happen
Actual behaviour
Tell us what happens instead

Just Use the below code to hide the keyboard inside listner
           ` CountryPicker.Builder builder = new CountryPicker.Builder().with(this)
                    .listener(new OnCountryPickerListener() {
                        @Override
                        public void onSelectCountry(Country country) {
                            countryCodeTv.setText(country.getDialCode());
                            getWindow().setSoftInputMode(WindowManager.LayoutParams.SOFT_INPUT_STATE_ALWAYS_HIDDEN);
                        }
                    });

`

Yes, that i have already handled at my end because it's already too late


Remove ' from testNamePatterns to fix failing tests

fixes #68
The suggestion done in #68 fixes my problem with Jest not finding the tests.
I'm not sure if this is a valid fix, but this works for me on Linux

@haydenmeade do you think this solution could break something we are not aware of ?

@JaapBarnhoorn could you please rebase your change with latest main to fix the conflict?

@haydenmeade do you think this solution could break something we are not aware of ?

Seems ok if it's been tested

@haydenmeade do you think this solution could break something we are not aware of ?

Seems ok if it's been tested

I have tested it for a while and did not encounter any issue so far


The deny list in a scope start filter file is not respected by the CLI

The CLI does not currently parse a deny list.
In the current state:

Processes specified in the deny list will have their service configurations modified to include scope.
Processes specified in the deny list will be attached to.

Deny list is being removed from the scope filter. Closing


Store key in System Environment is not working

Hi!
I want to store my key in the System Environment variables. I thought that if set there NODE_MASTER_KEY with my key, it will be enough, but it seems that I should copy the key file to the server. Should I do something else?
Error: The key file could not be found.
at loadKeyFromFile (D:\App\node_modules\schluessel\src\cred.js:17:11)
at loadKey (D:\App\node_modules\schluessel\src\cred.js:51:43)
at Object.loadVault (D:\App\node_modules\schluessel\src\cred.js:73:15)
at Object. (D:\App\node_modules\schluessel\index.js:9:34)
at Module._compile (internal/modules/cjs/loader.js:1137:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1157:10)
at Module.load (internal/modules/cjs/loader.js:985:32)
at Function.Module._load (internal/modules/cjs/loader.js:878:14)
at Module.require (internal/modules/cjs/loader.js:1025:19)
at Module.Hook._require.Module.require (C:\Users\AppData\Roaming\npm\node_modules\pm2\node_modules\require-in-the-middle\index.js:70:39)

I solved this by restarting the server.


Create utilities/spawn_coelixir

Branch: feature/75
remote:
remote: Create a pull request for 'feature/75' on GitHub by visiting:
remote:      https://github.com/zeam-vm/pelemay_backend/pull/new/feature/75
remote:


perf(map_height_fitter): more efficient query to find closest lanelet

Description

Improve performance of the map_height_fitter when getting the ground height.
Currently, it checks ALL lanelets to find the closest one. With this PR, the query is more efficient (thanks to the R-tree used in the lanelet layer).
Tests performed

Not applicable.
Effects on system behavior

Not applicable.
Interface changes

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[X] I've confirmed the contribution guidelines.
[X] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@maxime-clem
Thank you very much!
Sorry, I implemented it inefficiently due to my ignorance of lanelet modules.
I would like to ask an additional question that is not related to the pull request.
Is it reasonable to use centerline() from a retrieved item to get the z-coordinate?
I don't know what a centerline is, and I'm thinking that if this could be a very long lanelet element along the way, then there might be a slight probability that it is far from the query point.

Is it reasonable to use centerline() from a retrieved item to get the z-coordinate?

Thank you for checking the PR. It sounds like it would be better to retrieve the closest point. Which is also possible. I can use nearest(lanelet::BasicPoint2d{x, y}, 1); on the vector_map_->pointLayer instead of the vector_map_->laneletLayer. If that sounds okay I will update the PR.

I think that change is fine. :+1:
I apologize for the inconvenience, but please make the change.


Example for 1 NIC AZ Key Vault is using the incorrect Input Variable

Line 103 of the example terraform deployment - https://github.com/F5Networks/terraform-azure-bigip-module/blob/main/examples/bigip_azure_1nic_deploy_keyvault/main.tf
is using the incorrect input variable.
az_key_vault_authentication
On the main documentation - https://github.com/F5Networks/terraform-azure-bigip-module/tree/main, the input variable is defined as az_keyvault_authentication.

Hi,
Thanks for reporting. Added to the backlog and internal tracking ID for this request is: INFRAANO-1537.


Comment Spam

Thanks to Joseph (last name withheld) for helping dig this up.
A user steph88 is not a bot, but may as well be.

http://ludumdare.com/compo/ludum-dare-36/?action=preview&uid=51253
http://ludumdare.com/compo/ludum-dare-36/?action=preview&uid=50889
http://ludumdare.com/compo/ludum-dare-36/?action=preview&uid=56020
http://ludumdare.com/compo/ludum-dare-36/?action=preview&uid=46596
http://ludumdare.com/compo/ludum-dare-36/?action=preview&uid=85560

Hi, nice game and congratulations ! Simple mechanics that still offered a decent amount of challenge.We're realizing a video with several games of the Ludum Dare #36. We made the same thing at the previous jam.
Can you add your game on REDACTED ? (it's free)
So we can include also your game in the video ;)
p.s. write #LDJAM in the game's description.

As things stand, there really isn't a policy against this, it's just uncool. Very uncool.
Given our size, flagging comments is something that needs to be addressed. I don't want LD to devolve in to Reddit, with its Up and Downs, so Flagging must be a special case. It needs to be sort-of difficult to do. Care must also be taken so it isn't use for censorship (negative criticism). It should only ever be used and acted upon in the case of abuse (hateful things against others, and cases similar to the one above).
One flag shouldn't be enough to remove a suspicious comment, because of the possibility of abuse. That said, some people will, whether we like it or not, use flagging to mark comments they don't like.
That said, it's probably worth considering implementing local hiding. When a comment is flagged by a user, hide it and its children from them. When a comment gets enough flags, it gets removed for everyone. Where that fits in is TBD (a Flag-Count field on every comment?).
We can't rely on simple metrics like "5 flags" anymore, so we will have to go with trust scores.

Like blocking people on Twitter, it's probably a good idea to ask for a "flag reason". Options could include.

I don't want to see this comment
The comment contains inappropriate language
The comment is rude or mean
The comment is offensive
The comment is spam

If someone wants to be picky about the comments they see, they should be allowed to be.
Also kid friendly mode, if it becomes a thing, is going to need data to know what comments are/aren't safe. It's not a foolproof system. We can't promise to protect minors from inappropriate language or themes, but we can do a little better.

Implementation wise, this may make the most sense as a modification of Karma. No new tables need to be added, just another field and index. You're limited to setting one state per comment.
The majority of these are negative states, but could be assigned to an address space of a signed 8-bit integer, and spaced out accordingly depending on severity (offensive and spam being the worst).
Author Karma could become part of this system. The above new karmas being negative values, regular likes being positive, author karma and/or other fancy karmas being worth more than that.
Off-hand I'm thinking applying a 3-bit shift, meaning each severity level has 8 possible choices, and 16 possible levels (i.e. 4 bits, with the last bit reserved for the sign). Might not need 16 levels though, but 16 choices per level may be more important.
Probably need a graph.

I think instead of fully automated removal, flags above a threshold should be added to a moderator queue. Maybe they get temporarily removed at some level, but a human should have the final say.

One of my goals with the site is to remove the need for moderators (I'm my experience mods have a "life expectancy", before they mode on to other things). So you may see mention of a number of system that implies a hidden system of trust. There are of course cases that require an executive decision, but 99% of cases are not this.

I think any list of options should include the ability to flag things as criminal/harassment, but in general I agree that the best thing is if we build the site so it encourages good behavior in general but I would also feel more comfortable if we had good venues for reporting serious problems. It could also be a point in reaching out to some of those who have experience with (being) victims of online abuse for how to think about good and safe design.

The flip side problem to consider is the potential for abuse by flaggers to harass people or ideas they don't like. This is why avoiding human moderation is an idealist dream, not currently grounded in reality :) It will mostly work well until it really doesn't.

If this wasn't a niche platform for creators, I'd agree with the impossibility of automation. But we are niche, averaging 5000 active users during an event. I could see us getting as high as 10k once things are smoothed out, but unlikely more than 20k at any one time.
Because a line is drawn between those that made something and those that didn't, we avoid a significant number of problems. We also have a higher number of people that actually understand what they're talking about, unlike a sub-reddit or forum. Not a lot of things work this way. We quite literally have a vetting process, and it's participation.
Of course there will always be troublemakers, and I'm not suggestion no moderation tools. But only the most conniving abuses will get by people.

I'm not really trying to suggest that those who have suffered abuse would be better per se in figuring out how to design or not design safe online experiences. I'm however quite convinced that it is important to have a diverse group of people with different backgrounds and experiences involved in discussions about such system. And I guess I want to hold LD to higher than average standards with this regard. Because game making matters to me, and games and tech in general is a quite toxic place. So, even if I have zero negative online experience myself, I'm going to take the annoying questioning position until proven that I can relax.


[WIP] Enable cinder to handle cloud volume backup

This is the second phase of https://github.com/ManageIQ/manageiq/pull/11088. Now Cinder manager will refresh all the volumes, volume snapshots and volume backups.

Add Cinder storage manager support (https://github.com/ManageIQ/manageiq/pull/11088)

@roliveri @jerryk55 please review.

This looks good to me.
The test failures appear to be unrelated to the PR.
@Fryguy - is this a known failure on master right now?
 1) MiqAeMethodService::MiqAeService service models expose all expected active_record models as service_models

     Failure/Error: expect(failed_models).to eq([])

       expected: []

            got: ["ManageIQ::Providers::Azure::NetworkManager::LoadBalancer"]

       (compared using ==)

     # ./spec/lib/miq_automation_engine/miq_ae_service_spec.rb:135:in `block (3 levels) in <module:MiqAeServiceSpec>'

@hsong-rh https://github.com/ManageIQ/manageiq/pull/11497
Should fix the test failure. Once it's merged, rebasing your PR should fix it.

@hsong-rh One more thing needs to changes - in app/models/cloud_volume_backup.rb, you need to specify that the ext_management_system belongs to "ManageIQ::Providers::StorageManager"
instead of "ManageIQ::Providers::CloudManager"; otherwise the code won't be able to find the ems for a backup.
This change should also be made in cloud_volume.rb and cloud_volume_snapshot.rb, maybe you could add those updates here as well?

@tzumainn Thank you for your reminder. I added these changes in this PR.

@hsong-rh It looks like you need to update the tests to account for the change you made in the last commit.

@tzumainn In this PR we didn't change current schemas/tables. Cloud volumes/snapshots/backups are still use the model of Openstacks. The relationship between them and Storage::Manager is indirectly via its parent manager, namely Openstack::CloudManager. I'll revert the changes.

@hsong-rh Your reply to @tzumainn is not correct.
The cinder manager has:
has_many :cloud_volume_backups,          :foreign_key => :ems_id, :dependent => :destroy

Which means that the ems_id filed of cloud_volume_backups now points back to the cinder manager, so it can't point to the cloud_manager any longer. This is a two-way relationship, but you've only changed one side. You need to change cloud_volume_backups so it knows it's ems_id points to the cinder manager instead of the cloud manager. This also applies to volumes and snapshots.
We didn't make any changes that require migrations. Does this change require a migration?
The tables don't need to change, the ems_id is still there, it's just a model change. However, this may require a data migration for existing data - the ids need to be changed to point to the new manager. If the refresh process handles this switch, then we're ok, otherwise it may be a problem.

By the way, this should be euwe/yes, correct?

@hsong-rh , @tzumainn - Actually, at this point cloud_volumes can belong to varying types of management systems. So, the belongs_to relationship should be to ExtManagementSystem, and not ManageIQ::Providers::StorageManager nor ManageIQ::Providers::CloudManager.

@roliveri Ah, that makes sense - thanks for pointing that out!

@tzumainn @roliveri Thank you.

Euwe Backport details:
$ git log
commit 26af3849a29daf30d300d99f853517243f446315
Author: Richard Oliveri<EMAIL_ADDRESS>Date:   Mon Sep 26 11:51:53 2016 -0400

    Merge pull request #11473 from hsong-rh/cinder_volume_backup

    Enable cinder to handle cloud volume backup
    (cherry picked from commit 1cb93af9a8f54b6e0cce2f1c372a9f047887997d)


Map doesn't update on sidebar collapse or expand

When:
The user toggles sidebar statte maps doesn't update.

Use padding for country layout instead of resizing map on sidebar collapse.


[W3W] Fair SampleApp loading indicator

Currently pairing loading indicator disappears in 3 sec. Indicator should represent real pairing status

can you pls link me to the code @flypaper0 ?
Looked at the original PR and don't find the 3 second definition

@arein
The timeout is set to 3 seconds in that PR in Example/WalletApp/PresentationLayer/Wallet/Wallet/WalletPresenter.swift line #147 If we don't receive a session by that time it will show the timeout error, so it is timeout value rather than hardcoded time for how long to show loader.
Referring to the PR, not sure about the current state as it seems like it's different code in develop.

Gotcha. 3s timeout makes sense. AFK so not looking at code by maybe Artur's hypothesis of we show the loading until we get the first message was right. Instead we'd probably need to load until we received a or all 1100 tags.

My initial bug report and what Pedro's video show is that after 3s we don't see an error

Yeah as I say the logic currently in develop is different from what is in the PR you've linked. It no longer shows the error message for some reason.
https://github.com/WalletConnect/WalletConnectSwiftV2/blob/bcecbd5af552cb9ee8dc125658614b5e8c878cc0/Example/WalletApp/PresentationLayer/Wallet/Wallet/WalletPresenter.swift#L130

added pairing state publisher, will be included in the next release


Improved UI and corrected domain condition

Added Placeholders for maxX and minX.
improved dropdown by adding 'Select graph' hidden option.
Added alert for the case where no graph is selected.
Corrected domain condition from maxX > minX to maxX >= minX.

Changes Screenshot

I don't really think this is necessary. Sorry.

If you send a video comparing the two I might reconsider it.


Way to specify iconPath for treeview items in 'web extension'

Can a web extension have iconPath set to some /resource/xyz.svg, or other scheme, or file path, to show custom .svg as its icons for their treeview TreeDataProvider ?
Would that work? -> Maybe creating a FileSystemProvider that provides the .svgs like the https://github.com/microsoft/vscode-extension-samples/blob/main/fsprovider-sample/src/fileSystemProvider.ts that would provide the .svg file contents from ram on its own 'scheme' when making URIs for the iconPath?
Thanks!

More suggestions : Is there a way to provide the iconPath URi/pathstring for the icon of an item from a TreeDataProvider with instead a string holding the svg content itself? (or maybe even in base64 if it helps) ?

Hurray!
I found out that scheme 'data' is supported! So for further reference for anyone having this issue when compiling for web extension support, i.e.vscode.dev support, you can specify your icon images as such :
    const testUri = vscode.Uri.from({
        scheme: "data",
        path: 'image/svg+xml;utf8,<svg xmlns="http://www.w3.org/2000/svg" fill="none" height="16" width="16"><path fill="#4d4d4d" d="M0 4v9h15.15V4zm1 1h13.15v7H1z"/></svg>'
    });

Instead of a typical "resources/dark/document.svg" that is shown in the treeview provider extension code samples.
Have a great day! :)


Code proposal for remote install

This code will allow the users to deploy the app on all target servers remotely
param(
[Parameter(Mandatory=$true)][System.Management.Automation.PSCredential]$creds,
[Parameter(Mandatory=$true)][ValidateScript({(Test-Path $) -and ($ -like "*.exe")})]$PathToFile,
[Parameter(Mandatory=$true)][string[]]$ComputerName
)
$targetInstallerName = 'AdHealthAddsAgentSetup.exe'
$ToBeInstalled = [System.Collections.ArrayList]::new()
foreach($computer in $ComputerName){
try{
Test-WSMan -ComputerName $computer -ErrorAction Stop
$ToBeInstalled.Add($computer) | Out-Null
}
catch{
Write-Warning "$computer did not pass the WSman protocol test. Application will not be deployed"
}
}
Foreach($computer in $toBeInstalled){
if(Test-Path "\\$computer\C$\$targetInstallerName"){
    Write-Warning "File already present"
}
else{
    try{
        Copy-Item -Path $PathToFile -Destination "\\$computer\C$\$targetInstallerName" -Verbose
    }
    catch{
        Write-Warning "Unable to copy file. Removing $_ from installation list"
        $ToBeInstalled.Remove($computer)
    }
}

}
Write-Verbose "Attempting to install the app on $($ToBeInstalled -split ',')" -Verbose
Invoke-Command -ComputerName $ToBeInstalled -ScriptBlock {
try{
Get-Service AzureADConnectHealthAddsInsights,AzureADConnectHealthAddsMonitor -ErrorAction Stop | Select-Object DisplayName, Status
Write-Warning "Services are already present. Application will not be installed."
exit 0
}
catch{
Write-Warning "Service is not present will install the app"
}
Start-Process "C:\AdHealthAddsAgentSetup.exe" -ArgumentList "/quiet /log C:\ADHealthADDSAgentLog.txt" -Wait
try{
import-module "C:\Program Files\Azure Ad Connect Health Adds Agent\PowerShell\AdHealthAdds"
}
catch{
 return "Unable to load module"
 exit 1
}
Register-AzureADConnectHealthADDSAgent -Credential $args[0]

} -ArgumentList $creds

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

ID: c53448ff-443b-9d3e-9300-344636c30d5d
Version Independent ID: eae8e4cc-a4f3-d6ce-85cd-9c5f588c3174
Content: Install the Azure AD Connect Health agents in Azure Active Directory - Microsoft Entra
Content Source: articles/active-directory/hybrid/connect/how-to-connect-health-agent-install.md
Service: active-directory
Sub-service: hybrid
GitHub Login: @billmath
Microsoft Alias: billmath

@vasillilov
Thanks for your feedback! We will investigate and update as appropriate.

@vasillilov
Thanks for contributing towards enhancement of the document. I am going to assign this case to content Author who will update accordingly.
@billmath
Kindly take a look at this case.

@billmath, do you gave any update?

Thank for submitting this. I have created a work item for it and will reviewing this in the coming weeks. I am not sure what the timeline will be, so I am going to close this for now. But once I have an update I will drop it in here and let you know.
Thank you!
Bill
#please-close


Update user-submission.md

@JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks

@siosulli Please merge this PR. Thanks


Is get_firewall_policies() implemented correctly?

I am seeing some test failures on get_firewall_policies in napalm-fortios.
napalm-base defines get_firewall_polices as:
        {
            'policy_name': [{
                'position': 1,
                'packet_hits': 200,
                'byte_hits': 83883,
                'id': '230',
                'enabled': True,
                'schedule': 'Always',
                'log': 'all',
                'l3_src': 'any',
                'l3_dst': 'any',
                'service': 'HTTP',
                'src_zone': 'port2',
                'dst_zone': 'port3',
                'action': 'Permit'
            }]
        }

This is from base.py file.
The unit tests for napalm-fortios (mocked data) have the following for 'show firewall policy'
config firewall policy
    edit 1
        set uuid 9d747de8-dad4-51e5-7a7e-d078aa77140a
        set srcintf "any"
        set dstintf "any"
        set srcaddr "all"
        set dstaddr "all"
        set action accept
        set schedule "always"
        set service "ALL"
    next
    edit 2
        set name "bla"
        set uuid de597b64-aca8-51e6-5d00-b7874e6b72b8
        set srcintf "port3"
        set dstintf "port2"
        set srcaddr "all"
        set dstaddr "all"
        set schedule "always"
        set service "ALL"
        set logtraffic all
    next
end

The napalm-fortios driver returns the following (on get_firewall_policy):
{u'1': [{u'action': u'permit',
         u'byte_hits': -1,
         u'dst_zone': u'any',
         u'enabled': False,
         u'id': u'1',
         u'l3_dst': u'all',
         u'l3_src': u'all',
         u'log': u'False',
         u'packet_hits': -1,
         u'position': 2,
         u'schedule': u'always',
         u'service': u'ALL',
         u'src_zone': u'any'}],
 u'2': [{u'action': u'reject',
         u'byte_hits': -1,
         u'dst_zone': u'port2',
         u'enabled': False,
         u'id': u'2',
         u'l3_dst': u'all',
         u'l3_src': u'all',
         u'log': u'all',
         u'packet_hits': -1,
         u'position': 1,
         u'schedule': u'always',
         u'service': u'ALL',
         u'src_zone': u'port3'}]}

The key names being '1' and '2' looks wrong to me (should map to the policy_name per the definition in base.py).

@ebeahan This is the same issue I messaged about in the Slack channel.

@ktbyers Thanks! I'm taking a look.

On FortiOS CLI firewall policies are created within the config firewall policy context by stating edit <integer> where the integer is a unique value that becomes the policy id for that firewall policy. With that said, the key names being set to 1, 2, 3, etc. is correct since it's the policy id casted as a string. Also the policy ID is independent of the policies position (e.g. I created policy id 5 but move it's position above policy id 1).
I do see an issue with the position key of the inner dict in the output. If I understand the intent of position correctly, the value should represent where the policy lies in firewall evaluation order (ordered 1 to n with 1 being the top of the firewall policy, n being the final rule). The current way position is set is non-deterministic since the raw command output returned from pyFG is initial parsed into an intermediate dict structure potentially losing the original ordering.
One possible fix would be initializing an OrderedDict() vs a dict() for that intermediate dict to ensure the ordering remains accuracte for later on when position is being determined:
https://github.com/napalm-automation/napalm-fortios/blob/develop/napalm_fortios/fortios.py#L272

So with FortiOS is there only one firewall policy?
For example, on an ASA, I could have multiple firewall policies that would map to ACL names and could be bound to different interfaces.
I was expecting the 'policy_name' to correspond more to this.
What is the use of policy id in fortios. Is it just a way to reference the entries in the one firewall policy (so that you can delete them, move them, change them)?

Yes, there is only one firewall policy. Rules apply always to inbound/outbound interfaces pairs (which can be wildcards) but they are configured under a single flat list (the UI let's you filter/organize by interfaces)

I still think this use of policy name doesn't make sense for FortiOS. The policy name for FortiOS should be something like default (since there is only one firewall policy that always applies).
The FortiOS policy id corresponds more to ACL sequence number (on Cisco ASA; Cisco IOS firewall)...and doesn't correspond to policy_name.
Here is what I think it should be for FortiOS (which would work reasonable well for Cisco ASA and Cisco IOS firewall).
        {
            'policy_name': [{
                'packet_hits': 200,
                'byte_hits': 83883,
                'id': '230',
                'enabled': True,
                'schedule': 'Always',
                'log': 'all',
                'l3_src': 'any',
                'l3_dst': 'any',
                'service': 'HTTP',
                'src_zone': 'port2',
                'dst_zone': 'port3',
                'action': 'Permit'
            }]
        }

position should be removed and should be indicated instead by position in the list. Also, there is only service specified here and we need to add src_port, dest_port (and possible remove service). There also needs to be a specification of protocol (i.e. UDP/TCP/ICMP).
Note, id field already contains the FortiOS policy ID.
We also should have someone that is familiar with Juniper SRX review to make sure we are covering the major NAPALM firewall platforms. Also probably need to consider zone based firewall on Cisco IOS and make sure it covers that reasonably well.
I am also open to just eliminating this and re-implementing later (since current implementation has a lot of problems) and since FortiOS is currently the only platform it is implemented on.

@ktbyers Agree with what you're proposing. I was interpreting policies as the individual firewall rules vs a collection of rules.  As you pointed out, it's really indifferent in terms of FortiOS so whatever approach will apply best across the board can be adopted. Also prefer the idea of dumping position and allowing the index position with the list to serve the same purpose.

I have no particular opinion about this. If you plan to modify the definition of the object, could you contact the people involved in the original PR to synch with them, please?

@dbarrosop Yes, that makes sense.

@awlx @DiogoAndre We had some discussion here on whether to modify get_firewall_policies that would also better support Cisco ASA and IOS if that's something that is implemented in the future. It looks like you were both involved in the original discussions below:
https://github.com/napalm-automation/napalm-base/pull/136
https://github.com/napalm-automation/napalm-fortios/pull/26
Any thoughts? Like @dbarrosop mentioned above we wanted to sync up with those involved in the original PRs to add the feature.

@ebeahan I am totally fine with your idea to modify the behaviour of get_firewall_polices. If you are modifying it could you also add a field for NAT Status and NAT Pool?
The service field is a group of protocols and ports for the Fortigate so I would like to still have it. As this makes it easier to search for in the config.
The implementation of get_firewall_policies was only driven by the need to read the rules of a Fortigate. So yes it lacks functions for other platforms and I am happy if someone wants to improve it :). I have some ASAs and a SRX here so if you need help testing I'll be glad to help.

Hey guys,  I have a couple of thoughts to add to this:

policy ID servers as the name of the policy, and sequence number determines the order in which the policy is used for enforcement. I think both are important, so maybe do something like

...
'id': 10,
'seq': 3
...

Would it be useful to grab the comments also? Or does that typically fall under too much clutter, not enough value?

@jmcgill298

we already have position which should be equivalent to seq. Or do you mean something else?
Where would comment be? Policy or individual entry?

@dbarrosop

you do currently have position, but @ktbyers suggested format does not, so just saying I think that should be carried over.
The comment is per individual entry. My previous job of managing firewalls used it primarily as a reference to a ticket request, but some people use it to also provide the reason why it was added. So I see that it could be useful to some, and not useful to others.

Anyone at RIPE74 maybe it's better to discuss somethings of this in person.

I am but probably not the best person to talk with about this topic. Maybe @mirceaulinic.


error when multiple explosions affect the same thing (tested on wooden walls)

Specs:
Ryzen 9 3900x oc 4.4GHz
gtx 1080 oc
RimThreaded Version: 1.0.63
Modlist:
-Harmony
-Core
-HugsLib
-Mod Manager
-Smart Speed
-TicksPerSecond
-RimThreaded
-Camera+
Appears in vanilla: false
Reproducible: true (use devmodes explosions)
error when multiple explosions go off in close proximity affecting the same thing, tested multiple explosives, they pretty much all do the same thing.
got 3 main frequent errors:
https://gist.github.com/FervidFiend/36b1fc49d6d2cd0378fe7c95afd433cb
https://gist.github.com/FervidFiend/53076e115957b3f887c62e394bb49d99
https://gist.github.com/FervidFiend/2443c83ede6f80c78d84e655e8ecd157

Closing bug. Please reopen in new bug report using the new templates if this issue persists.


Add columns for previous timesteps; fix some sex behaviour bugs

(if you haven't seen already!) To avoid the circular imports due to the type hint, you can use a constant that's provided by the typing module; see a post with more details and this StackOverflow question. Unfortunately, it's a bit clunky. The other option is to remove the type hint.

I hadn't actually seen that before; I spent quite a bit of time yesterday trying to figure out how to get around the circular imports and just ended up having to take them out on my working branch! The trouble with that is that it makes development a lot harder because  a) you don't know off hand what type something is meant to be, and b) you don't get any of the additional information like what functions are available and function signatures. I did consider restructuring so that the other modules point to the population rather than the population pointing to the modules, so the evolve step wouldn't be called on population but would be a simulation method instead, and the simulation would tell each of the modules to do their thing and each module would act on the population. This would solve the circular import issue and still makes conceptual sense, as one might readily think of the modules as being aspects of the simulation rather than aspects of the population itself.

New methods for dealing with dataframe are working and are backwards compatible so work with existing code (as well as new code written for e.g. circumcision). I have not gone back through and changed all instances of previous code style as it may be a lot to do in one go and I'd like to move forward, but we can migrate to this way of doing things as we go ahead which will make it easier to change how we handle the dataframe in the future (especially in the case that Pandas makes changes to the way that it works!!)


[5.3] Introduce a Response contract

\Illuminate\Http\JsonResponse extend \Symfony\Component\HttpFoundation\JsonResponse, not \Illuminate\Http\Response.
A response contract would allows to match both \Illuminate\Http\Response and \Illuminate\Http\JsonResponse, as these two types of response should always be handled by Laravel middlewares
Note: in some ways this interface could also be completely empty, just as \Illuminate\Contracts\Queue\ShouldQueue is.

Related : #12313 #12309

No plans to define our own HTTP contracts. Especially with PSR-7 available.

Especially with PSR-7 available.

:+1:


Send container events with nil PodSandboxStatus

Signed-off-by: ruiwen-zhao<EMAIL_ADDRESS>This is a follow up action item of supporting Kubernete's Evented PLEG feature (https://github.com/kubernetes/enhancements/issues/3386). Now that https://github.com/kubernetes/kubernetes/pull/114351 merged, kubelet can handle nil PodSandboxStatus, containerd can send events with nil PodSandboxStatus.
Note that this PR also contains reverting part of https://github.com/containerd/containerd/pull/8007 (i.e. moving PLEG event generation back to sbserver) This is needed because we need the event generation to be in sbserver/ to be able to get PodSandboxStatus. More details in https://github.com/containerd/containerd/pull/8007#issuecomment-1416341393.
fixes https://github.com/containerd/containerd/issues/7785

/assign @mikebrow

While working on this, I found a change of behavior introduced in https://github.com/containerd/containerd/pull/8007, which should have broken the test but didn't due to a bug in the test.
cc @mxpv

sync'ed with @mxpv on Slack.
With current implementation, the PLEG events are generated before sandboxstore.Add is called. So the event cannot get sandbox status by either calling cri.PodSandboxStatus or Controller.Status.
The conclusion here is that we will move the event generations back to sbserver/, and will move them to podsandbox package after sandboxstore.Add is moved to Controller.

/ok-to-test

LGTM with a hold for the k8s PR being cherry picked to v1.26.. otherwise a v1.7 containerd would throw an exception on v1.26.. We'll need to mention in our 1.7 docs the requirement for the v1.26 patched release.

Thanks Mike. Created https://github.com/kubernetes/kubernetes/pull/115552 to cherry pick.

LGTM with a hold for the k8s PR being cherry picked to v1.26.. otherwise a v1.7 containerd would throw an exception on v1.26.. We'll need to mention in our 1.7 docs the requirement for the v1.26 patched release.

@mikebrow  https://github.com/kubernetes/kubernetes/pull/115552 is now merged and will be released with 1.26.2.  Can we get an approval on this PR?

/retest

/test all

@ruiwen-zhao looks good but needs a rebase to pick up the main branch fix for a path issue with cri-tools .. they moved the location of the test executable .. sigh

@ruiwen-zhao looks good but needs a rebase to pick up the main branch fix for a path issue with cri-tools .. they moved the location of the test executable .. sigh

uh thanks Mike! Somehow i was under the impression that github would rebase my change for me before running the tests.
Rebased on top of main.

@ruiwen-zhao looks good but needs a rebase to pick up the main branch fix for a path issue with cri-tools .. they moved the location of the test executable .. sigh

uh thanks Mike! Somehow i was under the impression that github would rebase my change for me before running the tests.
Rebased on top of main.

it would for prow side k8s changes and any "latest" type project pulls, but not for the containerd/containerd changes as they are stowed in your branch

/test pull-containerd-sandboxed-node-e2e

@ruiwen-zhao I'm just merged the sandbox related change. Please rebase. Thanks

/test pull-containerd-sandboxed-node-e2e

All tests have passed and corresponding kubelet change is merged and cherry-picked.
This PR should be OK to be merged.


attribute order changed on "modify"

Even if I dont actually modify anything, the order of the attributed in catalog is changed:

Well as of now there is no detection whether there is a change. It could be added with the new dialogs, yet that would not solve the core issue. We need to force an ordering of item in the UI not rely on backend structures.
@jakubklimek Any suggestion for ordering?

Maybe I am missing something, but the attributes should be ordered in backend structures - this order should be used in multiple places such as UI, generated documentation, schemas, etc.

As far as I know, the entities are made available as an JS object
https://github.com/mff-uk/dataspecer/blob/main/packages/core-v2/src/entity-model/entity.ts#L14
@sstenchlak Do we have an ordering in the backend structures and if so, how can I obtain it?
PS: This is not stating that there is no issue in CME code.

Hi, no, there is no ordering in the semantic model - not even between attributes, as they are represented as stand-alone relations. The only ordering currently in place is within the list of classes in the generated documentation, which is sorted alphabetically.
I agree that some form of ordering might be useful. The question is whether it should be at the semantic or the visual level, as both approaches could make sense..


DataStage connector should be sending SchemaType not SchemaAttribute qualifiedNames

ie. need to use the IGC prefixes for things like TabularColumnType, etc on the qualifiedNames for them to be appropriately found.

Actually OMAS is already expecting SchemAttribute, so no need to change.


[NTR/gene]

Preferred gene-related syndrome label
PIK3CA-related overgrowth syndrome
Synonyms
PROS
Parent term (use OLS, or your favorite ontology browser)
MONDO:0019716
Definition
Any overgrowth syndrome in which the cause of the disease is a mutation in the PIK3CA gene.
**Definition source (Please give PubMed ID, if applicable, in format PMID:#######)
PMID:25557259
See also: https://www.prosspectrum.com/
Children terms (if applicable) Should any existing terms be re-classified as children underneath this new proposed term?

Fibroadipose hyperplasia or Overgrowth (FAO) - [MONDO:0017812]
Hemihyperplasia Multiple Lipomatosis (HHML), [MONDO:0017177]
Scoliosis/Skeletal and Spinal (CLOVES) syndrome, [MONDO:0013038]
macrodactyly [MONDO:0007962]
Segmental progressive overgrowth syndrome with fibroadipose hyperplasia [MONDO:0017812]
Megalencephaly-Capillary Malformation (MCAP or M-CM) [MONDO:0011240]
Dysplastic Megalencephaly (DMEG) - mentioned in definition but not in either Monarch or ORDO.

Your nano-attribution (ORCID) or URL for a working group
0000-0003-1493-865X
Note - this is a high level disease grouping that I've seen people in our Rare as One cohort talk about and it's not represented in MONDO but does have a designation in ORDO: https://www.orpha.net/consor/cgi-bin/OC_Exp.php?lng=en&Expert=530313

Hi @GullyBurns! Looks like we do have this term: MONDO:0035162 'PIK3CA-related overgrowth syndrome'
I can classify the children under it, as you suggested above.

OK. That's interesting. I was searching for it on the Monarch webpage (not the ontology) and couldn't find it there. Apologies.

The extra terms were from a cursory read of  PMID:25557259 and may need some additional discussion for precise definitions. Would it be worth attempting to connect to members of the Rare as One community about this?

OK. That's interesting. I was searching for it on the Monarch webpage (not the ontology) and couldn't find it there. Apologies.
No worries! You are right, it is not in Monarch. Monarch is not fully synched up with the latest version of Mondo. I suggest using OLS: https://www.ebi.ac.uk/ols/ontologies/mondo/terms?iri=http%3A%2F%2Fpurl.obolibrary.org%2Fobo%2FMONDO_0035162

The extra terms were from a cursory read of PMID:25557259 and may need some additional discussion for precise definitions. Would it be worth attempting to connect to members of the Rare as One community about this?

That would be great to connect with the Rare as One experts to add definitions to these terms. Would you mind sending them an email and include me and I can coordinate a call. Thanks @GullyBurns!

@GullyBurns I think Dysplastic Megalencephaly (DMEG) is a phenotype, I'll make a ticket on the HPO tracker


🐛(LMSHandler) check resource_link matches lms COURSE_REGEX

Purpose
Enrollment widget can fail to enroll if course run resource_link is malformed.
Proposal

[x] Display enrollment widget only if there is a backend for the course run and a course_id can be extracted from its resource_link

I'm wondering if it's not the frontend's job to determine if the course "has_connected_lms"? Since the frontend and the backend regexes are different settings, It feels like each one should check this based on its regex!

I'm wondering if it's not the frontend's job to determine if the course "has_connected_lms" based on its regex?
Since the frontend and the backend regexes are different settings, It feels like each one should check this based on its regex!

@sampaccoud Delegate this check to the back allow us to prevent a useless react component generation.

I'm wondering if it's not the frontend's job to determine if the course "has_connected_lms" based on its regex?
Since the frontend and the backend regexes are different settings, It feels like each one should check this based on its regex!

Do this check from backend prevents rendering of a React component to just display a link.

Ok then. The only drawback I see is that the backend and frontend regexes must be the same regex translated in 2 languages. I suppose it's fine and will mention it in the documentation revamp.


Isolate.pauseCapability description is incorrect

The description for property Isolate.pauseCapability reads:

Capability pauseCapability
read-only
Capability granting the ability to pause the isolate.
This capability is used by pause. If the capability is not the correct pause capability of the isolate, including if the capability is null, then calls to pause will have no effect.
If the isolate is started in a paused state, use this capability as argument to resume to resume the isolate.

It is not clear how to use the pauseCapability.
It is read-only, so it is not clear how it can be changed in order to be used by pause.
Also the description of Isolate.resume() reads:

The capability must be one returned by a call to pause on this isolate, otherwise the resume call does nothing.

So it might be incorrect to use Isolate.pauseCapability with Isolate.resume()

The capabilty doesn't change, it is set when the object is created.
You don't use it actively, it is used by the pause function, which uses it to document to the isolate that the caller is allowed to pause it.
The capability is visible because you can extract it and use it to create a new Isolate object.
You can also create another Isolate objects without a pause capability. That Isolate object can't be used to pause the corresponding isolate.
The pauseCapability of an Isolate object should either be the one corresponding to the controlPort's isolate or it should be null (that's what the Isolate constructor tries to say).
You can create the Isolate object with any capability, the type checked won't notice, but it's not going to work. Since control port messages are generally fire-and-forget, there is no error if you use the wrong capability, it just doesn't do anything.
The resume documentation is correct. If you read the documentation for the paused parameter, it says:

If the [paused] parameter is set to true,  the isolate will start up in a paused state, just before calling the [entryPoint] function with the [message], as if by an initial call of isolate.pause(isolate.pauseCapability). To resume the isolate, call isolate.resume(isolate.pauseCapability).

That means that isolate.pauseCapability can be treated as the return value of a pause call.

@lrhn Should we close this, or is there some improvements we can make here?

I'm making some improvements (I hope, at least some changes) in https://codereview.chromium.org/2549363003/

Could you also correct the typo in the last sentence?

If the isolate is started in a paused state, use this capability as argument to resume to resume the isolate.

Absolutely. Good catch.

I think it is also worth to mention, that it will not be possible to resume the isolate, which is started in paused state, if the pauseCapability is not correct.

I don't think it should be necessary to repeat that an invalid capability doesn't work. That's stated in the resume method.  And it shouldn't be invalid.
In practice, an isolate is either created using a spawn function, and then the capability is correct, or it's created using the Isolate constructor, and then whoever created it is responsible if it doesn't work. The capability should never be wrong, it should either be null or correct - anything else is just somebody messing with you. The Isolate constructor does say that the capabilities should be the ones taken from the original isolate that the control port came from (or a subset of them).

Closing as @lrhn's doc fix landed.


Discuss Moving Build Paths To Environment Vars

Right now build paths in GB are relative paths from the locations of the vs project. This is useful b/c it means that after downloading the project and the required libs that developers dont have to do any work to make things build.
We start running into issues when we use GB as a sub-module of another project (like RE). b/c GB is then one folder deeper in the structure the paths it searches for are invalid. The current solution for this is to simply copy SFML and boost a second time and place them where GB expects them.
This will become inefficient as GB grows in size and requires more and more 3rd party dlls. This also becomes cumbersome as the number of projects using GB grows.
One possible solution is to change the compiler options for GB to use environment variables to compile.
Each developer would have to set the environment variables to the absolute path of their own location for the 3rd party dlls and libs. The benefits of this solutions is that no developer would have to keep several copies of each 3rd party resource that GB uses.

Well, the relative paths aren't really an issue for RE:

I hate that it's above the whole folder, so I moved it down.
a. GB and RE are sharing the same SFML contained one level above the GB and the RESln.
b. This is cleaner and keep all of our possible Dlls right close to where they will be used.

As for the multiple copies, I agree that could need worked on, but I still think I prefer relative paths. I don't particularly like the idea of separating my Dlls from my projects, even though I would be able to handle it. I have always thought that that sort of stuff belonged together.

Environment variables will be used.

see this page for information on how to set up environment vars


Adds codecov to CI

Changes proposed in this pull request

Adds codecov in preparation for removing code-climate to maintain some CI coverage metrics.

Instructions for Reviewers
[All PRs] - Confirm PR template filled
[Feature Branches] - Review code
[Production Merges to main]
    - Check story numbers included
    - Check for debug code
    - Check version

Codecov appears to be reporting the same coverage as codeclimate on the website 👍


contrast/background

Hi! Having a blast using the theme, albeit in short bursts -- is there any way you could and would be willing to provide a variant with a lighter, more grayish background akin to Nord or sth? The high contrast is unfortunately killing my eyes :)

Hi @effinsky ! I think you may have the wrong repo? There's no colour theme here, just syntax highlighting, snippets, and a formatter.

I guess you are looking for https://github.com/NicklasXYZ/vscode-gleam-themes ?


finding offset for the patchfinder

Hi Jsherman212 or contributors,
I was looking to work on adding support for iOS 16, currently digging through Kernelcache for the offset value, would just finding the offset be good enough to make xnuspy work on iOS 16?

Been trying to work on it myself, I am stuck on write_blr function for some reason this seems to cause an issue? could this because I don't have the correct address(I have looked and updated the mask and stuff), or is it that xnu now know when you write some assembly code in that region.


Add an init command

I work on a package that is published to NPM, but internally uses some JSR modules. Currently, if anyone wants to use the package, they have to manually create a .npmrc file and add @jsr:registry=https://npm.jsr.io into it. It would be great to have some kind of jsr init command which we could tell the users to use before installing our package.

Isn’t this something npx jsr add already does?

Running just npx jsr add throws with:
ERR_PNPM_MISSING_PACKAGE_NAME  `pnpm add` requires the package name
Child process exited with: 1

Therefore, we'd have to tell the users of our package to either explicitly install one of its dependencies, or to install a dummy JSR package, neither of which sounds ideal.

So you basically wanna initialize the .npmrc without having to add a specific package. Makes sense! Upvoted the idea.


Adds SetServiceStatus

This adds a SetServiceStatus function which accepts a handle returned by RegisterServiceCtrlHandlerEx and a SERVICE_STATUS struct. So an additional HSERVICESTATUS handle, a SERVICE_STATUS struct, and SERVICE_STATUS struct associated constants are also added.,
NOTE: I believe the currently implemented SERVICE_TYPE constant is a bitmask and not an ordinary constant. See the dwServiceType member of the SERVICE_STATUS structure for details
NOTE: I'm not sure why https://github.com/rodrigocfd/winsafe/pull/100 was closed. Please see my comments there and continue here. Unfortunately I lost the nice #100 number
Reference:
[1] SetServiceStatus
[2] RegisterServiceCtrlHandlerEx
[3] SERVICE_STATUS

Hi Tom,
I'm currently implementing RegisterServiceCtrlHandlerEx, which is tricky like all functions which deal with native callbacks. So I must be careful, since it's easy to segfault.
As soon as I finish it, I'll publish it so we can discuss whether it suits your use case. After that, I'll accept your PR, which I believe it will fit nicely together with the code I'm writing.
Thanks for your contribution, we're gonna talk again soon.

@rodrigocfd Ok sounds good.
I appreciate you taking time to add this feature. 👍

hello @rodrigocfd
I was about to start working on adding RegisterDeviceNotification which can be used to register a window handle (or a service) to receive messages of DEV_BROADCAST_HDR. But before doing that I wanted to check in with you and see how this PR was coming along.
Also I kind of concluded that it was not possible to wrap ServiceTableEntry with out doing something abstract, involving macros. There are probably ways you can do it by unnecessarily using the heap, but I was curious if you gave some thought about potentially wrapping this item if potentially supporting the service api's

Hi Tom,
I'm finishing the implementation of RegisterServiceCtrlHandlerEx. I was absent for a few days because I'm really busy with work, but I'll return as soon as I can. I'm sorry this is taking longer than usual.
Regarding SERVICE_TABLE_ENTRY, is it used solely with StartServiceCtrlDispatcher? If so, it's in my backlog. As far as I could see, it's one of those wildly unsafe constructs of Win32, therefore super hard to implement ergonomically in Rust. I'll still think about it, but probably I'll leave the struct with an open pointer – that is, the struct will be unsafe –, then forward the "problem" to LPSERVICE_MAIN_FUNCTION.
I'll need your help on all this. If you have good C/C++ examples of RegisterServiceCtrlHandlerEx and StartServiceCtrlDispatcher, please leave a link to them here. That would help me a lot.

Here is an example from the author of Win10 sys programing books.  The Chapter19 demonstrates how to use the Service API's pretty well. It is not a real world example however since it is educational book it kind of makes the parts we're looking for more clear.
Per the author, there is some history for why the API is the way it is. Basically in the early days, many services would run under the same process (hence the table array).  Each service would make itself available to be imported as a DLL. The main process would import the service functions and load them into an array and each service would share the same process and save memory.
Modern services typically are implemented as a single entry in the array however. (which could simplify the api significantly i think.)  However, if you want to be a pretty literal translation of the win32 api, and support legacy services to be wrapped, then you would need to support the table array.
I think it is a good idea to leave the "unsafe" function as part of the table.  Technically there is no way around it. Lets say a user wanted to use winsafe to import a bunch of legacy services from DLLs. they would need to manage the raw function handlers themselves.  The most hazardous thing would be to make sure the array is null terminated. To do that with out the heap you might need a macro I think.

That's very interesting.
AlarmSvc is the most straightforward example, I suppose?

Yes, the other examples are for creating and removing services which show how sc.exe work

Hi Tom,
Here is my implementation of RegisterServiceCtrlHandlerEx. Let me know if this is correct.

It is very close I think.
DeviceEvent
I believe all device events do include the DEV_BROADCAST_HDR so i do not believe it needs to be optional. The DEV_BROADCAST_HDR can be further parsed into:

DevBroadcastDeviceInterface
DevBroadcastHandle
DevBroadcastOem
DevBroadcastPort
DevBroadcastVolume

Also, the Interface, Handle, and Port blobs have variable length arrays at the end. Typically it is a name of something, IE: COM1 or something like that.  So they could be heaped with a Guard like is done else where.  The final type for the DeviceEvent could be something like
pub enum DeviceBroadcastType {
  DeviceInterface(DevBroadcastDeviceInterfaceGuard),
  Handle(DevBroadcastHandleGuard),
  Oem(DEV_BROADCAST_OEM),
  Port(DevBroadcastPortGuard),
  Volume(DEV_BROADCAST_VOLUMN)
}

You can make the DeviceEvent tuple look like below.
DeviceEvent(co::DBT,  DeviceBroadcastType)

or
struct DeviceEvent {
  event: co::DBT,
  type: DeviceBroadcastType
}

Note that when you fully describe the blobs (with guards) I think you can get rid of the lifetime.
PowerEvent
The POWER_BROADCAST_SETTING only happens when there is a PowerSettingsChange. none of the other PowerEvents include the POWER_BROADCAST_SETTING argument, so i think it misleading to make it optional. It could be enumified similar to the DeviceEvent.
pub enum PowerEvent {
  PowerStatusChange,
  ResumeAutomatic,
  ResumeSuspend,
  Suspend,
  PowerSettingChange(&'a POWERBROADCAST_SETTING)
}

Also the POWERBROADCAST_SETTING can be further parsed into a GUID describing the POWER change. Forexample, if on a laptop and the battery is decreasing, you would get a power event with a GUID of  GUID_BATTERY_PERCENTAGE_REMAINING. And the data field is a u32 between 0-100 representing the battery life...  The data field is different per GUID, so I imagine a final PowerBroadcastSetting type to be a enum as well.
pub enum PowerBroadcastSetting {
	AcDcPowerSource(co::SYSTEM_POWER_CONDITION),
	BatteryPercentageRemaining(u32),
	ConsoleDisplayState(co::DISPLAY_STATE),
	GlobalUserPresence(co::USER_PRESENCE),
	IdleBackgroundTask,
	MonitorPowerOn(co::MONITOR_STATE),
	PowerSavingStatus(co::POWER_SAVING_STATUS),
	PowerSchemePersonality(co::POWERSCHEME_PERSONALITY),
	SessionDisplayStatus(co::DISPLAY_STATE),
	SessionUserPresence(co::USER_PRESENCE),
	LidSwitchStateChange(co::LIDSWITCH_STATE_CHANGE),
	SystemAwayMode(co::SYSTEM_AWAYMODE),
}

Also note that the when fully describing the blob I believe you can also get rid of the lifetime here too.
SessionChange and TimeChange
These fields are less important for my use case, however, they are not dynamically lengthed, so the enum can fit the entire parameter in those since the size is fixed. In this case you could also get rid of their lifetimes too.
So concluding I think by fully describing the blobs of data you discover the whole lengths and therefor can remove lifetimes by sticking the known types into the enum (but you need guards for the variable length DEVICE events that describe names of devices).

I uploaded the structs derived from DEV_BROADCAST_HDR.
But I'm puzzled about DEV_BROADCAST_HANDLE. The binary dbch_data field is a BYTE, but the dbch_nameoffset field talks about a string... so is this BYTE* always a string?

I think the dbch_size member gives the length of the data as it gives the length of the whole structure, including the data. The name_offset seems like an index to a string within the data. I haven't found how it is used though. Seems strange.

Hi Tom,
I wrote an implementation for the RegisterServiceCtrlHandlerEx enum argument, does it look good?
The POWEREVENT stuff is not there yet, but if everything is fine so far, it will be the next one I'll tackle.

@rodrigocfd I think so far it looks pretty good.  As far as DEV_BROADCAST_HANDLE goes, somebody can raise issue if they ever need to parse the dbch_data field. Likely if a person needs to parse the dbch_data field they'll be in a position to understand it better. So the DeviceEvent looks fairly complete as far as I can tell.
Also my previous thoughts about getting rid of the lifetime is good to ignore, I think it is more correct how you have it because there is no need to copy these bytes.

Hi Tom,
I implemented the power event stuff, could you please take a look?

@rodrigocfd
I think the API distinguishes "Monitor" state from "Display" state, where Display can be tri positioned as on/off/dimmed. and the "Monitor" state is either on or off.  So you could probably replace the MonitorPowerOn enum member with a bool as similar to as you did for the PowerSavingsStatus.  So reusing the co::MONITOR_DISPLAY_STATE is (maybe) subtly inaccurate because it implies the MonitorPowerOn can be dimmed.
(Maybe it can be though - who knows what windows will stick in there - so perhaps your experience with windows is handing perhaps undocumented but probable behavior ::shrug::)
I think i'll be able to have have more valuable feedback when I am able to start using it though.
Thank you for the update.

I guided myself through the Windows headers. The MONITOR_DISPLAY_STATE does exist, but not a constant with just the off/on values. My guess is they simply reused the values without taking care about an invalid state for MonitorPowerOn, so in order be more accurate, we must diverge a little.
I made a small change, and now MonitorPowerOn is just a bool, which seems reasonable to me.
Do you think I can merge your commits now? If so, I need you to resolve the conflicts.
Anyway, even if you're not 100% sure of the implementation now, we can change some stuff in the future, when you really start using it. No problem.

@rodrigocfd I merged master and resolved conflicts.  The 3 commits are the original commit, rebasing with master to fix conflicts, and then updating the readme numbers.
Its hard to squash it all into 1 commit because there is a conflict with each README on each commit so rebasing is difficult im not sure best way to squash these.

When implementing a service, a lot of the preferable API's do not use window handles.  For example, NamedPipes, and CreateWaitableTimer to name some i need. So if interested I could add those. They should not be no where near as difficult as the Svc api so it should go easier.
can try in a seperate PR unless you have some thoughts against adding NamedPipes or waitable timers.

@rodrigocfd
Any thoughts on StartServiceCtrlDispatcher?
It accepts a Null terminated array of Name/Proc. And to make more complicated, the name cannot be static inside the struct, and must point to a string outside of the struct.  See its initialization here:
int main() {
  WCHAR name[] = L"foo";
  const SERVICE_TABLE_ENTRY table [] = {
    {name, proc},
    { nullptr, nullptr}
  }
  ::StartServiceCtrlDispatcher(table);
}

Notice how name cannot be statically created inside the table.
I know the goals are not to provide over abstraction over the windows API. So im curious what the api would look like.
In order to avoid heap allocations and keep simple i kind of like a macro approach which could handle the details of null terminating the table and setting up the name of the service.  something like this:
extern "system" fn service_main(nArgs: u32, args: *mut OsStr) { /* */ }

fn main() {
  winsafe::start_service_ctrl_dispatcher![
    "MyService", service_main
  ];
}

I haven't attempted that though as its fairly opinionated approach i think - maybe you know another way to do it more inline with the normal winsafe api's.
Inside the service_main is where you register the control handler and set service status, which would i think complete the Service Api.

When implementing a service, a lot of the preferable API's do not use window handles. For example, NamedPipes, and CreateWaitableTimer to name some i need. So if interested I could add those. They should not be no where near as difficult as the Svc api so it should go easier.
can try in a seperate PR unless you have some thoughts against adding NamedPipes or waitable timers.

I need more info on this. Create a PR or an issue with your proposal, and let's discuss the idea.

Any thoughts on StartServiceCtrlDispatcher?

An arbitrary array of callbacks looks tricky... I'll need to think about what can be done in such situation.


The prefix is scanned incorrectly in add/load safe flow

Description
The prefix is scanned incorrectly  : the selected network prefix is put before address instead of scanned prefix on loading /add safe with QR code scanning .
If the safe exists on a few networks we will allow to add wrong safe without taking prefix into account
The same if the owner address is scanned during safe creation
Environment

Browser: Chrome and mobile app
Wallet: MetaMask
Safe: rin:0x1230B3d59858296A31053C1b8562Ecf89A2f888b ( the safe exists on both rinkeby and mainnet)
Environment:

https://pr3015--safereact.review-safe.gnosisdev.com/

Steps to reproduce

Add 0x1230B3d59858296A31053C1b8562Ecf89A2f888b on rinkeby on mobile app
Go to the mainnet on web app
add existed safe
scan QR for rinkeby 0x1230B3d59858296A31053C1b8562Ecf89A2f888b safe ( from mobile to web)
Current result: Instead of set up rin prefix eth prefix is set up before the address, so the safe will be added on mainnet ( bit it's not mine from rinkeby)
The same for the safes from another networks: instead of displaying scanned prefix to show that wrong network is used for scanning the selected network prefix is put before the address

Expected result:
Show the scanned prefix instead of putting selected network prefix.

All the QR issues where grouped here https://github.com/gnosis/safe-react/issues/3055


Abram Bugfixes

I put a bare minimum implementation for fetching the event_group information on the live entry page. It does pull the right information, but it needs to be connected to the actual group id. How should we proceed?

I've made some changes in the structure of events and event_groups, namely:

organization_id, concealed, available_live, and auto_live_times are now attributes of event_group, not event

An organization no longer has many events; now an organization has many event_groups, and an event_group has many events.

Could you update the models and behavior?


Horizontal Scroll in Microsoft Browser (IE or Edge)

Environment

Browser: IE 11
Version of smooth-scrollbar: 8.2.4

Issue Summary
I'm using Windows 10 with IE.  I added the horizontal scroll plugin (jQuery) but it does not work in Edge.
Current Behavior
In Firefox I move the scroll mouse and I can see the horizontal information. But in Edge this does not work.
My plugin code is:
var Scrollbar = window.Scrollbar;

function ScaleDeltaPlugin() {
  Scrollbar.ScrollbarPlugin.apply(this, arguments);
}

ScaleDeltaPlugin.prototype = Object.create(Scrollbar.ScrollbarPlugin.prototype);
ScaleDeltaPlugin.prototype.transformDelta = function(delta, fromEvent) {
    if (fromEvent.type == 'touchmove' || fromEvent.type == 'touchend') return
    var deltaTransformation = {
        x: delta.y,
        y: delta.x
    };
    return deltaTransformation;
};

ScaleDeltaPlugin.pluginName = 'scaleDelta';
ScaleDeltaPlugin.defaultOptions = {
  nothing: [],
};

Steps to Reproduce
Scrollbar.use(ScaleDeltaPlugin);
var scrollbar = Scrollbar.init(document.getElementById('my-scrollbar'), {});

Online demo
The demo is here: https://codepan.net/gist/5e75e9584a1a2c739b5e61b19702aaf3

Also this another 2 url have the same issue:

http://workshop.ddd.it/
http://paack.fr/

It seems that wheel events do not work in IE Edge with a precision touchpad. But I remember when I tested it months ago on a Windows PC, it worked as expected 🤔.
There's an issue in the issue tracker which is marked as  "FIXED", though.

In Edge works. In IE 11 it does not work.


[DO-NOT-MERGE][DROOLS-7635] ansible-rulebook : Raise an error when a condition comp…

…ares incompatible types

WIP: This doesn't support alpha index. Some tests fail

Replaced by
https://github.com/apache/incubator-kie-drools/pull/6104
https://github.com/kiegroup/drools-ansible-rulebook-integration/pull/114


fix(telemetry): wrong average volume sizes due to V2 volume counts

https://github.com/longhorn/longhorn/issues/7380

@mergify backport v1.5.x


Improve session factory

Adds support for:

Custom TCPConnector to Application
Config verify_ssl, defaults to True

Todo:

[ ] Update docs

This should fix https://github.com/rbw/snow/issues/31

Codecov Report

Merging #36 into master will increase coverage by 0.00%.
The diff coverage is 0.00%.

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


Honda: Add FW for 2023 Honda Passport

Add missing firmware for the 2023 Passport/Pilot. Extend supported model-years forward to 2023.
Route: 214a25adc54636fc|2023-05-30--17-53-09
Thanks to community Passport owner RAB!

@nuwandavek did you check the max torque for this car yet?


Fixup runtime error in client.go

Without a c.init() in the UseOAuth method, BaseURL must be specified, or this code will result in a SIGSEGV on client.go:121 (122 after this commit).
This is one possible way to fix this issue. Alternatively, implementing a NewClient would probably be reasonable too.

@oxtoacart and @mpminardi are discussing this and considering options. The "v2" API design isn't quite nailed down yet and this might change.

Alternatively, implementing a NewClient would probably be reasonable too.

Personally, I like the style of exporting optional things and telling users to stop touching them after initialization. It makes more for a lot less ceremony than c, err := tailscale.NewClient(tailscale.ClientOpts{.....})) or the sea of zero values in tailscale.NewClient(foo, 0, nil, "", 0, nil)

Thanks for calling this out @hosom ! This should be addressed as of https://github.com/tailscale/tailscale-client-go/pull/112 (with https://github.com/tailscale/tailscale-client-go/pull/115 containing a follow-up bugfix).
Our README for the v2 of the client has also been updated to show the new method of configuring OAuth.

@hosom Thanks for bringing this to our attention and proposing a fix. I'm going to close this PR, but please let us know if you run into any lingering issues.


Fix NetworkStatus.isOffline() on Cordova/iOS

@danielpassos I think we need to replicate this with promise.

need change of usage in queue

@danielpassos What is the story here?

@wtrocki That is exactly that make https://github.com/aerogear/apollo-voyager-ionic-example/pull/7 works. Not sure if I got you


decoder option to ignore invalid UTF-8 on the end of input

Hello!
I have a case where client code may receive truncated byte string from server. The string completely valid as UTF-8 except cropped ending. It would be nice to allow ignoring broken endings.

Coverage increased (+0.4%) to 93.007% when pulling a893c9c2bb187ac8950aed896d66f749e9fae66f on amje:ignore-broken-ending into 5566334e1aa5347ba652c38dc186df08b47d8fb9 on mathiasbynens:master.

Coverage increased (+0.5%) to 93.103% when pulling 158c9b177618de04cbbe0fa36bfe6d4f64f09fcb on amje:ignore-broken-ending into 5566334e1aa5347ba652c38dc186df08b47d8fb9 on mathiasbynens:master.


improve getting started guide // include legacy files

Basically, the aim of this PR is to concentrate every information a person wanting to help Neomutt in the getting-started section

This is work in progress
After i finished, i think i keep this PR open for some days/a week, due to its big changes. At this step, i will ask people for a review.
You can comment of course at every time.
this is also part of this PR, because i want to include for instance devel-notes.txt

Questions:

What should be the relationship between CONTRIBUTING.md and the getting-started guide? What should be done in which place?

@flatcap can this also be merged then?


feat: Support Telegram Notification provider

#1382

https://github.com/casdoor/casdoor/assets/98513748/06707505-5af4-4963-bd0b-294c5f78f9fb