Update scala-library to 2.13.6

Updates org.scala-lang:scala-library from 2.13.1 to 2.13.6.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (2.13.1).
You might want to review and update them manually.
.travis.yml

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-lang", artifactId = "scala-library" } ]

labels: library-update, semver-patch, old-version-remains

Codecov Report

Merging #91 (3ff2f46) into master (339416a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #91   +/-   ##
=======================================
  Coverage   93.75%   93.75%
=======================================
  Files           1        1
  Lines          16       16
=======================================
  Hits           15       15
  Misses          1        1

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 339416a...3ff2f46. Read the comment docs.

Superseded by #102.


Fix few typos and rendering issue in Readme

I do not have rights to merge it myself

Solving the privileges thing...


Add support for parsing inline elements and CDATA in XLIFF 1.2

@ehoogerbeets apparently CI is failing due to SSH key issue, could you please take a look?


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-30-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 4669

Questions ?
Please refer to the Backport tool documentation


Support for isar database

Can you explain what you want exactly?
Isar is a local DB to be used with Flutter.
It is independent of Serverpod.

Clarification missing. Please open a new issue with a better clarification if this is still relevant.


太邪了！明明传入的只有2个参数……

Traceback (most recent call last):
  File "/root/ComfyUI/custom_nodes/ComfyUI-Hallo/Hallo/scripts/inference.py", line 376, in <module>
    inference_process(command_line_args)
  File "/root/ComfyUI/custom_nodes/ComfyUI-Hallo/Hallo/scripts/inference.py", line 182, in inference_process
    audio_emb, audio_length = audio_processor.preprocess(driving_audio_path, clip_length)
TypeError: AudioProcessor.preprocess() takes 2 positional arguments but 3 were given

怎么你也在哈哈哈


Declaratively set attributes on host component

Is there a way to mimic the functionality of Stencil's Host Element? https://stenciljs.com/docs/host-element
My use case is wai-aria

I guess I could do
@property({ reflect: true })
'role' = 'button';

or, for dashed attributes
@property({ reflect: true })
'aria-valuemin' = '0';

or
attribute: 'foo-bar'

We're considering features along these lines.


Watch support?

It doesn't appear watches are currently supported, having them would be great! (or letting me know they exist); something akin to https://github.com/python-zk/kazoo/blob/master/kazoo/recipe/watchers.py could be a pattern that is emulated.

Hi @harlowja. The usual pattern is to make a blocking call using the index parameter. This will return when new data has been added. Here's an example using the Vanilla Python async client (https://github.com/cablehead/vanilla.consul)
import vanilla
import pprint

h = vanilla.Hub()

# simulate a process that periodically updates a key value
@h.spawn
def _():
    c = h.consul()
    value = 0
    while True:
        h.sleep(3000)
        value = value + 1
        c.kv.put("foo", str(value)).recv()

# watch a key value
c = h.consul()
index = None
while True:
    index, data = c.kv.get("foo", index=index).recv()
    print "UPDATE:"
    pprint.pprint(data)
    print

Does this help?

We also need this feature.As the usual pattern,if we need some interface like Health.service.We need copy the code and finish this same logic with the pattern.

Perhaps start a patterns folder and share these common pieces of code there?
On Tuesday, October 20, 2015, Chen Chenglong<EMAIL_ADDRESS>wrote:

We also need this feature.As the usual pattern,if we need some interface
like Health.service.We need copy the code and finish this same logic with
the pattern.
—
Reply to this email directly or view it on GitHub
https://github.com/cablehead/python-consul/issues/71#issuecomment-149481471
.

--
facebook.com/jshharlow http://www.facebook.com/jshharlow
flickr.com/jshharlow
YIM: jshharlow

There's an example uses section in the docs:
http://python-consul.readthedocs.org/en/latest/#example-uses
Do you think it'd work to add these common patterns there?

I was thinking under https://github.com/cablehead/python-consul/tree/master/consul/patterns
Examples are great, and such, but I've always preferred having in-code-tree patterns like what kazoo also provides (as recipes): https://github.com/python-zk/kazoo/tree/master/kazoo/recipe
Which itself come from http://curator.apache.org/curator-recipes/index.html and others at https://github.com/Netflix/curator/wiki/Recipes

Can you help me with watching ?
import consul.tornado
import tornado

class Config(object):
    def __init__(self):
        self.foo = None
        loop.add_callback(self.watch)

    @tornado.gen.coroutine
    def watch(self):
        c = consul.tornado.Consul()

        # asynchronously poll for updates
        index = None
        while True:
            index, data = yield c.catalog.nodes(index=index)
            print("Index: {}".format(index))
            print("Data: {}".format(data))

if __name__ == "__main__":
    loop = tornado.ioloop.IOLoop()
    loop.make_current()
    c = Config()
    loop.run_sync(c.watch)

what am I doing wrong ? I'm still getting Timeouts (tornado.httpclient.HTTPError: HTTP 599: Timeout during request)
Thanks

watch seems to be a first class action in consul.io,  I think just having a recipe to achieve something similar is not enough. (not that I'm saying it's easy to implement for various types of clients)

after reading the documentation again, I take that back...  watch isn't first class in the http api

Hi @pipozzz.. were you able to get this to work? If a change doesn't occur before the timeout that's configured with your http client, then yeah, it'll timeout. It's normal to need to reestablish the watch from the long poll, if it returns before there's a change. Just keep the current index value that you have.

Going to close this. Totally agree more examples and documentation would be helpful. If anyone sends through a PR for them, we'll look to get them merged.


OOM when using GAS>1

I have been able to use any GAS value without any noticeable VRAM changes in the past, but recently, I have seen very weirdly timed OOMs. The first one happened with GAS=4 after over 500 steps, at around epoch 1.2 or so, i.e. the model had trained on everything once already successfully and was doing it again when it ran out of VRAM. Resuming the run resulted in OOM after some dozen or so steps. This is with pipeline_stages=2, i.e. the model trains each batch sequentially.
I have seen intermittent OOMs after this, for different models, around step 50-60 again with GAS=4. The exact same setup with GAS=1 does not OOM.
I wonder if the batching stuff could be the reason, but IIUC they should be identical for the same step across epochs, which means it should have OOM'd during epoch 1. Also, the code literally puts the biggest fattest batch at step 1 to ensure OOM happens early, and yet it continues on for 500+ steps before finally breaking.
It is possible that there is some subtle form of leak somewhere, but if that was the case, I would assume that the resume of the 500+ step run above would run for another 500 or so steps before crashing, but it didn't.

Don't think this is qlora-pipe related.

I can share a few things that might be the cause.
First, are you training with variable batch sizes? (meaning using batch_size_tokens along with variable length data) Because tensors are different sizes each step, it may cause more memory fragmentation. And this could theoretically get worse as time goes on. Might be worth implementing sample packing at some point to avoid this.
Another thing, GAS > 1 uses a tiny bit more memory, since it needs some buffers proportional to the maximum number of in-flight forward and backward passes. This is capped by your number of pipeline stages though.
I have seen in the past that resuming from checkpoint can use a very small amount of additional VRAM. Something in deepspeed or pytorch leaks a bit of memory. If you search you can find others reporting similar issues.
For what it's worth, a few weeks ago (on a commit prior to the most recent round of changes, if that matters), I trained a qlora on mistral-large across 4 4090s. This was with multiple resumes. Each GPU had a couple of GB of free memory, and I recall the usage being very consistent. This was with fixed sized batches though.

Thanks for the suggestions! I am using batch_sizer_tokens and variable length data, yes. I'm not sure how that would cause memory fragmentation. What's boggling to me is that qlora-pipe trained successfully on the batch once and then failed the second time.

I can share a few things that might be the cause.
For what it's worth, a few weeks ago (on a commit prior to the most recent round of changes, if that matters), I trained a qlora on mistral-large across 4 4090s. This was with multiple resumes. Each GPU had a couple of GB of free memory, and I recall the usage being very consistent. This was with fixed sized batches though.

Yeah, I'm currently training a qlora on mistral-large across 3 machines each with 2xA6000 (ie: pipeline_stages = 2) and I'm seeing steady 45/48GB used on each machine. I'm the past when I've OOMed it's always been after the first eval step so fingers crossed it will stay like this as only 1/4 of the way through so far.

I have only ever seen qlora-pipe crash on the first step before. It crashing on step 500-something was pretty disconcerting but I am also juggling a bunch of the dependencies as latest-github-commit versions, so I bet the issue is in some bug there. I just resorted to using activation_checkpointing = 'unsloth' which is slower but at least it doesn't OOM.

Did you ever get the the bottom of this?
I've had a weird one where it keeps running 1 more step at the end ("step:    563 /   562 loss: ..."), and before your fix to use min_lr; I tried using with cosine annealed schedule and (I think) the lr went negative (ie: gradient ascent) on this last step and screwed the model over :/
I'm not sure if it's because I've started using eval_after_last_step = true, but IIRC the time the model got screwed I wasn't actually using this setting...
I don't remember it running these extra steps before, but I guess it could be related to batch size and I just happened to get lucky before and the total number of batches was an even multiple of my 6 GPUs I'm running it on?

Did you ever get the the bottom of this?

Work kept me away so I haven't really done anything qlora-pipe related recently. Definitely intend to look closer at this when I get the chance.

I've had a weird one where it keeps running 1 more step at the end (eg: "step: 563 / 562 loss: ..."), and before your fix to use min_lr; I tried using with cosine annealed schedule and (I think) the lr went negative (ie: gradient ascent) on this last step and screwed the model over :/

I always see that extra step these days. Like you, I don't recall seeing it in the past, but I don't save the trainer output for runs so I can't check. Kinda tempted to propose a built-in tee feature to just log everything to stdout AND to the out-dir/trainer.log.

I'm not sure if it's because I've started using eval_after_last_step = true, but IIRC the time the model got screwed I wasn't actually using this setting...

That setting should not affect training at all, no.

I don't remember it running these extra steps before, but I guess it could be related to batch size and I just happened to get lucky before and the total number of batches was an even multiple of my 6 GPUs I'm running it on?

Don't think so. I consistently see that extra step on my end.

I've been looking to see where this extra step is coming from and nothing stands out in the code :/
Do you have any logs saved that might narrow down when it started occurring?
(Sadly I haven't been saving any, but will start using your suggestion of piping into tee now)

I've been looking to see where this extra step is coming from and nothing stands out in the code :/

I assume it is
steps_per_epoch = len(train_dataloader) // model_engine.gradient_accumulation_steps()

which will be truncated unless len(train_dataloader) is evenly divisible by the GAS. Easy fix, I think? Just ceil it. I'll PR.
This would explain why it happens sometimes and not others. I used GAS 1 for a long time as I heard > 1 was unstable. I then started using a higher value because I was recommended an effective BS of ~16-32.

Extra training step on epoch end is fixed by e9b1bfb.


Fix the bug of issue #37

The generated getter method doesn't need to create nan__xxx
instances for interface array attribute, only need to get the array
from impl and set it as return value. Add checks to separate interface
and interface array attributes, and also update ut tests.
@kenny-y

LGTM except a naming issue that is using point instead of Point. I'll merge this first and PR the naming issue later


lxd: 2.0.2 -> 2.12

Motivation for this change
Things done

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

[x] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

Can you test this with sandboxing enabled? With such a great leap in versions, seems like good potential for new implicit dependencies & whatnot.

Hi, I have done nox-review pr 25685.
I also activated the sandbox and builded the package in a sandbox.

Thank you

Hi!
I realize PRs are not a place for support requests, but the resources on the subject seem very sparse, so I'll indulge myself into asking here anyway.
Are you aware of any beginner friendly walk-through of running LXC container (using LXD) on NixOS? I'm interested in more recent versions of LXD (> 2.3) with networking features. Basically I want to recreate this setup from Ubuntu 16.04 into NixOS 17.03.
I'm very new to both LXC / LXD and NixOS (installed it for a first time this weekend).
Thanks in advance and sorry for spamming your PR ;-)

Nope sorry. Is something not working?

I can't start a container (neither as root or normal user).
Here is my /etc/nixos/configuration.nix:
{ config, pkgs, ... }:

{
  imports =
    [ # Include the results of the hardware scan.
      ./hardware-configuration.nix
    ];
  networking.hostName = "macbook-pro-nixos";
  networking.networkmanager.enable = true;

  # Open ports in the firewall.
  # networking.firewall.allowedTCPPorts = [ ... ];
  # networking.firewall.allowedUDPPorts = [ ... ];
  # Or disable the firewall altogether.
  # networking.firewall.enable = false;

  virtualisation.lxd.enable = true;

  # Define a user account. Don't forget to set a password with ‘passwd’.
  users.extraUsers.tadeusz = {
    isNormalUser = true;
    uid = 1000;
    extraGroups = [ "wheel" "networkmanager" "lxd" ];
    shell = pkgs.zsh;
  };

  # The NixOS release to be compatible with for stateful data such as databases.
  system.stateVersion = "17.03";
}

I've left everything that can even remotely affect the LXD.
Here is what a shell log:
 lxc<PHONE_NUMBER>3206.290 INFO     lxc_seccomp - seccomp.c:parse_config_v2:418 - processing: .kexec_load errno 1.
            lxc<PHONE_NUMBER>3206.290 INFO     lxc_seccomp - seccomp.c:parse_config_v2:590 - Adding native rule for kexec_load action 327681.
            lxc<PHONE_NUMBER>3206.290 INFO     lxc_seccomp - seccomp.c:parse_config_v2:593 - Adding compat rule for kexec_load action 327681.
            lxc<PHONE_NUMBER>3206.290 INFO     lxc_seccomp - seccomp.c:parse_config_v2:418 - processing: .open_by_handle_at errno 1.
            lxc<PHONE_NUMBER>3206.290 INFO     lxc_seccomp - seccomp.c:parse_config_v2:590 - Adding native rule for open_by_handle_at action 327681.
            lxc<PHONE_NUMBER>3206.290 INFO     lxc_seccomp - seccomp.c:parse_config_v2:593 - Adding compat rule for open_by_handle_at action 327681.
            lxc<PHONE_NUMBER>3206.290 INFO     lxc_seccomp - seccomp.c:parse_config_v2:418 - processing: .init_module errno 1.
            lxc<PHONE_NUMBER>3206.290 INFO     lxc_seccomp - seccomp.c:parse_config_v2:590 - Adding native rule for init_module action 327681.
            lxc<PHONE_NUMBER>3206.290 INFO     lxc_seccomp - seccomp.c:parse_config_v2:593 - Adding compat rule for init_module action 327681.
            lxc<PHONE_NUMBER>3206.290 INFO     lxc_seccomp - seccomp.c:parse_config_v2:418 - processing: .finit_module errno 1.
            lxc<PHONE_NUMBER>3206.290 INFO     lxc_seccomp - seccomp.c:parse_config_v2:590 - Adding native rule for finit_module action 327681.
            lxc<PHONE_NUMBER>3206.290 INFO     lxc_seccomp - seccomp.c:parse_config_v2:593 - Adding compat rule for finit_module action 327681.
            lxc<PHONE_NUMBER>3206.290 INFO     lxc_seccomp - seccomp.c:parse_config_v2:418 - processing: .delete_module errno 1.
            lxc<PHONE_NUMBER>3206.290 INFO     lxc_seccomp - seccomp.c:parse_config_v2:590 - Adding native rule for delete_module action 327681.
            lxc<PHONE_NUMBER>3206.290 INFO     lxc_seccomp - seccomp.c:parse_config_v2:593 - Adding compat rule for delete_module action 327681.
            lxc<PHONE_NUMBER>3206.290 INFO     lxc_seccomp - seccomp.c:parse_config_v2:603 - Merging in the compat Seccomp ctx into the main one.
            lxc<PHONE_NUMBER>3206.290 INFO     lxc_conf - conf.c:run_script_argv:427 - Executing script "/nix/store/pns5q1wwyy2f0glsrpnwrc4304lclw2p-lxd-2.0.2-bin/bin/lxd callhook /var/lib/lxd 7 start" for container "ubuntu-container", config section "lxc".
            lxc<PHONE_NUMBER>3206.290 DEBUG    lxc_monitor - monitor.c:lxc_monitord_spawn:330 - Going to wait for pid 1779.
            lxc<PHONE_NUMBER>3206.291 DEBUG    lxc_monitor - monitor.c:lxc_monitord_spawn:349 - Trying to sync with child process.
            lxc<PHONE_NUMBER>3206.291 INFO     lxc_start - start.c:lxc_check_inherited:235 - Closed inherited fd: 3.
            lxc<PHONE_NUMBER>3206.291 INFO     lxc_start - start.c:lxc_check_inherited:235 - Closed inherited fd: 8.
            lxc<PHONE_NUMBER>3206.291 DEBUG    lxc_monitor - monitor.c:lxc_monitord_spawn:387 - Using pipe file descriptor 5 for monitord.
            lxc<PHONE_NUMBER>3206.299 DEBUG    lxc_monitor - monitor.c:lxc_monitord_spawn:364 - Successfully synced with child process.
            lxc<PHONE_NUMBER>3206.300 DEBUG    lxc_monitor - monitor.c:lxc_monitord_spawn:333 - Finished waiting on pid 1779.
            lxc<PHONE_NUMBER>3206.300 INFO     lxc_monitor - monitor.c:lxc_monitor_sock_name:201 - using monitor socket name "lxc/d78a9d7e97b4b375//var/lib/lxd/containers" (length of socket name 44 must be <= 105)
            lxc<PHONE_NUMBER>3206.300 DEBUG    lxc_monitor - monitor.c:lxc_monitor_open:225 - opening monitor socket lxc/d78a9d7e97b4b375//var/lib/lxd/containers with len 44
            lxc<PHONE_NUMBER>3206.339 DEBUG    lxc_start - start.c:setup_signal_fd:273 - Set SIGCHLD handler with file descriptor: 5.
            lxc<PHONE_NUMBER>3206.339 INFO     lxc_confile - confile.c:config_idmap:1537 - read uid map: type u nsid 0 hostid 1000000 range 100000
            lxc<PHONE_NUMBER>3206.339 INFO     lxc_confile - confile.c:config_idmap:1537 - read uid map: type g nsid 0 hostid 1000000 range 100000
            lxc<PHONE_NUMBER>3206.339 DEBUG    console - console.c:lxc_console_peer_default:438 - process does not have a controlling terminal
            lxc<PHONE_NUMBER>3206.339 INFO     lxc_start - start.c:lxc_init:475 - Container "ubuntu-container" is initialized.
            lxc<PHONE_NUMBER>3206.339 DEBUG    lxc_start - start.c:__lxc_start:1325 - Not dropping CAP_SYS_BOOT or watching utmp.
            lxc<PHONE_NUMBER>3206.344 INFO     lxc_conf - conf.c:instantiate_veth:2709 - Retrieved mtu 1500 from vethK1CQKF
            lxc<PHONE_NUMBER>3206.344 ERROR    lxc_conf - conf.c:instantiate_veth:2728 - failed to attach 'veth722Y17' to the bridge 'lxdbr0': Operation not permitted
            lxc<PHONE_NUMBER>3206.352 ERROR    lxc_conf - conf.c:lxc_create_network:3021 - failed to create netdev
            lxc<PHONE_NUMBER>3206.352 ERROR    lxc_start - start.c:lxc_spawn:1099 - Failed to create the network.
            lxc<PHONE_NUMBER>3206.352 ERROR    lxc_start - start.c:__lxc_start:1354 - Failed to spawn container "ubuntu-container".
            lxc<PHONE_NUMBER>3206.352 INFO     lxc_conf - conf.c:run_script_argv:427 - Executing script "/nix/store/pns5q1wwyy2f0glsrpnwrc4304lclw2p-lxd-2.0.2-bin/bin/lxd callhook /var/lib/lxd 7 stop" for container "ubuntu-container", config section "lxc".
            lxc<PHONE_NUMBER>3206.371 WARN     lxc_commands - commands.c:lxc_cmd_rsp_recv:177 - Command get_cgroup failed to receive response: Connection reset by peer.
            lxc<PHONE_NUMBER>3206.371 WARN     lxc_commands - commands.c:lxc_cmd_rsp_recv:177 - Command get_cgroup failed to receive response: Connection reset by peer.
            lxc<PHONE_NUMBER>3206.372 INFO     lxc_confile - confile.c:config_idmap:1537 - read uid map: type u nsid 0 hostid 1000000 range 100000
            lxc<PHONE_NUMBER>3206.372 INFO     lxc_confile - confile.c:config_idmap:1537 - read uid map: type g nsid 0 hostid 1000000 range 100000
            lxc<PHONE_NUMBER>3219.573 INFO     lxc_confile - confile.c:config_idmap:1537 - read uid map: type u nsid 0 hostid 1000000 range 100000
            lxc<PHONE_NUMBER>3219.573 INFO     lxc_confile - confile.c:config_idmap:1537 - read uid map: type g nsid 0 hostid 1000000 range 100000
            lxc<PHONE_NUMBER>3219.577 INFO     lxc_confile - confile.c:config_idmap:1537 - read uid map: type u nsid 0 hostid 1000000 range 100000
            lxc<PHONE_NUMBER>3219.577 INFO     lxc_confile - confile.c:config_idmap:1537 - read uid map: type g nsid 0 hostid 1000000 range 100000

~ ⌚ 22:32:19
$ lxc network list                                                                                ~
+--------+----------+---------+-------------+---------+
|  NAME  |   TYPE   | MANAGED | DESCRIPTION | USED BY |
+--------+----------+---------+-------------+---------+
| wlp3s0 | physical | NO      |             | 0       |
+--------+----------+---------+-------------+---------+

~ ⌚ 22:32:31
$ ifconfig                                                                                ~
lo: flags=73<UP,LOOPBACK,RUNNING>  mtu 65536
        inet <IP_ADDRESS>  netmask <IP_ADDRESS>
        inet6 ::1  prefixlen 128  scopeid 0x10<host>
        loop  txqueuelen 1  (Local Loopback)
        RX packets 757  bytes 71143 (69.4 KiB)
        RX errors 0  dropped 0  overruns 0  frame 0
        TX packets 757  bytes 71143 (69.4 KiB)
        TX errors 0  dropped 0 overruns 0  carrier 0  collisions 0

vboxnet0: flags=4099<UP,BROADCAST,MULTICAST>  mtu 1500
        inet <IP_ADDRESS>  netmask <IP_ADDRESS>  broadcast <IP_ADDRESS>
        ether 0a:00:27:00:00:00  txqueuelen 1000  (Ethernet)
        RX packets 0  bytes 0 (0.0 B)
        RX errors 0  dropped 0  overruns 0  frame 0
        TX packets 0  bytes 0 (0.0 B)
        TX errors 0  dropped 0 overruns 0  carrier 0  collisions 0

wlp3s0: flags=4163<UP,BROADCAST,RUNNING,MULTICAST>  mtu 1500
        inet <IP_ADDRESS>  netmask <IP_ADDRESS>  broadcast <IP_ADDRESS>
        inet6 fe80::cd15:744c:de86:ff16  prefixlen 64  scopeid 0x20<link>
        ether f4:5c:89:9f:d0:0d  txqueuelen 1000  (Ethernet)
        RX packets 1576  bytes 1230552 (1.1 MiB)
        RX errors 0  dropped 0  overruns 0  frame 0
        TX packets 1323  bytes 152385 (148.8 KiB)
        TX errors 0  dropped 0 overruns 0  carrier 0  collisions 0

I'm probably missing something very obvious - as I've mentioned I'm new to both technologies. One thing I've noticed is that there is no lxdbr0 bridge network.

Have you executed the command lxd init?
See the following tutorial.

Thanks. I'll try it in the evening.
But... is this a proper NixOS way? Seems quite imperative :grimacing:

Not that I'm aware of. But after some experiments, I also gave up lxc and did not used it anymore. So, sorry.

Thanks again for your attention here. I've opened a proper issue for that here: #27967


Add support for Request<T> where T is a type that implements hyper::body::Body trait

Should resolve #1102

Problem
Attempts to use any other hyper request type aside Request<hyper::body::Incoming> fails with a compile-time error of type mismatch

Solution
Add generic support for valid supported hyper Request or any custom Request that has a body which implements hyper::body::Body
Checklist

[x]  ~Tests are added~ (not required)
[x]  CHANGELOG entry is added
[x]  ~Documentation is updated~ (not required)

CI Checks seem to fail and this is weird since the issues arise from the juniper crate.
Any pointers?

Thank you for the PR, sorry for the slow response! Are there backward compat considerations here? I see you have a default type but I am not sure. Also, can you add a test so we can be sure this doesn't regress?

Thanks for checking in. To answer your concerns..

Are there backward compat considerations here?

Yes, this PR is fully backward compatible. How do we know? There's a test already included here and it passes with no need for further configuration.

can you add a test so we can be sure this doesn't regress

Yes I can add a test for custom Request<T> type. However, I must point out that this PR makes no algo change whatsoever, it only allows passing generic request types making juniper compatible with different request types.

@elcharitas @LegNeato I'll look into it too in the next few working days.


Fix environment binding preview

Fixes https://github.com/mui/mui-toolpad/issues/3410
The env var file is only a subset of the available vars.

Use the env var file keys for the warning message
Use the actual env for evaluation of preview
Show a preview of the actual value
Fix issues with the tab switching to expression when you empty the env value

Does it make sense to have a more standard warning?

Does it make sense to have a more standard warning?

Maybe, but probably not within the scope of this PR. The message was already there, I just fixed some bugs around the binding editor for env vars.


Non-Windows support

Is your feature request related to a problem? Please describe.
It would be helpful if there was support for running the commands from a non-Windows host to a Windows host using PowerShell core.
I think it is almost there as-is, except for a few issues like $env:APPDATA in Set-DcnConfiguration, since so little is actually done locally.
Happy to work through any other issues on this front in a PR

We could try this out. I'm all for that.
The thing is though that the actions for creating all the images and clones rely on Windows functionality


[Doc] Relax Deep Dive

Similar as TensorIR Deep Dive, we also have Relax Deep Dive.

preview at http://<IP_ADDRESS>:58000/deep_dive_relax/

cc @tqchen

@tvm-bot rerun


Most of notification data in eventListener returned as undefined

Hey.
I've currently done full integration as per documentation (AppDelegate .h and .m), and notification event listener returns most of the data as undefined.
    RNCPushNotificationIOS.addEventListener('localNotification', notif => {
      console.log(notif);
    });
// {"_alert": undefined, "_badgeCount": undefined, "_category": "", "_data": {"id": "1231231"}, "_fireDate": undefined, "_isRemote": undefined, "_remoteNotificationCompleteCallbackCalled": false, "_sound": undefined}

Although in the native side you can easily get fireDate etc., it results undefined in the listener.
UIApplication *app = [UIApplication sharedApplication];
NSArray *eventArray = [app scheduledLocalNotifications];
for (int i=0; i<[eventArray count]; i++)
{
    UILocalNotification* oneEvent = [eventArray objectAtIndex:i];
    NSLog(@"notifs hello %@", oneEvent);
}
// notifs hello <UIConcreteLocalNotification: 0x600003561e00>{fire date = Wednesday, July 8, 2020 at 12:37:21 PM Eastern European Summer Time, time zone = (null), repeat interval = 0, next fire date = Wednesday, July 8, 2020 at 12:37:21 PM Eastern European Summer Time, user info = {

Notifications work alright and fire on time with message and text. What am I missing?
    RNCPushNotificationIOS.scheduleLocalNotification({
      fireDate: new Date(Date.now() + 15000).toISOString(),
      alertTitle: 'hello',
      alertBody: 'message 2',
      userInfo: { id: 1231231 },
      alertAction: 'view',
    });

This is happening because from native side we do get
 data {"body": "message 2", "category": "", "date": "2020-07-08T14:01:34.374+03:00", "identifier": "54D00615-D28C-4DDD-940D-D53A84AAEA4C", "thread-id": "", "title": "hello", "userInfo": {"id": 1231231}}

and in @react-native-community/push-notification-ios/js/index.js:398 there are wrong fields being accessed ! :
else {
      // Local notifications aren't being sent down with `aps` dict.
      this._badgeCount = nativeNotif.applicationIconBadgeNumber;
      this._sound = nativeNotif.soundName;
      this._alert = nativeNotif.body;
      this._data = nativeNotif.userInfo;
      this._category = nativeNotif.category;
      this._fireDate = nativeNotif.date;
    }

@Naturalclar do you have any ideas why this is done this way?

Turns out it was firebase messing arround with notifications

@lukebars Could you kindly provide some hints on what you did to solve this? We are having the exact same issue. Cheers!

@1st8 it was AppDelegate.m setup.
Make sure your AppDelegate.m matches this:

- (BOOL)application:(UIApplication *)application didFinishLaunchingWithOptions:(NSDictionary *)launchOptions
{
   ...
  // Define UNUserNotificationCenter
  UNUserNotificationCenter *center = [UNUserNotificationCenter currentNotificationCenter];
  center.delegate = self;
   ...
}

// Required to register for notifications
- (void)application:(UIApplication *)application didRegisterUserNotificationSettings:(UIUserNotificationSettings *)notificationSettings
{
 [RNCPushNotificationIOS didRegisterUserNotificationSettings:notificationSettings];
}
// Required for the register event.
- (void)application:(UIApplication *)application didRegisterForRemoteNotificationsWithDeviceToken:(NSData *)deviceToken
{
 [RNCPushNotificationIOS didRegisterForRemoteNotificationsWithDeviceToken:deviceToken];
}
// Required for the notification event. You must call the completion handler after handling the remote notification.
- (void)application:(UIApplication *)application didReceiveRemoteNotification:(NSDictionary *)userInfo
fetchCompletionHandler:(void (^)(UIBackgroundFetchResult))completionHandler
{
  [RNCPushNotificationIOS didReceiveRemoteNotification:userInfo fetchCompletionHandler:completionHandler];
}
// Required for the registrationError event.
- (void)application:(UIApplication *)application didFailToRegisterForRemoteNotificationsWithError:(NSError *)error
{
 [RNCPushNotificationIOS didFailToRegisterForRemoteNotificationsWithError:error];
}
// Required for the localNotification event.
- (void)application:(UIApplication *)application didReceiveLocalNotification:(UILocalNotification *)notification
{
 [RNCPushNotificationIOS didReceiveLocalNotification:notification];
}

//Called when a notification is delivered to a foreground app.
-(void)userNotificationCenter:(UNUserNotificationCenter *)center willPresentNotification:(UNNotification *)notification withCompletionHandler:(void (^)(UNNotificationPresentationOptions options))completionHandler
{
  completionHandler(UNAuthorizationOptionSound | UNAuthorizationOptionAlert | UNAuthorizationOptionBadge);
}
- (NSURL *)sourceURLForBridge:(RCTBridge *)bridge
{
#if DEBUG
  return [[RCTBundleURLProvider sharedSettings] jsBundleURLForBundleRoot:@"index" fallbackResource:nil];
#else
  return [[NSBundle mainBundle] URLForResource:@"main" withExtension:@"jsbundle"];
#endif
}

@end

@lukebars Thanks for your reply.
I don't think that removing firebase completely is an option though.
Or how do you receive remote notifications now?

@lukebars Thanks for your reply.
I don't think that removing firebase completely is an option though.
Or how do you receive remote notifications now?

Remote notifications are still working for me. I might be wrong that you need to remove firebase stuff, but make sure that your notification related functions match those that I've wrote and test it out.

I was not able to find a fix for this.
My code is matching yours (and the docs) in the relevant parts, with the addition of https://rnfirebase.io/#configure-firebase-with-ios-credentials
I tried the Firebase didFinishLaunchingWithOptions additions both before and the additions for this package.
There is also this issue https://github.com/zo0r/react-native-push-notification/issues/1488 but I think this problem is better discussed here.
Could you reopen this issue?

Hi @1st8 .
Does your AppDelegate contain anything else firebase related than this?
#import <Firebase.h>
...

  if ([FIRApp defaultApp] == nil) {
     [FIRApp configure];
   }

Nope, it does not @lukebars
Thanks for following up, really appreciate it!

Reopened @1st8

Hi,
I think there is a property name flipping in the library depending on user-interaction:
https://github.com/react-native-community/push-notification-ios/blob/master/ios/RNCPushNotificationIOS.m#L94

  formattedLocalNotification[@"alertAction"] = RCTNullIfNil(notification.alertAction);
  formattedLocalNotification[@"alertTitle"] = RCTNullIfNil(notification.alertTitle);
  formattedLocalNotification[@"alertBody"] = RCTNullIfNil(notification.alertBody);
  formattedLocalNotification[@"applicationIconBadgeNumber"] = @(notification.applicationIconBadgeNumber);
  formattedLocalNotification[@"category"] = RCTNullIfNil(notification.category);
  formattedLocalNotification[@"repeatInterval"] = @(notification.repeatInterval);
  formattedLocalNotification[@"soundName"] = RCTNullIfNil(notification.soundName);
  formattedLocalNotification[@"userInfo"] = RCTNullIfNil(RCTJSONClean(notification.userInfo));
  formattedLocalNotification[@"remote"] = @NO;

https://github.com/react-native-community/push-notification-ios/blob/master/ios/RNCPushNotificationIOS.m#L121
  formattedNotification[@"title"] = RCTNullIfNil(content.title);
  formattedNotification[@"body"] = RCTNullIfNil(content.body);
  formattedNotification[@"category"] = RCTNullIfNil(content.categoryIdentifier);
  formattedNotification[@"thread-id"] = RCTNullIfNil(content.threadIdentifier);
  formattedNotification[@"userInfo"] = RCTNullIfNil(RCTJSONClean(content.userInfo));

Data returned in the JS part, local-notification pressed by user, is:
{
    "body": "My Notification Message",
    "category": "",
    "date": "2020-08-03T19:43:56.622+02:00",
    "identifier": "D0386218-CEE1-42D2-AD92-62A295631124",
    "thread-id": "",
    "title": "Local Notification",
    "userInfo": {"id": "1", "screen": "home"}
}

A quick fix can be made in the JS part, but is it the right thing to do ?
https://github.com/react-native-community/push-notification-ios/blob/master/js/index.js#L400
      this._badgeCount = nativeNotif.applicationIconBadgeNumber;
      this._sound = nativeNotif.soundName;
      this._alert = nativeNotif.alertBody || nativeNotif.body;
      this._title = nativeNotif?.alertTitle || nativeNotif?.title;
      this._data = nativeNotif.userInfo;
      this._category = nativeNotif.category || nativeNotif.categoryIdentifier;
      this._fireDate = nativeNotif.fireDate || nativeNotif.date;

One more thing, not tested, but I think there is also a mistake on the remote part:
          this._alert = notifVal.alert;
          this._title = notifVal?.alertTitle;

When nothing in the objective-c provide a property call alert it's body or alertBody.
Best regards,

Hey, @Naturalclar, can you share your thoughts on this?

It was working 2 weeks ago, but I think I was using a previous version, not the last.

This was fixed in #175 as long as you have updated your AppDelegate to use UNUserNotificationCenter


generated wrong code

Expected Behavior

With the following code snippet, after obfuscation, it reports undefined object.
!function() {
if (true) {
let name = "charles";
}
if (true) {
let {name, age} = {name: "foo", age: 100};
console.log(name);
}
}()
Current Behavior

With above code, the generated code is like below
( javascript-obfuscate  --compact false --seed 1)
var _0x3eb7 = [
'foo',
'log',
'charles'
];
(function (_0x1c3ed4, _0x5087a2) {
var _0x4a5a26 = function (_0x1c70bb) {
while (--_0x1c70bb) {
_0x1c3ed4'push';
}
};
_0x4a5a26(++_0x5087a2);
}(_0x3eb7, 0x119));
var _0x2857 = function (_0x590bb1, _0x2ddabf) {
_0x590bb1 = _0x590bb1 - 0x0;
var _0x280a78 = _0x3eb7[_0x590bb1];
return _0x280a78;
};
!function () {
if (!![]) {
let _0x4f5231 = _0x2857('0x0');
}
if (!![]) {
let {name, age} = {
'name': _0x2857('0x1'),
'age': 0x64
};
console_0x2857('0x2');
}
}();
when running the generated code,  it reports:
console_0x2857('0x2');
^
ReferenceError: _0x4f5231 is not defined
Your Environment

Obfuscator version used:  0.18.1
Node version used:  v8.9.4

Stack trace

Minimal working example that will help to reproduce issue

I think the issue with variable name, the obfuscator creates the wrong scope for it. Try not to use the same variable's name in function's scope twice.

I think the issue with variable name, the obfuscator creates the wrong scope for it. Try not to use the same variable's name in function's scope twice.

Yes, the obfuscator creates the wrong scope for the destructuring  assignment variables, if that variable name is used in some other scope.
We are now trying to obfuscate our existing project, which has many similar case , so trying avoiding this syntax is a lot of work .   This should be fixed in obfuscator.
Our workaround is to first use babel to transcode before obfuscator.  But this should really be fixed in the obfuscator so that we can remove the babel transcoder.

Hi!
Is there any news on this issue?

This problem is seriously preventing this obfuscator from a real project use.  Anybody working on it ?

I made Pull Request to solve this problem, the link above.

Thank you

0.18.3 version is out

Great. It's now working !


Determine data store solution

There's been talk of using Firebase, PostgreSQL, and Mongo.  The decisions needs to be made. If you accept this ticket you need to get buy-in from the group.

I'll take this on. I'm going to start by getting a prototype of #22 going to see what the structure of the event is, and whether it'd be appropriate for an SQL DB. If it is, I'm going to advocate for Postgres.


Cannot Export to REG (After Applying Exploit Protection Settings)

After enable a few settings in Exploit Protection in Microsoft Defender (Windows Defender), cannot export to REG file.

Hmm, I cannot reproduce this after enabling "use a common set of exploit protection settings" (Microsoft.Policies.ExploitGuard:ExploitProtection_Name, the only policy I see in Windows Components | Microsoft Defender Exploit Guard | Exploit Protection). What exactly are you trying to enable and what exactly happens when you try to export to a REG file?

I'm using Windows 10 20H2 and in the App and browser control> Exploit protection settings
I set everything to On by Default and them import the exploit protection available here https://github.com/jdgregson/Windows-10-Exploit-Protection-Settings
After restart the computer, stop working the export REG

Thank you for the details! It turned out to be possible to reproduce the issue independently of the exploit protection settings: there were some bugs in the handling of list policy elements that clear the list key before adding values. Those are addressed in https://github.com/Fleex255/PolicyPlus/commit/6233795cec5e287400735257f4b2d93648e69307. Please let me know if the updated dev build fixes the problem.

Will try the new version here and them back to say if fix or not the problem

additional information, for the users with attack surface reduction rules, if include the
Block executable files from running unless they meet a prevalence, age, or trusted list criterion - 01443614-cd74-433a-b99e-2ecdc07bfc25
Microsoft Defender block the PolicyPlus to run, you may need add to the exclusion of attack surface reduction rules to run or turn it off when you want to use, changing the value from 1 to 0. in gpedit.msc
Here's a picture

Thanks for the info! And of course thank you again for the bug report. I'm glad to see that this seems to be working as intended now.


Documentation and understanding why maps don't work

I'm trying to use the IGN map, and I supplied a key, but there may be special parameters required. Do you have documents explaining how to do this?
It would be nice to have access to a request/response log view to be able to understand why a given map doesn't work.
My perspective is that I used to develop an Android-native map app, MyTrails, and I'm trying to find a successor. I was thinking of using React Native, but Nativescript doesn't sound too dissimilar. I really like AlpiMaps so far 👍 .

@paour thanks IGN is a special beast in the sense that it depends on what you subscribed for in the pro plan. I will try with my key to ensure all works fine.
As for the logs the production build does not produce any on purpose to be as fast and lightweight (battery wise) as possible.
Also keep in mind that alpimaps real potential is with offline generated data as explained in the readme.
About N yes a hundred time stronger than RN. We are a small community but we are moving fast

@paour i tested IGN and it works fine. What i can you is that as a protection i set the User-Agent to 'AlpiMaps' for IGN.
It works fine for my keys where i add the security on User-Agent.
Maybe you can try that


cloaning process not successful

after branch, there is a colon (:) sign at the end of the folder 04 _02b.  then, unable to further after this not even type anything.

Yes the same happening with me, when I re-start the Terminal, it was not finding gulp so I did a local install of gulp. When I run gulp I get assert.js:350
throw err;
^
AssertionError [ERR_ASSERTION]: Task function must be specified

This course needs an update, not just because of the gulp issue, but because of changes within the framework. It's on my schedule to re-do, but I don't have a due date to give you right now.


Error in setup/deploy. Proxy Healthcheck Fails even if web role is running.

Hi there!
kamal version 2.2.2
I'm trying to implement Kamal as our deployment tool. We achieved in Kamal 1.9.x with a Rails Application. Now I'm trying to do the same with Kamal 2.x and a Rails App (7.1.2). But I cannot do it. It always fails in the healthcheck step, even when I see the logs and my app is running.
config/deploy.yml
service: my-app

image: my-app/my-app

servers:
  web:
    hosts:
      - xx.xxx.xx.xx
    proxy:
      app_port: 3000
      healthcheck:
        interval: 3
        path: /healthcheck
        timeout: 3

registry:
  server: registry.digitalocean.com
  username:
    - DOCKER_REGISTRY_TOKEN
  password:
    - DOCKER_REGISTRY_TOKEN

env:
  secret:
    - RAILS_MASTER_KEY
    - SECRET_KEY_BASE
    - DOCKER_REGISTRY_TOKEN
    - REVISION

builder:
  arch: amd64
  args:
    SECRET_KEY_BASE: "<%= ENV['SECRET_KEY_BASE'] %>"
    REVISION: "<%= ENV['REVISION'] %>"
  secrets:
    - SECRET_KEY_BASE
    - REVISION

aliases:
  console: app exec --reuse -i "bin/rails console"

Dockerfile
ARG RUBY_VERSION=3.2.2
ARG NODE_VERSION=22.1.0

FROM node:${NODE_VERSION}-alpine3.18 AS node
FROM ruby:${RUBY_VERSION}-alpine3.16

COPY --from=node /usr/lib /usr/lib
COPY --from=node /usr/local/share /usr/local/share
COPY --from=node /usr/local/lib /usr/local/lib
COPY --from=node /usr/local/include /usr/local/include
COPY --from=node /usr/local/bin /usr/local/bin

RUN apk update \
  && apk upgrade \
  && apk add --update --no-cache \
  build-base curl-dev postgresql-dev \
  tzdata yarn imagemagick \
  font-adobe-source-code-pro font-noto \
  git

RUN mkdir -p /app
WORKDIR /app

ENV RAILS_ENV production
ENV NODE_ENV production

COPY Gemfile Gemfile.lock /app/

ARG BUNDLER_VERSION=2.5.10
RUN gem update --system
RUN gem install bundler -v ${BUNDLER_VERSION}
RUN bundle config set --local deployment 'true'
RUN bundle config set --local without 'development:test'
RUN bundle config set --local jobs $(nproc)

ARG SECRET_KEY_BASE
ARG REVISION

# Load env
ENV LANG=C.UTF-8 \
  BUNDLE_JOBS=4 \
  BUNDLE_RETRY=3 \
  RAILS_ENV=production \
  NODE_ENV=production \
  SECRET_KEY_BASE=${SECRET_KEY_BASE} \
  RAILS_MASTER_KEY=${SECRET_KEY_BASE} \
  APP_REVISION=${REVISION}

RUN bundle install

COPY package.json yarn.lock /app/
RUN yarn install

COPY . /app

# This folder is needed so propshaft can compile all stylesheets
RUN mkdir -p ./app/assets/builds
RUN bundle exec rake assets:precompile

EXPOSE 3000

CMD ["bundle", "exec", "rails", "s", "-b", "<IP_ADDRESS>"]

Logs of kamal setup
  INFO [c7ce6aff] Running /usr/bin/env mkdir -p .kamal/apps/my-app/env/roles on xx.xxx.xx.xx
  INFO [c7ce6aff] Finished in 0.560 seconds with exit status 0 (successful).
  INFO Uploading .kamal/apps/my-app/env/roles/web.env 100.0%
  INFO [60da576a] Running docker run --detach --restart unless-stopped --name my-app-web-54a09b7d34881bcf229655b9ea26bb0e11db3c5a --network kamal --hostname xx.xxx.xx.xx-33bd4aaba7c7 -e KAMAL_CONTAINER_NAME="my-app-web-54a09b7d34881bcf229655b9ea26bb0e11db3c5a" -e KAMAL_VERSION="54a09b7d34881bcf229655b9ea26bb0e11db3c5a" --env-file .kamal/apps/my-app/env/roles/web.env --log-opt max-size="10m" --label service="my-app" --label role="web" --label destination registry.digitalocean.com/my-app/my-app:54a09b7d34881bcf229655b9ea26bb0e11db3c5a on xx.xxx.xx.xx
  INFO [60da576a] Finished in 2.056 seconds with exit status 0 (successful).
  INFO [93dc4b22] Running docker container ls --all --filter name=^my-app-web-54a09b7d34881bcf229655b9ea26bb0e11db3c5a$ --quiet on xx.xxx.xx.xx
  INFO [93dc4b22] Finished in 0.671 seconds with exit status 0 (successful).
  INFO [7ff5e920] Running docker exec kamal-proxy kamal-proxy deploy my-app-web --target="6f45b922864d:3000" --deploy-timeout="30s" --drain-timeout="30s" --health-check-interval="3s" --health-check-timeout="3s" --health-check-path="/healthcheck" --buffer-requests --buffer-responses --log-request-header="Cache-Control" --log-request-header="Last-Modified" --log-request-header="User-Agent" on xx.xxx.xx.xx
 ERROR Failed to boot web on xx.xxx.xx.xx
  INFO First web container is unhealthy on xx.xxx.xx.xx, not booting any other roles
  INFO [e29b2635] Running docker container ls --all --filter name=^my-app-web-54a09b7d34881bcf229655b9ea26bb0e11db3c5a$ --quiet | xargs docker logs --timestamps 2>&1 on xx.xxx.xx.xx
  INFO [e29b2635] Finished in 1.259 seconds with exit status 0 (successful).
 ERROR 2024-10-31T19:02:07.774177335Z => Booting Puma
2024-10-31T19:02:07.774258825Z => Rails <IP_ADDRESS> application starting in production
2024-10-31T19:02:07.774263978Z => Run `bin/rails server --help` for more startup options
2024-10-31T19:02:32.083991580Z Puma starting in single mode...
2024-10-31T19:02:32.084313149Z * Puma version: 5.6.4 (ruby 3.2.2-p53) ("Birdie's Version")
2024-10-31T19:02:32.084424907Z *  Min threads: 5
2024-10-31T19:02:32.084542515Z *  Max threads: 5
2024-10-31T19:02:32.084883296Z *  Environment: production
2024-10-31T19:02:32.085029239Z *          PID: 1
2024-10-31T19:02:32.108918708Z * Listening on http://<IP_ADDRESS>:3000
2024-10-31T19:02:32.196310698Z Use Ctrl-C to stop
  INFO [2b457d89] Running docker container ls --all --filter name=^my-app-web-54a09b7d34881bcf229655b9ea26bb0e11db3c5a$ --quiet | xargs docker inspect --format '{{json .State.Health}}' on xx.xxx.xx.xx
  INFO [2b457d89] Finished in 0.627 seconds with exit status 0 (successful).
 ERROR null
  INFO [6538a4bd] Running docker container ls --all --filter name=^my-app-web-54a09b7d34881bcf229655b9ea26bb0e11db3c5a$ --quiet | xargs docker stop on xx.xxx.xx.xx
  INFO [6538a4bd] Finished in 1.504 seconds with exit status 0 (successful).
  Finished all in 74.7 seconds
Releasing the deploy lock...
  Finished all in 80.5 seconds
  ERROR (SSHKit::Command::Failed): Exception while executing on host xx.xxx.xx.xx: docker exit status: 1
docker stdout: Nothing written
docker stderr: Error: target failed to become healthy

I've tried to increase the servers.web.proxy.healthcheck.interval and servers.web.proxy.healthcheck.timeout but still is not working. Always I get that the container is unhealthy and following that error I see that my Puma Server is running.
I achieved deploy successfully disabling the proxy server, but that is not the idea...
servers:
  web:
    hosts:
      - <IP_ADDRESS>
    proxy: false

Currently we deploy our app with docker and K8s and we are trying to migrate to Kamal. And the docker and the App is running correctly.
Can you help me with this ??

@tomassiNico I believe your proxy config is in the wrong place.
Documentation mentions the following:

The proxy is configured in the root configuration under proxy

Your config should look something like:
proxy:
  app_port: 3000
  healthcheck:
    interval: 3
    path: /healthcheck
    timeout: 3

servers:
  web:
    hosts:
      - xx.xxx.xx.xx

Hope this helps.

Hi @Rohland, thanks for your help!
I tried changing my config/deploy.yml as you said but still I got the same problem :(

@tomassiNico can you tell how much time is elapsing between the start of the kamal-proxy deploy (that was the [7ff5e920] section in your example above), and the "Booting puma" line?
Your settings should allow for 30s before the deploy action fails, but I'm wondering if that's what you're seeing happen here. Is it possible that app is taking longer than that to start, for some reason?

hi @kevinmcconnell, I found the problem the healthcheck was been lasting too much until my app connect to my database. That was the problem.
I could fix this problem increasing the timeout: 30 for my app and my deploy_timeout: 120 .
Thanks for the help


ci: add workflow to comment new Opened PRs

Description
This PR  adds a workflow that analyzes the PRs and comments them with a small summary of the changeset (if doesn't pushes outside the ctx limits!)

A stray ' in the word what's was terminating the shell script improperly. I've slightly reworded the prompt to try and avoid this?


store references to failed Rogue rbs

What's this PR do?
Adds reportbacks that were successfully send to Rogue via cron to the dosomething_rogue_reportbacks table. Includes some helper methods to expedite that both in cron and when we send reportbacks to Rogue via the web form.
How should this be reviewed?
Make sure you can submit reportbacks normally.
Submit a reportback that ends up in the failed table. Run cron. Make sure that reportback ends up both in Rogue and in the dosomething_rogue_reportbacks table.
Any background context you want to provide?
We need all these little dudes to live in this table so they know that they made it!
Relevant tickets
Fixes #7223
Checklist

[ ] Documentation added for new features/changed endpoints.
[ ] Tested on staging.

@katiecrane can you synccccc with @chloealee on her PR, I think this one will cause conflicts with hers

📷

🚥 🍏


[BUG]Should we use Locale.ENGLISH for monthName/dayName function

What is the bug?
The monthName/dayName function uses Locale.getDefault to get the display name, which may return different results in different env and cause a test error.
org.opensearch.sql.sql.DateTimeFunctionIT > testMonthName FAILED
    java.lang.AssertionError:
    Expected: iterable with items [[September]] in any order
         but: not matched: <["九月"]>
        at __randomizedtesting.SeedInfo.seed([3027A175091234F:5336A188EBBE09E2]:0)
        at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:18)
        at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:6)
        at org.opensearch.sql.util.MatcherUtils.verify(MatcherUtils.java:173)
        at org.opensearch.sql.util.MatcherUtils.verifyDataRows(MatcherUtils.java:149)
        at org.opensearch.sql.sql.DateTimeFunctionIT.testMonthName(DateTimeFunctionIT.java:801)

Should we change the locale to Locale.ENGLISH?
BTW, I found that the locale of from_unixtime function is Locale.ENGLISH.

@LantaoJin friendly ping:)

Thanks for reporting this @bugmakerrrrrr .
MonthName and DayName are MySQL-like functions. I found this https://dev.mysql.com/doc/refman/8.0/en/server-system-variables.html#sysvar_lc_time_names. So in my opinion, it's not a bug as I said in https://github.com/opensearch-project/sql/pull/2685#issuecomment-2236908496. But I am not quite sure that should we continue to follow all MySQL principles. cc @dai-chen @penghuo.

Thanks for reporting this @bugmakerrrrrr . MonthName and DayName are MySQL-like functions. I found this https://dev.mysql.com/doc/refman/8.0/en/server-system-variables.html#sysvar_lc_time_names. So in my opinion, it's not a bug as I said in #2685 (comment). But I am not quite sure that should we continue to follow all MySQL principles. cc @dai-chen @penghuo.

Thanks for you context. I found that the MySQL document said that the default value is 'en_US' regardless of your system's locale setting, so if we need to follow the MySQL principle, should we change the locale to english?

so if we need to follow the MySQL principle, should we change the locale to english?

That's what I am not quite sure. OpenSearch SQL doesn't provide a system function/variable to set a lc_time_name, if we change it to Locale.ENGLISH, it will restrict to English.


flip version to 0.1.2-snopshot

@ZhijunZhao, flip version to 0.1.2-snapshot after release

@yungez,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

Codecov Report

Merging #6 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master       #6   +/-   ##
=======================================
  Coverage   77.59%   77.59%
=======================================
  Files          15       15
  Lines         424      424
  Branches       56       56
=======================================
  Hits          329      329
  Misses         57       57
  Partials       38       38

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 9c08213...2a4c1d5. Read the comment docs.


Ability to parse non HTTP logs

Sorry if this question was asked before.
I have logs produced by our product (custom debug logs really). I was hoping to use goaccess with that after seeing that I can specify various formats. However, from the description is claims it is HTTP specific. Can it be done: to customize to parse arbitrary logs?
Thanks.

Right now goaccess requires the following fields, so if you have any of those, you should be able to get data out of your log that goaccess can parse.
a valid IPv4/6 %h
a valid date %d
the request %r

There are some requests to have some custom fields #1202 and panels, so going forward they will add more flexibility as to what data can parse, the plumbing is already there so it shouldn't be too bad. Hope that helps!

Messages are typically like this:
2021-02-09T19:05:08.468899-05:00  FOO                 BAR:                Info     Message

Timestamp                                         Node               Component    Error    Message text.
Name              Name              Level
No IP or request in the log above. The source distinction is really the node name.
I can process the log with sed to add a fake IP field, I suppose.  But where would the fake request go?

You could try using Message as the request. e.g.,
goaccess log --log-format='%dT%t.%^ %v %h %^ %U' --date-format=%Y-%m-%d --time-format=%T --no-ip-validation


CIA Builds can't check if files exist

The method of checking if files exist on the SD card, does not work with romfs.

596530357be467fb22271c9a2d3cac4e93499bde should've fixed this.


Built-in timing and benchmarking

Give code timings by default, click on it to benchmark that cell.

Julia JIT compiles, so show <0.12 s for a first run, 0.12 s for a second.
120 ± 3 ms for a benchmark result.


ADDING SEARCH BAR/BOX FOR QUCIK SEARCHING OF THE APPS.

### ADDING SEARCH BAR/BOX FOR QUCIK SEARCHING OF THE APPS.
Adding a search bar to your webpage is a powerful way to enhance user experience by providing quick and seamless access to applications, content, and key information. A well-integrated search function allows users to navigate your site effortlessly, improving usability and driving engagement.
Here’s how a search bar can be highly beneficial:
Improves Navigation: Users can easily locate specific content or applications without scrolling through the entire webpage, saving time and enhancing efficiency.
Boosts User Engagement: With faster access to relevant information, users are more likely to stay on the site, explore further, and interact with additional content.
Streamlines Content Discovery: It helps surface hidden or less visible content, ensuring that users can access all valuable resources, increasing the site's overall value.
Increases Conversion Rates: By helping users find exactly what they’re looking for quickly, a search bar can lead to higher satisfaction, driving more conversions and actions.
Enhances Personalization: A search bar can gather insights into user behavior, allowing you to tailor content suggestions and improve the overall user journey.

As an experienced frontend developer, I am confident that I can resolve this issue and significantly enhance the page's functionality and user experience. By implementing improvements, I will elevate the design and usability by 10x. I kindly request that you assign this task to me, and I will ensure it is executed with precision and efficiency.
~thankyou

@dnyanesh99borse can you share any image or screenshot of your idea?

Sure, here it is.
#BEFORE
[image: image.png]
#ASSUMING:
[image: image.png]
I would like to add a search box to the page or header to enable users
to quickly locate elements or applications across the website. This feature
will significantly enhance user experience by providing a seamless way to
search for and access content, ensuring efficient navigation and improving
overall usability.
On Tue, Oct 1, 2024 at 9:35 PM Rahul Mahato @.***>
wrote:

@dnyanesh99borse https://github.com/dnyanesh99borse can you share any
image or screenshot of your idea?
—
Reply to this email directly, view it on GitHub
https://github.com/iamrahulmahato/master-web-development/issues/62#issuecomment-2386413844,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BCIMNV7PS363NIRTFHYHPVLZZLB3BAVCNFSM6AAAAABPFXHT7WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBWGQYTGOBUGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

@dnyanesh99borse image is not visible, bro

Really sorry for that ..., Sure, here it is.
### #BEFORE

### #ASSUMING:

I would like to add a search box to the page or header to enable users to quickly locate elements or applications across the website. This feature will significantly enhance user experience by providing a seamless way to search for and access content, ensuring efficient navigation and improving overall usability.

By integrating a search box, users will be able to directly access specific content or applications on the webpage with ease. This feature will streamline navigation, allowing users to quickly find what they’re looking for without having to browse through multiple sections, thereby improving the overall user experience and efficiency of the site.

Sorry to inform you, man, I want to keep the website design minimal. you can work on other issue

Ok, thanks for the reply.
On Tue, Oct 1, 2024 at 10:37 PM Rahul Mahato @.***>
wrote:

Sorry to inform you, man, I want to keep the website design minimal. you
can work on other issue
—
Reply to this email directly, view it on GitHub
https://github.com/iamrahulmahato/master-web-development/issues/62#issuecomment-2386536763,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BCIMNV47T45Q6HQMEFP6WYDZZLJGHAVCNFSM6AAAAABPFXHT7WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBWGUZTMNZWGM
.
You are receiving this because you were mentioned.Message ID:
@.***>


Error: Cannot find module

Using this starter - https://github.com/gatsbyjs/gatsby-starter-hello-world
Followed the directions to setup this theme and no matter how I configure it, using the sample data or the template provided, "gatsby develop" produces the error below every time and crashes.
Error: Cannot find module '../node_modules/gatsby-theme-guitar-book/src/assets/icon.png'

@john-rock probably using this starter is not the best the way to start with guitar-book plugin. If you copy the template directory and open it as a separate project it should work. The template directory is kind of starter so it doesn't require any other setup or starters.

I fixed this error, so with gatsby-theme-guitar-book, version 1.0.1-alpha.11 it should work.
I also add repo with just simple starter (it is same as it is in template directory but easier to start).

@Jozwiaczek Awesome thank you I will check this out later.
Great idea too, looking forward to future releases.


What about factual errors in alpaca_data.json?

Would PRs to main fixing errors in the output field of data set items be welcome? If so, the readme should clearly state that the data set is work in progress and researchers should refer to a specific commit number or release in their publications. Or would you prefer to collect corrections in a special branch?
Here an example for an error:
    {
        "instruction": "Find the second derivative of the following equation: y = 4x^2 + 3x \u2013 5",
        "input": "",
        "output": "The second derivative of the given equation is 8x + 3."
    },

That's the first derivative. The second one would be just 8.
(Also, one could argue that the use of EM DASH (U+2013) as a minus sign in the instruction field is an error. However, it's a good thing to train the model to be able to deal with such errors as they can be expected to occur in real life user inputs.)
In addition, or as an alternative, to fixing entries one could extend the data format to give each item an ID and author field, the latter set to self-instruct-on-text-davinci-003 for the initial data, a date field, an optional field derived_from referencing the ID of another item that the item is based on and derived_by that specifies in what way the new item is derived, e.g. fixing an error or varying some details such as the minus sign in the above example.
Building a simple web app for users who do not have the git/github skills but would like to contribute corrections or new/derived data set items may also pay off given the wide interest this technology currently receives.
Opinions?
Edit: suggested value for author field

    {
        "instruction": "Illustrate the phrase \"no way out\".",
        "input": "No input required",
        "output": "![No Way Out](https://www.wallfoy.com/wallpaper/661/no-way-out.jpg)"
    },

URLs are not really guaranteed to be valid or safe, but maybe answering with random URLs is good enough for training these kind of things.
The URL in this data leads to some unexpected and sketchy looking redirects. Is this the intended illustration of the phrase? I can't imagine the experience of being trapped in an endless redirect was the intention here. If it was the intention, this artist work may be a better / safer example https://permanent-redirect.xyz/
side-note: what are the implications for theory of mind when a class of possible future 'intelligences' have been built on models trained with hyperlinks leading to phishing or other exploits? perhaps this will lead to unique evolutionary/survival traits or hidden aspects of personality. It probably just means that some future boston dynamics dog weapon could have a 0.000000001% chance of evaluating this part of its language model. Does that mean that it would end up transfering all of its money to a prince if the dissidents that it is hunting yell 'no way out' before being assassinated? careful


RHBZ#2112559: VirtIO-FS: set guest side owner UID/GID to Everyone

Allow every user in guest system act like files and folders owner.
Remove session change handling and UID/GID change logic.

Confirmed works on my AD setup. Thanks a million!

Confirmed works on my AD setup. Thanks a million!

@Daniel-Trevitz Thank you for testing!


timezone/daylight savings

I have an ical with timezone UTC+1, and there is on top of this daylight savings for my local settings. Now here is the issue - my calendar shows an event today at 19:00-21:00 - but the magicmirror shows it at 18:00-20:00. How do you adjust for this? I have been looking all around for this...

Well, first, which calendar program are you using for making ics feed? Because Daylight savings information should be adjusted in ical format itself. As I know, normal ical file record time with UTC format with time gap like 19980119T070000Z, I suspect your feed is not proper.
But I'll search more about this, and if possible, I'll provide a manual patch for this. I hope.

I noticed when i recurring events on a google calendar the time doesn't display correctly. if i change it to a 1-time event it doesn't have an issue.


fix(simd): update wabt.js and remove/unskip simd tests

Update wabt.js, which suppors new opcodes, so we can unskip some tests.
Remove 2 outdated tests simd_boolean.{0,1}.wasm, which used old opcodes
that are now undefined.

Welcome! Please help me to merge it in, I do not have permissions, thanks :)

:tada: This PR is included in version 5.0.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


SSL documentation is misleading

At http://progrium.viewdocs.io/dokku/nginx it says:
PER APP
To enable TLS connection to to one of your applications, copy or symlink the .crt and .key files into the application's /home/dokku/:app/tls folder (create this folder if it doesn't exist) as server.crt and server.key respectively.
Redeployment of the application will be needed to apply TLS configuration. Once it is redeployed, the application will be accessible by https:// (redirection from http:// is applied as well).
However, at #1114 you can see I had to use
ssh<EMAIL_ADDRESS>nginx:import-ssl < server.tar myapp
Can the documentation please be updated so that the correct steps for per-app https are listed? Thanks!

@sglebs pull requests for documentation changes are more than welcome.

I would be glad to fix the docs but I have not been able to figure out the right steps. I mean, I am trying to do the same thing for multiple domains, but it fails:
tar cvf ~/Downloads/dokku-staging-certificates.tar ~/Downloads/.cert ~/Downloads/.key
dokku mqm$ ssh<EMAIL_ADDRESS>nginx:import-ssl < ~/Downloads/dokku-staging-certificates.tar core-server-staging
tar: /Users/mqm/Downloads/._27817776_staging.core.namoromobile.com.cert
Removing leading `/' from member names
Users/mqm/Downloads/27817776_staging.core.namoromobile.com.cert
/Users/mqm/Downloads/._30459181_get.v1.staging.core.namoromobile.com.cert
Users/mqm/Downloads/30459181_get.v1.staging.core.namoromobile.com.cert
/Users/mqm/Downloads/._3795813_get.staging.core.namoroon.com.cert
Users/mqm/Downloads/3795813_get.staging.core.namoroon.com.cert
/Users/mqm/Downloads/._48373792_get.v1.staging.core.namoroon.com.cert
Users/mqm/Downloads/48373792_get.v1.staging.core.namoroon.com.cert
/Users/mqm/Downloads/._67223910_v1.staging.core.namoroon.com.cert
Users/mqm/Downloads/67223910_v1.staging.core.namoroon.com.cert
/Users/mqm/Downloads/._84123428_staging.core.namoroon.com.cert
Users/mqm/Downloads/84123428_staging.core.namoroon.com.cert
/Users/mqm/Downloads/._9243972_v1.staging.core.namoromobile.com.cert
Users/mqm/Downloads/9243972_v1.staging.core.namoromobile.com.cert
/Users/mqm/Downloads/._9737983_get.staging.core.namoromobile.com.cert
Users/mqm/Downloads/9737983_get.staging.core.namoromobile.com.cert
/Users/mqm/Downloads/._27817776_staging.core.namoromobile.com.key
Users/mqm/Downloads/27817776_staging.core.namoromobile.com.key
/Users/mqm/Downloads/._30459181_get.v1.staging.core.namoromobile.com.key
Users/mqm/Downloads/30459181_get.v1.staging.core.namoromobile.com.key
/Users/mqm/Downloads/._3795813_get.staging.core.namoroon.com.key
Users/mqm/Downloads/3795813_get.staging.core.namoroon.com.key
/Users/mqm/Downloads/._48373792_get.v1.staging.core.namoroon.com.key
Users/mqm/Downloads/48373792_get.v1.staging.core.namoroon.com.key
/Users/mqm/Downloads/._67223910_v1.staging.core.namoroon.com.key
Users/mqm/Downloads/67223910_v1.staging.core.namoroon.com.key
/Users/mqm/Downloads/._84123428_staging.core.namoroon.com.key
Users/mqm/Downloads/84123428_staging.core.namoroon.com.key
/Users/mqm/Downloads/._9243972_v1.staging.core.namoromobile.com.key
Users/mqm/Downloads/9243972_v1.staging.core.namoromobile.com.key
/Users/mqm/Downloads/._9737983_get.staging.core.namoromobile.com.key
Users/mqm/Downloads/9737983_get.staging.core.namoromobile.com.key
Tar archive missing server.crt
Does anybody know how to make SSL work for an app with multiple domains? I was only able to make it work for a single domain. Maybe a workaround is to tar one by one, but that would be very tedious.
Ideas?

Doing manually for each key/cert pair worked (tar each pair, etc). PITA. Should I document it this way?

I made the doc change in this pull request: https://github.com/progrium/dokku/pull/1201
(sorry I was not able to make a pull request just with the doc change - an old vmware fix also survived)


[feature request] german and english audio?

Not sure if possible, but it would be nice to have both audio tracks in one video file.

Unfortunately, you'd have to download the full videos in both languages and can't just download the audio. So yes, technically it would be possible to add the other audio stream with FFMPEG, but at that point you may as well just download it in both languages IMO.

I'm going to close the issue for now since I don't see how this feature could be implemented efficiently. Thanks for the request though and feel free to open a new issue if you know how this could be implemented efficiently.


Validation Rules types

I can't find method documentation for method validate which is using /validation-rules/${encode(validationRuleId)}/validation api endpoint.

I've updated the types accordingly


Add ESLint config with all latest v0.24 rules

ESLint run

Most of these warnings/errors can be very quickly fixed
Also I did not do it here, but it is very cool to have eslint-loader added to webpack, so you can see warnings while you develop in realtime. Will add PR after this one.

I feel like this is way too much for the scope of what we have right now. I have local eslint config set up, it prevents a few common mistakes, but adding 200+ lines of lint makes no one wants to go back changing it; and I doubt it'll catch much useful lints.

@chenglou it actually catches pretty much all most common mistakes. It is large because it has all rules in there with descriptions. It may be unnecessary when you work alone but as we found in our company it works pretty well if you have distributed team. Additional comments are also very helpful to figure out why something is failing without going to the webz
It is up to you of course. Feel free to close. I just highly recommend to have a code style in place to avoid mistakes made by other contributors. And yourself as well =)

It is also very good to have when you add CI, so PR will just fail if it has style issues in there

I agreed with @chenglou, this is way too much and didn't see too much benefit from it. something like use const instead var because the value never changed is also opinionated and slow down the development in this early stage.

I'd just adopt a widely spread convention (e.g. airbnb javascript eslint rules).
Like @DenisIzmaylov did here: https://github.com/gaearon/redux/blob/4f620e6aa5d21a78c7e624725230a84645588c12/.eslintrc#L2

I really think we need this even at this stage. It will save us a lot of headaches down the road. Adopting AirBnB’s rules as @gaearon suggested I think is a very good idea.

@nkbt would you mind doing that? What @gaearon said

@chenglou for sure, np

@chenglou done, see updated code.

Great! Thanks =)

\o/


fix infinite setState onChangeDate

resolve infinte rendering issue on onChange Date
Related: https://github.com/acro5piano/react-native-big-calendar/issues/712

Thanks for your pull request!
  React.useEffect(() => {
    if (onChangeDate) {
      onChangeDate([dateRange[0].toDate(), dateRange.slice(-1)[0].toDate()])
    }
- }, [dateRange, onChangeDate])
+ }, [dateRange])

+ /* eslint-disable */
+ }, [dateRange])

I have tried it but it's not working

 /* eslint-disable */

by the way working MR is ready I have disable eslint for that specific part you can merge it now

@mubashiralisiddiqui Thanks! I'll confirm it and merge this ASAP

Thank you for your contribution!


Significant difference in simulation time between local environment and CI Pipeline

Background information gathered
I'm trying to run the test suite for the Black Parrot using Verilator on my local machine. There is a substantial difference in execution time compared to the CI pipeline on the GitLab repository. Specifically, it takes about 1886 minutes (~31 hours) locally, whereas the CI pipeline completes it in about 80 minutes. I assume this gap could be due to the differences in simulation tools (Verilator vs. VCS) and the available hardware (32 cores on my end vs. 144 cores on the CI pipeline).
Question
Could these factors account for such a significant difference in execution time? Could you provide any recommendations on optimizing my testing environment for faster performance (e.g., specific configurations or tool versions that might help decrease runtime on my end)?
Thank you in advance for any guidance you can provide.

Yeah VCS tends to be ~5-10x faster than Verilator for our designs (though it's very likely there's low-hanging fruit to speed it up).
One check first, you are setting the CI scripts to use those 32 cores, right?
The best way to speed up Verilator is to feed it more RAM -- it's ravenous. Our CI machine has hundreds of gigs. The multicore designs in particular will be massively slowed down by swapping

Oh, I see. Thanks a lot for your help, I appreciate it. I will try your suggestions.

No problem, definitely contribute back if you find any Verilator tricks!


not working for text files

Hi,
vscode-todo-parser doesn't add TODOs in simple text files (e.g. .txt or .graphql).
This would be a desirable default, since otherwise arbitrary TODOs in the project get lost, since they don't appear in the List. As a developer I don't want to memorize the list of supported file types and remember them each time I write a TODO into my files.

According to https://github.com/kantlove/vscode-todo-parser/blob/master/src/types/LanguageType.ts this plugin uses the Java Regex for plain text, which is defined at https://github.com/kantlove/vscode-todo-parser/blob/master/src/const/RegexStrings.ts as /*+([\s\S]*?)*/|//([^\r\n]+) so it should work when using a double slash (can't get it to work) or the /* */ notation, but I think there's a . missing in the first part of the regex... so I don't see how this should be possible to match...


TPU Client

is there or will be support for connecting to Solana blockchain using a TPU Client?

Probably not in this package. This sounds like it should be a standalone library.


I am having the same issue, but  built it outside of visual studio 2012.

I also get the 'HEAD' error.  You say I am supposed to have 'git' installed?  Where do I get 'git'? I don't see any place to download 'git' for .NET.  There are 5 or 6 .NET add-ins that reference 'git'.  Which one do I use?  I have never used git before, except to download zips and use things.  I really would like to also use this in a project, but already had an issue with:
The build is referencing a jar file that is x.12 and it receives a 404 error.  The jar file is actually x.13.  I could fix that in the build file and continue.  But I cannot get past the 'git' issue.  Please help!

Looks like the ReadMe could use some improvement.

Mention the git requirement and how to install git if you don't already have it.

@aahock Looks like you opened another issue for the same problem #52 I think the best solution is to update the readme and work towards making the build not care if git is missing.


dotnet aspnet-codegenerator identity Unrecognized option '-lf' but '--listFiles' OK

From @billschen on June 20, 2018 8:7
First I run command dotnet aspnet-codegenerator identity -h to show help
--files|-fi
List of semicolon separated files to scaffold. Use the --list-files option to see the available options
--listFiles|-lf
Lists the files that can be scaffolded by using the '--files' option.
then I run command dotnet aspnet-codegenerator identity --listFiles

But when I use '-lf' become Unrecognized:

Copied from original issue: dotnet/cli#9518

Closing, as this is a duplicate of #788


fix on macOS where strcasecmp not declared in scope

In tensorflow/io#309 was getting an error of "strcasecmp' was not declared in this scope".
Seems like we need either string.h or cstring on other platforms other than linux as well

n.b. Interesting, the macOS build (https://travis-ci.org/tensorflow/io/jobs/547994854) over on the tensorflow/io repo passed. But this failed because of this missing when building on my local machine.
I assume it's an issue for macOS build here, but something magical is happening on that other travis build........

Although not likely to happen, but have you validated if this PR so that it won't break the Travis build on the tensorflow/io repo?

Yes, this is the latest update on that repo https://github.com/tensorflow/io/pull/309. I'm currently using HEAD here and applying this as a patch on the latest build. See https://travis-ci.org/tensorflow/io/builds/549500080 with latest build on that repo

Great, then I don't have any concerns. Thanks for the contribution!


TLS local issuer certificate error

This happens when sending the websocket connection using the method described in the documentation. The same error occurs when using DXFeedStreamer and DXLinkStreamer.
Traceback (most recent call last):
  File "/Users/user/PycharmProjects/project/main.py", line 63, in <module>
    asyncio.run(main())
  File "/Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/asyncio/runners.py", line 194, in run
    return runner.run(main)
           ^^^^^^^^^^^^^^^^
  File "/Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/asyncio/runners.py", line 118, in run
    return self._loop.run_until_complete(task)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/asyncio/base_events.py", line 664, in run_until_complete
    return future.result()
           ^^^^^^^^^^^^^^^
  File "/Users/user/PycharmProjects/project/main.py", line 52, in main
    async with DXFeedStreamer(session) as streamer:
  File "/Users/user/Library/Caches/pypoetry/virtualenvs/EcDFx_qL-py3.12/lib/python3.12/site-packages/tastytrade/streamer.py", line 294, in __aenter__
    raise TastytradeError('Connection timed out')
tastytrade.utils.TastytradeError: Connection timed out
Task exception was never retrieved
future: <Task finished name='Task-2' coro=<DXFeedStreamer._connect() done, defined at /Users/user/Library/Caches/pypoetry/virtualenvs/EcDFx_qL-py3.12/lib/python3.12/site-packages/tastytrade/streamer.py:318> exception=SSLCertVerificationError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1000)')>
Traceback (most recent call last):
  File "/Users/user/Library/Caches/pypoetry/virtualenvs/EcDFx_qL-py3.12/lib/python3.12/site-packages/tastytrade/streamer.py", line 325, in _connect
    async with websockets.connect(  # type: ignore
  File "/Users/user/Library/Caches/pypoetry/virtualenvs/EcDFx_qL-py3.12/lib/python3.12/site-packages/websockets/legacy/client.py", line 629, in __aenter__
    return await self
           ^^^^^^^^^^
  File "/Users/user/Library/Caches/pypoetry/virtualenvs/EcDFx_qL-py3.12/lib/python3.12/site-packages/websockets/legacy/client.py", line 647, in __await_impl_timeout__
    return await self.__await_impl__()
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/user/Library/Caches/pypoetry/virtualenvs/EcDFx_qL-py3.12/lib/python3.12/site-packages/websockets/legacy/client.py", line 651, in __await_impl__
    _transport, _protocol = await self._create_connection()
                            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/asyncio/base_events.py", line 1126, in create_connection
    transport, protocol = await self._create_connection_transport(
                          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/asyncio/base_events.py", line 1159, in _create_connection_transport
    await waiter
  File "/Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/asyncio/sslproto.py", line 575, in _on_handshake_complete
    raise handshake_exc
  File "/Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/asyncio/sslproto.py", line 557, in _do_handshake
    self._sslobj.do_handshake()
  File "/Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/ssl.py", line 917, in do_handshake
    self._sslobj.do_handshake()
ssl.SSLCertVerificationError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1000)

Process finished with exit code 1

How to reproduce
from tastytrade import ProductionSession, DXFeedStreamer
from tastytrade.dxfeed import EventType

import os
import asyncio

async def main():

    session = ProductionSession('username', 'password')

    # get the current price of ES and NQ Futures
    async with DXFeedStreamer(session) as streamer:
        await streamer.subscribe(EventType.QUOTE, symbols=['SPY', 'QQQ'])
        quotes = []
        async for quote in streamer.listen(EventType.QUOTE):
            quotes.append(quote)
            if len(quotes) == 2:
                break
        print(quotes)

if __name__ == '__main__':
    asyncio.run(main())

I'm not sure if maybe this is related to the TLS cutoff?

Oddly had to go run the "Install Certificates" executable. https://stackoverflow.com/questions/50236117/scraping-ssl-certificate-verify-failed-error-for-http-en-wikipedia-org


Add Hot-Reload support

TUM-Live Server should serve the current GOHTML pages when they are changed.
Currently a server restart is needed.

Some context: currently the application embeds the .gohtml files into the binary, so restarting tum Live is required whenever something changed. When developing tum Live, these templates should be read from disk (there's a handy variable, VersionTag that indicates whether we're on a dev maschine or on prod).
The templates are also parsed using template.Must(...) which crashes tum Live at startup time if there are errors in the template, maybe this should be changes for dev envs as well.
Additionally, tailwind assets are currently not included in the build dev configuration, this would be cool as well.


Rename Adventure arguments

This essentially makes these changes to make arguments that deal with the same Minecraft argument type have identical names:

Spigot-related argument changes:

- dev.jorel.commandapi.arguments.ChatArgument
- dev.jorel.commandapi.arguments.ChatColorArgument
- dev.jorel.commandapi.arguments.ChatComponentArgument

+ dev.jorel.commandapi.arguments.spigot.ChatArgument
+ dev.jorel.commandapi.arguments.spigot.ChatColorArgument
+ dev.jorel.commandapi.arguments.spigot.ChatComponentArgument

Adventure-related argument changes:

- dev.jorel.commandapi.arguments.AdventureChatArgument
- dev.jorel.commandapi.arguments.AdventureChatColorArgument
- dev.jorel.commandapi.arguments.AdventureChatComponentArgument

+ dev.jorel.commandapi.arguments.adventure.ChatArgument
+ dev.jorel.commandapi.arguments.adventure.ChatColorArgument
+ dev.jorel.commandapi.arguments.adventure.ChatComponentArgument

The one mildly annoying thing with this PR is the developer experience. Developers now need to ensure that they import the "right one". There are no compile-time checks that will catch the case if a developer accidentally uses the wrong argument. (I'm only saying "mildly annoying", because the CommandAPI does have cases where it has classes which have the same name as other cases, such as NamespacedKey)
Not something worth rejecting the PR, just something to be aware of (of course, the fix for this is trivial on the developers point of view, and runtime errors will occur which should be detected during developer testing, so it's "easily identifiable"). We'll have to ensure that the documentation fully covers this to ensure that users don't mix them up (we should now have an intermediary "chat arguments" page, we can probably put a note about this there so developers know which package to import)

The one mildly annoying thing with this PR is the developer experience. Developers now need to ensure that they import the "right one". There are no compile-time checks that will catch the case if a developer accidentally uses the wrong argument. (I'm only saying "mildly annoying", because the CommandAPI does have cases where it has classes which have the same name as other cases, such as NamespacedKey)

Yeah, it is a little confusing. Similar things happened to me when I accidentally import java.awt.List instead of java.util.List.
Maybe this could be similar to those NamespacedKey examples. There could be a ChatArgument/ChatColorArgument... class for the Spigot case and ChatArgument.Adventure/ChatColorArgument.Adventure... for Adventure. Or, since Spigot doesn't have to be the 'default', there could be ChatArgument.Spigot/ChatColorArgument.Spigot... instead.
Going that way would pretty much be no change at all though. If it's not a big change, it's probably not going to solve the problem you are trying to solve.

Not something worth rejecting the PR, just something to be aware of (of course, the fix for this is trivial on the developers point of view, and runtime errors will occur which should be detected during developer testing, so it's "easily identifiable"). We'll have to ensure that the documentation fully covers this to ensure that users don't mix them up (we should now have an intermediary "chat arguments" page, we can probably put a note about this there so developers know which package to import)

As Skepter said, documentation could probably help with any possible confusion with the way it is right now.

Alright, I guess, this should be done.
Once again, the only thing I am really not sure about is documentation. In this case, the chatpreview.md page as that refers to different platform implementation several times. I personally think I found a good compromise between only naming the class name and mentioning the platform every time too.
So yeah, a code review would be appreciated for the whole PR :D

Just a few typos and small details I noticed. Otherwise, everything makes sense.
I'm still not sure what problem this is trying to solve. I've never used the chat arguments though, so I'm not sure how developers want to use them. One challenge I see with the setup here is providing support for Spigot and Adventure. Right now, if you want to use both ChatArguments in the same file, you need to use the fully qualified name (eg. dev.jorel.commandapi.arguments.spigot.ChatArgument) for at least one of them, which doesn't seem convenient. I don't know if anyone would want to do that, but this is currently happening in the ChatArgumentTests files.
I think placing the arguments as static classes like ChatArgument.Spigot and ChatArgument.Adventure fits better with existing Arguments (Eg. SoundArgument and SoundArgument.NamespacedKey both use the minecraft:resource_location Brigadier argument). I don't know though. The best solution probably depends on the problem that this is trying to solve.
If we think different packages is the best solution, this PR looks good to me 👍.

First, thanks for the review!
I am not really solving a problem with this since the arguments work fine the way they currently are, I just thought it would be nice if arguments that provide essentially the same functionality had the same name. My initial discord message
Of course, when I originally had the idea I didn't think about the case where someone would like to use both arguments in the same class (why would someone even want to do this, if you got Adventure, use Adventure).
Given that 10.0.0 isn't quite releasing any time soon, we still can discuss this package vs. inner class thing, I personally would continue to use this but using inner class would be fine as well.

Superseded by #517


chore(deps): add support for httpx 0.28

Pulls in patch (https://github.com/lundberg/respx/pull/278) for RESPX to support HTTPX 0.28 release. This is temporary and I will pin main RESPX as depedency again once this or a siilar fix is merged to master and released.

RESPX version 0.22.0 released


Example Travis Pro config?

I've tried to wrap my head around how to connect to a Travis Pro account with this wrapper but aside from the token setter, I couldn't find much.
Do you have an example you could add to the README? Thanks. :smiley:

This project is not ready to be used yet, unfortunately.
On Tue, Jan 27, 2015 at 6:09 AM, Olivier Lacan<EMAIL_ADDRESS>wrote:

I've tried to wrap my head around how to connect to a Travis Pro account
with this wrapper but aside from the token setter, I couldn't find much.
Do you have an example you could add to the README? Thanks. [image:
:smiley:]
—
Reply to this email directly or view it on GitHub
https://github.com/travis-ci/travis.js/issues/5.

Shucks. :-)
On January 28, 2015 at 4:05:22 PM, Konstantin Haase<EMAIL_ADDRESS>wrote:
This project is not ready to be used yet, unfortunately.
On Tue, Jan 27, 2015 at 6:09 AM, Olivier Lacan<EMAIL_ADDRESS>wrote:

I've tried to wrap my head around how to connect to a Travis Pro account
with this wrapper but aside from the token setter, I couldn't find much.
Do you have an example you could add to the README? Thanks. [image:
:smiley:]
—
Reply to this email directly or view it on GitHub
https://github.com/travis-ci/travis.js/issues/5.

—
Reply to this email directly or view it on GitHub.

I plan to revisit the idea of an official JS client soonish.
On Wed, Jan 28, 2015 at 5:41 PM, Olivier Lacan<EMAIL_ADDRESS>wrote:

Shucks. :-)
On January 28, 2015 at 4:05:22 PM, Konstantin Haase (
<EMAIL_ADDRESS>wrote:
This project is not ready to be used yet, unfortunately.
On Tue, Jan 27, 2015 at 6:09 AM, Olivier Lacan<EMAIL_ADDRESS>wrote:

I've tried to wrap my head around how to connect to a Travis Pro account
with this wrapper but aside from the token setter, I couldn't find much.
Do you have an example you could add to the README? Thanks. [image:
:smiley:]
—
Reply to this email directly or view it on GitHub
https://github.com/travis-ci/travis.js/issues/5.

—
Reply to this email directly or view it on GitHub.
—
Reply to this email directly or view it on GitHub
https://github.com/travis-ci/travis.js/issues/5#issuecomment-71868315.

@rkh Any news on this?


[Slide] Convert to function component

[x] I have followed (at least) the PR section of the contributing guide.

Breaking Change
Child must be able to hold a ref.

@joshwooding Branch has conflicts


use globalization for HeaderText

Globalization does not work for HeaderText in mvcgrid.config
how can i use Resources files for  HeaderText?

Hello shrdshrd  you found a solution to the problem?

Hi, did someone find any solution? :-)

I just merged in #35 which added the ability to customize the text in the configuration.


Lambda timesout while iterating over each permission set/account

https://github.com/aws-samples/aws-identitycenter-codepipeline-auto-assignment/blob/e41e5a5d0848e2e67ff6ef2f266bc2ee2903994a/src/lambda-code/identity-center-auto-assign/auto-assignment.py#L28

Hi,
Thanks for reporting the issue.
In general Lambda has the "15 minutes" hard timeout limit so if you have more than 250 permission sets, the function will probably takes more than 15 mins to complete. And in that case, you customize the solution to run the scripts in a ECS container or EC2 instance to get unblocked from the function timeout limit.


feat(types): add optional padding parameter to fitBounds method

fitBounds methods now supports an optional padding parameter.
https://developers.google.com/maps/documentation/javascript/reference/map#Map.fitBounds

Any progress on fixing the checks?
Seems the issue thrown is TypeError: Invalid attempt to destructure non-iterable instance. A quick google didn't provide much help, and I can't see anything wrong at a glance in the 2 places that the arguments are destructured (here).
Keen to get this type change merged 😄

@scoobster17 That error isn't related to this PR, it's reproducible on master. Also this repo seems to have died, there hasn't been a release in over a year. I've been using my fork with the new types ever since with no issues. You're welcome to use it as well.

@andreiho @scoobster17 please check npm @react-google-maps/api


#374: Upgrade go-jsonnet to 0.10.0

We can't take commits unless you sign your commits. Please review https://github.com/ksonnet/ksonnet/blob/master/CONTRIBUTING.md.