Non-obvious user code execution

If I'm reading the proposal correctly, the following code snippet would execute the getter:
const obj = {
  get foo() {
    console.log('user code execution!');
    return 5;
  },
};

// elsewhere

const { foo: void, ...obj1 } = obj;
// or
let obj2;
({ foo: void, ...obj2 } = obj);

I'm not convinced that discarding a binding should cause this kind of action-at-a-distance in the same way that assigning a binding does. Would it be possible to make a destructuring discard binding just skip over that property, instead of executing the getter and discarding the result?

If your goal is to avoid action-at-a-distance, you'll still run afoul of Proxy traps that can cause side-effects.
I don't believe we should special case void here as it means going from { x: void, y: Number } to { x: Number, y: Number } could result in different behavior at runtime. True, it may be bad to depend on such side-effects, but I'd prefer the path where side-effects are consistently applied vs. one where side-effect semantics are more complex.

If you consider alternate representations of void, they all indicate the property should be read:

Using _ in an AssignmentPattern:let _, obj2;
({ foo: _, bar: _, ...obj2 } = obj);

Using a custom matcher:const Any = { [Symbol.customMatcher]() { return true; } };

const { foo: Any, bar: Any, ...obj2 } = obj;

Using {} or not {}:const { foo: {} or not {}, bar: {} or not {}, ...obj2 } = obj;

All of these will trigger getters. void should be consistent here.

The analogy i'd apply is void f() still runs the function, it doesn't short-circuit.

To be clear, I don't feel that strongly about this, so if the answer is just no, that's fine with me.
But I do see void f() and const { x: void } = obj as two different things. I don't see side effects as an integral part of discard bindings the way I see them as an integral part of the void operator. (Without side effects, I don't think the void operator has any reason to exist, and also it's pretty obvious from reading void f() that f() is called right there; not hidden and nonlocal like in this case with discard bindings.)
A counter-analogy: While developing Temporal we were encouraged, particularly by implementations, to delete as many observable calls into user code as possible, in cases where the return value wasn't used or could be derived in another way (e.g., prevent Temporal.Now.plainDate(someCalendar).add({ days: 0 }) from calling someCalendar's dateAdd() method.)

ok, void obj.foo then :-p
i totally agree that observable calls into user code should be minimized! but if you've made a getter and destructuring from an object with one, i think there's already expected user code.

ok, void obj.foo then :-p

Yeah, fair. That is also nonlocal.
I was thinking of it more in terms of, the void operator would be how you'd more explicitly request the user code call, if the discard binding didn't do it:
const { foo: void, ...rest } = obj;
void obj.foo;  // yes, I really do want to execute the getter

but if you've made a getter and destructuring from an object with one, i think there's already expected user code.

I buy that that's expected in some use cases, yes. For the use case given in the explainer, "explicitly exclude properties from rest bindings without needing to introduce throw-away temporary variables for that purpose", I would not expect or want the excluded getter to be called. Like, "I want an object with only data properties, consisting of all the properties of obj except foo" I would otherwise write as a loop over the keys of obj, skipping foo, precisely because const { foo: _, ...rest } = obj; would execute the getter.
(But maybe that's a symptom of how my brain has been warped by diving too deep into JS's corners! I bet many people wouldn't anticipate a getter there, or write that loop, or care at all.)

I definitely see that possible interpretation; i love the idea of preventing getter execution, but in JS i've somewhat resigned myself to the reality that the only way to prevent that is to never define getters :-)


Contribution Guidelines page prepared

This guide should help contributors understand the steps needed to contribute to the project.

Thanks @neslihanatasever, merged. 🎉


Linear scheduler in multi-gpu training

So the Linear Scheduler for linearly increasing distortion (similar to dropblock) would in no way work for multi-gpu training since it uses a simple variable i (not tensor) so when we do the following
def step(self):
        if self.i < len(self.drop_values):
            self.disout.dist_prob = self.drop_values[self.i]
        self.i += 1

The value of i will never get updated. You can try if you want. My question is, how did you guys run this code to train imagenet and got those results?

The code works well on our device. Could you provide more detailed information?

unless you're running it on a single gpu, the value of i will never be updated. It would remain 0 or change between 0 and 1 as it is not a tensor. It's a known issue and you can read more about it at,
https://github.com/kjunelee/MetaOptNet/issues/41

In the code of disout, the update of i is not in the function 'forward()'.  It is updated in function 'train' of file 'train_imagenet.py '. Thus the problem in the mentioned issue does not occur.

Got it. Thanks


DDIM example - Can't load back the weights

Hello,
First of all, thank you for this example, it is exactly what i was looking for in my ML discovery !
I just have one problem though.
I've followed the code from the keras' website examples section, and ran a training for 500 epochs, saving the weights at every epoch with a callback using a ModelCheckpoints function (set to weights_only_true).
I ended up with a directory looking exactly like the one provided in the example, yet I can't load it back to generate images (or continue training).
On calling the load_weights method, i get either a Unable to open file (file signature not found) , when writing model.load_weights
or, if writing model.load_weights (without the ema_network), I get Unable to load weights saved in HDF5 format into a subclassed Model which has not created its variables yet. Call the Model first, then load the weights.
For more details, my colab is accessible here
Does anyone have an idea where this might be coming from and how to resolve it?
Thanks !

Hello Sachin,
Thanks for your answer.
In the meantime, I successfully loaded back the weights, using the following syntax:
model.load_weights('/content/drive/MyDrive/1.MelodiaAtomizacji/SenaTrainings/23.08.24 - Training 2/model_cp/senaDiff.ckpt')
I guess the error was in my writing of the filepath.
I am now trying to customize this notebook by implementing a DDIM-type model for the generating part, and using it to perform some interpolated generations (going between noise samples). I haven't managed to do either, but this isn't a bug related to the example anymore !

I just had to use the proper filepath syntax


arch: Rename _exit to up_exit to follow the naming convention

Summary
Impact
Testing

Humm..   It was probably a mistake to merge this one.  _exit is the standard name for this function.  See https://linux.die.net/man/2/_exit

Humm.. It was probably a mistake to merge this one. _exit is the standard name for this function. See https://linux.die.net/man/2/_exit

_exit still exist here:
https://github.com/apache/incubator-nuttx/pull/1194/files#diff-e83159a0d25bf3928fb3d29123fe383bR53
The difference is that the arch layer implement up_exit instead _exit. The change make the arch layer more consistent.


Explore options to reduce boilerplate code in config props

Can we use Java Record for config props now?
What are our thoughts about Lombok.

This is just an improvement ticket.

Looks like using Java records is supported now: https://github.com/spring-projects/spring-boot/wiki/Spring-Boot-2.6-Release-Notes#records-and-configurationproperties
The only thing I dislike is that we would have to put a lot of fields into the constructor if I understand it correctly

Oh that is great news @alpreu .
I am ok w/ having to add the fields in the <init> as we need to doc them so that the doc gen stuff works. (IOW - we would have to do that somewhere anyways). I think getting rid of the get/set would be a huge improvement.

Suggestion Let's pause this for now. Once SB supports descriptions on the record fields, we will continue.

The config props have moved into Spring Boot proper and any improvements or modification will be handled there on a user requested/needed basis.


Replace reshaping workaround with ReshapePorts

In the absence of ReshapePorts in Lava, we have implemented reshaping with dedicated Reshape Processes (ReshapeBool, ReshapeInt) and changed the implementation of the connect() function to insert these Processes:
Population (Source) -> ReshapeBool -> Dense -> ReshapeInt -> Population (Destination)
Once ReshapePorts are available, use them and remove the Reshape Processes.

In connect.py, replace

    # make connections from the source port to the connections process
    # TODO (MR) workaround in absence of ReshapePorts
    con_ip = connections.s_in
    rs1 = ReshapeBool(shape_in=src_op.shape, shape_out=con_ip.shape)
    src_op.connect(rs1.s_in)
    rs1.s_out.connect(con_ip)

    # make connections from the connections process to the destination port
    # TODO (MR) workaround in absence of ReshapePorts
    con_op = connections.a_out
    rs2 = ReshapeInt(shape_in=con_op.shape, shape_out=dst_ip.shape)
    con_op.connect(rs2.s_in)
    rs2.s_out.connect(dst_ip)

by
    # make connections from the source port to the connections process
    src_op_flat = src_op.reshape(connections.in_ports.s_in.shape)
    src_op_flat.connect(connections.in_ports.s_in)

    # make connections from the connections process to the destination port
    con_op = connections.out_ports.a_out.reshape(dst_ip.shape)
    con_op.connect(dst_ip)

In test_connect.py, replace

        # check whether 'source' is connected to 'connections'
        src_op = source.out_ports.s_out
        con_ip = connections.in_ports.s_in
        # TODO (MR): remove this after switching to Reshape ports
        rs1_op = src_op.get_dst_ports()[0].process.out_ports.s_out
        self.assertEqual(rs1_op.get_dst_ports(), [con_ip])
        self.assertEqual(con_ip.get_src_ports(), [rs1_op])

        # check whether 'connections' is connected to 'target'
        con_op = connections.out_ports.a_out
        dst_op = destination.in_ports.a_in
        # TODO (MR): remove this after switching to Reshape ports
        rs2_op = con_op.get_dst_ports()[0].process.out_ports.s_out
        self.assertEqual(rs2_op.get_dst_ports(), [dst_op])
        self.assertEqual(dst_op.get_src_ports(), [rs2_op])

by
        # check whether 'source' is connected to 'connections'
        src_op = source.out_ports.s_out
        con_ip = connections.in_ports.s_in
        self.assertEqual(src_op.get_dst_ports(), [con_ip])
        self.assertEqual(con_ip.get_src_ports(), [src_op])

        # check whether 'connections' is connected to 'target'
        con_op = connections.out_ports.a_out
        dst_op = destination.in_ports.a_in
        self.assertEqual(con_op.get_dst_ports(), [dst_op])
        self.assertEqual(dst_op.get_src_ports(), [con_op])

Remove directories lava/lib/dnf/connect/reshape_* including all content.

Fixed by https://github.com/lava-nc/lava-dnf/pull/31


Snow forming at low altitude inside ship

As of 0.9_prerelease_5, snow is forming on the ship while it's raining and the ship is at ocean level.
In vanilla, snow would only form if the block is high enough in a cold biome.

Fixed in the shown commit.


Yard rake task is missing require

Steps to reproduce
The yard rake task is not self-contained and is missing requires to yard itself.

add yard to rakefile:require "yard/rake/yardoc_task"
YARD::Rake::YardocTask.new

verify yard rake task:$ rake -T
# snip
rake yard                     # Generate YARD Documentation

run rake yard task

Actual Output
```
$ rake yard
rake aborted!
NameError: uninitialized constant YARD::CLI

          yardoc = YARD::CLI::Yardoc.new
                       ^^^^^

Tasks: TOP => yard
(See full trace by running task with --trace)
```

Expected Output
I expected roughly the same output as yard
It seems the yard task is missing requires.
Environment details:

OS: macos 15.2
Ruby version (ruby -v): ruby 3.1.6p260 (2024-05-29 revision a777087be6) [arm64-darwin23]
YARD version (yard -v): yard 0.9.37
Relevant software dependency/versions: rake, version 13.2.1

I have read the Contributing Guide.

This is by design. You should not be reaching into and requiring any files from the YARD package other than require 'yard' itself:
require 'yard'
YARD::Rake::YardocTask.new

as documented in the README.

Understood. I saw that when reading up on install, but assumed it was the "simple, but malperformant" setup that naive rake users typically fall into.
It's common to see many gems document the "simple" setup that requires their entire lib in the rakefile, only to later discover the performance impact that has on shell completion (rake <tab>) or when gem A negatively impacts gem B's completely unrelated rake task (because gem A is being fully loaded even when not run).
The common convention is to have a rake task entrypoint that only defines the task, without requiring the library. (gems that do this include: rdoc, bundler, rspec,  and rubocop)
The way I ended up defining the task was:
require "yard/rake/yardoc_task"
YARD::Rake::YardocTask.new :yard do |t|
  t.before = -> { require "yard" }
end

Comparing the two approaches (the above vs: require "yard"; YARD::Rake::YardocTask.new)
yielded:
  %   cumulative   self              self     total
 time   seconds   seconds    calls  ms/call  ms/call  name
  0.02     0.02      0.00        1     0.01    16.56  Object#full_yard
  0.04     0.02      0.00        2     0.00     0.21  Object#just_task

Would you consider a request to have an "official" rake entrypoint that didn't set up all of yard? (I realize it's only configuring autoloads and a few things, but it's quite a bit more than the tasklib needs.)
I would find this to also be helpful for consumers who look for patterns in Rakefiles and, seeing require "yard" could mistakenly conclude that require _gem_ is normal or good practice.

Still about 2 orders of magnitude.

How are you generating this benchmark? This isn't something I can reproduce. Something looks very off about those numbers given that I've never seen a sub-ms require call. Are you sure your benchmark isn't requiring an already-required file? This would be the case if you ran a bench for require "yard"; require "yard/rake/yardoc_task" within the same process.
Here's what I see:
❯ ruby -rrake -e "t=Time.now;require 'yard/rake/yardoc_task';p Time.now - t"
0.0276671

❯ ruby -rrake -e "t=Time.now;require 'yard';p Time.now - t"
0.0639603

Running this a number of times is fairly consistent-- requiring the task file directly takes anywhere between 2.5ms - 4ms, while requiring yard fairly consistently is ~6.5ms. In the "worst case" performance it's about 2-2.5x faster, but keep in mind we're talking about differences of ~2-3ms in a use case that is absolutely not going to be a tight loop.
There is probably a risk of disk cache miss where loading YARD can take a little longer, but usually this only happens once and accounts for a very small total cost of your total runtime. If the concern is that YARD will slow down the running of other tasks, then this one time possible disk cache miss is unlikely to be the primary contributor-- an extra ~20ms every 100 or so runs doesn't seem significant for the benefit of significantly improved backwards compatibility guarantees.
tl;dr you're free to use the "unsupported" path of directly requiring into YARD if you really need to tightly optimize your Rakefile load time. I think the approach you're using is perfectly reasonable if the load time is that stark on your machine. I don't see enough of a performance gap to justify increasing YARD's maintenance surface area for the whole project, though.


Angular 2 ?

Not familiar with Angular 1.x so I'm having a helluva time figuring our what applies anymore ;(
I need to start a new project using videojs and angular 2.
Shall I go bark up some other tree?

The next big update for vjs-video will be angular 2 support. In the meantime, video.js is already set up to use ES6 which should help with integration. I hope to have the new release in a couple of weeks.

Hi @LonnyGomes ,
any updates on Angular 2 support please?
Cheers

I have some thing working in dev, perhaps I an release a beta version within the next few days.


Lets keep the DCIDs same, even if wikidataId has changed

Refer - https://github.com/datacommonsorg/data/pull/743
This exception is required only for Warangal and Malerkotla.

District
WikiDataID Previous
Exists on DC
WikiDataID New
Exists on DC
Map to the existing DC Node

Warangal District
Q28169759
Yes
Q15399
No
Keep the same DCID and update the node with wikidataId

New Delhi District
Q110067863
No
Q8560886
Yes
Update the Node with just LGD code. No special care is required.

Malerkotla District
Q1470987
Yes
Q107016021
No
Keep the same DCID and update the node with wikidataId

Very old PR. Hence closing it.


Adaptive layout doesn't show player controls

Describe the bug
When resizing the window to the smallest layout, it hides the player and shows only the user's playlists. To switch, pause, resume music etc the user has to resize the window again so it shows the music playing.
To Reproduce
Steps to reproduce the behavior:

Play any music
Resize horizontally the window, until it has only one content column
Fail to see controls of the currently playing music

Expected behavior
A miniplayer should be visible
Videos
https://user-images.githubusercontent.com/71804353/166981325-207097ff-cd76-4eb2-b57e-ba7aca64b0df.mp4
General information:

Distribution: Fedora 34
Installation method: Flatpak from Flathub
Version: 0.3.3

sorry missed your issue, I'll move discussion over to #521 if you don't mind!


add rresult & tcpip library dependencies to _tags

Without these additions, applications attempting to use charrua-core's libraries can't build binaries easily:
$ ./build.sh
Finished, 0 targets (0 cached) in 00:00:00.
+ ocamlfind ocamlopt -linkpkg -g -package rawlink -package cmdliner -package ipaddr -package tuntap -package cstruct -package cstruct.lwt -package cstruct.unix -package lwt.unix -package charrua-core.server src/charruad.cmx -o src/charruad.native
File "_none_", line 1:
Error: No implementations provided for the following modules:
         Rresult referenced from /home/user/.opam/dhcharrua-client/lib/charrua-core/dhcp_server.cmxa(Dhcp_wire)
         Ipv4_packet referenced from /home/user/.opam/dhcharrua-client/lib/charrua-core/dhcp_server.cmxa(Dhcp_wire)
         Ethif_packet referenced from /home/user/.opam/dhcharrua-client/lib/charrua-core/dhcp_server.cmxa(Dhcp_wire)
         Ipv4_wire referenced from /home/user/.opam/dhcharrua-client/lib/charrua-core/dhcp_server.cmxa(Dhcp_wire)
         Udp_packet referenced from /home/user/.opam/dhcharrua-client/lib/charrua-core/dhcp_server.cmxa(Dhcp_wire)
         Ethif_wire referenced from /home/user/.opam/dhcharrua-client/lib/charrua-core/dhcp_server.cmxa(Dhcp_wire)
         Udp_wire referenced from /home/user/.opam/dhcharrua-client/lib/charrua-core/dhcp_server.cmxa(Dhcp_wire)
Command exited with code 2.
Compilation unsuccessful after building 4 targets (0 cached) in 00:00:00.

Can you break on 80columns please ?

Sorry, should have been more specific, can you squash them into one commit :D ?

Bonus typo fix in commit message, since you asked so nicely. :)

Awesome, thanks ! I'll wait for travis and then integrate.

Hmm it won't build with 4.02.3 because of result, is mirage3 dropping 4.02.3 support ?

Yes, partly because we're using result all over the place.  Still, it's your library -- I can add the dependency on result as well if you like.

Argh, don't merge this until I push a new fix; ocamlfind fails on the line break:
+ ocamlfind ocamldep -package rawlink -package cmdliner -package ipaddr -package tuntap -package cstruct -package cstruct.lwt -package cstruct.unix -package lwt.unix -ppopt -lwt-debug -package charrua-core.server -modules src/charruad.ml > src/charruad.ml.depends
ocamlfind: While parsing '/home/user/.opam/dhcharrua-client/lib/charrua-core/META': '=' must be followed by a string constant in line  at line 4 position 11

Sorry for all the churn; hopefully this is aesthetically and computationally acceptable.  (FWIW ocamlfind lint lib/META turns up some more suggestions, but you can choose to care about them or not as you find appropriate.)

sorry, it's all my fault -- in mirage-vnetif I just re-added 4.02 support (by requiring + opening Result) https://github.com/MagnusS/mirage-vnetif/pull/9  once that is in, travis should be slightly more happy...

So I should just wait for mirage-vnetif go in ?

once that is in, you can restart those travis thingies and they might work.. :)


body tag cannot be inside .sentinl div

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


synthv1: 0.9.0 -> 0.9.2

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build synthv1

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: synthv1
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: synthv1
Partial log (click to expand)
shrinking /nix/store/qj4qwjas64vh89hsrkxfg52wsf2j6alj-synthv1-0.9.2/lib/libsynthv1.so.0.9.2
shrinking /nix/store/qj4qwjas64vh89hsrkxfg52wsf2j6alj-synthv1-0.9.2/bin/synthv1_jack
gzipping man pages under /nix/store/qj4qwjas64vh89hsrkxfg52wsf2j6alj-synthv1-0.9.2/share/man/
/nix/store/vp40my263xk32f07788srrgl59rycsna-binutils-2.30/bin/strip is /nix/store/vp40my263xk32f07788srrgl59rycsna-binutils-2.30/bin/strip
stripping (with command /nix/store/vp40my263xk32f07788srrgl59rycsna-binutils-2.30/bin/strip and flags -S) in /nix/store/qj4qwjas64vh89hsrkxfg52wsf2j6alj-synthv1-0.9.2/lib  /nix/store/qj4qwjas64vh89hsrkxfg52wsf2j6alj-synthv1-0.9.2/bin
patching script interpreter paths in /nix/store/qj4qwjas64vh89hsrkxfg52wsf2j6alj-synthv1-0.9.2
checking for references to /build in /nix/store/qj4qwjas64vh89hsrkxfg52wsf2j6alj-synthv1-0.9.2...
postPatchMkspecs
postPatchMkspecs
/nix/store/qj4qwjas64vh89hsrkxfg52wsf2j6alj-synthv1-0.9.2

Success on aarch64-linux (full log)
Attempted: synthv1
Partial log (click to expand)
shrinking /nix/store/5lv352q9xal5aiyr2qaz18hfhkxwfc0s-synthv1-0.9.2/lib/libsynthv1_ui.so.0.9.2
shrinking /nix/store/5lv352q9xal5aiyr2qaz18hfhkxwfc0s-synthv1-0.9.2/lib/libsynthv1.so.0.9.2
gzipping man pages under /nix/store/5lv352q9xal5aiyr2qaz18hfhkxwfc0s-synthv1-0.9.2/share/man/
/nix/store/8wl4dykwswy42wxj671b6k7vk2ydxg86-binutils-2.30/bin/strip is /nix/store/8wl4dykwswy42wxj671b6k7vk2ydxg86-binutils-2.30/bin/strip
stripping (with command /nix/store/8wl4dykwswy42wxj671b6k7vk2ydxg86-binutils-2.30/bin/strip and flags -S) in /nix/store/5lv352q9xal5aiyr2qaz18hfhkxwfc0s-synthv1-0.9.2/lib  /nix/store/5lv352q9xal5aiyr2qaz18hfhkxwfc0s-synthv1-0.9.2/bin
patching script interpreter paths in /nix/store/5lv352q9xal5aiyr2qaz18hfhkxwfc0s-synthv1-0.9.2
checking for references to /build in /nix/store/5lv352q9xal5aiyr2qaz18hfhkxwfc0s-synthv1-0.9.2...
postPatchMkspecs
postPatchMkspecs
/nix/store/5lv352q9xal5aiyr2qaz18hfhkxwfc0s-synthv1-0.9.2


refactor(PX-4712): shuffle around the shipping options display to match the design

Ended up not adding space between the title and the description as this spacing seems fine now when things are in a table format.
Looks like:

Oh and for the record that was a reference figma: https://www.figma.com/file/J5bIQj4a0l7wYAHnhbVGTp/[Collector]-Checkout-Shipping?node-id=1761%3A0 but the spacing there is not consistent at all so I just did not do anything spacing wise.


Clair container failed to start due to permisson deny

Root cause it the umask is '027' on standalone ova.

https://github.com/vmware/harbor/pull/3652


refactor np mcmc, pass verbose arg.

pass show_progress_bar as verbose arg to slice_np to avoid showing pbars all the time.

Codecov Report

Merging #382 into main will increase coverage by 0.07%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #382      +/-   ##
==========================================
+ Coverage   74.69%   74.76%   +0.07%
==========================================
  Files          49       49
  Lines        2845     2845
==========================================
+ Hits         2125     2127       +2
+ Misses        720      718       -2

Flag
Coverage Δ

unittests
74.76% <100.00%> (+0.07%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

sbi/inference/posteriors/base_posterior.py
64.59% <ø> (ø)

sbi/mcmc/slice_numpy.py
78.49% <100.00%> (+2.15%)
:arrow_up:

Continue to review full report at Codecov.

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


employee_attendance_automator

Pull Request Template
script name - employee_attendance_automator
Complete end-to-end project. I have worked on it for 6 months. I can guarantee that there will be no bugs at all.
Issue no.(#380  ) -
Self Check(Tick After Making pull Request)

[x] This issue was assigned to me.
[x] One Change in one Pull Request
[x] My file is in proper folder (Name of folder should be in lowercase with no space in between)
[x] I am following clean code and Documentation and my code is well linted with flake8.
[x] I have added README.md and requirements.txt with my script

If issue was not assigned to you Please don't make a PR. It will marked as invalid.

Please approve my request i can also provide demo samples if needed.


Move the connection algorithm to the protocol crate

Moves the non-I/O details of RustConnection::connect_to_stream_with_auth_info to the x11rb_protocol crate, as struct Connect (not attached to the name). Connect is used to keep track of the state of the handshake between the client and X11 server.

Thanks for this! I like this idea and only left some minor comments.


Update doc

Just want to point out that I had to merge the branch with master first. For some reason the version number was 1.4.7

Codecov Report

Merging #436 (35c4f66) into devel (0a98355) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##            devel     #436   +/-   ##
=======================================
  Coverage   39.97%   39.97%
=======================================
  Files          75       75
  Lines       25154    25154
=======================================
  Hits        10056    10056
  Misses      15098    15098

Impacted Files
Coverage Δ

tofu/version.py
100.00% <100.00%> (ø)

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 0a98355...35c4f66. Read the comment docs.


Output vendor interface

I started to define an interface for the output vendors.
I created a Json exporter, and now it is shown in the UI, but it still doesn't work- the next step is to remove the MainTable and replace it with a big run button.
So I think we can merge this, and continue to work on

Implement the interface for the rest of the output vendors (it will require more decisions about the interface)
See all the TODOs I wrote, maybe before merging this PR.
Create a big run button.
Implement the full process- from importing to exporting.

I will start with the TODOs and then with the run.
Maybe you @brafdlog can convert the output vendors to the new interface, because you wrote them.

@baruchiro As we discussed this pr is now in a state where the application is working and ts is set correctly. I suggest we merge it and start a new branch for the output vendors and types. WDYT?

Right, let's merge it


Using PProgress.fn progress() always returns 1

Example code:
const got = require("got");
const PProgress = require('p-progress');
const { createWriteStream } = require("fs");

const settings = require("../settings.json");

const url = "https://media0.giphy.com/media/4SS0kfzRqfBf2/giphy.gif";
const name = "image.gif";

const fileDownload = PProgress.fn(async (url, name, progress) => {
    const downloadStream = got.stream(url);
    const fileWriterStream = createWriteStream(`${settings.download_target}/${name}`);

    downloadStream
        .on("downloadProgress", ({ transferred, total, percent }) => {
            const percentage = Math.round(percent * 100);
            // console.log(percent);
			// progress(percent);
			// progress(0.01);
            progress(1/10);
            // console.error(`${name} progress: ${transferred}/${total} (${percentage}%)`);
        })
        .on("error", (error) => {
            console.error(`Download failed: ${error.message}`);
            throw new Error(`Download Failed: ${error.message}`);
        });

    fileWriterStream
        .on("error", (error) => {
            console.error(`Could not write file to system: ${error.message}`);
            throw new Error(`File Writing Failed: ${error.message}`);
        })
        .on("finish", () => {
            console.log(`File downloaded to ${name}`);
            return true;
        });

    downloadStream.pipe(fileWriterStream);
});

(async () => {
	const progressPromise = fileDownload(url, 'Gather rainbows');

	progressPromise.onProgress(console.log);

	await progressPromise;
})();

No matter what I've tried it always returns 1 a single time and then doesn't report anything again. Am I missing something?
Node: v14.15.1
Environment: Ubuntu in WSL2 on Windows 10 (application is run in Ubuntu though)

Moving my progress() reporting outside the stream ons does have it return the value, but it still immediately jumps to 1 without a second input.
i.e.
const got = require("got");
const PProgress = require('p-progress');
const { createWriteStream } = require("fs");

const settings = require("../settings.json");

const url = "https://media0.giphy.com/media/4SS0kfzRqfBf2/giphy.gif";
const name = "image.gif";

const fileDownload = PProgress.fn(async (url, name, progress) => {
    const downloadStream = got.stream(url);
    const fileWriterStream = createWriteStream(`${settings.download_target}/${name}`);

	progress(1/10); <-- Returns 0.1, but without there being a second reporting it also returns 1 immediately after.

    downloadStream
...

Once a promise resolves with a value, its progress will be set to 1 as it assumes the operation is complete:
https://github.com/sindresorhus/p-progress/blob/402ad0fdd8e2490ee6e2919ecb8dfa2f575fceab/index.js#L78-L81


OP-Request-Unsinged test is failing due to ConnectOAuth2RequestFactory implementation

As per the OpenId spec if request not found error should be display "request_not_supported" but i am not able to find the error message in current implementation, Hence Op-request-unsigned test is always failing. May be due to below condition
if (client.getRequestObjectSigningAlg() == null) {
throw new InvalidClientException("Client is not registered for unsigned request objects (no request_object_signing_alg registered)");
} else if (!client.getRequestObjectSigningAlg().equals(Algorithm.NONE)) {
throw new InvalidClientException("Client is not registered for unsigned request objects (request_object_signing_alg is " + client.getRequestObjectSigningAlg() +")");
}

OP-request-unsigned test result
AuthorizationRequest
[ERROR] ResponseError:Missing or faulty response
Result
PARTIAL RESULT


Implement connection manager

The Issue
Per a discussion on call this week, we don't have anything currently handling/optimizing for multiple downloads at the same time. The connection manager would support concurrency and efficiency when finding peers.
System Configuration

LBRY Daemon version:
LBRY App version:
LBRY Installation ID:
Operating system:

Anything Else

Screenshots

Internal Use
Acceptance Criteria

Definition of Done

[ ]  Tested against acceptance criteria
[ ] Tested against the assumptions of user story
[ ] The project builds without errors
[ ] Unit tests are written and passing
[ ] Tests on devices/browsers listed in the issue have passed
[ ] QA performed & issues resolved
[ ] Refactoring completed
[ ] Any configuration or build changes documented
[ ] Documentation updated
[ ] Peer Code Review performed

nice to have: separate connection opening from blob requests
why: so peers will take the time it needs to establish a connection before being available to download, thus improving the speed score and peer picking since the time to open the connection wont favor ongoing slower peers.


Consider support/handling of System.Net

http://forums.bridge.net/forum/general/open-discussions/1320

Some basic support for System.Net would be helpful along with System.Net.WebSockets

For System.NET I am mainly missing the URI class. I think the ClientWebSocket class can simply wrap the built in browser WebSocket as it doesn't implement non-async members.

Relates to #776.

MSDN ClientWebSocket Class
Underlying implemention according to MDN WebSockets


Playlist move item doesn't seem to work

Version Number
2.0.2 (Docker: master)
What branch are you on?
master
Describe the Bug
After the first creation of the playlist, any attempt to reprocess and move items into the correct placement results in the following error, regardless of playlist:
[2024-10-10 15:07:13,844] [plex.py:645]               [ERROR]    | (404) not_found; (redacted)/playlists/23193/items/None/move <title>Not Found</title>
404 Not Found
 |
[2024-10-10 15:07:13,844] [builder.py:3505]           [ERROR]    | Failed to Move Constantine S01E01: Season 1: Episode 1                                             |

Relevant Collection/Overlay/Playlist Definition
playlist_files:
- file: config/playlists.yml
- default: playlist
  template_variables:
    libraries: Movies, TV Shows
    radarr_add_missing: true
    sonarr_add_missing: true

playlists:
  One Chicago (Timeline Order):
    sync_mode: sync
    libraries: TV Shows
    trakt_list: https://trakt.tv/lists/5415197
    summary: One Chicago
    radarr_add_missing: true
    sonarr_add_missing: true
  The Walking Dead (Timeline Order):
    sync_mode: sync
    libraries: TV Shows
    trakt_list: https://trakt.tv/lists/25146121
    summary: The Walking Dead
    radarr_add_missing: true
    sonarr_add_missing: true

### Logs

_No response_

I can't reproduce an issue here with Kometa 2.0.2.
I added each of the five shows in the Chicago playlist one at a time to a test library, running this playlist after each, so every run would be adding new episodes and inserting them into the playlist.
Each run went fine with no errors; the playlist was updated as expected each run.
I then ran twice after everything was in the library and playlist with no errors.

Just repeated the experiment with Kometa 2.1.0 with the same result; no issues seen.

What's causing the issue here is that None should be an internal Plex ID:
(redacted)/playlists/23193/items/None/move

This is telling Plex to move an item with the ID of None in the playlist with ID 23193, which of course is going to fail and lead to: 404 Not Found.
For some reason, Plex is having trouble getting its own ID for that item.

Any idea what to look for and where to see if something is happening on the plex side?


onAudioDeviceVolumeChanged need deviceId

Need deviceId for device MediaDeviceType.audioRecordingDevice to handle volume recording. Now when change one in all recording we will receive callback onAudioDeviceVolumeChanged but don't check recording selected change volume.

Thank you. I am going try this.


Automated cherry pick of #8347: fix: remote update metadata may cause metadata update loop

Cherry pick of #8347 on release/3.4.
#8347: fix: remote update metadata may cause metadata update loop

Codecov Report

Merging #8348 into release/3.4 will decrease coverage by 2.67%.
The diff coverage is 0.00%.

@@              Coverage Diff               @@
##           release/3.4   #8348      +/-   ##
==============================================
- Coverage         6.22%   3.54%   -2.68%
==============================================
  Files              759     303     -456
  Lines           117145   62249   -54896
==============================================
- Hits              7289    2205    -5084
+ Misses          109197   59891   -49306
+ Partials           659     153     -506

Flag
Coverage Δ

#aFlag
3.54% <0.00%> (-2.68%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/appsrv/stats.go
0.00% <0.00%> (ø)

pkg/compute/models/buckets.go
1.10% <0.00%> (+0.03%)
:arrow_up:

pkg/compute/models/cachedimages.go
1.70% <0.00%> (-0.11%)
:arrow_down:

pkg/compute/models/capabilities.go
0.00% <0.00%> (ø)

pkg/compute/models/cloudaccounts.go
0.48% <0.00%> (-4.66%)
:arrow_down:

pkg/compute/models/dbinstance_skus.go
2.83% <0.00%> (-0.02%)
:arrow_down:

pkg/compute/models/dns_recordsets.go
1.82% <0.00%> (-0.24%)
:arrow_down:

pkg/compute/models/dns_zones.go
1.62% <0.00%> (-0.06%)
:arrow_down:

pkg/compute/models/elasticips.go
1.08% <0.00%> (-0.01%)
:arrow_down:

pkg/compute/models/guest_actions.go
0.00% <0.00%> (ø)

... and 449 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 41bdfff...fbf62b2. Read the comment docs.

/lgtm
/approve


Fix STOARGE_ENCRYPTED type on create playbook variable

Bug Report
Type in STOARGE_ENCRYPTED variable in https://github.com/finos/cfi-ansible-aws-rds/blob/main/ansible/create-rds-db.yaml
The playbook works - this is just to fix spelling.

raised this PR #27


[6.4.x] [DROOLS-1058] osgi: upgrade jaxb bunles from 2.2.10 to 2.2.11

+1

Waiting for the build to finish. Will merge after that if all tests pass.


feat(ferry_generator): add option to reuse data classes for fragments with a single inline fragment  spread as selection

chore: update codegen
feat(ferry_generator): add data_class_config: reuse_fragment option in order to re-use selection sets with only a single inline fragment spread. see https://github.com/gql-dart/gql/pull/413 for details.

cc @dehypnosis


How to bundle this project for Mac OSX?

I love this project. It's so well done. Real kudos to the developers.
I was looking at this as inspiration for some generative art based GUI apps, and my question is what is the process to build the installer bundles?
Is there a script to bundle into the .dmg or the .exe? I'm not seeing anything like it in the repo.

I love this project. It's so well done. Real kudos to the developers.
I was looking at this as inspiration for some generative art based GUI apps, and my question is what is the process to build the installer bundles?
Is there a script to bundle into the .dmg or the .exe? I'm not seeing anything like it in the repo.

Thank you! I use Pyinstaller with a custom .spec to bundle the packages.


Issue with 'Linux' and 'Windows' notation in Korean docs

This is a Bug Report

Problem:
In some translated docs into Korean, 'Linux' and 'Windows' were not translated.
Proposed Solution:
Replace 'Linux' and 'Windows' with '리눅스' and '윈도우' in the following pages:

'Linux':

concepts/configuration/organize-cluster-access-kubeconfig.md
concepts/services-networking/service.md
concepts/workloads/pods/pod.md
concepts/workloads/pods/init-containers.md
concepts/policy/pod-security-policy.md
concepts/cluster-administration/networking.md
reference/glossary/docker.md
setup/production-environment/tools/kops.md
setup/production-environment/container-runtimes.md
setup/learning-environment/minikube.md
tasks/tools/install-minikube.md
tasks/tools/install-kubectl.md
tasks/access-application-cluster/configure-access-multiple-clusters.md

'Windows':

concepts/configuration/organize-cluster-access-kubeconfig.md
concepts/services-networking/service.md
setup/production-environment/windows/user-guide-windows-containers.md
setup/learning-environment/minikube.md
tasks/tools/install-minikube.md
tasks/tools/install-kubectl.md
tasks/administer-cluster/kubeadm/adding-windows-nodes.md
tasks/administer-cluster/kubeadm/upgrading-windows-nodes.md
tasks/access-application-cluster/configure-access-multiple-clusters.md

Page to Update:
https://kubernetes.io/...

/language ko

Fixed in PR #22362

/close

@sftim By any chances, do you know why this issue didn't close even though I added "Closes #22360" in the description of the PR?
https://kubernetes.io/docs/contribute/new-content/new-content/#open-a-pr

If there is a related GitHub issue, include Fixes #12345 or Closes #12345 in the description. GitHub's automation closes the mentioned issue after merging the PR if used. If there are other related PRs, link those as well.

/shrug
The ways of GitHub are sometimes mysterious


TensorFlow building issue

Fix the TensorFlow Lite build error with Raspberry Pi

no longer needed @raj99-code


use Bundler.with_unbundled_env instead of Bundler.with_clean_env

This fixes the following warning:
[DEPRECATED] Bundler.with_clean_env has been deprecated in favor of Bundler.with_unbundled_env. If you instead want the environment before bundler was originally loaded, use Bundler.with_original_env (called at /app/.bundle/vendor/ruby/3.0.0/gems/capistrano-locally-0.2.7/lib/capistrano/locally.rb:26)

Thank you for your excellent update!

I'd like to support old versions of Bundler for a little while, I'll check the compatibility and release the gem a little later. 🙇


Docs don't match CLI

Following the docs at https://prefix-dev.github.io/rattler-build/highlevel/#how-to-run-rattler-build
> rattler-build --recipe-file myrecipe/recipe.yaml

pixi run build
✨ Pixi task: rattler-build --recipe-file recipe.yaml
error: unexpected argument '--recipe-file' found

Usage: rattler-build [OPTIONS] [COMMAND]

rattler-build --version
rattler-build 0.7.0

Fixed in #503

Thanks!


Using yarn results in an error

# Installing project dependencies ...
# ========================

events.js:136
      throw er; // Unhandled 'error' event
      ^

Error: spawn yarn ENOENT
    at _errnoException (util.js:1031:13)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:201:19)
    at onErrorNT (internal/child_process.js:389:16)
    at _combinedTickCallback (internal/process/next_tick.js:138:11)
    at process._tickCallback (internal/process/next_tick.js:180:9)

Using develop template: vue init Toilal/vue-webpack-template#develop

Do you have yarn globally installed ? Does the problem occurs also with official webpack template ?

Yes to both.

I'm on Windows so yarn is installed using the MSI they provide, and it's global (or at least, in the path) C:\Program Files (x86)\Yarn\bin\yarn.cmd
It works correctly when using the webpack template.

If you choose to install dependencies manually (last choice) ? Everything is working properly when you run yarn manually ? It's strange it does work with official template because this part is exactly the same here.

Running yarn manually is fine, even after that error.


please auto install ghost cms with external PostgreSQL and Cloudinary image storage

hi Emmadi Sumith Kumar ,
at the first , thank you for make deploy to heroku button at github branch :)
please auto install ghost cms with external PostgreSQL and Cloudinary image storage as index.php file in web
PostgreSQL is free addon & not need to add credit card verify on heroku
this repo is complete productable , now need to auto install with your process :
https://github.com/SNathJr/ghost-on-heroku
waiting for your response
finest regards

Hi bazarbashe

It's very simple to do but it's take little more time , just wait I will explain you how you can do this.
Just clone my repository and add below  dependencies
to dependencies Object in package.json

    "attila": "github:zutrinken/attila#1.13.0",
    "bleak": "github:zutrinken/bleak#1.4.0",
    "casper": "github:tryghost/Casper#3.0.4",
    "ghost": "^3.2.0",
    "ghost-storage-adapter-s3": "^2.8.0",
    "ghost-storage-cloudinary": "^2.0.2",
    "ghost-webdav-storage-adapter": "^0.3.5",
    "london": "github:tryghost/london#1.0.0",
    "massively": "github:tryghost/massively#1.0.2",
    "mysql": "^2.17.1",
    "the-shell": "github:mityalebedev/The-Shell#v2.0.0",
    "vapor": "github:sethlilly/Vapor#v1.6.0",
    "pico": "github:tryghost/pico",
    "lyra": "github:tryghost/lyra",
    "liebling": "github:snathjr/liebling"

After adding above dependencies to package.json file add  "postinstall": "bash bin/copy-themes.sh && bash bin/copy-webdav-plugin.sh" to  Scripts   Objects and save the file.
Now open app.json file and add below add-ons  to add-ons object

    {
      "plan": "heroku-postgresql",
      "options": {
        "version": "9.5"
        }
    }
    "jawsdb",
    "mailgun",
    "cloudinary"

Now add your  environments  to env  object in app.json to know more https://devcenter.heroku.com/articles/app-json-schema
As per this repo  ghost-on-heroku the env will be

"env": {
    "APP_PUBLIC_URL": {
      "description": "The HTTPS URL of this app: either your custom domain or default 'herokuapp.com' hostname.",
      "value": "https://YOURAPPNAME.herokuapp.com"
    },
    "S3_ACCESS_KEY_ID": {
      "description": "(Optional) Set your AWS Access Key ID to enable S3 file storage. Defaults to Cloudinary storage",
      "required": false
    },
    "S3_ACCESS_SECRET_KEY": {
      "description": "AWS Access Secret Key, if using S3 file storage.",
      "required": false
    },
    "S3_BUCKET_NAME": {
      "description": "Name of your S3 bucket on AWS, if using S3 file storage.",
      "required": false
    },
    "S3_BUCKET_REGION": {
      "description": "Region of your S3 bucket on AWS, if using S3 file storage.",
      "required": false
    },
    "S3_ASSET_HOST_URL": {
      "description": "Optional custom CDN asset host url, if using S3 file storage.",
      "required": false
    },
    "WEBDAV_SERVER_URL": {
      "description": "URL of your WebDAV server, if using WebDAV file storage",
      "required": false
    },
    "WEBDAV_USERNAME": {
      "description": "Username for your WebDAV server, if using WebDAV file storage",
      "required": false
    },
    "WEBDAV_PASSWORD": {
      "description": "Password for your WebDAV server, if using WebDAV file storage",
      "required": false
    },
    "WEBDAV_PATH_PREFIX": {
      "description": "Optional path prefix for your WebDAV server, if using WebDAV file storage",
      "required": false
    },
    "WEBDAV_STORAGE_PATH_PREFIX": {
      "description": "Optional storage path prefix for  your WebDAV server, if using WebDAV file storage",
      "required": false
    }
  }

Then copy   all files and folders to this repo except Procfile ,app.json and package.json file.

then validate your app.json  file , refer to https://www.npmjs.com/package/app.json

Now push your repository to GitHub

Click on Deploy to Heroku button

Follow the instructions shown on Heroku  Dashboard.

But this type of installation is not good because Heroku will generate  new .onion link everytime when your app restarts  to use permanent onion domain go to my Repository  and follow from 7th step in Quick Start  session

Hello Again Emmadi Sumith Kumar ,
1 - I go all your steps but in case to :
Copy all files and folders to this repo except Procfile ,app.json and package.json file.
I also removed package-lock.json from ghost repo because overlap with your repo package-lock.json. is it correct ?
2 - server.js file is exist in both repositories , what to do ?
Please if it's possible make new branch diffrent as master & make changes of this work on it .
Waiting for your repsonse
Finest regards

Yesh sure ,I will create completely another repository .

hi Emmadi Sumith Kumar ,
I'm waiting for your changes bro :)
please if its possible do it faster ...
finest regards


Error inflating class on Android 4.x

Library Version [v1.0.5]
Affected Device(s) [AS emulator 4.2, 4.4]

Describe the Bug:
When inflating the layout containing expandablelayout, the app crashes. Unfortunately not much is reported to the logcat. Only "Error inflating class com.skydoves.expandablelayout"
Expected Behavior:
It should work as it does on Android 7 and 10; I tested on.
This is where it crashes
LayoutInflater.from(MainActivity.this).inflate(R.layout.activity_main_settings, null, false);
Below is how I use this library inside my layout
<com.skydoves.expandablelayout.ExpandableLayout
android:id="@+id/ABC"
android:layout_width="match_parent"
android:layout_height="wrap_content"
android:layout_marginTop="6dp"
app:expandable_duration="150"
app:expandable_isExpanded="false"
app:expandable_parentLayout="@layout/abc_parent"
app:expandable_secondLayout="@layout/abc_child"
app:expandable_showSpinner="true"
app:expandable_spinner="@drawable/ic_arrow_down"
app:expandable_spinner_color="@color/violetDark"
app:expandable_spinner_animate="true"
app:expandable_spinner_margin="0dp"
app:expandable_spinner_size="36dp"/>

Hi, @thereisonlywe!
It is released a new version 1.0.6!
Please check to work well using the new release.
Thank you for your issue :)

Fix confirmed.

Fix confirmed.
The problem remains!


 Specifically, we need the concourse security group which allows concourse to access the machines deployed with bosh init.
This fixes the error:
  Required variable not set: concourse_security_group_id

How to review this?
Deploy bosh as described in the README and destroy it.
Who?
Anyone but @keymon

Tested this as a part of work on https://www.pivotaltracker.com/n/projects/1275640/stories/110246622


Draft: Refactor + fix serialization

Closes #355 .
While moving ml_tools to qadence-libs, a refactoring of the qadence/serialization.py was needed. During the process, it was discovered a major issue on where an Python's eval was used to execute str that could come not only from internally serialized data but also from json files. While addressing the issues on the migration, this is an important fix while doing a good refactoring.

docs are failing and according to investigations made independently by @jpmoutinho it's braket related. On braket version 1.71.0 seems to be working.

@jpmoutinho @Roland-djee can you take a look whenever it's more appropriate

I expect that this might fail to deserialize old quantum models?
Could you verify that?
If so, could we wait to merge this until #366 is merged because I'd need that for the algorithm APIs, and once we change the serialization format it might take significant time to get the new format into production.

I expect that this might fail to deserialize old quantum models?
Could you verify that?
If so, could we wait to merge this until #366 is merged because I'd need that for the algorithm APIs, and once we change the serialization format it might take significant time to get the new format into production.

It's the same serialization process, just rewritten. If something is working now with the current serialization will work with this new one. A next step should be to propose a better serialization logic and structure.

Hey @Doomsk do you have time to finalise this ?

Hey @Doomsk do you have time to finalise this ?

I'll do it tomorrow!

can you take a look @dominikandreasseitz ?


Rolling Restart Alternative Stopping Mechanism

The current implementation of rolling restart or rolling deploys auto stops previous version (v1) of the app based on the good health of the new version's readiness (v2).     There is interest in doing an application version update however we want all of v2 up prior to v1 destroy AND there is a REST invocation to signal the v1 destruction.    The scenario would look like this:

app v1 running
marathon update v1 to v2
v2 fully update and running
external check of v2
successful external check leads to a message to marathon to remove v1
only v2 running

Hi @kensipe,
what about scripting the REST API for that?

(unchanged)
add an additional app with v2 (PUT app or POST app)
wait for deployment to finish
(unchanged)
DELETE app v2
(profit)

I think that @guenter just told me about a customer doing just that.

Just to keep our opened issues relevant, I will close this for now. If your requirements are different, you are more than welcome to reopen the issue.

it was @guenter that wanted this issue recorded for otto.

Hi @kensipe, I talked to @guenter and he elaborated that there approach was not to create a second app. Without creating a second app on upgrade you would need a different approach and additional features in Marathon.
But since they are doing blue/green deployment anyway it seems fairly natural to me.


urls that are not unique can destroy the build

If a story gets the same urls as a another story the build will not run
quick fix
Add a timestamp after the string to minimize this issue
Real fix
Validate a string before it gets submitted from netlifycms to test if its valid

This is a user issue and should not really be the role of the syste,


Add error reporting to bridge

Signed-off-by: Stoyan Gradev<EMAIL_ADDRESS>Signed-off-by: Plamen Doychev<EMAIL_ADDRESS>

Coverage decreased (-5.6%) to 87.5% when pulling adc0c148616e60c3accfc18eb56c282bce13bc73 on nmaslarski:master into 1f58fe92727974c97607f5310424c4e5a16a099a on cloudfoundry-incubator:master.

Coverage decreased (-0.3%) to 92.808% when pulling 208e09432449f0afb97b5368405d522d9d326550 on nmaslarski:master into 1f58fe92727974c97607f5310424c4e5a16a099a on cloudfoundry-incubator:master.

Coverage decreased (-0.3%) to 92.808% when pulling 208e09432449f0afb97b5368405d522d9d326550 on nmaslarski:master into 1f58fe92727974c97607f5310424c4e5a16a099a on cloudfoundry-incubator:master.

Coverage decreased (-0.05%) to 93.074% when pulling 208e09432449f0afb97b5368405d522d9d326550 on nmaslarski:master into 1f58fe92727974c97607f5310424c4e5a16a099a on cloudfoundry-incubator:master.


Investigate why the module can't delete Security groups and Subnets

As reported in https://github.com/seqeralabs/devops-backlog/issues/380, it seems the module fails to delete SGs and subnets:
module.terraform-seqera-aws.module.vpc.aws_subnet.private[0]: Still destroying... [id=subnet-0fa23ef68528640f6, 19m40s elapsed]
module.terraform-seqera-aws.module.vpc.aws_subnet.private[0]: Still destroying... [id=subnet-0fa23ef68528640f6, 19m50s elapsed]
module.terraform-seqera-aws.module.vpc.aws_subnet.private[0]: Still destroying... [id=subnet-0fa23ef68528640f6, 20m0s elapsed]
╷
│ Error: deleting Security Group (sg-01a597fdb9b6d498a): DependencyViolation: resource sg-01a597fdb9b6d498a has a dependent object
│ 	status code: 400, request id: 69257ac0-7cee-403d-b16d-c370123e0cab
│
│ Error: deleting EC2 Subnet (subnet-0fa23ef68528640f6): DependencyViolation: The subnet 'subnet-0fa23ef68528640f6' has dependencies and cannot be deleted.
│ 	status code: 400, request id: bcd72938-a453-465f-bb7a-3959148122d7

The reason was that the SG had an attached Network Interface which wasn't deleted by Terraform.
Once that was deleted I could also delete the Security Group, the Subnet and the VPC.


Repatriated Travellers - Record Not Found

Hitting https://api.opencovid.ca/summary?loc=RT returns 'Record not found'.  Documentation at https://opencovid.ca/api/ shows 'Repatriated Travellers' should exist.

You're right. Thanks. I'll work on a fix.

You're right. Thanks. I'll work on a fix.

Turned out to just be a typo. In prov_map.csv, I have specified Repatriated as RP. I've update the API docs to reflect this.

Turned out to just be a typo. In prov_map.csv, I have specified Repatriated as RP. I've update the API docs to reflect this.

https://api.opencovid.ca/summary?loc=RP

https://api.opencovid.ca/summary?loc=RP


Fixes after IP BOM 7.0.0.CR4 upgrade

All PRs mentioned in https://github.com/errai/errai/pull/180 need to be merged together, at the same time.

+1 to merge.


Cyclic dependency with advanced initialization

I'm having a cyclic dependency error when using the advanced initialization and a custom Http provider:
Cannot instantiate cyclic dependency! ApplicationRef_: in NgModule AppModule in ./AppModule
Bellow is the providers section of the module, note that if I remove any of the two providers the application will compile and run with each of them individually. Only when I try to use both of them together the error occurs:
providers: [
  {
    provide: APP_INITIALIZER,
    useFactory: initLocalization,
    deps: [LocalizationConfig],
    multi: true
  },
  {
    provide: Http,
    useFactory: (xhrBackend: XHRBackend, requestOptions: RequestOptions, router: Router) => {
      return new HttpInterceptor(xhrBackend, requestOptions, router);
    },
    deps: [XHRBackend, RequestOptions, Router]
  }
]

Full error stack trace:
Unhandled Promise rejection: Provider parse errors: Cannot instantiate cyclic dependency! ApplicationRef_: in NgModule AppModule in ./AppModule ; Zone: <root> ; Task: Promise.then ; Value: Error: Provider parse errors: Cannot instantiate cyclic dependency! ApplicationRef_: in NgModule AppModule in ./AppModule Stack trace: NgModuleProviderAnalyzer</NgModuleProviderAnalyzer.prototype.parse@http://localhost:5555/node_modules/@angular/compiler/bundles/compiler.umd.js:9916:21 [<root>] NgModuleCompiler</NgModuleCompiler.prototype.compile@http://localhost:5555/node_modules/@angular/compiler/bundles/compiler.umd.js:18498:11 [<root>] JitCompiler</JitCompiler.prototype._compileModule@http://localhost:5555/node_modules/@angular/compiler/bundles/compiler.umd.js:26572:52 [<root>] JitCompiler</JitCompiler.prototype._compileModuleAndComponents/createResult@http://localhost:5555/node_modules/@angular/compiler/bundles/compiler.umd.js:26514:22 [<root>] Zone.prototype.run@http://localhost:5555/node_modules/zone.js/dist/zone.js?1482335873252:113:24 [<root> => <root>] scheduleResolveOrReject/<@http://localhost:5555/node_modules/zone.js/dist/zone.js?1482335873252:520:52 [<root>] Zone.prototype.runTask@http://localhost:5555/node_modules/zone.js/dist/zone.js?1482335873252:151:28 [<root> => <root>] drainMicroTaskQueue@http://localhost:5555/node_modules/zone.js/dist/zone.js?1482335873252:418:25 [<root>] ZoneTask/this.invoke@http://localhost:5555/node_modules/zone.js/dist/zone.js?1482335873252:349:25 [<root>]

Did you see this question? http://stackoverflow.com/questions/39767019/app-initializer-raises-cannot-instantiate-cyclic-dependency-applicationref-w

Yep, that definitely fixes the problem.
I didn't associated with it because the custom Http provider was working standalone and the error message was referring the ApplicationRef_ and not the Router, but using the injector to get the router works fine.
@robisim74 thanks
Closing the issue.

Thank you for reporting. Greetings


Error handling for migrations

Migrations that fail should log and downgrade any changes that occurred.
Possibly allow developers to set a data iterator in their migration and pass that iterator to the downgrade callback.

I'm thinking users should handle their own exceptions in their migrations. No need for callbacks.


Retry Calibration with AHT20 command on failure

If try-except around legacy calibrate command fails, try to send new one instead afterwards.
Addressing #17 and possibly #18 as well
Moved try-except from PR #16 doesn't work with the latest AHT20s (at least the ones I have) but instead gives very cryptic OS error implying the device is not present. Found different first-byte to write for newer AHT20's and tried that (details in #17 ). Made updates that I think will work with AHT10 still as the execution path remains unchanged from before, but retries if that fails.
Tested on my setup (details in #17) and works great (takes the calibration_failed path, then calibrates and succeeds).

Confirmed. This update works for me. Requesting forum user also try.

thanks!


pull request for e2e test at 12/20/2016 11:15:19

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Is it possible to use pgsync only for update or delete triggers?

I'm not sure about this use case. I guess you can fork the repo and manually change the triggers to only handle UPDATE and DELETE ops.

@toluaina
Thanks for the update.


Could we please get an option similar to "Lock client to screen"

OSB and Konduit both have this option. I and a fair share of others like to be able to toggle the option off so the client will stick to one screen instead of moving to another.

The option should definitely be phrased in opposite way, to make it clear it is opt-in and not opt-out. Something like lock window to screen.

I agree, I was just quoting what OSB option for this was because I forget what the konduit option says. I think it's what you posted so i updated. Thank you!

Fixed in 1b7bdc616b5268bfc6a0de5cfabdab4bed2d3f49


error TS2687: All declarations of 'name' must have identical modifiers.

Bug Report
Problem
What is expected to happen?
successful build
What does actually happen?
Error: node_modules/cordova-plugin-file/types/index.d.ts:47:5 - error TS2687: All declarations of 'name' must have identical modifiers.

47     name: string;
       ~~~~

Error: node_modules/cordova-plugin-file/types/index.d.ts:49:5 - error TS2687: All declarations of 'root' must have identical modifiers.

49     root: DirectoryEntry;
       ~~~~

Error: node_modules/cordova-plugin-file/types/index.d.ts:49:5 - error TS2717: Subsequent property declarations must have the same type.  Property 'root' must be of type 'FileSystemDirectoryEntry', but here has type 'DirectoryEntry'.

49     root: DirectoryEntry;
       ~~~~

  node_modules/typescript/lib/lib.dom.d.ts:8316:14
    8316     readonly root: FileSystemDirectoryEntry;
                      ~~~~
    'root' was also declared here.

Error: node_modules/typescript/lib/lib.dom.d.ts:8314:14 - error TS2687: All declarations of 'name' must have identical modifiers.

8314     readonly name: string;
                  ~~~~

Error: node_modules/typescript/lib/lib.dom.d.ts:8316:14 - error TS2687: All declarations of 'root' must have identical modifiers.

8316     readonly root: FileSystemDirectoryEntry;

Information

Command or Code

npm run build in Ionic 7
Environment, Platform, Device

Windows 11
Version information

"typescript": "~5.1.6"
"cordova-plugin-file": "^8.0.0",

Checklist

[x] I searched for existing GitHub issues
[ ] I updated all Cordova tooling to most recent version
[x] I included all the necessary information above

any ideas? it conflicts with dom which is enabled by default in the tsconfig.json and is required for Angular

what's the best solution here?
module augmentation? overrides? special pathing in tsconfig.json?


proxy error

hi
when i try
npm install -g @getgauge/cli
and i'm behind proxy's campany.
i have this error
Fetching download url for Gauge version 1.0.6
Error: Error: getaddrinfo ENOTFOUND api.github.com api.github.com:443
    at Request.request.get [as _callback] (X:\node-v10.16.3-win-x64\nod
e_modules\@getgauge\cli\src\install.js:46:34)
    at self.callback (X:\node-v10.16.3-win-x64\node_modules\@getgauge\c
li\node_modules\request\request.js:185:22)
    at Request.emit (events.js:198:13)
    at Request.onRequestError (X:\node-v10.16.3-win-x64\node_modules\@g
etgauge\cli\node_modules\request\request.js:881:8)
    at ClientRequest.emit (events.js:198:13)
    at TLSSocket.socketErrorListener (_http_client.js:392:9)
    at TLSSocket.emit (events.js:198:13)
    at emitErrorNT (internal/streams/destroy.js:91:8)
    at emitErrorAndCloseNT (internal/streams/destroy.js:59:3)
    at process._tickCallback (internal/process/next_tick.js:63:19)
+<EMAIL_ADDRESS>updated 1 package in 9.739s

proxy is already set and only when call api.github.com:443 , connection is refused

Have you tried setting both the HTTP_PROXY and the HTTPS_PROXY environment variable.
Setting this should solve the issue.
Please refer this documentation
https://jjasonclark.com/how-to-setup-node-behind-web-proxy/
Closing this as this is not an issue with Gauge but related to having the right settings.

it s an gauge issue. because when install plugin. gauge get plugin from
other url. and proxy is block.
so it s not a gauge issue, but it can t work in stand alone.
without js plug in.
Le mar. 10 sept. 2019 à 17:06, Zabil Cheriya Maliackal <
<EMAIL_ADDRESS>a écrit :

Have you tried setting both the HTTP_PROXY and the HTTPS_PROXY
environment variable.
Setting this should solve the issue.
Please refer this documentation
https://jjasonclark.com/how-to-setup-node-behind-web-proxy/
Closing this as this is not an issue with Gauge but related to having the
right settings.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/getgauge/gauge/issues/1480?email_source=notifications&email_token=AAIIR325QRH42CG24ZZWYS3QI6ZXHA5CNFSM4IUZ273KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6LNXKQ#issuecomment-529980330,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAIIR3ZXGXOP6M3SODGBCHDQI6ZXHANCNFSM4IUZ273A
.

@spham Have you tried setting the proxy for Gauge? Refer https://docs.gauge.org/configuration.html#http-proxy. to set the proxy.

yes. but proxy have multi level.
and filter some website.
Le mer. 11 sept. 2019 à 07:25, Nehashri<EMAIL_ADDRESS>a écrit :

@spham https://github.com/spham Have you tried setting the proxy for
Gauge? Refer https://docs.gauge.org/configuration.html#http-proxy. to set
the proxy.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/getgauge/gauge/issues/1480?email_source=notifications&email_token=AAIIR3ZP6QCZYICH7WUZNRLQJB6NPA5CNFSM4IUZ273KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6NJN6Y#issuecomment-530224891,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAIIR3ZPEETWCQWZPTV6XZ3QJB6NPANCNFSM4IUZ273A
.

Gauge needs access to GitHub to download and install releases. Please try

White listing the GitHub urls in your proxy
Or downloading the zip archive from releases and installing it.

i think is not clear, jauge was download, but only js template can't install

This is what happens when you run npm install -g @getgauge/cli

Download a wrapper script
Download gauge binaries frongithub and install it on the path

After this you can initialise a javascript project using gauge init js which

Downloads the gauge plugin from GitHub
Runs npm install to install all dependencies

At all of this stages the proxy should not block GitHub or npm urls.

already done.
gauge init js failed
Le mer. 11 sept. 2019 à 16:23, Zabil Cheriya Maliackal <
<EMAIL_ADDRESS>a écrit :

This is what happens when you run npm install -g @getgauge/cli

Download a wrapper script
Download gauge binaries frongithub and install it on the path

After this you can initialise a javascript project using gauge init js
which

Downloads the gauge plugin from GitHub
Runs npm install to install all dependencies

At all of this stages the proxy should not block GitHub or npm urls.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/getgauge/gauge/issues/1480?email_source=notifications&email_token=AAIIR36U2WKFHU6HLBWE7PLQJD5NZA5CNFSM4IUZ273KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6OVEGY#issuecomment-530403867,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAIIR36234UH5C435A2JUXTQJD5NZANCNFSM4IUZ273A
.

Can you share the complete console output when running gauge init js ?

gauge init js
Compatible language plugin js is not installed. Installing plugin...
Failed to install plugin 'js'.
Reason: Invalid plugin. Could not download js-install.json file.

i try offline without success
C:\Users\XX\Desktop\gauge>gauge -f gauge-js-2.3.5.zip install
Error ----------------------------------

[Gauge]
File manifest.json doesn't exist.

Get Support ----------------------------
        Docs:          https://docs.gauge.org
        Bugs:          https://github.com/getgauge/gauge/issues
        Chat:          https://gitter.im/getgauge/chat

Your Environment Information -----------
        windows, 1.0.6, 2bc49db
        Could not retrieve plugin information.

@spham We currently can't install gauge-js plugin offline. https://github.com/getgauge/gauge-js/issues/200 should solve this issue, we will be working on this soon.

@spham We have pushed an offline installer for gauge-js.
Could you please try the following steps

Download the installer from https://bintray.com/gauge/gauge-js/download_file?file_path=gauge-js-offline-2.3.6.nightly-2019-09-24.zip

Run gauge install js -f <PATH_TO_DOWNLOADED_ZIP_FILE>

@spham Any update on this issue.

wonderful, it's work perfect, thank you

Compatible version of plugin html-report not found. Installing plugin html-report...
Failed to install plugin 'html-report'.
Reason: Invalid plugin. Could not download html-report-install.json file.
Installing required plugins.
fatal: not a git repository (or any of the parent directories): .git

Compatible version of plugin html-report not found. Installing plugin html-report...
Failed to install plugin 'html-report'.
Reason: Invalid plugin. Could not download html-report-install.json file.
Installing required plugins.
fatal: not a git repository (or any of the parent directories): .git

it have now an issue wirh html report

ok it's other issue


Chips js functionality

Adding some javascript functionality to create and delete chips in different chips containers. Keyboard feature is done. ToDo: autocomplete.

@acburst @TyrionGraphiste It's done!, sorry for the last PR :-P

Are you sure you forked the latest master? Your gruntfile is out of date which is why our tests are failing.

See https://help.github.com/articles/syncing-a-fork/


Update GEE Client wiki page

https://github.com/google/earthenterprise/wiki/Google-Earth-Enterprise-Client-(EC)

Fix current version to <IP_ADDRESS>21
Add link to download x64 installer for Windows

[Windows 32-bit](https://dl.google.com/dl/earth/client/advanced/current/googleearthecwin.exe)
[Windows 64-bit](https://dl.google.com/dl/earth/client/advanced/current/googleearthecwin-x64.exe)

I've made the requested changes @zedxxx. Let me know if you see any other problems.


Idea: Add helpers to avoid rewriting code in similar projects

Hi! This is really great library that helps us to build flexible API over ElasticSearch. Currently we do have a need to build our own helpers over this library to build some complex queries over our data. In our code we do have something like this (one of examples):
shared.BuildTermsQuery("tags", []string{"tag1", "tag2"})
and BuildTermsQuery function:
// BuildTermsQuery builds terms query for elasticsearch
func BuildTermsQuery(field string, values []string) elastic.Query {
	terms := make([]interface{}, len(values))
	for i, key := range values {
		terms[i] = key
	}

	return elastic.NewTermsQuery(field, terms...)
}

I would like to propose adding helper functions in this library that will add some syntax sugar to build  queries over nested data, to build list of term queries with must relation and some other examples. It will help users to not rewrite their own code in 2 or more projects.
I would like to add some syntax sugar, that will help users to write complex queries over their data.
WDYT about it? I can add more examples/add PR with examples if you think that it can be possible.

What about NewTermsQueryFromStrings(field string, values ...string)?


Name resolution refactoring

This is still very WIP, but it's becoming quite big and I want to make sure this isn't going in a completely bad direction :sweat_smile:. I'm not really happy with how the path resolution looks, and I'm not sure PerNs<Resolution> is the best return type -- there are 'this cannot happen in the (types/values) namespace' cases everywhere. I also want to unify the resolver and nameres namespaces once I'm done switching everything to Resolver. Also, Resolver only has a lifetime because it needs to have a reference to the ItemMap during import resolution :confused:
The differences in the completion snapshots are almost completely just ordering (except it completes Self as well now), so I changed it to sort the completions before snapshotting.

Love this interface!
bors r+
See also
https://github.com/rust-analyzer/rust-analyzer/issues/723 for some required changes in the implementation to support macros properly.