WithClientAddr overwrite WithClientHostname (need to change the from domain)

My preferred external sip registrar Placetel need always in the FROM header @fpbx.de. I can change this by using sipgo.WithClientHostname("fpbx.de")
client, err := sipgo.NewClient(ua, sipgo.WithClientHostname("fpbx.de"), sipgo.WithClientNAT())
Dec 31 21:42:39.543219 DBG UDP write to [::]:5061 -> <IP_ADDRESS>:5060:
REGISTER sip:xxx@fpbx.de:5060 SIP/2.0
Via: SIP/2.0/UDP :::5061;rport;alias;branch=z9hG4bK.3wJ7yK1Bw5TOLjvk
Contact<EMAIL_ADDRESS>P-Preferred-Identity<EMAIL_ADDRESS>From: "xxx" <sip:xxx@fpbx.de>;tag=CrK1AR2OweJrIn1R
To<EMAIL_ADDRESS>Call-ID: d2249932-b169-4c81-916b-fd516b9736e7
CSeq: 1 REGISTER
Max-Forwards: 70
Content-Length: 0

But If I need also to change the source IP and Port with WithClientAddr, the WithClientHostname value is no longer used. This means now the client ip is correct but it will decline the request because of the incorrect source domain.
client, err := sipgo.NewClient(ua, sipgo.WithClientAddr("<IP_ADDRESS>:5061"), sipgo.WithClientHostname("fpbx.de"), sipgo.WithClientNAT())
Dec 31 21:44:29.274552 DBG UDP write to <IP_ADDRESS>:5061 -> <IP_ADDRESS>:5060:
REGISTER sip:xxx@fpbx.de:5060 SIP/2.0
Via: SIP/2.0/UDP <IP_ADDRESS>:5061;alias;branch=z9hG4bK.KC0IV1B063fpY98A;rport
Contact<EMAIL_ADDRESS>P-Preferred-Identity<EMAIL_ADDRESS>From: "xxx" <sip:xxx@<IP_ADDRESS>>;tag=n2SxuHt2U3sVCCCK
To<EMAIL_ADDRESS>Call-ID: 8fc2c304-726a-4fab-962f-007f48ff6b0f
CSeq: 1 REGISTER
Max-Forwards: 70
Content-Length: 0

Dec 31 21:44:29.302124 DBG UDP read from <IP_ADDRESS>:5061 <- <IP_ADDRESS>:5060:
SIP/2.0 404 Not our domain
Via: SIP/2.0/UDP <IP_ADDRESS>:5061;alias;branch=z9hG4bK.KC0IV1B063fpY98A;rport=5061;received=xx.xx.xx.xx
From: "xxx" <sip:xxx@<IP_ADDRESS>>;tag=n2SxuHt2U3sVCCCK
To: <sip:xxx@fpbx.de>;tag=00cf985326c1d88126535b2db992fff3.b816c500
Call-ID: 8fc2c304-726a-4fab-962f-007f48ff6b0f
CSeq: 1 REGISTER
Content-Length: 0

Maybe you have some idea how to change both (source ip and from domain)?

I found some workaround:
	// Setup UAC
	ua, err := sipgo.NewUA(
		sipgo.WithUserAgent(*username),
	)
	if err != nil {
		log.Fatal().Err(err).Msg("Fail to setup user agent")
	}

	client, err := sipgo.NewClient(ua, sipgo.WithClientAddr(*addr), sipgo.WithClientNAT())
	if err != nil {
		log.Fatal().Err(err).Msg("Fail to setup client handle")
	}
	defer client.Close()

	// Create basic REGISTER request structure
	recipient := &sip.Uri{}
	sip.ParseUri(fmt.Sprintf("sip:%s@%s", *username, *dst), recipient)
	req := sip.NewRequest(sip.REGISTER, recipient)
	from := sip.FromHeader{
		DisplayName: *username,
		Address: sip.Uri{
			User:      *username,
			Host:      "fpbx.de",
			UriParams: sip.NewParams(),
			Headers:   sip.NewParams(),
		},
		Params: sip.NewParams(),
	}
	from.Params.Add("tag", sip.GenerateTagN(16))
	req.AppendHeader(&from)
	req.AppendHeader(
		sip.NewHeader("Contact"<EMAIL_ADDRESS>*username)),
	)
	req.AppendHeader(
		sip.NewHeader("P-Preferred-Identity"<EMAIL_ADDRESS>*username)),
	)
	req.SetTransport(strings.ToUpper(*tran))

	// Send request and parse response
	req.SetDestination(*dst)
	log.Info().Msg(req.StartLine())
	ctx, _ := context.WithTimeout(context.Background(), 60*time.Second)
	tx, err := client.TransactionRequest(ctx, req)
	if err != nil {
		log.Fatal().Err(err).Msg("Fail to create transaction")
	}
	defer tx.Terminate()

	res, err := getResponse(tx)
	if err != nil {
		log.Fatal().Err(err).Msg("Fail to get response")
	}

	log.Info().Int("status", int(res.StatusCode)).Msg("Received status")
	if res.StatusCode == 401 {
		// Get WwW-Authenticate
		wwwAuth := res.GetHeader("WWW-Authenticate")
		chal, err := digest.ParseChallenge(wwwAuth.Value())
		if err != nil {
			log.Fatal().Str("wwwauth", wwwAuth.Value()).Err(err).Msg("Fail to parse challenge")
		}

		// Reply with digest
		cred, _ := digest.Digest(chal, digest.Options{
			Method:   req.Method.String(),
			URI:      recipient.Host,
			Username: *username,
			Password: *password,
		})

		newReq := sip.NewRequest(sip.REGISTER, recipient)
		from := sip.FromHeader{
			DisplayName: *username,
			Address: sip.Uri{
				User:      *username,
				Host:      "fpbx.de",
				UriParams: sip.NewParams(),
				Headers:   sip.NewParams(),
			},
			Params: sip.NewParams(),
		}
		from.Params.Add("tag", sip.GenerateTagN(16))
		newReq.AppendHeader(&from)
		newReq.AppendHeader(sip.NewHeader("Authorization", cred.String()))
		newReq.AppendHeader(
			sip.NewHeader("Contact"<EMAIL_ADDRESS>*username)),
		)
		newReq.AppendHeader(
			sip.NewHeader("P-Preferred-Identity"<EMAIL_ADDRESS>*username)),
		)
		newReq.SetTransport(strings.ToUpper(*tran))

		// Send request and parse response
		newReq.SetDestination(*dst)

		ctx, _ := context.WithTimeout(context.Background(), 60*time.Second)
		tx, err := client.TransactionRequest(ctx, newReq)
		if err != nil {
			log.Fatal().Err(err).Msg("Fail to create transaction")
		}
		defer tx.Terminate()

		res, err = getResponse(tx)
		if err != nil {
			log.Fatal().Err(err).Msg("Fail to get response")
		}
	}

	if res.StatusCode != 200 {
		log.Fatal().Msg("Fail to register")
	}

	log.Info().Msg("Client registered")

@danielchristianschroeter I see your point. Currently those options are also used for routing or enforcing source IP, that should be in Via. Maybe here is probably confusion. WithClientAddr and WithClientHostname should not be used together.
Yes you can always add custom any header and it will be skipped during TransactionRequest.
Consider seperation of From and Source IP

The current implemented logic for getting the correct source IPs and listening interfaces is not working very well on my Windows 11 development environment. It will listen by default on IPv6 interface. Even if I try to disable IPv6. Also this internal IP is not reachable from outside. IPv6 is also not supported by the remote SIP registrar and the external IP is not detected.
I‘m now in the situation with IPv4 in a local Ubuntu VM to register and answer a call but no RTP media can be received.
I think I will not have these issues when I‘m outside of a NAT but maybe it makes more sense to get a more flexible and consistent configuration for the internal and external IPs / listing interfaces / used ports.

@danielchristianschroeter thnx for feedback. Can you only be a bit specific.
THere is some things in TODO, like ListenAndServe where you will be able to pass udp4, tcp4, but it may also go as server option to force IPV4 or IPV6
This actually should be quick to fix. For client side I think it needs more work, but this is mostly because initially lib focus was on server.

@danielchristianschroeter added WithUserAgentHostname option for building UA.
In more customization needed, passing manually set headers as you here described is needed.
Still in most cases I see this needs to be set and then reused.
Check main branch.
Closing...


Add input textContentType for registration BillingAddress

During last week's Auctions QA session, I tried to register for a sale on an iPhone. I realized that the billing address inputs lacked textContentType, which is a property that tells iOS what data goes in each input field. This allows users to tap (for example) their name to get it input into the name field; this drastically improves the user experience of entering an address.
Here's what it looks like in the simulator (SIM-VALUE would be replaced by the user's actual data, on-device):

During this work, I noticed that the StyledInput component was not well-typed, so I added some type annotations to get type-checking and autocomplete working:

LGTM, thanks!


Documentation typo "flexbix"

Typo in this paragraph :
http://v4-alpha.getbootstrap.com/getting-started/flexbox/#why-flexbox
flexbix instead of flexbox

Already fixed, we just haven't redeployed the site yet.

OK nice reactivity :)

And yet we're not reactive :stuck_out_tongue_closed_eyes:


Backporting to 5.2

Explain the changes

Backporting to 5.2

@nimrod-becker
Do we want to bump the package.json ?

@liranmauda yes, thanks


add a job to customize gateway for network interfaces

Add a job to configure gateway for network interfaces. This is mainly for the VM which has multiple network interfaces, with this job you will be able to direct traffic out of a Linux host via the specific gateways.

Thanks for the PR! This might make more sense in the networking-release. It looks like there's a gateway job in there already. It seems like this may supersede that functionality. There's also a routes job in networking-release which may have some of this logic as well. Do you think this would make more sense as a PR to that gateway job instead?

Thanks @dpb587-pivotal. I will submit this PR to networking-release.

Followup PR: cloudfoundry/networking-release#14


admin: Fix incorrect param name of HttpResponse constructor

Replace removed 'mimetype' parameter name with 'content_type'

Thanks, this is quite on spot. Invalid since Django 1.7 migration.


Pseudo-elements

I am trying to allow users to snap family trees that are build with CSS -- based on this plugin. Unfortunately, something goes wrong when running html2canvas:

Whereas the expected result would look like this:

As the rest of the CSS is rather straight forward, it seems to me that there's some problems with :after and :before and there positioning. Is that possible, or am I looking in the wrong direction?

Is this still an issue with v1.0.0? If so, could you please share an example on jsfiddle.

I am facing this same issue


ci: try using pip install .[tests]

Codecov Report

Merging #38 (b2d7a40) into main (1eb294a) will increase coverage by 80.24%.
The diff coverage is 81.57%.

:exclamation: Current head b2d7a40 differs from pull request most recent head 531cd9e. Consider uploading reports for the commit 531cd9e to get more accurate results

@@            Coverage Diff             @@
##            main      #38       +/-   ##
==========================================
+ Coverage   0.40%   80.64%   +80.24%
==========================================
  Files         18       25        +7
  Lines        496      682      +186
  Branches      64       89       +25
==========================================
+ Hits           2      550      +548
+ Misses       494       97      -397
- Partials       0       35       +35

Flag
Coverage Δ

Unit_Tests
80.64% <81.57%> (+80.24%)
:arrow_up:

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

Impacted Files
Coverage Δ

truelearn/learning/__init__.py
100.00% <ø> (+100.00%)
:arrow_up:

truelearn/preprocessing/_wikifier.py
34.28% <34.28%> (ø)

truelearn/utils/metrics/_base.py
60.00% <60.00%> (ø)

truelearn/preprocessing/_utils.py
62.50% <62.50%> (ø)

truelearn/learning/_ink_classifier.py
79.71% <72.91%> (+79.71%)
:arrow_up:

truelearn/learning/_interest_classifier.py
78.37% <81.81%> (+78.37%)
:arrow_up:

truelearn/models/_knowledge.py
88.42% <82.14%> (+88.42%)
:arrow_up:

truelearn/datasets/_base.py
82.35% <82.35%> (ø)

truelearn/learning/_novelty_classifier.py
64.51% <83.33%> (+64.51%)
:arrow_up:

truelearn/learning/_base.py
74.54% <86.27%> (+74.54%)
:arrow_up:

... and 30 more

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

Codecov Report

Merging #38 (531cd9e) into main (1eb294a) will increase coverage by 80.24%.
 Consider uploading reports for the commit df29626 to get more accurate results

@@            Coverage Diff             @@
##            main      #38       +/-   ##
==========================================
+ Coverage   0.40%   80.64%   +80.24%
==========================================
  Files         18       25        +7
  Lines        496      682      +186
  Branches      64       89       +25
==========================================
+ Hits           2      550      +548
+ Misses       494       97      -397
- Partials       0       35       +35

Flag
Coverage Δ

Unit_Tests
80.64% <81.57%> (+80.24%)
:arrow_up:

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

Impacted Files
Coverage Δ

truelearn/learning/__init__.py
100.00% <ø> (+100.00%)
:arrow_up:

truelearn/preprocessing/_wikifier.py
34.28% <34.28%> (ø)

truelearn/utils/metrics/_base.py
60.00% <60.00%> (ø)

truelearn/preprocessing/_utils.py
62.50% <62.50%> (ø)

truelearn/learning/_ink_classifier.py
79.71% <72.91%> (+79.71%)
:arrow_up:

truelearn/learning/_interest_classifier.py
78.37% <81.81%> (+78.37%)
:arrow_up:

truelearn/models/_knowledge.py
88.42% <82.14%> (+88.42%)
:arrow_up:

truelearn/datasets/_base.py
82.35% <82.35%> (ø)

truelearn/learning/_novelty_classifier.py
64.51% <83.33%> (+64.51%)
:arrow_up:

truelearn/learning/_base.py
74.54% <86.27%> (+74.54%)
:arrow_up:

... and 30 more

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

Codecov Report

Merging #38 (df29626) into main (1eb294a) will increase coverage by 80.24%.
The diff coverage is 81.57%.

@@            Coverage Diff             @@
##            main      #38       +/-   ##
==========================================
+ Coverage   0.40%   80.64%   +80.24%
==========================================
  Files         18       25        +7
  Lines        496      682      +186
  Branches      64       89       +25
==========================================
+ Hits           2      550      +548
+ Misses       494       97      -397
- Partials       0       35       +35

Flag
Coverage Δ

Unit_Tests
80.64% <81.57%> (+80.24%)
:arrow_up:

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

Impacted Files
Coverage Δ

truelearn/learning/__init__.py
100.00% <ø> (+100.00%)
:arrow_up:

truelearn/preprocessing/_wikifier.py
34.28% <34.28%> (ø)

truelearn/utils/metrics/_base.py
60.00% <60.00%> (ø)

truelearn/preprocessing/_utils.py
62.50% <62.50%> (ø)

truelearn/learning/_ink_classifier.py
79.71% <72.91%> (+79.71%)
:arrow_up:

truelearn/learning/_interest_classifier.py
78.37% <81.81%> (+78.37%)
:arrow_up:

truelearn/models/_knowledge.py
88.42% <82.14%> (+88.42%)
:arrow_up:

truelearn/datasets/_base.py
82.35% <82.35%> (ø)

truelearn/learning/_novelty_classifier.py
64.51% <83.33%> (+64.51%)
:arrow_up:

truelearn/learning/_base.py
74.54% <86.27%> (+74.54%)
:arrow_up:

... and 30 more

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

I dont think there is any need to have the quotes around .[tests] .[linters] you can just write it like that.

I dont think there is any need to have the quotes around .[tests] .[linters] you can just write it like that.

This is to ensure that pip will install truelearn correctly when using zsh. See zsh: no matches found: requests[security].


Table Result View Does not show all data

Describe the bug
Certain types of queries cannot display all results
Steps to reproduce
If possible, provide a recipe for reproducing the error.
Two Queries:
select ARRAY_CONCAT(
    [{"Test": "TestValue"},{"Test": "TestValue2"}],
    [{"Test2": "Test2Value"},{"Test2": "Test2Value2"}]
) as concatArrays

select concatArrays.*
FROM ARRAY_CONCAT(
    [{"Test": "TestValue"},{"Test": "TestValue2"}],
    [{"Test2": "Test2Value"},{"Test2": "Test2Value2"}]
) as concatArrays

What did you expect to see?
I have attached the CB query workbench console compared to vsCode output to show the resulting behavior

We have fixed this bug in the latest release. Thanks!


Add Default Deny Policy example

PR Summary
This PR example adds a default deny k8s manifest to the repo which is referenced by

The Wizard UI (screengrab below)
The BYO network CI/CD sample
The Private CI/CD sample

The following application helm charts have been updated to have Network Policies (in order to work in a namespace with a default deny on egress/ingress).
Azure Vote
Java hello world
CertManagerIssuer
ExternalDns
The Azure Vote workflow has been extended to include a Playwright test to properly test the East-West traffic between the pods.
Other changes
Added an NSG rule to allow public 80/443 access when the AGW is not deployed with a privateIP.
Screengrabs

Other notes
Ridiculous number of commits in this branch, candidate for a squash-merge if I've ever seen one.
CI/CD
Full testing in the branch has happened.
https://github.com/Azure/Aks-Construction/actions/runs/1688055862

https://github.com/Azure/Aks-Construction/actions/runs/1687900578

PR Checklist

[x] PR has a meaningful title
[x] Summarized changes
[x] This PR is ready to merge and is not Work in Progress
[x] Link to a filed issue

Closes #97
Closes #177

Added support for the Run Command, as per our review @khowling


Reduce the number of confusing instances of stackrox.io in charts

Description
@BradLugo got confused seeing registry: stackrox.io in the generated Helm values.
While that confusion is fair, Helm values files are meant to be examples illustrating usage rather than the set of currently selected values. I myself would prefer helm value files to give actual defaults, but this can easily make this change XL-sized.
Instead, I simply get rid of stackrox.io mentions where they may be confusing.
Context thread
https://srox.slack.com/archives/CELUQKESC/p1675462004575229?thread_ts=1675459102.811779&cid=CELUQKESC
Checklist

[ ] Investigated and inspected CI test results

None of these needed:

[ ] Unit test and regression tests added - the change does not deserve ones since values files are examples and not executables.
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

Testing Performed

Manually:

$ make cli
$ rm -rf stackrox-central-services-chart
$ rm -rf stackrox-secured-cluster-services-chart
$ roxctl helm output central-services --image-defaults=rhacs

$ grep -F '#image:' -B1 -A4 stackrox-central-services-chart/values.yaml
## Common settings for all image properties
#image:
#  # The image registry to use. Unless overridden in the more specific configs, this
#  # determines the base registry for each image referenced in this config file.
#  registry: registry.redhat.io/advanced-cluster-security
#

$ grep -F '# image:' -A7 stackrox-central-services-chart/values-public.yaml.example
# image:
#   # The registry relative to which all image references are resolved, unless
#   # a specific registry is provided for particular workloads which takes precedence
#   # (see `central.image`, `db.image`, `scanner.image`, and `scanner.dbImage` below).
#   # This can be just a registry hostname such as `stackrox.io`, or a registry hostname with
#   # a "remote" component such as `us.gcr.io/my-stackrox-mirror`.
#   registry: us.gcr.io/my-stackrox-mirror
#

$ grep -F '`image.registry`:' stackrox-central-services-chart/values-public.yaml.example
# - `image.registry`: if you are pulling images from a registry other than `registry.redhat.io/advanced-cluster-security`.

$ roxctl helm output secured-cluster-services --image-defaults=rhacs
$ grep -F '# Add registry defaults' -A6 stackrox-secured-cluster-services-chart/internal/defaults/50-images.yaml
# Add registry defaults
image:
  main:
    registry: {{ ._rox.image.registry }}
  collector:
    registry: {{ ._rox.image.registry }}

Would it make sense to change the default registry to quay.io/stackrox-io? I see that's what we use for the default in our deploy/common/deploy.sh. If so, I think we can change the default here: https://github.com/stackrox/stackrox/blob/master/scripts/ci/lib.sh#L602
That way users will have a working installation by default. Open to opinions on this

@BradLugo I might misunderstand your suggestion.
The way I read the push_helm_charts() function: it generates three sets of Helm charts for stackrox.io, for rhacs and for opensource image flavors. The respective image registries will be stackrox.io, registry.redhat.io/advanced-cluster-security and quay.io/stackrox-io.
What you seem to be asking should be covered by --image-defaults=opensource lines. Just need to figure out where do these charts get published.
Does this make sense?

Yes, that makes sense. Thank you for explaining


Submitting customized tests

Adjustments to the original tests are necessary e.g., if the class under test is placed in a namepsace.
Is it possible to have the test engine run submitted tests instead of the original one?

As far as I know, no there is no current plan to allow submitted test files to be used in place of the canonical ones.
Is it possible, yes, but it would be exceptional and this would probably be a product decision. @iHiD / @ErikSchierboom

Is it possible to have the test engine run submitted tests instead of the original one?

There are no plans, as this would allow students to just rewrite tests if their implementation does not pass them. This would make mentoring a lot harder too, as you can't rely on on "knowing" the test suite.

Adjustments to the original tests are necessary e.g., if the class under test is placed in a namepsace.

I don't exactly understand this comment. Are there exercises where you'd have to change the tests? Do you have any examples?

@ErikSchierboom I agree. Changing tests adds complexity for the reviewing process of a mentor as well as the automated tests.
I just like to add strictness and clarity to my tests by adding type declarations and putting functions for example in a namespace. So, it is more a personal preference and nothing where changes must be changed.
I close this for the moment as I feel this is not a common request. Thanks for your responses!


Introduce flags for fine-tuning maximum concurrent reconciles per resource

This commit aims to improve the performance and scalability of the ACK
service controllers by introducing support for configuring the maximum
number of concurrent reconciles for individual resources. The primary
motivation behind this change is to address varying workload demands and
resource requirements in Kubernetes environements.
This patch introduces two new flags (soon exposed to in the helm chart
values):

--reconcile-default-max-concurrent-syncs: allow users to specify the
default maximum concurrency level for all the resources.
--reconcile-resource-max-concurrent-syncs: enable users to define
resource-specific maximum concurrency settings, overriding the default
value.

A use case example would be the scenario of an admin wanting to manage
EKS resources using the ACK eks-controller. It is normal for each
cluster to have multiple nodegroups/PIAs/AccessEntries/FargateProfiles,
which can indicate the proportion of needed max concurrencies for each
resource.
For instance, you can configure the EKS Controller with a default
maximum of 2 concurrent reconciles for all the resources, and override
the maximum concurrency to 10 for AccessEntries and
PodIdentityAssociations
Signed-off-by: Amine Hilaly<EMAIL_ADDRESS>By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/lgtm
/test all


https://microsoft.github.io/dowhy/ lead to 404

The official documentation is not accessible under the link https://microsoft.github.io/dowhy/. The link leads to 404 error.

We are in the process of moving dowhy to a new org, py-why. You can access the docs at the new link: https://py-why.github.io/dowhy
Unfortunately, github does not allow redirection from the old docs page.


GE: Set up GeoEvent on National Security Portal

From @lfunkhouser on October 3, 2017 19:24

[x] Check status of the GEE component for NatSec Portal - Avi
[ ] Verify GEE and NatSec is working - Avi
[x] Fix GEE so it's up and running. - Avi
[ ] Create user logins for whole team. - Avi
[ ] Update wiki instructions on how to connect:
https://github.com/ArcGIS/national-security-development-portal/wiki

Copied from original issue: ArcGIS/national-security-development-portal#12

From @ACueva on October 5, 2017 17:40
I've identified the server that has GEE 10.5 installed on it, unfortunately I wasn't the one that installed it and I'm not sure of the credentials for ArcGIS Server and GEE.  @elinz would you happen to know this information?

From @ACueva on October 5, 2017 22:40
GEE is up and running.  Working on creating username/passwords for the team to be able to login to GEE.

From @ACueva on October 5, 2017 20:57
I was able to reset the admin password for the site.  I'm able to login to the ArcGIS Manager page, GEE is still not coming up.  I'm in the process of troubleshooting this issue. FYI @pHill5136


Support for JSON + SockJS

See #19 and #18.
Florent

See some comments inline, but overall it looks great!
Honza

Tell me when you're happy with the changes, I'll rebase my commits in order to just have two.
Florent

Looks good!
Honza

I rebased the commits. You can accept the pull request now :)

Awesome!
Honza


Mark linux_docs_test as flaky

Flaky: https://github.com/flutter/flutter/issues/84609
Pre-launch Checklist

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] I read the Tree Hygiene wiki page, which explains my responsibilities.
[ ] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[ ] I signed the CLA.
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test-exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Seems a lint error needs fixed.


Make use of paginated API by default on list-projects and list-jobs

Task linked: QF-2939 Make qfc-sdk use the new paginated API for projects by default, since the non-paginated API is (semi-)deprecated


I am getting 404 error

Hi,
I tried to use the SAP Cloud SDK in my SAP CAP application to send an email using destinations in SAP BTP. However, I am encountering the below issue. Kindly guide.
Please find attached herewith the screenshots of code snippets.

Please assist in this regard at an earliest.
Regards,
Syed

Hey @SyedBabaTajuddinHussaini,
it seems that the 404 comes from your app because you cannot POST against /mail/sendMail. Is there an actual error from the SAP Cloud SDK? I might be wrong but from the logs you shared it even looks like the sendMail() function is never reached. Could you check this and share a few more details?

Hi @marikaner,
You were correct. The issue has been fixed.
Thank you.


Fix 2nd Harmonic Minor

I think I fixed this.  Can you confirm?
https://earthmusicguide.com/harmonic_minor_2/

Holy smokes!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
Yes it's perfect.  Please let me pay you.  And talk on the phone sometime.
The human element.
On Tue, May 2, 2023 at 12:15 PM Tony Buckingham @.***>
wrote:

I think I fixed this. Can you confirm?
https://earthmusicguide.com/harmonic_minor_2/
—
Reply to this email directly, view it on GitHub
https://github.com/Christopher-Lacey/christopherlacey-web/issues/10#issuecomment-1531848742,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A35UY36XH2VZRXFSSTOA53LXEE6JJANCNFSM6AAAAAAXTKRIEM
.
You are receiving this because you were assigned.Message ID:
@.***>

--
Christopher Thomas Lacey
415-902-3588


Map - Live Filter Search Feature

Adds live search box to the top of the map.  Filters as you type.  Minor improvements to map spacing/margins.
Tested locally on Windows Chrome, Firefox, IE11.
Tested locally on iPad Safari (iOS 9.2.1) and Chrome Mobile (Tablet).

Closes #6586

Test ran on OSX Chrome, Safari, Firefox and Opera. iOS Chrome, Safari and Opera Mini (This last one is glitchy as expected, but it get's the filtering done, etc).

Fixed indentation and line length issues.

When I type something (hello for example) it live-searches and shows the result but when I refresh the page without removing #map-filter's text, search box still has that word (hello) in it but doesn't show any search results, instead it shows the whole map. This is natural because live-search works on keyup. But #map-filter is live-search feature so what do you think about either auto-searching on page load if #map-filter has text in it or clearing #map-filter's content on page load?

@bugron Clearing the search box might be better.

@AkiraLaine yep, I think so too.

After some research I added autocomplete="off" and value="" to the form, which should work in most browsers.

Latest Changes:

Changed Placeholder text to "Type a challenge name"
Set auto-focus on the search field on page load
Pressing esc either in or out of the text box clears the search

Awesome work @SaintPeter @hallaathrad. Merging!


Enable FIPS mode automatically

FreeRDP aborts if OpenSSL operates in FIPS mode and +fipsmode is not manually specified. Let's prevent the abortion and enable the necessary options in that case automatically.

@akallabeth Sure, done.


Fake disks capacity not supporting 4K logical sector size.

When using fake disks with a drive which has logical sector size of 4K, the capacity calculation when performing a set zones is incorrectly using the 4K sector size instead of 512

Good catch.
But your fix is equivalent to saying that zbd_lblock_size is 512. It is 4K in your case isn't it ?
Checking the code again, I noticed that the line 1307 is wrong anyway:
dev->zbd_info.zbd_lblocks = fdev->zbd_nr_zones * zone_sz;
zone_sz is 512B sector unit, so this ends up assuming that lblocks are 512B. Fixing this line, the remaining of the code should be correct as is. That is:
dev->zbd_info.zbd_lblocks = (fdev->zbd_nr_zones * zone_sz) / dev->zbd_info.zbd_lblock_size;
should fix the problem. Which also points to a missing check to see if the zone size aligns to the logical block size of the backing device.
I will push a fix asap.

Doesn't that make zbd_lblocks a ratio of 512-sector unit over 4K byte size?
So to truly get the zbd_lblocks you would need:
dev->zbd_info.zbd_lblocks = (fdev->zbd_nr_zones * zone_sz * 512) / dev->zbd_info.zbd_lblock_size;
Thanks

Yes, correct. I was trying to correct a bug with another bug :)
I just pushed some modifications to address the problem, and also enforce checks for set zones operation. Please check and confirm that this fixes the problem for you. Everything seem OK on my side with 512n, 512e and 4K drives, as well as regular files.

I made a comment - I think the zone descriptors also need to be in 4K logical sector size. Right now they are in the zone_sz 512 byte blocks.
https://github.com/hgst/libzbc/commit/700d1b55dd927c5e61af93eebea3f5f46b208c51

You are correct: a disk would use the logical block size (which may not be 512B) as the unit for a zone descriptor zone start, length and write pointer. But the fake backend uses directly the API unified 512B sector size in the metadata to avoid conversions between 512B and logical block size. This is just a simplification of the code and in no way does this reflects the working of a real disk. This a "fake" disk after all !
Thank you for looking into the code and helping improving it.

This still doesn't seem right to me. The sum of the zone descriptor lengths should equal the number of logical blocks; now we're mixing 4K logical blocks in the header and 512-byte sectors in the zone descriptors. Shouldn't we accurately emulate the zone descriptors to logical sector size as defined by T10?

The device info structure has the field zbd_sectors which represents the capacity of the disk in 512B sector unit, and that is equal to the sum of the all the zone descriptor length. Since v5, all functions of libzbc have been moved to use 512B sector unit, regardless of the disk LBA size. The values in the zone descriptors returned by zbc_report_zones() is no exception and the values of the start, length and wp fields of the zone descriptors are always converted to 512B sector unit (see for example zbc_scsi.c around line 445 and the use of zbc_dev_lba2sect()).
The fake backend driver purpose is to implement this API on top of a file or a regular block device. It is not to emulate perfectly a ZBC disk as defined by T10. Hence the direct use of the 512B sector unit in the zone descriptors rather than the chosen LBA size (which may be arbitrary or reuse the backing store LBA size). That is maybe a little confusing and better comments in the code could avoid this. I will add more explanation.
If you are more interested in a real full emulation of a ZBC disk (following T10 specs), I recommend that you take a look at tcmu-runner which has since a few weeks ago a ZBC handler to emulate a host aware or host managed ZBC disk over a file.
https://github.com/open-iscsi/tcmu-runner
This is a fully compliant emulation so that the kernel SCSI stack sees a well behaved ZBC disk through the target code connection. Please see tcmu-runner README file to get more information on how to set the ZBC handler.

Thanks for the through explanation.  👍


[No QA] Use lowercase "id" for policy object in Onyx

Details
Came across this minor inconsistency here for the policy objects in Onyx.
Fixed Issues
No issue
Tests
QA Steps
Nothing to QA yet as nothing is using this data yet AFAIK
Tested On

[x] Web
[ ] Mobile Web
[ ] Desktop
[ ] iOS
[ ] Android

Screenshots

Web

Mobile Web

Desktop

iOS

Android

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging in version: 1.0.71-1🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production in version: 1.0.73-3🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


fields missing in notice-subtype form definitions

The TransmissionDate & TransmissionTime fields are shown in the notice examples, and they are defined in the conceptual model, BUT they are not included into any of the notice subtype form definitions:
<!-- examples/notices/can_24_maximal.xml -->

	<ext:UBLExtensions>
		<ext:UBLExtension>
			<ext:ExtensionContent>
				<efext:EformsExtension>

					<!-- "id" : "BT-803(d)-notice", "name" : "Notice Dispatch Date eSender (date)"-->
					<efbc:TransmissionDate>2023-04-23+01:00</efbc:TransmissionDate>
					<!-- "id" : "BT-803(t)-notice", "name" : "Notice Dispatch Date eSender (time)"-->
					<efbc:TransmissionTime>21:00:00+01:00</efbc:TransmissionTime>

field ids:

BT-803(d)-notice
BT-803(t)-notice

Ok, we will look into this for the visual model, possibly do something about this in 1.9


CI: run cilium-docker in debug mode by default in CI

Change log-gathering command for cilium-docker.
Add "--debug" to cilium-docker service file to have it run in debug mode by default.

Signed-off by: Ian Vernon<EMAIL_ADDRESS>Relates-to: #3233

test-me-please

test-me-please

rebased to master with new Kafka CI fixes.

test-me-please

All tests passed.


Task Scheduler

So this is more of a general feature request/enhancement (also a question), but something that would be extremely useful in my use cases would be having a task scheduler so that scripts could, for example, yield back to the engine for a specified duration (like a sleep function). This could also potentially be used for calling callback functions bound to specific events through functions, rather than having specific hook functions.
An example of this might look like this:
-- another benefit of this is having multiple callbacks defined per script
physics_stepped_event:bind(function()
  -- do something
  sleep(5) -- sleep for 5 seconds
  -- do something else
end)

Rather than the current hook-based system:
-- it's a hook so it can only be defined once per script
function physics_stepped_event()
  -- do something
  -- no idea how you'd wait here, maybe call a function and pass the name of another hook to call as well as the time to sleep?
end

I do know this isn't currently possible with the current language implementations, but could something like this be implemented on top of the existing traits? I do know the current system probably works better under Bevy's ECS system, but for me the benefits from having this outweighs the extra work I'd have to do to create the necessary APIs.
It's something that would be extremely useful for my projects, and would be something I would consider implementing myself as well (and of course I'd be willing to consider submitting a PR if you think it'd be a worthy addition). I just don't know enough about how the code is structured to know if I could implement this on top of it or not.

I submitted a PR (#34) that allows for asynchronous functions, which would theoretically allow the scripts to yield in the background. I believe this is required to implement the task scheduler properly. The only issue is that the values returned by mlua are locked to the lifetime of the Lua struct, and cannot be passed between threads.
I don't know if you happen to know a solution to this issue, or if bevy just isn't a suitable choice for what I'm trying to do. Essentially I need a way of storing all of the Lua contexts and return values on a single thread. I did have the idea of just spawning a background thread that would own everything, but passing messages back and forth seems somewhat messy and I feel like it would defeat the purpose of this crate.

@katk0smos, are you trying to keep a hold of and pass around mlua threads? The way Bevy works, resources/components must be 'static i.e. have no lifetimes attached.
Currently all the scripts added via this plugin are stored in ScriptContexts inside of Mutexes. I am not sure what you mean by storing all of them on a single thread, when the Bevy game loop runs, systems are parallelized and run in their own threads, for this plugin, because the handler systems  are exclusive, they have sole control over the contexts when they run, i.e. all of these contexts are controlled by a single thread and events are handled in sequence.  The plugin does not dictate what the ScriptHost's do with the events, and the hosts can do whatever they want in the handle_events hook.
I am guessing you're trying to get Thread as a return value from the scripts? You won't be able to store that in a resource or anything, but I believe you should be fine manipulating those threads within the context of a single handle_events hook (although I am not very sure, I've not tried anything like this).
But then I am just guessing, it's a bit hard for me to imagine what you're trying to do (some examples would help), may I ask why coroutines don't suit your use case?

I think Bevy just may not be designed for what I'm attempting to do. I was looking to create a Lua API similar to that used in Roblox,
Since I don't believe Bevy will work for what I'm attempting to do with this, I'm just going to close this issue. Thank you for all the help!

@katk0smos looking at the roblox task scheduling, I'd say you'd be able to get the delay mechanics working, i.e. if all you want to do is yield an event call, put it in some queue then immediately after all normal events are handled handle that queue, that should be possible:
say you have:
function EventA()
return my_function_or_coruotine
end

EventA() is called
returned function is put on queue of "delayed" events
once all normal events are handled, call all of the delayed functions/coroutines untill the queue is empty

This sort of behaviour should be possible entirely within the handle_events hook, anything that requires tasks to be delayed through another frame I would just handle with coroutines as I've shown above


onDelete is not working for custom cells (fixed in main but not released!)

Hi all, thanks for this great package!
We're having issues with onDelete and custom cells. According to #578 it has been fixed in main, however there hasn't been a release to NPM with this. Could we get a patch release to get this code out there? I'd prefer to not have to point to a git branch in my package.json...
Thank you!

Update: to others looking at this, they did release it as a beta tag! Not sure why I didn't think of this before...
<EMAIL_ADDRESS>


NFT display tweaks

fixes #1494
also support nft.image key, as nft.image_data is meant for raw SVG image data
change no image found messaging
show OpenSea link even if no image has been set
typo fixes

This asset has empty tokenURI (for whatever crazy reason) and we were showing "This NFT was not created in Ocean Market" which was simply wrong. So replaced with this more generic message as we are checking exactly for this:

More important to get the fixes in here out, than figuring out just for this fix how to mock all our hooks for the tests

Issues in the publish preview:

owned by
accessed by
it is shown as algorithm instead of data set

tokenURI is empty so where does opensea get the image?

good find for #1515. OpenSea image is not the completely correct one either as it's scaled down. Would assume OpenSea picks it up from some other related asset or so


Made wesite Navbar with logo aswell as the home page with images buttons and styles elements

In this pull request I am submitting my project based off of figma design file 3 provided by Microverse.

I've added:
Basic html skeleton
background images and attractive colors to elements of the site
added buttons with animations
added social links with animations
added text logo based of of design recommendation
added burger menu design (Interactive)

Hi @andreacostabz ,
Great work on making the project requirements 👏
There are still some requirements that aren't done yet, please work on it and ask for another review.
Required Changes ♻

[] Please, consider my comment under review

Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.
Optional suggestions

N/A

Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

@elmar8287  I can’t see any of the changes you requested. Could you send them again? Cheers

@elmar8287 the only recommended change I see from you is literally just a pair of brackets[].

@andreacostabz If you scroll up to comments from the previous reviewer, you can see my comment under it.

@andreacostabz do not hesitate to ask me additional questions.

@elmar8287 great! Just saw the comment. Is that the only thing I’ve gotta fix?
i definitely dont want to miss something  mess up my last review request.

@andreacostabz that is nice 👍

hey @Davidosky007 !!
Thanks for taking your time to review my request! I appreciate your help. Have a awesome day my friend!


Reduce Latency in RTMP streamings

Describe the bug
I'm working on a low-latency streaming service.
Comparing an RTMP streaming with Cicada (iOS) versus FFPlay, the Cicada stream is always ~1 second behind FFPlay.
For FFPlay I use -fflags nobuffer, is there a way to pass a similar flag to Cicada or to the ffmpeg service inside it?
Could there be any other reason or configuration option why Cicada is behind FFPlay?
To Reproduce
Steps to reproduce the behavior:

Open an RTMP streaming with Cicada
Open the same streaming with FFPlay and -fflags nobuffer
See there is a diference and Cicada has more latency.

Expected behavior
Both streams should have similar latency
Smartphone (please complete the following information):

Device: iPhone 12 Pro
OS: 14.5
Version [e.g. 22]

AVFMT_FLAG_NOBUFFER will drop the packets read in avformat_find_stream_info(), you can add this flag to mCtx before avformat_find_stream_info for testing it , but I don't think this is a good method to reduce the latency, I think it will lead to slow down the first frame time, because it will drop the first key frame before decode, and the latency can't be keep by this way. You can use maxDelayTime in MediaPlayerConfig to keep the latency


Electron Window opens, yet, NO Forza data transmitting to window.

Describe the bug
The Electron window opens with all the elements within the window, yet, when I check on the "View/Toggle Developer Tools", it provides and error of "react-dom.development.js:67 Warning: validateDOMNesting(...):  cannot appear as a child of . Add a ,  or  to your code to match the DOM tree generated by the browser." at Line 67 of the "react-dom.development.js" file inside the "react-dom/cjs" directory.
I have not been able to figure this issue out, though I spent much time with my friend Google..lol...!
To Reproduce
Steps to reproduce the behavior:

npm run build:react | npm run build | npm run start
Click on 'View/Toggle Developers Tools'
Errors and Warnings are displayed on bottom, along with information on the culprit file(s).

Expected behavior
A expect that the data from Forza would be transmitted to and rendered by Electron.
Screenshots

Desktop (please complete the following information):

OS: MacOS Sonoma 14.2.1
Game: Forza Motorsport 2023

Additional context
I have used the RECORD button on the Electron window, yet, when I go to the *.csv file, only the titles of the Data Out from Forza are within the file (thus, no data). Not sure if the error explained above will address this data receiving/recording part of the SW package.

What output style are you using? Dash format with the new Forza changed a bit and needs to be handled in the data_packet file.

Sorry forgot to capture a method in my first comment

Hi Jeremy, thank you very much for looking into this.
I am using the DASH format (Data Out) on my FM settings.
I will check that file.
Again, thank you!

Just realized you were talking about the project you pulled together, and not what Austin has developed which is what I am currently using. However, does seem your pulling together some pretty cool stuff for the "backend" in order to get access to the Telemetry data. Thanks!

Just realized you were talking about the project you pulled together, and not what Austin has developed which is what I am currently using. However, does seem your pulling together some pretty cool stuff for the "backend" in order to get access to the Telemetry data. Thanks!

I'm so sorry I got myself a bit mixed up. I was thinking that I had branched from Austin's project so assumed my work of getting it running with FM8 would work for you. My project is actually python based rather than JS and C based but I believe the issue remains the same. FM8 changed the data packet format and added 5 new values to the dash output packet. It appears that Austin already made a switch for this new format in the code but I don't see the new values available in the dataPacket class.
Here are the new values directly from the Forza data out documentation

Looking at Austin's code the data packet formatting ends at the driving assist lines even when under the new FM8 format

@austinbaccus please let us know if we are off base or are missing something here.

Thanks Jeremy! Your response explains it spot on! I will take a look this file and edit for some testing. Will let you know if it works.
I am also wondering if these 5 new data elements are also set somewhere else.

Just made the edits to the DataPacket.cs file and proceeded to perform a fresh "npm install", followed by the run command "". It still has the same issue, and, the data file is empty (only including column headers).

It appears that there is also data packet information in the following files

FMData.cs
Program.cs

I would venture to guess you would need to add similar lines in each of those files to get it running. However, being unfamiliar with cs I'm a bit unsure of where else to dig for dependencies.

I will give that a try, thank you! I am also unfamiliar with cs.....just learning & troubleshooting.


When using the flag: useFEAPIDefinition the exported API-Spec must contain the backend base path

User story
If an API is exported with the flag: -useFEAPIDefinition, then the API specification of the Frontend API is exported. That is, in the format in which it would be presented to the consumer of the API. This cannot really be used to import it into another API-Manager.
An example API in the API Manager:
The original Backend has the base path: me/mapping/v4 and is configured with the backend: https://my.backend-server.com/restForwards. This creates the following config:

This is how the Frontend-API is configured:

When exporting this API, having the toggle: -useFEAPIDefinition set, the exported API specification (Swagger) contains the Frontend API exposure path (/my/api), which is incorrect, as it doesn't really reflects the actual API in API-Manager. Example:
{
  "swagger" : { "2.0",
  "info" : {
    "description" : "",
    "version" : "1.0",
    "title" : "My API"
  },
  "host" : "api-env.demo.axway.com:8065",
  "basePath" : "/my/api",
  "schemes" : { "https" ],
  "paths" : {
    "/items" : {

The API-Config looks like this:
{
  "name" : "My API",
  "path" : "/my/api",
  "state" : "unpublished",
  "version" : "1.0",
  "organization" : "API Development",
  "apiDefinition" : "My API.json",
  "backendBasepath" : "http://my-backend.host.com/restForwards",
  "clientOrganizations" : [ "API Development" ]
}

Solution approach
The CLI should export the original backend basepath by default in the exported API-Specification, even if the Frontend API is exported. Based on the example above the basePath must say /me/mapping/v4:
{
  "swagger" : "2.0",
  "info" : {
    "description" : "",
    "version" : "1.0",
    "title" : "My API"
  },
  "host" : "api-env.demo.axway.com:8065",
  "basePath" : "/me/mapping/v4",
  "schemes" : [ "https" ],
  "paths" : {
    "/items" : {

This behavior is supposed to be new default if the toggle: useFEAPIDefinition is set, but can be disabled via an option.

For reference, a very simple Backend API Definition. The fields: resourcePath and basePath are used to adjust the Frontend-API specification.
{
  "id": "8a624c14-0b7f-401d-92eb-6c57d1ca64d8",
  "name": "My API",
  "summary": "",
  "description": "",
  "version": "1.0",
  "basePath": "https://my-backend.host.com",
  "resourcePath": "/me/mapping/v4",
  "models": {
    "Item": {
      "type": "object",
      "required": [
        "id",
        "name"
      ],
      "properties": {
        "id": {
          "type": "string",
          "description": "The item's unique identifier"
        },
        "name": {
          "type": "string",
          "description": "The item's name"
        }
      }
    },
    "ItemArray": {
      "type": "array",
      "items": {
        "$ref": "Item"
      }
    }
  },
  "consumes": [

  ],
  "produces": [

  ],
  "integral": false,
  "createdOn":<PHONE_NUMBER>607,
  "createdBy": "f60e3e05-cdf3-4b70-affc-4cb61a10f4bb",
  "organizationId": "439ec2bd-0350-459c-8df3-bb6d14da6bc8",
  "serviceType": "rest",
  "hasOriginalDefinition": false,
  "importUrl": null,
  "properties": {

  }
}


Add instructions for compiling with Ubuntu 18.04 -- Anaconda

Thanks for sharing, this is really impressive work.
I got this running on Ubuntu 18.04 and can share my steps for getting it to work. Maybe you'd like to include these steps in INSTALL.md.

Remove the -D_GLIBCXX_USE_CXX11_ABI=0 flag for each line in tf_custom_ops/compile_op.sh. I think this problem is from Ubuntu 18.04 having a new version of gcc.
Also, the specific version of cuDNN you mentioned (cuDNN 7.4) wasn't available on Anaconda, so I found that 7.3.1 works fine with CUDA 9.0 and TF.12. The problem seems to be in 7.5 and 7.6 (and potentially future versions too).

Hello @noahtren,
Thanks a lot for your help. I will add these steps to INSTALL.md.


add react storybook

[x] Has Knobs
[x] Compiles Production code with Rollup
[x] Compile Development code in Storybook

Ignoring the codeclimate dupe issue for now as the options should be merged and this issue will be resolved when that fix is completed. At least for now a complete object that matches the Fuse.js pattern is expected.


This seems to fail could you make an example for where it works?

["quartz.jobStore.type"] = $"{nameof(RedisJobStore)}, {nameof(JobStore)}",

Hey @killerfrienddk, sorry the delay, I just pushed a new package's version in nuget, could you please update your version?
About the JobStore Type, It was wrong, thanks! You can use the following:
["quartz.jobStore.type"] = "QuartzRedis.Store.RedisJobStore, QuartzRedis.Store"
Let me know if you need any help!

hey @killerfrienddk, I am closing the issue, but if something went wrong just re-open it


Removed the terms and condition text on login page

In this PR terms and condition text on login page below Google sign in button has been removed as it looked like clickable link but didn't redirect any page and we don't have any term & conditions of our own as we are using google sign in only.

seems like the right thing to do to me, @amurphy-cl thoughts on just removing the link?

Yeah, I also think it's fine to remove it. Or maybe replace the text with something else like "To learn more about UpGrade, please visit www.upgradeplatform.org".
Before:

After:

Demo App Login (For reference):

Linking to the website seems fine.


Could not find a version that matches celery<4,<4.0,==3.*,>=2.5,>=3.1.0,>=3.1.15,>=4.0

We have our celery version pinned to 3.* but pip-tools fails with the above error.
Environment Versions

OS Type: Debian Jessie Docker
Python version: Python 2.7.14
pip version: pip 9.0.1 from /usr/local/lib/python2.7/site-packages (python 2.7)
pip-tools version: pip-compile, version 1.9.0

Steps to replicate

Pin celery like so in requirements.in
celery==3.*
Run pip-tools --upgrade

Expected result
Celery pinned to 3.x version
Actual result
Could not find a version that matches celery<4,<4.0,==3.*,>=2.5,>=3.1.0,>=3.1.15,>=4.0
Tried: 0.1.2, 0.1.4, 0.1.6, 0.1.7, 0.1.8, 0.1.10, 0.1.11, 0.1.12, 0.1.13, 0.1.14, 0.1.15, 0.2.0, 0.3.0, 0.3.7, 0.3.20, 0.4.0, 0.4.1, 0.6.0, 0.8.0, 0.8.1, 0.8.2, 0.8.3, 0.8.4, 1.0.0, 1.0.1, 1.0.2, 1.0.3, 1.0.4, 1.0.5, 1.0.6, 2.0.0, 2.0.1, 2.0.2, 2.0.3, 2.1.0, 2.1.1, 2.1.2, 2.1.3, 2.1.4, 2.2.0, 2.2.0, 2.2.0, 2.2.1, 2.2.1, 2.2.1, 2.2.2, 2.2.2, 2.2.2, 2.2.3, 2.2.3, 2.2.3, 2.2.4, 2.2.5, 2.2.6, 2.2.7, 2.2.8, 2.2.9, 2.2.10, 2.3.0, 2.3.1, 2.3.2, 2.3.3, 2.3.4, 2.3.5, 2.4.0, 2.4.1, 2.4.2, 2.4.3, 2.4.4, 2.4.5, 2.4.6, 2.4.7, 2.5.0, 2.5.1, 2.5.2, 2.5.3, 2.5.5, 3.0.0, 3.0.1, 3.0.2, 3.0.3, 3.0.4, 3.0.5, 3.0.6, 3.0.7, 3.0.8, 3.0.9, 3.0.10, 3.0.11, 3.0.12, 3.0.13, 3.0.14, 3.0.15, 3.0.16, 3.0.17, 3.0.18, 3.0.19, 3.0.20, 3.0.21, 3.0.22, 3.0.23, 3.0.24, 3.0.25, 3.1.0, 3.1.1, 3.1.2, 3.1.3, 3.1.3, 3.1.3, 3.1.4, 3.1.5, 3.1.6, 3.1.7, 3.1.8, 3.1.9, 3.1.9, 3.1.10, 3.1.10, 3.1.11, 3.1.11, 3.1.12, 3.1.12, 3.1.13, 3.1.13, 3.1.14, 3.1.14, 3.1.15, 3.1.15, 3.1.16, 3.1.16, 3.1.17, 3.1.17, 3.1.18, 3.1.18, 3.1.19, 3.1.19, 3.1.20, 3.1.20, 3.1.21, 3.1.21, 3.1.22, 3.1.22, 3.1.23, 3.1.23, 3.1.24, 3.1.24, 3.1.25, 3.1.25, 4.0.0rc3, 4.0.0rc3, 4.0.0rc4, 4.0.0rc4, 4.0.0rc5, 4.0.0rc5, 4.0.0rc6, 4.0.0rc6, 4.0.0rc7, 4.0.0rc7, 4.0.0, 4.0.0, 4.0.1, 4.0.1, 4.0.2, 4.0.2, 4.1.0, 4.1.0

Judging from the output it looks like it would have a hard time finding something >= 4.0 and also less than < 4.0

found it was the package django-celery-email being upgraded from 1.x to 2 requiring celery 4
but my django-celery-email line is unpined
so to satisfy requirements why didn't it pick the 1.x instead of trying to go to 2 since that is incompat?

pip-compile will act like pip: If no constraints are given, it will fetch the latest version of a package.

why didn't it pick the 1.x instead of trying to go to 2 since that is incompat?

Because pip-compile does not try to "backtrack" to find a matching version.
It's a shame, but it's not a trivial thing to do. See #397 for a bit more info.
In your specific case, I'd say downpinning django-celery-email <2.0 in your requirements.in would be your solution.
Hope this helps!

yep down pinned worked
instead of backtracking then, since it's hard
couldn't it display which package(s) are causing a conflict
or even just display which packages had any version specifications when it fails to find a package
so I could see which ones are probably the culprit
turned on verbose and had to grep the output of a big list (lots of requirements)

There's an issue on that: #473. It does gets asked a lot, and I agree it would really help.
I had started some work on this, but had no time to work on it for a while.
The general idea would probably to populate the InstallRequirements.comes_from property and to display it on failure, although it gets messy. grep is indeed your best friend right now.
I'll admit I'm waiting to #531 to be resolved before putting more energy in new features right now.
Contributions are still welcomed though!

Closing, keeping #473


Style inheritance and cascading

Thanks for opensourcing this, however, I do have a question. I’m not sure if anybody done this before and if it would make sense in react-canvas or react-native, but what about CSS-like inheritance and cascading of styles? Even though, everything seems like big step forward in performance and app development, styling, in my opinion, seems more complicated and difficult. Am I wrong?

You can export your styles in a new file, and require them. Since they are only Object you can merge them and have inheritance.
You can do whatever you want, make a function to load your styles objects, merge them etc...?

Cascading styles are something I've tried to avoid. Not only due to the added performance cost, but it often leads to bugs where updating styles can have unexpected side effects downstream.
That being said, you might take a look at the css-layout example to see how having a minimal subset of CSS (margins/padding/flexbox) changes the way components are styled.

@podo - see: https://speakerdeck.com/vjeux/react-css-in-js, everything is moving in this direction right now.

Thanks!


DM Not allowed

Hi people,
I've been using twython for quite some time now but lately I get an error message when trying to send a Direct Message to a new Twitter follower.
Twythonerror: Twitter API returned a 403 (Forbidden). There is an error sending your message. DM commands are no longer allowed!
How do I solve this? If this can be solved :-)
/Stefan

Can you share the piece of code you are trying to use?
The documentation that is linked to the twitter api is no longer available so a lot of documentation links need to be updated.


Add targets in the CLI

Implement to the kaprien-cli the functionality to add targets
It can be useful to work with CI/CD or when the admin need to reprocess/process a corner case.
Two specific feature to add targets:
Add Targets (Feature)
This target adds the target using the RSTUF REST API. This feature is an interface to add targets using the command line.

Simple one target file
kaprien admin add-targets -p/--path [file-path], -l/lenght [length] -h/hashes [b2sum] -c/--custom [dictionary]
From JSON
kaprien admin add-targets -d @file.json

{
  "targets": [
    {
      "info": {
        "length": "int",
        "hashes": {
          "blake2b-256": "str"
        },
        "custom": {
          "key": "value"
        }
      },
      "path": "str"
    },
    {
      "info": {
        "length": "int",
        "hashes": {
          "blake2b-256": "str"
        },
        "custom": {
          "key": "value"
        }
      },
      "path": "str"
    },
  ]
}

Import targets (Feature)
This feature adds a huge amount of targets, importing from CSV directly to RSTUF Database
The feature description is detailed in these links

https://github.com/vmware/repository-service-tuf/issues/188
BDD Feature: https://github.com/vmware/repository-service-tuf/pull/218

Add a feature rstuf admin add-target option to import directly to DB. It is related to https://github.com/vmware/repository-service-tuf/issues/188

CLI Implementation
The RSTUF user will be required to create a CSV file that contains the following information
path;size;hash algorithm;hash;
Example:
test/optimistic_lamport-v1.0.tar.gz;12345;blake2b-256;716f6e863f744b9ac22c97ec7b76ea5f5908bc5b2f67c61510bfc4751384ea7a;
The CSV file limit will be half-million of targets (500,000 lines)
The CLI will implement as an admin command, and it will require some parameters
metadata-url: Address where the CLI can download the RSTUF TUF Metadata. i.e.:  http://<IP_ADDRESS>:8080
db-uri: the RSTUF SQL DB URI. i.e.: postgresql://postgres:secret@<IP_ADDRESS>:5433
csv the CSV path. i.e targets-1of2.csv
Example:
rstuf admin add-targets -metadata-url http://<IP_ADDRESS>:8080 -db-uri postgresql://postgres:secret@<IP_ADDRESS>:5433 -csv ../repository-service-tuf/tests/data/targets-1of2.csv
The CLI will insert all the targets to the RSTUF SQL DB directly.
Running in a Macbook Pro (2019) 2,4 GHz 8-Core Intel Core i9/32GB 2667 MHz DDR4
import time: ~15 seconds

I updated the Issue description and the comment above.

I think we can close that one, am I right @kairoaraujo?

We added only part of this.

[ ] add targets
[x] import targets

Sorry, but the comment you are referring to has three todo items and all of them are marked as complete.
What do I miss?

Sorry, but the comment you are referring to has three todo items and all of them are marked as complete.
What do I miss?

There are two feauteres there (subtitles) there: Add targets (Feature) and Import targets (Feature)

@KAUTH
I suggest breaking down this feature into three sub-tasks

[ ] Implement rstuf artifact add giving a file
[ ] Implement rstuf artifact add giving file information
[ ] Implement rstuf artifact add giving JSON file

So we can delivery small features and discuss more details in each.

@kairoaraujo good point! I was thinking of breaking it similarly as well.
I was also considering that giving file information can also be substituted with giving JSON file with information for only one file. Adding an actual file I see the immediate value, but do you think we need both JSON file and file info as well?

The JSON file is for adding a list of files, but it is a low priority.

The JSON file is for adding a list of files, but it is a low priority.

Yeah, my point is one file is also a list of one. So, do we want both JSON file and passing file information?
In any case, I am working on "Implement rstuf artifact add giving a file. I will create tasks for each one and link them to this issue.


Fuzzer: Disable caching

Codecov Report

Merging #98 (ad53d81) into master (f513029) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master       #98   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            3         3
  Lines          487       487
  Branches        94        94
=========================================
  Hits           487       487

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 f513029...ad53d81. Read the comment docs.


Do not kill udev during boot 17.09

Backport of https://github.com/NixOS/nixpkgs/pull/40230/files
Fixes #39867

@edolstra @dezgeg

Any particular reason to backport? I don’t like to do it with such significant changes. At least let’s let this sit on unstable for a month or so before backporting to releases.

Well, it's needed for reliable boot on one of those larger AWS instances.

Ok nevermind then. I didn’t realize it was breaking booting.


DBZ-5213 Adding unique label when building multiarch images

Similar to https://github.com/docker/build-push-action/issues/452
Which seems to be related to a bug in containerd https://github.com/containerd/containerd/pull/6995
quay.io probably responds with redirect when pushing a new tag which has a matching hash of already existing tag. This unfortunately causes buildx builder to fail. Based on testing this seems to help.

@nilshartmann just FYI, another complication with surfaced when building form different tags in the repo. Truly coding on the edge :D

@jcechace 🙀  if there is anything I can help, please let me know


Modified travis config for env separation

Sorry, I think the Travis config is simple enough and not worth new tox-travis dependency.

@kmmbvnr probably it's my fault not to provide description of what this PR fixes.
In present configuration travis launches all tox env's for every python version re-running same tests multiple times (5 times in worst case). Example below.
 I added tox-travis to separate travis jobs according to tox env.
See travis run for this PR (https://travis-ci.org/kmmbvnr/django-fsm/builds/234050427). It's 2m46s versus 8m58s of unmodified config.
Just clarifying :)


HADOOP-16114 Ensure NetUtils#canonicalizeHost returns same canonicalized host name for a given host

Currently NetUtils#canonicalizedHost returns different canonicalized host name for the same host, this patch resolves it.

committed to trunk via patch; closing


Incorrect keyboard shortcut shown in menu.

VSCode Version: 1.40.2 (also happens in 1.41.0-insider)
OS Version: Pop!_OS 19.10 64-bit (which is very much like Ubuntu 19.10 Eoan Ermine)

Steps to Reproduce:

Press Ctrl+K, Ctrl+S to open the keyboard shortcut editor;
Find the entry for toggling the Activity Bar (workbench.action.toggleActivityBarVisibility)
Set its keybinding  to "Ctrl + Super + A", which shouldn't be used by anything else by default; "Super" is the "Windows" key on PC keyboards, also known as the meta key in Linux.
Close the keyboard shortcut editor;
Open the "View" menu --> "Appearance"
See that the keyboard shortcut displayed next to "Show Activity Bar" is "Ctrl+A" instead of "Ctrl+Super+A"

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

Does the assigned keybinding works?

The keybinding (Ctrl+Super+A) does work.

It seems that electron is just ignoring the super part of the shortcut even though is documented as supported. @deepak1556 let me know if we can't fix this on electron side and I will patch with our "non-native keybind" workaround

Any Super/Meta keybind is ignored. Super/Meta + C to copy instead of Ctrl + C becomes just C.
On linux no matters xorg or wayland it is completely broken. Unusable for me.

This requires patch in https://source.chromium.org/chromium/chromium/src/+/main:ui/base/accelerators/accelerator.cc;l=396-405 and cannot be made easily. @sbatten a workaround on our end would be better here.


Cannot issue with Ledger Nano

Our user complained that she is unable to issue certs with Ledger Nano on https://admin.opencerts.io today. When she clicked on Issue nothing had happened. She was able to until yesterday. Has any update been made to the website? How can we solve this?

Hi,
No changes has been made to the admin portal. Another user reported problem to issue using ledger as well: https://github.com/OpenCerts/opencerts-website/issues/588
We provided an alternative flow to use ledger through metamask in the issue.
Let me know if it helps you.

Hi,
No changes has been made to the admin portal. Another user reported problem to issue using ledger as well: https://github.com/OpenCerts/opencerts-website/issues/588
We provided an alternative flow to use ledger through metamask in the issue.
Let me know if it helps you.

Hi, Thanks for advice.
We have tried the recommendation in #588 but failed. We managed to add ledger nano into metamask hardware wallet and it can show up in admin website. However, issuance still cannot carry on.

Hi, Thanks for advice.
We have tried the recommendation in #588 but failed. We managed to add ledger nano into metamask hardware wallet and it can show up in admin website. However, issuance still cannot carry on.

Yes, the hardware is showing "Contract Data Allowed" already

Yes, the hardware is showing "Contract Data Allowed" already

Can you record the problem ?

Can you record the problem ?

How to record the problem? The balance is equivalent to around $100, so should be enough to pay for one transaction.

How to record the problem? The balance is equivalent to around $100, so should be enough to pay for one transaction.

you can use whatever tool that record part of the screen, there is one with window 10 for instance: https://www.laptopmag.com/articles/how-to-video-screen-capture-windows-10

you can use whatever tool that record part of the screen, there is one with window 10 for instance: https://www.laptopmag.com/articles/how-to-video-screen-capture-windows-10

Please check out the recorded video uploaded. We tried a few times but no luck.
OpenCerts_Metamask2.zip

Please check out the recorded video uploaded. We tried a few times but no luck.
OpenCerts_Metamask2.zip

That particular merkle root has already been issued

That particular merkle root has already been issued

Yes, the user has verified that it's been issued. I guess the transaction had taken too long time so she thought it failed. Thank you for all your efforts. I will close it.

Yes, the user has verified that it's been issued. I guess the transaction had taken too long time so she thought it failed. Thank you for all your efforts. I will close it.


[Fonts] Icon request: network_wifi_[X]_bar

Hi
I've noticed some icons are missing in the current Flutter codebase.
In my app I need to show a Wifis' list with the power/closeness specified by ad icon, as the OS does.
Although I should be able to set the desired icons, I'm here to notify.
Icons not present:

signal_wifi_1_bar
signal_wifi_2_bar
signal_wifi_3_bar

While 0 (min) and 4 (max) are present.
I've tried to use the web's code point but they point to other icons.
I.e.: Network Wifi 3 Bar in Google Fonts points to the printer icon in the Flutter code.
Can I help in any way?
Thanks.

Hi @FabrizioBilleciUNICT, thanks for the report. We're trying to consolidate all the missing icons into one issue so that it's easier to track. If you do find any more missing icons, kindly leave a comment in https://github.com/flutter/flutter/issues/99973.

I've tried to use the web's code point but they point to other icons.
I.e.: Network Wifi 3 Bar in Google Fonts points to the printer icon in the Flutter code.

See https://github.com/flutter/flutter/issues/75633 for why this may not be a good idea.
Closing from here as duplicate.
Thank you

Hi @FabrizioBilleciUNICT, see https://github.com/flutter/flutter/issues/99973#issuecomment-1088400026


Code Jump only in the same notebook

I am not sure if it's a bug or the intended behavior, (could not confirm in the documentation), but I have the following issue:
If I try "Jump to Definition" or "Jump to References" within a .py file or between .py files, it works fine.
But I try the same features in a notebook (.ipynb), it only works within the same notebook. That means:

I cannot jump to definition/references if I defined a function etc. in another notebook, even though Shift + Tab shows the definition, signature etc.

I cannot jump to definition/references if the definition is in a .py file, even though Shift + Tab shows the definition, signature etc.

Google Chrome version: 123.0.6312.88
JupyterLab v4.1.2
labextension list:
/usr/local/share/jupyter/labextensions
nbdime-jupyterlab v3.0.1 enabled OK
jupyterlab-plotly v5.18.0 enabled  X
jupyter-matplotlib v0.11.3 enabled OK
jupyterlab-logout v1.0.0 enabled OK (python, jupyterlab_logout)
jupyterlab-theme-toggler v1.0.0 enabled OK (python, jupyterlab_theme_toggler)
jupyterlab-topbar-text v1.0.0 enabled OK (python, jupyterlab_topbar_text)
jupyterlab_code_formatter v2.2.1 enabled  X (python, jupyterlab_code_formatter)
jupyterlab_pygments v0.3.0 enabled OK (python, jupyterlab_pygments)
@jupyterlab/git v0.50.0 enabled OK (python, jupyterlab-git)
@epi2melabs/jupyterlab-play-cell-button v1.0.0 enabled  X (python, jupyterlab_play_cell_button)
@jupyter-notebook/lab-extension v7.1.0 enabled OK
@jupyter-lsp/jupyterlab-lsp v5.1.0 enabled OK (python, jupyterlab-lsp)

Which language server are you using?

Please see the image:

Does it also happen if you were to use python-lsp-sever instead?

So sorry for the late response due to my workload. I had finally time to try your suggestion. But yes, I see the same issue with python-lsp-server.

I am also having this issue, but I am unfamiliar with Jupyterlab. Is anyone looking into this?


fix: pfe-modal local variable support; design updates

Component name

pfe-modal

Related issue

https://github.com/patternfly/patternfly-elements/issues/771
https://github.com/patternfly/patternfly-elements/issues/811

Preview

Netlify demo page

What has changed and why

fixes for local var support
padding cleanup
temporary theming fix for broadcast links (when modal is inside dark card)
built-in support for media
remove "jumpiness" on page when modal is opened

Testing instructions

Browser requirements
Your component should work in all of the following environments:

[ ] Latest 2 versions of Edge
[ ] Internet Explorer 11 (should be useable, not pixel perfect)
[ ] Latest 2 versions of Firefox (one on Mac OS, one of Windows OS)
[ ] Firefox 68 (or latest version for Red Hat Enterprise Linux distribution)
[ ] Latest 2 versions of Chrome (one on Mac OS, one of Windows OS)
[ ] Latest 2 versions of Safari
[ ] Android mobile device (such as the Galaxy S9)
[ ] Apple mobile device (such as the iPhone X)
[ ] Apple tablet device (such as the iPhone Pro)

Ready-for-merge Checklist
Check off items as they are completed.  Feel free to delete items if they are not applicable.

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

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

@TODO This needs an issue cc @starryeyez024

closing as stale, feel free to reopen if you'd like to rebase onto main


BUG:  create transaction for same debtor and creditor account (TBC)

Actual Behaviour
Debtor and creditor account can be the same, and there is no error message when sending a transaction

Expected behaviour
If the debtor and creditor entities are the same, display an error message "You can't pay yourself"

This is not an issue for the demo


Save / Restore / Render bending data

I made changes to plug-in to save data into edge.data  instead edge.scratch and now i can save serialized bending data along with edge data and restore it later.
var exported = cy.elements().jsons();
but my issue is when i restore it by rendering again using same element data then it does not display bends. I have to personally click on edge then it shows bend, to prevent manual click I had to apply a workaround, which works but there should be a better way.
cy.elements().each(function (i, ele) {
		if (ele.group() == "edges") {
			ele.select();
			ele.unselect();
		}
	});

Can you add method for export, import and apply bend data?
Attached file with changes. Search for vishal keyword in file to see changes.
cytoscape-edge-bend-editing.zip

@vishaltanwar any changes requested should be made in the form a pull-request. thanks

Can you please tell me a how to redraw an edge with bends. Need a function to which i pass edge and it redraws it. e.g. redrawWithBends(edge) assuming edge.data.bends has its bendpoints?

@metincansiper I don't know at what level this is / should be supported but I can currently persist bends in Newt. Is this issue obsolete?

@ugurdogrusoz In Newt we get bend points data of an edge using a method exposed in this extension and use that data upon save.
@vishaltanwar we have a method called 'getSegmentPoints' to get bend points information of an edge. You can check it here 'https://github.com/iVis-at-Bilkent/cytoscape.js-edge-bend-editing#api'

I don't see an issue here; closing it


Does not connect

When i try to login with my spotify account, it shows me this message:
INVALID_CLIENT: Invalid redirect URI
and i cannot login with my spotify account

Hello, @dagashy thank you for your feedback, I will see what's is happening with the extension ASAP.

@dagashy  was a problem with netlify they changed the suffix .com to .app and this broke the OAuth from Spotify, but now it's fixed


Compiler: fix a couple of issues with as, as? and unions

Fixes #5805
Fixes #7441
The first two commits use .type? instead of .type on a couple of nodes because their types might not be computed yet.
The third commit makes computing a union's type reactive (make it depend on the inner types, and if they change, recompute) and also use .type? instead of .type because of the same reason as above.

@ysbaddaden  hehe, yes, thanks! Eventually the compiler could be simplified, improved and documented a bit more so these changes aren't that cryptic :-)


Build fails when trying to add akmods to image

Here is the relevant terminal output:
0.161 ================================================ Start 'akmods' Module ================================================ 0.174 Installing akmods 0.175 Installing: openrgb xpadneo 0.198 error: Handling argument /tmp/rpms/kmods/*openrgb*.rpm: No such file or directory (os error 2)
Here is my recipe.yml:

image will be published to ghcr.io//

name: xps17-fedora
description will be included in the image's metadata
description: Personal Laptop ISO
the base image to build on top of (FROM) and the version tag to use
base-image: ghcr.io/ublue-os/bazzite-gnome-nvidia
image-version: latest # latest is also supported if you want new updates ASAP
module configuration, executed in order
you can include multiple instances of the same module
modules:

type: files
files:

source: system
destination: / # copies files/system/* (* means everything inside it) into your image's root folder /
# put configuration files you want in /usr/etc/ inside the image
# while putting files in /etc/ is done on booted system by system administrators.
# read more about this in files module reference.

type: rpm-ostree
repos:

https://download1.rpmfusion.org/free/fedora/rpmfusion-free-release-%OS_VERSION%.noarch.rpm
https://download1.rpmfusion.org/nonfree/fedora/rpmfusion-nonfree-release-%OS_VERSION%.noarch.rpm
https://copr.fedorainfracloud.org/coprs/atim/starship/repo/fedora-%OS_VERSION%/atim-starship-fedora-%OS_VERSION%.repo
https://copr.fedorainfracloud.org/coprs/atim/xpadneo/repo/fedora-%OS_VERSION%/atim-xpadneo-%OS_VERSION%.repo
https://copr.fedorainfracloud.org/coprs/ublue-os/akmods/repo/fedora-%OS_VERSION%/ublue-os-akmods-fedora-%OS_VERSION%.repo
https://copr.fedorainfracloud.org/coprs/sentry/kernel-fsync-staging/repo/fedora-%OS_VERSION%/sentry-kernel-fsync-staging-fedora-%OS_VERSION%.repo
install:

command line utilities

fish
starship
tldr
btop
bolt
mesa-vulkan-drivers
mesa-libGL
nvidia-settings
rclone
tlp
tlp-rdw

KVM/QEMU *********************

virt-install
libvirt-daemon-config-network
libvirt-daemon-kvm
overlayfs-tools
qemu-kvm
virt-manager
virt-viewer
guestfs-tools
python3-libguestfs
virt-top
swtpm
#***********
profile-sync-daemon
samba
freerdp
irqbalance
restic

remove:
- rom-properties
- wallpaper-engine-kde-plugin
- steamdeck-dsp
- steamdeck-gnome-presets
- steamdeck-kde-presets-desktop
- steam_notif_daemon
- jupiter-fan-control
- jupiter-hw-support-btrfs
- jupiter-sd-mounting-btrfs
- extest
- ryzenadj
- sdgyrodsu

type: default-flatpaks
notify: true # Send notification after install/uninstall is finished (true/false)
system:
If no repo information is specified, Flathub will be used by default
repo-url: https://dl.flathub.org/repo/flathub.flatpakrepo
repo-name: flathub
repo-title: "Flathub (system-wide)" # Optional; this sets the remote's user-facing name in graphical frontends like GNOME Software
install:
- com.github.IsmaelMartinez.teams_for_linux
- com.visualstudio.code
- org.gnome.FontManager
- org.remmina.Remmina
- sh.cider.Cider
- io.github.dvlv.boxbuddyrs #box buddy
- io.podman_desktop.PodmanDesktop
- org.videolan.VLC
- org.gnome.DejaDup

type: signing # this sets up the proper policy & signing files for signed images to work fully

type: fonts
fonts:
nerd-fonts:
- FiraCode # don't add spaces or "Nerd Font" suffix.
- Hack
- SourceCodePro
- Terminus
- JetBrainsMono
- NerdFontsSymbolsOnly
google-fonts:
- Roboto
- Open Sans

type: systemd
this example disables automatic flatpak updates and enables a custom service unit for all users
system:
enabled::
- smb.service
- nmb.service
- libvirtd.service
user:
enabled:
- psd.service # TODO: copy service description files to the repo

type: akmods
base: fsync # if not specified, classic "main" base is used by default
install:

openrgb
xpadneo

type: bling
install:

1password # install 1Password (stable) and op CLI tool
dconf-update-service # a service unit that updates the dconf db on boot
gnome-vrr # enables gnome-vrr for your image

type: gnome-extensions
install:

Blur my Shell
AppIndicator and KStatusNotifierItem Support
Bluetooth Quick Connect
Dash to Dock
Forge
Just Perfection
Space Bar
System Monitor
User Themes
uninstall:
GSConnect # Uninstall step here should be used only for the extensions
gnome-shell-extension-caribou-blocker
gnome-shell-extension-compiz-windows-effect
gnome-shell-extension-hanabi
gnome-shell-extension-hotedge

Just FYI, I have stripped out all my akmods except one, as a test, and it always fails:

type: akmods
base: fsync # if not specified, classic "main" base is used by default
install:

openrgb

Is this happening because there is no tag 'fsync-40'?

This issue should be fixed now:
https://github.com/ublue-os/akmods/pull/232

Now I have this error when building after the change:
[21:30:27 g.i/r/tacos-os:latest] => 5.896   - nothing provides kernel-uname-r = 6.10.4-201.fsync.fc40.x86_64 needed by kmod-framework-laptop-6.10.4-201.fsync.fc40.x86_64-0.0.git.75.7229df21-1.fc40.x86_64 from @commandline

Update:  I got it to build by modifying

type: akmods
base: fsync-40

to this:

type: akmods
base: fsync

Y'all might want to update the README on the akmods page but otherwise I'd say its good to go.


CNVs in the Platform

@deniseOme commented on Thu Sep 07 2017
We seem to be getting CNVs from ClinVar (via EVA) e.g. nsv1067886:
http://www.targetvalidation.org/evidence/ENSG00000080815/EFO_0000249?view=sec:genetic_associations
For consistency and the benefit of the user, it would be good to link out to Ensembl as we do for the rsIDs, @emepyc @ckongEbi
http://www.ensembl.org/Homo_sapiens/StructuralVariation/Explore?r=14:73204262-73209817;sv=nsv1067886;svf=121159025;vdb=variation

@ElaineMcA commented on Tue Jul 24 2018
Checked in system - looks complete.  Please confirm if other instances still where not working, please re-open.

@deniseOme commented on Tue Jul 24 2018
I'm afraid to say it seems it's still not linking out to Ensembl.
The CNV in the provided link (i.e. https://www.targetvalidation.org/evidence/ENSG00000080815/EFO_0000249?view=sec:genetic_associations) is nsv1067886.
It's yet not hyperlinked. If it were, it would be redirecting the user to the Ensembl variation page that for this CNV is:
http://www.ensembl.org/Homo_sapiens/StructuralVariation/Explore?r=14:73204262-73209817;sv=nsv1067886;svf=121954490;vdb=variation
This would be kept in line with the SNPs e.g. rs63751163 gets redirected to http://www.ensembl.org/Homo_sapiens/StructuralVariation/Explore?r=14:73204262-73209817;sv=nsv1067886;svf=121954490;vdb=variation.

@ckongEbi commented on Tue Jul 24 2018
@deniseOme @ElaineMcA from what I can see, the API didn't return the full hyperlinked to Ensembl (rather just the id ie. nsv1067886 & rs63751163). The hyperlink is built in the frontend code, this issue will need moving to webapp

The link URL is indeed "built" in the front end as CK suggested back in July.
There is a bit of logic to handle different types of variant ID and point them to different URLs.
In this bit of logic however we don't take into account IDs starting with "nsv", hence the link not being generated for that entry in the table.
I will add that in, so that (if I understood correctly) nsv and rs IDs will both link to "http://www.ensembl.org/Homo_sapiens/StructuralVariation/Explore" (obviously with the id param set correctly).

This was a quick fix and has been pushed along other minor bug fixes.
Fixed.


Scrollable window needed within the import modal to handle large csv file

Add a vertically scrollable window to handle large csv file with several number of rows.

Closing this issue, as we decided to hide that information on file upload anyway.


dpservice local test fails on one of the dell machines

dpservice local test fails on one of the dell machines (dell-3). The GDB shows the following error stack trace:
(gdb) bt
#0  0x0000000000000000 in ?? ()
#1  0x000055555597edbc in dp_ref_dec (ref=0x11fffd52c8) at ../include/dp_refcount.h:35
#2  0x000055555597f37f in dp_age_out_flow (flow_val=0x11fffd5200) at ../src/dp_flow.c:411
#3  dp_process_aged_flows_non_offload () at ../src/dp_flow.c:437
#4  0x0000555555702023 in dp_process_event_flow_aging_msg (m=0x17936a9c0) at ../src/monitoring/dp_event.c:120
#5  0x0000555555715eac in dp_process_event_msg (m=0x17936a9c0) at ../src/monitoring/dp_monitoring.c:22
#6  0x00005555558ed005 in handle_nongraph_queues () at ../src/nodes/rx_periodic_node.c:48
#7  rx_periodic_node_process (graph=0x11ffffd9c0, node=0x11ffffe380, objs=0x11fffea140, nb_objs=0) at ../src/nodes/rx_periodic_node.c:74
#8  0x00005555559d3850 in __rte_node_process (graph=0x11ffffd9c0, node=0x11ffffe380) at /usr/local/include/rte_graph_worker_common.h:186
#9  rte_graph_walk_rtc (graph=0x11ffffd9c0) at /usr/local/include/rte_graph_model_rtc.h:42
#10 0x00005555559d351f in rte_graph_walk (graph=0x11ffffd9c0) at /usr/local/include/rte_graph_worker.h:38
#11 0x00005555559d30df in graph_main_loop (arg=0x0) at ../src/dpdk_layer.c:100
#12 0x00007ffff7ae542f in eal_thread_loop () from /usr/local/lib/x86_64-linux-gnu/librte_eal.so.24
#13 0x00007ffff7afbe26 in eal_worker_thread_loop () from /usr/local/lib/x86_64-linux-gnu/librte_eal.so.24
#14 0x00007ffff70a645c in ?? () from /lib/x86_64-linux-gnu/libc.so.6
#15 0x00007ffff7126bbc in ?? () from /lib/x86_64-linux-gnu/libc.so.6

After a debugging session with @PlagueCZ, it is found out that swapping the order of union dp_ipv6 underlay_dst and uint8_t l4_type or make struct flow_nf_info packed can eliminate this error. The root cause still needs to be investigated.

We found out that flow_value->ref_count->release is NULL.
I suspect the problem is a possible double free of flows (once on LB destroy other one on aging). If you comment out dp_grpc_impl.c:150:dp_remove_lbtarget_flows(&ipv6); the problem stopped.
Because by using dp_ref references, the handler for flows is dp_free_flow() which calls rte_free() that actually memsets the memory to zero, which i think is the source of NULL in ref->release.
I have created a validation commit in https://github.com/ironcore-dev/dpservice/tree/fix/refcount that actually randomly fires on my machine.


BUG : Flat Files Separator is not correctly serialized

Reproduce :

Create a flat files connection with a tabulation chr(9) separator
Restart Dbeaver
Open your flat files connection driver properties : the separator is now a space chr(20)

5.0.1

is it still actual?

no, it seems to have been fixed. Well done :+1:


Running multi-threaded vkcts in slang mode produces errors

When I tried running vkcts multi-threaded in slang mode I see random failures in tests that pass when run serially. When I disable slang and run vkcts multi-threaded the errors do not occur.
The errors are all similar to:
"ERROR: Got non-white pixels on sub-case 1"
The command I run is (for each thread I have a different test list and I increase the log file name and cache file name by one):
deqp-vk.exe --deqp-archive-dir=. --deqp-caselist-file=parallel1.txt --deqp-log-images=disable --deqp-log-shader-sources=disable --deqp-log-flush=disable --deqp-log-filename=TestResults1.qpa --deqp-shadercache-filename=shadercache1.bin

This will take around 3 weeks of work.

There are few global static variables that needs to be added to a context so that it becomes thread safe.
Make sure the test_sever is able to process multiple queries simultaneously. It might need a different kind of initialization. @csyonghe can the test_sever handle multiple queries?
We will have to handle test_sever crash across multiple threads
Test the compelete list and make sure there are no issues. This takes the longest time as total time to test is high.

Approximately 3 weeks of effort including testing.

Instead of having test server handle multiple requests in parallel, the right thing to do is to launch multiple test servers and dispatch requests to them independently. I don't think it will take three weeks if we go this route. The slang-test infrastructure used this technique to run tests in parallel without getting into the mess of thread-proofing the rest of the code.

This can be done by having each CTS test thread create its own test server and handle the restart of test server independently in that thread. There should be no thread syncing required.

parallel1.txt
parallel2.txt
parallel3.txt
parallel4.txt
parallel5.txt
attaching test lists that I used just for reference. I'm not sure if this is the best way to run in parallel. It's just the way I set it up.

CTS has an option --deqp-fraction= that can run parallel threads.  I am investigating it.

It sounds like this may be resolved; Pankaj to confirm when he is back.

The issue happens when when multiple processes are launched simultaneously. Seems like a issue with windows mechanism of registering a mutex. The issue can be wok-around by launching each process after some delay.


make the OGRE plugin path discovery portable

Instead of checking the platform, check the existence of pkg-config to accommodate more environments.

@seanyen , mind signing your commit? We're using DCO on Ignition just like ROS 2. Thanks!

@chapulina Thanks for the reminder! Now I signed my commits.

@osrf-jenkins run tests please


Support multiple raw data files

Well we do already do that since #604, no? Maybe it's only half-baked though

yep, there is the problem of keeping track of recordings, which is not straightforward with the current architecture, and another unknown bug that causes a huge memory leak during threshold detection (only when there are more than 1 .dat file). Will fix all of these issues during the formatless work.


mc_pos_control: manual stick input fix; deletion of unusued function

This pr should fix the issue when flying forward with constant roll/pitch demand during velocity demand changes in z-direction.
In the past the velocity setpoint in xy changed as well:

In addition I deleted the function limit_pos_offset since not used anywhere.

Nice @Stifael, Tested today outside, completely solved the strange attitude reversal problem.
Has merge conflict with https://github.com/PX4/Firmware/pull/6625, I would take this one in first and I can easily rebase and resolve my conflicts just afterwards.


feat: use SCSS?

I noticed that the project uses CSS for components rather than SCSS. Is this intentional?
It seems like using SCSS would be the better choice as that is what Angular Material uses and that is what the generator produces for export.

It's a miss from my side when generated project. Did not specify style option for ng new


Add The Wheeler School

The Wheeler School is an N-12 school based in Rhode Island, US.

@BNault Please rename lib/domains/org/wheelerschool to lib/domains/org/wheelerschool.txt, .txt extension is mandatory in this repository.
Also, please notice that free educational license for JetBrains tools is for high and higher schools students, i.e. those who are over 13 of age. Thank you!

@philipto: The Wheeler School has a High School. The 12 refers to the US Grade 12.

@BNault Pull request merged. Thank you for the update!


[WIP] Manifest downloads

Codecov Report

Merging #125 into hannes-fix-file-version-leak will increase coverage by 0.09%.
The diff coverage is 91.89%.

@@                       Coverage Diff                        @@
##           hannes-fix-file-version-leak     #125      +/-   ##
================================================================
+ Coverage                         87.57%   87.66%   +0.09%
================================================================
  Files                                29       29
  Lines                              1038     1070      +32
================================================================
+ Hits                                909      938      +29
- Misses                              129      132       +3

Impacted Files
Coverage Δ

hca/util/__init__.py
90.87% <60%> (-1.35%)
:arrow_down:

hca/dss/__init__.py
88.38% <96.87%> (+3.34%)
: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 d039302...47411d0. Read the comment docs.


Basics of the incremental renderer

The basics should work, can connect to a TCP connection over http2, receive commands and render it.

This is 2/3rd of the way done. Currently blocked by https://github.com/canjs/dom-patch/issues/19 and https://github.com/canjs/node-route/issues/9

Closed by https://github.com/donejs/done-ssr/pull/259


APT fails on @Embbeded Java record

Observed vs. expected behavior
When adding a Java Record as an @Embedded field to an entity the code generation fails with:
Caused by: java.lang.IllegalArgumentException: Illegal type: com.project.SomeEntity.javaRecordField
    at com.querydsl.apt.TypeExtractor.visitDeclared (TypeExtractor.java:56)
    at com.querydsl.apt.TypeExtractor.visitDeclared (TypeExtractor.java:25)
    at com.sun.tools.javac.code.Type$ClassType.accept (Type.java:1171)
    at javax.lang.model.util.AbstractTypeVisitor6.visit (AbstractTypeVisitor6.java:108)
    at com.querydsl.apt.AbstractQuerydslProcessor.handleEmbeddedType (AbstractQuerydslProcessor.java:344)
    at com.querydsl.apt.AbstractQuerydslProcessor.getEmbeddedTypes (AbstractQuerydslProcessor.java:323)

Steps to reproduce
Add any record to an existing entity and generate the classes.
Environment
Querydsl version: Version 5.0.0
Querydsl module: querydsl-apt
Database:
JDK: JDK 17.0.1
Additional details
A small code change fixes the issue (not sure if it introduces other problems)
https://github.com/querydsl/querydsl/blob/c068ced1f741c86949daca12de81a667ea47e258/querydsl-apt/src/main/java/com/querydsl/apt/TypeExtractor.java#L52-L57
A new line case RECORD: above the case CLASS fixes the issue.

Definitely a bug, record support was added via #2798.

The fix doesn't seem to embedded in the latest released version.
Any plan for a new release?

The fix doesn't seem to embedded in the latest released version.
Any plan for a new release?(2)

The fix doesn't seem to embedded in the latest released version.
Any plan for a new release?(3)

The fix doesn't seem to embedded in the latest released version.
Any plan for a new release?(4)

It's in the 5.0.0 release so I see no reason this can't be closed

It's in the 5.0.0 release so I see no reason this can't be closed

No it is not fixed yet. It is about embedded records. For example, this fails with MongoDB:
@Document
public class TestDocument {
  @Id
  UUID id;

  SubRecord subRecord;

  public record SubRecord(String foo) {}

}

and it fails:
Fatal error compiling: java.lang.IllegalArgumentException: Illegal type: org.test.TestDocument.SubRecord


Minecraft Development Plugin - Recaf Edition

Because of popular demand and usage of Recaf, it would be advised to create as specific plugin handling those features:

Ability to download MC mappings (MCP, Yarn, Loom, etc.).
Presets of resolution of dependencies using #666 for Forge, Fabric, MC, Paper, Spigot, etc.

Didnt Filip already do almost this in his Recaf4Forge?

This would be very useful.   What might be a first step is, if Recaf could say give examples in the documentation of how to load the yarn/mappings:
https://github.com/FabricMC/yarn/tree/1.20.5-pre2/mappings
that would be really helpful.   What type of mappings are the yarn mappings?  Simple?  Enigma?  Proguard? SRG, TSRG?  Tiny?  Tiny V2?  JADX?   Yarn does not say, or does not use the same terminology as Recaf.
If Recaf already has the ability to read yarn/mappings then if it were simply explained in a page of documentation, that would be a large part of the problem solved.
If Recaf does not yet have the ability to read yarn/mappings,
a) if there are simple work-arounds, like, concatenate all the files from the yarn/mappings tree into a single text file, and load that using the mappings tab of Recaf, with the Simple/Enigma..../JADX setting that is needed then that would be useful to explain in docs.
b) if there is a simple utility to convert yarn/mappings to a format Recaf can use, then it would be really useful if the docs could provide links and explanations on how to do that.
c) If there were documentation about the mappings tab of Recaf, explaining the format, and giving examples for each type of mapping then someone reasonably skilled in Perl or TCL could write a utility to read yarn/mappings and write them out in a format that Recaf can read.
In summary, this might be just a documentation problem, or maybe documentation could go a long way, with available utilities, or ones we can write, to helping us solve the problem of using Recaf for Minecraft Mod Programming.
Thank you for your time.


WIP Tests e2e

Can we take a look at this together tomorrow @BugRoger? I'll ping you.
At least some feedback on the main parts (e.g. e2e_api.go) would be nice.
Basic tests for API and smoke tests for network (see snippet below) and pvc.
[success]                e2e-small-6ltm9 --> e2e-medium-jp3c8                 <IP_ADDRESS>     --> <IP_ADDRESS>
2017/11/06 15:35:06 Running '/usr/local/bin/kubectl exec --namespace=default e2e-nginx-e2e-small-6ltm9 -- /bin/sh -c wget --timeout=10 -O - http://<IP_ADDRESS>:80'
2017/11/06 15:35:09 stderr: "--2017-11-06 23:35:09--  http://<IP_ADDRESS>/\nConnecting to <IP_ADDRESS>:80... connected.\nHTTP request sent, awaiting response... 200 OK\nLength: 612 [text/html]\nSaving to: 'STDOUT'\n\n     0K

Thanks in advance.

missed the comment. will merge tomorrow.


Enforce TDD using grunt watch

"As a developer, I want grunt to monitor source file changes and trigger the appropriate test in order to enforce the TDD method"
Test programs are already capable of executing tests only for given sources.
Acceptance criteria:

The source test is triggered when the source is modified

Actually, all tests must be triggered as the impact may be on other sources: adding mocha to the watch task.