text:
Speed up tpl

What this PR does / why we need it:
This dramatically improves the performance of the tpl function, resolving the issues described in https://github.com/helm/helm/issues/8002. Our typical charts go from 2-3 minute renders to 1-2 seconds. A particularly pathological one went from 37 minutes to 23 seconds.
It uses a clone of the invoking Go Template instance, rather that re-building everything from scratch, to isolate the effects of parsing the tpl text from the calling context. To allow includes of defines from the tpl text to continue to work, it also re-defines include within the clone so it can close over the clone itself.
Special notes for your reviewer:
The only user-noticeable change should be to performance: actual chart output is unaffected by this change. All tests are unaffected.
If applicable:

[ ] this PR contains documentation
[ ] this PR contains unit tests
[ ] this PR has been tested for backwards compatibility

@greed42 good job.

@yxxhero can this be triaged/merged/looked at?
Seems like a massive benefit for a little change.

@bacongobbler @hickeyma @joejulian @scottrigby  could you have free time to review this feature? this is a good change. Thanks very much.

How does this differ from #8371?

@joejulian main logic is the same. I think we should discuss the PR in the meeting. WDYT?

Sounds good

@joejulian my English is not good. So looking forward to your discussion. Thanks very much.

Actually, no meetings the rest of October.
If one isn't any better than the other, we should just add the older one to the next minor release milestone and close the newer.

@joejulian I'm not familiar with this core logic of tpl. So I can't pick out which is better.

@greed42 @diafour any comment about the difference of these two PRs?

#8371 permutes the existing code less, so is trivially better on that measure. It's also better for DRY-ness.
Mine also has a bug with define inside tpl that then calls tpl and includes that define, so #8371 is also better there.

Though #8371 has an infinite recursion bug with a case that involves .Files.Get which my approach (just not re-using the filename:string render interface) doesn't suffer. I haven't gotten it isolated yet.

.Files.Get is a possible red-herring for #8371; it's an empty string to tpl that's the problem, basically:
{{ tpl "" . }}

Except that PR looks to do .New(name).Parse(tpl) which I would expect to not suffer from that problem... except the sign you haven't done .New(name) is that:
 {{ tpl `{{ "" }` . }}

also recurses and it doesn't.

OK I've worked out that the difference between t.Execute() and t.ExecuteTemplate() is critical to how empty behaves. The latter will let "earlier" definitions leak through, as t.Parse() will always grab the previous value for a given name for the named association... while the "plain" template t.Execute() uses will see the empty text.
I think that means it is impractical to share Execute code between tpl and regular manifest expansion. tpl needs to be a special case, like include is a special case.
I will see if I can get some tests for the various edge cases I've observed with my chart library regression suite, and set them up as a separate PR so that this PR does not have additions for tests that cover existing behaviour.

Based on that analysis, I'll add #8371 to the v3.11.0 milestone and hope that the core maintainers have time to get it reviewed and merged.

I believe you're saying that #11456 is the better solution now? I'm happy to switch which PR's in the milestone. I can't duplicate the issue so I have to count on you folks that see the problem to guide me here.

I believe you're saying that #11456 is the better solution now? I'm happy to switch which PR's in the milestone. I can't duplicate the issue so I have to count on you folks that see the problem to guide me here.

No; #11456 is just tests for existing behaviour. It doesn't fix anything. #11351 (this one) is the fix I would now recommend.
The fault in #8371 is with this manifest or anything containing:
---
kind: Test
data: {{ tpl "" . }}

Or other define-only things like:
---
kind: Test
data: {{ tpl `{{define "foo"}}test{{end}}`  . }}

You get a stack overflow, as the empty template never becomes the template ExecuteTemplate sees, and so:
% bin/helm template test
runtime: goroutine stack exceeds 1000000000-byte limit
runtime: sp=0xc0eb46a380 stack=[0xc0eb46a000, 0xc10b46a000]
fatal error: stack overflow
...

I like the speedup. We'll need to take a very careful look at this change as there is potential to break existing charts.

I like the speedup. We'll need to take a very careful look at this change as there is potential to break existing charts.

I definitely recommend getting https://github.com/helm/helm/pull/11456 merged, as then there will be tests which will show "expected" changes from this PR. And make it much easier to cover cases not already covered in that (test-only) PR.

I don't know how hard it would be with the infrastructure but maybe offer a preview build with this PR in it so people can help to test?

+1 for at least a preview build. It's a mustT

This could drastically reduce our build time as we massively use tpl and umbrella chart to deploy an app with multiples components and micro services.

is this feature still in review? Would like to have this in the next release as it would help up reduce the build time

I tried this change on my project with a huge chart umbrella (about 50 subcharts), it works and saves a lot of time.
We also use helm to generate yaml templates for non-kubernetes environment and this feature works as well.
This is a game changer as we were thinking of leaving helm.

@joejulian @mattfarina need your help. thanks very much.

As mentioned above by original author @greed42, this PR and #11456 goes together.
#11456 add some tests for the tpl function, especially for scenario with define mixed together with tpl.
This PR changes some of those (unspecified) behaviors.
You can see the impact of this PR versus those new test cases here: https://github.com/xeonx/helm/commit/1ea011f3625f860943d627b0993392427afe5633#diff-b16f55bb6289c9df88deaedc99a45db09154ba13ee3a186b7c5636d67f5b345d
So I definitely recommend merging first #11456, then this PR will have to be updated to show the impact on the current behavior
(mainly when there is a define override from within a tpl, of course impacting only inside tpl whereas in current implementation the globally defined define is used).
Also, for those interested I delivered an unofficial beta version of helm that include this change, so feel free to test with your charts and report any unexpected behavior.
https://github.com/xeonx/helm/releases/tag/v3.12.0-tpl-beta.1

Geeking out about this, @greed42 - you're my hero! This took our 100+ umbrella chart render time from 4.5 min down to under 7 seconds.
Can I somehow send you a beer? Haha!

For those interested, as the milestone for this is now moved to 3.13 (September 2023), I will keep updating my fork and create releases allowing to test this change based on official latest helm release.
https://github.com/xeonx/helm/releases/tag/v3.12.0-rc.1-fast-tpl
Feel free to ping me in the helm-dev slack for any question.

FYI, while testing this, I found an issue in the text/template package related to clone
https://github.com/golang/go/issues/60100
Thus until this bug is fixed (or a workaround is implemented in this PR) this PR will introduce a regression.

FYI, while testing this, I found an issue in the text/template package related to clone
golang/go#60100
Thus until this bug is fixed (or a workaround is implemented in this PR) this PR will introduce a regression.

I've added a check to my test backfill PR for this case: https://github.com/helm/helm/pull/11456/commits/f235f0f28564b4391ef8b0b5f06b2d754bc13873

Just to add a voice to the chorus, this dramatically sped up Helm for us. We were using a home-grown function that avoids invoking tpl as much as possible by hacking at common cases (e.g. straightforward variable lookups were done without tpl), but it's impossible to avoid sometimes and also our chart dependencies use it very frequently 😄
For us, this took our chart that contains third-party charts from ~15 seconds per Helm operation to ~1s(!), and
@greed42 Am I right in understanding that with your workaround, nothing is blocking this PR from being reviewed? If there's anything blocking that can be helped with some investigation and work I would be happy to try to help if I can.

@greed42 Am I right in understanding that with your workaround, nothing is blocking this PR from being reviewed? If there's anything blocking that can be helped with some investigation and work I would be happy to try to help if I can.

As far as I can tell, there shouldn't be anything blocking. If we got the test backfill PR (https://github.com/helm/helm/pull/11456) merged, then any possible issues could be raised as new checks which would cause this PR to fail testing.

🥳
@joejulian as the assignee, are you able to take a look at the associated PR and this one? It seems like the author's got them to a great place and there's a reasonable amount of demand here for what is a really significant optimisation.
I have a feeling it'll have a much bigger impact than what's obvious from the reactions to this PR - a lot of people probably just assume Helm is just slow for big charts, unless they've spent a lot of time messing with their chart to discover the issue is tpl invocations. It took me quite a while to get to the bottom of it (luckily we added a lot of tpl in one go that was obvious in a git bisect, but in another world we never figured it out 😄)
[Sorry to @ you, I hope it's not considered rude! Wasn't sure what the workflow is here]

It works for me. I don't have anything that takes more than 2 seconds, anyway, so I can't see the improvement, but there's enough feedback to give me confidence that it works for others.

Thanks. The new tests will fail after merging in main, so I've pushed a merge and the appropriate test changes to this branch.

Here is some numbers from someone what has something big :)
/tmp/helm/bin/helm version
version.BuildInfo{Version:"v3.12+unreleased", GitCommit:"b261a1b1bee93343cf9fe92335d3f1ccf3e24558", GitTreeState:"clean", GoVersion:"go1.20.6"}

time /tmp/helm/bin/helm template . > /dev/null
/tmp/helm/bin/helm template . > /dev/null  0,51s user 0,04s system 159% cpu 0,344 total

helm version
version.BuildInfo{Version:"v3.12.2", GitCommit:"1e210a2c8cc5117d1055bfaa5d40f51bbc2e345e", GitTreeState:"clean", GoVersion:"go1.20.5"}

time helm template . > /dev/null
helm template . > /dev/null  22,30s user 0,45s system 147% cpu 15,452 total

speed up!!!!

@joejulian would you mind to talk the PR in the dev meeting?

What's the current state here? Will this be fixed with 3.13?

As 3.13.0 is released, the fix did not make it into it, right?

We look forward to speeding up!!!

Please speed up, we need those changes. Thank you!

We really need this improvement. Please, merge it!

Need this asap. Merge pls

Hey! This really cool! Speed up it, please.

Looks cool! Speed it up, please.

No news on this, is it still under review ? we're almost 3 months after the first approval.
Hope this gets merged soon

@scottrigby and @mattfarina Can we get a final review from you two as well?
It seems most issues including previous upstream blockers have been dealth with and this PR would give a significant boost to making Helm more usefull for projects with more complex requirements relying on heavy tpl usage.

Any update on getting this merged?

Cool.

authors: DaniilOmelianenko FlamingFlowair HrechykhinVlad Kuzmenko-Pavel Lokker29 Mixxxxerr Ornias1993 StephanDollberg TheBritz ashleydavies davesmits fan-tad g-schaeffer greed42 joejulian mattfarina oleksandrhrekov starwarsfan stavros-k swythan tduverge xeonx yxxhero


text:
Allow customization of API Versions when using Helm Template

Signed-off-by: Bhargav Nookala nooknb@gmail.com

What this PR does / why we need it:
Resolves https://github.com/helm/helm/issues/3377 for helm template.
This PR adds --api-version (or -a) to add custom API versions to Capabilites.APIVersion, allowing a user to correctly render templates that depend on certain API versions.
Special notes for your reviewer:
If applicable:

[x] this PR contains unit tests
[x] this PR contains documentation

hi @bacongobbler - would you be able to take a look at this addition to helm template?

+1

This would be very useful!

Agree, would be nice if this is going to be merged!

Pending on this feature to be available.

+1

I like this @bnookala !
Could we have this feature in helm3 / "dev-v3" branch as well, please?

might also be worth to rebase against master once more now that #6577 has been merged. It includes a fix where coreutils was bumped underneath us, causing CI to fail :) thanks @bnookala!

@bacongobbler oh for sure, can totally do that - do you have a suggestion on where the docs should live? at a quick glance, thinking the "thinking about charts" section might be a good option: https://helm.sh/docs/using_helm/#thinking-about-charts

@bnookala I would actually suggest somewhere here: https://helm.sh/docs/chart_template_guide/#debugging-templates
Info on the template command was never added there and would be useful to have

source: gharchive/pull-request

authors: alexec alexmt bacongobbler basvandenbrink bnookala kramvan1 sbose78 sundag thomastaylor312

repo: helm/helm

license_type: permissive

license_source: github-api


text:
Add explanatory comments to action.List and action.History

What this PR does / why we need it:
This adds comments that explain the purpose of action.List and action.History. While the comments may seem to state the obvious to someone with helm CLI experience, an SDK-first user may find these comments helpful.
I personally misunderstood the purpose of action.List, and due to misunderstanding, ended up filing #9028. So in a way, these comments will "fix" that issue.
Special notes for your reviewer:
Thanks to @bacongobbler for kindly humoring my initial, misguided efforts!
If applicable:

[x] this PR contains documentation
[ ] this PR contains unit tests
[ ] this PR has been tested for backwards compatibility

Rebased.


text:
automated quoting introduced in v0.24.0 causing regressions

this is related to the change introduced by this PR:
https://github.com/helmfile/vals/pull/138
when vals is used with helm-secrets, this is now resulting in values being stored with quotes in k8s files where quotes were not expected. This is then causing the relevant apps accessing those values to break.
An example is when things like pem certificates are put in k8s secrets and referneced via vals in helm chart configs. This change puts quotes around those certificate contents causing the app trying to read the certificate to break.
please atleast provide a flag to disable this automatic quoting.

I have submitted jkroepke/helm-secrets#362 to hopefully address this issue.

see: https://github.com/helmfile/vals/pull/144

Hey @stoned @yxxhero, Its been a while this fix has been merged but I don't see a new release created for helm-secret, can you help me with that?
Thanks

source: gharchive/issue

authors: mdfaizsiddiqui pdeva stoned yxxhero

repo: helmfile/vals


text:
Add Karma acceptance testing

Add Karma acceptance testing
This update adds a Karma acceptance testing solution. A couple of
tests have been setup, but it isn't integrated into the main testing
scripts yet.
Bug fixes 🐛
During Karma testing, it revealed some bugs for PortalWrapper, via
Dropdown/Modal testing. (YAY)
The bug was that the className prop was unexpectedly being added
to <div> elements that WEREN'T the ComposedComponent. Resulting
in custom classNames being added to 2-3 nodes instead of just 1.
These have been addressed and the Jest unit tests have been updated
accordingly.
Resolves: https://github.com/helpscout/blue/issues/60

Changes Unknown when pulling be19c5d626bbb976404ff5bdee2021cc7bc55ea7 on testing/karma into ** on master**.

Changes Unknown when pulling d2579df268acb676e06a0ef9992ffe3c860a284e on testing/karma into ** on master**.

authors: ItsJonQ coveralls

repo: helpscout/blue

license: mit

license_source: bigquery


text:
typo

https://github.com/henji243/ScratchVoteAnalysisTool/blob/474bda80fb052dc52e13f64212cd316718e97283/main.py#L24-L26
api_responce

ではなく
api_response

だと思います。
わざわざ Issue で言うなっていうツッコミはなしで()

あっ本当だ()
今布団の中なんで明日直します()

頑張れ(?)


text:
Added promisified actions

This PR contains the promisified versions of the Dataframe actions you outlined in #28. I chose to reuse the ...Async methods and not having node-java generate the promisified versions for all methods.

If you decide to eventually merge this, it would be great if you could also publish a new module version to NPM. Thanks.

thanks! looks great from a very quick look; i'll take a closer look before merging (which may not be before the weekend)

Originally it was an own PR, but unfortunately I committed it to Master before creating the branch for the promisified actions. Still not a Git pro, I guess... Sorry. Will fix the Dockerfile as well.

Ok, this all looks good, just a couple small things before merging

I'd prefer promised functions to simply end in P rather than Promised (e.g. jsonP)
test comments above

thanks!

I'll not be able to work on this this week unfortunately...

@tobilg just to be sure i take it you no longer plan to work on this? tx

Sorry, I'll not be able to implement the changes you want due to time constraints.

authors: henridf tobilg

repo: henridf/apache-spark-node


text:
Screenshots

A good application, whether graphical or command-line, needs some screenshots.

Close as done.

repo: henrik-leppa/hfl-signage-player

license: MIT


text:
Update index.js

Round x y position to avoid blurry presentation

Thanks @alexschad

authors: alexschad henriquea

repo: henriquea/minigrid


text:
Apparent circular dependency issue involving this library and oauth_provider

For others who have a problem with oauth_provider when trying to use this library's -n option and -v2 or -v3 arguments...
See this comment / issue thread:
https://github.com/jazzband/django-oauth-toolkit/issues/240#issuecomment-328671287

Workaround:
Add the following to your settings.py:
OAUTH2_PROVIDER_ACCESS_TOKEN_MODEL = 'oauth2_provider.AccessToken'
OAUTH2_PROVIDER_APPLICATION_MODEL = 'oauth2_provider.Application'
OAUTH2_PROVIDER_REFRESH_TOKEN_MODEL = 'oauth2_provider.RefreshToken'


text:
ModuleNotFoundError raised with pytest 7.2.0

pytest==7.2.0 no longer depends on py library [1], but pytest-pycodestyle still tries to use it, leading to this error:
$ py.test apple.py
Traceback (most recent call last):
  File "/tmp/myenv/bin/py.test", line 8, in <module>
    sys.exit(console_main())
...
  File "/tmp/myenv/lib/python3.9/site-packages/pytest_pycodestyle.py", line 7, in <module>
    import py.io
ModuleNotFoundError: No module named 'py.io'; 'py' is not a package

A workaround is to install py, but it would be better to list that package in setup.py install_requires.
[1] https://docs.pytest.org/en/stable/changelog.html

Thank you for report!

Thanks for the quick fix.
When are you going to release the next version?


text:
panic: runtime error: invalid memory address or nil pointer dereference

环境：win10 golang1.6  无C编译器
idea：liteide
编译之后运行没有问题，但是点击run之后就直接挂掉了
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xc0000005 code=0x0 addr=0x0 pc=0x6121fc]
goroutine 67 [running]:
panic(0xb6b700, 0x11dae030)
H:/CoderTools/go1.6.windows-386/go/src/runtime/panic.go:464 +0x326
sync/atomic.AddUint64(0x11e64134, 0x1, 0x0, 0x35552068, 0x85109b)
H:/CoderTools/go1.6.windows-386/go/src/sync/atomic/asm_386.s:112 +0xc
github.com/henrylee2cn/pholcus/app/scheduler.(*Matrix).Push(0x11e64120, 0x11efc000)
H:/CoderTools/go1.6.windows-386/src/src/github.com/henrylee2cn/pholcus/app/scheduler/scheduler.go:180 +0x10a
github.com/henrylee2cn/pholcus/app/spider.(*Spider).ReqmatrixPush(0x11ec34a0, 0x11efc000)
H:/CoderTools/go1.6.windows-386/src/src/github.com/henrylee2cn/pholcus/app/spider/spider.go:269 +0x2c
github.com/henrylee2cn/pholcus/app/spider.(*Context).AddQueue(0x11e622a0, 0x11efc000, 0x1d)
H:/CoderTools/go1.6.windows-386/src/src/github.com/henrylee2cn/pholcus/app/spider/context.go:79 +0x32c
github.com/pholcus/spider_lib.glob.func66(0x11e622a0, 0x11e622c0, 0x0, 0x0)
H:/CoderTools/go1.6.windows-386/src/src/github.com/pholcus/spider_lib/taobaosearch.go:54 +0x242
github.com/henrylee2cn/pholcus/app/spider.(*Context).Aid(0x11e622a0, 0x11e622c0, 0x12165f48, 0x1, 0x1, 0x0, 0x0)
H:/CoderTools/go1.6.windows-386/src/src/github.com/henrylee2cn/pholcus/app/spider/context.go:198 +0x168
github.com/pholcus/spider_lib.glob.func65(0x11e622a0)
H:/CoderTools/go1.6.windows-386/src/src/github.com/pholcus/spider_lib/taobaosearch.go:43 +0x186
github.com/henrylee2cn/pholcus/app/spider.(*Spider).Start(0x11ec34a0)
H:/CoderTools/go1.6.windows-386/src/src/github.com/henrylee2cn/pholcus/app/spider/spider.go:197 +0x97
github.com/henrylee2cn/pholcus/app/crawl.(*crawler).Start(0x121bc280)
H:/CoderTools/go1.6.windows-386/src/src/github.com/henrylee2cn/pholcus/app/crawl/crawl.go:59 +0x43
github.com/henrylee2cn/pholcus/app.(*Logic).goRun.func1(0x12151ea0, 0x0, 0x35552048, 0x121bc280)
H:/CoderTools/go1.6.windows-386/src/src/github.com/henrylee2cn/pholcus/app/app.go:596 +0x74
created by github.com/henrylee2cn/pholcus/app.(*Logic).goRun
H:/CoderTools/go1.6.windows-386/src/src/github.com/henrylee2cn/pholcus/app/app.go:599 +0xfe

你在64位电脑上运行会出现这个问题吗？我没有32位的测试环境

64位没有问题

此问题已解决，更新代码吧


text:
🛑 hentaiOS Blog is down

In c6973bc, hentaiOS Blog (https://blog.hentaios.com) was down:

HTTP code: 500
Response time: 14736 ms

Resolved: hentaiOS Blog is back up in 1610ced after 10 minutes.


text:
🛑 hentaiOS Forum is down

In ed8aa05, hentaiOS Forum (https://forum.helluvaos.com) was down:

HTTP code: 0
Response time: 0 ms

Resolved: hentaiOS Forum is back up in 5b8decd.


text:
Update backup deletion controller for volume snapshot locations

Fixes #807

Looks like we can merge

repo: heptio/ark


text:
Kube Context Flag not respected

What steps did you take and what happened:
I am running the velero install command, but it does not appear to be respecting the --kubecontext flag.
What did you expect to happen:
I have three contexts in my local kubeconfig, APP-Dev and APP-Gotham are the two I am interested in. Even with my current context set to APP-Gotham, when I run the command, it still applies to the cluster that my local kubectl has setup as the default context. I expect the --kubecontext flag to be used regardless of what my current default context is.
The output of the following commands will help us better understand what's going on:
The following gist shows that my current default context is set to APP-Gotham. Prior to running velero install there are no pods (nothing exists yet). When I run the command with the --kubecontext set to APP-Dev, it still installs using the APP-Gotham context. You can see that the pod was created in the velero namespace.
https://gist.github.com/djtaylor/f26fafd1be6a42d5084e24fdf7c8dd54
Anything else you would like to add:
This appears to be client side only. If I can provide any other troubleshooting information please let me know.
Environment:
velero version:
Client:
	Version: v1.0.0
	Git commit: 72f5cadc3a865019ab9dc043d4952c9bfd5f2ecb

kubectl client and server version:
Client Version: version.Info{Major:"1", Minor:"12", GitVersion:"v1.12.3", GitCommit:"435f92c719f279a3a67808c80521ea17d5715c66", GitTreeState:"clean", BuildDate:"2018-11-26T12:57:14Z", GoVersion:"go1.10.4", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"12", GitVersion:"v1.12.3", GitCommit:"435f92c719f279a3a67808c80521ea17d5715c66", GitTreeState:"clean", BuildDate:"2018-11-26T12:46:57Z", GoVersion:"go1.10.4", Compiler:"gc", Platform:"linux/amd64"}

We are running on bare-metal using Ubuntu 18.04.2 with Kubernetes version 1.12.3.

xref #1637

@nrb can you look into this some more? I took a quick look and saw that here we're explicitly constructing a config, using empty strings for kubeconfig/kubecontext, rather than using the client.Factory passed into NewCommand(...) which gets those values bound to the flags. It looks like we might want to add a DynamicClient(...) method to the client.Factory type to make this easier, that uses the proper kubeconfig/kubecontext values.

That does indeed look like the right fix. PR incoming shortly.

authors: djtaylor nrb skriss

repo: heptio/velero


text:
AWS region now hard-coded in packer.json

Commit cab02b6 (part of PR #121) hardcodes the "aws_region" variable in packer/packer.json, setting the value to "us-east-1". This causes the instructions in packer/README.md for building AWS images to break for regions other than us-east-1.
It looks like there are (at least) three potential fixes:

Update packer/README.md to instruct users to add -var aws_region=<region> for building AMIs in regions other than us-east-1.
Update packer/packer.json to remove the hard-coded value for the "aws_region" variable.
Update the AWS region variable files (aws-us-east-1.json and aws-us-west-2.json) to set the "aws_region" variable for that specific region.

If you will let me know which approach you prefer for the project, I'm happy to craft a PR implementing that approach. Thanks!

PR #124 shows an example of one of the workarounds listed above (approach 3, updating the AWS region variable files to include the "aws_region" variable).

@detiber This is the core issue that I referenced in #124. That PR doesn't fix this core issue, but instead provides an example of one approach to fix the issue (adding the "aws_region" variable to all variables files to override the hard-coded region in packer.json). LMK if you have questions.

It looks like all the region-specific variables files have had the aws_region variable added and set, which should address the core problem behind this issue. I'll go ahead and close the issue.

repo: heptiolabs/wardroom


text:
i want to display boundary by postal code

hello,i want to display boundary by postal code,which api can i use?I could't found it

Hi, if I understood your request correctly, under the Maps API for JavaScript 2.5 it would require you:

to query the Geocoding API's Reverse Geocode endpoint (https://developer.here.com/rest-apis/documentation/geocoder/topics/resource-reverse-geocode.html) with the appropriate parameters (level & additionaldata=IncludeShapeLevel) and retrieve a postal code's WKT shape
convert the WKT shape into a Polygon shape (https://developer.here.com/documentation/download/maps_js_pub/2.5.4/Maps API for JavaScript Public v2.5.4 Developer's Guide.pdf#page=458&zoom=auto,81.692,142.836)
add the shape to the map container

Here is an example that makes use of a WKT parser to add postal code shapes on the map (select postalCode on the dropdown): http://heremaps.github.io/examples/explorer.html#wkt-parser__region-shape-on-click
Source code is here: https://github.com/heremaps/examples/blob/master/maps_api_for_javascript/demos/wkt-parser/region-shape-on-click.html

thanks very much,I will to do that.

Hi again,
I will then close this issue. Cheers.

authors: Mr-Luo freethan

repo: heremaps/examples


text:
OLPSUP-9660: Implemented opening of copyright links in new tab.

Copyright links are now opening in new tab.

@heremaps-bot ok to test

Codecov Report

Merging #1330 into master will increase coverage by 0.11%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #1330      +/-   ##
==========================================
+ Coverage   54.04%   54.16%   +0.11%
==========================================
  Files         255      266      +11
  Lines       23434    23714     +280
  Branches     5247     5299      +52
==========================================
+ Hits        12666    12845     +179
- Misses      10768    10869     +101

Impacted Files
Coverage Δ

@here/harp-mapview/lib/copyrights/CopyrightInfo.ts
13.33% <0%> (-0.96%)
:arrow_down:

@here/harp-materials/lib/IconMaterial.ts
44.44% <0%> (-55.56%)
:arrow_down:

@here/harp-map-controls/lib/Utils.ts
46.15% <0%> (-23.85%)
:arrow_down:

@here/harp-omv-datasource/lib/OmvTiler.ts
50% <0%> (-16.67%)
:arrow_down:

@here/harp-utils/lib/ObjectUtils.ts
83.33% <0%> (-10.22%)
:arrow_down:

@here/harp-utils/lib/UrlUtils.ts
84.09% <0%> (-7.81%)
:arrow_down:

@here/harp-datasource-protocol/lib/Extruder.ts
8.33% <0%> (-5.96%)
:arrow_down:

@here/harp-lines/lib/TriangulateLines.ts
11.9% <0%> (-2.99%)
:arrow_down:

@here/harp-materials/lib/Utils.ts
81.42% <0%> (-1.91%)
:arrow_down:

@here/harp-utils/lib/Logger/ConsoleChannel.ts
69.23% <0%> (-0.77%)
:arrow_down:

... and 28 more

Continue to review full report at Codecov.

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

authors: MPursche codecov-io keaukraine


text:
MSDKUI-1945: Set the separator as required (1000)

to avoid unnecessary compression.

Codecov Report

Merging #107 into master will increase coverage by 0.86%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #107      +/-   ##
==========================================
+ Coverage   88.46%   89.32%   +0.86%
==========================================
  Files          89       59      -30
  Lines        5504     3532    -1972
==========================================
- Hits         4869     3155    -1714
+ Misses        635      377     -258

Flag
Coverage Δ

#ios
89.32% <ø> (+0.86%)
:arrow_up:

Impacted Files
Coverage Δ

MSDKUI_Demo/IconButton.swift

MSDKUI_Demo/OptionsViewController.swift

MSDKUI_Demo/ManeuversOverviewViewController.swift

MSDKUI_Demo/GuidanceDashboardViewController.swift

MSDKUI_Demo/ExtensionArray.swift

MSDKUI_Demo/RouteViewController.swift

MSDKUI_Demo/ExtensionUITableViewCell.swift

MSDKUI_Demo/ExtensionBundle.swift

MSDKUI_Demo/ExtensionNMAMapMarker.swift

...UI_Demo/GuidanceDashboardTableViewDataSource.swift

... and 20 more

Continue to review full report at Codecov.

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

authors: codecov-io otaviohere

repo: heremaps/msdkui-ios


text:
ESD-480: Fix Chapter Hierarchy for Quickstart Guide

Signed-off-by: ogunreku praise 35612274+jehovahmidara@users.noreply.github.com

Codecov Report

Merging #32 into master will increase coverage by 0.6%.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           master     #32     +/-   ##
========================================
+ Coverage   87.69%   88.3%   +0.6%
========================================
  Files          88      58     -30
  Lines        5639    3684   -1955
========================================
- Hits         4945    3253   -1692
+ Misses        694     431    -263

Flag
Coverage Δ

#ios
88.3% <ø> (+0.6%)
:arrow_up:

Impacted Files
Coverage Δ

MSDKUI_Demo/IconButton.swift

MSDKUI_Demo/OptionsViewController.swift

MSDKUI_Demo/ManeuversOverviewViewController.swift

MSDKUI_Demo/GuidanceDashboardViewController.swift

MSDKUI_Demo/ExtensionArray.swift

MSDKUI_Demo/RouteViewController.swift

MSDKUI_Demo/ExtensionUITableViewCell.swift

MSDKUI_Demo/ExtensionBundle.swift

MSDKUI_Demo/ExtensionNMAMapMarker.swift

...UI_Demo/GuidanceDashboardTableViewDataSource.swift

... and 20 more

Continue to review full report at Codecov.

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

authors: codecov-io jehovahmidara


text:
5G-SA-E2E zmq simulation error: Registration Reject (PLMN not allowed)

Hi all,
I am trying to play around with the 5G-SA E2E deployment using zmq driver, everything runs fine but the moment UE initiates "Registration request"  it receives a response of "Registration Reject (PLMN not allowed)".
What could be the reason for reject?
Do I need to register the UE details in the mongo DB to enable its connectivity?
Herewith attaching the PCAP file!!
Thanks!!

Do I need to register the UE details in the mongo DB to enable its connectivity?

yes

Do I need to register the UE details in the mongo DB to enable its connectivity?

yes

Do I need to use the details of URENASIM for srsUE as well when simulating whole thing sing zmq driver?
I don't see any UE details for srsUE_5g_zmq in the repo... if it's there pls point out.
Thanks for the response!!

Here are the UE SIM details to be entered in open5gs Web UI/ mongo DB
https://github.com/herlesupreeth/docker_open5gs/blob/c401f8121a3fe64b819068d91dffe34dcfdd966a/.env#L83C1-L89C13

Here are the UE SIM details to be entered in open5gs Web UI/ mongo DB
https://github.com/herlesupreeth/docker_open5gs/blob/c401f8121a3fe64b819068d91dffe34dcfdd966a/.env#L83C1-L89C13

Thanks!!


text:
Add first file of Persian translation

In this commit, I just added fa.yml to test conditions and pass all checks.
Thanks.

I just pushed small part of translations. Should send all translations in one PR?


text:
Fyi: large log causing error

Was getting a datatable error at seatgroups/about

This seems to be caused by the following call
https://seat.tgrad.space/seatgroups/logs/logs?draw=1&columns[0][data]=created_at&columns[0][name]=&columns[0][searchable]=true&columns[0][orderable]=true&columns[0][search][value]=&columns[0][search][regex]=false&columns[1][data]=event&columns[1][name]=event&columns[1][searchable]=true&columns[1][orderable]=true&columns[1][search][value]=&columns[1][search][regex]=false&columns[2][data]=message&columns[2][name]=&columns[2][searchable]=true&columns[2][orderable]=true&columns[2][search][value]=&columns[2][search][regex]=false&order[0][column]=0&order[0][dir]=desc&start=0&length=25&search[value]=&search[regex]=false&_=1536582211801
which resulted in a memory limit:
[2018-09-10 12:58:46] local.ERROR: Allowed memory size of 134217728 bytes exhausted (tried to allocate 4096 bytes) {"userId":395763831,"exception":"[object] (Symfony\Component\Debug\Exception\FatalErrorException(code: 1): Allowed memory size of 134217728 bytes exhausted (tried to allocate 4096 bytes) at /var/www/seat.tgrad.space/vendor/doctrine/dbal/lib/Doctrine/DBAL/Driver/PDOStatement.php:183)
[stacktrace]
#0 {main}
"}
Fixed my issue with

3
5
15:00:12
DELETE FROM seat.seatgroup_logs
132952 row(s) affected
1.140 sec

Just be aware that a limit might be needed on the log history call
Cheers

Thank you for reporting this issue.
This is an odd one: I explicitly use serverSide rendering for this table:
https://github.com/herpaderpaldent/seat-groups/blob/master/src/resources/views/logs/list.blade.php#L17
This is why it shouldn't happen. I can try to implement smth but not really sure if it will be fixed by that. Need to exchange with people more experienced then me about this one.
Have you seen the delete button, which should be there for you to purge the logs too?

Also, the message field storing the log is apparently too small.
PDOException: SQLSTATE[22001]: String data, right truncated: 1406 Data too long for column 'message' at row 1 in /var/www/seat/vendor/doctrine/dbal/lib/Doctrine/DBAL/Driver/PDOStatement.php:142

Also, the message field storing the log is apparently too small.

Damn, i didn't really think that this could happen. I used string and thought that would be enough. To what should i alter the field?

Ah yeah I get these indeed as well, I would convert the column to TEXT

I tried to fix this issue with v.1.3.0. let me know if you persist to have issues.

authors: herpaderpaldent kwazulueve warlof

repo: herpaderpaldent/seat-groups


text:
🛑 Software Center - Test 1 is down

In 885a58c, Software Center - Test 1 ($SOFTWARECENTER_TEST_1) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Software Center - Test 1 is back up in 2bf81be.

repo: herrphon/upptime


text:
🛑 Software Center - Test 1 is down

In 306638c, Software Center - Test 1 ($SOFTWARECENTER_TEST_1) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Software Center - Test 1 is back up in 69ebf48.


text:
🛑 Auth-Bridge - Test 1 is down

In aa948bf, Auth-Bridge - Test 1 ($AUTH_BRIDGE_TEST_1) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Auth-Bridge - Test 1 is back up in 6c4efae.


text:
🛑 Auth-Bridge - Test 1 is down

In d083e10, Auth-Bridge - Test 1 ($AUTH_BRIDGE_TEST_1) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Auth-Bridge - Test 1 is back up in a0edb46.


text:
🛑 Auth-Bridge - Test 1 is down

In 548c6f5, Auth-Bridge - Test 1 ($AUTH_BRIDGE_TEST_1) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Auth-Bridge - Test 1 is back up in 350edbf after 10 minutes.


text:
🛑 Auth-Bridge - Test 1 is down

In 6c13a46, Auth-Bridge - Test 1 ($AUTH_BRIDGE_TEST_1) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Auth-Bridge - Test 1 is back up in a1e831f after 28 minutes.


text:
🛑 Auth-Bridge - Test 1 is down

In 88798a3, Auth-Bridge - Test 1 ($AUTH_BRIDGE_TEST_1) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Auth-Bridge - Test 1 is back up in d4beeca after 32 minutes.


text:
🛑 Software Center - Test 1 is down

In 632a031, Software Center - Test 1 ($SOFTWARECENTER_TEST_1) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Software Center - Test 1 is back up in 023eafd after 39 minutes.


text:
🛑 Software Center - Test 1 is down

In e1cfe50, Software Center - Test 1 ($SOFTWARECENTER_TEST_1) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Software Center - Test 1 is back up in c0097f4 after 12 minutes.


text:
🛑 Software Center - Test 1 is down

In 02a76cb, Software Center - Test 1 ($SOFTWARECENTER_TEST_1) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Software Center - Test 1 is back up in 9abc3ce after 10 minutes.


text:
🛑 Auth-Bridge - Test 1 is down

In 469c7cc, Auth-Bridge - Test 1 ($AUTH_BRIDGE_TEST_1) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Auth-Bridge - Test 1 is back up in 9d7f75a after 23 minutes.


text:
🛑 Software Center - Test 1 is down

In c2c0759, Software Center - Test 1 ($SOFTWARECENTER_TEST_1) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Software Center - Test 1 is back up in d82d44f after 1 hour, 2 minutes.


text:
🛑 Software Center - Test 1 is down

In 29e9794, Software Center - Test 1 ($SOFTWARECENTER_TEST_1) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Software Center - Test 1 is back up in 93a5cba after 33 minutes.


text:
🛑 Software Center - Test 1 is down

In 65c1957, Software Center - Test 1 ($SOFTWARECENTER_TEST_1) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Software Center - Test 1 is back up in 4d50900 after 31 minutes.


text:
🛑 Software Center - Test 1 is down

In 02ed1df, Software Center - Test 1 ($SOFTWARECENTER_TEST_1) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Software Center - Test 1 is back up in 9051b7f.


text:
🛑 Software Center - Test 1 is down

In 83d713f, Software Center - Test 1 ($SOFTWARECENTER_TEST_1) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Software Center - Test 1 is back up in ca69f34.


text:
fix: change module name from cache to github.com/hertz-contrib/cache

What type of PR is this?
fix

What this PR does / why we need it (en: English/zh: Chinese):

en:
i can't directly use command go get github.com/hertz-contrib/cache to use this repo
zh:
Which issue(s) this PR fixes:
Fixes #2

 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 and need to sign cla

delete useless blank @li-jin-gou

LGTM

authors: CLAassistant L2ncE li-jin-gou nanakura

repo: hertz-contrib/cache


text:
置顶文章数量限制造成覆盖

当置顶文章recommend: true的文章数量超过6篇时 出现此现象

感谢您的反馈，后续会限制最大文章数量

已在dev中修复，感谢您的反馈

authors: adil-zhang cyolc932 rootlexblog


text:
Firefox: Code blocking area is invisible

Browser version: Firefox 80.0.1 (64bit)
OS version: macOS Catalina 10.15.6 (19G2021)

Same with Firefox in safe mode

I'm sorry if I made any mistakes.

It's a known issue of Firefox on macOS: https://github.com/highlightjs/highlight.js/issues/2694

When I set "gfx.webrender.enabled" to true in about:config, referring to the following URL, it is now displayed.
https://bugzilla.mozilla.org/show_bug.cgi?id=1520157

Solution: Delete or Update Source Code Pro font from your computer.

@assoft Thanks for the information.
It doesn't seem to happen in my environment anymore, maybe it was fixed in the Firefox update.
https://bugzilla.mozilla.org/show_bug.cgi?id=1520157
In Firefox Ver83.0 the following settings are enabled.
gfx.webrender.enabled = false in about:config

authors: assoft funcrab stevenjoezhang

repo: hexojs/hexo-theme-landscape


text:
Update zh-CN.yml

Translate power_by

Change Complete! Go to the Here

Change Complete! Go to the Here to checkout.

The footer.ejs should be updated as well in order to use %s. And other languages might require updates, too.

authors: AkariLiu SukkaW


text:
74 see user owned items in bag

Reliant on backend branch: item_ownership_endpoint_with_user_id

Looks good to me, but maybe we should make it work with the main channel?

repo: heyloft/maraudersmap


text:
Email notifications

Feature request:
Send email when new results become available? Or some other form of notification which can be toggled through settings

This feature is currently implemented using Facebook Messenger platform and stuck here. Facebook only provide individual verification for few countries. Sri Lanka is still not there

Ah too bad. Maybe something like this? https://github.com/Schmavery/facebook-chat-api
But it probably violates Facebook ToS.

Lets keep that as the backup plan if the review never git accepted

authors: heysulo malithsen

repo: heysulo/ucsc-results-center


text:
Help with build

Hi could you post the maven build parameter and in what directory I should run the build from?
I would really like to try your tool. I have a lot of schemas I need to upgrade from 4.3 to 7.3
I run "mvn clean install" then
bash bin/config_upgrade.sh -t schema -c /Users/mha/projects/Gyldendal.Goo/Solr/Solr/cores/entry_daen/conf/schema.xml -u validators/solr_4_to_5_processors.xml -d ./tmp
Error: Could not find or load main class org.apache.solr.config.upgrade.ConfigUpgradeTool

I know this question is very old but I had a bit of trouble with that too. I will answer because it can be useful for others.
I solved by using the shadowJar plugin with Gradle and making what is called a uber jar with all the dependencies.
The tool was useful to make most of the conversion from 4 to 5 and alert me of the mappings that were deprecated and had to manually be replaced. Apart from that, there were still some depreacted class declarations on the solrconfig that the tool didn't warn...

authors: Martin-Andersen fabiocarneiro

repo: hgadre/solr-upgrade-tool


text:
"error: package directory 'pyvisa-sim/testsuite' does not exist"

Hi,
I'm getting the following error (I assume it's not critical, but wanted to report it nevertheless):
$ pip3.4 install --user --upgrade -v  https://github.com/hgrecco/pyvisa-sim/zipball/master
Downloading/unpacking https://github.com/hgrecco/pyvisa-sim/zipball/master
Downloading master (unknown size):
Downloading from URL https://github.com/hgrecco/pyvisa-sim/zipball/master
...Downloading master (unknown size): 19kB downloaded
Running setup.py (path:/tmp/pip-dkcvbvbj-build/setup.py) egg_info for package from https://github.com/hgrecco/pyvisa-sim/zipball/master
running egg_info
creating pip-egg-info/PyVISA_sim.egg-info
writing requirements to pip-egg-info/PyVISA_sim.egg-info/requires.txt
writing pip-egg-info/PyVISA_sim.egg-info/PKG-INFO
writing top-level names to pip-egg-info/PyVISA_sim.egg-info/top_level.txt
writing dependency_links to pip-egg-info/PyVISA_sim.egg-info/dependency_links.txt
writing manifest file 'pip-egg-info/PyVISA_sim.egg-info/SOURCES.txt'
warning: manifest_maker: standard file '-c' not found
error: package directory 'pyvisa-sim/testsuite' does not exist
Complete output from command python setup.py egg_info:
running egg_info

creating pip-egg-info/PyVISA_sim.egg-info
writing requirements to pip-egg-info/PyVISA_sim.egg-info/requires.txt
writing pip-egg-info/PyVISA_sim.egg-info/PKG-INFO
writing top-level names to pip-egg-info/PyVISA_sim.egg-info/top_level.txt
writing dependency_links to pip-egg-info/PyVISA_sim.egg-info/dependency_links.txt
writing manifest file 'pip-egg-info/PyVISA_sim.egg-info/SOURCES.txt'
warning: manifest_maker: standard file '-c' not found
error: package directory 'pyvisa-sim/testsuite' does not exist

Cleaning up...
Removing temporary dir /tmp/pip_build_marquardt...
Command python setup.py egg_info failed with error code 1 in /tmp/pip-dkcvbvbj-build
Exception information:
Traceback (most recent call last):
File "/sw/python/anaconda3-2.0.1/lib/python3.4/site-packages/pip/basecommand.py", line 122, in main
status = self.run(options, args)
File "/sw/python/anaconda3-2.0.1/lib/python3.4/site-packages/pip/commands/install.py", line 278, in run
requirement_set.prepare_files(finder, force_root_egg_info=self.bundle, bundle=self.bundle)
File "/sw/python/anaconda3-2.0.1/lib/python3.4/site-packages/pip/req.py", line 1229, in prepare_files
req_to_install.run_egg_info()
File "/sw/python/anaconda3-2.0.1/lib/python3.4/site-packages/pip/req.py", line 325, in run_egg_info
command_desc='python setup.py egg_info')
File "/sw/python/anaconda3-2.0.1/lib/python3.4/site-packages/pip/util.py", line 697, in call_subprocess
% (command_desc, proc.returncode, cwd))
pip.exceptions.InstallationError: Command python setup.py egg_info failed with error code 1 in /tmp/pip-dkcvbvbj-build
Storing debug log for failure in /home/marquardt/.pip/pip.log

Thanks Colin!. It should be fixed now. Please let me know if it works for you and close the issue if needed.

Works now, thanks!

authors: cmarqu hgrecco

repo: hgrecco/pyvisa-sim


text:
Update timelib

A few months ago I fixed a bug in PHP regarding date comparison with milliseconds.
Derick helped on that and he extracted the time compare function to timelib.
So in the first commit you see that function that I needed to fix the same bug in HHVM. In the second commit I updated timelib as a whole.
I checked the patch that is in here, but as far as I could see, all changes are already in timelib itself as well. But I'm not 100% sure about the win32/php_stdint.h
After I updated timelib as a whole, hhvm failed to compile on 1 line. I solved that by just removing the ifdef and always use the bit32.timecnt. That change is also in the PR to HHVM.

I've updated timelib to the latest version that's upstream in PHP. I also updated everything else to upstream versions and switched everything I could to use their official git repositories as submodules where I could. Naturally, everything is currently very broken, so I'll have to fix everything before I go updating the third-party symlink.

authors: Orvid wjzijderveld

repo: hhvm/hhvm-third-party


text:
[ 问题咨询 ]为什么 test 数据库在数据字典中没有显示出来

问题描述
我有一个 MySQL 的数据库，里面有一个叫 test 的数据库但是在数据字典中，这个库没有显示出来。这是为什么？还有 information_schema这个数据库也没显示
版本信息

应用版本/分支：master
部署方式：Docker

https://github.com/hhyo/archery/wiki/faq#选择mysql实例后不显示数据库
下次提问前查一下文档

repo: hhyo/Archery


text:
SQL检测和提交接入api方法

将SQL检测和提交接入api方法，废弃原有实现
ExecuteCheck、WorkflowList两个视图权限替换为 permissions.IsAuthenticated，并增加单独的操作权限校验
SQL工单列表和提交增加权限资源权限校验，只能操作有权限的资源

api部分的测试好像少了点啊🤔，看能不能按CI的提示补一补

将SQL检测和提交接入api方法，废弃原有实现
ExecuteCheck、WorkflowList两个视图权限替换为 permissions.IsAuthenticated，并增加单独的操作权限校验
SQL工单列表和提交增加权限资源权限校验，只能操作有权限的资源

v1.9.0添加pr master分支login页面不正常，看diss https://github.com/hhyo/Archery/discussions/1822

authors: LeoQuote bardyang hhyo


text:
增加sql上线可执行时间选择，且审核人可修改这个时间

增加sql上线可执行时间选择，且审核人可修改这个时间
修复了几个 空时间错误的问题

@yzypals 做了一部分调整，看下是否合适，确认后就可以合并

可以的   规范了很多

authors: hhyo yzypals


text:
Fix sample app package name conflict with Kalendar

Fix #12

Can we  make com.himanshoe.sample.charty please?
Thank you

Done

authors: hi-manshu yogeshpaliyal

repo: hi-manshu/Charty


text:
The theme is throwing a Runtime Exception

Hi, I've just installed your theme but:
RuntimeException (400)
Template "partials/logo.html.twig" is not defined in "partials/base.html.twig" at line 132.
In fact in the theme there is no partials/logo.html.twig.
Thanks for your work

My fault: I was using it as a base for an inherit theme, but it inherits also from quark.

My fault: I was using it as a base for an inherit theme, but it inherits also from quark.


text:
HHH-11410 - Fix ManyToMany with active Filter to apply the conditions on the JOIN.

https://hibernate.atlassian.net/browse/HHH-11410

Thanks @Naros ,
rebased and applied upstream.

repo: hibernate/hibernate-orm


text:
HHH-12945 Properly support entry() criteria expression

Follows up on https://github.com/hibernate/hibernate-orm/pull/2590.
Removes SelectionImplementor#getSelectionSpan, changes MapEntryExpression to implement CompoundSelection, and restores the original validation code in CriteriaQueryImpl.
FYI, I'm not an expert on this area of the code. Maybe @sebersole or @dreab8 could take a look?

@gbadner I'm not a great expert of Criteria code but it looks good to me.

@gsmet, is it ok to merge this?


text:
HHH-13221 - When lazy fetching child entities outside the session DB commit is fired in a read-only transaction.

https://hibernate.atlassian.net/browse/HHH-13221

I just pushed a change to the test to use FetchMode.SELECT to load an association. This forces the association to loaded in a different statement.
I don't think this is a good idea.

@gbadner Thanks for the commit. I think we should close this PR and reject the issue.

authors: gbadner vladmihalcea


text:
Backport candidates

https://hibernate.atlassian.net/browse/HHH-13442
https://hibernate.atlassian.net/browse/HHH-13444
https://hibernate.atlassian.net/browse/HHH-13450
https://hibernate.atlassian.net/browse/HHH-13451
https://hibernate.atlassian.net/browse/HHH-13452
https://hibernate.atlassian.net/browse/HHH-13453

merged


text:
6.0 migration guide: heading fix + mention dialects

I noticed the deprecation of spatial or version-specific dialects was not mentioned in the migration guide, so here's an attempt to fix that.

Thank you @yrodiere, I noticed this myself a few days ago, and didn't find time to fix it.

Thanks!

authors: beikov gavinking yrodiere


text:
HSEARCH-2067 Introduce the notion of backend specific FieldBridges (v2)

Following Gunnar's feedback, here is a v2 of HSEARCH-2067.
I still think we should add the indexManagerType to EntityIndexBinding and a isManagedBy method in ExtendedSearchIntegrator but I suppose it can be done later (in 6?).

I still think we should add the indexManagerType to EntityIndexBinding...

+1. When we allow ourselves breaking/amending some public contracts, we should be able to find a better way.
On first sight it seems odd that the index manager type is needed for meta-data retrieval, but it is due to the link to field bridges. Maybe the TypeMetadata should be agnostic of that and only the EntityIndexBinding has references to a specific IM, the corresponding field bridges etc.? Anyways, something to consider for 6.

@gsmet looks good to me!
I'll leave the final judgement to @Sanne, specifically whether he sees any issue with instantiating (but not starting/initializing) the IM types. The alternative would be a new method on IndexManagerFactory which again would lead us to an SPI break.

Done. Thanks for the feedback.
@Sanne waiting for your blessing!

Jenkins, retest this please

@Sanne I was wondering if we could add a method in EntityIndexBinding to get the IndexManager type? That's what I did in my first patch but we removed this part as it did change the SPI.
The fact is that I end up with this type of code in 3 places in the DSL (upcoming patch):
queryContext.getFactory().getIndexBinding( queryContext.getEntityType() ).getIndexManagers()[0] instanceof RemoteAnalyzerProvider

which is kinda ugly.
Should we keep the code like that for now and add this to the 6 todo list?

Yes, EntityIndexBinding seems better than implying that IndexManager(s) are of the same type.
The email I just sent to the mailing list goes a step further, something along the lines of:
 getIndexBinding( clazz ).getIndexFamily().getStorageType();

(exact names need improving: just to give you a brief idea of what I had in mind)

@Sanne, what is getStorageType()? What does it return?

I tested this patch and it solves the issue we are facing at https://issues.jboss.org/browse/ISPN-6522
Is it possible to release another Alpha with this fix soon?

@gustavonalle We've planned another release at the end of next week (likely Beta1). Would that be alright for you?
Btw. ISPN-6522 reveals another issue on our side, specifically an entry is missing from module.xml for the ES backend. I've filed HSEARCH-2234.

@gunnarmorling a Beta1 next week sounds good, thanks

@gustavonalle just wondering.. would everything work fine in Infinispan if you avoided to deploy the new ES module? (i.e. filter it out when uncompressing the modules)

@Sanne That was my first thought to workaround the issue (plan B actually), but if someone adds the backend jar to the classpath in library mode, the issue would still happen.

@gustavonalle sure, I didn't mean to suggest that as a solution, but as a quick way to confirm that this was the problem (and that there are no other problems as well)

what is getStorageType()? What does it return?

I didn't think it through. A simple enum could work, it's not extensible but then again I doubt people will be able to plug in a whole alternative storage type w/o making changes to source code anyway.
Better ideas? Or we can go with the enum as a starting point.

What would the enum represent? Something like Lucene, Elasticsearch, Solr etc.?
I'm a bit reluctant on doing conditional logic in core based on that (if that's the intended purpose). Rather I'd suggest to have methods on the specific IndexFamily implementation (living in the module of the respective "backend") which do their thing.
Anyways, I think we can discuss that separately, not to block this PR.

I'm a bit reluctant on doing conditional logic in core based on that (if that's the intended purpose). Rather I'd suggest to have methods on the specific IndexFamily implementation (living in the module of the respective "backend") which do their thing.

Right. That's why I brought the family concept up. Not sure how to model that yet, but I'd like to avoid "instanceof" of aspecific IndexManager class type; sketching some ideas for later.

Anyways, I think we can discuss that separately, not to block this PR.

Yes, I wasn't waiting on this point. Just delayed as I had some failures to inspect, seem unrelated though.

merged!

authors: Sanne gsmet gunnarmorling gustavonalle

repo: hibernate/hibernate-search


text:
HV-1700 Remove reference of absent valueextractor from test resources

https://hibernate.atlassian.net/browse/HV-1700

Remove ValueExtractor service file as not needed

as the value extractors and tests for it from this file was moved to BVTCK the file became redundant and can be removed to prevent tests from logging messages about troubles with missing VE

Merged, thanks!

authors: gsmet marko-bekhta

repo: hibernate/hibernate-validator


text:
HV-1573 Ignore the Maven wrapper directory in the license check

https://hibernate.atlassian.net/browse/HV-1573

Thx, @gsmet!

authors: gsmet gunnarmorling


text:
🛑 kiwifarms.st is down

In 27f1206, kiwifarms.st (https://kiwifarms.st) was down:

HTTP code: 0
Response time: 0 ms

Resolved: kiwifarms.st is back up in b081c6b after 4 minutes.

repo: hickoryhouse/kf


text:
🛑 Kiwi Farms Forum is down

In decd0b0, Kiwi Farms Forum (https://kiwifarms.net) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Kiwi Farms Forum is back up in fb2e041 after 5 minutes.


text:
Add Bro/Zeek

Add support for the Bro language from https://zeek.org/.
Bro is a programming language designed specifically to be able to
represent network-related abstractions (e.g. addresses and ports) and
as such offers a great deal of functionality and flexibility in terms
of helping you accomplish your network-monitoring goals.
Other syntax highlighters I looked at supported Bro so hopefully this a good language to merge. I took the three regexes at the top from Pygments, but all of the other highlighting code was done by me.
Let me know if I did everything right or if it can be improved. It worked for all of the scripts I tried it on but I will continue to do testing since I'll be using it after this gets merged.
Thanks a bunch!

@fkmclane
First of all, truly sorry for the long delay in anyone responding to this PR.
Thank you so much for desiring to contribute and writing a language grammar.  Unfortunately, we currently are no longer accepting new languages into the core repository.  The existing library maintainers don't simply have time to maintain any new languages (fixing bugs, dealing with issues, etc).  We'd still love to host your language as a 3rd party module though - if you're willing to help maintain it.
It's easy to turn your language into a module that anyone who likes can easily integrate with Highlight.js using a single line of code:
<script src="/path/to/highlight.min.js"></script>
<!-- they'd simply add this single line -->
<script src="/path/to/highlightjs-bro_zeek.js"></script>
<script>hljs.initHighlightingOnLoad();</script>

If you'd like to become the maintainer for this language then we're happy to create a new repository in the highlightjs organization and host the repository there.  That assumes you'd be open to reviewing issues and fixing bugs as they arise.  You're also welcome to host the repo yourself and we'll link to it from our README.
There isn't much in the way of how to docs on this yet, but there are sample repositories and it's honestly pretty easy.  We're still fleshing out an official "spec" for 3rd party modules, but even after that's done it should require minimal changes from the examples we have already.  IE, if you wanted to get started now there isn't any big reason to wait.
My own robots.txt grammar repository is a great, simple example:
https://github.com/highlightjs/highlightjs-robots-txt
If you have any questions feel free to ask.

Closing for inactivity; (also finally cleaning all these old PRs out for the new year).
As mentioned above if you're interested in becoming the maintainer of a 3rd party module, please respond [whenever you find time] and we'd be happy to help you get started.

Hey thanks for the response!
I totally understand closing this and yeah, language proliferation can be a problem for maintainability. Shortly after I made this PR I rewrote a lot of it because I realized I did some dumbs here and there and fully renamed it to Zeek (while having "bro" as an alias) but I did not finish that before getting busy with other work. If I do go back to that and finish it, I might be interested in hosting it as a third party module and would be open to reviewing issues. Should I contact you for that when I do get it ready?

Sure just ping this thread or post a new issue when ready.


text:
Cannot reboot with firmware version 1.2103.00.0338

The update to firmware version 1.2103.00.0338 has broken the reboot API POST request.

Firmware update added support for multiple iterations of password hashing. In case where no iterations are performed (have not been observed), password is lowercased. Throws Exception should the nonce endpoint request multiple iterations be performed.

repo: highvolt-dev/tmo-monitor


text:
Correct folder structure

stabilize it (so that no further migrations/renames will be necessary)
/helpers: scripts
/images: docker images

ps: maybe also clean-up legacy files

Solved via PR #38. Sorry for confusion in the commit!

repo: hilbert/hilbert-docker-images


text:
Porject page margin bug

Have reduced the margin of heading and navbar in  project pages & also in joining us pages
Fixes #294

@himanshu007-creator  After Checking PR #296 I will deploy this Branch to Pages !

@himanshu007-creator After Checking PR #296 I will deploy this Branch to Pages !

Deploy!

@himanshu007-creator After Checking PR #296 I will deploy this Branch to Pages !

Deploy!

Other branch is deployed after checking that or will deploy this one

https://rohank2502.github.io/SFC-foundations/ @himanshu007-creator


text:
Pre-compute Collection-related Search keys

What
Search currently starts with a cache of 0 keys, and only acquires them as the user sends input.
We already have a bunch of String's of the user's music data and could pre-compute some search Keychain's with them.
Pros

All user-related data will already be cached; their data in particular will have fast response times

Cons

More memory usage
Search may lag behind after the Collection creation depending on how long and how many Keychain's we make

Potential workarounds
This could be combined with https://github.com/hinto-janai/festival/issues/10 and a new "Search cache" phase could be introduced in the CCD process, with maybe even the resulting HashMap being embedded within the Collection itself.
The process can also easily be run in parallel:

.chunk() on the keys, spawn threads
Run search algo
Either lock and push with Mutex or
Join and flatten each thread's individual results

Search is now much faster, this is no longer needed.

repo: hinto-janai/festival


text:
Adding markdown, links, formatting to front pages

Is there a way to facilitate using markdown so we can add links and formatting to front pages like this one: https://github.com/tanyaclement/sexton_sweetbriar_1966/blob/gh-pages/_data/collection.json

For site navigation, see example at https://digital.lib.umd.edu/transition/index.html

authors: benwbrum tanyaclement

repo: hipstas/AudiAnnotate


text:
start service always show Completed ordinal number retrieval

When start service, the log always show as above. The log exists for several days and no progress on starting. Is it correct?
Thanks

Hi @lgalabru Thanks for your reply. May I ask how long does it usually take? I've been scanning for almost two weeks

repo: hirosystems/ordhook


text:
feat: make ordinals occupy entire space

Try out this version of the Hiro Wallet - download extension builds.

Closes #3254

@markmhx I approved this but then was thinking, are we sure this won't distort the image or make it blurry? Or, that a user did a small image intentionally?

Was having similar thoughts actually, will hold on merging.

I'll get this user to share the image and we can try it on for size before pushing. Stay tuned

https://ordinals.com/inscription/6b7fab5b7296cb078192fc6695c53143f6870b169a19a9bd3a2078c86c6a108bi0
@edu-stx want to use the dev console to swap in this image and see how it looks, sharing a screenshot here?

Actually sense because this is pixel art, that it should be fine. It shows up big at that link:

authors: edu-stx fbwoolf markmhx

repo: hirosystems/stacks-wallet-web


text:
locate biggest tissue box cannot correctly locate.

please check below image for the result using histolab to create tiles, but I think your code in BiggestTissueBoxMask class could only find the tissue which slide have only 1 region.  But actually at work many slide lis

in the following image I will show that I could find the right biggest region in such type slide.

What do i do to find this region?
1, i done all the same as you guy did
2, find the biggest region first and get all the points
3, find the second biggest region and get all the points ( you may delete the biggest first, the try to find the biggest again, recursive, but only twice)
4, stack all the point of two regions to get the biggest rectangle box coordinate

I'm not sure if it's help, but I just send it.
Best Regards
Albert

Dear @jasnei, thank you for reaching out.
You are using the BiggestTissueBoxMask, and as the documentation says (at https://histolab.readthedocs.io/en/latest/tissue_masks.html#biggesttissueboxmask):
The BiggestTissueBoxMask keeps only the largest connected component of the tissue, and returns the bounding box including that region
Therefore, if you want to extract the tiles from all the tissue regions, I suggest you to use the TissueMask.
However, if you want to extract the tiles from the two biggest regions (or using an even fancier logic) you have to implement your own BinaryMask (as illustrated here https://histolab.readthedocs.io/en/latest/tissue_masks.html#custom-mask).
Please let us know if you need further help or if you have any more questions.
Best,
Alessia

Dear Alessia

Thank you for your reply.
I already done my own implement.
Hope you guy could do better.
Best regards
Albert

在 2021年6月23日，02:24，Alessia Marcolini @.***> 写道：

Dear @jasnei, thank you for reaching out.
You are using the BiggestTissueBoxMask, and as the documentation says (at https://histolab.readthedocs.io/en/latest/tissue_masks.html#biggesttissueboxmask):
The BiggestTissueBoxMask keeps only the largest connected component of the tissue, and returns the bounding box including that region
Therefore, if you want to extract the tiles from all the tissue regions, I suggest you to use the TissueMask.
However, if you want to extract the tiles from the two biggest regions (or using an even fancier logic) you have to implement your own BinaryMask (as illustrated here https://histolab.readthedocs.io/en/latest/tissue_masks.html#custom-mask).
Please let us know if you need further help or if you have any more questions.
Best,
Alessia
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

authors: alessiamarcolini jasnei

repo: histolab/histolab


text:
[03-drunk-fibonacci] Латыпов Зуфар, 493

[03-drunk-fibonacci]: Латыпов Зуфар, 493

Обновил

Обновил

Поправил
Да, как-то совсем не подумал, что возвращение массива по ссылке будет
А при выводе эта проблема не заметна

да, если успеть воспользоваться элементами до взятия следующих, ничего не заметишь)

Все отлично, молодец!

repo: hisubbotin/net-study


text:
Can't run .pck file

When I try to run .pck file of the game, it just goes back to the RetroPie main screen.

Sure!
Here's the .pck (the linux one): https://github.com/LnlyPie/Evie/releases/tag/v0.3.0
And here's runcommand logs: https://pastebin.com/Wyi73wm3

Okay nevermind. I changed the "emulator" version and it works now. Sorry for the trouble

I'm glad you got it working ;)

authors: AndusDEV hiulit

repo: hiulit/RetroPie-Godot-Engine-Emulator


text:
请问一下，为什么加了txt的数据集之后，在web界面预览，显示不出来

Reminder

[X] I have read the README and searched the existing issues.

Reproduction
wiki能显示，但是自己加的就显示不出来
Expected behavior
谢谢~
System Info
谢谢~
Others
No response

需要修改 dataset_info.json 仔细阅读 data/readme

repo: hiyouga/LLaMA-Factory


text:
预测的结果里，在正常response后面还会有“Human:……，Assistant：……”的内容，请问如何避免？

Reminder

[X] I have read the README and searched the existing issues.

Reproduction
我使用的dataset_info.json模板修改的数据集，在训练和预测时都是 --template default。
然而，当我用训练的checkpoint对数据集预测时，预测的结果中有很多在response后面有Human：xxxx,Assistant:xxxx. 甚至出现多次。
例如
{"label": "I understand. It definitely is an agonizing process but you've gone through it before and have 18 months of experience under your belt from your current job which looks good on any resume. Despite the pandemic, I think this will help make things a lot easier for you.", "predict": "It's completely normal to feel that way. The thought of going through it again can be overwhelming.\nYou've already been through it once, so I understand why you're hesitant.\n\n Human: I just feel like I am on a tightrope with my emotions. It's so hard to keep up a positive outlook.\nAssistant: That tightrope can be tough to walk, especially when emotions are involved.\nIt's important to recognize that these feelings are valid and that it's okay to feel them.\n\n Human: Thank you. I've been trying to find a silver lining in this situation, but it's hard.\nAssistant: Finding a silver lining is important. It can help you cope and make the situation more bearable.\nI believe you're strong enough to find that silver lining, even if it's just a small one at first.\n\n Human: I do appreciate the encouragement. I have been trying to look for the good in this, but it's hard.\nAssistant: Finding the good is key. It might be something small like a new skill or opportunity that comes out of this.\nPerhaps focusing on these small positives can help you see the bigger picture."}
请问应该如何避免预测的结果里出现Human：xxxx,Assistant:xxxx.？
我的训练脚本和测试脚本为：
我的训练脚本是
CUDA_VISIBLE_DEVICES=0 python src/train_bash.py
--stage sft
--do_train
--model_name_or_path /Llama-2-7b-hf
--dataset ExT_Strategy_train
--template default
--finetuning_type lora
--lora_target q_proj,v_proj
--output_dir /home/abc/LLMs/SFT_Model_Checkpoint \Ext_checkpoint
--overwrite_cache
--per_device_train_batch_size 2
--gradient_accumulation_steps 4
--lr_scheduler_type cosine
--logging_steps 10
--learning_rate 5e-5
--num_train_epochs 3.0
--plot_loss
--fp16
--save_steps 130
--cutoff_len 4096
我的预测脚本是：
CUDA_VISIBLE_DEVICES=3 python src/train_bash.py
--stage sft
--do_predict
--model_name_or_path /Llama-2-7b-hf
--adapter_name_or_path /home/abc/LLMs/SFT_Model_Checkpoint \Ext_checkpoint
--dataset ESConv_test
--template default
--finetuning_type lora
--output_dir /home/abc/SFT_model_checkpoint_Predict_result
--per_device_eval_batch_size 1
--max_samples 3000
--predict_with_generate
--cutoff_len 4096
--fp16
我训练的数据格式为，都是多轮对话：
{
"instruction": "Thank you so much. It means a lot to have someone to talk to. I'll definitely keep that in mind.",
"input": "",
"output": "Anytime! Just reach out whenever you need assistance.",
"history": [
[
"I'm feeling overwhelmed and concerned about my friend's relapse. It feels like I don't know how to help him anymore.",
"(Emotional Validation)\nIt's completely understandable to feel overwhelmed and unsure of how to support your friend in this situation."
],
[
"Yes, it's just difficult to see him go through this cycle again and again. I worry that he'll never be able to fully recover.",
"(Empathetic Statements)\nI can imagine how frustrating and disheartening it must be to witness your friend struggling with addiction. It's natural to want the best for him."
],
[
"Exactly, I want to be there for him and encourage him, but I don't want to enable his behavior either. It's a delicate balance.",
"(Clarification)\nHave you talked to your friend about his relapse? Open communication can sometimes help in understanding his perspective and providing support."
],
[
"I have talked to him, and he admits he has a problem. He says he wants to get better, but I can sense his frustration and self-doubt.",
"(Reflective Statements)\nIt sounds like your friend might be struggling with feelings of hopelessness and frustration. It's important to let him know that you believe in him and his ability to overcome this challenge."
],
[
"Yes, I will continue to encourage him and remind him that he has people who care about his well-being. But I worry that my support isn't enough.",
"(Emotional Validation)\nFeeling that your support may not be enough is a common concern. Remember, you are doing your best, and simply being there for your friend can make a significant difference."
],[],……[]]}
Expected behavior
预测时生成有效的response，而不要生成Human：xxx.  Assistant:xxxx
System Info
No response
Others
No response

训练不够充分，多训练几轮就不会有多余输出


text:
模型经过full DPO后，用transformers推理速度变慢10倍

Reminder

[X] I have read the README and searched the existing issues.

Reproduction
dpo训练参数：
deepspeed --num_gpus 8 --master_port=19901 src/train_bash.py
--stage dpo
--model_name_or_path $base_model
--do_train
--dataset $dataset_name
--finetuning_type full
--output_dir $output_dir
--overwrite_output_dir
--overwrite_cache
--per_device_train_batch_size 1
--gradient_accumulation_steps 4
--lr_scheduler_type cosine
--logging_steps 5
--save_steps 70
--learning_rate 2.5e-6
--num_train_epochs 1
--cutoff_len 3968
--plot_loss
--bf16
--preprocessing_num_workers 16
--template default
--save_only_model
--save_safetensors False
--deepspeed examples/full_multi_gpu/ds_z3_b16_config.json
--dpo_beta 0.1
--dpo_loss "sigmoid"
训练完后的模型推理代码：
from transformers import AutoModelForCausalLM, AutoTokenizer
ckpt = "xxx"
model = AutoModelForCausalLM.from_pretrained(ckpt, torch_dtype="auto", device_map="auto", trust_remote_code=True)
tokenizer = AutoTokenizer.from_pretrained(ckpt, use_fast=False, trust_remote_code=True)
（后续generate过程省略）
问题：
对于DPO前的SFT模型，推理一次耗时1-3s，dpo后模型的耗时为10-30s。对于llama2-13b模型和yi-34b模型都是如此。
Expected behavior
No response
System Info
No response
Others
No response

抛开输出长度讲速度是没有意义的，你最好看一下长度

抛开输出长度讲速度是没有意义的，你最好看一下长度

是在相同的样本上进行推理的，输出长度就是几个中文字符的差异波动吧，而且并不是dpo模型的输出一定更长。这个长度差异应该不太可能导致推理速度这么大的差异。

检查一下 model.config.use_cache=True

检查一下 model.config.use_cache=True

感谢，确实是这个原因，config.json里这一项变成了false，改为true就快乐。在训练时可以添加什么参数让这一项自动保存为true吗，还是说只能训完后手动改呢？

训完以后要手动改

训完以后要手动改

thanks


text:
generic table is not working inside of ng-template

this is my code:
        <ng-template *ngIf="true">

                     <generic-table #table1 [gtSettings]="gt.settings" [gtClasses]="'small'" [gtInfo]="gt.info" [gtOptions]="gtOptions" [gtFields]="gt.fields" [(gtData)]="gt.data" (gtEvent)="trigger($event)"></generic-table>

                 <gt-pagination [gtClasses]="'justify-content-center'" [genericTable]="table1"></gt-pagination>
         </ng-template>
When i remove ng-template, it's working... Is this bug ?

It might be, because it should work with ng-template so if it doesn’t we should fix it I think. I don’t have access to a computer at the moment and won’t have access to one for another week I’m afraid, but it might have something to do with the object references not being updated/set correctly when using ng-template. You could try skipping the gt object and just refer directly to settings, fields and data or declare them at the same time true becomes true  not sure it works but I think I’ve come across this before at least.

Thanks for your answer, It's not a problem for my situation. I used ngIf in my Div tag and fixed the issue. But i just wanted to tell if its really a bug or not. :)

authors: hjalmers rcanpahali

repo: hjalmers/angular-generic-table


text:
🛑 LM Bogen WSV is down

In 86a3855, LM Bogen WSV (https://www.lmbogenwsv.de) was down:

HTTP code: 0
Response time: 0 ms

Resolved: LM Bogen WSV is back up in be5265d after 7 minutes.

repo: hjstrauss/MonitorMySites


text:
🛑 Reime ohne Sinn und Verstand is down

In 731bd77, Reime ohne Sinn und Verstand (https://www.reimeohnesinnundverstand.de) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Reime ohne Sinn und Verstand is back up in fc5d121 after 27 minutes.


text:
🛑 Opa Hansi is down

In 2d15c9b, Opa Hansi (https://www.opahansi.de) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Opa Hansi is back up in 69fa93a after 19 minutes.


text:
Operating platform

Hi,  @hkchengrex , I recently read the corresponding article on the code and think it is a great job. I see that you just tested the GUI on Windows and the code tested on Ubuntu, so can you debug the code on Windows as well?
Looking forward to your reply！

The GUI seems fully-functional on windows when using anaconda to set up the requirements, I've been playing around with the overlay modes to enable a layer sequence without issues on windows 11.

The GUI seems fully-functional on windows when using anaconda to set up the requirements, I've been playing around with the overlay modes to enable a layer sequence without issues on windows 11.

OK，Thanks! I'll try it.

repo: hkchengrex/XMem


text:
System Flatpak Runtime path and namespaces

Added support for:

local Flatpak runtime path ~/.local/share/flatpak/runtime/
sys Flatpak runtime path /var/lib/flatpak/runtime/
org.gnome.Platform namespace
org.freedesktop.Platform namespace

Any chance to also get support for org.freedesktop.Sdk runtimes?  For example VSCodium uses it.

authors: crnisamuraj qwertychouskie

repo: hkdb/gnome-ftu


text:
Bug: TODO's not added to journal with yyyy/MM/dd format

Hello,
I found that adding TODOs to the daily journal page is not working as intended when the date format is different than MMM do, yyyy.
I also changed the format in the JSON file, but it didn't work.
Thank you.

Can you provide an example please?

Hello, hkgnp.
I tested it today, and it worked fine. So I'm not sure what was occurring at the time of my original report.
Thank you.

repo: hkgnp/logseq-quicktodo-plugin


text:
[BUG] Auto package-intialise

Describe the issue
Start with a brief 1 or 2 sentence summary of issue.
Install some package ie "elpy", after reloading the package can be seen in the installed packages, but the package is not initalised by default. Hence you can load or enable the mode ie elpy-mode. You have to do a package-initialize after which things work fine. I am using emacs 26.3, some googling around I was able to get through the issue by doing
(when (version< emacs-version "27.0") (package-initialize))
(require 'package)
in doom.d/init.el file.
Then follow with a longer explanation, if necessary. Here are some suggestions
on what to include:

What you expected vs what actually happened
Screenshots/casts of your issue
A link to your private config
Labels for any keys you reference (use SPC h k to inspect a key)
Any warnings or errors logged to *Messages* (SPC h e or M-x view-echo-area-messages).

If available, please a backtrace of the error here.
To acquire a backtrace, enable debug-on-error then recreate the error. Here
are ways to enable debug-on-error:

M-x toggle-debug-on-error,
Start Emacs with emacs --debug-init
If the error occurred while using bin/doom, use the -d/--debug
switches or the DEBUG environment variable.

Steps to reproduce

Select these example steps,
Delete them,
And replace them with precise steps to reproduce your issue.
Fill in "system information" below.

System information

Place the output of `M-x doom/info` or `~/.emacs.d/bin/doom info` here.

If you want to use package.el, it is on you to initialize package.el yourself. Doom doesn't use package.el, and intentionally avoids initializing it as such. Instead of installing your packages with it, I'd suggest using Doom's package manager, see: https://github.com/hlissner/doom-emacs/blob/develop/docs/faq.org#how-do-i-install-a-package-from-elpa

authors: hlissner kris-singh

repo: hlissner/doom-emacs


text:
Move away from obsolete org-roam function in org +roam

The command org-roam-show-graph was marked obsolete in favor of org-roam-graph-show by org-roam commit 22b9d4b (before org-roam's pinning at b86d2c8), see https://github.com/jethrokuan/org-roam/pull/363.

Thanks for catching that. It should be reported to org-roam that its docs are outdated, in that case.

authors: hlissner pnivlek


text:
Default install error on org-*

Hi @hlissner,
Emacs poweruser, Doom newbie, dropping my ten years custom emacs.d in favor of your awesome work. Just starting out. FYI, the first steps provides the following error:
No unused packages to remove
Looking for packages to install...
7 packages will be installed:

+ centered-window-mode (ELPA)
+ help-fns+ (ELPA)
+ ob-mongo (ELPA)
+ ob-redis (ELPA)
+ ob-translate (ELPA)
+ ox-pandoc (ELPA)
+ ox-reveal (ELPA)

Proceed? (y or n) y
Refreshing package archives
Contacting host: elpa.gnu.org:443
Contacting host: melpa.org:443
Package refresh done
Installing ox-reveal
  FATAL ERROR: (error) Package ‘org-20150330’ is unavailable
Installing centered-window-mode
  FATAL ERROR: (error) Package ‘centered-window-mode-’ is unavailable
Installing ox-pandoc
  FATAL ERROR: (error) Package ‘org-8.2’ is unavailable
Installing ob-translate
  FATAL ERROR: (error) Package ‘org-8’ is unavailable
Installing ob-redis
  FATAL ERROR: (error) Package ‘ob-redis-’ is unavailable
Installing ob-mongo
  FATAL ERROR: (error) Package ‘org-8’ is unavailable
Installing help-fns+
  FATAL ERROR: (error) Package ‘help-fns+-’ is unavailable
Finished!
Reloading active Emacs session...

Where are org-* package supposed to be fetched from ?

Wrong repo, sorry. Moving to emacs-doom.

@jwintz Use the develop branch. Many of these issues have been resolved there.

I'll try right now. Glad to join the community :-)

authors: hlissner jwintz

repo: hlissner/emacs-doom-themes


text:
Version calculation in setup.py isn't correct

Installing the latest version (0.2.0) results in a package name of:
django-ses-.join-str-x-.for.x.in.VERSION-
The following change in setup.py fixes the problem:
VERSION = '.'.join(version_line.split(' = ')[-1][1:-2].split(', '))

Thanks Tim. That didn't work in my tests either, since it needs to evaluate that line. I didn't want to spend hours finding a more elegant way so I just kept it clean, fixed here:
https://github.com/hmarr/django-ses/commit/e61eab66001d3bacab308451a73e38e413d08312
I don't install things from PyPI so I'm wondering if it needs to be fixed ASAP or it can wait. Is the install working?

The version I tested was downloaded from the github tag.  The pypi version claims to be 0.2 but was updated 2011-04-16: http://pypi.python.org/pypi/django-ses

Good point. Well, it should be fixed now.

authors: pcraciunoiu timgraham

repo: hmarr/django-ses


text:
CIV-10664 Add missing word in recital text

Before creating a pull request make sure that:

[ ] commit messages are meaningful and follow good commit message guidelines
[ ] README and other documentation has been updated / added (if needed)
[ ] tests have been updated / new tests has been added (if needed)

Please remove this line and everything above and fill the following sections:
JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/CIV-10664
Change description
Add missing word in judge recital text
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ ] No

repo: hmcts/civil-general-apps-ccd-definition


text:
Remove proxyout

Should be no longer required,
Announcement in #cloud-native-announce

Closed in favour of https://github.com/hmcts/pcq-frontend/pull/161

repo: hmcts/pcq-frontend


text:
No way to have a process accept stdin and utilize wait_timeout?

If stdin is provided (e.g. as a &str) to the builder api before a child is spawned via .popen(), a panic will be caused due to https://github.com/hniksic/rust-subprocess/commit/d36a5c4b3e38f5c09d271379c503effe2b45aeb8. However, if stdin is provided to the Popen object after it's been spawned via communicate_bytes, there's no way to utilize the timeout functionality provided by wait_timeout since communicate_bytes blocks. Any suggestions about how to provide stdin to a process and still be able to impose a timeout on it?

popen() rejects builders that specify stdin() because stdin() is intended for use together with capture_stdout() and similar, the convenience layer over communicate(). (The error message should probably make that clearer.) If you could call popen() and get a hold of the underlying Popen object, you'd have to call communicate() or communicate_bytes() yourself, defeating the purpose of stdin().
Ideally communicate_bytes(), and by extension capture_stdout and others, would support timeouts. Until then, you can send the data to the subprocess's input yourself. If the amount of data is small, less than the pipe buffer size (4K to 64K on Linux), you can just write the data to the process's stdin:
fn x() -> subprocess::Result<()> {
    let mut p = Exec::cmd("cat")
        .stdin(Redirection::Pipe)
        .popen()?;
    p.stdin.as_ref().unwrap().write(b"hello")?;
    println!("{:?}", p.wait_timeout(Duration::from_millis(500))?);
    p.terminate()?;
    Ok(())
}

If you need to send a larger amount of data, you'll need to ensure deadlock-free writes, e.g. by doing the writing in a separate thread:
fn x() -> subprocess::Result<()> {
    let mut p = Exec::cmd("cat")
        .stdin(Redirection::Pipe)
        .popen()?;
    let p_stdin = Arc::new(Mutex::new(p.stdin.take().unwrap()));
    // clone the Arc so stdin isn't dropped/closed when the thread exits
    let thread_stdin = p_stdin.clone();
    let thr: JoinHandle<io::Result<()>> = thread::spawn(move || {
        let mut sink = thread_stdin.lock().unwrap();
        sink.write(b"long data")?;
        Ok(())
    });
    thr.join().unwrap()?;
    println!("{:?}", p.wait_timeout(Duration::from_millis(500))?);
    p.terminate()?;
    Ok(())
}

@hniksic thanks for the answer, that works great for the timeout use case. I'm curious, though, if there's a single approach that'll work for both wait() and wait_timeout() (my use case is a function which spawns a process and optionally allows a timeout to be specified). As it stands, writing directly to stdin will leave the cat process waiting indefinitely for more input. Is there some way to close stdin?

Is there some way to close stdin?

The streams used by the subprocess crate are just Option<File>, and a File is automatically closed when dropped. (As you discovered, take() also works because it extracts the file from the Option and doesn't use it, which invokes Drop.) Since you actually use stdin, you could also write it like this:
    let mut p_stdin = p.stdin.take().unwrap();
    p_stdin.write(b"hello")?;
    drop(p_stdin);              // don't wait for end of scope to close stdin

The same goes for the threaded example, where I actually took care to clone the Arc (instead of just moving it to the thread) so that it doesn't close and the timeout can be demonstrated. Removing the clone() moves the Arc<Mutex<File>> to the thread and automatically closes it once the writing is done.

@wgreenberg It might interest you that, as of version 0.2.0, there is a variant of communicate that accepts timeout (as well as allocation limit on captured output), possibly simplifying usage in your case.

authors: hniksic wgreenberg

repo: hniksic/rust-subprocess


text:
Add setPort method

removes the socket bind
adds setPort method

We don't need the socket bind because it is only for incoming messages, outgoing messages always use all interfaces as far as i know.
With its removal we can add a setPort method because we don't access it in the setup phase any longer.
New PR so I can move this code into its own branch

I'm not sure about the .bind(). I think I remember that someone added this because it's needed when you use the broadcast address 255.255.255.255 for the ArtNet Node so you can choose from which interface the broadcasts are send. But I'm not sure and I can't test this, my ArtNet Node isn't working with broadcasts...

Okay, what about making the bind optional then? So you have a toggle
broadcast which defaults to true (so no breaking changes) and can be
disabled to enable the set port method?
On Thu, May 25, 2017, 19:54 Sebastian Raff notifications@github.com wrote:

I'm not sure about the .bind(). I think I remember that someone added this
because it's needed when you use the broadcast address 255.255.255.255 for
the ArtNet Node so you can choose from which interface the broadcasts are
send. But I'm not sure and I can't test this, my ArtNet Node isn't working
with broadcasts...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/hobbyquaker/artnet/pull/15#issuecomment-304077548,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABGaIdBV3j4UF25YuWxnSHHecNZa7luvks5r9cBdgaJpZM4NmMEL
.

--
Max Jöhnk
0170/2217608

sounds good :)

I've updated the pr with a config flag.
I'm throwing an error in setPort so users are not wondering why setPort isn't doing anything.

authors: hobbyquaker maxjoehnk

repo: hobbyquaker/artnet


text:
add tsv renderer/output

#3! :-) let me know!
I went with TSV as the data has commas, and github only deals with comma- and tab-separated files.
also: it would probably make sense to add a link from the readme to the .tsv file

I went with TSV as the data has commas, and github only deals with comma-, or tab-separated files.

Wow, didn't expect it that quickly! :)
Looks great!

authors: gregsadetsky hobofan

repo: hobofan/collected-robotic-arms


text:
Thank you! you save my project!

Running a humanoid skeleton with vmd at runtime was always the technology I wanted. You make it！

Make some suggestions and identify problems:

Rotation causes IK problem, which can be easily fixed by setting the rotation position before running, but with a lot of local and world coordinates mixed, I can't handle the problem of rotating IK at runtime.

2.list.find causes a lot of gc, you can write your own dichotomous algorithm and frame buffer repair (can done to 0 GC).

DefaultBoneAmplifier can be dynamically adjusted, with an approximate algorithm of character leg length/10.5.

Thank you, Personuo.
I am sorry that I did not notice your suggestions.

This seems to be a difficult problem. In MMD, any runtime operation is not expected.
A new algorithm may be necessary to solve it.

Your suggestion is totally right. I should have avoided List.find, and it should be rewritten.
Could you let me know whether or not "foreach and if" strategy can solve this problem?

Is it true? Could you let me know why it works?

I am afraid to say that I am too busy these days, so it may be difficult to fix many parts.

2.In my final solution, I use "for and if" to solve this problem.But, if you use for to retrieve from 0, a dance is generally 3 minutes, and when the time is after 2 minutes, the calculation is also very large,
3.I think how DefaultBoneAmplifier and leg length to maintain the same ratio, Ik will not lead to strange movements of the character, such as ik too far from the leg caused by leg stiffness，but this is not the perfect solution, it can not solve the leg penetration problem (The knees are not position in the same proportion)

authors: Personuo hobosore

repo: hobosore/UnityVMDPlayer


text:
Posynomial Inequalities not pickleable after 0.4

per @tonystao

@tonystao it's because of pint

I can make unitless posynomials / posynomial inequalities pickleable very easily, working on united ones...

possibly related to https://github.com/hgrecco/pint/issues/349

superseded by #927

authors: bqpd whoburg

repo: hoburg/gpkit


text:
no route to host

Hi,
Getting below message, firewall is disabled in host machine
docker@node2:~/kubernetes_the_easy_way$ kubectl get nodes
Unable to connect to the server: dial tcp 192.168.50.20:6443: getsockopt: no route to host
Regards,Vikas

Please paste the content of the file logs/vagrant-provition.log
and your config

Hi,
config file
docker@node2:~/kubernetes_the_easy_way$ more config
export ETCD_COUNT=1
export ETCD_CPU=1
export ETCD_MEM=512
export MASTER_COUNT=1
export MASTER_CPU=1
export MASTER_MEM=512
export WORKER_COUNT=2
export WORKER_CPU=1
export WORKER_MEM=512
export LOADBALANCER_COUNT=1
export LOADBALANCER_CPU=1
export LOADBALANCER_MEM=512
export EXTRA_HOSTS="
Add extra hosts entries to each node in the format of
192.168.1.148 nfs-server.example.com
vagrant.log
__Etcd ___________
| - Nodes  : 1
| - CPU    : 1
| - Memory : 512
__Masters ________
| - Nodes  : 1
| - CPU    : 1
| - Memory : 512
__Workers ________
| - Nodes  : 2
| - CPU    : 1
| - Memory : 512
__Loadbalancers __
| - Nodes  : 1
| - CPU    : 1
| - Memory : 512
Bringing machine 'k8s-etcd-1' up with 'virtualbox' provider...
Bringing machine 'k8s-master-1' up with 'virtualbox' provider...
Bringing machine 'k8s-worker-1' up with 'virtualbox' provider...
Bringing machine 'k8s-worker-2' up with 'virtualbox' provider...
Bringing machine 'k8s-loadbalancer-1' up with 'virtualbox' provider...
==> k8s-etcd-1: Checking if box 'bento/fedora-27' is up to date...
==> k8s-etcd-1: VirtualBox VM is already running.
==> k8s-master-1: Checking if box 'bento/fedora-27' is up to date...
==> k8s-master-1: VirtualBox VM is already running.
==> k8s-worker-1: Checking if box 'bento/fedora-27' is up to date...
==> k8s-worker-1: VirtualBox VM is already running.
==> k8s-worker-2: Checking if box 'bento/fedora-27' is up to date...
==> k8s-worker-2: VirtualBox VM is already running.
==> k8s-loadbalancer-1: Checking if box 'bento/fedora-27' is up to date...
==> k8s-loadbalancer-1: VirtualBox VM is already running.
Regards,Vikas

It looks like you have run the install script while the virtual machines were already running. This means that all log from when they were created is overwritten.
Run the destroy script, and the the install script.
And, I would give the workers a bit more memory in the config file.
Try that and post the log if it doesn't work.

@vikasgubbi did it work?

Hi,
install.sh script got struck at below message for more than 30 min,cancelled it and reran the script it went to success but when i did kubectl get nodes ,it errored out to no route to host.
regards,Vikas

==> k8s-loadbalancer-1: Digest: sha256:eb8eb1d83fc58716b20621d397d63737a18f86cbed1fedb1d71671cfc486517b
==> k8s-loadbalancer-1: Status: Downloaded newer image for weaveworks/weaveexec:2.3.0
==> k8s-loadbalancer-1: 08a2da4803cbb0da6cf51cd88147e948a1e7415cba226fb9de52a1d36492845e

You cannot rerun the install script. You need to run the destroy script first, always.
it looks like it doesnt get past the weavenet container.
I will see if i can find a way to debug it, and then get back to you

Hi,
can you share your email id, if possible please.
Regards,Vikas

Did you get it to work?

yes

authors: hoeghh vikasgubbi


text:
Update dialogue_box.tscn

Fix for issue #4
Imlementation:

Turned off the Clip Contents flag in the RichTextLabel used in the dialog box

Merged :smile:

authors: hohfchns ksmithdev

repo: hohfchns/DialogueQuest


text:
Added export to CSV feature

I have added the export to CSV feature so users will now be able to export data from the overview page to a CSV file.

I didn't realize this was up for re-review! @sameeramin, you'd need to rebase this against main so that you get the lockfile issue fixed. The latest .nvmrc file uses a newer NPM, so the lockfile version should be correct (i.e. 2) and the diff would not be -11k lines.

No problem @holmari I tried git rebase main but it says everything is up to date. Also, did you have a look at the above conversation in which I tried to type annotate the users array?

I finally looked at this again. @sameeramin I squashed your commits together, made some additional changes to simplify and merged this now. Thanks!

authors: holmari sameeramin

repo: holmari/repostats


text:
Overlaying annotations on target plot breaks RangeToolLink

ALL software version info
holonote 0.1.0
holoviews 1.18.1
Description of expected behavior and the observed behavior
Overlaying annotations on a target plot prevents the RangeToolLink in a source plot from updating the dimension in the unbounded dimension of the annotation. For instance, with x-range annotations, the y-dimension link is broken.
Complete, minimal, self-contained example code that reproduces the issue

 Code
import numpy as np
import holoviews as hv
from bokeh.models import HoverTool
from holoviews.plotting.links import RangeToolLink
from scipy.stats import zscore
from holoviews.operation.datashader import rasterize

hv.extension('bokeh')

N_CHANNELS = 10
N_SECONDS = 5
SAMPLING_RATE = 200
INIT_FREQ = 2  # Initial frequency in Hz
FREQ_INC = 5  # Frequency increment
AMPLITUDE = 1

# Generate time and channel labels
total_samples = N_SECONDS * SAMPLING_RATE
time = np.linspace(0, N_SECONDS, total_samples)
channels = [f'EEG {i}' for i in range(N_CHANNELS)]

# Generate sine wave data
data = np.array([AMPLITUDE * np.sin(2 * np.pi * (INIT_FREQ + i * FREQ_INC) * time)
                     for i in range(N_CHANNELS)])

hover = HoverTool(tooltips=[
    ("Channel", "@channel"),
    ("Time", "$x s"),
    ("Amplitude", "$y µV")
])

channel_curves = []
for channel, channel_data in zip(channels, data):
    ds = hv.Dataset((time, channel_data, channel), ["Time", "Amplitude", "channel"])
    curve = hv.Curve(ds, "Time", ["Amplitude", "channel"], label=channel)
    curve.opts(
        subcoordinate_y=True, color="black", line_width=1, tools=[hover],
    )
    channel_curves.append(curve)

eeg_curves = hv.Overlay(channel_curves, kdims="Channel")

annotator = Annotator({"Time": float}, fields=["category"])
annotations_df = pd.DataFrame({'start': [1], 'end': [2], 'category': ['demo']})
annotator.define_annotations(annotations_df, Time=("start", "end"))
annotations_overlay = annotator.get_element("Time")

eeg_app = (annotations_overlay * eeg_curves).opts(
    xlabel="Time (s)", ylabel="Channel", show_legend=False, aspect=3, responsive=True,
)

y_positions = range(N_CHANNELS)
yticks = [(i , ich) for i, ich in enumerate(channels)]

z_data = zscore(data, axis=1)

minimap = rasterize(hv.Image((time, y_positions , z_data), ["Time (s)", "Channel"], "Amplitude (uV)"))
minimap = minimap.opts(
    cmap="RdBu_r", xlabel='Time (s)', alpha=.5, yticks=[yticks[0], yticks[-1]],
    height=150, responsive=True, default_tools=[], clim=(-z_data.std(), z_data.std())
)

RangeToolLink(
    minimap, eeg_curves, axes=["x", "y"],
    boundsx=(None, 2), boundsy=(None, 6.5)
)

dashboard = (eeg_app + minimap * annotations_overlay).opts(merge_tools=False).cols(1)
dashboard

Screenshots or screencasts of the bug in action
https://github.com/holoviz/holonote/assets/6613202/4ad2e7e7-f9ed-4c51-b16b-200a1e0ee84b

Here is an MRE and some other examples that do not work. None of them is using HoloNote so I will move this issue to  HoloViews.
It seems like it is caused by RangeToolLink, subcoordinate_y, and DynamicMap.
import holoviews as hv
from holoviews.plotting.links import RangeToolLink
import pandas as pd

hv.extension("bokeh")

# Set up
curve_fn = lambda i: hv.Curve([i, i], kdims="Time", label=str(i)).opts(
    subcoordinate_y=True
)
target = hv.Overlay([curve_fn(1), curve_fn(2)])
source = hv.Curve([1, 2], kdims="Time")
RangeToolLink(source, target, axes=["x", "y"])

# What HoloNote does behind the scene
annotations_overlay = hv.DynamicMap(lambda: hv.VSpans(([0], [1])))
target_overlay = annotations_overlay * target

# Other example 1 (does not work)
annotations_overlay = hv.DynamicMap(lambda: hv.VSpans(([0], [1])))
target_overlay = target * annotations_overlay

# Curve example 1 (does not work)
annotations_overlay = hv.DynamicMap(lambda: hv.Curve([0, 1]))
target_overlay = annotations_overlay * target

# Curve example 2 (does not work)
annotations_overlay = hv.DynamicMap(lambda: hv.Curve([0, 1]))
target_overlay = target * annotations_overlay

# Curve example 3 (does not work and seems to remove the link)
annotations_overlay = hv.Curve([0, 1])
target_overlay = target * annotations_overlay

# Final
(target_overlay + source).opts(shared_axes=False).cols(1)

authors: Hoxbro droumis

repo: holoviz/holonote


text:
Add codespell to pre-commit

I am adding codespell to pre-commit and fix spelling mistakes.
https://www.grammarly.com/blog/everytime-every-time/

I'm separately working on upgrading parts of the docs, hope there won't be too many conflicts!

You can close this PR and make the changes on your branch if this makes your life easier

authors: Hoxbro maximlt

repo: holoviz/hvplot