Selecting maps using constants other than binary literals raises an error

Environment

Elixir version (elixir -v): Elixir 1.9.4 (compiled with Erlang/OTP 22)
Database and version (PostgreSQL 9.4, MongoDB 3.2, etc.): PostgreSQL 12
Ecto version (mix deps): ecto 3.2.5 (Hex package) (mix)
Database adapter and version (mix deps): postgrex 0.15.1 (Hex package) (mix)
Operating system: Linux amd64

Current behavior
This doesn't work (schema reference):
# Let target = %{id: 5}

Tagging
|> where(image_id: ^source.id)
|> select([t], %{image_id: ^target.id, tag_id: t.tag_id})
|> Repo.all()

# ** (DBConnection.EncodeError) Postgrex expected a binary, got 5.

In order to work around it, I do this:
Tagging
|> where(image_id: ^source.id)
|> select([t], %{image_id: ^to_string(target.id), tag_id: t.tag_id})
|> Repo.all()
|> Enum.map(&%{&1 | image_id: String.to_integer(&1.image_id)})

Expected behavior
You should be allowed to select whatever you want in a query (assuming it can be casted to a database type) as an interpolated literal and expect it to come back as the same thing.

The issue here is that when you are using a map, we don't actually know the type of image_id. So your schema does not matter. Postgres does infer the type later as a binary - which is a weird choice but one that is out of our control.
Luckily, you can also fix this by explicitly telling Postgres the expected type:
Tagging
|> where(image_id: ^source.id)
|> select([t], %{image_id: type(^target.id, :integer), tag_id: t.tag_id})
|> Repo.all()

Thanks @josevalim . Exactly the problem I faced and I solved it using your advice :)


Upgrade muxado to muxado v2, Yamux, or SPDY

And consider using Protocol Lab's go-stream-muxer

go-stream-muxer doesn't seem to provide much benefit to us, and it adds a dependency. We don't need interop with multiple stream protocols.

I don't think go-stream-muxer provides interop between protocols. I think it simply provides a common go-interface that conforms to https://github.com/diasdavid/interface-stream-muxer
We will either have to write a similar wrapper ourselves, or write more version branching to deal with backwards compatibility.

go-stream-muxer doesn't appear to be go-gettable, which makes it even more of a hassle to use. We'd need to install their gx dependency management tool.
Wrappers are easy to write, and we would need to perform version branching anyway.

Ah, I forgot they used gx. Wrappers are easy to write, I just didn't want to write them myself :P

This issue is now a bit deprecated now, as we're now going for smux, or has that already changed?

indeed. I'll close this once we merge the smux PR (I'm currently putting it together mostly using your code, should be ready soon).


Figures in tutorials are erroneous.

The issue refers to the plots of the various tutorials - e.g. westeros_baseline.ipynb
Problem description

Versions
ixmp:        3.2.1.dev80+g40fc589
     40fc589 (HEAD -> master, origin/master, origin/HEAD) Merge branch 'master' of https://github.com/iiasa/ixmp
message_ix:  3.2.1.dev62+g4cde6cc
     ed74a6f (HEAD -> tutorial_update_addon2, origin/tutorial_update_addon2) black test_tutorials.py
message_data: installed
     3bfb76b (HEAD -> RES_add_5_year_timesteps2, origin/RES_add_5_year_timesteps2) added configuration files for ENGAGE submission 20210331 (ENGAGE 4.1.7)

click:       7.1.2
dask:        2020.12.0
graphviz:    0.13.2
jpype:       1.2.1
… JVM path:  C:\Program Files\Java\jre1.8.0_231\bin\server\jvm.dll
openpyxl:    3.0.5
pandas:      1.1.3
pint:        0.11
xarray:      0.15.1
yaml:        5.3.1

iam_units:   installed
jupyter:     installed
matplotlib:  3.3.2
plotnine:    0.7.0
pyam:        0.7.0+4.gc1ed1f8
     c1ed1f8 (HEAD -> master, origin/master, origin/HEAD) Add a tutorial how to read data from GAMS gdx to pyam (#424)

GAMS:        33.1.0

python:      3.7.9 (default, Aug 31 2020, 17:10:11) [MSC v.1916 64 bit (AMD64)]
python-bits: 64
OS:          Windows
OS-release:  10
machine:     AMD64
processor:   Intel64 Family 6 Model 94 Stepping 3, GenuineIntel
byteorder:   little
LC_ALL:      None
LANG:        None
LOCALE:      None.None

Hi @OFR-IIASA ,
I also noticed that issue when testing the release 3.2.0. See this comment.
From @khaeru's comment, it seems that the value set for the width in this line is too big.

@francescolovat thanks for triaging here. Please keep in mind that links into non-public Slack conversations won't be clickable for others visiting the issue; so in the future, better to copy the text/description into your GitHub comment.
As you said, I think the culprit is this line in the simplified plots used in the tutorials:
https://github.com/iiasa/message_ix/blob/93b65e5f7d8f2b96224179a82e2b1f3d95f11540/message_ix/reporting/computations.py#L96-L106
In Slack, I wrote:

I was looking at the plots for the Austria tutorials, and increased the width of those stacked bars so that those looked reasonable. But it seems like the width is too much for the Westeros examples.

So whoever fixes this should check both sets of tutorials to ensure the output looks reasonable.


Fix #103: Disabling readonly anyOf/oneOf fields

Thanks for the PR.
Can you please include only the src files in the commit and exclude dist and other files outside of src.
dist builds are committed only for new releases.
Other than that, everything looks good.

Done!

@bhch just checking back in - anything else you need from me on this?

Sorry to have kept you waiting. I've merged the changes. I'll do a release in a few days.
BTW, I'm currently working on a new library for generating JSON Schema forms. It will be completely customizable with the ability to write custom widgets, plugins etc, and a good development experience (with storybook etc.).
Repo: https://github.com/topspin-js/topspin. I haven't pushed anything yet but I will soon (I'm trying to clean up my local copy before pushing the code).
Again, thanks for the PR and wish you a happy new year.


Adds support for loading Windows Installs

Requires the user to specify an HTTP network location of extracted ISO.  This is experimental and it may break.

:+5:

Yeah, let me know how it works.  I may also add some hooks into netboot.xyz-custom so that you can preset your url variables by github user.  I haven't tested yet as I need to upload the bits somewhere to try it out.


Add project views

Description
This project will implement views to perform:

user sign up
user sign in
tasks CRUD operations
comment the task

User profile page should be also available.
This page will be used to set user personal information.
DoD

[ ] Views are implemented
[ ] Views respond with appropriate content

Task commenting will be implemented within "[Extras] Django project" milestone.


Completion of Assignment 1

Changed the readme.md to be my own rather than a filled template.

I'm still waiting for your "hello world" issue.


⚠ Bofrost.de has degraded performance

In 7498ad2, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 984 ms

Resolved: Bofrost.de performance has improved in 41fa92b.


how to know the uihost for the AuthenticationURL is missing

Issue description
Hi ,
We are referring to "browser based" type of authentication here as described here:
Exposing the SQL Service for Data Federation with Business User Access from the server
https://help.sap.com/docs/btp/sap-business-technology-platform/exposing-sql-service-for-business-user-access
and
Creating an ODBC Data Source on Windows (Business User Access) which is the consumer gets only the restricted values based on the user authorization assigned to that user
https://help.sap.com/docs/btp/sap-business-technology-platform/creating-odbc-data-source-on-windows-business-user-access.
AuthenticationURL=https:///sap/bc/sec/reentrance
While configuring DSN for a business user, First we have tried hostname from the project properties-> ABAP Development -> System URL, but it didn't work. Next, we tried the uihost from the Fiori Launchpad, which did function as intended.
After making the above correction we are able to consume SQL service from external ODBC-based clients for the business user access.
It would be nice if details provided on how to get the  for the ABAP Environment.
Best Regards,
Biru
Feedback Type (Optional)
clarity
Page Title on SAP Help Portal (prefilled)
Creating an ODBC Data Source on Windows (Business User Access)
Page URL on SAP Help Portal (prefilled)
https://help.sap.com/docs/btp/sap-business-technology-platform/creating-odbc-data-source-on-windows-business-user-access?q=uihost&version=Cloud

Hi @birusahu , thank you for opening an issue. We're looking into it right now. Before we can continue, we'd like to ask you to please clarify what you need details provided on. Your sentence "It would be nice if details provided on how to get the for the ABAP Environment" is missing a word. How to get the what exactly?
Best regards,
Yasmina

Hi there 👋 There's been no activity on this issue for a while, so we're closing it. If you've further information to provide, please comment below and add that information. We'll review it and reopen the conversation if necessary. Until next time, thanks!


networking.networkmanager.dns = systemd-resolved fails because of dbus

Describe the bug
I tried to change the default networkmanager DNS resolver with "systemd-resolved" (to test it mostly)
       networking.networkmanager.dns
           Set the DNS (‘resolv.conf’) processing mode.

           A description of these modes can be found in the main section of https://developer.gnome.org/NetworkManager/stable/NetworkManager.conf.html[1] or in NetworkManager.conf(5).

           Type: one of “default”, “dnsmasq”, “unbound”, “systemd-resolved”, “none”

but after a nixos-rebuild, I would get
resolvectl query google.be
google.be: resolve call failed: Unit dbus-org.freedesktop.resolve1.service not found.
org.freedesktop.resolve1.service': Unit dbus-org.freedesktop.resolve1.service not found.

Steps To Reproduce
with networkmanager enabled, rebuild nixos with  networking.networkmanager.dns = "systemd-resolved"; and run resolvectl query google.be
Expected behavior
Either fix the issue but I feel like with dbus it's not straightforward, otherwise add some more doc in man configuration.nix
It's on unstable

so apparently I needed services.resolved.enable = true;. Leaving this open since it might make sense for network manager to enable or assert if not enabled ?


How to reuse bull connection across module like TypeORM ?

Currently if I setup BullModule in ApplicationModule, I counldn't inject the queue in other module/controller/service.
If I want to use queue in AdminModule, I have to setup BullModule again in AdminModule configuration.
How to archive global inject like TypeORM + nestjs

Yes, I checked the document and developing an app with Nest but I'm still not sure the best way to do it with BullModule. My queue will be doing various tasks, interact with database so I'll need to import other services.
Thanks for you help, I think your solution is clean and easy to implement.


Improve error message when unable to save patient

When creating a new patient via "Build your call list" the patient creation requires these fields:

phone number (must be valid_
name

If they are missing or invalid, an error appears:

An error prevented this patient from being saved

It would be nice to make this error more specific, such as

Error: missing patient name

or

Error: invalid phone number

Right now it's not super clear why things aren't saving, so this is an opportunity to reduce an identified point of frustration for our users.
It should be as simple as correcting the error rendering in the patients controller (though please do some googling around to follow best practices etc).

When the save fails we just need to update the flash msg to use the .errors hash in the create action on the patient's controller

Looking into it 👀


72fps, ATW/ASW in tandem

Hi, first of all, great software you have here! Working stellar with my PC and OGO.
I was wondering if you are planning to introduce 72fps mode for Oculus GO?
Also, what about Asynchronus Space Warp together with your implementation of ATW? My thinking is that with ASW, you only have to stream 50% of the frames to the HMD, reducing latency and requirements even further. Maybe as a toggle in ALVR server.

Yeah you may be right, AWS does seem to be available in mobile SDK.
This is what I've found on 72fps mode: https://developer.oculus.com/documentation/unity/latest/concepts/unity-advanced-go/
OVRManager.display.displayFrequency = 72.0f;

Its possyble 72FPS i any mode of configuration¿?¿?
something like this¿?
vrapi_SetDisplayRefreshRate( app->GetOvrMobile(), 72.0f );

Thank you for the information.
I have release v1.9 which support 72Hz mode!
I can not test because I don't have Go, so please check yourself.

Perfect my friend an incredible work for your part it works perfect

Great!
Thank you for report!


Disable Cascade-publishing of Element in ElementalArea

In a current project i need to allow sorting Elements inside a Elemental area where unpublished elements need to stay unpublished.
Is it possible to remove cascade publishing trough $owns on the Page or ElementalArea by configuration or is there a workaround?
Greetings
Xari

Hi there, since it is defined as an array without keys, you cannot remove it with YAML. You can use Config::modify()->set() in a config.php dile to remove it at runtime though, which would put the publishing burden onto individual elements. Ensure to remove other cascading API configuration as well if you don’t want them.


temperature parameter is not respected

LocalAI version:
cc31c58235496ad7b703b55096172efb1e37feb8
Environment, CPU architecture, OS, and Version:
Linux 6.1.31_1 #1 SMP PREEMPT_DYNAMIC Wed May 31 05:53:37 UTC 2023 x86_64 GNU/Linux
Describe the bug
the temperature parameter seems to be being ignored, as if it's always set to 0. the same output is always returned - and when I run the model using llama.cpp directly with a temp param of 0 I get the same text as what's returned by the /completions endpoint. I'm using a custom model which wasn't trained for chatting, so I'm not sure if this is an issue only for the /completions endpoint, or for all endpoints.
To Reproduce
❯ curl http://localhost:57541/v1/completions -H "Content-Type: application/json" -d '{
     "model": "ggml-model-q4_0.bin",
     "prompt": "",
     "max_tokens": 16,
     "temperature": 0.7
   }'
{"object":"text_completion","model":"pesterbot_5mins-2023_05_19/ggml-model-q4_0.bin","choices":[{"text":" 😂 | I'm so tired of this | I've"}],"usage":{"prompt_tokens":0,"completion_tokens":0,"total_tokens":0}}%
❯ curl http://localhost:57541/v1/completions -H "Content-Type: application/json" -d '{
     "model": "ggml-model-q4_0.bin",
     "prompt": "",
     "max_tokens": 16,
     "temperature": 0.7
   }'
{"object":"text_completion","model":"pesterbot_5mins-2023_05_19/ggml-model-q4_0.bin","choices":[{"text":" 😂 | I'm so tired of this | I've"}],"usage":{"prompt_tokens":0,"completion_tokens":0,"total_tokens":0}}%
❯ curl http://localhost:57541/v1/completions -H "Content-Type: application/json" -d '{
     "model": "ggml-model-q4_0.bin",
     "prompt": "",
     "max_tokens": 16,
     "temperature": 0.7
   }'
{"object":"text_completion","model":"pesterbot_5mins-2023_05_19/ggml-model-q4_0.bin","choices":[{"text":" 😂 | I'm so tired of this | I've"}],"usage":{"prompt_tokens":0,"completion_tokens":0,"total_tokens":0}}%

Expected behavior
receive different outputs if the temp is non-zero.
Logs

Additional context

looking at the logs, the temperature param gets correctly parsed into the config:
11:20PM DBG Parameter Config: &{OpenAIRequest:{Model:pesterbot_5mins-2023_05_19/ggml-model-q4_0.bin File: Language: ResponseFormat: Size: Prompt:<nil> Instruction: Input:<nil> Stop:<nil> Messages:[] Stream:false Echo:false TopP:0.7 TopK:80 Temperature:1.2 Maxtokens:32 N:0 Batch:0 F16:false IgnoreEOS:false RepeatPenalty:0 Keep:0 MirostatETA:0 MirostatTAU:0 Mirostat:0 FrequencyPenalty:0 TFZ:0 Seed:0 Mode:0 Step:0 TypicalP:0} Name: StopWords:[] Cutstrings:[] TrimSpace:[] ContextSize:512 F16:false Threads:4 Debug:true Roles:map[] Embeddings:false Backend: TemplateConfig:{Completion: Chat: Edit:} MirostatETA:0 MirostatTAU:0 Mirostat:0 NGPULayers:0 MMap:false MMlock:false LowVRAM:false TensorSplit: MainGPU: ImageGenerationAssets: PromptCachePath: PromptCacheAll:false PromptCacheRO:false PromptStrings:[] InputStrings:[] InputToken:[]}

Temperature:1.2

I have the same experience.


Issue with Forge Tests Freezing on High Verbose Levels

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (6822860 2024-07-29T00:23:34.055980000Z)
What command(s) is the bug in?
forge test -vvvvv
Operating System
macOS (Apple Silicon)
Describe the bug
Every run with forge test -vvvv has become significantly slower, taking between 3-5 minutes to complete
Please look at it for me. This problem has been bothering me for a long time：
Issue with Forge Tests Freezing on High Verbose Levels #2130.

I even deleted VS Code and its configuration folder ~/Library/Application Support/Code, but that didn’t help either.

Hi @meitedaf we will need more information for this ticket to be actionable.
Can you link to the test you are running or provide a minimal example?
Presumably it is this repo: https://github.com/Cyfrin/foundry-nft-cu
Running with forge test -vvvvv is not unexpectedly slow. Could it be due to your internet connection? If so, try passing the --offline flag.

Hi @meitedaf we will need more information for this ticket to be actionable.嗨，我们需要更多的信息，这张票是可操作的。
Can you link to the test you are running or provide a minimal example?您能链接到您正在运行的测试或提供一个最小的示例吗?
Presumably it is this repo: https://github.com/Cyfrin/foundry-nft-cu大概是这个回购:https://github.com/Cyfrin/foundry-nft-cu
Running with forge test -vvvvv is not unexpectedly slow for me. Could it be due to your internet connection? If so, try passing the --offline flag.跑步对我来说并不是出乎意料的慢。会不会是你的网络问题?如果是，试着传递旗帜。

Yes, you are absolutely correct. I tried adding the --offline flag, and forge test -vvvvv completed very quickly. I’m curious, though, what does adding --offline change? Does it only stop checking dependencies from the network?
Even though I don’t understand why running the project directly from GitHub works fine when I have network issues, but running my own project (with the same version of the libraries imported) has problems, this issue that has troubled me for a long time is now resolved. Thank you!

--offline prevents you from resolving information from Etherscan (this is useful for decoding traces of fork tests, this kicks in at > -vvv) and skips checking for missing solc
After the initial run the selectors are cached until you run forge clean


Use GraphicsMagick

Swap to GraphicsMagick

@pastudan

I think this will break webp conversions as well as OCR functionality

Webp is fixed but OCR functionality is till broken.

We are not using the OCR functionality as per Dan and the Devops team. Can we ignore/kill it?

Looks like only DuelOCR test is failing.

Seems okay to me! I can look at fixing the OCR test in another PR

Thank you! I'll merge this PR for now. I can help with fixing DuelOCR later.


[ci] Enable remaining RMA variants of JTAG tests

This PR enables 4 new JTAG-based tests on CI. I estimate that this should only add ~1 minute of additional CI latency.
cc @mundaym

@dmcardle can you also check that we didn't drop any tests by comparing the log output with a run without these changes? Thanks!

@dmcardle can you also check that we didn't drop any tests by comparing the log output with a run without these changes? Thanks!

I cheated and used bazel query on this PR's branch and origin/master:
dmcardle@dmcardle ~/REPOS/opentitan/wt0 % ./bazelisk.sh query 'tests(//sw/device/silicon_creator/rom/e2e/...)' | sha256sum
Loading: 0 packages loaded
Loading: 1 packages loaded
Loading: 1 packages loaded
09fbc20e58ec91f25bba65d96a29744af8037840f4f3c1dc0565f8b45d0ba6ef  -
dmcardle@dmcardle ~/REPOS/opentitan/wt0 % git checkout origin/master
...
dmcardle@dmcardle ~/REPOS/opentitan/wt0 % ./bazelisk.sh query 'tests(//sw/device/silicon_creator/rom/e2e/...)' | sha256sum
Loading: 0 packages loaded
Loading: 1 packages loaded
Loading: 1 packages loaded
09fbc20e58ec91f25bba65d96a29744af8037840f4f3c1dc0565f8b45d0ba6ef  -

Great, thanks for confirming that we didn't drop any tests!


Any possibility to implement push on top of etcd just like zookeeper.

LGTM.

Can you provide more details?

Closing due to low activity. You can reopen it if there is a follow-up.


portals-list not working on zoom less than 15

This is because most fields in portal.options.data are unassigned

This is local tested and working solution. If somebody can check it and rewrite according good practice... I'm not strong in js
portals-list.user.js.patch.zip


Parallel deploy and failure

Hi! So currently if we have multiple servers and running a parallel deploy, if rocketeer encounters an error on one connection it won't stop the deploy of the other servers.
It would be great if we could change the behavior of the parallel deploy to:

Check/Setup/Deploy on every server in parallel
Wait for all to complete
Change the symlinks on all connections

In this way the release won't be activated until (and only if) all servers were successful. Otherwise we encounter the situation when only 8 out of 10 servers are updated (with migrations, queues etc working on the new codebase) and 2 on the old version. Makes sense?

This is something that would hugely benefit me. Is there anything I could do to help, @Anahkiasen ? Happy to test, give logs, heck, I'll even write some code if you don't got the time. :)

No the branch is currently blocked by https://github.com/phpseclib/phpseclib/issues/506, phpseclib doesn't seem to really play well with being forked by PCNTL which is what we use for parallelism currently unfortunately

Thank you, I'll subscribe to that for updates.
So, you did not want to go the route of a new connection for each fork? Although there would be more overhead, it would allow you to progress here and work backwards later for the extra gain. For me this would still be beneficial since there are many long-running tasks that run on each host of 20, followed by sub-second updates on load balancers and such. My basic goal here is to do all the code updates, preparation, etc, then ping the load balancers last. For now, this is accomplished through roles and without parallelism, and that works ok for now with just a few nodes in the pool, but we're hoping to start pushing to a much larger production pool Soon™
I'm going to take a peek at the other branch this week and see if I can monkey something. Will post a PR if it's promising. I just really don't want to get into phpseclib if at all possible :(

I had thought about disabling the connections cache but I'm afraid it would have a negative impact on performances, will give it a shot though, might work


Can not remove directory inside container created during image build

Description
I am not able to remove non empty directory in docker container. Directory is created as part of image build.
Steps to reproduce the issue:

Dockerfile

FROM ubuntu
RUN mkdir -p /usr/local/bin/test
RUN echo "test" > /usr/local/bin/test/test.txt

Build Image

docker build -t deletetest .

Run container

docker run --rm -it deletetest bash

Describe the results you received:
try to delete directory
root@d284c6c5e4f6:/# cd /usr/local/bin
root@d284c6c5e4f6:/usr/local/bin# rm -rf test/
rm: cannot remove 'test/': Directory not empty

Additional information you deem important (e.g. issue happens only occasionally):
[centos@test]$ docker build -t testdelete .
Sending build context to Docker daemon 2.048 kB
Step 1/3 : FROM ubuntu
 ---> f49eec89601e
Step 2/3 : RUN mkdir -p /usr/local/bin/test
 ---> Running in 1483f6170161
 ---> 0a41a1f634ea
Removing intermediate container 1483f6170161
Step 3/3 : RUN echo "test" > /usr/local/bin/test/test.txt
 ---> Running in 4f27c36a0a05
 ---> 39019e424b60
Removing intermediate container 4f27c36a0a05
Successfully built 39019e424b60
[centos@test]$ docker run -it testdelete bash
root@d284c6c5e4f6:/# cd /usr/local/bin
root@d284c6c5e4f6:/usr/local/bin# ll
total 4
drwxr-xr-x.  3 root root   17 Feb 23 08:30 ./
drwxr-xr-x. 10 root root 4096 Jan 24 09:27 ../
drwxr-xr-x.  2 root root   21 Feb 23 08:30 test/
root@d284c6c5e4f6:/usr/local/bin# rm -rf test/
rm: cannot remove 'test/': Directory not empty
root@d284c6c5e4f6:/usr/local/bin# ll test/
ls: cannot access 'test/test.txt': No such file or directory
total 0
drwxr-xr-x. 1 root root 21 Feb 23 08:37 ./
drwxr-xr-x. 1 root root 17 Feb 23 08:30 ../
??????????? ? ?    ?     ?            ? test.txt

Output of docker version:
[centos@test]$ docker version
Client:
 Version:      1.13.0
 API version:  1.25
 Go version:   go1.7.3
 Git commit:   49bf474
 Built:        Tue Jan 17 09:55:28 2017
 OS/Arch:      linux/amd64

Server:
 Version:      1.13.0
 API version:  1.25 (minimum version 1.12)
 Go version:   go1.7.3
 Git commit:   49bf474
 Built:        Tue Jan 17 09:55:28 2017
 OS/Arch:      linux/amd64
 Experimental: false

Output of docker info:
[centos@test]$ docker info
Containers: 10
 Running: 0
 Paused: 0
 Stopped: 10
Images: 201
Server Version: 1.13.0
Storage Driver: overlay
 Backing Filesystem: xfs
 Supports d_type: false
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: bridge host macvlan null overlay
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Init Binary: docker-init
containerd version: 03e5862ec0d8d3b3f750e19fca3ee367e13c090e
runc version: 2f7393a47307a16f8cee44a37b262e8b81021e3e
init version: 949e6fa
Security Options:
 seccomp
  Profile: default
Kernel Version: 3.10.0-514.6.1.el7.x86_64
Operating System: CentOS Linux 7 (Core)
OSType: linux
Architecture: x86_64
CPUs: 2
Total Memory: 6.896 GiB
Name: <removed>
ID: BQID:S2LK:OB74:DLPS:ICCZ:I2AI:L4WX:EMNM:2OQP:O7NN:FQ6F:SJJA
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): false
Registry: https://index.docker.io/v1/
Experimental: false
Insecure Registries:
 <IP_ADDRESS>/8
Live Restore Enabled: false

Additional environment details (AWS, VirtualBox, physical, etc.):
Working on M3 large EC2 instance

To use overlayfs over xfs, you have to format the xfs filesystem with mkfs.xfs -n ftype=1 /path/to/dev
See https://access.redhat.com/documentation/en-US/Red_Hat_Enterprise_Linux/7/html/7.2_Release_Notes/technology-preview-file_systems.html

Thanks @mountkin, yes that's the cause of this issue. Docker 1.13 adds a warning if the filesystem is not formatted correctly, but will refuse to start without ftype=1 in future; see https://github.com/docker/docker/blob/v1.13.1/docs/deprecated.md#backing-filesystem-without-d_type-support-for-overlayoverlay2 (and https://github.com/docker/docker/pull/27433)
The output of docker info shows if the backing filesystem has this option enabled here;
Supports d_type: false

I'm closing this issue, because this is a duplicate of https://github.com/docker/docker/issues/27358, but feel free to comment after I closed.


getLocalesGlobal() does not gracefully fall back to English on failure

General information
Project issue is for:
Server
Expected result:
DataCallbacks.getLocalesGlobal() Should fall back to English if the locale key doesnt exist.  https://dev.sp-tarkov.com/SPT/Server/src/commit/d13e86ba467deb904a96235757967b3412800ded/project/src/callbacks/DataCallbacks.ts#L180
Actual result:
If the locale key doesn't exist it returns undefined instead of gracefully falling back to English.

Originally written by Cj

Fixed with https://github.com/sp-tarkov/server/commit/e2dd677b40edbfe628130a29d3d7ef9da2b3f496

Originally written by chomp


Remove outdated MNIST build file and edit samples README

Fixes #47

/azp run onnxruntime-inference-examples-PullRequest


Added link to RDS Basics for comparing instances

http://www.ec2instances.info/rds/

👍 Good addition!


Use optional chaining

If you're using a version of TypeScript that supports optional chaining, you can simplify the conditional checks further. Here's an example

Code search shows 54 results for ?..
If you wish to help with these trivial code style changes, please open a PR instead of multiple issues.


from plotly_future import v4 is not compatible with cufflinks 0.15

Maybe it's a bit hasty.
The current cufflinks 0.15 is not compatible with _plotly_future_ introduced recently in 3.8.0 https://github.com/plotly/plotly.py/pull/1474, meaning you cannot use them both.
from _plotly_future_ import v4
import cufflinks as cf
>>><ipython-input-4-3178e833bc72> in <module>
----> 1 import cufflinks as cf

~/.local/share/virtualenvs/arche-lm24e12g/lib/python3.7/site-packages/cufflinks/__init__.py in <module>
     21 from .plotlytools import *
     22 from plotly.graph_objs import *
---> 23 from plotly.plotly import plot
     24 from .colors import cnames, get_colorscale
     25 from .utils import pp

ImportError: cannot import name 'plot' from 'plotly.plotly' (/Users/valery/.local/share/virtualenvs/arche-lm24e12g/lib/python3.7/site-packages/plotly/plotly/__init__.py)

plotly 3.9.0

When I reinstalling my environment yesterday, I notice that if I do:
pip install cufflinks
pip install plotly

when I does
import cufflinks as cf

it will throw an error:
ImportError:
The plotly.plotly module is deprecated,
please install the chart-studio package and use the
chart_studio.plotly module instead

If I install plotly first, the cufflinks install will down version plotly to 3.10.0, and there will be no problem. Will this issue be fix in the next version?

I was able to fix this by editing the cufflinks config files.
Wherever "import plotly.plotly as py" statement was used, I replaced it with "import chart_studio.plotly as py"
And you need to install chart_studio package as well
My packages:
plotly 4.0.0
cufflinks 0.16

@nishu3210 yes, it is how I fix the issue myself as well:

use plotly 4.0.0
install chart_studio
in the cufflinks config files: init.py, plotlytools.py, tools.py, changes plotly.plotly to chart_studio

I had use cufflinks for the past couple days without issue.

I did the same as @TengGao and the solution worked out.

From what I understand chart-studio is meant to be the library that interacts with the plotly hosted service (or perhaps their on-prem offering).
Is there any reason cufflinks would need chart-studio instead of just plotly?

@TengGao Thank you! It worked for me as well.


IntelliSense does not contain entry for read() on open().re

open().re
IntelliSense doesn't contain read

Using our current rules for determining what members are on a particular class, read doesn't actually qualify here. We will need to adjust our approach here and/or special case well-known functions like this.

Already fixed


vlc: glconv_vaapi_x11 gl error: vaInitialize: unknown libva error causes failure to play the video

Describe the bug
Watching a video using vlc causes mentioned error which results in video breaking and switching in-between two frames until restarted
[kreyren@leonid:~]$ vlc Downloads/Rick.and.Morty.S05E07.1080p.WEBRip.x264-CAKES\[rarbg\]/rick.and.morty.s05e07.1080p.webrip.x264-cakes.mkv
VLC media player 3.0.12 Vetinari (revision 3.0.12-1-0-gd147bb5e7e)
[0000000000dea610] main libvlc: Running vlc with the default interface. Use 'cvlc' to use vlc without interface.
libva info: VA-API version 1.12.0
libva info: Trying to open /run/opengl-driver/lib/dri/iHD_drv_video.so
libva info: Trying to open /usr/lib/dri/iHD_drv_video.so
libva info: Trying to open /usr/lib32/dri/iHD_drv_video.so
libva info: va_openDriver() returns -1
libva info: Trying to open /run/opengl-driver/lib/dri/i965_drv_video.so
libva info: Trying to open /usr/lib/dri/i965_drv_video.so
libva info: Trying to open /usr/lib32/dri/i965_drv_video.so
libva info: va_openDriver() returns -1
[00007f63e4001fd0] glconv_vaapi_x11 gl error: vaInitialize: unknown libva error
libva info: VA-API version 1.12.0
libva info: Trying to open /run/opengl-driver/lib/dri/r600_drv_video.so
libva info: Found init function __vaDriverInit_1_12
libva info: va_openDriver() returns 0
[00007f63f8c144a0] avcodec decoder: Using Mesa Gallium driver 21.1.4 for AMD TURKS (DRM 2.50.0 / 5.10.48, LLVM 12.0.0) for hardware decoding
[00007f63f8cfce50] main decoder error: Timestamp conversion failed for 847484001: no reference clock
[00007f63f8cfce50] main decoder error: Could not convert timestamp 0 for faad
[00007f63f8c144a0] main decoder error: Timestamp conversion failed for 847513001: no reference clock
[00007f63f8c144a0] main decoder error: Could not convert timestamp 0 for FFmpeg
[h264 @ 0x7f63f8c894c0] co located POCs unavailable
QObject::~QObject: Timers cannot be stopped from another thread

Steps To Reproduce

Install vlc
Open random video
Let it play and wait for it to fail
2.1. Using arrow keys to skip the time seems to make it more reproducible

Reproducible: Seems to be on random
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[kreyren@leonid:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.10.48, NixOS, 21.11pre301985.23cd13167a1 (Porcupine)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.3.12`
 - channels(root): `"nixos-21.11pre301985.23cd13167a1"`
 - channels(kreyren): `"home-manager-21.05, master"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

I have a similar issue where VLC hardware accellerated video fails to initialize.
libva info: VA-API version 1.11.0
libva info: Trying to open /run/opengl-driver/lib/dri/radeonsi_drv_video.so
libva info: Found init function __vaDriverInit_1_11
libva info: va_openDriver() returns 0
[00007f4f84001790] glconv_vaapi_x11 gl error: vaCreateSurfaces: resource allocation failed
[00007f4f54009fb0] main video output error: video output creation failed
[00007f4f6057b7f0] main decoder error: failed to create video output
[00007f4f6057b7f0] avcodec decoder: Using G3DVL VDPAU Driver Shared Library version 1.0 for hardware decoding
[h264 @ 0x7f4f6053fe00] hardware accelerator failed to decode picture
[h264 @ 0x7f4f6012f000] hardware accelerator failed to decode picture
[h264 @ 0x7f4f6021d7c0] hardware accelerator failed to decode picture

I think the root causes of our issues are probably different, but anyway as a workaround you could try out the different hardware accelerated video backends to see if selecting a specific one resolves the issue.
https://wiki.archlinux.org/title/VLC_media_player#Hardware_video_acceleration

VLC automatically tries to use an available API, but you can override it by going to Tools > Preferences > Input & Codecs and choosing the suitable option under Hardware-accelerated decoding, e.g. Video Acceleration (VA) API for VA-API or Video Decode and Presentation API for Unix (VDPAU) for VDPAU.

I have to manually select "VA-API video decoder" to play video. If none of them work, you might try selecting "Disable" from that menu. It'll use more battery/cpu but you should at least be able to play video.

unstale


get_function_call_parameters() support for dynamic varaibles

As I was working on a sniff for deprecated parameter values. I noticed that for strings the start position did not include any white spaces but when using other values like functions, constants, arrays, or properties the start position included whitespaces.
This is test code
echo bloginfo( function_name() );
echo bloginfo( CONSTANT );
echo bloginfo( array( 'home') );
echo bloginfo( $this->value );

My code to check the output.
$parameters[ $position ]['raw']
$this->tokens[ $parameters[ $position ]['start'] ]['type']
$this->tokens[ $parameters[ $position ]['start'] ]['content']

These are the results

Raw output
"function_name()"
"CONSTANT"
"array( 'home' )"
"$this->value"

Code type for start position
"T_WHITESPACE"
"T_WHITESPACE"
"T_WHITESPACE"
"T_WHITESPACE"

Content for start position
" "
" "
" "
" "

What I expected

Raw output
"function_name()"
"CONSTANT"
"array( 'home' )"
"$this->value"

Code type for start position
"T_STRING"
"CONSTANT"
"T_ARRAY"
"T_VARIABLE"

Content for start position
"function_name"
"CONSTANT"
"array"
"$this"

I noticed that for strings the start position did not include any white spaces

@grappler It's unclear to me what your issue is.
AFAICS the function does exactly what it says: it gives the start and end position of the parameter and the trimmed raw content.
Whitespace is not disregarded when determining the start/end position as the output of the function should also be usable to do code style checks on parameters. All token types inside parameters are treated the same, string or no string token type.

@jrfnl Sorry, I made a mistake, the white space is included if there is only a string too.
I have created a PR which should fix this for all types of data. I find that the start point should be from the point the parameter starts and not after the bracket opens or after the comma.

@grappler I do not agree and strongly object to your PR.
If all you need to check is the content of a param, you have the raw value already.
If you need to check for specific token types or combinations, you should always walk the tokens using the start and end values. That's what they are for.
Having accurate start and end tokens available makes for more efficient and accurate results using this function than walking everything between brackets in various inconsistent ways across different sniffs.
Your proposed change has the following drawbacks:

The results of the function can no longer be used for checking whitespace code style.
Having the whitespace removed will very easily mislead people into writing sniffs which just check the start token instead of properly walking the start to end tokens to check whatever they need to check which will lead to buggy sniffs.

I'm seriously wondering why you want to change this. The only thing I can think of is for the above mentioned second point which would often be the wrong way to go.
If that however is your underlying issue, I would rather suggest an alternative: adding an extra utility function is_function_param_one_token() (or something along those lines) which will return false if there are more than one non-whitespace tokens in the param and will return the token array for the one token which is non-whitespace in the param if the param only has one non-whitespace token.

Sorry, I see what you mean. I have adapted my code as it is a lot simpler. Closing as it is not needed anymore.


feat: adding codeforces scrapper

Pull Request Template
script name - Codeforces scraper
Brief about script
This python script will let the user scrape problem statements from Codeforces. In the case of contests, this can quickly sum all the questions in the contest and thus save problems for challenging problems.
Issue no.(must) - #569
Self Check(Tick After Making pull Request)

[x] This issue was assigned to me.
[x] One Change in one Pull Request
[x] My file is in proper folder (Name of folder should be in lowercase with no space in between) (E.g. meet_schedular)
[x] I am following clean code and Documentation and my code is well linted with flake8.
[x] I have added README.md and requirements.txt (Include version numbers too e.g. pandas==0.0.1) with my script
[x] Just including required dependencies in requirements.txt (Don't include Python version too)

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

Good work @decipher07


UseDataClass - exclude classes with interfaces

Expected Behavior
Class below shouldn't be data class
Current Behavior
Class below is data class from detekt prospective
Context
interface I1
interface I2

class Container(
     val i1: I1,
     val i2: I2
)

Class above is just container which walks between methods. It has interfaces only, so in the most cases the toString/equality methods are useless here.
Another example - context classes, which are used as service locators.

Thanks for your this report @imanushin
From my point of view, this looks like a special scenario that you should suppress locally in your code (nit: you could also define a custom annotation and use the excludeAnnotatedClasses config of UseDataClass to exclude such classes).
To follow up on your sample, this class:
class Mailbox(
    val inbox : List<String>,
    val outgoing : Queue<String>
)

that contains only interfaces seems like a good candidate for a data class.
Leaving open to hear others' opinion.

I agree with Nico here. It’s hard to find a rule that decides, whether this is a candidate or not.

I think that there's very little that we can do here. I think that the custom annotation is the best option, as @cortinico say.
If you have a lot of this classes other option is to disable this rule all together. Other option is to disable it. I don't think that this rule could fit every project. And there is not point to force it.

Closed due to the reasons mentioned.

@schalkms, @BraisGabin , agree, I see two ways here: close issue or create AI to split service interfaces from dto interfaces. It is reasonable just to close this issue, to avoid overcomplicated logic for this rule.

If you can come up with something reasonable, we can discuss about merging it. For now, I don’t think it’s worth to put more effort into it also considering the number of items with higher priority.


Exit with a non-zero exitcode if the kernel panics.

Uhyve should exit with a non-zero exitcode if the kernel panics. This is important for testing rusty-hermit with uhyve and CI, since I'm not a fan of parsing the output of stdout or stderr to determine if a program failed.
I'm not sure if Uhyve always returns a zero, but at least in the cases I've seen where uhyve shuts down due to a panic of the kernel, the exit code was zero.

This is can easily be implemented, as the signal for the shutdown of the hypervisor is writing an value to a certain COM port. So you can return an arbitrary return code from the application and for example return another value in the panic handler than on regular shutdown.
I've already implemented this for uhyve's little sibling - ehyve. It can be easily backported for uhyve.
Would this be sufficient?

Yes, That would be absolutely sufficient. Thanks for your help :)

Took me embarrasingly long to realize, but this is actually the behaviour that uhyve has by default.
Panicing
#[cfg(target_os = "hermit")]
extern crate hermit_sys;

fn main()  {
        println!("Hello World!");
        panic!("asdf");
}

> cargo build -Z build-std=std,core,alloc,panic_abort --target x86_64-unknown-hermit
# [...]

> uhyve target/x86_64-unknown-hermit/debug/hermit-new
Hello World!
thread 'main' panicked at 'asdf', src/main.rs:6:9
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

> echo $?
255

Return Error
#[cfg(target_os = "hermit")]
extern crate hermit_sys;

use std::io::{Error, ErrorKind};

fn main()  -> Result<(), std::io::Error> {
        println!("Hello World!");
        Err(Error::new(ErrorKind::Other, "oh no!"))
}

> uhyve target/x86_64-unknown-hermit/debug/hermit-new
Hello World!
Error: Custom { kind: Other, error: "oh no!" }

> echo $?
1

I'd like to reopen this issue. I checked and the two examples you posted work for me too. However this does not work for all cases.
To reproduce do the following:

Clone rusty-hermit
Checkout devel branch
Switch to a new branch at commit 4b74fa4889f60c27eda89c40d3b22598ffc54ff5 (newer commits are not building for me)
Initialize and update submodules
Run cargo build -Z build-std=std,core,alloc,panic_abort --target x86_64-unknown-hermit
Run  uhyve -v target/x86_64-unknown-hermit/debug/rusty_demo
Verify that this works and return code is 0 (no problem here)
cd libhermit-rs/src/arch/x86_64
Comment out lines 411 to 415 so that the we run into the "expect".
cd back to rusty-hermit
Run cargo clean
Run cargo build -Z build-std=std,core,alloc,panic_abort --target x86_64-unknown-hermit
Run  uhyve -v target/x86_64-unknown-hermit/debug/rusty_demo
Run echo $?. For me this gives 0, even though rusty-hermit crashed.

This is shows an issue that rusty-hermit may have when handling panics, but I should probably create a seperate issue for that, since that isn't uhyve related.
Output of step 13

[0][INFO] Welcome to HermitCore-rs 0.3.26
[0][INFO] Kernel starts at 0x200000
[0][INFO] BSS starts at 0x445600
[0][INFO] TLS starts at 0x443828 (size 264 Bytes)
[0][INFO] Total memory size: 64 MB
[0][INFO] A pure Rust application is running on top of HermitCore!
[0][INFO] Heap: size 50 MB, start address 0x600000
[0][INFO] Heap is located at 0x600000 -- 0x3800000 (0 Bytes unmapped)
[0][INFO]
[0][INFO] ===================== PHYSICAL MEMORY FREE LIST ======================
[0][INFO] 0x00000003800000 - 0x00000004000000
[0][INFO] ======================================================================
[0][INFO]
[0][INFO]
[0][INFO] ================== KERNEL VIRTUAL MEMORY FREE LIST ===================
[0][INFO] 0x00000003800000 - 0x00800000000000
[0][INFO] ======================================================================
[0][INFO]
[0][TRACE] __sys_malloc: allocate memory at 0x6000e8 (size 0x408, align 0x8)
[0][ERROR] Page Fault (#PF) Exception: ExceptionStackFrame {
    instruction_pointer: 0x38c66b,
    code_segment: 0x8,
    cpu_flags: 0x10287,
    stack_pointer: 0x1ffb90,
    stack_segment: 0x10,
}
[0][ERROR] virtual_address = 0x3830000, page fault error = The fault was caused by a non-present page.
The access causing the fault was a read.
The access causing the fault originated when the processor was executing in supervisor mode.
The fault was not caused by reserved bit violation.
The fault was not caused by an instruction fetch.
[0][ERROR] fs = 0x0, gs = 0x447728
[0][ERROR] General Protection (#GP) Exception: ExceptionStackFrame {
[Redacted everything inbetween here since I hit the Character limit]

[0][ERROR] General Protection (#GP) Exception: ExceptionStackFrame {

Dump state of CPU 0

Registers:
----------
kvm_regs { rax: 3026652, rbx: 3026652, rcx: 19, rdx: 3026652, rsi: 19, rdi: 3026652, rsp: 73728, rbp: 0, r8: 19, r9: 1, r10: 0, r11: 0, r12: 19, r13: 73824, r14: 19, r15: 0, rip: 3799712, rflags: 65554 }kvm_sregs { cs: kvm_segment { base: 0, limit: 0, selector: 8, type_: 11, present: 1, dpl: 0, db: 0, s: 1, l: 1, g: 0, avl: 0, unusable: 0, padding: 0 }, ds: kvm_segment { base: 0, limit: 0, selector: 16, type_: 3, present: 1, dpl: 0, db: 0, s: 1, l: 0, g: 0, avl: 0, unusable: 0, padding: 0 }, es: kvm_segment { base: 0, limit: 0, selector: 16, type_: 3, present: 1, dpl: 0, db: 0, s: 1, l: 0, g: 0, avl: 0, unusable: 0, padding: 0 }, fs: kvm_segment { base: 0, limit: 65535, selector: 0, type_: 3, present: 1, dpl: 0, db: 0, s: 1, l: 0, g: 0, avl: 0, unusable: 0, padding: 0 }, gs: kvm_segment { base: 4486952, limit: 65535, selector: 0, type_: 3, present: 1, dpl: 0, db: 0, s: 1, l: 0, g: 0, avl: 0, unusable: 0, padding: 0 }, ss: kvm_segment { base: 0, limit: 0, selector: 16, type_: 3, present: 1, dpl: 0, db: 0, s: 1, l: 0, g: 0, avl: 0, unusable: 0, padding: 0 }, tr: kvm_segment { base: 6291456, limit: 103, selector: 24, type_: 11, present: 1, dpl: 0, db: 0, s: 0, l: 0, g: 0, avl: 0, unusable: 0, padding: 0 }, ldt: kvm_segment { base: 0, limit: 65535, selector: 0, type_: 2, present: 1, dpl: 0, db: 0, s: 0, l: 0, g: 0, avl: 0, unusable: 0, padding: 0 }, gdt: kvm_dtable { base: 58720256, limit: 65535, padding: [0, 0, 0] }, idt: kvm_dtable { base: 4482844, limit: 4095, padding: [0, 0, 0] }, cr0:<PHONE_NUMBER>, cr2: 6291500, cr3: 65536, cr4: 329312, cr8: 0, efer: 3329, apic_base:<PHONE_NUMBER>, interrupt_bitmap: [0, 0, 0, 0] }
Segment registers:
------------------
register  selector  base              limit     type  p dpl db s l g avl
cs        kvm_segment { base: 0, limit: 0, selector: 8, type_: 11, present: 1, dpl: 0, db: 0, s: 1, l: 1, g: 0, avl: 0, unusable: 0, padding: 0 }
ss        kvm_segment { base: 0, limit: 0, selector: 16, type_: 3, present: 1, dpl: 0, db: 0, s: 1, l: 0, g: 0, avl: 0, unusable: 0, padding: 0 }
ds        kvm_segment { base: 0, limit: 0, selector: 16, type_: 3, present: 1, dpl: 0, db: 0, s: 1, l: 0, g: 0, avl: 0, unusable: 0, padding: 0 }
es        kvm_segment { base: 0, limit: 0, selector: 16, type_: 3, present: 1, dpl: 0, db: 0, s: 1, l: 0, g: 0, avl: 0, unusable: 0, padding: 0 }
fs        kvm_segment { base: 0, limit: 65535, selector: 0, type_: 3, present: 1, dpl: 0, db: 0, s: 1, l: 0, g: 0, avl: 0, unusable: 0, padding: 0 }
gs        kvm_segment { base: 4486952, limit: 65535, selector: 0, type_: 3, present: 1, dpl: 0, db: 0, s: 1, l: 0, g: 0, avl: 0, unusable: 0, padding: 0 }
tr        kvm_segment { base: 6291456, limit: 103, selector: 24, type_: 11, present: 1, dpl: 0, db: 0, s: 0, l: 0, g: 0, avl: 0, unusable: 0, padding: 0 }
ldt       kvm_segment { base: 0, limit: 65535, selector: 0, type_: 2, present: 1, dpl: 0, db: 0, s: 0, l: 0, g: 0, avl: 0, unusable: 0, padding: 0 }
gdt                 kvm_dtable { base: 58720256, limit: 65535, padding: [0, 0, 0] }
idt                 kvm_dtable { base: 4482844, limit: 4095, padding: [0, 0, 0] }

APIC:
-----
efer:<PHONE_NUMBER>000d01  apic base: 00000000fee00900

Ok, seems that a userspace panic produces a correct return value, but a panic in the kernel code does not.

I'd like to link to the Travis CI build for commit 4a2381ccc192b3d699fd4b54fb007e297bac9e2d. In release mode it seems a race condition occured that caused a Panic due to a BorrowError: [0][!!!PANIC!!!] /home/travis/.rustup/toolchains/nightly-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/src/libcore/cell.rs:798: already mutably borrowed: BorrowError
Given the output I assume that the BorrowError occured in the kernel or demo and not in uhyve.
The thing that makes this related to this isssue is that uhyve doesn't exit after the panic.
I've saved the log output here so it doesn't get lost if someone reruns the build.
log.txt

I'm updating this Issue, since apparently it is still relevant.
https://git.rwth-aachen.de/acs/public/hermitcore/libhermit-rs/-/jobs/1378271 shows a CI run where the kernel crashed, but uhyve exited with 0 and the test was marked as passed.

I'm updating this Issue, since apparently it is still relevant.
https://git.rwth-aachen.de/acs/public/hermitcore/libhermit-rs/-/jobs/1378271 shows a CI run where the kernel crashed, but uhyve exited with 0 and the test was marked as passed.


Refueling with over 20 items in FC inventory

Hi and thanks for the good work.  I used this for another account and it was fine, however on the current account I can't get the refuel to work and am wondering if it is because I have too many different items in my FC inventory?
I also converted it to work for my 4K screen... so far, so good.

That is likely it.   The current version is set to trying going up and down up to 10 key presses each looking for tritium.   So if your inventory is large it’s probably not looking far enough.  It’s a simple change to make it 20 instead of 10.   Glad you got it working on 4K.
I’ve been having trouble with the auto jump timing as it seems to be trying to jump just a few seconds before the cooldown completes.   So I’m going to have to slow that down by 30 seconds or so.    I’ll try and remember to add in a longer look for tritium in the update with the increased delay.
Cheers.
On 19 Sep 2022, 11:21 +0100, rmw00 @.***>, wrote:

Hi and thanks for the good work. I used this for another account and it was fine, however on the current account I can't get the refuel to work and am wondering if it is because I have too many different items in my FC inventory?
I also converted it to work for my 4K screen... so far, so good.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

I just jumped my carrier to Colonia using the macros.  I found refueling to be unreliable and haven't had time to look into it.  So, I just did 10 jumps then manually refueled.  Wat better than having to do the 40 jumps manually.

It is definitely struggling with the refuel.  Seems to work when I watch but enroute it struggles.  Still 10 jumps at a time is pretty good.
On 11 Oct 2022 at 15:36 +0100, rmw00 @.***>, wrote:

I just jumped my carrier to Colonia using the macros. I found refueling to be unreliable and haven't had time to look into it. So, I just did 10 jumps then manually refueled. Wat better than having to do the 40 jumps manually.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>


Tree view pages The property 'IndentationControls' cannot be used in this contextAL(AL0124)

Describe the bug
When creating a new page, which I want to use as tree page, I Get
The property 'IndentationControls' cannot be used in this contextAL(AL0124)
To Reproduce
This gives an error whitout any explication
page 95000 "PBL Contact List"
{
ApplicationArea = All;
UsageCategory = Lists;
SourceTable = Contact;
Caption = 'Valcredo Contacts';
CardPageID = "Contact Card";
PageType = List;
IndentationControls = "No.";
The property 'IndentationControls' cannot be used in this contextAL(AL0124)
Expected behavior
Screenshots
If applicable, add screenshots to help explain your problem.

5. Versions:

AL Language: 5.0.254558

Business Central: BC

Hi,
The property has to be set on the repeater control, not on the page directly.
See: https://docs.microsoft.com/en-us/dynamics365/business-central/dev-itpro/developer/properties/devenv-indentationcontrols-property

My bad.
Thanks for the quick feedback


Taichi_colon_030518

Latest

Hi Olga,
Sorry, I don’t know how I know your change.
Please let me know how to do it.
Best,
Taichi
Taichi Isobe, M.D., Ph.D.
Research Scientist
Michael F Clarke Lab.
Institute for Stem Cell Biology and Regenerative Medicine
Stanford University
265 Campus Drive, G2015
Stanford, CA 94305, United States
E-mail<EMAIL_ADDRESS>TEL:<PHONE_NUMBER> FAX:<PHONE_NUMBER>

On Mar 8, 2018, at 10:20, Olga Botvinnik<EMAIL_ADDRESS>wrote:
@olgabot commented on this pull request.
In 00_data_ingest/02_tissue_analysis_rmd/Colon_facs.Rmd https://github.com/czbiohub/tabula-muris/pull/144#discussion_r172690946:

NA, # 'Enterocyte (Proximal)',

NA, # 'Lgr5+ amplifying undifferentiated cell',
NA, # 'Lgr5+ undifferentiated cell (Distal)',
NA, # 'Lgr5- amplifying undifferentiated cell',
NA, # 'Enterocyte (Proximal)',
NA, # 'Goblet cell (Distal)',
NA, # 'Goblet cell (Distal)',
NA, # 'Enterocyte (Distal)',
NA, # 'Goblet cell (Proximal)',
NA, # 'Goblet cell (Proximal)',
NA, # 'Goblet cell (Distal)',
NA, # 'Tuft cell',
NA  # 'Chromaffin Cell'
)

cell_ontology_class <-
@tisobe1 https://github.com/tisobe1 can you make sure these are correct?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/czbiohub/tabula-muris/pull/144#pullrequestreview-101752168, or mute the thread https://github.com/notifications/unsubscribe-auth/AimgezckaKQyCOnDW5_gZlNJjK57uzcLks5tcXZWgaJpZM4Sd41C.


Add a button to rename resolved identifiers' aliases in Connections

Is your feature request related to a problem? Please describe.

Nope. This feature is not related to a problem.

Describe the solution you'd like

Add an option to rename the alias of each resolved identifier.
Perhaps, a "Rename" button could be added just below the "View" button in the same manner as those in the "Identifiers" tab.

Describe alternatives you've considered

At the moment, the alias can be changed by re-resolving the same OOBI URL with a new alias. This is not obvious for users.

Additional context

I believe we already have this. Do you not see this in your UI?

@dhh1128 I only see it in the "Identifiers" tab, not in the "Connections" (OOBI) tab.

Ah, thank you for the clarification. You're right, we should add the same option in Connections.

[from twin in jira]
This issue tracks jira  https://eipi.atlassian.net/browse/DF-2313.
To send new comments there, start them with 'Tell {location}:'.
The issue is assigned in jira to nobody.
As of 2024-08-21T17:10Z, the status of the issue in jira is 'to do'.

tell jira: note that we already have this in the Identifiers tab; we just need to expose in Connections as well.

[from twin in jira]
The issue is assigned in jira to Vlad Radulescu.

[from twin in jira]
As of 2024-09-25T15:34Z, the status of the issue in jira is 'doing'.

[from twin in jira]
As of 2024-09-27T09:59Z, the status of the issue in jira is 'to do'.


Add fixed NNDVI

Adds

NNSpacePartitioner, NNDVI partitioning and detection algorithms
examples, documentation, tests

Fixes

ongoing false/indiscriminate alarm issue with NNDVI

Closes #36

@tms-bananaquit I think you can begin review of this PR while I work out remaining issues with formatting/coverage. The major thing I anticipate you'll notice is the hard-coded unit tests for some of these compute functions (and elsewhere), for which I couldn't really find an alternative for now

The sklearn dependency is already listed as part of install_requires, so we shouldn't need to add it to the other settings in setup.cfg. The logs show the build step is pulling python ver 3.10.8 instead of 3.10.7, and the sklearn .tar is a different file, but installation is apparently finishing successfully.
Apparently installing from the sklearn alias has been deprecated for a while, so I switched the dependency to scikit-learn. That fixed the build issue. Now there are some unit tests failing for other modules, which doesn't make a whole lot of sense given what code has(n't) been modified.

The only full failure I see is the coverage 100% due to un-tested lines in nndvi.py -- addressing that now

The mu == 0 case was removed because it's effectively no longer possible with the new change (and was cutting warnings in a very unlikely case to begin with). The remaining line to be covered is just the if drift, reset line which for some reason isn't being counted by one of the unit tests.

Should be ready for review now

Did the validation / docstring changes, waiting on some example business to resolve that comment too.


Capability to advertise different address than the hostname of host.

We are trying to implemenent multi data center Couchbase clusters, we will have two or more Couchbase servers running in one data center and two or more running in another data center, providing HA across data center as well as failover capabilites within data center.
Question:
Can we setup a Couchbase Server to advertise a different hostname or IP address than is defined for the host running the Couchbase Server?
Details:
Each data center will have its own Kubernetes Cluster and Couchbase clusters will be deployed on Kubernetes Cluster. As per our current understanding and analysis, Couchbase servers need to reach each individual server in the cluster and across data center. First it reaches to Admin Port and from there it gets the Hostname/IP address to connect on intra node ports.
In a single kubernetes cluster we could have used Stateful Sets and each node could discover each other with internal pod ips, however these pods ips are internal to cluster only and are not routeable outside cluster. So this approach will not work with multiple kubernetes cluster.
To mitigate this issue, we decided to use Demonsets and pinned different Clouchbase Nodes to particular hosts. This way we can port forward the ports from known host to internal pods. This works well for admin port 8091, however when to reach to intra node ports, they use the Hostname/IP of the pod itself which is not routeable. We tried to set the hostname with the name which can be resolved by our dns server, but it seems that Couchbase should be able to listen to that address, only then it will accept it as Hostname.
To mitigate this issue we decided to use use the hostNetwork feature of Kubernetes, which essentially replicates the same network interfaces inside the pod. This worked perfectly as now advertised address is host address and pod is able to listen on that address. But this is not advisable from security standpoint as now pod can influence host network.
Is there a way in which we can listen/bind to internal pod ip (without host networking) and Advertise the host address and then forward the port from host to pod?

@nitinmidha Checking with the team. Do you mind asking this question on Couchbase Forums?


delete-empty-src-dirs not deleting folders.

Command used: rclone move -vv -L --delete-empty-src-dirs --ignore-existing --fast-list --stats=30s --drive-chunk-size=256M --buffer-size=256M "/home/user1/data/upload/" gdrive:
After uploading everything, every folder inside /home/user1/data/upload/ will still be there.
The log doesn't show anything interesting at the end:

2018/07/02 11:34:56 DEBUG : 5 go routines active
2018/07/02 11:34:56 DEBUG : rclone: Version "v1.42" finishing with parameters ["rclone" "move" "-vv" "-L" "--delete-empty-src-dirs" "--ignore-existing" "--fast-list" "--stats=30s" "--drive-chunk-size=256M" "--buffer-size=256M" "/home/user1/data/upload/" "gdrive:"

I can confirm this...
$ mkdir -p src/dir
$ echo hello > src/dir/file.txt
$ tree src
src
└── dir
    └── file.txt

1 directory, 1 file
$ rclone move -vv --delete-empty-src-dirs --disable DirMove src dst
2018/07/02 14:04:14 DEBUG : rclone: Version "v1.42-016-g97beff53" starting with parameters ["rclone" "move" "-vv" "--delete-empty-src-dirs" "--disable" "DirMove" "src" "dst"]
2018/07/02 14:04:14 DEBUG : Using config file from "/home/ncw/.rclone.conf"
2018/07/02 14:04:14 DEBUG : Reset feature "DirMove"
2018/07/02 14:04:14 DEBUG : Reset feature "DirMove"
2018/07/02 14:04:14 INFO  : Local file system at /tmp/dst: Waiting for checks to finish
2018/07/02 14:04:14 INFO  : Local file system at /tmp/dst: Waiting for transfers to finish
2018/07/02 14:04:14 INFO  : dir/file.txt: Moved (server side)
2018/07/02 14:04:14 INFO  :
Transferred:      0 Bytes (0 Bytes/s)
Errors:                 0
Checks:                 0
Transferred:            1
Elapsed time:          0s

2018/07/02 14:04:14 DEBUG : 2 go routines active
2018/07/02 14:04:14 DEBUG : rclone: Version "v1.42-016-g97beff53" finishing with parameters ["rclone" "move" "-vv" "--delete-empty-src-dirs" "--disable" "DirMove" "src" "dst"]
$
$ tree src
src
└── dir

1 directory, 0 files
$ tree dst
dst
└── dir
    └── file.txt

1 directory, 1 file

It does work with 1.41 - a bit of bisecting reveals this is the problem
commit 737aed8412c28641fcd82755b74346b00e9e1cd9
Author: ishuah<EMAIL_ADDRESS>Date:   Mon May 14 20:16:56 2018 +0300

    Ensure items in srcEmptyDirs are actually empty

:040000 040000 e86f9cf64558491328cef3833970e26ac9fed51d 8da1b952c1eb90a2064b46714dcc38faaab6333d M	fs

Any ideas why 737aed8412c28641fcd82755b74346b00e9e1cd9 broke this @ishuah ?
It would be a good software engineering to first make a failing test then fix it

@KurtzPT I see where it's breaking.
I added a line to remove a dir from the srcEmptyDirs map if the dir has/had a file. So when processing a move, all dirs which had files before the move won't be in the srcEmptyDirs map, meaning they won't be deleted.
Testing with an empty second src dir proves this:
$ mkdir -p src/dir
$ mkdir src/dir2
$ echo hello > src/dir/hello.txt
$ tree src
src
├── dir
│   └── file.txt
└── dir2

2 directories, 1 file
$ rclone move -vv --delete-empty-src-dirs --disable DirMove src dst
2018/07/11 15:29:15 DEBUG : rclone: Version "v1.42" starting with parameters ["rclone" "move" "-vv" "--delete-empty-src-dirs" "--disable" "DirMove" "src" "dst"]
2018/07/11 15:29:15 DEBUG : Using config file from "/Users/ishuahkariuki/.config/rclone/rclone.conf"
2018/07/11 15:29:15 DEBUG : Reset feature "DirMove"
2018/07/11 15:29:15 DEBUG : Reset feature "DirMove"
2018/07/11 15:29:15 INFO  : Local file system at /Users/ishuahkariuki/Workshop/go_workshop/src/github.com/ncw/lab/dst: Waiting for checks to finish
2018/07/11 15:29:15 INFO  : Local file system at /Users/ishuahkariuki/Workshop/go_workshop/src/github.com/ncw/lab/dst: Waiting for transfers to finish
2018/07/11 15:29:15 INFO  : dir/file.txt: Moved (server side)
2018/07/11 15:29:15 DEBUG : Local file system at /Users/ishuahkariuki/Workshop/go_workshop/src/github.com/ncw/lab/dst: copied 1 directories
2018/07/11 15:29:15 DEBUG : dir2: Removing directory
2018/07/11 15:29:15 DEBUG : Local file system at /Users/ishuahkariuki/Workshop/go_workshop/src/github.com/ncw/lab/src: deleted 1 directories
2018/07/11 15:29:15 INFO  :
Transferred:      0 Bytes (0 Bytes/s)
Errors:                 0
Checks:                 0
Transferred:            1
Elapsed time:          0s

2018/07/11 15:29:15 DEBUG : 4 go routines active
2018/07/11 15:29:15 DEBUG : rclone: Version "v1.42" finishing with parameters ["rclone" "move" "-vv" "--delete-empty-src-dirs" "--disable" "DirMove" "src" "dst"]
$ tree src
src
└── dir

1 directory, 0 files
$ tree dst
dst
├── dir
│   └── file.txt
└── dir2

2 directories, 1 file

@ncw I'll write proper tests for this before fixing it.

Thanks @ishuah :-)

@ishuah has fixed this now
https://beta.rclone.org/v1.42-050-gdcc74fa4/ (uploaded in 15-30 mins)
This will be in the latest beta from now on and v1.43

I am using version 1.43 and I am still experiencing this. Is it possible that the fix has been missed out of 1.43?

@WeKnowMC that fix is definitely in 1.43
The problem as I reformulated in my reply to the original issue is fixed (I just checked), but there may be more problems!
Can you make an example like the one I made above showing the problem and make a new issue on github with that in?
Thanks


How to set Video Duration Same as Audio

I want to stop video recording when mp3 audio is finished (+ some gap).
May be i can use video.addAudio() somehow?
Another idea is to use pagePrepareFn and "onloadedmetadata" event. But how to update "duration" setting at this moment?

@ivangusev
Currently WVC supports the following methods to set audio:

wvc.createSingleVideo({ ... }).addAudio({ ... });
Call captureCtx.addAudio({ ... }) on the page
Insert the <audio> element into the DOM tree

If you want to terminate the video capture when the audio playback ends, this may not be easy to control. The most direct method is to set the video duration to be the same as the audio duration, which requires predicting the audio duration.
Another solution is to use captureCtx.currentTime on the page to determine whether the audio duration is currently reached, and when the duration is reached, set captureCtx.stopFlag to true to automatically trigger termination.

@Vinlic, i can't confirm. I updated to 0.0.32. Now i got error:
Error: Cannot set properties of undefined (setting 'duration')
Also, is it possible to access this.audios this way?

@ivangusev
Have you confirmed that the "this" pointer is pointing to the correct object?

@Vinlic, "this" is undefined inside pagePrepareFn


Orchestrator support: Meltano

Is your feature request related to a problem? Please describe.
https://docs.elementary-data.com/deployment-and-configuration/collect-job-data#cant-find-your-orchestrator-missing-info mentions that I should open an issue for an orchestrator that doesn't appear in the list.
We run dbt-core within a Meltano project and it would be very valuable to have job info from the Meltano invoker captured within elementary metadata.
Describe the solution you'd like
It would be great to have Meltano metadata captured in the same orchestrator metadata model as others:
Orchestrator name: meltano
Job name: job_name (Meltano job_name)
Job ID: n/a ?
Job results URL: n/a ?
The ID of a specific run execution: job_run_id (Meltano run_id )
Job run results URL: n/a ?
Describe alternatives you've considered
not sure
Additional context
Would you be willing to contribute this feature?
Yes!

Hi @menzenski, you can do it by filling the values in this macro. Here's our contribution guide. Let me know if you need any help.


Rviz not generating a map, but Turtlebot 4 is visable

Please provide the following information:

OS: Ubuntu 20.04
ROS Distro: Humble 1.2
Built from source or installed:
Package version:
sudo dpkg -s ros-humble-turtlebot4-navigation
Package: ros-humble-turtlebot4-navigation
Status: install ok installed
Priority: optional
Section: misc
Installed-Size: 2470
Maintainer: rkreinin<EMAIL_ADDRESS>Architecture: amd64
Version: 1.0.2-1jammy.20230516.173216
Depends: ros-humble-nav2-bringup, ros-humble-nav2-simple-commander, ros-humble-slam-toolbox, ros-humble-ros-workspace
Description: Turtlebot4 Navigation

Expected behaviour
I want to generate a map of the room I am in using Rviz as per the guide
Actual behaviour
When launching Rviz I can see the Turtlebot4, but the room is not being mapped with the lidar.

These are the errors that are being presented in Rviz:

To Reproduce
Provide the steps to reproduce:

run ros2 launch turtlebot4_navigation slam.launch.py
run ros2 launch turtlebot4_viz view_robot.launch.py
see the error

Other notes
The terminal that I run Rviz in generates this warning, but I do not know if it is relevant:
Warning: Invalid frame ID "" passed to canTransform argument source_frame - in tf2 frame_ids cannot be empty
And the terminal which runs SLAM prints this:
LaserRangeScan contains 1014 range readings, expected 1006
I have tried to run asynchronous SLAM, but that does not change anything. I have also tried to change the Fixed Frame, in Global Options, to odom (which does not change anything) and rplidar_link which removes the error with the LaserScan, but does not generate a map.
Here is the output from ros2 topic list:
/battery_state
/cmd_audio
/cmd_lightring
/cmd_vel
/color/preview/camera_info
/color/preview/image
/diagnostics
/diagnostics_agg
/diagnostics_toplevel_state
/dock
/dock_status
/hazard_detection
/hmi/buttons
/hmi/display
/hmi/display/message
/hmi/led
/imu
/interface_buttons
/ip
/ir_intensity
/ir_opcode
/joint_states
/joy
/joy/set_feedback
/kidnap_status
/mobility_monitor/transition_event
/mouse
/odom
/parameter_events
/robot_description
/robot_state/transition_event
/rosout
/scan
/slip_status
/static_transform/transition_event
/stereo/camera_info
/stereo/depth
/stop_status
/tf
/tf_static
/wheel_status
/wheel_ticks
/wheel_vels
And here is from ros2 node list:
/_internal/composite_hazard
/_internal/kinematics_engine
/_internal/mobility
/_internal/stasis
/analyzers
/joy_linux_node
/mobility_monitor
/motion_control
/rgb_stereo_node
/robot_state
/robot_state_publisher
/rplidar_composition
/static_transform
/system_health
/teleop_twist_joy_node
/turtlebot4_base_node
/turtlebot4_diagnostics
/turtlebot4_node
/ui_mgr

Hi,
I'm having the exact same issue and haven't been able to run SLAM in neither the real nor simulated robot. Just wondering if there's something I'm doing wrong as well.

I have now flushed the Turtlebot 4 so that it is a clean install with humble. All parts of the system are now running ROS 2 Humble, but the issue is still the same. My Ubuntu version is 22.04 and not 20.04 as I stated above so sorry for that mistake. Do you have any other tips that I could try?

Luckily we had access to a Turtlebot 4 lite, so we took the RPLIDAR from the lite version and put it on the standard version. This solved our problem and we can now generate a map. I do not know if we could have solved it another way, but maybe it can be of help to others that the RPLIDAR seems to be the problem in this case.


Can't find any deserts

I have tried creating several worlds lowering moisture and raising temperature but only biome I can find is either a mesa or savannah. The strangest thing that deserts are shown in generation preview but are not there when I fly to their supposed location. Am I doing something wrong or deserts are not implemented yet?

Deserts only occur at low elevations and transition into mesa at higher elevations. This coupled with the requirement for high temperature and low moisture make them quite rare currently. It's being worked on for version 0.1.0


Add gitlens as activity bar item instead of panel in explorer item

More details about this:
https://code.visualstudio.com/updates/v1_23#_extension-authoring

I thought about going down this route (and maybe I will in the future), but my plan is to move GitLens into the SCM activity. I originally thought that I could make it a user choice, but currently that really isn't possible with the provided apis/structures.
Hopefully moving to the SCM view is an acceptable compromise.

I'm in favour of git lens being either integrated into the Source Control view or its own view using the new 1.23 API. Either way it doesn't make much sense to me being in the Explore view.
I think it could look rather neat if it was in the SC view, but I have no idea how difficult this is compared with having its own view.


Minor fix to the README

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
We've detected an issue with your CI configuration that might affect the accuracy of this pull request's coverage report.
To ensure accuracy in future PRs, please see these guidelines.
A quick fix for this PR: rebase it; your next report should be accurate.

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 84.174%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5154

Relevant Lines:
6123

💛  - Coveralls


File pipeline parameters expression

Pipeline parameter expressions are incorrect. They should contain brackets after pipeline i.e. @pipeline().parameters... instead of @pipeline.parameters...

@syedhassaanahmed : Thanks for your contribution! The author, @douglaslMS, has been notified to review your proposed change.

#sign-off


Sg memcpy

Fix issue #3700
https://github.com/shogun-toolbox/shogun/commit/f269f25a5d565bf035e8421eeeeac0b22c4e8dac

Pls dont open and close PRs, creates a lot of junk email. Thanks!


Send retries over longer period of time.

Change times for existing send retries so they are more spread out (last retry is 24hrs).
Notes:

There is NO logic change, just existing the times being updated.
Because the retries are spread out more, the time until a send is considered "failed" is no much longer. Before it was ~4hr (231 minutes), now it's closer to 1d 17hr.

Test Steps:

Deploy to staging and break settings (e.g. bad password) so a sends break.
Use Metabase to watch failures/retries.

Changes

See comments inline on PR.

Checklist
Testing

[ ] Tested locally?
[ ] Ran ./prime test or ./gradlew testSmoke against local Docker ReportStream container?
[ ] (For Changes to /frontend-react/...) Ran npm run lint:write?
[ ] Added tests?

Process

[ ] Are there licensing issues with any new dependencies introduced?
[ ] Includes a summary of what a code reviewer should test/verify?
[ ] Updated the release notes?
[ ] Database changes are submitted as a separate PR?
[ ] DevOps team has been notified if PR requires ops support?

Linked Issues

Fixes #issue

To Be Done
Create GitHub issues to track the work remaining, if any

#issue

Specific Security-related subjects a reviewer should pay specific attention to

Does this PR introduce new endpoints?

new endpoint A
new endpoint B

Does this PR include changes in authentication and/or authorization of existing endpoints?
Does this change introduce new dependencies that need vetting?
Does this change require changes to our infrastructure?
Does logging contain sensitive data?
Does this PR include or remove any sensitive information itself?

If you answered 'yes' to any of the questions above, conduct a detailed Review that addresses at least:

What are the potential security threats and mitigations? Please list the STRIDE threats and how they are mitigated

Spoofing (faking authenticity)

Threat T, which could be achieved by A, is mitigated by M

Tampering (influence or sabotage the integrity of information, data, or system)
Repudiation (the ability to dispute the origin or originator of an action)
Information disclosure (data made available to entities who should not have it)
Denial of service (make a resource unavailable)
Elevation of Privilege (reduce restrictions that apply or gain privileges one should not have)

Have you ensured logging does not contain sensitive data?
Have you received any additional approvals needed for this change?

@jimduff-usds
While testing this I found something odd with retries. It always starts with retry count = 2. Stepping through the code shows something odd with the resendEvent() and I could use another set of eyes. You seem to be the original author.
First issue is that retryItems is assigned a constant and then tested. I just want to make sure this is a simple bug (e.g. it since we reach this point it the code we know from the code above retryItems should be constant.

Tested with my local host with stoping the prime-router_sftp_1 container
'==============================================
2023-03-09T00:41:14.209392317Z [2023-03-09T00:41:14.209Z] FAILED Sftp upload of inputReportId a71cd171-b63d-40d2-b19c-2c58894aeeb5 to SFTPTransportType(host=sftp, port=22, filePath=./upload, credentialName=DEFAULT-SFTP) (orgService = md-phd.elr), Exception: sftp
2023-03-09T00:41:14.209457477Z [2023-03-09T00:41:14.209Z] 00:41:14.208 [pool-2-thread-60][906] WARN  gov.cdc.prime.router.azure.SendFunction - Send Failed.  Will retry sending report: a71cd171-b63d-40d2-b19c-2c58894aeeb5 to md-phd.elr in 10 minutes and -13 seconds at 2023-03-09T00:51:01.208753Z
'======================
2023-03-09T00:51:00.999410784Z [2023-03-09T00:51:00.999Z] FAILED Sftp upload of inputReportId a71cd171-b63d-40d2-b19c-2c58894aeeb5 to SFTPTransportType(host=sftp, port=22, filePath=./upload, credentialName=DEFAULT-SFTP) (orgService = md-phd.elr), Exception: sftp
2023-03-09T00:51:00.999668920Z [2023-03-09T00:51:00.999Z] 00:51:00.999 [pool-2-thread-65][1304] WARN  gov.cdc.prime.router.azure.SendFunction - Send Failed.  Will retry sending report: a71cd171-b63d-40d2-b19c-2c58894aeeb5 to md-phd.elr in 60 minutes and -11 seconds at 2023-03-09T01:50:49.998855Z
'========================
2023-03-09T01:50:51.179896588Z [2023-03-09T01:50:51.179Z] FAILED Sftp upload of inputReportId a71cd171-b63d-40d2-b19c-2c58894aeeb5 to SFTPTransportType(host=sftp, port=22, filePath=./upload, credentialName=DEFAULT-SFTP) (orgService = md-phd.elr), Exception: sftp
2023-03-09T01:50:51.180358893Z [2023-03-09T01:50:51.180Z] 01:50:51.179 [pool-2-thread-73][1575] WARN  gov.cdc.prime.router.azure.SendFunction - Send Failed.  Will retry sending report: a71cd171-b63d-40d2-b19c-2c58894aeeb5 to md-phd.elr in 240 minutes and -282 seconds at 2023-03-09T05:46:09.179394Z
'=========================
2023-03-09T05:46:08.720046748Z [2023-03-09T05:46:08.719Z] FAILED Sftp upload of inputReportId a71cd171-b63d-40d2-b19c-2c58894aeeb5 to SFTPTransportType(host=sftp, port=22, filePath=./upload, credentialName=DEFAULT-SFTP) (orgService = md-phd.elr), Exception: sftp
2023-03-09T05:46:08.720401818Z [2023-03-09T05:46:08.720Z] 05:46:08.719 [pool-2-thread-124][3567] WARN  gov.cdc.prime.router.azure.SendFunction - Send Failed.  Will retry sending report: a71cd171-b63d-40d2-b19c-2c58894aeeb5 to md-phd.elr in 720 minutes and -94 seconds at 2023-03-09T17:44:34.719565Z
'===========================
2023-03-10T17:41:31.787880640Z [2023-03-10T17:41:31.787Z] FAILED Sftp upload of inputReportId a71cd171-b63d-40d2-b19c-2c58894aeeb5 to SFTPTransportType(host=sftp, port=22, filePath=./upload, credentialName=DEFAULT-SFTP) (orgService = md-phd.elr), Exception: sftp
2023-03-10T17:41:31.788115563Z [2023-03-10T17:41:31.787Z] 17:41:31.787 [pool-2-thread-554][21061] FATAL-ALERT gov.cdc.prime.router.azure.SendFunction - FAILED Sftp upload of inputReportId a71cd171-b63d-40d2-b19c-2c58894aeeb5 to SFTPTransportType(host=sftp, port=22, filePath=./upload, credentialName=DEFAULT-SFTP) (orgService = md-phd.elr), Exception: sftp, All retries failed.  Manual Intervention Required.  Send Error report for: a71cd171-b63d-40d2-b19c-2c58894aeeb5 to md-phd.elr

Very minor note:   I think there was one more retry that occurred than shows in the logs above.    Total time is about 41 hours, which looks right.


Member status test failures

OpenStack Release
Mitaka (Was previously passing in Liberty)
Description
test_member_status.py reports two tests failing:
test_online:
assert 'OFFLINE' == 'ONLINE'
test_no_monitor:
assert 'OFFLINE' == 'NO_MONITOR'
Members never go online and appear to always remain offline.

The test failed because Liberty changes were not merged forward to Mitaka. They now have and tests are passing.


Remove dead code

Just something I noticed 🙂

Hello, thank you for contributing :D
This is not dead code. It follows the naming pattern of the LSP binary release packages https://github.com/tekumara/typos-lsp/releases.
The code you removed allowed non-windows users to download the right package for their OS (Linux/MacOS), and also broke the unit tests ( run cargo test).
Removing it will break the plugin for those users, so I will not merge your PR.
I appreciate your will to contribute to this project though :D

Oh wait, no, my bad you are absolutely right.
I confounded the function for downloading and the one for the binary release. Nice catch, thank you for contributing :D

No worries. But it's weird the compiler didn't catch that 🤔


Maybe show selected surfaces+topographies also in analysis detail view

Otherwise there is no reference on the page which surfaces the analyses are related to.

I think this is no longer an issue. The UI has changed quite a bit since this issue was created.


/Games/ is blank if user navigates there - solution in card

There is an off chance a user might visit /Games/ if they manually trim the URL
(It's the kind of thing I might do out of curiosity)
Right now, they get a blank screen, except for the menu, with no error message
On my version, I added the following to the const routes object in main.js
  { path: "/Games/", redirect: "/" },
This seems to solve it without errors. Haven't tested extensively but will keep an eye out
I didn't do a full PR since it's so small, @randylubin

Based on: https://router.vuejs.org/guide/essentials/redirect-and-alias.html

Good idea - I'm redirecting it toward /Gallery/ in mine


fix: enable organization paging

Enable organization paging in InfluxDB. The paging can be enabled independent from UI side
I closed the previous pull-request for org paging: https://github.com/influxdata/influxdb/pull/20985 because it contained code for UI which should now be directed into a different repository: influxdata/ui

[x] CHANGELOG.md updated with a link to the PR (not the Issue)
[x] Well-formatted commit messages
[x] Rebased/mergeable
[x] Tests pass
[ ] http/swagger.yml updated (if modified Go structs or API)
[ ] Feature flagged (if modified API)
[ ] Documentation updated or issue created (provide link to issue/pr)
[x] Signed CLA (if not already signed)

Is this PR still work in progress? I'm very interested in this issue being solved because we want to use influx with more than 20 organizations in one project.


remove lint, add typing, modernize, cleanup

Codecov Report

Merging #104 (899e56f) into main (f313165) will decrease coverage by 0.28%.
The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@             Coverage Diff             @@
##              main     #104      +/-   ##
===========================================
- Coverage   100.00%   99.72%   -0.28%
===========================================
  Files            2        2
  Lines          363      363
===========================================
- Hits           363      362       -1
- Misses           0        1       +1

Impacted Files
Coverage Δ

things/database.py
99.57% <100.00%> (-0.43%)
:arrow_down:

Besides comments LGTM.


Synchronous operation

I'm submitting a ...

[ ] bug report
[x] feature request
[ ] other (Please do not submit support requests here (below))

Sometimes a third party api forces you to run synchronously (this was the reason I could not integrate universal-router into my project). Or you want to just keep it simple or avoid extra unnecessary ticks.
Since consumers may already expect that resolve always returns a promise, this may be implemented by adding a resolveSync method.

@futpib hi, could you please show an example when/how it could be useful?

Or you want to just keep it simple or avoid extra unnecessary ticks.

Btw, do you know that promises are executed in the current event loop (tick) on a microtask queue?

could you please show an example when/how it could be useful?

I find myself writing an application inside a proprietary app container that expects that frontend code (which started as a simple template, which ran synchronously) to return html string synchronously. The code now contains a bunch of if testing the request path and begs for a router.

Btw, do you know that promises are executed in the current event loop (tick) on a microtask queue?

It's a queue anyway, right? Let's call ticks on the microtask queue microticks if this lessens confusion. Do you think microtask queue does not inflict overhead compared to synchronous code?

Ok, I think we're pretty much on the same page.
I don't want to add resolveSync method into the core because it will increase library size with not profit for most of the cases since loading a page usually is asynchronous process. I also believe that any web app could be designed with this in mind and maybe it is better to change design of your app earlier than later.
But I agree that synchronous router could be useful in some certain cases. What do you think about an add-on approach like we already have with url generation? For example something like:
import UniversalRouter from 'universal-router'
import resolveSync from 'universal-router/resolveSync'

const router = new UniversalRouter(routes)

resolveSync(router, pathnameOrContext) // => result

or
import UniversalRouter from 'universal-router'
import withResolveSync from 'universal-router/withResolveSync'

const Router = withResolveSync(UniversalRouter)
const router = new Router(routes)

router.resolveSync(pathnameOrContext) // => result

or
import UniversalRouter from 'universal-router/sync'

const router = new UniversalRouter(routes)

router.resolve(pathnameOrContext) // => result


ONNX conversion for GlowTTS

Has the conversion step from GlowTTS to ONNX been solved? According to a post on the Mozilla TTS repo, it appears to be the case. If so, could the steps taken to make this conversion be provided? Thanks.

I have same request. @sayakpaul

looking forward to hearing from you


Add support for multiple platforms to docker build

Description
The goal of this PR is to add support for ARM processors to our docker images.
Instructions for Reviewers
Verify that the docker images (still) work for your processor type
Checklist

[x] My PR is small in size (e.g. less than 1,000 lines of code, not including comments & integration tests). Exceptions may be made if previously agreed upon.
[x] My PR passes Checkstyle validation based on the Code Style Guide.
[x] My PR includes Javadoc for all new (or modified) public methods and classes. It also includes Javadoc for large or complex private methods.
[x] My PR passes all tests and includes new/updated Unit or Integration Tests based on the Code Testing Guide.
[x] If my PR includes new, third-party dependencies (in any pom.xml), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.
[x] If my PR modifies the REST API, I've linked to the REST Contract page (or open PR) related to this change.

The docker builds seem to have passed. I did have to remove linux/arm/v7 as the maven image we're using doesn't support it, but this PR still adds linux/arm64 which I hope will suffice to use them on M1 macs without emulation


Update virt-handler daemonset when set kubelet-pods-dir in kubevirt cr

What this PR does
Before this PR:
When we set kubelet-pods-dir in kubevirt cr. Virt-handler dosen't update, and the set dosen't realy work.
After this PR:
When we set kubelet-pods-dir in kubevirt cr. Virt-handler daemonset update, and the set realy work.

Fixes #8998
Why we need it and why it was done in this way
The following tradeoffs were made:
The following alternatives were considered:
Links to places where the discussion took place:
Special notes for your reviewer

Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[ ] PR: The PR description is expressive enough and will help future contributors
[ ] Code: Write code that humans can understand and Keep it simple
[ ] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note

NONE

/test all


fix: Adjust the elapsed method to a more suitable shape.

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

Hi @haroldofurtado thanks for your contribution.
We do have an old ticket about using monotonic counters for profiling (https://puppet.atlassian.net/browse/PUP-7520) but I'm not sure if that's similar to the issue you're trying to solve?

Hi @haroldofurtado could you review my last comment?

Hi @haroldofurtado we haven't heard from you and will close this in a few weeks if there's still no reply.

If we already have a solution, this PR can be closed. Thanks for reporting it.


Refactoring BigQuery Read utilities into internal file

This is an intermediate step for #13170 , along with #13154 .
This change is meant to be pure moving of utilities without changing the functionality. Internal symbols are moved to the interna file.
R: @chamikaramj

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

 Lemme know if you need a deeper look. Thanks.


Add QuickTime:ContentCreateDate to photo exporter for --exiftool

Reference #888

Included in v0.55.6](ttps://github.com/RhetTbull/osxphotos/releases/tag/v0.55.6)

Quick question: once I upgrade to 0.55.6 will osxphotos re-export all the videos (MOV, MP4, etc) -- running with options --update and --cleanup -- to update the exported files?
Just want to be prepared if I have to run a huge export/update!

@oPromessa yes, likely as the new exif metadata won't match and that's what osxphotos checks for update


Generate cue def from go struct

We should generate a cue def from its corresponding go struct when the latter exists, to avoid desynchronization between go & cue datamodels.
Today applies to

http proxy
(soon) plugin definition

In the same way, it would be great also to have the typescript types being generated from the backend ones (cuelang and/or go), if achievable

Since https://github.com/perses/perses/pull/1664 we now have the core Golang data model being made automatically available as CUE definitions under cue/model/api/v1. However, the automatic generation provided by CUE has limitations - as explained in the _patch files manually added:
https://github.com/perses/perses/blob/d34492405fabe3da8ccf27612dce73d6d58d60ea/cue/model/api/v1/dashboard_patch.cue#L14-L20
E.g the http proxy model mentioned above would get generated if this limitation was removed, and we could then consider removing the manual http proxy def available at cue/schemas/common/proxy/http.cue.
Anyway, closing this issue since "generate cue def from go struct" is now the case, now waiting the CUE limitations to be sorted out.


add tests for visualize and data methods on explanation

Adding tests for the data() and visualize() methods on our explanations in order to increase code coverage.

Codecov Report

Merging #530 (e9947be) into main (6d1f31b) will increase coverage by 0.17%.
The diff coverage is n/a.

:exclamation: Current head e9947be differs from pull request most recent head 623d5c6. Consider uploading reports for the commit 623d5c6 to get more accurate results

@@            Coverage Diff             @@
##             main     #530      +/-   ##
==========================================
+ Coverage   86.74%   86.91%   +0.17%
==========================================
  Files          62       62
  Lines        4074     4074
==========================================
+ Hits         3534     3541       +7
+ Misses        540      533       -7

Flag
Coverage Δ

unittests
86.91% <ø> (+0.17%)
:arrow_up:

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

Impacted Files
Coverage Δ

...hon/interpret_community/explanation/explanation.py
86.63% <0.00%> (+0.94%)
:arrow_up:

Continue to review full report at Codecov.

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


Samsung M2070 still not working

Describe the bug
@tohl
Even with the following in configuration.nix:
{config,pkgs,...}:{
  imports=[
    <nixpkgs/pkgs/misc/cups/drivers/samsung/1.00.36/module.nix>
  ];
  services.printing.enable=true;
  hardware.sane.enable=true;
  services.samsung-unified-linux-driver_1_00_36.enable=true;
}

scanning still doesn't work and scanimage -L still doesn't give me any scanner
To Reproduce
Steps to reproduce the behavior:

Add the above to configuration.nix
nixos-rebuild switch
scanimage -L

Expected behavior
scanimage -L recognize the Samsung M2070 as scanner
Metadata

system: "x86_64-linux"
host os: Linux 5.2.20, NixOS, 20.03pre197685.f35f0880f2c (Markhor)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.4pre20190922_382aa05
channels(root): "nixos-20.03pre197685.f35f0880f2c"
channels(paki): "nixos-20.03pre196201.07d4df59626"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
# a list of nixos modules affected by the problem
module:
  - services.samsung-unified-linux-driver_1_00_36

@pasqui23
I have M2070W
not sure if you have the right driver

scanning still doesn't work

does printing work?

and scanimage -L still doesn't give me any scanner

and sudo scanimage -L?

pasqui23<EMAIL_ADDRESS>writes:

Printing does work,but sudo scanimage -L still doesn't list I did
succed in scanning by using an Ubuntu docker image and installing the
samsung unified driver v1.00.36 with apt,then issuing manual scanimage

I just successfully scanned a few pages (even without restarting the
scanner).  My scanner is M2070W, not M2070.
$ nixos-version
19.09.1019.c5aabb0d603 (Loris)
$ scanimage -L
[bjnp] create_broadcast_socket: ERROR - bind socket to local address failed - Address already in use
device `smfp:usb;04e8;3469' is a Samsung  on USB Scanner
$ sudo scanimage -L
[bjnp] create_broadcast_socket: ERROR - bind socket to local address failed - Address already in use
device `smfp:usb;04e8;3469;...' is a Samsung M2070 Series on USB Scanner

Maybe it's some other file that you have not yet put in that workaround

How could we check for unwrapped paths in a binary?

pasqui23<EMAIL_ADDRESS>writes:

Maybe it's some other file that you have not yet put in that
workaround

I doubt it.
I installed it on another machine too so it must be in nix store.
How are you connecting the scanner?
With USB?
What does lsusb say?

pasqui23<EMAIL_ADDRESS>writes:

How could we check for unwrapped paths in a binary?

you can use
strings file | grep unwrapped
or
strings file | grep /nix/store

this should be closed
since nixos 19.09 scanner works without any problems

Tested with latest nixos-unstable,works both as printer and as scanner


vendor: containerd/cri 40071878d7392be4193b33b556a69827710b00b1

full diff: https://github.com/containerd/cri/compare/64aa9da76fc0ab333119f455f3b292244c1fae8c...40071878d7392be4193b33b556a69827710b00b1

Is there a reason for bumping this change?

If we don't have a reason, we can just synchronize with the next beta release

Sorry, probably should've described; yes, there's no real "urgency" in this update; my motivation was to get the upstream dependencies  "up to speed" for the 1.4 release. Some of these dependencies started implementing go modules, and (lacking a release for those), consumers of containerd that use go modules will most likely ending up with "master" of those dependencies, so I thought it would be good to match that expectation in this repository as well.
The changes around error handling (using errors.Is() instead of errors.Cause()) are likely good to have, as they prevent possible bugs creeping in if we update a dependency and that dependency has switched to using "native" Go wrapping of errors (masking underlying errors if errors.Cause() is used).

Updating the dependencies is certainly part of the release process. There are still in flight changes in cri that are expected for the next beta, let's synchronize updating cri when we ship the next beta, probably next week.

I also want to make sure that we are updating deps, any deps, because there is a need/requirement for it.  I don't want to have bugs because we updated a dep just because.  We need to be careful about what we bump and when.