RFC: A simpler implementation

This is a rewrite of Reactive to make it more robust
How it works

A Node contains the current value and a vector of Actions

type Node{T}
    value::T
    actions::Vector{Action}
    alive::Bool
end

immutable Action
    recipient::WeakRef
    f::Function
end
isrequired(a::Action) = a.recipient.value != nothing && a.recipient.value.alive

add_action!(f, node, recipient_node) creates an Action with weakref to recipient_node and function f. send_value!(node, val, timestep) sets val as node's value and calls all the actions with the recipient_node and timestep as the two arguments to f. timestep is a counter that increases with every new message.
connect_* functions (e.g. connect_map) take both input and output nodes and add appropriate actions to them (spot the difference with this example in sicp)
exported functions like map, foldp, etc create a new output node and connect them to input nodes using the respective connect_* function
_messages is a global channel. push! writes (node, new_value, metadata) to the channel.
Reactive.run(n) reads n messages off this channel and propagates the changes.
It throws a ReactiveException  with data to debug the crash. a ReactiveException prints a comprehensive crash message, for example:

ERROR: Failed to push!
    1
to node
    Node{Int64}(1, nactions=1)
at timestep 1

Debug info:
    ["meta"=>"kwarg","passed"=>"to push!","comes back"=>"here"]

error message goes here blah blah
 in buggy_function at /home/shashi/.julia/v0.3/Reactive/test/example.jl:10
 in anonymous at /home/shashi/.julia/v0.3/Reactive/src/operators.jl:39
 in send_value! at /home/shashi/.julia/v0.3/Reactive/src/core.jl:66
 in run at /home/shashi/.julia/v0.3/Reactive/src/core.jl:121
 in run at /home/shashi/.julia/v0.3/Reactive/src/core.jl:111
 in include at ./boot.jl:245
 in include_from_node1 at loading.jl:128
 in process_options at ./client.jl:285
 in _start at ./client.jl:354
 in run at /home/shashi/.julia/v0.3/Reactive/src/core.jl:125
 in run at /home/shashi/.julia/v0.3/Reactive/src/core.jl:111
 in include at ./boot.jl:245
 in include_from_node1 at loading.jl:128
 in process_options at ./client.jl:285
 in _start at ./client.jl:354
while loading /home/shashi/.julia/v0.3/Reactive/test/example.jl, in expression starting on line 20

no @lift macro - it is too complex as it stands

API changes
This is a breaking change

Signal and Input are both typealiases to Node.Input{Type}(val) is now Input(Type, val)
lift is now map
foldl is now foldp - for fold-over-past
merge breaks the guarantee of keeping the update from the earliest
argument if more than one arguments update in the same timestep - the
oldest node gets priority (the propagation is depth-first)
the main process needs to run Reactive.run() and watch for exceptions. A ReactiveException is raised which captures exceptions, backtraces and more info. exceptions don't get lost into the ether
push! takes optional meta argument which shows up in the ReactiveException object if an  exception occurs
close(node) makes the node refuse further updates
added delay, debounce and boundcopy

You can call push! inside a consume this simply queues up the update for the next timestep.
This fixes #54 #62 #61 #24
TODO

[x]  #35 - windowing/throttle

[x] right now the automatic gc'ing of nodes does not work: make it work with at least with some explicit api (relevant code is here)

[ ] Move old docs and change them

cc @SimonDanisch @Keno @stevengj @aviks @amitmurthy

I'm pretty new to Reactive and haven't yet used it in a serious project, but a big :+1: for this. I'm especially excited to see the removal of all of those heap operations and like the lift->map rename. I'm planning on becoming an adopter.
While you're renaming things, I'll confess that the push!, value pair seems a little strange to me: I guess I kind of expect that push! implies a growing container, but a Node just stores a single value. What about set! and get? Or put! and get/take? But the status quo is fine if it makes more sense to you. Whatever words you choose here, the same ones should apply to widgets in Interactive.jl.

@timholy great to hear! :)
I agree push! was ill-conceived. It seems put! or set! would indeed be better names for this. Both good candidates.

Would be good to also have the names reflect the "imperative"/"declarative" distinction clearly. set!/put! seems like it should take effect immediately to me, kind of like the implementation of send_value! (which to me implies something more like a remote-ref operation...). Now that I look at it, push! is not such a bad name for what it does now; however, one slightly confusing thing is that you're effectively pushing onto _messages rather than n---but it's complicated because it will eventually set! n. set!_queued(n, x) or queue_set!(n, x)??
Wow, naming is hard.

Or maybe @queue set!(n, x). I like that better than my other two suggestions.

why a @queue macro when the job can be done by a function..? I think queue_set! is equally good, and better than send_value! :+1:

I was thinking that @queue set!(n, x) would just rewrite the code as queue_set!(n, x), but somehow I thought the macro syntax (a little remiscent of @async) made it clearer that you are queuing a set! operation.

(Almost like queue(set!(n, x)), which of course wouldn't work.)

Ah, I see the similarity with @async, but I think queue_set! will do here. It's not part of the public API anyway. One of the changes here (I just added to the list) is foldl being renamed to foldp - I'm more concerned how that will go...

To clarify, I'm essentially proposing that
send_value! -> set! # you'll have to put this inside the let block so timestep can be filled in
push! -> queue_set!

I suspect this is inconsistent with queue_set! not being part of the public API. I'd argue you want an imperative form at your fingertips, and not rely on polling for everything.

Oh I confused what you meant. I thought you were suggesting send_value! be renamed to queue_set!, well it now makes more sense, and so does @queue set! - because queue_set! can be confused for setting something on a queue... I think enqueue! and push! are very similar, so push! would work here as well...

I'd be happy with push!. As I stated above, really you're pushing onto _messages rather than the n, but conceptually it's basically the same thing. One other issue, though, with the choice of push!: currently push!(::Input, value) is currently imperative, and you're proposing to make it be asynchronous. I don't know how much it matters, but that does seem like a tricky deprecation path, especially for people who are using this in event driven code (i.e., GUI toolkit handles the event loop and issues callbacks).
That said, we know which public packages use Reactive, so it's possible to ping their authors about this.

Yes, this change for example affects this Escher example by @aviks. It intermittently push!-es in a tight for loop. I think to alleviate this in most cases (where only one such simulation is running), you can run yieldto(Reactive.runner_process) to wake it up to read the message. It may be worth adding an optional boolean argument to push! for this.

Well, in that case I'd just call set!---that's the new imperative function.
What I'm advocating is a good deprecation strategy. Now push! is imperative, and we're discussing changing the imperative to set! and making push! be asynchronous---which means that old code will not immediately throw an error, but may behave differently in ways that are hard to debug. A better approach might be to emit a deprecation warning for push! (encouraging users to switch to set!) and choose a different name for the asynchronous function (like enqueue!(n,x) or @queue set!(n,x).

I agree that we should have deprecation. I've added warnings for renamed / deleted functions. I'm not sure how to deprecate push! if the name and signature is not going to change... One more case I'm not sure about is Input{T}(val) -> Input(T, val) (I think the latter is more Julian, so decided to make the change)
about calling the thing set! - I don't think that's what it precisely does. You would be simply yielding to the runner task, but that doesn't guarantee that the next message it reads is the one put via the current push! call. (e.g. it could be a message from an fps signal, or if the runner task is already blocked elsewhere, there could be other messages)

If that's how it works, I agree. I may have been slow to realize that you don't intend to have any events happen imperatively, that everything goes through the queue. So you can probably scratch a lot of what I said above (sorry).

Out of curiosity, what's your current thinking here? I've been planning to rebase GtkUtilities on Reactive, but had originally thought to wait until this was merged. If you have reservations about this PR, then perhaps I'll either proceed with the current Reactive or find another approach.

@timholy Very sorry for the delay here. I'd like to get this merged. Right now the thing blocking this is GLVisualize examples which freeze with this branch but work if gc_enable(false) is run at the beginning. This is because of some node getting gc'd when it goes out of scope - a new feature in this branch. I haven't been able to track where this is happening yet, but will take a closer look this weekend I'm looking forward to fixing this. I've migrated Escher to this branch and things work well.
cc @SimonDanisch

No apologies needed, I just wondered if you were reconsidering. I have projects of my own that have lingered; sometimes it's just getting around to finishing, sometimes there's a blocker, but sometimes I become less sure if it's a good approach.

+1 to merging to master and fixing the issues with GLVisualize before tagging. Chances are we might find the bug as other packages update to the new Reactive.

Sounds like a reasonable strategie! I'd be surprised if the bug is so bad, that it makes the whole of reactive NEXT unusable ;)

@timholy I still believe this is a strict improvement. :)
@sjkelly Okay I will rebase and merge soon
I'm trying to track down the GLVisualize issue by saving the backtrace when a Node is constructed. see this commit's  (next-debug branch). This gist runs a simple GLVisualize script and prints out the backtraces for cases where the node got gc'd...
but, the backtraces seem to be completely unrelated.... http://codepad.org/fJnYub1T

How weird... MacroTools must stem from Mustache.jl, which uses Requires.jl, which uses MacroTools...

I do some shady stuff with reactive and mustache right here:
https://github.com/JuliaGL/GLAbstraction.jl/blob/sd/reactive_next/src/GLShader.jl#L92

I see if I can debug a bit as well ;)

Update: I removed the WeakRef logic from nodes. This means all old code will be much happier. I realized this does not preclude the GC from helping out cleaning up nodes that go out of scope... Entire subtrees will get removed when all its nodes go out of scope! This is good. For example, with Reactive.debug_memory=true:
julia> using Reactive

julia> map(+, Input(1), Input(3))
Node{Int64}(4, nactions=0)

julia> gc() # might have to run this many times, but will eventually work
Node got gc'd. Creation backtrace:
call at /home/shashi/.julia/v0.4/Reactive/src/core.jl:49 (repeats 2 times)

julia> gc()
Node got gc'd. Creation backtrace:
call at /home/shashi/.julia/v0.4/Reactive/src/core.jl:49 (repeats 2 times)

julia> gc()
Node got gc'd. Creation backtrace:
call at /home/shashi/.julia/v0.4/Reactive/src/core.jl:50
 in _map at /home/shashi/.julia/v0.4/Reactive/src/operators.jl:19
 in map at /home/shashi/.julia/v0.4/Reactive/src/operators.jl:25

This means that when you reload a page in Escher, for example, the previous signals will go away. (I'm pretty sure this was the behavior in the current master as well... I have just been too dumb to think of it)
However, for some reason I cannot figure out, every or fps nodes refuse to go away...  Cleaning up these nodes is the most important (you want an animation to stop if no one is looking at it)
julia> every(1)
julia> gc() # as many times, but nothing happens Reactive.nodes still holds a reference to it...

I figured this should be because the timer callback points to the output node, so I tried to use WeakRef in the callbacks but that does not help. every_connect does seem to not hold any reference to the Node and does what you expect, as can be verified by running something like: Reactive.every_connect(1,Int[])
Is there some way of checking what references an object has? That would help quickly figure out the problem.
(test failure is due to delay caused by printing of errors, will get fixed once I disable debug_memory flag)

Are you sure you can do without WeakRef? I'm still trying to wrap my head around Reactive, but my concern is the following (illustrated by the TwoSharedNodes example in #75): suppose you start out with nodes A and B and merge them into C. Now later you want to replace B with B2; so you close(C) and then say C = merge(A, B2). The problem is, A still holds an Action whose recipient is the old C, and so I don't think that old C can be garbage-collected.

However, for some reason I cannot figure out, every or fps nodes refuse to go away...

With the new implementation, the timer never fires:
julia> using Reactive
INFO: Recompiling stale cache file /home/tim/.julia/lib/v0.4/Reactive.ji for module Reactive.

julia> n = every(1)
Node{Float64}(1.447256309980414e9, nactions=0)

julia> Reactive._messages
Channel{Any}(sz_max:1024,sz_curr:0)

julia> Reactive._messages
Channel{Any}(sz_max:1024,sz_curr:0)

julia> Reactive.nodes
WeakKeyDict{Any,Any} with 0 entries

(The delay there was much longer than a second.)

This works fine for me....
julia> using Reactive

julia> every(1)
Node{Float64}(1.447269625871374e9, nactions=0)

julia> x=map(println, ans)
1.44726963488805e9
Node{Void}(nothing, nactions=0)

julia> 1.447269635888282e9
1.447269636888132e9
1.447269637888482e9
1.447269638889957e9
1.447269639890409e9
1.447269640891841e9
1.447269641893283e9
1.447269642893697e9
1.447269643895142e9
1.44726964489559e9

Ah, you are correct.
I wondered if it might be a bug elsewhere, but this works:
julia> cleanup = Ref(false)
Base.RefValue{Bool}(false)

julia> z = [1]
1-element Array{Int64,1}:
 1

julia> y = Timer(x->WeakRef(z), 0.1, 0.1)
Timer(Ptr{Void} @0x00000000032e22a0,Condition(Any[Task (waiting) @0x00007f362f3d6d60]),true)

julia> finalizer(z, x->(cleanup[] = true; close(y)))

julia> isopen(y)
true

julia> z = 0
0

julia> gc()

julia> isopen(y)
false

So it seems like something else must be holding a reference?

We seem to have correct implementations of everything we need now.
Couple of questions to answer to get this merged:

Do we need bottom-up GC of the graph like it was before dfabab7 ? @SimonDanisch doesn't seem to like sprinkling preserves very much. @timholy what do you think?
The names Signal, Node and Input all point to the same thing... This one is a trifling matter, but it is slightly important that we bikeshed this. We ideally should to keep just one name, the best candidate seems to be Signal. We should deprecate Input for Signal if that makes sense to everyone (I feel the removal of Input will help understanding). Node is a good name internally (they are like wires which are connected to become or hold a signal) Signal is a good name in the public API. My odd preference here is to keep the type's name as Node but export Signal (typealias of Node). Here is the code. I will reluctantly agree to rename Node to Signal internally if that's what people want.

1.)
Can you explain a little more? What's possible and what isn't right now? I think I'd be fine with sprinkling preserves, as long as they follow very understandable rules! If at some point the general rule of thumb would be: "rather put a preserve around your signal to be on the safe side", we'd done something wrong ;)
2.)
I agree with your preferences... Signal makes most sense to the user, while Node is a good name internally!

Right now if a signal a is used to create several other signals, then either they all stop updating when they all go out of scope or all of them sit around and continue to sit around and update unless you close them explicitly. What's possible is: if a is used to create several signals, so as to make a graph of nodes, then any node whose children have gone out of scope (or a leaf node which has gone out of scope) will automatically stop updating after a gc pause. This means you will require preserve in places where you are doing side-effects e.g. preserve(map(println, a)) instead of just map(println, a) so that the map node will continue to exist as long as a does. I don't know if any other reactive programming systems do this... I couldn't find out much by googling.
I'm relieved one more person agrees! :)

We seem to have correct implementations of everything we need now.

Congratulations!

Do we need bottom-up GC of the graph like we had before dfabab7

I am torn on this one. On balance, I suspect that it would be better that way.
But I do have a question: suppose (like fps) we return a node/signal:
a = foo(7)

but internally in the course of creating a, we also need to create a "derived" node/signal b that we don't return to the user. We could, of course, call preserve(b), but then I think that will keep a from being gced?

Node/Signal/Input

Yes, definitely good to reduce the number of redundant names. You know about @deprecate_binding? (Used for deprecating types or other names.)
I agree with the preference Node internally. Externally, I have a slight preference for keeping it Node (rather than Signal) just for reasons of consistency (to facilitate blurring the lines between "user" and "developer" :wink:). But I will be fine with whatever you choose.

but internally in the course of creating a, we also need to create a "derived" node/signal b that we don't return to the user. We could, of course, call preserve(b), but then I think that will keep a from being gced?

I have another implementation of preserve which will allow b and a to be gc-ed together.

You know about @deprecate_binding? (Used for deprecating types or other names.)

I didn't know about it, will use it. Thanks!
Node seems to be too general / meaningless a name to export...

Here's the preserve we need.

I agree that Node is too general to export. That's a good reason for exporting Signal.

Awesome! Thanks for the great work :)

Yes indeed!


feature(middleware): Add logging middleware

:heavy_check_mark: Checklist:

[x] Have only one commit (if not, squash them into one commit)
[ ] Tests for the changes have been added (for bug fixes / features / refactoring)

Codecov Report

Merging #224 (11c5c0f) into main (da87702) will increase coverage by 8.29%.
The diff coverage is 69.68%.

@@            Coverage Diff             @@
##             main     #224      +/-   ##
==========================================
+ Coverage   66.84%   75.14%   +8.29%
==========================================
  Files          10       12       +2
  Lines         368      519     +151
==========================================
+ Hits          246      390     +144
+ Misses        111      100      -11
- Partials       11       29      +18

Impacted Files
Coverage Δ

internal/apischema/find.go
0.00% <0.00%> (ø)

internal/openapi3/parse.go
47.05% <50.00%> (+12.44%)
:arrow_up:

internal/apischema/apischema.go
72.41% <72.41%> (ø)

internal/openapi3/openapi3.go
75.00% <75.00%> (ø)

internal/openapi3/schema.go
75.00% <75.00%> (ø)

internal/faker/faker.go
96.92% <100.00%> (+1.64%)
:arrow_up:

internal/openapi3/example.go
100.00% <100.00%> (ø)

internal/openapi3/mediatype.go
100.00% <100.00%> (+14.28%)
:arrow_up:

internal/server/handler.go

internal/server/server.go

... and 1 more

Continue to review full report at Codecov.

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


Currently 'bucc fly' defaults target to 'bucc'; can now be overridden by 'fly_target:' in vars.yml

Closing. Submitting new PR for both fly_target and bosh_alias.

@drnic would it make sense to configure both with an environment variable? (for example BUCC_NAME?) it seems like an advanced usecase which probably will only be used from an other wrapper script. I'd like to keep the number of configuration options in vars.yml to a minimum.

@rkoster I really would prefer to ensure that anyone cloning the repo sets up the same name for the aliases.
If you think its polluting vars.yml for most users, then I can remove the examples from the vars.tmpl, and it can be a less well discoverable feature.

@drnic agreed, will remove examples from vars.tmpl, so we ensure people are less likely to change it (so alias stay the same). Don't want people to start using it for scoping their environment's they should use director_name for that.


How to use openvpn to connect to casdoor？And use Wecom as Casdoor's provider?

Those related features are not planned.

This issue will be deleted shortly


Add Safari & Safari on iOS 15.5

https://webkit.org/blog/12669/new-webkit-features-in-safari-15-5/
There‘s at least 3 functional changes, see top of the article, plus more being discovered (https://twitter.com/dannymoerkerke/status/1526526459573149697), which is why I’m arguing for independent version :)

A little note that 15.5 isn't getting picked up by the Google Analytics importer, either.

These were added, importer should work as expected too. Let me know if not.


Removing @generated functions

Right now, there is a mess of @generated functions in run/fractal_flame.jl. In principle, these can be removed by...
@kernel function naive_chaos_kernel!(points, n, H_fxs, H_kwargs,
                                     H_clrs, H_clr_kwargs,
                                     H_probs, H_fnums,
                                     layer_values, layer_reds, layer_greens,
                                     layer_blues, layer_alphas, frame, bounds,
                                     bin_widths, num_ignore, max_range)

    tid = @index(Global,Linear)

    pt = Ref(point(0.0,0.0))
    pt[] = points[tid]
    dims = Fae.dims(pt[])

    clr = Ref(RGBA{Float32}(0,0,0,0))

    seed = quick_seed(tid)
    fid = create_fid(H_probs, H_fnums, seed)

    for i = 1:n
        # quick way to tell if in range to be calculated or not
        sketchy_sum = absum(pt[])

        if sketchy_sum < max_range
            if length(H_fnums) > 1 || H_fnums[1] > 1
                seed = simple_rand(seed)
                fid = create_fid(H_probs, H_fnums, seed)
            else
                fid = UInt(1)
            end
            choices = find_random_fxs(fid, H_fnums, H_probs)

            #ntuple(Val(choices)) do k
            for k in NTuple(Val.(choices))
                Base.@_inline_meta
                pt[] = H_fxs[k](pt[].y, pt[].x, frame; H_kwargs[k]...)
                clr[] = H_clrs[k](pt[].y, pt[].x, clr[], frame; H_clr_kwargs[k]...)
            end

            histogram_output!(layer_values, layer_reds, layer_greens,
                              layer_blues, layer_alphas, pt[], clr[],
                              bounds, dims, bin_widths, i, num_ignore)
        end
    end

    @inbounds points[tid] = pt[]

end

ie, finding some way to iterate through known values with ntuple, like: https://github.com/CliMA/ClimateMachine.jl/blob/2e0b6b7d97719e410d12a8596c98d5db7f891dbf/src/Numerics/DGMethods/remainder.jl#L510
I couldn't quite figure out how to go from the tuple of ints to an ntuple of vals.
The core issue is that LLVM cannot optimize on the tuple of functions because each function is a unique type, so it cannot be stored as an ntuple to begin with.
This discussion was introduced in #64

Might be time to put some adult pants on and "just do it" by creating my own @generated macro for Fable: https://github.com/JuliaLang/julia/blob/d49a3c74c97c9ca9ef711d644a3f2b1c02b38d63/base/expr.jl#L1074
Here is @generated:
macro generated(f)
    if isa(f, Expr) && (f.head === :function || is_short_function_def(f))
        body = f.args[2]
        lno = body.args[1]
        return Expr(:escape,
                    Expr(f.head, f.args[1],
                         Expr(:block,
                              lno,
                              Expr(:if, Expr(:generated),
                                   body,
                                   Expr(:block,
                                        Expr(:meta, :generated_only),
                                        Expr(:return, nothing))))))
    else
        error("invalid syntax; @generated must be used with a function definition")
    end
end

I believe we can just save all the exprs from user fums and splat them all into a big function at the end. I don't know if this will actually cut compile time, but it does...

reduce the number of times we are metaprogramming.
It also allows us to do #74.
Should be the same cost as us going through GLSL for everything.

Lots of notes to toss from Slack:

No way around @generated in Julia. You can do it in C because ptrs are defined in modules that all know each other.
Try @noinline for @generated functions
Precompile tools
https://github.com/JuliaGPU/CUDA.jl/pull/1853

julia> struct VTable{T}
          funcs::T
       end

julia> @generated function (VT::VTable{T})(fidx, args...) where T
           N = length(T.parameters)
           quote
              Base.Cartesian.@nif $(N+1) d->fidx==d d->return VT.funcs[d](args...) d->error("fidx oob")
           end
       end

julia> VT = VTable(((x)->x+1, (x)->x+2))
VTable{Tuple{var"#2#4", var"#3#5"}}((var"#2#4"(), var"#3#5"()))

julia> VT = VTable(((x)->x+1, (x)->x+2))^C

julia> VT(1, 2)
3

julia> VT(2, 2)
4

julia> VT(3, 2)
ERROR: fidx oob
Stacktrace:
 [1] error(s::String)
   @ Base ./error.jl:35
 [2] macro expansion
   @ ./REPL[3]:4 [inlined]
 [3] (::VTable{Tuple{var"#2#4", var"#3#5"}})(fidx::Int64, args::Int64)
   @ Main ./REPL[3]:1
 [4] top-level scope
   @ REPL[7]:1

julia> @code_typed VT(3, 2)
CodeInfo(
1 ─ %1 = (fidx === 1)::Bool
└──      goto #3 if not %1
2 ─ %3 = Core.getfield(args, 1)::Int64
│   %4 = Base.add_int(%3, 1)::Int64
└──      return %4
3 ─ %6 = (fidx === 2)::Bool
└──      goto #5 if not %6
4 ─ %8 = Core.getfield(args, 1)::Int64
│   %9 = Base.add_int(%8, 2)::Int64
└──      return %9
5 ─      invoke Main.error("fidx oob"::String)::Union{}
└──      unreachable
) => Int64


As a student who has not used flashcard applications before, I can see the list of available commands when I type 'help'.

higher priority!


github webhook event/action

implement more event/actions
https://developer.github.com/webhooks/

do this when there are more concrete function for this


frontend saving to backend blocks to execute

generate blocks
save to the backed
make a controller to run them

Almost done by this package :
https://github.com/richorama/IronBlock/issues/28
However, the Swagger / and so on blocks / should be translated into C#

does it works now?

No , it was too difficult to do this.


Error messages not appearing

Excellent work Balin!
Tried testing out the error messages for cstring with this program:
  1 #include "cstring.h"
  2 #include <stdio.h>
  3 int main() {
  4   string *test = cstring("hello");
  5   insert(test, NULL, 0);
  6   printf("%s\n", test->str);
  7 }

But it doesn't display any error messages (i.e. in _verify), just segfaults. Is this intended?
Also, thoughts on renaming insert, copy, etc. to something more specific to avoid naming conflicts in user code?

YOO, thanks for the bug report. Was a missing part of the assertion statement in the insert function. Probably more in this that I haven't caught yet, need to refactor this entire thing one day. As for your later concern, its very dependent on the situation. I could rename them to cstring_add but then that would become tedious. However, I think that if I use long, unique function names and have a bunch of wrapper functions which you can include, which can be in a separate file, would be the best solution that I can think of right now given that C is not object oriented.


Conflicting with the core sls invoke command (v1)

Once I've added this plugin to my Serverless (v1) project, I can't invoke my function via the usual "serverless invoke -f hello" command. I get this debug output:
$ serverless invoke -f hello

  Type Error ---------------------------------------------

     Cannot read property 'forEach' of undefined

     For debugging logs, run again after setting SLS_DEBUG env var.

  Stack Trace --------------------------------------------

TypeError: Cannot read property 'forEach' of undefined
    at PluginManager.getEvents (/Users/rowan/.nvm/versions/node/v4.3.2/lib/node_modules/serverless/lib/classes/PluginManager.js:216:39)
    at PluginManager.run (/Users/rowan/.nvm/versions/node/v4.3.2/lib/node_modules/serverless/lib/classes/PluginManager.js:77:25)
    at Serverless.run (/Users/rowan/.nvm/versions/node/v4.3.2/lib/node_modules/serverless/lib/Serverless.js:77:33)
    at /Users/rowan/.nvm/versions/node/v4.3.2/lib/node_modules/serverless/bin/serverless:18:50

Looking closer, the commandDetails variable in PluginManager.js (line 216) at run time is:
{ usage: 'Invoke mocha tests for service / function',
  commands:
   { test:
      { usage: 'Invoke test(s)',
        lifecycleEvents: [Object],
        options: [Object] } } }

I'm just getting started with Serverless, but it looks like it's not expecting this object to have so many levels (i.e. it's expecting the lifecycleEvents property to be at the top level of commandDetails).
Is this a configuration issue on my side? I'm adding this snippet at the bottom of my serverless.yml file to load the plugin:
plugins:
  - serverless-mocha-plugin

If I don't load this plugin, the invoke command works as expected.

I think it's a serverless issue since serverless documentation gives me the understanding that this kind of command nesting is allowed. See: https://github.com/serverless/serverless/issues/1987

Thanks for the response! I'm still getting a hang of the framework. I'll track it in the upstream ticket.


initial error message reformat

This PR is aimed to rewrite the pretty print for vaildator error types to include details about the bounds and the name of a newtype, addressing #105

@danma3x Thank you!
Looks pretty good!
Could please also fix the assertions in examples/serde_complex/src/main.rs ?


ReadDocumentCollectionAsync not returning any storage information

I'm following this guide on how to see the current storage size of a collection.
However, it appears that the values for all of these are 0 - is there a different way to do this?

It looks like you need to specify the PopulateQuotaInfo property as true in the RequestOptions:
var collection = await documentClient.ReadDocumentCollectionAsync(collectionLink, new RequestOptions { PopulateQuotaInfo = true });

Maybe the docs should be updated?


VoxelGridBasedEuclideanCluster spam logs when an empty cloud is received

Checklist

[X] I've read the contribution guidelines.
[X] I've searched other issues and no duplicate issues were found.
[X] I'm convinced that this is not my fault but a bug.

Description
For example:
[component_container-40] Failed to find match for field 'x'.
[component_container-40] Failed to find match for field 'y'.
[component_container-40] Failed to find match for field 'z'.
[component_container-40] [pcl::KdTreeFLANN::setInputCloud] Cannot create a KDTree with an empty input cloud!
[component_container-40] Failed to find match for field 'x'.
[component_container-40] Failed to find match for field 'y'.
[component_container-40] Failed to find match for field 'z'.
[component_container-40] [pcl::KdTreeFLANN::setInputCloud] Cannot create a KDTree with an empty input cloud!
[component_container-40] Failed to find match for field 'x'.
[component_container-40] Failed to find match for field 'y'.
[component_container-40] Failed to find match for field 'z'.
[component_container-40] [pcl::KdTreeFLANN::setInputCloud] Cannot create a KDTree with an empty input cloud!
[component_container-40] Failed to find match for field 'x'.
[component_container-40] Failed to find match for field 'y'.
[component_container-40] Failed to find match for field 'z'.
[component_container-40] [pcl::KdTreeFLANN::setInputCloud] Cannot create a KDTree with an empty input cloud!

There are a few cases where it could happen. For example, when the vehicle lidar scan perfectly matches the pointcloud map, the output of VoxelBasedCompareMapFilterComponent is empty.
I think it would be nice to handle this case and avoid meaningless spam. Maybe a timed warning could do.
Expected behavior
no spam, or at least meaningful messages.
There are many kdtree->setInputCloud in autoware, so it took me ages to find which one was having an issue.
Actual behavior
SPAM
Steps to reproduce
Edge use case, so difficult to reproduce. Maybe in a simulator it is easier.
Versions
No response
Possible causes
No response
Additional context
No response

Solved by:

https://github.com/autowarefoundation/autoware.universe/pull/4029


feat!: enable interchain accounts

Closes https://github.com/celestiaorg/celestia-app/issues/3062
Remaining tasks

[x] Update specs
[ ] Add list of allowed messages from CIP
[ ] Add tests

Links

https://github.com/strangelove-ventures/interchaintest
https://github.com/cosmos/ibc-go/blob/main/e2e/tests/interchain_accounts/base_test.go

Docker image was pushed to https://github.com/celestiaorg/celestia-app/pkgs/container/celestia-app/191669081?tag=pr-3182

    ica_test.go:75:
        	Error Trace:	/Users/rootulp/git/rootulp/celestiaorg/celestia-app/interchaintest/ica_test.go:75
        	Error:      	"[]" should have 1 item(s), but has 0
        	Test:       	TestICA

        -2024-03-15T19:56:10.332900Z	debug	Error querying block data	{"chain_name": "celestia", "chain_id": "celestia", "height": 2, "error": "error in json rpc client, with http response metadata: (Status: 200 OK, Protocol HTTP/1.1). RPC error -32603 - Internal error: node is not persisting abci responses"}

    ica_test.go:103:
        	Error Trace:	/Users/rootulp/git/rootulp/celestiaorg/celestia-app/interchaintest/ica_test.go:103
        	Error:      	Received unexpected error:
        	            	exit code 1:  Error: unknown command "intertx" for "tx"

If we want the ICA tests to run on every PR, we're going to need to build Docker images on every PR and then plumb the appropriate label (PR # or commit hash) into make test-interchain. Since we currently only build Docker images for branches on the upstream repo (a.k.a not on forks), we could alternatively only run this test on new commits to main which should produce new Docker images already.

I'll pause work on the interchaintests because I'm making extremely slow progress. I'll try to get this PR to a review-able state without the end to end tests and then try adding interchaintests in a follow-up PR.

Oof it looks like the testApp doesn't spin up a GRPC server so the test was only passing locally because I have a celestia-app running that exposes the GRPC server on localhost. Will prob revert back to just querying via the param keeper b/c I can't find an example where testApp spins up a GRPC server.

Gahhhh another merge conflict 💀


[WFLY-16277]: NetworkHealthTestCase failures on Windows.

Fixing the default windows ping command.

Jira: https://issues.redhat.com/browse/WFLY-16277

@ehsavoie If this solves the problem should it be mentioned in https://github.com/wildfly/wildfly/blob/main/docs/src/main/asciidoc/_admin-guide/subsystem-configuration/Messaging_Discovery_Configuration.adoc ?

@bstansberry the windows failure is not related :)
You can merge this :)

Thanks, @ehsavoie


Category scroll gesture doesn't work with plain EmojiKeyboard and enableCategoryChangeGesture

Describe the bug
Hi folks! I noticed the scroll gesture for each category doesn't work when using the plain EmojiKeyboard (no modal) and enableCategoryChangeGesture. See screenshots for more.
To Reproduce
Wrap the EmojiKeyboard in a BottomSheet, similar to:
<BottomSheet
    index={0}
    snapPoints={['60%']}
  >
    <EmojiKeyboard
      disableSafeArea
      enableCategoryChangeAnimation
      enableCategoryChangeGesture={false}
      enableSearchAnimation={false}
      enableSearchBar
      onEmojiSelected={handleEmojiSelected}
      styles={emojiKeyboardStyles}
    />
</BottomSheet>

Unfortunately my bandwidth is short to create a full working repro, but lmk if you have troubles reproing.
Expected behavior
I can scroll vertically through a category, without EmojiKeyboard attempting to change the focused category.
Screenshots
With enableCategoryChangeGesture={true}
Vertical scroll doesn't work, it tries to change categories:
https://github.com/TheWidlarzGroup/rn-emoji-keyboard/assets/8047841/c9169b2a-3971-4bb9-a391-1d4b6a2a4d49
With enableCategoryChangeGesture={false}
Vertical scroll works as expected
https://github.com/TheWidlarzGroup/rn-emoji-keyboard/assets/8047841/68d04c71-7deb-4ad8-bb8e-ef60bab45a1d
Additional context
Add any other context about the problem here.

Hi @MarceloPrado,
Thanks for reporting that. I've figured that the problem is not the modal itself, but a horizontal margin added on the parent of EmojiKeyboard component.
A temporary solution may be to remove the extra margin on the modal (of course, I guess that this is not the best advice 🙈), meanwhile, we will definitely take a closer look at this issue and try to fix it in the next release.


I don't have all the tweets

Hello,
I'm using this API for my final project of the university. I need to consult tweets between two dates. I run the command line and I have compared the result with a search in Twitter and I don´t have all the tweets. Somebody know what is the problem?
Thanks

Same problem here.
I realize that depends on where I run. On my local machine, in a Brazilian IP, I receive almost all my tweets. At Google App Engine I receive only the "most relevant", independent on search queries. For famous profiles I receive almost all tweets at both IPs.

Same here. Change of f=realtime to f=tweets is also not doing any good.


Connect to WSL Services with Other Devices

Hi.
My Windows version is 10.0.15063.
I start service in WSL. But it does not have outside IP address. So when I use an outside device, for example, a phone , cannot connect to the service in WSL.
These are service in WSL. And I can connect them in my computer locally. But the other devices in LAN cannot.
 Thanks!!

But it does not have outside IP address.
It should have the same IP address configuration as Windows (basically):

I can connect them in my computer locally. But the other devices in LAN cannot.

@ChillingVan - Those videos might be helpful, I guess;

How to turn the firewall on or off in Windows 10
How to add a rule or port to a Windows 10 firewall


fix getting user's home dir

When running binwalk in non-interactive session some environment variables may not be set. This causes binwalk to fail to correctly determine user's home directory. This commit fixes that (at least for linux systems).
It should be also tested on Windows systems.

My bad, sorry for that. There are already pull requests for that - https://github.com/ReFirmLabs/binwalk/pull/411 and https://github.com/ReFirmLabs/binwalk/pull/414


ICA tutorial: wrong components (random seed?)

The ICA tutorial is a bit off.

I can't promise I can fix this for 0.12 ...
@dengemann

I will. No worries.
On Sun, Apr 24, 2016 at 9:52 PM jona-sassenhagen<EMAIL_ADDRESS>wrote:

The ICA tutorial is a bit off.
[image: screen shot 2016-04-24 at 21 49 53]
https://cloud.githubusercontent.com/assets/4321826/14770019/bdb267de-0a66-11e6-9250-8955bff46fa4.png
[image: screen shot 2016-04-24 at 21 50 03]
https://cloud.githubusercontent.com/assets/4321826/14770021/bf88cd32-0a66-11e6-91f7-91df130d875a.png
I can't promise I can fix this for 0.12 ...
@dengemann https://github.com/dengemann
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/mne-tools/mne-python/issues/3168

Fixed in https://github.com/mne-tools/mne-python/pull/3174. See if you agree.

Oh oh ...

I could take care of it eventually.

@Eric89GXL reopen please :(

This was recently fixed. Do you know the cause? Was there a regression,
errant commit, or is this example fragile? (Will reopen when I'm off mobile)

This is very weird indeed.

Bisecting now...

the fun thing is that this got fixed 2 times at least.
On Wed, May 11, 2016 at 4:03 PM Eric Larson<EMAIL_ADDRESS>wrote:

Bisecting now...
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/mne-tools/mne-python/issues/3168#issuecomment-218468720

Found it -- @dengemann this commit makes things non-deterministic, which is funny since it's title is "fix seed issue in ICA tutorial":
https://github.com/mne-tools/mne-python/commit/4205bf09fec99067d4c5da667242146e01a10c94#diff-862a5a4a87f90baa4d64932537a53851R154
Notably, the seed= kwarg was removed in that commit :(

... or rather, the "reference ICA" has no seed, so when (in that commit) it was switched from using the first of the section-based ICAs (which were deterministic) to the "reference ICA", it lost its deterministic-ness.

In any case, I'm not sure of the proper fix here (seed the original, or go back to seeding the others and use the first one again), but I'm not surprised it's non-deterministic, because the random state isn't ever set. @dengemann can you fix it?

Ok I'll fix it once more. ... I hope for the last time.
On Wed, May 11, 2016 at 4:17 PM Eric Larson<EMAIL_ADDRESS>wrote:

In any case, I'm not sure of the proper fix here (seed the original, or go
back to seeding the others and use the first one again), but I'm not
surprised it's non-deterministic, because the random state isn't ever set.
@dengemann https://github.com/dengemann can you fix it?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/mne-tools/mne-python/issues/3168#issuecomment-218473100

@dengemann will you have time for this soon, or should someone else have a look (@jona-sassenhagen maybe?)

I will have time on Monday for some MNE hacking.

@dengemann it's been a while, any time for this one or should we get someone else on it?


Add Osu! 2020.613.0

Auto Generated by<EMAIL_ADDRESS>on Fri, 12 Jun 2020 17:05:29 +0000
Microsoft Reviewers: Open in CodeFlow

Greetings. When I run the app Osu!laser, I do not see anything happen.  It starts to launch, then nothing.  Is it missing a dependency?


$containerManager->logs() causes empty response

here is my code to show a container logs via browser, based on Zend Framework:
public function getLogsOfContainerAction(){
        try{
            $containerId = $this->_request->getParam("id");
            if(empty($containerId)){
                $this->ajaxError("container ID not provided!");
            }
            $container = My_Docker::getContainerById($containerId);
            if(!$container){
                $this->ajaxError("This container does not exist!");
            }
            $containerManager = My_Docker::getContainerManager();
            $log = $containerManager->logs($container->getId(),["stderr"=>true, "stdout"=>true]);
            $output = htmlentities(ltrim($log->getBody()->getContents()));
            $this->ajaxSuccess($output);
        }catch (Exception $e){
            $this->ajaxError($e->getMessage());
        }
    }

for some containers, it does show me the logs, however, some just get empty response:
[22:58:58]:~/test$wget http://docker-host/docker-container/get-logs-of-container/id/a3ac18fe5e26c53fdc3000db2f8ff6465cf98c09b6b5722e32844f4e25147a0a
--2016-03-17 22:59:00--  http://docker-host/docker-container/get-logs-of-container/id/a3ac18fe5e26c53fdc3000db2f8ff6465cf98c09b6b5722e32844f4e25147a0a
Resolving docker-host... <IP_ADDRESS>
Connecting to docker-host|<IP_ADDRESS>|:80... connected.
HTTP request sent, awaiting response... No data received.
Retrying.

--2016-03-17 22:59:01--  (try: 2)  http://docker-host/docker-container/get-logs-of-container/id/a3ac18fe5e26c53fdc3000db2f8ff6465cf98c09b6b5722e32844f4e25147a0a
Connecting to docker-host|<IP_ADDRESS>|:80... connected.
HTTP request sent, awaiting response... No data received.
Retrying.

^C

PS. this container is running well, I can see the logs via docker client:
[23:27:22]:~/test$docker logs a3ac18fe5e26c53fdc3000db2f8ff6465cf98c09b6b5722e32844f4e25147a0a
== |2016-03-17 22:58:26| Start monitoring LOCAL TEST REDIS ============
...

when I say empty response, it actually means, looks like my apache died when executing this line:
$log = $containerManager->logs($container->getId(),["stderr"=>true, "stdout"=>true]);

I tried:
var_dum($log);
exit();

still no http response from my web server.

[23:38:55]:~/test$docker version
Client:
 Version:      1.9.1
 API version:  1.21
 Go version:   go1.5.1
 Git commit:   a34a1d5
 Built:        Sat Nov 21 00:49:19 UTC 2015
 OS/Arch:      darwin/amd64

Server:
 Version:      1.9.1
 API version:  1.21
 Go version:   go1.4.3
 Git commit:   a34a1d5
 Built:        Fri Nov 20 17:56:04 UTC 2015
 OS/Arch:      linux/amd64

Have you tried to surround your code with a try/catch ?

Can you try also to add tail => 'all' to the parameters list ?


How to change button color

This might be a trivial issue, but how do I change the color of the button in the dropdown?
I've tried the following, but this approach does not work. The CSS class btn2 is not being applied in this case.
Settings:
mySettings: IMultiSelectSettings = {
buttonClasses: 'btn btn2',
};
CSS:
.btn2 {
background-color: #ff0000;
}

That looks correct to me, is that the only configuration you are applying? If so the issue might be in the application of settings - are you assigning the settings properly on initialisation of the control?
You can see it working on my demo site here:
https://chris--jones.github.io/angular-2-dropdown-multiselect/
e.g. Change buttonClasses to btn btn-danger
buttonClasses: 'btn bg-danger'

Thank you Chris. The example you gave did work. The issue seems to be that I can't apply css properties defined in the Angular 2 component css file. I guess this could be an issue with css scoping in Angular 2.

I found the issue. I was not properly applying the css to the child component. The following css works from the parent component:
:host /deep/ ss-multiselect-dropdown .btn2 {
background-color: #ff0000;
}

Hi @Ezeon, could you elaborate on your solution? I failed to understand that..
What is this:

:host /deep/ ss-multiselect-dropdown

Thx!

@gangfang If you want to add scoped css in the parent component, you would need to use ':host /deep/' for it to affect a child component such as the ss-multiselect-dropdown.
Scoped css is called component styles in Angular. You can read about it here: https://angular.io/guide/component-styles
Of course, you could also just add it to the global css file.


[Bug]: Out of memory and crash while exporting

What happened?
While exporting my vault I am getting two kinds of issues:

First (and sometimes second) click on the icon on the ribbon does nothing. Only the next click opens the modal for me. No errors in the console.

 It's during the writing files to disk step. The Modal becomes white (see bug4.png) and the entire Obsidian crashes, leaving only the white window intact.

Now, when the console is open I can catch an out-of-memory exception (see bug1.png) and continue execution until I see the same white screen (see bug4.png)
Reproduction
(I am not able to share the vault)
Last Working Version
Was working before the new modal (with folder selector) was introduces (a long time ago)
Version
1.8.01
Operating System
Windows 11
Obsidian Version
1.6.5
Which browsers can you reproduce this in?
 Obsidian has a memory limit which is much lower than your system's memory limit. The solution is to allow the files to be written to disk as they go rather than saving everything at the end, but I have not yet implemented this feature.

Hey, I just released a beta version that should address this. If you could try it out and let me know if his issue is fixed and if you find any other issues. There are instructions for installing the beta in the readme :)
Thanks!


Automatic plist caveat always shows "sudo"

[x] ran brew update and can still reproduce the problem?
[x] ran brew doctor, fixed all issues and can still reproduce the problem?
[x] ran brew config and brew doctor and included their output with your issue?

What you were trying to do (and why)
I am trying to provide a Formula that provides a service that can be managed with brew service. This service needs to run as the logged in user, not as root. This formula comes with a plist stanza.
https://github.com/FiloSottile/yubikey-agent/blob/master/HomebrewFormula/yubikey-agent.rb
What happened (include command output)
On installation, a caveat is automatically shown, which is convenient. However, the suggested command includes sudo which starts the service as root, and causes the service to break (FiloSottile/yubikey-agent#16, FiloSottile/yubikey-agent#25).
To have launchd start yubikey-agent now and restart at startup:
  sudo brew services start yubikey-agent

brew services start without sudo works well.

Command output

brew reinstall yubikey-agent
==> Downloading https://homebrew.bintray.com/bottles/go-1.14.3.catalina.bottle.tar.gz
==> Downloading from https://akamai.bintray.com/e3/e3c4f834acba4e4db3f2b848860a040de1d8dec158545e8aaa3dd2f380c9c950?gda=exp=1590253514~hmac=23ef415fd0892b05a6003490bae373809d23cf5524c4474f1171b39210
######################################################################## 100.0%
==> Downloading https://github.com/FiloSottile/yubikey-agent/archive/v0.1.1.zip
Already downloaded: /Users/filippo/Library/Caches/Homebrew/downloads/53c03a96f64ebea6b983932b643ff0fa33b1390ea9e376033c0d6469850776f6--yubikey-agent-0.1.1.zip
==> Reinstalling yubikey-agent
==> Installing dependencies for yubikey-agent: go
==> Installing yubikey-agent dependency: go
==> Pouring go-1.14.3.catalina.bottle.tar.gz
🍺  /usr/local/Cellar/go/1.14.3: 9,441 files, 424.6MB
==> Installing yubikey-agent
==> go build -ldflags -X main.Version=v0.1.1 /usr/local/Cellar/yubikey-agent/0.1.1/bin filippo.io/yubikey-agent
==> Caveats
To set up a new YubiKey, run this command:
yubikey-agent -setup
To use this SSH agent, set this variable in your ~/.zshrc and/or ~/.bashrc:
export SSH_AUTH_SOCK="/usr/local/var/run/yubikey-agent.sock"
To have launchd start yubikey-agent now and restart at startup:
sudo brew services start yubikey-agent
==> Summary
🍺  /usr/local/Cellar/yubikey-agent/0.1.1: 6 files, 6.0MB, built in 21 seconds
==> Caveats
==> yubikey-agent
To set up a new YubiKey, run this command:
yubikey-agent -setup
To use this SSH agent, set this variable in your ~/.zshrc and/or ~/.bashrc:
export SSH_AUTH_SOCK="/usr/local/var/run/yubikey-agent.sock"
To have launchd start yubikey-agent now and restart at startup:
sudo brew services start yubikey-agent

What you expected to happen
I would like to be able to configure the caveat not to suggest using sudo, or be able to turn it off so I can replace it with my own text.
Step-by-step reproduction instructions (by running brew commands)
brew tap filippo.io/yubikey-agent https://filippo.io/yubikey-agent
brew install yubikey-agent

Output of brew config and brew doctor commands

brew config
HOMEBREW_VERSION: 2.2.17-78-g9265ea0
ORIGIN: https://github.com/Homebrew/brew
HEAD: 9265ea0c505cd06773d89ccd26770f0d0683592c
Last commit: 22 hours ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: c40b0f25552c3d1a254ef61635ccff6a2dac286b
Core tap last commit: 18 hours ago
HOMEBREW_PREFIX: /usr/local
HOMEBREW_AUTO_UPDATE_SECS: 3600
HOMEBREW_DISPLAY: /private/tmp/com.apple.launchd.eOllW3lTPU/org.macosforge.xquartz:0
HOMEBREW_EDITOR: code -w
HOMEBREW_MAKE_JOBS: 4
HOMEBREW_NO_ANALYTICS: set
HOMEBREW_NO_INSECURE_REDIRECT: set
CPU: quad-core 64-bit kabylake
Homebrew Ruby: 2.6.3 => /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/bin/ruby
Clang: 11.0 build 1103
Git: 2.26.2 => /usr/local/bin/git
Curl: 7.64.1 => /usr/bin/curl
Java: 1.8.0_141
macOS: 10.15.4-x86_64
CLT: <IP_ADDRESS>.1.1586360307
Xcode: N/A
XQuartz: 2.7.11 => /opt/X11

Please note that these warnings are just used to help the Homebrew maintainers
with debugging if you file an issue. If everything you use Homebrew for is
working fine: please don't worry or file an issue; just ignore this. Thanks!

Warning: Some installed formulae were deleted!
You should find replacements for the following formulae:
  gomod-ww
  gomod-gotests
  go-get-go1.13rc2
  gomod-dlv
  go-get-gvt
  gomod-staticcheck
  go-get-goimports
  go-get-go1.8.7
  go-get-go1.8
  go-get-gosumcheck
  go-get-benchstat
  go-get-git-cleanup
  go-get-go1.9.5
  go-get-vgo
  go-get-ct-submit
  go-get-go1.11.13
  go-get-go1.11rc1
  go-get-gostatus
  go-get-rmapi
  go-get-tcat
  go-get-vendorcheck
  go-get-go1.14rc1
  go-get-unconvert
  go-get-git-codereview
  go-get-git-codereview
  go-get-maintserve
  phantomjs
  go-get-go1.9rc2
  go-get-go1.12.14
  go-get-go1.11beta1
  go-get-gohack
  gomod-gotip
  go-get-go1.11rc2
  go-get-macos-roots-test
  go-get-pprof

Warning: Unbrewed dylibs were found in /usr/local/lib.
If you didn't put them there on purpose they could cause problems when
building Homebrew formulae, and may need to be deleted.

Unexpected dylibs:
  /usr/local/lib/libykcs11.copy.dylib

The sudo here is due to using plist_options :startup => true. If you remove that line it will start as the current user at user login rather than system startup. We do not provide a supported way to run at system startup as the current user/not using sudo in this situation.
Does that explain the behaviour?
Thanks!

The sudo here is due to using plist_options :startup => true. If you remove that line it will start as the current user at user login rather than system startup. We do not provide a supported way to run at system startup as the current user/not using sudo in this situation.

Oh that makes sense, I misinterpreted :startup to mean "not just on request", but that's controlled by using brew services run instead. Fixed, thanks.
Maybe it should be an error to run a non-:startup service with sudo, or a :startup one without?

@FiloSottile Good idea, done in https://github.com/Homebrew/homebrew-services/pull/226

(@FiloSottile well, to be specific: not erroring as there is a use-case for that but instead printing warning noting if the user is doing what the formula doesn't recommend as it might not be what they expect).

Awesome!


Add support for '' (backspace) escaped double quotes in double-quoted string literals

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Benefit
TerraForm and its auto-completion library https://github.com/posener/complete are unable to offer autocomplete functionality because of the lack of support for the otherwise-standard '\'-escaped quotes. This also breaks many other scripts written scripts and commands (e.g. those written for cmd.exe and batch files for one example) that have been written to rely on the standard '\'-escaped quotes.
Steps to reproduce
Enter following on the command line:
PS> "name"=\"value\""

Expected behavior
Command line should accept the string, similar to when using '""' double quote-escaped double quotes or as in other modern shells such as BASH, cmd.exe, and BATCH (.bat) files:
PS> "name=""value"""
name="value"

Actual behavior
ParseError: Unexpected token 'value""' in expression or statement.
PS> "name=\"value\""
ParserError:
Line |
   1 |  "name=\"value\""
     |          ~~~~~~~~
     | Unexpected token 'value\""' in expression or statement.

Error details
Type        : System.Management.Automation.ParseException
Errors      :
    Extent  : value\""
    ErrorId : UnexpectedToken
    Message : Unexpected token 'value\""' in expression or statement.
Message     : At line:1 char:9
              + "name=\"value\""
              +         ~~~~~~~~
              Unexpected token 'value\""' in expression or statement.
ErrorRecord :
    Exception             :
        Type    : System.Management.Automation.ParentContainsErrorRecordException
        Message : At line:1 char:9
                  + "name=\"value\""
                  +         ~~~~~~~~
                  Unexpected token 'value\""' in expression or statement.
        HResult : -2146233087
    CategoryInfo          : ParserError: (:) [], ParentContainsErrorRecordException
    FullyQualifiedErrorId : UnexpectedToken
    InvocationInfo        :
        ScriptLineNumber : 1
        OffsetInLine     : 9
        HistoryId        : -1
        Line             : "name=\"value\""
        PositionMessage  : At line:1 char:9
                           + "name=\"value\""
                           +         ~~~~~~~~
        CommandOrigin    : Internal
TargetSite  :
    Name          : Invoke
    DeclaringType : System.Management.Automation.Runspaces.PipelineBase, System.Management.Automation,
Version=<IP_ADDRESS>0, Culture=neutral, PublicKeyToken=31bf3856ad364e35
    MemberType    : Method
    Module        : System.Management.Automation.dll
Source      : System.Management.Automation
HResult     : -2146233087
StackTrace  :
   at System.Management.Automation.Runspaces.PipelineBase.Invoke(IEnumerable input)
   at Microsoft.PowerShell.Executor.ExecuteCommandHelper(Pipeline tempPipeline, Exception& exceptionThrown,
ExecutionOptions options)

Environment data
Name                           Value
----                           -----
PSVersion                      7.3.4
PSEdition                      Core
GitCommitId                    7.3.4
OS                             Microsoft Windows 10.0.22621
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals

Let me share some options with you:

Write your own powershell tab completion code.
You can refer to: kubectl completion powershell | Out String

https://github.com/KaiKorla/TerraformCompletion

Use cloud native powershell commands instead of terraform.
AWS: https://learn.microsoft.com/zh-cn/powershell/module/az.accounts/?view=azps-8.3.0#accounts
Azure: https://docs.aws.amazon.com/powershell/index.html

My project Kasini3000 provides many multi cloud consistent powershell scripts
https://github.com/kasini3000/kasini3000

PowerShell already includes an escape character (the backtick). AFAIK this was chosen over the more common backslash to make it easier to deal with Windows paths like: "C:\Windows".
Some may not like it but every time I have to add a Windows path to a JSON file I think about how annoying it is to escape \ so I've come to appreciate this design choice.
Regardless of how you feel about this design choice, changing it now seems unrealistic. It would be a huge breaking change for practically every script out there that uses a Windows path for anything.


I am interested in using your game on my website. Could you add a license file like the MIT one in this pull request so I know how you want your software used?

Sure, feel free to use it as you see fit.


Made some variable names more consistent with the other parts.

Hello, we're developing an automated system that detects inconsistent variable names in a large software project. Our system checks if each variable name is consistent with other variables in the project in its usage pattern, and proposes correct candidates if inconsistency is detected. This is a part of academic research that we hope to publish soon, but as a part of the evaluation, we applied our systems to your projects and got a few interesting results. We carefully reviewed our system output and manually created a patch to correct a few variable names. We would be delighted if this patch is found to be useful. If you have a question or suggestion regarding this patch, we'd happily
answer. Thank you.

Coverage decreased (-0.1%) to 86.613% when pulling 8fae1705b69c6c27976e97801b9bdbff37c075f0 on euske:master into 205876d6f9eb60ac31f3ec848b23d025f32995ab on apache:master.

Coverage decreased (-0.1%) to 86.613% when pulling 8fae1705b69c6c27976e97801b9bdbff37c075f0 on euske:master into 205876d6f9eb60ac31f3ec848b23d025f32995ab on apache:master.

I am on the fence with this PR.
Personally I don't see any benefit in consistent naming of local variables as long as the local variables have names that can be understood. To me a local variable named length is in no way clearer than one called len.
We will never get a consistency at the level your tool is striving for on a project that relies on numerous volunteer contributors. I don't see myself frustrating contributors by asking them to rename their local variables on patches.
@garydgregory if you see value in this and want to merge the PR after the changes you've requested have been made I won't stand in the way but I'll otherwise ignore it myself.

I don't care that the changes come from a tool, I am just looking to see if the code is easier to read, which admittedly is subjective of course. But yeah, I can bring in the PR at some point. But if this is just a PR where the poster does not come back and update the code or comment, then it will go nowhere.

Personally I don't see any benefit in consistent naming of local variables as long as the local variables have names that can be understood. To me a local variable named length is in no way clearer than one called len.
We will never get a consistency at the level your tool is striving for on a project that relies on numerous volunteer contributors. I don't see myself frustrating contributors by asking them to rename their local variables on patches.

I think that's a valid concern. Our system proposed numerous small changes, but I only sent some of the most conspicuous ones. I would say that overall the variable naming in this project is pretty consistent. We're not insisting the change.

Please rebase on master. Recent changes should allow all builds to be green including Java 16 and 17-EA.

I think the consensus is that is not really worth it. Closing.


Could not install package 'System.IO.FileSystem 4.3.0'. You are trying to install this package into a project that targets '.NETPortable,Version=v4.5,Profile=Profile259', but the package does not contain any assembly references or content files that are compatible with that framework.

Working on Xamarin Cross Platform project(PCL) in Visual Studio 2017. Need to do some file IO operations but not able to add reference to System.IO.FileSystem nugget. Getting this error Could not install package 'System.IO.FileSystem 4.3.0'. You are trying to install this package into a project that targets '.NETPortable,Version=v4.5,Profile=Profile259', but the package does not contain any assembly references or content files that are compatible with that framework.

This isn't an ASP.NET question, it appears to be one for Xamarin. So I'd suggest finding the right place in their org, https://github.com/xamarin but I don't know what you'd choose, so I can't move the issue for you.

I have raised it under Xamarin.Thanks


.NET demo application

Equivalent apart from structured logging. We can open an issue for that if we need to merge this quickly for the live office hours.
Deploys with the exact same Helm chart (literally a symlink).

Requested review from @cristiklein

Closing in favor of this one, which makes pre-commit happy: https://github.com/elastisys/compliantkubernetes/pull/245


17 integrate into vantage6

Oh, also the tests seem to be failing - is that something that you're going to work on next, or should that already be working?

I removed the useless push script. In the end I decided it's not necessary because github actions will take care of pushing the images

OK I fixed the tests now


ability to define end-point URLs

I wanted to do a testride but its not connecting to docker-run ollama. (see below)
which could maybe worked around, if we could  define the endpoints for the ollama-server (might run on another machine)
and the own IP:PORT to bind to.
any plans for that?
and how to use it with docker?
  62 |
  63 | function getInitialModel() {
> 64 |   fetch("http://localhost:11434/api/tags")
     |  ^
  65 |     .then((response) => response.json())
  66 |     .then((data) => {
  67 |       console.log(data);

I've got an implementation working and am putting together a PR.

Following up, I've added this feature under PR #22.

merged!


Build error in Mac 10.12.4

I tried installing caffe2 from source and got the following error.

Undefined symbols for architecture x86_64:
  "cv::_InputArray::_InputArray(cv::Mat const&)", referenced from:
      caffe2::ImageInputOp<caffe2::CPUContext>::GetImageAndLabelAndInfoFromDBValue(std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > const&, cv::Mat*, caffe2::ImageInputOp<caffe2::CPUContext>::PerImageArg&, int) in image_input_op.cc.o
  "cv::_OutputArray::_OutputArray(cv::Mat&)", referenced from:
      caffe2::ImageInputOp<caffe2::CPUContext>::GetImageAndLabelAndInfoFromDBValue(std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > const&, cv::Mat*, caffe2::ImageInputOp<caffe2::CPUContext>::PerImageArg&, int) in image_input_op.cc.o
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

Does anybody know how to fix this? I can get it to build by disabling opencv when doing cmake, but can't get it to work with opencv. I tried reinstalling opencv with homebrew but that didn't work.

I have the same issue. clang -v gives
Apple LLVM version 8.0.0 (clang-8<IP_ADDRESS>)
Target: x86_64-apple-darwin16.5.0
Thread model: posix
InstalledDir: /Library/Developer/CommandLineTools/usr/bin

I'm going through and closing old issues. If this is still a problem, please feel free to open a new issue.
FWIW, we have been running continuous builds on macOS 10.13 (High Sierra).


[SPIKE] Evaluate current state of AKS CAPI provider

[SPIKE] Evaluate current state of AKS CAPI provider.

Workshop in March


commentmessage can be too big and fail

Problem Description
There is a limit to comment in checks api summary. When there are a large number of changes (like changing a setting across all repos), this API call can fail, casuing no completion status to ever be posted
What is actually happening
Check run stays pending
What is the expected behavior
Check run completes
A simple solution we are using is to just make sure the comment fits in summary limits

but its not super pretty. I can implenent this solution if we want

cc @decyjphr

We are experiencing the same bug with pull requests that do not have any safe-settings files, and therefore validation sends the report.
Invalid request.
Only 65535 characters are allowed; 66662 were supplied.

HttpError: Invalid request.

Only 65535 characters are allowed; 66662 were supplied.
    at /opt/safe-settings/node_modules/@octokit/request/dist-node/index.js:86:21
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (internal/process/task_queues.js:95:5)
    at async sendRequestWithRetries (/opt/safe-settings/node_modules/@octokit/auth-app/dist-node/index.js:466:12)
    at async Job.doExecute (/opt/safe-settings/node_modules/bottleneck/light.js:405:18)

We have more than 20 repositories managed with safe-settings, and now it always fails. Furthermore, we created a workflow (we discuss it here) to cancel the status.


Add TS-PostDiff

Some Major Changes:

Change arms DataFrame in ArmData: Add three columns

count: number of assigned arm;
success: success of assigned arm, used in TS-PostDiff policy;
failure: failure of assigned arm, used in TS-PostDiff policy.

Add some columns in TS-Contextual simulations to show number of assigned arms.
Encapsulate evaluation to an Evaluator class object in metrics/evaluator.py.

Use Factory design pattern for choosing metrics for corresponding policy.

Add TS-PostDiff policy for simulations.

Some columns specified to TS-PostDiff simulation results: (1) success and failure for each arm; (2) number of assigned arms.
See policies/tspostdiff/ts_postdiff.py for TS-PostDiff algorithm details.

Move sample configs file to examples/ and add configs file for both TS-Contextual & TS-PostDiff simulations.

Note

Now "simulation" and "evaluation" in the sample configs file are names for two directories for storing simulation results and evaluation results, respectively.


Replace Glide with Coil

Closes #352

What's the impact on release APK size?

@topminipie Is right. I just ran a couple builds and got these results for APK size.
main - 2759KBs
coil - 2895KBs

What are the advantages of switching to Coil?

I generally prefer to use Coil in my Android apps nowadays because of the Kotlin-first syntax. Working with it in the Dolphin, Citra, and yuzu Android apps has been very plesant whether it be for loading images with the built-in options or creating custom loaders for the job.
As for performance or special features, I don't know of any advantages Coil has over Glide.

That's fair
I'll just leave this open if you ever become interested in it


chore: update ethereum-mainnet.md

Modify Network Subgraph Endpoints for Indexer-Agent and Indexer-Service

Old, closing


Riwyat website has changed

Source information
Riwyat
Steps to reproduce
The source doesn't work cause the web site has changed to
riwyatspace.com
Expected behavior
Its should work
Actual behavior
Its doesn't work
LNReader version
1.1.19
Android version
Android 11
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 1.1.19.
[X] I have updated all installed extensions.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

please update to beta 2.0


Compiling on HP-UX 11iv3 using the HP aCC compiler.

Hi, i was able to compile the -master branch on HP-UX 11iv3 (11.31) Itanium using the HP aCC compiler with the following changes:

Install GNU Make 4.2.1 from HP-UX porting archive and use "gmake" instead of "make"

Update "Makefile" as follows:

46,47d45
< REAL_CC=$(CC) -std=c99 -pedantic
< REAL_LD=$(CC)
54c52
< DYLIB_MAKE_CMD=$(REAL_LD) -shared -Wl,-soname,$(DYLIB_MINOR_NAME) -o $(DYLIBNAME) $(LDFLAGS)
---
> DYLIB_MAKE_CMD=$(CC) -shared -Wl,-soname,$(DYLIB_MINOR_NAME) -o $(DYLIBNAME) $(LDFLAGS)
70,78d67
< ifeq ($(uname_S),HP-UX)
<   WARNINGS=
<   DEBUG_FLAGS= -g
<   REAL_CFLAGS=$(OPTIMIZATION) +Z $(CFLAGS) $(WARNINGS) $(DEBUG_FLAGS) $(ARCH)
<   REAL_LDFLAGS+= -ldl -lnsl
<   REAL_LD=ld
<   DYLIB_MAKE_CMD=$(REAL_LD) -b -G -o $(DYLIBNAME) +h $(DYLIB_MINOR_NAME) $(LDFLAGS)
<   REAL_CC=$(CC) -AC99 +strict
< endif
167c156
<       $(REAL_CC) -c $(REAL_CFLAGS) $<
---
>       $(CC) -std=c99 -pedantic -c $(REAL_CFLAGS) $<

Update "net.h" as follows:

40c40
< #if defined(__sun) || defined(__hpux)
---
> #if defined(__sun)

Here's the compile output (redis server was not running)
SERVER:/hiredis-master# gmake clean
rm -rf libhiredis.so libhiredis.a hiredis-test hiredis.pc examples/hiredis-example* *.o *.gcda *.gcno *.gcov

SERVER:/hiredis-master# gmake
cc -AC99 +strict -c -O3 +Z   -g  net.c
"net.c", line 196: warning #4276-D: relational operator ">" always evaluates
          to 'false'
          if (msec < 0 || msec > INT_MAX) {
                               ^

cc -AC99 +strict -c -O3 +Z   -g  hiredis.c
cc -AC99 +strict -c -O3 +Z   -g  sds.c
cc -AC99 +strict -c -O3 +Z   -g  async.c
"async.c", line 66: warning #2191-D: type qualifier is meaningless on cast type
                                 sdslen((const sds)key));
                                         ^

"async.c", line 80: warning #2191-D: type qualifier is meaningless on cast type
      l1 = sdslen((const sds)key1);
                   ^

"async.c", line 81: warning #2191-D: type qualifier is meaningless on cast type
      l2 = sdslen((const sds)key2);
                   ^

cc -AC99 +strict -c -O3 +Z   -g  read.c
ld -b -G -o libhiredis.so +h libhiredis.so.0.13  net.o hiredis.o sds.o async.o read.o
ar rcs libhiredis.a net.o hiredis.o sds.o async.o read.o
cc -AC99 +strict -c -O3 +Z   -g  test.c
cc -O3 +Z   -g  -o hiredis-test   -ldl -lnsl test.o libhiredis.a
Generating hiredis.pc for pkgconfig...

SERVER:/hiredis-master# gmake test
./hiredis-test
#01 Format command without interpolation: PASSED
#02 Format command with %s string interpolation: PASSED
#03 Format command with %s and an empty string: PASSED
#04 Format command with an empty string in between proper interpolations: PASSED
#05 Format command with %b string interpolation: PASSED
#06 Format command with %b and an empty string: PASSED
#07 Format command with literal %: PASSED
#08 Format command with printf-delegation (int): PASSED
#09 Format command with printf-delegation (char): PASSED
#10 Format command with printf-delegation (short): PASSED
#11 Format command with printf-delegation (long): PASSED
#12 Format command with printf-delegation (long long): PASSED
#13 Format command with printf-delegation (unsigned int): PASSED
#14 Format command with printf-delegation (unsigned char): PASSED
#15 Format command with printf-delegation (unsigned short): PASSED
#16 Format command with printf-delegation (unsigned long): PASSED
#17 Format command with printf-delegation (unsigned long long): PASSED
#18 Format command with printf-delegation (float): PASSED
#19 Format command with printf-delegation (double): PASSED
#20 Format command with invalid printf format: PASSED
#21 Format command by passing argc/argv without lengths: PASSED
#22 Format command by passing argc/argv with lengths: PASSED
#23 Format command into sds by passing argc/argv without lengths: PASSED
#24 Format command into sds by passing argc/argv with lengths: PASSED
#25 Error handling in reply parser: PASSED
#26 Memory cleanup in reply parser: PASSED
#27 Set error on nested multi bulks with depth > 7: PASSED
#28 Works with NULL functions for reply: PASSED
#29 Works when a single newline (\r\n) covers two calls to feed: PASSED
#30 Don't reset state after protocol error: PASSED
#31 Don't do empty allocation for empty multi bulk: PASSED
#32 Returns error when host cannot be resolved: FAILED
#33 Returns error when the port is not open: PASSED
#34 Returns error when the unix_sock socket path doesn't accept connections: PASSED
#35 Don't fail when redisFree is passed a NULL value: PASSED
#36 Don't fail when freeReplyObject is passed a NULL value: PASSED

Testing against TCP connection (<IP_ADDRESS>:6379):
Connection error: Connection refused
gmake: *** [Makefile:158: test] Error 1

Will update ticket after performing functionality test.

Here's the rest of the output for testing a TCP connection:
Testing against TCP connection (xx:xx):
#37 Is able to deliver commands: PASSED
#38 Is a able to send commands verbatim: PASSED
#39 %s String interpolation works: PASSED
#40 %b String interpolation works: PASSED
#41 Binary reply length is correct: PASSED
#42 Can parse nil replies: PASSED
#43 Can parse integer replies: PASSED
#44 Can parse multi bulk replies: PASSED
#45 Can handle nested multi bulk replies: PASSED
#46 Successfully completes a command when the timeout is not exceeded: PASSED
#47 Does not return a reply when the command times out: FAILED
#48 Reconnect properly reconnects after a timeout: PASSED
#49 Reconnect properly uses owned parameters: PASSED
#50 Returns I/O error when the connection is lost: PASSED
#51 Returns I/O error on socket timeout:

And at the last test it hangs ... i left-it runfor almost a day and didn't finished - that's one heck of a timeout :)

I'm not sure if there's much we can do without access to a machine running HP-UX.
Going to close this since it's been nearly three years since the last activity but if it's still an issue feel free to reopen and we can try to figure out some kind of solution.


feat(aws-openapi-lambda): make names unique

Issue #, if available:
Description of changes:

Add the method id to the name of the Lambda function, so that a construct that creates 2 Lambda functions creates functions that can be identified from the name
Create a unique, stack specific name for each RestAPI rather than naming them all SpecRestApi

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: f90739c75ab84fe373eaafd83a3accfc992d98f5
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: 691fb103ea2376969f7894d6bd389c15539d62de
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: 441d3f8e803f1ec6095fc35d4369f3433e76ea30
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Release 1.4.0

Coverage increased (+0.006%) to 31.517% when pulling c4408057bbcf222d11b5f556231d7f0c5b863b44 on release_1.4.0 into d9269c4930760fd95f852915022c1b234ed0c17c on master.


Api error

Api error  when trying to search for anything

Api error  when trying to search for anything

Sorry to hear this is happening - is this occurring with the Youtui application, or ytmapi-rs library? Which version and OS are you on? (youtui -v for binary)

Windows 11 and this is youtui when i do -V my output is youtui 0.0.6

Unfortunately I've been unable to replicate so far, but merge of PR #78 may have resolved this.
If you're able to assist, could you please test locally?
git clone -b https://github.com/nick42d/youtui.git
cd .\youtui\youtui\
cargo r

Api error  new error


The error "go_vncdriver was installed without OpenGL support" is still existed

ubantu 14.04, python 2.7
I am trying to install universe, The error occurs: go_vncdriver was installed without OpenGL support.
The output of ./build.sh is
"zqc@zqc-P453UJ:~/go-vncdriver$ ./build.sh
Env info:
export GOPATH='/home/zqc/go-vncdriver/.build'
export CGO_CFLAGS='-I/home/zqc/anaconda2/lib/python2.7/site-packages/numpy/core/include -I/home/zqc/anaconda2/include/python2.7'
export CGO_LDFLAGS='/usr/lib/x86_64-linux-gnu/libjpeg.so -L/home/zqc/anaconda2/lib -lpython2.7'
export GO15VENDOREXPERIMENT='1'
Running build with OpenGL rendering.
Building with OpenGL: GOPATH=/home/zqc/go-vncdriver/.build go build -buildmode=c-shared -o go_vncdriver.so github.com/openai/go-vncdriver. (Set GO_VNCDRIVER_NOGL to build without OpenGL.)
github.com/openai/go-vncdriver
In file included from /home/zqc/anaconda2/include/python2.7/Python.h:8:0,
from .build/src/github.com/openai/go-vncdriver/main.go:5,
from $WORK/github.com/openai/go-vncdriver/_obj/_cgo_export.c:3:
/home/zqc/anaconda2/include/python2.7/pyconfig.h:1193:0: warning: "_POSIX_C_SOURCE" redefined [enabled by default]
#define _POSIX_C_SOURCE 200112L
^
In file included from /usr/include/stdlib.h:24:0,
from $WORK/github.com/openai/go-vncdriver/_obj/_cgo_export.c:2:
/usr/include/features.h:230:0: note: this is the location of the previous definition
define _POSIX_C_SOURCE 200809L"
^
The output of pip install -e ./ is:
"zqc@zqc-P453UJ:~/go-vncdriver$ pip install -e ./
Obtaining file:///home/zqc/go-vncdriver
Requirement already satisfied: numpy in /home/zqc/anaconda2/lib/python2.7/site-packages (from go-vncdriver==0.4.17)
Installing collected packages: go-vncdriver
Found existing installation: go-vncdriver 0.4.17
Uninstalling go-vncdriver-0.4.17:
Successfully uninstalled go-vncdriver-0.4.17
Running setup.py develop for go-vncdriver
Successfully installed go-vncdriver"
When I run the example of universe again in Pycharm with the interperter: /usr/bin/python2.7, the error " go_vncdriver was installed without OpenGL support." is still existed.
$which python
/home/zqc/anaconda2/bin/python
$ /usr/bin/env python -c 'import sys; print(sys.executable)'
/home/zqc/anaconda2/bin/python

I think this is fixed by the clarified instructions in https://github.com/openai/go-vncdriver/commit/1a818cd581d86bffdc16fe9cafe9bc1091874f75


Unable to complete packer build, vercmp and imds: command not found

Environment:

AWS Region: ap-southeast-1
Instance Type(s): c5a.4xlarge
EKS Platform version (use aws eks describe-cluster --name <name> --query cluster.platformVersion): AWS EKS 1.25
Kubernetes version (use aws eks describe-cluster --name <name> --query cluster.version): 1.25
AMI Version: amzn2-ami-kernel-5.10-hvm-2.0.*, we are using the latest
Kernel (e.g. uname -a):5.10.173-154.642.x86_64
Release information (run cat /etc/eks/release on a node): We are try to build an AMI using tag, v20230322

2023-04-01T00:35:06Z:     amazon-ebs: /tmp/script_4512.sh: line 330: vercmp: command not found
2023-04-01T00:35:07Z:     amazon-ebs: /tmp/script_4512.sh: line 370: vercmp: command not found
2023-04-01T00:35:07Z:     amazon-ebs: Loaded plugins: extras_suggestions, kernel-livepatch, langpacks, priorities,
2023-04-01T00:35:07Z:     amazon-ebs:               : update-motd, versionlock
2023-04-01T00:35:07Z:     amazon-ebs: Package amazon-ssm-agent-3.1.1732.0-1.amzn2.x86_64 already installed and latest version
2023-04-01T00:35:07Z:     amazon-ebs: Nothing to do
2023-04-01T00:35:07Z:     amazon-ebs: /tmp/script_4512.sh: line 506: imds: command not found

Hello, there good day!
We are having issue during the packer build, vercmp and imds command cannot be found. Could we get some insight on what package we need to install in the AL2 ami?

Found the place to copy the required file closing, thanks.

Similar issue here do you remember what you did by any chance?

These scripts: https://github.com/awslabs/amazon-eks-ami/tree/master/files/bin
Need to be on your PATH. The AMI template puts them in /usr/bin. If that isn't on your PATH, put them somewhere that is. 😄

Yeah thx I found it in the meantime! thx


ios click to send text will wrap instead of triggering  blurOnSubmit

Ask the bigwigs, TextInput component, multiline is true, and blurOnSubmit is false, ios click Send (enter)  the text will be wrapped without firing onSubmitEditing, there is no way to make it not to line but trigger onSubmitEditing (android is like this Effect). I do is a similar WeChat chat, click to send is sent, do not wrap.

@facebook-github-bot no-template


[IT-1801] Update lambdas

Update lambdas to use newer version of python.  This PR updates to AWS
development accounts only so we can test it before updating to prod
accounts.
depends on Sage-Bionetworks-IT/cfn-cr-sc-actions-provider/pull/17 Sage-Bionetworks-IT/lambda-send-budget-alert/pull/11 Sage-Bionetworks-IT/cfn-macro-ssm-param/pull/7 Sage-Bionetworks-IT/cfn-cr-oidc-idp/pull/2

@xschildw creating a tag will trigger the CI to build and deploy it to the S3 bucket in admincentral.  tags are created manually


Create account content updates

Fixed:

typos in email address validation
link and language style

@tracy-content can you take a look please?


How do you know whether more steps or more images create a more accurate result?

How do you know whether more steps or more images/annotations create a more accurate result?

If you stick to below 0.05 for steps, to get a better result, create more images/annotations.


Possible memory leak

When inserting 15,000 records, the memory usage climbed from ~140MB to ~165MB. Debug this possible memory leak. It may also be that Docker uses memory for its logs.

Not sure if this is the same but I get:
FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - process out of memory
Aborted (core dumped)

I'll send you the (docker) log that I have

Oh, it is probably that I did not update the docker image

Hmm with v0.8.0 I'm still getting the same error
><EMAIL_ADDRESS>start /data
> node lib/connector.js

{"name":"concava-connector-spul","hostname":"45b693abd7da","pid":18,"level":50,"type":"uncaught-exception","stack":"Error: read ETIMEDOUT\n    at exports._errnoException (util.js:874:11)\n    at TCP.onread (net.js:544:26)","msg":"Error: read ETIMEDOUT","time":"2016-04-30T18:02:05.958Z","v":0}

FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - process out of memory

This also happened in v0.7? I haven't seen this before and can only imagine that by not closing the socket in the connector, some variables are kept in memory, which causes a memory leak.
In the other log you've send me I saw that the SPUL connector uses ~1.4GB of RAM. This is a lot, since it normally uses ~65MB. Also unfortunately the ETIMEDOUT error still occurs.
I will be adding a DEBUG option today (which logs to /logs/debug.log) that'll log extensive (connection) information. This hopefully gives us some insights.
FYI: the logs are now handled by Bunyan and can be found in /logs/info.log.

Also: any thoughts on adding a timeout to the SPUL connector? Say: close socket after 30 seconds?

v0.9 is now running on spul.kukua.tech and d.kukua.cc. It appears that the connections aren't closed properly. I see the total amount of connections increasing and no disconnects.

I don't know about the timeout. It is strange that the server does not see the close.
Maybe the GPRS device tell me the socket is closed, but in reality it may be so that it has to negotiate this close with the server. I'll have to skim through the GPRS device manual to see if something like that could be happening.
Meanwhile a timeout of 30 seconds sounds good to me.

Since yesterday around 14:00 up until now the SPUL connector has 535 open sockets. However the memory usage rose from ~44MB to only ~51MB. This is on d.kukua.cc.

v0.10 is now running on spul.kukua.tech and d.kukua.cc. See .env.sample for the latest configuration options.

I stopped running my own SPUL connector.
It keeps running into
FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - process out of memory
This is with 0.10.0
I'm using d.kukua.cc right now. Let's see how that holds up ...


Add workflow to deploy dev

I mostly just copied over from qa.yml and adapted to dev.
Before merging we'll need:

confirmation that infra has been deployed
put NFCS_DEV_KEY and NFCS_DEV_SECRET in GitHub secrets

@jackmurdoch just checked and infra has been deployed on dev, so we should be ok to review and merge.
Where can I find NFCS_DEV_KEY and NFCS_DEV_SECRET to add to GitHub secrets?


Add densitiy scale

I added a overall matter density scale as a free parameter to the osc stage. So far it is just a float scaling the entire density profile.

41 commits? So weird how these still all show up for you..
Changes look fine and easy enough for me, would you care to write a test, checking the scaling does what it's supposed to?

yeah, I have no idea why all these old commits show up every time. Maybe I have to start with a fresh pisa fork to get rid of them.
Do you mean a test as part of pisa (like a unit test) or just a general test? (the second was already done by Sharmistha)

I meant a unit test checking just the functionality of this additional scaling.. should be pretty easy, but getting to know the unit tests of prob3 is probably the harder part :grimacing: - ok, snap, I don't see any tests for prob3.. that's not great :eyes:
That's not on you then, I'm merging..


Deploy hangs on empty Repo

I recently tried to a link an empty repo by accident. This just caused it to hang with a 500 error in the console.
This is obviously user error but just leaving this here in case you weren't aware. Feel free to close if this isn't where you'd like feedback.

Was this a newly created repo or a repo that was created but with no files?
Best to report in our discord if you want support (sounds like you are good?) https://discord.cyclic.sh/support


Fix invalid scala-cli-signing artifact downloads

Fixes #2036

it seems the CI won't pass until https://github.com/VirtusLab/scala-cli-signing/pull/33 gets merged...


Document the "default NaN values" in Parser

For example from docs (http://pandas.pydata.org/pandas-docs/stable/generated/pandas.io.parsers.read_csv.html):
keep_default_na : bool, default True
If na_values are specified and keep_default_na is False the default NaN values are overridden, otherwise they’re appended to

But it is not clear what those default values are.

@jreback : IINM, the docs now enumerate the default values.  IMO, this is no longer an issue.

yes this is ok

Closed by https://github.com/pydata/pandas/pull/14030


Update index.js

Changed product name in Index.js

Update Oasis Juice to Del monte Juice in Index.js


Change default apiVersion to "v1beta"

Setting the default apiVersion to "v1beta" matches the other GoogleAI server SDKs (Go and Python).
For web users, this will be slightly different from other client SDKs such as Swift and Kotlin, which default to "v1".

can you tell me then how to access the file upload and file list methods? using this Prompting with media files and here File prompting strategies


Adjust sync committee size and duration

If we merge #2370 then we will also want to tune the sync committee parameters to be friendly to light clients (including bridges, etc.) while still maintaining a floor for security.

@djrtwo good catch on the LIGHT_CLIENT_UPDATE_TIMEOUT, i've addressed that concern and a few other minor things in #2378.
that PR should be merged along w/ this one


Status?

Hey there, what's the status of this project? I maintain Stylus, and would love to see this integration make it. Currently working on v2, so if there's any particular reason why you stopped development on this (other than loss of interest, or perhaps it's stable?) I'd love to hear it!
SimenB/stylint#269

Hello, Simon! It's stable version, but a new release the reviewbot is coming soon . We support stylint v2.

Great! Shout out if there are any particular things making it difficult to integrate with


fix(calendar): add rule on picasso calendar styles component

SDE-462 :bug:
Description
On version 26.7.0 of Picasso there were implemented a couple of new props for DatePicker, a rule (today) on styles.ts file of Calendar component was removed but it was still being referenced, in results, on staff-portal jest tests were failing because of this lack.
How to test

On Staff-Portal run yarn test libs/filters/src/components/Filters/__tests__/date-range-filter.test.tsx
The tests are going to fail with Warning message Warning: [JSS] Could not find the referenced rule "today" in "PicassoCalendar"

Screenshots

Development checks

[x] Add changeset according to guidelines (if needed)
[x] Read CONTRIBUTING.md and Component API principles
[x] Make sure that additions and changes on design follow Toptal's BASE design, and it's been already discussed with designers at #-base-core
[x] Annotate all props in component with documentation
[x] Create examples for component
[x] Ensure that deployed demo has expected results and good examples
[x] Ensure the changed/created components have not caused accessibility issues. How to use accessibility plugin in storybook.
[x] Self reviewed
[x] Covered with tests

Breaking change

[N/A] codemod is created and showcased in the changeset
[N/A] test alpha package of Picasso in StaffPortal

All development checks should be done and set checked to pass the
GitHub Bot: TODOLess action

PR commands

List of available commands:

@toptal-bot run all - Run whole pipeline
@toptal-bot run build - Check build
@toptal-bot run deploy:documentation - Deploy documentation
@toptal-bot run package:alpha-release - Release alpha version

PR Review Guidelines

When to approve? ✅
You are OK with merging this PR and

You have no extra requests.
You have optional requests.

Add nit: to your comment. (ex. nit: I'd rename this variable from makeCircle to getCircle)

When to request changes? ❌
You are not OK with merging this PR because

Something is broken after the changes.
Acceptance criteria is not reached.
Code is dirty.

When to comment (neither ✅ nor ❌)
You want your comments to be addressed before merging this PR in cases like:

There are leftovers like unnecessary logs, comments, etc.
You have an opinionated comment regarding the code that requires a discussion.
You have questions.

How to handle the comments?

An owner of a comment is the only one who can resolve it.
An owner of a comment must resolve it when it's addressed.
A PR owner must reply with ✅ when a comment is addressed.

@toptal-anvil ping reviewers
@toptal-bot run required

@toptal-anvil ping reviewers

@toptal-anvil ping reviewers

@toptal-anvil ping reviewers


How to create custom dataset from colmap?

Dear author,
I have many images captured by the cellphone, and I can run colmap to obtain the extrinsics and intrinsics.  Can you tell me how to proceed to run the train and inference stage?  What is the key procedure to make colmap result work for SparseNeus?
Thanks so much !

Can you provide detailed information for custom dataset adaption in the readme.md ?


NNEF V-1.0.3 Support

NNEF has fixed minor bugs in the Spec and released a new version.

Match Model Compiler to V 1.0.3
Most of the fixed issues were filled by us, so not major changes
Test with the latest NNEF Version

@hansely any updates on this? Check for 1.0.3 support.

I updated nnef to the latest version and ran the profile script and all the models were passing.
The new updates in V1.0.3 includes changes in min_max_linear_quantize node which is not supported by current nnef_to_nnir script. It will be added in the future since we don't have any models with min_max_linear_quantize node.
Here is the result of profile script.
|alexnet-caffe   |  1|   6.176|   6.176
|alexnet-onnx    |  1|   4.220|   4.220
|vgg16-caffe     |  1|  60.477|  60.477
|vgg16-onnx      |  1|  12.445|  12.445
|vgg19-caffe     |  1|  48.834|  48.834
|vgg19-onnx      |  1|  14.153|  14.153
|googlenet-caffe |  1|  12.014|  12.014
|googlenet-onnx  |  1|   4.343|   4.343
|inceptionv1-onnx|  1|   4.957|   4.957
|inceptionv2-onnx|  1|   8.110|   8.110
|resnetv1-18-onnx|  1|   3.515|   3.515
|resnetv1-34-onnx|  1|   6.775|   6.775
|resnetv1-50-onnx|  1|   7.083|   7.083
|resnetv1-101-onnx|  1|  13.434|  13.434
|resnetv1-152-caffe|  1|  19.897|  19.897
|resnetv2-18-onnx|  1|   3.598|   3.598
|resnetv2-34-onnx|  1|   6.870|   6.870
|resnetv2-50-onnx|  1|   6.791|   6.791
|resnetv2-101-onnx|  1|  13.366|  13.366
|mobilenetv1-caffe|  1|   2.425|   2.425
|mobilenetv2-onnx|  1|   3.337|   3.337
|mobilenetv2-caffe|  1|   3.435|   3.435
|squeezenet1.0-caffe|  1|   7.862|   7.862
|squeezenet1.0-onnx|  1|   1.689|   1.689
|squeezenet1.1-caffe|  1|   5.090|   5.090
|squeezenet1.1-onnx|  1|   1.702|   1.702

V-1.0.3 supported


Attach .datepicker to dynamically created input

Hi,
How can I attach the datepicker to input elements that have been dynamically inserted to the site?

Hey @eriktobben, how are you inserting the input dynamically? Can you show some code?

Hi @pixelcommerce. Here is the code:
jQuery:
$('.new-line').on('click', function () {
    var source = document.getElementById("row-template").innerHTML;
    var template = Handlebars.compile(source);
    var html = template();

    $('.productcontainer').append(html);
});

HTML Handlebars template (the same code as rendered on page load):

        <input type="text" class="form-control date" name="delivery_date[]" autocomplete="off" />

So the datepicker is working great on the first row that in rendered on page load, but every new row that is created with JS does not work. I assume that it is because the DOM is changed and it does not pick up on that.

Why are you using native JS while inside jQuery like that?
Just go for:
$( '.productcontainer' ).append( $( '#row-template' ).html() );

And then:
$( '.date' ).datepicker({
  format: 'dd.mm.yyyy'
});

I assume you're trying to create an infinite record table for adding/updating multiple rows.
I'd recommend .clone or template literals for that matter.


Improve stylePanelLayout type

The documentation says Set a different layout render mode. Can be either 'integrated', 'compact', and/or 'circle'. So we can combine integrated and compact with circle?

Maybe there's also a way to allow the terms to be in different order 'circle integrated'?


Add generic tag prefix support

A generalized version of @robinbowes PR.
Without options the script functions as it is now.
Add -p <prefix> to have semtag derive the version from tags prefixed with <prefix>. This can be handy if multiple artifacts are released from the same repository. Or if you have local standards that require a different prefix.

This moves away from the semtag standard, any particlar reason for this?

I agree that the values returned by getcurrent and such are now not fully semver. But this is similar to the current situation with the 'v' prefix, refer to https://semver.org/#is-v123-a-semantic-version
The functionality added allows stripping of the 'v' prefix so it's easier to integrate with build tools (when I made this PR I had to always strip of the 'v' in jenkins scripts).
I also needed support to release multiple artifacts from one repository with their own version numbers so the prefix provides a namespace to achieve this. Since I already needed to strip the 'v' off, stripping off the different prefix was not that much extra trouble.
I fully agree it's better to have one versioned artifact per repository, it's messy if one tag has no meaning for one part of the repository and another for the other part of the repository. Sadly you sometimes cannot fix the environment you're in.
Would modifying the PR to strip of the prefix of the returned version make it acceptable? E.g. semtag -p 'artifactA-' getcurrent would return just the major.minor.patch-etc.
If that doesn't work I hope you would reconsider to at least merge the PR #6 . Not having the 'v' prefix saves some clutter in jenkins scripts to strip the 'v' off.