AssemblyLevel::FULL on device not working

Hello!
I am trying to use AssemblyLevel::FULL on a (Par)BilinearForm in an attempt to get the assembly of my linear system to be done on an AMD GPU.  Following is a minimal example of what I am trying to do.
#include "mfem.hpp"

using namespace mfem;

int main(int argc, char *argv[])
{

    Mesh mesh(argv[1], 1, 1);

    Device device("hip");
    device.Print();

    H1_FECollection fec(1, mesh.Dimension());

    FiniteElementSpace fespace(&mesh, &fec);

    ConstantCoefficient one(1.0);
    BilinearForm a(&fespace);
    a.AddDomainIntegrator(new DiffusionIntegrator(one));
    a.SetAssemblyLevel(AssemblyLevel::FULL);
    a.Assemble();

}

This does not work, however, and running the program fails with the following error:
Verification failed: (mode == DofToQuad::FULL) is false:
 --> invalid mode requested
 ... in function: virtual const mfem::DofToQuad &mfem::ScalarFiniteElement::GetDofToQuad(const mfem::IntegrationRule &, DofToQuad::Mode) const
 ... in file: fem/fe/fe_base.cpp:500

Loading the program into rocgbd and breaking on the location of the failing MFEM_VERIFY gives the following:
Reading symbols from ./test-fa...
(gdb) break fem/fe/fe_base.cpp:500
Breakpoint 1 at 0x127f47c: fem/fe/fe_base.cpp:500. (2 locations)
(gdb) run
Starting program: /home/iverh/test-fa/test-fa /home/iverh/D1/data/meshes/martinez-navarro-etal/mesh/gmsh/heart01.msh
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib/x86_64-linux-gnu/libthread_db.so.1".
Boundary elements with wrong orientation: 3108 / 3108 (fixed)
[New Thread 0x155536ab9700 (LWP 78221)]
Device configuration: hip,cpu
Memory configuration: host-std,hip

Thread 1 "test-fa" hit Breakpoint 1, mfem::ScalarFiniteElement::GetDofToQuad (this=0x1ab75b0, ir=..., mode=mfem::DofToQuad::FULL) at fem/fe/fe_base.cpp:500
500        MFEM_VERIFY(mode == DofToQuad::FULL, "invalid mode requested");
(gdb) p mode
$1 = mfem::DofToQuad::FULL
(gdb) c
Continuing.

Thread 1 "test-fa" hit Breakpoint 1, mfem::ScalarFiniteElement::GetDofToQuad (this=0x1a5ffd0, ir=..., mode=mfem::DofToQuad::TENSOR) at fem/fe/fe_base.cpp:500
500        MFEM_VERIFY(mode == DofToQuad::FULL, "invalid mode requested");
(gdb) p mode
$2 = mfem::DofToQuad::TENSOR
(gdb) c
Continuing.

Thread 1 "test-fa" hit Breakpoint 1, mfem::ScalarFiniteElement::GetDofToQuad (this=0x1a5ffd0, ir=..., mode=mfem::DofToQuad::TENSOR) at fem/fe/fe_base.cpp:500
500        MFEM_VERIFY(mode == DofToQuad::FULL, "invalid mode requested");
(gdb) up
#1  0x000000000112baf7 in mfem::DiffusionIntegrator::AssemblePA (this=0x1453830, fes=...) at fem/bilininteg_diffusion_pa.cpp:391
391        maps = &el.GetDofToQuad(*ir, DofToQuad::TENSOR);
(gdb) up
#2  0x0000000000f8fda4 in mfem::DiffusionIntegrator::AssembleEA (this=0x1453830, fes=..., ea_data=..., add=false) at fem/bilininteg_diffusion_ea.cpp:249
249        AssemblePA(fes);
(gdb)
#3  0x0000000000fd70a9 in mfem::EABilinearFormExtension::Assemble (this=0x1724440) at fem/bilinearform_ext.cpp:528
528           integrators[i]->AssembleEA(*a->FESpace(), ea_data, i);
(gdb)
#4  0x0000000000fda649 in mfem::FABilinearFormExtension::Assemble (this=0x1724440) at fem/bilinearform_ext.cpp:852
852        EABilinearFormExtension::Assemble();
(gdb)
#5  0x0000000000b6c122 in main (argc=2, argv=0x7fffffff61a8) at test-fa.cpp:22
22          a.Assemble();
(gdb)

The mesh I am using is a heart model consisting of tetrahedron elements. The performance page (https://mfem.org/performance/) says that

Note that partial assembly is currently implemented for tensor-product elements (i.e. quadrilaterals and hexahedra). Partial assembly for simplex elements (triangles and tetrahedra) is planned.

and from the call stack in rocgdb it seems that some partial assembly is performed when doing full assembly. Is this the reason why it is failing? Is it not possible to use  AssemblyLevel::FULL on a tetrahedron mesh?
I am using MFEM version 4.5 compiled with HIP support, HYPRE 2.24 compiled with HIP support, and AMD ROCm version 5.1.3. The node I am running on has an AMD MI100 GPU.

I compiled the minimal example with builds of mfem and hypre without HIP support and I am getting the same error, so it seems not to be related to device execution.

Hello @ivhak,

and from the call stack in rocgdb it seems that some partial assembly is performed when doing full assembly. Is this the reason why it is failing? Is it not possible to use AssemblyLevel::FULL on a tetrahedron mesh?

AssemblyLevel::FULL does end up calling some of the "partial assembly" routines as part of the algorithm. PA is only supported in MFEM on elements with tensor-product elements (quads, hexes). For this reason, I believe FA is not expected to work on meshes containing any other element type (there is some support for PA on simplex meshes using the libCEED backend, but I don't know if that will work with full assembly — this would be a question for @YohannDudouit).

Hi @pazner @YohannDudouit @ivhak, for non-tensor element support for AssemblyLevel::FULL, the code currently does not make use of libCEED but it looks like it could: libCEED provides functions CeedOperatorLinearAssembleSymbolic and CeedOperatorLinearAssemble which could use the libCEED operators constructed/assembled from the PABilinearFormExtension object and then call these routines. The output I/J/data arrays could then be copied into the a->mat SparseMatrix as FABilinearFormExtension does already.
I'm not sure the cleanest way to implement this alongside the current native MFEM AssemblyLevel::FULL support (which uses EA, which in turn uses PA), but there should be way to take advantage of this functionality I would think. Perhaps we use this issue as a place to brainstorm implementations?

Hi @sebastiangrimberg ,
This is correct, the change should be fairly straightforward from the libCEED interface side, this was on my TODO list but didn't get the chance to get to it. Regarding the inheritance relation between PA -> EA -> FA this was my design, but as I said a couple of times in the past I don't think it was a particularly good idea, I think composition, and/or using temporary objects would have made a better design.
If I'm not mistaken libCEED assembles into a COO sparse matrix format, so in order to be compliant with MFEM, we would have to call some of the CUDA/HIP functions to transform COO into CSR (this could potentially also be an option).


navigate through cells in interactive window

As far as I can see , there is no way at the moment to navigate through cells in interactive window -- at the moment, one has to do it by hand , clicking on 'delete' button of every cells
That would a very nice feature  to have, to clean up the interactive window, before exporting , without having to re run the notebook , pretty much like it happens already in a jupyter notebook -- arrows 🔽  / 🔼   + delete
(also, sometimes , we wanna maybe just show few cells / sections of the interactive script )
thanks

you can set a shortcut for interactive.history.focus to do this, though I should probably also just add a default keybinding.

nice! thanks a lot

(I figure out that I can delete outputs of previous cells with cmd+backspace )


Modified job to detect "JavaScript heap out of memory" errors.
Related issue(s): #2262
Fixes #

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 75.15%.  Consider uploading reports for the commit 9ab4317 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2263   +/-   ##
=======================================
  Coverage   75.15%   75.15%
=======================================
  Files          13       13
  Lines         644      644
  Branches      118      118
=======================================
  Hits          484      484
  Misses        115      115
  Partials       45       45

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Addressed here https://github.com/hashgraph/hedera-json-rpc-relay/pull/2305
So we can maybe close this PR : )

Replaced by https://github.com/hashgraph/hedera-json-rpc-relay/pull/2395


prevent null backgroundColor on BottomNavigationBarType.shifting #117991

fixes issue #117991
Description
This PR prevents null backgroundColor BottomNavigationBarType.shifting
Screenshot :
BottomNavigationBarType.fixed :

BottomNavigationBarType.shifting and the backgroundColor is null :

fixed BottomNavigationBarType.shifting backgroundColor null issue :

Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Thanks for contributing.
Please see https://github.com/flutter/flutter/issues/117991
As a result, closing this PR.

i think you should merge this pr again to prevent null backgroundColor


Issue with createGlobalState

createGlobalState stops tracking changes after component that first used global state is disposed.
Tried it with vue@3 and can reproduce it there too, so it is not  @vue/composition-api issue.
Added pull request with reproduction tests: #129

This is a limitation of current Vue's API design: https://github.com/vuejs/vue-next/issues/1532
I am already working on an RFC to addressing this: https://github.com/vuejs/rfcs/pull/212
Thanks

Thanks for info. Will monitor RFC then.
Maybe this caveat should be added to vueuse docs for now?

Inspired by effectScope I have fixed #129 by creating new Vue component to hold reactive scope. Seems to work fine with Vue 2 and 3

I think that will do for now, thanks!

Released in 4.0.0-beta.16.


Add coarse level class names for OnlineProductsDataset

Is it possible to provide fine_label_names (simply label_names)?

It has to be generated by a program.
For instance, the first class belonging to bicycle super-class would be bicycle_0000.

@yuyu2172 Could you solve the conflict?

OK


Add support for overriding IdP Entity ID per SAML Service Provider in WSO2 IS

The following tasks are completed :

Implement support for overriding the IdP Entity ID of resident IdP by an Alias specified in SAML SSO inbound configuration of Service Providers.
Implement support for registering multiple SAML Service Providers with the same SAML ‘issuer’ by using an extra qualifier to identify correct SP.
Implement support for validating SAML assertions with overridden IdP Entity ID values in SAML2 Bearer Grant Profile for OAuth.
Implement unit tests for the implementations.
Prepare documentation on the new features added to WSO2 IS.

Pending task:

Implementing integration tests to test SAML SSO flow with entity id overriding enabled.

Following are the links of the prepared documents :

Configuring SAML SSO (Introducing new attributes - Issuer Qualifier and IdP Entity ID Alias) - https://docs.google.com/document/d/1m67T85r42qeTsJuzQtgUZmqM_7xpbDMeQn7hpr3FGis/edit?usp=sharing
Federating Multiple Office 365 Domains with WSO2 IS using SAML 2.0 - https://docs.google.com/document/d/1vxLF9o3qJpujOg92GjxyYT2EREfWKJFkE4Z0bik3g5Q/edit?usp=sharing
Overriding IdP Entity ID per SAML SP - https://docs.google.com/document/d/1V5dD-50AqrSLrp4S_4Cad-Y8_mi89Hkxsa0UYj2JVRI/edit?usp=sharing

Following are the PRs :

https://github.com/wso2/carbon-identity-framework/pull/1999
https://github.com/wso2-extensions/identity-inbound-auth-saml/pull/201
https://github.com/wso2-extensions/identity-inbound-auth-oauth/pull/994

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


"could not create val_spec" is not very helpful

When one is missing the penultimate unit parameter in the *_specs.txt file,
this is the error message to get.
There could be a more helpful message, or at least that would be nice.

luckily, the function spec for which the generation failed is printed just before, but still
it took me some time to realize what was wrong with my spec.

Yeah, you've hit one of the items on my to do list...the error messages are generally pretty unhelpful.  I still haven't gotten the hang of getting angstrom to produce nice error messages, to be honest.  But I agree, they aren't that helpful as is, and need some work.

I added a bit more info to some of the failures.  I wouldn't call the problem fixed, but it's moving in a better direction at least.


Configuration broken when upgrading from 2.0.0 to 2.0.1

Hello,
Here is my config:
// .php_cs
$finder = PhpCsFixer\Finder::create()
    ->in([__DIR__.'/src', __DIR__.'/tests', __DIR__.'/bin'])
;

return PhpCsFixer\Config::create()
    ->setUsingCache(false)
    ->setFinder($finder)
    ->setRules([
        '@Symfony' => true,
        'psr0' => false,
        'unalign_equals' => false,
        'unalign_double_arrow' => false,
        'ordered_imports' => true,
        'array_syntax' => ['syntax' => 'short'],
        'header_comment' => [
            'header' => '[header]',
            'commentType' => PhpCsFixer\Fixer\Comment\HeaderCommentFixer::HEADER_COMMENT,
        ],
    ])
;

It runs fine on 2.0.0, but since last composer update providing 2.0.1:

[PhpCsFixer\ConfigurationException\InvalidConfigurationException (16)]
The rules contain unknown fixers (unalign_equals, unalign_double_arrow).

Is it a merge mistake? Fabbot.io seems to be broken too, see last PRs on symfony/symfony.
Thanks

That is odd as the fixers have been removed in 2.0
(these are no part of binary_operator_spaces through configuration of that fixer);
Details can be found here:
https://github.com/FriendsOfPHP/PHP-CS-Fixer/blob/v2.0.0/UPGRADE.md#renamed-rules
Looking at your configuration you can remove both the lines;
  'unalign_equals' => false,
  'unalign_double_arrow' => false,

as these are the default configuration values for the binary_operator_spaces which is part of the @Symfony set (which is all so part of your config).

keradus@keradus:~/github/FriendsOfPHP/PHP-CS-Fixer$ git co v2.0.0-alpha
HEAD is now at d0d76b43... minor #1875 renamed Composer package (fabpot)
keradus@keradus-workLap:~/github/FriendsOfPHP/PHP-CS-Fixer$ php php-cs-fixer fix --rules=unalign_equals
Loaded config from "/home/keradus/github/FriendsOfPHP/PHP-CS-Fixer/.php_cs.dist".
^C
keradus@keradus:~/github/FriendsOfPHP/PHP-CS-Fixer$ git co v2.0.0-beta
Previous HEAD position was d0d76b43... minor #1875 renamed Composer package (fabpot)
HEAD is now at 962b2c53... prepared the 2.0.0-beta release
keradus@keradus-workLap:~/github/FriendsOfPHP/PHP-CS-Fixer$ php php-cs-fixer fix --rules=unalign_equals
Loaded config default from "/home/keradus/github/FriendsOfPHP/PHP-CS-Fixer/.php_cs.dist".

  [UnexpectedValueException]
  Rule "unalign_equals" does not exist.

fix [--path-mode PATH-MODE] [--allow-risky ALLOW-RISKY] [--config CONFIG] [--dry-run] [--rules RULES] [--using-cache USING-CACHE] [--cache-file CACHE-FILE] [--diff] [--format FORMAT] [--] [<path>]...

keradus@keradus:~/github/FriendsOfPHP/PHP-CS-Fixer$ git co v2.0.0
Previous HEAD position was 962b2c53... prepared the 2.0.0-beta release
HEAD is now at f3baf72e... prepared the 2.0.0 release
keradus@keradus-workLap:~/github/FriendsOfPHP/PHP-CS-Fixer$ php php-cs-fixer fix --rules=unalign_equals
Loaded config default from "/home/keradus/github/FriendsOfPHP/PHP-CS-Fixer/.php_cs.dist".

  [UnexpectedValueException]
  Rule "unalign_equals" does not exist.

fix [--path-mode PATH-MODE] [--allow-risky ALLOW-RISKY] [--config CONFIG] [--dry-run] [--rules RULES] [--using-cache USING-CACHE] [--cache-file CACHE-FILE] [--diff] [--format FORMAT] [--] [<path>]...

rule was available on v2.0.0-alpha, but not beta or stable release.
for fabbot, ask it's maintainer


Unsupported key: options

Category

[ ] fzf binary
[ ] fzf-tmux script
[ ] Key bindings
[ ] Completion
[ ] Vim
[X ] Neovim
[ ] Etc.

OS

[ X] Linux
[ ] Mac OS X
[ ] Windows
[ ] Windows Subsystem for Linux
[ ] Etc.

Shell

[ ] bash
[X ] zsh
[ ] fish

Just upgraded to newest version of neovim. Now when I reload and open fzf, I am getting error about unsupported key: options
Thoughts?

This should fix it:
diff --git i/plugin/fzf.vim w/plugin/fzf.vim
index 065701c..cb80b61 100644
--- i/plugin/fzf.vim
+++ w/plugin/fzf.vim
@@ -465 +465 @@ function! s:execute_term(dict, command, temps) abort
-  function! fzf.switch_back(inplace)
+  function! fzf.switch_back(inplace) dict
@@ -474 +474 @@ function! s:execute_term(dict, command, temps) abort
-  function! fzf.on_exit(id, code)
+  function! fzf.on_exit(id, code) dict

Thanks @mhinz, but I'm still seeing errors with the patch :/

on_exit() needs 3 arguments.

I'm seeing the same errors with latest HEAD neovim and fzf

@justinmk is right, adding a param to on_exit fixes things
-  function! fzf.on_exit(id, code)
+  function! fzf.on_exit(id, code, wat)

Still getting this with:
NVIM v0.2.0
0.16.8 (brew)
MacOS El Capitan 10.11.4

see this issue again with 0.21.1 (334a4fa)


[Question] invalid operation: cannot index routine.NewThreadLocal (value of type func() routine.ThreadLocal)

Is there an existing issue for this?

[X] I have searched the existing issues

Question
The example "Use ThreadLocal" does not work, it reports "invalid operation: cannot index routine.NewThreadLocal (value of type func() routine.ThreadLocal)", "invalid operation: cannot index routine.NewInheritableThreadLocal (value of type func() routine.ThreadLocal)
"
golang version 1.21
Code of Conduct

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

@Castlebin You can use v1.1.3 now. Please try again

@Castlebin You can use v1.1.3 now. Please try again

Thanks!


Django Template Language Breaks Highlighting/Intellisense for Javascript Blocks

If` you insert any Django Template Language tags such as {{csrf_token}} or {% comment %} within a script block inside an html page, all of the javascript highlighting and intellisense will be removed from everything within that block and it will only highlight and provide intellisense for the DTL tag. Other javascript blocks will be unaffected.
Example:
<script type="text/javascript">
    var a = "foo";
</script>

vs
<script type="text/javascript">
    var a = "{{ bar }}";
</script>

If this is in fact a bug, it was also present in VS2013 with Pytools installed.

Thanks! We already have a bug for this, so I'll close this as a duplicate.
https://github.com/Microsoft/PTVS/issues/1098


normalize capacity by number of batteries

On laptops with more than one battery the calculation of capacity is broken: the sum of capacities of batteries is displayed (and if this turns out to be bigger than a 100%, it gets clamped to 100%).
The total_capacity is computed as a sum over individual capacities of batteries, which can be in the range from 0% to number of batteries * 100%. If total_capacity was calculated, the calculated_capacity is set to the same value. However, all values over 100% are clamped, since, as explained in the code comment:

// This can happen when the battery is calibrating and goes above 100%

I propose the calculated_capacity should be normalized by the number of batteries.

Interesting. This highlights the issue with the battery module which existed way before the change.
My hardware (ThinkPad X1Y6) has only one battery. But there's two batteries reported:
% cat /sys/class/power_supply/BAT0/capacity
78
% cat /sys/class/power_supply/wacom_battery_0/capacity
100
% cat /sys/class/power_supply/wacom_battery_0/type
Battery

The latter one is a built-in Wacom stylus, which in its dormant state reports capacity 0. If you use it even once since boot, it'll start reporting the actual capacity and affect the total (don't know if it's a firmware or a kernel bug, doesn't matter for this discussion). I don't use it often, so it was on 0 most of the time.
So after the change I see either 39% or 89% instead of the actual 78% and will have to configure acceptable battery names in the waybar config :)
...which seems to be a good idea overall. I expect that some complaints could be fixed by limiting the module to BAT* or even a set of common battery name patterns.

Thank you @alebastr, this might very much be related to @Estebiu s issue from #1792! I like the idea of filtering down to BAT* by default.

@Estebiu, could you post your ls /sys/class/power_supply/?

For what it's worth, i also have the issue with battery percentage getting clamped. I just tried building master and this PR has indeed fixed it for me :)
I'm on an Lenovo X260 with two batteries
My /sys/class/power_supply looks like this.
ls -la /sys/class/power_supply
total 0
drwxr-xr-x  2 root root 0 Nov 17 07:44 .
drwxr-xr-x 72 root root 0 Nov 17 07:44 ..
lrwxrwxrwx  1 root root 0 Nov 17 07:44 AC -> ../../devices/LNXSYSTM:00/LNXSYBUS:00/PNP0A08:00/device:00/PNP0C09:00/ACPI0003:00/power_supply/AC
lrwxrwxrwx  1 root root 0 Nov 17 07:44 BAT0 -> ../../devices/LNXSYSTM:00/LNXSYBUS:00/PNP0A08:00/device:00/PNP0C09:00/PNP0C0A:00/power_supply/BAT0
lrwxrwxrwx  1 root root 0 Nov 17 07:44 BAT1 -> ../../devices/LNXSYSTM:00/LNXSYBUS:00/PNP0A08:00/device:00/PNP0C09:00/PNP0C0A:01/power_supply/BAT1

Hey, sorry for the late reply. Busy week.
As @alebastr it's saying, I think that I'm also encountering his problem.
I have a laptop with a wacom pen- thing is, it's never been able to report it's battery correctly. Even on GNOME it says that it's battery it's low- 0%.
So waybar it's probably counting the pen's battery togheter with the laptop one.

@Estebiu nice! That means ignoring batteries different than BAT* and CMB* when autodetecting should work.
Speaking of which, I restored the branch, rebased and added a commit that should do just that. Could someone test it? Sadly I don't have any unusual batteries like you guys, so I'm coding in the dark.

Hm, sad thing is I think github is no longer tracking changes on that branch, maybe bc the PR is closed. Should I open a new one? Kinda don't wanna lose this discussion. Anyway, here's a direct link to the branch:
https://github.com/grfrederic/Waybar/tree/normalize-battery-capacity

I just tried a "half test": if I set the string to ignore batteries other than BAT1, it works correctly on my machine, that is it only uses BAT0 when autodetecting. I am now relatively confident that the fix should work for other people. If no one has the time to try it next 1-2 days I'll open a new PR with the fix.

I'm just going to make things worse and tell you that CMBx/BATx are the names specific for Linux ACPI battery driver.
Ex. Pinebook: /sys/class/power_supply/axp20x-battery, PBP: /sys/class/power_supply/cw2015-battery, M1: /sys/class/power_supply/macsmc-battery.
I also suspect there are some ancient laptops where the battery is not routed through ACPI 😞
Maybe a list of globs would work? Something like "bat": ["BAT*", "CMB*", "*battery"], with these values as the defaults.

@grfrederic I built your branch and it shows correct battery capacity now.

@alebastr i think it's a good idea, it should work out of the box most times and give a lot of leeway to fix things in configuration in trickier cases.
@abcdw nice! i'll check then if i can work in @alebastr's suggestion easily and we're good to go.

i pushed code that allows for regexes, @alebastr wdyt?
also, how do we set a default? do we just update the one in the reference config and assume that that's the one users will copy over?

btw i notice that your suggested *battery regex would exactly match @Estebiu 's pen...

i'm thinking maybe the regexes are a bit overkill after all, we could go back to BAT* and CMB* if battery is not given, and besides that allow for a string (single battery) or list of strings (for multiple batteries). i don't think people will often have to configure this, and if they do, they can just write out the batteries they want.

I've just found some time and figured out how UPower distinguishes system batteries from various peripheral devices - #1862.
If that works, all the regex/name checks I suggested are unnecessary. Sorry about that.
@Estebiu can you please check if /sys/class/power_supply/hid-0018:04F3:2BB3.0001-battery/scope exists and contains "Device"? If it is, my PR should address the issue.

Yes, ´/sys/class/power_supply/hid-0018:04F3:2BB3.0001-battery/scope´ exist and contains 'Device'.

Thanks @alebastr, thats a neat solution ;)


Check al links on website, many might be broken

Is there a way to check that the links on the posts are not broken? Example post

Related, should we have something that says "this project is no longer maintained/has moved to..." ?

ok.
so I had a browse though the pages and here is a sum up of broken links/images (there might be more, but I think it would be a pretty good advance if we dealt with these):
Update (some have faulty html code, some miss image, some have layout issues, etc)

[ ] Super releaser page needs an image. It also has turned to a company, and I couldn't find the documentation on a quick look at their page
[ ]  data repositories
[ ]  autoreward 2
[ ] big neuron
[ ] BPM biosignal (image)
[ ] brain browser (image)
[ ]  data acquisition and signal processing
[ ]  fingertip laser (image)
[ ]  green brain (image)
[ ]  ipipet
[ ] lab management software (image)
[ ] neuromorpho (image)
[ ] neurobytes (embedded video)
[ ] nibabel
[ ] nipy
[ ] nosepoke device
[ ] open hand
[ ] open microscopy environment
[ ] open flexure
[ ] open fuge (image)
[ ] pulse pal
[ ] psychophysics toolboxes
[ ] python, numpy, scipy
[ ] raspberry pi based
[ ] red pitaya
[ ] school of data
[ ] spike gadgets
[ ] Skinner box with RPi + python

Remove

[ ] Open Prosthetics and Robots page is an umbrella page (just references other internal pages). But then we need to add openrobothardware.org and sofrobotstoolkit.con (as they are only referenced on this page)
[ ] Arduino posner test
[ ] schematics posner test
[ ] Neuroimage database is umbrella
[ ] microcontrollers is umbrella
[ ]  computer clusters is umbrella
[ ]  computers is umbrella
[ ]  data analysis page is umbrella
[ ]  database & sharing is umbrella
[ ] microscopes is umbrella
[ ]  human psychophysics using arduino
[ ] opening MCD files with python
[ ]  python record script posner test

Cool, I will do the remove ones first.
I will try to do the others as I can. Sometimes it happened that I didn't know what the solution was, I might need help for those (e.g., locating the actual picture or video that was supposed to show). I guess #1 can help identify those.

no worries, let me know the ones you need help with and I can try to work them out on my end!

Just dropping a message here to ping you, since I've worked on some posts!

ok, mostly done. the two projects that need new page, I'll add as new issues and we can contact the maintainers of the projects when our form system is up and running..


background-image from resources

Hi,
when I set background image in css for a section  from resources folder it is not appear. (it seems the renderer looking for it in dist folder, how I can add my images to dist folder or add to bundle?)
Thanks for your good boilerplate and Support :)

The resources directory is actually intended to be more for build assets. I would recommend adding a ./app/img folder. See this project as an example.


Nextcloud 11 login test not working

Steps to reproduce the behaviour

Go to setting and back up on owncloud
Input nextcloud11 server adress(http) and credentials
When pressing test it says oc server ok, but login details incorrect

Expected behaviour
Nextcloud 11 should still be fully compatible with owncloud, or did they change the login Api?
Actual behaviour
Software specifications

GnuCash Android version: 2.1.3
System Android version: 5.1
Device type: x86 Asus Zenfone2

Same with version 2.1.5 on a Moto G 4G (peregrine). Neither user/password nor user/app-pin currently works.

@poVoq can you use the new Save As.. functionality in v2.2.0 with your NextCloud successfully?

Seems that GnuCash is using /index.php/ocs/cloud/user for logging-in, but at least on NextCloud 11 the GET request will get 302 redirected to the login page (/login)

@wenxin-wang are you referring to the "Save As.." functionality, or the logging in via ownCloud integration?

You can use the "Send to..." export target for use with the Nextcloud app. It is not that much convenient (you have to choose the upload folder each time) but it works.
IMHO, this is a more general solution and should work with other cloud storages as well - as long as they provide an Android client. ;-)

Did you set a new APP-Pin for gnucash in the nextcloud settings in the webpanel?

Go to the nextcloud webpanel->press your nickname in the top right-> personal.
Then set a new APP-PIN and copy the new passwort. Paste the passwort into the gnucash panel with your nextcloud Nickname and your Serveradress.

AFAIK the app pin should only be necessary if an external auth system like SAML or LDAP is used in Nextcloud and thus no password is stored in the Nextcloud database.

@poVoq @Johnjustdoit : An app pin is necessary if you activated TOTP for your user. AFAIK, it is fully transparent to the client if you enter your password or an app pin.
Just do as follows:

Log into your Nextcloud Android client with your password.
Activate TOTP via web interface.
Watch your Android client mourn about invalid credentials.
Create an app pin and use that pin instead of your password.
Watch your Android client successfully connect to your NC instance.

That means it won't make any difference for GnuCash.
BTW: Has anyone tried to use GnuCash app with NC12?

Thank you for clearing it up!
I found a workaround to sync it automatically in my private network. (Should also work with a nextcloud reachable in the www)
I use the app Foldersync Pro (3,19€) and the app tasker (2,99€). First I set a periodic export in GnuCash to my SD Card. Then I set a task in tasker to sync all my previous added syncronisations(GnuCash, Photos,...etc) in FolderSync when my phone log into my home WLAN. This is posible for any folder and file on your phone. On my PC I use the nextcloudapp to use the file in my nextcloud folder with GnuCash.

Yep, that should do the trick. :-)
Nevertheless, the NC Android client v1.5 (and above) has the potential to render this toolchain obsolete. See nextcloud/android#285 (it is a meta ticket gathering the work done in separate tickets).

I'm having the exact same problem with Nextcloud 11.0.3. @sehucke tip didn't work (I don't have 2FA enabled atm, which may be the cause)

Which tip did you mean @denysvitali ? I suppose you are referring to https://github.com/codinguser/gnucash-android/issues/651#issuecomment-315597218 ? My explanation of app pin usage was not meant to be a tip for getting GnuCash to work with NC. I tried to explain the exact opposite: It does not matter if you use your password or an app pin inside GnuCash. The client does not know there is a difference.
At the moment there seem to be only two ways to get GnuCash working with NC:

To use "Send to..." as described in https://github.com/codinguser/gnucash-android/issues/651#issuecomment-312599156 or...
... to set up a toolchain like @Johnjustdoit explained in https://github.com/codinguser/gnucash-android/issues/651#issuecomment-315598825. (Which I have not tested so far by myself. Should work anyway.)

Besides that: App pins should work regardless of whether you have enabled 2FA or not. If I remember correctly, I have successfully tried that before enabling 2FA in my NC installation.
You can easily verify if your app pins work correctly. Just generate a new pin and use an app to log in with it (NC, DAVdroid, CalDAV, CardDAV...). However, atm this won't help you with this GnuCash problem.

I'm already working on a fix to integrate NextCloud into GnuCash...
I thought NC and OC had the same authentication mechanism

With Nextcloud 12 I could successfully work around the Issue with an entry in my webserver config (in this case apache2 with virtualhosts):
RewriteEngine On
RewriteRule ^/index.php/ocs/cloud/user /ocs/v1.php/cloud/user

Fixed develop branch (fc261f7).


Enable operator installation for IBM Cloud

It is convenient that the operator supports installation on IBM Cloud. Potential work items are:

[x] Providing a kustomize overlay for IBM Cloud
[ ] Enable multi-architecture (i.e. amd64 and s390x) artifacts for installation

Any other?

FYI - I have issue https://github.com/confidential-containers/operator/issues/107 to cover getting the coco operator working with s390x. Once that is done (and depending on how). I guess we might need to update the cloud-api-adaptor payload build to base on the new multi-arch image, or have a separate s390x version of the build?

This has been completed with the multiarch work finished.


Render from simulation does not work.

The test was made with the simuator for fluid.
After the simulation baked, prman only render the first frame over and over.
If an other frame is selected in the timeline, only this frame will be rendered (only if the frame 1 already have been rendered)- and it is not possible to change frames in the timeline.
Link to forum thread: https://renderman.pixar.com/forum/showthread.php?s=&threadid=29631
Kind regards
Kristian

Thanks!  I took a quick look yesterday but will get to it more today!
On Tue, Nov 24, 2015 at 1:50 AM, Kristian Mollenborg <
<EMAIL_ADDRESS>wrote:

The test was made with the simuator for fluid.
After the simulation baked, prman only render the first frame over and
over.
If an other frame is selected in the timeline, only this frame will be
rendered (only if the frame 1 already have been rendered)- and it is not
possible to change frames in the timeline.
Link to forum thread:
https://renderman.pixar.com/forum/showthread.php?s=&threadid=29631
Kind regards
Kristian
—
Reply to this email directly or view it on GitHub
https://github.com/bsavery/PRMan-for-Blender/issues/153.

--
<EMAIL_ADDRESS>508-274-8700


Withings OAuth2 not working

Description:
Withings has since moved to OAuth2 and they have since added a required parameter for all endpoints. This causes the the auth to fail since we are not providing action when we request the access token.
As per the new docs, we need to provide requesttoken to the action parameter. This is on top of what is already provided.
What is the best way to customise this section?
open func postOAuthAccessTokenWithRequestToken(byCode code: String, callbackURL: URL?, headers: OAuthSwift.Headers? = nil, completionHandler completion: @escaping TokenCompletionHandler) -> OAuthSwiftRequestHandle? {

        var parameters = OAuthSwift.Parameters()
        parameters["client_id"] = self.consumerKey
        parameters["code"] = code
        parameters["grant_type"] = "authorization_code"

        // PKCE - extra parameter
        if let codeVerifier = self.codeVerifier {
            parameters["code_verifier"] = codeVerifier
            // Don't send client secret when using PKCE, some services complain
        } else {
            // client secrets should only be used for web style apps where they can't be decompiled (use pkce instead), so if it's empty, don't post it as some servers will reject it
            // https://www.oauth.com/oauth2-servers/client-registration/client-id-secret/
            if !self.consumerSecret.isEmpty {
                parameters["client_secret"] = self.consumerSecret
            }
        }

        if let callbackURL = callbackURL {
            parameters["redirect_uri"] = callbackURL.absoluteString.safeStringByRemovingPercentEncoding
        }

        OAuthSwift.log?.trace("Add security parameters: \(parameters)")
        return requestOAuthAccessToken(withParameters: parameters, headers: headers, completionHandler: completion)
    }

OAuth Provider? (Twitter, Github, ..):
Withings
OAuth Version:

[ ] Version 1
[x] Version 2

OS (Please fill the version) :

[x] iOS :
[ ] OSX :
[ ] TVOS :
[ ] WatchOS  :

Installation method:

[ ] Carthage
[ ] CocoaPods
[x] Swift Package Manager
[ ] Manually

Library version:

[x] head
[ ] v2.1.0
[ ] v2.0.0
[ ] v1.4.1
[ ] other: (Please fill in the version you are using.)

Xcode version:

[ ] 11.4 (Swift 5.2)

[ ] 11.x (Swift 5.1)

[ ] 10.x (Swift 5.0)

[x] other: 12

[ ] objective c

I see nothing the with current code
you could subclass and override  postOAuthAccessTokenWithRequestToken
and copy all code (I expect all is public)
if not or to do a better job all code from postOAuthAccessTokenWithRequestToken except the last line could be extracted to
parametersForPostOAuthAccessTokenWithRequestToken() -> OAuthSwift.Parameters()
and then you could also subclass and override only this function, call the super and add only your new attribute
or just a function(need override class too) or closure to customise it


AllOf is not rendering

Merging parts of different objects is fun. OpenAPI understands it but unfortunately cant render it properly in editor and offline documentation.
Simplest example:
asyncapi: 1.0.0
info:
  title: Example API
  version: "1.0"
topics:
  example.topic:
    publish:
      payload:
        allOf:
          - type: object
            properties:
              foo:
                type: string
          - type: object
            properties:
              bar:
                type: string

Results in empty payload table:

AsyncAPI does not mention about its limitations and OpenAPI plays with XOf nicely.

Yes, it's a known limitation. I should probably add it to the documentation meanwhile it's not supported. PRs are welcome too 😊

@approxit a solution has been released and pushed to npm under version 1.8.0.


Failure in parsing ipv4 with leading 0

See the coffeescript code below:
coffee> ipaddr.parse '<IP_ADDRESS>'
{ octets: [ 192, 12, 1, 1 ] }
coffee> ipaddr.parse '<IP_ADDRESS>'
{ octets: [ 192, 10, 1, 1 ] }

Why is the address wrong if i add a leading 0?

Octal. Not a bug but standard behavior in Unix inet_ntoa. E.g.:
$ ping <IP_ADDRESS>
PING <IP_ADDRESS> (<IP_ADDRESS>) 56(84) bytes of data.

This is one of the most confusing address interpretations. A good solution should allow for a flag to auto-trim zeroes before conversion, to avoid the confusion.

It may be confusing, but diverging from the ubiquitous Unix's inet_ntoa would be just more confusing.
Use IPv6, it has no such problems ;p


windows height after split

win10 1809:
2.1 stable:

Issue
When I split the windows with ctrl+shift+d/e, the new windows always at the end of the windows, and I need scroll up to the top position

I have the same problem

I have the same problem

I solve it stupid by add clear in .bashrc, because I found it will going to the end when I clear it. At least we can use it with this method.


Can we get UUID / path of the content store and snapshotter?

We need these information to monitor image filesystem usage, and properly remove images. :)
This may be related to https://github.com/containerd/containerd/issues/1398 in some way.
/cc @abhinandanpb Related to image filesystem metrics.

@Random-Liu We added Usage calls for this use case. We really don't want to expose these paths for the caller, because that becomes a leaky interface. If Usage is not sufficient, let us know why and we'll see if we can make it work.
#1398 should cover removal. Is there not a way to remove snapshots from the API?

@stevvooe Actually, we need to know the capacity of the partition the image filesystem is on, so that we know whether the partition is nearly full, and trigger eviction.

@stevvooe Any thoughts? :)

@Random-Liu That configuration is passed to containerd in the config.toml file when specifying the snapshotter. It is going to depend on how containerd is setup and run.

@stevvooe Hm, for the first stage, we could make this a flag in cri-containerd, and user should make sure it corresponds to what containerd is using on the node.
If that is not enough, in the future, we may need to parse containerd config file, or probably containerd could return its current config in some way.

Close this one for now, we agree that this should be done through configuration.


License Inquiry

Would you be willing to release this under a BSD-style license?
I would like to include it in projects at work in the future, and I'm hoping it can go with me from position to position.

Of course, why not?
On Thu, Mar 25, 2021 at 9:11 PM Todd Millecam @.***>
wrote:

Would you be willing to release this under a BSD-style license?
I would like to include it in projects at work in the future, and I'm
hoping it can go with me from position to position.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ArunaLakmal/Terraform-Backend/issues/3, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AG3UIYXZEJQZEYOOCCFKGSLTFNKS5ANCNFSM4ZZTJNEA
.

--
--
Best Regards,
Aruna Prasad Lakmal
Mobile: +94779515040
[image: http://lk.linkedin.com/in/arunalakmal]
http://lk.linkedin.com/in/arunalakmal
[image: https://www.facebook.com/arandunu]
https://www.facebook.com/arandunu [image:
https://twitter.com/ArunaRandunu] https://twitter.com/ArunaRandunu [image:
https://plus.google.com/+ArunaRandunu]
https://plus.google.com/+ArunaRandunu | https://www.techcrumble.net

Thank you!

@tyggna I have added a MIT License to this and you  can do anything. Let me know if you need anything else. Thank you for the query.


Pool Monitor - Darker GUI for Unity Pro Skin

Added new MpmView settings that are bound if the editor skin is dark.

Thanks


Unable to get property 'getDependencies' of undefined or null reference

Category

[ ] Question
[ ] Typo
[x] Bug
[ ] Additional article idea

I kept on getting this error message on IE11 after bundle + package-solution --ship
[SPLoaderError.loadComponentError]:
***Failed to load component "4710c79c-7ef3-4d1b-8092-a1c7399014f5" (GeneralSearchItemsWebPart).
Original error: Unable to get property 'getDependencies' of undefined or null reference

***INNERERROR:
Unable to get property 'getDependencies' of undefined or null reference
***CALLSTACK:
Error
   at t._generateErrorStackForIE (https://spoprod-a.akamaihd.net/files/sp-client-prod_2018-11-09.013/sp-classic-page-assembly_en-au_ba1edfce542707ee4f1396e9304c3a2e.js:187:16308)
   at t (https://spoprod-a.akamaihd.net/files/sp-client-prod_2018-11-09.013/sp-classic-page-assembly_en-au_ba1edfce542707ee4f1396e9304c3a2e.js:187:15844)
   at t (https://spoprod-a.akamaihd.net/files/sp-client-prod_2018-11-09.013/sp-classic-page-assembly_en-au_ba1edfce542707ee4f1396e9304c3a2e.js:925:22000)
   at e.buildErrorWithVerboseLog (https://spoprod-a.akamaihd.net/files/sp-client-prod_2018-11-09.013/sp-classic-page-assembly_en-au_ba1edfce542707ee4f1396e9304c3a2e.js:925:16595)
   at e.buildLoadComponentError (https://spoprod-a.akamaihd.net/files/sp-client-prod_2018-11-09.013/sp-classic-page-assembly_en-au_ba1edfce542707ee4f1396e9304c3a2e.js:925:12620)
   at Anonymous function (https://spoprod-a.akamaihd.net/files/sp-client-prod_2018-11-09.013/sp-classic-page-assembly_en-au_ba1edfce542707ee4f1396e9304c3a2e.js:925:62209)
   at T (https://spoprod-a.akamaihd.net/files/sp-client-prod_2018-11-09.013/sp-classic-page-assembly_en-au_ba1edfce542707ee4f1396e9304c3a2e.js:8:2116)
   at L (https://spoprod-a.akamaihd.net/files/sp-client-prod_2018-11-09.013/sp-classic-page-assembly_en-au_ba1edfce542707ee4f1396e9304c3a2e.js:8:2232)
   at O (https://spoprod-a.akamaihd.net/files/sp-client-prod_2018-11-09.013/sp-classic-page-assembly_en-au_ba1edfce542707ee4f1396e9304c3a2e.js:8:2026)
   at g (https://spoprod-a.akamaihd.net/files/sp-client-prod_2018-11-09.013/sp-classic-page-assembly_en-au_ba1edfce542707ee4f1396e9304c3a2e.js:8:1608)

I have similar kind of issue when i open a page in IE browser and on other browser like Chrome and Edge its working fine.
Error in IE is given below
[SPLoaderError.loadComponentError]:
***Failed to load component "ecd50544-ca18-4cfc-a797-4c824436a6ff" (LeadershipWebPart).
Original error: Unable to get property 'getDependencies' of undefined or null reference

@paigeflourin - your test app is working for me in chrome, but fails in IE11.  However the failure seems to be different - an error stating "Out of stack space".  I also get a slew of errors in IE11 from within your code.  It's hard to know if this is something that is in your code due to the code itself or from one of the numerous dependencies, or if it is a problem in the framework.  Unfortunately, the debugging experience in IE11 is lacking.
Can you take a look at the exceptions thrown during load and see if they are expected in IE due to included packages?

For the others encountering this - we tracked down the issue and are working on a fix.

Hi @patmill i have commentwd it out yesterday but let me bring it back and we can try again. Thank you!

Hi @patmill  checked IE11 and had getDependencies error instead of stacktrace. I do have errors in IE11 and i have posted it here https://github.com/pnp/pnpjs/issues/360 .

Hi @patmill ,
not sure if i should paste it here but this is the error in IE

Yes - the "unable to get property 'getDependencies'" is the error we are aware of and are working on.  When I ran your sample, I was getting a stack overflow.  Maybe I had a weird environment that you don't have.

@patmill
This happens on spfx 1.7 classic page.
I just managed to fix this issue by removing SPComponentLoader.loadScript and use npm package instead.

So I think there are two things going on.  The first is that there is an issue with some solutions in how they load specific js libraries, which drops into a different codepath that has an error in the error reporting.  When the fix for the error reporting rolls out (hopefully today), we should have a better idea of the underlying issue with the script being loaded.  So @jonathanhotono - the solution to use an npm package instead is probably the right long term one.  Do you mind if I ask what package / script you were trying to load?

@jonathanhotono - is there any chance you could share a repro (or source code) for the failure case?

@patmill you're right, using the npm package is the right long term solution but previously in classic page we had to share certain dependencies with what in the masterpage until everything is fully ported to spfx + modern site.
I loaded a shared momentjs script from cdn to load it and set a globalExportsName to moment, then I rendered the element after.
SPComponentLoader.loadScript("https://cdnjs.cloudflare.com/ajax/libs/moment.js/2.22.2/moment.min.js",{
        globalExportsName: "moment"
      }).then(script=>{
        ReactDom.render(element, this.domElement);
      });

From a comment by MSFT, seems like a fix was rolled out. Can you please verify if you are still seeing the issue?


Replace ElasticSearch with lightweight datastore

ElasticSearch is a very heavy method to run on a Pi just to store a few configuration values. This issue is to replace it with Apache Derby or another lightweight datastore.
Bounty: https://www.bountysource.com/issues/34471155-replace-elasticsearch-with-embedded-derby

i agree.  Some can argue that derby is heavyweight as well.  Others have proposed something as simple as storing the configuration as json on disk.  What do you think?

On-disk JSON would be fine too, I've updated the issue to indicate such.

Might want to consider SQLite as well.

Agreed.. Anything to reduce system load would be great.  I'm running this on a Pi3, and it takes 13 MINUTES from start to ports open and ready to work.

sorry all, my day yob is taking up most if not all of my time.  If someone
has time to do this i can accept the pull, else please be patient!
On Thu, Jul 7, 2016 at 6:50 PM, Michael J. Kidd<EMAIL_ADDRESS>wrote:

Agreed.. Anything to reduce system load would be great. I'm running this
on a Pi3, and it takes 13 MINUTES from start to ports open and ready to
work.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/armzilla/amazon-echo-ha-bridge/issues/76#issuecomment-231240578,
or mute the thread
https://github.com/notifications/unsubscribe/ABTxZoIN7vQdCmnmnfegXeJyp-aP8Cs8ks5qTZDMgaJpZM4IkB-f
.

--
chicken goes cluck cluck, cow goes moo, lemur goes phh phh how bout you?

Hey, no worries Arm... I know I certainly understand life getting in the way.

For the record: startup on my raspberry 2 took about 250s, so I tried to replace ElasticSearch mit h2 and hibernate. Got me down to 240s. You can find my attempt at pvdh/amazon-echo-ha-bridge.
Doesn't seem to be worth the effort. Perhaps better results can be achieved by accessing a standalone database server, instead of an embedded one, perhaps ditching hibernate might be effective as well. However, I don't expect people running the bridge on a raspberry to have a database sever nearby, so I don't think it's worth exploring.

@pvdh 10s doesn't sound worth that effort, but the configuration is simple enough that ditching the database entirely for a simple json file should be sufficient.


JS numeric separators docs task for Fx68

See https://trello.com/c/ZJ6IVDrd/206-numeric-separators-fx-68 for the original trello card.
Acceptance criteria

[ ] Implement Numeric Separators — documented appropriately

This is another one that is only available in Nightly builds right now, so documenting it as part of Firefox 68 is not urgent.

Closing this one; this is going to actually be enabled in Fx 70.


Feat: Added minimum node version to package.json template

I added the minimum node version to the package.json template. This is a good practice, and also necessary for various deployment environments. Added in pursuit of #320.

👍  LGTM


System.InvalidCastException: Unable to cast object of type 'Godot.Resource' to type 'Blackboard'

Godot version
4.2. beta 5 - 4.2 rc2 Mono (C#)
System information
Fedora 39 (XFCE, Gnome, KDE) - Godot 4.1 - 4.1.1, Dotnet 7,8, Ryzen 5950, Geforce 3080, 128gb DDR4 RAM
Issue description
Currenlty using Godot 4.2 rc2 tested also on beta 5. I have a custom resource which I can create from the editor: https://pastebin.com/3Vwu3n6B and assign it to array of resources in my CutsceneAction script which also inherits from the resource and is also marked [Tool] [GlobalClass]. Here are the relavant parts of the other scripts:
using Godot;
using System;
using Godot.Collections;

[Tool]
[GlobalClass]
public partial class WaitForBlackboardValueCutsceneAction : CutsceneAction
{
    [Export]
    public BlackboardSourceType BlackboardSourceType;
    [Export]
    public NodePath UnitPath;
    [Export]
    public Blackboard ReferenceBlackboard;
    [Export]
    public BlackboardCheckType BlackboardCheckType;
}
rest of the code

and then I use the cutscene action here:
using Godot;
using System;
using System.Collections.Generic;
using System.Linq;

public partial class Cutscene : Node2D
{
    public CutsceneController cutsceneController;

    [Export]
    public string CutsceneName = "";

    [Export]
    public Blackboard blackboard = new Blackboard();

    [Export]
    public Godot.Collections.Array<CutsceneAction> CutsceneActions;
    private Queue<CutsceneAction> CutsceneActionsQueue = new Queue<CutsceneAction>();

    private List<CutsceneAction> CutsceneActionsInExecution = new List<CutsceneAction>();
    private CutsceneState state = CutsceneState.Idle;
}

The main collection I have is the exported one. Others are temporary for the cutscene purposes. I'm running into this issue only witht the WaitForBlackboardValueCutsceneAction and I have 15 other cutscene action types where I also use custom resource.
The errors I get on runtime:
ERROR: System.InvalidCastException: Unable to cast object of type 'Godot.Resource' to type 'Blackboard'.
at Cutscene.SetGodotClassPropertyValue(godot_string_name& name, godot_variant& value) in /home/name/Development/GodotProjects/project/godot-project/Godot.SourceGenerators/Godot.SourceGenerators.ScriptPropertiesGenerator/Cutscene_ScriptProperties.generated.cs:line 65
at Godot.Bridge.CSharpInstanceBridge.Set(IntPtr godotObjectGCHandle, godot_string_name* name, godot_variant* value) in /root/godot/modules/mono/glue/GodotSharp/GodotSharp/Core/Bridge/CSharpInstanceBridge.cs:line 57
at: void Godot.NativeInterop.ExceptionUtils.LogException(System.Exception) (/root/godot/modules/mono/glue/GodotSharp/GodotSharp/Core/NativeInterop/ExceptionUtils.cs:112)
But the thing is that the Blackboard can be created via editor:

But after I run the scene the blackboard in editor is gone and not assignable (options left are load, no option to create):

But if I restart the project it's assignable again. Any clue what is going on here?
Steps to reproduce
/
Minimal reproduction project
/

Removing new everywhere in declaration seemed to help to resolve the issue.

Happens to me as well. I got my CustomResource deriving from resource. In two different scenes I do
[Export] private CustomResource resource;
Result is that in one scene I can attach my resource via editor in other I can't;
It also does not work when I right click field and select "new CustomResource" in the "broken" one.
But this:
[Export] private Array<CustomResource> arr;
Works fine.
Also "broken" in my case scene has [Tool] attribute added to it. When I remove [Tool] it works fine.

Same issue here. The [Tool] annotation seem to be the cause for me. I can't assign custom resources to exported variables from nodes with this annotation. Neither by dragging it into the exported variable in the inspector, nor by creating a new resource directly in the inspector.
I always get this same error message in the Output panel.
  modules/mono/glue/runtime_interop.cpp:1324 - System.InvalidCastException: Unable to cast object of type 'Godot.Resource' to type 'Raele.QuadTerrainData'.
     at Raele.QuadTerrain.SetGodotClassPropertyValue(godot_string_name& name, godot_variant& value) in C:\Users\Leonardo Raele\Workspace\mygame\Godot.SourceGenerators\Godot.SourceGenerators.ScriptPropertiesGenerator\Raele.QuadTerrain_ScriptProperties.generated.cs:line 32
     at Godot.Bridge.CSharpInstanceBridge.Set(IntPtr godotObjectGCHandle, godot_string_name* name, godot_variant* value) in /root/godot/modules/mono/glue/GodotSharp/GodotSharp/Core/Bridge/CSharpInstanceBridge.cs:line 57

I had the same problem just now, after assigning a custom resource to my [Tool] class. I seemingly fixed it by adding the [Tool] attribute to the custom resource I was assigning. I got the fix from seeing someone post this in Godot's C# Discord channel:

In case of a [Tool] script, all the classes involved in the hierarchy have to be [Tool] scripts
But I didn't see anything in the docs about this.

@jcnoon Good point. Now that you mentioned it, I do remember reading something about that in the docs.
Going back to the docs to look for it, I found this quote:

Code from other nodes doesn't run in the editor. Your access to other nodes
is limited. You can access the tree and nodes, and their default properties,
but you can't access user variables. If you want to do so, other nodes have
to run in the editor too. Autoload nodes cannot be accessed in the editor at
all.

It talks about other nodes also requiring the [Tool] annotation, but it doesn't say anything about Resources.
Adding the [Tool] annotation to my custom Resource class fixed the issue for me.

This confused me a good amount too recently. Adding [Tool] to the affected Resources worked for me.
I think this would be a good opportunity to suggest documentation improvements. I've opened an issue for it here:
https://github.com/godotengine/godot-docs/issues/8692

This is also my issue:
Thing : Node have a exported property ThingParameters Parameters, uses ThingParameters : Resource marked with GlobalClass.
When i run i get the same error as OP. All are NOT marked with Tool.
Also every single time i rebuild, all my exported properties are defaulted, it doesnt save nothing. Its also another issue.

I think it should be a bug? As soon as a Resource is nested as a property in another Node/Resource class, it is not cast to the proper C# type when the property is set, resulting in the InvalidCastException.
it's worth in mention that I don't use [Tool] anywhere in my case, so it can't be explained by one of the resources being a [Tool] scripts thus causing the other to need the Tool attribute as well. The only factor seems to be nesting.
I have a setup similar to this example here:
[GlobalClass]
public partial class ResourceA : Resource
{
    // This can be assigned in the editor just fine
    [Export]
    public ResourceB B { get; set; }
}

[GlobalClass]
public partial class ResourceB : Resource
{
    // When assigning this one through the editor we get the InvalidCastException,
    // The setter is called with a 'Godot.Resource' type instead of 'ResourceC', a cast to the correct type is missing
    // Setting ResourceC as [Tool] solves the issue but this makes no sense
    [Export]
    public ResourceC C { get; set; }
}

[GlobalClass]
// This solves the issue: [Tool]
public partial class ResourceC : Resource
{
}

I made a minimal reproduction of the issue here. What I found is that this appears to work if at any point a default custom resource is created within your C# scripts, OR if any of your custom resources are [Tool] scripts.
The workarounds are:

Don't use default instantiations within your Custom Resource scripts, and don't assign default instantiation Custom Resources in your Node scripts.
If you need to use a [Tool] script, try to structure your nesting so the [Tool] is at the BOTTOM of the chain. This part of the issue seems to be more egregious to me, as I can see this being a huge problem in developing [Tool] helpers. For example I want to create a [Tool] script which will access the custom sub-resource and create a UI representation of the sub-resource's configurations. Given this issue, that's just not possible.

Tests I thought about but didn't do:

I didn't test this with gdscript. I don't use gdscript in any of my projects (so far), so it's less relevant and testing this bug has already been a pretty long process so I just left it. Someone else is welcome to test.

So if you have

// --- MyCustomChildResource.cs
[GlobalClass]
public partial calss MyCustomChildResource : Resource { [Export] Vector2I cords; }

// ---  MyCustomResource.cs
public partial calss MyCustomResource : Resource { [Export] MyCustomChildResource myCustomChildResource }

// --- MyClass.cs
public class MyClass : Node {  public MyCustomResource myCustomResource; }

And attach the MyClass to a node, it will work fine in godot 4.3 as of today.
However, if you make one change to MyClass.cs, you'll start getting the error message from this issue report.
// --- MyClass.cs
public class MyClass : Node {  public MyCustomResource myCustomResource = new MyCustomResource(); }

Hopefully this helps someone else not spend too much time trying to figure out (and can help the devs who know C++ identify the bug and solve it!)

Even though workaround exists, it doesn't make sense to be satisfied with it. It's not always possible to make [Tool] be at the bottom of the chain.
I have a specific case where I use _GetPropertyList override to display one of 3 fields depending on enum value. This bug makes it impossible to work.


Repeated calls to System.import are not cached

I'm submitting a feature request
Webpack version:
2.1.0-beta.20
Expected/desired behavior:
Is it possible to cache repeated calls to System.import within Webpack?
Similar to require.cache
https://webpack.github.io/docs/api-in-modules.html#require-cache
Scenario: React-Router async routes
Test: do a second import after the first import, doesnt look like the result from the first is cached
function getChildRoutes(location, callback) {
    let time1 = new Date();
    Promise.all([
        System.import('./p/includes')
    ]).then(function (modules) {
        console.log("[/p]:getChildRoutes1: " + timeSince(time1));
        Promise.all([
            System.import('./p/includes')
        ]).then(function (modules) {
            console.log("[/p]:getChildRoutes2: " + timeSince(time1));
            let module = modules.shift();
            callback(null, module.RoutesList)
        }).catch(err => {

        });
    }).catch(err => {

    });
}

gives me something like this:
[/p]:getChildRoutes1:  [527ms] -> request to /public/bundle/0.chunk.js
[/p]:getChildRoutes2:  [528ms] -> no request

then just prototyping to see what caching i can do in app-space:
if(typeof(window) != "undefined" && !((window as any).modulecache)) {
    (window as any).modulecache = {};
}

function getChildRoutes(location, callback) {
    let time1 = new Date();
    if((window as any).modulecache["./p/includes"]) {
        console.log("[/p]:getChildRoutes: [CACHE-HIT] " + timeSince(time1));
        let module: any = (window as any).modulecache["./p/includes"];
        callback(null, module.RoutesList);
    } else {
        Promise.all([
            System.import('./p/includes') //note: seem to be effected by IE cache ignore debug mode
        ]).then(function (modules) {
            console.log("[/p]:getChildRoutes: [CACHE-SET] " + timeSince(time1));
            let module = modules.shift();
            (window as any).modulecache["./p/includes"] = module;
            callback(null, module.RoutesList)
        }).catch(err => {

        });
    }
}

this is better:
[/p]:getChildRoutes: [CACHE-SET]  [676ms] -> request to /public/bundle/0.chunk.js
[/p]:getChildRoutes: [CACHE-HIT]  [0ms]

What is the motivation / use case for changing the behavior?

Caching similar to require.cache

Actually with further testing with my contrived test, caching works when on same route, however it doesnt seem to work if you change route then come back to the same route

    Promise.all([
        System.import('./p/includes')
    ]).then(function (modules) {
        console.log("[/p]:getChildRoutes1: " + timeSince(time1));
        let time2 = new Date();
        Promise.all([
            System.import('./p/includes')
        ]).then(function (modules) {
            console.log("[/p]:getChildRoutes2: " + timeSince(time2));
            let module = modules.shift();
            callback(null, module.RoutesList)
        }).catch(err => {

        });
    }).catch(err => {

    });

Caches find on the current route
[/p]:getChildRoutes: START
[/p]:getChildRoutes1:  [671ms]
[/p]:getChildRoutes2:  [1ms]


Change: Stop using region short code in State dropdown

Description
Some country regions have numeric short codes ([https://en.wikipedia.org/wiki/ISO_3166-2:VN](such as the regions in Vietnam)). Because of this, we shouldn't be sending the ISO 3166 numeric short codes, but instead just send the actual name of the region.
That Being Said...
The Database still doesn't support special characters, so most of the Vietnam regions, for example, will result in question mark characters when the form is actually submitted and another GET /account request is made.
IMO, this is not something we should be working around.
Type of Change

Non breaking change ('update', 'change')

@Jskobos That's just great

Just put in a DB request to remove this limit.

Closing....this solution isn't going to work with the current DB limits


kafka nozzle

Thanks for adding this.

Just noticed that this was against the master branch. In future, please open PRs against the develop branch.


Added initial scraping files

scraping.ipynb is the notebook I used to scrape. jobs.csv is the job list.

Creating a pull request to add my scraping files, so far.


Renamed "specifity" to "specificity"

fixes #463

Perfect, great job 👍  - thanks for your patience and commitment!


Adding more fine grained percentage reporting

Codecov Report

Merging #750 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master     #750      +/-   ##
============================================
+ Coverage     65.72%   65.72%   +<.01%
- Complexity     3855     3858       +3
============================================
  Files           334      334
  Lines         25644    25645       +1
  Branches       3205     3205
============================================
+ Hits          16855    16856       +1
- Misses         7891     7893       +2
+ Partials        898      896       -2

Impacted Files
Coverage Δ
Complexity Δ

.../main/java/com.github.ambry/store/StoreCopier.java
62.22% <100%> (+0.42%)
9 <0> (ø)
:arrow_down:

...c/main/java/com.github.ambry.network/Selector.java
78.48% <0%> (-3.59%)
44% <0%> (-1%)

...in/java/com.github.ambry.network/SocketServer.java
82.78% <0%> (-0.41%)
13% <0%> (ø)

...ain/java/com.github.ambry.router/PutOperation.java
89.95% <0%> (-0.23%)
69% <0%> (-1%)

.../main/java/com.github.ambry.store/HardDeleter.java
81.93% <0%> (ø)
42% <0%> (ø)
:arrow_down:

...java/com.github.ambry.router/GetBlobOperation.java
88.59% <0%> (+0.26%)
31% <0%> (ø)
:arrow_down:

.../main/java/com.github.ambry.router/PutManager.java
84.82% <0%> (+0.68%)
22% <0%> (ø)
:arrow_down:

.../com.github.ambry.router/GetBlobInfoOperation.java
81.25% <0%> (+1.38%)
31% <0%> (+1%)
:arrow_up:

.../java/com.github.ambry.router/DeleteOperation.java
93.23% <0%> (+1.5%)
44% <0%> (+1%)
:arrow_up:

.../src/main/java/com.github.ambry.store/Journal.java
91.37% <0%> (+1.72%)
20% <0%> (+1%)
:arrow_up:

... and 1 more

Continue to review full report at Codecov.

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


Add exporting/importing document feature

Currently app uses database to store app data (documents) on native platform. Prior to storing data in database it was stored in json files in the first version of the app. PR that introduced database usage: https://github.com/StaffEngineer/rusticify/pull/17 and removed saving/loading data from json file. It's useful feature to have ability to just export/import document to json file to share it with a buddy. Let's have an option to export/import document from database to json file.

Once this issue is implemented let's implement logic for loading document via URL.
Once loading document via URL is implement let's implement providing document via URL param for wasm target like so:
https://staffengineer.github.io/velo/?gist=https://gist.githubusercontent.com/StaffEngineer/351edf0525c104a572dbe122c2ec60de/raw/c2bbe9f73e6ebd7710258226543f0a6705436a88/my-drawing.json
Opening this link will insert new document with data from public gist.


ReferenceError: regeneratorRuntime is not defined

Had the following error when using this script (via generator-github-project):
events.js:167
      throw er; // Unhandled 'error' event
      ^

ReferenceError: regeneratorRuntime is not defined
    at /Users/.../node_modules/yo-option-or-prompt/lib/index.js:10:46
    at Object.<anonymous> (/Users/.../node_modules/yo-option-or-prompt/lib/index.js:70:2)
    at Module._compile (internal/modules/cjs/loader.js:689:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:700:10)
    at Module.load (internal/modules/cjs/loader.js:599:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:538:12)
    at Function.Module._load (internal/modules/cjs/loader.js:530:3)
    at Module.require (internal/modules/cjs/loader.js:637:17)
    at require (internal/modules/cjs/helpers.js:20:18)
    at Object.<anonymous> (/Users/.../initializing.js:12:31)
Emitted 'error' event at:
    at Immediate.setImmediate (/Users/.../node_modules/yeoman-generator/lib/index.js:433:22)
    at runCallback (timers.js:693:18)
    at tryOnImmediate (timers.js:664:5)
    at processImmediate (timers.js:646:5)

Did this fix your error?

I think the regenerator runtime error is because it requires babel-polyfill to be imported.
I could add idempotent-babel-polyfill so everyone isn't forced to require babel-polyfill.

Commit 44baaa7c9dee494b1132aabb05085fc33a71aaee should have fixed this. Please updated to 0.2.0 and let me know if it works.

Can you confirm if this worked?


Publish build scan URL

Hello @eskatos
I tried to include the build scan URL in the github message but didn't succeed.

Build scans
If your build publishes a build scan the gradle-command-action action will emit the link to the published build scan as an output named build-scan-url.
You can then use that link in subsequent actions of your workflow.

Here is my yaml config
name: CI unit tests
on: [pull_request]
jobs:
  unittests:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v1
      - uses: actions/setup-java@v1
        with:
          java-version: 8
      - uses: eskatos/gradle-command-action@v1
        with:
          arguments: testDebugUnitTest
      - name: Comment PR
        uses: thollander/actions-comment-pull-request@master
        if: failure()
        with:
          message: Build failed ${{ steps.gradle.outputs.build-scan-url }}
          GITHUB_TOKEN: ${{ secrets.GITHUBTOKEN }}

Expected result: Build failed https://gradle.com/s/rjfaddrnmi5dq
Actual result: Build failed
PR https://github.com/jmfayard/android-kiss/pull/5
Actions: https://github.com/jmfayard/android-kiss/actions

Hey @jmfayard,
Sorry for the lag answering your question. It works for me™.
Your workflow file points to steps.gradle.outputs.build-scan-url but your step running the gradle action doesn't declare an id.
Here's the excerpt from the README:
    - uses: eskatos/gradle-command-action@v1
      with:
        arguments: build
      id: gradle
    - uses: example/action-that-comments-on-the-pr@v0
      if: failure()
      with:
        comment: Build failed ${{ steps.gradle.outputs.build-scan-url }}

The key is the 4th line id: gradle.
I'm optimistically closing this, feel free to reopen if it doesn't work for you.


Add a grid feature

This refactors the contourFeature to avoid code duplication, as the grid and contour features are nearly the same.  Specifically, the contour feature has data values at each vertex of a mesh and the grid feature has data values on the elements of the mesh.  The grid feature has solid colors for each mesh element, while the contour feature has interpolated values between the vertices.
This adds a small tutorial.
This closes #545.

@manthey build issues - holding review until it passes

It passes locally (and in a docker build environment).  I'm not sure what is failing in travis.  This is built on top of PR #1079, so if that gets merged, I'll rebase and see if kicking off travis again fixes it.


Kubernetes connection Problem Kong-Manager

Hello ich have tried to install Kong with custom values.

A postgres Database ist running in the same namespace.
I can access the Kong-Manager WebUI, but there is not data displayed. I also tried to connect to kong but this also not worked.

I would be thankful about help :)

@wefersmasasana Could you open the browser dev tools and look at it if there is an error?
Thank you.

admin_gui_path

Above configurations. It should be like follows:
KONG_ADMIN_GUI_PATH: '/manager'
KONG_ADMIN_GUI_URL: 'http://localhost:18802/manager'


Update to valid 2020.1 icons

Addresses issue #278
Version 2020.1 removed the NextFile and ShowEditorHighlighting icons and the recommend icons are AllIcons.Actions.Edit and AllIcons.Actions.Forward
https://github.com/JetBrains/intellij-community/blob/1ef4ef2466913559c46118ac01ce44cc99b45721/platform/util/src/com/intellij/icons/AllIcons.java#L381
https://github.com/JetBrains/intellij-community/blob/1ef4ef2466913559c46118ac01ce44cc99b45721/platform/util/src/com/intellij/icons/AllIcons.java#L2653

Thanks a lot! I tagged a new release and uploaded it to JetBrains. v1.6.8 is up: https://plugins.jetbrains.com/plugin/7389-floobits/update/83783


Enable shortened/normal mixed record definition

Fixes #238.
Enable record definition in mixed style:
x = 1
z = 100
t = {x; y = 10; z}
print! t.x  # prints 1
print! t.y  # prints 10
print! t.z  # prints 100

@mtshiba

Thank you!
I found another problem during the review, so I made a new issue.
#244


[Bug] Balloon values no longer parsed for Expert & Master branches

Balloons placed in Expert & Master branches will no longer receive custom values designated by the TJA file & will always be given the default value of 5. Normal will still be parsed as expected.
In addition, Dans with multiple branching charts will take balloon values that initially belonged in Expert/Master branches & put them in later Normal balloons instead.

For this part, how balloons in branches are normally written within the .tja file ?

In the Dan test file provided, the balloon values are written in order of N,E,M,N,E,M,N,E,M
Within the file itself are several instances of these bars in each song :
#BRANCHSTART p,-1,0

#N
7008,
7008,
#SECTION

#E
7008,
7008,
#SECTION

#M
7008,
7008,
#SECTION

branch and balloon dan test.zip

Thank you for the details, are they how they currently work on TJAPlayer3 and TNDE ?
For this case I would think more about N,N,E,E,M,M, considering the parsing order.
Additionally, shouldn't #SECTION be stated only once, after the 3 branches #BRANCHEND tags ?

Using the latest version of TJAP3 from TJADB resources channel. Can't test dans w/ branching charts since that breaks TJAP3, so I ended up testing the first song on its own.
Same behavior in TJAP3, balloons in Expert/Master branches default to 5 everytime. However, all balloons parse correctly when using BALLOONNOR:/BALLOONEXP:/BALLOONMAS: instead. Same behavior for OpenTaiko for single & dan.
I had taken the N,E,M,N,E,M,N,E,M, route as a suggestion you wrote in the past. The #SECTION was a brainfart on my part, and was corrected during testing. Results were the same before/after, though.

I will close this ticket considering you confirmed that the BALLOONNOR: / BALLOONEXP: / BALLOONMAS: options work, and will document them as standard.
It makes more sence this way, thereforce BALLOON: should only concern the balloons for the main course (it would make charts no longer compatible with other simulators if I implement them differently).


Add article Bitcoin Cash Node - Portugues

For #110
https://read.cash/@phabulu/bitcoin-cash-node-portuguese-00bee10a

Waiting until conflicts can be resolved, @ChaituVR


Building wheel for steup.py takes forever

Its taking more that 1 hour for building wheel
Building wheel for flash-attn (setup.py)
is this expected behavior? I don't see anything heavy in setup.py

im running on T4 GPUs btw

Compiling takes 5-6 minutes for me on a multi-core machine. We do parallelize the compilation if you have ninja installed. ninja usually comes with PyTorch, but you can check by pip install ninja. Can you make sure that ninja is installed and then try compiling again?
There's a lot of templating in CUDA code for max efficiency, so compiling time is indeed an issue. With parallel compilation it's not so bad, but eventually we'd love to provide pre-built wheels. I just haven't had the bandwidth to tackle this.

Hey there thanks a ton for your reply!
Yes i have indeed ninja installed
On debugging the wheel of flash-attention is erroring out
error: subprocess-exited-with-error

   python setup.py bdist_wheel did not run successfully.

     note: This error originates from a subprocess, and is likely not a problem with pip.
  Running setup.py install for flash-attn ... error

error: legacy-install-failure

any idea whats causing this/how to solve this?

Are you saying that python install setup.py succeeds but python setup.py bdist_wheel fails?
I don't think I've seen this error, but can you paste the full compilation log?
What's your OS, nvcc version, and gcc version?

Hello, I've encountered the same problem. Have you resolved it?

it takes forever to install with 100% loading of my 12 threads cpu, i've stopped it after 10 minutes, just want to make sure is it normal?


add activity_timeout env variable

In some cases, I need to configure this variable. Kubernetes resets ws connection before 120 seconds

Hey thanks for the PR! 🎉
The build is breaking because it's not formatted with mix format. If you could run mix format then push the changes? 👍

Thanks for help! I added this env to readme

Thank you! I will release a new version soon! There should be soon a new edgurgel/poxa-automated:latest docker image soon automatically built by docker hub if you want to try it out 👍


Added workarounds for AMBARI-17367 and BUG-60040 and removed SQOOP

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/cloudbreak-pull-request/2774/

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/mock-integration-test/265/


UIPCIR-2: Create holdings record via fast add form

https://issues.folio.org/browse/UIPCIR-2

yarn run v1.22.4
$ eslint .
/home/jenkins/workspace/in-create-inventory-records_PR-7/project/src/components/HoldingAccordion/HoldingAccordion.js
2:8  warning  'defer' is defined but never used              no-unused-vars
20:8  warning  'ContributorFields' is defined but never used  no-unused-vars
/home/jenkins/workspace/in-create-inventory-records_PR-7/project/test/bigtest/tests/create-records-plugin-test.js
30:14  error  describe.only not permitted  no-only-tests/no-only-tests
✖ 3 problems (1 error, 2 warnings)
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Test errors found. See https://jenkins-aws.indexdata.com/job/folio-org/job/ui-plugin-create-inventory-records/job/PR-7/1/

yarn run v1.22.4
$ eslint .
/home/jenkins/workspace/in-create-inventory-records_PR-7/project/test/bigtest/tests/create-records-plugin-test.js
30:14  error  describe.only not permitted  no-only-tests/no-only-tests
✖ 1 problem (1 error, 0 warnings)
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Test errors found. See https://jenkins-aws.indexdata.com/job/folio-org/job/ui-plugin-create-inventory-records/job/PR-7/2/


groovy-gradle-plugin Not Listed In Gradle Plugin Reference

Issue type
Wrong or misleading information
Problem description
The groovy-gradle-plugin I am presuming is a core plugin (*). The Gradle Plugin Reference claims at the top to contain "links and short descriptions for all the core plugins provided by Gradle itself" however this seems to be incorrect because the groovy-gradle-plugin is not to be found on that page.
Ideally, there would be dedicated docs added for the groovy-gradle-plugin, but at the least, the Gradle Plugin Reference page should be modified to say that it doesn't list all core plugins, or one better would be to list the core plugins that are not included in the page.
(*) I presume it is a core plugin because where I have seen it used in the docs (eg in the Sharing convention plugins with build logic build Sample) its id is unqualified, ie plugins { id 'groovy-gradle-plugin' }.
Context (optional)
I'm currently learning Gradle and it's hard for me to work out what is a core plugin and what isn't.
Without a comprehensive list of core plugins, if I see a plugin like groovy-gradle-plugin used briefly in the docs, I don't currently have a way of knowing for sure if it is a core plugin or just a legacy plugin that happens to have an unqualfied name.
Page with the problem
https://docs.gradle.org/current/userguide/plugin_reference.html

Thank you for your interest in Gradle!
This is a valid documentation issue that we will address.


Add hooks to the SplitView

Add callback functions to customize the behaviour of the split view component. For example, onMouseDown, onMouseUp, onResize should be customizable.

OnGrabHandle, OnMoveHandle and OnReleaseHandle have been added to release 1.2.0


Add ArgonJsonConverterAdapter for adapting a Newtonsoft.Json.JsonConverter to Argon.JsonConverter

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: ldeluigi sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

can u add some tests to ensure this works with scrubbers https://github.com/VerifyTests/Verify/blob/main/docs/scrubbers.md and ignoring members https://github.com/VerifyTests/Verify/blob/main/docs/serializer-settings.md#ignore-member-by-name

and add some docs showing how people should use it

@SimonCropp done

can u change that test to use the generic variant of IgnoreMember. since that is the approach most people use
settings.IgnoreMember<IgnoreExplicitTarget>(_ => _.PropertyThatThrows);

can u change that test to use the generic variant of IgnoreMember. since that is the approach most people use
settings.IgnoreMember<IgnoreExplicitTarget>(_ => _.PropertyThatThrows);

it's not possible since I'm using VerifyJson. I'll make another test

It seems to not be working with the generic version. I can't debug because you've put debugger skips

you've put debugger skips

not sure i understand?

you've put debugger skips

not sure i understand?

it seems that if I try to F11 during debug into Verify(employee, settings) the debugger skips some method calls because there is a DebuggerStepThrough attribute and I can't understand what's happening

i have not used DebuggerStepThrough anywhere in Verify

i have not used DebuggerStepThrough anywhere in Verify

Weird. I'll investigate deeper in my next free time. 🤔

Ok I went deeper and I understood everything. Basically any adapter won't work with scrub/ignore members because it uses a different contract resolver. That's it. The best I can do is document that scrubbers and ignore members don't work.

Otherwise we close the entire PR and I will implement a JsonConverter manually again for my classes in my projects

I decided that this implementation does not meet the standards I was looking for. I'm concluding that reimplementing converters is the only possible right way to solve the issue. Thanks.


Roll Flutter Engine from 6cff45c25c34 to b18909ec14ad (1 revision)

https://github.com/flutter/engine/compare/6cff45c25c34...b18909ec14ad
2022-06-28<EMAIL_ADDRESS>Roll Skia from b007c1909b83 to<PHONE_NUMBER>0f (1 revision) (flutter/engine#34352)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


dotenvy silently fails to parse strings with dollar signs

I ran into this issue when storing a password in a .env file and not understanding why I couldn't authenticate. After some testing, I was able to figure out that the Dollar Sign $ was causing the string parser from dotenvy::var("KEY_NAME") to fail.
Minimum Reproduceable Example
// main.rs
use dotenvy::dotenv;

fn main() {
    dotenv()
        .and_then(|_| {
            println!("{}", dotenvy::var("NO_DOLLARS").expect("NO_DOLLARS failed"));
            println!("{}", dotenvy::var("DOLLARS").expect("DOLLARS failed"));

            Ok(())
        })
        .unwrap();
}

Store variables in .env file
NO_DOLLARS="Hello%World^&FooBar*()_+"
DOLLARS="H$ello$%World^$&FooBar*()_$+"

As you can see, the DOLLARS variable includes random dollar signs throughout. However, if you run this script the strings returned do not match what's saved in the .env file
$ cargo run
    Finished dev [unoptimized + debuginfo] target(s) in 0.03s
     Running `target\debug\foobar.exe`
Hello%World^&FooBar*()_+
H%World^&FooBar*()_+

This issue can be closed. After further digging, I believe this is actually intended behavior, as $foobar$ is used to indicate evaluating the value of foobar.


[WIP] Add --waitTimeout flag to dapr stop command

Description
Adding a new --waitTimeout flag to the dapr stop command - This flag allows users to specify a timeout duration in seconds, which determines the maximum time the command will wait for the process to exit gracefully. If the process remains active after this timeout, the command will forcefully terminate it.
The default --waitTimeout value is set to 15 seconds.
WIP - This is implemented only for linux for now. Currently learning how windows event signals work.
Unit tests run successfully.
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #1160
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[ ] Created/updated tests
[ ] Extended the documentation

Hi @nikitasarawgi, welcome to Dapr 👋🏻 Could you please fix the DCO check? https://docs.dapr.io/contributing/contributing-overview/#developer-certificate-of-origin-signing-your-work


A0-860: Switch fork-off to use WS instead of HTTP

Description
This makes it very easy to multiplex 1000s of requests in a single
connection which speeds up the download considerably. Previously the
worker would often crash/hang when setting --num-workers=1000.
On main, with the default --num-workers=5:
time target/release/fork-off --http-rpc-endpoint=http://rpc.test.azero.dev --initial-spec-path=../chainspec.json --combined-spec-path=combined.json
[...]
real	7m22,744s
user	0m35,144s
sys	0m15,549s

On main with --num-workers=1000 it just hangs. With --num-workers=100 still often crashes, but when it works:
time target/release/fork-off --http-rpc-endpoint=https://rpc.test.azero.dev --initial-spec-path=../chainspec.json --combined-spec-path=combined.json --num-workers=100
[...]
real	0m37,785s
user	0m14,808s
sys	0m4,818s

With these improvements (and the default --max-requests=1000):
time target/release/fork-off --ws-rpc-endpoint=wss://ws.test.azero.dev --initial-spec-path=../chainspec.json --combined-spec-path=combined.json
[...]
real	0m10,405s
user	0m1,533s
sys	0m0,562s

I also tried this out on westend-rpc.polkadot.io. While doing that, I noticed that about half the time was spent in the serial part that fetches the list of all keys. So I made it fetch the keys in parallel with fetching values (second commit). Current time for westend:
time target/release/fork-off --ws-rpc-endpoint=wss://westend-rpc.polkadot.io --initial-spec-path=../chainspec.json --combined-spec-path=combined.json
[...]
[2022-06-13T13:15:42Z INFO  fork_off::fsio] Writing snapshot of 961834 key-val pairs and 736062098 total bytes
[...]
real	4m14,338s
user	0m53,071s
sys	0m13,124s

Lastly, I added a --at-block option, and used that to verify that the file generated by the code on main is the same as what's generated by this code.
Type of change
Not sure about this one - is the command-line API of the fork-off tool official enough to consider this change breaking?

Bug fix (non-breaking change which fixes an issue)

Checklist:

I have made corresponding changes to the existing documentation
I have created new documentation

I think I addressed all points, except for the output.into_inner() thing, which I'm not quite sure how to do.


tracing: Improve decoding of functions output

Motivation
Currently, while decoding result of function execution in traces, the following is done:

If function appears in ABIs of some contract seen during execution and identified by it's bytecode, ABI is taken from this contract bytecode.
Otherwise, ABI is fetched from openchain.xyz

This works fine for decoding inputs, however, openchain.xyz does not (and can not) tell anything about output types of a function.
Due to this, outputs for any calls to external contract accessed by an interface in forked environment are not decoded at the moment.
An example of this behavior is following test:
pragma solidity 0.8.20;

import {Test} from "forge-std/Test.sol";

interface IERC20 {
    function balanceOf(address) external view returns (uint256);
}

contract TestTracing is Test {
    function test() public {
        vm.createSelectFork("mainnet");

        IERC20 usdc = IERC20(0xA0b86991c6218b36c1d19D4a2e9Eb0cE3606eB48);
        usdc.balanceOf(address(0));
    }
}

Traces for it will look like this:
  [15795] TestTracing::test()
    ├─ [0] VM::createSelectFork("mainnet")
    │   └─ ← 0x0000000000000000000000000000000000000000000000000000000000000000
    ├─ [9815] 0xA0b86991c6218b36c1d19D4a2e9Eb0cE3606eB48::balanceOf(0x0000000000000000000000000000000000000000) [staticcall]
    │   ├─ [2529] 0xa2327a938Febf5FEC13baCFb16Ae10EcBc4cbDCF::balanceOf(0x0000000000000000000000000000000000000000) [delegatecall]
    │   │   └─ ← 0x0000000000000000000000000000000000000000000000000000000000000000
    │   └─ ← 0x0000000000000000000000000000000000000000000000000000000000000000
    └─ ← ()

However, it should be decoded into this:
  [15795] TestTracing::test()
    ├─ [0] VM::createSelectFork("mainnet")
    │   └─ ← 0
    ├─ [9815] 0xA0b86991c6218b36c1d19D4a2e9Eb0cE3606eB48::balanceOf(0x0000000000000000000000000000000000000000) [staticcall]
    │   ├─ [2529] 0xa2327a938Febf5FEC13baCFb16Ae10EcBc4cbDCF::balanceOf(0x0000000000000000000000000000000000000000) [delegatecall]
    │   │   └─ ← 0
    │   └─ ← 0
    └─ ← ()

And this gets much uglier when working with contracts returning tuples/addresses/bytes
Solution
There is already with_events function which is used for importing events from known contracts when constructing a CallTraceDecoder, so I've just added similar logic for functions as well
https://github.com/foundry-rs/foundry/blob/fdad9fb0dde45d3476fc5d1fe6f40e8dc7c17caa/crates/forge/bin/cmd/test/mod.rs#L692-L695

Also cc @onbjerg @Evalir tracing change

@klkvr I meant repro tests in crates/forge/tests/it/repros.rs

Hey @DaniPopes I am not sure if it makes sense to add this as a repro test

It seems to me that it isn't possible to get known_contracts object and configure trace identifier with it while not changing tests too much

Even if it is possible, the test will not really make a lot of sense. I think test should check that

traces are decoded correctly when testing through CLI

rather than

traces are decoded correctly when known contract abis are included into scope of call trace decoder

The former is what this PR fixes, and the latter is how it does that, and this logic already worked before this PR


Feature -  AI Simulator

Add the ability to simulate a game against a team.
Just use their stats of the season to calculate the probabilities of each throw
Since missed throws is not stored try to calculate based upon throw differences and add a difficulty setting

@glifchits let me know if you got any ideas on how to enhance this.


expose annotations to help dart2js optimizations

I'd like to create a package that exposes some annotations understood by dart2js.
Some of the use cases for this are:

frameworks could indicate hot-paths that can benefit from inlining. For example, see https://github.com/dart-lang/sdk/issues/25270 request to inline looseIdentical in ng2 apps.
we identified something curious and are deciding whether it is worth writing an analysis for it. Before doing so, we could include annotations in the source code to test out a transformation idea. For example, we could use @notnull to help the compiler understand nullability on certain cases that the current inferrer failed to do so.
we can provide one-day a more fine-grain control on trusting types (e.g. trust the types of this package, but not of some other third-party package).

For now, I suggest we make public the same annotations we have today on package:expect for our own unit tests. As part of doing that we can then see what would be the proper way to add more.

We have @dart2js.noInline and @dart2js.tryInline:
///     import 'package:meta/dart2js.dart' as dart2js;
///
///     @dart2js.noInline
///     String text() => 'A String of unusual size';

We will put more annotations in the same package as they become available.