Update node SDK to use v0.15 of the JS SDK

This should be available by 4/10. This will most likely break the following: https://github.com/Microsoft/AppInsights-node.js/blob/master/applicationInsights.ts#L62-L69
The contains a few fixes to JS in addition to supporting the latest bond schemas https://github.com/Microsoft/bond.

New contracts are here: https://github.com/Microsoft/ApplicationInsights-node.js/blob/feature/v15-schema-and-auto-collection/Library/Contracts.ts

this is fixed in version 0.15


Error:- cannot find cgroup plugin for cgroup/v2, slurmd initialization failed

i am trying to create a sclurm cluster on kubernetes (azure kubernetes service). but the slurmd pod keeps crashing giving error "Couldn't find the specified plugin name for cgroup/v2 looking at all files". Have mentioned the errors below for pods  slurmctld and  slurmd.

I have tried to debug it a lot but no luck. Any idea on how to fix this on kubernetes cluster.

i could see that slurmdb is unable to connect with slurmctld as shown below.

> k logs -f slurmdbd-6f59cc7887-4mwwq

slurmdbd: debug2: _slurm_connect: failed to connect to <IP_ADDRESS>:6817: Connection refused
slurmdbd: debug2: Error connecting slurm stream socket at <IP_ADDRESS>:6817: Connection refused
slurmdbd: error: slurm_persist_conn_open_without_init: failed to open persistent connection to host:<IP_ADDRESS>:6817: Connection refused
slurmdbd: error: slurmdb_send_accounting_update_persist: Unable to open connection to registered cluster linux.
slurmdbd: error: slurm_receive_msg: No response to persist_init
slurmdbd: error: update cluster: Connection refused to linux at <IP_ADDRESS>(6817)

> k logs -f slurmctld-0

slurmctld: debug:  sched: Running job scheduler for full queue.
slurmctld: debug:  create_mmap_buf: Failed to open file `/var/spool/slurmctld/job_state`, No such file or directory
slurmctld: error: Could not open job state file /var/spool/slurmctld/job_state: No such file or directory
slurmctld: error: NOTE: Trying backup state save file. Jobs may be lost!
slurmctld: debug:  create_mmap_buf: Failed to open file `/var/spool/slurmctld/job_state.old`, No such file or directory
slurmctld: No job state file (/var/spool/slurmctld/job_state.old) found
slurmctld: debug2: accounting_storage/slurmdbd: _send_cluster_tres: Sending tres '1=40,2=10,3=0,4=10,5=40,6=0,7=0,8=0' for cluster
slurmctld: debug2: Testing job time limits and checkpoints
slurmctld: debug:  slurm_recv_timeout at 0 of 4, recv zero bytes
slurmctld: error: slurm_receive_msg [<IP_ADDRESS>:7132]: Zero Bytes were transmitted or received
slurmctld: debug2: Processing RPC: MESSAGE_NODE_REGISTRATION_STATUS from UID=0
slurmctld: debug2: found existing node slurmd-1 for dynamic future node registration
slurmctld: debug2: dynamic future node slurmd-1/slurmd-1/slurmd-1 assigned to node slurmd-1
slurmctld: debug2: _slurm_rpc_node_registration complete for slurmd-1
slurmctld: debug:  slurm_recv_timeout at 0 of 4, recv zero bytes
slurmctld: error: slurm_receive_msg [<IP_ADDRESS>:38712]: Zero Bytes were transmitted or received
slurmctld: debug2: Processing RPC: MESSAGE_NODE_REGISTRATION_STATUS from UID=0
slurmctld: debug2: found existing node slurmd-0 for dynamic future node registration
slurmctld: debug2: dynamic future node slurmd-0/slurmd-0/slurmd-0 assigned to node slurmd-0
slurmctld: debug2: _slurm_rpc_node_registration complete for slurmd-0
slurmctld: debug2: Testing job time limits and checkpoints
slurmctld: debug2: Performing purge of old job records
slurmctld: debug:  sched: Running job scheduler for full queue.
slurmctld: debug2: Testing job time limits and checkpoints
slurmctld: debug2: Testing job time limits and checkpoints
slurmctld: debug2: Performing purge of old job records
slurmctld: debug:  sched: Running job scheduler for full queue.
slurmctld: debug2: Testing job time limits and checkpoints

> k logs -f pod/slurmd-0
---> Set shell resource limits ...
core file size          (blocks, -c) unlimited
data seg size           (kbytes, -d) unlimited
scheduling priority             (-e) 0
file size               (blocks, -f) unlimited
pending signals                 (-i) 3547560
max locked memory       (kbytes, -l) unlimited
max memory size         (kbytes, -m) unlimited
open files                      (-n) 131072
pipe size            (512 bytes, -p) 8
POSIX message queues     (bytes, -q) 819200
real-time priority              (-r) 0
stack size              (kbytes, -s) unlimited
cpu time               (seconds, -t) unlimited
max user processes              (-u) unlimited
virtual memory          (kbytes, -v) unlimited
file locks                      (-x) unlimited
---> Copying MUNGE key ...
---> Starting the MUNGE Authentication service (munged) ...
---> Waiting for slurmctld to become active before starting slurmd...
-- slurmctld is now active ...
---> Starting the Slurm Node Daemon (slurmd) ...
slurmd: CPUs=96 Boards=1 Sockets=2 Cores=48 Threads=1 Memory=886898 TmpDisk=0 Uptime=37960 CPUSpecList=(null) FeaturesAvail=(null) FeaturesActive=(null)
slurmd: debug:  Log file re-opened
slurmd: debug2: hwloc_topology_init
slurmd: debug2: hwloc_topology_load
slurmd: debug2: hwloc_topology_export_xml
slurmd: debug:  CPUs:96 Boards:1 Sockets:2 CoresPerSocket:48 ThreadsPerCore:1
slurmd: error: Couldn't find the specified plugin name for cgroup/v2 looking at all files
slurmd: error: cannot find cgroup plugin for cgroup/v2
slurmd: error: cannot create cgroup context for cgroup/v2
slurmd: error: Unable to initialize cgroup plugin
slurmd: error: slurmd initialization failed

Went through the same issue, it seems that the provided image ghcr.io/stackhpc/slurm-docker-cluster does not contain the build of the cgroup v2 plugin:
sh-4.4# ls /usr/lib64/slurm/cgroup*
/usr/lib64/slurm/cgroup_v1.a  /usr/lib64/slurm/cgroup_v1.la  /usr/lib64/slurm/cgroup_v1.so

only cgroup_v1 is supported in this image.
I will check how to build an image with cgroup_v2 or go down with cgroup_v1

After adding the cgroups_v2.so plugin I am getting:
slurmd: error: cgroup_dbus_attach_to_scope: cannot connect to dbus system daemon: Failed to connect to socket /run/dbus/system_bus_socket: No such file or directory
slurmd: error: _init_new_scope_dbus: scope and/or cgroup directory for slurmstepd could not be set.
slurmd: error: cannot initialize cgroup directory for stepds: if the scope /sys/fs/cgroup/kubepods.slice/kubepods-besteffort.slice/kubepods-besteffort-poda8507382_5d59_4c0a_8119_c6651b541881.slice/system.slice/slurmstepd.scope already exists it means the associated cgroup directories disappeared and the scope entered in a failed state. You should investigate why the scope lost its cgroup directories and possibly use the 'systemd reset-failed' command to fix this inconsistent systemd state.
slurmd: error: Couldn't load specified plugin name for cgroup/v2: Plugin init() callback failed
slurmd: error: cannot create cgroup context for cgroup/v2
slurmd: error: Unable to initialize cgroup plugin
slurmd: error: slurmd initialization failed

Probably better to disable cgroups by configuration.

Hi, I faced the same issue when setting up a k3s cluster on GCP. The problem stems from the cgroup version on the VM. My VM was running on a Debian environment. Here’s how I solved it:

Issue: cgroup/v1
First, check the cgroup version:
$ mount | grep cgroup
cgroup2 on /sys/fs/cgroup type cgroup2 (rw,nosuid,nodev,noexec,relatime,nsdelegate,memory_recursiveprot)

The VM defaulted to cgroup v2, so I had to switch it back to cgroup v1. To do this, edit the GRUB configuration:
$ sudo nano /etc/default/grub

Add this line to set the system to use cgroup v1:
GRUB_CMDLINE_LINUX="... systemd.unified_cgroup_hierarchy=0"

Update the GRUB configuration:
$ sudo update-grub
$ sudo reboot

After reboot, verify the cgroup version again:
$ mount | grep cgroup
cgroup on /sys/fs/cgroup/cpu,cpuacct type cgroup (rw,nosuid,nodev,noexec,relatime,cpu,cpuacct)
cgroup on /sys/fs/cgroup/memory type cgroup (rw,nosuid,nodev,noexec,relatime,memory)

Install NFS on Slurm client nodes:
sudo apt-get update
sudo apt-get install -y nfs-common

Once these steps were complete, I was able to redeploy the repository, and the Slurm daemon pods started running successfully. This should help you resolve the issue!


Clockface menu improvements

As discussed here, instead of having separate "menu item" functions, do require("ClockFace_menu").addItems(menu, callback, items) instead.
And then I figured that if clocks don't have any other settings, it would be nice to get rid of the Storage boilerplate as well, so you can just do require("ClockFace_menu").addSettingsFile(menu, settingsFile, ["showDate", "loadWidgets"])

That looks great, thanks! I reckon that'll make a huge difference to clocks.


[AutoDiff] Improve invalid stored property projection diagnostics.

Add a request that resolves the "tangent stored property" corresponding to an
original stored property in a Differentiable-conforming type.
Improve non-differentiability diagnostics regarding invalid stored
property projection instructions:
struct_extract, struct_element_addr, ref_element_addr.
Diagnose the following cases:

Original property's type does not conform to Differentiable.
Base type's TangentVector is not a struct.
Tangent property not found: base type's TangentVector does not have a
stored property with the same name as the original property.
Tangent property's type is not equal to the original property's
TangentVector type.
Tangent property is not a stored property.

Previously, these cases were undiagnosed and crashed with assertion/verification failures.
Resolves TF-969 and TF-970.

@swift-ci Please smoke test


shell.openItem，Can't get the file launcher

Electron version: all
Operating system: all

Can't get the file launcher,So open the program is actually not know.
I want to get the initiator's path, so the use of selective "open appPath filePath" open way, rather than having to default.
Excuse me, can you provide relevant interface to obtain the default launcher path？

There isn't currently any API for this I believe in Electron


Honor pre 1.0.0 release breaking changes

According to semver the API may change with releases before the 1.0.0 release. https://semver.org/#spec-item-4
I would still like to track the breaking changes being made to the API using conventional commits so we can easily communicate to early adopters that the API did change.
The current behavior is to perform a major release version bump on all BREAKING CHANGES. I propose that for versions < 1.0.0 that BREAKING CHANGE commits only trigger a minor release.

Hi @nathanielc ! Thank you very much for your feedback.

The current behavior is to perform a major release version bump on all BREAKING CHANGES. I propose that for versions < 1.0.0 that BREAKING CHANGE commits only trigger a minor release.

If follow this rule, how do I change the version from < 1.0.0 to 1.0.0 ? always manually ?

Yes,it would always be a manual process. that way the developers explicitly choose when to say that their API is now complete and will no longer break in future minor releases.


How to know the emitting target of onWatch event?

How to know, which target (dev, md, or web) has fired the onWatch event?
I wish to handle the filepath different suitable the calling watch task target.
    grunt.event.on('watch', function(action, filepath) {
        // Handle filepaths concerning their calling target
        switch (callingTarget) {
            case "dev":
                ...  // handle the filepath
            case "md":
                ...
            case "web":
                ...
            }
    });

This is the sample config:
        watch: {
            options: {
                spawn: false,
            },
            dev: {
                files: [
                    "<%= dir.src %>/**/*.ts",
                    "!<%= dir.src %>/**/*.d.ts"
                ],
                tasks: [ "build:dev", "clean:dev", "copy:dev" ]
            },
            md: {
                files: [
                    "<%= dir.src %>/**/*.md"
                ],
                tasks: [ "string-replace:md" ]
            },
            web: {
                files: [
                    "<%= dir.web %>/**/*.html"
                ],
                options: {
                    livereload: 35728
                }
            }
        }

Solution is to simple:

    grunt.event.on('watch', function(action, filepath, callingTarget) {
        // Handle filepaths concerning their calling target
        switch (callingTarget) {
            case "dev":
                ...  // handle the filepath
            case "md":
                ...
            case "web":
                ...
            }
    });


Issue with Dyson and IQVIA in 98.5

Home Assistant release with the issue:

0.98.3
Operating environment (Hass.io/Docker/Windows/etc.):

Component/platform:

Description of problem:
Dyson component fails to setup.
IQVIA fails to pull down data for most configured sensors.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
dyson:
username<EMAIL_ADDRESS>password: RANDOMGARBLE
language: US
devices:
- device_id: NM7-US-JXXXXX # eg. Serial number: XXX-XX-XXXXXXXX
device_ip: 10.x.x.x.
iqvia:
zip_code: "98034"
Traceback (if applicable):
Sat Sep 14 2019 14:26:20 GMT-0700 (Pacific Daylight Time)
iqvia: Error on device update!
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 292, in _async_add_entity
await entity.async_device_update(warning=False)
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 439, in async_device_update
await self.async_update()
File "/usr/src/homeassistant/homeassistant/components/iqvia/sensor.py", line 156, in async_update
[period] = [p for p in data["periods"] if p["Type"] == key]
ValueError: not enough values to unpack (expected 1, got 0)
Sat Sep 14 2019 14:26:16 GMT-0700 (Pacific Daylight Time)
Setup failed for dyson: Integration failed to initialize.

**Additional information:**

Dyson account is working as I can connect to it via my phone app.

Same for me. Hoping this is just a server issue, as it worked before with the same config.

Closing as another thread although newer than mine, has more traction with users.


Trojan:Win32/Zpevdo.B

Affected items:
file: C:\Program Files (x86)\Myrtille\bin\wfreerdp.exe
2020-10-16 07:52:12,065 [4] INFO  System.Diagnostics redirection [(null)] - Connecting remote session *, type RDP, security AUTO, server (:port) *, vm (none), domain (none), user , program (none)
2020-10-16 07:52:12,518 [4] ERROR System.Diagnostics redirection [(null)] - Failed to start the host client process, remote session * (System.ComponentModel.Win32Exception (0x80004005): Operation did not complete successfully because the file contains a virus or potentially unwanted software
at System.Diagnostics.Process.StartWithCreateProcess(ProcessStartInfo startInfo)
at Myrtille.Services.RemoteSessionProcess.StartProcess(Guid remoteSessionId, HostType hostType, SecurityProtocol securityProtocol, String serverAddress, String vmGuid, String userDomain, String userName, String startProgram, Int32 clientWidth, Int32 clientHeight, Boolean allowRemoteClipboard, Boolean allowPrintDownload, Boolean allowAudioPlayback))

Hi,
Thanks for your report. Yes, I also saw that a few days ago, a Windows update of Windows Defender decided that FreeRDP is a threat. This is a false positive of course, but it breaks Myrtille by quarantining the FreeRDP executable. The next entry you will have in Myrtille.Services.log is that wfreeedp.exe is missing (...)
This is what I posted a fews ago on the Myrtille group (I should had created an issue here after that):

I was reported today a zpevdo.B trojan detection by Windows Defender on wfreerdp.exe (the rdp client used by Myrtille) after a Windows update on a Windows Server 2016 machine.
This is a false positive, as already stated this summer by the FreeRDP team (https://github.com/FreeRDP/FreeRDP/issues/6336) and is probably due to the fact FreeRDP uses low level APIs calls to handle remote access. The Virus Total scan of wfreerdp.exe, as bundled into Myrtille 2.9.0, doesn't report this threat (even if there are 3/68 heuristic detections).
So, don't worry if you have such detection on FreeRDP. Problem is, Windows Defender may quarantine wfreerdp.exe, thus breaking Myrtille. If this happens, you can safely restore it and add a security exclusion for it so that it doesn't happen anymore. You can also use another anti-virus (as a double check in case such a thing happen).

This happened on a Windows Server 2016. On which OS did you had this problem ?

Hello, i`m not sure that this was decide by mistake, because ~10 days aolder builds have not this issue. Had on windows server 2019.
Anyway is it possible to take screen shot from remote desktop with pure php via curl or some manipulation with sockets?
Best regards

It must be due to a recent Windows update (less than 10 days), which in turn updated the Windows Defender threats database. That's why this issue didn't happened before.
It might be due to the fact the FreeRDP executable is not signed (same issue here: https://github.com/stride3d/stride/issues/285 or here). I will see to do that, along with the other myrtille binaries (including the installer).
Meanwhile, false positives can be reported to Microsoft here: https://www.microsoft.com/en-us/wdsi/filesubmission
To reply to your last question, it would be difficult taking screenshots with php, moreover in virtualized sessions, and with a huge loss of performance if even possible.

I verified this happen since Windows Defender 1.325.805.0 (latest: https://www.microsoft.com/en-us/wdsi/defenderupdates).
As a side note, Windows Defender can be configured to prompt the user for action instead of quarantining the file silently...

I reported the false detection to Microsoft which reacted quite switfly. It should be fine now:

@The-Compiler you should also do a report at https://www.microsoft.com/en-us/wdsi/defenderupdates
@Jonerrok could you please update Windows Defender to definition 1.325.945.0 and re-scan (myrtille install path)\bin\wfreerdp.exe?
I will anyway sign the Myrtille executables and installer in the next release to avoid such troubles

@The-Compiler you should also do a report at https://www.microsoft.com/en-us/wdsi/defenderupdates

I assume you mean https://www.microsoft.com/en-us/wdsi/filesubmission, right?

Yes (wrong link copy&paste)

Turns out it doesn't actually get detected anymore - I guess whatever fixed it for your project also fixed it for mine. Thanks for taking care of it!

Hi!
Today I tried to install Myrtille 2.9.0.
Got the same Windows Defender detection so I uploded the .msi to virustotal.com: 16 different engines detected various malwares. Probably false positives, but should I be worried?

Hi @AceMoneus thanks for your input. MSI files aren't executable files, they are a database containing compressed data for msiexec.exe, the Windows installer engine. It's like having an .mp4 video file detected as a virus, although it's run by an external player and doesn't contain any executable code by itself.
After installation, the Myrtille binaries (exe and dlls) aren't detected by any AV (except wfreerp.exe, as a false positive).
The Myrtille 2.8.1 MSI (and the versions before) wasn't detected by any AV. Nothing changed in 2.9.0 except the false positive detection of wfreerdp; so I guess it must have come from that. As explained, FreeRDP does many things low level, a recent feature might have triggered this detection (for example usbredirect, recently added to FreeRDP).
I will continue to investigate. I already checked my build machine and it's clean (I guess the other executables would have been infected otherwise, which is not the case).

By the way, did you updated Windows Defender (via Windows Updates) to latest definition? (>= 1.325.945.0)

Ah, ok. I was thinking of msi:s as a sort of kompressed folders. Like most antivirus agents do open zip-files and examine the content.
Actually, I don't know my version. Where do I see that? Under "Virus & threats" it only says that I'm using Microsoft Defender.
I have all the updates that MS Update gives me, so at least I want to think that I'm on the latest of everything...
Anyway, your explanation seems legit so I'll go for it. Thx 👍

https://www.bleepingcomputer.com/tutorials/how-to-find-windows-defender-version-number-installed-in-windows-10/

Closing now. I will sign the Myrtille executables (including the FreeRDP fork) into a next version and re-check then.


update pub DEP to include build/serve deprecation notice

Commit https://github.com/dart-lang/pub/commit/875d35005a7d33f367d70a3e31e9d3bad5d1ebd8

CL https://dart-review.googlesource.com/c/sdk/+/47682


Seeking advice on how to choose between ACT and DP algorithms

Hello,
Thank you very much for the work you have done in bringing together the current excellent imitation learning collections for convenient use. Regarding the ACT algorithm and DP algorithm, besides the basic differences in the algorithms themselves, how should one choose between them for different tasks? Do they have specific types of tasks they are particularly suited for? I have just started using your project and am unsure how to select the appropriate algorithm. I would greatly appreciate any advice you can provide.
Thank you!

DP is going to be more compute heavy as it typically needs multiple forward passes through the model. ACT is much faster, even when outputting longer horizons. So ACT might be the way to go for faster control loops.
DP has some nice flexibility in conditioning via inpainting. We don't have this feature yet, but we can use inpainting to warm-start or constrain parts of temporally adjacent action chunks.
Word is on the street that DP might be better at handling multimodality? I'd take this with a grain of salt and try both ways.
Our DP has a 1D CNN backbone which might make it harder to deal with sudden changes in dynamics, for example when making contact. The paper covers this, and proposes a transformer backbone to mitigate this issue. but we don't have it implement yet. If you have sharp transitions in dynamics around critical points of your trajectory (eg hanging keys on a hook) you might want ACT for its transformer backbone.

@alexander-soare Thank you very much for your response, it has been incredibly helpful. I can prioritize trying one approach in practice. Thank you again for your guidance.

@alexander-soare Thank you very much for your response, it has been incredibly helpful. I can prioritize trying one approach in practice. Thank you again for your guidance.In practice, we also found that ACT is indeed less flexible than DP. However, ACT learns certain details better than DP, such as control over angles and distances. Do you have any good suggestions or methods to improve the flexibility of ACT? Our attempts to enhance images or add small amounts of noise to the trajectories have not yielded satisfactory results.

@le-wei thanks for the tidbits!
When you say "improve the flexibility of ACT" can I check what you mean exactly? I think by "enhance images and add small amounts of noise" it seems you are referring to test-time generalization? We do have a new data augmentation feature which we have experimentally been shown to improve results https://github.com/huggingface/lerobot/pull/234

"Thank you very much for your advice, I have learned a lot from your suggestions." @alexander-soare

"Have you considered standardizing the data representation in ACT and DP? This way, the generated dataset can adapt to both strategies without requiring significant code modifications."
ACT:

DP:

@le-wei it's not well documented yet, but this is part of the "standard" format. We allow "observation.image" if there is one image. If there is more than one image we require "observation.images.x", "observation.images.y" etc. It's an experiment and we will be more careful about documenting it once we think it's the way to go.
Does that help answer your question?

Thank you very much for your patient explanation. @alexander-soare

Great, well, I'll close this issue and you're more than welcome to reopen if it's not resolved or open another one.


Attempting to perform action on non-existing agent

BUG REPORT
See:
https://github.com/huawei-noah/SMARTS/blob/c071d5349a8046ef512e4deb523c900eff5b56b5/examples/multi_agent.py#L46-L53
Observation is generated for done agents on the last step when the agent becomes done. This done agent's observation is processed in the next environment iteration. A sample print-out of the simulation state is shown below for better illustration.
-----------------------------------------------
obs before =  dict_keys(['Agent 0', 'Agent 1'])
actions =  {'Agent 0': 'keep_lane', 'Agent 1': 'keep_lane'}
dones =  {'Agent 0': True, 'Agent 1': False, '__all__': False}
obs after=  dict_keys(['Agent 0', 'Agent 1'])
===============================================

-----------------------------------------------
obs before =  dict_keys(['Agent 0', 'Agent 1'])
actions =  {'Agent 0': 'keep_lane', 'Agent 1': 'keep_lane'}
WARNING:SMARTS:Attempted to perform actions on non-existing agent, Agent 0
dones =  {'Agent 1': False, '__all__': False}
obs after=  dict_keys(['Agent 1'])
===============================================

-----------------------------------------------
obs before =  dict_keys(['Agent 1'])
actions =  {'Agent 1': 'keep_lane'}
dones =  {'Agent 1': False, '__all__': False}
obs after=  dict_keys(['Agent 1'])
===============================================

SMARTS version
0.4.15
Previous associated issues
None
Steps to reproduce the bug
Run the following:
$ cd /path/to/SMARTS/
$ python3.7 ./examples/multi_agent.py ./scenarios/intersections/6lane/

Resulting and expected behaviour
Result:

Warning message is printed: WARNING:SMARTS:Attempted to perform actions on non-existing agent, Agent_001
Attempted to step done agent

Expected:

No message printed.
Don't attempt to step done agent.

Error logs and screenshots

System information

Operating system & Environment [Please note if using Docker]: Ubuntu 16.04
Number of system cores available: 16
Python version [Please note if using Jupyter]: Python3.7
Python packages [e.g. from pip freeze]
SUMO version: 1.8.0
Date when bug occurred: 19 April 2021

Impact [If known]

Unnecessary consideration of done agent

Also experienced in ULTRA's train.py and evaluate.py when simulating multiple ego agents.

In hindsight, having trained multi-agent multi-environment examples, printing a warning to alert the user of done agents might be better than silently ignoring the done agents in step.

Has this problem been solved, please? I'm having this problem too.


feat(extras): add neogen

Can we add the following config for Neogen?
Differences with the proposed one.

One shortcut to generate the annotations. The plugin automatically selects the type of annotation.
Automatically select the snippet engine in the following priority order: "nvim", "luasnip", "snippy", and "vsnip"

return {
  "danymat/neogen",
  dependencies = "nvim-treesitter/nvim-treesitter",
  cmd = "Neogen",
  keys = {
    {
      "<leader>ga",
      function()
        require("neogen").generate({})
      end,
      desc = "Generate Annotations",
    },
  },
  ---@param _ LazyPlugin
  ---@param opts table
  opts = function(_, opts)
    local snippet_engine = nil

    if vim.snippet ~= nil then
      snippet_engine = "nvim"
    else
      local util = require("lazyvim.util")
      if util.has("luasnip") then
        snippet_engine = "luasnip"
      elseif util.has("snippy") then
        snippet_engine = "snippy"
      elseif vim.g.loaded_vsnip then
        snippet_engine = "vsnip"
      end
    end

    opts.snippet_engine = snippet_engine
  end,
}

@rabinadk1 This way when vim.snippet is available (which is on Neovim 0.10+) it will be hardcoded to use the native snippets engine by default, which some users might not like and prefer to use Luasnip instead. I believe it should be left up to the user to define what he wants. It's one line in the personal config after all.

@rabinadk1 This way when vim.snippet is available (which is on Neovim 0.10+) it will be hardcoded to use the native snippets engine by default, which some users might not like and prefer to use Luasnip instead. I believe it should be left up to the user to define what he wants. It's one line in the personal config after all.

Since LazyVim already prefers native snippets on default for nvim >=0.10, I though it would be a sensible default than not using a snippet at all. As you said, you can change the config later according to their preferences.

Yes, but at least the checks could be reverted, so it first checks if any of the other snippet engines exist first (since the user would have explicitly enabled them) and as last fallback default to vim.snippet.


Add serde feature

Would you guys mind accepting this small PR, which adds serde's Serialize and Deserialize traits to AsciiChar and AsciiString behind a serde feature?

Thank you, optional serde support is a no-brainer.
I don't think adding it is a breaking change either: While serde requires Rust 1.13 and this crate Rust 1.9, adding it as an opt-in feature should not affect current users.
That requirement should be documented in the crate-level documentation (in lib.rs and the readme) though.
On the other hand, small is not the word I'd use to describe a "+164 -100" commit.
Could you at least split out the rustfmt run (I assume) into a later commit?
(or drop it completely; I'm not a big fan of rustup in the first place, and most of these changes makes the code longer (=less code on screen) while not improving readability IMO)

Yes, the other changes are rustfmt, which I'll turn off. I'll also adds some documentation.

There we go. Three commits:

One that prevents rustfmt from running (so future contributors with rustfmt turned on in their IDE won't make the same mistake I did)
Another that adds serialization and deserialization support for AsciiChar and AsciiString behind a serde feature flag, with the minimum Rust version documented.
Another that bumps the version to 0.9.1 (because, as you mentioned, this is not a breaking change)

This PR implements serialization exposing the inner structure of the AsciiString and AsciiChar type.
Example: {"vec":["t","e","s","t","Space","s","t","r","i","n","g"]}
I think we should serialize and deserialize to a string (serde datatype).

Your earlier question prompted me to the same conclusion; so I've been working on a version that uses custom impls of Serialize and Deserialize to basically have AsciiString and AsciiChar behave like regular String and regular Char.

So here's a version that has a lot more code, but it should make serialization and deserialization work exactly like the standard Char, &'a str, and Stringtypes.
Even though it's relatively complicated code, all I did was basically copy/paste the impls that serde itself has for Char, &'a str, and String and adapt them slightly for the various ascii types; so hopefully it won't be too hard to review.

I assume that you finished working on this PR? I'd merge it then.

I don't have any additional changes I'd like to make, but happy to continue incorporating any comments you guys have.


Quest helper fails to distinguish between different Clue Scroll teleport scrolls

In this example, the Quest helper mistakes the Lunar Isle teleport scroll in the bank for the Nardah teleport scroll.
The result of this is that the Nardah teleport scroll is marked as white (item in bank) in the Quest helper when the player does not have the Nardah teleport scroll in any capacity

Hey, is this for Beneath Cursed Sands? My hunch is that it's actually detecting you have a desert amulet rather than the teleport scroll.
If you click the bank quest filter button in the top right of the bank, what item is inserted for the nardah teleport slot?

Hey I'm going to close this assuming ^ is correct, but if you know it's not then please feel free to comment again.


Adding GitHub actions

@yqiqichen thanks for this. Next time please provide more informative PR name;) Merging this in.


has any update ?

I think is a very interesting project, has any updated?

Hi kamly,
this project is unfortunately not really maintained anymore. I currently don’t really have the time to do it, so it’s mostly just doing nothing. I am however open to reviewing and accepting pull requests.
Best regards
Karim

On 31. Jan 2021, at 15:49, kamly<EMAIL_ADDRESS>wrote:
I think is a very interesting project, has any updated?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/SimpleRegex/SRL-PHP/issues/49, or unsubscribe https://github.com/notifications/unsubscribe-auth/AARY2SUPSXED5N347OY2EV3S4VNXDANCNFSM4W3L2VDQ.

Hi kamly,
this project is unfortunately not really maintained anymore. I currently don’t really have the time to do it, so it’s mostly just doing nothing. I am however open to reviewing and accepting pull requests.
Best regards
Karim

On 31. Jan 2021, at 15:49, kamly<EMAIL_ADDRESS>wrote:
I think is a very interesting project, has any updated?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/SimpleRegex/SRL-PHP/issues/49, or unsubscribe https://github.com/notifications/unsubscribe-auth/AARY2SUPSXED5N347OY2EV3S4VNXDANCNFSM4W3L2VDQ.


Fix AttachPtvsd test to pass proper arguments to IronPython to allow ptvsd to work.

Increase waiting period for process startup in AttachPtvsd test, as the original value is often not long enough.

+1 with a property or a half-good reason not to bother right now.

:ok:


Remove dashbuilder Jenkins jobs for master.

@psiroky @mbiarnes The goal of this PR is to remove the Jenkins jobs for building/releasing the Dashbuilder repository master branch.
Please take a careful look. I did not fully understand the scripts and am almost certain I messed something up. In particular, I did not remove references to dashbuilder from FlowJob. It looks to me like a template you use for generating new job configurations. Is that correct?

looks OK to me and I tested it and has no errors.
will uberfire and dashbuilder have in future the same verison as kie?

@mbiarnes Yeah, my next step is to consolidate versions and version management so that versions are aligned with kie and version management is in droolsjbpm-build-bootstrap.


[5.4] Backport str_before helper

Fixes #20125
It's currently on the docs for the 5.4: https://laravel.com/docs/5.4/helpers#method-str-before
However it exists only on 5.5, maybe you'd prefer to remove it from the docs...

Prefer to remove it from docs.


Added converters | IntToBool | DoubleToInt | BoolToObject | IndexToArrayItem | LowerCaseText | UpperCaseText

Description of Change
Added a few more common converters.
Note: I haven't added comments yet, because I don't know which converters will be accepted.
Bugs Fixed

Related to issue #116

API Changes
Added:

IntToBoolConverter
DoubleToIntConverter
BoolToObjectConverter
IndexToArrayItemConverter
TextCaseConverter

PR Checklist

[ ] Has tests (if omitted, state reason in the description)
[ ] Has samples (if omitted, state reason in the description)
[X] Rebased on top of the master at the time of PR
[X] Changes adhere to the coding standard
[ ] Updated documentation

I'm trying to think about what would be a use case for the ToArray converters. You already mentioned that they're probably the most confusing but I'm trying to find a way in my day to day that I would use these :D Do you happen to have a sample from the real world on how I could use these (apart from the conceptual one above)?

I am afraid that if the converter's name is not self explainable, it will be never used.
That seems the case for BoolToArray, BoolToObject, IntToArray

Yeah, so I am open for conversation :)

I'm trying to think about what would be a use case for the ToArray converters. You already mentioned that they're probably the most confusing but I'm trying to find a way in my day to day that I would use these :D Do you happen to have a sample from the real world on how I could use these (apart from the conceptual one above)?

Seems BoolToArray doesn't make sense since BoolToObject exists.
As for IntToArray: for example, I have five color themes for my page. And by setting values from 0 to 4 I can pick a Color from the array.

All requests are resolved, please re-review.

Added two new converters (Upper/Lower case)

Perhaps we could do a single TextCaseConverter that has an enum property defining the casing? That way we can expand in the future easier e.g. for CamelCase, TitleCase etc.

Added two new converters (Upper/Lower case)

Perhaps we could do a single TextCaseConverter that has an enum property defining the casing? That way we can expand in the future easier e.g. for CamelCase, TitleCase etc.

Very good point. I like this idea

Added two new converters (Upper/Lower case)

Perhaps we could do a single TextCaseConverter that has an enum property defining the casing? That way we can expand in the future easier e.g. for CamelCase, TitleCase etc.

Should we use converter parameter for this?

Should we use converter parameter for this?

We could do that or we could just introduce a typed property for it:
public class TextCaseConverter
{
    ...

    public TextCase TextCase { get; set; }

    ...
}

And use it as:
 <Label Text="{Binding MyString, Converter={xct:TextCaseConverter TextCase=UpperCase}}" />

@sthewissen Don't you think we should use converter parameter for this?

@sthewissen How about such rule

We try to parse parameter from the "ConverterParameter"
IF NO LUCK THEN
We take value from the property

Oh @sthewissen for doing as you described our converters should conform IMarkupExtension interface.
Perhaps it will work for us as implemented right now?

Oh @sthewissen for doing as you described our converters should conform IMarkupExtension interface.
Perhaps it will work for us as implemented right now?

I don't see a problem in also implementing IMarkupExtension if that brings some more type-safety into the mix (and thus also Intellisense). I'm personally not a fan of the string matching / integer matching on a type object parameter as it doesn't really provide any feedback to the user from the IDE on what to put in.
To move forward with this PR and get some stuff done/in we could merge it, but I would consider the above.

Oh @sthewissen for doing as you described our converters should conform IMarkupExtension interface.
Perhaps it will work for us as implemented right now?

I don't see a problem in also implementing IMarkupExtension if that brings some more type-safety into the mix (and thus also Intellisense). I'm personally not a fan of the string matching / integer matching on a type object parameter as it doesn't really provide any feedback to the user from the IDE on what to put in.
To move forward with this PR and get some stuff done/in we could merge it, but I would consider the above.

I propose to merge it and create a separate ticket to make the necessary converters conforming IMarkupExtension.

@AndreiMisiukevich Looks like this PR is ready to go, but you mention

I haven't added comments yet, because I don't know which converters will be accepted.

Do you still want to add anything?

@AndreiMisiukevich Looks like this PR is ready to go, but you mention

I haven't added comments yet, because I don't know which converters will be accepted.

Do you still want to add anything?

No, I don't think so. I've already added comments

Oh @sthewissen for doing as you described our converters should conform IMarkupExtension interface.
Perhaps it will work for us as implemented right now?

I don't see a problem in also implementing IMarkupExtension if that brings some more type-safety into the mix (and thus also Intellisense). I'm personally not a fan of the string matching / integer matching on a type object parameter as it doesn't really provide any feedback to the user from the IDE on what to put in.
To move forward with this PR and get some stuff done/in we could merge it, but I would consider the above.

I propose to merge it and create a separate ticket to make the necessary converters conforming IMarkupExtension.

Then LGTM.


Add region_kms_key_ids option to Packer config

What would you like to be added:
Allow the region_kms_key_ids parameter to be passed to the Packer config.
Why is this needed:
This feature goes hand-in-hand with encrypt, kms_key_id, and ami_regions - if you specify any AMI regions and also a kms_key_id, the key is only respected for the region in which the AMI is being built, and other regions will use the default key. This is a Packer EBS provisioner option already, just allow us to use it here without using jq or something to modify the settings json. See the Packer options here: https://developer.hashicorp.com/packer/integrations/hashicorp/amazon/latest/components/builder/ebs
Currently testing this out in a fork: https://github.com/quercy/amazon-eks-ami

That's kind of a bummer, but IIWII I guess. Thanks for the details!


Correction for raw affiliation ChemHouse Research Group; Sciences Pour l'Oenologie

Correction needed for raw affiliation ChemHouse Research Group; Sciences Pour l'Oenologie
raw_affiliation_name: ChemHouse Research Group; Sciences Pour l'Oenologie
new_rors: 04w07sc25
previous_rors: 04w07sc25;05fe7ax82
works_examples: W3213895109
contact: a3b8b64e18eee5eb8701f32028c480ed:d40e8eebf2fd531b13396ffd3fc8970463d8 @ inrae.fr

This issue was accepted and ingested by the OpenAlex team on 2024-10-09. The new affiliations should be visible within the next 7 days.


Problem sending 204 status code with content-type header

Hi, i'm using adonis to create a JSON API for a ember js application. According to JSON API docs, deleting resources needs to send a 204 status code with no content on success, and, as with the whole concept, needs to send the application/json header. The problem is that when i send the status 204, it deletes the content type header, no matter if i use the .header() or .json() method. This is my code:
* destroy (request, response) {
    const contact = yield Contact.find(request.param('id'))
    let result = yield contact.delete()
    if(result){
      response.status(204)
      response.header('Content-type', 'application/json')
      response.send()
      //response.status(204).json() //this fails also
    }
  }

Just in case, this is my dependency list:
"dependencies": {
    "adonis-ace": "^3.0.4",
    "adonis-auth": "^1.0.4",
    "adonis-commands": "^2.1.2",
    "adonis-fold": "^3.0.3",
    "adonis-framework": "^3.0.3",
    "adonis-jsonapi": "^0.4.1",
    "adonis-lucid": "^3.0.8",
    "adonis-middleware": "^1.0.8",
    "mysql": "^2.11.1",
    "standard": "^8.4.0",
    "youch": "^1.0.1"
  }

Not sure why JSONAPI has that requirement with 204 status code since Content-Type is required when there is content in response.
https://tools.ietf.org/search/rfc2616#section-7.2.1

Closing since no reply from the issue creator.

Sorry about the delay. If someone is having this same issue, what i did to solve it is using the 200 status code response and returning the meta body like this
{
  "meta": {
    "copyright": "Copyright 2015 Example Corp.",
    "authors": [
      "Yehuda Katz",
      "Steve Klabnik",
      "Dan Gebhardt",
      "Tyler Kellen"
    ]
  }
}


fix(customMQ): extend --HDcolor media query with color-gamut check

Fix issue #443
Extends the --HDcolor custom media query the check for either a dynamic-range: high or color-gamut: p3

Oh, by the way, I often use this one:
--forcedColors: (forced-colors: active)
It would be nice to have it in the default Open Props set.
I can add it too.

Oh, by the way, I often use this one: --forcedColors: (forced-colors: active) It would be nice to have it in the default Open Props set. I can add it too.

oh yes! how'd i miss it. ty ty, please add if you are down 🙂


Training loss graph is updated when "discretize output" is selected or deselected

The graph visualizing the training loss is updated every time the "discretize output" box is selected or deselected, please see attached screen recording. Because no step is actually performed this is assumed to be a bug.
https://user-images.githubusercontent.com/5433242/227735348-754e36d2-727e-45f8-add5-f9591a4fe497.mov

I agree that this is clearly a bug. Nevertheless, I wanted to report an other issue related to this one. If you click the Discetize output, it does have to update the training loss numerical value and graph, but just once. Whenever you discretize the space and neglect the existence of the"gradient" colours, you only have 2 static values for the predicted values, hence calculating the loss would result in a different result.


network connection should not depend on spike pattern?

@rythorpe you might be interested in this. I was looking into how a cell receives connections and realized that there are conditional statements in the network connection that depend on the input spike pattern. See here:
https://github.com/jonescompneurolab/hnn-core/blob/master/hnn_core/pyramidal.py#L534-L593
do you have any idea why the extpois and extgauss might have different connection patterns?

I believe extgauss input times and synaptic weights are distributed to cells based on a spatial Gaussian distribution. extpois feeds probably don't have a spatial center, so their target connection pattern should be different.

@samnemo @blakecaldwell is that correct?

so their target connection pattern should be different.

I see, interesting. Can we document what these different input types do somewhere? What are the connection and spike patterns etc? I still don't know what are the differences between them :-(
Also, if we want to separate the cortical column model from the input, this method should be part of a ExtGauss class rather than part of the Pyramidal or Basket class

I see, interesting. Can we document what these different input types do somewhere? What are the connection and spike patterns etc? I still don't know what are the differences between them :-(

Yes, this definitely should be done. Somewhere in all of this is the explanation for #115.

Also, if we want to separate the cortical column model from the input, this method should be part of a ExtGauss class rather than part of the Pyramidal or Basket class

Agreed, however, we might need to keep the NEURON references within the Pyramidal and Basket cell classes.

Also, if we want to separate the cortical column model from the input, this method should be part of a ExtGauss class rather than part of the Pyramidal or Basket class

Agreed, however, we might need to keep the NEURON references within the Pyramidal and Basket cell classes.

@rythorpe's explanation sounds right. It also makes sense that there should be a separation. Ideally feeds connected/removed from the cortical column independently. However, the need to be connected at some point. What's the argument for putting these functions in ExtFeed. They could certainly be made separate functions because there's no code reuse with the conditionals now.

The only argument to put them as methods of ExtFeed is if the connections depend on the feed type. If the connections don't depend on the feed type, I can see how you need only a single function to connect. It's a bit hazy in my mind though, we'll have to start touching the code :)

I believe this is also closed by #164
the lines originally referred have changed but I think it has been fixed. Please feel free to reopen if you disagree


High low

What is a good number to look for when pinning?

I think the the most important thing is that ping should be stable.
I think that ~30ms is perfect but even 100ms could be good if it is stable.
stratum-ping]$ ./build/bin/stratum-ping -tls eth.2miners.com:12020
PING stratum eth.2miners.com (<IP_ADDRESS>) TLS port 12020
eth.2miners.com (<IP_ADDRESS>): seq=0, time=30.844285ms
eth.2miners.com (<IP_ADDRESS>): seq=1, time=32.667843ms
eth.2miners.com (<IP_ADDRESS>): seq=2, time=32.037485ms
eth.2miners.com (<IP_ADDRESS>): seq=3, time=30.88984ms
eth.2miners.com (<IP_ADDRESS>): seq=4, time=32.420842ms
--- eth.2miners.com ping statistics ---
5 packets transmitted, 5 received, 0% packet loss, time 5.509857194s
min/avg/max = 30.844285ms, 31.772059ms, 32.667843ms


Mode for GFM with admonitions

Solves #6425

Sorry, but as the CONTRIBUTING.md file states, we aren't adding new extensions to this distribution. So this would best be maintained and distributed as a separate npm package.


Jamie/member expression dialog improvement

:nut_and_bolt: Description: What code changed, and why?
Currently, the member expression builder consists of a single text input, where the user must consult documentation in order to correctly input a member expression.  This branch updates the user experience of this modal so that the user is guided through the building of the member expression one step at a time.
:chains: Related Resources
:+1: Definition of Done
User is stepped through the member expression builder modal.
Add Expression button is disabled unless formatting is acceptable
Errors fire when the member expression is already being used
Documentation is updated
:athletic_shoe: How to Build and Test the Change
build components/automate-ui-devproxy && start_all_services
Navigate to https://a2-dev.test/settings/policies/editor-access/add-members
Click on Add Member Expression in bottom left corner
Build Member expression step by step --> click add Add Expression when enabled
:white_check_mark: Checklist

[x] Tests added/updated?
[ ] Docs added/updated?

:camera: Screenshots, if applicable
Before:

After:

@SEAjamieD i took a quick look, can take a better look tomorrow..
(1) can you make the cancel button tertiary?
(2) for token, the next field should be ID instead of Name, that may have been wrong in my prototype

@susanev I was looking at the documentation and I'm not sure it really needs any updating.  The concepts are still the same, only difference is now we're helping them build the expression so they don't have to figure it out as much.  Thoughts?
https://automate.chef.io/docs/iam-v2-guide/#member-expressions

@SEAjamieD yah actually we are doing a huge overhaul on those docs right now, so it doesnt make sense to have you do an update now anyway.. so feel free to ignore that content from the issue, sorry about that.

@SEAjamieD a few more things to address if possible...

that cancel button looks weird, it should be re-sizing to have 16px margin on either side instead of stretching to match the size of the other button
if you use the close-x it sends me back to the list page, but it should stay on the add members page
can we get a space between the label and the asterisks
lets change that wildcard text on the name/id fields to A single wildcard (*) is allowed
itd be cool to get the select styles that we added for ingest rules into this, but that can be a separate pr if you want

if you use the close-x it sends me back to the list page, but it should stay on the add members page

@susanev - I'm going to suggest we roll this into its own ticket.  Current functionality now is closing the entire page out.  I haven't touched this piece of the logic and when investigating it looks like this is being passed down via chef-page stencil component and may take a little longer to diagnose.

new issue for close-x issue #1934


Changes not applied - Unable to find feature

A bunch of changes failed with Unable to find feature and could not be applied.
The changes came in in small chunks (3 deltas). Applying the change again does fail with the same error.
This is related to https://github.com/opengisch/qfieldcloud/issues/324

Can you give me any details about username/projectname and ideally, delta ids that you want to know more about. Also what is the data format of the layers, GeoPackage, PostGIS?

Dataformat is GeoPackage. Change eff4a084-aef5-4b9b-8efa-321fb23f0ab0 Project Eifel-Large-N/Re-Deployment

@suricactus when you see patch dd6fff05-b9ad-47c8-a253-8ae54c405c10 the sourcePk and localPk is the same. However the sourcePk should be 383.
Is this due to an old QField client?

Now that QfieldCloud is offered as a paid service (which I am 💯 supportive of) bugs like this  should be ironed out!! I see hundreds of changes which appear applied, but are not. We were confident to use qfieldcloud in production, but I am getting a little nervous now.

All that was changed in-between the working versions was the styling in the project file.

Hi, @miili . This is definitely something that is of high importance.

when you see patch dd6fff05-b9ad-47c8-a253-8ae54c405c10 the sourcePk and localPk is the same. However the sourcePk should be 383.

How do you know the source PK is wrong in this one? Adding a bit more context will help finding the root cause.

Is this due to an outdated QField client app?

Should not be the case, can you please share the version you are using?

The matching sourcePk seems to be a common issue: https://github.com/opengisch/qfieldcloud/issues/324#issuecomment-1385168652

Seems there is some root problem might be from QgsOfflineEditing plugin. Will keep you updated what are our findings.

See for example these features:

Only the diamond shaped features should have been modified: Move feature, update attributes
(probably due to bad sourcePk another feature was moved and edited

This already is a big mess.

Hi,
Maybe I have the same problem as yours :

I wrote a ticket and open a discussion about this problem because as miili, it's quiet critical because I can't put Qfield in production anymore...
Project URL :

I uploaded the GeoPackage from the beginning of the working day, re-applied all changes from the day (~500 deltas), and reproduced the same bad state.
Something is really odd!

After more investigation it looks I am pretty confident that the localPk and sourcePk are refering to the same fid_1, where instead sourcePk should point to fid.
Subsequently the wrong feature pointing to fid_1 is modified and everything is messed up.

@miili thanks for doing the deep parallel investigation. We have identified the issue as I have posted 4 hours ago. All the team resources are focused on fixing this issue, so expect results very soon.
We know how important is data for our users and we are looking for a solution where you do not need to download the changes and apply them yourself. Since the issue escalated a few hours ago, it will take a bit of time to be evaluated and implemented, but I completely understand your urgency. Please give us a window of a few hours and we will provide you with all the information you need.
After we make sure we prevent this from hapening again, we can also provide you with all your changes.
Hope makes you feel a bit better, apologies for the inconvenience.

Great, thank you!
I got all the deltas from the API and will use the day to apply changes locally. If needed I can share the scripts.

Thanks for the heads up. Will keep you posted on our side too.

Hi @miili  we we've clearly identified what the issue is and have assessed the impacted projects. The issue seem to be related to a very specific use case and we have 23 users affected by the issue out of 45K users.
We are now investigating what project configuration leads to the issue.
we'll keep you updated as soon as we've more information.
cheers Marco

Thanks for the heads up. What exotic configuration did we use for our project?

Hello together,
My Projects are also affected of this bug and i can reproduce it too.
A Workaround to fix this is to delete and download the Project again.
If you need more Information or betatesting (i have a selfhosted cloud currently), where i could apply a patch e.g.

Can you please send me an email at<EMAIL_ADDRESS>for additional information?

Perhaps I am also involved...
All the 'not applied' changes are because of "unable to find feature". I tried to re-apply them, but that didn't work out.

Apart from that, some features of collaborators and myself where registered twice. I don't understand how this could happen in the field.

Here is another user who has the same problems when synchronizing. The error "Unable to find feature" appears many times, but this entity does exist in the project.

I don’t know if it has anything to do with it, but the theme that was producing the errors refers to a symbology that has embedded SVGs in the project and they are rotated with the value that a specific field has.

Same problem here... Several changes not applied because "Unable to find feature". I checked and the features are there (id are matching).
Any workaround before patching this ?

I'm also affected:

{
  "conflicts": null,
  "delta_file_id": "4bd43c74-e88b-4ae8-b004-eaa91dbeeea8",
  "delta_id": "01042462-700b-45d6-8ba2-3453ccaa0361",
  "delta_index": 0,
  "e_type": "ERROR",
  "feature_pk": "{9a8a24b4-6fbe-4a1d-aca2-05425f951cb8}",
  "layer_id": "test_01c1f56e_701b_44fc_b85a_5a400914d8be",
  "method": "patch",
  "modified_pk": null,
  "msg": "Unable to find feature",
  "provider_errors": null,
  "status": "status_apply_failed"
}

The problem persists and apparently without any reproducible logic.
Unfortunately, it makes data collection unreliable. Is there any news?


💪 #01.01: Account ✅ - HS23

Check-in area

Larissa Gies
Schiphol, Nederland
Published on February 27, 2020
HUAWEI, ELE-L29
Free to use under the Unsplash License

👍

👍🏻

👍

👍

👍

👍

👍

👍🏻

👍

👍

👍

👍

👍

👍

👍

👍

🤙

👍

👍

👍

👍

👍

👍

👍

👍

👍

👍

Onboarding done up to here!

👍

👍

👍

https://etherpad.wikimedia.org/p/bfh-ch-module-eoss-hs23

👍

👍

👍

👍

👍

👍

👍

👍

👍

👍

👍

👍

👍

👍
Unfortunately i couldn‘t attend the first lesson… could you please check me in

👍
Hi! I was unable to be present at last week's lecture. could i please still be checked in? Thank you

👍

👍

👍

👍

👍

👍

👍

👍

👍

👍unfortunately I could not attend the class, would you please check me in!

👍 Hi, could you also please check me in? Thanks

👍  Could you check me in aswell? Thank you!

👍

👍

👍

👍


Hotfix: Fix MasterSolutionLibrary indexing for multiple architecture build (#1888)

Fix MasterSolutionLibrary indexing to avoid index collision when building for multiple architectures which causes getAllSolutions gtests to fail.

The code change looks OK.
90a extended test fail is a known issue (1sum fail).

Fix for https://ontrack-internal.amd.com/browse/SWDEV-433699


exchange param in search_assets has unexpected behaviour

Discussed in https://github.com/alvarobartt/investiny/discussions/33

Originally posted by longlie1109 October 12, 2022
Is there a way to get the "exchange" in search_assets function ?
NASDAQ works fine but other market isn't work out for me
So as the discussion says, it seems that the results provided by search_assets contain a field named exchange which doesn't seem to be matching the actual filter for the search, which means that if the results contain e.g. exchange="RANDOM_EXCHANGE", if you use that filter in a next call to search_assets using the exchange param, it won't work, as the printed exchange is not the same one as the one used by the filter, even though those refer to the same.
Note that something similar happens with the type parameter, which was already solved at https://github.com/alvarobartt/investiny/commit/8cf3e0eabc17e689975b63e8793b3fdab1b7fe34

Related to #34


📝 Styleguide for responsive table

Add instructions on adding responsive table.

Thanks @lachlanhardy for the review.
@karensawrey I'll hold on merging, be keen to get your thoughts on the language that I used.

Awesome addition to the style guide, @olyism! I merged in 2 tiny improvements. Please ship the whole thing when the tests pass. 👍

Ship it! 🚢🚢 🚢  ;)


Adds missing meta data to Entry, Asset and ContentType responses

This will add missing meta information back into the resolved responses:
Assets
import Contentful.Query
alias Contentful.Asset
alias Contentful.Delivery.Assets

{:ok, asset} = Assets |> fetch_one("my_asset_id")

"my_asset_id" = asset.id
"<a timestamp for updated_at>" = asset.sys.updated_at
"<a timestamp for created_at>" = asset.sys.created_at
"<a locale string>" = asset.sys.locale

Entries
import Contentful.Query
alias Contentful.{ContentType, Entry, SysData}
alias Contentful.Delivery.Entries

{:ok, entry} = Entries |> fetch_one("my_entry_id")

"my_entry_id" = entry.id
"<a timestamp for updated_at>" = entry.sys.updated_at
"<a timestamp for created_at>" = entry.sys.created_at
"<a locale string>" = entry.sys.locale
%ContentType{id: "the_associated_content_type_id"} =  entry.sys.content_type

ContentTypes
import Contentful.Query
alias Contentful.ContentType
alias Contentful.Delivery.ContentTypes

{:ok, content_type} = ContentTypes |> fetch_one("my_content_type_id")

"my_content_type_id" = content_type.id
"<a timestamp for updated_at>" = content_type.sys.updated_at
"<a timestamp for created_at>" = content_type.sys.created_at

Why is the id not part of sys?
Because I am lazy and the id is most likely just important enough to warrant being access by entity.id instead of entity.sys.id. I am open to feedback here :wink:
Shoutout to @OldhamMade for bringing this to my attention in #44.
fixes #44.

Pull Request Test Coverage Report for Build 61f49c93bd0388eb2e80d45e2f94811343db4f12-PR-45

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 67.568%

Totals

Change from base Build a6abd60cc5669f1709058a46396f4a84fbb52dfc:
0.0%

Covered Lines:
75

Relevant Lines:
111

💛  - Coveralls

Why is the id not part of sys?

When I was working on a similar (more hobby) project/SDK for another language I also struggled with the sys property. In the end I think it is fine to not have structural equality between API presentation and language environment, the only important thing is that the necessary information there. Esp. for the CDA, which is a read-only API, we can flatten or strip presentational structure if it makes handling the data more convenient, since we don't really need to keep it for updates (a CMA concern).
tl;dr: I'm totally fine with structural changes for a "one-way" API.

Ultimately, there probably will be a CMA integration with this library - that's when the question of sys.id or id will probably pop up again, but it should be easy to serialize/deserialize this on the fly.
Don't quote me years from now please.

Pull Request Test Coverage Report for Build d7bc398c5630db514f89bd8833632e6d3b0b1e74-PR-45
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 67.568%

Totals

Change from base Build a6abd60cc5669f1709058a46396f4a84fbb52dfc:
0.0%

Covered Lines:
75

Relevant Lines:
111

💛  - Coveralls


ip-reconciler: Add all non default interfaces to Pod IP list

This fixes https://github.com/k8snetworkplumbingwg/whereabouts/issues/130
In code below,
	for _, network := range networkStatusList {
		// we're only after multus secondary interfaces
		if network.Default {
			continue
		}

		if network.Interface[:multusPrefixSize] == multusInterfaceNamePrefix {
			for _, ip := range network.IPs {
				ipSet[ip] = empty
			}
		}
	}

I don't know of any case where there would be a network attachment that isn't default, but not a Multus attachment? The way I understand it, we have a primary CNI (such as Calico, for us). This is implicitly attached to every Pod as "eth0", and in describe Pod, it is always set to default: True
So, is there any case where the Pod would have multiple Networks, and one of them not be Default yet not Multus?
Either way, I don't think this check is correct, as in the Pod's network annotation, it is allowed to change the interface name. At very least there should be some other way to detect Multus interfaces (if simply checking for not default isn't sufficient). For example, see this example: https://github.com/k8snetworkplumbingwg/multus-cni/blob/master/docs/how-to-use.md#launch-pod-with-json-annotation-with-interface.
With my fix, I set Pod's interface to be named "eth1" for my macvlan network. Per the Log I added to ip-reconciler, I can see it's now added the the IPSet for the Pod:
xagent@pop-os:~/luigi/samples/macvlan$ kubectl logs ip-reconciler-1632195360-9vgrk -n kube-system
2021-09-21T03:36:07Z [debug] NewReconcileLooper - Kubernetes config file located at: /host/etc/cni/net.d/whereabouts.d/whereabouts.kubeconfig
2021-09-21T03:36:08Z [debug] successfully read the kubernetes configuration file located at: /host/etc/cni/net.d/whereabouts.d/whereabouts.kubeconfig
2021-09-21T03:36:08Z [debug] listing IP pools
2021-09-21T03:36:08Z [debug] **Added IP <IP_ADDRESS> for pod kube-system/samplepod1**
2021-09-21T03:36:08Z [debug] **Added IP <IP_ADDRESS> for pod kube-system/samplepod2**
2021-09-21T03:36:08Z [debug] the IP reservation: IP: <IP_ADDRESS> is reserved for pod: kube-system/samplepod1
2021-09-21T03:36:08Z [debug] pod reference kube-system/samplepod1 matches allocation; Allocation IP: <IP_ADDRESS>; PodIPs: map[<IP_ADDRESS>:{}]
2021-09-21T03:36:08Z [debug] the IP reservation: IP: <IP_ADDRESS> is reserved for pod: kube-system/samplepod2
2021-09-21T03:36:08Z [debug] pod reference kube-system/samplepod2 matches allocation; Allocation IP: <IP_ADDRESS>; PodIPs: map[<IP_ADDRESS>:{}]
2021-09-21T03:36:09Z [debug] pod reference kube-system/samplepod1 matches allocation; Allocation IP: <IP_ADDRESS>; PodIPs: map[<IP_ADDRESS>:{}]
2021-09-21T03:36:09Z [debug] pod reference kube-system/samplepod2 matches allocation; Allocation IP: <IP_ADDRESS>; PodIPs: map[<IP_ADDRESS>:{}]
2021-09-21T03:36:09Z [debug] no IP addresses to cleanup

Sorry, submitted wrong private branch. Will re-submit


Add splashtopbusiness Label

Test run output:
% ./utils/assemble.sh splashtopbusiness
2022-08-19 09:12:42 : REQ   : splashtopbusiness : ################## Start Installomator v. 10.0beta1, date 2022-08-19
2022-08-19 09:12:42 : INFO  : splashtopbusiness : ################## Version: 10.0beta1
2022-08-19 09:12:42 : INFO  : splashtopbusiness : ################## Date: 2022-08-19
2022-08-19 09:12:42 : INFO  : splashtopbusiness : ################## splashtopbusiness
2022-08-19 09:12:42 : DEBUG : splashtopbusiness : DEBUG mode 1 enabled.
2022-08-19 09:12:43 : INFO  : splashtopbusiness : BLOCKING_PROCESS_ACTION=tell_user
2022-08-19 09:12:43 : INFO  : splashtopbusiness : NOTIFY=success
2022-08-19 09:12:43 : INFO  : splashtopbusiness : LOGGING=DEBUG
2022-08-19 09:12:43 : INFO  : splashtopbusiness : LOGO=/System/Applications/App Store.app/Contents/Resources/AppIcon.icns
2022-08-19 09:12:43 : INFO  : splashtopbusiness : Label type: pkgInDmg
2022-08-19 09:12:43 : INFO  : splashtopbusiness : archiveName: Splashtop Business.dmg
2022-08-19 09:12:43 : INFO  : splashtopbusiness : no blocking processes defined, using Splashtop Business as default
2022-08-19 09:12:43 : DEBUG : splashtopbusiness : Changing directory to /Users/liam.steckler/Installomator/build
2022-08-19 09:12:43 : INFO  : splashtopbusiness : name: Splashtop Business, appName: Splashtop Business.app
2022-08-19 09:12:43.496 mdfind[10012:99378] Couldn't determine the mapping between prefab keywords and predicates.
2022-08-19 09:12:43 : WARN  : splashtopbusiness : No previous app found
2022-08-19 09:12:43 : WARN  : splashtopbusiness : could not find Splashtop Business.app
2022-08-19 09:12:43 : INFO  : splashtopbusiness : appversion:
2022-08-19 09:12:43 : INFO  : splashtopbusiness : Latest version of Splashtop Business is <IP_ADDRESS>
2022-08-19 09:12:43 : REQ   : splashtopbusiness : Downloading https://download.splashtop.com/macclient/STB/Splashtop_Business_Mac_INSTALLER_v<IP_ADDRESS>.dmg to Splashtop Business.dmg
2022-08-19 09:12:43 : DEBUG : splashtopbusiness : No Dialog connection, just download
2022-08-19 09:12:44 : DEBUG : splashtopbusiness : File list: -rw-r--r--  1 liam.steckler  staff    38M Aug 19 09:12 Splashtop Business.dmg
2022-08-19 09:12:44 : DEBUG : splashtopbusiness : File type: Splashtop Business.dmg: zlib compressed data
2022-08-19 09:12:44 : DEBUG : splashtopbusiness : curl output was:
*   Trying <IP_ADDRESS>:443...
* Connected to download.splashtop.com (<IP_ADDRESS>) port 443 (#0)
* ALPN: offers h2
* ALPN: offers http/1.1
*  CAfile: /etc/ssl/cert.pem
*  CApath: none
* (304) (OUT), TLS handshake, Client hello (1):
} [327 bytes data]
* (304) (IN), TLS handshake, Server hello (2):
{ [122 bytes data]
* (304) (IN), TLS handshake, Unknown (8):
{ [19 bytes data]
* (304) (IN), TLS handshake, Certificate (11):
{ [4962 bytes data]
* (304) (IN), TLS handshake, CERT verify (15):
{ [264 bytes data]
* (304) (IN), TLS handshake, Finished (20):
{ [36 bytes data]
* (304) (OUT), TLS handshake, Finished (20):
} [36 bytes data]
* SSL connection using TLSv1.3 / AEAD-AES128-GCM-SHA256
* ALPN: server accepted h2
* Server certificate:
*  subject: CN=*.splashtop.com
*  start date: Jan 19 00:00:00 2022 GMT
*  expire date: Feb 17 23:59:59 2023 GMT
*  subjectAltName: host "download.splashtop.com" matched cert's "*.splashtop.com"
*  issuer: C=US; O=Amazon; OU=Server CA 1B; CN=Amazon
*  SSL certificate verify ok.
* Using HTTP2, server supports multiplexing
* Copying HTTP/2 data in stream buffer to connection buffer after upgrade: len=0
* h2h3 [:method: GET]
* h2h3 [:path: /macclient/STB/Splashtop_Business_Mac_INSTALLER_v<IP_ADDRESS>.dmg]
* h2h3 [:scheme: https]
* h2h3 [:authority: download.splashtop.com]
* h2h3 [user-agent: curl/7.83.1]
* h2h3 [accept: */*]
* Using Stream ID: 1 (easy handle 0x136012000)
> GET /macclient/STB/Splashtop_Business_Mac_INSTALLER_v<IP_ADDRESS>.dmg HTTP/2
> Host: download.splashtop.com
> user-agent: curl/7.83.1
> accept: */*
>
* Connection state changed (MAX_CONCURRENT_STREAMS == 128)!
< HTTP/2 200
< content-type: binary/octet-stream
< content-length: 39504001
< date: Fri, 19 Aug 2022 15:33:37 GMT
< last-modified: Wed, 03 Aug 2022 00:30:20 GMT
< etag: "59d2c157d57414b6678cceb648801ee5-5"
< accept-ranges: bytes
< server: AmazonS3
< x-cache: Hit from cloudfront
< via: 1.1 2c6b5dd77f1abe60653ce0454f344b64.cloudfront.net (CloudFront)
< x-amz-cf-pop: SEA19-C3
< x-amz-cf-id: _IfLA1h0og50dBm0Qt0kII2LCPcdGfprvxpiu1t6ZHdHN2aJx-uBPQ==
< age: 2346
<
{ [15990 bytes data]
* Connection #0 to host download.splashtop.com left intact

2022-08-19 09:12:44 : DEBUG : splashtopbusiness : DEBUG mode 1, not checking for blocking processes
2022-08-19 09:12:44 : REQ   : splashtopbusiness : Installing Splashtop Business
2022-08-19 09:12:44 : INFO  : splashtopbusiness : Mounting /Users/liam.steckler/Installomator/build/Splashtop Business.dmg
2022-08-19 09:12:46 : DEBUG : splashtopbusiness : Debugging enabled, dmgmount output was:
Checksumming Protective Master Boot Record (MBR : 0)…
Protective Master Boot Record (MBR :: verified   CRC32 $308FDF24
Checksumming GPT Header (Primary GPT Header : 1)…
GPT Header (Primary GPT Header : 1): verified   CRC32 $AFA72DF9
Checksumming GPT Partition Data (Primary GPT Table : 2)…
GPT Partition Data (Primary GPT Tabl: verified   CRC32 $4DDB491A
Checksumming  (Apple_Free : 3)…
(Apple_Free : 3): verified   CRC32 $00000000
Checksumming disk image (Apple_HFS : 4)…
disk image (Apple_HFS : 4): verified   CRC32 $8462426B
Checksumming  (Apple_Free : 5)…
(Apple_Free : 5): verified   CRC32 $00000000
Checksumming GPT Partition Data (Backup GPT Table : 6)…
GPT Partition Data (Backup GPT Table: verified   CRC32 $4DDB491A
Checksumming GPT Header (Backup GPT Header : 7)…
GPT Header (Backup GPT Header : 7): verified   CRC32 $BE6BD74E
verified   CRC32 $008E5B19
/dev/disk5              GUID_partition_scheme
/dev/disk5s1            Apple_HFS                       /Volumes/Splashtop Business 1

2022-08-19 09:12:46 : INFO  : splashtopbusiness : Mounted: /Volumes/Splashtop Business 1
2022-08-19 09:12:46 : DEBUG : splashtopbusiness : Found pkg(s):
/Volumes/Splashtop Business 1/Splashtop Business.pkg
2022-08-19 09:12:46 : INFO  : splashtopbusiness : found pkg: /Volumes/Splashtop Business 1/Splashtop Business.pkg
2022-08-19 09:12:46 : INFO  : splashtopbusiness : Verifying: /Volumes/Splashtop Business 1/Splashtop Business.pkg
updateDialog:26: no such file or directory:
2022-08-19 09:12:46 : DEBUG : splashtopbusiness : File list: -rw-r--r--  1 liam.steckler  staff    38M Aug  1 23:47 /Volumes/Splashtop Business 1/Splashtop Business.pkg
2022-08-19 09:12:46 : DEBUG : splashtopbusiness : File type: /Volumes/Splashtop Business 1/Splashtop Business.pkg: xar archive compressed TOC: 4531, SHA-1 checksum
2022-08-19 09:12:46 : DEBUG : splashtopbusiness : spctlOut is /Volumes/Splashtop Business 1/Splashtop Business.pkg: accepted
2022-08-19 09:12:46 : DEBUG : splashtopbusiness : source=Notarized Developer ID
2022-08-19 09:12:46 : DEBUG : splashtopbusiness : origin=Developer ID Installer: Splashtop Inc. (CPQQ3AW49Y)
2022-08-19 09:12:46 : INFO  : splashtopbusiness : Team ID: CPQQ3AW49Y (expected: CPQQ3AW49Y )
2022-08-19 09:12:46 : DEBUG : splashtopbusiness : DEBUG enabled, skipping installation
updateDialog:26: no such file or directory:
2022-08-19 09:12:46 : INFO  : splashtopbusiness : Finishing...
2022-08-19 09:12:49 : INFO  : splashtopbusiness : name: Splashtop Business, appName: Splashtop Business.app
2022-08-19 09:12:49.645 mdfind[10113:99694] Couldn't determine the mapping between prefab keywords and predicates.
2022-08-19 09:12:49 : WARN  : splashtopbusiness : No previous app found
2022-08-19 09:12:49 : WARN  : splashtopbusiness : could not find Splashtop Business.app
2022-08-19 09:12:49 : REQ   : splashtopbusiness : Installed Splashtop Business
2022-08-19 09:12:49 : INFO  : splashtopbusiness : notifying
2022-08-19 09:12:49 : DEBUG : splashtopbusiness : Unmounting /Volumes/Splashtop Business 1
2022-08-19 09:12:50 : DEBUG : splashtopbusiness : Debugging enabled, Unmounting output was:
"disk5" ejected.
2022-08-19 09:12:50 : DEBUG : splashtopbusiness : DEBUG mode 1, not reopening anything
2022-08-19 09:12:50 : REQ   : splashtopbusiness : All done!
2022-08-19 09:12:50 : REQ   : splashtopbusiness : ################## End Installomator, exit code 0

Thank you!


BUG: curveDCA bot is missing in stats endpoint

The service stats endpoint does not have any data on the curveDCA bot, but we had some bots already.
https://service.palomabot.ai/stats

Do we have any price info though that we can derive volume with currently?
If not, I propose 2 things.
1/ update the stats endpoint to include the number of DCA bots
2/ update the UI to not try to pull the Volume number for the DCA bots until we figure out a better solution.


query return "71 Protocol error" after "11 Resource temporarily unavailable"

I'm using mongodb without replset mode(single server).
Query ok on mongod running ok.
The same query will return "11 Resource temporarily unavailable" after I run db.fsyncLock() to the mongod server. There is no problem.
But after I turn the server run normally(db.fsyncUnlock()), The following same query will return "71 Protocol error".
After debug, I notice that this is because  (h.resp_to != rid) in _mongo_sync_packet_recv().
algernon, can you fix it?

Seeing as the library is obsolete, unmaintained, and not developed further anymore, I'm closing this issue.


feat: add Astro support

Description

This PR adds support for Astro.
Linked Issues
Closes #154
Related to https://github.com/natemoo-re/astro-icon/issues/23
Additional context

NOTE: There is an outstanding issue I discovered while developing this integration. I created a ticket as well #242

@stramel astro type shouldn't be included in package.json exports?

@stramel astro types shouldn't be included in package.json exports entry?

Looks like you're correct. I missed that spot. I will amend the original commit so that there is a single commit showing all spots necessary for modification outside of documentation.

EDIT: can you also keep the example minimal?

This example was the "Basic" scaffold provided by Astro using npx create astro@latest and selecting the "basic" option. What do you feel needs trimmed out?

This example was the "Basic" scaffold provided by Astro using npx create astro@latest and selecting the "basic" option. What do you feel needs trimmed out?

Remove astro components to a minimal and remove readme and external info on entry point (index route), we only need to check the icon is working, we dont need those extra info.

Remove astro components to a minimal and remove readme and external info on entry point (index route), we only need to check the icon is working, we dont need those extra info.

Alright, I have updated my PR with those changes.
NOTE: Just for consistency, the next and nuxt3 examples should probably be updated to reflect this minimal approach as well. (These were the 2 I looked at initially for reference)

@stramel I found on latest Astro versions a lot of sourcemap points missing source files (also for astro pages), for example, running your astro example:
Sourcemap for "~icons/icon-park/abnormal.astro" points to missing source files
Sourcemap for "~icons/mdi/store-24-hour.astro" points to missing source files
Sourcemap for "~icons/mdi/alarm-off.astro" points to missing source files
Sourcemap for "~icons/logos/astro.astro" points to missing source files

@stramel I found on latest Astro versions a lot of sourcemap points missing source files (also for astro pages), for example, running your astro example:
Sourcemap for "~icons/icon-park/abnormal.astro" points to missing source files
Sourcemap for "~icons/mdi/store-24-hour.astro" points to missing source files
Sourcemap for "~icons/mdi/alarm-off.astro" points to missing source files
Sourcemap for "~icons/logos/astro.astro" points to missing source files

@userquin I also noticed that when running the example. I wasn't sure if that was a side-effect of the virtual components or an issue? I'm happy to dig into it but unsure where to start digging, can you point me in a direction?

@userquin I also noticed that when running the example. I wasn't sure if that was a side-effect of the virtual components or an issue? I'm happy to dig into it but unsure where to start digging, can you point me in a direction?

I'm no an expert on Astro, if the sfc comes from disk or from virtual module doesn't matter, we also have the same problem with other compilers, Anthony just include an empty sourcemap here https://github.com/antfu/unplugin-icons/blob/main/src/index.ts#L50, maybe we can ask Nate.

@userquin Seems like a recent bug on the Astro side. withastro/astro#5053

Yeah, I just send a direct message to Nate, it seems there is a few reports about those warnings.

Rebased onto main to remove merge conflicts

Rebased onto main to remove merge conflicts

Can you run pnpm run lint from root? Just to confirm it is working with Astro files and there are no errors...

@stramel I will push changes running pnpm run lint-fix...


Specifying cache directory for models

When I was going through the RAGMultiModalModel class's from_pretrained, I saw there is no mentions of specifying cache directory. However, ColPali engine supports that.
Would be great if an additional optional parameter can be passed to from_pretrained function where we can specify the cache_dir as well.

Yes I was wondering about this also


feat(integrations): Add support for slack integration

Env vars
SLACK_CLIENT_ID
SLACK_CLIENT_SECRET

To start the oath flow we make a get request to
https://{address}/api/v1/environments/<environment_key>/integrations/slack/oauth/
which  redirects the user(after oauth) to our callback endpoint(with auth token)
The view set to configure a channel for an environment is hosted at:
/environments/{environment_api_key}/integrations/slack/
ps: you can use something like https://ngrok.com/ to test oauth callback
Open question:
where should we redirect the user after successful oauth.

Requesting the review again because of redirect url


Solution of Minimum Number of Refueling Stops (Leetcode: 871)

Description
This pull request contains the solution to Leetcode problems that is to find minimum number of refueling stops and to find longest valid parenthesis using dp in Java. This includes the addition of files named "Min_No_Of_Refueling_Stops.java" and "Longest_valid_parenthesis.java".
Link of the problem: https://leetcode.com/problems/minimum-number-of-refueling-stops/
Link of the problem: https://leetcode.com/problems/longest-valid-parentheses/
Fixes #757, #756
Checklist:

[✔] My code follows the style guidelines(Clean Code) of this project
[✔] I have performed a self-review of my own code
[✔] I have commented my code, particularly in hard-to-understand areas

@Ayush7614, @Amit366 Please review and merge my pull request.

@Ayush7614, @Amit366 Please review and merge my pull request.

@sejal1430 make separate PR for both issues


Add Explicit Tag to Search

Change Log:

When searching for a song, it’s not always clear whether it is explicit, which can lead to the wrong version playing. To address this issue, I have added the text "Explicit" to the subtitle for better clarity.
Fixed the path for the plugin folder in the bat file

Before:

After:

https://github.com/waaverecords/PowerToys-Run-Spotify/issues/38

Good improvement.
This will be part of the next release.
Thanks!


arg input doesn't work properly, fixing promise

arg input doesn't work properly, fixing promise

gotta review async in JS :p

fixed

gota update logic a bit more, need to make it step thru each command THEN combine, almost done :p

bam!


FlushConsolidationHandler remove conditional

Motivation:
FlushConsolidationHandler#flushIfNeeded has a conditional which is fixed based upon code path. This conditional can be removed and instead just manually set in each fixed code path.
Modifications:

Remove boolean parameter on FlushConsolidationHandler#flushIfNeeded and set readInprogess to false manually when necessary

Result:
Less conditionals in FlushConsolidationHandler

@normanmaurer - PTAL

Cherry-picked into 4.1 (cebf255951a257068bbcab4781d4c6085e3289bd) and 4.0 (44ad7ec)


QAOA().compile appears to 'lock-in' values after first run?

Prerequisites
Before raising this issue, I have already checked that I am:

[x ] running the latest version
[x ] made sure that this issue has not already been filed

Describe the bug
Hello, when performing testing using the QAOA package, I find that the initial run of the program will seem to execute correctly. However, if subsequent runs are executed, the program will re-execute as if no variables had changed.
To Reproduce
In a new folder, install the necessary dependencies, and then execute this code:
import numpy as np
import networkx as nx
from openqaoa.utilities import plot_graph

# Generating a graph of 100 nodes
n = 100  # Number of nodes in graph

G = nx.generators.fast_gnp_random_graph(n, p=0.4, seed=42)
plot_graph(G)
from openqaoa.problems import MaximumCut

maxcut_prob = MaximumCut(G)
maxcut_qubo = maxcut_prob.qubo

print(maxcut_qubo.hamiltonian.expression)
maxcut_qubo.asdict()
print('this is maxcut_qubo', maxcut_qubo.asdict())

plot_graph(G)

from openqaoa import QAOA, create_device

q = QAOA()
analytical_device = create_device(location='local', name='analytical_simulator')
q.set_device(analytical_device)

q.compile(maxcut_qubo)
q.optimize()

print('this is answer',q.result.most_probable_states)

You will get an error saying 'ValueError: Maximum allowed dimension exceeded'
Then make a slight modification to the code, setting n to 6.
The code will run as if n still equals to 100, and you will get the same error.
Expected behavior
The code should run as id n=6 on the second try.
Additional context
Please note that I tested this on two separate PC laptops, in both virtual and non-virtual environments, and got the same error each time. Also, there is a follow-up question as to why the analytical machine cannot solve the initial max-cut problem at 100 nodes...?
Thank you,

Hi @robliou , thank you for raising this issue.
I have tried to reproduce your error on a clean install of openqaoa through pip install openqaoa as well as using the current version of dev without success. What version of openqaoa are you using and how did you install the package?
I first execute your code with n=6 and then modify the value of n and as soon as I use q.compile(<MODIFIED QUBO>) the internal number of qubit is updated and the rest follows as expected.
Maybe I misunderstood your question but I still want to add that you need to use q.compile(maxcut_qubo) every time you modify a parameter for the changes to take action. In your case, when you modify n, you need to recreate the graph, then the problem, generate qubo form, and finally compile.
I am quite surprised to see that the analytical simulator produces such an error, a quick search through the code base indicates that it does not appear in our tests anymore. The closest similar ValueError I could find is: ValueError('number of qubits exceeds maximum of 20') but it should not be raised in that case.
My first guess concerning the ValueError is that you tried to compile on another device/simulator where the number of qubits is limited for computational reasons (matrix multiplication in a $2^n$-dimensional vector space is not scalable).
To answer your follow-up question, the analytical simulator should be able to support larger problem sizes since it only computes the analytical expectation value. Naturally, the larger the problem, the longer it takes to compute that value and the optimization becomes more difficult since you also increase the number of parameters linearly in the problem size. However this can only allow you to find the optimal angles. In order to find the most probable state as showed in your code, you would still need to execute/simulate the large circuit with these parameters you found.
Let us know if that helps!


Enhance reporting

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.7%) to 96.979%

Totals

Change from base Build<PHONE_NUMBER>:
-0.7%

Covered Lines:
642

Relevant Lines:
662

💛  - Coveralls


Uncaught DOMException: Failed to execute 'createElementNS' on 'Document': The qualified name provided ('[object HTMLDivElement]') contains the invalid name-start character '['.

This error may indicate that it makes bootstrap modal not able to be used

I tried to see if anything I could help.

It appears to be this code after researching

this might resolve problem

it seems to be conflict between loading-bar and other 3rd libraries. For a quick fix you can try if removing g.append works since this preventing potential issue from conflicts. :)


enhancement make HyperGraph can be built from plan

Proposed changes
make HyperGraph can be built from plan

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall

run buildall

run buildall

run buildall

run buildall

run buildall

run buildall


Use asynchronous wasm instantiation methods where possible

This fixes a problem (introduced in #167) on Chrome, which would complain about the module being too big. It also significantly simplifies the code on the browser entry points, because instantiateStreaming is widely supported.

I've fixed the tsconfig so that tsc actually check JS files in https://github.com/matrix-org/matrix-rust-sdk-crypto-wasm/pull/176. You should probably rebase on that and check that everything typechecks

Will do, but you need to fix the CI for it first ;)

I've merged #176, you should rebase on top of it/merge back main


fix  update status indicator & some notifications are not shown

some notifications are not shown

@fregante Can you help to release a new version?

Done. I’ll have to manually upload the chrome version though. I’ll do it today

Both stores are now up to date


Add GPU support

Before we merge, test that all DEC CUDA matrices load properly

[x] Exterior derivative
[x] Boundary
[x] Dual derivative hodge
[x] Inverse hodge
[x] Primal-Primal Wedge products

We don't have CUDA testing through Github actions but on HiPerGator with CUDA we see that all CUDA tests pass.

Not sure why the ExteriorCalculus tests take so long here but I don't see this behavior on my own machine. I think this is ready to merge.


Persist photo state and upload assets to s3

This makes it possible for the Overview to trigger uploads to S3 and Gemini.
Relies on https://github.com/artsy/metaphysics/pull/786

Fails

:no_entry_sign:

No CHANGELOG added.

  Generated by :no_entry_sign: dangerJS

Fails

:no_entry_sign:

No CHANGELOG added.

:no_entry_sign:

Missing Test Files:

src/lib/Components/Consignments/Submission/__tests__/uploadPhotoToGemini-tests.ts

If these files are supposed to not exist, please update your PR body to include "Skip New Tests".

TSLint Issues:

:no_entry_sign: uploadPhotoToGemini.ts - no-debugger - Use of debugger statements is forbidden

  Generated by :no_entry_sign: dangerJS

Fails

:no_entry_sign:

Missing Test Files:

src/lib/Components/Consignments/Submission/__tests__/uploadPhotoToGemini-tests.ts

If these files are supposed to not exist, please update your PR body to include "Skip New Tests".

  Generated by :no_entry_sign: dangerJS


fix: persistent bar: export buttontype and type cleanup

SUMMARY:
Persistent Bar uses an internal type. Need to Export ButtonType to expose all functionality. Also cleans up types by alphabetizing them.
JIRA TASK (Eightfold Employees Only):
ENG-29273
CHANGE TYPE:

[X] Bugfix Pull Request
[ ] Feature Pull Request

TEST COVERAGE:

[X] Tests for this change already exist
[ ] I have added unittests for this change

TEST PLAN:
pull the PR branch and do yarn and yarn storybook. Verify the Persistent Bar stories behave as expected.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@39d916f). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #373   +/-   ##
=======================================
  Coverage        ?   89.27%
=======================================
  Files           ?      484
  Lines           ?    13071
  Branches        ?     4395
=======================================
  Hits            ?    11669
  Misses          ?     1378
  Partials        ?       24

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Run time error with input dimensions

Hi,
I am trying to run ccpflow for a simple regression example and I keep getting the following error runtime error:
2022-08-29 16:00:11.005482: I tensorflow/cc/saved_model/loader.cc:228] Restoring SavedModel bundle.
2022-08-29 16:00:11.063842: I tensorflow/cc/saved_model/loader.cc:212] Running initialization op on SavedModel bundle at path: ../tfmodel/
2022-08-29 16:00:11.080403: I tensorflow/cc/saved_model/loader.cc:301] SavedModel load for tags { serve }; Status: success: OK. Took 102661 microseconds.

terminate called after throwing an instance of 'std::runtime_error'
  what():  In[0] and In[1] has different ndims: [1] vs. [1,64]
    [[{{function_node __inference__wrapped_model_51816}}{{node sequential_9/dense_16/Relu}}]]

My model is very simple (from https://www.tensorflow.org/tutorials/keras/regression) and saved with regular save.
horsepower = np.array(train_features['Horsepower'])
horsepower_normalizer = layers.Normalization(input_shape=[1,], axis=None, name='hp')
horsepower_normalizer.adapt(horsepower)

horsepower_model = tf.keras.Sequential([
    horsepower_normalizer,
    layers.Dense(64, activation='relu'),
    layers.Dense(64, activation='relu'),
    layers.Dense(1, name='mpg')
])

My cpp code also very simple
#include <iostream>
#include "cppflow/cppflow.h"

int main() {

    // Load the model
    cppflow::model model("../tfmodel/");

    // Set the input
    float input = 100.0;

    // Run

    // Get the input and output graph nodes from saved_model_cli
    auto output = model({{"serving_default_hp_input:0", input}}, {"StatefulPartitionedCall:0"});

    // Show the predicted value
    std::cout << output << std::endl;

   return 0;
}

The code compiles fine and the libraries are linked with CMake on Windows. Other cppflow functions work, but I have this problem with the dimensions
Does anyone knows how to proceed? I have not seen any regression or numeric examples, only image classification.
Thanks for any pointer,
Best
David

Hi @davidissamattos
Probably the issue is that you are feeding a scalar instead of a tensor. Try feeding a tensor of shape [1, 1].
auto input_vector = std::vector<float>({1.0});
auto input_tensor = cppflow::tensor(input_vector, {1, 1});

Awesome! Thanks @serizba! I did try before setting the input as a tensor but (as I realize now) I was also having runtime errors with the output (printing output instead of output[0].
For future reference, this is the working code:
#include <iostream>
#include "cppflow/cppflow.h"

int main() {

    // Load the model
    cppflow::model model("../tfmodel/");

    // Set the input (should be a tensor)
    auto input_vector = std::vector<float>({100.0});
    auto input_tensor = cppflow::tensor(input_vector, {1, 1});

    // Run

    // Get the input and output graph nodes from saved_model_cli
    auto output = model({{"serving_default_hp_input:0", input_tensor}}, {"StatefulPartitionedCall:0"});

    // Show the predicted class
    std::cout << output[0] << std::endl;

    return 0;
}


⚠ ku-s has degraded performance

In 433bcc1, ku-s ($KU_API_STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 19485 ms

Resolved: ku-s performance has improved in f927304 after 49 minutes.


ZEPPELIN-3382 Installing interpreter from UI

What is this PR for?
Installing a new interpreter from helium's UI. You just can click enable button to download a new interpreter you want to install
What type of PR is it?
[Improvement]
Todos

[x] - Changed frontend side to enable users to click an install button
[x] - Added status message for installation
[x] - Added Backend logic to handle an interpreter installation

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-3382

How should this be tested?

Make sure you don't have solr interpreter in an interpreter create menu
Move to Helium/Interpreter menu and click the solr enable button
See the messages for downloading it and having downloaded it
Check interpreter creation menu to make sure solr exists

Screenshots (if appropriate)

Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? Might be

@felixcheung @zjffdu @Leemoonsoo Please review this PR.

Not sure whether periodically thread checking is a good idea. What will happen if the checking thread run in the middle of interpreter copying.
@jongyoul What is your scenario of adding new interpreter ? Another option is adding some kind of trigger in UI (maybe a button) so that admin can trigger the interpreter checking via this trigger.
So this is what I imagine:

First copy new interpreter artifact to interpeter folder
Admin trigger the interpreter checking by clicking some kind of UI in frontend.

@zjffdu Yes, right. I thought about the copying interpreter into the directory. I'll fix it. Actually, I thought this would help to add a new interpreter by the shell script of "install-interpreter.sh" or a simillar way. To go further, I made a method to update it and will add a rest point to update immediately. I'll add a rest endpoint to call register interpreter immediately.

@zjffdu @felixcheung Can you take a look at this PR? I've changed the goal of this PR so that Installing a new interpreter become possible in helium/interpreter tab

@zjffdu Can you check it again?

@jongyoul I tried this PR and download the pig interpreter, and found several issues.

The helium.json still use plugin of 0.7.2, but my zeppelin server is on 0.9 (master). Zeppelin should display the valid interpreter plugins.
I enable zeppelin-pig plugin, and it is downloaded to my local folder interpreter/zeppelin-pig, but I don't think it is correct, because it should be interpreter/pig

@zjffdu

It's what we need to discuss. I, basically, make a new PR for handing version issue for it, but we need to discuss how to manage interpreters' version. Before that, we need to provide a script to get valid helium menu. I plan to make a new issue for handling this.
This is also similar issue the above one. We, currently, don't have any rule for matching installed one and installable one because in case of pig, there's no strict naming rule for pig directory, but you just coded to copy files into interpreter/pig directory. In this time, I agree that I should map heuristic to prevent well-known interpreters from installing twice in a different directory.
I know this issue. To handle this one, we need to solve the first and second one. How about solving this issue with seperate PR?

Concerning what you mentioned the above, I've implemented the second one to check different names with same interpreters. To clarify it, if you install zeppelin-pig in a UI, the logic would check if zeppelin-pig and pig exist. If both don't exist, the logic would install zeppelin-pig in the directory of interpreter/zeppelin-pig. That's because I don't think it's good to change interpreter name with our current unruled format. currently, zeppelin-pig is extracted by maven artifact name. If we made a helium file with interpreter name 'pig', we could install interpreter/pig. I think it's one of the realistic solutions to handle the name of interpreters. WDYT?
And about the first and third one, I think it would be better to handle separate issues.

@zjffdu I addressed all of what you commented before and test passed. https://travis-ci.org/jongyoul/zeppelin/builds/369969538

Thanks @jongyoul  LGTM, and please create apache ticket for the follow up work

Helium menu: https://issues.apache.org/jira/browse/ZEPPELIN-3424
Enable/Disable: https://issues.apache.org/jira/browse/ZEPPELIN-3425


Bug fix

Fixes:

Storing theme in cookies (now with next.js cookies)
Flickering text on home screen
Sidebar now fades in.

Type bug from next js until it is fixt by next team no push