[Discussion] Compilation, intermediary functions, statements, expressions, etc

Opening this issue so that we can share our ideas here, may help with figuring out the best strategy.
My understanding (and correct me if I'm wrong) is that a lot (it not all) the intermediary functions introduced inside the body of a compiled defun are compiled let forms.
And since most functions end being async, this also introduces a lot of asyncs once those functions have been introduced.
If we have an expression, most of the time it can be converted into an equivalent statement, with its last expression becoming a return or an assignment.
When some piece of code can be compiled into a Javascript expression, then it is better to do so because it results in a smaller code size, statements would be used when it is not possible to produce an expression.
What comes ahead is an incomplete analysis, and there are probably many holes in my undestanding, so please point those out.
let expressions
Since lets are expressions, using blocks requires inserting a return to the expresion that
produces the value of the let expression.
Example:
(let X 1 (+ X X))

Gets compiled to:
(X => $.asNumber(X) + $.asNumber(X))(1)

With blocks and statements the equivalent would be:
{ const X = 1;
  return $.asNumber(X) + $.asNumber(X); }

But this only works when the result of the let expression is also the result of the defun.
If the let expression shows up someplace where its result will be bound to a variable or used as an argument to a function call, then a return cannot be used. What can be done instead, in the case of a variable bind, is to declare the variable as let Var; (instead of const), and then introduce the block as before, but with an assigment to Var instead of a return.
Example:
Source:
(let X (let Y 1 Y) (+ X X))

Current:
(X => $.asNumber(X) + $.asNumber(X))((Y => Y)(1))

With statement
{ let X;
  { const Y = 1;
    X = Y; }
  return $.asNumber(X) + $.asNumber(X); }

The compiler should include this information in the context, so that when the let is compiled it knows what to do (either return or variable assignment).
Since expressions are compiled "from the outside to the inside", annotating the context with this information shouldn't be hard.
The situation where the result of the let is used as a function call parameter is more complicated: (+ (let X 1 X) 2).
If the expression doesn't have side-effects, then it can just be assigned to a gen-symed variable using the method described above, and then that variable used as an argument.
But if there are side effects, this will not work because by doing this transformation naively we are changing the order of execution (Kl should be evaluated left-to-right).
A solution in such cases is to perform a conversion to A-normal form, so that every argument passed to the function call is bound to a variable first, then the variable passed (the transformation is easy to do, I wrote code to do this long ago because it was required for Chibi because it evaluates right-to-left, and also an OCaml port I was working on, this conversion can be done to the Kl code before the compiler even sees it).
It also has a cascading effect, every time this conversion is done, the function call that was an expression becomes wrapped in a let, which gets compiled into a statement.
And unless side-effectful code is tracked, this conversion would be required every time a let shows up as an argument to a function.
The good news is that such lets are very rare, so all this would only happen in very few cases, if at all.
Example conversion:
(+ (let A (get-A)
        B (get-B)
     (some-func B A))
   10)

becomes:
(let Dn664 (let A (get-A)
                B (get-B)
             (some-func B A))
  (+ Dn664 10))

if expressions
if expressions are compiled into Javascript's ternary operator, which work quite well because it is an expression. But statements cannot show up inside such expressions, which is a problem now once let expressions get compiled into statements.
In such cases ifs can be compiled into Javascript's ifs, and the same rules as with let can be used to handle the results.
For example (basic case, this one wouldn't require any of this, but once compiled this way then statements can be part of if branches):
Source:
(let X (if (> 2 1) 2 1) (+ X X))

Current:
(X => $.asNumber(X) + $.asNumber(X))(2 > 1 ? 2 : 1)

Proposed:
{ let X;
  if (2 > 1) {
      X = 2;
  } else {
      X = 1;
  }
  return $.asNumber(X) + $.asNumber(X); }

trap-error expressions
I see these are wrapped in a function too, but for a try/catch same rules as above would apply.
And throwing is done with $.raise, so throwing an exception doesn't convert an expression in a statement like using throw would.
Factorization
If intermediary functions are removed, the constructs introduced by the factorise-defun extension can be compiled trivially into labelled blocks and labelled breaks.
Since the constructs are new and separate there is no ambiguity and the transformation is direct (none of the analysis required for the above cases is required here, and at the same time, the above cased cannot be confused with these constructs).
The translation is described here: https://github.com/rkoeninger/ShenScript/issues/13#issuecomment-565065693
Since the conversion done by factorise-defun already adds %%return expressions, the Kl->Js compiled will not have to figure out where to insert returns.
Sync/Async
This part is something I'm not very clear on. Is it important for the async stuff to be implicit? Is it possible to instead use sync versions of the primitives that Shen requires, and then provide a new construct to make async calls explicit? if yes, what would the drawbacks be?
I'm adding libuv support on Shen/Scheme, so I'm interested in this not just because of ShenScript, and it is something I haven't given much tought to.
Code size
The statement versions require more bytes than the expressions, but I don't think the difference is THAT big, and if it results in considerably faster code, I think it is worth it.
Many calls that are repeated use functions that are properties of $, I think that with some analysis some can be removed (at least when typeckecking and the optimise flag are enabled).
Also I think expressions like (+ A B) can be compiled into $.add(A, B) instead of $.asNumber(A) + $.asNumber(B).

For reference, the previous attempt to use statement-oriented syntax is document here. The implementation used a similar approach but the result was far from perfect - there were a lot of unnecessary intermediate variables and block scopes. I'm not sure how of a performance impact that would have had.

Also I think expressions like (+ A B) can be compiled into $.add(A, B) instead of $.asNumber(A) + $.asNumber(B).

The build functions also use type-tracking so if  A and B were known to evaluate to numeric types, the rendered code would just be A + B. This is described in the docs.
Previous experiments showed calling the as* functions had trivial impact on performance, so I would remove the type-tracking and just put as* calls everywhere before removing the type checks.

Is it important for the async stuff to be implicit? Is it possible to instead use sync versions of the primitives that Shen requires, and then provide a new construct to make async calls explicit? if yes, what would the drawbacks be?

If you had to use special syntax, function, macro when using I/O in ShenScript, that would mean that any Shen code using I/O wouldn't be portable. That's quite a lot of the code since f_error and other stuff in the kernel calls I/O functions and f_error gets inserted into any unexhaustive functions.
There's some attempt to cut down on the amount of async-ing and await-ing with this fixed list of known non-async functions, but it's limited and hard to extend since functions and be re-defined to start using I/O even if they originally didn't.
https://github.com/rkoeninger/ShenScript/blob/0e990504139b08d2a44816ead38796d49619b3a7/lib/backend.js#L184-L196
There's a docs section about this too.

If you had to use special syntax, function, macro when using I/O in ShenScript, that would mean that any Shen code using I/O wouldn't be portable.

But can't those Shen functions just use the blocking I/O versions of the underlying platform? this wouldn't require any special syntax, just the regular one, and those calls would behave as expected without incurring any overhead.
This would not prevent the use of the underlying async functionality, it would just not be done through Shen's primitives. If you wanted to for example write a Web server in Shen, you would not use those primitives anyway, or even worry about portability much, because the way it will be written to perform and behave well will depend a lot on the characteristics of the underlying platform.
Will go through those links latter, just wanted to ask about this but this is the part I'm a bit confused about.

For reference, the previous attempt to use statement-oriented syntax is document here. The implementation used a similar approach but the result was far from perfect - there were a lot of unnecessary intermediate variables and block scopes. I'm not sure how of a performance impact that would have had.

Ok, from the description, Fabrs look very much like what I have in mind, so back then you were already way ahead of where I am now.
I understand that the resulting code will be bigger because statements require more bytes than expressions.
What I don't understand is what kind of runtime overhead statements could have. Lets say the Javascript compiler is really good at optimizing IIFEs, what it will produce is equivalent to the statements version, right? it will not be faster.
Did those benchmarks include compilation time? Thats where I can see some overhead happening.

The build functions also use type-tracking so if A and B were known to evaluate to numeric types, the rendered code would just be A + B. This is described in the docs.

Ok, thats great. I saw that there was type tracking code, but for the functions I tried the wrapping functions were still present, but now that I think about it it was obvious why (the variables where parameters to the functions, not results of any operation).

But can't those Shen functions just use the blocking I/O versions of the underlying platform?

I didn't want to design around special cases where you happen to be able to do sync I/O for a particular operation on a particular platform. async/await allows for a transparent merging of styles on all modern JS engines.
The idea was that a function like load would behave ostensibly the same why in ShenScript in Node and in the browser as it behaves in CL or wherever else.

This would not prevent the use of the underlying async functionality, it would just not be done through Shen's primitives. If you wanted to for example write a Web server in Shen, you would not use those primitives anyway, or even worry about portability much...

...eh, we seem to be of two minds here. I remember the main original Shen website saying "write once, run everywhere" and every platform-specific thing makes that less true. This goes back to comments I've made before like when we say Shen, is supported on platforms X, Y and Z what's "Shen" in each of those cases? Is it just the kernel? Which version of the kernel? According to the color-coded chart on the open source site, "Shen" is a little different per platform so which "Shen" are we talking about? And to a developer, "Shen" might mean the kernel plus common libraries, which might be different per-platform... it's kind of a mess and I don't want to make it any worse.

Did those benchmarks include compilation time? Thats where I can see some overhead happening.

They did - the only AOT compilation in ShenScript is when you build it - the kernel is pre-rendered. All other code gets read and compiled JIT, just like JavaScript.

What I don't understand is what kind of runtime overhead statements could have. Lets say the Javascript compiler is really good at optimizing IIFEs, what it will produce is equivalent to the statements version, right?

That's what I was thinking, but the results spoke for themselves. Maybe the code getting generated was excessively noisy in some way, but I'm not sure how. There were a lot of intermediate temp let variables but I don't know why that would matter so much.

...eh, we seem to be of two minds here. I remember the main original Shen website saying "write once, run everywhere" and every platform-specific thing makes that less true

Ideally, yes, but once you start building anything useful that goes beyond manipulation of data, you are going to have to depend on functionality provided by the platform.
This is already a problem in Scheme, which is far more defined than Shen is, has been around for much longer  and is less varied when it comes to platforms on which it runs on.
When you want to write code to solve a more complicated problem that involves interaction with the outside world, it is far more likely for it to be non-portable than portable.
To make Scheme code portable, what you do is write implementation-specific code for all implementations you want to support, and use cond-expand so that different code is evaluated on different implementations.
But then again, the kind of stuff you see that can be made portably, is not stuff that talks to the operating system, it is stuff like XML parsers, pattern matching macros, etc, not HTTP servers or libraries to talk to databases or anything like that.
All the above applies to Common Lisp too, but it is just not as bad, because the Common Lisp spec includes faaaaaar more than Scheme's or Shen, so there is more common ground covered. But even to a lesser degree the problem exists there too.
There are only a few I/O primitives in Shen, if you want to write something more complicated, those will be enough, and in that case you will have to depend on Nodejs stuff (same if you were to write something that runs on the browser and manipulates the DOM).

That's what I was thinking, but the results spoke for themselves. Maybe the code getting generated was excessively noisy in some way, but I'm not sure how. There were a lot of intermediate temp let variables but I don't know why that would matter so much.

Ok, I'm going to do the following. I'm going to compile a bunch of Shen functions, then try to manually translate them to a statement oriented version, then run then and compare (using a profile if I can figure out how to do it in node).

Just a quick update on this. I tried the following functions:
(define do-times
  0 F -> done
  N F -> (do (F void)
             (do-times (- N 1) F)))

(define simplify
  [Op A B] -> (s Op (simplify A) (simplify B))
  A -> A)

(define s
  + M N -> (+ M N)      where (and (number? M) (number? N))
  + 0 F -> F
  + F 0 -> F
  + A [+ B C] -> (simplify [+ [+ A B] C])
  * M N -> (* M N)    where (and (number? M) (number? N))
  * 0 F -> 0
  * F 0 -> 0
  * F 1 -> F
  * 1 F -> F
  * A [* B C] -> (simplify [* [* A B] C])
  Op A B -> [Op A B])

(let Exp [* x [+ [+ [* 12 0] [+ 23 8]] y]]
  (time (do-times 200000 (/. _ (simplify Exp)))))

I ran this loop, then a modified version without all the async/await, and then one where the code was translated into the statements equivalent, which looks like this:
 ((s$c, simplify$c) =>
    $.d('simplify', $.l(V2 => {
        if ($.isCons(V2) && ($.isCons(V2.tail) && ($.isCons($.asCons(V2.tail).tail) && null === $.asCons($.asCons(V2.tail).tail).tail))) {
            return $.b(s$c.f, $.asCons(V2).head, $.t(simplify$c.f($.asCons($.asCons(V2).tail).head)), $.t(simplify$c.f($.asCons($.asCons($.asCons(V2).tail).tail).head)));
        } else {
            return V2;
        }
    })))($.c('s'), $.c('simplify'));

 (($2b$s, simplify$c, $2a$s) =>
    $.d('s', $.l((V6, V7, V8) => {
        if ($2b$s === V6 && ($.isNumber(V7) && $.isNumber(V8))) {
            return $.asNumber(V7) + $.asNumber(V8);
        } else if ($2b$s === V6 && 0 === V7) {
            return V8;
        } else if ($2b$s === V6 && 0 === V8) {
            return V7;
        } else if ($2b$s === V6 && ($.isCons(V8) && ($2b$s === V8.head && ($.isCons(V8.tail) && ($.isCons($.asCons(V8.tail).tail) && null === $.asCons($.asCons(V8.tail).tail).tail))))) {
            return $.b(simplify$c.f, $.r([$2b$s, $.r([$2b$s, V7, $.asCons($.asCons(V8).tail).head])], $.asCons($.asCons(V8).tail).tail));
        } else if ($2a$s === V6 && ($.isNumber(V7) && $.isNumber(V8))) {
            return $.asNumber(V7) * $.asNumber(V8);
        } else if ($2a$s === V6 && 0 === V7) {
            return 0;
        } else if ($2a$s === V6 && 0 === V8) {
            return 0;
        } else if ($2a$s === V6 && 1 === V8) {
            return V7;
        } else if ($2a$s === V6 && 1 === V7) {
            return V8;
        } else if ($2a$s === V6 && ($.isCons(V8) && ($2a$s === V8.head && ($.isCons(V8.tail) && ($.isCons($.asCons(V8.tail).tail) && null === $.asCons($.asCons(V8.tail).tail).tail))))) {
            return $.b(simplify$c.f, $.r([$2a$s, $.r([$2a$s, V7, $.asCons($.asCons(V8).tail).head])], $.asCons($.asCons(V8).tail).tail));
        } else {
            return $.r([V6, V7, V8]);
        }
    })))($.s `+`, $.c('simplify'), $.s `*`);

Times:

Original: run time: 1.378000020980835 secs
Without async/await: run time: 0.4609999656677246 secs
With statements: run time: 0.46000003814697266 secs

It seems async/await adds considerable overhead, not sure about functions yet as this code I tried happens to not introduce intermediary functions, so the runtime for both non-async versions is the same.
Next time I will try something with a bunch of let expressions to see what happens.

On another topic, based on the code I see on backend.js I thought the asX wrappers would not be generated for this case:
* M N -> (* M N)    where (and (number? M) (number? N))

but I got this:
$2b$s === V6 && ($.isNumber(V7) && $.isNumber(V8)) ?
         $.asNumber(V7) + $.asNumber(V8)

Just wanted to point that out.

Using const vs IIFEs:
(define let-test
  N -> (let N+1 (+ N 1)
            N+2 (+ N+1 1)
            N+3 (+ N+2 1)
            N+4 (+ N+3 1)
            N+5 (+ N+4 1)
            N+6 (+ N+5 1)
            N+7 (+ N+6 1)
            N+8 (+ N+7 1)
            N+9 (+ N+8 1)
          N+9))

(time (do-times 2000000 (/. _ (let-test 100))))

Statements version:
$.d('let-test', $.l(V22 => {
    const N$2b1 = $.asNumber(V22) + 1; {
        const N$2b2 = $.asNumber(N$2b1) + 1; {
            const N$2b3 = $.asNumber(N$2b2) + 1; {
                const N$2b4 = $.asNumber(N$2b3) + 1; {
                    const N$2b5 = $.asNumber(N$2b4) + 1; {
                        const N$2b6 = $.asNumber(N$2b5) + 1; {
                            const N$2b7 = $.asNumber(N$2b6) + 1; {
                                const N$2b8 = $.asNumber(N$2b7) + 1; {
                                    const N$2b9 = $.asNumber(N$2b8) + 1;
                                    return N$2b9;
                                }}}}}}}}}))

Times:

Original: run time: 1.9229998588562012 secs
Statements: run time: 1.7259998321533203 secs

So, the intermediary functions add some overhead, but it is much less than async/await.
What commit/tag do you recommend for me to check the "fabrs" producing code? I want to try more complicated code examples, but translating them by hand gets tedious.

Just out of curiosity, I tried the version with the asNumber wrappers removed, and as you said, there is almost no runtime overhead (or not at all, because of measuring error).
Btw on this example I also got those wrappers, but it was not what I was expecting.

But can't those Shen functions just use the blocking I/O versions of the underlying platform?

Just wanted to update on this, turns out I was very wrong here, it seems Node doesn't provide what I thought it provided, and a C extension would be required, which is not ideal. I will keep thinking about this and let you know if I figure out anything.


Adding cisco spark notifier plugin

Woops sorry about that, fixed it here:
https://github.com/jenkinsci/cisco-spark-notifier-plugin/blob/master/build.gradle#L32

:+1:

@taylorp36 Merged, thanks!


feat(blankslate): add props and max-width

Proposed Changes

add max-width to blank slate description
add buttonClasses for allowing customize margin
set padding to 0 when applying p0 to .blankslate-row td

https://coveo.invisionapp.com/share/A3V885JSHJ8#/screens/402858961
Potential Breaking Changes
no
Acceptance Criteria

[ ] The proposed changes are covered by unit tests
[ ] The potential breaking changes are clearly identified
[ ] README.md is adjusted to reflect the proposed changes (if relevant)

Codecov Report

Merging #1460 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master    #1460   +/-   ##
=======================================
  Coverage   98.99%   98.99%
=======================================
  Files         496      496
  Lines       10271    10271
  Branches     2370     2370
=======================================
  Hits        10168    10168
  Misses        103      103

Impacted Files
Coverage Δ

...act-vapor/src/components/blankSlate/BlankSlate.tsx
100% <100%> (ø)
:arrow_up:

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 c07b3ff...91a4e24. Read the comment docs.


Release Page Text Changes

Miro reference: https://miro.com/app/board/o9J_kj6tycQ=/?moveToWidget=3074457366776693470&cot=14
[x] Remove HTML download button
[x] Remove XML download button
Link “Check DP release in Dataverse”

[x] Link appropriately (back to the Dataset page or DV download file)

Table with “DV Installation”, “Remaining time to complete release”, etc.

[x] Move this table after "DP Release Information" and before the accordion panels
[x] If the process is complete, remove the row "Remaining time to complete release"
ref: https://miro.com/app/board/o9J_kj6tycQ=/?moveToWidget=3074457368233364946&cot=14

[x] for support email at the bottom, use<EMAIL_ADDRESS>

reviewed


T8170 S340 looking to get support homebridge-eufy-security v2.2.21

Describe the solution you'd like
Looking to get the T8170 S340 new camera set up in Homebridge. I have the s40 camera up and running on the S380 home base and I am running  homebridge-eufy-security v2.2.21. Tried it today and it does not recognize as a camera

Looks like it was recently added to the main eufy-security-client so hopefully it will come to the plugin soon.
https://github.com/bropat/eufy-security-client/pull/410

any update about it? I also have s40 and I can't get it to work :(

A new beta should be coming this week that should add support. :)

Did it happened?On 4 Mar 2024, at 23:51, Ben Towne @.***> wrote:
A new beta should be coming this week that should add support. :)
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: @.***>

No new beta yet.
Guessing the dev has been busy w/ their real job & life stuff.
Hoping they have some free time this weekend to throw a beta version using v3.0.0 together for the community. 🤞🏼

Good news tho is it does look like they were making some changes to the hksv version 4 days ago. 🥳

It works got my new S340 working with my older S230 (S40) and both are on the Home base 3. I am using 2.3.13 Version. Thanks for the new updates...

It works got my new S340 working with my older S230 (S40) and both are on the Home base 3. I am using 2.3.13 Version. Thanks for the new updates...

I dont have the homebase, is this a requirement?
I had them showing as cameras but they wouldnt work, but now they are switches and I cant change it. Ive used 2.3.13 and 2.3.14 same issue

No it should not be a issue I had the S230 (s40) running for a long time without the Homebase 3 on Homebridge. Both the s340 and S40 work without the Homebase 3. Do you have the S340 camera.

Yes I just got a couple s340 cameras. I havent used homebridge in a long time so maybe I made a mistake. Also my setup doesnt have the images like yours.
Nothing happens in homekit unless I use the child bridge but then I still only get switches. Im using 2.3.14 now, Ive also noticed that even tho I have node v 20.11.0 its showing 20.11.1 in the homebridge ui. I deleted 2.11.1 altogethrr and have 20.11.0 confrimed I use.

Did you try do Reload and signing into Eufy again. This will load all your camera's but also make sure you have another ID from the original set on the camera's. So in other words you have to share a Id with another person and sign in with that. Here is a good video.

I just reinstalled homebridge and Eufy plugin and the UI is different, shows the pics now. Ive done that a few times but only changed this time.
I am using a secondary account I created in Eufy yes.
So now I have no child bridge but the cameras show up in homekit as 2 icons per camera, neither are cameras, looks like switches still
Do I need do make a chil bridge for it to work?

Yes I am running it as a child bridge  here is a video that help me a lot different camera but help me a lot. https://www.youtube.com/watch?v=n7YjKEVnFvE

Thank you! iIll take a look

v2.3.13 also back up to this version .14 did not work for me.

Thank you! looks like I have them working!
Only issues I see now is no snap shots in home app and im not sure if we are able to control the camera movement but I dont see a way to do so.

its also very very slow to load. Ive read that most people have seen it load quite quickly. Im gonna have to keep messing with it I guess, its not functional as it unfortunately

iCloud will give you a old picture

Here is my set up and yes it can be slow

I am running a Raspberry Pi 3 Model B for Homebridge maybe is
my speed problem also...


firstgenrepeaterv2

Could you squish the last 3 commits so all the changes relative to the previous PR are easy to read?

Could you squish the last 3 commits so all the changes relative to the previous PR are easy to read?

I should have said "squash". And I misread the number of commits. Could you squash all the new commits into a single commit with a description of what was done? If we need some additional steps after this is done we will squash them too (or just use commit --amend)


Backup solution for 1Password Vaults

User Need
As a (type of user)
I want (some action)
so that (some result)
Value
Functional Requirements:

Non-Functional Requirements:

Acceptance Criteria:

Notes:

Not going to do.


Problem with driver for Epson SX215 (SX210)

When printing labels of Avery type J8163 (7x2) the print moves up the label about 1mm on each row of labels down the page. Can this be corrected please?
Please advise if this is not the correct place to raise this issue.
Thanks John

This is probably a problem of the printer driver. Which printer driver are you using?

Not a problem with the driver. The default configuration scaled page to fit rather than print designed size. The label document was shrunk when printed. Changed the setting to print actual size - problem cured.


Add support for zwstring_view to natvis. Fix parse error in natvis XML

Closes #265
Why is this change being made?
I recently began using wil::zwstring_view in a new project and when debugging in windbg the visualization for these string types is not very good.  They must be manually expanded to see the contents of the string.  The experience is worse than std::wstring_view.
Briefly summarize what changed
This string type is a thin wrapper over std::basic_string_view so I just copied the natvis for that type and updated the name so it knows to use it.
There was also a preexisting warning when loading the XML because the <?xml...> element was not the first bytes of the file.  I'm not sure if that was breaking things or just a warning but I fixed it regardless.
How was this change tested?
I created an empty console app and added WIL to it.  I added a wil::zwstring_view variable and debugged the project with windbg.  The edits were made to the wil.natvis that came with the WIL NuGet package.
Before this change

After this change

Note that the value is visible without expanding it.  The size is also newly visible.

are your before/after pictures reversed?

No, they are not.  The before required expanding to see the string value.  After has it top-level.  I expanded it to also show that the underlying buffer and size are now visible.

I do think the "size" is more relevant than the array of characters, though. Is there a mode to show "{size} {string-content}" or similar without getting the full dump of characters?

That's fair.  Iteration 2.


Black entities in nether with sildurs shaders

When using sildurs vibrant shaders, all entities (Mobs, dropped items and xp) are completely black, including the arm/item I am holding.
Tested with multiple different versions of sildurs shaders. High and medium, older and newer versions.
Gonna test older versions of immersive portals.

Restart the game and try again and see whether entities are black. The shader compatibility is very hacky and sometimes problems get fixed by restarting the game.

Ok I can reproduce this issue now. You can use sildurs shader 1.22 which does not have this problem.

I'm using 1.22, and restarted. Still black

Already tried using sildurs 1.22 (medium), and restarting.
Nothing works.

what's your optifine version? And show the log

Possibly fixed.

Fixing this also leads to the fix of chocapic shader compatibility and beyond belief shader compatibility

I will release the new version possibly tomorrow

fixed in 0.26


Option to not compress

Can an option be added to linuxdeploy so the AppImage won't be compressed when it's created?

I think what makes more sense is to expose appimagetool's compression setting directly, this way you can configure this completely yourself.

I'm not sure how needed this is actually. Does anyone else think it would be a good option to have?
I'm making app images for 0ad and that project distributes their data in a compressed zip file already, over 1G, and that zip file normally just stays zipped when the game is installed. When making an appimage, the included libraries take up about 180 MB.
I'm just not sure if any performance hits are significant from the appimage decompression at run time, plus 0ad decompressing data at run-time.
But I don't expect anyone here to have intimate knowledge of 0ad implementations, just kind of wondering if anyone else thinks having an option to not-compress an appimage might generally be a useful feature.

I think what makes more sense is to expose appimagetool's compression setting directly, this way you can configure this completely yourself.

That makes sense to me. (I didn't see your comment until after I finished posting mine)

It's totally fine to have that option, it's not a lot of work anyway.
Regarding decompression, one of my projects, Blue Nebula, uses the standard compression algorithm (zlib), and the performance is reasonably good.
Overall, what might be suited best for games is Zstandard, since its decompression speed is a fair bit better than zlib, even though compressing takes a lot longer.

I'm making app images for 0ad and that project distributes their data in a compressed zip file already, over 1G, and that zip file normally just stays zipped when the game is installed.

Currently we don't decompress data for 0 A.D. Its zip files are plane data (kind of tar without compression) to be able to do a fast read.

Implemented $APPIMAGE_COMP. Note that:

appimagetool does not support "no compression" at the moment, please open an issue in https://github.com/AppImage/AppImageKit or comment to an existing one to bring it to attention
linuxdeploy needs to be rebuilt to contain an up-to-date AppImage plugin, so you need to download the AppImage plugin yourself currently to use the latest version


[CORE] Upgrade the version of master to 0.7.0-SNAPSHOT.

What changes were proposed in this pull request?
XSQL have been released 0.6.0, so we should upgrade the version of master to 0.7.0-SNAPSHOT.
I think this is a good deal references Apache Spark.
The branch 0.6 will continue to fix bug and release 0.6.1, 0.6.2 and so on.

LGTM


Allow profiles to have arguments member that can be used that can be used as parameter to underlying text application

Sample profile for cmd.exe for example (Bold is the what I want)
{
"arguments" : "f:\GIT\Tools\Razzle.cmd x64fre",
"startingDirectory" : "f:\GIT",
"name" : "Razzle",
"commandline" : "cmd.exe",
"acrylicOpacity" : 0.75,
"closeOnExit" : true,
"colorScheme" : "Vintage",
"cursorColor" : "#FFFFFF",
"cursorShape" : "bar",
"fontFace" : "Consolas",
"fontSize" : 11,
"foreground" : "#00FF00",
"guid" : "{0caa0dad-35be-5f56-a8ff-afceeeaa6101}",
"historySize" : 9001,
"icon" : "ms-appx:///ProfileIcons/{0caa0dad-35be-5f56-a8ff-afceeeaa6101}.png",
"padding" : "0, 0, 0, 0",
"snapOnInput" : true,
"useAcrylic" : true
},

?
"commandline": "cmd /k f:\git\tools\razzle.cmd x64fre"

I have tried this and tried again, the arguments are completely ignored, also the startup directory doesn't change

If you're seeing settings not changing, you should 100% check to make sure you haven't duplicated your guid :wink:

An easy to way to reset the guid is to just delete it.


Add launch-navigator plugin

It lets you call the phone navigator.
This is my first plugin here and I don't know if it's ok. Please, tell me if something is wrong. :dancer:

Hi @gortok. Docs added in https://github.com/driftyco/ng-cordova/pull/942


[Python] GCS FileSystem - support federated identity

Describe the enhancement requested
Hello,
It seems that federated / external identities are not supported in the Arrow GcsFileSystem implementation? It would be great to support this. I'm using it in CI with GitHub as an IdP as per the instructions and it works great for the Google CLI tools.
I'm not sure how this is implemented, is there a standard library from Google that can just be linked / updated? The federated auth mechanism creates a regular Google application creds file, with the "type" set as "external_account". Is there an easy (ish) way to bring this in if the Google libs handle the different standard credential types? Or is that wishful thinking on my part?
I'm interested in the Python component but guess this would come to other languages that use the same underlying code if it was added.
Here is the stack trace from Arrow:
File "pyarrow/_fs.pyx", line 571, in pyarrow._fs.FileSystem.get_file_info
File "pyarrow/error.pxi", line 144, in pyarrow.lib.pyarrow_internal_check_status
File "pyarrow/error.pxi", line 100, in pyarrow.lib.check_status
pyarrow.lib.ArrowInvalid: google::cloud::Status(INVALID_ARGUMENT: Permanent error GetObjectMetadata: Could not create a OAuth2 access token to authenticate the request. The request was not sent, as such an access token is required to complete the request successfully. Learn more about Google Cloud authentication at https://cloud.google.com/docs/authentication. The underlying error message was: Unsupported credential type (external_account) when reading Application Default Credentials file from [/path/to/credentials.json].). Detail: [errno 22] Invalid argument
Component(s)
Python

CC @coryan perhaps?

Yup, I am probably the right person to ask about this.  Workload identity federation is supported starting with v2.6.0:
https://github.com/googleapis/google-cloud-cpp/releases/tag/v2.6.0
We actually use the integration with GitHub actions in our own testing:
https://github.com/googleapis/google-cloud-cpp/actions/workflows/external-account-integration.yml
I am not sure how one goes about updating pyarrow to require google-cloud-cpp >= v2.6.0, I assume it depends on how it is getting installed?
HTH

I am not sure how one goes about updating pyarrow to require google-cloud-cpp >= v2.6.0, I assume it depends on how it is getting installed?

In theory we just need to update this line to cover most users (e.g. pyarrow): https://github.com/apache/arrow/blob/f10f5cfd1376fb0e602334588b3f3624d41dee7d/cpp/thirdparty/versions.txt#L52
In practice things typically aren't that simple (e.g. breaking changes or changes in the way the gcs library is built).

I can give this a try later this month.  FWIW, the only backwards incompatible change between 1.42.0 and 2.x is the requirement for C++14.  Since arrow already requires C++17 that should be trivial.  I do not recall any changes to the build requirements, but that is always risky.
I assume there is no way for @martin-traverse to upgrade before then?


fixed nil panic on the ChangeLogMessage for the servers

Fixes #2126

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/1403/
Test FAILed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/1404/
Test FAILed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/1406/
Test PASSed.


[WIP] Integration Testing for Quantized ONNX export

Asana ticket: https://app.asana.com/0/1201735099598270/1201949554543506/f
There are some discrepancies that are probably harmless and expected. For instance, when comparing two pruned models, the sequences of last operations (before logits) differ slightly (this is probably due to how the newer PyTorch version handles the model-->onnx export pipeline).
The prediction head for the original onnx model ("before")

The prediction head for the retrieved onnx model ("after").

We do not have a system in place to catch this error in CI/CD, but this PR will not be complete unless we also merge:
https://github.com/neuralmagic/sparsezoo/pull/141

@dbogunowicz any reason not to use load_numpy_list? that should handle tar'd directories right?

Oh, only now I have realized that it calls load_numpy_from_tar.  You are absolutely right.

@dbogunowicz @bfineran Should transformers testing be added to GHA here? It seems those tests are getting skipped, as they are a new category not yet included in GHA


TLS is disabled by default

Hello,
I've noticed what seems a change in behavior as of this PR, that now requires starttls: true to be explicitly set to true to enable TLS.
I noticed my email sending using AWS SES no longer works in my drone builds with error
time="2021-11-01T07:15:45Z" level=error msg="Error while dialing SMTP server: unencrypted connection"

However explicitly setting starttls: true works, but from how I understand the PR above it should be enabled by default
Thank you

It should default to true, yes. I thought I tested this as well... I will have a look at this later today.

I failed to notice that my testing mail server just accepted whatever starttls setting I gave it... Either way, I fixed the issue by flipping the starttls flag. It's now called no_starttls which you have to explicitly set to true to disable StartTLS.


Downloading file-content when user upload a file/s in bot conversation.

I am using Bolt-JS SDK for my bot development.
I have a requirement, where user is prompted to upload a file based on which further communication will continue.
Now i want to read contents of that respective file. I need the knowledge on how to handle the file upload event or under which event shall I check in Bolt SDK in JavaScript.
Flow example goes like:
User: Hi
Bot: Hello User. Good Day
User: I want to upload a file.
Bot: Sure. Please upload the excel file.
User: Uploads file. ----> (I need to handle this event)
Bot: Thank You.
Please suggest me a code solution or reference.
I need something file_shared event or something similar. (Since I want to read the file contents)
Thanks in advance.

Hey @Sudip-byte! 👋 Downloading file contents can be a bit tricky but if your app has the right setup, should be possible with just the message event and the addition of a files:read scope using the url_private_download in a message.
This snippet can be used for retrieving contents from files shared to a channel:
app.message(async ({ context, event }) => {
    if (!event.files) {
        return;
    }
    const downloads = await Promise.all(event.files.map(async (file) => {
        const download = await fetch(file.url_private_download, {
            headers: {
                'Authorization': `Bearer ${context.botToken}`
            },
        });
        return download.text();
    }));
    console.log(downloads);
});

Including the bot token in this request is pretty important since files are restricted to a team by default. Hopefully this is helpful, but let me know if this doesn't seem to work for you!

I think this works well for text (.txt) files.
My requirement here is that a excel will be uploaded in the slack bot conversation.
Then I need to extract the contents of file either in binary or in base64 and send to a different server for uploading.

I think you can change the download.text() to possibly download.arrayBuffer() or .blob() instead? A few other typed options for the body are listed here, but ArrayBuffer should represent binary pretty well.

@zimeg Thanks a lot for your support in helping me with the solution.
I was able to achieve my requirement with the solution that you mentioned above.
In addition to your above suggestion, I did some additional exploration and was able to achieve the same using the file_shared event and axios.
Regards,
Sudip Rana


til::coalesce<T> should be able to take a function that returns T

From a teams chat
example 1
int Foo() {...}
optional<int> whatever { /*...*/ };
...

int a = til::coalese(whatever, Foo)

Foo only gets called if whatever is empty.
example 2
class Foo{
  optional<int> _cached;
  int GetThing();
  int _generate();
}

int Foo::GetThing()
{
    _cached = coalesce(_cached, [&](){ return _generate(); });
    return _cached;
}
int Foo::_generate() { /*...*/ }

Look ma, now it's easy to implement lazy properties. So easy in fact...
example 3 - til::lazy_property<T>
class Foo {
  til::lazy_property<int> Thing { { this, &Foo::_generate } }; // same syntax as winrt event handlers
  int _generate();
}

// ...

Foo foo{};
int x = foo.Thing(); // Calls _generate
int y = foo.Thing(); // Doesn't
foo.Thing.invalidate();
int z = foo.Thing(); // calls generate again

(maybe we don't do this one. Seems like it's potentially footgun-y)

This is a truly one of the ideas I had while working on Profile::EvaluatedIcon in #15843

Large parts of coalesce code will become obsolete with C++23 and the 3 monadic functions listed here: https://en.cppreference.com/w/cpp/utility/optional
If we want to do this, I think we should take inspiration from those for the design.


forward error of mxnet-cu100 with version 1.4.1

When I run the CNN model (resnet50) on GPU with mxnet-cu100, the error occurs as below:
Traceback (most recent call last):
File "test_score.py", line 84, in
test_imagenet1k(model=args.model, task_mode=args.task_mode, **kwargs)
File "test_score.py", line 39, in test_imagenet1k
score(model=m, rgb_mean=mean, rgb_std=std, scale=s, num_layer=nl, label_name=l, **kwargs)
File "/app/score.py", line 225, in score
predicts = mod.get_outputs()[0].asnumpy()
File "/usr/local/lib/python2.7/dist-packages/mxnet/ndarray/ndarray.py", line 1980, in asnumpy
ctypes.c_size_t(data.size)))
File "/usr/local/lib/python2.7/dist-packages/mxnet/base.py", line 252, in check_call
raise MXNetError(py_str(_LIB.MXGetLastError()))
mxnet.base.MXNetError: [03:20:41] src/operator/nn/./cudnn/cudnn_convolution-inl.h:159: Check failed: e == CUDNN_STATUS_SUCCESS (7 vs. 0) cuDNN: CUDNN_STATUS_MAPPING_ERROR
Stack trace returned 10 entries:
[bt] (0) /usr/local/lib/python2.7/dist-packages/mxnet/libmxnet.so(+0x4114ba) [0x7f5c93d054ba]
[bt] (1) /usr/local/lib/python2.7/dist-packages/mxnet/libmxnet.so(+0x411ad1) [0x7f5c93d05ad1]
[bt] (2) /usr/local/lib/python2.7/dist-packages/mxnet/libmxnet.so(+0x3765922) [0x7f5c97059922]
[bt] (3) /usr/local/lib/python2.7/dist-packages/mxnet/libmxnet.so(+0x3795cd9) [0x7f5c97089cd9]
[bt] (4) /usr/local/lib/python2.7/dist-packages/mxnet/libmxnet.so(+0x2ceb179) [0x7f5c965df179]
[bt] (5) /usr/local/lib/python2.7/dist-packages/mxnet/libmxnet.so(+0x2cf1ec6) [0x7f5c965e5ec6]
[bt] (6) /usr/local/lib/python2.7/dist-packages/mxnet/libmxnet.so(+0x2cd01c4) [0x7f5c965c41c4]
[bt] (7) /usr/local/lib/python2.7/dist-packages/mxnet/libmxnet.so(+0x2cd44b3) [0x7f5c965c84b3]
[bt] (8) /usr/local/lib/python2.7/dist-packages/mxnet/libmxnet.so(+0x2cd4706) [0x7f5c965c8706]
[bt] (9) /usr/local/lib/python2.7/dist-packages/mxnet/libmxnet.so(+0x2cd08f4) [0x7f5c965c48f4]
The setting of environment are:
CUDA10.0
cuDNN7.5 or cuDNN7.6
mxnet-cu100 1.4.1
GPU driver 410.93
2080ti

@mxnet-label-bot add [question]

How did you solve this? Or bypass?


table 38 "Purchase Header" - new integration event in procedure SendRecords

Similar to the event "OnPrintRecordsOnBeforeTrySendToPrinterVendor" in the procedure "PrintRecords", we would like to have an event in the procedure "SendRecords". We subscribe to this in our product knkPublishing to call the procedure DocumentSendingProfile.SendVendorRecords with a different "ReportSelections.Usage" based on the values of product relevant fields in the Purchase Header.
changes in procedure SendRecords
    procedure SendRecords()
    var
        DocumentSendingProfile: Record "Document Sending Profile";
        ReportSelections: Record "Report Selections";
        DocTxt: Text[150];
        //added handled flag
        IsHandled: Boolean;
    begin
        CheckMixedDropShipment;
        OnSendRecordsOnAfterCheckMixedDropShipment(Rec);

        GetReportSelectionsUsageFromDocumentType(ReportSelections.Usage, DocTxt);
        //added event call
        IsHandled := false;
        OnSendRecordsOnBeforeSendVendorRecords(ReportSelections.Usage.AsInteger(), Rec, DocTxt, IsHandled);
        if not IsHandled then
            DocumentSendingProfile.SendVendorRecords(
              ReportSelections.Usage.AsInteger(), Rec, DocTxt, "Buy-from Vendor No.", "No.",
              FieldNo("Buy-from Vendor No."), FieldNo("No."));
    end;

new integration event OnSendRecordsOnBeforeSendVendorRecords
    [IntegrationEvent(false, false)]
    local procedure OnSendRecordsOnBeforeSendVendorRecords(ReportUsage: Integer; var PurchaseHeader: Record "Purchase Header"; DocTxt: Text[150]; var IsHandled: Boolean)
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 47054.


What i can do?

/usr/local/lib/python3.10/dist-packages/torchvision/transforms/functional_tensor.py:5: UserWarning: The torchvision.transforms.functional_tensor module is deprecated in 0.15 and will be removed in 0.17. Please don't rely on it. You probably just need to use APIs in torchvision.transforms.functional or in torchvision.transforms.v2.functional.
warnings.warn(
Testing 0 CODE_OF_CONDUCT
Traceback (most recent call last):
File "/content/Real-ESRGAN/inference_realesrgan.py", line 166, in
main()
File "/content/Real-ESRGAN/inference_realesrgan.py", line 138, in main
if len(img.shape) == 3 and img.shape[2] == 4:
AttributeError: 'NoneType' object has no attribute 'shape'

same here, it finds the file but i get the error :
(RESRGAN) C:\Users\X\DEV\Real-ESRGAN>python inference_realesrgan.py -n RealESRGAN_x4plus_anime_6B -i inputs
Testing 0 test_video_1
Traceback (most recent call last):
File "inference_realesrgan.py", line 166, in
main()
File "inference_realesrgan.py", line 138, in main
if len(img.shape) == 3 and img.shape[2] == 4:
AttributeError: 'NoneType' object has no attribute 'shape'

Any solution @SuperVirtuel  and @Keiser04 ??

I don't remember, I kind of made a simple collab but it only has the upscae of anime style and that's where it goes.

Any solution @SuperVirtuel and @Keiser04 ??

Ouki, could you update me on the things you are using nowadays, or youhave used and found useful since that time?
Thanks

/usr/local/lib/python3.10/dist-packages/torchvision/transforms/functional_tensor.py:5: UserWarning: The torchvision.transforms.functional_tensor module is deprecated in 0.15 and will be removed in 0.17. Please don't rely on it. You probably just need to use APIs in torchvision.transforms.functional or in torchvision.transforms.v2.functional. warnings.warn( Testing 0 CODE_OF_CONDUCT Traceback (most recent call last): File "/content/Real-ESRGAN/inference_realesrgan.py", line 166, in main() File "/content/Real-ESRGAN/inference_realesrgan.py", line 138, in main if len(img.shape) == 3 and img.shape[2] == 4: AttributeError: 'NoneType' object has no attribute 'shape'

Line 137, which is reading the image using cv2.imread() is getting null value. It should be an numpy array (like these: (612, 402, 3) ), for any RGB image. Try to check whether you are passing the right path/correct file of input images in the terminal argument.

I'm getting this same error after running the command:
python inference_realesrgan.py -n RealESRGAN_x4plus_anime_6B -i inputs
for the second time, first time I ran it I got torchvision.transofmrs.functional_tensor not found, another thread told me to add an underscore and that worked.
so IDK about it being a path to images if I did not specify images, it should be whatever comes with the repo right?
full output:
`Downloading: "https://github.com/xinntao/Real-ESRGAN/releases/download/v<IP_ADDRESS>/RealESRGAN_x4plus_anime_6B.pth" to C:\Users\John\Documents\video2x\shangar21_stuff\Real-ESRGAN\weights\RealESRGAN_x4plus_anime_6B.pth
100%|███████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 17.1M/17.1M [00:01<00:00, 17.5MB/s]
Testing 0 00003
Testing 1 00017_gray
Testing 2 0014
Testing 3 0030
Testing 4 ADE_val_00000114
Testing 5 OST_009
Testing 6 children-alpha
Testing 7 tree_alpha_16bit
Input is a 16-bit image
Testing 8 video
[<EMAIL_ADDRESS>global loadsave.cpp:248 cv::findDecoder imread_('inputs\video'): can't open/read file: check file path/integrity
Traceback (most recent call last):
File "C:\Users\John\Documents\video2x\shangar21_stuff\Real-ESRGAN\inference_realesrgan.py", line 166, in
main()
File "C:\Users\John\Documents\video2x\shangar21_stuff\Real-ESRGAN\inference_realesrgan.py", line 138, in main
if len(img.shape) == 3 and img.shape[2] == 4:
AttributeError: 'NoneType' object has no attribute 'shape'`


Switch go-sqlit3 to modernc.org/sqlite

This enables static compiled binaries without cgo in order to run on armv5 devices for example.
I didn't commit the files from dep ensure because they would polute the PR.

Thanks for opening this PR! Another solution might be to use xgo: https://github.com/karalabe/xgo
xgo --targets=linux/arm64 github.com/cloudflare/utahfs/cmd/utahfs-server

Would that work for you?  It seems less risky than changing the db driver

Hey there @Bren2010 I did that but for target linux/arm so I could run on a device with armv5 32 bit and then it compiled a stub just to say that go-sqlite3 needs cgo for that to work.
And some other times it would link to a libc version that was not present on this device.
With these changes I managed to get a fully static binary that could run on a dd-wrt router

Thanks again :)

modernc.org/sqlite was removed from master ?

Yes, I believe at the time the modernc implementation didn't support macOS or Windows. Though it may now.


Group by heading beta feature failing

Describe the bug
With the beta group_by_heading set to true I see fresh content added to a Discussion. However, the content is not grouped by headings: it appends all existing Discussion comments in sequence.
Steps to reproduce the behavior

Use a /snippet or similar slash command to ensure the a new Discussion comment has predictable and exactly the same headings across multiple comments.
Configure the a workflow as follows:

on:
  discussion_comment: {} # Remove to only rollup issue comments

name: Rollup weekly snippets

jobs:
  comment_rollup:
    runs-on: ubuntu-latest
    name: Comment rollup

    steps:
      - name: Rollup comments
        uses: benbalter/comment-rollup@v2
        with:
          token: ${{ secrets.GITHUB_TOKEN }}
          group_by_heading: true

Add two comments on the same discussion using the exact same headings.

Expected behavior
See the content from both discussion posts grouped under a singular heading-per-section in the collapsed Details area in the Discussion.
Running this in the TPM LT repo.

Specific example. I created to comments to one discussion topic. The comments have the same heading values, although some comments have headings that others do not. For example, I would expect the 📣 section in the rollup to contain both the 'Adding something.' and 'completely different escalation' rather than repeating the 📣 section.

From: BitsByD
Testing
📣 What are the important updates that Eng LT should receive?
Adding something.
💡 What's top of mind for you this week?
mindful ness
🚢 Key Ships & Accomplishments
accomplishments
✨ Shoutouts
Shout out.
📊 Weekly Metrics
Shipped a lot
From: BitsByD
📣 What are the important updates that Eng LT should receive?
completely different escalation
💡 What's top of mind for you this week?
🏟 a different top of mind.
🔴 Risks and Blockers
a risk!


Instaloader is terminated with HTTP 502 error

Starting instaloader with the following arguments:
instaloader "#selfie" --only-if="not is_video" --filename-pattern={date:%H:%M:%S}_-_{profile} --count 55
but also in the most simplistic form:
instaloader "#lol"
will yield the following error message:
JSON Query to graphql/query: HTTP error code 502. [retrying; skip with ^C]
JSON Query to graphql/query: HTTP error code 502. [retrying; skip with ^C]
JSON Query to graphql/query: HTTP error code 502.

Errors occured:
JSON Query to graphql/query: HTTP error code 502.

Also, adding --no-geotags or --no-captions won´t help. A week ago, instaloader did not produce this error with any argument-variations. Nothing besides the system (OSX) was modified since then.

Might be a duplicate of #59, which has been fixed yesterday. Are you running the latest version of Instaloader, 3.2.1?

thanks a lot, updating it did the trick.


Canonicalize phi(x, redefinition(x)) to x when x and redefintion(x) have the same effects

With the NNBD unfork, code like ListIterator.current can wind up with floating pieces of flow graph that refine a type, and then discard the refinement.
B1:
  ...
  x := ... (type = E?)
  StrictCompare(x, null, B2, B3)
B2[target]:
  y := Redefinition(x) (type = E ^ *)
  goto B3
B3[join]: B1, B2
  z := phi(x, y) (type = E?)

Here we can replace use of z with uses of x, because B2 and the strict compare have no effects.
Currently the phi and the branch stick around, reducing performance in benchmarks like MegaNeverOverridden and ForInLoop.

This pattern is probably a result of 65254a45188a7813e8884573ca6974110abb80bf. We can revert that change after fixing https://github.com/dart-lang/sdk/issues/40892.


Alleviates unit-test exception when Org's email service is limited.

The following unit-test caused exception occurs when the org's email deliverability's 'Access Level' setting is not 'All Email'.
System.EmailException: SendEmail failed. First exception on row 0; first error: NO_MASS_MAIL_PERMISSION, Single email is not enabled for your organization or profile.: []
This fix uses mock, send-email class, which verifies DoWork method was called, presuming the platform's Messaging feature functions, as intended and used in the default SendEmailWork class.
Attempted to use the FFLIB Apex Mocks functionality to mock the new IEmailWork interface, but Test.createStub method generated an exception pertaining inner classes not being supported.

@stohn777 very cool fix, nice use of the UOW extensibility thats been added since this test. Also i am working with the Apex team to get inner class support in as soon as possible. Meanwhile @stephenwillcock @agarciaodeian this looks good to me!

inner class support for apexmocks will be so great!


Errors in production

I try to build the repo for production with:
npm run build
but no success when I publish the code on a live server (GitHub or my server on apache)
it works in local with npx live-server dist --port=4000 --no-browser
This the repo:
https://github.com/electroheadfx/ARPA3
Page error on github: https://electroheadfx.github.io/ARPA3/
404 error
on my website:
http://efxdesign.fr/working/ARPA3/
I have differents errors:
[Error] Did not parse stylesheet at 'http://efxdesign.fr/assets/index.44fc10bc.css' because non CSS MIME types are not allowed in strict mode.
[Error] TypeError: 'text/html' is not a valid JavaScript MIME type.
Any idea  about ?
thanks

it works when I publish in netify: https://619cffa56d486817654ecb33--goofy-shannon-42d245.netlify.app
The setup is only for netify services?

Upgrading unocss should fix it for you

I updated, weird it works in local inside a MAMP server, but on my production server I have MINE TYPE errors:
https://github.com/electroheadfx/ARPA3
do you know what setup I should done on my apache/ubuntu server ?

Sorry I am not familiar with backend setup, since your local env works, I guess your question is out-of-scope of this project and you will need to figure it out yourself


Fix: Line Comment Refactor

This MR overhauls the logic that we previously had in place for determining payloads for creating comments.
Gitlab's API is notoriously awful when it comes to posting comments, because the actual content of the POST has to change depending on whether or not you are sending a comment on a modified line, an unchanged line, or a deleted line.
This MR takes a very meticulous approach to determining what sort of change has occurred on the currently focused line. It does this by parsing the hunk headers and actual content of the diff as well, to figure out whether the line has been added, changed, or untouched. It also takes advantage of Neovim's scroll lock functionality. If you are scrolling in the reviewer panel, the location of your cursor in the changed file and in the new file will be "locked" together. This let's us send not only the line in the current buffer (if you are in the new panel) but also the line number from the old buffer as well.
This MR is meant to resolve all issues with leaving single-line comments, as outlined in issue #128
The plan is to keep this feature in a feature branch for some time and to have some users of the application make sure that it's working as intended. Once it's deemed stable enough it'll be merged into main.

There could still be issues with multi-line comments where the range is off for some reason. This MR will only fix all isues related to line codes being wrong.

@jakubbortlik Hey would you be interested in using this branch for a while to see if it resolves the issues with incorrect line codes? I'm going to use it for the next week or so but the more people using it the better. Once your toggle branch merges into main I'll bring that into this MR as well, so it should have all the features you want.
The goal for now is to fix single-line comment/note creation issues completely. Once you are reliably able to create single-line comments without any issues, we can start to address the other issues, where those line codes become out of date with the current diff, which is a separate issue.

@jakubbortlik Hey would you be interested in using this branch for a while to see if it resolves the issues with incorrect line codes? I'm going to use it for the next week or so but the more people using it the better. Once your toggle branch merges into main I'll bring that into this MR as well, so it should have all the features you want.
The goal for now is to fix single-line comment/note creation issues completely. Once you are reliably able to create single-line comments without any issues, we can start to address the other issues, where those line codes become out of date with the current diff, which is a separate issue.

I'm exited about this! Of course I'll test the branch. I've been filing some fun UI feature requests recently, but creating comments successfully is of course a priority, I just didn't know how I could help other than providing logs and descriptions of cases that don't work.
Just a clarification - this only addresses the comments created with gitlab.create_comment in normal mode not the gitlab.create_multiline_comment in virtual mode even if the selection contained just one line, right? I guess from the point of gitlab.nvim this would still be a "multiline" comment.

Yes that's right. I'm going to try and track down any issues with multi-line comments later. This also doesn't address comments being in the wrong place after an update, etc. -- just the initial POST.


No need for forwardRef with React 19

Description
With React 19 forwardRef can be replaced by a simple ref: https://react.dev/blog/2024/04/25/react-19#ref-as-a-prop
Additional Information
No response

Hi, thanks for the issue!
We still need to be backwards compatible with react 18, which we are also still on.
React 19 is still not released, and we would need to update to be able to use it, which is still a bit further down the road for us at the moment.
Due to the need to support backwards compatibility, we might need to keep using forwardRef for the forseeable future. It won't be removed in react 19, so it is fine to keep it as is for now,


MaskRCNN

It looks like constants.py missed several const variables. I believe I need to add below:
mllogger.event(key=mllog_const.MIN_IMAGE_SIZE, value=cfg.INPUT.MIN_SIZE_TRAIN)
mllogger.event(key=mllog_const.MAX_IMAGE_SIZE, value=cfg.INPUT.MAX_SIZE_TRAIN)
mllogger.event(key=mllog_const.EVAL_ACCURACY, value=uncased_score,
metadata={mllog_const.BBOX: bbox_map, mllog_const.SEGM: segm_map})

+1, please feel free to create a PR:)

Sorry for the late update. Just added one PR #25


Flexible ways of working with ApolloClient

[x] Setup queries to work with the createInsertMutation, etc
[x] Implement useQuery() inside useData() so we benefit of caching
[ ] Integration tests

Published @bluelibs/x-ui-collections-bundle to 1.2.0


IPCW with Missing Observations

Found an error in the IPCW code that caused it to raise an error for missing predicted values. I have updated the code, but need to verify it.

Fixed the problem. IPCW now calls propensity_score() rather than fitting regression models itself. Still need to fully verify against a sample SAS 9.4 dataset


Initial set of files and changes for helm management repo operator.

Helm operator changes for IBM management repo

/retest

Closing this PR and ll reopen for the tide build to start.


VOIP calls failing to connect

haha! found the culprit, if anyone interested: when using openvpn and it has the same subnet as the other client which we are calling (different network) openvpn goes crazy and can't forward the packet, because it assumes that the adress is in the same subnet as the origin
that's why my call didn't connect from riot-android
Gergő (@gergoe:matrix.bgsoft.hu)
https://matrix.to/#/!GnEEPYXUhoaHbkFBNX:matrix.org/%241499182419120lRiDX:matrix.bgsoft.hu

This is so ridiculously specific it can't be a FAQ. With emphasis on the F.


[Bug Report] a bug with "SELECT" component - need to click twice to chose a option on safari  with iOS13

Element UI version
2.12.0
OS/Browsers version
safari 13 on iOS13
Vue version
2.6.10
Reproduction Link
https://jsfiddle.net/mmx38qxw/
Steps to reproduce
(1) open the page with on safari 13 with iOS 13,
(2) click the SELECT component and try to chose an option, but it need to click twice and then it can chose the option.
What is Expected?
it should just click once to chose the option.
What is actually happening?
click the SELECT component and try to chose an option, but it need to click twice and then it can chose the option.

Your demo only has input component.

add scss code:
.el-scrollbar {
  > .el-scrollbar__bar {
    opacity: 1;
  }
}

can resolve it.

@acgotakuatdji It's worked~ but i don't know what is the cause of this problem ? Can you explain ?

@my9074  https://run.iviewui.com/v8lHkSrE
This is the test demo. Just click Content area. And check console log.
You will get the answer.

Still have this problem on ^2.13.0

add scss code:
.el-scrollbar {
  > .el-scrollbar__bar {
    opacity: 1;
  }
}

can resolve it.

Where should i place this? i tried right on a component where i was using select but it didn't work
thanks in advance

add scss code:
.el-scrollbar {
  > .el-scrollbar__bar {
    opacity: 1;
  }
}

can resolve it.

@my9074 https://run.iviewui.com/v8lHkSrE
This is the test demo. Just click Content area. And check console log.
You will get the answer.

Could you elaborate what you trying to say

Adding styles didn't do the trick. How to overcome the issue?

Will this bug be fixed this year?

/*Fixes double tap on iOS*/ .el-scrollbar__bar {     opacity: 1!important; }
add this to index.scss

I've added this line of code, but ios still needs two clicks to respond

@acgotakuatdji Is there any other solution to ios compatible EL-Select?

@acgotakuatdji i can't see there any answer and the problem is still here v2.15.6
So, can anyone explane why this bug is appear?

This may not be an element but a Safari bug.
When there is code.

1

2

3

4

test 23 baidu.com

The same problem can be reproduced in mobile Safari

so!who can tell me why happen this

I want to know reason


ci: overwrite jenkins build url in gke cluster lock

test-gke

Bug fixed, merging: https://jenkins.cilium.io/job/Cilium-PR-K8s-GKE/1118/execution/node/83/log/

Coverage decreased (-0.03%) to 37.846% when pulling 15f20841e890d0950036b3a13d7363bfa9b9e785 on pr/fix-build-url-update into dac186c2279952ebe48fca77e18a3ed51796a06e on master.


Adding an empty style throws error

Environment
Doesn't matter.
Reproduction
useHead({
  bodyAttrs: {
    style: ''
  }
});

Describe the bug
Same as the nuxt/nuxt#15488, if you provide an empty string for style property, it throws an error value.split is not a function.
We need to change
for (const c of value.split(';').filter(Boolean))
to
for (const c of (value || '').split(' ').filter(Boolean)) {
Same as class attribute.
Additional context
No response
Logs
No response

Thanks, this should be fixed in 1.8.12

@harlan-zw
Unfortunately, unlike the case for class attribute, This didn't fix the error for style.
I don't know why but empty style is converted to boolean true somewhere, {class: '', style: true}.
So the code for (const c of (value || '').split(' ').filter(Boolean)) { actually does nothing.

@harlan-zw I did a bit of debugging, it seems something is wrong inside normalizeTag it converts empty string to boolean true.

Sorry about this, I only had a short amount of time and didn't test the fix properly.
I've now pushed a proper fix and also added support for handling styles similar to classes with minimal runtime overhead.
Meaning, you can now do things like this:
const isNavActive = ref(false)
useHead({
  bodyAttrs: {
    style: {
      'color': 'white',
      // fully reactive styles
      'background-color': () => isNavActive.value ? 'red' : '',
    },
  },
})


Fix #624: --data=libs should output in compiler-, not linker-format.

I'm going to leave --data=linker-libs and --data=linker-lflags I suggested in #624 up to a separate PR later on.

Can you re-target this against the 0.9.24 branch? There are already some changes on master that shouldn't go into this release.
Hm, --data=linker-lflags sounds a bit strange and is inconsistent with the "lflags" field in the package recipe, we should probably turn this around and let  --data=compiler-lflags output in compiler format (or whatever the name).


webhook not working on heroku

last_error_message":"Wrong response from the webhook: 503 Service Unavailable"

Could you provide more info?

how can fix this error from telegram?
....
"has_custom_certificate":false,"pending_update_count":1,"last_error_date":1504179338,"last_error_message":"Wrong response from the webhook: 503 Service Unavailable","max_connections":40}}


[documentation] Remove duplicate section in Messaging > Consumers > Acknowledgement

Motivation
In Messaging > Consumers > Acknowledgement section in the documentation, it is described the number of ways the messages get acknowledged. This description was duplicated; once in a paragraph form and then in a point form. One should be removed to keep the documentation flow.
Modifications
The description in the paragraph form is removed since the point form is more clearer.

@yehancha good catch!
Could you please help update all affected doc versions? Thanks

@yehancha good catch!
Could you please help update all affected doc versions? Thanks

@Anonymitaet Thanks! Will do.

@yehancha good catch!
Could you please help update all affected doc versions? Thanks

@Anonymitaet Thanks! Will do.

@Anonymitaet Fixed in all versions.

@yehancha thanks! You can ping me once all tests pass, then I can merge this PR.

@Anonymitaet Noted with thanks! I think everything has passed.

@Anonymitaet - Are we waiting for @Huanli-Meng's review?


Enable Gradle Worker API by default

TL;DR: I think that we should enable Gradle Worker API by default on detekt.
#4128 introduced the option to run detekt by using the Worker API. This allows gradle to execute detektMain and detektTest in parallel and also execute detektMain in parallel with compileTestKotlin. This helps gradle to run more tasks in parallel.
For example, in a normal build as soon as compileKotlin finishes it can start detektMain and compileTestKotlin at the same time. Also, as soon as compileTestKotlin ends the task detektTest can start. Before these 4 task should run one after the other.
At the beginning of #4128 I did some benchmarks and we didn't saw any improvement but I executed other benchmarks today and the picture is really different (also I'm benchmarking slightly different scenarios, but I think that these ones are "more "real").
New benchmarks:
I created 3 different scenarios:

Execute detektMain and detektTest on ~250 modules (this is a real project)
Execute detektMain and detektTest on a sinle module (same project)
Execute detektMain on a sinle module (same project)

And those scenarios with and without Gradle Worker API enabled.
Also, between each run I removed the folder inside build/report/detekt to ensure that detekt is executed again and also with --no-build-cache.
I tested this on detekt 1.23.4.
Results:

detektMain detektTest
:module:detektMain :module:detektTest
:module:detektMain

without workers (mean)
39684,6
1507,7
1404,0

with workers (mean)
29129,8
1573,1
1669,2

difference
-27%
~+4%~ (no significant)
+19%

So, we end up that if you executes detekt several times the parallelization wins and makes huge improvements (27%). If you only executes detekt in one single module main and test it runs more less the same. And if you executes detekt only once it's slower by (19%).
Even we have one case that improves and other that don't I would say that we should also see the total numbers. In one we gain 10 seconds and in the other we lose 265ms. And, in general, I think that no one will only run detekt on main and not on test.
For that reasons I think that we should enable it by default. As an extra this would close #2957. And that would improve the workflow for us and the developers of third party rules.

Thanks for revisiting, I confess that after implementing it I never used it due to the performance concerns you'd raised, the performance issue I mentioned in the original PR, and the other issues with workers not being invalidated when classpath contents change. Not sure where those are at now?
Gradle has made performance improvements in the worker implementation so this might be the result.
Agree that enabling for 2.0 (at least the RC) should help us understand if beneficial for users, even if detekt project itself may not benefit much from it.

Ok, I'm going to repeat the same benchmark that I did in that PR and share the results here. Depending in what we see there we can take an informed decision.

detekt parallel
detekt no-parallel
detekt-plain parallel
detekt-plain no-parallel

without workers (mean)
156642.86
165339.37
88921.55
89305.88

with workers (mean)
163460.65
164514.45
98738.76
98074.95

difference
+4.2%
~-0.5%~ (no significant)
+9.9%
+8.9%

So, in the detekt project it is still slower to run it with the worker enabled but not on my project. Not as bad as before, but still bad.
Other thing to take into account is that here I'm just running the detekt tasks. I'm not getting the full benefit of the parallelization. In real scenarios the user would also compile the code and maybe even run tests in parallel.
I think that, anyway, we should enabled it. But I'm probably biased so I would like to know what someone else think to take the final decision. @detekt/maintainers.

@BraisGabin go for it 🙂

This issue is stale because it has been open 90 days with no activity. Please comment or this will be closed in 7 days.

This issue is stale because it has been open 90 days with no activity. Please comment or this will be closed in 7 days.


How to navigate between different pages?

When clicking between different items in the bottom bar, how can I navigate between new pages without setting up a default index.

any luck?


NotifyNeighborBlockEvent not being fired since Redstone Tracking merged into master.

The title basically says it all, nothing worked in the notify event, so I put a System.out.println in there to test if anything actually happened. At the top of the method nothing is outputted at all.

Thanks blood


[bug]: unable to kill app during model initial registration

Is there an existing issue for this problem?

[X] I have searched the existing issues

Operating system
Linux
GPU vendor
Nvidia (CUDA)
GPU model
No response
GPU VRAM
No response
Version number
a386544a1d78bdbc9d5a3f8bad6462ae379a1c68
Browser
FF
Python dependencies
No response
What happened
During the initial app startup, we do model registration, including hashing. I'm unable to interrupt this process with Ctrl - C - I have to forcibly end the process (or close the terminal window).
What you expected to happen
I should be able to interrupt the startup without killing the parent process.
How to reproduce the problem

Have a few models on disk
Ensure hashing is enabled
Enable memory db (or use a fresh db) and start the app
Press Ctrl - C once it starts registering models.

Additional context
No response
Discord username
No response

This is a bit beyond me, @lstein . I think we need to signal to the model manager that SIGINT has occurred on the main thread. Maybe we could do this with the events service? Or add a stop event to ApiDependencies or Invoker?


[FEATURE REQ] Support AzurePipelinesCredential via AzureComponentFactory

Library name
Microsoft.Extensions.Azure
Please describe the feature.
Add support for configuring and instantiating an AzurePipelinesCredential via AzureComponentFactory.

@jviau:  Can you please share the customer scenario where this would be used?

//cc: @scottaddie, @christothes for thoughts.

@jsquire the precise scenario I am thinking of is for integration tests of Azure Functions extensions in Azure Pipelines. Today many extensions leverage AzureComponentFactory to parse configuration and get a token for the Azure resource (example: event hubs extension). Adding azure pipelines support to component factory will simplify integration tests in Azure pipelines as a service connection can be used. This would be a preferred approach over existing auth as they have limitations:
Connection string: not secretless.
Managed identity: need to assign identity to agent. All pipelines running on agent now have access to identity. Service connection will allow restricting to specific pipelines.

We don't generally add public features for internal purposes such as this.   Can you help us understand why using the Functions Startup and the UseCredential method for the client builder wouldn't work for the Functions test infrastructure?  That would be the recommended approach and works with any Identity credential or custom implementation based on TokenCredential.
Example

This is not specifically for internal purposes only. It is for customers that have integration tests for their function apps as well. Azure functions extensions do not support manually supplying token credentials for the most part and are limited to connection strings or what AzureComponentFactory supports. Additionally, it out-of-process scenarios customers would not have any code running in the host process to manually provide a token credential to begin with, and are limited to only config-based solutions.


Update autofill to 6.6.0

Task/Issue URL: https://app.asana.com/0/1204572548387759/1204572548387759
Autofill Release: https://github.com/duckduckgo/duckduckgo-autofill/releases/tag/6.6.0
Description
Updates Autofill to version 6.6.0.
Autofill 6.6.0 release notes
What's Changed

Ema/fix release again by @GioSensation in https://github.com/duckduckgo/duckduckgo-autofill/pull/311
Update Dax logo by @alistairjcbrown in https://github.com/duckduckgo/duckduckgo-autofill/pull/312
New subdomain matching rules by @GioSensation in https://github.com/duckduckgo/duckduckgo-autofill/pull/297
Update release workflow by @alistairjcbrown in https://github.com/duckduckgo/duckduckgo-autofill/pull/313

Full Changelog: https://github.com/duckduckgo/duckduckgo-autofill/compare/6.5.1...6.6.0
Steps to test
This release has been tested during autofill development. For smoke test steps see this task.

Closing off in favour of next released version


[3.x] SCons: Add explicit dependencies on thirdparty code in cloned env

Since we clone the environments to build thirdparty code, we don't get an
explicit dependency on the build objects produced by that environment.
So when we update thirdparty code, Godot code using it is not necessarily
rebuilt (I think it is for changed headers, but not for changed .c/.cpp files),
which can lead to an invalid compilation output (linking old Godot .o files
with a newer, potentially ABI breaking version of thirdparty code).
This was only seen as really problematic with bullet updates (leading to
crashes when rebuilding Godot after a bullet update without cleaning .o files),
but it's safer to fix it everywhere, even if it's a LOT of hacky boilerplate.
(cherry picked from commit c7b53c03ae7f7feb45a6023ee5cf764025ebb5e1)

Cherry-picked for 3.3.1.


100% height

Is it somehow possible to set 100% of the height explicitly? I.e. it allign main-footer to bottom of the page, while content will have most of the area?

I'm not sure I understand the question, but the theme works this way out of the box - but requires app.js, for some reason, to make the footer hug the bottom.

Closing due to inactivity. If this is still relevant, please let us know.
Thanks.


add row index to grid

Hi expert
I do not know why my previous issue has been removed.
I was asked How do I add row index (row sequence) to the grid.
thanks.

 Search it in closed issues.


Window size way too large

For some reason the default window size stretches all the way over to my second monitor.
Trying to pass window settings like size or max_size don't seem to have any effect.
I cannot remember it doing this in earlier versions, so it might just be a recent regression.
Shown here with the todo example application on the master branch (589f68df).

System:
OS: Arch Linux 64bit
CPU: AMD FX-8350
GPU: AMD Radeon RX480

One thing I noticed is that forcing 96 DPI in KDE settings seems to produce a reasonably scaled window.
So most likely something changed in the way X11 reports scaling that broke this.

Could you try if the winit examples have the same issue? In that case, we should report the issue there!
I'm closing this since it seems very likely to be an issue in winit, or maybe related to your specific environment. Feel free to reopen if you find any evidence that iced may be at fault here!


Exit code is always 0, should return 1 when there are errors

After running on command line, echo $? always returns 0 even if there are errors. This prevents apigeelint from being used in CI/CD to detect errors.

I've committed a fix for non-zero return on test error - 6a87a5b06bebc6a7762a7ad40a702574c6e74e7d
This contains @tom-ashe fix   #90

Please confirm this fix Kurt. Looked good to me, would like your review on the merge.


[Feature request] relativeTimeThreshold for weeks

Description of the Issue and Steps to Reproduce:
Feature request : Add a relativeTimeThreshold for weeks.
Please include the values of all variables used.
    moment.relativeTimeThreshold('d', 21);
    moment.relativeTimeThreshold('w', 16); // Doesn't exist yet
    console.log(moment(someDate).toNow(true));

What is the use-case or motivation for changing an existing behavior?
I want to display age of animals :octocat:
My problem is for puppy, a veterinary count in days (1 to 21days) AND weeks (4 to 16weeks) because the vaccines (and some other medicines) use this timeline.

I'd like to try and tackle this feature: it'd be my first contribution to the project, but I think I have a decent idea of how this would work.  Is there anyone who can lead me in the right direction concerning this?  Thanks :)

(I'm working with @lad475) So, right now there's a threshold "d" for days to month. In order to implement this feature, would we need to change the "d" threshold to be days to week and add a "w" threshold for weeks to month? It doesn't seem right to get rid of days to months (not everyone wants to use weeks), but I'm not sure how else to go about it - maybe have two thresholds for days to week and days to month? Any ideas / help would be appreciated!

See 2.25.3


sigmaVector for smoothing in preprocessfMRI bad format (string should be vector)

Hello,
in the preprocessfMRI function, when smoothing is selected to be "gaussian" the size of the smoothing kernel (e.g., 5) results in the follwoing string being sent to the smoothImage function "5x5x5x0" which then returns an exception:
STD ExceptionObject caught!
Error in .smoothImageHelper(inimg, sigma, sigmaInPhysicalCoordinates,  :
not compatible with requested type
the input to smoothImage should be c(5,5,5,0)
line 82 in preprocess fMRI should be:
sigmaVector <- c(rep(spatialSmoothingParameters[1],3),0)

Yeah, that was fixed recently.  Also, you should migrate to preprocessRestingBOLD as we'll probably deprecate preprocessfMRI soon.

thanks.
On Fri, May 12, 2017 at 9:21 AM, Nick Tustison<EMAIL_ADDRESS>wrote:

Yeah, that was fixed
https://github.com/stnava/ANTsR/commit/0b971d8c862ddff3c6999ddb9339918af8eba4c9#diff-8de6ca795b058492b666e862741189f5
recently. Also, you should migrate to preprocessRestingBOLD as we'll
probably deprecate preprocessfMRI soon.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/stnava/ANTsR/issues/152#issuecomment-301074498, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ATDN3fAO6FcjvLVe7q75f3DS1CBgNs1Dks5r5Fy_gaJpZM4NZO2G
.


I can't find the tensorflow-lite model at the specified location(new->other->tensorflow-lite model) What should i do?

Thank you for submitting a TensorFlow documentation issue. Per our GitHub
policy, we only address code/doc bugs, performance issues, feature requests, and
build/installation issues on GitHub.
The TensorFlow docs are open source! To get involved, read the documentation
contributor guide: https://www.tensorflow.org/community/contribute/docs
URL(s) with the issue:
Please provide a link to the documentation entry, for example:
https://www.tensorflow.org/versions/r2.0/api_docs/python/tf/MyMethod
Description of issue (what needs changing):
Clear description
For example, why should someone use this method? How is it useful?
Correct links
Is the link to the source code correct?
Parameters defined
Are all parameters defined and formatted correctly?
Returns defined
Are return values defined?
Raises listed and defined
Are the errors defined? For example,
https://www.tensorflow.org/versions/r2.0/api_docs/python/tf/feature_column/categorical_column_with_vocabulary_file#raises
Usage example
Is there a usage example?
See the API guide: https://www.tensorflow.org/community/contribute/docs_ref
on how to write testable usage examples.
Request visuals, if applicable
Are there currently visuals? If not, will it clarify the content?
Submit a pull request?
Are you planning to also submit a pull request to fix the issue? See the docs
contributor guide: https://www.tensorflow.org/community/contribute/docs,
docs API guide: https://www.tensorflow.org/community/contribute/docs_ref and the
docs style guide: https://www.tensorflow.org/community/contribute/docs_style

@DhilipKumarDK,
In order to reproduce the issue reported here, could you please provide the TensorFlow version, the complete code and the dataset you are using. Thanks!

@DhilipKumarDK,
In order to reproduce the issue reported here, could you please provide the TensorFlow version, the complete code and the dataset you are using. Thanks!

I was using android studio 4.0.1 ver which didnt have the tfmodel and Thank
you for your concern
On Mon, 11 Jan 2021, 2:14 pm Abhilash Mahendrakar<EMAIL_ADDRESS>wrote:

@DhilipKumarDK https://github.com/DhilipKumarDK,
In order to reproduce the issue reported here, could you please provide
the TensorFlow version, the complete code and the dataset you are using.
Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/46307#issuecomment-757732240,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASB5TIB6KTGTCFZA3QHUI2LSZK25XANCNFSM4V3L3TLQ
.

I was using android studio 4.0.1 ver which didnt have the tfmodel and Thank
you for your concern
On Mon, 11 Jan 2021, 2:14 pm Abhilash Mahendrakar<EMAIL_ADDRESS>wrote:

@DhilipKumarDK https://github.com/DhilipKumarDK,
In order to reproduce the issue reported here, could you please provide
the TensorFlow version, the complete code and the dataset you are using.
Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/46307#issuecomment-757732240,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASB5TIB6KTGTCFZA3QHUI2LSZK25XANCNFSM4V3L3TLQ
.

TFLite ML Model Binding plugin for Android Studio is only available from Android Studio 4.1.
Please see here for more details.

TFLite ML Model Binding plugin for Android Studio is only available from Android Studio 4.1.
Please see here for more details.


Fixes the issue I just introduced.

There's an ordering problem.
make ci calls for validation. Validation requires go-bindata but go-bindata isn't installed until the build command.
This moves the prereqs into an explicit build target and then make sures dependent targets require it.

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

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

TTYL, MSBOT;

(This regressed because I took it out of the Dockerfile so that there was only a single source of truth for how to acquire this dependency).
Normally, I'd just install go-bindata by go get . but that doesn't work if the project doesn't build locally, etc, etc.


Allow addition of new skills

Admins should be able to add a new skill to the set of skills instructors are able to teach.  There should be an easy way for the admin adding the skill to indicate which of the people already in the database are qualified to teach it.

Note: we need bulk addition from a CSV (can be a command-line tool).

What's the status on this issue? Do we postpone it to v0.4?

We postponed it, but it should be early in v0.4.
Thanks,
Greg

@gvwilson what's the status on this?

Let's move this to 0.5, since it ties into getting instructors' responses to the skills form into the database.


Flower in recipe

Why are there flowers in this recipe? It made my cake taste bad. And why isn't there any flour?

Hello, happy new year
Thanks for filling this issue, and sorry for the typo 🤣
fix #2
Have a good day.
Cheers


Package Scope
@suspensive/react-query-4
Bug description
I found a type error that occurs when passing the return value of queryOptions as a property to SuspenseInfiniteQuery.
Check it out here: Codesandbox Link
Expected behavior
The return value of queryOptions should be compatible with the props of SuspenseInfiniteQuery without causing any TypeScript errors.
To Reproduce
No response
Possible Solution
If queryOptions cannot adequately handle the infiniteQuery type, we might consider creating a separate infiniteQueryOptions as an alternative approach.
etc.
No response

https://github.com/toss/suspensive/pull/1016#issuecomment-2198900615

It looks like the error is occurring due to type mismatches in refetchOnMount, refetchOnReconnect, and refetchOnWindowFocus. The queryOptions are typed as data: TData, while in the Infinity Hook, data: InfiniteData<TData> should be used.
I agree with both of you that creating infiniteQueryOptions to address this is the perfect solution! If this task takes a long time, wrapping it as a temporary measure and gradually updating it could also be a good approach.

@manudeli I think there might be users currently using this component. How about temporarily wrapping it with the code I suggested and proceeding with infiniteQueryOptions in the next version?
like this: https://github.com/toss/suspensive/issues/1015#issuecomment-2199263821

@manudeli I think there might be users currently using this component. How about temporarily wrapping it with the code I suggested and proceeding with infiniteQueryOptions in the next version?
like this: #1015 (comment)

Cool! Could you make it as Pull Request!? Actually I need this infiniteQueryOptions at now too for my other work


Trigger becameInactive event when active spy destroyed.

When a spied-upon element is destroyed, and that spy is currently active, trigger the duScrollspy:becameInactive event.  This doesn't matter if the client code is just using the class for styling, but if it's actually making use of the becameActive/becameInactive events then the client will end up in the wrong state.  This also maintains symmetry with addSpy, which forces the handler to run (and possibly trigger becameActive) as soon as the spy is added.

Looks great, thanks!


Add IBU doc.

Co-authored by: Jun Chen<EMAIL_ADDRESS>

/hold

/test ci-job

/unhold

/approve

/lgtm

/override ci/prow/integration


How do I stop both front- and backend?

When I do sbt run, everything runs fine. If I stop the application with Ctrl + C (Windows), the React part keeps running.
Is it possible to shut down both together?

Same issue here. Any news?

@eluchsinger I fixed this issue in #20


Cannot retrieve the type of the current position in OneWayMode

futures_position_information
If the user is in One-Way-Mode cannot know if the current position is LONG or SHORT.
In Hedge-Mode its displayed.
Is any way to know it in OWM?

Here is an explanation but I wasn't able to apply it yet to python-binance. Did you solve it?

Yes, but I used another library https://github.com/m4st3rb0ts/Binance_Futures_python
Simply call client.get_position_v2()

I ended up deducing the position side like this
info = client.futures_position_information(symbol=symbol)

# check if position is long or short
entryPrice = float(info[0]['entryPrice'])
markPrice = float(info[0]['markPrice'])
unRealizedProfit = float(info[0]['unRealizedProfit'])
quantity = abs(float(info[0]['positionAmt']))

positionSide = ''
if(entryPrice < markPrice):
    if(unRealizedProfit > 0):
        positionSide = 'long'
    if(unRealizedProfit < 0):
        positionSide = 'short'

if(entryPrice > markPrice):
    if(unRealizedProfit < 0):
        positionSide = 'long'
    if(unRealizedProfit > 0):
        positionSide = 'short'

print(f'positionSide: {positionSide}\n')

# cancel open orders
cancelled_orders = client.futures_cancel_all_open_orders(symbol=symbol)
print(f'cancelled_orders: {cancelled_orders}\n')

side = 'BUY' if positionSide == 'long' else 'SELL'

# market close
client.futures_create_order(
            symbol=symbol,
            side=side,
            type='MARKET',
            quantity=quantity,
            isIsolated='TRUE')


Secondary Subs Not Loading

Just started having this issue today. Seconary subs section is blank.
NflxMultiSubs> Fatal:  TypeError: Cannot read properties of undefined (reading 'downloadUrls')
Can you look into this? Thanks

same here, the console showed this:
btw im using version 2.2.8 on win 10 with chrome version 98.0.4758.102 (Official Build) (arm64)
nflxmultisubs.min.js:2 NflxMultiSubs> Fatal:  TypeError: Cannot read properties of undefined (reading 'downloadUrls')
at Function._buildImageBased (chrome-extension://jepfhfjlkgobooomdgpcjikalfpcldmm/nflxmultisubs.min.js:2:102918)
at Function.build (chrome-extension://jepfhfjlkgobooomdgpcjikalfpcldmm/nflxmultisubs.min.js:2:102537)
at chrome-extension://jepfhfjlkgobooomdgpcjikalfpcldmm/nflxmultisubs.min.js:2:113419
at Array.map ()
at chrome-extension://jepfhfjlkgobooomdgpcjikalfpcldmm/nflxmultisubs.min.js:2:113409
at chrome-extension://jepfhfjlkgobooomdgpcjikalfpcldmm/nflxmultisubs.min.js:2:113441

Same here 🥲

Thanks for the reports everyone! Fixed and pushed v2.2.9 to the store. I'll leave this issue open until it is approved (also you can let me know if the fix works for you when it's up).

@WingCH Hi, can you follow up this on safari extension?

Thanks for your reminder
I have uploaded the new version to apple, waiting for review

v2.2.9 is published on the Chrome and Firefox stores. 🥰 Does it work for everyone now?

mac version already pass review and deployed in mac app store!

I checked both safari, chrome extension and they works well :)


Packet Decoding : ftbteams

The Stack Trace of the decoding error can be found Here!
On connection to server, PROXY server resports the above stacktrave for Decoding error for ftbteams:team.
No lightfall version decleration on jar, but it's your latest 1.18.2 support download with lightfall client 1.0.0

You need to enable bungeecord mode in spigot.yml in Arclight

You need to enable bungeecord mode in spigot.yml in Arclight

I  did; I'll provide more information as I'm sure it's coming down to a backend server issue.
Proxy config.yml Here!
BackEnd spigot.yml Here!
Addons to the proxy/ aka plugins: None
Backend server modifications: A Mohist Forge 1.16.5 SpongeForge does not work with Lightfall and Mohist has what looks like custom patches to work with lightfall.
No crash reports, just logs which are a bit long, the error I reported is directly from the console and matches directly with the log and debug log.

You are using Mohist which is out of my touch.


Not working when Vue.js is started after Tom Select

To Reproduce
I tried to create a fiddle, but couldn't even get Tom Select to work :-( https://jsfiddle.net/f4jsau3c/
So here's a copy-pastable code:
<!DOCTYPE html>
<html>
    <head>
        <meta charset="utf-8">
        <title>Tom Select</title>
        <link<EMAIL_ADDRESS>rel="stylesheet">
        <script<EMAIL_ADDRESS>        <script<EMAIL_ADDRESS>    </head>
    <body>
        <main>
            <select multiple="multiple">
                <option>One</option>
                <option>Two</option>
            </select>
        </main>
    <script>
        new TomSelect('select', {});
        let vue = new Vue({
            el: 'main'
        });
    </script>
    </body>
</html>

Here, Tom Select isn't working at all (Firefox + Chrome).
Solution
Move the new TomSelect(...) line below Vue.js.
If this can't be fixed (or is the expected behavior ;-) it should be documented somewhere at https://tom-select.js.org/

I'm not an experienced Vue user at all but, correct me if I'm wrong, it seems like this kind of conflict is not uncommon with Vue.js

See https://github.com/orchidjs/tom-select/discussions/260#discussioncomment-1666971