Package is vulnerable due to dependency on old version of log-update

This package is currently using<EMAIL_ADDRESS>which is vulnerable - that version has a sub-dependency on<EMAIL_ADDRESS>https://snyk.io/test/npm/ansi-regex/3.0.0

See also https://github.com/advisories/GHSA-93q8-gq69-wqmw
It would be great to get an update here.

updating to log-update ^4.0.0 should be sufficient without code-changes. 5.0.0 is an ESM and would require code changes.

@SamVerschueren this repo doesn't seem to be very active. Would you accept a PR for this? Would certainly be open to make one but don't wan't to do the effort if it's not going to be merged.

@SamVerschueren any updates on this issue? This dependency breaks a few of our repos

@SamVerschueren This issue is definitely a security risk and will not pass Snyk validation. The GraphQL Codegen CLI relies on this package which is quite popular.


Docs: "Go doesn't appear in the drop down for Python"

SCENARIO
Please see this issue in the Go repo: https://github.com/GoogleCloudPlatform/google-cloud-go/issues/455
Seems like we need to refresh the docs for Python perhaps?
Either way, "Go" is missing from the drop-down:

@omaray, this should get updated on the next docs push.


Improve Release pipeline support

Is your feature request related to a problem? Please describe.
Today, if a user tries to add the task to a release pipeline, they get a confusing error message when the task attempts to upload the results artifact. We have an internal FAQ on how to avoid the error, but it's not thrilling that the error message when you try it naively is unactionable.
Describe the desired outcome
Ideally, I think we'd detect that we're running in a release and automatically switch to using ##vso[task.uploadfile] rather than uploading an artifact
Describe alternatives you've considered
If there are issues with doing uploadfile, a smaller improvement would be to automatically detect that we're in a release and early-exit with a more actionable error message (similar to the FAQ, or maybe pointing to the FAQ) explaining what the user needs to configure.
Additional context
n/a

Had another customer hit this today

Unfortunately, this is out of scope given our current priorities.
Thanks for using Accessibility Insights!


Quest requirement correct but cannot load quest correctly

What is your Quests version / build number (do not say "latest")?
Quests-4.0.0-rc.3.jar 1.8.6
Describe the problem and how to replicate it. Any console errors to post using Pastebin.com?
upgrade Quests-4.0.0-rc.3.jar from 178 to 179, it show error Requirement quests has unknown quest name NULL, see quest of ID custom1103  i know this is a bug fix from 179 , however it seems like lead to the following problem.
To test this problem, i removed quests down to only 3 quest.
Quest requirement maybe broke if the quest name is using foreign language, I use Chinese as quest name. I set up a quest with requirement of another quest, but it say the requirement quest quest name is NULL, which does not make sense as the quest is correctly loading, if I remove the quest requirement,  it load normally.
I saw the update of #179 changed will Throw error if requirement quest name is unknown, but the requirement name is correct.
entire server log
https://pastebin.com/AbyC3GLs
bug part
https://pastebin.com/C4R0dCzU
originally, i have 300+ quest but use to demonstrate the bug, i cut the quests until only have 3 in my dev server.

--
If applicable, can you provide an example quest from quests.yml using Pastebin.com?
quest.yml:
https://pastebin.com/uLgF01WX

You have manually edited your quests.yml file therefor it drops errors.

You have manually edited your quests.yml file therefor it drops errors.

yes, but actually no, there is no way for me to remove 300 quest become 3 to test this bug

First of all you should always keep up to date your plugin. Build 186 is current latest one, relating to all bugfixes.
Secondly the issue is caused because you most likely self edited ItemStacks within quests.yml
When using editor ItemStacks never generate & see #1527

It's not a syntax error in this case. It's unsupported characters being used within it.
This is why you should never ever edit your quests.yml file unless you want to fix a typo or any text somewhere.

First of all you should always keep up to date your plugin. Build 186 is current latest one, relating to all bugfixes.
Secondly the issue is caused because you most likely self edited ItemStacks within quests.yml
When using editor ItemStacks never generate & see #1527

i forgot to say, i have tested from 186 to 170 , each version, this problem start from #179 to #186 build
i am already using quest editor to set up requirement

The way Spigot works is pretty basic when using names within YML there are few basic rules.
Your Item name cannot start with special character also it must be within quotes.
try to use ยง5 as color tag on the name and make sure to ' ' quote it.
So it becomes like display-name: 'ยง1 LV 1  頭盔'

The way Spigot works is pretty basic when using names within YML there are few basic rules.
Your Item name cannot start with special character also it must be within quotes.
try to use ยง5 as color tag on the name and make sure to ' ' quote it.
So it becomes like display-name: 'ยง1 LV 1 頭盔'

These items are generated by mythicmobs and is not manually edited by anyone using external method, we used the quest editor to load the item and save it properly, if you say the color tag will become 'ยง1 LV 1 頭盔' , then why not the quest system done it automatically for me? you are telling me to edit the file by hand now but last comment you tell me to use in game editor.

I'm not saying to edit it everytime. This right now is literally just investigating where the exact issue is. This is why I asked you to edit it. So you can test it out and see. Normally quests should auto set up the character but from other hand I'm using b186 myself. So I can't tell it 179 or any older would normally generate proper sign. As everyday there are different fixes. I'm trying to help you investigate if its color issue or Chinese characters issue. It leads me to color code issue because normally those signs are unsupported and cause issues in quests. Also it say quest name is null. Perhaps check on that name in requirements: quests: Chinese name. And remove it and test it out if it works without it but the Itemstack with color codes. If that works then you either have to wait for pika to find solution for Chinese or rename your quests to English

This might be the issue specifically not the Itemstacks at all. Check on this and test it out using English related names only. If it works then it's Chinese characters issues and you gotta wait on pika for a fix

ok, i try to change it to english, thank you for your suggestions, i am sorry that i goes emotional

i start to see why this problem happen, we have been digging a wrong way, english or chinese language is not related.
I suspicious the quest report NULL is due to the secondary quest is before the primary quest.

The above picture show 3 quest.
custom 1101 is primary, linked with 1103 and 1102 with quest requirement 1101.
as custom1103 load, 1101 haven't load and 1103 cannot find 1101.
So it say 1103 quest is null.
If so, quest 179 version maybe changed how the quests load, before can be randomly loaded, but after 179, all quest are loaded in sequence.

Thanks for the report. The chosen language should not make any difference. I would think the order of quests in the file should not matter either.
Since you have (correctly) updated to the latest development build, could you post the new console error? The line numbers will be different since it's a newer version.

yes i think the order of the quest is the problem.
i have moved the custom1103 entirely before custom1102, now it loaded.
However, this can only be done with text editor but not in game.
This would be a big problem if quests are designed modulely in teams, if a quest is designed then add a quest requirement which created after, the quest cannot load.

plz wait, i try to load with #186

same in #186
order of quests:

however, no problem with loading if order is

I appreciate you being so thorough. Yes, it would seem the order does matter for those with a Requirement quest or quest blocks. Not sure if there's an easy way to fix this, but I'll look into it.

i am sorry, i have read your fix which will show this message if this error occurs:

however, in my server i use quest build in item gui to display the quest in order, to make quest in item set up in the following order, i have to more some quest after another quest, which will lead to the above bug

The above highlighed quest will unlock all other quest in this gui requirement.

This gui order in gui is design to have special meaning, if the required quest is moved to front, the meaning will lost

should i open another issue bug or suggestion again 🤔

I opted for a more detailed error message because I couldn't see a way to properly address the issue. Even if a proper fix were presented, there's no guarantee that the load order will remain the same going forward. GUI ordering is really something that should be addressed with QuestsGUI eventually. You posted about this in the discussion thread a while ago; I have not forgotten.

I opted for a more detailed error message because I couldn't see a way to properly address the issue. Even if a proper fix were presented, there's no guarantee that the load order will remain the same going forward. GUI ordering is really something that should be addressed with QuestsGUI eventually. You posted about this in the discussion thread a while ago; I have not forgotten.

does that mean you are leaving me 2 options...

fixed the order in order to update latest version, but gui also need to change
stay # 178, do not upgrade any further

This problem does not occur in # 178 or other version of quest, i saw # 179 build change due to some bugs(?) or not? will it be possible for using another method to fix the bug so that the next patch, the requirement check also any order 🤔🤔


Sync hmpps-template-typescript and update dependencies

FYI: app insights still works in dev so should be all fine


Add version: Cryptomator.Cryptomator version 1.15.17

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[X] Does your manifest conform to the 1.0 schema?

Microsoft Reviewers: Open in CodeFlow

There's a typo in the version number. The correct version should read 1.5.17 and not 1.15.17 .
This leads to winget showing the package as upgradeable, even when the current version has been installed already.


Documents and Templates

As a user, I want to see my documents and templates in the AutoDoc plugin so that I can always manage them there.
ACs:

[ ] there is a tab: Documents that contains all documents
[ ] Documents are displayed in a list containing: Name, type, description and version
[ ] there is a tab: Templates that contains all templates
[ ] Templates are displayed in a list that contains Name and Description

@PhilipLeddin still needed? because with dont have a document overview, only a template overview. This was implemented in sprint 4 #141

@LissiEitel , you are right -> closed

dont have a document overview, only a template overview. This was implemented in sprint 4 https://github.com/sprinteins/oscd-plugins/issues/141


Bug: The per page dropdown does not work

Describe the bug

Steps to reproduce

Navigate to repo list
Select 30 per page
Note there are still 10 per page

Affected services
opensauced.pizza
Platforms
No response
Browsers
No response
Environment
No response
Additional context
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Contributing Docs

[ ] I agree to follow this project's Contribution Docs

This is only an issue in beta.insights.opensauced.pizza. Not sure it's worth looking into.

I'm currently updating that select option on #705. I will look into it and see if it can be handled in one PR

This is only an issue in beta.insights.opensauced.pizza. Not sure it's worth looking into.

After some playing around, if you re-order the list by clicking on Repositories label or Activity or any of those, it will fix itself. I guess this issue is related to #662 and would propagate to production if kept without fixing

This is only an issue in beta.insights.opensauced.pizza. Not sure it's worth looking into.

After some playing around, if you re-order the list by clicking on Repositories label or Activity or any of those, it will fix itself. I guess this issue is related to #662 and would propagate to production if kept without fixing

I did mention it here -> https://github.com/open-sauced/insights/pull/662#issuecomment-1358090062, I'm not sure though if I did change something that makes this happened, but I will try to switch to a previous commit before that PR and investigate on the issue

I can confirm this is bug on production now.

I think it was in beta, but was introduced in main -> https://github.com/open-sauced/insights/pull/723

We did a revert. The sorting and query params were breaking quite a bit. Opening a new issue soon https://github.com/open-sauced/insights/commit/5436712ce203bdb43c5f6fec92751c0d2bf711e2


Add missing require parameter to NgAdminConfiguration

Thanks!

No problem. I've got a couple more drive-by requirejs tweaks coming as well -- I've been integrating ng-admin into my own project and fixing things up as I run into them. Very nice job on the code, it's been quite smooth so far.

Glad your integration was without bumps - we've worked a lot on that.


New to git

is it possible for you to make a youtube vid. I was not able to get it to work :(

got it to run but its not catching any pokemon though? I activated the bot and it did say its activated in cmd window but its not catching the pokemon.

#93


Spelling suggestion on ReadMe.

I am not a bash wizard, and am dying for access to the copilot CLI, and got impactient.

I am not a bash wizard, and am dying for access to the copilot CLI, and got impatient.

Thanks @ongerit!


JIT class: Internal error at resolving type of attribute

With numba v0.23.0, the following code:
from numba import jitclass, double
import numpy as np

spec = [
        ('a', int),
        ('x', double[:]),
        ]

@jitclass(spec)
class Test:
    def __init__(self, a):
        self.a = a
        self.x = np.empty(self.a, dtype = double)

t = Test(5)

gives the following error:
numba.errors.TypingError: Failed at nopython (nopython frontend)
Failed at nopython (nopython frontend)
Internal error at resolving type of attribute "a" of "self":
--%<-----------------------------------------------------------------
Traceback (most recent call last):
  File "/home/david/anaconda3/lib/python3.4/site-packages/numba/typeinfer.py", line 111, in propagate
    constraint(typeinfer)
  File "/home/david/anaconda3/lib/python3.4/site-packages/numba/typeinfer.py", line 356, in __call__
    typeinfer.add_type(self.target, attrty)
  File "/home/david/anaconda3/lib/python3.4/site-packages/numba/typeinfer.py", line 572, in add_type
    unified = tv.add_type(tp)
  File "/home/david/anaconda3/lib/python3.4/site-packages/numba/typeinfer.py", line 33, in add_type
    assert isinstance(tp, types.Type), type(tp)
AssertionError: <class 'type'>
--%<-----------------------------------------------------------------

That's because you must specify a Numba type (such as int32, etc.), not Python int.
We should catch those errors early and output a nice error message, though.

Thanks.

Reopening to make sure we add type checking in the API.


Add Resource Well Pressuirizer

Resource wells don't extract resources and can be accounted for by other machines.


Improve invite rejection

When someone attempts to connect to a share using an invite, we want them to be told if the invite is good or bad. This PR adds a test with the desired behavior and then fixes that test with the fix Herb suggested in slack.
Tests get a little flaky with this fix, so there's probably something else we'll need to do

lgtm
I haven't seen any flakiness with the tests - if you still are, could you open an issue?


David/ogc features fixes

feat: introduce Features Router to support consistent redirection behaviour i.e. "/" redirects to "" on the end of routes. Fix count bug where count is zero.

Have realised this creates a bug where /items on the non Features route doesn't work now.


[BUG] Widget for PiHole not working after PiHole update

Environment
Self-Hosted (Docker)
System
Rasperry Pi , Docker, Portainer v2.16.1
Version
2.1.1
Describe the problem
PiHole widget shows the following:
An error occurred, see the logs for more info. Unable to fetch data Retry
Dashy logs:
 [15:48:01] API Key is required, please see the docs [15:48:01] Unable to fetch data
My config:

 "widgets": [
        {
          "type": "pi-hole-stats",
          "options": {
            "hostname": "http://{pihole ip}"
          },
          "id": "0_577_piholestats"
        },
        {
          "type": "pi-hole-top-queries",
          "options": {
            "hostname": "http://{pihole ip}",
            "apiKey": "{my api key}"
          },
          "id": "1_577_piholetopqueries"
        }

Additional info
Issue was reported by someone else on the PiHole discourse.
https://discourse.pi-hole.net/t/dashy-doesnt-show-all-widget-data-after-pi-hole-update/59995
Response from PiHole support:
Suspect it is related to "More API endpoints now require auth" from the announcement for the most recent updates. Not used Dashy so can't be sure.
Originally published at: Pi-hole FTL v5.20 and Web v5.18 released – Pi-hole As always, please read through the changelog before updating with pihole -up. (A new tag for docker image will arrive in due course.) Pi-hole FTL changes: This release finally ships the remaining bits of the latest dnsmasq release and updates the help text of FTL. We also update the embedded LUA engine and increase the SQLite3 cache size to get some more performance due to relying more on our internal caching comp…
Sample image of what Dashy displays

Please tick the boxes

[X] You have explained the issue clearly, and included all relevant info
[X] You are using a supported version of Dashy
[X] You've checked that this issue hasn't already been raised
[X] You've checked the docs and troubleshooting guide
[X] You agree to the code of conduct

Adding the following to the widgets,
retrieves 'Top Ads Blocked' and 'Top Queries'
          "useProxy": true,
However pi-hole-stats still does not work.

Pi-hole made some changes and now it requires the API key for the stats. It should work if you add the apiKey option. (https://github.com/Lissy93/dashy/blob/master/docs/widgets.md#pi-hole-stats)

Pi-hole made some changes and now it requires the API key for the stats. It should work if you add the apiKey option. (https://github.com/Lissy93/dashy/blob/master/docs/widgets.md#pi-hole-stats)

Thanks @albcp
Yes, that has resolved the issue.
I must have been reading some outdated doco.
Will mark this issue as closed.

Pi-hole made some changes and now it requires the API key for the stats.
It should work if you add the apiKey option.
(https://github.com/Lissy93/dashy/blob/master/docs/widgets.md#pi-hole-stats)
Thanks @albcp for the issue fix.

It appears there's a typo in the documentation.
https://github.com/Lissy93/dashy/blob/master/docs/widgets.md#public-holidays
I got this working by changing the "region" field (as shown in the example)
to "state" as defined in the options.


[BUG] k3d registry create + cluster create --network host --registry use don't seem to work together

What did you do
Following the docs,
k3d registry create...
k3d cluster create --registry-use ...
works well together. While trying to debug a connectivity issue, I added --network host to the cluster create command. This fails to start the cluster due to:
Command '/usr/local/bin/k3d cluster create XYZ --kubeconfig-update-default=0 --image rancher/k3s:v1.25.3-k3s1 --wait --timeout=240s --registry-use ABC --network host' returned non-zero exit status 1.
/usr/local/bin/k3d cluster create XYZ --kubeconfig-update-default=0 --image rancher/k3s:v1.25.3-k3s1 --wait --timeout=240s --registry-use ABC --network host
b"\x1b[36mINFO\x1b[0m[0000] [SimpleConfig] Hostnetwork selected - disabling injection of docker host into the cluster, server load balancer and setting the api port to the k3s default \n\x1b[36mINFO\x1b[0m[0000] [ClusterConfig] Hostnetwork selected - disabling injection of docker host into the cluster, server load balancer and setting the api port to the k3s default \n\x1b[36mINFO\x1b[0m[0000] Prep: Network                                \n\x1b[36mINFO\x1b[0m[0000] Re-using existing network 'host' (1da4a9f896cefba85d1d35694b84122b832079332322eea6d19796d528de2972) \n\x1b[36mINFO\x1b[0m[0000] Created image volume XYZ \n\x1b[36mINFO\x1b[0m[0000] Deleting cluster 'XYZ'           \n"
b"\x1b[33mWARN\x1b[0m[0000] Failed to connect to network 'host': Connection failed \n\x1b[33mWARN\x1b[0m[0000] Error response from daemon: container cannot be disconnected from host network or connected to host network \n\x1b[31mERRO\x1b[0m[0000] Failed Cluster Preparation: Failed to connect registry node 'ABC' to cluster network: Failed to connect to one or more networks \n\x1b[31mERRO\x1b[0m[0000] Failed to create cluster >>> Rolling Back    \n\x1b[31mERRO\x1b[0m[0000] failed to get cluster: No nodes found for given cluster \n\x1b[31mFATA\x1b[0m[0000] Cluster creation FAILED, also FAILED to rollback changes! \n"

How was the cluster created?
/usr/local/bin/k3d cluster create XYZ --kubeconfig-update-default=0 --image rancher/k3s:v1.25.3-k3s1 --wait --timeout=240s --registry-use ABC --network host

What did you do afterwards?

k3d commands?
docker commands?
OS operations (e.g. shutdown/reboot)?

What did you expect to happen
k3d cluster create --network host respects previously created k3d registry create
Screenshots or terminal output
If applicable, add screenshots or terminal output (code block) to help explain your problem.
Which OS & Architecture
ubuntu x64
Which version of k3d
k3d version v5.6.0
k3s version v1.27.4-k3s1 (default)
Which version of docker
Client: Docker Engine - Community
Version:           25.0.3
API version:       1.44
Go version:        go1.21.6
Git commit:        4debf41
Built:             Tue Feb  6 21:13:09 2024
OS/Arch:           linux/amd64
Context:           default
Server: Docker Engine - Community
Engine:
Version:          25.0.3
API version:      1.44 (minimum version 1.24)
Go version:       go1.21.6
Git commit:       f417435
Built:            Tue Feb  6 21:13:09 2024
OS/Arch:          linux/amd64
Experimental:     false
containerd:
Version:          1.6.28
GitCommit:        ae07eda36dd25f8a1b98dfbf587313b99c0190bb
runc:
Version:          1.1.12
GitCommit:        v1.1.12-0-g51d5e94
docker-init:
Version:          0.19.0
GitCommit:        de40ad0
Client: Docker Engine - Community
Version:    25.0.3
Context:    default
Debug Mode: false
Plugins:
buildx: Docker Buildx (Docker Inc.)
Version:  v0.12.1
Path:     /usr/libexec/docker/cli-plugins/docker-buildx
compose: Docker Compose (Docker Inc.)
Version:  v2.24.5
Path:     /usr/libexec/docker/cli-plugins/docker-compose
Server:
Containers: 2
Running: 1
Paused: 0
Stopped: 1
Images: 56
Server Version: 25.0.3
Storage Driver: overlay2
Backing Filesystem: extfs
Supports d_type: true
Using metacopy: false
Native Overlay Diff: true
userxattr: false
Logging Driver: json-file
Cgroup Driver: systemd
Cgroup Version: 2
Plugins:
Volume: local
Network: bridge host ipvlan macvlan null overlay
Log: awslogs fluentd gcplogs gelf journald json-file local splunk syslog
Swarm: inactive
Runtimes: io.containerd.runc.v2 runc
Default Runtime: runc
Init Binary: docker-init
containerd version: ae07eda36dd25f8a1b98dfbf587313b99c0190bb
runc version: v1.1.12-0-g51d5e94
init version: de40ad0
Security Options:
apparmor
seccomp
Profile: builtin
cgroupns
Kernel Version: 6.5.0-25-generic
Operating System: Ubuntu 22.04.4 LTS
OSType: linux
Architecture: x86_64
CPUs: 4
Total Memory: 9.704GiB
Name: xxx
ID: db1d1237-4a37-49d5-a777-e17f0e010bec
Docker Root Dir: /var/lib/docker
Debug Mode: false
Experimental: false
Insecure Registries:
<IP_ADDRESS>/8
Live Restore Enabled: false

I should also add that I found the undocumented --default-network flag for registry create that seems pretty much what I want: https://github.com/k3d-io/k3d/blob/main/cmd/registry/registryCreate.go#L111
but then cluster create on use fails with
b"\x1b[36mINFO\x1b[0m[0000] [SimpleConfig] Hostnetwork selected - disabling injection of docker host into the cluster, server load balancer and setting the api port to the k3s default \n\x1b[36mINFO\x1b[0m[0000] [ClusterConfig] Hostnetwork selected - disabling injection of docker host into the cluster, server load balancer and setting the api port to the k3s default \n\x1b[36mINFO\x1b[0m[0000] Prep: Network                                \n\x1b[36mINFO\x1b[0m[0000] Deleting cluster XYZ           \n"
b'\x1b[31mERRO\x1b[0m[0000] Failed Cluster Preparation: Failed Network Preparation: failed to create cluster network: failed to check for duplicate docker networks: failed to parse IP Prefix of network "host"'s member <>: ParseAddr(""): unable to parse IP \n\x1b[31mERRO\x1b[0m[0000] Failed to create cluster


[BUG] create refine-app EXDEV: cross-device link not permitted

Describe the bug
i have separated partition between root and home directory in my ubuntu 22.
and i got this error when i want to start refine project
Remote source found. Downloading...node:events:491
      throw er; // Unhandled 'error' event
      ^

Error: EXDEV: cross-device link not permitted, rename '/home/path/directory/1670582808831-862229917806892/superplate-core-plugins-master' -> '/tmp/d-2022119-408232-p6iijn.ucncd'
Emitted 'error' event on GithubDownloader instance at:
    at /home/user dir/.npm/_npx/678fa4c4b5e44e85/node_modules/github-download/lib/github-download.js:131:26
    at FSReqCallback.oncomplete (node:fs:192:23) {
  errno: -18,
  code: 'EXDEV',
  syscall: 'rename',
  path: '/home/path/directory/1670582808831-862229917806892/superplate-core-plugins-master',
  dest: '/tmp/d-2022119-408232-p6iijn.ucncd'
}

what should i do?
thanks
Steps To Reproduce

separated home directory and root
run npm create refine-app my-app

Expected behavior
refine installed successfully
Screenshot
No response
Desktop

OS ubuntu 22
node v16.18.0
npm 9.2.0

Mobile
No response
Additional Context
No response

I am a completely new user and just followed the quickstart steps, i got the exact same error! I will include a screenshot .

Hey @asule90 @zubSero ,
Thank you for contacting us! Could you try with the download argument?
npm create refine-app my-project -- --download git

awesome!
yes that solved the problem..
thanks..


Provide documentation on resource types, input and access

Enhance documentation on the various access and input types for resources. We have examples at various places but lack an overview and a systematic description:

What is the difference between access and input types and when to use what
Typical use case for each method/type
Most important: Provide an example (ideally both on CLI level and programmatically with Go)
Custom types

Even after working with this for weeks each time I need to create something I copy/paste a fragment from before and start from there
To me it seems this would help not only to get users up-to-speed but this is also in other means supportive:

As a basis for integration tests (which should cover as many scenarios as possible)
Maybe also in the VSCode-Plugin in the OCM creation wizard by providing a context-sensitive link from the input mask to the web site

Added https://github.com/open-component-model/ocm-website/pull/85

So we can close this one, let's briefly discuss in our review meeting later.


SearchFilter - support multiple values for collection-valued association

Can you add some tests please?

I also had to switch to league/url to avoid problems with http_build_query

Can you make league/url optional? Move it in the suggest section of composer.json and throw a RuntimeException if the package is not installed at the line it is necessary?
Can you also update the doc to explain how to use this filter?

Nice improvement! Just some last issues to fix.

So this support is for search on properties of a collection but stop at the first level right?

@theofidry Only the first level (the resource's own relations), yes. I might need to expand the work on this soon when the need for nested search arises in my project, or look for an alternative (AFAICT, your LoopBackApiBundle still doesn't support that yet, right?).

Not yet I'm currently working on that but it will be handled at DunglasApiBundle level. Actually there is not much there difficulty is about easing the declaration to the user and securing parameters placeholders and aliases. I will push a first draft shortly.

@dunglas I've removed the dependency on league/url (for now...)

Thank you @teohhanhui


Dump test details

The failing test is passing locally in my VM:

The failing test is passing locally in my VM:

yeah but the test is flacky, we need to understand why


[pack] Add crosswalk-lite support for android building.

Please hold this PR for merge, I'm investigating if it's a feature bug for app-tools

Please verify with an older Crosswalk-lite lib, there's a regression issue with latest one.

Verified, LGTM


How to maintain the datetime format same as input

First check

[ ] I added a very descriptive title to this issue.
[ ] I used the GitHub search to find a similar issue and didn't find it.
[ ] I searched the FastAPI documentation, with the integrated search.
[ ] I already searched in Google "How to X in FastAPI" and didn't find any information.
[ ] I already read and followed all the tutorial in the docs and didn't find an answer.
[ ] I already checked if it is not related to FastAPI but to Pydantic.
[ ] I already checked if it is not related to FastAPI but to Swagger UI.
[ ] I already checked if it is not related to FastAPI but to ReDoc.
[ ] After submitting this, I commit to one of:

Read open issues with questions until I find 2 issues where I can help someone and add a comment to help there.
I already hit the "watch" button in this repository to receive notifications and I commit to help at least 2 people that ask questions in the future.
Implement a Pull Request for a confirmed bug.

I have created a model as below.
class Inventory(BaseModel):
locationId: str = Field(..., example="LOC44242", description="Unique Store/Location identifier.")
inventoryDate: datetime = Field(..., example="2019-04-01T00:00:00.000Z", description="ISO 8601 format")
I am sending inventoryDate in 2019-04-01T00:00:00.000Z format I want FastAPI to maintain this format but it is changing it to 2019-05-01T11:53:21.988000+00:00
is there a way yo maintain the format?
Example
Here's a self-contained, minimal, reproducible, example with my use case:

from fastapi import FastAPI

app = FastAPI()

@app.get("/")
def read_root():
    return {"Hello": "World"}

Description

Open the browser and call the endpoint /.
It returns a JSON with {"Hello": "World"}.
But I expected it to return {"Hello": "Sara"}.

Environment

OS: [e.g. Linux / Windows / macOS]:
FastAPI Version [e.g. 0.3.0]:

To know the FastAPI version use:
python -c "import fastapi; print(fastapi.__version__)"

Python version:

To know the Python version use:
python --version

Additional context

Actually The converting is executed by Pydantic, not by FastAPI.

@waynerv Thanks for correcting. But is there any way to achieve what I am looking for? Either in FastAPI or Pydantic?

My description above is wrong, in fact the conversion from datetime to string is done by datetime.isoformat() which is a standard library function.
Unfortunately it will handle UTC time as +00:00 instead of Z, you may have to manually use str.replace to achieve the desired effect.

Here is a simple example using jsonable_encoder of FastAPI:
from datetime import datetime

from fastapi.encoders import jsonable_encoder
from pydantic import BaseModel, Field

class Inventory(BaseModel):
    locationId: str = Field(..., example="LOC44242", description="Unique Store/Location identifier.")
    inventoryDate: datetime = Field(..., example="2019-04-01T00:00:00.000Z", description="ISO 8601 format")

a = Inventory(locationId='LOC44242', inventoryDate='2019-04-01T11:53:21.980Z')

print(jsonable_encoder(a.dict(), custom_encoder={datetime: lambda dt: dt.isoformat().replace("+00:00", "Z")}))

Output:
{'locationId': 'LOC44242', 'inventoryDate': '2019-04-01T11:53:21.980000Z'}

Thanks for the help here everyone! 👏  🙇
This will also probably change in Pydantic v2, that the date parsing/validation/handling is done in a more robust way in Rust. :nerd_face:
If you solved the original problem, then you can close this issue @yantrikskaran  ✔

Sorry for the long delay! 🙈  I wanted to personally address each issue/PR and they piled up through time, but now I'm checking each one in order.


#169 Parse target list from text file

Fixes #169
Fixes #83
Not unit testing the wrapper for stdlib, coverage will slightly decrease.

Pull Request Test Coverage Report for Build 578

16 of 18 (88.89%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.5%) to 96.066%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

loaders.go
16
18
88.89%

Totals

Change from base Build 572:
-0.5%

Covered Lines:
293

Relevant Lines:
305

💛  - Coveralls


generate docker file for RabbitMQ to use it with AMQP

generate a docker file to host RabbitMQ to run applications using AMQP.

Maybe you want to have a look at Docker Compose.

actually thats what I'm doing. sorry for the misleading title :)
version: '2'
services:
  rabbitmq:
    image: rabbitmq:3.6.2-management
    ports:
      - 5672:5672
      - 15672:15672

we can close this as the code is merged in develop branch


Improve README

I will add Diff tags like lines of code, repo size, forks ,stars ,last commit ,total contributor, issues, PRs ,license and add Repo banner
Please assign me under SWOC'21
@aakankshabhende @shivshankar9

Hey @aakankshabhende please add label

Hey @aakankshabhende @shivshankar9
Please add labels


set temp folder as target folder or an option to set it

currently, the temp folder (that is the temporary location where the video, audio and subtitle file download before going to output location) is defaulted to gui location.
it would be great to have it as the output or target location/folder, or have an option in the settings to set my user.

I have not been able to implement this implementation in a short period of time, for the following reasons

With the current working conditions, I can't find a better reason to add this option.
For the foreseeable future features, I will use the temporary requirements, and I will not implement this implementation until the whole process is conceived

Sorry, I hope you can understand, this requirement should not be added in the recent update

Hello, thank you for using my GUI.
Sorry for the wait.
I have released a new version which includes the ability to customize the temp folder.
If you are interested, please try the new version.


Run CI against rails/master

This adds Rails master to CI as an allowed failure. While builds will
continue to pass regardless of this run, it makes it easier to keep an
eye on if changes need to be made for the upcoming version of Rails, as
well as automatically test shims to add compatibility

Ironically, this works even though https://github.com/airblade/paper_trail/blob/6411ab1ce6f661075092d7597d32e4c1f9eae7ff/paper_trail.gemspec#L27 is present because 5.1.0.alpha and any other prerelease version is still considered to be < 5.1

I think that https://github.com/airblade/paper_trail/pull/898 might conflict with this. Let's merge this first so we don't have to ask Sean to rebase again and fix a conflict. :)

👍


Can we donate money?

Is there a way to donate money? For years YouTubeCenter has been my #1 most important extension along with ad blockers and every day I am thankful it exists. Now the new Youtube design has finally motivated me to find out if we can donate. Just as a way to say thank you. I don't need any promise that it'll speed things up.

Yeah, simply open the settings and there's a donate tab to the left.

That's correct. I don't accept any donations because I'm not actively working on YouTube Center and I wouldn't feel right if I did. Anyways, if you want to donate you should donate to a charity instead as your money then will go to a good cause instead of into my pockets.


OCPBUGS-26114,OCPBUGS-26081,OCPBUGS-26145: add snyk config file for SAST scan

Per the documented policy from ART for the SAST scans, and following the pattern established in the openshift/oc repo, we are introducing a snyk config file that excludes the vendor tree, as that is currently some minor warnings.

/jira refresh

/assign @coreydaley
/assign @otaviof

/lgtm

/cherrypick release-4.15


Update example so that it outputs all books before finishing

Previously it would exit on line 40 after outputting the first line

Thanks!


Add default googleapi route creation to net-vpc

All good just maybe change the var name? These are not default routes (in the networking sense).

One more point, a bit more controversial. I see today we don't default to true delete_default_routes_on_create. We could consider also adding a TF managed "default" route in addition to these in the module (create_special_routes), or at least do that just for FAST.

One more point, a bit more controversial. I see today we don't default to true delete_default_routes_on_create. We could consider also adding a TF managed "default" route in addition to these in the module (create_special_routes), or at least do that just for FAST.

Good point. Can you open an issue for this?

All good just maybe change the var name? These are not default routes (in the networking sense).

You right, "default routes" is a terrible name. Any suggestions?


[UI] Canont click on "K3s" hyperlink in cluster import description

master-2424-head
What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):

Launch Rancher UI, click on the Add Cluster button
Click on the "K3s" hyperlink in the description for the cluster import (See image below)

Result:

If you click on the "K3s" hyperlink, a new tab will not be opened bringing you to https://k3s.io, instead, you will be navigated to the import cluster page. We should either fix this or just not provide a hyperlink here.

Minor, nice-to-have.

The bug fix is verified with Rancher server rancher\rancher:master-2425-head version 881b3e40d
k3s hyperlink can now be clicked and a new tab will be opened to https://k3s.io


[mimir.rules.kubernetes] - Alert rules path naming

What's wrong?
Can't find a possibility to change alert rules path name, as example for something like -  $cluster:$namespace:$alertRuleName.
Current naming is - agent/$namespace/$alertRuleName/$id
Steps to reproduce
Regular alert rule creation by grafana-agent.
System information
No response
Software version
0.37.0
Configuration
mimir.rules.kubernetes "prometheus_rules" {
  address = "endpoint"
  sync_interval = "5m"
}

Logs
No response

Would love that, I think by default it is pretty long and the id doesn't make it really user friendly when navigating the UI and searching for a specific alert namespace. Allowing to customise it for Loki as well would be a great enhancement !
I'm guessing this would be done in this file (? https://github.com/grafana/alloy/blob/main/internal/component/loki/rules/kubernetes/events.go#L201


BigInteger should have a less strictly typed constructor

BigInteger from System.Numerics could be "serialized" and "deserialized" to/from a byte array.
However, currently, it only has a constructor that takes byte[]. Since the constructor (according to the reference source) actually only uses .Length property and indexed access, it would have been enough to require IList<byte> as an argument.
First, asking for an interface instead of a specific type seems more in line with the ideology of the language. Second, there are situations when this would have been more convenient.
As an example, we are implementing some public key cryptosystems and use big integers internally. Many PK cryptosystems produce ciphertexts that are actually a pair of bigintegers, which are stored as a single byte array; e.g., in a byte[96] we have two big integers, 48 bytes long each. Reconstructing these would require an extra step of copying first and second halves of the array into individual arrays — an expensive operation, and it adds up when we are working with large data sets. Instead, if there was an IList<byte> constructor, we could have used ArraySegment<byte> and skip an unnecessary array copying.

Noted, thanks, will do a proposal in the corefx repo as well.

Agreed. This repo isn't the right one for this. Closing issue.

As an example, we are implementing some public key cryptosystems and use big integers internally.

Unless your product is a full crypto library, a thousand to one you shouldn't be doing this.

(Everyone thinks they are the exception, but to be honest...)

It's like speeding or not wearing a seat belt. Everyone is sure they can get away with it but the risk vs benefit just doesn't make any sense if you look at the consequences.

Our product is a crypto library.

That's awesome, I'm glad to hear it!


Model performance degrades when moved to Multi-GPU

Hi,
When I run your code on multi-gpu, performance degrades severely (compared to the single-gpu version).  To make the code multi-gpu competable, I've only added 2 lines of code:

model = nn.torch.DataParallel(model) between your model = model_class.from_pretrained(args['model_name'])  and model.to(device) calls

loss = loss.mean() after the loss = outputs[0] line in the train function. Do you have any idea how can I get the same (or similar) performance on Multi-GPU setting?

These are the results I got with these two settings:

With Multi-GPU training:
evaluate_loss: = 0.3928874781464829
fn = 116
fp = 81
mcc = 0.5114751200090137
tn = 1291
tp = 136

With Single-GPU Training:
evaluate_loss: = 0.39542119007776766
fn = 82
fp = 126
mcc = 0.5465463104769824
tn = 1246
tp = 170

Although avg loss values are similar, there are big differences in other metrics.

Those changes should be sufficient to enable multi-gpu training in my experience. Is there any other difference (e.g. batch size) between the two runs?

Nope, I did not change any of the variables in args dictionary.

This is probably a silly question, but did you try this multiple times and receive the same results?

Yes, I run the code with the same configuration multiples times. There is no difference across different runs.

Sorry, I am not sure why this is happening. I recommend that you try the Simple Transformers library as it supports multi-gpu training by default and I have used multi-gpu training with that library without any performance degradation.


Risc-v Cross compile error

hi, i'm try make for riscv,but find something error  :(
make HOSTCC=gcc TARGET=RISCV64_GENERIC CC=riscv64-unknown-linux-gnu-gcc FC=riscv64-unknown-linux-gnu-gfortran
cc1: error: '-march=native': ISA string must begin with rv32 or rv64
cc1: error: requested ABI requires '-march' to subsume the 'D' extension
cc1: error: ABI requires '-march=rv64'
cc1: error: '-march=native': ISA string must begin with rv32 or rv64
cc1: error: requested ABI requires '-march' to subsume the 'D' extension
cc1: error: ABI requires '-march=rv64'
make: *** [Makefile.prebuild:70: getarch] Error 1
Makefile.system:1351: Makefile.: No such file or directory
make: *** No rule to make target 'Makefile.'.  Stop.
riscv64-unknown-linux for rv64g

You can comment out one line in Makefile.system as a workaround.
# On x86_64 build getarch with march=native unless the compiler is PGI. This is required to detect AVX512 support in getarch.
ifeq ($(HOSTARCH), x86_64)
ifeq ($(findstring pgcc,$(HOSTCC))$(findstring nvc,$(HOSTCC)),)
// GETARCH_FLAGS += -march=native
endif
endif

@brada4 that line is only there for the host compiler, used in building the getarch tool. The problem must be elsewhere.

What is your build host, is it x86_64 or something else ? And which release of OpenBLAS are you trying to build here ? (I assume either 0.3.15 or the develop branch - do not use the master branch as that is no longer updated)

What is your build host, is it x86_64 or something else ? And which release of OpenBLAS are you trying to build here ? (I assume either 0.3.15 or the develop branch - do not use the master branch as that is no longer updated)

build host for x86_64 Ubuntu20.04  ,relrese for 0.3.15 or the develop branch

Which cross-compiler do you use on Ubuntu ? (I am not getting the error on openSuSE, but the cross-riscv64-elf toolchain there appears to be broken or incomplete, missing headers later in the build)

Which cross-compiler do you use on Ubuntu ? (I am not getting the error on openSuSE, but the cross-riscv64-elf toolchain there appears to be broken or incomplete, missing headers later in the build)

i use riscv64-unknown-linux-gnu -arch=rv64g

Thanks for the additional information. I still get different errors with a build from source (errors related to the toolchain missing some headers), but I guess it is safe to say that your gcc (as used in HOSTCC) probably does not point to the host (x86_64) compiler at the moment but to the toolchain compiler instead.

Try HOSTCC=/usr/bin/gcc (or clang , or gcc-10 or something like that)

Right, that was what I was trying to say (and my build went fine once I noticed I had omitted the linux option on the make command to build the crosscompiler - a bit exhausted today)

I'm sorry to trouble you all， thanks everyone。
i try to use HOSTCC=/usr/bin/gc  but it was the same

so i change to use CBLAS lib to Cross compile
it‘s work！
But I still can't error this mistake
Thank  again

The error message suggests that /usr/bin/gcc on your system is a riscv compiler as well, maybe something went wrong with the installation of riscv64-unknown-linux-gnu-gcc and it put a link /usr/bin/gcc -> riscv64-unknown-linux-gnu-gcc ? /usr/bin/gcc should normally be a compiler for your host system (x86_64), and the OpenBLAS Makefile needs to build a few tools on the host system to configure the build even when cross-compiling.

The error message suggests that /usr/bin/gcc on your system is a riscv compiler as well, maybe something went wrong with the installation of riscv64-unknown-linux-gnu-gcc and it put a link /usr/bin/gcc -> riscv64-unknown-linux-gnu-gcc ? /usr/bin/gcc should normally be a compiler for your host system (x86_64), and the OpenBLAS Makefile needs to build a few tools on the host system to configure the build even when cross-compiling.

yes，gcc->riscv64-unknown-linux-gnu-gcc
change gcc for x86_x64 will different

No idea where the -march=tigerlake and -mtune=tigerlake is coming from - this is definitely not set by the OpenBLAS build, all we do is set -march=native for building the getarch tool. Perhaps you have set CFLAGS in your environment ?
(-march=tigerlake is supported by gcc version 10 and newer)

No idea where the -march=tigerlake and -mtune=tigerlake is coming from - this is definitely not set by the OpenBLAS build, all we do is set -march=native for building the getarch tool. Perhaps you have set CFLAGS in your environment ?
(-march=tigerlake is supported by gcc version 10 and newer)

your are right！
for echo "" | gcc -v -x c++ -E -

COLLECT_GCC_OPTIONS='-v' '-E' '-mtune=generic' '-march=x86-64'
So how do I set it up?can i use HOSTCC=gcc -march=..

You'd need to add quotes around the argument, like HOSTCC="gcc -march=native" - but I think it would be easier to find out where the -march=tigerlake is getting set. Does echo $CFLAGS  show it ?

You'd need to add quotes around the argument, like HOSTCC="gcc -march=native" - but I think it would be easier to find out where the -march=tigerlake is getting set. Does echo $CFLAGS show it ?

no，nothing。i can‘t find

I made it！
thanks @martin-frbg suggest。
My gcc-9 doesn't support - March = tiger lake
but I can't find this option
This option is not supported until gcc-10, so I apt install gcc-10
And using gcc-10 make, it is successful
：）

Glad to know you got it working 🙂


fix(client-presence): restore initial state values

Corrects regression from #23120

can there be a test added for this? regression tests are some of the best tests, as things that break tend to be non-obvious so frequently break again

can there be a test added for this? regression tests are some of the best tests, as things that break tend to be non-obvious so frequently break again

There are tests pending which is how we caught this. We wanted typescript to detect this problem for us :/


Did @riotjs/register stop exporting register ?

Hi!
Thank you for your hard work in responding to the major version upgrade!
Well, I have recently tried v9 myself.
I noticed that if I try to import register from @riotjs/register in node.js, it will throw an error as follows.
> node node_scripts/html.js

file:///Users/genki/Sites/Getup/node_scripts/html.js:6
import register from '@riotjs/register'
         ^^^^^^^^
SyntaxError: The requested module '@riotjs/register' does not provide an export named 'register'
    at ModuleJob._instantiate (node:internal/modules/esm/module_job:124:21)
    at async ModuleJob.run (node:internal/modules/esm/module_job:190:5)

Node.js v18.17.1
ERROR: "html-from-riot" exited with 1.
ERROR: "html" exited with 1.

and here is the file I ran.
https://github.com/nibushibu/Getup/blob/d79644794d0f6869de8f39b90ff5a39ca3c632ed/node_scripts/html.js#L6
I am continuing to work on solving my problem, but I would appreciate any tips you can give me.

Hi @nibushibu in v9 I have dropped the support for commonjs projects so you will not be able to import Riot components in that way. If you want to migrate your project using modern nodejs syntax with esm modules you will need to call your scripts registering the riot loader. For example in your package.json:
{
  "type": "module",
  "scripts": {
    "start": "node --loader @riotjs/register app.js"
  }
}

If you want to keep using commonjs syntax the older @riotjs/register version should just be fine and you don't need to update it.

@GianlucaGuarini Thank you for your reply and advice!  Finally I was able to solve the problem at hand ✨


Add neural search plugin to 2.4.0 manifest linux

Description
Adds neural search plugin to 2.4.0 manifest. Adds support for linux platform. Skip windows platform for now because k-NN dependency does not support windows.
neural search needs to be added after ml-commons and k-NN because it takes these as dependencies.
Issues Resolved
#2736
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

manifests / manifest-checks-jdk17 (manifests/3.0.0/opensearch-3.0.0.yml) (pull_request) Failing after 11m

Failure should be unrelated given we are updating 2.4, not 3.0

@jmazanec15 that issue is fixed in CU now could you pull again from main?
Thanks.


feat: Profile has assets on other chains

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[x] Feature

Needs Design check

@exezbcz please review

Context

[x] Closes #8742

Did your issue had any of the "$" label on it?

[x] My DOT address: Payout

Screenshot 📸

[x] My fix has changed UI

It has nft/collection on any other chains.

It have no collection on any other chains.

Copilot Summary
copilot:summary
copilot:poem

merry xmas - the (4) there is indicator of how much networks the user has art on

also the cta should be to a chain where the user has any art

it will be displayed whenever there are no assets on the current chain, but are on others

then, if it is on more chains than one (for example ahp and RMRK2) it shows a redirect based on the hierarchy mentioned - in this case ahp

merry xmas - the (4) there is indicator of how much networks the user has art on

also the cta should be to a chain where the user has any art

it will be displayed whenever there are no assets on the current chain, but are on others

then, if it is on more chains than one (for example ahp and RMRK2) it shows a redirect based on the hierarchy mentioned - in this case ahp

✅ Done. Could you please check again? @exezbcz

can you please use this message for the empty state

when a user does not have anything on the chain not even on others

otherwise, it's working nicely! thanks

can you please use this message for the empty state

when a user does not have anything on the chain not even on others

otherwise, it's working nicely! thanks

✅ Yes, I have updated it.

suggestions:
perhaps add skeleton and then show one of the two messages?
https://github.com/kodadot/nft-gallery/assets/36627808/56a0858a-0c8f-4e56-85d4-325887bab311
provide better chain naming instead of AHP or KSM, but only if easy to implement

suggestions:
perhaps add skeleton and then show one of the two messages?
skel.mp4
provide better chain naming instead of AHP or KSM, but only if easy to implement
edit: functionality working nicely tho

✅DONE
I have added a loading skeleton and updated the chain name for it.

Thanks!
pay 50 usd

😍 Perfect, I’ve sent the payout
💵 $50 @ 8.64 USD/DOT ~ 5.787 $DOT
🧗 16SjUbGKSdjCdWTy3NNT3JxbRVGGqD4mwkHpc6BD9U2Rp29Z
🔗 0xe5c965e471b951ab566018e0bf4f7ca3666f1936b89fe482aaa30fad6424d9c4
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


Remove hdf5_hl hack

This resolves LiDO codevelop from failing. This hack is no longer needed since we've updated MFEM.
https://lc.llnl.gov/gitlab/topopt/lido-2.0/-/merge_requests/589#note_435971

@tepperly , if you approve this we can get it moved over to your code.

Codecov Report

Merging #1029 (3a312d5) into develop (fbf391f) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #1029   +/-   ##
========================================
  Coverage    89.25%   89.26%
========================================
  Files          144      144
  Lines        11311    11311
========================================
+ Hits         10096    10097    +1
+ Misses        1215     1214    -1

see 1 file with indirect coverage changes
:mega: Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!


Default port changed

Changed according to https://github.com/prometheus/prometheus/wiki/Default-port-allocations


Bring back mistakenly removed Loaded_Chart()

This JS function is still being used by ziya charts.
Mistakenly removed in commit bffad7221f68cac34ee8a9b154640395d46a3534

Do we still use ziya charts on upstream or is this downstream only?  If it's downstream only we should leave it out (and only put it in the productization code base).

@mzazrivec @romanblanco Can you write a test so we know how this is used and protect us in the future?

@Fryguy @mzazrivec @jrafanie I'm trying to search, where is this function used, but I can't find any place

@romanblanco It's called by ziya charts, so you'd have to look in the ziya gem.

Do we still use ziya charts on upstream or is this downstream only?

Don't think this was answered.  @mzazrivec @romanblanco @h-kataria ?

@Fryguy ziya supports the flash charts which are only used downstream.  I'm not sure how much effort we should put into moving this (unless it's trivial) as we are looking to replace all charts with C3.

I added a short spec testing presence of the problematic JS function.

Regarding the move to the productization repo -- not really worth the effort,
seeing that the ziya charts are going to be gone soon.
Though I need this function back in master now, since it blocks another work (PR) of mine.

I :heart: removing ziya.  It's old, not maintained, and monkey patches rails.


Multiple cameras

Hello!
Tell me please. How can I connect two cameras and publish them to different topics?

Hi Ashley
Certainly.
Connect one camera and run ros2 run pylon_instant_camera node. A line like this should appear in the log:
Using device [daA1600-60uc] with full name [2676:ba03:2:2:4], user defined name [] and serial number [123456789].

Repeat with the other camera.
You can use either the full name, the user defined name or the serial number to select a particular camera. I just added the serial number option because I found it to be more reliable than the full name (which in case of USB cameras includes topology information which can change).
You may also want to remap the namespace so the output topic will not clash:
ros2 run pylon_instant_camera node --ros-args --param serial_number:="12345678" -r __ns:=/front_camera
ros2 run pylon_instant_camera node --ros-args --param serial_number:="23456789" -r __ns:=/back_camera

Good luck with your project!

Hi Ashley
Certainly.
Connect one camera and run ros2 run pylon_instant_camera node. A line like this should appear in the log:
Using device [daA1600-60uc] with full name [2676:ba03:2:2:4], user defined name [] and serial number [123456789].

Repeat with the other camera.
You can use either the full name, the user defined name or the serial number to select a particular camera. I just added the serial number option because I found it to be more reliable than the full name (which in case of USB cameras includes topology information which can change).
You may also want to remap the namespace so the output topic will not clash:
ros2 run pylon_instant_camera node --ros-args --param serial_number:="12345678" -r __ns:=/front_camera
ros2 run pylon_instant_camera node --ros-args --param serial_number:="23456789" -r __ns:=/back_camera

Good luck with your project!

Hello!
I will check it today and write an answer.
I'm afraid if i run two instances of the program, the performance will decrease.
Thanks.

Hi Ashley
Certainly.
Connect one camera and run ros2 run pylon_instant_camera node. A line like this should appear in the log:
Using device [daA1600-60uc] with full name [2676:ba03:2:2:4], user defined name [] and serial number [123456789].

Repeat with the other camera.
You can use either the full name, the user defined name or the serial number to select a particular camera. I just added the serial number option because I found it to be more reliable than the full name (which in case of USB cameras includes topology information which can change).
You may also want to remap the namespace so the output topic will not clash:
ros2 run pylon_instant_camera node --ros-args --param serial_number:="12345678" -r __ns:=/front_camera
ros2 run pylon_instant_camera node --ros-args --param serial_number:="23456789" -r __ns:=/back_camera

Good luck with your project!

Sorry for the long answer. I checked, it works.
I want to ask, how can I reduce the load on the CPU? If two cameras are started, there is a high load on the CPU and "buffer error" errors appear

@fhwedel-hoe
By the way. I want to ask you something else. The camera's FPS is lower than in PylonViewer, why is that?

I checked, it works.

Nice to hear that.

how can I reduce the load on the CPU? If two cameras are started, there is a high load on the CPU and "buffer error" errors appear

I reduced the CPU overhead as much as I could. As part of it's communications design, ROS2 will always create a copy of the data which is published.  Keep in mind that I cannot know for sure what causes the CPU load in your particular set-up.
There has been some extensive discussion here at https://github.com/ros-perception/image_common/issues/216. Bottom line is: If you want speed, do not use ROS2 for image transport. Have one ROS node which connects to one camera. Do the image analysis immediately within that node. Publish only the relevant data (small amounts of data e.g. position of detected markers). Of course, you will not be able to view the image via ROS2 tools. :(

The camera's FPS is lower than in PylonViewer, why is that?

Unfortunately, I have no idea. By default, this node just connects to the camera and starts grabbing. The settings used in PylonViewer are kept. However, I have been told that some Basler cameras revert to their default settings when power cycled. You can save the PylonViewer settings into a file and then supply that file to this node by using the parameter camera_settings_pfs (it expects a path to the pfs file).

@fhwedel-hoe hi, Thanks for answer.
Last question. Let me ask you something. Why did you decide to write your own frame capture code instead of using the available official driver?

The official node uses the camera in software trigger mode. The node has the authority. That means:

Request frame 1
Capture frame 1
Transfer frame 1
Process frame 1
Repeat from 1 for frame 2

This driver can use the "overlap" mode (you need to configure your camera to use that in PylonViewer) available in dart cameras (and possibly other models, too). Che camera has the authority.With this, the procedure is:

Capture frame 1
Transfer frame 1, capture frame 2 at the same time
Process frame 1, transfer frame 2, capture frame 3 at the same time
Repeat 3

Since the camera can transfer one frame while capturing the next one, the effective frame-rate is doubled.


Added Japanese locale

Added Japanese locale

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


nixos/systemd-shutdown: use prebuilt shutdown ramfs derivation

Description of changes
I changed the shutdown ramfs to be built alongside the system config rather than be built on-the-fly at shutdown. See discussion starting at https://github.com/NixOS/nixpkgs/pull/185085#issuecomment-1368503199 for details on motivation, but basically there's not much reason to only do this at shutdown (even if it's quick to run), dracut does not do this, and for cross systems it requires building a cross rustc toolchain for make-initrd-ng even if no other Rust programs exist on the target image just to run the one script.
This change technically could be breaking, but only if the user manually made a unit that dependend on generate-shutdown-ramfs.service, which doesn't seem like a use-case that would give someone any benefit anyway with the options available in the module. I can change the service name back to generate if someone feels otherwise that this will be a real problem.
I opted not to unpack the booted initrd image for shutdown because unpacking it is a hard problem, and that would ignore the options already available in this module (and are already used at least by the ZFS module and some people's configs).
It would be nice if we could try to use the same systemd version and other software versions that we booted with (just to ensure no issues), but that would involve modifying system top-level build and loading from something like /run/booted-system/shutdownramfs, which would on the whole be more complicated than the potential benefit I feel it would give.
cc: @ElvishJerricco @oxalica
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

@ofborg build nixosTests.systemd-shutdown

Will isn't in the systemd team, so I've requested his review directly, for posterity if nothing else :)

@lilyinstarlight I'm sure it's small, but could you comment the difference in system closure size caused by this?
If we care about the shutdown ramfs matching the initrd, we could unpack it at boot instead of shutdown, and just leave it in there all boot long. This of course consumes extra system memory though. Tradeoffs, tradeoffs :)

@lilyinstarlight I'm sure it's small, but could you comment the difference in system closure size caused by this?

Yep, will do!

If we care about the shutdown ramfs matching the initrd, we could unpack it at boot instead of shutdown, and just leave it in there all boot long. This of course consumes extra system memory though. Tradeoffs, tradeoffs :)

I also thought about that but idk how much I trust /run/initramfs to continue existing from bootup to shutdown on every person's system every time...

Okay I just checked and the closure size difference for nixosTests.systemd-shutdown.config.nodes.machine.system.build.toplevel for before and after 03dcc0d79 is 598.25 MiB -> 619.32 MiB
Is that acceptable @ElvishJerricco?

That's about what I expected. We should also probably confirm that this correctly fixes the cross compiling situation before we merge.

So, funny thing about this... It does remove the need to cross compile make-initrd-ng. But a cross-targeting rustc is still in the build graph because of #214153

So, funny thing about this... It does remove the need to cross compile make-initrd-ng. But a cross-targeting rustc is still in the build graph because of #214153

Ah, neat (guess I waited too long to actually open this PR...). Well I'll probably rethink this change then if that's no longer a motivation
What we really should be doing anyway is pivoting back to booted initrd
Do you think we could keep the unpacked version of the initrd somewhere or at least the contents file reachable from toplevel? Or is there a reliable way to unpack an arbitrary /run/booted-system/initrd?

We could always copy the files out of the initrd during initrd so that we don't have to think about any kind of archive format stuff. Of course that relies on those files remaining intact all along until shutdown. Could mount the tmpfs at /run/initramfs with ro though I guess. Not sure... I'd like to know how other distros do this part, if at all.

We could always copy the files out of the initrd during initrd so that we don't have to think about any kind of archive format stuff. Of course that relies on those files remaining intact all along until shutdown. Could mount the tmpfs at /run/initramfs with ro though I guess. Not sure...

Yeah I just worry that it may disappear while a user is mucking with mounts or something, but perhaps if they are mucking with that it's their own fault if shutdown can't pivot to back to initrd... (it's not like that's fatal, systemd just will silently not use it if /run/initramfs/shutdown does not exist)

~I'd like to know how other distros do this part, if at all.~ Ah, you pointed out in the other PR that they extract the initrd from the initrd manually. Looks like they do this at shutdown via ExecStop on an otherwise dummy oneshot service that's started at boot with RemainAfterExit=yes

Yes and dracut does so via trying multiple different directories for the initrd associated with the running kernel version, using their own skipcpio executable (which I may extract to a separate output or even derivation from the dracut derivation in nixpkgs), and then trying different compression programs until one works
Which seems fragile and we can probably do a lot better than that given we'll have /run/booted-system/initrd guaranteed available and can theoretically know the compressor used (although we probably can't support an arbitrary compressor and also for u-boot systems that initrd link is only to the u-boot image and the original initramfs file is not in the output and idk whether that matters or not because I've never mucked with u-boot)

So I'll look more at this soon and may just implement a sketchy /run/initramfs dir/mount during bootup that it gets copied to (tbh I wonder if you can have the original initramfs / remounted at /run/initramfs during the pivot and prevent the kernel from freeing it :thinking:)

tbh I wonder if you can have the original initramfs / remounted at /run/initramfs during the pivot and prevent the kernel from freeing it thinking

This is an interesting thought. I don't think systemd will let you do that though; it literally wipes the old root if it can (I've read this is because the initrd's root fs can't actually be unmounted for some reason, but I have no idea where that idea comes from or if it's true)

I'll go ahead and close this and open a new PR soon, since we'll try to do something more "correct" for this

So, funny thing about this... It does remove the need to cross compile make-initrd-ng. But a cross-targeting rustc is still in the build graph because of #214153

That's unfortunate. Maybe it's time to make cross-targeting rustc more fast to compile instead, if it's required anyway? Only the std and friends need recompilation actually.

we'll try to do something more "correct" for this

I don't like the idea of keeping shutdown-ramfs in memory all the time, even after cherry picking some of them from original initrd (~22M systemd-shutdown closure for me, larger if zfs is also inside). Memories are much expensive than even high end SSDs. Users, especially with no swap enabled, don't want a permanent memory usage during the whole uptime just for "a more graceful shutdown".
I'm not sure if there a better way to do this now. :cry:

That's unfortunate. Maybe it's time to make cross-targeting rustc more fast to compile instead, if it's required anyway? Only the std and friends need recompilation actually.

Yea rustc definitely has the chops to use the same build for a lot of targets, so it would be great if we could separate stuff like std into separate target-specific derivations. AFAIK though, that's basically unsupported by the build system. So the actual binary can do it just fine, but the tooling is not so friendly.

I'm pretty sure (and/or hopeful) more rust will end up in low-level system services, so having to have a rustc in the build closure is probably not too bad after all :-)
Agreed that we should probably see how we can get this to be a bit more enjoyable, and maybe have it in the binary caches for the most part.

I don't like the idea of keeping shutdown-ramfs in memory all the time, even after cherry picking some of them from original initrd (~22M systemd-shutdown closure for me, larger if zfs is also inside). Memories are much expensive than even high end SSDs. Users, especially with no swap enabled, don't want a permanent memory usage during the whole uptime just for "a more graceful shutdown".

Noted. Thinking about it more, maybe just doing best-effort /run/booted-system/initrd decompression isn't that bad of an idea, since failure to create /run/initramfs is non-fatal
Status-quo isn't the most terrible too, since I mean it is effectively equivalent to what mkinitcpio does (mkinitcpio just sketches me out a lot in general, so I'm pretty apprehensive about replicating any decisions it makes...)

I think that what we currently have is probably better than a "best effort" sort of thing.

Noted. Thinking about it more, maybe just doing best-effort /run/booted-system/initrd decompression isn't that bad of an idea.

I've tried it once but soon gave it up. Despite we can append arbitrary binary content before initrd via boot.initrd.prepend (mostly microcodes), we also allows custom compression command boot.initrd.compressor{,Args}. A quick search on GitHub shows there are many complex usages of these arguments, like custom compressor and options. It's hard to calculate the inverse of the given command, and it may require extra packages.

I'm pretty sure (and/or hopeful) more rust will end up in low-level system services, so having to have a rustc in the build closure is probably not too bad after all :-)

I think that what we currently have is probably better than a "best effort" sort of thing.

Yeah, especially now Linux also has some Rust code. I agree that the current solution is good enough. We should probably just keep it. The shutdown ramfs can always be disabled if necessary via systemd.shutdownRamfs.enable = lib.mkForce false;.


In cluster mode do i have to give different urls to different clients for load balancing ?

Environment

OS:
CentOs
Erlang/OTP:
EMQ:
2.3.6

I now wanted to use emqttd broker in cluster mode. From the docs i could understand that i can run different nodes on different machines and then join those nodes to form a cluster.
But if i am using 2 different machines for different nodes, how do clients know about which node to connect to ? Do i have to tell different clients, different urls.. ? how can i implement automatic distribution of load between nodes using only one endpoint, because in my case there are thousands of devices and i cannot update mqttServer Url for the devices frequently.
And does this mean that if a client who was connected to one of the node and if the node goes down the client wont connect automatically to the other node ? until this one node comes up again.. ?

You should setup a reverse proxy before EMQ nodes such as nginx or haproxy.

Ohk, Thank you @terry-xiaoyu.


Can't sppourt all standard library?

I can run the example given.But when i try my file, iwasm occurs some errors!
My code:
#include <stdio.h>
#include <string.h>
#include <ctime>
int fibR(int n)
{
    if (n < 2) return n;
    return (fibR(n-2) + fibR(n-1));
}
int main()
{
    clock_t t = clock();
    char str[50];
    int len;
    strcpy(str, "This is runoob.com");
    len = strlen(str);
    t = clock() - t;
    printf ("%f\n",((float)t)/CLOCKS_PER_SEC);
    return 0;
}

error
Load import section failed: resolve import function (env, _clock) failed.

What confused me is that the functions of <stdio.h> and <string.h> can be called correctly, but the functions of <ctime.h> are not.
I have noticed that /core/iwasm/products/linux/main.c contains code like that:
#ifndef _GNU_SOURCE
#define _GNU_SOURCE
#endif
#include <stdlib.h>
#include <string.h>
#include "bh_platform.h"
#include "wasm_assert.h"
#include "wasm_log.h"
#include "wasm_platform_log.h"
#include "wasm_thread.h"
#include "wasm_export.h"
#include "wasm_memory.h"
#include "bh_memory.h"

The main.c only contains <stdlib.h> and <string.h> ,so i add #include <time.h>.I reran the make and cmake command .But iwasm still occurs the same error!
Is there a way to allow iwasm to support other standard library files?

Currently wamr supports a subset of libc API's, including printf/memcpy/strcpy/malloc/free and so on, please refer to "Built-in application library" in README. And developer may extend the libc API's, you can add the clock wrapper function in file: core/iwasm/lib/native/libc/libc_wrapper.c, registering the wrapper API to array native_func_defs, please refer to the implementation of other wrapper functions, such as strcpy, memcpy, strlen. Or you can also add the wrapper function to file ext_lib_export.c.
In the future, we may enable WASI to support libc, and in that mode, developer needn't implement the wrapper function of libc API. But the size of the wasm binary file will increase, since part of the libc source code is compiled into the wasm file.


Raid Toolkit does not run with new PlariumPlay

Hi there,
RaidToolkit looks for gamestorage.gsfn, which was located in 'C:\Users...\AppData\Local\Plarium\PlariumPlay\gamestorage.gsfn'. The new installation of PlariumPlay v8 stores that file in 'C:\Users...\AppData\Local\PlariumPlay\gamestorage.gsfn' instead. this results in a "System.IO.DirectoryNotFoundException: Could not find a part of the path ".
The installation from scratch of both, RaidToolkit and PlariumPlay, did not help.
Best Regards
Sachmet-Netjer

With the actual version RaidToolkit, v<IP_ADDRESS>52, the problem seems to be solved.
In this context I have an additional question, that bothers me for quite some time.
After installing the version above the tooltip tells me I run v<IP_ADDRESS>617. Where does that mismatch come from? In former versions aswell it happened that for several releases the tooltip did not change.
Best Regards
Sachmet-Netjer

After installing the version above the tooltip tells me I run v<IP_ADDRESS>617. Where does that mismatch come from? In former versions aswell it happened that for several releases the tooltip did not change.

I only recently figured this one out, it's been driving me nuts.  The mismatch is caused by a change in the Nerdbank.GitVersioning dependency that was causing a different version number to be emitted for the release name than was being stamped into the assembly. This is fixed in 2.0, but I haven't ported back to 1.10.x yet.

Latest <IP_ADDRESS>75 release should have matching version numbers


#2220 use parsed_hosts_file_ptr only if it has valid value

https://github.com/apache/trafficserver/issues/2220
When ATS is configured to use a hosts file (via proxy.config.hostdb.host_file.path) and fails to open it, there's a crash:
#0 0x00000000006e4ccc in std::_Rb_tree<ts::ConstBuffer, std::pair<ts::ConstBuffer const, IpAddr>, std::_Select1st<std::pair<ts::ConstBuffer const, IpAddr> >, CmpConstBuffferCaseInsensitive, std::allocator<std::pair<ts::ConstBuffer const, IpAddr> > >::_M_begin (this=0x10) at /usr/include/c++/4.9/bits/stl_tree.h:518
#1 0x00000000006e4f05 in std::_Rb_tree<ts::ConstBuffer, std::pair<ts::ConstBuffer const, IpAddr>, std::_Select1st<std::pair<ts::ConstBuffer const, IpAddr> >, CmpConstBuffferCaseInsensitive, std::allocator<std::pair<ts::ConstBuffer const, IpAddr> > >::find (this=0x10, __k=...) at /usr/include/c++/4.9/bits/stl_tree.h:1913
#2 0x00000000006e48a9 in std::map<ts::ConstBuffer, IpAddr, CmpConstBuffferCaseInsensitive, std::allocator<std::pair<ts::ConstBuffer const, IpAddr> > >::find (this=0x10, __x=...)
at /usr/include/c++/4.9/bits/stl_map.h:860
#3 0x00000000006dd237 in HostDBContinuation::do_dns (this=0x7f8e1a9d65e0) at HostDB.cc:1993
#4 0x00000000006dcd91 in HostDBContinuation::probeEvent (this=0x7f8e1a9d65e0, e=0x7f8df0252b00) at HostDB.cc:1932
#5 0x000000000050ce28 in Continuation::handleEvent (this=0x7f8e1a9d65e0, event=2, data=0x7f8df0252b00) at ../iocore/eventsystem/I_Continuation.h:153
#6 0x00000000007b7d11 in EThread::process_event (this=0x7f8e16c0a010, e=0x7f8df0252b00, calling_code=2) at UnixEThread.cc:148
#7 0x00000000007b8120 in EThread::execute (this=0x7f8e16c0a010) at UnixEThread.cc:230
#8 0x00000000007b72bb in spawn_thread_internal (a=0x1612870) at Thread.cc:86
#9 0x00007f8e193eb6ba in start_thread (arg=0x7f8e161ff700) at pthread_create.c:333
#10 0x00007f8e1867582d in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:109
Found on ATS 6.2.1 but looks applicable to current master.

[approve ci].

Please run "make clang-format" and push the updated changes (squash the commits as well, so you will have to force push).

I've done a forced push after running "make clang-format"

[approve ci].

Cherry picked to 7.0.x


Plugin broken: Google Analytics Link Tagging

Sorry for the crossposting, i know this is que is not meant for plugins, but i still dare to post here, delete if i'm unappropriate.
The plugin Google Analytics Link Tagging seems broken since YOURLS 1.7 (or earlyer?)
https://github.com/katzwebservices/google-analytics-for-yourls/issues/6
AND a bit abandoned.
So i thought it's maybe ok to try attract some attention from 'outside'.
This plugin seems quite handy for people who use YOURLS for cross-media tracking. It allows to easily add tags to your url like

http://www.my-long-url.com/?utm_source=MySource&utm_medium=Banner&utm_campaign=MyCampaign

So, maybe someone familiar with plugins could quite quickly spot the problem?
My own coding skills would take me days to get into it.

Did you check the other plugins? There are a few others about Google Analytics. I pinged the plugin author to be notified if we should remove this plugin from the "official" list

Thanks. No, i did not try the others, but from their description they clearly are no replacement.
They allow to add the analytics js code to track yourls-usage.
Whereas Google Analytics Link Tagging adds some input fields to facilitate adding url-parameters like utm_source=somevalue to the long URL's


AppStateAsserts: Ensure WebView loaded before verifying test #625

Fixes #625

v1
@Zhiyuan-Amos submitted v1 for review.

[v1 1/2] AppStateAsserts: Resolve browser loading race condition
[v1 2/2] Remove WebViewLoadedEvent

(:books: Archive)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/626/1/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you wish to fetch this PR to.


Automatic Video Conversion - Audio Passthrough not working

Hi,
I configured a profile for the automatic video conversion and for the audio part configured passthrough option as follows:

Auto passthrough settings. If no match, falls back to the 2)
E-AC3 encoder settings in case there is no match for auto passthrough

End result is that all audios are encoded with the E-AC3 configured there. If no encoder is configured in part 2), audio is ignored and missing in the final file.
My question is how to configure profile for the automatic video conversion to passthrough all audio tracks present in the original file.
Thanks

Also interested

In 2), the first encoder should be Auto Passthru to handle what you selected in 1).

You are right. I made it work at the end with this settings:

Here want to confirm, if there is no match of selected auto passthru, audio will get encoded with whatever is chosen in Fallback and Encoder settings from Auto Passthru are selected (as displayed above)?
Another question, for what purpose is e.g. E-AC3 Passthru ? From documentation is not clear to me..
Thanks

Here want to confirm, if there is no match of selected auto passthru, audio will get encoded with whatever is chosen in Fallback and Encoder settings from Auto Passthru are selected (as displayed above)?

I think that settings related to the "Auto Passthru" encoder are not used.  If you want to configure passthru the fallback encoder, you need to add a second encoder that matches the fallback.  Again, I'm not 100% sure of this, I guess the best is to try it :)

Another question, for what purpose is e.g. E-AC3 Passthru ? From documentation is not clear to me.

Not sure to understand your question?  E-AC3 is another audio format that HandBrake can copy directly to the output video.

I also find this part confusing :) You may have better luck to seek help from the HandBrake forum directly.

You sure? I think the automated conversion through folder scan is your magic doesn't it?

I was talking about the behaviour of audio settings of a preset.
Once you configure (from the UI) the preset the way you want, the automatic video converter using this same preset should behave the same way as when encoding using the UI.


Add a possibility to run/serve the site via Docker

Description
This PR provides an easy way to setup/serve the site locally using a light way (Docker) instead of dealing with the dependency hell while installing all the dependencies for using Jekyll.
Issues Resolved
n/a
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[X] New functionality has been documented.

[ ] New functionality has javadoc added

[X] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

(I'm not a Ruby person, excuse my ignorance)
When I run docker-compose up -d, the Gemfile.lock gets modified and adds x86_64-linux-musl. I wouldn't expect running docker-compose to modify a file.
In this case, I'm guessing it is just adding a platform dependency so nbd, right? @dblock, you're a Ruby person - care to weigh in?

(I'm not a Ruby person, excuse my ignorance)
When I run docker-compose up -d, the Gemfile.lock gets modified and adds x86_64-linux-musl. I wouldn't expect running docker-compose to modify a file.
In this case, I'm guessing it is just adding a platform dependency so nbd, right? @dblock, you're a Ruby person - care to weigh in?

When you run bundle install, bundler downloads and installs all the dependencies from rubygems and creates Gemfile.lock if it doesn't exist. Commit the file if you want to lock the dependencies.
When you run docker-compose up -d it just starts a container. What we want is that the bundle install runs inside that container in its own file system, which it's currently not doing, so it just is "adding docker on top of an already working jekyll serve", which seems like providing a bunch of overhead (docker) to run what otherwise would work today (jekyll serve) which doesn't install anything.
Hope I explained it clearly enough? AMA! :)

This LGTM - @dblock Any issues before I merge?

Thanks for the contribution, @GezimSejdiu!


Feat/add privatelink docs

https://konghq.atlassian.net/browse/CSRE-2806
Description

Add docs page for privatelink.

Testing instructions
Preview link:
Checklist

[x] Review label added
[ ] Conditional version tags added, if applicable.

Hi @healthy-pod, I've moved the section and made some structure/wording changes, can you take a look?
My understanding from the product brief is that it will eventually be possible to create private connections to other cloud providers, so it made sense to me to create a section for that and put the privatelink docs under it. Let me know what you think. Thanks!

@lmilan Left some comments! I like where you put the doc in the nav and I think making that new Private connections section makes sense to me. I changed the HTML to markdown as well in my suggestions.

Thanks for your comments! When I started reviewing I also changed the HTML to markdown, but then I realized that it caused issues with the navtabs inside a step, so I put the HTML back 😄 do you know if there a better way to do this?

the problem with the navtabs is probably the indentation.
if they are inside a list, they should be indented

@fabianrbz Could you help fix the failing smoke test? I updated the branch since I thought it might be an issue with the Kong Mesh 2.9 nav file, but that didn't fix it and I couldn't figure out what went wrong.

@cloudjumpercat we need to merge this one first. Or drop the submodule update from this one

I accidentally merged this early so I reverted it and created a new PR #8149


Fix Horizontal Scrolling of Result Views

The Issue
Unlike vertical scrolling, horizontal scrolling inside a ResultView scrolled both the ResultView and the Atom Editor Pane.
The expected behavior is to stop Atom Editor Pane scrolling while scrolling inside a ResultView.
This behavior was already implemented for vertical scrolling, but had not been implemented for horizontal scrolling.
This PR fixes that issue.
Notes

For reproduction, see #1644
Upon merge, closes #1644.

This PR is ready for review

Nice catch @wadethestealth !


[WIP] Add plot_trace_dens

This is my attempt to reproduce the legacy plot_trace.

we have ecdf too!

And also a lot of mistakes.

I added the function scale_fig_size (needs more works). Is this the approach we are going to use for the new ArviZ?
I use matplotlib keywords to get/set default values, which will fail for bokeh or other backends.
compact=true is not doing the right thing (I guess school is stacked ), see below:

I added some logic to set the size of ticks, remove ticks and add x/y labels I am not sure I am doing the right thing, in particular for setting the size of ticks and remove ticks.

I did not add any code for bokeh...

Docstring and even variable names may need to be cleaned...

Should we call this plot_trace_dist instead of plot_dens?

To reduce redundancy and make the plots more compact, we may want to have a title per row with the variable names and spanning both columns.  Not sure how easy is that

📚 Documentation preview 📚: https://arviz-plots--27.org.readthedocs.build/en/27/

Codecov Report
Attention: 102 lines in your changes are missing coverage. Please review.

Comparison is base (e1a36c5) 76.22% compared to head (a6bc8af) 65.60%.

:exclamation: Current head a6bc8af differs from pull request most recent head 1602acf. Consider uploading reports for the commit 1602acf to get more accurate results

Files
Patch %
Lines

src/arviz_plots/plots/tracedensplot.py
10.29%
61 Missing :warning:

src/arviz_plots/plots/utils.py
8.69%
21 Missing :warning:

src/arviz_plots/backend/matplotlib/__init__.py
25.00%
12 Missing :warning:

src/arviz_plots/visuals/__init__.py
33.33%
8 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##             main      #27       +/-   ##
===========================================
- Coverage   76.22%   65.60%   -10.62%
===========================================
  Files          10       11        +1
  Lines         572      692      +120
===========================================
+ Hits          436      454       +18
- Misses        136      238      +102

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Should we call this plot_trace_dist instead of plot_dens

me gusta más con dist, si

Merging this. We can keep working on separate PRs, I will open issues.


Changing file mode for secret volumes - required for ssh keys.

Trying to mount a private ssh key to a container using secret volume.
However the secret file is always mounted with mode 664. Which will cause SSH to reject the key with following error:
$ ssh -o BatchMode=yes -i /volumesecrets/key some.internal.host command

@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@
@         WARNING: UNPROTECTED PRIVATE KEY FILE!          @
@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@
Permissions 0664 for '/volumesecrets/key' are too open.
It is required that your private key files are NOT accessible by others.
This private key will be ignored.
Load key "/volumesecrets/key": bad permissions

Trying to change the mode from within the container does not work since the volume mount is readonly:
$ chmod 0400 /volumesecrets/key
chmod: changing permissions of '/volumesecrets/key': Read-only file system

Is there any way we can configure the chmod of the files mounted?

Considering this repo does not appear to be tracked anymore by the ACI team, I wonder what would be the proper method for relaying this request to the team?
One work-around is to use a regular Azure File Share, configure write access and set the container to change the file permissions of the ssh key at startup. The ability to configure the file permission for folders/files mounted from an Azure File Share, would be another useful feature in this case (now they are always 777, from my understanding).
Any other work arounds?


Move OnConditionalAwsCloudEnvironment to autoconfigure module

I was thinking on moving AwsCloudEnvironmentCheckUtils too.
@maciejwalkowiak WDYT?

Merged in ec784c4b8ae12a6282633f185b5187420e9ac4b2


[BUG] Context menu active color effect

Hi Richard it's me again. I like to change themes often to resist visual fatigue. Frost Glass offers corresponding blurred UI under both light themes and dark themes and that's excellent.
Here's what it looks like when I use a dark theme:

And when I use Ra Spring light theme:

It has the correct effect. Strange things happen when I use the Brackets Light Pro light theme:

The active menu item has the white color on the text, making it hard to recognize characters in such a light theme. I guess, since I don't know the underlying mechanism of Frost Glass, that's probably because something went wrong when detecting it's currently light or dark theme.
Exaust all possible color themes is apparently impossible, so I'll appreciate it if you add a option in the settings to change the active color or something. I really like Brackets Light Pro and a workaround is enough for me.

I am working on it

You can try adding the following code to your settings.json
"workbench.colorCustomizations": {
    "[Your theme name]": {
        "menu.selectionForeground": "#000000"
    },
}

In the current version the effect color is the same as selectionForeground.
In the next release I will change it to be the same as menu.selectionBackground.
So you can customize the color with workbench.colorCustomizations.
Many themes use colors other than black or white for menu.selectionBackground. However in my experience, only black or white is good for the light effect. Thus you will need to try different combinations by yourself.

It works! I can now use my favorite light theme and favorite blur effect ☺

I have released a new version (0.7.2).
After the update, menu.selectionForeground will control the text color when selected, menu.selectionBackground will control the light effects color. And there is still frosted-glass-theme.revealEffect.opacity that will control the transparency of light effect.  You will need to find suitable combinations for each theme.


Use string interpolation

Used string interpolation
Used var instead of actual type when it's clear from the right side
Modernizing sample

@rpetrusha , I updated the samples you mentioned.
I also noticed that I get some different results when I run the code, can you check those commits ? (I still didn't check the parallel C# for those, I'll wait your review first)

Thanks, @rpetrusha :)
I've committed it. What about the commits with message Fixed wrong output, should I apply same changes to the C# sample?

Thanks for the additional changes, @Youssef1313. I'll merge your PR now.


Starting application with "electron ." instead of "npm start"

Hi,
Is there any way to force starting electron for testing with "electron ."? Should there be a choice how app is started?
The problem I face under Windows OS is related to recompiled SQLite. I cannot start my electron app with npm start but fortunately i can with "electron .". It generates another issue related to testing which I wanted to add using Spectron.
Thank you for help in advance.

GitHub issues are for feature requests and bug reports, questions about using Electron should be directed to the community or to the Slack Channel.
FWIW typically npm start maps to electron . in your package.json.


Parse.Query.or ignores query on relation field

Hello,
I'm trying to run a compound query where in I want to match a pointer to a field which is of type relation. Here's my request:
GET /parse/classes/Activity { 'user-agent': 'node-XMLHttpRequest, Parse/js1.7.0 (NodeJS 4.2.6)',
  accept: '*/*',
  'content-type': 'text/plain',
  host: 'localhost:1337',
  'content-length': '422',
  connection: 'close' } {
  "where": {
    "$or": [
      {
        "toUsers": {
          "__type": "Pointer",
          "className": "_User",
          "objectId": "NSME8tT0tf"
        }
      },
      {
        "toUser": {
          "__type": "Pointer",
          "className": "_User",
          "objectId": "NSME8tT0tf"
        }
      }
    ]
  },
  "limit": 100,
  "order": "-updatedAt"
}

In the above request toUsers is a column of type relation while toUser is of type pointer. The results simply ignore the 'toUsers' query and returns results matching pointer column. Running these two queries separately works.
Here's the cloud code of the above request:
Parse.Cloud.define("GetActivity", function(request, response) {

  var Activity = Parse.Object.extend("Activity");
  var User = Parse.Object.extend("User");

  var user = request.user;

  var queryPings = new Parse.Query(Activity);
  queryPings.equalTo("toUsers", user);

  var queryFollows = new Parse.Query(Activity);
  queryFollows.equalTo("toUser", user);

  var query = Parse.Query.or(queryPings, queryFollows);
  query.descending("updatedAt");
  query.limit(100);

  query.find().then(function(activities) {
    response.success(activities);
    }, function(error) {
      response.error(error);
  });

});

Just to try out whether compound queries were working or not, I tested it with values on normal string based columns and that worked fine. It seems the problem only occurs on compound queries involving relation type column. I'd also like to add that this code works well with the parse hosted server.

Have similar issue here.

+1 Same issue here

Looks related to #344..

This should be fixed with #769, please test against that version and let me know if that's still occuring

@flovilmart just tried and at least in this particular case it seems to be working well! Will test more in depth later and re-open if I find an issue. Thanks man and as always great job 👍


Remove utils.value module

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (024c862) to head (9f7fbdc).

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #23   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            8         7    -1
  Lines          117       106   -11
  Branches        15        15
=========================================
- Hits           117       106   -11

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Unwatch deleted file & renamed file

If you are in the navigation actions, renaming or deleting a file will not unwatch it and will also trigger change events.
We want to avoid this by unwatching the path first.
Also, for the case of renaming, we want to then watch the new path.

Resolved by https://github.com/chipsenkbeil/distant.nvim/commit/2024af2bd84b29e12a39891daac3edc7f8cd737d.