drop tail undisturbed/mask agnostic

Is there a reason to keep tail undisturbed?
Tail agnostic includes both tail undisturbed for in-order cores and tail one for out-of-order cores, therefore tail agnostic is good enough for all core implementations.

If tail undisturbed is required to be implemented in the out-of-order cores, then every vector load has to be treated as masked vector load, i.e. it should depend on/read previous vector register value/physical register, because in case of exception the rest of vector register, starting from vstart, must remain undisturbed. This can have performance impact on vector loads and add non-trivial design and verification complexity.

Some instructions always operate as tail agnostic, so dropping tail undisturbed will make ISA more consistent.

Additional mode has a non-trivial verification cost.

Similar arguments for dropping mask agnostic.

I'm currently using tail-undisturbed as part of a matrix repacking and zero-padding routine. In particular, I'm using tail-undisturbed loads into zero-initialized registers to simulate tail-zeroing.
I'm not arguing that my niche application justifies the tradeoffs you discuss, just mentioning that there is an application.

@nick-knight this can be done with just a couple of instructions and mask undisturbed:
vsetvli t1, x0, vtypei
vmv.v.i v1, 0
vid.v v0
vmslt.vx v0, v0, t2 # t2 is the number of non-zero elements
# use v0 as a mask

Sure. At the cost of keeping around an extra mask register. My code doesn't need masking otherwise, so this increases register pressure in the inner loop. And I suspect on some implementations, masked forms may be less performant than unmasked forms (I don't have data to support this). I'm just saying, it's a tradeoff.

LLVM uses tail undisturbed to implement insertion of a small vector into a larger vector. Using the offset and the VL you can control where the insertion starts and ends.

LLVM uses tail undisturbed to implement insertion of a small vector into a larger vector. Using the offset and the VL you can control where the insertion starts and ends.

Hmm, this looks like a hack.
vslide with mask should do it.

IIRC @aswaterman told us vslide is slow? or maybe this its' very implementation dependent.

All the performance characteristics are implementation dependent.
However, if tail undisturbed is one of the two fill options for virtually
all instructions it is highly likely to be optimized. Whereas slide is just
one of many instructions an thus has less impetuous to optimize/commit many
transistors
On Thu, Apr 15, 2021, 22:59 Kito Cheng, @.***> wrote:

IIRC @aswaterman https://github.com/aswaterman told us vslide is slow?
or maybe this its' very implementation dependent.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/riscv/riscv-v-spec/issues/664#issuecomment-820872476,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFAWIKNRSSHVIEKSBAPTEDLTI6R2FANCNFSM426SUE7Q
.

IIRC @aswaterman told us vslide is slow? or maybe this it's very implementation dependent?

Hacky use of tail undisturbed is likely to have even more performance problems in high perf OOO implementation.

The following heuristic should apply to any high-quality implementation: VSLIDE will be no slower than using VSE followed by dependent VLE.
On some implementations, VSLIDE will perform similar to simple ALU ops like VADD.  On other implementations, it will be noticeably slower than VADD, depending on the slide distance.  (I'd expect < 2x overhead in practice.)
On some implementations, the tail-undisturbed operations will have noticeable overhead, as @solomatnikov mentioned.  This overhead generally shouldn't be extreme (often <= 1 cycle per instruction in practice) but will add up.
So, this might be -mtune territory.  Both approaches should get decent performance on any implementation, but one or the other will be preferable for specific implementations.

On 2021-04-15 11:32 p.m., Alex Solomatnikov wrote:

IIRC @aswaterman told us vslide is slow? or maybe this it's very implementation dependent?

Hacky use of tail undisturbed is likely to have even more performance problems in high perf OOO implementation.

Essentially by definition, the actual use of tail undisturbed [whether hacky or not] is expected to have performance impact on OoO and non-OoO massive vector implementations.
Agnostic is the means to allow efficient implementation across all micro-archs.
If the facility provides no opportunity for optimization to the u-arch, then it can operate as undisturbed with no performance overhead.
It was an allowance specifically because it was anticipated that YMMV across multiple implementations.
Your concern is certainly more nuanced than this explanation gives credit.
However, the software folk early on suggested undisturbed would be useful and it thus became the default standard. Thus the basis for the tradeoff.
We now recommend that agnostic be the software standard and undisturbed be used only when needed to improve algorithms and hence performance.
This "hack" is thus intentional and quite in keeping with the hacking nature of toolchain and OS developers, many of whom take pride in squeezing out performance from even obscure ISA features.

@nick-knight @topperc @kito-cheng could you post code sequences with actual values of vl and lmul? It would be useful for discussion and performance analysis.

A data point that may not be very conclusive, in case it helps.
Reductions benefit from tail undisturbed when using vl to avoid having epilog code. We can easily accumulate partial reductions (using vl) to a vector that has been initialised (under vl=VLMAX) with the neuter element. That said, this is the only scenario I've seen where it is very convenient (at least for the programmer) to have this behaviour. Almost every other code I've seen seems to do fine with tail agnostic (as in the code does not care about the tail).
That said, one could argue that this particular algorithm is not suited to be implemented without epilog code and we could do better if we do a first VLMAX loop (so the tail policy is not relevant) followed by epilog code that does a final accumulation (the partial vector reduced from the previous VLMAX loop and the values from the epilog) using tail-agnostic and mask-undisturbed (followed by the reduction itself v{,f}red<op>.vs).
Dot products are essentially reductions and they seem important in many applications these days.

Undisturbed allows for avoiding register spills.
Software can retain data in the "tail" of a register to be used in subsequent operations without memory ops.
This may be its most beneficial use, and a sufficiently positive use for it to have justified including it [or so thought the TG].
However, software folk are ingenious and other important variations and usages of undisturbed are undoubtedly discoverable.

We're keeping undisturbed.

according to compiler engineer tail undisturbed is completely useless for vectorizing compiler

according to compiler engineer tail undisturbed is completely useless for vectorizing compiler

Completely useless is in my opnion a bit of an extreme stance: I'd say that the common approaches to vectorization do not benefit from undisturbed policy in the tail.
However as I've stated above, reductions do benefit from it. Think of the partial reductions being accumulated on a register fully initialised before the loop (with vlmax) and then partially accumulating on that vector, the last iterations may not accumulate all the elements but you still want to preserve their earlier values (earlier partial values). After the loop the final reduction is done horizontally with this vector (using a v{,f}red<op>.vs instruction, again with vlmax).  If we do not have undisturbed, partial accumulation is a bit more involved as it requires doing the partial accumulation and then merge it with partial reductions vector with a mask vid < vlmax. (Perhaps at the CPU implementation level that would not be a big deal?)
Beyond vectorization, others have pointed there are uses for tail undisturbed.
(Now if you ask me: we could get rid of the mask policy and we could hardcode it to mask undisturbed. I don't think anyone has presented a use case for mask agnostic)

Not all vector code is generated by a vectorizing compiler.

I am relieved that Tail Undisturbed will remain in the specification as I use it a lot (actually most of my code). But I was wondering why this issue exists in the first place and how, using Tail Agnostic, code can be as fast (and compact) as using Tail Undisturbed for, say, the following vector max code (as an example):
https://godbolt.org/z/d9xMTj3rd
#include <float.h>
#include <riscv_vector.h>

void max_flt( const float *p_src, uint32_t len, float *p_res )
{
   float   new_val;
   float   max_val = FLT_MIN;

   while ( len > 0 )
   {
       new_val = *p_src++;

       if ( max_val < new_val )
       {
           max_val = new_val;
       }

       len --;
   }

   *p_res = max_val;
}

Also, what does the output of a vectorizing compiler look like for the above code? How does it do it?
Here is my Tail Undisturbed version and assembly output:
https://godbolt.org/z/xdaf1EMr8
#include <float.h>
#include <riscv_vector.h>

/* Find the maximum value in the vector *p_src of length len, using RISC-V Vector strip-mining */
void max_flt( const float *p_src, uint32_t len, float *p_res )
{
    uint32_t        len_ctr;    /* The remaining number of elements to process */
    vfloat32m8_t    v_new_vals; /* New values read from memory */
    vfloat32m8_t    v_max_vals; /* Current maximum values */
    vfloat32m1_t    v_max_val;  /* Final max value */
    size_t          vl;

    len_ctr = len;

    v_max_vals = vfmv_v_f_f32m8( FLT_MIN, len_ctr ); /* Set the max vector to the minimum value */
    v_max_val  = vfmv_v_f_f32m1( FLT_MIN,       1 ); /* Set the final reduction result to the minimum value */

    while ( len_ctr > 0 )
    {
        vl = vsetvl_e32m8( len_ctr );

        /* Load the next values from memory */
        v_new_vals = vle32_v_f32m8( p_src, len_ctr );

        /* Find the maximum of the current and new values *** REQUIRES TAIL UNDISTURBED *** */
        v_max_vals = vfmax_vv_f32m8( v_max_vals, v_new_vals, len_ctr );

        /* Update the pointer and remaining length */
        p_src   += vl;
        len_ctr -= vl;
    }

    /* Reduce the max vector to the scalar result */
    v_max_val = vfredmax_vs_f32m8_f32m1( v_max_val, v_max_vals, v_max_val, len );

    /* Save the scalar max result */
    vse32_v_f32m1( p_res, v_max_val, 1 );
}

Using LLVM compiles to:

.LCPI0_0:
        .word   0x00800000                      # float 1.17549435E-38
max_flt(float const*, unsigned int, float*):                       # @max_flt(float const*, unsigned int, float*)
        lui     a3, %hi(.LCPI0_0)
        flw     ft0, %lo(.LCPI0_0)(a3)
        vsetvli zero, a1, e32, m8, ta, mu
        vfmv.v.f        v8, ft0
        vsetivli        zero, 1, e32, m1, ta, mu
        vfmv.v.f        v25, ft0
        beqz    a1, .LBB0_3
        mv      a3, a1
.LBB0_2:                                # =>This Inner Loop Header: Depth=1
        vsetvli a4, a3, e32, m8, ta, mu  ##### This should be tu TAIL UNDISTURBED (currently no way to specify in LLVM)
        vle32.v v16, (a0)
        vfmax.vv        v8, v8, v16
        slli    a5, a4, 2
        sub     a3, a3, a4
        add     a0, a0, a5
        bnez    a3, .LBB0_2
.LBB0_3:
        vsetvli zero, a1, e32, m8, tu, mu
        vfredmax.vs     v25, v8, v25
        vsetivli        zero, 1, e32, m1, ta, mu
        vse32.v v25, (a2)
        ret

If someone can use the vector intrinsics and port this function using Tail Agnostic (& LLVM) in the most optimal way possible, then we can compare and discuss the issues so I (and others) can better understand the implications on different platforms.

I think this is a great use-case for tail undisturbed. The point @solomatnikov was raising with this Github Issue is that tail-undisturbed use-cases, like yours, can be simulated using tail-agnostic and mask-undisturbed. Here is the prototypical code sequence he suggested earlier in this thread:
vsetvli t1, x0, vtypei
vmv.v.i v1, 0
vid.v v0
vmslt.vx v0, v0, t2 # t2 is the number of non-zero elements
# use v0 as a mask

Integrating this into a strip-mined loop like yours will require adding some "boilerplate" code reminiscent of fringe-case handling in the SIMD ISAs of yore. In other words, I claim that a tail-undisturbed implementation like yours leads to more compact code than a tail-agnostic implementation.
I feel this addresses your question of program compactness. You also asked about speed. That is implementation-dependent. For example, an implementation could execute tail-undisturbed instructions much more slowly than tail-agnostic mask-undisturbed instructions, plus the aforementioned boilerplate/overhead. (I'm not claiming this is a useful design point, just that it's possible.)
A third question you asked concerns the autovectorization capabilities of current compilers. I'm afraid I can't address this one.
And in case anyone would like context on comments in your code snippets concerning limitations of the RVV intrinsics API, there is extensive discussion over at https://github.com/riscv/rvv-intrinsic-doc/issues/27 (my summary: https://github.com/riscv/rvv-intrinsic-doc/issues/27#issuecomment-870713856).

@nick-knight thank you. I have another question. You said:

You also asked about speed. That is implementation-dependent. For example, an implementation could execute tail-undisturbed instructions much more slowly than tail-agnostic mask-undisturbed instructions, plus the aforementioned boilerplate/overhead.

Would there be a difference in speed between Tail Agnostic and Undisturbed when the vector length vl = VLMAX? I would assume they would be the same speed?
If they are the same speed then my code will be faster for all iterations except the last when vl < VLMAX.
I also don't see why Mask Undisturbed will be faster than Tail Undisturbed when vl = VLMAX? I would assume they would be the same speed.
Anyway, this is a simple example. In more complex code I have used masking (undisturbed) and require Tail Undisturbed as well when vl < VLMAX. Setting up the mask to also cater for Tail Undisturbed will add extra code and registers (making it more complex, bigger, slower on vector units that don't have agnostic speed ups, use more registers and making it less elegant).
That's the point, Tail Agnostic designed code will be slower on vector units that don't have register renaming or other agnostic speed ups. So as programmers we will have to produce two performance versions, something I think the Vector design team are against.

I'm not clear what you are asking: are you hoping for an architectural guarantee that the relative speed of tail-undisturbed must be sufficiently close to that of tail-agnostic? Currently the RISC-V ISA says nothing, anywhere, about relative instruction speeds, and I don't see this changing. The architecture describes the functionality, the microarchitecture decides the relative speeds, and implementors risk an uncompetitive product by introducing performance cliffs. From my (cynical) software perspective, it's best to write as much tail-undisturbed software as possible, driving the market forces to ultimately reduce performance cliffs.
Pragmatically, my strategy is to write tail-undisturbed code where it is appropriate (like your example), but to revisit this decision when tuning for a particular microarchitecture. In that context, I'm already dealing with several "performance versions". For example, if the cost of vfredmax is sufficiently large relative to vfmax --- which depends on the implementation --- I may want to consider targeting LMUL < 8. Additionally, I am considering a long list of variations for cases of small application vectors (len), where "small" is understood relative to things like VLEN and datapath width(s). These are all decisions that a vectorizing compiler could (in principle) automate, but the RVV intrinsics (or assembly) programmer has to address explicitly.

Just to add to this closed issue discussion, the conversation seems to be missing the obvious point that in-order implementations can implement tail-undisturbed at full speed with no cost.   If we were to force these smaller implementations to execute longer code sequences to synthesize tail-undisturbed when useful, they would be slower (not to mention code size growth issue).
Of course, assumption throughout is that software should only use tu when it is actually needed.
To reduce implementation complexity, renamed machines can synthesize tail-undisturbed using the underlying microarchitecture needed for mask-undisturbed (e.g.., execute tail-undisturbed as a full-length mask-undisturbed uop with an internal mask generated from vl - basically generating the suggested code sequence on the fly).  This still adds some complexity versus only supporting tail-agnostic, but should not perform worse than executing the equivalent macro code sequence.  If tu is not commonly used in code, then this slow implementation would be fine.  If people actually use it (as I suspect they will), then further optimizations will be found to be necessary, but then having to run common instructions fast is not exactly a bad thing.

Did x86 AAA instruction have use (and abuse) cases? - Yes, https://www.hugi.scene.org/online/coding/hugi 17 - coaax.htm
Did it make the code that used it more compact? - Yes.
Did it reduce the number of registers in the code that used it? - Most likely.
Why is it not in RISC-V ISA? - Because it doesn't make sense to include an instruction just because there are some marginal, rare use cases (with obvious and efficient workarounds).
Another example is VAX: "The INDEX instruction is used to calculate the address of an array element while at
the same time checking to see that the index fits in the array bounds. This is clearly an important
function to accurately detect errors in high-level languages statements."
Why is the INDEX instruction not in RISC-V ISA? - Because it doesn't make sense if you believe the arguments from this paper: D. Patterson, D. Ditzel, "The Case for the Reduced Instruction Set Computer".
I argue that the complexity cost of having tail undisturbed in addition to tail agnostic is significantly worse than AAA and INDEX because it affects most of RVV instructions, which means verification space explosion.
It also contradicts the open source spirit by raising the complexity barrier and preventing a variety of implementations.
Another downside: confused SW people will use (and abuse) tail undisturbed in all kinds of unnecessary ways (as evident from this thread). And then they will be even more confused by subtle, hard to predict/hard to explain performance issues of tail undisturbed in high performance RVV implementations.

Also, with renaming and tail undisturbed some widening instructions need to read 5 vector registers instead of at most 4 without tail undisturbed.
Register file ports are expensive, which means some implementations might choose to execute such instructions at half throughput in tail undisturbed mode.


How to empty data?

How to empty data object without delete methods? Thank you

You can't delete a property without the delete keyword, but if you just want to set everything to undefined, you can do:
for(let key in this.$data) {
  this.$data[key] = undefined;
}

this.build();

This will set to undefined methods too.
There is a way to set to undefined just data?

Ahh, well you can do something like:

Closing for now, let me know if you need more help with clearing data.


Add better metrics reservoir tick overflow handling

The current implementation of SlidingTimeWindowArrayReservoir handles clock tick overflows much better, than SlidingTimeWindowReservoir. This is a change for SlidingTimeWindowReservoir to postpone tick overflow as much as possible, as it is done in SlidingTimeWindowArrayReservoir.
Maybe it would be even better to clear all measurements in case of overflow as it is done in SlidingTimeWindowArrayReservoir, but this may be treated as a backward-incompatible change from an end-user perspective.

The question is: why would you even want to use SlidingTimeWindowReservoir if you can use SlidingTimeWindowArrayReservoir since it is a full drop-in replacement.
SlidingTimeWindowArrayReservoir is used in production by many systems for years now, so maybe it is time to go one step further get fully replace SlidingTimeWindowReservoir with SlidingTimeWindowArrayReservoir implementation. What do you think @arteam, @joschi, @ryantenney?

I agree that SlidingTimeWindowArrayReservoir is a better choice, but from my perspective, it would be nice to at least backport the tick overflow handling to 3.x branch as it would not hurt anyone.


Plex container stopping (s6 errors)

I am getting some version of these S6 error before Plex server is asked to close.  Seems to happen randomly every few days.
FIRST:
[s6-finish] waiting for services.

SECOND:
[s6-finish] waiting for services.
s6-svwait: fatal: timed out
[s6-finish] sending all processes the TERM signal.
[s6-finish] sending all processes the KILL signal and exiting.
Seems maybe related to random restarts of OmniMount.
Anyone else? Thoughts
thanks

This is a Plex error in their stop algorithm, not OmniStream.  OmniMount will restart if the mounts fail and this could be for a number of reasons.  The problem is that, when it restarts, any container mapped to it's shares ends up remapped to the empty directory on the host where the mounts are overlaid.  I specifically wrote this process for Plex since it seems to like to remove entries from your library when shares disappear (and yes, this is even sometimes when you have Plex configured to NOT delete missing videos).
Since the failure can be tracked via Docker's healthchecks, this serves to raise a flag that OmniMount is restarting and any container dependent on it's mounts will subsequently restart once OmniMount is back up and the shares are verified.
Plex's kill switch triggers their own scripts and, at least one of them has had incorrect parameters for years.  It doesn't cause any damage since all the "real" data is written outside of Plex's container.


"View the assets in the file explorer" link broken

see:

This and the problem with the link's root not pointing to the repository subdomain are both fixed on staging with #129
Address-only submission

Sources submission


Incomplete type annotation for nextFocus* props in ViewProps

https://github.com/react-native-tvos/react-native-tvos/blob/f5004239e33bb527c26303addb8eecb2a9e72534/packages/react-native/types/public/ReactNativeTVTypes.d.ts#L10-L35
Minor issue that causes TypeScript to complain.
The nextFocus* props should ideally also include the View type, since refs to a View component can also be passed as a value to these props despite the type not specifying it. As an example, the DirectionalNextFocus examples under rn-tester use View refs for the nextFocus* values.

@jschoeny please have a look at #850 and see if that will solve the issue you are seeing.


macOs monterey support

My neo-cli 3.1.0 is not working after I upgraded to Monterey.

Are you running on Apple Silicon?

Are you running on Apple Silicon?

No. Intel


ghidraDev misc issues

The ghidraDev plugin works with a sufficiently new eclipse (4.10 I think?), which can be retrieved from a sufficiently new nixpkgs. There is a gdk problem currently causing a lot of segfaults, which is worked around by the unset.
unset GDK_PIXBUF_MODULE_FILE; nix-shell -I "nixpkgs=https://github.com/NixOS/nixpkgs-channels/archive/nixos-unstable.tar.gz" -p eclipses.eclipse-java
Originally posted by @deliciouslytyped in https://github.com/NixOS/nixpkgs/pull/56959#issuecomment-474289837

[ ] Creating a new Ghidra Module project will fail with IOError: Failed to find skeleton directory. if you use nix-build and specify a path like /home/user/ghidra/result. My hypothesis is that symlinks are not followed properly. Specifying a direct path to the nix store is successful.

The gdk_pixbuf issue looks like #54278.

I seem to have successfully made a working plugin system, I should be filing  a PR in the coming days.


Emojis not indexed? They return no results

Using PageFind with tweetback I've noticed that searching for emojis return no results (issue here)
Testing it on your demo page confirmed that it did not work: searching for 🌋 should return this page as a result: https://xkcd.pagefind.app/comics/2017-11-24-emoji-sports/
Is it known behavior? Would there be a way to index emojis to enable searching for them like any other caracters?

Hey @Julianoe 👋
The latest Pagefind beta (1.0.0-beta.1) includes support for emoji. There are also a couple of changes on the beta, most notably a rename from the default /_pagefind/ to /pagefind/, which I haven't finalized a set of release notes for.
Let me know if you give this a go! It will be part of a full 1.0 release next week or so, so no worries if you just want to wait til then 🙂
In lieu of running it, here's the new test that now passes: https://raw.githubusercontent.com/CloudCannon/pagefind/main/pagefind/features/characters.feature
(Search Pagefind matches emoji)

Hey @Julianoe  ! 👋
Good news — this has landed in Pagefind v1.0.0! ✨
See the full release notes here: https://github.com/CloudCannon/pagefind/releases/tag/v1.0.0 💙

Awesome! Thanks!


Callback not working (iOS/Android)

Hi, playback works fine on both iOS and Android, but callbacks (onError, onSuccess, onStatusChange) are not being called.
I tried using org.cordova.plugin.media and callbacks work, but stream playback does not work on iOS.
I'm using latest Cordova.
Thanks

I've noticed the same problem.  To be fair cordova does seem to have the same problem.
in CDVStream.m we do have this call:
[self.commandDelegate sendPluginResult:result callbackId:command.callbackId];
Being a non objective-c programmer I cant say why that doesn't send the callback, but it appears to be not functioning as expected

Callbacks are not working, because these are not implemented in ios version of this plugin.


2.4.0 sess.run freezing in Windows 10 with RTX 3090

my DeepFaceLab software uses tf 2.3.1 with compat.v1, it works fine.
When I upgrade to 2.4.0rc4, tf sess.run is freezed without any errors.
Seems like it depends on model complexity.
Models with big graph freeze instantly.
Model with simpler graph may be freezed in 30 minutes.
I don't know how to write reproduction code.
OS: Windows 10
Windows 10 setting
Hardware-accelerated GPU scheduling to “ default graphics setting ”, solves the problem.

Hardware-accelerated GPU scheduling to “ default graphics setting ”, solves the problem.

TF 2.X packages come with gpu support however if the gpu environment setup is not compatible it silently falls back on cpu for computations.
TensorFlow 2.4 pip packages are built with CUDA11 and cuDNN 8.0.2.
See release notes.

 I have exactly same issue on my large scale CNN network, and it solved by enabling Hardware accelerated GPU scheduling in Windows 10. (Python 3.8 + TensorFlow 2.4.0)

@iperov I have exactly same issue on my large scale CNN network

do you also use compat.v1 ?

No, I used keras API. TF 2.4.0 makes application freezed (0% cpu/gpu usage) without errors.

@KichangKim

I used keras API.

so it is mean the problem not only with v1 graph system, but also with v2 eager execution?

2.4.0 release does not fix the issue.

The same with me. I have tried 2.4.0-rc0 rc1 rc2 rc3 rc4 and 2.4.0 but unexpected thread freezing usually but not always  appear.

@ymodak still no solution?
why users should do the fix in windows settings?

I don't know what the "Hardware accelerated GPU scheduling" feature does (perhaps it uses the new CUDA-graph HW support in Ampere?), so I don't understand why it is interacting poorly with TensorFlow.  Someone familiar with this new setting in Windows will have to chime in.

I am using RTX 2080, it is not ampere.

reference, https://github.com/tensorflow/tensorflow/issues/45716

@psycho2012  already referenced, you are blind flooder

also in Windows Server 2019 no solution at all.
Stable freeze with tf 2.4.0 and rtx 3090

https://github.com/fo40225/tensorflow-windows-wheel/tree/master/1.15.4%2Bnv20.12/py38/CPU%2BGPU/cuda111cudnn8avx2
This repo build tf1.15 whl with cuda11 support，maybe you can try it

@Fannhhyy
py3.8 - no
avx2 - no

@Fannhhyy
py3.8-没有
avx2-没有

https://github.com/fo40225/tensorflow-windows-wheel/issues/167#issuecomment-767277221
The author fixed the nvidia-tensorflow build on windows. Maybe you can try to build it.
'https://drive.google.com/file/d/1Zeln6Buadpokml3wmTwyJipRo2JvRKP8/view?usp=sharing'
I tried to build it , but I don't have time to test the whl.

Hi, could you please check if the issue still persists with latest Tensorflow version or tf-nightly version.
Also, make sure you update the drivers and the tested build configurations from here. Thanks!


can't use shortcuts

We have written the needed data into your clipboard because it was too large to send. Please paste.Hi,
I'm having an issue while using VS (I'm a newbie, it could totally be me). The shortcut to run the code (control+f5) sometimes works and sometimes doesn't. Same thing happen when I want to "clear" the terminal. I know, it's strange... Any clue what it may be?
Thank you!
Antonio

@wing0621 The command Developer: Toggle Keyboard Shortcuts Troubleshooting needs to be ran in VS Code. You can press F1 or ctrl+shift+p / cmd+shift+p to bring up the command palette in VS Code and execute the command


Terminal: Cannot set property 'innerHTML' of undefined

Seeing this when making the terminal visible in selfhost after having quite some output. It seems to try to access a div that is not existing in the array:

Duplicate, already fixed https://github.com/Microsoft/vscode/issues/7357


Applying operation-operationId to callbacks

Describe the bug
operation-operationId rule triggers warning when there is no operationId defined for the operations under callback object.
To Reproduce

Define an operation with callback.
Run openapi lint with recommended ruleset configuration

Expected behavior
I'm not actually sure whether it's a bug or not, but I couldn't come up with the real word usage for operationId in callbacks. Thus, I'd expect this rule to be checked only against Oas3Operation directly under Oas3Paths.Oas3PathItem. Or have it configurable.
Logs
➜ openapi lint example.yaml
No configurations were defined in extends -- using built in recommended configuration by default.

validating example.yaml...
[1] example.yaml:31:15 at #/paths/~1test/post/callbacks/notification/{$request.body#~1callbackURL}/post

Operation object should contain `operationId` field.

29 | '{$request.body#/callbackURL}':
30 |   post:
31 |     summary: 'Creation completed'
32 |     requestBody:
 … |     < 10 more lines >
43 |           description: OK
44 |

Warning was generated by the operation-operationId rule.

example.yaml: validated in 18ms

Woohoo! Your OpenAPI definition is valid. 🎉
You have 1 warning.

OpenAPI definition
openapi: 3.0.3
info:
  title: test
  version: 1.0.0
  description: test
  license:
    name: Apache 2.0
    url: https://www.apache.org/licenses/LICENSE-2.0.html
servers:
  - url: https
paths:
  /test:
    post:
      summary: test
      operationId: doTest
      requestBody:
        content:
          application/json:
            schema:
              type: object
              properties:
                callbackURL:
                  type: string
      responses:
        '202':
          description: Accepted
      callbacks:
        notification:
          '{$request.body#/callbackURL}':
            post:
              summary: 'Creation completed'
              requestBody:
                required: true
                content:
                  application/json:
                    schema:
                      type: object
                      properties:
                        id:
                          type: string
              responses:
                '200':
                    description: OK

openapi-cli Version(s)
1.0.0-beta.46
Node.js Version(s)
v14.8.0

I heard this is the same issue with webhooks.


Typed arrays exported from tool scripts cannot be edited

Godot version
9f12e7b52d944
System information
arch linux
Issue description
Attempting to edit from the editor a typed array exported from a tool script fails, any value is immediately reset, setters are never called.
Steps to reproduce
Create a script such as follows and try to add and then change items of the array in the inspector.
@tool
extends Node

@export var x: Array[String]

Minimal reproduction project
n/a

Reverting the commit from #76378 resolves this issue.

cc @KoBeWi, regressioception. :laughing:

No idea why this happens. The text sub-property is re-created for some reason, but only in tool script.

This seems to be fixed on the latest version. I tested both on the commit referenced by OP and on (https://github.com/godotengine/godot/commit/8c729f0f34a92b92a1a8204e1d56b6ad05cfd27b). While the issue did appear in the first one, it is no longer present on the latest build.
Tested on Windows 11, Forward+ mode.

Try restarting the editor and checking again, because you seem to have some bug (the <null>s).
I can still reproduce this issue.

You're right, after restarting the editor the issue does still appear on master


Ading content json asociated to atoms

Is possible to have a special section (like @JSONcontent) where I can spcify the content for every molecule?

Hi cromozooom!
Could you give some more details, please? What are you referring to? The markup?
Maybe provide an example DocBlock how you imagine this.

I wish to have the standard content inside a JSON file for every component, if this is possible

One of you just implemented @script annotation. This is practically same, but for CONTENT. (https://github.com/holidaypirates/nucleus/pull/15/files#diff-0)


COOPR-675 provisioner refactor master

this is the first part of https://issues.cask.co/browse/COOPR-675
this PR moves the provisioner master into coopr-provisioner/lib (eliminating the master directory), and updates a few related paths.
This PR will break standalone on develop.  https://github.com/caskdata/coopr/pull/910 and a submodule ref update will fix it.

Update the Rakefile to point to the new specs.


Scrape AnimeFillerList and skip filler episodes

Ability to be able to skip downloading of filler episodes given an AnimeFillerList url.
Example for One Piece, https://www.animefillerlist.com/shows/one-piece
Afaik, AFL has no API so you need to manually enter the url.

Great PR, I'll add a search function.

man it took a while but here it is
https://github.com/Chr1st-oo/anime-downloader/pull/1
also fixed a bug that your code didn't skip some episodes e.g 100-150 would only skip 100 and 150


When adding a row, duplicate the OrigQuote field

Currently, when adding a row in tC Create, it duplicates the SupportReference field from the previous row. That is almost never desired. What usually is desired is to add a Note that comments on the same or mostly the same OrigQuote string. Please duplicate the OrigQuote field instead of the SupportReference field.

@mandolyte will this be solved in 1.2?

Probably won't be too hard to do, but I think 1.2 is imminent and I have been working on 818, which is now in review. Perhaps you are thinking of the editing/rejuvenation of deleted rows?


Update documentation-government-networking.md

The documentation around the boundary was not specific to what the boundary is. The proposed update specifies the boundary to be an ITAR boundary.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@mark-beaupre : Thanks for your contribution to the Azure documentation! The author, @jawalte, has been notified to review your proposed change.

#sign-off

#sign-off


Fix daily combo

Fixed a logical error that prevented the daily combo combo from being purchased if one of the cards had already been purchased

Thanks


Add nGlide SDK

SteamDB app page links to a few games using this
https://steamdb.info/app/245010
Brief explanation of the change
Find nglide_config.exe

After running changed rules in this pull request (this bot only tests rules.ini changes):
Changed technologies: SDK.nGlide
These apps now match: (24) 38450<PHONE_NUMBER>10<PHONE_NUMBER>40<PHONE_NUMBER>10<PHONE_NUMBER>70<PHONE_NUMBER>60<PHONE_NUMBER>50<PHONE_NUMBER>20<PHONE_NUMBER>90<PHONE_NUMBER>20<PHONE_NUMBER>90 1112570 1257910 1710170
Please confirm that all these changed apps are correct.


Mark a badge as not counted in totals

Badges like Bug Hunter and Flames of Prometheus are not counted in the badge totals.
Will require a new flag like virtual or additional... some decent name for marking a badge as not counted. Preferably something in the negative, so the default false or undefined value means it is a normal badge, for backward compatibility.

Settled on field name ignoreInTotals. By default, all badges will be included this way, and only badges with this set to true will be ignored.


Identity not use is chained

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-identity


My files in the "pages" can't decrypt

I had decrypt metadata.edn
but my files in the "pages" ie still encrypt
how can I  decrypt them?

@kanru what happened here?

From the screenshot it seems there is a nested logseq folder? Make sure the layout is correct before trying to decrypt. You can try to decrypt the inner logseq folder directly.


Account Profile Dropdown Inconsistent with New Post Dropdown

The account profile dropdown will pop open when a mouse hovers overtop while the New Post dropdown requires a click. Ideally, both will require a click. This would be an opportune time to lower the desktop dropdown to be under the Gear Icon in a similar fashion to what we see on mobile screens.

This has been resolved with 15F100. Closing Ticket.


note-parser not working when using as a library in another project

Hi!
Synthesizers work great for me in the demo project. However, when I add react-music as a dependency to another project, I'm unable to use Synth or Monosynth, seemingly because of an error using note-parser:
Uncaught TypeError: Cannot read property 'freq' of undefined
    at Monosynth.createOscillator (monosynth.js:147)
    at Monosynth.playStep (monosynth.js:163)
    at Scheduler.process (scheduler.js:119)
    at loop (scheduler.js:34)

The Sampler instrument works fine.

I think this is a note-parser package issue.   and  both use parser.freq and it seems like depending on what kind of syntax you're using it works on some environments where in other it doesn't (In my case, i was working on a react app using create-react-app)
I had to move the entire react-music src/ folder to inside of my react project and had to switch out all the import parser from 'note-parser' to  var parser = require('parser')
and now it seems like it's working! Not elegant but a workaround for now.
Inside my apps
import {
  Analyser,
  Song,
  Sequencer,
  Sampler,
  Synth,
} from '../ReactMusic';

Inside Monosynth, Synth
var parser = require('note-parser');

Is there an update on this?

Is there an update on this?

It's not the right solution but it works temporarily for me if I change line 128 in node_modules/react-music/lib/components/synth from this:
// osc.frequency.value = _noteParser2.default.freq(transposed);
to this:
osc.frequency.value = _noteParser2.freq(transposed);
_noteParser2.default is undefined
Would be nice if that could be fixed in a new release.

This worked for me 👍!


Keybinding editor isn't focused under Metacity window manager

What steps will reproduce the problem?
1. Start X session with Metacity window manager
2. Open ibus-mozc keymap editor
3. Click the keymap table twice to open keybinding editor

What is the expected output? What do you see instead?

Keybinding editor should be raised and focused, but it appears behind
the keymap editor and isn't focused. This behavior is very annoying
because we must click the keybinding editor every time to bring up it.

What version of the product are you using? On what operating system?

I use ibus-mozc r25 in Ubuntu 10.04.

Please provide any additional information below.

I think the keybinding editor must send a client message
"_NET_WM_STATE = _NET_WM_STATE_ABOVE" to the root window when using
Metacity window manager.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 28 May 2010 at 7:36
Attachments:

mozc_keybinding_editor_screenshot.png

[deleted comment]

Sorry, I made a mistake in additional information.

The client message that the keybinding editor needs to send is
_NET_ACTIVE_WINDOW.

Original comment by<EMAIL_ADDRESS>on 28 May 2010 at 8:04

Thanks for the report. Reproduced on Ubuntu Lucid x86_64.

Original comment by<EMAIL_ADDRESS>on 30 May 2010 at 12:32

Changed state: Accepted

Thanks for the report.

Seems that this might be a bug of setFocusProxy() in Qt4, since the same code
does work as expected on Windows/Mac.
We are going to pass a special window flag Qt::WindowStaysOnTop to the
keybinding editor so that it is always displayed at the top of all other
windows.
This is just a workaround but better than current situation.

Original comment by<EMAIL_ADDRESS>on 1 Jun 2010 at 7:20

Fixed it in revision 30, it is just a workaround though. Please check it.

Original comment by<EMAIL_ADDRESS>on 16 Jun 2010 at 11:30

Changed state: Fixed

Umm... The keybinding editor is actually raised but not focused, so it still
needs mouse clicking.

Please reopen this issue.

Original comment by<EMAIL_ADDRESS>on 17 Jun 2010 at 12:08

Original comment by<EMAIL_ADDRESS>on 17 Jun 2010 at 12:20

Changed state: Accepted

Original comment by<EMAIL_ADDRESS>on 1 Apr 2012 at 2:20

Added labels: OpSys-Linux


Docs improvement

Can't figure how to run this
If I simply add all hosts to internal section of config.json and run, it should work, right?
Getting empty (Timed Out) results so far

Running in kubernetes cluster containers with NET_RAW capability added

Is the project still alive? No activity from June...

The SYN-ACK packets are denied by Calico iptables rules, even though all traffic is allowed between hosts. My suspect is the packet is marked as NEW, not ESTABLISHED.. Will kepp investigating


FIX primal computations with infinite weights

@sehoff this explains why your case 3.1 was slow. use Lasso(verbose=1) or verbose=2 to get more info about the solver progress; here the primal objective was nan at first iteration, hence it was just running until the max number of iterations and epochs, so a very long time.
@Badr-MOUFAD have a look at this, when this is merged can you add a unit test testing this case for Lasso, LogisticRegression and GroupLasso ?
Basic script to reproduce the issue on main:
import numpy as np
from celer import Lasso, GroupLasso

np.random.seed(1)
X = np.random.randn(50, 60)
y = np.random.randn(50)

weights = np.ones(60)
weights[30:] = np.inf
alpha_max = np.max(np.abs(X.T @ y) / weights) / len(y)
clf = Lasso(alpha=alpha_max / 10, weights=weights,
            max_iter=1, max_epochs=20, verbose=2).fit(X, y)

Codecov Report

Merging #232 (0d6b37d) into main (32147a2) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #232   +/-   ##
=======================================
  Coverage   86.55%   86.55%
=======================================
  Files          14       14
  Lines         967      967
  Branches      129      129
=======================================
  Hits          837      837
  Misses        100      100
  Partials       30       30

Flag
Coverage Δ

unittests
∅ <ø> (∅)

Flags with carried forward coverage won't be shown. Click here to find out 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 32147a2...0d6b37d. Read the comment docs.


VACMS-8688 Component: block_content--alert

I noticed that the links don't have google analytics events firing like we have on the banners, is there anything that needs to be added for tracking banner interactions?

@amponce there was no logic in the current implementation that recorded GA events so I don't think its necessary


A small issue of Table. 1

Thanks for your good work. I would like to kindly clarify a small detail in Table 1: MLLA is actually not causal.

Oh sorry for that! It's a typo. We will fix it right now and upload an updated version!


Fix abnormal page token cases in Ruby sample gen

Addresses https://github.com/googleapis/toolkit/issues/692.

Rebased.

Should be submitted concurrently with https://github.com/google/google-api-ruby-client/pull/528.

Codecov Report

Merging #986 into master will increase coverage by -1.17%.

@@             Coverage Diff              @@
##             master     #986      +/-   ##
============================================
- Coverage     85.18%   84.02%   -1.17%
+ Complexity     3341     3303      -38
============================================
  Files           340      340
  Lines         13340    10060    -3280
  Branches       1513     1424      -89
============================================
- Hits          11364     8453    -2911
+ Misses         1543     1177     -366
+ Partials        433      430       -3

Impacted Files
Coverage Δ
Complexity Δ

...former/ruby/RubySampleMethodToViewTransformer.java
97.29% <100%> (-0.94%)
20 <ø> (+2)

.../transformer/py/PythonSampleTypeNameConverter.java
42.85% <ø> (-36.64%)
4% <ø> (-1%)

...very/transformer/js/JSSampleTypeNameConverter.java
38.46% <ø> (-28.21%)
7% <ø> (-1%)

...ry/transformer/php/PhpSampleTypeNameConverter.java
50% <ø> (-27.28%)
6% <ø> (-1%)

.../transformer/ruby/RubySampleTypeNameConverter.java
46.66% <ø> (-23.71%)
6% <ø> (ø)

...very/transformer/go/GoSampleTypeNameConverter.java
50% <ø> (-22.42%)
8% <ø> (-1%)

.../java/com/google/api/codegen/SnippetSetRunner.java
33.33% <ø> (-16.67%)
1% <ø> (ø)

...n/transformer/ruby/RubyModelTypeNameConverter.java
65% <ø> (-16.34%)
15% <ø> (-1%)

...odegen/discovery/MainDiscoveryProviderFactory.java
72.72% <ø> (-15.96%)
5% <ø> (-1%)

...ansformer/nodejs/NodeJSModelTypeNameConverter.java
68.29% <ø> (-14.38%)
16% <ø> (-1%)

... and 151 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 c9450a4...2c7cb50. Read the comment docs.

lgtm


KAFKA-8040: Streams retry initTransactions on timeout

As of 2.0, Producer.initTransactions may throw a TimeoutException, which is retriable. Streams should retry instead of crashing when we encounter this exception.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@guozhangwang @bbejeck , Can you take a look at this if you get the chance?
https://issues.apache.org/jira/browse/KAFKA-8040
I took Guozhang's suggestion from the ticket and kicked off a rebalance if we get a TimeoutException from the init transactions call.
This is definitely safe to do, but I'm wondering if it might be too heavy weight. If something as simple as a network hiccough breaks the tcp connection to the broker, the Producer will currently just hang until the "max block" timeout expires and then throw this exception.
Under those circumstances, it would be sufficient just to try again. But in this PR, we'll actually do a rebalance, pausing processing and creating a whole bunch more broker API calls until the rebalance completes.
I'm wondering if we should instead just retry in a loop just the initTransactions call. It seems like this preserves the semantics, which is to block until the call returns.
Is there some aspect to this that I'm missing which would require a rebalance to resolve?
Thanks,
-John

Actually, after looking more closely at the code, I realized that the StreamsException will be fatal. It might have been a mistake, but I thought the intention was to re-try the call.
It certainly seems appropriate; TimeoutException seems like a retryable exception, and initTransactions doesn't retry internally.
WDYT?

Java 8 failures were:

org.apache.kafka.common.network.SslSelectorTest.testCloseConnectionInClosingState
kafka.api.AdminClientIntegrationTest.testConsumerGroups

Retest this, please.

@mjsax , thanks for the review.
Yes, the idea is to log a more informative message, but still shut down. I was confused when I wrote the original log message.
I thought it would be easier to target 2.0 and forward-port it to 2.1, 2.2, and trunk. (the exception was introduced in 2.0)
I'm not sure if we want to do something different on trunk... We can talk about whether Streams should try to stay alive after retriable exceptions, but it's a much larger discussion. In the short term, it seems like we'd just want to merge this change to trunk as well.

I thought it would be easier to target 2.0 and forward-port it to 2.1, 2.2, and trunk. (the exception was introduced in 2.0)

Works for me. For this case, the PR should be against trunk and we cherry-pick to older branches (this is the common pattern).

Java 8 passed, Java 11 failed with kafka.server.LogDirFailureTest.testIOExceptionDuringLogRoll
retest this please

As requested, I've rebased this PR on trunk.

Ok! The Jenkins tests have passed. Thanks, @mjsax , @bbejeck , and @guozhangwang !

Merged #6372 to trunk
Thanks @vvcephei!

cherry-picked to 2.1 and 2.2 will merge separate PR for 2.0


Feature adult classifier

Classify adult content when it contains xxx in the name and try to get info from metadataapi.net using an api key
I tried to use the code from the resolver tmdb, it may contains some duplicate code.
We may add a button NSFW and blur the image but I don't know anuglar.
Let me know if something is wrong.

Hi, I'm not sure about the metadataapi integration - what's their track record, it looks a bit fly-by-night (I might be wrong)? As I understand most adult content is available on TMDB that we've already integrated (is there a lot of adult content on metadataapi but not on TMDB?) - it might just be a case of adapting the existing movie/TV regexes to adult content and querying TMDB. Whereas the movie/TV classifiers require an exact API match we probably just want to put things in the XXX category if the name contains relevant keywords; if possible we'll find a TMDB match but generally XXX releases are more likely to be "unofficial" or not something we can find on any API.

Closing this PR due to lack of response.


[DO NOT MERGE] Disable travis kube until it can be fixed

Coverage remained the same at 41.003% when pulling e5ac44fff5a73045540f51758d5676aba5e89e64 on eriknelson:disable-kube-travis into 2d991adcad64ff3c51389cc1b72c3a2846096994 on openshift:master.


Thumb position gap

Hello,
I really like this slider and how it is accessible and CSS agnostic, however there is one tiny thing that I wonder if something can be done about it.
I created a sandbox to explain it: https://codesandbox.io/s/festive-shannon-swbmg?file=/src/App.tsx
Is it possible to fix this gap between the track and the thumb? It's literally 1px 😓

The gap is a result of the border radius on the thumb. If you remove the border radius, you'll no longer see any gap:

If you want to remove the gap, you can make the thumb larger and offset it to cover the gap.


Safari is only loading footer on all pages but Home

Safari is only loading footer on all of the pages but the Home Page. this is when navigating to by URL/Tabs/Buttons.
Safari version 11.1
https://openforge-website-dev.herokuapp.com/

This appears to have been fixed with other footer related fixes. Closing the ticket


cant set any Startup Connections

I cant setup any connections as a startup
i cant create any connection shortcuts
i like to have an automatic connection after reboot, but  it does not work either
my system: windows 10 pro  x64 21H1  19043.1023  Windows Feature Experience Pack 120.2212.3530.0
SoftEtherVPN v436  Build 9754
i know this workeda long time ago  on my win 10.
but since ~2  years  it doesnt work  for me.
see  attached pic

How is the picture related to your problem?

How is the picture related to your problem?

As u can see, the "Set as Startup Connection" is greyed out and not usable for me!

You did not select a connection so even connect is greyed out.

..as u can see, i have an active connection, but i cant chose. how to do that?
thank

s

Can you try on normal (non-VPN gate) connections?

how to do that inside the SoftEther App?
I only use that app for VPN!

Create a fake connection from Add VPN Connection and try setting that as startup

yes sir, that fake connection works.
ii can set up as start up, and i can make an shortcut

but i  like to have this from that included VPN list.

That seems impossible since you don't know what server you are connecting to before selecting one from the list.

..but that worked before. i think 2 years ago maybe?
after i chosed an connection from that list, it appears in the main tab as connected. so why cant i pick that connection up?

i never had setup any other (fake) connections... but i can promise that it has worked before!

OK. You can download and install the old version.

phew, i really dont know which version that was before... i permanently keep my pc updated...

would it be that hard for you to bring us those functions back?

First you need to show that it did work like that before, which I don't think so.

when i had installed this package a long time ago, there was directly an connect to an vpn after reboot!

Well what I can say is that the relevant code hasn't changed in the past 8 years.


"Memory does not start at address 0" => Always?

In the test of the memory map, there is an assertion that ensures that the first physical address in the UEFI memory map is not zero.
As far as I know, there is nothing which guarantees that this will be the case. The EFI_MEMORY_DESCRIPTOR spec does not dictate it, and there is actually something at physical address 0 on x86, even though the EFI firmware may not show it (not to mention other CPU architectures where there is nothing special about a null pointer).
So spontaneously, I would replace this assertion with one that the number of pages of the first map entry is nonzero, which is guaranteed by the UEFI spec.

Ah, sorry, I was confused by the assertion style :( It does check that it is zero.


Web Demo - Unable to Paste or Add Quotes

I encountered issues while trying out the web demo. I’m unable to paste my HTML code for parsing. Additionally, when I attempted to type the code manually, I found that I couldn’t add quotes (").
https://github.com/user-attachments/assets/9852b63e-e2e0-4cc3-a7b9-c83f5a454bb7

I updated the web demo, this should be fixed now.

Note that images may not work on the web demo, I'm checking it.


Poor Yuzu Performance on Hyprland - Not on Sway

When running any games through Yuzu on Hyprland, performance is extremely poor.  Switching to Sway resolves this.
(I have previously run Yuzu through GNOME and got similar results to Sway)
My Setup:

NixOS - Config Here
Hyprland from the Flake
ASUS Zephyrus G14 - 2021
Yuzu 1484 (Flatpak) (NVIDIA Vulkan)

Steps to Reproduce
Run any Yuzu game
FPS Comparison (In Menu)

Hyprland => 8-14
Sway => 60 (capped)

Observations
When comparing Hyprland and Sway, the microprofile traces from Yuzu yielded vastly different results. One major observation was that the "Copy to Swapchain" step was taking drastically longer on the Hyprland side. For the most part, the "Execute Command Buffer" step remained unchanged between the two.  The other was that the frametimes were seemingly much less consistent on Hyprland's end.
@fufexan

Doesn't seem to be a Nix-specific issue. In any case, @end-4 is on the hunt for the commit(s) that indroduced performance regressions so we may know more about what causes this when we have some of those commits figured out.

My yuzu is running great on the latest Hyprland though...

Sway

sway or sway-git? always test on -git

Write the following shell.nix to build Sway from source.
{pkgs ? import <nixpkgs> {} }:
pkgs.mkShell {
  nativeBuildInputs = with pkgs.buildPackages; [
    bash-completion
    basu
    cairo
    cmake
    fish
    gdk-pixbuf
    glslang
    hwdata
    json_c
    libdisplay-info
    libdrm
    libevdev
    libinput
    libliftoff
    libxkbcommon
    mesa
    meson
    ninja
    pango
    pcre2
    pixman
    pkg-config
    scdoc
    seatd
    systemd
    vulkan-loader
    wayland
    wayland-protocols
    xorg.libX11
    xorg.xcbutilrenderutil
    xorg.xcbutilwm
    xwayland
  ];
}

When running through the latest Sway, the performance was nearly identical between the two (bad). Does that meant hat the problem is with wlroots?

your hardware is bad, or did you forget to plug in the charger or something?
i ran mario kart 8 deluxe on both the git version of hyprland and sway at 60fps and 16.67ms frame time. not to even mention that was using integrated radeon graphics.

Can confirm that it isn't the hardware (I had played TOTK for ~30hrs on Fedora before switching to NixOS). Out of curiosity, I reverted to the nixpkgs version of Hyprland and the performance went back up. The only common factor I've noticed (between both Sway and Hyprland both) is a change upstream in wlroots.

can you git bisect to narrow down the wlroots rev change that broke this?

I'm actually going to do a 180 and say the issue is completely on my end.  For a sanity check, I built Sway 1.8.1 (current release) and the corresponding wlroots version from source to test it out and got the same performance as I did on Sway-git.
Now why this is happening is a mystery to me, I built in release mode (meson setup --buildtype=release --wipe build/) and also tried manually adding -Ofast -g3 to the meson build script but it would seem that this regression happens whenever I'm compiling it (I didn't use the cached flake builds). I can try using clang instead of gcc but in my experience, that has never made a noticeable difference.

Can you check the output of nix run nixpkgs#glxinfo? If it doesn't report the correct GPU in renderer info and instead says llvmpipe, there's a mesa mismatch.

I get my AMD iGPU as the output which would make sense since I have supergfxctl set up.
I was fairly certain it wasn't using software rendering since my laptop doesn't sound like a jet engine and CPU usage was around ~(0-1)% according to btop.
That being said, I know Yuzu is using my NVIDIA GPU.  As an initial troubleshooting step, I had nvidia-smi open and was watching power usage and utilization and that was being registered in nvidia-smi.  I even turned on the force max clock option in Yuzu which ensured the GPU drew all it could of its 60W allowance and that had no effect.


Select OS

In SolusIO you have a OS with subversios like
Ubuntu

Subversion: 18.04
Subversion: 20.04

But in WHMCS it's not possible to select this subversion.
It's only possible to set "Ubuntu".

It displays OsId instead of OsImageId - Will be fixed soon

Fixed in v1.0.3


didn't know how to start any example in Python

Whenever i run python examples in folder testpy (python test*.py) i get this error:
in init
rules.error: Could not create connection, error code: 301

Hi, thanks a lot for posting the question. By default the rules engine assumes there is a redis instance listening on localhost port 6379.
You can install a local redis instance:
https://github.com/jruizgit/rules/blob/master/docs/py/reference.md#local-setup
If you already have a redis instance, you can configure your ruleset to use it by passing host_name and port to the run_all function:
run_all([{'host': 'host_name', 'port': port, 'password': 'password'}])

Closing this issue. No action item left.


Ibm plugin node scenario

Adding ibm plugin node-scenario

@chaitanyaenr updated after seeing error below yesterday. No longer seeing today
error out listing instanceslist indices must be integers or slices, not str


Error occurs during compilation in Touch.h

An error occurs during compilation
_In file included from C:\Users\wszaj\Dysk Google\Arduino\FT6336U\Touch\Touch.ino:1:0:
C:\Users\wszaj\Dysk Google\Arduino\libraries\FT6336U/FT6336U.h:502:2: error: 'Finger' does not name a type
Finger _finger[2];
^_

C:\Users\wszaj\Dysk Google\Arduino\FT6336U\Touch\ does not seem to be part of M5Core2 folder structure.
You can't expect the example sketch from an external driver to work natively with M5Core2 code.
You should try the example from the M5Core2 group instead:

Hi, as @tobozo mentioned this is not part of our library and therefore won't work, you can try the examples in the library to work your way around them.


Docs: Guide page for decorators

decorators are a pretty nifty part of angular that is currently only documented as part of $provide. We should have a guide page that shows some common use cases: decorating a (core) service and decorating a directive. Probably also a good place to mention some limitations.

@Narretz I was surprised there was no guide for decorators when I came across this. So, I am putting together a PR and I will have it opened soon, but I need to address backwards compatibility.
I see this is in the 1.5.x milestone but there aren't any branch labels. The document I created gives examples using a couple 1.5.x exclusive features and I then realized that it should be probably be backwards compatible. The question is: how far back should this document support? If it is prior to v1.4.0-rc.0 then it looks like the initial version shouldn't include mentions of the decorator exposure through the module (e57138d) either. I can prepare a pre v1.4.0-rc.0 document and then a current document noting the module.decorator (a simple, but sometimes confusing aspect that I'd like to cover in the guide). Forgive me if I am making this more difficult than need be, but I am not sure how the core team handles previous version support and patches.
Please get back to me when you can, and I will get a PR opened and ping you for a code review.
Thanks!

@sjbarker The guides are versioned, e.g. https://code.angularjs.org/1.5.1/docs/guide. So BC is not an issue. To cover pre-v1.5, you'll need to backport.

@awerlang Fantastic thanks!


Dataflow issue after a devops pipeline

Just follow you step, and when I run a release with a devops pipeline, DataFlow column on the left of the datafactory is not show
however, when I go to a pipeline and double click on a dataflow, it open it
why I cannot see dataflow on the left then?
Thanks

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

ID: 1e22e4d6-366f-fbf8-e141-8905ca7ab7ae
Version Independent ID: 5be2acf2-66e7-cadf-6666-5a85a65b9771
Content: Continuous integration and delivery in Azure Data Factory - Azure Data Factory
Content Source: articles/data-factory/continuous-integration-deployment.md
Service: data-factory
GitHub Login: @djpmsft
Microsoft Alias: daperlov

@kilagg What is your permission on the release pipeline? If you are only a reader, there is a known issue with viewing data flows in the UX.

Thanks for the feedback and bringing this to our notice . At this time we are reviewing the feedback and will update the document as appropriate .

@kilagg Could you please try refreshing / reloading the Data Factory in your web browser after publishing and let me know if that helps?

@kilagg
We will now proceed to close this thread as we have not heard back from you. If there are further questions regarding this matter, please tag me in your reply and reopen the issue. We will gladly continue the discussion.


Add try..finally around RunAsync and WaitForTokenShutdownAsync

The try..finally ensures that the ManualResetEventSlim which AttachCtrlcSigtermShutdown uses is set even when an exception occurs in these two methods.
Fixes #1194
I also looked at if I could any tests to test this behavior (besides manually testing it with a repo sample project) and looked at https://github.com/aspnet/Hosting/blob/dev/test/Microsoft.AspNetCore.Hosting.FunctionalTests/ShutdownTests.cs but saw in the code that it was disabled because of #1214 and so I didn't add upon it. If anyone has any idea how to add tests for this behavior I am all ears and will add it to this PR.

 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: Rutix sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Thanks


Separate routs for GraphQL and GraphiQL

I'd like to configure two different routes for GraphQL such as /api/graphql and for GraphiQL such as /graphiql .
This currently is not possible.

Can you not add the path explicitly?
public void Configure(IApplicationBuilder app, IHostingEnvironment env, ILoggerFactory loggerFactory)
{
    app.UseGraphiQl("gaphiql");
    app.UseMvc();
}

@pouladpld I'm doing some work around routes at the moment so I can add the option for separate routes.

@poulad This has been added in the latest release thanks to @FrankyBoy, I'll now close this issue. Feel free to reopen it if needed.


How to redirect to other custom list page after success on update button

Redirect to other custom list page after success on update button
in short i have to redirect user to custom listing page after don update on "save-and-close-button"
i am not getting idea how can i do it.

Put this into existing DialogUtils.ts:
export function dialogCloseRedirect(url: string, element: JQuery) {
        element.bind('dialogbeforeclose', function (e) {
            if (!Serenity.WX.hasOriginalEvent(e)) {
                return;
            }
            e.preventDefault();

            // *** Redirect to url where the new list is located ***
	    window.location.href = url;

        });
    }

and in the xyzDialog.ts, use it like this:
DialogUtils.dialogCloseRedirect("http://url-to-new-list", this.element);

With kind regards,
John


chore: remove unused code

Changes

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#779
#778  👈
#777
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @AngelOnFira and the rest of your teammates on  Graphite


additional symbols are pasted (ref. to issue 2131)

From @alexeyshishkin01 on June 9, 2017 6:21
This bug-tracker is monitored by developers and other technical types.  We like detail!  So please use this form and tell us, concisely but precisely, what's up.  Please fill out ALL THE FIELDS!
If you have a feature request, please post to the UserVoice.  If you're reporting a BSOD, don't post here!  Instead, e-mail<EMAIL_ADDRESS>and if possible attach the minidump from "C:\Windows\minidump".

Your Windows build number:  (Type ver at a Windows Command Prompt)
D:>ver
Microsoft Windows [Version 10.0.15063]
root@Alexey-HP:~# az --version | head -1
azure-cli (2.0.7)

What you're doing and what's happening:  (Copy&paste specific commands and their output, or include screen shots)
I copy-paste a command into BashOnWindows and additional symbols are pasted:

what was copied is:
az vm image list-publishers \

--location northeurope
--query "[?starts_with(name, 'Microsoft')]"

what was pasted is:
root@Alexey-HP:~# az vm image list-publishers
ion> --location northeurope \

--query "[?starts_with(name, 'Microsoft')]"

What's wrong / what should be happening instead:

additional symbols are pasted
no additional symbols are expected

Strace of the failing command, if applicable:  (If <cmd> is failing, then run strace -o strace.txt -ff <cmd>, and post the strace.txt output here)
N/A

See our contributing instructions for assistance.
Copied from original issue: Microsoft/WSL#2204

From @sunjoong on June 9, 2017 8:30
@alexeyshishkin01 - I think.... Screenshots of Get-Clipboard on windwos PowerShell and output of paste like below could be helpful to figure it out;

Hmm.... and ignore the conents of above screenshots; In there, I pasted ">" to show the output of "i> > --location northeurope".

From @alexeyshishkin01 on June 9, 2017 9:54
additional symbols also vary

From @alexeyshishkin01 on June 9, 2017 9:52
I'm attaching the screens from:

notepad

power shell

bash-on-windows

the main issue it doesn't repeat all the times - sometimes it does happen, the other time it doesn't

From @sunjoong on June 9, 2017 10:24
@alexeyshishkin01 - I see. You meant "ion" of your description and "_" of your bash-on-windows screenshot before ">", alike "li" and "u" before ">" on the below sceenshot;

@bitcrazed I think something went wrong copying this over, it is the first comments of the same issue as #92

Thanks for the ping. Alas, GitHub doesn't provide a way to move issues between repro's, and their API's are gated to ~5000 calls per hour. Looks like they started gating which confused the moving tool I was using, and not all move operations were completed successfully :(
Sorry for any corruption during this one-time move. I'll try to go back and retroactively restore missing comments.

Closing this issue as it's a corrupted dupe of #92.


Having issues with local paths

Hi,
Im having a few problems locally, where the plugin can't load the JS and CSS assets.
Looking into it I've noticed that it is trying to load the assets with an invalid path:
http://webipack.dev/booklet/wp-content/plugins/Users/luismartins/Sites/webi…v/wp-content/themes/wip-roots/lib/kirki//assets/css/customizer.css?ver=0.5

It is adding my local path to an URI and that will not work.
Worth mentioning this is a site running on a multisite install, perhaps it may have a relation to it.
Any way I can get around this?

Hey there!
Are you using Kirki as a plugin, or are you including it in your theme?
If you're including it in your theme, have you defined a url_path in your configuration?
What are you using there?

Hi again Aristeides!   :D
Im embbeding Kirki in my theme and setting the url_path like this:
'url_path'     => get_template_directory_uri() . '/lib/kirki/',

I've already try switching the order my code, so that I would include the library before/after passing the filter kirki/config but can't make it behave differently.
Im also a bit confused with the documentation. I was passing a property url_path for the $args object, but it seems that now it should be a variable called $url, is that correct?
Im asking because the docs refer to both.

Doing some testing, here's what I found so far.
In the Dependencies class the $controls var is getting the value im passing through the config filter. For some reason the $kirki_urlis not getting the correct value, as you can see here:
https://cloudup.com/cyz_Fqzs8_w

Aristeides, I now managed to have it load all the files but somethings not right yet.
I've found two problems:

In dependencies.php you were reading the wrong variable name when setting $kirki_url
In customizer.php you're not checking for the existence of a url_pathsetting, so it always assumes the default value.

I would do a PR but im not sure my fix is the most efficient way to solve it.
The issue though, is that the customiser UI's still broken:
https://cloudup.com/cPOt9sPPJMK
As far as I can tell those settings are coming from customize.php, so im not sure where to look for next.
Can you help please?
Thanks.

Hi there, got the same issue, also got it in my theme without the usage of the plugin methode.
http://website.nl/secret/wp-content/plugins/C:/domains/website.nl/www…cret/wp-content/themes/website/kirki//assets/css/customizer.css?ver=0.5
serialize.js, customizer.js also got this problem, for some reason tries to load those files from the plugin folder.
i got the latest files from github with these lines in my function.php
include_once ( dirname( FILE ) . '/kirki/kirki.php' );
and ( http://kirki.org/#configuration ):
 function kirki_demo_configuration_sample() {
$url  = get_stylesheet_directory_uri() . '/kirki/';

/**
 * If you need to include Kirki in your theme,
 * then you may want to consider adding the translations here
 * using your textdomain.
 *
 * If you're using Kirki as a plugin then you can remove these.
 */

$strings = array(
    'background-color' => __( 'Background Color', 'kirki' ),
    'background-image' => __( 'Background Image', 'kirki' ),
   .......
);

$args = array(
    'logo_image'   => get_stylesheet_directory_uri() . '/images/logo.png',
    'description'  => __( 'The theme description.', 'kirki' ),
    'url_path'     => $url,
);
return $args;  } add_filter( 'kirki/config', 'kirki_demo_configuration_sample' );

So for example the slider does not show etc..
But when installing it as an plugin those 'error' were fixed and all works well.

Hey guys! Can you please try the develop branch and let me know if that fixes it for you?

Sorry, couldn't test this out at home yesterday.
When I switch to dev im getting a 500 with the following in the console:
[01-Apr-2015 08:12:45 UTC] PHP Fatal error:  Allowed memory size of 268435456 bytes exhausted (tried to allocate 72 bytes) in /Users/luismartins/Sites/webipack.dev/wp-includes/functions.php on line 4470
[01-Apr-2015 08:12:45 UTC] PHP Stack trace:
[01-Apr-2015 08:12:45 UTC] PHP   1. {main}() /Users/luismartins/Sites/webipack.dev/index.php:0
[01-Apr-2015 08:12:45 UTC] PHP   2. require() /Users/luismartins/Sites/webipack.dev/index.php:17
[01-Apr-2015 08:12:45 UTC] PHP   3. require_once() /Users/luismartins/Sites/webipack.dev/wp-blog-header.php:16

hi there!
Deleted the old kirki folder and pasted in the develop one.

Fatal error: Class 'Kirki_Fonts' not found in C:\domains\reply-none.nl\wwwroot\secret\wp-content\themes\reply-none\inc\customizer.php on line 331

line 331 =

$controls[] = array(
'type'     => 'select',
'setting'  => 'base_typography_font_family',
'label'    => __( 'Font Family', 'example' ),
'section'  => 'base_typography',
'default'  => 'Roboto',
'priority' => 20,
      'choices'  => Kirki_Fonts::get_font_choices(),
'output' => array(
'element'  => 'body',
'property' => 'font-family',
),
);

'choices'  => Kirki_Fonts::get_font_choices(),  this gives the font error.
So deleted the font choices for some further testing:

Warning: call_user_func_array() expects parameter 1 to be a valid callback, function 'slider' not found or invalid function name in C:\domains\reply-none.nl\wwwroot\secret\wp-includes\plugin.php on line 213

Warning: call_user_func_array() expects parameter 1 to be a valid callback, function 'text' not found or invalid function name in C:\domains\reply-none.nl\wwwroot\secret\wp-includes\plugin.php on line 213

used the '$control' examples from http://kirki.org/#field-types
Hope this will help, good luck.

Im rebuilding the affected site to see if I can isolate the issue, where's what I got so far:
[01-Apr-2015 09:08:39 UTC] PHP Warning:  Invalid argument supplied for foreach() in /Users/luismartins/Sites/webipack.dev/wp-content/themes/wip-roots/lib/kirki/includes/Helpers/helpers.php on line 31

[01-Apr-2015 09:08:39 UTC] PHP Warning:  Invalid argument supplied for foreach() in /Users/luismartins/Sites/webipack.dev/wp-content/themes/wip-roots/lib/kirki/includes/Scripts/Frontend/GoogleFonts.php on line 22

[01-Apr-2015 09:08:39 UTC] PHP Warning:  Invalid argument supplied for foreach() in /Users/luismartins/Sites/webipack.dev/wp-content/themes/wip-roots/lib/kirki/includes/Styles/Frontend.php on line 178

[01-Apr-2015 09:08:40 UTC] PHP Warning:  Invalid argument supplied for foreach() in /Users/luismartins/Sites/webipack.dev/wp-content/themes/wip-roots/lib/kirki/includes/Scripts/Customizer/PostMessage.php on line 47

I got to the point where I have a blank, fully configured site without the 500, so im guessing it's not related to Kirki. I've also managed to resolve the problem where text and background where at the same color.
Apart from the notices above, I don't have standing issues in my project. Im not using the Google fonts control in this project.

Jesus, im having a terrible time with Kirki this morning.
I've just noticed that with developer branch I can't make any change to the defaults, said in other way, get_theme_mods() returns an empty array no matter what.
Anything I can do about this?

did some testing with the twentyfourteen theme, got the same issues as stated above.
with the normal kirki it gives an error in console: double http request pluging folder
with the develop kirki it gives errors on the $controls

@replynone sorry about that, I deleted the Kirki_Fonts class by mistake yesterday. I just restored it...
Examining the other issues you guys reported right now.
I'll get back to you ASAP, hopefully with a fix.

Thank you @aristath, don't let my grumpiness bother you as it wasn't the intention. It's just one of those days ;)

The main issue here is that I still can't replicate this issue so I don't know exactly what goes on.
Here's exactly what I did:

I copied Kirki to the twentyfifteen theme, in inc/kirki
I included the kirki main file from the functions.php file:

require get_template_directory() . '/inc/kirki/kirki.php';

I added some demo fields and then added this configuration:

/**
 * Configuration sample for the Kirki Customizer
 */
function kirki_demo_configuration_sample() {

    $args = array(
        'logo_image'    => 'http://kirki.org/images/logo.png',
        'description'   => __( 'This is the theme description.', 'kirki' ),
        'color_accent'  => '#00bcd4',
        'color_back'    => '#455a64',
        'url_path'      => get_template_directory_uri() . '/inc/kirki',
    );

    return $args;

}
add_filter( 'kirki/config', 'kirki_demo_configuration_sample' );

It looks fine on my end!!
@lmartins would it be possible for you to send me some FTP & admin credentials for that site so that I may take a look at what goes on?
If yes, then please send them via email to aristath at gmail dot com

@aristath The main topic of this issue was the loading of assets files and that is solved.
The 500 im getting in one site I've already disregarded as Kirki's triggered. I still don't know what happened but I can't replicate it in other sites either.
At this point the single issue Im having is that once I checkout the "develop" branch, no customisations get to the front-end, as if they were being stored with different identifiers. It saves the values in the backend, but nothing gets to front. Anything changed in this regard recently?

@lmartins oh! ok....
I didn't realise the assets issue had been resolved and it was driving me insane. :smile:
Yeah, I was experimenting with https://github.com/aristath/kirki/issues/104 last night and just pushed my latest changes before going to sleep. It was an incomplete commit, I'll get that fixed in a couple of minutes. :+1:

oh man, what a relief... thank you :D

Can you please check the develop branch now?

I'm out of the office but I'll check it soon as I get there, in about 30 minutes. Thanks!
Sent from my iPhone

On 01/04/2015, at 13:16, Aristeides Stathopoulos<EMAIL_ADDRESS>wrote:
Can you please check the develop branch now?
—
Reply to this email directly or view it on GitHub.

Im getting a fatal error:
[01-Apr-2015 13:05:06 UTC] PHP Fatal error:  Call to undefined method Kirki::controls() in /Users/luismartins/Sites/webipack.dev/wp-content/themes/wip-roots/lib/kirki/includes/Styles/Frontend.php on line 27

Try deleting that line. From what I can tell it's not used anymore.

It used again later in the same file to retrieve in the function loop_controls:

	function loop_controls() {

		$controls = Kirki::controls()->get_all();
		$styles   = array();

		foreach ( $controls as $control ) {
			$element  = '';

So far so good!  Did some small testing with a textbox, got the value and no errors in the console. Going to setup a small customizer section tomorrow, I will keep you posted.
tnx for all your good work :+1:

@replynone what branch did you tested?
For me, in branch develop, is outputting the fatal error stated above.

martins, yes I took the latest one from develop tab, deleted all the old files.
function.php
include_once ( dirname( FILE ) . '/kirki/kirki.php' );
/**

Configuration sample for the Kirki Customizer
*/
function customizer_config() {
$args = array(
 // Change the logo image. (URL) Point this to the path of the logo file in your theme directory
         // The developer recommends an image size of about 250 x 250
 'logo_image'   => get_template_directory_uri() . '/images/logo.png',

 // Only use this if you are bundling the plugin with your theme (see above)
 'url_path'     => get_template_directory_uri() . '/kirki/',

 // The color of active menu items, help bullets etc.
 'color_active' => '#444',

 // Color used for secondary elements and desable/inactive controls
 'color_light'  => '#eee',

 // Color used for button-set controls and other elements
 'color_select' => '#34495e',

 // You can add your own custom stylesheet for full control as well
 // For the parameter here, use the handle of your stylesheet you use in wp_enqueue
 'stylesheet_id' => 'customize-styles',

);
return $args;

}
add_filter( 'kirki/config', 'customizer_config' );
then took some pre settings from the kirki site.

Im not sure why it is working for you. In my project Kirki is a gitsubmodule, so im just pulling directly from github and checking out the develop branch.
The function in question does a loop for each control and as far as I can see isn't declared anywhere, so im a bit surprised is working for you.
Anyway, this is dev branch, master is ok, although in master there some styling issues.
@aristath Do you wan't me to help with styling in any way?

But is is declared... https://github.com/aristath/kirki/blob/develop/includes/Controls.php#L38-L49
That's what troubles me!
As for styling sure! If you have an improvement in mind just jubmit a PR. :)

@aristath I said earlier that the assets were already loading ok, turns out I was mistaken. Damn. Sorry about that.
To saves us all from back and forth messages, im doing a PR with my proposed resolution that im asking you then to confirm if you don't mind.
There are three files missing, CSS and JS causing multiple issues. I'll update you once I submit the PR.

@lmartins, what method are you using to set the url_path for Kirki?  After some commits last night, I can't replicate the issue anymore, (though previously I had)
 * Configuration sample for the Kirki Customizer
 */
function kirki_demo_configuration_sample() {

	$url  = get_stylesheet_directory_uri() . '/inc/kirki/';

	$args = array(
		'logo_image'    => 'http://kirki.org/images/logo.png',
		'description'   => __( 'This is the theme description. You can edit it in the Kirki configuration and add whatever you want here.', 'kirki' ),
		'color_accent'  => '#00bcd4',
		'color_back'    => '#455a64',
		'url_path'      => $url,
	);

	return $args;

}
add_filter( 'kirki/config', 'kirki_demo_configuration_sample' );```

Can you ensure that you're using get_stylesheet_directory_uri() as outlined in the block above?

@aristath
https://github.com/aristath/kirki/pull/151 should fix the issues with assets loading if user sets the url_path option.
@briancwelch Im doing the same, but after reading the code I am surprised it works for you. As you can see in https://github.com/lmartins/kirki/commit/1b5b25bfd6015eb51f0ec000bcedc380e4d5963f the var wasn't being set for loading from the user specified option.  Are you sure you have the plugin bundled with your theme? If it was installed as a plugin this setting doesn't make a difference I believe.

@lmartins No plugin installation.  I started a fresh WordPress vagrant with TwentyFifteen and the Kirki develop branch included under /wp-content/themes/twentyfifteen/inc/kirki
The reason I asked, is because as soon as I change get_stylesheet_directory_uri() to get_stylesheet_directory(), I am able to replicate the issue instantly.

The switch from get_stylesheet_directory_uri() it is normal to break as they produce different values.
Im guessing that it works for you because you have the files inside the theme itself, which you shouldn't if you wan't to update it later. Because you're using the same path as Kirki gets by default, it ends up working with the default value. Just a guess, @aristath can say better im sure.

@lmartins I'm afraid I don't understand then. If Kirki is installed as a plugin, what benefit is there to setting the url_path?

@lmartins can you please try pulling the develop branch now and test this again?

Ok, some weirdness going on.
I have the project as a git submodule on my now project. I've checked out develop and pulled from github.
Lines 26-27 from Frontend.php still shows me
$config   = Kirki::config()->get_all();
$controls = Kirki::controls()->get_all();

but on github site I see those lines are different.
Manually deleted every file and replaced it and it is now working. What the actual ******.
I have no ideia what happened here, and why git was showing me different code after checking out the repo but I am now quite hesitant about git submodules.
Well, it is loading just fine (code from the dev branch), so im closing this.
Thank you all for all the help.

@lmartins hey there! Out of curiosity did it work?
Do you still have an issue with this?

Hi @aristath, sorry, I should have been clearer about that.
Yes, is working now, still scratching my head over the git remote thing but when it comes to Kirki everything seems fine now.

Awesome!
Thanks for the update man! :+1:

Thank you for all the work.
Btw, do you have any ETA when will these get merged into master?

Hopefully by Monday I'll have any remaining bugs squashed, merge the branch and release an update.
If you want to help what we really need is just lots and lots of testing on the develop branch!!!
If you happen to find any bugs (no matter how large or small) just create a new issue so that we may deal with it. :+1:


Update robot_arm.py

Looks like the class name was changed from URRobot to BaseURRobot

Good point. The example folders also need to be updated


IPython version detection not working correctly

From<EMAIL_ADDRESS>on 2012-05-26T19:20:53Z
Spyder Version:  2.2.0dev rd98df4092e16 Python Version:  2.7.2
Qt Version:      4.7.4, PyQt4 (API v2) 4.8.6 on Windows
Preferences...Console...External Modules is incorrectly detecting the state of the current IPython install. As a result, it is not possible to set command line options for the IPython kernel, and the user is forced to use the default "--pylab=inline" for all IPython consoles.
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=1060

From<EMAIL_ADDRESS>on 2012-05-26T17:34:11Z
I have implemented a fix for this here: https://code.google.com/r/jedludlow-spyderlib-default/source/browse/?name=jedludlow-issue-1060 Please review and pull as desired. Incidentally, the same problem exists in the v21 repo, and we should decide if it's worth back porting.
Status: Started
Cc: pierre.raybaut ccordoba12<EMAIL_ADDRESS>Labels: -Type-Defect

From pierre.raybaut on 2012-05-27T00:52:21Z
Fixed by revision 01d32ba360a4 on default branch, and by the following revision on v21 branch: https://code.google.com/p/spyderlib/source/detail?r=b9ab513bec1ab37dd90a5bdd0084de6835a67357&repo=v21
Status: Fixed

From<EMAIL_ADDRESS>on 2012-05-29T09:00:50Z
I am no longer seeing the Ipython kernel option in my menu.  In the past I have had to change the check to check for 0.13dev, but in this case is_module_installed('IPython', '>=0.12) is returning true.  However, I still don't have the option to fire up a kernel...any thoughts?

From<EMAIL_ADDRESS>on 2012-05-29T08:27:27Z
Looks like the merge at r55999e3e509c created a partial regression on this issue.
I have volunteered to check out compatibility of IPython 0.13 with the new IPython console changes that have recently been committed, so allowing versions of IPython greater than 0.12 is required. See code review on the original submission here: https://code.google.com/r/jedludlow-spyderlib-default/source/detail?r=01d32ba360a4862ba921f91cce3d2ecb99864751 @-Carlos: Would you mind making a quick change that modifies the merge at r55999e3e509c to take the line with "is_module_installed('IPython', '>=0.12')"?
Status: Started

From<EMAIL_ADDRESS>on 2012-05-29T09:17:16Z
Yes, it was the check in externalconsole that I missed.
Thanks

From<EMAIL_ADDRESS>on 2012-05-29T09:08:57Z
@-Matt: The is_module_installed('IPython',...) check occurs in more than one place in the code. I experienced the same symptoms as you and ran a full find-in-files to locate all the locations where it might not be checking for >=0.12, and that's when I found the one location mentioned in comment 3. Try looking there.

From ccordoba12 on 2012-05-29T14:14:52Z
Ok, that was a mistake introduced by the merge because I did it before Pierre committed his changes. I'll fix it right now.
Jed we trust on you to tell us of problems with 0.13dev. Hope there aren't much :)

From ccordoba12 on 2012-05-29T14:23:48Z
This issue was updated by revision 4913e5253eb1 .
This was a problem introduced by the merge in revision 55999e3e509c
Status: Fixed

From<EMAIL_ADDRESS>on 2012-05-29T14:30:52Z
I also have been, and will continue to use 0.13dev...so far so good.

From<EMAIL_ADDRESS>on 2012-05-29T15:28:31Z
Verified the fix on Windows 7.
Status: Verified

From ccordoba12 on 2012-05-29T14:32:34Z
Ok, really good to know!


Flair Recognizer Initialization Error

Hello,
I am getting an error while trying to instantiate an object from the FlairRecognizer class defined in this document:
flair_recognizer.py.
The error happens at line 92 of the file:

Error(s) in loading state_dict for XLMRobertaModel:
Unexpected key(s) in state_dict: "embeddings.position_ids".

Steps to reproduce the behavior:

Clone this code flair_recognizer.py
Run it and wait for the weights of the model to be downloaded.
See the error.

Did you find a solution or workaround for this error?

Could the bug be related to this? https://github.com/flairNLP/flair/issues/3284

My issue came from the incompatibility of one of the Flair package requirements with Apple silicon processors (arm64), lxml.
I could fix that by manually building the package based on the provided instructions using the following code:
STATIC_DEPS=true pip install lxml
Note that there might be cached data saved from pip that could prevent you from building the package. You may delete the caches before building the package or use the following code to ignore the saved caches:
STATIC_DEPS=true pip --no-cache-dir install lxml
I hope that helps!
P.S. I wasn't sure if that would be something related to presidio, and that's why I closed the issue.


Multi-segment execution trace

This PR builds on #77 and adds multi-segment execution trace support.

[x] Update TraceInfo struct.
[x] Update prover to commit to multiple trace segments.
[x] Update proof struct to support queries against multiple trace segments.
[x] Update Air trait to support handling of auxiliary trace columns, including for transition and boundary constraints.
[x] Update verifier to support constraint evaluation and composition for auxiliary trace segments.
[x] Update prover to support constraint evaluation and composition for auxiliary trace segments.

Thank you so much for reviewing! The idea for the example looks good to me (I haven't checked it out in detail - but will do that probably a bit later today).

I've opened a PR against this branch to add the Rescue example, which works with the above comment fixed.
I did it really quickly so there may be things I've overlooked. One thing I noticed though is that calling evaluate_constraints() is pretty slow with RAPs, and is scaling with the length of the trace (+35ms for the non-RAPs Rescue chain vs 1200/2500/3500 for the RAPs chain with length 2^10/11/12). I'll try looking deeper into it if I have some free time.

Thank you! I'll fix the issue with OOD frame handling a bit later today - that should be pretty easy. The part about slowdown is interesting - I'll need to investigate this a bit more. But to confirm: is it specifically evaluate_constraints() that slows down? Or does constraint evaluation slow down in general (because we also need to call evaluate_aux_constraints())?.

I benchmarked the different chunks on the verifier side to track the slowdown, below for 8192 hashes chain:
Evaluated all main transition constraints in 5 us
Evaluated all auxiliary transition constraints in 1 us
Evaluated all transition constraints in 12693 us
Evaluated main boundary constraints in 6 us
Evaluated aux boundary constraints in 3 us
Part of it comes from the fact that I was using iterated powers of the random element for the constraints, instead of distinct ones. Switching to several ones brings the cost down to ~ 7000 us for evaluating the auxiliary transition constraints, still heavy but already better.

I've updated the verifier to process OOD frame in the same was as the prover does. Hopefully, this fixes the bug.
I haven't had a chance yet to look into the performance issue in detail yet - but I have two comments:

It makes sense that using independent random variables (rather than computing powers of a single variable) would be more performant as, I guess, powers are computed on each invocation of the evaluate_aux_transition() function, but independent variables are drawn only once before we start evaluating constraints. We could make the procedure for building random aux elements more custom that powers could be pre-computed ahead of time, but I'm not sure if that's worth it (though, open to feedback on this).
Yes, using such long cycles would lead to inefficiencies (ideally, we would want to keep cycles in periodic columns fairly small). I'll need to think a bit more about a different way to identify the first and the last row of the trace.

@Nashtare - do you think there are more things to review here? Or is it OK to merge?

Everything looks good to me! I guess we're good to merge this to main once #85 is merged to this branch.

Great! I'll merge now and we can merge the example into main once it's ready.


Bug: md-menu

Bug
What is the expected behavior?
When a menu is open and you click another action outside of the menu it should follow through with that action.
What is the current behavior?
When a menu is open and you click another action outside of the menu it closes the menu and doesn't follow through with the action.
What is the use-case or motivation for changing an existing behavior?
When a component uses a transparent overlay this indicates that it should not affect the users other actions.
Which versions of Angular, Material, OS, browsers are affected?
Latest

This is working as intended


[Removal]:

Hotspot Name
Orbiting Bone Jay
Hotspot b58 Address
112cQt4KMvkYiijxYxd5HLrkLj9JS6rak3VVAKsxZJNjrBTtxnqy
Discord Handle
No response
Hotspot Manufacturer
PantherX
Removal Reason
I bought a used hotspot
Modifications
No
Extra forwarders
No
Extra antennas
No
Additional Information
No response

We have taken into account this removal request in the next iteration of the denylist. It may take some time before it's processed. This is an automated comment due to the volume of addition and removal requests. A future tool is in the works to provide more insight into the analysis approach made by the Helium team and community members that maintain this list.


Fix avoid-returning-unused-results edge cases

app/assets/javascripts/admin/lib/autocomplete_v2/tags_autocomplete.coffee
app/assets/javascripts/admin/modules/rte/mixed_content_protection.coffee
app/assets/javascripts/admin/modules/shipping/carrier_shipping_rate_provider.coffee
app/assets/javascripts/admin/modules/shipping/shipping_rate_available_services.coffee
Error: {type: ReturnStatement, start: 3412, end: 3419, loc: [object Object], argument: null} does not match field "expression": Expression of type ExpressionStatement
    at Array.forEach (native)
    at NodePath.<anonymous> (avoid-returning-unused-results.js:24:41)
    at Array.forEach (native)
    at avoid-returning-unused-results.js:22:14
    at Array.forEach (native)
    at NodePath.<anonymous> (avoid-returning-unused-results.js:16:10)
    at Array.forEach (native)
    at avoidReturningUnsusedResults (avoid-returning-unused-results.js:12:6)

app/assets/javascripts/admin/modules/multi_progress_bar.coffee
TypeError: Cannot read property 'arguments' of undefined
    at Object.getFieldValue
    at NodePath.NPp.getValueProperty
    at getChildPath
    at NodePath.get
    at NodePath.<anonymous> (avoid-returning-unused-results.js:14:10)
    at Array.forEach (native)
    at Collection.forEach
    at avoidReturningUnsusedResults (avoid-returning-unused-results.js:12:6)
    ```

The ReturnStatement error comes from a returnsans argument:
    _.each('', token => {
      if (!foo) {
        return; // This is the problem.
      }

      return bar();
    });

The arguments error comes from nested _.each calls:
function zoo() {
  return _.each(foo, bar => {
    return _.each(qux, buzz => {

    });
  });
}


go-snaps permission denied when running from packagestest

When trying to run snaps.Matchsnapshot with packagestest
returns error mkdir /redacted/..../go/packages/packagestest/__snapshots__: permission denied.
reporter: @rtfpessoa

This is also happening when calling from "github.com/stretchr/testify/suite".
func (suite *ExampleTestSuite) TestExampleSnapshot() {
	snaps.MatchSnapshot(suite.T(), "hello world")
}

// In order for 'go test' to run this suite, we need to create
// a normal test function and pass our suite to suite.Run
func TestExampleTestSuite(t *testing.T) {
	suite.Run(t, new(ExampleTestSuite))
}


%classpath add mvn results in message rate exceeded

It seems that printing the Maven status causes an error. Can we raise this rate limit for Maven by default somehow?
IOPub message rate exceeded.
The notebook server will temporarily stop sending output
to the client in order to avoid crashing it.
To change this limit, set the config variable
`--NotebookApp.iopub_msg_rate_limit`.

Current values:
NotebookApp.iopub_msg_rate_limit=1000.0 (msgs/sec)
NotebookApp.rate_limit_window=3.0 (secs)

IOPub message rate exceeded.
The notebook server will temporarily stop sending output
to the client in order to avoid crashing it.
To change this limit, set the config variable
`--NotebookApp.iopub_msg_rate_limit`.

Current values:
NotebookApp.iopub_msg_rate_limit=1000.0 (msgs/sec)
NotebookApp.rate_limit_window=3.0 (secs)

we plan to improve the progress display to fix this without changing the iopub limit:
https://github.com/twosigma/beakerx/issues/6367