API cannot be reached

Is there any changes on the api or url? I can no longer access the api. Thank you

same here. Tried using http://phisix-api.appspot.com/stocks.{json|xml}. Is this a dead project?

Is there any changes on the api or url? I can no longer access the api. Thank you

@32x0lf  You can get uptime stats here: https://statuspage.freshping.io/29889-phisixapi

same here. Tried using http://phisix-api.appspot.com/stocks.{json|xml}. Is this a dead project?

@maclaine-san you can change the API response payload by choosing either json or XML.
Example:

JSON payload: http://phisix-api.appspot.com/stocks.json
XML payload: http://phisix-api.appspot.com/stocks.xml

@edgedalmacio yes just to be clear I tried each xml and json formats yesterday :). Guess it's working now. Thanks!

Can you please show me what did you do to make it work on your side?. I tried everything from json to xml and api to api4 but none of them seems to work on my side. but this works
previously with me around 2020.

On Feb 26, 2021, at 5:09 PM, John Maclaine Danao<EMAIL_ADDRESS>wrote:

@edgedalmacio yes just to be clear I tried each xml and json formats yesterday :). Guess it's working now. Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

All their GET endpoints are working for me, so I'm not sure what you mean
here, but you can submit a new issue with detailed steps to replicate your
issue.
On Mon, Mar 1, 2021 at 12:20 PM 32x0lf<EMAIL_ADDRESS>wrote:

Can you please show me what did you do to make it work on your side?. I
tried everything from json to xml and api to api4 but none of them seems to
work on my side. but this works
previously with me around 2020.

On Feb 26, 2021, at 5:09 PM, John Maclaine Danao <
<EMAIL_ADDRESS>wrote:

@edgedalmacio yes just to be clear I tried each xml and json formats
yesterday :). Guess it's working now. Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/phisix-org/phisix/issues/153#issuecomment-787624707,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE4MHQMZ24O7KAI5OB3SSJTTBMIZ3ANCNFSM4YGEEWFA
.

I figured it out. I changed my DNS from dynamic to public google dns
<IP_ADDRESS> and I got results. Thanks
On Mon, Mar 1, 2021 at 12:20 PM Flote Fuertes<EMAIL_ADDRESS>wrote:

Can you please show me what did you do to make it work on your side?. I
tried everything from json to xml and api to api4 but none of them seems to
work on my side. but this works
previously with me around 2020.
On Feb 26, 2021, at 5:09 PM, John Maclaine Danao<EMAIL_ADDRESS>wrote:

@edgedalmacio https://github.com/edgedalmacio yes just to be clear I
tried each xml and json formats yesterday :). Guess it's working now.
Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/phisix-org/phisix/issues/153#issuecomment-786513418,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFAWZLPKOWHS5RYNASOTTK3TA5QLZANCNFSM4YGEEWFA
.


revocation list model

Initial attempt to add the revocation list model to the anoncreds spec. It is based on #108 and replaces revocation registry entries. We should probably add some of the stuff about deltas and revocation registry entries to an implementers guide, and add the transformation from deltas/entries to the revocation status list model to the indy anoncreds method spec (is this already defined somewhere?)

Looks good.  The change to the term "Status List" is one we should confirm with the Working Group -- we'll have it for discussion next Monday (2023.01.09).

Discussed on 2023.01.09 AnonCreds Spec Working Group Meeting -- agree to merge/no objections.


Move homebrew

The new HomeBrew is now in https://github.com/maplibre/homebrew-martin
cc: @rubenpoppe

Maybe update martin book either?

Worked on my MacOS.

Maybe update martin book either?

be my guest :)


Validate and describe Tracks events for the WP-CLI run via VIP-CLI

Currently the vip_cli_wpcli_* Tracks Events are unvalidated and undescribed, we should validate them and describe them :)

What does validating and describing entail?

Looking at the current master branch. It looks like we somehow lost the tracking we had and now we are tracking the confirm_cancel event only https://github.com/Automattic/vip/blob/master/src/bin/vip-wp.js#L146
I think this is a good opportunity to pause and ask what questions are we trying to answer and build the events that can help us responding to them.
@simonwheatley lets brainstorm some questions and work out how we can track them.

Off the top of my head:

What users are running commands?
What organisations are running commands?
What commands are being run?
How long do commands take to run?
Success vs failure rates
What commands are failing at the CLI, at the API, and at the container?

Can we tell what we were tracking? It’s pretty worrying if that’s just been accidentally removed from the code.

Cross-linking some internal discussion: https://wp.me/p6jPRI-2iF/#comment-15731


Fix filtering top repositories (#783, #1281) (recreated for dev branch)

Recreated pull request #1279 for dev branch.

Coverage increased (+0.07%) to 51.405% when pulling 3c7a09a8c46131f7b4e7dd68f989dfdc45f1709e on laz2:fix-783-filter-top-repos into ce0a44010ddf02aebcb7078d89d1b57b6e547baa on vmware:dev.


EventSourceWriter - Thrown on non-existing RF

Consider best way of throwing an exception when writing to a non-existing resilient function

It could be a method with a boolean indicating if the invoker wants to take the performance hit for making this assertion


RFC:  problem on course Software Testing

Course software testing isn't open and I think this the new link https://www.udacity.com/course/software-analysis-testing--ud333

Course software testing isn't open

Thank you for pointing this out!
The course that you linked is a new course, by different instructors. If we're recommending a new course (not simply finding where the current recommendation has moved), then we should go through an actual RFC process that recommends a course and details how it is superior to the alternatives. It looks like there are a number of options for a course on software testing.

I'll also note that the Software Debugging course is no longer offered and instead points to a Udacity Nanodegree on becoming a Site Reliability Engineer. Not something that must be addressed in the Testing RFC.

Do these links work? They may require a Udacity log-in first.
https://learn.udacity.com/courses/cs258
https://learn.udacity.com/courses/cs259

Do these links work? They may require a Udacity log-in first.
https://learn.udacity.com/courses/cs258
https://learn.udacity.com/courses/cs259

No, they redirect to "My Programs" page, and do not enroll or anything.

Do these links work? They may require a Udacity log-in first.
https://learn.udacity.com/courses/cs258
https://learn.udacity.com/courses/cs259

They work for me

Confirming the links do not work for me. When I click them and I am not logged in I am asked to log in. If I click them while logged in I just see My Programs.

Seems like videos from the original course is still available under this YouTube playlist.

Thanks @sams05, I've dropped that link into the curriculum as a stop-gap.
This is still an opportune time to analyze the best course currently available. I'll leave this issue open for another 4 weeks in hopes that a contributor can compare the video playlist to the course noted above:
https://www.udacity.com/course/software-analysis-testing--ud333

Closing as 4 weeks have elapsed.


HARVESTER: Implement pod affinity in harvester manager

Summary

Occurred changes and/or fixed issues

https://github.com/harvester/harvester/issues/2317

Technical notes summary

Areas or cases that should be tested

Areas which could experience regressions

Screenshot/Video

Hi,  @richard-cox   @mantis-toboggan-md  Will this PR be merged into Rancher 2.7.3 ?

According to the #8643 changes, I propose that harvester should clone this component to the pkg, and revert the changes for harvester. I think we should separate the similar component timely. @WuJun2016
I am going to send PR to revert the changes for harvester.

Hi @mantis-toboggan-md ! Could you take a look at this PR?


x/pkgsite: package removal request for "github.com/iambpn/go-schema-validator"

What is the path of the package that you would like to have removed?
https://pkg.go.dev/github.com/iambpn/go-schema-validator
Are you the owner of this package?
Yes, I am the owner of this module. I have only published this module few days back.
What is the reason that you could not retract this package instead?
I want to remove this package because I didn't like the different path for documentation and package import. Also, importing package required to use deep path like github.com/iambpn/go-schema-validator/pkg/schema I would prefer to have a path like github.com/iambpn/go-schema-validator/schema. I tried redaction, but redaction didn't solve this problem. So I am requesting to remove this package.

Related Issues and Documentation

x/pkgsite: package removal request for github.com/fastbpmn/go-study #57805 (closed)
x/pkgsite: package removal request for [type path here] #59882 (closed)
x/pkgsite: package removal request for [type path here] #50341 (closed)
x/pkgsite: package removal request for [type path here] #55027 (closed)
x/pkgsite: package removal request for [type path here] #54154 (closed)
x/pkgsite: package removal request for [type path here] #50666 (closed)
x/pkgsite: package removal request for [type path here] #52429 (closed)
x/pkgsite: package removal request for [type path here] #59647 (closed)
x/pkgsite: package removal request for [type path here] #53147 (closed)
x/pkgsite: package removal request for [type path here] #54155 (closed)

(Emoji vote if this was helpful or unhelpful; more detailed feedback welcome in this discussion.)

your repository still exists and is public, what do you wish to achieve through removal?

I want to make my repo public and also index in pkg.go.dev, but I have messed up the package import path and semantic versioning. So deleting it would solve that problem.

removal only means hiding the generated documentation, it doesn't allow you to restart semver for your module

Oh. I didn't know that.
If that's the case, then I will mark my v1 as depreciated and work on v2.
I am closing this ticket now. Thanks for your clarification.


Create Dutch translation

Add a nl.yaml for the Dutch translation.

What this PR does / why we need it:

Special notes for your reviewer:
Release note:

Awesome \o/
Thanks. May I ping you for further additions later on?

Sure!


Non-RegExp patterns

Motivation
Regexes are easy to use and powerful enough for most of the highlighting problems we face. However, sometimes regexes are not powerful enough.
Since some tokens can only be matched by a context-free grammar, we usually resorted to a number of tricks ranging from supporting a finite number of recursion steps to using the usually surrounding of the token. We use these tricks out of necessity but what we really need in those cases is some more powerful than regexes.
Description
I want to propose that we relax grammars to allow regex-like objects. The basic idea is that Prism's matching algorithm only uses the exec and lastIndex properties of RegExp objects, so any object that implements those properties can be used.
Speaking in types, I want to change:
interface GrammarToken {
  pattern: RegExp;
  ...
}
interface Grammar {
  [name: string]: RegExp | GrammarToken | Array<RegExp | GrammarToken>;
}

to:
interface RegExpLike { // RegExp trivially implements this interface
  lastIndex: number;
  exec(value: string): RegExpExecArray | null;
}
interface GrammarToken {
  pattern: RegExpLike;
  ...
}
interface Grammar {
  [name: string]: RegExpLike | GrammarToken | Array<RegExpLike | GrammarToken>;
}

This will allow us to implement custom matchers that can be more powerful than regexes.
Required changes to Prism Core
The only thing we have to change is how we try to enable the global flag here.

Hi! I like how this is presumably such a small change in the core mechanism while still giving the grammars more power.
I quickly scanned through the original issue that lead to this idea. But could you maybe give an explicit example of the problem it would solve and what the exec function would look like?

But could you maybe give an explicit example of the problem it would solve and what the exec function would look like?

Yes, I'll add an example of nested XML today. I originally wanted to add it yesterday but I gotta sleep at some point.

Before I show the example: Implementing the exec function isn't easy because of the required lastIndex logic. This logic has to be implemented by all regex-like matchers, so I moved it to its own function:

toRegExpLike:
function toRegExpLike(matchFn: (input: string, offset: number) => RegExpExecArray | null): RegExpLike {
	return {
		lastIndex: 0,
		exec(value: string): RegExpExecArray | null {
			let lastIndex: number = this.lastIndex;
			if (typeof lastIndex !== "number" || lastIndex !== lastIndex || lastIndex < 0) {
				lastIndex = 0;
			}
			lastIndex = Math.floor(lastIndex);

			if (lastIndex <= value.length) {
				const result = matchFn(value, lastIndex);
				if (result) {
					this.lastIndex = result.index + result[0].length;
					return result;
				}
			}

			this.lastIndex = 0;
			return null;
		}
	}
}

This function allows us to implement matchers as pure functions.
Here is the promised example for nested XML structures:

Example:
function toRegExpExecArray(matches: string[], input: string, index: number): RegExpExecArray {
	const array = matches as RegExpExecArray;
	array.input = input;
	array.index = index;
	return array;
}

const enum TagType {
	OPEN,
	CLOSE,
	SELF_CLOSE,
}
function getTagType(tag: string): TagType {
	if (tag[1] === "/") {
		// </ ...>
		return TagType.CLOSE;
	} else if (tag[tag.length - 2] === "/") {
		// <... />
		return TagType.SELF_CLOSE;
	} else {
		// <...>
		return TagType.OPEN;
	}
}

function matchXML(input: string, offset: number): RegExpExecArray | null {
	const tag = /<\/?(?!\d)([^\s>\/=$<%]+)(?:\s(?:\s*[^\s>\/=]+(?:\s*=\s*(?:"[^"]*"|'[^']*'|[^\s'">=]+(?=[\s>]))|(?=[\s/>])))+)?\s*\/?>/g;

	tag.lastIndex = offset;
	let initialMatch = tag.exec(input);
	while (initialMatch) {
		let type = getTagType(initialMatch[0]);
		if (type === TagType.SELF_CLOSE) {
			return initialMatch;
		} else if (type === TagType.OPEN) {
			const stack: RegExpExecArray[] = [initialMatch];
			let m;
			while ((m = tag.exec(input))) {
				type = getTagType(m[0]);

				if (type === TagType.OPEN) {
					stack.push(m);
				} else if (type === TagType.CLOSE) {
					const open = stack.pop();
					if (open[1] !== m[1]) {
						// mismatched tag names. Abort
						break;
					}
					if (stack.length === 0) {
						// we just found the closing tag to the initial opening tag
						const matches = [
							input.substring(open.index, m.index + m[0].length) // whole match
						];
						return toRegExpExecArray(matches, input, open.index);
					}
				}
			}
		}

		// move pattern to the next index
		tag.lastIndex = initialMatch.index + 1;
		initialMatch = tag.exec(input);
	}

	return null;
}

A regex-like object is created by calling toRegExpLike(matchXML).
Example usage in NodeJS shell
> var r = toRegExpLike(matchXML)
undefined
> r.exec(`foo <foo> <br/> </foo> <div/> <<BAZ/> </g> <fan> <b> </fan> </b>`)

[
  '<foo>
 </foo>',
  input: 'foo <foo>
 </foo>
 <<BAZ/> </g> <fan>  </fan> ',
  index: 4
]
> r.exec(`foo <foo> <br/> </foo> <div/> <<BAZ/> </g> <fan> <b> </fan> </b>`)

[
  '
',
  'div',
  index: 23,
  input: 'foo <foo>
 </foo>
 <<BAZ/> </g> <fan>  </fan> ',
  groups: undefined
]
> r.exec(`foo <foo> <br/> </foo> <div/> <<BAZ/> </g> <fan> <b> </fan> </b>`)

[
  '<BAZ/>',
  'BAZ',
  index: 31,
  input: 'foo <foo>
 </foo>
 <<BAZ/> </g> <fan>  </fan> ',
  groups: undefined
]
> r.exec(`foo <foo> <br/> </foo> <div/> <<BAZ/> </g> <fan> <b> </fan> </b>`)

null
> r.exec(`foo <foo> <br/> </foo> <div/> <<BAZ/> </g> <fan> <b> </fan> </b>`)

[
  '<foo>
 </foo>',
  input: 'foo <foo>
 </foo>
 <<BAZ/> </g> <fan>  </fan> ',
  index: 4
]

Both toRegExpLike and toRegExpExecArray are reusable across matchers.

Even though I'm the one proposing it, I don't like my current matcher implementation. It's not as declarative as regexes and fairly complex which makes it easy for bugs to hide. Ideally, we would have some way to create the AST of a CF grammar and a function/class that creates a matcher for that grammar.
I also want to point out that these matchers have a worst-case complexity of Ω(n^2) assuming that the formal language the matcher accepts is CF.

Interesting approach! I don't know enough about building tooling on language grammars to know what could possibly help here, but I do wonder if any inspiration could come from any prior art, such as the TextMate grammar approach? (eg. https://macromates.com/manual/en/language_grammars)
Shiki uses these for highlighting, such as this TSX language definition: https://github.com/shikijs/shiki/blob/master/packages/languages/data/tsx.tmLanguage.json
Repl.it demo: https://repl.it/repls/GroundedEmbellishedEmbeds

I think the main question I have about this, and this applies broadly to a lot of (great!) suggestions you have for Prism, is the tradeoff between bundle size & correctness. My understanding is the original implementation was done w/ regex because it would enable us to keep the project small while producing relatively accurate (but not perfect) highlighting. If we start taking advantage of these sorts of things, we potentially start blowing up the size of the language definition in order to improve highlighting correctness.
That isn't to say I'm opposed to this change, but means we should be aware of what the trade-offs are here. I think it would also be helpful, in advance of something like this, to add a GH bot that tells us the bundle size impacts of our changes so we can discuss those tradeoffs with data as they come in.

the tradeoff between bundle size & correctness.

Yeah. I usually value correctness more than bundle size, so please do hold me back if I went overboard.
That being said, when I propose (or implement) these ideas that are motivated by one or a few specific cases (e.g. #2190, #2002), I usually think in terms of how well the idea will scale. I.e. if it took 1kb to implement a logic that saves 100bytes every time it's used, then it might be worth it.
I also want to say that most of these ideas are motived from cutting down complexity. This one is too. This might seem al little contradictory because implementing these matchers as described above is no easy feat.

the tradeoff between bundle size & correctness.

Yeah. I usually value correctness more than bundle size, so please do hold me back if I went overboard.
That being said, when I propose (or implement) these ideas that are motivated by one or a few specific cases (e.g. #2190, #2002), I usually think in terms of how well the idea will scale. I.e. if it took 1kb to implement a logic that saves 100bytes every time it's used, then it might be worth it.
I also want to say that most of these ideas are motived from cutting down complexity. This one is too. This might seem contradictory because implementing these matchers as described above is no easy feat.
My end game with this issue to implement something like an LL parser. We would supply a CF grammar and the matcher will be generated for us. Easy to use and as declarative as regexes. Implementing this won't be easy but we only have to it once. I also intend this to replace constructs like this I created out of necessity.

add a GH bot that tells us the bundle size impacts of our changes so we can discuss those tradeoffs with data as they come in.

#1787

@RunDevelopment Duh, yes, I knew I had an issue about that. Let me look into doing that soon. Maybe we consider doing a migration from TravisCI -> GH Actions in the process.


Fix error display string template in Ps4.tsx. Add missing Web API import.

I wasn't able to get the PS4 Mode addon to work, it just showed the following error at me after clicking Verify and Save:  'ERROR: Could not verify required files: ${e}'
Turns out that was a string literal instead of a template. Replacing the ' with ` in Ps4.tsx made it a template and enabled printing the actual error.
The actual error was that WebAPI wasn't found. I added the missing import and it started working.

Great fix, thanks for the PR!
The missing webApi import was probably my bad when separating all the addons from the addonConfigPage.


E2E repeated failure

I've notices some repeated test failure because of:
Unable to retrieve the operator version: no matches for kind "IntegrationPlatform" in version "camel.apache.org/v1"
Error: the server could not find the requested resource (get integrationplatforms.camel.apache.org)

Apparently, when we delete the resource after the tests are completed, we sometimes are not able to delete the IntegrationPlatform. We should figure out why and avoid the test to fail for such reason.

fyi @tadayosi

Where do you find this?  I looked at the upgrade testing for release-1.9.x branch but it seems to be failing with the same error.
https://github.com/apache/camel-k/runs/6647282436?check_suite_focus=true
=== RUN   TestOLMAutomaticUpgrade
    olm_upgrade_test.go:64: Testing cross-OLM channel upgrade stable-1.9 -> stable-dev-1.9
Using external kamel binary on path /tmp/kamel-1.9.1
OLM is available in the cluster
Error: exit status 2
Usage:
   [flags]

Flags:
  -h, --help   help for this command

    olm_upgrade_test.go:80:
        Expected success, but got an error:
            <*exec.ExitError | 0xc00079b320>: {
                ProcessState: {
                    pid: 50946,
                    status: 512,
                    rusage: {
                        Utime: {Sec: 0, Usec: 240206},
                        Stime: {Sec: 0, Usec: 36641},
                        Maxrss: 67340,
                        Ixrss: 0,
                        Idrss: 0,
                        Isrss: 0,
                        Minflt: 4642,
                        Majflt: 30,
                        Nswap: 0,
                        Inblock: 4768,
                        Oublock: 0,
                        Msgsnd: 0,
                        Msgrcv: 0,
                        Nsignals: 0,
                        Nvcsw: 1099,
                        Nivcsw: 659,
                    },
                },
                Stderr: nil,
            }
            exit status 2
    dump.go:44: -------------------- start dumping namespace test-65e61bbb-89ec-43e5-90bd-4fc693ebd592 --------------------
    test_support.go:1752: Error while dumping namespace test-65e61bbb-89ec-43e5-90bd-4fc693ebd592: the server could not find the requested resource (get integrationplatforms.camel.apache.org)
--- FAIL: TestOLMAutomaticUpgrade (39.77s)

Note the last log line just comes from test_support.go where it tries to run the final clean up of the test, which I think just fails because the main part of the test has failed and there's no resource to clean up.

It's in the latest merge of main branch (https://github.com/apache/camel-k/runs/6647187769?check_suite_focus=true). I've also seen it in other PR on main as well.

Ah I see. It looks like a false alarm as well, which is coming from the clean up script. For the current main branch, it seems the problem is in the CI script itself.
make: 0.0: Command not found
make: *** [Makefile:530: bundle] Error 127
Error: Process completed with exit code 2.
Error: Unable to process file command 'env' successfully.
Error: Invalid environment variable format '0.0'

This error makes me feel that somewhere in the script there is code that tries to process the revision from 1.10.0 and fails to extract, which used to be 1.9.n and thus worked. I'll see if I can fix it quickly.

The root cause is that this Makefile variable has to be a single-line string but is actually multi lines:
BUNDLE_METADATA_OPTS=--channels=stable-dev-1.1
0.0 --default-channel=stable-dev-1.1
0.0 --package=camel-k

This makes make bundle fail on CI.


Apply tolerations to ray pods defined by env var in KFP v2

Search before asking

[X] I searched the issues and found no similar issues.

Component
KFP workflows
Feature
Related to #620 and PRs https://github.com/IBM/data-prep-kit/pull/643 and https://github.com/IBM/data-prep-kit/pull/627
Following @roytman request in this pr tolerations for ray pods (and kfp pods) can be applied for KFP v1 when exporting the following env var before compiling the pipeline; however, for KFPv2 its is not supported yet.
export KFP_TOLERATIONS='[{"key": "key","operator": "Equal", "value1": "value", "effect": "NoSchedule"}]'

In addition, for both KFP v1 and v2 the tolerations for ray pods can be specified in the pipeline params:  ray_head_options and ray_worker_options.
for example:
  "cpu": 1,
  "image": "quay.io/dataprep1/data-prep-kit/noop-ray:latest",
  "memory": 4,
  "tolerations": [
    {
      "key": "preprocessing",
      "operator": "Equal",
      "value": "dedicated",
      "effect": "NoSchedule"
    }
  ]
}

Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Done with #627 and #643


Native editorconfig first support

What would you think of the following two things for Oni?

using an editorconfig file for the core editor settings
automatically using editorconfig files in a project when present

Having an editorconfig at the core allows for easy config sharing between machines (dotfiles) and straightforward fallback per configuration rule from project > core level. Secondly it means having support for every editorconfig rule without concessions which happen when integrating editorconfig support as a plugin. What usually happens with plugins is they get in conflict with the core editor settings, e.g. in Atom using the editorconfig plugins means having to disable the core whitespace package.
Finally as a bonus Oni would get a nice place in the native support list at https://editorconfig.org/ :)

@bryphe thoughts?

Hey @Siilwyn ! Thanks for pinging me on this.
As I've been testing VSCode extensions - I've been thinking about editor-config in the context of the VSCode extension (https://github.com/editorconfig/editorconfig-vscode) - it'd be a good test case for our configuration synchronization.
Exposing it via the VSCode extension has a few benefits, IMO:

Keeps our implementation simpler (we get the functionality via the VSCode extension host). Our configuration story is already complicated - as we have a configuration.json and plan on supporting some customization via VimL as well. There's no ReasonML / OCaml core, yet, which means we'd have to implement it ourselves - whereas the editorconfig-core-js seems pretty well maintained.

We get to exercise our extension host integration - the more we exercise it, and work through the issues, the better and broader our extension support becomes. The editor-config plugin is a great test case for the configuration APIs.

So at least for our first round of editor-config integration - I plan on integrating with the VSCode extension for the dual benefit of us saving us work and exercising our extension integration.
If there's gaps beyond that, that could be solved by a native implementation, and it's something users want - I'm open to exploring it. But interested to see how far the extension can get us.

Hey @bryphe no problem! Good points, especially for the foreseeable future going for 'integration' of the EditorConfig VSCode extension makes the most sense.
I'll close this issue until maybe someday it becomes relevant again. :)


Android Nougat: Email Context Menu at Wrong Position

Expected behavior
If I long tap on a message the context menu should appear besides the finger position.
Actual behavior
The context menu appears at the lower end of the screen and is only 1-2 lines in size.
Steps to reproduce

Select a message
Long tap
See what happens

Environment
K-9 Mail version: 5.010
Android version: Nougat (7) Release
Account type (IMAP, POP3, WebDAV/Exchange): All (type independent)

This is probably a bug in Android Nougat. We'll have to find a way to work around it.

I reported the bug to AOSP: https://code.google.com/p/android/issues/detail?id=221643


SetEndpointNameFormatter prefix for multibus configuration does not work.

Contact Details
<EMAIL_ADDRESS>Version
8.x
On which operating system(s) are you experiencing the issue?
Windows, MacOS, Linux
Using which broker(s) did you encounter the issue?
RabbitMQ
What are the steps required to reproduce the issue?
1. Register MassTransit:

public static void AddMassTransit(this IServiceCollection services, string connectionString, string anotherConnectionString)
    {
        services.AddMassTransit(x =>
        {
            x.SetEndpointNameFormatter(new KebabCaseEndpointNameFormatter("", false));
            x.UsingRabbitMq((context, bus) =>
            {

                bus.Host(connectionString);
                bus.ConfigureEndpoints(context);
            });
        });

        services.AddMassTransit<ISecondBus>(x =>
        {

            x.SetEndpointNameFormatter(new KebabCaseEndpointNameFormatter("mobile", false));
            x.UsingRabbitMq((context, bus) =>
            {
                bus.ConfigureEndpoints(context);
                bus.Host(anotherConnectionString);

            });
        });
    }

    public interface ISecondBus :
    IBus
    {
    }

### What is the expected behavior?

- It is expected to add "mobile" prefix on second bus. But it is not added.
- It works with one bus.
- I can register default formatter in DI like this
 `builder.Services.AddSingleton<IEndpointNameFormatter>(provider => new KebabCaseEndpointNameFormatter("mobile", false));`,
 but it changes both buses.

### What actually happened?

Endpoint prefix does not exist for second bus.

### Related log output, including any exceptions

_No response_

### Link to repository that demonstrates/reproduces the issue

_No response_

If you are using different endpoint name formatters with multiple bus instances, you can specify the endpoint name formatter directly on the ConfigureEndpoints call (rather than setting it in the container, which as you've experienced, doesn't work the way you would like since it's a single registration in the container).
You'd set it using:
bus.Host(anotherConnectionString);
bus.ConfigureEndpoints(context, new KebabCaseEndpointNameFormatter("mobile", false));

Also, notice that order can matter, and ConfigureEndpoints should always be the last call in your configuration.


Add source details to mail deliveries

Right now librato-rails adds mailer gauges by default which is great. It would be 10x better IMO if it were to include source details (what type of email is being sent). could this be added?

It looks like unfortunately the only relevant info received from the subscription is the mailer and subject. Feel free to disregard this.

@sgringwe You could definitely add support for including this as a custom instrumentation yourself, but yeah this info isn't included via the built-in ActiveSupport::Notifications instrumentation.


Fixing SM1 shaders on SPIR-V after TEX* opcodes changes

I ran into two problems:

SPIR-V emittor relies on input attributes implied by ps_1_3 (and below) TEX* opcodes, but those have been recently removed, causing SPIR-V emittor to produce invalid binaries. However, once that was fixed, I noticed that some shaders were declaring these implicit input multiple times. Which brings me to ...
Only TEX opcode had its implicit input attribute removed. However, other TEX* opcodes (TEXBEM, etc.) have not only been still adding theirs, they were also re-adding those from TEX.

Both problems are fixed by the PR, but it might be worth discussing problem 2. with Ryan, as I'm only guessing his intention was to remove these implicit attributes everywhere.

Commit in question: https://github.com/FNA-XNA/MojoShader/commit/1d735fd921a9d3f7fa3e038d0ea159b2efd9eb09
Will send this for review...

These patches look good to me; feel free to push them upstream!

Applied to upstream: https://hg.icculus.org/icculus/mojoshader/rev/02d36217591b


Variabilize backdrop class name (#34094)

To resolve #34094, create a new class offcanvas-backdrop which extends modal-backdrop. It helps avoid confusion if an offcanvas and a modal is open and two backdrop are shown.

@romdum I did some cleanup in order to help this proceed. I hope you don't mind
@twbs/css-review any help on the sccs?
The main cause for now is just to extend the modal-backdrop class, use it on offcanvas-backdrop, (initial idea)
In order to avoid duplication, one Idea is to use a new file backdrop.scss create a new class backdrop and use it to both as extend

Js  scoped, seems ok

Thanks for the fix lol, I must have been braindead earlier. I pushed another change to fix our z-indexes. I'll make sure to cover this in our highlights for the blog post btw. New z-indexes make more sense now IMO.

I'm on airplane wifi and I screwed up the push, so this was merged to main. Working to at least squash those commits in main and we can iterate in subsequent PRs if needed. Sorry about that folks!

I'm on airplane wifi and I screwed up the push, so this was merged to main. Working to at least squash those commits in main and we can iterate in subsequent PRs if needed. Sorry about that folks!


QUESTION: is it possible to customize which folders Zeus monitors for file changes?

We have many files in non-standard locations, and I would like to understand if it's possible to customize which folders each command should be watching?
Thanks in advance!

+1 to this question.

Yes you can customize this as part of your custom_plan.rb file. Basically zeus will track all files which are loaded by mokeypatching load see https://github.com/burke/zeus/blob/master/rubygem/lib/zeus/load_tracking.rb so depending on which files are loaded by which command the those files will be watched. https://github.com/burke/zeus/blob/master/rubygem/lib/zeus/rails.rb would be the default rails setup which you would want to override given your file setup. For the way it works take a look at Zeus::LoadTracking.add_feature(file) Hope this helps.


Format causes breaking change for "<>"

Versions

Elixir: Elixir 1.16.0 (compiled with Erlang/OTP 26)
Styler: locked at 0.11.1 (styler) 8d995b3e

Example Input
"foo" <>
      ( for value <- 1..100 do
          "bar"
      end
      |> Enum.join(", "))

formats to invalid elixir:
    <>(
      "foo",
      for_result =
        for value <- 1..100 do
          "bar"
        end,
      Enum.join(for_result, ", ")
    )

   error: syntax error before: '<>'

Thanks much for the repot + minimal repro example =) Will release 0.11.2 with a fix later today

0.11.2 is released on hex with the fix :) cheers


Fix hang on Invalid Namespace

If you try and connect to an Invalid namespace you get an error back - but without any handler you end up just hanging (this issue was previously described in more detail here:  https://github.com/invisibleroads/socketIO-client/issues/84)
I'm not sure whether this belongs in this SocketIO "namespace" or the parent EngineIO - welcome to adjust. Also welcome to adjust on the exception (whether Socket IO specific or another generic Python one)

Just read contributing.md - moving it to develop :)


changed font family, weight, and spacing for logged in user and gave …

…drop down unique id

Is this good to go?
@birdage @dlangston


Refactor minds to be entities with components, make roles components

Requires https://github.com/space-wizards/RobustToolbox/pull/4328
About the PR
It would probably be better if roles were entities too but that's for a future PR.
Technical details
Minds are now entities with their own components attached to them, such as MindComponent and all the role components.
Currently it is not possible to have multiple traitor roles at the same time, for example. Neither was it before, but the code only checked that sometimes, now its consistent.
Most cases of code interacting with minds should go through MindSystem.TryGetMind
Media
mindrefactor.webm

Traitor and briefings (after breakpointing the code to force captain to not block antag selection):

End game screen:

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes
Minds have been made entities with attached components.
Roles have been made components.
Any code using minds should go through MindSystem now.
PlayerData stores an EntityUid representing the mind entity instead of a Mind object.

Reviewing


skikoMain API are shown as available in commonMain

Compose 1.5.0-beta02, Kotlin 1.9.0, IDEA 2023.1.3

open compose-multiplatform template
try to write Modifier.onPointerEvent in commonMain and PopupProperties(scrimColor = Color.Black)

Actual

These functions are available in commonMain
Expected
These functions shouldn't be available in commonMain, and IDE shouldn't suggest them, because this API isn't avaialable in androidMain
It happens for all API that is put in skikoMain sourceSet in the Compose sources. It doesn't happen for API in desktopMain/uikitMain

Link to IDEA's YT: https://youtrack.jetbrains.com/issue/KTIJ-27360

onPointerEvent is defined in foundation module in skikoMain sourceSet.
Build files:
build.gradle
redirecting android target to Google artifacts

ComposeTest.zip
Here is the really simplified example with just a KMP gradle plugin applied and with couple dependencies.
In this project I have the following gradle script:
kotlin {
    androidTarget()
    iosArm64()
    sourceSets.commonMain {
        dependencies {
            implementation("org.jetbrains.compose.ui:ui:1.5.11")
        }
    }
}

So I declared a dependency to the org.jetbrains.compose.ui:ui. And you can see in this project that in commonMain we don't see the ui:skikoMain sourceSet, but we can see the API from  ui-graphics:skikoMain library. That is odd. What is the difference?
Well... it is complicated. To explain what is going on we need to dive into how the KMP project resolution for IDE works.
Basically, we have 2 targets in the project -- android and iosArm64. And commonMain has dependencies to the org.jetbrains.compose.ui:ui:1.5.11. So for every target we need to find corresponding variant in gradle metadata.
Gradle does it by looking into the ui-1.5.11.module and finding there following relevant variants:
   {
      "name": "releaseApiElements-published",
      "attributes": {
        "org.gradle.category": "library",
        "org.gradle.usage": "java-api",
        "org.jetbrains.kotlin.platform.type": "androidJvm"
      },
      "dependencies": [
        {
          "group": "androidx.compose.ui",
          "module": "ui",
          "version": {
            "requires": "1.5.4"
          }
        }
      ]
    }
....
      {
      "name": "uikitArm64ApiElements-published",
      "attributes": {
        "artifactType": "org.jetbrains.kotlin.klib",
        "org.gradle.category": "library",
        "org.gradle.usage": "kotlin-api",
        "org.jetbrains.kotlin.native.target": "ios_arm64",
        "org.jetbrains.kotlin.platform.type": "native"
      },
      "available-at": {
        "url": "../../ui-uikitarm64/1.5.11/ui-uikitarm64-1.5.11.module",
        "group": "org.jetbrains.compose.ui",
        "module": "ui-uikitarm64",
        "version": "1.5.11"
      }
    }

So, for android it founded the releaseApiElements-published and also added it's dependencies into the resolution (androidx.compose.ui:ui:1.5.4) and for iosArm64 we found uikitArm64ApiElements-published and went by available-at link to the ui-uikitarm64-1.5.11.module:
{
      "name": "uikitArm64ApiElements-published",
      "attributes": {
        "artifactType": "org.jetbrains.kotlin.klib",
        "org.gradle.category": "library",
        "org.gradle.usage": "kotlin-api",
        "org.jetbrains.kotlin.native.target": "ios_arm64",
        "org.jetbrains.kotlin.platform.type": "native"
      },
      "dependencies": [
        {
          "group": "org.jetbrains.compose.runtime",
          "module": "runtime",
          "version": {
            "requires": "1.5.11"
          }
        },
        {
          "group": "org.jetbrains.compose.runtime",
          "module": "runtime-saveable",
          "version": {
            "requires": "1.5.11"
          }
        },
        {
          "group": "org.jetbrains.compose.ui",
          "module": "ui-geometry",
          "version": {
            "requires": "1.5.11"
          }
        },
        ...
      ],
      "files": [
        {
          "name": "ui.klib",
          "url": "ui-uikitarm64-1.5.11.klib",
          "size": 1727779,
          "sha512": "755a7681ed3c852268ab7a3e5e2a175aa5b036d38ad4e6cb1de42d47cd48df7c152c235a82062fa13c4726037e15632aa5cd4e2fb988ae33fe08e6448bb10e46",
          "sha256": "1203245b23387d425798001ad060e60492a2af7a529440836335c2ee6619f394",
          "sha1": "f2fe26d45d52e07cec972c97247903338a20af20",
          "md5": "7a7ab2f53f8776c1a482aea0a0a95a2f"
        }
      ]
    },

As you can see for iOsArm64 variant we have dependencies to other org.jetbrains.compose parts declared, so we added then as well. And that is the key difference here! For android target we don't adding dependency to the org.jetbrains.compose.ui:ui-geometry:1.5.11 because it isn't declared in the corresponding variant, and because of that, when we finished the resolution we see the following picture (I highlighted only org.jetbrains.compose.ui:[ui|ui-geometry] dependencies.

androidMain has dependencies to the:

org.jetbrains.compose.ui:ui:1.5.11, variant releaseApiElements-published

iosMain has dependencies to the:

org.jetbrains.compose.ui:ui:1.5.11, variant uikitArm64ApiElements-published
org.jetbrains.compose.ui:ui-geometry:1.5.11, variant uikitArm64ApiElements-published

Later on, for every KMP library we are trying to determine what should be visible from commonMain source set and to do that we need the information about what variants of the KMP library were used in every targets that user has in their project. And unfortunately, for org.jetbrains.compose.ui:ui-geometry:1.5.11 we got an incorrect situation -- this library is not used in android target at all.
Here the corresponding code in Kotlin repo: SourceSetVisibilityProvider#getVisibleSourceSets
And comment in the code:
Returning null if we can't find the given dependency in a certain platform compilations dependencies.
This is not expected, since this means the dependency does not support the given targets which will
lead to a dependency resolution error.

Esoteric cases can still get into this branch: e.g. broken publications (or broken .m2 and mavenLocal()).
In this case we just return null, effectively ignoring this situation for this algorithm.

Ignoring this will still lead to a more graceful behaviour in the IDE.
A broken publication will potentially lead to 'too many' source sets being visible, which is
more desirable than having none.

So, how to fix that?
The answer is quite simple -- we need to add dependencies to the org.jetbrains.compose libraries into the releaseApiElements-published variant as well.
Btw, the reason, why the compilation is fine also clear -- all the releaseApiElements-published variants of Compose libraries don't have any files, so they don't add any actual files to the classpath or IDE resolution for Android target. But these "fake" variants are crucial for KMP Gradle plugin to figure out what should be visible in commonMain.
To fix that properly in Compose KMP publication you probably should consult with @antohaby. Also cc @Tapchicoma.

Temporary reverted

Should be fixed in 1.7.0-dev1686

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Looking forward to USB Memory Stick support！

Am planning to support USB Memory Stick, can you provide some ideas?
I found that USB Memory Stick and Fat filesystem support has been implemented in this link, clueless as to how to embed it in an ftp server. Thanks for your reply.
https://github.com/espressif/esp-dev-kits/tree/ac453a16cc46b04304431dbca2acdd4130df2737/esp32-s3-lcd-ev-board/examples/usb_msc_file_sys

There is sample code here.
However, Memory Stick is extremely slow.
https://github.com/espressif/esp-idf/tree/master/examples/peripherals/usb/host/msc

Has there been any consideration of webDav support?

👍 Good Job  ,Thx


[stable/nginx-ingress] Add hide-headers option to nginx-ingress.

What this PR does / why we need it:
This PR adds the hide-headers option to the controller-configmap.yaml for nginx-ingress.
Special notes for your reviewer:
N.A.
For fast feedback, please @-mention maintainers that are listed in the Chart.yaml file:
@jackzampolin
@mgoodness
@chancez

Why do you need a special settings for this? You can already configure it.

I guess you mean by using controller.config? To be honest, I totally missed that value.

I assume this can be closed now.


Create minimum viable documentation

Fixes #108 , see https://deeprankcore.readthedocs.io/en/minimum-viable-documentation/ for preview :)
Also fixes #122

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
0

Relevant Lines:
0

💛  - Coveralls


4.0 Theme Documentation Does Not Exist

On the Examples page of the 4.0 Documentation, there is a section for Themes that links to the Options page, presumably to describe that functionality in greater detail. However, there is no Themes section on the Options page, the link takes you to a non-existent anchor, and there is only a single mention of themes at all (in passing about setting default options).

:+1:

I would really appreciate some documentation for the theme option too

There is documentation for basic usage here, but a theme authoring guide would be helpful - if someone wants to contribute to that.

The documentation was completely redone a while ago and is much, much improved now. This is definitely able to be closed now

5 years is passed and yet there is nothing on documentation specifying how to create a new theme:
https://select2.org/appearance


Improve library

Fixes #30
Fixes #13
Fixes #10
Fixes #5 (though maybe could be better?)

Released this in v0.2.0 FYI @Sytten - thanks again.  Especially for writing the README, been putting that off far too long.


Add hyperopt for experiments

Add experiment.experiment, where class of Experiment, and LGBExperiment subclass.
A minimal running example in check_data.py

replace #49

Close this one since it is replaced by #59


Logic bug in arrow_writer?

https://github.com/huggingface/datasets/blob/88a902d6474fae8d793542d57a4f3b0d187f3c5b/src/datasets/arrow_writer.py#L475-L488
I got some error, and I found it's caused by batch_examples being {}. I wonder if the code should be as follows:
-        if batch_examples and len(next(iter(batch_examples.values()))) == 0:
+       if not batch_examples or len(next(iter(batch_examples.values()))) == 0:
            return

@lhoestq

Hi @alvarobartt ,
Thanks for answering. Do you know when and why an empty batch is passed to this function? This only happened to me when processing with multiple workers, while chunking examples, I think.

Thanks, I added a if-print and I found it does return an empty examples in the chunking function that is passed to .map().

Glad I could help you out @cccntu :hugs:

Hi ! We consider an empty batch to look like this:
empty_batch = {
    "column_1": [],
    "column_2": [],
    ...
}

While {} corresponds to a batch with no columns.
Therefore calling this code should fail, because the two batches don't have the same columns:
writer.write_batch({"a": [1, 2, 3]})
writer.write_batch({})

If you want to write an empty batch, you should do this instead:
writer.write_batch({"a": [1, 2, 3]})
writer.write_batch({"a": []})

Makes sense, then the if-statement should remain the same or is it better to handle both cases separately using if not batch_examples or len(next(iter(batch_examples.values()))) == 0: ...?
Updating the regressions tests with an empty batch formatted as {"col_1": [], "col_2": []} instead of {} works fine with the current if, and also with the one proposed by @cccntu.

Great thanks for the response! So I'll just add that regression test and remove the current if-statement.

Hi @lhoestq ,
Thanks for your explanation. Now I get it that {} means the columns are different. But wouldn't it be nice if the code can ignore it, like it ignores {"a": []}?

BTW,

There's a check later in the code that makes sure that the columns are the right ones, so I don't think we need to check for {} here

I remember the error happens around here:
https://github.com/huggingface/datasets/blob/88a902d6474fae8d793542d57a4f3b0d187f3c5b/src/datasets/arrow_writer.py#L506-L507
The error says something like arrays and schema doesn't have the same length. And it's not very clear I passed a {}.

But wouldn't it be nice if the code can ignore it, like it ignores {"a": []}?

I think it would make things confusing because it doesn't follow our definition of a batch: "the columns of a batch = the keys of the dict". It would probably break certain behaviors as well. For example if you remove all the columns of a dataset (using .remove_colums(...) or .map(..., remove_columns=...)), the writer has to write 0 columns, and currently the only way to tell the writer to do so using write_batch is to pass {}.

The error says something like arrays and schema doesn't have the same length. And it's not very clear I passed a {}.

Yea the message can actually be improved indeed, it's definitely not clear. Maybe we can add a line right before the call pa.Table.from_arrays to make sure the keys of the batch match the field names of the schema


Tests for pollset_set.

Note: I initially created https://github.com/grpc/grpc/pull/9009 for this but that did not do it the right way.
This is a test for pollset_set. This is meant to be a test for pollset_set implementations in all polling engines - but currently it only works for epoll engine (the poll engine's pollset_set implementation currently fails this test and I will be modifying that soon)

Adding DO NOT MERGE label since it contains changes to build.yaml file (and since we are holding off all merges that modify build.yaml for now)

All test passed. The failure 'Based Full Build & Tests' is a known issue.

ping. @ctiller , can you take a look at this ? (else, this might result in merge conflicts and I'll have to wait for all the tests to run :) )

Bunch of compilation issues:
test/core/iomgr/pollset_set_test.c:271:16: error: implicit declaration of function 'GRPC_TIMEOUT_MILLIS_TO_DEADLINE' is invalid in C99 [-Werror,-Wimplicit-function-declaration]
    deadline = GRPC_TIMEOUT_MILLIS_TO_DEADLINE(2);

https://grpc-testing.appspot.com/job/gRPC_pull_requests_asan_c/1652/console

Fixed the compilation issues. The failures now are all known. This can be merged.


Allow ticks as escape character to avoid backslash escapes inside of template strings in JS?

If you don't want to use the AQL query builder in JavaScript, you can still embed plain AQL queries in your JS code. There are multiple options how to do multi-line strings for that, but the most convenient way in ECMA2015 times is probably to use template strings:
let query = `some
multi-line
string`

If you got collection names which conflict with AQL keywords, they need to be wrapped in backticks. Not allowed:
FOR f IN filter
  RETURN f.sort

Valid:
FOR f IN `filter`
  RETURN f.`sort`

Backticks are also used for template strings in JS, which means, you can't do this:
let query = `
FOR f IN `filter`
  RETURN f.`sort`
`

You need to do:
let query = `
FOR f IN \`filter\`
  RETURN f.\`sort\`
`

... which is not very pretty. If Arango supported forward ticks instead, or in addition, it could become:
let query = `
FOR f IN ´filter´
  RETURN f.´sort´
`

Please discuss!

@CoDEmanX : the devel branch supports the usage of forward ticks as an alternative. Can you give it a try and check whether it's any helpful?

Works! @pluma and me believe it's useful, even if it's limited to JS users.
We actually wonder if forward ticks have meaning in any programming language, or if they are a neglected character...


Typo

[Enter feedback here]
First ref to 'https://localhost:5001/api/TodoItem' should be 'https://localhost:5001/api/TodoItems' (as per Postman screenshot)

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

ID: 583ae3e7-5915-f22b-e54c-6c100cf2a7da
Version Independent ID: 95f2549e-29f9-e31a-b4d0-432b971643ff
Content: Tutorial: Create a web API with ASP.NET Core
Content Source: aspnetcore/tutorials/first-web-api.md
Product: aspnet-core
Technology: aspnetcore-tutorials
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

It is.

Did this get fixed recently? As soon as I returned to the page it appears fixed...

Yes, take a look at the history
Merge to live was an hour ago.


Direct and Inverse Mappings Owlapy objects into DL syntax in string and Manchester systax string

Previously,
we use the comma in an Enumaration of Individuals to seperate individuals in the DL syntax.
Yet, such usage can lead to confusion as the meaning of comman does not indicate AND or OR,e.g. "∃ hasParent.{F8F137 , F5M63 , F6F77}"
C = OWLObjectOneOf((OWLNamedIndividual("http://example.com/example#a"), OWLNamedIndividual("http://example.com/example#b")))
assert renderer.render(C)=="{a ⊔ b}"

This idea of using logical and/or also traltes to OWLDataOneOf as shown below
c = OWLDataSomeValuesFrom(property=has_age,
                           filler=OWLDataIntersectionOf([OWLDataOneOf([OWLLiteral(32.5), OWLLiteral(4.5)]),
                                                         IntegerOWLDatatype]))
rendered_c = renderer.render(c)
self.assertEqual(c, parser.parse_expression(rendered_c))
self.assertEqual(rendered_c, "∃ hasAge.({32.5 ⊔ 4.5} ⊓ xsd:integer)")

Issues to be fixed in this branch
1
During refactoring, found the following test
c = OWLDataAllValuesFrom(property=has_age, filler=OWLDataUnionOf([owl_datatype_min_max_inclusive_restriction(40, 80), IntegerOWLDatatype]))
rendered_c = renderer.render(c)
self.assertEqual(c, parser.parse_expression(rendered_c))
self.assertEqual(rendered_c, "∀ hasAge.(xsd:integer[≥ 40 , ≤ 80] ⊔ xsd:integer)")

Meaning of xsd:integer[≥ 40 , ≤ 80] is also unclear. Could you please replace the the comma with logical AND @alkidbaci ?
2
test_owlapy_render.py and test_owlapy_parser.py should be included in test_owlapy_conversions.py
could you take care of it @LckyLke
3
C and CC must be equal right ? @alkidbaci @LckyLke ?
C = OWLObjectOneOf((OWLNamedIndividual("http://example.com/example#a"), OWLNamedIndividual("http://example.com/example#b")))
rendered_c = renderer.render(C)
assert rendered_c=="{a ⊔ b}"
print(rendered_c)
CC = OWLObjectOneOf((OWLNamedIndividual("http://example.com/example#b"), OWLNamedIndividual("http://example.com/example#a")))
print(C==CC)

This problem is caused by the usage of tuple instead of a set.

Currently, print(C==CC) does only hold if we use a set as the iterable like this:
C = OWLObjectOneOf({OWLNamedIndividual("http://example.com/example#a"), OWLNamedIndividual("http://example.com/example#b")})
CC = OWLObjectOneOf({OWLNamedIndividual("http://example.com/example#b"), OWLNamedIndividual("http://example.com/example#a")})
print(C==CC)

Cause the equality method of OWLObjectOneOf is not order independent:
    def __eq__(self, other):
        if type(other) is type(self):
            return self._values == other._values
        return NotImplemented

so maybe we should change it? We could convert both values to sets first?

Currently, print(C==CC) does only hold if we use a set as the iterable like this:
C = OWLObjectOneOf({OWLNamedIndividual("http://example.com/example#a"), OWLNamedIndividual("http://example.com/example#b")})
CC = OWLObjectOneOf({OWLNamedIndividual("http://example.com/example#b"), OWLNamedIndividual("http://example.com/example#a")})
print(C==CC)

Cause the equality method of OWLObjectOneOf is not order independent:
    def __eq__(self, other):
        if type(other) is type(self):
            return self._values == other._values
        return NotImplemented

so maybe we should change it? We could convert both values to sets first?

c723e7f fixes this at the initialization

We should refrain from returning an exception return NotImplemented.

test_owlapy_render.py and test_owlapy_parser.py should be merged into test_owlapy_conversions.py and then be deleted?


Zcash deprecated

Verus-Desktop shows in GUI: No running daemon found for Zcash.
Debug.log supplies this info:
2020-05-29 12:36:34 UpdateTip: new best=0000000001f16164aea6249bb1df1bdf7cb42c484844923f0d483556276b9f24  height=848058 bits=469931424 log2_work=57.684841  tx=6425354  date=2020-05-29 12:35:57 progress=0.999999  cache=162.9MiB(16663tx)
2020-05-29 12:36:34 *** This version has been deprecated as of block height 848058. You should upgrade to the latest version of Zcash.
2020-05-29 12:36:34 Error: This version has been deprecated as of block height 848058. You should upgrade to the latest version of Zcash.

After which the deamon shut down.

This has been tested, confirmed, and should be resolved in the latest release, feel free to reopen if this problem persists.


Add a reaction effects section to the guidebook

About the PR

Added a short guidebook entry explaining a few reaction effects.
Why / Balance

To make the information more readily available.
Technical details

N/A
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

N/A
Changelog

:cl:

add: Added a guidebook entry for some reaction effects.

this should really be automated the same as every other chem guidebook is

this should really be automated the same as every other chem guidebook is

Yeah, you're right, it should.
This is frankly rather basic, but I hoped it'd be good enough to convey the concepts.

Closing this because it's not gonna merged in its current state and fixing it would require enough changes that it might as well be done in another PR.


Running pre-commit on local machine gives error

Checklist

[X] I've read the contribution guidelines.
[X] I've searched other issues and no duplicate issues were found.
[X] I'm convinced that this is not my fault but a bug.

Description
I'm having issues running pre-commit following these instructions:

https://autowarefoundation.github.io/autoware-documentation/main/contributing/pull-request-guidelines/ci-checks/#pre-commit

Expected behavior
pre-commit run -a works.
Actual behavior
mfc@mfc-SG13:~/projects/autoware$ pre-commit run -a
[INFO] Installing environment for https://github.com/pre-commit/pre-commit-hooks.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
An unexpected error has occurred: CalledProcessError: command: ('python', '-mpip', 'install', '.')
return code: 1
expected return code: 0
stdout:
    Executable `python` not found
stderr: (none)
Check the log at /home/mfc/.cache/pre-commit/pre-commit.log

Then I try to fix it by installing sudo apt install python-is-python3.
This time I get this:
mfc@mfc-SG13:~/projects/autoware$ pre-commit run -a
[INFO] Installing environment for https://github.com/pre-commit/pre-commit-hooks.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
An unexpected error has occurred: AssertionError: BUG: expected environment for python to be healthy immediately after install, please open an issue describing your environment

more info:

virtualenv python version did not match created version:
- actual version: <<error retrieving version from /home/mfc/.cache/pre-commit/repo4fv1sw1o/py_env-python3/bin/python>>
- expected version: 3.10.6.final.0

Check the log at /home/mfc/.cache/pre-commit/pre-commit.log

Steps to reproduce

Run pre-commit run -a

Versions

OS: Ubuntu 22.04
ROS: humble

Possible causes
I'm not sure what is causing this, I've installed pre-commit following instruction in here.
Additional context
No response

@xmfcx I think it's https://github.com/pre-commit/pre-commit/issues/2336. Could you try these commands?
rm -rf ~/.local/share/virtualenv/ && curl -fsSL https://raw.githubusercontent.com/deadsnakes/python3.10-jammy/ubuntu/jammy/debian/patches/venv-scheme.diff | sudo patch /usr/lib/python3.10/sysconfig.py

@kenji-miyake this fixed my problem, thanks!


[Autocomplete] / Types - Move 'defaultValue' definition below 'multiple'  - useAutocomplete.d.ts

[X] I have followed (at least) the PR section of the contributing guide.

Description:
Fix:
Reordered the defaultValue prop in packages/mui-base/src/AutocompleteUnstyled/useAutocomplete.d.ts, to be below multiple, which its definition depends on (ternary).
In #33269, the props in useAutocomplete.d.ts were re-ordered alphabetically. The definition of defaultValues is:
/**
   * The default value. Use when the component is not controlled.
   * @default props.multiple ? [] : null
   */
  defaultValue?: AutocompleteValue<T, Multiple, DisableClearable, FreeSolo>;

As the definition now appears above the definition of multiple, an extra [] is added to the definition and TypeScript errors are encountered:

Moving the defaultValues definition below multiple resolves this issue.
First PR, apologies if anything isn't quite right. Thanks in advance.

Hi @DaniBedz, could you please create a reproduction so we can verify the fix? It would be best if you created an issue with the description of the problem you're solving and linked this PR to it.


Revive lol-monitor

Before 0.9, there was a software called lol-monitor.
https://github.com/akiradeveloper/lol/tree/v0.8.0/lol-monitor
Because it's hard to maintain the software, I removed it. But I think it is time to revive it.

I will use ratatui. I am learning it now.


[composer] jumpy popovers

https://github.com/user-attachments/assets/4f03a48b-be54-4abf-9dac-da0f14ae2d36

Ah, the popover anchor rendering delay.
We can probably fix this with global popovers now that we can use virtualRef.


requests removes characters from valid url

Requests strips the question mark when a hash tag follows. urlopen doesn't do it. Why?
Recreate bug with:
#!/usr/bin/python3

from urllib.request import urlopen
from requests import get

url = 'http://incolumitas.com?#param=value'

native = urlopen(url)
req = get(url)

assert native.url == req.url, '{} vs {}'.format(native.url, req.url)

"""
AssertionError: http://incolumitas.com?#param=value vs http://incolumitas.com/#param=value

Requests strips the question mark when a hash tag follows. urlopen doesn't do it. Why?
"""

The question mark indicates the beginning of the query portion of the URL. The octothorpe indicates the beginning of the fragment portion of the URL. We conclude that you have an empty query string and normalise it by removing the section entirely.
If the octothorpe is intended to be part of the query string you need to percent-encode it to avoid ambiguity.

On 10 Jan 2015, at 12:52, NikolaiT<EMAIL_ADDRESS>wrote:
Requests strips the question mark when a hash tag follows. urlopen doesn't do it. Why?
Recreate bug with:
#!/usr/bin/python3
from urllib.request import urlopen
from requests import get
url = 'http://incolumitas.com?#param=value'
native = urlopen(url)
req = get(url)
assert native.url == req.url, '{} vs {}'.format(native.url, req.url)
"""
AssertionError: http://incolumitas.com?#param=value vs http://incolumitas.com/#param=value
Requests strips the question mark when a hash tag follows. urlopen doesn't do it. Why?
"""
—
Reply to this email directly or view it on GitHub.

I understand. Many thanks for the quick response. Will close.

@NikolaiT just to give you some more detail about this:
The URI structure (that URLs follow) is of the form {scheme}://{authority}{/path}{?query}{#fragment}. (On Python 3) The urllib.parse module gives us the urlparse function to look at these components of a URL.
>>> import urllib.parse
>>> uri = urllib.parse.urlparse('http://incolumitas.com?#param=value')
>>> uri
ParseResult(scheme='http', netloc='incolumitas.com', path='', params='', query='', fragment='param=value')
>>> uri.geturl()
'http://incolumitas.com#param=value'

According to this module (and the RFC that defines handling of URIs) these two URLs are equivalent. This can be seen by visiting both in your browser. In essence, if we didn't modify the URL it would be as valid as our current approach is. We do normalize URLs though because servers (and sometimes users) give us some rather bizarre URLs that will only "just work" when normalized. As core developers of a library whose core design goal is to make users' lives better, we need to take this approach to satisfy that goal.
I hope that helps give you a deeper understanding of what's happening, why it is okay, and why it should happen.


Fix MSPT cavity receiver system capacity calculation

Add default pumping power of 2% if not reported from receiver design method

Fixes https://github.com/NREL/SAM/issues/1500

Simple fix to https://github.com/NREL/SAM/issues/1500#event-10680676101
A more detailed fix would add a design-point performance calculation to the cavity model


C# Project Library Error

The problem described in this forum post is reproducible. Following the tutorial from CLI's docs will yield the same error.

Local code sharing was fixed by https://github.com/QuantConnect/lean-cli/pull/96 for both C# and python projects.
Please follow the docs here for code sharing: https://www.quantconnect.com/docs/v2/lean-cli/projects/libraries/project-libraries
Currently, code sharing in the cloud is not supported

This issue is solved already, see https://github.com/QuantConnect/lean-cli/pull/175, https://github.com/QuantConnect/lean-cli/pull/191, https://github.com/QuantConnect/lean-cli/pull/197, https://github.com/QuantConnect/lean-cli/pull/207, https://github.com/QuantConnect/lean-cli/pull/208


docs: improve input doc for plugins [APE-1344]

We were maybe having some confusions with the default behavior of this input, and it just doesnt work when using versions constraints in your ape-config.yaml, which is confusing.
A better fix that I don't have time for right now it to check if there are version constraints in the ape-config and if there, set the default to be just ..

closing in favor of https://github.com/ApeWorX/github-action/pull/26/files


Resp: Prevent site to be accesed in portrait mode

Is your feature request related to a problem? Please describe.
To prevent site to be accessed in portrait mode
Describe the solution you'd like
Adding the Switch to landscape mode screen so that the user can only access the site in landscape mode.
Describe alternatives you've considered
N/A
Additional context

Are you working on this? (Yes/No)
Yes

@Prerna-0202 @Arnabdaz, I am going to start up this feature
Thank You : )


New Feature: Rush Typescript SDK: Solana Storage Class

Description
This is a feature to enable the game developers using the Rush SDK done in Typescript. Starting with the component. The storage class is initialized and made the placeholder for its methods to connect to the actual solana program.
Link to issue
notion: card:: ts sdk:: solana storage
Type of change

[x] New feature (non-breaking change that adds functionality)
[x] This change requires a documentation update

Test plan (required)
This is still in development with collaboration with my co-candidates of JRBE
To start testing, from the project go to dir:
- cd ecs/web/sdk/typescript
Install the dependencies
- npm i
While in development, you may enable dev environment by
- npm run dev
To start the function running and do function testing, go the file '/src/index.ts'
In the src file

uncomment the line test_call_storage() to test out calling the class

In the modules/storage/ file

uncomment the line test() to test out this class

To test, build it first and start
- npm run build && npm run start
Screenshots/Screencaps
This is a screenshot showing the success of initializing the Solana Class and returns the four parameters required to be processed inside the class when you uncomment the test_call_storage()

LGTM as initial base. Merging for team to have initial structure.


Release Google.Cloud.Functions.V2Beta version 1.0.0-beta06

Changes in this release:
New features

Add IServiceCollection extension methods for client registration where an IServiceProvider is required. (commit 022fab2)

Created release for Google.Cloud.Functions.V2Beta-1.0.0-beta06


chore(repo): Update remaining GHA workflows

Description
Updates existing GHA workflows to utilized the provided actions.
Fixes cache being constantly re-created.
Checklist

[x] npm test runs as expected.
[x] npm run build runs as expected.
[ ] (If applicable) JSDoc comments have been added or updated for any package exports
[ ] (If applicable) Documentation has been updated

Type of change

[ ] 🐛 Bug fix
[ ] 🌟 New feature
[ ] 🔨 Breaking change
[x] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

Packages affected

[ ] @clerk/backend
[ ] @clerk/chrome-extension
[ ] @clerk/clerk-js
[ ] @clerk/clerk-expo
[ ] @clerk/fastify
[ ] gatsby-plugin-clerk
[ ] @clerk/localizations
[ ] @clerk/nextjs
[ ] @clerk/clerk-react
[ ] @clerk/remix
[ ] @clerk/clerk-sdk-node
[ ] @clerk/shared
[ ] @clerk/themes
[ ] @clerk/types
[x] build/tooling/chore

@tmilewski i have also applied this fix, you probably need to apply this to your changes: https://github.com/clerk/javascript/pull/2048


Missing: Object Tracking

Need to implement part tracking using something similar to: https://livecodestream.dev/post/object-tracking-with-opencv/
Tacking will be used to determine the speed of the part moving along the identification zone and to know if predictions are being made on the same part.

https://pysource.com/2021/01/28/object-tracking-with-opencv-and-python/


hotkeys: Replace C with x for composing PM.

Update of #8308.
@aedorado, could you review?
I think maybe_show_deprecation_notice should most naturally take a key and a message, but didn't want to clutter hotkey.js with the message text.

I'm test-deploying this on chat.zulip.org.
One thing we'll definitely want to change, though, is that this doesn't check if local storage is available in the browser; we should have a reasonable fallback (probably just notifying once per session).

Merged, thanks @rishig!


Enhance document for building container image and packages for local development

What would you like to be added:
I would like to enhance the existing document to include instructions for building container images locally and generating packages using the container image.
Why is this needed:
This update is necessary because the current document lacks certain steps and details required for developers to successfully build container images and generate packages. By providing comprehensive instructions, developers will have a clearer understanding of the process and can effectively build container images and generate packages for local development purposes.

Thanks! Could you add some requirements to this issue? Please correct me if I am wrong, but it looks like you already encountered some issues. Maybe you can share those issues and we can enhance the documentation accordingly.

There are some steps are missing in the document. I have draft one on https://github.com/gardenlinux/package-build/pull/20
Please review. :)


transfering DataFrame to JSON using dictionary style with Type Error

Code Sample, a copy-pastable example if possible
I want to transfer the follwing dataframe to JSON using a dictionary style.
   weightedRisk  riskAsset  safetyAsset  weightedAsset  riskRank
0      0.029839   0.029839     0.220161           0.25         2
1      0.053840   0.053840     0.446160           0.50         3
2      0.015485   0.015485     0.234515           0.25         4

Expected Output
the expected ouput should be:
   "groupedWeightedRisk": {
        "safetyAsset": [
            0.2201611228,
            0.4461597981,
            0.2345148013
        ],
        "riskAsset": [
            0.0298388772,
            0.0538402019,
            0.0154851987
        ],
        "riskRank": [
            2,
            3,
            4
        ],
        "weightedRisk": [
            0.0298388772,
            0.0538402019,
            0.0154851987
        ],
        "weightedAsset": [
            0.25,
            0.5,
            0.25
        ]
    }

But in pandas, all of the to_json method makes the ouput contains the index.
So.I  use to_dict(orient='list") and then using  json.dumps to serialize the data.
this method works well in most of the time. However , in the pervious case, the
json.dumps raise an error saying:
TypeError: 2 is not JSON serializable
then I checked the dataframe and locate the problem.
type(groupedWeightedRisk['riskRank'][0])
<class 'numpy.int64'>

one of the series is made up by numpy.int64, and this data type could not be JSON serializable directly.
Finally, I solve this problem using a function with a stupid way,that I serialise the data columns one by one.
def dataFrameToDict(df,indexMethod=None):
    result={}
    for i in df.columns:
        result[i]=json.loads(df[i].to_json(orient='records'))
    if indexMethod is not None:
        if df.index.name is not None:
            indexName=df.index.name
        else:
            indexName='index'

        if type(df.index) == pd.tseries.index.DatetimeIndex:
            result[indexName]=list(df.index.strftime("%Y-%m-%d"))
        else:
            result[indexName] = list(df.index)
    return result

So, is there any possibility that pandas could add a method in to_json that could provide the a JSON data using dictionary style?
Thanks a lot

Sorry, can't understand what you mean... Could you attach copy-pastable script and pd.show_version() as indicated in github issue template.


Introduce bounds to search for sparkles!

In c368692a3, the recursive search was changed to iterative. The big problem with both approaches, however is that as more and more positions are scanned, it takes longer to actually determine if the position was scanned. This PR introduces a radius which is used to get a fixed set of block positions to scan and as each position is scanned, particles are generated. This reduces memory footprint and avoids a O(n^2) search.
Not quite working yet; pending me not being on a plane. :)

Have you measured performance from checking the already iterated set before adding to the queue? (would help reduce at least some of the memory impact in this case)

In reference to your comment @thiakil - it was already checking if a coordinate was in the set, but it was having to check n^2 times if it was in the set, due to the traversal. The lookup wasn't that expensive, but that coupled with memory allocation and the creation of a thread could push total generation time into 10s of milliseconds at times. With this setup, it's reliably 1ms or less and we avoid memory allocation and thread creation.


array options items must have item & id properties   plz  how  to change there properties

I fixed it by mapping my array and adding a new field name, probably this is fastest way to keep going until this fix go live

thanks :D

No description provided.

const dataList = data?.map(s => ({
item: s.title,
id: s.count
}))
in my case it works


New command: Get details about Microsoft Teams device usage by user

@waldekmastykarz @VelinGeorgiev
With Microsoft Graph, we can access Office 365 usage reports resources to get the information about how people using Office 365 services. For example, use the Microsoft Teams device usage reports to get information about Microsoft Teams device usage in an organization.
graph reports teamsdeviceusageuserdetail --period D7
graph reports teamsdeviceusageuserdetail --date 2019-05-01
API Reference - https://docs.microsoft.com/en-us/graph/api/reportroot-getteamsdeviceusageuserdetail?view=graph-rest-1.0&tabs=cs
Note: This command needs Reports.Read.All permission.

Very nice suggestion! Would you like to pick it up yourself or let others chime in?

@waldekmastykarz
Sure, I will on this.
Do we need to change anything to get the needed permission?

Yes, I need to update the AAD app used by the CLI and we need to update this page in docs https://pnp.github.io/office365-cli/concepts/authorization-tokens/ with the additional permission. You can include it in your PR.

Thank you @waldekmastykarz.
Sure, I will update the documentation. Should I wait for your permission update or I can start from my side?

I've granted the permission to the CLI AAD application. You're ready to go. Don't forget to reconsent the application in your AAD to use the new permission. You can do this by executing in the command line o365 --reconsent

Sure @waldekmastykarz. I will start this tomorrow or day after.

Excellent! Thank you!

Submitted the PR for your review.

Excellent! Thank you very much @sprider. We'll review it shortly.


[tests-only][full-ci]Updated expected to failure for closed issue to new issue

This PR backports https://github.com/owncloud/ocis/pull/6769

https://drone.owncloud.com/owncloud/ocis/24348
SonarCloud did not run because the skip-i-unchanged logic knows that the unit tests did not need to be run.
At the time that CI started, the PR did not have full-ci in the title.
GitHub will not let us merge, because SonarCloud is a required status.
I will rebase and force-push - that should make everything run.


Allow date filters with only one bound

When using date_range, the filter only worked when entering both a from
and until date. This allows the query to also work when only entering
one of the two.
@jgeskens a review if you would.

Can you add a unit test for this change? :-) Otherwise looks good.

That's a bit tricky, since the unit test setup is in your polishing-branch. Can I merge this into that branch, and write the test there?

Merged into polishing branch.


📃: Interactive Periodic Table

:red_circle: Title :  Interactive Periodic Table
:red_circle: Aim :  Remember 11th class Chemistry, Periodic Table . Interative Periodic table helps you to get details of the elements with a click.
:red_circle: Tech Stack Used : HTML CSS JS
Screenshots 📷

N/A

:white_check_mark: To be Mentioned while taking the issue :

Full name : Amruta Jayanti
What is your participant role? (Mention the Open Source Program name. Eg. GSSOC, SSOC, JWOC, etc.) GSSOC

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

@AmrutaJayanti go ahead.


Template Section fails to render variable within when using dot-Paths

Hi,
I noticed some weird behaviour when using sections.
When I use {{#InCa}} the {{data.client.name}}-Variable within can be rendered.
If I use a dotted path as a section the behaviour is different. I used  {{#parameter.letterpaper_S}} and the {{data.client.name}}-Variable within wasn't rendered.
Template:
{{#InCa}}
    Test 1
    {{data.client.name}}

 {{/InCa}}
 {{#parameter.letterpaper_S}}
    Test 2
    {{data.client.name}}
{{/parameter.letterpaper_S}}

Output:
 Test 1
 STEC

 Test 2

My expectations would be, that the {{data.client.name}}-variable can be rendered in both cases.

What does the input data model look like?

Thanks for the enlightening question. The input model is added below. I did some more digging and tried to narrow it down. Instead I found, that the issue isn't the dot-syntax.
The big difference seems to be the type of the object in the input model. Once I provided "True" as string and once True as bool. The string version seems to cause the trouble.
To validate my theory I added a third test. All tests used the input model below.
Template:
{{#data.logo}}
    Test 3
    {{data.client.name}}
{{/data.logo}}

Output:
   Test 3

Input Data model:
var data = new Dictionary<string, object>()
            {
                { "data", new Dictionary<string, object>()
                    {
                        { "client", new Dictionary<string, object>()
                            {
                                { "name", "STEC" },
                            }
                        },
                        { "logo", "somethingFooBar"}
                    }
                },
                { "parameter", new Dictionary<string, object>()
                    {
                        { "letterpaper_S", "True" },
                    }
                },
                { "InCa", true },
            };

So I currently see two open questions:

Why doesn't {{data.client.name}} get rendered in each block?
Is the .net Type system too strict in the evaluation of section true/false evaluations? As the template was already used successfully in a JavaScript-Context, my current theory is that Handlebars.Net behaves different because of the type system. Javascript uses a "truthy" (Javascript ==-operator) evaluation for the sections. I am not sure if this point is a bug or a feature. A problem in the section evaluation (e.g. a supressed exception) could also be the reason for point 1.

I looked closer at your template, and it's not clear to me what you're trying to do. #data.logo is not valid syntax - the # prefix indicates a block helper name. So the block isn't valid to begin with. Another issue is when a block is created, the implicit root of that block's data scope is the model for the block. So even if #data.logo was a valid way to create a block, the data context inside would be relative to data.logo, which means for the {{data.client.name}} to work, logo would need to have a child property data which in turn have child properties client and respectively, name.
I suggest you read up on the Handlebars documentation - they do a better job explaining the basics than I can here.


Make it easier to use an existing stream reader

Potential fix for #100

Yick, unit tests failed remotely, succeeded on my compiler

Thanks!

Yick, unit tests failed remotely, succeeded on my compiler

std::string argument to iostream is only supported from C++11. I'll fix it after the merge.


同优先级规则deny优先于allow规则生效

这个 PR 实现什么功能/修复什么问题:
同优先级规则deny优先于allow规则生效
是否需要 backport 到之前的 release 分支:

release/2.11.0
release/2.10.0
release/2.9.0
release/2.8.0

/area region
/cc @swordqiu @yousong

Codecov Report

Merging #1699 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #1699   +/-   ##
======================================
  Coverage    7.65%   7.65%
======================================
  Files         412     412
  Lines       63111   63111
======================================
  Hits         4834    4834
  Misses      57869   57869
  Partials      408     408

Flag
Coverage Δ

#aFlag
7.65% <ø> (ø)
:arrow_up:

Impacted Files
Coverage Δ

pkg/compute/models/guests.go
0.71% <ø> (ø)
: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 9921c4e...2b81873. Read the comment docs.

/approve
/lgtm


CreateDirectory will fail if UNICODE macro is set in Windows.

Suggest to use CreateDirectoryA.

Resolved by 117b01f.


Provide link to other install methods on the default website

If it can't detect the platform it should provide links to the other downloads in the readme. Soon we're going to have a variety of low-tier platforms where we're less likely to guess correctly.

  More about rustup.

This is too terse to the level that it looks like a useless cliche on a website. Compare it with a wording by the Python team:

https://www.python.org/downloads/

Looking for Python with a different OS? Python for Windows, Linux/UNIX, Mac OS X, Other


ISPN-16772 GH/CI fix credentials and check run

fix quay creds and create pending check runs for running tests
also excluded all rocksdb xml test report since they often have syntax error

Thanks @rigazilla


Update stock should set

Update stock should probably set things, not add them as it is a put request.

Fixed by #34


feature: disable check for security policy in Repolint configuration files

The Repolinter Rulesets check for existence of a SECURITY.md link. We should consider disabling this. There is now a default security policy in the .github repo at https://github.com/newrelic/.github/blob/main/SECURITY.md. There is a link to that global default in the "About" menu on every repository.As a result any other repository that lacks a policy of its own will still have a security policy and a well-known UX path to it.

The global default policy is probably better than most projects will do on their own, so nudging projects to make their own security policy may actually decrease security.
Eliminating a Ruleset check reduces work for maintainers and creates engineering efficiencies.


Data Key Service Integration Tests

Do we want a test that hits the ping endpoint and the healthcheck endpoint and gets 200s and the appropriate response body in each case?

Do we want a test that hits the '/ping' endpoint and the '/healthcheck' endpoint and gets 200s and the appropriate response body in each case?
Also, noserunner only seems to run tests if the function name starts with 'test_' ?

Included the additional Endpoint tests
Correct - nose requires 'test_' to run - changes have been implemented

/ping response is plain text, not JSON.
/healthcheck test needs to check response keys contain only 'encryptionService', 'masterKey', 'dataKeyGenerator', 'encryption', 'decryption', and the values should contain only 'OK'.
the encrypted key should be passed as the 'data' parameter of the decrypt POST.

Ammended


chunk: limit number of upload goroutines

Codecov Report

Merging #1625 (e617fe1) into main (f99460e) will increase coverage by 0.03%.
The diff coverage is 63.20%.

@@            Coverage Diff             @@
##             main    #1625      +/-   ##
==========================================
+ Coverage   61.55%   61.58%   +0.03%
==========================================
  Files         133      133
  Lines       22474    22475       +1
==========================================
+ Hits        13833    13841       +8
+ Misses       7077     7073       -4
+ Partials     1564     1561       -3

Impacted Files
Coverage Δ

pkg/meta/redis.go
72.76% <29.41%> (+0.02%)
:arrow_up:

pkg/meta/utils.go
71.67% <44.44%> (-1.50%)
:arrow_down:

pkg/chunk/cached_store.go
79.41% <72.50%> (+2.88%)
:arrow_up:

pkg/vfs/writer.go
78.64% <0.00%> (-1.02%)
:arrow_down:

pkg/meta/sql.go
67.65% <0.00%> (-0.21%)
:arrow_down:

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 bc9c186...e617fe1. Read the comment docs.


redirect to default login url only after access unothorized page

I got the same problem, have anyone know how to fix this yet?
I have 2 actors: admin and user
when I try to access the user homepage without authentication, the server redirect me to the login url of admin.

+1
I'm trying to work this out too.

So not the best solution, but this post https://github.com/Hesto/multi-auth/issues/50 led me to this solution
Adding this to app/Exceptions/Handler.php render()
        if ($exception instanceof \Illuminate\Auth\AuthenticationException) {
            if ($request->is('admin')) {
                return redirect()->guest('/admin/login');
            }
        }

I got the same problem, but i'm resolved this issue.
Remove web middleware from mapAdminRoutes function in app\Providers\RouteServiceProvider.php


"." should be a valid char when creating a point label

"." should be a valid char when creating a point label.
We need it to make puns great again !

Fixed!


Fix npm start crash due to the absence of the logs folder

Actually there is a problem when you try to start the project after a build because there is no logs folder. It is easy to reproduce, just clone the repository and do: npm install && npm start.
This commit add the logs folder and ignore everything inside it.

Thank you for this.

published as v0.3.2


Missing greater than or equal and less than or equal matchers

Hi,
i'm currently using custom matchers for greater then or equal and less than or equal in lots of projects.
In my opinion they should be in the standard matcher set.
I would happily implement and document them it that's something that you would like to see within shpec.
i've named mine ge and le to have them the same as in bash, but maybe gte or lte is more recognizable.
What are your thoughts about that?

@schorsch3000 that seems like a common enough use case that it should probably be standard! What do your custom matchers look like?


Expose information about transformers

Transformers work now, but are hardly discoverable. It would be great if tools like the Editor could show a user what transformers are available in the packages they are using, and as much info about them as the transformer author can provide.  Then we just need a command in pub that reads that and prints it out in a way the Editor can consume.

This issue has been moved to dart-lang/pub#700.


Does not calculate Cisco style wildcard masks.

I would like to request that this library generates Cisco style wildcard masks, possibly with Network.wildcard().
For example, <IP_ADDRESS>/30 = <IP_ADDRESS>/<IP_ADDRESS> with a wildcard mask of <IP_ADDRESS>.

Is this part of an RFC, or is it vendor specific?

I don't believe is part of an RFC but they are still in use in networking. Obviously Cisco ACLs and OSPF configuration are the predominant features using wild mask or inverse mask. You will find them in other vendors like Juniper, Quagga, Cisco, Dell, Alcatel and many others. They are also used for discontiguous subnetwork support in routers (See RFC 1812 <IP_ADDRESS> Subnetting) and firewall configurations.
I would like to join the request to support wildcard mask or inverse mask in this library as well.

When instantiating a new Network object, how do I differentiate between a netmask and a wildcard mask? Because <IP_ADDRESS>/<IP_ADDRESS> could both mean 10/32 (netmask) and 10/0 (wildcard).

No updates. The .wildcard() method was implemented, closing issue.


[SMALLFIX] Fix unmount journal entry replay

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/10030/
Test PASSed.

Merged build finished. Test PASSed.

LGTM


Is7458 reveal animation speed

Fix #7458 for the base cases of fast/slow.

visual test works for me
only thing the assets and motion-ui symlinks don't work on a windows machine, have to mklink them for me

Based on @designerno1 's assessment that it works, going to merge.


Error with type aliases (assignment) in python<3.10

Input file:
A = Union[str, int]
B = Tuple[str]

flake8-future-annotations suggests FA100 Missing from __future__ import annotations but imports: Union, Tuple
But when using from __future__ import annotations this is the result:
from __future__ import annotations

A = str | int
B = tuple[str]

Both assignments will fail at runtime in python<3.10 (as they aren't postponed like with type hints)

Yes, this is the correct behavior. Code before 3.10 cannot always be converted, however the long time is telling the user to do something more subtle.
From __future__ import annotations
From typing import Tuple, Union

A = Union[str, int]
B = Tuple[str]
C: list[int] =  [3]

This will pass lint checks and work correctly.
Pyupgrade will now update C but now lines A or B.

Yes, this is the correct behavior. Code before 3.10 cannot always be converted, however the lint check is telling the user to do something more subtle.
from __future__ import annotations
from typing import Tuple, Union

A = Union[str, int]
B = Tuple[str]
C: list[int] =  [3]

This will pass lint checks and work correctly. Pyupgrade will now update C but now lines A or B.

Ah, that makes sense. Maybe this should be included in the documentation (README.md)?

I just realized, you don't even need to keep the Tuple, Union imports, you can put the type aliases in an if TYPE_CHECKING: block. I think that should be mentioned instead of keeping them and adding from __future__ import annotations.

You do need to keep the imports in cases like Python 3.7:
a = ["hi"]
x = cast(List[Union[int, str]], a)

You do need to keep the imports in cases like Python 3.7:
a = ["hi"]
x = cast(List[Union[int, str]], a)

Yes, for casting it's needed, but casting isn't that common (at least I don't use it) and then you can still keep the imports, but for type aliases you can put them in an if TYPE_CHECKING block. And as the plugin enforces using from __future__ import annotations, an if TYPE_CHECKING block is a good solution for it.

Casting is fairly common in coffee that I've read. In any case, if the solution doesn't cover all cases it's not worth pursuing imo


Adding check for Changes in replicas flag too

Without this isReplicaSet was set to false even when --replicas was changed.
This affected in the long run because it failed the check in line #1635 and was assigned the minimum computeNodes that was 3.
Signed-off-by: Vicente Zepeda Mas<EMAIL_ADDRESS>fixes #946

/lgtm


Remove mentions to containers in favour of pods

change landing page animation to switch from container to pod
Also switch from container mention to pod in the repo

thanks <3


CYP IAPT Young Advisors Feedback

•         How many Fruit or veg are you eating section: this needs to be made more clear that when moving up the tree, the number is increasing. Add numbers to this part
•         How are you feeling section: It is not clear what to do in this section. Why is the banana there?
•         What are you interested in section: can users have the option to write their own interests in? Or at least have a free text ‘other’ box? Some of the options might be of no interest to the users
•         What do you like and dislike about your friendships? This wording is confusing – should it be separated out to like and dislike? Is there a need to have dislike? It seems strange seeing it paired next to the word friendship.

Thank you @amgoforth 😀

Thanks Alex, I received similar feedback but from different people. What age group gave you this feedback?

Between 18-23 so not your demographic.

I am closing this issue as we have have addressed the points highlighted.


Visit default branch button is wrongly appears

My repo have different default branch, so it's not master anymore, and visit default branch button is still showing up, and when clicked it does refresh the page.

Mentioned repo (for testing):
https://github.com/ftis-admin/pinjam-ruang
RefinedGithub Ver:
<IP_ADDRESS>6
Thank you!

Hmm it's just solved now, the button are now missing.

Should i worry that this is GH's bug?
Anyway, thank you.

Nah it wasn't a GH bug, probably just cache issues.


character not working anymore

Describe your bug here.
I am adding a character when I go to debug and select the character it says "Uncaught object reference" Idk why it says this cuz I have fixed it multiple times and still doesn't work so I think there is a problem with something cuz this also happen when I was making a json so maybe there are some bugged stuff.
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
no

OH and also nvm I added a character so yeah I Did edit one thing in this build? If so, mention or summarize your changes.

The character is missing an animation

"The character is missing an animation"
which animation

Can't tell if you don't show the log or the files, that's literally the whole point of reporting a bug

@Wither362 that doesn’t matter


Last Rain (& possibly other calculated sensors) unknown

We had a short power outage yesterday - a couple of hours. After everything came back online, I noticed the Last Rain sensor is showing as unknown.

I believe this is because it has not rained since then. You can see nothing has been transmitted.

However if you look at the logbook, it was getting updated earlier yesterday morning just fine.

Looking at other calculated fields, I think they may have experience a similar issue, but because they are updated much more frequently, it wasn't noticed at the time. However looking at history I can see small gaps after the power came back on, but before a new value was sent - usually just a couple minutes. The gap can be seen here on the Feels Like sensor, and it shows similarly in the others as well.

I have not noticed this with restarts or anything like that in the past, although I can experience once we get some rain. I am curious if there is something we can do about this though. Shouldn't it retain the previous values even in this scenario? Would it be as simple as setting the retain flag on the MQTT message, assuming it's not already set? Or is there something else that we would need to do.
Thanks

Probably not on all of them simply due to the update frequency, but specifically in the instance of Last Rain, it seems like retain is essential to it's function. Depending on where one lives, there could easily be months between rainfall events. You wouldn't want to lose that value just because you rebooted or something during that time.
I started looking through my other MQTT devices and while not all, a number of them do appear to have retain set for sensor values.


Add guarded combinator

After some time I find myself needing it several times. So let's return it back. Apparently, it's occasionally useful
This combinator has the following type:
guarded :: Alternative f => (a -> Bool) -> a -> f a

I would like to take a run at this, although I am not sure where it should live? Should I make a new file Alternative.hs?

Thanks for taking this @JonathanLorimer !
I would expect it in the Relude.Bool.Guard module. @chshersh, could you please confirm that you had the same place in mind?

@vrom911 you're absolutely right. It should go to the Relude.Bool.Guard module