perlPackages.buildPerlModule: allow pre/post hooks

buildPerlModule did not allow to specify hooks like preCheck or postInstall
it also did not allow to override buildPhase or installPhase
Working with hooks was a subtle difference between buildPerlPackage and buildPerlModule and replacing buildPerlPackage with buildPerlModule (or the other way around) resulted with unexpected effects.
@aanderse @srhb @rycee

I built staging with this PR + #64736
All perl package built successfully on x86_64-linux
on aarch64-linux only perlPackages.SysCPU failed

Is it possible to fix the SysCPU issue?

Sys-CPU-0.61 dated 2001, I doubt it is aarch64-aware
And Sys::CPU is removed from CPAN
Although http://cpan.metacpan.org/authors/id/M/MZ/MZSANFORD/Sys-CPU-0.61.tar.gz is not 404, the package in not in the index.

I found a CPAN issue for it with a patch: https://rt.cpan.org/Public/Bug/Display.html?id=95400#txn-1388036
Not sure if it's worth applying or we can simply skip the package for aarch64.


Conflict between Elevated Vision and Perfect Vision

Expected Behavior
Expecting scenes to load normally when PV and EV are both active.
 This is something that started happening recently.
Steps to Reproduce
I'm not sure what settings I have that might be creating a conflict, but this is from Caewok (developer of EV)
There must be some setting in PV that is causing that. When I load PV and EV, I don't get any errors. But it is likely that some setting specific to your scenes triggers the conflict, and I just don't have that setting enabled in my testing. I recommend starting with filing an issue in the PV github (https://github.com/dev7355608/perfect-vision/issues) because none of those errors directly implicate EV. The PV mod developer will probably flag if there is something I should change in EV to accommodate it.
Errors are here.

Context
Most recent Foundry. Hosted in Oracle instance. Running Levels, EV, and PV.
Version
4.1.17
Foundry VTT Version
v10.291
Operating System
Windows
Browser / App
Chrome
Game System
dnd5e (2.0.3)
Relevant Modules
elevated vision 0.4.0, levels 3.7.5

EV 0.4.1 should fix this issue.


c++ regex_match native Ubuntu vs WSL

I have some strange behavior while using both Ubuntu and Ubuntu on Win 10 (WSL). Both versions are 18.04 and GCC, and I'm compiling with these flags: -std=c++11 -g.
My problem is that I have this regular expression: ^[\\t ]*(?:([.A-Za-z0-9_]+[:]))?(?:[\\t ]*([A-Za-z]{2,4})(?:[\\t ]+(@[A-Za-z0-9_]+(?:(?:\\+|-)[0-9]+)?|\".+?\"|\'.+?\'|[.A-Za-z0-9_]+)(?:[\\t ]*[,][\\t ]*(@[A-Za-z0-9_]+(?:(?:\\+|-)[0-9]+)?|\".+?\"|\'.+?\'|[.A-Za-z0-9_]+))?(?:[\\t ]*[,][\\t ]*(@[A-Za-z0-9_]+(?:(?:\\+|-)[0-9]+)?|\".+?\"|\'.+?\'|[.A-Za-z0-9_]+))?)?)?
for matching some assembler-like string (yeah, I know it's long).
I'm using it like this :
    std::ifstream infile(this->inFile.c_str());
	while (std::getline(infile, line)) {
		lineNumber++;
		line = line.substr(0, line.find("#"));
		if (line == "")
			continue;
		line = reduce(line);

		if (regex_match(line, m, op_reg)) {
			std::vector<std::string> ops;
			ops.push_back(std::to_string(lineNumber));
		    for (int i = 1; i < m.size(); i++) {
		    	if (m[i] != "") {
		    		ops.push_back(m[i]);
		    	}
		    }
		    this->maps.opMap.push_back(ops);
		}
	}

This piece of code is working well on my native Ubuntu installation, but on the WSL version of Ubuntu regex_match always returns false.
Have some of you already encountered this? And if so, how have you managed it ?

I found the issue is that even in WSL we have \r\n line ending style

I found the issue is that even in WSL we have \r\n line ending style

Neither WSL/RealLinux kernel nor the Windows kernel "have \r\n" line ending style. Your input file has a line ending style. This snippet will behave the same Real Linux and WSL given the same input.
You'll need more code to handle files with differing eol conventions, if that's a design goal. Bonus points for handling all the charset eol conventions, a common one on Windows being UTF16. [Which is needed if your program is going to consume assembler like strings containing characters other than 'merican.]

how have you managed it?

Short answer is strip the \r chars from the ifstream before handing off to std::getline(ins, line, '\n'). Pull the whole file into a std::stringstream is a big hammer way to do that. Boost iostreams are another way to go.
Long answer is use a library like libicu because you can't even assume a [.A-Za-z0-9_] regex has much meaning let alone the eol convention. [Shorter long answer: you don't want the long answer.]


Create esri leaflet app try2 al

you checked in part of a merge conflict and a typo here.  perhaps you could let someone else code review/merge?


Loads config from yaml file if present

If config/favicon.yml is present, and Rails.application.config_for is defined (>= 4.2), then this will prefer that file over config/favicon.json, as it conforms better to Rails best-practices, and allows for ERB templating.
I have a use-case where I want to use a different file for the master image based on an environment variable, and using ERb inside Yaml is a common practice (which is supported by config_for), whereas trying to add templating to the json file felt hacky, so this felt like an addition that others could benefit from.

Thanks! This is part of v0.0.1, freshly deployed.
I didn't know about config_for. I realize that, instead of producing config.json, RealFaviconGenerator could dump config.yml directly.
RoR 4.2 was released more than 3 years ago. I need to decide if it is safe to drop the JSON fil or if it must be maintained.


Continuing the FUNdamentals

[x] the next few modules

[x] definitions with examples - 10
[x] complete practice examples

[ ] merge progress to master

[ ] merge PR from dev => master
[ ] next commit on dev, open new PR from dev => master

#2


[RubyGemsGemInstaller] Validate checksums from the compact index

Closes #4464

woo @homu r+

:pushpin: Commit 9231ceb has been approved by indirect

Ugh, of course now that I've approved it I realized that we should add a setting to disable this check, because some people will have existing gems with invalid checksums that they want to keep using.

@homu r-

I am super happy with this, though!

@indirect added a setting to disable validation

This is failing because index.rubygems.org is inconsistent between hex & base64 encoding the checksums. Is it worth accounting for that or should we convert to one or the other server-side?

The config key should probably also to be added to the bundle config manpage section about available config keys.

Running a data migration right now to replace all checksums in the bundler-api database with the longer hex versions. I'm torn about what we should put in the actual index files, though... using base64 checksums would decrease the file size. :/

:umbrella: The latest upstream changes (presumably #4869) made this pull request unmergeable. Please resolve the merge conflicts.

Rebased

This should be good now that the checksums in the database are all consistent.
@homu r+

:pushpin: Commit 54baa4e has been approved by indirect

:hourglass: Testing commit 54baa4e with merge a238b61...

:broken_heart: Test failed - status

This should pass now

@homu retry

oops, @homu r+

:pushpin: Commit ae465eb has been approved by indirect

:hourglass: Testing commit ae465eb with merge cb24b6d...

:sunny: Test successful - status


ARROW-2945: [Packaging] Update argument check

This is follow up change for #2341.

Do this, #2344 and #2345 need to go into 0.10?

cc @kou @wesm

Ah, this one does for sure

I'm testing the changes.

build-300 https://github.com/kszucs/crossbow/branches

Leonidas would be glad: https://github.com/kszucs/crossbow/branches/all?utf8=✓&query=build-300


[UNDERTOW-1724] Remove the @RunWith from SameSiteCookieHandler, so th…

…at the server is not started twice, once by run and another time by the test
https://issues.redhat.com/browse/UNDERTOW-1724

Retest this please


Bug Library

When clicking a game, it selects a different one

Fixed this. Next time I will branch of off the issue so no separate PR needs to be made.


Fix chgrp test failure

testChgrpHPSS fails with AssertionError: Command=`hsi ls -l zstash_test`. Errors=['This was supposed to be found, but was not: 3.', 'This was supposed to be found, but was not: m.']. This occurs as far as back as v1.0.0 -- I'm unsure why this is happening, since the unit tests have been run successfully multiple time since then.
In tests/test_chgrp.py, expected_present is a string, but should be a list.
                expected_present = 'e3sm'
            else:
                expected_present = 'chgrp: HPSS is unavailable'
            self.check_strings(cmd, output + err, expected_present, ['ERROR'])

Note, however, that even changing it to a list does not make the test pass.

Partially fixed by #179.


⚠ Fortnite has degraded performance

In 7e0038c, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1404 ms

Resolved: Fortnite performance has improved in df1c172.


(refactor) Module renaming & REPL context

Signed-off-by: Jerome Simeon<EMAIL_ADDRESS>

Would rename ErgoC to ErgoCalculus everywhere, even if it's longer.


When using "Pull", provide a way to override the name that is stored.

With docker, you can docker pull godarch/void, and it will pull from "docker.io/godarch/void:latest", but when creating the image, will store it as "godarch/void".
Currently, I have to pull, then make a new image manifest, and delete the old one.
I should be able to provide this value.

Same thing with push.

I was able to hack at the remote.Resolver for the pull.
type overrideNameResolve struct {
	RealResolver remotes.Resolver
	Name         string
}

func (r *overrideNameResolve) Resolve(ctx context.Context, ref string) (name string, desc ocispec.Descriptor, err error) {
	_, d, e := r.RealResolver.Resolve(ctx, ref)
	return r.Name, d, e
}
func (r *overrideNameResolve) Fetcher(ctx context.Context, ref string) (remotes.Fetcher, error) {
	return r.RealResolver.Fetcher(ctx, ref)
}
func (r *overrideNameResolve) Pusher(ctx context.Context, ref string) (remotes.Pusher, error) {
	return r.RealResolver.Pusher(ctx, ref)
}

This can be achieved on the client side. Which is what docker does. My own client does so by making use of the label functionality.
I personally don't think such an association is needed as a native part of the daemon itself.
/cc @stevvooe @dmcgowan

Ya, the client side is what I'm taking about.
The client inserts the image manifest with the name. It is just that the client libraries don't support  changing the name that is actually imported.
It turns out, push already supports this. I can push any image to a different url.
Pull doesn't support this

In general, I would recommend against this. Storing the short name was a huge mistake in docker and the interfaces and behavior in containerd depart from this to avoid those mistakes. Having a namespace on the image is a huge advantage and ensures that you can have images from multiple locations without name collisions.
If you want reproduce the experience of docker, I would just recommend dropping the namespace in the ui. Anything else is going to cause you problems down the line.

It is too verbose for my users (and even docker users) to constantly type docker.io everywhere.
To drop the domain in the UI would lead to confusion if they actually typed in the domain, or another domain.
It is my understanding that the intention is to move the docker image layer storage to containerd. What will happen then?
I see can see the the "library/ubuntu" vs "ubuntu" normalization causing weird behaviors, but I'm not seeing why the domain would cause issues. Containerd doesn't have networking support in it, aside from the client.

@pauldotknopf like I said, nothing prevent all this to be handled in the client and my guess is that Docker will do just so.
For instance in my pet project, I automatically try a few prefixes if I don't get a full ref on input, it works for my case and doesn't require much code. I'm still for letting this handling up to the client.

@pauldotknopf just be clear, when referring to the "client", I mean a program interacting with containerd via the grpc interface. So docker qualifies, ctr doesn't since it's just a development tool.
The program interacting with containerd has the leeway to do all that it wants before sending a request to the daemon. If it want to magically translate names, it can do so. I haven't found a need to have changes to the "client API" (i.e. the containerd package) to achieve this result.
The docker resolver provided by the containerd client API uses the canonical name as expected by the registry. My own client does all the needed translation before calling the Pull command and that works as expected. If you rather have this in the resolver,  Nothing prevents you from writing your own resolver if you so wish, you just need to implement the interface.

Will docker use a custom resolver when it migrates to storing images in containerd?

@pauldotknopf I don't think so. I assume it'd do as I do currently and translate the path before calling Pull(), but it'd depend on who ends up implementing it :)

What do you mean by "before calling Pull()". The Pull() method expects the fully qualified name.
See here.
In this code pullRef has to be the fully qualified name. Where are you talking about about doing the translation?

@pauldotknopf The docker code will resolve the full path according to its database before calling in Pull()

What about when you later execute docker images? What will it return? The image the user entered, or the fully qualified image? If you show the image the user entered, will you be keeping a meta-database that maps images the user entered to their fully qualified name?

@pauldotknopf docker will keep working exactly as it works today. If you have more question about how docker will handle this, please redirect those to github.com/moby/moby.
I'm going to close this issue now, as I think we clarified the expectations by now.


Host API remove restart

Metal trace will undergo a bit of spec redesign and code refactor.
We've determined that restart is not needed and we should remove it

https://github.com/tenstorrent-metal/tt-metal/pull/5501 merged to main


Is this still being developed?

@benryan2010 Are you developing this at all?
I'd be very interested in collaborating with you.

@TimothyMalahy this is an active project, and would love to collaborate!


CR22: Search: Add Food groups text box to search area

Tools: JQuery

Text area for food groups added to the search options


Use url pattern for [nexus] + minor cleanup

Ref #3329

Messages

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

  Generated by :no_entry_sign: dangerJS against ca3217bd7a734b8a2d4875e7e88ec776d9b815a0

The changes in here look good and I'm ready to approve. However, while playing with the examples I noticed we're getting invalid response data on many of them.
I don't think anyone has raised an issue yet on this specific item, but definitely needs a review. I'd suggest doing so later on/outside this PR.
Thoughts?

https://developers.redhat.com/auth/realms/rhd/protocol/openid-connect/auth?scope=openid&response_type=code&redirect_uri=https%3A%2F%2Frepository.jboss.org%2Fnexus%2Fsec&state=6a99060e-0c6e-480c-b968-b938ae92aba4&login=true&client_id=nexus
Looks like RH is requiring auth now for https://repository.jboss.org/nexus 🤔

Is that why the tests are failing too?

Is that why the tests are failing too?

Yes I believe so. We could swap the test targets to the OSS Sonatype instance (although that's where I was having trouble with the example).
The failing service tests are all now showing the below inaccessible error:
AssertionError: message mismatch: expected 'inaccessible' to equal 'artifact or version not found'

I found the issue with the schema that was rejecting our examples (like https://oss.sonatype.org/service/local/lucene/search?g=com.google.guava&a=guava).
TL;DR that query will include every published version for that artifact, and the format for one of the fields can be in a different format (like HEAD-android-SNAPSHOT)
I've updated the schema accordingly, and I'm going to update the examples and tests to use the the OSS Sonatype instance (as the RH instance would require auth and a self-hosted Shields server)

I've approved but I am going to defer to another maintainer to merge since I've also added some commits to this PR


fix documentation in hierarchy.rst

Some of the import statements didn't work when following the examples in the documentation, so I have corrected the documentation here. Also example in the documentation didn't run because the method name is HierarchyTree.from_nodes(), so I have also made this correction.

Thanks for the fixes! Merged


Control sensu version

Hi there,
Thanks for the awesome module!
Here are two minor changes:

ability to control sensu version
Ensure Sensu SSL directory exists even if ssl_generate is not used

Thanks! Apologies for the delay.


doc: code.html page assumes GOPATH is not a list

What version of Go are you using (go version)?

$ go version
go version go1.4.3 linux/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
GOARCH="amd64"
GOBIN="/home/matthew/go/bin"
GOCHAR="6"
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/matthew/go:/usr/share/go1.4/contrib"
GORACE=""
GOROOT="/usr/lib64/go1.4"
GOTOOLDIR="/usr/lib64/go1.4/pkg/tool/linux_amd64"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0"
CXX="g++"
CGO_ENABLED="1"

cat /etc/issue
Welcome to openSUSE Tumbleweed 20181103
What did you do?
Use the documentation page https://golang.org/doc/code.html to try and set up and use go
to run initial examples
The documentation assumes that the go env GOPATH returns a single path not a list of paths.
go env GOPATH
/home/matthew/go:/usr/share/go1.4/contrib
This means that passages like this do not work.

To compile and run a simple program, first choose a package path (we'll use github.com/user/hello) > and create a corresponding package directory inside your workspace:
$ mkdir $GOPATH/src/github.com/user/hello

An obvious workaround in my case is to use sed but it would make the documentation page less readable.
$ mkdir $(echo "$GOPATH" | sed 's/:.*//g')src/github.com/user/hello
What did you expect to see?
What did you see instead?

The documentation explicitly mentions how to set a GOPATH here - https://golang.org/doc/code.html#GOPATH and goes forward with that assumption.
Technically, the documentation is correct in that regard.
Maybe we can add another line saying that GOPATH can have multiple values. But given the fact that we are sunsetting GOPATH, I wouldn't worry about adding further corner cases to it.

(People who like this bug also like #18119)

This is originally (some later modifications added) of what I wanted a Wiki page to look like.
https://nurmi-labs.blogspot.com/p/filetreedoc.html

R.C. once wrote about something like $GOROOT/HACKING.md (to match src/runtime/HACKING.md and maybe others), maybe such (circa 2017) consideration should be revisited, as that content could go in such an .md file.

/cc @bcmills for decision.

At this point I don't think it's worth the effort to update all of our tutorial content work with multiple-element GOPATHs: the use-cases for them are specific to GOPATH mode and will hopefully go away soon anyway (#30228, #4719).


[windows] double newline in codeblocks

This looks like a problem with pypandoc or pandoc on windows.
pandoc on windows generates lineendings as \r\n (normal windows line endings)
but when the file is saved as html, this produces two newlines so I get codeblocks like this:

I didn't catch this as an issue first because I thought, it was just a css-issue. Also, I haven't seen any differences in the output but in the codeblocks.
my first workaround is to just replace \r\n with \n in frames.py (function convert)

That's funny. I did a checkout of the code for the first time on a windows machine today, and notice this exact problem!
Could you confirm that it works?

Sorry, it doesn't.
 Writing it to the file applies the windows convention again...

@baudren
Yes, now it works


[Algo] Tower of Hanoi in C++

Algorithm

[ ] I have gone through the repository and made sure this algorithm has not been implemented

Information about my new algorithm

Algorithm Name: Tower of Hanoi

Programming Language: C++

Time complexity: O(2^n), n = Number of disks.

Is this your first contribution to this repository?
Yes

i am intrested

I would to like to contribute code of Tower of Hanoi in CPP

Hey @vyom153069 @Vivek0712, this algorithm was requested by @SundeepChand hence I'm assigning this to him.


Logiciel Reflectarray Designer

Good morning, sir,
Could you please put ReflectArrayDesigner_Installer.exe online.

Hi of course,
Please find the installer here:
https://github.com/Chen-XueWen/Reflectarray-Designer-Software/releases/tag/v1.0.0

Hello,
I am currently working on the transmitarray. I have made a code for 1D (along the ox axis) and now I am having problems putting it in 2D (along the xoy axis). Can you help me please?
I thank you in advance for your reactivity.

Cordialement ,
Sincerely,
C уважением
Wilfrid RENKANGA MBATCHI
<EMAIL_ADDRESS>
De : Tan Xue Wen @.>
Envoyé : jeudi 23 mars 2023 20:06
À : Chen-XueWen/Reflectarray-Designer-Software @.>
Cc : WillJr2907 @.>; Author @.>
Objet : Re: [Chen-XueWen/Reflectarray-Designer-Software] Logiciel Reflectarray Designer (Issue #1)
Hi of course,
Please find the installer here:
https://github.com/Chen-XueWen/Reflectarray-Designer-Software/releases/tag/v1.0.0
—
Reply to this email directly, view it on GitHubhttps://github.com/Chen-XueWen/Reflectarray-Designer-Software/issues/1#issuecomment-1481563042, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A54OT6A64W4Y5KKG3C624ALW5R7K7ANCNFSM6AAAAAAV377EZU.
You are receiving this because you authored the thread.Message ID: @.***>


Fix the condition for noobaa reconcile

BZ : https://bugzilla.redhat.com/show_bug.cgi?id=2251741

Verify generated may need to be rerun.

@iamniting if everything looks good, can we get this merged?


fix(providers): update form field labels for better readability

This PR updates the form field labels for the add provider modal. For RHV/oVirt I made the hostname/username/password prefixed with RHV Manager/Engine depending on the BRAND_TYPE env variable. The other provider form labels should be unchanged.

should help close https://github.com/konveyor/forklift-ui/issues/655

@apinnick what about in upstream builds? We're using the oVirt branding on providers instead of RHV when building the UI with Forklift rather than MTV branding. Is "oVirt Manager" still the correct term in that case?

Super helpful, thx @mturley - I'll make adjustments and send an update soon.

@apinnick should we also put different labels on these fields when adding a VMware provider? They are the same, just "Hostname or IP address", "Username" and "Password".

@apinnick what about in upstream builds? We're using the oVirt branding on providers instead of RHV when building the UI with Forklift rather than MTV branding. Is "oVirt Manager" still the correct term in that case?

@mturley The correct upstream term for RHV Manager is oVirt Engine.

@apinnick should we also put different labels on these fields when adding a VMware provider? They are the same, just "Hostname or IP address", "Username" and "Password".

@mturley These labels are not correct for VMware. I'm sure that we updated the provider fields a while ago. All lthose fields are for vCenter. That is very important because we don't want the user to mistake "host" for ESXi host.

@apinnick

These labels are not correct for VMware

Which ones, the ones in my screenshot? I do remember we updated them at some point, but I just wasn't sure if we should give them more specific labels since the equivalent fields for RHV are getting more specific labels.
Those field descriptions are from the documentation, right? Should they also be in the UI under help icons like the name and fingerprint fields?

@mturley

Which ones, the ones in my screenshot? I do remember we updated them at some point, but I just wasn't sure if we should give them more specific labels since the equivalent fields for RHV are getting more specific labels.

It's not so much that they are wrong (I overstated that) but that they needed documentation because users were confused about whether they needed the vCenter or ESXi credentials, etc.
If the fields do not have to be re-used for other providers and if the RHV fields will be specific, I think it makes sense to add "vCenter" to the labels:

vCenter host name or IP address
vCenter admin user name
vCenter admin password

Those field descriptions are from the documentation, right?

Yes, that is a screenshot of the user guide.

Should they also be in the UI under help icons like the name and fingerprint fields?

If you are not planning to update the field labels, then they should be in the tooltips or help icons. But I think it would be easier for the user if you update the field labels with "vCenter ..."

@apinnick I pushed some updates and updated the PR description to reflect the latest changes, let me know what you think when you have time to take another look. Thank you!


HAWQ-158. Remove legacy command line tools and help.

The overall fix is good. +1
Why we have gpssh-exkeys and clsAddMirrors.py in Makefile as they are deprecated?


How are Consensus IDs Created?

Hi! Thanks for making and maintaining this program!
I have a situation where I had to split my single-cell fastqs in half in order to run TRUST4 until I can get access to more computing resources, and I'm trying to wrap my head around merging them back together now. It seems like the consensus IDs are unique to each CDR3 (up to the _XXX number at the end), is this in fact the case? The idea being that I could recreate some of the output files by merging on the consensus ID. I'd appreciate any information about how these IDs are assigned and whether this makes sense. Thanks!

In single-cell setting, the alphabetic part of the consensus ID is the cell id , and the _XXX is the index for the consensus in that cell. For example, the consensus for IGH and IGK chains from the same cell could be cell_0 and cell_1.
I think a more convenient approach might be to split the reads based on the barcode, so you can just concatenate the report in the end.

Thank you for a very quick response!
I see, so then in the _report.tsv, the cid corresponds to the barcode with the most reads of that CDR3?

It seems like the _barcode_airr.tsv will be useful for me here. Can you confirm that the consensus-count column in that output is the number of reads (UMIs?) of that consensus contig in that barcode?

Depends on your running command for TRUST4. If you provide the --UMI option, it will be UMI count, otherwise it will be raw read fragment counts.

This is what I had run, but maybe what I'm understanding now after looking at other issues is that I may have needed ... -1 *R1* -2 *R2* --barcode *R1* --UMI --barcode *R1* --readFormat bc:0:15,um:16:-1 ...
!run-trust4 \
-f ../References/hg38_bcrtcr.fa \
--ref ../References/human_IMGT+C.fa \
-1 ../gdTCR/fastqs/*_R1_*.gz \
-2 ../gdTCR/fastqs/*_R2_*.gz \
--barcode ../gdTCR/fastqs/*_R1_*.gz \
--readFormat bc:0:15,um:16:-1,r1:0:-1 \
--barcodeWhitelist ./737K-august-2016.txt \
-t 4 \
-o ../gdTCR/output/

I appreciate your insight here, it's hard to parse these arguments from the documentation.

It seems read1 is contains barcode and UMI, I'm wondering is your data is actually single-end? If so, the parameter can be
run-trust4 \
-f ../References/hg38_bcrtcr.fa \
--ref ../References/human_IMGT+C.fa \
-u ../gdTCR/fastqs/*_R2_*.gz \
--UMI ../gdTCR/fastqs/*_R1_*.gz\
--barcode ../gdTCR/fastqs/*_R1_*.gz \
--readFormat bc:0:15,um:16:-1,r1:0:-1 \
--barcodeWhitelist ./737K-august-2016.txt \
-t 4 \
-o ../gdTCR/output/

I guess the large memory cost from the previous run could be due to that you used the barcode and UMI sequence as read sequence, which could create a lot of divergent paths in assembly and cause the blow-up of memory.

I was wondering if I had done something wrong with the setup! I'll give it a try and close this issue if everything seems ok.
Thank you so much for your help, I really appreciate it.

If the read length from R1 file is 28bp, then your data is single-end for sure. Otherwise, you may need to use --readFormat to specify the actual sequence range in read1.

You were entirely correct, and this was not exactly what I was expecting from the data based on the protocol I was looking at. Thank you very much for your help.
For everyone else's reference this call worked for me (although it did still take a long time and consume a lot of memory, and it's basically the same as what @mourisl indicated above):
run-trust4 \
-f ../References/hg38_bcrtcr.fa \
--ref ../References/human_IMGT+C.fa \
-u ../gdTCR/fastqs/*_R2_*.gz \
--UMI ../gdTCR/fastqs/*_R1_*.gz \
--barcode ../gdTCR/fastqs/*_R1_*.gz \
--readFormat bc:0:15,um:16:-1,r1:0:-1 \
--barcodeWhitelist ./737K-august-2016.txt \
-t 8 \
-o ../gdTCR/output/


liquiding users every time a trade is submitted

https://github.com/nexlabs22/Nex-Dex-Contracts/blob/d07cf743027b1330c22411620f3d1ddf1c741598/contracts/Exchange.sol#L206
Do you agree that liquidating users every time somebody wants to trade is a viable idea?

When a user trade, the price changes. So by each trade we can run the liquidation functions. It does not take significant gas (maybe one or two users get liquidated in that price)

We are letting the liquidations go first at the moment.
However, the liquidation functions are run on the premise of the old/current price, which should not liquidate them,
so the function will do nothing.
For example, let's say we have one long position and one short position.
The longer can liquidate the shorter by buying a huge amount. The transaction would go first since the liquidation would not happen based on the old price.
So I suggest to run the
liquidateUsers();
partialLiquidateUsers();
on the premise of the new price by plugging in: _assetSize and calculating the new price. (might need a new function for that)
Furthermore, I suggest disabling transactions that move the pool away (in the wrong direction) of the oracle price by more than f.e. 10%.
Example price 10, oracle price 13. Diff = (13-10)/10 * 100 = 30%
Suggested new price 11 after transaction: (13-10.5/10.5) = 23.8% (allowed)
Suggested new price 9 after transaction: (13-9/9) = 44% (not allowed)
Suggested new price 20 after transaction: (13-20/20) = -35% (too much, not allowed, needs to stay within (-10%,10%) of oracle.
Example:
max_range = mr = 10
//current diff = cd (example 30% see above)
// current diff distance = cdd =  abs(cd) - mr  (20%)
// diff after suggest trade = sd (23.8%, example see above)
// suggested diff distance = sdd = abs(sd) - 10  (13.8%)
if ( cdd>0):
//we are outside the target range and need to be brought back
if ((cdd - sdd) > 0):
trade allowed, we move closer
else:
//we are inside the target range
if (sdd < 10):
//trade allowed we stay within target range.
Then we can update the oracle price more than every hour (without running the fundingrate function) so the pool will stay aware of the target price and is allowed to trade accordingly.

https://github.com/nexlabs22/Nex-Dex-Contracts/blob/c7b8a4164694afa61cabd8f3e642e94c8fc30a5e/contracts/Exchange.sol#L243-L255
First In the function we check the price will not move alot after that we liquidate users by new price.

Great work!


Typescript version available

Hi,
Thanks for your great contrib.
Just so you know, I created a typescript wrapper for your lib.
https://github.com/anwfr/x18n-ts
Best regards

This is pretty awesome, thanks for that :) I'll add the link to the README!


Security Advisory for Keptn 0.5.x and 0.6.0.beta2 (do not publicly exposed Keptn's Bridge)

Keptn Security Advisory

Date: 2019-01-24
Keptn Versions: <= 0.6.0.beta2 (we have confirmed it for versions 0.6.0.beta2, 0.6.0.beta1, 0.5.2, 0.5.1, 0.5.0; older versions may also be affected)

We have been made aware of a potential security issue with Keptn's Bridge, and we would like to provide the following Security Advisory:
In case you are using Keptn 0.6.0.beta2 or older and you have exposed Keptn's Bridge publicly we advise you to delete that ingress gateway (see Solution below) and access Keptn's Bridge only via port-forward (also, see Solution below).
Am I affected?
Within our tutorials on https://keptn.sh we always recommend to use kubectl port-forward for a port-forward to your local computer (e.g., https://keptn.sh/docs/0.5.0/reference/keptnsbridge/ ). Therefore, if you followed the tutorials on https://keptn.sh you should not be affected by this.
However, we have been made aware that there are tutorials and demos that recommend exposing Keptn's Bridge publicly (e.g., via the istio ingress gateway) for easier use.
As we take security of our end-users serios we wanted to inform our community about this issue and provide you with the necessary steps to fix this issue.
You can check if you are affected by using kubectl get ingress --all-namespaces as well as kubectl get virtualservice --all-namespaces and looking for bridge in the output.
Threat:
We are aware that certain events (e.g., configuration-change events) contain sensitive data that could allow an attacker to read logs of Keptn Services (e.g., helm-service, jmeter-service). If you believe that there might be any information in your logs that are sensitive (e.g., access tokens to services), you should recycle those tokens.
Solution:

Please expose Keptn's Bridge as stated in our docs via kubectl port-forward svc/bridge -n keptn 9000:8080.
To remove the ingress gateway find out if you are affected using kubectl get ingress --all-namespaces as well as kubectl get virtualservice --all-namespaces, and delete those that are related to Keptn's Bridge (e.g., bridge-ingress) using kubectl delete ingress NAME -n keptn or kubectl delete virtualservice NAME -n keptn respectively.

Last but not least: We have fixed the underlying problem of exposing this token via the API of Keptn's Bridge with Keptn 0.6.0.

@jetzlstorfer This is the issue I talked about - it can be added to our security docs

I'm closing this advisory, as Keptn 0.7.x is currently the version we support.


Breaking change in 1.0.1 - causing the SQL CE provider to break

Steps to reproduce
Get the latest build of 1.0.1-rtm from myget feed and use in EFCore.SqlCe provider
Build, expecting the build to pass (no breaking changes)
The issue
The build breaks, as the FindProperty method on MigrationsSqlBuilder is no longer available, but replaced with FindProperties
https://github.com/aspnet/EntityFramework/commit/82fdf18aeed4d061cb70535a4aab9310d5520ada
Further technical details
EF Core version: 1.0.1-rtm-201608241155
Operating system: Windows 10
Visual Studio version: VS 2015 Update 3

Thanks @ErikEJ for the heads up on this one.

I double-checked, and this is the only non-Internal API we've changed in 1.0.1.

We ended up finding a few additional binary compatibility breaks in types in internal namespaces. We have a fix for all of them in https://github.com/aspnet/EntityFramework/pull/6430.
@ErikEJ thanks a lot of reporting this.

Closing as fixed. CC @bricelam in case there was something else left to do.


Fix for SSL Verify on MacOS

Ref:

https://github.com/epinio/epinio/issues/1551

This PR fixes the bug that happens during the Verify and its error check with the MacOS architecture.
We actually don't need the type check, so we can probably skip it like this.
We could eventually do a check only for the darwin/ios arch, but it seems overkill to me.
A brief explanation.
The SSL verification in the windows and darwin/ios platforms is done with the system verifier:
https://cs.opensource.google/go/go/+/refs/tags/go1.18.3:src/crypto/x509/verify.go;l=748-761
// Use platform verifiers, where available, if Roots is from SystemCertPool.
if runtime.GOOS == "windows" || runtime.GOOS == "darwin" || runtime.GOOS == "ios" {
	if opts.Roots == nil {
		return c.systemVerify(&opts)
	}
	if opts.Roots != nil && opts.Roots.systemPool {
		platformChains, err := c.systemVerify(&opts)
		// If the platform verifier succeeded, or there are no additional
		// roots, return the platform verifier result. Otherwise, continue
		// with the Go verifier.
		if err == nil || opts.Roots.len() == 0 {
			return platformChains, err
		}
	}
}

In the Windows implementation the UnknownAuthorityError error is checked and returned:
https://cs.opensource.google/go/go/+/refs/tags/go1.18.3:src/crypto/x509/root_windows.go;l=91-107
// checkChainTrustStatus checks the trust status of the certificate chain, translating
// any errors it finds into Go errors in the process.
func checkChainTrustStatus(c *Certificate, chainCtx *syscall.CertChainContext) error {
	if chainCtx.TrustStatus.ErrorStatus != syscall.CERT_TRUST_NO_ERROR {
		status := chainCtx.TrustStatus.ErrorStatus
		switch status {
		case syscall.CERT_TRUST_IS_NOT_TIME_VALID:
			return CertificateInvalidError{c, Expired, ""}
		case syscall.CERT_TRUST_IS_NOT_VALID_FOR_USAGE:
			return CertificateInvalidError{c, IncompatibleUsage, ""}
		// TODO(filippo): surface more error statuses.
		default:
			return UnknownAuthorityError{c, nil, nil}
		}
	}
	return nil
}

but in the Darwin arch it's probably not possible and a generic errorString is returned:
https://cs.opensource.google/go/go/+/refs/tags/go1.18.3:src/crypto/x509/root_darwin.go

Codecov Report

Merging #1561 (80f0ab5) into main (151121b) will decrease coverage by 0.39%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #1561      +/-   ##
==========================================
- Coverage   62.63%   62.23%   -0.40%
==========================================
  Files         167      167
  Lines       12291    12290       -1
==========================================
- Hits         7698     7649      -49
- Misses       3603     3650      +47
- Partials      990      991       +1

Flag
Coverage Δ

acceptance-api
47.99% <0.00%> (-0.65%)
:arrow_down:

acceptance-apps
38.04% <0.00%> (-0.52%)
:arrow_down:

acceptance-cli
57.60% <0.00%> (-0.27%)
:arrow_down:

unittests
8.62% <0.00%> (+<0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

internal/cli/usercmd/login.go
35.62% <0.00%> (-0.40%)
:arrow_down:

internal/cli/settings.go
44.21% <0.00%> (-22.11%)
:arrow_down:

helpers/error.go
0.00% <0.00%> (-16.67%)
:arrow_down:

internal/api/v1/namespace/create.go
68.08% <0.00%> (-12.77%)
:arrow_down:

helpers/termui/ui.go
60.18% <0.00%> (-4.63%)
:arrow_down:

pkg/api/core/v1/client/apps.go
47.57% <0.00%> (-2.43%)
:arrow_down:

internal/cli/chart.go
40.42% <0.00%> (-2.13%)
:arrow_down:

internal/cli/usercmd/chart.go
51.48% <0.00%> (-1.00%)
:arrow_down:

internal/api/v1/application/stage.go
76.13% <0.00%> (-0.49%)
:arrow_down:

pkg/api/core/v1/client/chart.go
35.89% <0.00%> (+17.94%)
: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 151121b...80f0ab5. Read the comment docs.

Does this work in macosx ? I tested 0.90 and linux works... but it did not work in osx.

I think 1.0.0-rc1 is not in brew tap yet

I think 1.0.0-rc1 is not in brew tap yet

https://github.com/epinio/epinio/blob/02d3e28408eec7be805c1f9c10d434e037e43d6e/.goreleaser.yml#L185


Generate documentation for the Fortran API

In GitLab by @ksiero on Jun 1, 2020, 13:13

unassigned @ksiero

The python part is covered by issue #187 , refocusing this one for Fortran only.


Release info for 0.16.0 tag

Where can we find the release info for tag 0.16.0? Running pnpm outdated indicates a major change but I have been unable to locate information about any possible breaking changes.

https://github.com/thumbmarkjs/thumbmarkjs/releases/tag/v0.16.0


Playback controls missing in some cases

In some cases, for some feeds, playback controls are there, but sometimes, they're not. In some cases, they don't appear at all.
Here's how to consistently reproduce the case where they are missing:

visit https://mbrukman.github.io/delayed-replay/ or open index.html file locally
search for france24
click on "C'est  en France"
note the missing playback controls

The controls are present for other feeds, e.g., for other searches such as rfi, etc. Once they disappear via the france24, other searches (that work initially, such as rfi) don't work either; a full page reload is required to get them to appear on the working searches.
Note that even with missing playback controls, audio is playing, it's just not controllable by the user, which is frustrating and confusing.

Looks like this is happening because this is actually a video podcast ("vodcast"?), so the file that's included is an *.mp4 file, which is not an audio file, and our audio controls are checking for this to be specifically an audio file.
The fact that it's still playing means that we can probably treat video files as audio files as well and use the controls to listen to the audio track from the video file anyway.


Table To Polyline - form not allowing selecting  "Line Field" or "Sort Field" from drop down.

Table To Polyline - form not allowing selecting  "Line Field" or "Sort Field" from drop down.  The drop down does function, but the field needed is not shown in the list.  The list does show "Model Variables".
Tested with Pro 1.3 Build 5792.
Tested using the sample text file from CSV folder  TableToPolyline.csv, your type "Group_" into the "Line Field" input box, the model will run.

This is now a known issue that needs to be addressed via redesign.

Fixed in #146


Add support to read and write properties file in Ballerina

Description:
Add support to read and write properties file in Ballerina, like XML and JSON file support in [1][2]

https://ballerina.io/v1-2/learn/by-example/xml-io.html
https://ballerina.io/v1-2/learn/by-example/json-io.html

Is this going to be added to the ballerina/io module? Or is this going to be a separate module of its own?

@pubudu91 We are thinking of adding this to the ballerina/io module as we support JSON and XML

The APIs are as follows.
Read a property
string propertyFilePath = "<PATH_TO_PROPERTIES_FILE>/config.properties";

io:ReadableByteChannel rbc = check io:openReadableFile(propertyFilePath);
io:ReadableCharacterChannel rch = new (rbc, "UTF8");
io:println(rch.readProperty("name"));

Read a property with default value
string propertyFilePath = "<PATH_TO_PROPERTIES_FILE>/config.properties";

io:ReadableByteChannel rbc = check io:openReadableFile(propertyFilePath);
io:ReadableCharacterChannel rch = new (rbc, "UTF8");
string defaultValue = "John";
io:println(rch.readProperty("name", defaultValue));

Read all properties - return as a map
string propertyFilePath = "<PATH_TO_PROPERTIES_FILE>/config.properties";

io:ReadableByteChannel rbc = check io:openReadableFile(propertyFilePath);
io:ReadableCharacterChannel rch = new (rbc, "UTF8");
io:println(rch.readAllProperties());

Write properties
string propertyFilePath = "<PATH_TO_PROPERTIES_FILE>/config.properties";

io:WritableByteChannel wbc = check io:openWritableFile(propertyFilePath);
io:WritableCharacterChannel wch = new (wbc, "UTF8");
map<string> properties = {
    name: "John Smith",
    age: "25",
    occupation: "Banker"
};

var writeResults = wch.writeProperties(properties, "comments");


[Bug] Brush selection - missing most points after progressive re-render

Version
5.5.0
Link to Minimal Reproduction
https://echarts.apache.org/examples/en/editor.html?c=scatter-large&code=GYVwdgxgLglg9mABAcwKZgCIEMpYBQA26ANInMMAM6pQCUiA3gFCKJFSJYBOXiAvIjCoA7ogBiBODgDMAJgCCPLAE9C6RACpEs2gG4WbGmQr9EABn2tgcXmo4xTFxA4A8hsLucBqL_WatWdkQAD1MvAFkcAAsAOi4sMAATOABbPHotAEYLA0CjZX5cgIjomMoYMDxg-gBaItZQrTUkAFJtRAB-cxjMxAAuRBqzHozESKhY-KTU9MQvesQ8ciojAB9V83oAekRsywDuLgBtZZ8AXVNg_dZDk4pz02V9gF8DLhoQLiRD_VeIBEoHESOCwvQEaEwILwAFZUNC9Ex_mBAYhgbhZKYIdhcDC4aRsgi4AAHWAIUz-RCwKBEAYU1hQVDBKB9BaoCBRbhQSgxaxcFI4GJYRKJADCqX5lDwaNBMSIYGQE0QOwxXlRINksvQCqiSu09FVAHJEAAFOAVLkGgzPYgGKBwOAEWBE2nW232ggAIzgwVpRSIwGZiANEHQDK4BptAUQwFQOE-qF9UdY0oAWvaUi6iq9WK68hDEomAjYYKGBgaAG6oLiwCBYAgRopcGDIKKB7JWyMheTBGCUAZHBjPM6d5Td3v9wfDgwerggShRF2d1Pp_tFOkBKDKIkJoMVcqJVCWqO5gLr-lbncGvcwA9HgLZxBTm5gGD80lgAbAOvUTvUJuoPtECONcFjALAUkveQGyTSkLzLShaygMNoKTaUBmlTJOyjRJX3QcoAX7A1ggjINlANJ8k0oZQUi9AgAGUYAALx3aQsICGAGRSOjNxpRgFlYYksAgDjlAGYYABYFhPKMth2AhuDQAYoFnVAszYs9BHAy8ACEUKjTdt3gxDkLY5MQXQ9VTNRXDkXgZFCOI0gDTIiioyomiHQY5iBlYhYONQLieJ3DSBKJISRLEmJJJg6SAlktgFJ3ZSQFU48DDOJhnl0IA
Steps to Reproduce

jump to minimum example
enable selection tool
select some points
zoom or select another selection tool
progressive re-rendering takes place

Current Behavior
After re-render only few points remain selected. From few first frames probably.

Expected Behavior
All points within brushed zones remain selected.
Environment
- OS:Windows 10,11
- Browser: Chrome, Edge
- Framework:

Any additional comments?
No response

This also happens to me even if I only have 10,000 points. After any re-rendering, only a few of the originally selected ones remain.
And  re-rendering begins to occur when you zoom in/zoom out or simply even if you click on any icon of the Toolbox panel.

official example has large:true and works fine. Any reason why you want to disable optimization with large ?

official example has large:true and works fine. Any reason why you want to disable optimization with large ?

Yes. The obvious reason. Brush selection doesn't work with large: true at all.

ok, agree, related to #17079 and #9945

progressive re-rendering takes place

Looks like disabling progressive thru progressiveThreshold fixes the zoom problem - Demo.

So basically there is no option to brushselect on large amount of data and have any perfomance option enabled.

Agree.

Looks like disabling progressive thru progressiveThreshold fixes the zoom problem - Demo.

It is clear that this will solve the issue. Problems initially sounded like problems with progressive rendering. In your example there are 5000 points so it works well. In my application for example we need to display up to 100,000 points (or 150,000 points) and be able to select them and zoom in/zoom out this brush area. Performance for 100 000 points with progressive rendering is satisfactory, but the fact that the selected dots disappear during Zoom - this is a blocker for us.
In my opinion this is a progressive rendering error with brush area. Because if you move the selection area even just a little, all the points are immediately selected correctly.  Also if I click on "Keep selections" and select additionally some empty area, it will also correct the currently selected brush area and display all the points correctly.
So these actions trigger some internal events of ECharts that force it to render the selected area correctly. And zoom does not trigger these processes during rendering (progressive rendering).
If we can’t fix it, then at least find what event needs to be dispatched after Zoom In/Out to force ECharts to render correctly all points in the selected area and get the correct list of selected points  in "brushSelected" event.

...if you move the selection area even just a little, all the points are immediately selected correctly.

@DespotMagic, frankly I cannot reproduce your findings.
What would be helpful is to update the Demo (more data, add dataZoom?) and explain the steps on how to reproduce the problem.

@DespotMagic, frankly I cannot reproduce your findings. What would be helpful is to update the Demo (more data, add dataZoom?) and explain the steps on how to reproduce the problem.

@helgasoft thank you for trying to help) This can be easily reproduced using the example from ilius33 https://github.com/apache/echarts/issues/19857#issue-2256195799.  We are taking the same steps that ilius33 indicated. And then we just move (drag) the area with the mouse.
Steps:

jump to minimum example
enable selection tool
select some points
zoom or select another selection tool
:exclamation: AR: only few points remain selected (current problem described in this bug)
move (drag) the selected area with the mouse
AR: All points within brushed zones remain selected :ok_hand:.

Example for 100,000 points on video
There you can see the problem: how the points are not rendered after Zoom. and how everything rendered corrected (all selected points) if I drag the selection area a little.

Thank you for the detailed explanation. Just step 4 might be better defined as "zoom with mouse scroll wheel".
Indeed the selection area drag is able to refresh its content successfully. So I think the zoom bug may not be that difficult to fix ...if a brave & knowledgeable volunteer has time to dive in.

On reflection and looking from a different perspective - zoom and brush seem to be completely unrelated actions in ECharts.
As seen in the example, selected area size does not change on zoom. But it should if zoom and brush-select are integrated!
My guess is that brush-select is pixel(canvas) based, and zoom is based on chart coordinates, so they are incompatible.
Brush-select is an isolated, singular event and its result does not persist in further chart manipulations like pan and zoom.

@DespotMagic @helgasoft Hello, Is There any plan or stopgap to address the issue? I also face that issues


Update nlu.yaml

Removed "value": "mexican" from Mexican cuisine Entity the default value given did not match the cuisine name in the knowledge_base_data.json and was causing issues in the trained model.
The trained model would recognize the Entity Mexican but use the provided value that was mexican and then the action action_query_knowledge_base would utter_ask_rephrase because of the lowercase value and the uppercase value in the knowledge_base_data.json file.
Proposed changes:

Remove the given value for Mexican in the Training Data

Status (please check what you already did):

[x] added some tests for the functionality
[x] updated the documentation
[ ] updated the changelog (please check changelog for instructions)
[ ] reformat files using black (please check Readme for instructions)

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

Thanks for submitting a pull request 🚀 @erohmensing will take a look at it as soon as possible ✨

Any idea why the bot Could not update branch? Do I need to do something?

Looks like there is a merge conflict, can you try merging master into your branch locally and resolving the conflicts?

@abdullahnaseem you can resolve it by hand in bottom of this page

Its says 'these conflicts are too complex to resolve in the web editor'

Yes, you'll have to check out your branch, run a git merge origin master, and resolve the conflicts.


cmd/cue: cue def is very slow on recursive jsonschema file

What version of CUE are you using (cue version)?

$ cue version
v0.9.2

Does this issue reproduce with the latest stable release?
Yes
What did you do?
exec timeout 5s cue def schema.json

-- schema.json --
{
  "$schema": "https://json-schema.org/draft/2019-09/schema",
  "$id"<EMAIL_ADDRESS>  "$comment": "https://json-e.js.org",
  "$defs": {
    "jsone-value": {
      "oneOf": [
        {
          "$ref": "#"
        },
        {
          "type": "array",
          "items": {
            "$ref": "#"
          }
        },
        {
          "type": ["null", "boolean", "number", "string", "integer"]
        }
      ]
    },
    "jsone-array": {
      "type": "array",
      "items": {
        "$ref": "#/$defs/jsone-value"
      }
    },
    "jsone-object-array": {
      "type": "array",
      "items": {
        "$ref": "#"
      }
    }
  },
  "additionalProperties": {
    "$ref": "#/$defs/jsone-value"
  },
  "properties": {
      "$if": {
      "type": "string"
    },
    "$json": {
      "$ref": "#/$defs/jsone-value"
    },
    "$let": {
      "type": "object",
      "additionalProperties": {
        "additionalProperties": {
          "$ref": "#"
        }
      }
    },
    "$map": {
      "$ref": "#/$defs/jsone-array"
    },
    "$match": {
      "$ref": "#"
    },
    "$merge": {
      "$ref": "#/$defs/jsone-object-array"
    },
    "$mergeDeep": {
      "$ref": "#/$defs/jsone-object-array"
    },
    "$reverse": {
      "$ref": "#"
    },
    "$then": {
      "$ref": "#/$defs/jsone-value"
    }
  },
  "title": "JSON-e templates",
  "type": "object"
}

What did you expect to see?
A passing test. Conversion of JSON Schema to CUE should not take more than 5s.
What did you see instead?
> exec timeout 5s cue def schema.json
[exit status 124]
FAIL: /tmp/testscript1916985807/x.txtar/script.txtar:1: unexpected command failure

On my machine, when I let it run to completion, I get the following timings:
82.62user 0.43system 0:46.49elapsed 178%CPU (0avgtext+0avgdata 76356maxresident)k

In the original schema, I thought it had gone into an infinite loop: I don't know how
long it would take to run to completion.

With eval v3 this cue def completes for me in < 2s and gives identical output as v2.

I think this means we can resolve this issue as fixed by evalv3; under 2s is still not particularly fast, but it finishes in reasonable time, and there's still plenty of optimization to be had in evalv3. @rogpeppe do you disagree?


There's no application executable file

Prerequisites

[ yeah ] Did you read FAQ section in readme.md?
[ yeah ] Did you test with the latest releases or commit ?
[ yeah ] Did you search for existing issues in Issues?

Description
It seems that there is no application executable downloading the latest master version, so I can't really run the code using windows terminal. Is there a solution to that besides trying to run it using python?

This is the newest release version with executable.
pixivutil202305.zip
You can find it in the release page, and find assets at the bottom, you'll find the executable zip file.

is there any way to get this to work on a pc with word?

is there any way to get this to work on a pc with word?

Like... Microsoft Word? I'm not sure what you mean here...

when I try to download and open the program it states that it doesn't work with windows 11

@Mac-Gyver14 Did you try downloading this?

This is the newest release version with executable. pixivutil202305.zip

You can find it in the release page, and find assets at the bottom, you'll find the executable zip file. https://github.com/Nandaka/PixivUtil2/releases/releases/latest

when I try to download and open the program it states that it doesn't work with windows 11

Do you have Python installed?

also do you have another PC to test, it shouldn't require Python.
After extracting and running PixivUtil2.exe it will load console window and it will ask for username/password, but won't work due to login changes in pixiv and you need add the cookie values (see readme.txt)

I see thanks for all the feed back everyone.

got it to work, thanks everyone. y'll have ben a great help.

okay now I'm getting this message every time I try to get it to work: (************Press enter to exit) I'm not sure why it's not working if y'll could give me some feed back that would be great.

also do you have another PC to test, it shouldn't require Python. After extracting and running PixivUtil2.exe it will load console window and it will ask for username/password, but won't work due to login changes in pixiv and you need add the cookie values (see readme.txt)

Did you use cookie login as said above?
Q3. I cannot login to Pixiv!
    - Check your password.
    - Try to login to the Pixiv Website.
    - Try to use the config.ini on the [Authentication] section.
    - Check your date and time setting (e.g.: https://www.timeanddate.com/)
    - Disable Daylight Saving Time and try again.
    - Copy your session values from browser:
      1. Open Firefox.
      2. Go to Pixiv website and login, remember to enable [Remember Me]
          check box.
      3. Press F12 to open Developer Tools, and select the Storage tab.
      4. Click the Cookies and select for the pixiv.net.
      5. Look for Cookie named = PHPSESSID.
      6. Copy the content value. https://imgur.com/a/BppHOoQ
      7. Open config.ini, go to [Authentication] section, paste the value
         to cookie. https://imgur.com/VB2g3qn

ok so I did that and I came back with this long error code: (Any feed back on what I did wrong would be great)
Trying to log in with saved cookie
Error at doLogin(): (<class 'mechanize._response.get_seek_wrapper_class..httperror_seek_wrapper'>, <httperror_seek_wrapper (urllib.error.HTTPError instance) at 0x20f89256170 whose wrapped object = <HTTPError 403: 'Forbidden'>>, <traceback object at 0x0000020F8740FCC0>)
Traceback (most recent call last):
File "PixivUtil2.py", line 1524, in doLogin
File "PixivBrowserFactory.py", line 347, in loginUsingCookie
File "PixivBrowserFactory.py", line 207, in open_with_retry
File "mechanize_mechanize.py", line 257, in open
File "mechanize_mechanize.py", line 313, in _mech_open
mechanize._response.get_seek_wrapper_class..httperror_seek_wrapper: HTTP Error 403: Forbidden
Cannot Login!

Go to config.ini, open with notepad. At the top change UserAgent to Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:109.0) Gecko/20100101 Firefox/115.0 If it doesn't work try Mozilla/5.0

thank you so much PatrickL546 with your help I finely got it to work. thanks so much and God Bless. 🙏🙏🙏


Externalize quickfix function returning text

Useful when one wants to use this function for something else and to know if a QuickFix window is actually a Location list with qf as &filetype.
References:

https://stackoverflow.com/questions/18522086/what-is-the-best-way-to-distinguish-the-current-buffer-is-location-list-or-quick
http://vim.1045645.n5.nabble.com/detect-QuickFix-window-list-or-LocationList-td4952180.html

I am not opposed to it, but why do you need it? Current Vim provides the getwininfo() function, that allows an easy distinction between those two.

Well just like in your implementation for s:get_text (now called airline#extensions#quickfix#get_type), If I would write a function like that by myself, I would have to parse the dictionary returned by getwininfo(win_getid()) and I think it can be very useful for airline users to have a function like that ready to be used.
Personally, I use it now with this in my ftplugin/qf.vim:
function! s:jump_to_current_line()
	let l:qf_type = airline#extensions#quickfix#get_type()
	if l:qf_type == g:airline#extensions#quickfix#location_text
		execute('.ll')
	elseif l:qf_type == g:airline#extensions#quickfix#quickfix_text
		execute('.cc')
	endif
endfunction

nnoremap <buffer> <CR> :call <SID>jump_to_current_line()<CR>

okay thanks.

Thank you too!


Problem install

Hi
I cannot find workspace.xml in Phpstorm for install this plugin
I need to this plugin
Do you can help me?

I am sorry, the project built is not friendly, it is recommended that you build with gradle. This plugin is no longer maintained.


Add remark field to metadata

A random remark field is probably a good idea. It can lead to additions to the schema. Also I think we can allow for additional RDF if people want that.

Have a look at the example yaml here. Almost all sections do have a "additional_**_information_field" which is meant as a free text field to comment (as defined in SIO http://semanticscience.org/resource/SIO_001167). Does this answer your question?

Does this answer your question?

Partly. One free text field for the whole form would be great. I am also thinking of allowing the injection of RDF, however. Maybe this is evil, but it would allow for a lot of freedom in adding metadata. Question is how you keep track of it ;)

Maybe I accept an additional comment file for the whole thing. However, I really don't know what additional metadata you want, especially if we include https://github.com/arvados/bh20-seq-resource/issues/77? You'd be lucky if you have all this metadata to go along with a submission, don't you think so? I'd say, let's try to keep up with what we have planned so far ... for now we have enough to address.

If someone wants to add a field for eye color they could do that immediately. We can follow with pyshex after if we think it is worth the trouble. It is that type of flexibility for end users. But then it may be better we just handle it ourselves (to avoid SPAM, for one).

Can be done when someone has a use case.


Bump to Gradle 7.6

release notes
migration guide

Can I handle this issue?

@EmmanuelDav Hi! That would be great!
Please also check that everything works as expected via tests from PR #63

This has been finished in #92.


Data connector KQL validations

Required items, please complete
Change(s):

Added validations for Data connector KQL validations

Reason for Change(s):

Added validations for Data connector KQL validations

Version Updated:

N/A

Testing Completed:

Yes

Checked that the validations are passing and have addressed any issues that are present:

Yes

Hi @mkchiliveri, please have a look on comments from @daxesht, also check for branch conflicts. Thanks

Hi @mkchiliveri, please address the comments given by @daxesht, thanks!

Hi @mkchiliveri please provide update on this, thanks!

Hi @mkchiliveri please provide updates on this, thanks!

Hi @mkchiliveri please provide update on this, thanks!

Hi @mkchiliveri please provide update on this, thanks!

Hi @mkchiliveri please provide update on this, thanks!

Hi @mkchiliveri we are waiting for your update on this, thanks!

Hi @mkchiliveri we are waiting for your update on this, thanks!

Hi @mkchiliveri we are waiting for your update on this, thanks!

Hi @mkchiliveri we are waiting for your update on this, thanks!

Hi @mkchiliveri please provide update on this, thanks!


Split-Path works with uri, but Join-Path does not

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
If Join-Path fails on uris and Windows-style absolute paths when on Linux (I suspect it also fails with Unix-style absolute paths on Windows, but don't have a machine to confirm). This is different to Resolve-Path failing on non-existant files (see https://github.com/PowerShell/PowerShell/issues/2993), as those work just fine. It is also inconsistent with Split-Path, which does not care about anything.
I discovered it when trying to use Join-Path to form uri for Invoke-RestMethod, as ${Parent}/${Leaf} would sometimes fail due to trailing /.
It is best demonstrated with the following test function:
Test-RejoinPath($Path) {
    Split-Path -Parent $Path | Tee-Object -Variable Parent
    Split-Path -Leaf $Path | Tee-Object -Variable Leaf
    Join-Path $Parent $Leaf
}

Expected behavior
> test-RejoinPath '/home/me/file.txt'
/home/me
file.txt
/home/me/file.txt
dkaszews@Cuddles-Berry:~/code/powershell/DeLimit/pwsh [feature-culture]

> Test-RejoinPath 'C:/Users/me/file.txt'  # On Linux
C:/Users/me
file.txt
C:/Users/me/file.txt

> Test-RejoinPath 'example.com/file.txt'
example.com
file.txt
example.com/file.txt

> Test-RejoinPath 'https://example.com/file.txt'
https://example.com
file.txt
https://example.com/file.txt

Actual behavior
> test-RejoinPath '/home/me/file.txt'
/home/me
file.txt
/home/me/file.txt
dkaszews@Cuddles-Berry:~/code/powershell/DeLimit/pwsh [feature-culture]

> Test-RejoinPath 'C:/Users/me/file.txt'  # On Linux
C:/Users/me
file.txt
Join-Path:
Line |
   4 |      Join-Path $Parent $Leaf
     |      ~~~~~~~~~~~~~~~~~~~~~~~
     | Cannot find drive. A drive with the name 'C' does not exist.

> Test-RejoinPath 'example.com/file.txt'
example.com
file.txt
example.com/file.txt

> Test-RejoinPath 'https://example.com/file.txt'
https://example.com
file.txt
Join-Path:
Line |
   4 |      Join-Path $Parent $Leaf
     |      ~~~~~~~~~~~~~~~~~~~~~~~
     | Cannot find drive. A drive with the name 'https' does not exist.

Error details
Exception             :
    Type                 : System.Management.Automation.DriveNotFoundException
    ErrorRecord          :
        Exception             :
            Type    : System.Management.Automation.ParentContainsErrorRecordException
            Message : Cannot find drive. A drive with the name 'https' does not exist.
            HResult : -2146233087
        TargetObject          : https
        CategoryInfo          : ObjectNotFound: (https:String) [], ParentContainsErrorRecordException
        FullyQualifiedErrorId : DriveNotFound
    ItemName             : https
    SessionStateCategory : Drive
    TargetSite           :
        Name          : GetDrive
        DeclaringType : System.Management.Automation.SessionStateInternal, System.Management.Automation,
Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35
        MemberType    : Method
        Module        : System.Management.Automation.dll
    Message              : Cannot find drive. A drive with the name 'https' does not exist.
    Source               : System.Management.Automation
    HResult              : -2146233087
    StackTrace           :
   at System.Management.Automation.SessionStateInternal.GetDrive(String name, Boolean automount)
   at System.Management.Automation.SessionStateInternal.GetDrive(String name, Boolean automount)
   at System.Management.Automation.LocationGlobber.GetDriveRootRelativePathFromPSPath(String path,
CmdletProviderContext context, Boolean escapeCurrentLocation, PSDriveInfo& workingDriveForPath, CmdletProvider&
providerInstance)
   at System.Management.Automation.LocationGlobber.GetProviderPath(String path, CmdletProviderContext context,
Boolean isTrusted, ProviderInfo& provider, PSDriveInfo& drive)
   at System.Management.Automation.SessionStateInternal.MakePath(String parent, String child,
CmdletProviderContext context)
   at Microsoft.PowerShell.Commands.JoinPathCommand.ProcessRecord()
TargetObject          : https
CategoryInfo          : ObjectNotFound: (https:String) [Join-Path], DriveNotFoundException
FullyQualifiedErrorId : DriveNotFound,Microsoft.PowerShell.Commands.JoinPathCommand
InvocationInfo        :
    MyCommand        : Join-Path
    ScriptLineNumber : 4
    OffsetInLine     : 5
    HistoryId        : 276
    Line             : Join-Path $Parent $Leaf

    Statement        : Join-Path $Parent $Leaf
    PositionMessage  : At line:4 char:5
                       +     Join-Path $Parent $Leaf
                       +     ~~~~~~~~~~~~~~~~~~~~~~~
    InvocationName   : Join-Path
    CommandOrigin    : Internal
ScriptStackTrace      : at Test-RejoinPath, <No file>: line 4
                        at <ScriptBlock>, <No file>: line 1
PipelineIterationInfo :

Environment data
Name                           Value
----                           -----
PSVersion                      7.4.0-preview.4
PSEdition                      Core
GitCommitId                    7.4.0-preview.4
OS                             Ubuntu 22.04.2 LTS
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
No response

Good find.
It makes sense that Join-Path shouldn't care about whether drives exist, just like Split-Path doesn't.
(It already doesn't care if a path on an existing drive exists or not).

I suspect it also fails with Unix-style absolute paths on Windows

It doesn't, given the interchangeable use of \ and /; that is, something like /foo/bar is equivalent to \foo\bar and is considered a root path relative to the current - by definition existing - drive.

However, note that Join-Path currently normalizes the path separators to the platform-native ones, because it assumes that its arguments are file-system paths.
E.g., on Windows Join-Path /foo bar yields '\foo\bar.
Thus, without additional effort, even making Join-Path accepts _URLs) such as 'file:///some/path' wouldn't yield the expected result. (Join-String allows you to use a -Separator, but it doesn't handle removing duplicate path separators).

@mklement0

However, note that Join-Path currently normalizes the path separators to the platform-native ones, because it assumes that its arguments are file-system paths.

Yeah, you can see in the third example that without https:// it just treats example.com as a current directory, which may in fact absolutely be the case. And I suspect on Windows, it will result in example.com\file.txt.
I think that is an edge case that is not that important, as there is nothing we can do to distinguish between directory which does not exist and website which does not exist (without checking for both, and that is not a good idea). Best I can think of is a flag -Separator (Environment|Unix|Linux|Current) = Environment where Current uses the one already in string, erroring out if mix.
Still, because the uris currently error out, we can just check whether the path has uri protocol. If so, always use /, even on Windows.

The following seems to do a good job in all cases I tested, ignoring stuff like $Path being an array and $AdditionalChildPath  which can easily be implemented with iteration:
function Join-Path2([string] $Path, [string] $ChildPath) {
  $IsUri = [Uri]::IsWellFormedUriString($Path, [System.UriKind]::Absolute)
  $Separator = $IsUri ? '/' : [Io.Path]::DirectorySeparatorChar
  $Path.TrimEnd('/', '\') + $Separator + $ChildPath.TrimStart('/', '\')
}

[Uri]::IsWellFormedUriString($Path, [System.UriKind]::Absolute) only matches absolute paths with explicit Uri protocol, so only cases which error out due to the protocol being mistaken for drive name.
@daxian-dbw Can you please check how Join-Path 'C://Windows' 'System32' is treated on Windows? If this also errors out due to double slash, then we are safe even if somebody has a drive with the same name as some Uri protocol.

This is my workaround, it's a one liner. Just add it to the top of the script and it'll just work as one'd expect. I had issues when I tried to join paths intended for a remote system on drive letters the controlling computer didn't have.
This workaround basically just prefixes the input with "Filesystem::" and stripping it from the output.
"Filesystem::" is the name of the provider. The PSPath syntax allows for it being specified explicitly. This way the Join-Path module won't try to use the Get-PSDrive data to find it.
function Join-Path ([String]$Path,[String]$ChildPath,[Switch]$Resolve) {return (Microsoft.PowerShell.Management\Join-Path -Path "Filesystem::$Path" -ChildPath $ChildPath -Resolve:$Resolve) -replace "^Filesystem::", ""}
Another workaround would be to use New-PSDrive to first register a dummy drive using the correct provider. Only the drive has to exist using the correct provider, and as long as "-Resolve" isn't specified it doesn't matter what data it has.

The WG discussed this and believe that this could be achieved with a new ParameterSet (to avoid a breaking change) and new parameter (Uri) which would only support parameters uri and ChildPath . The implementation should call the Uri constructor with both elements (new uri(, ), and then return the newly created Uri.

The WG discussed this and believe that this could be achieved with a new ParameterSet (to avoid a breaking change) and new parameter (Uri) which would only support parameters uri and ChildPath . The implementation should call the Uri constructor with both elements (new uri(, ), and then return the newly created Uri.

@JamesWTruher
I suggest allowing the user to choose / or \. Like:
param (
        [ValidateSet('System', '/', '\')]
        [string]$Separator
    )
# System is [System.IO.Path]::DirectorySeparatorChar

Related issues: #24193


Add App::name set function to use crate_name!() macro

Currently, you can further specify the about, authors, and version field of an App through setter methods of the same, and pull in the appropriate value through crate_about!() and similar macros. Only crate_name!() is special, in that we can only use it when constructing a new App through App::new.
If we construct an app through App::from_yaml, we have no way of setting the name to the one in Cargo.toml through the macro. It would be great to be able to set the name after App was constructed.
Is it possible to add this or is there a reason why this was omitted?

It's not omiited on purpose,probably just overlooked. Until I get a chance to add this* there is a workaround (that is technically public, but not meant to be so may change in the future without warning):
let mut app = // load YAML like normal
app.p.meta.name = "Some Name".into();
let m = app.get_matches();

If someone would like to submit a PR with this, it'd be an easy one just adding the fn to src/app/mod.rs!

I recently broke my finger so am typing with one hand and it's not feasible to get real work done 😞

Sorry about your hand. :( To make it heal faster I submitted #910.

What I find a bit strange here is that I need to use a dummy name in the YAML file, because otherwise clap does some strange stuff:
        let mut a = if let Some(name) = yaml["name"].as_str() {
            App::new(name)
        } else {
            let yaml_hash = yaml.as_hash().unwrap();
            let sc_key = yaml_hash.keys().nth(0).unwrap();
            is_sc = Some(yaml_hash.get(sc_key).unwrap());
            App::new(sc_key.as_str().unwrap())
        };


Update Daum.PotPlayer.installer.yaml

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Hi @drshtmstry,
This manifest adds Apps and Features entries that aren't present in previous PR versions. These entries should be added to the previous versions, or removed from this version.
(Automated response - build 887.)

https://github.com/microsoft/winget-pkgs/pull/168375


Server backup costs off by factor of 100

Describe the bug
I think that the monthly costs (haven't checked the daily ones) for server backups are off by a factor of 100.
To Reproduce
Steps to reproduce the behavior:

Create servers with backups enabled.
Check server prices and server backup prices.

Screenshot

I calculated the right backup costs. They're exactly 23.228 which is what's displayed divided by 100.
Expected behavior
That server backup prices are ~10% of the server prices.
Additional context
I think the error is the following:

Here you get a percentage from the Hetzner API, i.e., 10.
Then you multiply with the server prices without adjusting the percentage, i.e., without dividing by 100.

Hello @hziegeldorf,
Thank you very much for letting me know.
I will take a look next week and am optimistic that the fix is fairly trivial. If you want, you can take a swing at a fix as well if you want to have it fixed before next week 😄.
With kind regards

Hey @hziegeldorf,
Thank you again for the report. I have release a version containing the fixed backup calculation. You can get it as v0.3.1 from DockerHub or as 0.3.1-r2 from the Helm repository.
With kind regards


PoS tx tests

closes #462
depends and based on #927

pls update wasm

I think there's an issue with the bonds in the client caused by the split of the Bond pos_deltas and neg_deltas in bf0b686 - the client RPC only uses the pos_deltas. Ideally, we should hide the implementation details somewhere behind a nicer API in PoS

I think there's an issue with the bonds in the client caused by the split of the Bond pos_deltas and neg_deltas in bf0b686 - the client RPC only uses the pos_deltas. Ideally, we should hide the implementation details somewhere behind a nicer API in PoS

I fixed this in a rebase, for now manually in place

moved to https://github.com/anoma/namada/pull/254


Syntax highligher is slow and does not highlight as much has it could

Working recently on the documentation made me realize 2 things about the syntax highlighter:

It is pretty slow when highlighting big files such as the one for the string class (which is roughly 40Kb of code). Example here. Do not open it unless your browser freezing/crashing isn't a big deal! The same file in GitHub render in a eye blink.  Same thing in VsCode.

Currently there is not much word that are highlighted. Below is a screen comparison of VsCode with the pony extension and the same piece of code in the documentation. This may come down to preference but colors are used much more (ref, while, if , end are just in bold in the documentation) in vscode. Plus VsCode highlight more words (fun, be, and  members of class/structs) and comments are grayed out. All those differences combined in the highlighting make "visually" parsing the source is much in VsCode than in the documentation (at least for me).

I think point 1 should be handled no matter what.
Point 2 may be a matter of preference and as such I would like to know what kind of highlighting you prefer. Personally I would favor an highlighting such as the one in VsCode.

Regenerating the highlight.js  script fixes the long parsing and highlight more words which is now close to what VsCode does. So I think the problem come down to what version of highlight.js is packed when generating the documentation.

At worst, it is possible to override the current highlight.js with a more recent one fixing the problem by putting it in the extra_javascript section.


Update Multiple Record Names

Hi,
I am using your Script on my Unraid Server at Home and it Works great, but I am wondering if its possible to update more than one Record Name?
Maybe its possible to add more Record Names and seperate them with commas, but I dont find any Documentation for that.
The other solution would be to run a second docker container for a second Record Name.

Hi there,
thank you for the idea. Would you like to update multiple zones or multiple records in a zone?
Kind regards,
kutzilla

I would love to see this feature too. Maybe you can seperate multiple subdomains with a comma (like already suggested) so it would look like
-e RECORD_NAME="sub1,sub2,sub3"

Hey,
I think I can add the feature to update multiple zones and records with the same feature. Maybe you look into my suggestion in #8. I think the suggested possibility to update multiple zones could be a good way to update multiple records as well. What do you think?
Kind regards,
kutzilla

Hey,
Yeah sounds like a good solution.

Do you have any Idea, when this Feature will be finished and added to the Docker Image?

Hello there. Sorry for the long silence. I'm still working on this feature. I will keep you updated on that


update doc

doc updates

Also looks like it needs a rebase (to add changes from your last PR)

Also looks like it needs a rebase (to add changes from your last PR)

@Maxusmusti Thanks, I think I have addressed your comments, please review


Update HummingBoard to latest 2.X

update to morty
make sure it has all the requirements for 2.0 (ex initramfs, uboot resin etc).
test the final build

I will be pushing changes to our meta-layer https://github.com/SolidRun/meta-solidrun-arm-imx6 and adding a pull request here to include support for our iMX6 boards.


How to automatically redraw input values after user intervention?

Description
Input element ignores model changes and always displays the value entered by user.
Steps to Reproduce

Use keyboard or mouse to change the values of the two input elements.
Click button to change model.
Notice that these input elements do not redraw model's new values. They keep displaying the values you entered although their value attributes do reflect model's new values.

On the other hand, these input elements redraw model's new values if you never manually enter data for them.
code
import {html,LitElement} from 'lit-element';

class MyElement extends LitElement {
	render() {
		return html`
			<input type="checkbox" ?checked="${this.checkBox}">${this.checkBox}

			<input type="text" value="${this.myString}">${this.myString}

			<button @click="${this.changeModel}">Change Model</button>
		`;
	}
	static get properties(){
		return {
			checkBox:{Boolean}
			,myString:{String}
		};
	}
	constructor(){
		super();
		this.checkBox=false;
		this.myString='A';
		this.counter=1;
	}
	changeModel(){
		this.checkBox=!this.checkBox;
		this.myString=++this.counter;
	}
}

Browsers Affected

[ X] Opera
[ X] Firefox

check out the live directive. Also, I believe that value is a property, try .value=...

check out the live directive. Also, I believe that value is a property, try .value=...

check out the live directive. Also, I believe that value is a property, try .value=...

Binding values with properties in stead of attributes as you suggested appears to have automated the update to input without resorting to live directive.
I need to test more.

check out the live directive. Also, I believe that value is a property, try .value=...

Binding values with properties in stead of attributes as you suggested appears to have automated the update to input without resorting to live directive.
I need to test more.

Below are my findings:

element input

Use notation .value="${this.myValue}" instead of value="${this.myValue}"
Use notation .checked="${this.myValue}" instead of ?checked="${this.myValue}"

element mwc-switch

Use notation .checked="${live(this.myValue)}"
Do not use notations ?checked="${this.myValue}" or .checked="${this.myValue}"

Below are my findings:

element input

Use notation .value="${this.myValue}" instead of value="${this.myValue}"
Use notation .checked="${this.myValue}" instead of ?checked="${this.myValue}"

element mwc-switch

Use notation .checked="${live(this.myValue)}"
Do not use notations ?checked="${this.myValue}" or .checked="${this.myValue}"


Updating readme from branch example

Updates
I updated the readme file. Please review and accept.

Thougts
I probably shouldn't, but okay. :+1:


why there's a parameter "offset"?

https://github.com/jayelm/gisting/blob/acd78b49111db30c3a24c32f625b85ae59934585/src/gist_llama.py#L206
the apply_rotary_pos_emb() function does not accept the offset argument？
def apply_rotary_pos_emb(q, k, cos, sin, position_ids):
# The first two dimensions of cos and sin are always 1, so we can squeeze them.
cos = cos.squeeze(1).squeeze(0)  # [seq_len, dim]
sin = sin.squeeze(1).squeeze(0)  # [seq_len, dim]
cos = cos[position_ids].unsqueeze(1)  # [bs, 1, seq_len, dim]
sin = sin[position_ids].unsqueeze(1)  # [bs, 1, seq_len, dim]
q_embed = (q * cos) + (rotate_half(q) * sin)
k_embed = (k * cos) + (rotate_half(k) * sin)
return q_embed, k_embed

The offset parameter is used because, when caching an instruction, the model needs to know the length of the cached instruction to apply the position embeddings correctly. (It's not needed for T5 due to T5's relative position embedding scheme). You can see how the position embeddings are shifted here:
https://github.com/jayelm/gisting/blob/main/src/gist_llama.py#L114-L125
apply_rotary_pos_emb does not require the gist offset argument because it transforms the cos and sin tensors which already have the offset applied.
Note the offset parameter is not used during standard training or evaluation, because we don't actually modify any sequence lengths—the model gets the entire instruction/input/output in one go, with attention masking used to control compression, and the position embeddings are correctly applied with the original instruction length.
The offset parameter is only used in compress.py—the GistActivations class accepts a "gist offset" argument which records the length of the instruction before caching:
https://github.com/jayelm/gisting/blob/main/src/gist_llama.py#L633-L656

thanks for reply, but my question is: apply_rotary_pos_emb() function doesn't need the off-set para but in the code I mentioned above, in the 206 line of /gisting/src/gist_llama.py file , the function use off_set as input parameter，Doesn't that make an error？

The gist_offset parameter is needed because, when caching an instruction, the model needs to know the length of the cached instruction to apply the position embeddings correctly. (It's not needed for T5 due to T5's relative position embedding scheme). You can see how the position embeddings are shifted here:
https://github.com/jayelm/gisting/blob/acd78b49111db30c3a24c32f625b85ae59934585/src/gist_llama.py#L114-L125
apply_rotary_pos_emb does not require the gist offset argument because it transforms the cos and sin tensors which already have the offset applied.
Note the offset parameter is not used during standard training or evaluation, because we don't actually modify any sequence lengths—the model gets the entire instruction/input/output in one go, with attention masking used to control compression, and the position embeddings are correctly applied with the original instruction length.
The offset parameter is only used in compress.py—the GistActivations class accepts a "gist offset" argument which records the length of the instruction before caching:
https://github.com/jayelm/gisting/blob/acd78b49111db30c3a24c32f625b85ae59934585/src/gist_llama.py#L633-L656
https://github.com/jayelm/gisting/blob/acd78b49111db30c3a24c32f625b85ae59934585/src/gist_caching.py#L112-L113

so when I run the llama model, it occurs "apply_rotary_pos_emb() got an unexpected keyword argument 'offset'"

The gist_offset parameter is needed because, when caching an instruction, the model needs to know the length of the cached instruction to apply the position embeddings correctly. (It's not needed for T5 due to T5's relative position embedding scheme). You can see how the position embeddings are shifted here:
https://github.com/jayelm/gisting/blob/acd78b49111db30c3a24c32f625b85ae59934585/src/gist_llama.py#L114-L125
apply_rotary_pos_emb does not require the gist offset argument because it transforms the cos and sin tensors which already have the offset applied.
Note the offset parameter is not used during standard training or evaluation, because we don't actually modify any sequence lengths—the model gets the entire instruction/input/output in one go, with attention masking used to control compression, and the position embeddings are correctly applied with the original instruction length.
The offset parameter is only used in compress.py—the GistActivations class accepts a "gist offset" argument which records the length of the instruction before caching:
https://github.com/jayelm/gisting/blob/acd78b49111db30c3a24c32f625b85ae59934585/src/gist_llama.py#L633-L656
https://github.com/jayelm/gisting/blob/acd78b49111db30c3a24c32f625b85ae59934585/src/gist_caching.py#L112-L113

the error occurs in my llama training stage is
File "/data/wupf/gisting/src/gist_llama.py", line 206, in forward
query_states, key_states = apply_rotary_pos_emb(
TypeError: apply_rotary_pos_emb() got an unexpected keyword argument 'offset'

Are you using the version of transformers specified in requirements.txt, specifically commit fb366b9a? You'll see that the function signature is different:
https://github.com/huggingface/transformers/blob/fb366b9a/src/transformers/models/llama/modeling_llama.py#L136-L141
(and more generally if you run into other issues it could be because you're not using the package versions specified in requirements.txt)

Are you using the version of transformers specified in requirements.txt, specifically commit fb366b9a? You'll see that the function signature is different:
https://github.com/huggingface/transformers/blob/fb366b9a/src/transformers/models/llama/modeling_llama.py#L136-L141
(and more generally if you run into other issues it could be because you're not using the package versions specified in requirements.txt)

I try to install the transformers version fb366b9a, but I can't install it on my server,  All other packages are installed according to requirement except transformer

Unfortunately the codebase is only verified to work with commit fb366b9a. You might be able to get around this specific issue by just pasting in the apply_rotary_pos_emb function from the link above instead of importing it from modeling_llama, but I can't guarantee you won't run into additional issues.

Unfortunately the codebase is only verified to work with commit fb366b9a. You might be able to get around this specific issue by just pasting in the apply_rotary_pos_emb function from the link above instead of importing it from modeling_llama, but I can't guarantee you won't run into additional issues.

ok I fixed it， thanks！

Great!


Fix a couple issues related to PPS

From commit log:
    Fix a bug where a worker might try to return a datum even if the workerpool has been canceled

    Fix a bug where jobManager doesn't exist after the job has encountered a failure

    Fix a bug where the microsoft object client would return an error if it writes to an object that already exists

@jdoliner, ah, good catch.  That's what retCh is for.  Fixed it.

Is sending the datum back through retCh actually correct there? It seems like the code was sending it back through dataCh because the datum had failed to process for unknown reasons and it wanted to requeue it for processing. Returning it on retCh seems to suggest that it's going to be interpreted as having been processed.

Yeah retCh is what you use when you wanna requeue a datum.  If you wanna return a response, you use respCh.  The names are arguably confusing.


FCHL19 with periodic boundary conditions and OpenMP.

FCHL19 now works with periodic boundary conditions and is parallelized with respect to atoms.
Minor OpenMP addition in kernels/fgradient_kernels.f90.

Fantastic contribution, thank you so much for this!

I don't know why that OMP parallel loop was only running in serial, thanks for fixing that
I am also not sure why the CI is not running properly, I will investigate!
In the meantime could you add your name to the headers of the files you've changed? I'll also add you as co-author on the qmlcode.org website

Thanks again, this is awesome!

Thank you for the reply!

The commented-out draft of OpenMP parallelization included applying a "REDUCTION" clause to arrays whose size scale as natoms and natoms^2, which can drastically increase memory requirements during parallel runs; I suspect that's why it wasn't used. I ended up allocating a separate temporary array with a size scaling as natoms^2 to avoid the problem.
I've updated the file headers.


⚠ Secret Website 3 has degraded performance

In cb57779, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 2935 ms

Resolved: Secret Website 3 performance has improved in 582514a.


cmd_floating: Don't add non-float as sibling to float.

When turning a float to a non-float, get_focused_container might
return another floating view, causing the active view to be inserted
into the floating list on its workspace instead of the normal child list
which it should. (Since it has is_floating as false the resulting
discrepency triggered other bad behaviour eventually leading sway to
crash.)
This patch fixes that by simply checking floating status before making
it a sibling.
This should fix the extra issue reported at #241 .

(I also removed that comment because it doesn't make any sense / looks outdated.)


Sending Get Request With Proxy Mode Enabled Still Causes Token Invalidation Error

I learned recently that it was a new security feature implemented by Roblox, that when a Get Request is sent from another IPv4 Address, or proxy, the cookie/authorization token changes. Thus causing an error to occur which is token invalidation. I don't know if there are any bypasses to this or even if it would work on the premium version. But it seems proxy mode has no use anymore. Which in return also makes snipers almost useless as sending so many requests at the same time on the same IP would cause a rate-limit very very fast. Is there a fix to this?

i just realized it might be an issue on my end, but i dont understand why it would be caused.

your proxy has to be in the same region as your authenticated cookie


Add user documentation

Reference Issues or PRs
Closes https://github.com/nebari-dev/jupyterlab-gallery/issues/23
What does this implement/fix?
Put a x in the boxes that apply

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds a feature)
[ ] Breaking change (fix or feature that would cause existing features not to work as expected)
[x] Documentation Update
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build related changes
[ ] Other (please describe):

Testing

[ ] Did you test the pull request locally?
[x] Did you add new tests?

Documentation

Access-centered content checklist
Text styling

[ ] The content is written with plain language (where relevant).
[ ] If there are headers, they use the proper header tags (with only one level-one header: H1 or # in markdown).
[ ] All links describe where they link to (for example, check the Nebari website).
[ ] This content adheres to the Nebari style guides.

Non-text content

[ ] All content is represented as text (for example, images need alt text, and videos need captions or descriptive transcripts).
[ ] If there are emojis, there are not more than three in a row.
[ ] Don't use flashing GIFs or videos.
[ ] If the content were to be read as plain text, it still makes sense, and no information is missing.

Any other comments?

bot please update snapshots

Note: the failure of  "Check links" CI job is expected and will go away once this PR gets merged (because we are using absolute URLs for the snapshots which are being added in this PR).


Update activerecord-session_store gem

https://github.com/rails/activerecord-session_store/pull/66 added rails 5 compatibility to the activerecord-session_store gem
Without this, we were not able to support UI load balancing which requires session information be stored in the database.
The logging gem also had to be updated as it was causing a dependency issue versioned as it was.
https://bugzilla.redhat.com/show_bug.cgi?id=1300018

Specifically the reason for updating the logging gem was the following error from a clean bundle update
Bundler could not find compatible versions for gem "logging":
  In Gemfile:
    logging (~> 1.8)

    google-api-client (~> 0.8.6) was resolved to 0.8.6, which depends on
      googleauth (~> 0.3) was resolved to 0.4.1, which depends on
        logging (~> 2.0)

    winrm-elevated (~> 0.4.0) was resolved to 0.4.0, which depends on
      winrm-fs (~> 0.4.2) was resolved to 0.4.3, which depends on
        logging (< 3.0, >= 1.6.1)

    winrm-elevated (~> 0.4.0) was resolved to 0.4.0, which depends on
      winrm-fs (~> 0.4.2) was resolved to 0.4.3, which depends on
        logging (< 3.0, >= 1.6.1)
Bundler could not find compatible versions for gem "memoist":
  In Gemfile:
    memoist (~> 0.14.0)

    google-api-client (~> 0.8.6) was resolved to 0.8.6, which depends on
      googleauth (~> 0.3) was resolved to 0.3.0, which depends on
        memoist (~> 0.11.0)
Bundler could not find compatible versions for gem "multi_json":
  In Gemfile:
    activerecord-session_store was resolved to 1.0.0, which depends on
      multi_json (>= 1.11.2, ~> 1.11)

    google-api-client (~> 0.8.6) was resolved to 0.8.6, which depends on
      autoparse (~> 0.3) was resolved to 0.3.3, which depends on
        multi_json (>= 1.0.0)

    google-api-client (~> 0.8.6) was resolved to 0.8.6, which depends on
      googleauth (~> 0.3) was resolved to 0.4.0, which depends on
        multi_json (= 1.11)

    jbuilder (~> 2.3.1) was resolved to 2.3.2, which depends on
      multi_json (~> 1.2)

    jbuilder (~> 2.3.1) was resolved to 2.3.2, which depends on
      multi_json (~> 1.2)

    google-api-client (~> 0.8.6) was resolved to 0.8.6, which depends on
      googleauth (~> 0.3) was resolved to 0.4.0, which depends on
        logging (~> 1.8) was resolved to 1.8.2, which depends on
          multi_json (>= 1.8.4)

    omniauth-google-oauth2 (~> 0.2.6) was resolved to 0.2.10, which depends on
      multi_json (~> 1.3)

    omniauth-google-oauth2 (~> 0.2.6) was resolved to 0.2.10, which depends on
      multi_json (~> 1.3)

    google-api-client (~> 0.8.6) was resolved to 0.8.6, which depends on
      googleauth (~> 0.3) was resolved to 0.4.0, which depends on
        signet (~> 0.6) was resolved to 0.7.3, which depends on
          multi_json (~> 1.10)

    google-api-client (~> 0.8.6) was resolved to 0.8.6, which depends on
      googleauth (~> 0.3) was resolved to 0.4.0, which depends on
        signet (~> 0.6) was resolved to 0.7.3, which depends on
          multi_json (~> 1.10)

Updating logging allowed googleauth to rev to 0.5.1 which removed the strict check on multi_json (= 1.11) and also allowed a memoist version bump to 0.14.0

The ziya gem was dependent on logging (but did not have a strict version check). I allowed it to work with logging 2.0.0 in the PR here
This is not strictly necessary as we supply a logger when initializing Ziya here which means we will never error in the Ziya gem here because we will never attempt to initialize a logger in Ziya here

@gtanzillo @jrafanie please review

@miq-bot add_label bug, darga/yes, core

@miq-bot add_label wip

@miq-bot rm_label wip

👏 🎉 Nice work @carbonin


Use functions from this client in the mock's tests

e.g. wait_for_target_processed.

Not necessary.


Arc not being striked correctly in Canvas mode

Version

Phaser Version: 3.11.0

Operating system: OS/X 10.13.6

Browser:

Description

I get a different result when I draw an arc in Canvas mode compared to WebGL.
This is the WebGL code:
https://codepen.io/weyseal/pen/JaoMzr
And the Canvas code:
https://codepen.io/weyseal/pen/LJEeae
Any ideas why?
I have to use Canvas because I have had better performance on mobile devices using it.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

This is a duplicate of #3886 so I'm going to close this issue. Please track progress on the issue listed - thank you!

Fixed by adding required beginPath(): https://codepen.io/anon/pen/vzOyRb