When does this control fire?

At what point does this get the URL?   Can I cause it to be delayed?
I am using the OnChange event to set a VAR.  Is that the correct way to get the URL?
I have used a label and just set its text property to CurrentOrgUrl.cdsEnvironmentUrl.
My test app seems to get/show the url most of the time, but not all the time so I am trying to figure out if it is related to the browser, my test app or the control.
This control is super helpful in being able to open UCI records in a tab!!!
It should be built into PA out of the box!
-Art

Okay, so after chasing my tail a bit.  I think I have this figured out.
I built a URL Test app  just to test this one control.  Here's what I found:

If I added one CDS entity to my app, but did not connect it to a gallery on a screen, the URL is blank

I tried adding an entity and in the app start code, adding its records to a local collection.  However, the URL wasn't returned.

Next, I add the Organizations entity to a gallery. The URL populated in the control and VAR via onChange code.  (I used "Organizations" entity as it only returns one record)

So I ran another test in my calendar app.  Here I modified my URLControl onChange code to:
Set(_CurrentOrgURL, _CurrentOrgURL & " " &CurrentEnvOrgUrl.cdsEnvironmentUrl)
When the app runs I get two or three copies of the URL in my VAR. Not sure why it's sometimes two vs three. This app has three galleries that pull data from three CDS entities.
Browsers Tested: Chrome, New Edge, Edge Dev Channel, FireFox, FireFox Developer, Brave and Vivaldi.  All worked with both tested apps.
I would be interested in your feedback on what is the minimum configuration requirements to be able to get a URL.
Does the fact that the control runs per connected entity cause any performance issue?
Hope this helps
-Art


Compatability with Cloudbees folder plugin

Hi there,
We use your build monitor plugin and love it, but we're having issues with downstream compatibility with the folders plugin.
What we would like to do is create a folder which contains a bunch of build monitors, so that they do not clutter our project's top level view. However, once we are in the folder and go to create a build monitor, we do not see any of our jobs and cannot write a regex to get them, either.
Thanks,
Morgan

Hi Morgan and thanks for your kind words! I'm glad that you find this little plugin useful :smile:
The functionality you're thinking of is not (yet) available.
I'll be looking into making Build Monitor support Cloudbees Folder more fully as part of #130, and there's a good chance that I'll be able to work on your requirement as well (of course pull requests are also very welcome!).
In the meantime though you might want to give the Team Views Plugin a try. It allows you to define team areas in Jenkins and give each team its Build Monitor(s). Alternatively you could create a "Build Monitors" team and put your different Build Monitor views in there, which then prevents the proliferation of tabs on the main Jenkins page.
Hope this helps!

Closing as duplicate of #130


PD Milestone: test setup and iteration 1

Background
On #26 we identified an opportunity to test if we could use Milestones on GitHub Projects to identify the levels of PD work as well as min/max number of coursework.
Some background information about how milestones work at CYF:

Milestones Descriptions

Acceptance criteria

Create my coursework board
2 Batch clone 1 module (use Kamino) OR per competency
Define milestones
1. Key and Mandatory coursework only
2. 3 levels: Level 1, Level 2, Level 3 - to be defined with SIFA
3. Dealbreaker: is "Key" the dealbreaker?
Define min/max count of coursework per Level and per timeframe
Make notes of any constraints/issues in the process

Owner: Lara and Karen
Stakeholder: 1 volunteer from PD Syllabus and 1 from Coursereview
Approver: Sally
Technical notes
GitHub Milestone API
Must have info in bold
id integer or string yes The ID or URL-encoded path of the project owned by the authenticated user
**title string yes The title of a milestone**
description string no The description of the milestone
**due_date string no The due date of the milestone (YYYY-MM-DD)**
start_date string no The start date of the milestone (YYYY-MM-DD)

The coursework planner has been cloned. We need to think about the levels, @LaraHuzjan

Lara and I created the first version for Communication/English Language.
We need to validate it with Sally.

This was reviewed with @SallyMcGrath today and is a good starting point and seems to be going in the right direction. I will raise the tickets discussed in the meeting and link them back to this one as well as add it to the same milestone.

👋 I'm looking for the resources -- the boards, repos, issue, spreadsheets etc related to this project. Is there an obvious place I should be looking?

We are still finalising this work (ticket 47 was the last one). I will add details to your ticket if Lara and I are comfortable with it after our meeting tomorrow.
Thanks for raising this.


Update fade-edge mixin

transparent can't be used as a color value in a linear-gradient in iOS Safari, so it has been replaced with a rgba.
Also added a color param to the mixin.


fix: Fix storing same named files from different pkgs in memory plugin

The memory plugin was not correctly writing tarballs that have the same name but are from different packages and have different content.
I noticed this when using the memory plugin and installing json-buffer and @types/json-buffer.
This resulted in two things:
First verdaccio memory-plugin was logging a warning:
this package is already present

and secondly, yarn was throwing an error on subsequent installs:
error http://localhost:7460/json-buffer/-/json-buffer-3.0.0.tgz: Integrity check failed for "json-buffer" (computed integrity doesn't match our records, got "sha512-3YP80IxxFJB4b5tYC2SUPwkg0XQLiu0nWvhRgEatgjf+29IcWO9X1k8xRv5DGssJ/lCrjYTjQPcobJr2yWIVuQ== sha1-hcH/DwlI/BWYENS1vjW/jCCHX2Q=")

The underlying issue is that both of these packages have the tarball named json-buffer-3.0.0.tgz, but obviously with different content. So yarn received the wrong tarball, because only one was stored for both of these packages.
For the review, it is the easiest to hide whitespace changes.
If this is okay, I would also like to backport this to 5.x. Is this okay with you?

I love mornings 🫠 PR with new testS added, I will check it when I back from 🏖. Thanks in advance @danez

LGTM, could you add the changeset file (please check contributing.md about how to) then we can merge. About 5.x, yes, you can find same plugin https://github.com/verdaccio/monorepo for 5.x

Thanks for the fast review, I added the changeset and created the backport.
https://github.com/verdaccio/monorepo/pull/571

All looks good, unfortunatelly I cannot release until review if the tokens are still valid  for the automated CI pipeline (https://www.theregister.com/2022/05/27/github_publishes_a_post_mortem/) which I guess ( and I hope not) that will take few days. Thanks for fix this.

@danez It was published already.


Build fails on task ignite-api:signArchives

When attempting to build release 0.6.0 using gradlew build on Windows with AdoptOpenJDK 8 and 16, I'm getting the following error:
> Task :ignite-api:signArchives FAILED

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':ignite-api:signArchives'.
> Cannot perform signing task ':ignite-api:signArchives' because it has no configured signatory

I suspect that this has something to do with signing keys not being set up locally. I'm expecting the main branch to either build without additional actions, or the README to contain information about how to set this up in the building section.

Handled and closed after 42 minutes. I'm impressed.


1 test failed. week year:week numbers 2012/2013 (2446.7)

1 test failed. week year:week numbers 2012/2013 (2446.7)
Client info
Date String   : Fri Jan 01 2016 12:48:01 GMT-0800 (PST)
Locale String : 1/1/2016, 12:48:01 PM
Offset        : 480
User Agent    : Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_2) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/47.0.2526.106 Safari/537.36

Date String   : Fri Jan 01 2016 11:50:07 GMT-0800 (PST)
Locale String : January 1, 2016 at 11:50:07 AM PST
Offset        : 480
User Agent    : Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_2) AppleWebKit/601.3.9 (KHTML, like Gecko) Version/9.0.2 Safari/601.3.9

====
week year:week numbers 2012/2013 (2446.7)
// Expected 53
// Actual   52
52 === 53

Also, looks like expected/actual got reversed?

Also, looks like expected/actual got reversed?

Dup #2998


Search in table works only with text

It's not possible to find a date, a year or a value with the search functionality in table.
Bug or feature ?

I cannot reproduce this issue in the latest version on dev.visualize.admin.ch anymore.

Another sample here
The table

Search 326 (or 326,9 or 326.9)

If there is a 326 in a text (in the Text ID) it will find it, but not in a numeric or date value.

Dates should work but numbers not. This is intentional since the table search index would potentially get too big.

OK


A2A: Adding new properties and correcting few fields for zone to zone replication

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

/azp run automation - sdk

Can one of the admins verify this patch?

@yungezz : Can you please review the PR

hi @zikalino @qiaozha will it be sdk breaking change to change x-ms-disclaiminator?

pending on sdk confirmaiton on a potential breaking change

hi @zikalino @qiaozha will it be sdk breaking change to change x-ms-disclaiminator?

Yes. For user who use the old subclass discriminator value now might be unavailable.

hi @zikalino @qiaozha will it be sdk breaking change to change x-ms-disclaiminator?

Yes. For user who use the old subclass discriminator value now might be unavailable.

We have added the class RecoveryPlanA2ADetails recently, for supporting zone to zone replication in azure site recovery. , as part of PR: https://github.com/Azure/azure-rest-api-specs/pull/7985/files
-We have not declared GA for the zone to zone feature.

So we dont have any current users who are using this field, and for whom this will be a breaking change.

more info from @ayfathim about the change
[1:06 PM] Ayesha Fathima
    about the change in discriminator value
[1:06 PM] Ayesha Fathima

We have added the class RecoveryPlanA2ADetails recently, for supporting zone to zone replication in azure site recovery. , as part of PR: https://github.com/Azure/azure-rest-api-specs/pull/7985/files
-We have not declared GA for the zone to zone feature.

So we dont have any current users who are using this field, and for whom this will be a breaking change.

hi @NullMDR @isra-fel  could you pls help to look at the .NET CI task failure? thanks


Slurm module

We are currently accessing information from slurm in the results/init.py file. It would be better to include this as a new module instead.
I propose we add a module tcutility.slurm which will contain tools to use slurm, such as detecting if we can use slurm, running squeue and maybe more stuff later on. This module can then be used to detect if a job is pending, running, cancelled, etc. It can also be used to easily set dependencies between jobs using the slurmjobids.

Yes, one of the functions is for checking exactly that. See tcutility.slurm.has_slurm.


Add local pkg-cache for slm project

With the current compiler implementation, the compiler will create pre-compiled packages in $STANZA/pkg-cache by default (where $STANZA is the stanza installation directory).
If you attempt to use the #env-var macro to include the VERSION into in a build, it will work the first time. Then it will cache the package where #env-var is used in pkg-cache and when you try to increment the version, it will still refer to the old versison.
We have a .slm/pkgs directory which also creates pre-compiled packages. slm clean will remove the packages from here but it can't do anything about the packages in $STANZA/pkg-cache.
This was REALLY CONFUSING.
I propose that we add a pkg-cache: ".slm/pkg-cache" statement in the default stanza.proj for the project created by slm init. I further suggest that we update the slm clean command to remove the pre-computed packages in .slm/pkgs and .slm/pkg-cache. This will cause a true clean slate for the build.
Note - just passing -pkg-cache ./pkg-cache as a commandline option to stanza build was not sufficient due to what I think is a bug in stanza's pkg-cache setting resolution.

Tracking in Linear


Support Rails 5.1

Let's support Rails 5.1.

Coverage remained the same at 90.455% when pulling 97150cf44ddd61b806a749ab767e99c8cd64149b on support-rails-5.1 into a167d077ba35bea203cc5afbc3f62dc568b92c67 on master.

Coverage remained the same at 90.455% when pulling 97150cf44ddd61b806a749ab767e99c8cd64149b on support-rails-5.1 into a167d077ba35bea203cc5afbc3f62dc568b92c67 on master.

Coverage remained the same at 90.455% when pulling 97150cf44ddd61b806a749ab767e99c8cd64149b on support-rails-5.1 into a167d077ba35bea203cc5afbc3f62dc568b92c67 on master.

Coverage remained the same at 90.455% when pulling 97150cf44ddd61b806a749ab767e99c8cd64149b on support-rails-5.1 into a167d077ba35bea203cc5afbc3f62dc568b92c67 on master.

Coverage remained the same at 90.455% when pulling 1b010ab6763311304c6708bc8e541d92175dc172 on support-rails-5.1 into 89b62ee073565f842faf7b20c40710d847a700cb on master.

Coverage remained the same at 90.455% when pulling 1b010ab6763311304c6708bc8e541d92175dc172 on support-rails-5.1 into 89b62ee073565f842faf7b20c40710d847a700cb on master.


Use Docker to compile

create or find a docker image of moddableSDK enviromnent
write npm script

Added devcontainer for VSCode in 6995046.


Add current State to HandleUpdate.

Since Channel.State() can not be called from an Update handler and
having the current state in the ChannelUpdate does not make sense,
i moved added it as argument to HandleUpate.
Closes #15

@ggwpez Why do you think it does not make sense to add the CurrentState to the ChannelUpdate struct ?
From the framework user point of view, I see that it is easy to propagate the CurrentState and ProposedState of the channel to the module that will have to decide whether to accept/reject the update.

@ggwpez please also have a look at Mano's comment. What do you think?

@manoranjith It does make sense to do it, but it would imply many changes in the framework code because.
We internally use the struct and pass it around a lot. ChannelUpdate would then also not fulfill the Encode/Decode pattern anymore since the CurrentState should never be sent over the wire.

@manoranjith It does make sense to do it, but it would imply many changes in the framework code because.
We internally use the struct and pass it around a lot. ChannelUpdate would then also not fulfill the Encode/Decode pattern anymore since the CurrentState should never be sent over the wire.

Okay.
If making it a part of the ChannelUpdate would be inconsistent with the current design of Encoder/Decoder pattern. I think, we could still comply with Encoder/Decoder pattern by excluding the current state when Encoding/Decoding and add it by reading from state machine after Decoding. But this would mean redundantly storing the current state and passing it around.
Since the only purpose of this change is to make the current state available in HandleUpdate function, I think the current solutions of passing it as an additional parameter makes more sense.


Test integration with raiden without releasing a pypi package

@rakanalh has some ideas around this.  I'll ask and document them.

The idea behind this is as simple as doing exactly like: https://github.com/raiden-network/raiden/pull/3646/
Provided that Raiden client is a user of the contracts, if nothing breaks, then contracts are released. Otherwise,

Any breaking changes in the contracts need some rework in the client so first step is to adapt the client to changes in the contracts
Once the client is changed, then the contract changes can be tested agains the collection of integration tests we have to make sure no bugs are introduced by the contracts.

So first file a PR in raiden and then release?

pretty much... but the requirements for raiden-contracts has to point to the "latest" master... which means that the PR is not mergable but creating the PR would trigger Raiden's integration tests.

Is there a way to point requirements.txt to a particular git commit?

Sure, you can refer to: pip's VCS support

I'd first make a release and file a mergeable PR to raiden.  Because this way, I have to prepare only one PR.  By the proposal, two PRs need to be prepared.  One with the VCS version and the other with the PyPI release.


bug: lineTo() removes previous line

I was just searching for some time to find out why my lines weren't showing up on screen during development of a path creation tool. I didn't expect pixi to erase my lines just by drawing them, but it does it for some reason;
When drawing a line with the Graphics object with lineTo(x, y)  and after that draw another line with lineTo(x, y) and the same x and y values, none of both lines will be drawn!!!
I understand drawing a line to to the same spot is not very useful, but during debug process this can happen, for example for quick testing, like happened to me today.
It's confusing if, like in my case, no line is drawn at all if this happens. I would never expect drawing two lines would result in no line at all.
So this is drawing a line:
g.moveTo(0, 0);
g.lineTo(200, 200);

But this is not drawing anything:
g.moveTo(0, 0);
g.lineTo(200, 200);
g.lineTo(200, 200);

Expected Behavior
The second lineTo() should be ignored by pixi, because it's having no length. But the first line should not be removed.
Current Behavior
None of the lines are drawn when there are more following lineTos with same values
Possible Solution
Ignore lineTo's if they have zero length but don't change previous lines.
Steps to Reproduce
I've just created this pen with an example: https://codepen.io/Friksel/pen/OZjpBg
The first line is removed by the second lineTo.
Just remove the second lineTo in the example and the first line is visible again
Environment

pixi.js version: 4.7.3
Browser: Chrome, Firefox, Edge etc.
Device: Tested on Windows 10

I understand drawing a line to to the same spot is not very useful, but during debug process this can happen, for example for quick testing, like happened to me today.

There are many operatsions that cause PIXI.Graphics to lose screws. I want to port something better from one of existing vector libraries :)
Here is what i did with repeated lineTo and moveTo's one of those libs, that one is used by Google Chrome canvas2d context: https://bugs.chromium.org/p/skia/issues/detail?id=7792

Thanks @ivanpopelyshev . I see you have been busy up there.


Add e2e config/env debugging helper

What does this PR do?

Adds new invoke task that will check your tools and configuration and try to let you know if anything is wrong
aws-vault exec sso-agent-sandbox-account-admin -- inv new-e2e-tests.debug

Current checks

pulumi installed
aws cli installed with right version
aws-vault installed
aws creds available+valid
correct aws sso profile name
ssh-agent is running
key pair name and path are configured
configured keys loaded into ssh-agent
configured keys match keys in EC2
finds keys in ~/.ssh that match keys in EC2
ssh key is in ascii

Motivation

Additional Notes

Other common issues that I'm not sure how to validate are

appgate running/connected
pulumi plugins updated/synced
user/team does not have access to AWS

Possible Drawbacks / Trade-offs

currently shells out to openssl and ssh-keygen to get key fingerprints. Tested on debian and Windows (openssl coming from git-for-windows), but it could be made more platform agnostic by using a python crypto package.
Describe how to test/QA your changes

Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

I love this ! What about moving it to the inv setup task in test-infra-definitions though ? Could be a --debug mode, part of these steps are covered in the setup, for instance pulumi is installed there, ideally if we spot something is missing we could fix it if we can.
I understand it is confusing to have things across both repos, ideally I'd like to move tasks in test-infra-definitions to conda commands that we can install with go install <command> and execute from wherever.

move to https://github.com/DataDog/test-infra-definitions/pull/592


Add node detail preview panel based on selected node

Todo:

add container for graph
add ability to zoom in / out of graph

idk why all the merge changes are showing up in the "files changes" section. Maybe it's worth recreating this PR now that the branch should be in a healthy state?


convience script for starting manager

you used to be able to just start manager.py and it would build and start it, now that they are separated into build.py and manager.py it is more convenient to have a single script to run when killing/starting manager inside the existing tmux session

launch_openpilot.sh is the convenient, single script you can run in tmux after killing manager


Add timeout to genericItems in Appwrapper

This feature could be modeled after Kubernetes batchv1 job feature called activedeadlineseconds. The idea is to check the duration when appwrapper changes to Running state.  Once duration == userTimeout, we kill all the items in the appwrapper and generated a new state called FailedDeadlineExceeded.

Adding @dmatch01 to this issue

For reference timeout spec: https://github.com/dmatch01/multi-cluster-app-dispatcher/blob/carbon-demo-donot-merge/pkg/apis/controller/v1alpha1/schedulingspec.go

For reference timeout spec: https://github.com/dmatch01/multi-cluster-app-dispatcher/blob/carbon-demo-donot-merge/pkg/apis/controller/v1alpha1/schedulingspec.go

For timeout feature I think only is needed:
type RuntimeDurationSpec struct {
	Expected metav1.Duration `json:"expected,omitempty"`
	Limit    metav1.Duration `json:"limit,omitempty"`
}

your thoughts @dmatch01 ?

@dmatch01 Few questions:

How will the timeout work for the items that are not gang scheduled? if the timeout is at the appwrapper level does it get applied to all the objects like deployments, podgroups, jobs, etc? could there be a scenario where some items run more than other items? suppose an appwrapper has 10 items, 5 of them start in the first half and 5 of them in the second half, the first group ran for a longer duration than the second, if the timeout is set at appwrapper level.
Do we provide another configuration for graceful termination after the deadline is exceeded?


fix README

add brew update

I don't know think is necessary.. Unless it is otherwise stated in a README you should always be using the current version of any related tool.

If you run brew doctor without having brew updated in the last 24 hours, you get this warning:
Warning: Your Homebrew is outdated
You haven't updated for at least 24 hours, this is a long time in brewland!

so I think running brew update frequently is recommended strongly by brew.


Updates PHP DocBlocks for better compatibility with IDEs

The update intended to simplify Kahlan usage in IDEs that rely on DocBlock annotations (e.g. @return etc.)
No functional changes have been made.

:+1:

Good job!


Type renames

Renaming some of the sequence types.

Codecov Report

Merging #2 into api-redux will increase coverage by 0.03%.
The diff coverage is 91.28%.

@@              Coverage Diff              @@
##           api-redux       #2      +/-   ##
=============================================
+ Coverage      84.02%   84.05%   +0.03%
=============================================
  Files             42       44       +2
  Lines           3036     3048      +12
=============================================
+ Hits            2551     2562      +11
- Misses           485      486       +1

Impacted Files
Coverage Δ

src/twobit/record.jl
75.75% <ø> (ø)
:arrow_up:

src/composition.jl
83.82% <ø> (ø)
:arrow_up:

src/search/exact.jl
91.81% <ø> (ø)
:arrow_up:

...tablesequences/site_counting/site_types/certain.jl
0% <ø> (ø)

src/fasta/record.jl
77.08% <ø> (ø)
:arrow_up:

src/search/approx.jl
94.62% <ø> (ø)
:arrow_up:

src/search/pwm.jl
94.23% <ø> (+0.96%)
:arrow_up:

src/minhash.jl
69.04% <ø> (ø)
:arrow_up:

src/BioSequences.jl
0% <ø> (ø)
:arrow_up:

src/demultiplexer.jl
93.25% <ø> (ø)
:arrow_up:

... and 28 more

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 8bffb31...acc0e0d. Read the comment docs.


Implemented GitHub Enterprise Authentication

This PR implements support for using GitHub Enterprise as an authentication back end.

@mistercrunch Anything special I need to do to get this looked at? No rush, just want to make sure I didn't miss a step.

LGTM!


⚠ T:Riza Api has degraded performance

In f79d802, T:Riza Api (https://infra.triza.dev/api) experienced degraded performance:

HTTP code: 502
Response time: 21165 ms

Resolved: T:Riza Api performance has improved in 1e66e5f.


Hello Vavr 0.9.0

Now, the infrastructure and dependent projects needs to be fixed...
The new version is 0.9.0

bye-bye


Make debug log entries (from Vespa log) go to debug log method

@mpolden please review.

Replaced by https://github.com/vespa-engine/vespa/pull/10661


Repo Creation Request: colcon_hardware_acceleration (PENDING REPO)

Request Description
The Hardware Acceleration Working Group @ros-acceleration has made a repository for introducing some colcon verbs for hardware acceleration, mostly authored by @vmayoral.
Could the package be transferred to the colcon org, or a new one made that the extensions can be slated for release?
Is transferring the repo or creating a new one preferred? I think either way @vmayoral has to handle the transfer process...
The work to get it ready for release is tracked in this issue: https://github.com/ros-acceleration/colcon-acceleration/issues/10
Additional Notes
There's a pending draft REP-2008 that this extension package is a part of the implementation for, but I'm not sure if the pending status of the REP will factor into the decision to pull this package into the colcon org or slate it for release.

Overall, I think this extension is in fairly good shape. I'd like feedback from other colcon folks about the name colcon-acceleration, as the name isn't specific to Hardware Acceleration so might be confusing.
Some specific points:

[x] Licensing looks good. Apache 2.0.
[x] No ridiculous overhead for additional entry points - only a single verb.
[ ] Should advertise a colcon_core.extension_point entry point for the colcon_acceleration.subverb extension point.
[ ] Should either drop the tests_require or add SOME tests. Copyright checks would be highly recommended. It looks like it might be "too late" for flake8 without some substantial investment, so probably best to drop the flake8 section from the setup.py if it's not going to get used.
[ ] The stdeb.cfg should be updated to align with other colcon repositories (namely the Suite list).
[ ] The README.rst should be dropped if a README.md is used.

[ ] Should advertise a colcon_core.extension_point entry point for the colcon_acceleration.subverb extension point.

https://github.com/ros-acceleration/colcon-acceleration/commit/48efc96f8a3594bc68855f5a46fb111b08927d0f fullfills it.

Should either drop the tests_require or add SOME tests. Copyright checks would be highly recommended. It looks like it might be "too late" for flake8 without some substantial investment, so probably best to drop the flake8 section from the setup.py if it's not going to get used.

https://github.com/ros-acceleration/colcon-acceleration/commit/3e69285c0145ff5c4b6a71602d0986dbcaee7020
https://github.com/ros-acceleration/colcon-acceleration/commit/c03e8ca916113981acc6188cb3bbb65ad2db5780

Get us to to some basic tests for the package. Let me know @cottsay if you're missing anything else.

https://github.com/ros-acceleration/colcon-acceleration/commit/6d4ddbaaee5e66d67743e8e299c35e3f547507b5 cleans up flake8 deps which aren't needed for now and adds publish-python.yaml file.

Ok, looks ok to me. I think once this gets integrated, I'll open an issue for refinements.

@cottsay asked for feedback on the naming internally, so...
It seems to me that just "colcon-acceleration" is pretty generic. I know that's what's been used in the "ros-acceleration" branding, but did you guys consider something more specific like "hardware acceleration"? Even that is kind of vague, because if I hear "colcon-acceleration" my first thoughts are nothing to do with enabling hardware acceleration for the software being built. Instead, it seems like it should accelerate some part of colcon or the build process...
I'm not 100% sure what the role of colcon is in enabling this functionality either. Perhaps the question should be "does this belong in colcon"? For instance, CLI things like vcs and rosdep are not part of colcon and I think that's a good thing, to be honest. Maybe this should just be its own set of CLI scripts that don't use colcon? Maybe that doesn't make sense given the technical requirements, but I'm not up to speed on those.
Not a blocker, but I'm +0 leaning towards -1 on the name due to how generic it is.

I also have been hung up on the "Acceleration" name in the context of the ros2-gbp team name. I would recommend something like hardware_acceleration or hw_accleration or even hwa but ultimately left it as is with acceleration_wg (where wg == working group) because ros-acceleration is the name of the upstream org.
For colcon, which is not exclusively a ROS community project I think there's even more weight behind specificity since there is no immediate mapping between the colcon extension and ROS organization via the team / working group name.

I'm open to renaming from colcon-acceleration to colcon-hardware-acceleration if that gets us moving forward. Let me know if that'll do.

I think with rename we're good to go
There's small naming changes in the README that can be done, but I'll mention it in the issue. (:

All right, thanks everyone for the comments:

[x] rename repo to colcon-hardware-acceleration

[x] Make appropriate changes to adapt source code 3a1b3e18e225c0acf97c1bac5e6d4bc4cce81533

[x] Replace ROS workspace mentions for colcon workspace ones in README bc3ca604dd0b4fda2d1b7782da5705ccd069f049
[x] released 0.6.0 989a451fb54a60dffdca23b708563d08df317b4f
[x] created a new tag for it

I kept the acceleration verb in https://github.com/ros-acceleration/colcon-hardware-acceleration/blob/main/setup.cfg#L47 because I'd like to type as little as possible but if we really need to change this now (we can always go change the verb afterwards if another "acceleration" use case appears), I'd suggest hwacceleration instead of hardware_acceleration.

I wonder if hwa is possible to use if acceleration is an issue?

I wonder if hwa is possible to use if acceleration is an issue?

Mmm let me check internally and think about it. Before I dig too deep, is acceleration an issue?

I'm not sure yet, will need to see what the team thinks.

Mmm let me check internally and think about it. Before I dig too deep, is acceleration an issue?

I have a strong preference for hwa over acceleration for the verb name. I find it both more specific and easier to type at the cost of being slightly more opaque ("what's a Hwah?") but that's a preference for hwa not an outright objection to acceleration as a colcon verb.

If there's no strong objection, may I suggest we stick with acceleration as a colcon verb? there's quite a bit of documentation and content generated around it already and it'll be a bit painful 😭 to change it all again (we already changed it a while ago from krs to acceleration to make it more generic and community-accepted).

Repo migrated! https://github.com/colcon/colcon-hardware-acceleration
Thanks for the support here! We'll move on with the release steps, but I'll close this issue for now.


feat(links): distinguish links to other Quartz sites or digital gardens?

The significance of digital gardening is in its links and associations. Should we have special CSS classes for links to other Quartz sites (and a way of detecting that, maybe by meta headers, with fallback to typical external link style if it's a CORS error) and a way to manually specify that the site is another digital garden?
How would we represent the distinction? As for going to another Quartz site, I think just adding an outline-only Quartz icon would work.
For other digital gardens, I made this abomination from a material design icon as a mockup, like you're going down the rabbit hole. Other intuitive pictographs might be a door, a node graph, or a plant/tree/forest.

you can check the head meta tag for generator as Quartz

https://github.com/jackyzha0/quartz/blob/8141cb1587295b05277a5d380e8006deca6e88c6/quartz/components/Head.tsx#L198

you can check the head meta tag for generator as Quartz

That's not the only part of this issue. Should Quartz check that by default and render the results in the resulting link?

I don't think adding distinction is a necessary feature, if not at all. We don't crawl or do anything by default, and I don't think we should at all.


⚠ Turnos 24x7 has degraded performance

In 5a99c0b, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 5242 ms

Resolved: Turnos 24x7 performance has improved in 6733f5f.


perf debug: remove things that could have caused a perf regression

validated blob sending

/netsim

/netsim

I think we have determined that the perf is mostly as expected, except for a minor regression involving validate on send. For the latter, there is an issue https://github.com/n0-computer/iroh/issues/941


Use fluent API to configure a model

It would be very nice to have this option.
Example for EF https://docs.microsoft.com/en-us/ef/core/modeling/index
@joeconwaystk what do you think?

This is neat, but there currently aren't enough contributors to maintain multiple interfaces to building the data model. This would be worth looking at in the future.


zoom_us: add xdg_utils to PATH

This allows Zoom to open a web browser other than Google Chrome,
Firefox or Opera when xdg_utils is not installed.

Motivation for this change
It may help with https://discourse.nixos.org/t/no-sso-in-zoom-unless-firefox-is-installed/9453.
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/no-sso-in-zoom-unless-firefox-is-installed/9453/2

It may help with https://discourse.nixos.org/t/no-sso-in-zoom-unless-firefox-is-installed/9453

So we are not sure if it fixes the issue?

It may help with https://discourse.nixos.org/t/no-sso-in-zoom-unless-firefox-is-installed/9453

So we are not sure if it fixes the issue?

Honestly, I have no idea, I don't use zoom myself. I'm doing this because https://github.com/qt/qtbase/blob/dev/src/gui/platform/unix/qgenericunixservices.cpp#L126 suggests it may help.

It can't hurt anything, and it does appear Qt is indeed shelling out to xdg-open for this interaction. This may explain the SSO/Google login issues, as I have xdg-open in my $PATH but other users may not.

I would have tested that myself, but I don't have a way to sign in with SSO to reproduce the behavior. OTH, if someone else has SSO, uninstalling xdg_utils for the sake of testing this shouldn't be difficult.


[quickstart] configure the controller to schedule tasks when minion is enabled

Without this configuration, controller will not run cron jobs to schedule minion tasks.

Codecov Report

Merging #10338 (5cc84e0) into master (d3c8ec2) will decrease coverage by 56.51%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             master   #10338       +/-   ##
=============================================
- Coverage     70.27%   13.77%   -56.51%
+ Complexity     5902      221     -5681
=============================================
  Files          2027     1973       -54
  Lines        109953   107491     -2462
  Branches      16711    16418      -293
=============================================
- Hits          77273    14803    -62470
- Misses        27259    91511    +64252
+ Partials       5421     1177     -4244

Flag
Coverage Δ

integration1
?

integration2
?

unittests1
?

unittests2
13.77% <ø> (+0.01%)
:arrow_up:

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

 Learn more


Rewrite customizer using hooks

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @paulmelnikow!

  Generated by :no_entry_sign: dangerJS against 96bf3ce71728f77c942988730a6c5f7cfe596950


sql: Querying from a committed transaction record causes error

While operating on a created entity of a successful transaction, using QueryXXX APIs that may be present for edges will fail with sql: transaction has already been comitted or rolled back

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
Instead of a successful edge query, sql: transaction has already been comitted or rolled back is returned.
Expected Behavior 🤔
On a successful transaction commit, any newly created records that were created under the transaction should have config.driver point to a non-transactable version of the driver (if I understand what's going on behind the scenes).
Steps to Reproduce 🕹
Visit Bug Repo
Your Environment 🌎

Tech
Version

Go
1.17.1

Ent
87e21993097e

Database
Postgres

Driver
https://github.com/lib/pq

Hey @tankbusta
Thanks for reaching out! Ca you please submit a repo with a test reproducing your issue? We have a shiny new bug template repo just for that.
Thanks 🙏🙏🙏

Hey @tankbusta 👋, and thanks for taking the time to create a bug repository.
Each generated entity has a method named Unwrap that unwraps the entity from the transaction after it was closed so that all future queries will be executed through the driver which created the transaction. That means, the following change will fix your issue:
+	createdProduct = createdProduct.Unwrap()
	users, err := createdProduct.QueryCreatedBy().All(ctx)

A PR for updating the Transactions docs is more than welcome 🙏 😃

Ah, dang how did I miss Unwrap! I was looking for a method to potentially reset things.
I'd be more than happy to submit a PR to include Unwrap in the documentation. Although I'm curious why this wouldn't be the automatic behavior. Is there something I'm not thinking of that would make it undesirable behavior?
Thanks!

Although I'm curious why this wouldn't be the automatic behavior. Is there something I'm not thinking of that would make it undesirable behavior?

I assumed this question will come up 😃. Before we added this, we found out that when users use "transactional entities"  outside the transaction (e.g. after GraphQL mutation was finished), it's either because they do something wrong or it's intentional and the use case is valid. However, both cases are not so common, and I didn't want to add "magic support" for something that can prevent bugs or lead to undesired behavior. For example, the transaction was set to isolation level sql.LevelSerializable, and 2 identical queries can return different records.
u.QueryPets().All(ctx)

// ....

// Magically unwrap returns different records.
u.QueryPets().All(ctx)

I assumed this question will come up 😃

You can count on that from me! ;)
Thanks for that explanation, that all tracks!

I found this issue right before I reviewed the code of the same case.

I'd be more than happy to submit a PR to include Unwrap in the documentation.

@tankbusta, let me know if you plan to open a PR as I want to close the issue. Thanks 🙏

I do, I'll try to have it up at some point today.
You're free to close it and I'll just reference it :)


added a search system in heropedia web application

the video link of new feature is here https://drive.google.com/file/d/1wnjpudxOgIn-lLvjDopMAUBxqHjoozX_/view?usp=sharing
click;

if you add new character using the same " card " it will automatically sync the new character .

@kaustubhdidit

Good Work!!!


Stripe Extension not working inside a container running the stripe cli

ISSUE:
The stripe vscode extension does not work (nothing happens when you click on any "event" buttons such as "Trigger new default event", "Create new customized event". "Start streaming events" hangs indefinitely.
The following issue is displayed within vscode devtools when clicking the buttons:
[Extension Host] Error fetching CLI version TypeError: g is not a function     at h.checkCLIVersion (/home/devuser/.vscode-server/extensions/stripe.vscode-stripe-2.0.10/src/stripeClient.ts:179:11)     at processTicksAndRejections (internal/process/task_queues.js:93:5)
When pressing
"Start streaming API logs" also hangs indefinitely. and shows the following in vscode devtools:
[Extension Host] Error fetching CLI version TypeError: g is not a function     at h.checkCLIVersion (/home/devuser/.vscode-server/extensions/stripe.vscode-stripe-2.0.10/src/stripeClient.ts:179:11)     at runMicrotasks (<anonymous>)     at processTicksAndRejections (internal/process/task_queues.js:93:5)
TESTED:
The extension is working perfectly fine within windows 11 and WSL (Debian). but fails within the container. here is my docker file
# Start with base image (python 3)
FROM python:3.10.1-buster

ARG USERNAME
ARG USER_UID
ARG USER_GID=$USER_UID
ARG STRIPE_CLI_VERSION

# ***** Install & cleanup all unused packages *******
# Switch to using bash instead of the sh shell
SHELL ["/bin/bash", "-c"]
# Update all packages in the container
RUN DEBIAN_FRONTEND=noninteractive apt-get update && \
    apt-get install apt-utils curl dialog openssh-client wget build-essential sudo bash-completion --no-install-recommends  -yq \
    # Cleanup any unused packages
    && apt-get upgrade -y \
    && apt-get autoremove -y

# ***** Register Github as an allowed key *******
# Prevent the "git clone" operation from failing with an "unknown host key" error.
RUN mkdir -m 700 /root/.ssh; \
    touch -m 600 /root/.ssh/known_hosts; \
    ssh-keyscan github.com > /root/.ssh/known_hosts

# ***** Create non root user and enable sudo *******
# Create the user and group and enable sudo
RUN groupadd --gid ${USER_GID} ${USERNAME} \
    && useradd --uid ${USER_UID} --gid ${USER_GID} -m ${USERNAME}\
    && echo ${USERNAME} ALL=\(root\) NOPASSWD:ALL > /etc/sudoers.d/${USERNAME} \
    && chmod 0440 /etc/sudoers.d/${USERNAME}
# Create the application folder and set permissions
RUN mkdir /home/${USERNAME}/app \
    && chown -R ${USERNAME}:${USERNAME} /home/${USERNAME}
# Switch to the user that was created
USER ${USERNAME}
# Add the current user's .local folder to the path so installed binaries and python packages can be found
ENV PATH /home/${USERNAME}/.local/bin:$PATH

# ***** Install NVM for nodejs & NPM requirements *******
WORKDIR /home/${USERNAME}
# Set NVM environment variables
ENV NVM_DIR /home/${USERNAME}/.nvm
ARG NODE_VERSION 16.13.1
# Install nvm
RUN mkdir ${NVM_DIR} \
    && curl --silent -o- https://raw.githubusercontent.com/nvm-sh/nvm/v0.38.0/install.sh | bash \
    # Source the shell commands for NVM
    && . ${NVM_DIR}/nvm.sh \
    # Install node and npm
    && nvm install ${NODE_VERSION} \
    && nvm alias default ${NODE_VERSION} \
    && nvm use default
# Add node and npm to path so the commands are available
ENV NODE_PATH $NVM_DIR/v${NODE_VERSION}/lib/node_modules
ENV PATH $NVM_DIR/versions/node/v${NODE_VERSION}/bin:$PATH

# ***** Install The stripe cli tool *******
# Set working directory to the global binariy location
WORKDIR /home/${USERNAME}/.local/bin
# Download the tar.gz package for stripe cli, uncompress and unpack it with tar and gz, then remove tar.gz
RUN wget https://github.com/stripe/stripe-cli/releases/download/v${STRIPE_CLI_VERSION}/stripe_${STRIPE_CLI_VERSION}_linux_x86_64.tar.gz \
    && tar -xvf stripe_${STRIPE_CLI_VERSION}_linux_x86_64.tar.gz \
    && rm stripe_${STRIPE_CLI_VERSION}_linux_x86_64.tar.gz \
    && chmod 775 stripe
# Enable stripe cli auto completions
RUN mkdir /home/${USERNAME}/.stripe
WORKDIR /home/${USERNAME}/.stripe
RUN stripe completion --shell bash
RUN echo ". /home/${USERNAME}/.stripe/stripe-completion.bash" >> /home/${USERNAME}/.bashrc \
    && stripe config set --color on

# ***** Install the default python django libraries from requirements.txt *******
WORKDIR /home/${USERNAME}/app
# Disable buffering to not cut off any error messages
ENV PYTHONUNBUFFERED 1
# Install a base collection of tools usefull for django development
COPY requirements.txt requirements.txt
RUN pip3 install --upgrade pip\
    && pip install -r requirements.txt\
    && rm requirements.txt

I have also been able to replicate this issue on another one of my computers but in WSL(Debian) only.

I am getting this exact same error trying to run the stripe cli vscode extension inside a docker container


Disallow 'return' statements in pyfora 'with' blocks

Return statements are supposed to exit the enclosing function - not the with block, and we don't have a good way to implement that.

I'm wondering, do we need to also check yield statements or do we support these?

good call. we support those inside functions, and with block bodies get
turned into functions, so we should disallow them too
On Thu, Nov 5, 2015 at 5:25 PM, Alexandros Tzannes<EMAIL_ADDRESS>
wrote:

I'm wondering, do we need to also check yield statements or do we support
these?
—
Reply to this email directly or view it on GitHub
https://github.com/ufora/ufora/issues/68#issuecomment-154214450.

--
Thomas Peters | Software Engineer | *U**FORA *
115 West 27th Street 7th Floor, New York, NY 10001
Office:<PHONE_NUMBER> | Mobile:<PHONE_NUMBER>
<EMAIL_ADDRESS>| www.ufora.com

Fix ready, someone please review

With the typo fixes, I just pushed this to dev.


[Enhancement] Simplify Account Linking Process

Hey there! So, I noticed that bind codes can be quite complex and make linking accounts more difficult than necessary. My suggestion is to switch to a simpler system where users can just run /mgb link in Minecraft, then use /mgb link (username) on Guilded, and finally confirm with /mgb confirm in Minecraft. This would make connecting accounts much easier. Best regards.

I've made the bind code simpler in the latest commit. Have a try.


[Node.js] Expose native error to Javascript

Native error messages are exposed to javascript and
unrecoverable errors will generate TypeError exceptions.
Code refactor

@halton PTAL, thanks.


Error while removing the solution

I try to remove the solution and get the error:
Sql error: Generic SQL error. CRM ErrorCode: -2147204784 Sql ErrorCode: -2146232060 Sql Number: 5074

It is impossible for me to remove the solution.
What can I do ?

Stepping deeper into the problem I've found this error message:
The column 'proj_remainingbudgetallocation_Base' is dependent on column 'proj_ProjectBudget'. ALTER TABLE DROP COLUMN proj_ProjectBudget failed because one or more objects access this column.
Seems like there is nothing I can do about ?

I have this exact same issue and currently have a ticket open with Microsoft Support looking at it for me, no resolution as of yet with them though.

I'm meeting with them today to find a solution,, i'll post the fix once i get it

Hey @jbejai , did you get anywhere? I also had a call yesterday with someone from support, 1st line support only I'm guessing as they didn't have a clue how to fix it, but did take some additional information, and I did work out some additional information too I think.
It's a bit of a guess this but it seems like some recent update to Project for the Web is now using the Program table, part of the new portfolio stuff would be my guess, which has created a dependency meaning that accelerator now cannot be uninstalled due to that dependency. In particular, the Remaining Budget Allocation field as per the error message is on the Information Main Form which seems to be the sticking point.
I spent hours last night even trying the very much nuclear option of fully removing Project for the Web as per the instructions here...
https://learn.microsoft.com/en-us/project-for-the-web/remove-roadmap-from-office-365
But even that fails once you get to msdyn_ProjectServiceCore as you need to remove the accelerator first, which of course you can't due to the dependency.

@markheydon I got nowhere also,,,, the engineers at INFOSYS is currently trying to get a solution for me.. I have a call later on today with them.. they have never seen an issue such as this hence its taking so long...

I'm afraid it's an all too familiar story with Microsoft Support generally these days. Takes days and often I just give up.

Good news @jbejai @Tom-CK !!! I've solved it!!!!!

Ok so went back to basics on this to sort it out. Basically, I checked for dependencies on the Project Budget field which is my case was...

I then went into each and removed the Project Budget field including the chart. My money is on the calculated column to be honest but I was getting sick of the disappointment of waiting ages only to be greeted with the 'Failed' message, so I got rid of them all, then did Publish All, then I was able to uninstall the accelerator.
Hope the same works for you!

@Tom-CK

The column 'proj_remainingbudgetallocation_Base' is a calculated column. But I can not edit the formula, the button is grayed out.

Go in through the Default Solution (if you weren't already), and edit the 'proj_remainingbudgetallocation' column (i.e. NOT the _Base version), you should then be able to edit it. I replaced the msdyn_projectbudget field with '0' just to get it out of calculation. Remember to do a Publish All after editing it and check the deps again make sure it's gone.

Gonna try this now... hey @markheydon thanks sooo much

Hey! @markheydon thanks so much.... I got tru ... really appreciated this help


Create calculate_descriptor function

Closes #8

Try to install conda in trivis CI envrionment. Ref. https://docs.conda.io/projects/conda/en/latest/user-guide/tasks/use-conda-with-travis-ci.html

Install conda in trivis CI envrionment and passed it! 🎉


⚠ Bofrost.be has degraded performance

In aa9bd3e, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 2433 ms

Resolved: Bofrost.be performance has improved in e383a67.


Incompatible with new bleach

In order to adapt to the API changes in the current html5lib release, bleach is making substantial changes of its own:
https://github.com/mozilla/bleach
It does not appear to me that django-html_sanitizer is compatible with the bleach changes.  If I run test.py in python2.7 with the new bleach I get:
python tests.py
Traceback (most recent call last):
File "tests.py", line 6, in
from sanitizer.templatetags.sanitizer import (sanitize, sanitize_allow,
ImportError: No module named templatetags.sanitizer
The new bleach is expected to be released next week.

bleach 2.0 is released now, so I guess that means this is more urgent.

Could you please open a PR for this?

I don't understand what you are asking me to do (sorry).  What do you want in the pull request?  As far as I know, there's no code available for django-html_sanitizer to address the incompatibility.

@selwin, I think @kitterma is asking django-html_sanitizer maintainers to add support for bleach v2.0.


Rename 'delegatable' permissions to 'delegated'

Trello
This came out of work to fix 'delegatable' permissions. We've gradually settled on some new terminology, the main change being that we now refer to them as 'delegated' permissions (all permissions are delegatable). This updates the codebase, including the UI and documentation, to reflect that shift
Screenshots
Before

After

This application is owned by the publishing platform team. Please let us know in #govuk-publishing-platform when you raise any PRs.
⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠
Follow these steps if you are doing a Rails upgrade.

Makes sense to me as well. It take me reading your other comment explaining this to George for me to figure out exactly what's going on, so it's not super natural, but I do understand and agree with the terminology.


Fix RPM post-install script

We will need to add rpm test as well.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[X] Does not have any changes that may affect Istio users.

@jasonwzm: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
2dac2e944656dd019076b0ce63dde170309af334
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@jasonwzm: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio
dcd6107e78192799bd6296015a89a9ded651a3ca
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

In response to a cherrypick label: new pull request created: #26832


Fill-in QuotaResponse for Quota API calls.

This weaves a method return value through the layer in a way similar to how I weaved method arguments through the layers. This change introduces a TTL at the quota adapter layer, needed for prefetching of rate limiting quotas.
Also note the lack of tests for the changes in api/handlers.go. That whole file has almost non-existent testing for the time being. I plan to rewrite this file nearly completely to support OOO evaluation of incoming mixer API calls. When I do the rewrite, I'll add a full complement of tests.

This change is

Added a QuotaResult struct for use by the adapters.
PTAL.

Jenkins job mixer/presubmit passed


V4 config allow Env vars to be lowercased print entityName on debug payload log

The v4 config has a field  on the instances called env with this format:
env:     HOST: a-host     PORT: a-port
We want to allow the variable names to be lowercase.
Additionally, the new V4 config gets the binary name from the instance name, that makes all the instances for a given integration to have the same name. When troubleshooting and looking at the logs in verbose mode, the users identify each payload by the name field:
[...] level=debug msg="Received payload." component=integrations.emitter.Emitter **name=nri-kafka** payload="{\"nam [...]
Since now the name is the same for all the instances, it's harder for them to find which payload corresponds to each instance, to improve this, we want to add the entityName on the log as instance="the-name-of-the-entity".

What would be the output if the integration is not able to run correctly? There would be no payload and therefore no entityName, right? And how would this interact with integrations that return more than one entity?

I think in the end we agreed to print same full env as we do per integration execution. In that case there will be always uniq data, even from discovery.


Expand checkstyle

This introduces the charset property to help avoid issues on Windows machines, and adds rules regarding braces (required except for empty-body loops) and import wildcards (disallowed, except static imports).
Depends on !25

Seems like this isn't wanted.


Multi-producer multi-consumer ZMQ model

The original issue title wasn't addressing very important problems. When speed is heavily concerned this should be a better solution.

Not of any interest for the moment. Closing..


Remove redundancy in entitlement checks.

Checking proper package access rights is done on a deeper level. Disambiguating here is redundant.

PG3 875 🔵


(fix test regards changes on 0.11) - Start to check API rule violations and fix mock data

Description of the change:

Add check in the tests to ensure that no issues related the API rule validations should be faced.
See here that it worked all.
Fix/update mock data for Memcached implementation.  (All CI tests should finish with success in order to ensure the fix)
Add Ddokcerfile to test-framework for we are able to run operator-sdk commands on it to update the mock data
Add test in the sanity to ensure that the test-framework mock data will be always updated with teh current version

Motivation for the change:
Closes #2114
Related to: https://github.com/operator-framework/operator-sdk/issues/2042

/test e2e-aws-go
/test e2e-aws-helm
/test e2e-aws-subcommand
/test e2e-aws-ansible

HI @joelanford,

So the // +listType=set annotation currently has no impact on crd_test.go

It is not the point. The crd_test.go will do the test using a version which is outdated of the CRD since it is checking the test-framework which has not been updated and verified.
However, I will change this PR as you suggest no problem at all.

Hi @joelanford,
The TODO was added as requested. Please, let us know if we could move forward with it now?

/test e2e-aws-subcommand


Unable to drag titlebar on high-resolution display

On a screen with a 2K resolution and a 125% scale, I'm unable to drag the titlebar.
After investigating, I found that the issue lie in the calculation of the qtScenePos:
https://github.com/wangwenx190/framelesshelper/blob/b864e398940cb8fb9d9e51ec3a689aecad6418de/src/core/framelesshelper_win.cpp#L1004-L1005
When I changed fromNativePixels to fromNativeLocalPosition, the problem was resolved:
https://github.com/wangwenx190/framelesshelper/blob/b864e398940cb8fb9d9e51ec3a689aecad6418de/src/core/utils.cpp#L410
However, I'm not sure if this is the correct solution.

Added info:
When using only one 2K screen, this issue does not occur. It only happens when extending the display with a 1080p screen at 100% scale and a 2K screen at 125% scale.

Thanks for your investigation! It seems your solution is more sensible indeed! I'll address this issue as soon as possible!

Can you check whether commit 5ade3cfb74dbcb8a154cde596e3ab8b066d0526c fixed the issue or not? Thanks!


Rebuild for boost-cpp-1.70.0

Fixes #24

Checklist

[x] Used a fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@xhochy thanks for the PR! It seems like the Boost detection is now working better (no more weird messages about Boost 1.55 or references to Boost's CMake, as far as I can see), but there's still a link error on the linux build. Googling around, it seems like we may need to add -lrt to the compile flags. We can (and will) fix this in the next release, but what way would you suggest to temporarily patch the issue in the recipe?

@xhochy it passes! Thanks a lot!


Doesn't work with ssl

this  is the log of rendertron app running and try to render.
Chrome launched with debugging on port 41152 Listening on port 8080 Cannot render requested URL { message: 'net::ERR_NAME_NOT_RESOLVED' } Cannot render requested URL { message: 'net::ERR_NAME_NOT_RESOLVED' } Cannot render requested URL { message: 'net::ERR_NAME_NOT_RESOLVED' } Cannot render requested URL { message: 'net::ERR_NAME_NOT_RESOLVED' } Cannot render requested URL { message: 'net::ERR_NAME_NOT_RESOLVED' } Cannot render requested URL { message: 'net::ERR_NAME_NOT_RESOLVED' } Cannot render requested URL { message: 'net::ERR_NAME_NOT_RESOLVED' }
code of deploy:
https://github.com/pablo384/zoo-webapp-deploy
Angular App:
https://github.com/pablo384/Zoo-Front-end
I tried to use rendertron from a VPS in DigitalOcean. the domain is here so after using https rendertron doesn't work may I did a mistake in the process.

I solved, my nginx config was wrong. I put the domain name and fix it, but I knew thanks a log in prerender.io.
upstream pablo384.com {
          server <IP_ADDRESS>:3443;
    }

Please close this issue.

I'm having similar issue, could you provide more details? Your nginx config maybe?


Add FieldMask and regenerate Elixir code

This PR adds FieldMask to the PROTOS array in generate_google_protos.sh. It also includes the result of running the script to generate the Elixir code to make field_mask available.

Comment/question: I'm noticing that as well as adding the new field_mask.pb.ex, after running the codegen script transform_module functions have been removed, as well as the jsonName properties on some of the fields.
Have I missed something in running the script, or might it need to be updated to include these fields?
This was generated with protoc-gen-elixir 0.7.1, protoc libprotoc 3.17.3 on Elixir 1.10.3.

We are getting the following

19:53:16.659 [error] ** (UndefinedFunctionError) function Google.Protobuf.FieldMask.message_props/0 is undefined (module Google.Protobuf.FieldMask is not available)

Using protoc_gen_elixir_version: "0.10.0"

Hey, @redink @ericmj do you have some time to look at this one? 🙏🏻 the current codegen from the protoc is broken, we have some prod issue due to it, and would be amazing if we can get this one release before I have to copy the code over.
Thanks in advance.

the current codegen from the protoc is broken

What is broken?

@ericmj https://github.com/elixir-protobuf/google-protos/pull/10#issuecomment-1150448933
The module is not present so the codegen from the protoc is failing to unmarshal the JSO
defmodule CoreApi.V1.Accounts.UpdateAccountRequest do
  @moduledoc false
  use Protobuf, protoc_gen_elixir_version: "0.10.0", syntax: :proto3

  field :request_id, 1, type: :string, json_name: "requestId"
  field :account, 2, type: Hmbradley.ThoughtMachine.Protos.CoreApi.V1.Accounts.Account

  # here the usage of the `FieldMask`
  field :update_mask, 3, type: Google.Protobuf.FieldMask, json_name: "updateMask"
  field :overrides, 4, type: Hmbradley.ThoughtMachine.Protos.CoreApi.V1.Common.Overrides
end

N

It looks like you are using google_protos from Hex. But the FieldMask is only added to this PR and has not been released yet.

@ericmj correct, I am using everything from hex at the moment, and the Hex version of the protoc plugin is generating the code that depends on such module. The reason why I said it is "kind" of broken.

Hey peeps, any updates on this one?

@whatyouhide Are you also a maintainer on this one, along with the protobuf library?


All commands are required, slowing down the CLI

Do you want to request a feature or report a bug?
Bug
What is the current behavior?
I'm seeing all commands being required, both with ts-node and when using OCLIF_TS_NODE=0:
time DEBUG=* OCLIF_TS_NODE=0 npx mycli init --help
  @oclif/config reading core plugin /Users/matthewshwery/dev/src/github.com/<repo>/packages/cli +0ms
  @oclif/config loadJSON /Users/matthewshwery/dev/src/github.com/<repo>/packages/cli/package.json +0ms
  @oclif/config loadJSON /Users/matthewshwery/dev/src/github.com/<repo>/packages/cli/oclif.manifest.json +3ms
  @oclif/config loadJSON /Users/matthewshwery/dev/src/github.com/<repo>/packages/cli/.oclif.manifest.json +0ms
  @oclif/config:@mycli/cli loading IDs from /Users/matthewshwery/dev/src/github.com/<repo>/packages/cli/dist/commands +0ms
  @oclif/config:@mycli/cli found commands [ 'init', 'push', 'register', 'generate:action', 'generate:types' ] +10ms
  @oclif/config:@mycli/cli require /Users/matthewshwery/dev/src/github.com/<repo>/packages/cli/dist/commands/init.js +2ms
  @oclif/config:@mycli/cli require /Users/matthewshwery/dev/src/github.com/<repo>/packages/cli/dist/commands/push.js +317ms
  @oclif/config:@mycli/cli require /Users/matthewshwery/dev/src/github.com/<repo>/packages/cli/dist/commands/register.js +225ms
  @oclif/config:@mycli/cli require /Users/matthewshwery/dev/src/github.com/<repo>/packages/cli/dist/commands/generate/action.js +1ms
  @oclif/config:@mycli/cli require /Users/matthewshwery/dev/src/github.com/<repo>/packages/cli/dist/commands/generate/types.js +186ms
  @oclif/config reading user plugins pjson<EMAIL_ADDRESS>+0ms
  @oclif/config loadJSON<EMAIL_ADDRESS>+789ms
  @oclif/config loading plugins [ '@oclif/plugin-help' ] +3ms
  @oclif/config loadJSON /Users/matthewshwery/dev/src/github.com/<repo>/packages/cli/package.json +3ms
  @oclif/config loadJSON /Users/matthewshwery/dev/src/github.com/<repo>/packages/package.json +1ms
  @oclif/config reading core plugin /Users/matthewshwery/dev/src/github.com/<repo>/node_modules/@oclif/plugin-help +0ms
  @oclif/config loadJSON<EMAIL_ADDRESS>+0ms
  @oclif/config loadJSON<EMAIL_ADDRESS>+1ms
  @oclif/config:@oclif/plugin-help using manifest from<EMAIL_ADDRESS>+0ms
  @oclif/config config done +2ms
  @oclif/config start init hook +0ms
  @oclif/config init hook done +1ms
  mycli init version<EMAIL_ADDRESS>argv: [ 'init', '--help' ] +0ms

Note: this happens across multiple commands, with and without --help
What is the expected behavior?
The documentation site claims that only the command to be executed will be required (emphasis mine):

The overhead for running an oclif CLI command is almost nothing. It requires very few dependencies (only 35 dependencies in a minimal setup—including all transitive dependencies). Also, only the command to be executed will be required with node. So large CLIs with many commands will load just as fast as a small one with a single command.

=> Found "<EMAIL_ADDRESS>=> Found "<EMAIL_ADDRESS>=> Found "<EMAIL_ADDRESS>=> Found "<EMAIL_ADDRESS>
On macOS 11.4 (Big Sur), tried on multiple Node versions (12x/14x/16x), Yarn 1.22

Closing this due to lack activity. Please open a new issue if needed - thanks!


Implement Auxiliary Plan

AuxPlan is to process auxChan stream with a full plan instead of a simple endpoint.  This would allow users to create a flow around auxiliary streams.

New design does not use this anymore.


arch: POSIX: Add Kconfig option to stop on all faults

Due to popular demand:
Added an option to stop the execution of the posix arch based
executable on the first fault, even if the fault stemmed from a
non essential thread.
Having it fail faster, in the first fault, will ease debugging
in many cases.
The option is disabled by default to preserve the old behavior.
Signed-off-by: Alberto Escolar Piedras<EMAIL_ADDRESS>

Codecov Report

Merging #13439 into master will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           master   #13439   +/-   ##
=======================================
  Coverage   48.57%   48.57%
=======================================
  Files         319      319
  Lines       46569    46569
  Branches    10761    10761
=======================================
  Hits        22619    22619
  Misses      19360    19360
  Partials     4590     4590

Impacted Files
Coverage Δ

arch/posix/core/fatal.c
20.68% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

recheck

Will that trap in gdb?

That function ends in posix_exit, which ends in exit().
The point is that all exists in the posix arch go thru posix_exit() which does a proper cleanup and exists.


bypassed printing toggled key.

this is regarding issue 34

Thank you for your pr,But it is not very clean and not very good,I removed it.


transform: rotate slow down autoscroll and eventually stop it

Checks

[X] Not a duplicate.
[X] Not a question, feature request, or anything other than a bug report directly related to Splide. Use Discussions for these topics: https://github.com/Splidejs/splide/discussions

Version
v4.0.2
Description
Seems like whatever (body, wrapping div, div.splide) have CSS attribute transform: rotate(Xdeg) it slows down autoscroll and eventually stop it.

Reproduction Link
No response
Steps to Reproduce

Visit https://splidejs.com/extensions/auto-scroll/
Add CSS attribute transform: rotate(5deg) to slider or even body
...

Expected Behaviour
Autoscroll would work normally.

This is the design limitation. Splide does not work inside an element with transform since it uses bondingClientRect.


Copy contents folder for generic files

We have a folder Introduction which only contains generic files.
The contents folder isn't copied over for this folder, which suggests that we only copy Contents folder for files that are receiving special processing.
The issue can be seen as a publish error in PR https://github.com/DeepBlueCLtd/LegacyMan/pull/585

The reason this folder isn't being copied is because it is called Contents and the equivalent folder for all other places in the data is called Content. Is this just a typo? Or should we be accepting any folder that looks like Content or Contents or similar?

Yes @robintw - you were quite right.  The folder title was wrong.


[Feat] Multi-line reversed cards

i.e.
Multi-line
question
(or answer when reversed)
??
Multi-line
answer
(or question when reversed)

Configurable...

+1 for this feature

+1

Fix should be available in the next release once I'm done with testing.


Question - Annotation Aggregation.ApplySupported.Transformation

In risks-analysis-service-ui.cds, there is an annotation "Aggregation.ApplySupported.Transformation" - however there seems to be no difference in behavior when Transformations is empty when I actually run CAP Node.js for OData V4.
https://github.com/SAP-samples/cloud-cap-risk-management/blob/analytics/srv/risks-analysis-service-ui.cds#L48
Why is this annotation configured on this code? I thought that if it was unnecessary, it could be cut down.

Thank you for your answer. Now it is clear.


Update api.rst

Minor grammar corrections to the API documentation

Mostly updates to documentation.  Some small code style things to comply with pystyle

I think this fixed the numpy doc string style issues


Refactoring key

Pull Request Test Coverage Report for Build 685

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

Totals

Change from base Build 566:
0.0%

Covered Lines:
389

Relevant Lines:
408

💛  - Coveralls


Release as a NET standard library

PoE Sidekick is also looking into using Winook to deal with mouse lag from mouse wheel event hooks but we currently can't try it out since it is running on NET framework

Hi,
Thank you for the interest you have in Winook. I agree that targeting .NET Standard is a welcomed addition.
The challenge I have with this is updating my nuspec file so that I can distribute a Winook package that works with both .NET Framework and .NET Core. I think there are significant differences between how this is handled (packages.config vs PackageReference). The main road block is the fact that Winook uses pre-compiled support files that need to be distributed as well. I can spend some time on that matter, but if you already know how to make it work, please feel free to share this with me.

My knowledge of C# and its package ecosystem is limited. I was following this guide: https://www.michaeltaylorp3.net/porting-a-net-framework-library-to-net-standard/ and it does mention having to create a new package  (IIUC). Maybe it will just be easier to declare the current code as the final version of Winook and create a new package called WinookStandard (or something else).

Sounds good, I'll check the guide and try it out. Thanks.

Can you please try this out?


⚠ TorrentLeech has degraded performance

In 171bcde, TorrentLeech ($TL) experienced degraded performance:

HTTP code: 500
Response time: 1174 ms

Resolved: TorrentLeech performance has improved in 90064bf after 6 minutes.


config: delete vip "hides" other entries

Not entirely sure if this can be reproduced so for now just record what happened: it fits the "entries disappeared" reports that happen sporadically... I was using "die;" for debuggin in a POST request. Removed "die;", hit reload and resubmit in the browser and that happened:
--- /conf/backup/config-1542039272.2488.xml	2018-11-12 17:14:32.249928000 +0100
+++ /conf/backup/config-1542039332.9464.xml	2018-11-12 17:15:32.947471000 +0100
@@ -1165,8 +1165,8 @@
     <filterlogentriesacts>Pass Block Reject</filterlogentriesacts>
   </widgets>
   <revision>
-    <time>1542039239.9586</time>
-    <description>/firewall_virtual_ip_edit.php made changes</description>
+    <time>1542039272.25</time>
+    <description>/firewall_virtual_ip.php made changes</description>
<EMAIL_ADDRESS>   </revision>
   <openvpn>
@@ -3128,27 +3128,21 @@
   </wol>
   <virtualip>
     <vip>
-      <type>single</type>
-      <subnet_bits>8</subnet_bits>
-      <mode>ipalias</mode>
-      <interface>lo0</interface>
-      <descr>testing</descr>
-      <subnet><IP_ADDRESS></subnet>
-    </vip>
-    <vip>
-      <type>single</type>
-      <subnet_bits>24</subnet_bits>
-      <mode>ipalias</mode>
-      <interface>wan</interface>
-      <descr>Mock</descr>
-      <subnet><IP_ADDRESS></subnet>
-    </vip>
-    <vip>
-      <type>single</type>
-      <subnet_bits>64</subnet_bits>
-      <mode>ipalias</mode>
-      <interface>lan</interface>
-      <subnet>::123</subnet>
+      <vip>
+        <type>single</type>
+        <subnet_bits>8</subnet_bits>
+        <mode>ipalias</mode>
+        <interface>lo0</interface>
+        <descr>testing</descr>
+        <subnet><IP_ADDRESS></subnet>
+      </vip>
+      <vip>
+        <type>single</type>
+        <subnet_bits>64</subnet_bits>
+        <mode>ipalias</mode>
+        <interface>lan</interface>
+        <subnet>::123</subnet>
+      </vip>
     </vip>
   </virtualip>
   <staticroutes/>

Maybe also reported via: https://forum.opnsense.org/index.php?topic=10276.0
Offending issue could be: bd148f1dd2b52

@AdSchellevis maybe you could look at this, I'm suspecting bd148f1 again, I have it queued up for 18.7.8 but the forum thread makes me worry...

I'm having similar issues when I delete rules it's wiping out my entire Rule list for all firewall rules.

@mstrdraco which version?

@mstrdraco @AdSchellevis thanks ❤

@fichtner found the culprit, unset() doesn't reindex, which kills https://github.com/opnsense/core/blob/master/src/opnsense/mvc/app/library/OPNsense/Core/Config.php#L80
It's just very stupid that you can't distinct associative and sequential arrays easily.

it's in the same corner, yes

https://github.com/opnsense/core/commit/0df243422bfc449a43f1e94360985458f3a72771 should fix the problem.

@mstrdraco you can try the patch from the command line:
# opnsense-patch 0df2434

Worked great! Thank you

perfect, thanks. will be in the next devel update 👍


Use user exception for ShellTask

TL;DR
Please replace this text with a description of what this PR accomplishes.
Type

[ ] Bug Fix
[ ] Feature
[ ] Plugin

Are all requirements met?

[ ] Code completed
[ ] Smoke tested
[ ] Unit tests added
[ ] Code documentation added
[ ] Any pending items have an associated Issue

Complete description
How did you fix the bug, make the feature etc. Link to any design docs etc
Tracking Issue
https://github.com/flyteorg/flyte/issues/3559

@wild-endeavor is this ready to be reviewed?


Bad documentation of tokens usage in template literals

Subject
Token Usage
Description
The example https://panda-css.com/docs/theming/usage#styled-components in the documentation shows this usage of tokens:
const Button = styled.button`
  background: ${token('colors.blue.200')};
`;

but this doesn't work and even the Typescript says that the tag function expects only one argument, so the string without expression.
The correct example should be:
const Button = styled.button`
  background: token(colors.blue.200);
`;

Hi @adiosneo,
That example refers to using Panda with a runtime css-in-js like emotion or styled-components. In that sense, it's not a bad example.
Of course, to get it working with the styled factory in Panda, you need to use the string equivalent.
Thanks for bringing it up though.


Problem detecting libc

I noticed a regression in iconv when built with musl. This is actually caused by crystal not being able to detect the libc when building and then defaults to gnu. This in turn is caused by a regression of ldd in musl which doesn't recognize the --version flag anymore and fails. See here: https://github.com/crystal-lang/crystal/blob/b038213e636c055ad7fdf98d0fc294200ab849a5/src/compiler/crystal/config.cr#L60-L63 how crystal handles this.
This raises the question why crystal isn't using a build flag to bake the libc variant into the compiler as a constant when it's built for the platform. It's not possible to use the crystal compiler linked against gnu libc to build for musl and vice versa, is it?
See the discussion with alpine maintainers at https://dev.alpinelinux.org/irclogs/%23alpine-linux-2019-07.log from 2019-07-02 13:32:11 on.

We want to detect at runtime whether the binary wants to build against musl-libc or glibc, because we build a static binary for Crystal releases on Alpine Linux, then use it on whatever Linux distribution.
It hack should have been safe, because we follow what autoconf or automake does to detect musl, but apparently this has changed :/
Please look at the Git history to find the merge of the change and the actual discussion to understand more.

For now a workaround to be able to build for musl is to add -Dmusl to the crystal build or shards commands.

From
2019-07-02 04:44:22 <mikeee__> running 'ldd --version' results in '/lib/ld-musl-x86_64.so.1: cannot load --version: No such file or directory'
2019-07-02 04:44:51 <mikeee__> after https://github.com/alpinelinux/aports/commit/d4a7955cc72b9395779c20755df88f983763c43b#diff-1f663dd56ac26d76d510b53ab0a6c975
2019-07-02 04:45:38 <mikeee__> the problem is that the (now) wrapper script /usr/bin/ldd will turn it into '/lib/ld-xxx --list -- --version'

A solution would seems to be to extract the first part of the output of ldd --version or check if the string includes (and not just start).

This raises the question why crystal isn't using a build flag to bake the libc variant into the compiler as a constant when it's built for the platform.
It's not possible to use the crystal compiler linked against gnu libc to build for musl and vice versa, is it?

The libc was hard-coded before #7479, but we wanted more flexibility, Because, in fact the compiler doesn't care whether itself it linked against musl or glibc, it can link to both.

The actual issue here is that musl's ldd wrapper script is broken. I suppose this is going to be fixed. This behaviour was copied from GNU's config.guess, so it's rather commonly used and the breakage is simply on musl. ldd --version should just work for both glibc and musl.
If there are any better ideas to tell whether a system uses musl or gnu as default C toolchain, we could certainly work on improving this. But I doubt anyone knows a better and more reliable method.
Apart from that, I don't think there is anything to do right now.

This issue has been fixed in alpine 3.11 and we don't need to bother.


Rename inconsistent methods to be more consistent

We have 2 equivalent methods for retrieiving rest and realtime clients, they are called getRest and realtimeWithHandle. In this PR, I rename them to restFrom(handle) and restFrom(handle).

It's clear that the Android 21 CI failure is unrelated to this change so I'm going to land this pull request.


Lattice planner dev

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.2 out of 3 committers have signed the CLA.:white_check_mark: wanglei828:white_check_mark: yifeijiang:x: liyunADUYou have signed the CLA already but the status is still pending? Let us recheck it.


Pathway heatmap-adding protein to group function just re write pop-up window

I selected a column of proteins from the heatmap viewer, then double clicked on Add Proteins to Group...it just redrew the pop-up box a little to the right of where it originally was.

#1031

Closing as duplicate of #1031


Adding Calender in Classes Page of the App #SWoC 2021

The classes Page does not have a calendar integrated into it. I would like to work on this by adding customizable calender.

Sure Go ahead @Imadarshsri

17 days have passed ,
If nobody working on this feature  please assign this task to me

Sure Go Ahead @ajay963


Update examples for user creations

The ansible-role-users can be updated. Unfortunately it would mean that sites would need to update their group_vars - and we could have them do that before we update the role. If they don't, the "users" role will fail. But that is run very rarely so maybe that's ok.
The error is that in examples/group_vars/all/root_keys.yml we have

adminusers:
  - { name: cscluis, uid: 5001, group: csc, state: "present", groups: "{{admingroup}}", shell: "{{adminshell}}", pubkey: "ssh-rsa KEYHERE" }

4 changed so this role adds the csc group as the primary group of the user.
Before it only added the {{ admingroup }} and didn't do anything with the csc group.
When a user is added to a group the group must exist first, it doesn't and the task fails.
Two methods (both by changing group_vars):

create the csc group (users' groups will change - potentially causing some permission issues.)
set the admingroup as the group: on the csc users in the example files (hypothetically no change, needs to be tested)

Release notes we could use when we are ready:

users:
set comment= and remove= and only remove passwords on present users 3
use package module, install libselinux and use group and groups 4
actually remove users and keys 5
use block and default(omit) to reduce tasks 6

Examples updated.
Informing users via usual communications on Monday.
Updating fgci-ansible in a week or three.

Role updated in https://github.com/CSC-IT-Center-for-Science/fgci-ansible/commit/938118a3f29113643b45cf210c3041e14ede5bcf


Fix issue with: "First argument must be a string, Buffer, ArrayBuffer, Array, or array-like object."

While trying to figure out the issue with posting transactions to Terra Classic from a dApp, that Terra Station also has see print screen bellow:

Found out that when using wallet-provider/use-wallet package, you can pass isClassic as an option so that the extension can transform the transaction msgs correctly and be able to post them to the classic chain.
Like so:
import { useWallet } from "@terra-money/wallet-provider";

const { post } = useWallet();

const res: TxResult = await post({
  msgs,
  fee,
  // @ts-ignore
  isClassic: true,
});

For that reason I'm making this PR to add that option so it's actually discoverable, typescript doesn't scream saying it doesn't exist.
PS: For Terra Station to work with classic chain it needs this option passed.

it won't fix the error until wallet-provider is updated.
also many functions which are using CreateTxOptions have to be updated if we add isClassic to that interface.
I think we should just wait for station update.

You are correct this doesn't fix the actual issue, but does give an hint to anyone that's trying to use the post method that it should set it as isClassic if they want to interact with the classic chain.
Nothing needs to be changed, the commit makes it optional, so it literally just gives an hint that option is available and also Typescript doesn't yell at us saying it doesn't.
But anyway this doesn't block any issue, just brought it up so people are aware.

CreateTxOptions is used inside terra.js itself so many people might be confused.
The best way is to keep using terra.js 3.0.11 and wallet-provider 3.8.1 for classic dapps.


tests: speed up non-push/merge group cases.

only run readall or audit on the whole tap for push and merge_group runs.
always run brew style on the formula in case tap_syntax isn't run or there's macOS/Linux differences as it's really fast.

Thanks for review, all!

This is likely going to be annoying for recursive dependency checks, or CI-syntax-only PRs, where it will only fail when merging.

Is that less annoying than having an extra >5m on every PR? It's all trade-offs here.

Though I'm ok with this for now until we do something like brew audit --readall.

I'm not sure what this would be and how it would work?


Saturation in compiler generated Stickify

This algorithm adds the equivalent of this (per William Jones) to the stickify compiler generated code.
static const __vector float min_vec = {-8573157376.F, -8573157376.F, -8573157376.F, -8573157376.F};
static const __vector float max_vec = {8573157376.F,<PHONE_NUMBER>.F,<PHONE_NUMBER>.F,<PHONE_NUMBER>.F};

float* a = ...;
float* b = ...;
int16_t* out = ...;

__vector float* vec_a = (__vector float*)a;
__vector float* vec_b = (__vector float*)b;
__vector short* vec_out = (__vector short*)out;

*vec_out = vec_round_from_fp32(
    vec_max(vec_min(*vec_a, max_vec), min_vec),
    vec_max(vec_min(*vec_b, max_vec), min_vec),
    0);

When running on z16 with Roberta base 11 and 6x384 in sequential mode
zdnn
  zhigh.Stick, 85, 0.5742549, 48.8116667, 5.6%

compiler gen code without saturation
  zhigh.Stick, 85, 0.3593294, 30.5430000, 3.7%

compiler gen WITH saturation
  zhigh.Stick, 85, 0.4358196, 37.0446667, 4.4%

As shown, saturation runs still faster than zdnn based code.
Checked the asm generated, pattern use vector compare and vector select, which I believe is the best pattern at this time for floating point numbers
    81e6:	e7 91 10 00 24 ea 	vfchesb	%v9,%v17,%v1                    <<<<==== vadd
    81ec:	e7 20 20 00 21 8d 	vsel	%v2,%v0,%v2,%v18                   <<<<==== vshuffle

Updated the code to gen the min/max directly.
update performance:
zdnn
  zhigh.Stick, 85, 0.5742549, 48.8116667, 5.6%

compiler gen code without saturation
  zhigh.Stick, 85, 0.3593294, 30.5430000, 3.7%

compiler gen WITH saturation
  zhigh.Stick, 85, 0.3676314, 31.2486667, 3.8%

For a performance reduction of 3% (compared to compiler generated without saturation), and still 55% faster than zDNN.
Verified that it generates the right code
    8050:	e7 11 00 40 2c ee 	vfminsb	%v17,%v17,%v0,4                 <<<<==== vadd
...
    8062:	e7 22 10 40 20 ef 	vfmaxsb	%v2,%v2,%v1,4                   <<<<==== vadd

Jenkins Linux amd64 Build #15138 [push] Saturation in compiler g... started at 21:33

Jenkins Linux s390x Build #15143 [push] Saturation in compiler g... started at 22:33

Jenkins Linux ppc64le Build #14168 [push] Saturation in compiler g... started at 22:44

Jenkins Linux amd64 Build #15138 [push] Saturation in compiler g... passed after 1 hr 13 min

Jenkins Linux s390x Build #15143 [push] Saturation in compiler g... passed after 1 hr 39 min

Jenkins Linux ppc64le Build #14168 [push] Saturation in compiler g... passed after 2 hr 9 min


output video is the same fps

When I run 'test_single_frames.py' or 'test_multiple_frames.py', the number of output frames is the same number of input frames. For example, for a specific video with 20 LR frames, the output would be 20 HR frames in the evaluations folder. I would have thought there would be 40 HR frames, similarly to how zooming slow-mo works.
How do I fix this?

Hi, @nadimra. Although there are N pictures in LR folder and HR folder, the network will sample at intervals of certain frames in LR folder during the test stage, which can make the network to output N images for evaluation.


Open google maps sending street and city name

Hi, is possible (with this) to open maps writing street and city? Instead of lat and long

Duplicate of #5.

Closing this one since it is a duplicate of #5. Nevertheless, this might still be something that we might build in the near future, so please subscribe to #5 to stay up to date.


Label issue for armor resilient runes

In the lang/en.json is it actually PF2E.ResiliencyRuneLabel and not PF2E.ResilientRuneLabel

I also mention this here https://github.com/foundryvtt/pf2e/issues/12230


Addressing a lot of security vulnerabilities in the Cadence release v1.2.6

Version of Cadence server, and client(which language)
This is very important to root cause bugs.

Server version: 1.2.6

Describe the bug
There are several CVEs found in the latest release image 1.2.6
To Reproduce
Is the issue reproducible?

Yes

Steps to reproduce the behavior:
A clear and concise description of the reproduce steps.
Expected behavior
Some vulnerabilities can be fixed by upgrading the version of affected packages as below.

CVE
SEVERITY
CVSS
PACKAGE
VERSION
STATUS

CVE-2016-5397
high
8.80
github.com/apache/thrift
v0.0.0-20161221203622-b2a4d4ae21c7
fixed in 0.10.0

GHSA-m425-mq94-257g
high
7.50
google.golang.org/grpc
v1.47.0
fixed in 1.58.3, 1.57.1,   1.56.3

CVE-2023-39325
high
7.50
golang.org/x/net
v0.7.0
fixed in 0.17.0

CVE-2019-0190
high
7.50
openssl
3.1.4-r1

PRISMA-2023-0056
medium
6.20
github.com/sirupsen/logrus
v1.9.0
fixed in v1.9.3

CVE-2023-3978
moderate
6.10
golang.org/x/net
v0.7.0
fixed in 0.13.0

CVE-2023-48795
moderate
5.90
golang.org/x/crypto
v0.1.0
fixed in 0.17.0

CVE-2023-6992
medium
5.50
zlib
1.2.13-r1

CVE-2023-44487
moderate
5.30
golang.org/x/net
v0.7.0
fixed in 0.17.0

CVE-2023-44487
moderate
5.30
google.golang.org/grpc
v1.47.0
fixed in 1.56.3, 1.57.1,   1.58.3

CVE-2024-0727
low
0.00
openssl
3.1.4-r1
fixed in 3.1.4-r5

CVE-2023-6237
low
0.00
openssl
3.1.4-r1
fixed in 3.1.4-r4

CVE-2023-6129
low
0.00
openssl
3.1.4-r1
fixed in 3.1.4-r3

SEVERITY
DESCRIPTION

high
(CIS_Docker_v1.5.0 - 4.1) Image   should be created with a non-root user

high
Private keys stored in image

Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here, E.g. Stackstace, workflow history.

I'd close the issue since new version is released and addressed many reported CVEs. The vulnerability for new version v1.2.7 is raised in another ticket https://github.com/uber/cadence/issues/5770


Resizing to a really small screen causes a crash.

While loading, before the main menu shows up, if you resize it to a small size can cause the entire terminal to crash on you.
Screenshot taken a second after it crashes:

Update: This can also happen in game aswell.

This issue seems to be Windows-specific, as I've been unable to replicate it on Linux.

Will try and test it more later

Apparently, for some reason, on Windows, resizing a window to a height of 0 terminates the app instantly.
https://user-images.githubusercontent.com/47950669/211149602-16d90091-6db2-498a-b52d-0dfcc4a93045.mp4
This issue therefore seems to not be related to the game itself.


Update pseudocode.txt

Pretty sure functions shouldn't be processed if they have no arguments

Sorry, head not screwed on properly. I'm less convinced about this now that I'm trying to implement the t/f functions


python310Packages.transmission-rpc: 4.1.4 -> 4.2.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.transmission-rpc is: Python module that implements the Transmission bittorent client RPC protocol
meta.homepage for python310Packages.transmission-rpc is: https://github.com/Trim21/transmission-rpc
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 4.2.0 with grep in /nix/store/yqd7vibg6j0pqqqdd0czwbzr82658nls-python3.10-transmission-rpc-4.2.0
found 4.2.0 in filename of file in /nix/store/yqd7vibg6j0pqqqdd0czwbzr82658nls-python3.10-transmission-rpc-4.2.0

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
home-assistant-component-tests.transmission
python310Packages.transmission-rpc
python311Packages.transmission-rpc

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/yqd7vibg6j0pqqqdd0czwbzr82658nls-python3.10-transmission-rpc-4.2.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.transmission-rpc https://github.com/r-ryantm/nixpkgs/archive/a358c71e3e3ae86415b22e1fa0da166cea9795f0.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/yqd7vibg6j0pqqqdd0czwbzr82658nls-python3.10-transmission-rpc-4.2.0
ls -la /nix/store/yqd7vibg6j0pqqqdd0czwbzr82658nls-python3.10-transmission-rpc-4.2.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

3 packages built:

home-assistant-component-tests.transmission
python310Packages.transmission-rpc
python311Packages.transmission-rpc

Maintainer pings
cc @eyJhb for testing.

Addressed by https://github.com/NixOS/nixpkgs/pull/225603