Filter out images at ImageRepository level

In a given ImagePolicy object you may want to select a subset of the images, e.g., those marked as destined for a particular environment dev-*. You might also want to filter in an ImageRepository which images are available to policies; for example, if there are images that are known to be faulty, or cause problems for scanning.
I don't think this needs a big up front design, but it might pay to think (then write) about

the "little language" for describing what to exclude or include
how it fits in with filtering in image policies.

Adding my comment from #58 here:

An extra thought on the policy level prefix matching feature. I think it would be best if we implement this after we integrate a "real" database into the controller so that we can use its capabilities to filter out the tags dataset.

I've been thinking some more about this and it might not be a good idea to have this capability at the level of the repository, but rather more generic (as in not specific to a particular policy type) at the level of an image policy resource. The way I see things is that users can probably have a single repository resource with multiple policies responsible for different stages of a product's release lifecycle, some policies would match against a particular set of tags, some another (e.g. rc-<timestamp> for staging deployments and rel-<timestamp> for production).

Could one use this to approximate the glob: tag matching functionality of fluxv1?
For example, our CI system generates docker images and tags them with the pattern <branch name>-<build number>-<git commit sha> e.g. a build on develop might look like develop-199-dae1ee8 and the next one develop-200-7aeb34c. We've been using this convention with fluxv1 and it works well with the annotation flux.weave.works/tag.foo-service: glob:develop-* in our deployments. A different branch might produce an image tag awesomeFeature-201-2bbd164 and this tag is ignored by flux since it doesn't match the glob.
It seems like fluxv2 is pushing for semver-compliant tags. I was initially thinking of just using something like 1.0.<build number>+<branch name>-<git commit sha> but then this breaks our workflow since (using the example above) 1.0.201+awesomeFeature-2bbd163 gets deployed to our cluster, when we want the latest image created from the develop branch.
Perhaps I'm missing something obvious, and am open to suggestions for how to achieve our desired outcome: for a given deployment, deploy the latest image tag generated from a given branch. Perhaps what I'm looking for is #15?
Also, am I right in deducing that annotations on k8s deployments are going away in fluxv2 i.e. there's no way within a deployment manifest file (and only a deployment manifest file) to define whether/how flux will perform automated deployments when a new docker image tag is pushed?

Thanks for your suggestions, @TonyLovesDevOps. Flux2 actually does have support for an Alphabetical ordering policy that you can rely on, however, it does not yet support tag globbing (or prefix matching).
A potential example of how you can define such a policy with your use case would be:
apiVersion: image.toolkit.fluxcd.io/v1alpha1
kind: ImagePolicy
metadata:
  name: podinfo
  namespace: flux-system
spec:
  imageRepositoryRef:
    name: podinfo
  policy:
    alphabetical:
      order: asc
    # note that this part is not yet implemented
    matchTags:
      prefix: develop-

Also, am I right in deducing that annotations on k8s deployments are going away in fluxv2 i.e. there's no way within a deployment manifest file (and only a deployment manifest file and not an ImagePolicy) to define whether/how flux will perform automated deployments when a new docker image tag is pushed?

Yes, this is correct. The automation logic is currently implemented in a dedicated controller and is independent of the workloads running inside the cluster. It relies on markers defined in your manifests (either workload resources or any CRD) e.g. # {"$imagepolicy": "flux-system:podinfo:tag"}. To be able to define an automation workflow you would need to create a resource using the ImageUpdateAutomation, detailed here.

Thanks for the detailed response. I think we'll hold off on upgrading to Flux2 until more of the image tag matching is ironed out and implemented. Also, (and I know an only-tangentially-related Github issue comment is not the place to effectively share this kind of feedback but I'm not sure where else to do it), as an end user simply trying to "upgrade" from Flux/helm-operator v1 to the whole GOTK ecosystem is overwhelmingly complex. All I want is for my manifests/helm charts in git to be synchronized to my clusters, and my docker image pushes to be deployed automatically and synchronized back to git. Yes, GOTK can do it, but it seems like using an (admittedly awesome) aircraft carrier to kill a housefly.
Or, to illustrate the point using maybe my favorite animated GIF of all time, just replace "systemd" with "GOTK" and all of its snacks with "kubectl", "kustomize", "git", "docker", etc.

That is an excellent animated gif :-D
Is it fair to paraphrase your feedback as "It's difficult to know where to start with Flux v2 because there are many more parts to it"? I wonder if we could have a simple menu of "If you use Flux v1 for X then the equivalent is Y (go here to see how to set it up)", so people coming from Flux v1 have obvious signposts into the relevant docs.

That is an excellent animated gif :-D
Is it fair to paraphrase your feedback as "It's difficult to know where to start with Flux v2 because there are many more parts to it"? I wonder if we could have a simple menu of "If you use Flux v1 for X then the equivalent is Y (go here to see how to set it up)", so people coming from Flux v1 have obvious signposts into the relevant docs.

I wonder if we could have a simple menu of "If you use Flux v1 for X then the equivalent is Y (go here to see how to set it up)", so people coming from Flux v1 have obvious signposts into the relevant docs.

Such a doc exists here: https://toolkit.fluxcd.io/guides/flux-v1-migration/#in-place-migration

I wonder if we could have a simple menu of "If you use Flux v1 for X then the equivalent is Y (go here to see how to set it up)", so people coming from Flux v1 have obvious signposts into the relevant docs.

Such a doc exists here: https://toolkit.fluxcd.io/guides/flux-v1-migration/#in-place-migration


No scroll to navigate the diagram

No vertical/horizontal scroll to see the whole diagram.

Sorry, but I consider scrollbars a thing of the past. They take a lot of space and are usually sluggish to use. Why do you miss them exactly?

I agree the scroll bar take some space, but we need a fast way to navigate and see a portion of the diagram. When the diagram has a lot of boxes in it, the space require to display the complete diagram will not fit into the screen display without scrolling

Indeed and having them doesn't mean you have to use them.


move to proper location

To what it worth, plugin itself works fine after I moved files, I am not good at writing tests though.

closing this one in favour of https://github.com/choria-plugins/shell-agent/pull/6


SSH Tunneling test returns normal response even with incorrect password

In DBeaver, instead of connecting remotely, there's a button to simply test the creation of an SSH Tunnel. For other typical SSH implementations, if you input the wrong password at this time, an error is returned. But with shellhub, even if you don't enter the password or enter it incorrectly, a normal response is displayed. This can give the wrong signal to the user.

Originally posted by @civilizeddev in https://github.com/shellhub-io/shellhub/issues/3233#issuecomment-1789993330

Logs (shellhub-ssh)
level=trace msg="trying to use password authentication" sshid=ubuntu@(redacted SSHID)
level=info msg="using password authentication method to connect the client to agent" sshid=ubuntu@(redacted SSHID)

Logs (shellhub-ssh)
If you proceed with port forwarding with the wrong password, this is what happens.
level=trace msg="trying to use password authentication" sshid=ubuntu@(redacted)
level=info msg="using password authentication method to connect the client to agent" sshid=ubuntu@(redacted)
level=info msg="handling direct-tcpip channel" sshid=(redacted) username=ubuntu
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0xa8 pc=0xa51fd7]

goroutine 85 [running]:
github.com/shellhub-io/shellhub/ssh/session.(*Session).NewClientConnWithDeadline(0xc00022a6e0, 0xc0001da680)
    github.com/shellhub-io/shellhub/ssh/session/session.go:233 +0x3f7
github.com/shellhub-io/shellhub/ssh/server.NewServer.TunnelDefaultDirectTCPIPHandler.func6(0xc0000f90e0, 0xc000010480?, {0xe36658, 0xc0001300c0}, {0xe3c5a0, 0xc000010048})
    github.com/shellhub-io/shellhub/ssh/server/channels/tcpip.go:122 +0xe05
created by github.com/gliderlabs/ssh.(*Server).HandleConn in goroutine 47
    github.com/gliderlabs/ssh@v0.3.5/server.go:305 +0x52b

Sorry, I think I didn't get it because I was using a public key for authentication.
I have forced a push with a small modification to check if it solves that panic problem.
I will test here

Okay, I seam to be solved. Could you, please, test it again?

Now, a panic does not occur due to a password error.
level=trace msg="trying to use password authentication" sshid=ubuntu@(redacted)
level=info msg="using password authentication method to connect the client to agent" sshid=ubuntu@(redacted)
level=info msg="handling direct-tcpip channel" sshid=(redacted) username=ubuntu
level=error msg="Error when trying to create the client's connection" error="ssh: handshake failed: ssh: unable to authenticate, attempted methods [none password], no supported methods remain" session=(redacted)
level=error msg="failed creating agent connection" dest_addr=(redacted) dest_port=5432 error="ssh: handshake failed: ssh: unable to authenticate, attempted methods [none password], no supported methods remain" origin_addr=26587 origin_port= sshid=(redacted) username=ubuntu

However, it still responds as normal when the test ssh tunnel is executed.

Now, a panic does not occur due to a password error.
level=trace msg="trying to use password authentication" sshid=ubuntu@(redacted)
level=info msg="using password authentication method to connect the client to agent" sshid=ubuntu@(redacted)
level=info msg="handling direct-tcpip channel" sshid=(redacted) username=ubuntu
level=error msg="Error when trying to create the client's connection" error="ssh: handshake failed: ssh: unable to authenticate, attempted methods [none password], no supported methods remain" session=(redacted)
level=error msg="failed creating agent connection" dest_addr=(redacted) dest_port=5432 error="ssh: handshake failed: ssh: unable to authenticate, attempted methods [none password], no supported methods remain" origin_addr=26587 origin_port= sshid=(redacted) username=ubuntu

However, it still responds as normal when the test ssh tunnel is executed.
level=trace msg="trying to use password authentication" sshid=ubuntu@(redacted)
level=info msg="using password authentication method to connect the client to agent" sshid=ubuntu@(redacted)

I think that the second problem is a bit more complicated to be solved. I will verify what we can do.

Hi, @civilizeddev.
Thank you for your patience.
This is a normal behavior of ShellHub. Before user connect to the device itself, they connect to our gateway, what validates some information and redirect the data to the device.
The DBeaver tests only this first step, as a result, it is always successful.


Correct cluster-api project path

This PR corrects cluster-api project path

/assign @mfahlandt The fix is in the release crawler

@lwkd-maintainers
PTAL

Yeah, sorry, I keep meaning to merge this when we're not in the middle of editing a weekly edition, and I keep forgetting.

/lgtm
/approve


Format selection deletes regex inside brackets; javascript

Create a javascript file with the following code: (/a/)
select the first / of the regex
right click => Format Selection
The regex /a/ is now deleted

What VS Code version and TypeScript version are you using?

Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!

Fixed in latest insiders


Update Traditional Chinese translation

I found that the Traditional Chinese is outdated so I added translations to it. And fix some terms.

A new inspection was created.

The inspection completed: No new issues

Thank you @hirakujira ! Merged, will be available in 2-3 days officially, as patch release 3.1.20
Cheers!


Port build processes over to voltrazord

Generator changes per last week's pairing. Let me know how it works out.
Added

cf-type-testing to ignore list.

Changed

Migration to Voltrazord: Build processes now use npm instead of Bower.

Fixed

cf-theme-overrides.less now uses new brand palette color variables.

Removed

All Bower references.

Review

@Scotchester

Notes

https://github.com/cfpb/capital-framework/pull/256 needs to be merged before this PR.

Code looks good. Pulling down for a local test now.

Update to fix the earlier identified issue pushed. @contolini Can you pull this down and try a fresh generation and see if it works as expected? Also see if you run into that hanging issue.

Niiiiice! Both the grunt and gulp versions build without errors for me. And the generated dist directories look A-OK. Even the pesky icons are loading correctly.

Mergin' and publishin'.

Just to double-check: Do you mean that you ran the generator and it didn't hang just before running setup.sh?

Correct. :+1:
On Wednesday, January 6, 2016, Scott Cranfill<EMAIL_ADDRESS>wrote:

Just to double-check: Do you mean that you ran the generator and it didn't
hang just before running setup.sh?
—
Reply to this email directly or view it on GitHub
https://github.com/cfpb/generator-cf/pull/107#issuecomment-169349781.

--
Chris

Cool. Must've been a local issue for me.


Updated Poison Semver and Enhanced Config

I really like this library. I updated the deps that were outdated. I also want to disable sending events to Mixpanel altogether during tests (an not receive warnings). So I modified the config to include an active variable.

Thanks!


Fix an issue where selected import format ignored if multile action forms presented on the page

Problem
When an admin page has both actions_on_bottom and actions_on_top set to True the selected data format is ignored and You must select an export format displayed to the user.

Coverage decreased (-2.5%) to 94.193% when pulling 5d4d45458b7a88e5f0bfe9e59ab3bdb771d3bec6 on sprymak:master into b367e106794ef3a029f933702644b4f81453a0be on django-import-export:master.

Coverage decreased (-2.5%) to 94.193% when pulling 5d4d45458b7a88e5f0bfe9e59ab3bdb771d3bec6 on sprymak:master into b367e106794ef3a029f933702644b4f81453a0be on django-import-export:master.

Thanks for the contrib. Im going to convert this to a draft since there is no test coverage.

Hi @sprymak it's been a year since the last activity on this PR.  Please let us know if you want to proceed, in which case we'll need a reproducible test steps, and test coverage.
thanks


Add GitHub actions setup

It would be nice to ask the user if they want a GitHub or GitLab setup during the creation.

Added on version 1.3.1


[v2.12.2] 'BaseListener' object has no attribute 'output_queue'

I tried to run the example given on readme:
import betfairlightweight
from betfairlightweight.filters import (
    streaming_market_filter,
    streaming_market_data_filter,
)

trading = betfairlightweight.APIClient('myusername', 'mypassword', app_key='myappkey')
trading.login_interactive()

betfair_socket = trading.streaming.create_stream()

market_filter = streaming_market_filter(
    event_type_ids=['7'],
    country_codes=['IE'],
    market_types=['WIN'],
)
market_data_filter = streaming_market_data_filter(
    fields=['EX_ALL_OFFERS', 'EX_MARKET_DEF'],
    ladder_levels=3
)

betfair_socket.subscribe_to_markets(
    market_filter=market_filter,
    market_data_filter=market_data_filter,
)

betfair_socket.start()

but got AttributeError at betfair_socket.subscribe_to_markets():
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File ".../.venv/lib/python3.8/site-packages/betfairlightweight/streaming/betfairstream.py", line 132, in subscribe_to_markets
    self.listener.register_stream(unique_id, "marketSubscription")
  File ".../.venv/lib/python3.8/site-packages/betfairlightweight/streaming/listener.py", line 30, in register_stream
    self.stream = self._add_stream(unique_id, operation)
  File ".../.venv/lib/python3.8/site-packages/betfairlightweight/streaming/listener.py", line 71, in _add_stream
    return MarketStream(self, unique_id)
  File ".../.venv/lib/python3.8/site-packages/betfairlightweight/streaming/stream.py", line 21, in __init__
    self.output_queue = listener.output_queue
AttributeError: 'BaseListener' object has no attribute 'output_queue'

I tried on Python 3.8.9 with betfairlightweight[speed].

I found I need to pass a stream listener with bound queue:
class PrintQueue:
    def put(self, item):
        print('Received:', item)

betfair_socket = trading.streaming.create_stream(listener=betfairlightweight.StreamListener(PrintQueue()))

would be worth to add it to readme.

Ah fix would be to change this line to use the StreamListener instead


Router receive the packet itself broadcast and keep sending again

I used the example project router to transfer one packet from <IP_ADDRESS>/<IP_ADDRESS> to <IP_ADDRESS>/<IP_ADDRESS>. The router transferred the packet from one gateway to another  successfully, but when the other gateway tried to ship it away, it received the packet it send away and tried to send it again. This makes the gateway keeps receiving and sending the same packet again and again.
I think there is a BUG in the link layer for checking the MAC address, but I dont konw how to fix it. The results are as bellow:

What should I do?

actually I have the route(gateway) and packet here, how can I get their MAC address and make a compare?

What function can I use?

Hi. It looks like you're using the net/integration/router as a base, correct? I would say this code is only good for that test alone. We do have a router facility that's much better suited for this, and more tested/used: https://github.com/includeos/IncludeOS/tree/master/examples/router (the routes are setup in config.json).
If you curious about how the forwarding works, it's at the bottom of this header: https://github.com/includeos/IncludeOS/blob/master/api/net/router.hpp
To answer your second question:
To get the interface's MAC addr you can just say stack.link_addr().
To get the MAC addr of the packet, we need to make some hacky stuff... ;)
The packet on the way up the stack gets "promoted" from Ethernet (link) to IP4 (network) by having its pointer to the payload moved beyond the Ethernet frame and up to the IP4 header. So what we could do is the get the start of the IP4 header and subtract the size of the Ethernet header to find out where it starts (this assumes Ethernet with no VLAN tag).
Hopefully this will work (haven't tried it):
#include <net/ethernet/header.hpp>

auto* hdr_ptr = reinterpret_cast<char*>(&pckt->ip_header());
hdr_ptr -= sizeof(net::ethernet::Header);
const auto& eth_hdr = *reinterpret_cast<net::ethernet::Header*>(hdr_ptr);
printf("src=%s dst=%s\n",
  eth_hdr.src().to_string().c_str(),
  eth_hdr.dest().to_string().c_str());

Thanks for replying. I am using net/integration/router as a base and it going on pretty good, I used it to link my Linux and another Unikernel Web service. The Linux requests the web from the Web service through the Router Unikernel. It works, just exists the small BUG. About the way to get the MAC address of the Packet seems exists some problem:

What should I do? Thanks for replying again, your tips are very useful and considerate.

Ok I see we can simplify it a bit. Try changing first line to:
auto* hdr_ptr = pckt->layer_begin();

@ggbbr Did this solve your problems? Can we close this issue?

Yes, my problem solved, sorry I am doing the test these days and forgot to reply. Thank you guys so much for your hard-working in this field and your kindness, I benefit from this a lot.


vscode web showing error/notification "create workspace"

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.63.0-dev
OS Version: ubuntu 20.04

Steps to Reproduce:

I followed https://github.com/Microsoft/vscode/wiki/How-to-Contribute#run to run vscode in my browser
git clone https://github.com/microsoft/vscode.git .
sudo apt-get install build-essential g++ libx11-dev libxkbfile-dev libsecret-1-dev python-is-python3
yarn
yarn watch
yarn watch-web
yarn web
everything is good except that I am not able add extensions or use terminal.
The notification message says that I need to create workspace.
Can anyone tell me how to create and add workspace for vscode to run in my browser

The yarn web command brings up a very lightweight web instance that does not support terminals. If you use resources/server/bin-dev/server.sh instead and click on the link that is printed, you get a more functional web instance.
//cc @aeschli

@bpasero do I need to run any other commands after installing packages with yarn and running the command you told?
Here is the error I got
➜  vscode git:(main) resources/server/bin-dev/server.sh
yarn run v1.22.15
$ node build/lib/builtInExtensions.js
[19:42:52] Syncronizing built-in extensions...
[19:42:52] You can manage built-in extensions with the --builtin flag
[19:42:52] Downloading extension<EMAIL_ADDRESS>...
[19:42:52] Downloading extension<EMAIL_ADDRESS>...
[19:42:52] Downloading extension<EMAIL_ADDRESS>...
[19:42:52] Downloading extension<EMAIL_ADDRESS>...
[19:42:52] [marketplace] ms-vscode.references-view ✔
[19:42:52] [marketplace] ms-vscode.js-debug-companion ✔
[19:42:52] [marketplace] ms-vscode.vscode-js-profile-table ✔
[19:42:54] [marketplace] ms-vscode.js-debug ✔
Done in 3.10s.
yarn run v1.22.15
$ node --max_old_space_size=8192 ./node_modules/gulp/bin/gulp.js node
[19:43:00] Using gulpfile ~/git-prjs/vscode/gulpfile.js
[19:43:00] Starting 'node'...
[19:43:07] Finished 'node' after 6.97 s
Done in 12.31s.
internal/modules/cjs/loader.js:883
throw err;
^
Error: Cannot find module '/home/ashwin/git-prjs/vscode/out/vs/server/main.js'
at Function.Module._resolveFilename (internal/modules/cjs/loader.js:880:15)
at Function.Module._load (internal/modules/cjs/loader.js:725:27)
at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:72:12)
at internal/main/run_main_module.js:17:47 {
code: 'MODULE_NOT_FOUND',
requireStack: []
}

@bpasero Thank you. It worked.
I pulled fresh code and ran the following commands.

yarn
yarn watch
resources/server/bin-dev/server.sh

Then I got terminal. May I know how to load extensions too. Now the extensions are empty even after I search in the extensions panel.


Invalid fee calculation (hideout profit page)

Actual value:
Total fee = 93111 * 2 - 145471 - 8765 = 31986
Fee per item = 15993

Expected:

Most likely, fee calculation function arguments are in wrong order: https://github.com/the-hideout/tarkov-dev/blob/b79d97f8b6bcf43374ba25b5a6f5440b3c276675/src/components/crafts-table/index.js#L301-L305
but should:
 fleaMarketFee(
     craftRow.rewardItems[0].item[priceToUse],
     craftRow.rewardItems[0].item.basePrice,
     craftRow.rewardItems[0].count,
 )

@edward-raven First of all, thank you so much for this issue! This is exactly the kind of issue that makes me happy because you put in a lot of work to document it with screenshots from the game and even some bread crumbs to a possible solution ❤
I will go ahead and self-assign this issue and take a look!

Hmm, looking at the flea-marget-fee.js file, it looks like the arguments are in the correct order given their definition: https://github.com/the-hideout/tarkov-dev/blob/09b9714c7518b54d29ec03fa24237a642d9f951e/src/modules/flea-market-fee.js#L3-L23

basePrice, sellPrice, count = 1 looks accurate to me if we are passing in arguments like so:
fleaMarketFee(
     craftRow.rewardItems[0].item.basePrice,
     craftRow.rewardItems[0].item[priceToUse],
     craftRow.rewardItems[0].count,
 ) *

@edward-raven you are correct! Can you confirm that things are looking better here? https://1910e837.tarkov-dev.pages.dev/hideout-profit/

Oh yep that looks good! I was looking in the wrong place

reopened cc @Razzmatazzz

@edward-raven does this look better to you?
https://flea-fee-calc-fix.tarkov-dev.pages.dev/item/corrugated-hose

Yep. Thank you 👍


Create 2 modes of selection: ChoiceMode or LikeMode

@rafamarts this issue was created for you. Please, feel free to ask if you have any doubts and post your suggestions here so we can discuss.
The solution should support 2 modes of interaction: ChoiceMode between options or LikeMode where the user mark as like or deslike. This mode could be a URL parameter so all the components can autoconfigure based on this parameter.
At this issue you don't need to take care of the styling aspects. We will deal with it in another issue. The main goal here is to make possible for the component to have this 2 interaction modes.
Choice Mode
Two (or more) cards, side by side and the user is choosing sequentially between 2 options

LikeMode
One card with two options: like and deslike and the user is judging sequentially the itens. There could be the option to drag the card to left or right as an option of interaction/
This video illustrates this "like by dragging" interaction. The video has more interactions that are out of scope, but it definetly shows some potential of stop-analyzing use.

Hello,
I just committed the both model in static version, and now I will implement the Drag&Drop and the images reload when the user choose the image.
Please, if someone (@tiagostutz ) could check and comment about the implementation.
Thank you!

Hi,
I've added the Choice Mode. Please, check out if it is what you had in your mind.
Thank you!

Nice. I've just added some review notes in your commit.
One thing I mentioned there and I noticed I forgot to discuss here is that I think we should keep this Choice component we are building in this issue to 2 cards options, like the one shown in https://github.com/bancodobrasil/stop-analyzing/blob/master/README.md. This will simplify our design and code and still have great value. We can discuss later, on another issue, a multiple option/multiple choice approach.
What do you think?

You are right, I think only 2 cards for now is enough. Just made it. Please, review the commit and let me know if need something else for ChoiceMode.
I will work on LikeMode right now. Any suggestion for implement drag&drop? Should we use some library, if so, did you already work with some? I could review some and brings options.
Thank you!

Hi Tiago,
I have finished the LikeMode. I made a simple drag&drop, please, if you have suggestion to do it in different way, let me know. I will work on tests.
Thank you!

Awesome, @rafamarts !
I'll wait for the PR to review the drag&drop feature. Thanks for the surprise! Drag&Drop will be a very handy interaction.

@rafamarts I'm a bit worried with your PR as the project is growing. That's my fault because I opened an issue too big for you, sorry about that.
So, if you want to open the PR with the LikeMode only, I think that's a good ideia. I don't know if you have checked the CONTRIBUTING.md lately, but I updated it with some guildelines on how to keep your fork up to date with the master/origin. I think it would be a good ideia to update your fork (if you haven't done so).
Again, sorry for the low quality issue I opened for you. Let's fix this. 💪

Hello, @rafamarts!
As suggested by @tiagostutz and given that this issue has a big amount of work, can I help you in any way?

Hi @beatrizrezener !
Thank you! I've finished the rebase right now, and I will push it soon. Could you please review and check if is missing something? We can make a to-do list and divide the work.  Maybe, @tiagostutz could also review it and help us to make the to-do list. Is it good for you?

@tiagostutz
Wow! It was a huge rebase. I left the ChoiceMode as it was. The merge only added the id in ChoiceCardProps, and also passed through the cards props to onClick function.
I didn't worry about the design (CSS), maybe it could be done by @beatrizrezener in a new issue. What do you think?
Please, let me know if I can create a PR, or if we will review and make a to-do list.
Thank you!

Hi @rafamarts !! Yoo.. huge rebase indeed! Thanks for your patience on that... ehehehehe...
You can plase open the PR and then we will review together there. I think it will be smoother now.! 😄


etcd and caddy do not run properly when run from cuisine

##goal
-fix both so they run properly
##comments
no error will appear but the process halts directly afterwards ;
to build and start :
j.tool.cuisine.local.apps.etcd.build(start=True)

commit on branch issue_216

@abdulrahmantkhalifa could you explain more what goes wrong?

i believe this is done closed , tested


Mongomock updating objects inserted in the database

Hi,
I've came across weird issue with mongomock. When trying to insert many documents to the collection, insert_many method updates original dictionaries. Consider this example:
import mongomock

documents = [
    {
        'a':5,
        'b':6
    },
    {
        'a': 67,
        'b': 43
    }
]

collection = mongomock.MongoClient().db.collection
collection.insert_many(documents)
print(documents)

Effect in the console is:
[{'a': 5, 'b': 6, '_id': ObjectId('5b60683dabcb193acfa4c21d')}, {'a': 67, 'b': 43, '_id': ObjectId('5b60683dabcb193acfa4c21e')}]

I suppose it is not intended to modify documents?
I am happy to help with fixing this, as long as it is not intended behaviour.

Easiest way would be simply modify Collection._insert method to make deep copy of object before inserting it:
import copy
# (...)
    def _insert(self, data):
        if isinstance(data, list) or isinstance(data, types.GeneratorType):
            return [self._insert(item) for item in data]

        # Like pymongo, we should fill the _id in the inserted dict (odd behavior,
        # but we need to stick to it), so we must patch in-place the data dict
        data = copy.deepcopy(data)
        for key in data.keys():
# (...)

That way modifications of data object will not change original objects passed by user to e.g. insert_many method. Currently one has to make a deep copy of the whole iterable passed to the method, which can be heavy on memory.


How to preserve the order of transduced items?

First of all thanks for sharing your work with the world!
I'm using the Javascript version, and I have a question regarding the .dictionary(locations, false) bit here:
(code below is in es6)
import levenshtein from 'liblevenshtein';
const transducer = new levenshtein
  .Builder()
  .dictionary(locations, false) // What does false mean?
  .algorithm('transposition')
  .include_distance(false)
  .build();
...
const matches = transducer.transduce(query, 1);

When should I pass true and when should I pass false to .dictionary()?
How could I guarantee that matches are in the same order as locations?
For example, if query = 'elbourne', and locations is:
['melbourne', 'sydney', 'london', 'selbourne', 'adelaide', 'delbourne'];

I want to make sure that matches is:
['melbourne', 'selbourne', 'delbourne']

(the order of locations is preserved in matches)

The API is described in the wiki.  See the breakdown of the builder attributes:
https://github.com/universal-automata/liblevenshtein/wiki/usage.coffeescript
The second parameter specifies whether the list of terms is already sorted lexicographically.  If you pass-in false then the list will be sorted by the library.  If you have a very large list then you probably want to sort it before loading it into your application, and let the library know it's been sorted by passing-in true as the second parameter.
At the moment, the JS library only supports ordering candidates lexicographically or in whichever order it identifies them (non-deterministic).  You would have to extend the classes to override the sorting logic.  I'm not happy with the current design, and have been working on a new version, but I've been busy with other projects so I haven't had time to finish the implementation.

You could apply the ordering once you've extracted the spelling candidates.  That would be the easiest solution.

@dylon Thanks for your reponse!


Some Quest Items added through console Staying flagged

And being unable to unequip them.

@MrButterfl7 Hi can you give some examples ? Did you applied the method on various inventories you have ?
See, I have many of them. It has to be done on all of them separately.

Cheers

@MrButterfl7 Hi can you give some examples ? Did you applied the method on various inventories you have ?
See, I have many of them. It has to be done on all of them separately.

Cheers

Yes I tried on all inventories. For example one of Johnny’s pants stay flagged. Or judy’s diving suit.
Sent from my iPhone
On 3 Jan 2021, at 09:00, Psype<EMAIL_ADDRESS>wrote:

@MrButterfl7https://github.com/MrButterfl7 Hi can you give some examples ? Did you applied the method on various inventories you have ?
See, I have many of them. It has to be done on all of them separately.
[image]https://user-images.githubusercontent.com/13078156/103475061-3e4dd780-4daa-11eb-881a-d57c64993453.png
Cheers
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/PixelRick/CyberpunkSaveEditor/issues/6#issuecomment-753587652, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASKEKI67PLSQNLMA42H6XYTSYAWZHANCNFSM4VPYQ73A.

Yes I tried on all inventories. For example one of Johnny’s pants stay flagged. Or judy’s diving suit.
Sent from my iPhone
On 3 Jan 2021, at 09:00, Psype<EMAIL_ADDRESS>wrote:

@MrButterfl7https://github.com/MrButterfl7 Hi can you give some examples ? Did you applied the method on various inventories you have ?
See, I have many of them. It has to be done on all of them separately.
[image]https://user-images.githubusercontent.com/13078156/103475061-3e4dd780-4daa-11eb-881a-d57c64993453.png
Cheers
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/PixelRick/CyberpunkSaveEditor/issues/6#issuecomment-753587652, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASKEKI67PLSQNLMA42H6XYTSYAWZHANCNFSM4VPYQ73A.

It is believed that those are special items, re-given on save load by game scripts based on game facts. It may be possible later to bypass it. It is also possible that future game updates will remove these scripts, probably there to patch faulty quest logic.

It is believed that those are special items, re-given on save load by game scripts based on game facts. It may be possible later to bypass it. It is also possible that future game updates will remove these scripts, probably there to patch faulty quest logic.

I got that issue, the flag was at 02 instead of 01.
I guess that's why your button didn't caught them.

I got that issue, the flag was at 02 instead of 01.
I guess that's why your button didn't caught them.

Added a checkbox for this second flag. The "unflag all" won't change it though as it is not yet fully understood what this flag means.

Added a checkbox for this second flag. The "unflag all" won't change it though as it is not yet fully understood what this flag means.


Read data with NEMO 4.0, z coordinate absent of mesh_mask file

Hello,
it seems that from the version 4.0 of NEMO the mesh_mask dimension in the vertical is nav_lev and not z. I use a test case where all the domain is defined in the usr_def files and then output by NEMO.
The absence of the z dimension breaks the opening of the output files with load_xorca_dataset in the function create_minimal_coords_ds, as "z" is not a dimension (a KeyErroris raised at line 89):
https://github.com/willirath/xorca/blob/9ccbfb6fc880247b71503c12417e1e284e5738a8/xorca/lib.py#L89
I coded a very simple patch like following, if appropriated I can submit a PR for this.
def create_minimal_coords_ds(mesh_mask, **kwargs):
    """Create a minimal set of coordinates from a mesh-mask dataset.

    This creates `"central"` and `"right"` grid points for the horizontal grid
    and `"central"` and `"left"` grid points in the vertical.

    """
    try:
        N_z = len(mesh_mask.coords["z"])
    except KeyError:
        N_z = len(mesh_mask.coords["nav_lev"])
    N_y = len(mesh_mask.coords["y"])
    N_x = len(mesh_mask.coords["x"])

    [...]

Attached is the output of ncdump -h with the mesh_mask.nc file, with an extract here:
dimensions:
	x = 20 ;
	y = 40 ;
	nav_lev = 36 ;
	time_counter = UNLIMITED ; // (1 currently)
variables:
	float nav_lon(y, x) ;
	float nav_lat(y, x) ;
	float nav_lev(nav_lev) ;
        [...]

xorca_issue_mesh_mask.txt

Hi @rcaneill, thanks for wanting to contribute here. A PR would be very welcome!
While you’re  here: This notebook shows some tests I’m doing (too busy with other things, however…) for autodetecting most of the grid specifics. Ideally, I'd like to really strip down Xorca to the bare minimum of hard-coded variable names etc.
If you have any NEMO 4 example data, it could be a good thing to post them on Zenodo. I've done so with

https://zenodo.org/record/3755721
https://zenodo.org/record/3755517
https://zenodo.org/record/3634491
to have an easy way of referring to some representative data and to have something (smallish) I can pull in for testing and demos.

Something I'd love to go for is to isolate the part that transforms Xarray Datasets with NEMO grid and mesh data into XGCM-compatible Xarray datasets from the data loading part. This way, we'd be a lot more flexible to use NEMO data in any format we'd like. (There's many people looking into transforming NEMO data to, e.g., Zarr or other things for making it cloud-ready. So the more format agnostic the tool is, the better!)

I'll have a proper look on the notebook next week.
For the detection of the position of the variables, I am using the filename (grid_T for T points, grid_U for U point, etc), which seemed to be the easiest for me. That's what I do in the package  xnemogcm I developed before hearing of xorca.
I have a ~120MB test dataset, I can upload it to zenodo if you're interested to have them. They are in an idealized basin, terrain following coordinates with non flat-bottom.


Add project folder path to Contracts tab

It would be nice to have the project folder from where each contract came from

to clarify here: the contracts come from a regex that's roughly matching against /<project_name>/out/***. we assume that first name is the project's name, and we can use that to separate different projects in the contracts tab
it could also be worth changing to regex to also cover the fact that a lot of projects also do /<project_name>/contracts/out/***


Role for Azure AD connect Health dashboard

Is there a specific role in which this dashboard can be viewed,

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: add29a81-b72c-09c4-5d51-ba7b6fa99df9
Version Independent ID: 87df83ce-de7c-6948-931f-3b91b0b2ff87
Content: Using Azure AD Connect Health with sync
Content Source: articles/active-directory/hybrid/how-to-connect-health-sync.md
Service: active-directory
GitHub Login: @zhiweiw
Microsoft Alias: billmath

@amel00 Thanks for your feedback! We will investigate and update soon on it.

@amel00 all of the access to the dashboard is configurable. In order to get started with AD Connect Health, install it, and configure roles you need to be a global administrator of the account. However, you can configure Role-Based Access Control to grant access to other users in your organization and configure this access however you would like.

See: https://docs.microsoft.com/en-us/azure/active-directory/hybrid/how-to-connect-health-agent-install
See also: https://docs.microsoft.com/en-us/azure/active-directory/hybrid/how-to-connect-health-operations#manage-access-with-role-based-access-control

@amel00 hope this answers your question! I'm closing this out for now but if you have further questions, feel free to leave a comment and I will gladly continue the discussion.

Marilee,
That doesnt answer anything.. We have Azure AD Connect Health installed and running fine. I am asking if there is a built in role that grants access to Health, something like Health Viewer
Roles and administrators -
Security Viewer - does not grant access to this
Access can be granted individually but I was wonder why there isnt a built in role for people to be able to view this

Testing for synchronization in WT


org.rabix.bindings.cwl.processor.CWLPortProcessorException: Failed to set input file properties

Couple of tools where this happens:
inputbinding.zip

I'm encountering what's probably a similar/related issue, although the error meesage is org.rabix.bindings.sb.processor.SBPortProcessorException: Failed to set inputs propert ies.  This is with rabix 1.0.3.  Attached is a zip with a CWL file, an inputs file, and a log of a run.
JN.zip

BTW: This error happened on a couple of jobs I attepted to run; I had also tried rabix 1.0.1 on some of those jobs, and got the error message: org.rabix.bindings.sb.processor.SBPortProcessorException: Failed to set input file size.

Seems to be an issue when  using relative paths both for the cwl file and in input files. In this case the location of input file is attempted to be resolved against the parent folder of the cwl job file, but as cwl file is referenced as a relative path it fails.
I've made some changes on this branch to always convert to absolute path when processing files and they will be added to the 1.0.4 release.

OK, thanks.  Any idea when that release will be out?  (FWIW, this worked with cwltool, in terms of the pattern of where the cwl/input files were, and the paths specified in the input files.)
In the meantime, I tried re-running some jobs with the full/absolute path to the input files, and one of them is failing with a cryptic (to me) error message: [ERROR] Job root failed with exit code 127..   There's really no other messages to go on.  Any ideas on how I can get more information to help troubleshooting?

When jobs fail during execution you can usually find more info in docker logs. Find the id of the docker container that failed (docker ps -a or something) and call docker logs {containerId}.
We'll see about automatically printing this info in the rabix command line in the future.

OK, I looked at the output of docker logs for the ones that failed with exit code 127, and I got the following:
/bin/sh: error while loading shared libraries: libreadline.so.7: cannot open shared object file: No such file or directory
The thing is, we've used this container in other contexts (including running the analogous CWL with cwltool), and we didn't get this error, it worked fine.  Is there some reason this error would occur with rabix but not in these other contexts?  Maybe it's not directly related to rabix, but I don't know why this would happen.

When you do "docker inspect" on that container there should be a Cmd part which is the executed command and Mounts which are the mounted files. Might be that a file is referenced in the command line that isn't in the mounted folders.

OK, looking at the mounts/binds, it includes the working directory, the input file, and /bin/sh (to itself); the command is sh run_dc.cmd (run_dc.cmd is a file built by the CWL).  I think the problem is related to the bind of /bin/sh (it overwrites the one in the container).  I ran ldd on it on both my machine and inside the container, on my machine it requires libreadline.so.7, inside the container it doesn't.  So that could explain the problem.
OK, so why is it binding /bin/sh.  I have that as the default value for an input file in my CWL (this CWL was originally developed for cwltool, and that's how I made that input be optional; I've modified the CWL for the CGC, it's now using javascript for that, so that's no longer necessary here).  In this case, it's specified as sbg:toolDefaultValue and default.  When I remove the default, it works (the sbg:toolDefaultValue didn't matter).
Now this isn't a problem with cwltool.  There I found that it's binding /bin/bash, but not to itself, but rather in a temporary/working directory:
--volume=/bin/bash:/var/lib/cwl/stg75a0446e-5714-4428-8067-bce25e63d447/sh:ro \
Why /bin/bash and not /bin/sh?  On my machine, /bin/sh is a symbolic link to /bin/b ash.
So I think this explains the problem, and perhaps suggests some changes for rabix.
Thanks.

OK, I got past the problem in the previous post (and the earlier ones), but I'm still getting an error.  It's the Failed to set inputs properties. again.  Although it seems to be finding the file, and it's not a relative path issue (it's an intermediate file in a workflow).  There's also an [ERROR] Failed to remove container with id null, not sure if that's significant.  Attached is a zip file with the json CWL, the yaml input parameters, the input file, and a log of the output I'm getting.  Thanks.
GSCW.zip


chore(deps): update renovatebot/github-action action to v40.1.7

This PR contains the following updates:

Package
Type
Update
Change

renovatebot/github-action
action
patch
v40.1.6 -> v40.1.7

Release Notes

renovatebot/github-action (renovatebot/github-action)
v40.1.7
Compare Source
Miscellaneous Chores

deps: update dependency @types/node to v20.11.29 (01dcd82)
deps: update dependency @types/node to v20.11.30 (0103716)
deps: update dependency semantic-release to v23.0.5 (3a0b39f)

Build System

deps: update dependency typescript to v5.4.3 (18cb877)

Continuous Integration

deps: update renovate docker tag to v37.270.0 (9e52fb1)
deps: update renovate docker tag to v37.271.1 (473c572)
deps: update renovate docker tag to v37.272.0 (4616036)

Configuration
📅 Schedule: Branch creation - "every weekday after 4am" (UTC), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Edited/Blocked Notification
Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.
You can manually request rebase by checking the rebase/retry box above.
⚠ Warning: custom changes will be lost.


Neofetch is reporting macOS 12 on Ubuntu 22.04.

Description
The image is quite self-explanatory. I have installed Ubuntu 22.04 (Dev. branch) and neofetch says I am on macOS Monterey.

[x] Does this issue still occur in the master branch? (Required if issue)

Neofetch version

develc@machine-nr5c9pv:~$ neofetch --version
Neofetch 7.1.0

Screenshot

Config file
None I suppose
Verbose log

Run neofetch -vv 2> neofetchlog
Upload the contents of neofetchlog to pastebin, gist or equivalent.
The pastebin containing a Verbose log is: https://pastebin.com/yPBu2gAd

Nevermind. I had to reinstall Ubuntu anyway, so I close this now that (of course) neofetch is working.


ignore option

Hi,
I can't seem to get the --ignore option to work with simple patterns whereas regexes do.
chokidar './src/**/*.js' -i '*-test.js'  nope
chokidar './src/**/*.js' -i '/-test.js$/' yep
Probably missing something obvious.

It seems that the ignore case flag was preventing my regex to work (even though an example with /i flag is described in the help
Works:
chokidar '**/*.go' -d 1000 -t 15000 -c 'npm run godev:watch' -i '/_(?:sql2var|ffjson).go$/'

Fails:
chokidar '**/*.go' -d 1000 -t 15000 -c 'npm run godev:watch' -i '/_(?:sql2var|ffjson).go$/i'

I have found the cause of the problem. The ignore option currently only supports regular expressions. The specific code is in: https://github.com/open-cli-tools/chokidar-cli/blob/214494a336c49129d8828f22838b7a0b36980c8e/index.js#L186


Update helm 3+ installing CRDs

Based on https://cert-manager.io/docs/installation/kubernetes/#steps installation process for helm 3+ is different

@yazdipour : Thanks for your contribution! The author(s) have been notified to review your proposed change. @caya

Hi @mscatyao Could you review the proposed update to your article?
Also, could you create a pull request in the private repo to update the metadata author value for your articles to use your current GitHub user name? PRMerger uses that attribute to assign PRs and send review notifications when one of your articles is updated in the public repo. Thank you!

#sign-off

@mscatyao: I'm sorry - only the author of this article, @caya, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


with this simple changes it compiles with node > 0.12

Is this one still necessary with the new 0.8.0?

Now it does compile post-0.12.


Update Pages “IsTheTreatmentSafePage”

Automatically generated by Netlify CMS

👷 Deploy Preview for chic-shortbread-66a66b processing.

Name
Link

🔨 Latest commit
b00ad83ee15e0221e1a329e1834af6ca1b27bfb0

🔍 Latest deploy log
https://app.netlify.com/sites/chic-shortbread-66a66b/deploys/6570024924626100084cd45e


test: useDxtradeAccountHandler

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.6%) to 39.701%

Totals

Change from base Build<PHONE_NUMBER>:
0.6%

Covered Lines:
694

Relevant Lines:
1321

💛  - Coveralls


Add validation of parsed yaml

Since the YAML parser just takes the dict passed from the YAML and uses that to set the state in the objects, we might not be setting the correct things.
We need to validate that all required fields are set, and possibly make sure that they are the right type and convert if needed.
The type aspect is probably relevant when used with environment variables, e.g. #265 since I believe that values specified as env vars will be handled as strings.

Although an alternative could perhaps be not to use a schema and have each class use set_state to make sure all required fields are set and have the right type?


fix(accordion): keyboard a11y

Adds unit tests for keyboard accessibility fixes in <pfe-accordion>
Related issues

(#1779)

Preview

Link(s) to demo page(s) where this element can be viewed:

Link

What has changed and why

use delegatesFocus when attaching <pfe-accordion-header> shadowroot
unit tests for wai-aria practices

Testing instructions

[ ] Test case 1

npm run test -- --watch --group default --files elements/pfe-accordion/test/pfe-accordion.spec.ts

[ ] Test case 2

npm start
Manual keyboard testing according to wai aria best practices

Ready-for-merge Checklist

[x] Expected files: all files in this pull request are related to one request or issue (no stragglers or scope-creep).
[x] Tests have been updated to cover these changes.
[ ] Browser testing passed.
[ ] Changelog updated.
[ ] Documentation (README.md, WHY.md, etc.) updated or added.
[ ] Link to the demo recording:
[ ] Approved by designer.

Merging
Please squash when merging and ensure your commit message uses conventional commit formatting.
Be sure to share your updates with the<EMAIL_ADDRESS>mailing list!

paging @kelsS This PR attempts to fix accordion keyboard a11y within the pfe rewrite, #1804
To try it out for yourself, first ensure you have the prerequisites (nvm recommended) clone the pfe repo
git clone git@github.com:patternfly/patternfly-elements.git

then setup the node version with nvm use and install dependencies
npm ci

run the unit tests on the accordion with
npm run test -- --watch --group default --files elements/pfe-accordion/**/*.spec.ts

run the development server
npm start

I was able to get some of the tests passing with minimal code changes. I have some questions about behaviour before I continue implementing fixes:

on the WAI-ARIA practices page (linked above) accordion is specified to close other panels, meaning only 1 panel is ever supposed to be open. Our accordion doesn't work like that, is that intended behaviour?

Note to reviewers: at the target branch gets updated, a github action workflow tries (and fails) to update this branch. That makes it hard to find the relevant changes, so I'm linking to the test cases for keyboard accessibility here:
https://github.com/patternfly/patternfly-elements/blob/feat/pfe-accordion-keyboard-a11y/elements/pfe-accordion/test/pfe-accordion.spec.ts#L452-L904

@markcaron helped me understand the authoring practices

Some implementations require one panel to be expanded at all times and allow only one panel to be expanded; so, they do not support a collapse function.

That means our implementation is fine. I'll update my tests

I'm going to merge this into #1804 now, further review should proceed there


Documentation unclear on how to specify multiple restore-keys

Documentation at https://docs.github.com/en/actions/using-workflows/caching-dependencies-to-speed-up-workflows states that restore-keys is "An ordered list of alternative keys to use for finding the cache if no cache hit occurred for key.". However, if I supply an ordered list for restore-keys in a job, I get an error:
      - name: Cache CVE database for OWASP dependency-check
        uses: actions/cache@v3
        with:
          path: dependency-check-data
          key: "${{ runner.os }}-dependency-check-data-${{ hashFiles('**/nvdcve-1.1-modified.meta') }}"
          restore-keys:
            - "${{ runner.os }}-dependency-check-data-${{ hashFiles('**/nvdcve-1.1-modified.meta') }}"
            - "${{ runner.os }}-dependency-check-data-"

Invalid workflow file: .github/workflows/maven.yml#L37
The workflow is not valid. .github/workflows/maven.yml (Line: 37, Col: 13): A sequence was not expected

If this is not how an ordered list should be specified, a more detailed explanation in the documentation would be useful.
The example that does specify multiple restore-keys does it as a multi-line string, but when I try this the entire string appears to be treated as a single key?
      - name: Cache CVE database for OWASP dependency-check
        uses: actions/cache@v3
        with:
          path: dependency-check-data
          key: "${{ runner.os }}-dependency-check-data-${{ hashFiles('**/nvdcve-1.1-modified.meta') }}"
          restore-keys: |
            ${{ runner.os }}-dependency-check-data-${{ hashFiles('**/nvdcve-1.1-modified.meta') }}
            ${{ runner.os }}-dependency-check-data-

Debug output on post-step:
##[debug]Evaluating: format('{0}-dependency-check-data-{1}
##[debug]{2}-dependency-check-data-
##[debug]', runner.os, hashFiles('**/nvdcve-1.1-modified.meta'), runner.os)
##[debug]Evaluating format:
##[debug]..Evaluating String:
##[debug]..=> '{0}-dependency-check-data-{1}
##[debug]{2}-dependency-check-data-
##[debug]'
##[debug]..Evaluating Index:
##[debug]....Evaluating runner:
##[debug]....=> Object
##[debug]....Evaluating String:
##[debug]....=> 'os'
##[debug]..=> 'Linux'
##[debug]..Evaluating hashFiles:
##[debug]....Evaluating String:
##[debug]....=> '**/nvdcve-1.1-modified.meta'
##[debug]Search root directory: '/home/runner/work/********/********'
##[debug]Search pattern: '**/nvdcve-1.1-modified.meta'
##[debug]Starting process:
##[debug]  File name: '/home/runner/runners/2.289.2/externals/node16/bin/node'
##[debug]  Arguments: '"/home/runner/runners/2.289.2/bin/hashFiles"'
##[debug]  Working directory: '/home/runner/work/********/********'
##[debug]  Require exit code zero: 'False'
##[debug]  Encoding web name:  ; code page: ''
##[debug]  Force kill process on cancellation: 'False'
##[debug]  Redirected STDIN: 'False'
##[debug]  Persist current code page: 'False'
##[debug]  Keep redirected STDIN open: 'False'
##[debug]  High priority process: 'False'
##[debug]Updated oom_score_adj to 500 for PID: 2010.
##[debug]Process started with process id 2010, waiting for process exit.
##[debug]Match Pattern: **/nvdcve-1.1-modified.meta
##[debug]::debug::followSymbolicLinks 'false'
##[debug]::debug::followSymbolicLinks 'false'
##[debug]::debug::implicitDescendants 'true'
##[debug]::debug::omitBrokenSymbolicLinks 'true'
##[debug]::debug::Search path '/home/runner/work/********/********'
##[debug]/home/runner/work/********/********/dependency-check-data/nvdcache/nvdcve-1.1-modified.meta
##[debug]Found 1 files to hash.
##[debug]Hash result: 'a65b3939924c996fc1daedc8e7c7c4cdf4ea9ec5b7a3c2b495ff37333213'
##[debug]undefined
##[debug]STDOUT/STDERR stream read finished.
##[debug]STDOUT/STDERR stream read finished.
##[debug]Finished process 2010 with exit code 0, and elapsed time 00:00:00.2069719.
##[debug]..=> 'a9394c996fc1da62edc8e7c7c4cdf4ea9ec5b7a3c2b495ff37333213'
##[debug]..Evaluating Index:
##[debug]....Evaluating runner:
##[debug]....=> Object
##[debug]....Evaluating String:
##[debug]....=> 'os'
##[debug]..=> 'Linux'
##[debug]=> 'Linux-dependency-check-data-a65b5739924c996fc1da62edc8e7c7c4cdf4ea9ec5b7a3c2b4ff37333213
##[debug]Linux-dependency-check-data-
##[debug]'
##[debug]Result: 'Linux-dependency-check-data-a65b573939924c9fc1da62edc8e7c7c4cdf4ea9ec5b7a3c2b495ff37333213
##[debug]Linux-dependency-check-data-
##[debug]'

@fransf-wtax  Can you share the full logs please, along with the cache step.

The example that does specify multiple restore-keys does it as a multi-line string, but when I try this the entire string appears to be treated as a single key?

This should work, and not be treated as a single key. How did you infer that this is being treated as a single key ?

@vsvipul OK I might be mis-interpreting the output. Here's the full log: https://gist.github.com/fransf-wtax/b8db372355bebefe79ccb038041a223a
The part confused me is:
##[debug]..Evaluating Index:
##[debug]....Evaluating runner:
##[debug]....=> Object
##[debug]....Evaluating String:
##[debug]....=> 'os'
##[debug]..=> 'Linux'
##[debug]=> 'Linux-dependency-check-data-a65b573939924c996fc1da62edc8e7c7c4cdf4ea9ec5b7a3c2b495ff37333213
##[debug]Linux-dependency-check-data-
##[debug]'
##[debug]Result: 'Linux-dependency-check-data-a65b573939924c996fc1da62edc8e7c7c4cdf4ea9ec5b7a3c2b495ff37333213
##[debug]Linux-dependency-check-data-
##[debug]'

So between the opening ' and closing ' there are two newlines which makes it look like the entire string is interpreted the single key. But maybe there's processing happening after that that's not reflecting in the debug logs?
I guess that just leaves improving the documentation to indicate that an "ordered list" is in fact not a yaml list but a string with items separated by newlines.
Happy to submit a PR, just not quite sure where the source code to https://docs.github.com/en/actions/using-workflows/caching-dependencies-to-speed-up-workflows lives, would be great if you can point me in the right direction!

Adding @lucascosti to help with the docs change.

Docs change opened in https://github.com/github/docs/pull/16922

Fixed the docs. @fransf-wtax  Thank you for opening.


check & init chromedriver

checks if chromedriver is installed
checks if chromedriver is newer than the minimum version

What is this doing?

@justdvl  see above (just edited)

Oh ok. I thought it could also auto download Chromedriver.
BTW I wonder, is there a way to add this script, or Instapy to npm library, to have it maintained automatically?

that would be nice but i think instapy has a lot of overhead for this project


[Snyk] Security upgrade react-syntax-highlighter from 11.0.2 to 13.0.0

Snyk has created this PR to fix one or more vulnerable packages in the `npm` dependencies of this project.

Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

package.json
package-lock.json

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Breaking Change
Exploit Maturity

661/1000  Why? Recently disclosed, Has a fix available, CVSS 7.5
Regular Expression Denial of Service (ReDoS) SNYK-JS-PRISMJS-1076581
Yes
No Known Exploit

(*) Note that the real score may have changed since the PR was raised.

Commit messages

Package name: react-syntax-highlighter
    The new version differs by 124 commits.

e75a4c7 13.0.0
3ec6e08 Merge pull request #274 from conorhastings/simmerer/improved-demo
6dc2b66 re-auto-generate async-languages and prism styles
b308629 allow style-switching on Prism example, change languages for selector
25f51f9 add language selector to default demo
d85780f consistent control ordering
56e1487 remove useless "defaultStyle" option
d7f6358 specify language for example diff code
21c1377 improved font sizing & title line height
b903c03 Merge branch 'master' into simmerer/improved-demo
3204faa Merge pull request #269 from conorhastings/simmerer/update-highlight-lowlight
ee21ba2 Merge branch 'master' into simmerer/update-highlight-lowlight
1ece955 Merge pull request #259 from napagroup/master
ac3f409 Merge pull request #268 from conorhastings/simmerer/update-refractor
9cf19dd update note on showInlineLineNumbers prop
434547e enable line numbering in virtualized demo
65b606c Merge pull request #272 from conorhastings/dependabot/npm_and_yarn/lodash-4.17.19
2612aab Merge pull request #273 from conorhastings/dependabot/npm_and_yarn/codecov-3.7.1
9fca845 corrected snapshots (finally!)
89b2007 fix for nonexistent getLanguage() func
6fdd6ca Merge branch 'simmerer/update-highlight-lowlight' into improved-demo
688c50c better astGenerator inspection
dd9a2cb consistent markup for other examples
7893abf clean up layout for virtualized example

See the full diff

Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

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


bounds checking in fancy indexing

from neon.backends import gen_backend
import numpy as np
ng = gen_backend(backend='gpu', device_id=0)

x = ng.zeros((5, 5))
i = ng.zeros((3, 1), dtype=np.int32); i[:] = 7
j = ng.zeros((1, 1), dtype=np.int32); j[:] = 7

x[i] = 1 # no errors or apparent changes in x
x[j] = 1 # no errors, but sets x[0, :] to 1

In deference to performance, the GPU backend does little if any bounds checking.  We will add something to the documentation about this.


Use maximum compression for git archives

Followup to https://github.com/artichoke/nightly/pull/173.
Current archives are ~22MB. It seems worth spending some compute on compressing these further, especially since these jobs aren't the long pole of the release process.

this does better, but not by much. Saves maybe 200KB on the archives.

Build is successful: https://github.com/artichoke/nightly/releases/tag/2023-06-05-package-source-archive-v4


[Bug]: Building blocks DS do not get configured for Staging environment

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When I drag and drop a building block on staging environment, it does not work out of the box, because the datasources are configured only for production environment
Can we have some solution for this - either by default both prod and staging env should have the datasource configured by default, or atleast when we drop it on staging env, the datasource should be configured for production
Steps To Reproduce

Open an app on release-ee
Change the env to staging
Drag and drop a building block
it does not work out of the box.

Public Sample App
No response
Environment
Release
Severity
High (Blocker to building or releasing)
Issue video log
No response
Version
Cloud

Bumping this issue up to prod because it should have made it's way there by now if it was not fixed


pip install scButterfly produced an error

Thank you! This is fantastic work.
However, I'm encounting an error when trying to install scButterfly using pip.
Could the authors please look into this issue and help me resolve it?

My interface is the same, and I've been studying it for a long time without solving the issue.


chore: generate coverage report by PHPUnit only on PHP 8.0

Description

To reduce test time

add --no-coverage option to PHPUnit other than on PHP 8.0

Checklist:

[x] Securely signed commits
[] Component(s) with PHPDoc blocks, only if necessary or adds value
[] Unit testing, with >80% coverage
[] User guide updated
[] Conforms to style guide

PHPUnit / PHP 7.4 & 8.1 testing finished in 3m to 6m!


rework silero vad

remove pytorch dependency

we only use onnxruntime now (avg size of onnxruntime is 7MB, silero model is 1.5MB)

use right window sizes (enforce the same one used while training the model)
allow 16Khz*X sample rates (not technically right, we sample each sr//16000 step)
fixed warning when loading the silero model
the model is now embedded inside the pypi package
move vad paramters to silero.VAD constructor

EDIT: the tests should now be fixed

I still need to plot the accuracy before merging

Note that I'm using a default window size of 1024 at 16Khz (roughly 60ms)


Chore: deprecate NIXTLA_ID_AS_COL or issue warning

What happened + What you expected to happen
We need to make NIXTLA_ID_AS_COL the default behavior or start showing deprecation messages, as the default behavior of NF is now inconsistent with the API of our other libs (SF, MF, HF).
Versions / Dependencies
1.7.7
Reproduction script
n/a
Issue Severity
Low: It annoys or frustrates me.

This is what is done in #1220


Allow User to pick a topic of choice and build his own knowledge graph automatically

Thinking of a tool for CRISP DM/ML

will be implemented in ishmael


Add support for basic alerts for the operator pod

Right now afaik we don't  have any alerts/dashboards for when the operator pod has issues/goes down.
We should start analysing the oob metrics and create alerts/grafana dashboards for making the operator helm production ready.

https://github.com/open-telemetry/opentelemetry-helm-charts/pull/266


Adds Image faker

Connection with issue(s)
Resolve issue #19
Testing and Review Notes
Generates a url to a image random image. The size of image is determined  by width and height paramters. You can choose the image categories related  by the keywords parameter separeted by comma .
Example:
   faker.image.image(width: 1200, height: 900, keywords: 'nature,people');

Result: https://source.unsplash.com/1200x900?nature,people
To Do

[X] double check the original issue to confirm it is fully satisfied
[X] add testing notes and screenshots in PR description to help guide reviewers

@igorgoncalves Thanks a lot! I had a thought that instead of passing a comma separated string for the keywords we could actually accept an List of strings e.g: faker.image.image(width: 1200, height: 900, keywords: ['nature', 'people']);. What do you think?

Make sense to me, I will change that :D

Great, thanks!

The tests fails in Travis because this #22  I create a PR to resolve  #23


Race condition on map

There's a race condition in the observable on the map, while a dirty fix, a 1ms delay will probably fix it.

Was resolved in 0.2.11


Use db pull instead of introspect as default in docs

Note: introspect is used on landing page here https://www.prisma.io/docs/


Support watchosX64 Kotlin Native target

Description
We would like to support the watchosX64 Kotlin Native target.
See the targets supported by Kotlin Native for more details about it.

Checklist

[ ] Add the target in our Multiplatform Gradle plugin, add the target in the integration and the delivery workflows, add badge in README and update the unreleased changelog.
[ ] Close this issue as completed and update tracking ones if present.

We should attach a discussion to this issue before implementing it.

This issue will be resolved by #621.


[Feature Request] Shuffle the (n-1) last addresses of each peer

🚀 Feature Request
Since
https://github.com/libra/libra/commit/5e2d74a88466c01fbad4a53dfbf9610f1dfefce7
we dial the alternate addresses of peers in round-robin fashion.
We could be shuffling the (n-1) last elements of addrs. It does not entail other changes, can be done easily and:

makes the dial behavior less predictable (so there's less incentive to knock off a node because your controlled host is next on the list),
serves as a poor man's load balancer,

It seems like it's a straightforward improvement, because the intended meaning from the configuration is often "here's the guy I want you to dial, but you can try any of the others if it doesn't work"

@huitseeker, @phlip9 : I'm not sure I agree that we need to shuffle at all. The addrs that we chose to connect in a round-robin fashion all belong to a single node (validator/full node). So do we really need to worry about predictability in dial order or load balancing?

Hmm, I guess the load balancing story doesn't really check out, since it's the same validator node behind the endpoints. The predictability also seems unnecessary since the ordering is independent between different validators; an attacker controlling one validator cannot affect the ordering of our dials to another validator. Likewise, an attacker controlling one addr in the list of a validator's listen addrs still has to pass authentication checks when we dial them.

The load balancing, as for the control issues, is of course between network links, not validators.


metadata for 3.0.0 Beta 02

Summarise the changes this Pull Request makes.
metadata for 3.0.0 Beta 02 which will be used to test last changes to add Timer overloads.

Would be good to come up with a way to automate this at some point.


/opt folder and running the app

Hi Arunoda,
first of all a massive thanks for this wonderful program my life would be more dreadful without :) I did the first deployment to my server and all is good, I find my app living in /opt folder, which is exactly what the docs say. My rookie question is, how do I get access to the app? This isn't usual var/htdocs situation that I'm used to. I've specified ROOT_URL, but this just leads to empty folder on my server (yes, I pointed the app to an empty folder in htdocs). I'm bit confused as what to do now, can you help please? Thanks.

What did you do here: "yes, I pointed the app to an empty folder in htdocs" ?

I did set the ROOT URL to a folder in www/htdocs. I also noticed a readme file in the /opt folder, trying to run what it says:
" $ (cd programs/server && npm install)
$ export MONGO_URL='mongodb://user:password@host:port/databasename'
$ export ROOT_URL='http://example.com'
$ export MAIL_URL='smtp://user:password@mailhost:port/'
$ node main.js ".
Now, I try to execute this part ssh'ed to my server, but I'm getting this error:
"npm WARN package.json<EMAIL_ADDRESS>No description
npm WARN package.json<EMAIL_ADDRESS>No repository field.
npm WARN package.json<EMAIL_ADDRESS>No README data"
No clue where to go next, googling for these errors doesn't get lead to anything interesting here..

I dont know if the root_url matters that much but mine is set to the domain that is used on the server. E.g. http://mymupapp.microsoft.com

Does setting up http://mymupapp.microsoft.com as your url in mup.json makes the app automatically available at that url for you then?

Not for me at least. As far as I use it, it is basically just making sure that any links that I create, let's say in an email that is sent to the user, has the correct URL. To make meteor aware under which domain it is working.
Personally I am fronting my mup installs with nginx so any domain translation/forwarding is handled at that level.
But maybe it is used for more, under the hood, I've just never heard of anyone writing a directory path. Usually it is an IP adress or domain.
Do you get any more useful error messages if you check
mup logs -n 400
you do that in the same directory where you run mup deploy
In your mup.json file what have you written for port?
// Configure environment
"env": {
"PORT": 3000,
"ROOT_URL": "http://microsoft.com",
if you have the above you would access it via http://microsoft.com:3000
If you happen to own that domain name and it points to your servers IP adress.
If you haven't pointed a domain to the server you would use your servers IP address instead.
if you access your server terminal via ssh and write
curl localhost:3000
you should get the HTML output from the meteor app in your terminal.

Yes, I've got the port set up correctly, the url I use is a subdomain, as in IPAddress/dev/projectName/, thus cannot access it with port number appended.. It would make sense to use some tool to manage DNS (I'm really no expert here), and point my domain name to the right physical directory where the app lives (which happens to be in the /opt folder, I reckon).
Another thing is that my /opt folder contains my app after deployment, but the app folder contains just main.js, programs folder, readme, server folder and start.json. Really not sure what's going on.
Luckily, mup logs -n 400 throws some errors I've noticed now. It's bunch of variations for the following:
"Error: listen EADDRINUSE
at errnoException (net.js:904:11)
at Server._listen2 (net.js:1042:14)
at listen (net.js:1064:10)
at net.js:1146:9
at dns.js:72:18
at process._tickCallback (node.js:419:13)
error: Forever detected script exited with code: 8
error: Script restart attempt #125".
Quick search says the port number is already in use.
I'd really like to find some good tutorials leading through this, but there seem to be nothing detailed out there, nothing for complete novices like myself. I really enjoy working with Meteor; it delivers with front/backend solutions,  but deployment seems to be quite a tricky thing for myself.

The port u r using seems to be in use
fre 6 feb 2015 kl. 19:27 skrev abtx<EMAIL_ADDRESS>
Yes, I've got the port set up correctly, the url I use is a subdomain, as
in IPAddress/dev/projectName/, thus cannot access it with port number
appended.. It would make sense to use some tool to manage DNS (I'm really
no expert here), and point my domain name to the right physical directory
where the app lives (which happens to be in the /opt folder, I reckon).
Another thing is that my /opt folder contains my app after deployment, but
the app folder contains just main.js, programs folder, readme, server
folder and start.json. Really not sure what's going on.
Luckily, mup logs -n 400 throws some errors I've noticed now. It's bunch
of variations for the following:
"Error: listen EADDRINUSE
at errnoException (net.js:904:11)
at Server._listen2 (net.js:1042:14)
at listen (net.js:1064:10)
at net.js:1146:9
at dns.js:72:18
at process._tickCallback (node.js:419:13)
error: Forever detected script exited with code: 8
error: Script restart attempt #125
https://github.com/arunoda/meteor-up/issues/125".
Quick search says the port number is already in use.
I'd really like to find some good tutorials leading through this, but
there seem to be nothing detailed out there, nothing for complete novices
like myself. I really enjoy working with Meteor; it delivers with
front/backend solutions, but deployment seems to be quite a tricky thing
for myself.
—
Reply to this email directly or view it on GitHub
https://github.com/arunoda/meteor-up/issues/254#issuecomment-73287320.

same question here. still unclear about what happen after a mup deploy. is it live yet? how do we access the app?

The app will be available at the host and port you setup.

@protometa i want to upload it to a subdomain. what should i do? changing the host and root url setting to sub.domain.com doesn't work. the subdomain still shows the default index page.
here's my settings:
{
  "servers": [
    {
      "host": "app.mysite.com",
      "username": "ubuntu",
      "pem": "~/path/to/app.pem"
    }
  ],
  "setupMongo": true,
  "setupNode": true,
  "nodeVersion": "0.10.36",
  "setupPhantom": true,
  "appName": "AppName",
  "app": "/local/path/to/app",
  "env": {
    "ROOT_URL": "http://app.mysite.com/"
  },
  "deployCheckWaitTime": 15
}

i finally managed to get my app up and running. but i still haven't found a way to deploy to a a sub-folder or a sub-domain. any ideas?


DNS for demo server cannot be resolved

Accessing the demo does not work as I cannot resolve the DNS name.
$ host demo.10forge.org
Host demo.10forge.org not found: 3(NXDOMAIN)

Fixed!


[Android] systemNavigationBar reappears when clicked on screen

Use the dummyapp (increment counter) and add
SystemChrome.setEnabledSystemUIMode(SystemUiMode.manual, overlays: [SystemUiOverlay.top]);
in main-Method after runApp
void main() {
  runApp(const MyApp());
  SystemChrome.setEnabledSystemUIMode(SystemUiMode.manual, overlays: [SystemUiOverlay.top]);
}

App starts on android without systemnavigationbar but it reappears after click on screen.
[√] Flutter (Channel stable, 2.8.1, on Microsoft Windows [Version 10.0.19042.1526], locale de-DE)
• Flutter version 2.8.1 at C:\SDKs\flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 77d935af4d (10 weeks ago), 2021-12-16 08:37:33 -0800
• Engine revision 890a5fca2e
• Dart version 2.15.1

Hi @chrisDK1977, Thanks for filing the issue. I was able to reproduce the issue on the stable channel, but it looks like it is fixed on the master channel. The fix should be available on the stable channel in the upcoming releases.

stable
master

code sample
import 'package:flutter/material.dart';
import 'package:flutter/services.dart';

void main() {
  runApp(const MyApp());
  SystemChrome.setEnabledSystemUIMode(SystemUiMode.manual,
      overlays: [SystemUiOverlay.top]);
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  static const String _title = 'Flutter Code Sample';

  @override
  Widget build(BuildContext context) {
    return const MaterialApp(
        title: _title,
        home: Material(
          child: Center(
            child: Text('tap on screen'),
          ),
        ));
  }
}

flutter doctor -v
[✓] Flutter (Channel stable, 2.10.2, on macOS 12.1 21C52 darwin-arm, locale en-GB)
    • Flutter version 2.10.2 at /Users/mahesh/Documents/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 097d3313d8 (2 days ago), 2022-02-18 19:33:08 -0600
    • Engine revision a83ed0e5e3
    • Dart version 2.16.1
    • DevTools version 2.9.2

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.10.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] IntelliJ IDEA Community Edition (version 2021.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 61.2.4
    • Dart plugin version 212.5080.8

[✓] VS Code (version 1.63.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.34.0

[✓] Connected device (3 available)
    • Redmi K20 Pro (mobile) • <IP_ADDRESS>:5555 • android-arm64  • Android 11 (API 30)
    • macOS (desktop)        • macos            • darwin-arm64   • macOS 12.1 21C52 darwin-arm
    • Chrome (web)           • chrome           • web-javascript • Google Chrome 98.0.4758.102

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

[✓] Flutter (Channel master, 2.11.0-0.0.pre.582, on macOS 12.1 21C52 darwin-arm, locale en-GB)
    • Flutter version 2.11.0-0.0.pre.582 at /Users/mahesh/Documents/flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision<PHONE_NUMBER> (5 days ago), 2022-02-16 00:00:11 -0500
    • Engine revision b09cf53269
    • Dart version 2.17.0 (build 2.17.0-105.0.dev)
    • DevTools version 2.10.0

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.10.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] IntelliJ IDEA Community Edition (version 2021.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 61.2.4
    • Dart plugin version 212.5080.8

[✓] VS Code (version 1.63.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.34.0

[✓] Connected device (3 available)
    • Redmi K20 Pro (mobile) • <IP_ADDRESS>:5555 • android-arm64  • Android 11 (API 30)
    • macOS (desktop)        • macos            • darwin-arm64   • macOS 12.1 21C52 darwin-arm
    • Chrome (web)           • chrome           • web-javascript • Google Chrome 98.0.4758.102

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

Closing as already fixed on master.


Evaluate variable default values in determining if blank

If I create a Dockerfile with the following contents
FROM ${VAR:-ignore}

I get a diagnostic error:
dockerfile-utils: base name (${VAR:-ignore}) should not be blank [51]

The :- syntax provides a default value so if $VAR is blank, ignore is used instead, so there's no way for the image name to be blank.

@duckbrain Thank you for bringing this to our attention. I have opened https://github.com/rcjsuen/dockerfile-utils/issues/126 to track this.

@duckbrain Thanks again for the bug report. The latest 0.13.0 release of the language server should not flag this as an error anymore.


Fix formatting for DG

Codecov Report

Merging #205 (de1208d) into master (12b4c80) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #205   +/-   ##
=========================================
  Coverage     73.63%   73.63%
  Complexity      783      783
=========================================
  Files           101      101
  Lines          2401     2401
  Branches        271      271
=========================================
  Hits           1768     1768
  Misses          544      544
  Partials         89       89

Continue to review full report at Codecov.

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


Add a CODEOWNERS file

🚀 feature request
Relevant scope
What is the scope of this request?

[ ] Frontend specs
[ ] Backend specs
[x] Other... Please describe:

Description
CODEOWNERS feature allows to automatically add a list of reviewers to each open pull request.
Describe the solution you'd like
Here are the steps to follow for this task :

create a file named CODEOWNERS into the .github folder
include the following content :

@geromegrignon

Describe alternatives you've considered
No other alternative has been considered.

I would like to take this one.

Hello, thanks for your interest.
Feel free to open a pull request.

Closed by #688


Could it be possible to parse new lines (or markdown) on the hover information?

Recently, ZLS introduced integer literal hover support: https://github.com/zigtools/zls/pull/1911
However, in Kakoune that table is all presented in one line:

This is how it looks in VS Code

The problem might be that we don't parse markdown tables yet.
If true, there should be an Unsupported Markdown tag in the log.
Should be easy to fix, in case you're interested.
I don't have a the latest zls here but the full hover response should be enough to diagnose/fix it

Sorry for the late reply. Thanks for pointing me in the right direction. I will check it out, but I can't promise it will be soon though.

I was quickly looking at the logs, it seems there's no complaint about unsupported markdown tags:
Jun 26 01:39:58.484 DEBG From server zls: {"jsonrpc":"2.0","id":14,"result":{"contents":{"kind":"markdown","value":"| Base | Value      |\n| ---- | ---------- |\n| BIN  | 0b10010111 |\n| OCT  | 0o227      |\n| DEC  | 151        |\n| HEX  | 0x97       |"},"range":{"start":{"line":30,"character":34},"end":{"line":30,"character":37}}}}, module: kak_lsp::language_server_transport:169
Jun 26 01:39:58.485 DEBG To editor `44879`: evaluate-commands -client client0 -verbatim -- lsp-show-hover 31.36 %§| Base | Value      | | ---- | ---------- | | BIN  | 0b10010111 | | OCT  | 0o227      | | DEC  | 151        | | HEX  | 0x97       |§ %§§ %§§, module: kak_lsp::editor_transport:91

It seems that it's just not parsing the \n characters when rendering the hover info.

Thank you!

A bit late to the party.
pulldown_cmark definitely supports tables: Tag::Table. But because they are not part of the CommonMark spec, they need to be enabled explicitly: Parser::new_ext.
And as a side note: Cargo.lock lists v0.9.3. That could be updated to v0.11.0.

yeah we should parse and render tables eventually


Package request: Universal Android Debloater

Project description
Cross-platform GUI written in Rust using ADB to debloat non-rooted android devices. Improve your privacy, the security and battery life of your device.
Metadata

homepage URL:
source URL: https://github.com/0x192/universal-android-debloater
license: gpl-3.0
platforms: linux, darwin

This is duplicate to https://github.com/NixOS/nixpkgs/issues/199619


NFC loading and storing implementation for Achievement Rally

Issue

close #354
close #353

Overview (Required)

The NFC has been configured to trigger animations upon reading, and the results are saved locally.
Commands for verifying NFC

$ adb shell am start -W -a android.nfc.action.NDEF_DISCOVERED -d "https://confsched2023.page.link/e" io.github.droidkaigi.confsched2023.dev

A ResolveDynamicLinksActivity is being added to resolve DynamicLink's DeepLink, and a CompleteAchievementActivity to trigger animations and save them locally.
The Compose part of the CompleteAchievementActivity has been added to a module called Animation.

Movie (Optional)

I've changed some files to refactor. 🙏

@momomomo111
I've refactored some classes. Please check it out 🙏
https://github.com/DroidKaigi/conference-app-2023/pull/1043/files/acd3b622d78081008692f82a58bdcda1d520367b..88f53071af1f108387c7903b07094c209f349904

I don't check if this works and we don't have any tests for this. Please check it by yourself 🙇

I don't check if this works and we don't have any tests for this. Please check it by yourself 🙇

I have actually tried it from A to E and found it works.🙏

I'm not sure if @jmatsu is comfortable with this PR. I believe we can address any issues post-merge. Please let @momomomo111 know if there are any concerns.


Make Centroid (Pick and Place) files available

For some reason I am unable to open the Kicad project to create a centroid file as I am not sure how to get it from github in a non text file format. Kicad is new to me so this may be my issue, but I still think it is a good idea to upload some centroid/pickandplace files in the form of a csv file for easily getting it pre soldered from pcb manufacturers like PCBWay or JLPCB.

For some reason when I downloaded the whole project natively the kicad files seemed to be corrupted or something and would not open in kicad, however when I downloaded on a separate device and moved it over via usb drive they seem to be fine. I have no idea. I still think having the centroid files in a csv on the repository would be nice though since it may circumvent any similar witchcraft in the future and simply provide greater ease of access.


Inconsistent "BadUnexpectedError" from OpenSecureChannel

Before I start I just wanted to say thank you for making this library, this is a much more accessible alternative to the official repo.  This issue is the only thing currently giving me pause.
client.OpenSecureChannel(MessageSecurityMode.None, SecurityPolicy.None, null);
Irregularly returns BadUnexpectedError when connecting to Omron NJ PLC embedded OPC UA Server. I say irregularly because it seems to depend on the configuration the code is run in - the issue almost never occurs in debug, sometimes in release, and quite often when running in docker.
Any insight here would be helpful, I'm thinking this might be a race condition of some kind

I can't reproduce this locally. Can you take a Wireshark capture of the connection attempt when it fails?

BadUnexpectedErrorCap.zip
Occurs 8 times before finally connecting

It looks like the TCP connection is aborted on the client's side before a response is received. Can you try running TestServer and TestClient together?

TestClient&Server.zip
FYI Platform is Ubuntu 24.04, .NET 8.0.110

I don't see any disconnects in this capture, did it work fine?
If so, it may be related to Ubuntu or the docker setup. The initial connection with security disabled is blocking, so I don't see anything that would cause a timing issue. The only error I can find is the TCP connection gets cut on the client side and the client doesn't receive any response from the server.

I’m happy to dig into this further, would you be able to point out where this would be most likely to occur and any logging that could be done to track this down?

TestClient: https://github.com/nauful/LibUA/blob/0794efda13076ad0bbf572a6dd2e001d39535f41/NET Core/TestClient/Program.cs#L137
This is the method in the UA client: https://github.com/nauful/LibUA/blob/0794efda13076ad0bbf572a6dd2e001d39535f41/NET Core/LibUA/Client.cs#L134
Incoming bytes are consumed here, perhaps check this for failures: https://github.com/nauful/LibUA/blob/0794efda13076ad0bbf572a6dd2e001d39535f41/NET Core/LibUA/Client.cs#L1382
If a message fails to process due to an error, this will trigger: https://github.com/nauful/LibUA/blob/0794efda13076ad0bbf572a6dd2e001d39535f41/NET Core/LibUA/Client.cs#L1207
If your environment closes the connection, this will trigger: https://github.com/nauful/LibUA/blob/0794efda13076ad0bbf572a6dd2e001d39535f41/NET Core/LibUA/Client.cs#L1127

Closed as not reproducible. You can reopen if you find anything I can help with.


space issue in right side in SmartAdmin theme v1.5.2

in smart admin theme if we open any grid after that open any modal form and after that open any inside modal form then its append space in right side..if we open again and again this modal form then right side space will be increased.
this is because of bootstrap.min.js
in previous version of bootstrap there is
c.prototype.resetScrollbar = function () { this.$body.css("padding-right", "")
but now in SmartAdmin - v1.5.2 - 2015-04-23 bootstrap.min.js
c.prototype.resetScrollbar = function () { this.$body.css("padding-right", this.originalBodyPad)
so is there any issue with originalBodypad ??

Bootstrap does not include any such theme. We cannot help you with it. Please contact the SmartAdmin theme's actual vendor for support. Good luck.


Count on 5' libraries gives all cells ummapped

Greetings,
When @Adeebius and I run CITE-seq-Count (v 1.4.2) on a 5' library with the following parameters:
CITE-seq-Count -R1 <Route to R1 Fastq GZ file>
                                -R2 <Route to R2 Fastq GZ file>
                                -t  <Route to Tag file>
                                -cbf 1
                                -cbl 16
                                -umif 17
                                -umil 28
                                -cells 1000000
                                --max-error 1
                                -o <Output directory>

The file run_report.yaml yields that the percentage of unmapped cells is 100%:
Date: 2019-08-20
Running time: 18.0 minutes, 0.8353 seconds
CITE-seq-Count Version: 1.4.2
Reads processed: 10775807
Percentage mapped: 0
Percentage unmapped: 100
Uncorrected cells: 10
Correction:
        Cell barcodes collapsing threshold: 1
        Cell barcodes corrected: 0
        UMI collapsing threshold: 2
        UMIs corrected: 1206489
Run parameters:
        Read1_filename: <Route to R1 Fastq GZ file>
        Read2_filename: <Route to R2 Fastq GZ file>
        Cell barcode:
                First position: 1
                Last position: 16
        UMI barcode:
                First position: 17
                Last position: 28
        Expected cells: 1000000
        Tags max errors: 1
        Start trim: 0

However, when we run CITE-seq-Count for a 3' sample, the percentage of ummapped cells drops to 8%.
Can you please advise on what parameters we should adjust so that we can obtain a lower percentage of unmapped cells for 5' samples?
Thank you,

Hello @manugarciaquismondo
I would need a few reads to look at them and check out how the structure is. Could you send me 10'000 reads?

Thank you for your e-mail. If I enable --sliding-window the count is done properly; I think that this issue can be now closed.

Ok. Maybe there is a sequence before the actual tag. The sliding window would catch that but you could also find a fixed length to trim from the start with the --start-trim option.
I'm closing it then.
Best wishes


Cannot use 'JSON.SET' with redis-cluster.

I'm using AWS Redis Elasticache with cluster mode on, when trying to run JSON.SET/JSON.GET, I get "NotSupportedException" as you can see below:
Fatal error: Uncaught Predis\NotSupportedException: Cannot use 'JSON.SET' with redis-cluster. in predis-1.1.1/src/Connection/Aggregate/RedisCluster.php:380 Stack trace: #0
predis-1.1.1/src/Connection/Aggregate/RedisCluster.php(550): Predis\Connection\Aggregate\RedisCluster->getConnection() #1
predis-1.1.1/src/Connection/Aggregate/RedisCluster.php(593): Predis\Connection\Aggregate\RedisCluster->retryCommandOnFailure() #2
predis-1.1.1/src/Client.php(294): Predis\Connection\Aggregate\RedisCluster->executeCommand() #3

I'm running Predis 1.1.1(due to compatibility with 3rd party softwares)
This is my code:
    require('predis-1.1.1/autoload.php');
    try{
        $redis = new Predis\Client(['xxxxx.clustercfg.region.cache.amazonaws.com:6379'],['cluster'=>'redis', 'scheme' => 'tcp', 'port' => 6379]);
    }
    catch(Exception $ex){
        error_log("Error => ".$ex->getMessage());
    }

  $redis->executeRaw(['JSON.SET','teste123abc','.','[{"name":"nick","years":"20"}]']));
 //This same command works in redis with cluster mode disabled

Isn't this supposed to work?
These commands works properly:
  $redis->set('MyExampleTestKey1','testing'); // this works fine
  $redis->executeRaw(['SET', 'MyExampleTestKey2', 'testing']); // this works fine

What can I do to fix this?

@ederjbezerra Hello! Compatibility with Redis cluster for JSON.SET command will be available in next minor release (v2.2). But I'm not sure if v1.x is still supported? @tillkruss

When using the Redis Search Coordinator, the Search commands should be available too.
See: https://forum.redis.com/t/redis-cluster-proxy-with-redisjason-redissearch-is-possible/2375
See: https://github.com/RediSearch/RediSearch/tree/v2.6.9/coord
How can I whitelist commands locally for Predis to say they're allowed to be used in a cluster?

When using the Redis Search Coordinator, the Search commands should be available too.
See: https://forum.redis.com/t/redis-cluster-proxy-with-redisjason-redissearch-is-possible/2375 See: https://github.com/RediSearch/RediSearch/tree/v2.6.9/coord
How can I whitelist commands locally for Predis to say they're allowed to be used in a cluster?

Hey! You can specify commands here, within this array:
https://github.com/predis/predis/blob/v2.x/src/Cluster/ClusterStrategy.php#L41
For now Redis modules command are not available in Cluster context. But it's on our roadmap.

When using the Redis Search Coordinator, the Search commands should be available too.
See: https://forum.redis.com/t/redis-cluster-proxy-with-redisjason-redissearch-is-possible/2375 See: https://github.com/RediSearch/RediSearch/tree/v2.6.9/coord
How can I whitelist commands locally for Predis to say they're allowed to be used in a cluster?

Hey! You can specify commands here, within this array:
https://github.com/predis/predis/blob/v2.x/src/Cluster/ClusterStrategy.php#L41
For now Redis modules command are not available in Cluster context. But it's on our roadmap.

Niiiiice. Is there any way to override that array at runtime? I'd rather not have to hack away at my vendor folder if possible. I'm in a Symfony / SncBundle context if that helps at all?

@schodemeiss You should specify it without dot and uppercase, like this: JSONSET JSONGET etc.

@schodemeiss Nope, it's not publicly accessible