FlowRouter.getQueryParams() suggestion

Hi again @arunoda !
So, I was using your package, and I came up with a really ugly solution for a problem, and I figured others might like to see an official solution!
Frankly, FlowRouter.watchPathChange() is overkill for my purposes. It exhibits very strange behavior.  but FlowRouter.getQueryParam('requiredArgument') is too little. I need to watch my path for 4 or 5 query params, but not all 20. In fact, watching all 20 wouldn't even be bad. But FlowRouter.watchPathChange() comes with even extra baggage and is less predictable.
Suggestion: FlowRouter.getQueryParams() called with no arguments, which returns the entire reactive queryParams object, or it can be given a list of params to watch One could still call it with a string argument to obtain the old behavior.
This is a function I am using:
getQueryParamsReactive = function(list){
    //list is an array of query params I want to "watch".
	return _.reduce(list, function(memo,query){
		var param = FlowRouter.getQueryParam(query);
		if (param){
			memo[query] = param;
		}
		return memo;
	},{})
}

Basically, it watches the path for changes only in the desired query params I care about, but it's kind of ugly.
Anyone else wanna see this feature? +1 this.
Thanks :D!

I think i'll keep this open, but I'm not going to implement this right now since there's a working solution for this.
In Flow, we are not going to implement all the features we need, but layout the foundation. May be you can build a new package which expose this functionality.

My solution resorts to calling FlowRouter.getqueryParam('param_k') for all N params - it just seems like there's a better way. i'll see if i cant find a good way to add onto FlowRouter object for a package!

I think your solutions seems good. I can't think of another way.

Closing this for now.


Operator-sdk help for Helm/Ansible shows that GO text

Bug Report
What did you do?
Run operator-sdk -h in a Helm/Ansible project (it is possible to use the testdata/ samples to check)
What did you expect to see?
A specific helper message according to the type
What did you see instead? Under which circumstances?
The GO help message.
Environment
Operator type:
/language ansible
/language helm

operator-sdk version: master (1cba71cfbc496879dce1ecc12ed421ba009d6f76)

Possible Solution
Update the CLI helper based on the type. See: internal/cmd/operator-sdk/cli/cli.go

Duplicate of #3798


txmgr: handle invalid SubmitPFB response when height is zero

There should be a sanity check on the SubmitPFB response to make sure we're not writing height = 0 as the frame reference in case of an error.

Closed in #157


(PDB-5135) notify when resources are oversized

This PR is closed because the termini implementation was preferred. PR https://github.com/puppetlabs/puppetdb/pull/3539


Fix spawning glass shard for each glass sheet in stack

About the PR
Fixed the issue of SpawnEntitiesBehavior not executing multiple times on entities with stack conponent.
Why / Balance
Stackable entities loose their spawn entity behaviour when there are multiple of them in single stack, which causes in case of https://github.com/space-wizards/space-station-14/issues/25287 to punish players that have not known about it by deleting resources.
Technical details
Now, Execute method loops its spawn code stack size times if the owner entity had StackComponent.
Media

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

Breaking changes
Now, Execute method loops its spawn code stack size times if the owner entity had StackComponent.
Changelog

:cl:

fix: Damaging stacked glass sheets now gives appropriate amount of glass shards

Does this cause 10 bananium stack to leave 10 peels too?


Hosts values not prepulated when editing

I am not sure if this is just a specific version of <IP_ADDRESS>5 or after it was upgraged, but when editing a content security policy, the original value is not prepopulated when clicking on a policy to edit.
To reproduce.  Add a host value to style-src, save it and then try to edit, it appears the other fields are populated with the original values you saved, but hosts are not.

@JoshuaFolkerts I will take a look


MAINT: Update meson build according to private API changes

See: https://github.com/rgommers/scipy/issues/36#issuecomment-957342970

Update for the following modules are completed:

[x] cluster
[x] constants
[x] fft
[x] fftpack
[x] integrate
[x] interpolate
[x] io
[x] linalg
[x] misc
[x] ndimage
[x] odr
[x] optimize
[x] signal
[x] sparse
[x]  spatial
[x] special
[x] stats

cc @rgommers

Cool, looks like a good start. The separate commits like MAINT: Update meson build files with private API  should be easy to review; when CI turns green we should be good here.

The test failures(scipy/sparse/linalg/_eigen/tests/test_svds.py) are related to: https://github.com/scipy/scipy/pull/14868#issuecomment-964444731

@rgommers I see the similar failures in scipy master too. There are no failures on scipy CI because the spare.linalg._eigen.tests is not installed. See: https://github.com/scipy/scipy/blob/master/scipy/sparse/linalg/_eigen/setup.py

There are no failures on scipy CI because the spare.linalg._eigen.tests is not installed. See: https://github.com/scipy/scipy/blob/master/scipy/sparse/linalg/_eigen/setup.py

Ouch! That's a good catch, explains the discrepancy.

So is this good to review/merge? If this eigen/tests thing is the only thing for getting CI green, then I think we can disable them here temporarily, until things are fixed in SciPy master.

Should I add it back in scipy?

So is this good to review/merge?

Yes, please!

Should I add it back in scipy?

Yes, that sounds good - but only if you can get a handle on the test failures, otherwise it's a one-line change that we cannot merge because all CI turns red. See https://github.com/scipy/scipy/pull/14868#issuecomment-964444731, it may be that before that PR things all worked. If so, you can see if it's easy to remove those tests as the last comment says, and otherwise revert that whole PR.

If this eigen/tests thing is the only thing for getting CI green, then I think we can disable them here temporarily, until things are fixed in SciPy master.

For this PR should we skip testing eigen.tests to check if CI turns green? Should I commit it with a note to remember reverting it back?

For this PR should we skip testing eigen.tests to check if CI turns green? Should I commit it with a note to remember reverting it back?

Yes. The easiest fix is to just not install tests I guess. xfail-ing will need to be done per tests otherwise.

All macOS failures are due to:
RuntimeWarning: invalid value encountered in reciprocal

just like before.

I see the similar failures in scipy master too. There are no failures on scipy CI because the spare.linalg._eigen.tests is not installed. See: https://github.com/scipy/scipy/blob/master/scipy/sparse/linalg/_eigen/setup.py

@rgommers, Should we open an issue in scipy so that we have a track?

Yes, please do!


taking a swing at adding things#list

Here's the /things/ route I alluded to in comments on #21 .

I tend to prefer the explicitness of the named params primarily for testing. But I have no problem reverting to the initial commit, especially if that's how the larger Ember community does it.

I've reverted back to using positional params.


Make Login Email Case Insensitive

Login email should be case insensitive.
Include tests with fix

We have code commented out that will do that, waiting for the end of this hackathon to deploy it

Is there a reason we can't deploy it now?

Couldn't deploy before due to some existing users who would be unable to login by the change without a db migration. We can now though


Try to hack-around the broken pdfjs-dist links, such that they will point to the main library releases (issue 10391)

I verified manually that this indeed fixes the problem. Thank you!


Can embedded html tags be interpreted

1st, thx for your lib !
In the RSS scraping I'm setting up, I'm stuck with one of my section  that html tagged. So instead of getting  well-formatted final text I get :

First paragraph

Second paragraph

and so on... All other sections are fine.
Based on samples I found, here's my source code
...
doc.Find("item").Each(
		func(index int, item *goquery.Selection) {
                       //
			titleTag := item.Find("title")
			title := titleTag.Text()

			descriptionTag := item.Find("description")
			description := html.UnescapeString((descriptionTag.Text())

			temp := Article{NumArticle: index, TitleArticle: title, ContentArticle: description}
			ArticlesPageData = append(ArticlesPageData, temp)
		})
               return ArticlePageData
....

I used UnescapeString() to get rid of specific HTML codes (as é or ’) and retrieve accented letters but all embedded HTML tags remains. Does goquery able to interpret them ?

Hello Eric,
Can you provide the full input text that you're parsing (the RSS document)? I suspect that the html is escaped in the RSS, and so once unescaped it (correctly) returns html content. You could get around that by re-parsing the text that is returned with goquery, and extract the text-only content that you want.
Also note that goquery is for parsing html, using it to parse arbitrary XML-based data may return weird results as some tags have specific meaning in html and the parser may apply html5-specified corrections.
Martin

Thank you Martin for your fast answer. Here's,for example, what RSS looks like:
<?xml version="1.0" encoding="utf-8"?>
<rss version="2.0" xml:base="http://www.montrealmatin.com"  xmlns:dc="http://purl.org/dc/elements/1.1/">
<channel>
 <title>Montreal Matin</title>
 <link>http://www.montrealmatin.com/rss.php</link>
 <description></description>
 <language>fr</language>
<item>
 <title>Apple devient la première entreprise privée à valoir 1.000 milliards de dollars en Bourse</title>
 <link>http://www.montrealmatin.com/article-383915-Apple-devient-la-premiere-entreprise-privee-a-valoir-1000-milliards-de-dollars-en-bourse.html</link>
 <description><p>Apple, le groupe am&eacute;ricain connu pour avoir r&eacute;volutionn&eacute; le monde de l'informatique avec ses t&eacute;l&eacute;phones et ses tablettes, est devenu jeudi la premi&egrave;re entreprise priv&eacute;e &agrave; valoir plus de 1.000 milliards de dollars en Bourse.&nbsp;<br><span style="color: #333333; font-family: Georgia, Arial, Helvetica, sans-serif; font-size: 18px; font-style: normal; font-variant-ligatures: normal; font-variant-caps: normal; font-weight: 400; letter-spacing: normal; orphans: 2; text-align: justify; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-text-stroke-width: 0px; background-color: #ffffff; text-decoration-style: initial; text-decoration-color: initial; display: inline !important; float: none;"><span><br></span></span></p><p style="text-align: center;"><iframe src="//www.dailymotion.com/embed/video/x6cbn9q?autoPlay=1&amp;mute=1" allowfullscreen="" allow="autoplay" width="770" height="420" frameborder="0"></iframe></p>
</description>
</item>
...
<item>
...
</item>
</channel>
</rss>

Eric

Merci Eric,
Indeed there is escaped html inside the <item><description> tag, so when you get the unescaped text (by calling Text()), it returns the content with html tags, that's the proper behaviour of Text().
What you can do then, since this is html, is to parse that content with goquery and retrieve only the text, e.g.:
        // "data" is the RSS document string
	doc, err := goquery.NewDocumentFromReader(strings.NewReader(data))
	if err != nil {
		log.Fatal(err)
	}
        // Find with a simplified selector string just for this example, you should still loop on item of course
	t := doc.Find("item description").Text()

	d2, err := goquery.NewDocumentFromReader(strings.NewReader(t))
	if err != nil {
		log.Fatal(err)
	}
	t2 := d2.Text()
	fmt.Println(t2)

That being said, for the RSS parsing you should definitely use an RSS parser instead (https://godoc.org/?q=rss), as I mentioned earlier goquery is an html parser and follows rules that are meant for html that may not apply or may do surprising things with RSS. The content of the description, however, can definitely be parsed with goquery.
Hope this helps!
Martin

Parsing twice is a solution not to see all these tags. But it then loose layout.... The ideal would be to make them interpreted...
Anyways, you pinpoint my mistake about parsing an RSS file.
Merci encore !
;)

No problem Eric. Sorry I misunderstood your issue, I thought you wanted to get the text without the html tags. I'm not sure I understand what you mean by interpreting them? This is html, so if you render them in a web browser, it will display as expected (without losing the layout) - you don't even have to unescape the string for accents and all, this is correctly encoded for html and will render as expected.
So basically if what you want is to parse RSS and render the descriptions in a web browser, you don't need goquery at all (unfortunately! :D just kidding, right tool for the job is the goal). You need an RSS/Atom parser, and then save the descriptions to be rendered at some point in a web browser (assuming that's what you want to do). Note that there are some security concerns with that (displaying arbitrary html from a source you don't control), but I don't know the context of your app, just keep that in mind if that's for anything more than hobby / experimentation!
Have fun!
Martin

Sorry for my english, Martin. You write:

"This is html, so if you render them in a web browser, it will display as expected"

Unfortunately not in my case  :( as tags are first encoded.
Chrome returns description section with all HTML tags obtained as-is from my code and not layout'd (not interpreted) by browser as I could expect..
Indeed, I summarize my issue for future readers. The line of my source code:
description := descriptionTag.Text()
took this description section of RSS source file:
<description><p>Apple, le groupe am&eacute;ricain connu pour avoir r&eacute;volutionn&eacute; le monde...
and rendered into this:

Apple, le groupe américain connu pour avoir révolutionné le monde...
I then added the UnescapeString() command :
description := html.UnescapeString((descriptionTag.Text())
to render accents properly:

Apple, le groupe américain connu pour avoir révolutionné le monde...
The next step would be to get these remaining tags being executed by brower in order to get a non-monolithic description section...
I targeted RSS thinking that it could easier my task. Maybe I need to fully rethink my approach.

Sorry, what I meant is that this (that you currently get via goquery):
description := descriptionTag.Text()

Gets you valid html, and that html can then be rendered by a browser properly (without the need to unescape description, as this is a correct way to encode accents and browsers will handle it).
E.g. if you save the string
Apple, le groupe américain connu pour avoir révolutionné le monde...
 into a file, and then use File -> Open File in your favorite browser, it will render as Apple, le groupe américain connu pour avoir révolutionné le monde... (inside a
 tag).
I will close the issue, since I don't think there's much more that involves goquery, but feel free to reopen (or open a new issue) if there's something else!
Thanks,
Martin


Avoid counting twice overlapping base calls from the reads that come from the same target.

The CalculateTargetCoverage tool can either count reads or base calls (then averaged across position in the target).
We need to add the option to avoid double counting base calls that come from the same molecule; that is when to read-mates overlap, the overlapped bases should only be counted once.

Closed by PR #365.


Address partiality of applyDistribLedgerTables

applyDistribLedgerTables is partial. If we try to project into Allegra tables that have entries for Alonzo it will fail. This won't happen in absence of consensus bugs, but we might want to make this explicit and fail the query instead of killing the node with an error.

I've been looking into this, but I haven't come up with a plan on how to address the partiality. For the time being, I don't think it's problematic to keep the code as is

Blocked on #261


Instana instrumentation code for sarama

Hi team,
We used to use this https://pkg.go.dev/github.com/instana/go-sensor/instrumentation/instasarama for the instana integration with our code, but now as the ownership has been changed for the sarama, how much time will we be able to get the updated one?
Any idea? Any thoughts?

I'm guessing this will be handled by that team under https://github.com/instana/go-sensor/issues/447 — assuming that they're happy to only support 1.40 and newer going forwards, then it would be a simple matter of sed -e 's|/Shopify/sarama|/IBM/sarama|' -i on all files


Extension - Re-check bug do not show root screen after remove account

Describe the task
Re-check bug do not show root screen after remove account
To reproduce the task
Steps:

Open the SubWallet extension expand view
Perform remove any account
Observe the screen displayed

Actual: Do not show black screen with URL chrome-extension://chlkaejaldjknaipdoljkebfknendkmh/index.html#/

Expect: Navigate to the Home screen

Version test: http://<IP_ADDRESS>:8900/s/PaCYMmkCd9SmFsZ

[ ] Still reproduce the bug
Browser: Brave

Version test: http://<IP_ADDRESS>:8900/s/QLTCTMCS5ErS2T7
Validation passed.
Checklist: https://www.notion.so/subwallet/Checklist-theo-issue-13c9e3215ec24762973dc680aec67198?pvs=4#536db033a637425dae96c35330b6e8d7


List transitive dependencies with exact versions.

Exact requirements generated by pip freeze.
Fixes #108

Excellent, thanks!

Ignore CI


Support for custom comparator function

As far as I can tell, there is not support for setting a custom key comparator via the function call
int mdb_set_compare	(	MDB_txn * 	txn, MDB_dbi 	dbi, MDB_cmp_func * 	cmp  )
Is this correct ? I understand that callback functions suffer from big performance problems, but not sure if that's the reason for not supporting it.
Any hint how could I use a custom comparator ?
Thanks,
Luis

Sorry for noticing this after such a long time. Is it still of interest?

@shmul I would definitely like to see this implemented. It would be awesome if you could!

What is the usage scenario? The performance implications can be quite noticeable which makes me wonder whether it will be of real use.

Well I store custom data structure inside lmdb, and I need to iterate
records in a custom sorted manner. If it has a performance impact, I think
it might be best if I wrap it using luajit's ffi.
On May 23, 2017 3:24 PM, "Shmulik Regev"<EMAIL_ADDRESS>wrote:

What is the usage scenario? The performance implications can be quite
noticeable which makes me wonder whether it will be of real use.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/shmul/lightningmdb/issues/14#issuecomment-303349749,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAaAK4sVs-Thg766Im14K4Nyzrtss0Jlks5r8qzDgaJpZM4LFYzS
.


editor - use bulk descriptor update API

This is a...

[x] Feature request
[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report
[ ] Documentation issue or request

Description
When adding the split step we had to add a backend API so that the UI could fetch updated data shape info for the split step.  Since then a bulk descriptor update operation has been introduced, we should port the changes that were done to the angular app to the react editor.  The PR where this was done is here and the original bulk API was added here

@christophd if you fancy getting familiar with the react codebase this could be a gateway :-)

My first inclination would be to add a function to call the bulk api from this file and then actually make that call from before here and before here


Add SetPixel as separate r g b values

Description

Improve SetPixel implementation to have separate RGB values.

Motivation and Context
How Has This Been Tested?

Screenshots
Types of changes

[x] Improvement (non-breaking change that improves a feature, code or algorithm)
[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] Config and build (change in the configuration and build system, has no impact on code or features)
[x] Dependencies (update dependencies and changes associated, has no impact on code or features)
[ ] Unit Tests (work on Unit Tests, has no impact on code or features)
[ ] Documentation (changes or updates in the documentation, has no impact on code or features)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

 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.:x: LwServices sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@LwServices don't mind the last message from nfbot about the build failing. It was a glitch with one of the build tools.

@LwServices please rebase your PR and fix the merge conflicts with the versions.

@josesimoes rebase is done

@Ellerbach are you good with this PR?


Upgrade CMake to version: 3.23.3

As part of upgrading the downstream libraries in K-NN to add the support for filtering[https://github.com/opensearch-project/k-NN/pull/942], we need to upgrade the CMake version otherwise builds for the K-NN Native Libraries will start to fail. The ask for the issue to upgrade the Cmake version to 3.23.3.
Places to change:
A simple search of Cmake in build repo these many places where changes needs to be done: https://github.com/search?q=repo%3Aopensearch-project%2Fopensearch-build cmake&type=code
Can you provide what is the best way to make these changes? Can this change be picked by the Build team?
Once the CMake is upgraded I will start to merge the PR in 2.x branch otherwise the build for 2.x will start to fail

@peterzhuamazon will take a look this , most likey it may be pushed to after 2.9.0, but Peter will provide more insights of it .. thanks ... here is a link to it for the reference .. https://github.com/opensearch-project/opensearch-build/issues/1563.

@jordarlu this needs to go in 2.9. There are changes in k-NN plugin that requires this.

The code change from k-nn is merged in 2.x branch. The build might start to fail now. Can we prioritize the 2.9 release distributions first

Hi @navneet1v this is a last minute change and we are working on it already.
Thanks.

PRs:

https://github.com/opensearch-project/opensearch-build/pull/3722

This task is complete by now.
Thanks.


How to decouple grpc folder for other projects?

Hi,all! Thanks for the repo, it make me learn scalable rl more efficiently!
I want to only use the grpc for my projects now, i.e. i want to compile grpc ops for transform TF Tensors. How to decouple the grpc folder and only compile it in my project.
Thanks a lot~

You should be able to just copy the grpc/ directory and use that. It already has a compiled .so file, but you can compile it on your own with this docker file:
https://github.com/google-research/seed_rl/blob/master/docker/Dockerfile.grpc

Hi, @lespeholt
Thanks for your reply, i will have a try with that~

Hi, @lespeholt
I have tried to build from https://github.com/google-research/seed_rl/blob/master/docker/Dockerfile.grpc. As for my project, I need to use the latest nightly tensorflow version, so i modified the version of tensorflow from tensorflow-gpu==2.1.0 to tf-nightly-gpu in the dockerfile(I also changed the base image to tensorflow/tensorflow:nightly-custom-op-gpu-ubuntu16). But I met the error when i build the image which show as follows:
Extracting Bazel installation...
Starting local Bazel server and connecting to it...
Loading:
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
INFO: SHA256 (https://github.com/grpc/grpc/archive/ac1c5de1b36da4a1e3d72ca40b0e43f24266121a.tar.gz) = 031c9dfca989eee2b60d65ef0ff24c706e198ac6a09e58a22bf84d77b1948cb8
Loading: 0 packages loaded
Loading: 0 packages loaded
DEBUG: Rule 'com_github_grpc_grpc' indicated that a canonical reproducible form can be obtained by modifying arguments sha256 = "031c9dfca989eee2b60d65ef0ff24c706e198ac6a09e58a22bf84d77b1948cb8"
DEBUG: Call stack for the definition of repository 'com_github_grpc_grpc' which is a http_archive (rule definition at /root/.cache/bazel/_bazel_root/731c0d19406ca90b97ddc93a2cc43eaf/external/bazel_tools/tools/build_defs/repo/http.bzl:296:16):
 - <builtin>
 - /custom-op/WORKSPACE:9:1
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
    currently loading: grpc
Analyzing: 2 targets (1 packages loaded, 0 targets configured)
Analyzing: 2 targets (12 packages loaded, 84 targets configured)
Analyzing: 2 targets (13 packages loaded, 87 targets configured)
Analyzing: 2 targets (13 packages loaded, 87 targets configured)
Analyzing: 2 targets (13 packages loaded, 87 targets configured)
Analyzing: 2 targets (13 packages loaded, 87 targets configured)
Analyzing: 2 targets (13 packages loaded, 87 targets configured)
Analyzing: 2 targets (13 packages loaded, 87 targets configured)
WARNING: /root/.cache/bazel/_bazel_root/731c0d19406ca90b97ddc93a2cc43eaf/external/local_config_tf/BUILD:5697:1: target 'libtensorflow_framework.so' is both a rule and a file; please choose another name for the rule
Analyzing: 2 targets (24 packages loaded, 742 targets configured)
Analyzing: 2 targets (24 packages loaded, 742 targets configured)
Analyzing: 2 targets (24 packages loaded, 742 targets configured)
Analyzing: 2 targets (24 packages loaded, 742 targets configured)
Analyzing: 2 targets (24 packages loaded, 742 targets configured)
INFO: Call stack for the definition of repository 'boringssl' which is a http_archive (rule definition at /root/.cache/bazel/_bazel_root/731c0d19406ca90b97ddc93a2cc43eaf/external/bazel_tools/tools/build_defs/repo/http.bzl:296:16):
 - <builtin>
 - /root/.cache/bazel/_bazel_root/731c0d19406ca90b97ddc93a2cc43eaf/external/com_github_grpc_grpc/bazel/grpc_deps.bzl:100:9
 - /custom-op/WORKSPACE:18:1
WARNING: Download from https://boringssl.googlesource.com/boringssl/+archive/83da28a68f32023fd3b95a8ae94991a07b1f6c62.tar.gz failed: class java.io.IOException connect timed out
ERROR: An error occurred during the fetch of repository 'boringssl':
   java.io.IOException: Error downloading [https://boringssl.googlesource.com/boringssl/+archive/83da28a68f32023fd3b95a8ae94991a07b1f6c62.tar.gz] to /root/.cache/bazel/_bazel_root/731c0d19406ca90b97ddc93a2cc43eaf/external/boringssl/83da28a68f32023fd3b95a8ae94991a07b1f6c62.tar.gz: connect timed out
ERROR: /custom-op/grpc/BUILD.bazel:4:1: //grpc:ops/grpc.so depends on @boringssl//:ssl in repository @boringssl which failed to fetch. no such package '@boringssl//': java.io.IOException: Error downloading [https://boringssl.googlesource.com/boringssl/+archive/83da28a68f32023fd3b95a8ae94991a07b1f6c62.tar.gz] to /root/.cache/bazel/_bazel_root/731c0d19406ca90b97ddc93a2cc43eaf/external/boringssl/83da28a68f32023fd3b95a8ae94991a07b1f6c62.tar.gz: connect timed out
ERROR: /custom-op/grpc/BUILD.bazel:4:1: //grpc:ops/grpc.so depends on @boringssl//:crypto in repository @boringssl which failed to fetch. no such package '@boringssl//': java.io.IOException: Error downloading [https://boringssl.googlesource.com/boringssl/+archive/83da28a68f32023fd3b95a8ae94991a07b1f6c62.tar.gz] to /root/.cache/bazel/_bazel_root/731c0d19406ca90b97ddc93a2cc43eaf/external/boringssl/83da28a68f32023fd3b95a8ae94991a07b1f6c62.tar.gz: connect timed out
ERROR: Analysis of target '//grpc:ops/grpc.so' failed; build aborted: no such package '@boringssl//': java.io.IOException: Error downloading [https://boringssl.googlesource.com/boringssl/+archive/83da28a68f32023fd3b95a8ae94991a07b1f6c62.tar.gz] to /root/.cache/bazel/_bazel_root/731c0d19406ca90b97ddc93a2cc43eaf/external/boringssl/83da28a68f32023fd3b95a8ae94991a07b1f6c62.tar.gz: connect timed out
INFO: Elapsed time: 97.183s
INFO: 0 processes.
 Iam looking forward to your reply, thank you very much!


Clicking on VALID button in swagger UI-viewer makes page blank

When you click on the lower right button, the Swagger UI-viewer section becomes completely blank. Only page refresh gets docs back.

@shaliko Here is one of the cases where we run into trouble with http. If I press "Valid" in Swagger UI in nightly2, I get a blank page (as described above) and the following error in console:
Mixed Content: The page at ... was loaded over HTTPS, but requested an insecure resource 'http://online.swagger.io/validator/debug?url=https://nightly2.apinf.io/gridfs/fs/id/0ea6886fd4a9dfbbc2a6b163'. This request has been blocked; the content must be served over HTTPS.
Is there a possibility to fix this on server side for Swagger validator? (See also #1369 for Swagger viewer itself)

@bajiat The issue in swagger-ui.js file https://nightly2.apinf.io/swagger/swagger-ui.js.
      // Default validator
      if(window.location.protocol === 'https') {
        this.model.validatorUrl = 'https://online.swagger.io/validator';
      }
      else {
        this.model.validatorUrl = 'http://online.swagger.io/validator';
      }

window.location.protocol will return "https:" instead of "https". https://developer.mozilla.org/en-US/docs/Web/API/HTMLHyperlinkElementUtils/protocol
As I can see swagger-ui have that issue in version version v2.1.0, that currently on nightly2.apinf.io. And we already updated swagger-ui in main repo of https://raw.githubusercontent.com/apinf/api-umbrella-dashboard/develop/public/swagger/swagger-ui.js on version v2.1.5 that dose not have that issue.

@bajiat Please create new release with that commit https://github.com/apinf/api-umbrella-dashboard/commit/580df6599b9ef94f1a42c0770991d66ac1924afc and I will create new docker image for nightly2.

@shaliko we will create a new release on Tuesday, we will ping you then

@bajiat How should works "Clicking on VALID button"? Now after fix issue with mixed content and click on VALID button, browser open tab and show me just empty JSON response

@shaliko Tested this by adding a new Swagger and I get the same response. Should be ok. Closing the issue.


Support .ttc files

TTC files are TrueType Collection file, which contains several TrueType fonts.
It would be best if the support of .ttc files can be added.

I will look into it if you can send me a file to test with

How about using this: https://github.com/be5invis/Sarasa-Gothic/releases
It might be too large for me to send a TTC file, sorry for the inconvenience.


Uncaught TypeError: Arguments to path.join must be strings

[Enter steps to reproduce below:]

...
...

Atom Version: 0.174.0
System: Mac OS X 10.10.2
Thrown From: linter package, v0.9.1
Stack Trace
Uncaught TypeError: Arguments to path.join must be strings
At path.js:390

TypeError: Arguments to path.join must be strings
  at Object.exports.join (path.js:390:15)
  at LinterJshint.Linter.getNodeExecutablePath (/Users/jakedanforth/.atom/packages/linter/lib/linter.coffee:114:10)
  at LinterJshint.Linter.getCmdAndArgs (/Users/jakedanforth/.atom/packages/linter/lib/linter.coffee:90:25)
  at LinterJshint.Linter.lintFile (/Users/jakedanforth/.atom/packages/linter/lib/linter.coffee:122:24)
  at /Users/jakedanforth/.atom/packages/linter/lib/linter-view.coffee:138:18
  at Array.forEach (native)
  at /Users/jakedanforth/.atom/packages/linter/lib/linter-view.coffee:137:18
  at Object.oncomplete (fs.js:93:15)

Commands

Config
{
  "core": {
    "disabledPackages": [
      "relative-numbers",
      "welcome"
    ],
    "themes": [
      "unity-ui",
      "monokai"
    ]
  },
  "linter": {}
}

Installed Packages
# User
autocomplete-plus, v1.1.0
color-picker, v1.2.6
highlight-selected, v0.7.0
linter, v0.9.1
linter-jshint, v0.1.0
monokai, v0.9.0
unity-ui, v1.0.7
vim-mode, v0.26.0

# Dev
No dev packages

/cc @atom/core

Closing as dupe of #329. This is already fixed, please upgrade! :smile:


[BUG] - Keycloak does not start on the first try

Describe the bug
Locally I always have to restart keycloak after reboot. I see the following:
Connecting to github.com (<IP_ADDRESS>:443)
wget: note: TLS certificate validation not implemented
Connecting to objects.githubusercontent.com (<IP_ADDRESS>:443)
wget: can't open '/data/keycloak-metrics-spi-2.5.3.jar': File exists
Stream closed EOF for dev/keycloak-0 (initialize-spi-metrics-jar)

https://github.com/nebari-dev/nebari/blob/200099631c2f25e9d0eaa0a23b62df4cf6597651/src/_nebari/stages/kubernetes_keycloak/template/modules/kubernetes/keycloak-helm/values.yaml#L26-L40
This code was most recently modified in https://github.com/nebari-dev/nebari/pull/1810 (CC @Adam-D-Lewis)
It also means that the jupyterhub pod does not start up: https://github.com/nebari-dev/nebari/issues/2547
Expected behavior
Keycloak starts on the first try
OS and architecture in which you are running Nebari
Ubuntu
How to Reproduce the problem?

Deploy locally on Ubuntu
Reboot
See that keycloak is not up

Command output
No response
Versions and dependencies used.
No response
Compute environment
None
Integrations
No response
Anything else?
No response

Locally I always have to restart keycloak after reboot.

I'm trying to understand the issue.  I guess when you reboot, the k8s pods restart and run the init containers again, but somehow the file system of the keycloak pod is from the prior run?  I'll test and see if it happens to me.

I am also not entirely sure; in theory, unless the keycloak metadata for the Kubernetes resource has changed, it should not restart. @krassowski why do you need it to restart? Also, because the helm provider handles this, some changes made to the values.yaml, do not trigger a change in the pod. This is a bug with the provider itself, as it can be seen in some other cases (I think this is related to how the provider calculates the diff)

I need to restart it because when I boot my laptop it is always dead, I do not recall if it stuck in a restart loop or not but simply put nebari does not work until I get to k9s and restart it manually. I applied @Adam-D-Lewis's patch and will let you know if this helped or not as soon as I restart my laptop (but I think that patch is useful anyways because even if it is not the root cause, the logs will now be cleaner).

Also, because the helm provider handles this, some changes made to the values.yaml, do not trigger a change in the pod. This is a bug with the provider itself, as it can be seen in some other cases (I think this is related to how the provider calculates the diff)

I am not sure if I follow you here @viniciusdc, my problem is not during redeployment but after rebooting.

This is my k9s when it fails:


Fix to work with Middleman4

middleman-inliner does not work with Middleman 4.x because they changed the interface by which an extension registers itself, but the change is trivial.
I also removed the bit in the constructor that references app.compass_config because I wasn't using Compass, and it did not seem to be required for Inliner to work.  If this is still necessary to work with projects using Compass, then I apologise for ripping it out, and I would like to request that Compass is an optional dependency.
Finally, I suspect these changes, though minor, will break compatibility with Middleman 3, but I'm not sure how you would want to handle that (maybe it would be appropriate to bump the major version number of Inliner?).

can we merge this?


Add built_in_css to blueprint to enable configuration save in admin p…

Hi,
Found a "bug", built_in_css must be in blueprint to save the configuration via admin panel.
Cheers

Hi, thanks to the PR but i preferred add the built_in_css immediately after enabled because they are the first original parameters used by rhukster
Greetings

No pb :)


Removed duplicate manifest for CoreTemp

Microsoft Reviewers: Open in CodeFlow

 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.:x: VIKTORVAV99 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


ValueError: Found array with 0 sample(s) (shape=(0, 1)) while a minimum of 1 is required by check_pairwise_arrays.

multiprocess.pool.RemoteTraceback:
"""
Traceback (most recent call last):
  File "/usr/local/lib/python3.8/dist-packages/multiprocess/pool.py", line 125, in worker
    result = (True, func(*args, **kwds))
  File "/usr/local/lib/python3.8/dist-packages/tuneta/optimize.py", line 240, in fit
    ke.fit(correlations)
  File "/usr/local/lib/python3.8/dist-packages/yellowbrick/cluster/elbow.py", line 316, in fit
    self.k_scores_.append(self.scoring_metric(X, self.estimator.labels_))
  File "/usr/local/lib/python3.8/dist-packages/yellowbrick/cluster/elbow.py", line 104, in distortion_score
    distances = pairwise_distances(instances, center, metric=metric)
  File "/usr/local/lib/python3.8/dist-packages/sklearn/metrics/pairwise.py", line 1884, in pairwise_distances
    return _parallel_pairwise(X, Y, func, n_jobs, **kwds)
  File "/usr/local/lib/python3.8/dist-packages/sklearn/metrics/pairwise.py", line 1425, in _parallel_pairwise
    return func(X, Y, **kwds)
  File "/usr/local/lib/python3.8/dist-packages/sklearn/metrics/pairwise.py", line 299, in euclidean_distances
    X, Y = check_pairwise_arrays(X, Y)
  File "/usr/local/lib/python3.8/dist-packages/sklearn/metrics/pairwise.py", line 156, in check_pairwise_arrays
    X = check_array(
  File "/usr/local/lib/python3.8/dist-packages/sklearn/utils/validation.py", line 797, in check_array
    raise ValueError(
ValueError: Found array with 0 sample(s) (shape=(0, 1)) while a minimum of 1 is required by check_pairwise_arrays.
"""

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "tuneta_opti.py", line 29, in <module>
    tt.fit(X_train, y_train,
  File "/usr/local/lib/python3.8/dist-packages/tuneta/tune_ta.py", line 137, in fit
    self.fitted = [fit.get() for fit in self.fitted]
  File "/usr/local/lib/python3.8/dist-packages/tuneta/tune_ta.py", line 137, in <listcomp>
    self.fitted = [fit.get() for fit in self.fitted]
  File "/usr/local/lib/python3.8/dist-packages/multiprocess/pool.py", line 771, in get
    raise self._value
ValueError: Found array with 0 sample(s) (shape=(0, 1)) while a minimum of 1 is required by check_pairwise_arrays.

Went smoothly before that until:
[I 2021-11-09 11:17:59,067] Trial 71 finished with value: 0.15180702108510608 and parameters: {'length': 40, 'atr_length': 8}. Best is trial 22 with value: 0.15180702108510608.
[I 2021-11-09 11:17:59,091] A new study created in memory with name: pta.kc(X.high, X.low, X.close, length=trial.suggest_int('length', 2, 48), scalar=trial.suggest_int('scalar', 2, 48), )
[I 2021-11-09 11:17:59,115] Trial 72 finished with value: 0.1516975180327597 and parameters: {'length': 38, 'atr_length': 13}. Best is trial 22 with value: 0.15180702108510608.
[I 2021-11-09 11:17:59,162] Trial 0 finished with value: 0.20201019266930986 and parameters: {'length': 34, 'scalar': 15}. Best is trial 0 with value: 0.20201019266930986.

So might have been caused by pta.kc
Any ideas how to solve this?
Thank you for this great package!

Thanks for reporting the error.  This is a new one that I haven't seen before and can't replicate.  Could you provide me the code you are using as well as the output of the following from your environment:
from importlib.metadata import version
print(f"Pandas-TA: {version('pandas-ta')}")
print(f"FinTa: {version('finta')}")
print(f"Ta-Lib: {version('ta-lib')}")
print(f"Pathos: {version('pathos')}")
print(f"Tabulate: {version('tabulate')}")
print(f"Dcor: {version('dcor')}")
print(f"yFinance: {version('yfinance')}")
print(f"Optuna: {version('optuna')}")
print(f"Yellowbrick: {version('yellowbrick')}")
print(f"Sckit-Learn: {version('scikit-learn')}")
print(f"Pandas: {version('pandas')}")
print(f"Numpy: {version('numpy')}")
print(f"Numba: {version('numba')}")

I used 'all'. And it happens later of the optimization. Maybe RAM?

I think it's the same:
https://github.com/jmrichardson/tuneta/pull/11#issuecomment-933037167

Hmmmm... I've replicated your environment and was unable to replicate.  Could you run this code and test on your end:
from tuneta.tune_ta import TuneTA
import pandas as pd
from pandas_ta import percent_return
from sklearn.model_selection import train_test_split
import yfinance as yf

if __name__ == "__main__":
    # Download data set from yahoo, calculate next day return and split into train and test
    X = yf.download("AAPL", period="10y", interval="1d", auto_adjust=True)
    y = percent_return(X.Close, offset=-1)
    X_train, X_test, y_train, y_test = train_test_split(X, y, test_size=.3, shuffle=False)

    # Initialize with x cores and show trial results
    tt = TuneTA(n_jobs=7, verbose=True)

    # Optimize indicators
    tt.fit(X_train, y_train,
        indicators=['all'],
        ranges=[(2, 100)],
        trials=30,
        early_stop=10,
    )

I am wondering if it has something to do with the data?  If the above works, then perhaps you could send me your X_train, y_train?

This might be it. I use a custom data source.
I pickled it. You should be able to load it with this, if you are using python 3.8, too. :
    with open('./candles.pkl', 'rb') as f:
        X = pickle.load(f)

Thank you for this incredible support!
Just fired up a test with yfinance. Will report back if that works.

Strange yfinance doesn't work too:
[I 2021-11-09 17:15:34,498] A new study created in memory with name: tta.DX(X.high, X.low, X.close, timeperiod=trial.suggest_int('timeperiod', 2, 100), )
[I 2021-11-09 17:15:34,500] Trial 14 finished with value: 0.11147144074882838 and parameters: {'timeperiod': 25}. Best is trial 13 with value: 0.11913746577305458.
.RemoteTraceback[I 2021-11-09 17:15:34,510] Trial 11 finished with value: 0.11583889796679482 and parameters: {'timeperiod': 12}. Best is trial 1 with value: 0.13536161141313785.
:
"""
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/multiprocess/pool.py", line 125, in worker
    result = (True, func(*args, **kwds))
  File "/home/src/tuneta/tuneta/optimize.py", line 240, in fit
    ke.fit(correlations)
  File "/usr/local/lib/python3.9/site-packages/yellowbrick/cluster/elbow.py", line 316, in fit
    self.k_scores_.append(self.scoring_metric(X, self.estimator.labels_))
  File "/usr/local/lib/python3.9/site-packages/yellowbrick/cluster/elbow.py", line 104, in distortion_score
    distances = pairwise_distances(instances, center, metric=metric)
  File "/usr/local/lib/python3.9/site-packages/sklearn/utils/validation.py", line 63, in inner_f
    return f(*args, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/sklearn/metrics/pairwise.py", line 1790, in pairwise_distances
    return _parallel_pairwise(X, Y, func, n_jobs, **kwds)
  File "/usr/local/lib/python3.9/site-packages/sklearn/metrics/pairwise.py", line 1359, in _parallel_pairwise
    return func(X, Y, **kwds)
  File "/usr/local/lib/python3.9/site-packages/sklearn/utils/validation.py", line 63, in inner_f
    return f(*args, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/sklearn/metrics/pairwise.py", line 272, in euclidean_distances
    X, Y = check_pairwise_arrays(X, Y)
  File "/usr/local/lib/python3.9/site-packages/sklearn/utils/validation.py", line 63, in inner_f
    return f(*args, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/sklearn/metrics/pairwise.py", line 146, in check_pairwise_arrays
    X = check_array(X, accept_sparse=accept_sparse, dtype=dtype,
  File "/usr/local/lib/python3.9/site-packages/sklearn/utils/validation.py", line 63, in inner_f
    return f(*args, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/sklearn/utils/validation.py", line 726, in check_array
    raise ValueError("Found array with %d sample(s) (shape=%s) while a"
ValueError: Found array with 0 sample(s) (shape=(0, 1)) while a minimum of 1 is required by check_pairwise_arrays.
"""

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/live/tuneta_opti.py", line 38, in <module>
[I 2021-11-09 17:15:34,514] Trial 16 finished with value: 0.09896255266983982 and parameters: {'timeperiod': 19}. Best is trial 16 with value: 0.09896255266983982.
[I 2021-11-09 17:15:34,521] Trial 34 finished with value: 0.06372177855344523 and parameters: {'timeperiod': 48}. Best is trial 4 with value: 0.09774473093630688.
[I 2021-11-09 17:15:34,521] Trial 0 finished with value: 0.04227110044971143 and parameters: {'timeperiod': 70}. Best is trial 0 with value: 0.04227110044971143.
[I 2021-11-09 17:15:34,522] Trial 10 finished with value: 0.13836197640888684 and parameters: {'timeperiod': 10}. Best is trial 10 with value: 0.13836197640888684.
    tt.fit(X_train, y_train,
  File "/home/src/tuneta/tuneta/tune_ta.py", line 137, in fit
[I 2021-11-09 17:15:34,528] Trial 18 finished with value: 0.08295114460901958 and parameters: {'fastperiod': 30, 'slowperiod': 26}. Best is trial 10 with value: 0.11425592453571463.
[I 2021-11-09 17:15:34,529] Trial 15 finished with value: 0.1155967683245075 and parameters: {'timeperiod': 24}. Best is trial 13 with value: 0.11913746577305458.
[I 2021-11-09 17:15:34,529] Trial 70 finished with value: 0.05935943353020685 and parameters: {'timeperiod': 8}. Best is trial 13 with value: 0.062016877300818736.
[I 2021-11-09 17:15:34,530] Trial 36 finished with value: 0.05644099122647902 and parameters: {'timeperiod': 24}. Best is trial 11 with value: 0.0974994756356342.
[I 2021-11-09 17:15:34,531] Trial 40 finished with value: 0.055003654253323424 and parameters: {'timeperiod': 38}. Best is trial 11 with value: 0.06219734835957756.
    self.fitted = [fit.get() for fit in self.fitted]
  File "/home/src/tuneta/tuneta/tune_ta.py", line 137, in <listcomp>
[I 2021-11-09 17:15:34,537] Trial 12 finished with value: 0.1364460842741541 and parameters: {'timeperiod': 27}. Best is trial 12 with value: 0.1364460842741541.
[I 2021-11-09 17:15:34,540] A new study created in memory with name: tta.MACD(X.close, fastperiod=trial.suggest_int('fastperiod', 2, 100), slowperiod=trial.suggest_int('slowperiod', 2, 100), signalperiod=trial.suggest_int('signalperiod', 2, 100), )
[I 2021-11-09 17:15:34,548] Trial 17 finished with value: 0.09248373225478898 and parameters: {'timeperiod': 15}. Best is trial 16 with value: 0.09896255266983982.
[I 2021-11-09 17:15:34,557] Trial 11 finished with value: 0.1164897633704645 and parameters: {'timeperiod': 3}. Best is trial 10 with value: 0.13836197640888684.
[I 2021-11-09 17:15:34,562] Trial 35 finished with value: 0.0675826808773287 and parameters: {'timeperiod': 33}. Best is trial 4 with value: 0.09774473093630688.
[I 2021-11-09 17:15:34,563] Trial 16 finished with value: 0.11807535107417427 and parameters: {'timeperiod': 19}. Best is trial 13 with value: 0.11913746577305458.
[I 2021-11-09 17:15:34,564] Trial 1 finished with value: 0.05137525234234987 and parameters: {'timeperiod': 30}. Best is trial 1 with value: 0.05137525234234987.
[I 2021-11-09 17:15:34,565] Trial 37 finished with value: 0.0801308186933188 and parameters: {'timeperiod': 81}. Best is trial 11 with value: 0.0974994756356342.
    self.fitted = [fit.get() for fit in self.fitted]
  File "/usr/local/lib/python3.9/site-packages/multiprocess/pool.py", line 771, in get
    raise self._value
ValueError: Found array with 0 sample(s) (shape=(0, 1)) while a minimum of 1 is required by check_pairwise_arrays.

Had trials=500,  early_stop=100, - with your 30 and 10 it works now.

Ha that is interesting, your data works fine for me (ran twice without issue):

from tuneta.tune_ta import TuneTA
import pandas as pd
from pandas_ta import percent_return
from sklearn.model_selection import train_test_split
import yfinance as yf
import pickle

if __name__ == "__main__":
    with open('candles.pkl', 'rb') as f:
        X = pickle.load(f)
    y = percent_return(X.Close, offset=-1)
    X_train, X_test, y_train, y_test = train_test_split(X, y, test_size=.3, shuffle=False)

    # Initialize with x cores and show trial results
    tt = TuneTA(n_jobs=7, verbose=True)

    # Optimize indicators
    tt.fit(X_train, y_train,
        indicators=['all'],
        ranges=[(2, 100)],
        trials=50,
        early_stop=20,
    )

Hmmm... what version of tuneta are you using:
    from importlib.metadata import version
    print(f"TuneTA: {version('tuneta')}")

Ok, let me run with 500, 100

What ranges are you using?

TuneTA: 0.1.37
ranges=[(2, 100)],

I think this may the problem:
https://github.com/DistrictDataLabs/yellowbrick/issues/1185
This was merged into yellowbrick but a release hasn't been generated.  Can you please do this:
pip install git+https://github.com/DistrictDataLabs/yellowbrick.git -U
It may be why I couldn't replicate because I likely installed from github repo too (and forgot) but still shows as Yellowbrick: 1.3.post1

Indeed! This worked. Thank you a lot.


move item datasource logic into store

Description
Tried to do the minimal (yes minimal ;-)) change to move the item/breadcrumbs/result fetching from the item data source to the store.
I have also added e2e tests covering that. (and unit tests for effects)
Notes
This requires to add a new concept of "identifier" on the fetch state, which tags for what content is this state. I thought about it already in the past for other usages but here it ensures the state that we select matches the current content (select from router, not via a reducer that we control)
Test cases

[ ] Case 1:

Given I am the demo user
When I go to a chapter
Then the behavior looks correct
When I go to a regular task
Then the behavior looks correct
When I go to a regular task with a wrong path
Then the behavior looks correct
When I go to a explicit-entry content
Then the behavior looks correct
When I go to a locked content
Then the behavior looks correct

[ ] Case 2:

Given I am a temp user
When I go to a chapter
And I visit a few chapters
Then the behavior looks correct (the results start as expected)

[ ] Case 3

Given I am a temp user
When I go to a content I cannot see
I see an error message which invites me to go back to the home page

[auto-generated message] E2E test report on: https://algorea-static.s3.eu-central-1.amazonaws.com/branch/item-content-fetching-in-store/playwright-report/index.html

First I was thinking that the selectors doing a bit a lot, but then I've found this article which has a good arguments<EMAIL_ADDRESS>
Selectors are quite light.. it is just that they check that the current content is what we expect before returning the value. Then there is these 2 selectors aggregating several data, they are there for backward compat mainly, I didn't want to change directly all uses of itemData.


icub-head 02/24 – Gather information about the current status of the head

Before doing the kick-off of the new head, it'd be great to collect a list of the current major problems in order to get everyone on the team properly aligned.

Regarding the recent failures recorded on the neck cables broken, here's a list[^1]:

https://github.com/robotology/icub-tech-support/issues/1721
https://github.com/robotology/icub-tech-support/issues/1722
https://github.com/robotology/icub-tech-support/issues/1725
https://github.com/robotology/icub-tech-support/issues/1730
https://github.com/robotology/icub-tech-support/issues/1736
https://github.com/robotology/icub-tech-support/issues/1737

[^1]: Credits to @sgiraz

cc @pattacini @Nicogene @Lawproto @ale-git


New versioning is using a bad release version

The new versioning should be using a version like form, please use 2022.02 with a dot . instead of dash -.
But the main issue right now is that the released version does not even follow the dash version, the release tag is using v2202-02 which is clearly wrong as it should be v2022-02 instead.
So as a proposal for a new release, the versioning might be v2022.03 or at least use the correct release tag if the dash is keep.

It's the release tag going to be fixed?

Yes, it's going to be fixed.


Publish Google Buildpacks to the Buildpack Registry

We'd love to see the Google Buildpack in the Buildpack Registry. The Buildpack Registry is in preview, and will soft launch later this month.
Three things need to be done to get the Google buildpack into the registry:

Package the Google buildpacks with the pack buildpack package command (which creates an OCI image representing/containing a buildpack)
Push the Buildpackage images to any OCI Registry
Register the image(s)/buildpack(s) with the Buildpack registry.

Documentation is forthcoming, and you can see some prior art in the jkutner/terraform-buildpack and a few other buildpacks.

oh hey hi. I think i'd be interested in picking this up again

@jama22 cool! let me know if you'd like assistance. We can chat in buildpacks Slack too.


Can we add ansible galaxy goss install to README?

I would like to ask for adding ansible galaxy goss installation which I am maintaining. Long time ago I was working with goss on regular basic. Today I am using this on my local machine for different tasks. I've created Ansible Galaxy role for installation.
If all maintainers agrees - this can be added to goss README file Community Contributions section.
Links for Repos and Ansible Galaxy:

https://github.com/marcinpraczko/ansible-goss-install
https://galaxy.ansible.com/ui/standalone/roles/marcinpraczko/goss-install/install/

You can also check Issues (closed) as there are my old suggestions and feature requests (few of them has been implemented in current version of goss).
Thank you in advance for reply for this question.

@marcinpraczko Thanks for submitting this, looks great! Curious how this differs from https://github.com/naftulikay/ansible-role-degoss
@naftulikay I've used degoss in the past, great project! Wondering if it's still maintained?

Hi @aelsabbahy.
I've done some quick check of this repository https://github.com/naftulikay/ansible-role-degoss, and those are difference comparing with mine:

Above repository is installing always the latest version of goss,
Above repository is having also ansible module (not sure whether is still working) - to run goss and present result
Which means that above repo can install goss and run goss tests.

My Ansible role is only installing goss (so far for amd64), however my intention is to add more options and support development for now.
I am not asking replace above repository, but kindly asking add to goss readme as possible option of installing goss as user or as root (my ansible role allows to user choose whether this should be installed as root or not).
Hope that I answered to your questions?

That answers my question clearly, thank you.
Yes, feel free to submit a PR adding your repo to community contributions.

Hi @aelsabbahy
Thank you for confirmation - new PR created as requested.

Hi @aelsabbahy  - I've done PR - however from some reasons CI is failing - my change was only with README.md - so not sure how this is failing. Is there anything which we can check what needs to be done to allow this PR pass CI?

Closed by #995


migrate to the latest org.apache.pdfbox 2.0.15+ dependency

upgrading will require code refactoring as compilation failures detected
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.8.0:compile (default-compile) on project carina-utils: Compilation failure: Compilation failure:
[ERROR] carina/carina-utils/src/main/java/com/qaprosoft/carina/core/foundation/utils/PDFUtil.java:[24,30] cannot find symbol
[ERROR]   symbol:   class PDFTextStripper
[ERROR]   location: package org.apache.pdfbox.util
[ERROR] carina/carina-utils/src/main/java/com/qaprosoft/carina/core/foundation/utils/PDFUtil.java:[43,9] cannot find symbol
[ERROR]   symbol:   class PDFTextStripper
[ERROR]   location: class com.qaprosoft.carina.core.foundation.utils.PDFUtil
[ERROR] carina/carina-utils/src/main/java/com/qaprosoft/carina/core/foundation/utils/PDFUtil.java:[50,46] incompatible types: java.io.InputStream cannot be converted to org.apache.pdfbox.io.Ra
ndomAccessRead
[ERROR] carina/carina-utils/src/main/java/com/qaprosoft/carina/core/foundation/utils/PDFUtil.java:[53,31] cannot find symbol
[ERROR]   symbol:   class PDFTextStripper
[ERROR]   location: class com.qaprosoft.carina.core.foundation.utils.PDFUtil
[ERROR] -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException
[ERROR]
[ERROR] After correcting the problems, you can resume the build with the command
[ERROR]   mvn  -rf :carina-utils

6.1.25+
https://github.com/qaprosoft/carina/pull/781


chore(test): less verbosity

Fix #351
Browser logs can still be inspected by going into focus mode (F) and turning on browser debugging (D).

:tada: This PR is included in version 7.1.3 :tada:
The release is available on:

GitHub release
v7.1.3

Your semantic-release bot :package::rocket:


Create Codeforces  D.Backspace

Greedy Algorithm
This is my code for https://codeforces.com/contest/1553/problem/D

I have done the task, is there anything else I have to do before it is merged?

@aditya-11-19
Don't worry it will be approved soon as you may know there are a lot PR's right now and also it's a one month program so don't worry it will be approved as soon as possible also I can't merge all PR's blindly as it may result in the repo being excluded from the program like many other repo so be patient full you PR will be accepted soon and also Don't forget to star the repo and if possible do provide a good description of what type of contribution you've done. Happy Hactoberfest

Ok thanks


[NDTensors][ITensors] Refactor BlockSparse system

Description
The goal of this PR is to refactor the BlockSparse system.  A few things will go into this refactor: 1. I will be organizing the code to match the previous NDTensors refactors, 2. I will be working to make a more understandable and documented blocking system in ITensors, 3. I will be working to make the BlockSparse system stand-alone from the ITensors package, 4. I will be updating the blockoffset system to utilize either Tensors or StorageTypes which will allow more flexibility in the sparsity of these Tensors.
This PR is based on https://github.com/ITensor/ITensors.jl/pull/1097
Checklist:

[ ] My code follows the style guidelines of this project. Please run using JuliaFormatter; format(".") in the base directory of the repository (~/.julia/dev/ITensors) to format your code according to our style guidelines.
[ ] I have performed a self-review of my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have added tests that verify the behavior of the changes I made.
[ ] I have made corresponding changes to the documentation.
[ ] My changes generate no new warnings.
[ ] Any dependent changes have been merged and published in downstream modules.

@kmp5VT can this be closed? We will be moving to the BlockSparseArray type anyway.


Reduce bundle size

We have our performance budgets configured to warn us at 2mb and error at 5mb. It would be great if we could get this app under the 2mb line before launch.

Partially addressed in PR #163


"macOS" prefix is not required at the display_string_workflow_title variable

The "${macos_installer_marketing_name_target}" variable is already capturing the full macOS Title name. Hence, the "macOS" prefix is unnecessary in the "display_string_workflow_title" variable.
https://github.com/Macjutsu/super/blob/a3ba601a571466379d858aaf483c0233325b86fc/super#L7127

Please try the latest release of super as it may resolve this issue: https://github.com/Macjutsu/super/releases/tag/v4.1.0-beta1


Is there any update about the document?

Hi, is there any update about the document?
It seems that something has changed.
For example, in the document, it says
const char* rocprofiler_error_string();

but in fact, the function is
hsa_status_t rocprofiler_error_string(
    const char** str);

Thank you.

Hi @adfc93, apologies for the lack of response. An internal ticket has been created to update and fix the documentation. Thanks!

@adfc93, thanks again for notifying us of the issue. https://github.com/ROCm/rocprofiler/commit/865e64753ac01fd26dcbe9d408ac6fdd43586991 has been merged into staging. The documentation will be updated for the ROCm 6.3 release.


Use FQCN at any point I found

Fixes: #3481
I found some places without FQCN, here they are ;)

@muellerbe
Good job


math: Add defines for max and min values in header

Linked with #24 and #25

Ok, since previous issues were handled, this might be deprecated, but...
Header defines for MAXFLOAT or INFINITY, or NAN can be useful (see OpenCL).
So, for now, I'll leave it open.

Added to include, closing issue


feat: add application alteration creation for handler UI (HL-1253)

API branch is deployed to platta: https://helsinkilisa-pr3075.api.dev.hel.ninja/healthz :rocket::rocket::rocket:

API branch is deployed to platta: https://helsinkilisa-pr3075.api.dev.hel.ninja/healthz :rocket::rocket::rocket:


Search Cloud Drives in Powertoys Run

Description of the new feature / enhancement
Hey,
Can you add Search Cloud Drives in Powertoys Run?
I have onedrive + google drive + iCloud setted up on my Windows, But I can't search them from my windows only from the browser.
I was wondering if someone can add search cloud drives feature in powertoys run this would be a huge productivity plus.
Scenario when this would be used?
searching for documents on my work drive
searching for passports on my private drive... etc.
Supporting information
I tried to index the drives in windows settings, there is no option for that. Tried troubleshooting attributing the drives in cmd
attrib "C:\Onedrive" -I
nothing worked.

Agreed - Powertoys Run cannot return results from Google Drive and that'd be a huge bonus as that's where I keep a lot of my most important personal files (e.g. passport scans, manuals, personal finance spreadsheets....)

Check out #23159 - I fixed this on my machine by syncing my cloud files on OneDrive to my local machine, and then in Advanced Indexing Options, making sure the synced folders were included.

@bigplayer-ai did this help:

Check out https://github.com/microsoft/PowerToys/issues/23159 - I fixed this on my machine by syncing my cloud files on OneDrive to my local machine, and then in Advanced Indexing Options, making sure the synced folders were included.

/needinfo


Update Yejin's Korea-Yonsei 학술교류 발표회

Need QR code for this poster presentation
(Later) add photo of the event

Related event: https://github.com/AIML-K/bamboo-forest/issues/183

Added the photo and the poster. See https://aiml-k.github.io/publication/2024koreayonsei-science-jeong-etal/
@JeongYejin1226 do you think adding abstract would be useful here? let me know.

Yes, I think it would be helpful. I’m attaching the abstract of the paper.

Abstract
Background context: Accurate and consistent measurement of sagittal alignment is challenging, particularly in patients with severe coronal deformities, including degenerative lumbar scoliosis (DLS).
Purpose: This study aimed to develop and validate an artificial intelligence (AI)-based system for automating the measurement of key sagittal parameters, including lumbar lordosis, pelvic incidence, pelvic tilt, and sacral slope, with a focus on its applicability across a wide range of deformities, including severe coronal deformities, such as DLS.
Design: Retrospective observational study.
Patient sample: A total of 1,011 standing lumbar lateral radiographs, including DLS.
Outcome measure: Interclass and intraclass correlation coefficients (CC), and Bland-Altman plots.
Methods: The model utilizes a deep-learning framework, incorporating a U-Net for segmentation and a Keypoint Region-based Convolutional Neural Network for keypoint detection. The ground truth masks were annotated by an experienced orthopedic specialist. The performance of the model was evaluated against ground truth measurements and assessments from two expert raters using interclass and intraclass CC, and Bland-Altman plots.
Results: In the test set of 113 patients, 39 (34.5%) had DLS, with a mean Cobb’s angle of 14.8° ± 4.4°. The AI model achieved an intraclass CC of 1.00 across all parameters, indicating perfect consistency. Interclass CCs comparing the AI model to ground truth ranged from 0.96 to 0.99, outperforming experienced orthopedic surgeons. Bland-Altman analysis revealed no significant systemic bias, with most differences falling within clinically acceptable ranges. A 5-fold cross-validation further demonstrated robust performance, with interclass CCs ranging from 0.96 to 0.99 across diverse subsets.
Conclusion: This AI-based system offers a reliable and efficient automated measurement of sagittal parameters in spinal deformities, including severe coronal deformities. The superior performance of the model compared with that of expert raters highlights its potential for clinical applications.
Keywords: Pelvic parameter, convolutional neural network, sagittal alignment, artificial intelligence, coronal deformity, segmentation, keypoint detection.


fix: handle names with endings that might parse as versions better

fixes #4855

@lars - this is a proposed replacement for #4862.
Either still need tests (and probably a light refactor of the method to make it more testable).

Either should also try to fix #4898 at the same time.

Let's use this one as a base, will close #4862.

OK - I've looked more closely and there's not a quick refactor that would let us test the small bit of logic that's in this functions dispatch, there are already tests of the fuzzy matcher that cover what caused the pain that led to this PR, and #4898 is a symptom of the much more general issue with viewing the last version of a draft before it became RFC. Lets solve the last one after the RFC-as-type-of-document major refactor. Thus, I think this simple fix should go forward so the oauth wg's issue gets resolved.


Alternative to gulp-strip-debug for Javascript

Hi,
Currently gulp-strip-debug doesn't seem to be compatible with gulp-sourcemaps.
I've looked over the compatible options on the wiki but wasn't able to find any other package to replace it:
Are there any other options to remove console and debugger statements from JavaScript code that are compatible with gulp-sourcemaps?
Thank you :)

No, none I'm aware of.
Btw: please use the Github issues for actual bug reports. There's other places (e.g. StackOverflow) where your question will probably get more answers. Thanks


Please add a license file

I'd like to add this to awesome-zsh-plugins, but it doesn't have a license file.
Please add a license file - if you don't have one in mind,  I recommend using choosealicense.com to help choose an appropriate one.
Thanks!

Thanks for the suggestion! I've added a license file (MIT). Also renamed and hugely refactored the theme.


Embedding mechanism

Current Mechanism used on http://jupyter.org/widgets

Embed manager script tag from cdn

Script tag with type=application/vnd.jupyter-embedded-widgets containing the widget manager state.
The format for this state is the one described here:
https://github.com/ipython/ipywidgets/blob/master/ipywidgets/state.schema.json
before we wrote the spec, so without the version information.
Besides, each widget has a views attribute of type array. If it is not empty, the widget is a top-level  widget.
This script tag is picked up by the embed manager and replaced by a rendered version of all the top-level widgets in a row.

Proposed mechanism for web pages

Embed manager script tag from cdn

Script tag with type=application/vnd.jupyter-widgets-state+json containing the widget manager state, following precisely the schema for the manager state
https://github.com/ipython/ipywidgets/blob/master/ipywidgets/state.schema.json

Script tags with type=application/vnd.jupyter-widgets-view+json containing
{
    "model_id": {
        "type": "string",
        "description": "Model id to be displayed inplace"
    }
}

Proposed mechanism for nbviewer

nbviewer loads the embed manager JavaScript, either from cdn or bundles is

Each output area containing widgets gets its type=application/vnd.jupyter-widgets-view tag in the html mime bundle.

Global widget state following the state schema is saved in the notebook metadata.
The html nbconversion picks up that metadata and incorporates a type=application/vnd.jupyter-widgets-state+json.

Ping @michaelpacer @minrk @jasongrout

type=application/vnd.jupyter-embedded-widgets

We've been using application/vnd.jupyter.<something> for mime types, in accordance with https://tools.ietf.org/html/rfc6838#section-3.2. (We should register the jupyter name for this to be really official, though...)

type=application/vnd.jupyter-embedded-widgets

We've been using application/vnd.jupyter. for mime types, in accordance with https://tools.ietf.org/html/rfc6838#section-3.2. (We should register the jupyter name for this to be really official, though...)

OK edited the description.

@michaelpacer this is orthogonal to the current png repr in the mime bundle that is used to achieve
this: https://github.com/ipython/ipywidgets/blob/master/docs/source/examples/Factoring.ipynb

OK edited the description.

Thanks. The example I quoted still should be changed :).

OK edited the description.

Thanks. The example I quoted still should be changed :).

...

^ @jasongrout

Doh!

Ok, thanks for the heads up.
I'm still not sure what the best way to deal with widgets in static documents is.
I have some thoughts but I'm guessing that looking at the actual code will help to refine those thoughts. Could someone point me to the widget drawing code (for live drawing on display) and the current mechanism for generating the png representations?


Do command checks after choosing command and make required_permissions backend agnostic

Former is done in 38aaa1c3c1299acbaf4dfeecedd4e3d9b61847cd, latter was already implemented idk why I thought it wasn't


Using Bootstrap CDN instead of hosting on our server

Hi,
So the issue title is pretty self-explanatory. It would be wise to use the static bootstrap files using CDN so that the loading time is faster.
Bonus: Cleans our code a little bit. (~22% reduction in our repo size)

Hey @armudgal ! I am working on it.

@sanctophagus Hey, send a PR for the previous issue and then you can take this one. We don't want to block issues if they can't be worked upon :)

@armudgal  Should I delete all the files of public/bootstrap-3.3.7

@sanctophagus After making sure that everything works locally (UI and front-end JS part) you can. Once you make the PR, I will test as well.


Update for 0.20.6

Once we publish 0.20.6 to fix https://github.com/creditsenseau/zeebe-client-node-js/issues/74, this will pull it in, and also make the gateway address optional.
The library can now get its configuration from the environment if no gateway address is passed in (added in 0.20.5). It will default to localhost if no gateway address is supplied and no gateway address is passed to the constructor.
This allows you to write more portable code, without having to demarshall environment variables into the constructor.

Also, 0.20.6 will remove complete() - you need to use complete.success() instead. I updated the README.

0.20.6 is out.

Thanks!

Also, 0.20.6 will remove complete() - you need to use complete.success() instead. I updated the README.

This isn't a bug fix, not even a minor BTW, it's a breaking change SemVer wise...


Fix variable fetching on remote machines

For #4006

[x] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[x] Title summarizes what is changing.
[x] Has a news entry file (remember to thank yourself!).
[x] Appropriate comments and documentation strings in the code.
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[ ] Unit tests & system/integration tests are added/updated.
[ ] Test plan is updated as appropriate.
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed).

Thinking a bit on testing here. We already have tests that cover local testing. However this issue only shows up in remote cases. And in particular you can't test it with "fake" remote by starting a server on your own machine, as the sys paths will be there in that case.
Can we run Docker in tests? That's how I was reproing this locally without needing a second machine.

Thinking a bit on testing here. We already have tests that cover local testing. However this issue only shows up in remote cases. And in particular you can't test it with "fake" remote by starting a server on your own machine, as the sys paths will be there in that case.
Can we run Docker in tests? That's how I was reproing this locally without needing a second machine.

I believe so, yes.
See the answers to this issue:
https://github.community/t/running-automated-tests-inside-the-docker-container-after-building-it/18047

Widget test only failures.


Add multiple monitor support to Set-TaskbarOptions

This request is to enable support for the "Multiple displays" settings found in Windows 10:

Show taskbar on all displays
Show taskbar buttons on

All taskbars
Main taskbar and taskbar where window is open
Taskbar where window is open

Combine buttons on other taskbars

Always, hide labels
When taskbar is full
Never

These are set with the following registry values under HKCU\Software\Microsoft\Windows\CurrentVersion\Explorer\Advanced (thanks to https://finnzan.blogspot.com/2017/01/windows-10-taskbar-customization.html):
MMTaskbarEnabled
0 = Don't show taskbar on multiple displays
1 = Show taskbar on multiple displays
MMTaskbarMode DWORD
0 = Show taskbar buttons on all taskbars
1 = Show taskbar buttons on main taskbar and taskbar where window is open
2 = Show taskbar buttons on taskbar where window is open
MMTaskbarGlomLevel
0 = Always combine, hide labels
1 = Combine when taskbar is full
2 = Never combine
This seems pretty straightforward so I'm happy to contribute a PR for this.

Sounds good!

PR has been submitted (#400).

@pauby - This was closed in February, and the documentation refers to the new function, but I just installed BoxStarter moments ago and it's still got the old function.  Am I doing something wrong?  Does the new function need to be exported somewhere?

@pauby - Okay, just figured this out.  The bootstrapper.ps1 script has been updated to 2.13.0, in the repo, but the one pointed to by the installation instructions (https://boxstarter.org/bootstrapper.ps1) is still the old version.

Issue #458 opened to resolve some additional problems with this new implementation.


GlosSI-Target crash after several minutes

Expected Behavior

GlosSI-Target shouldn't be crashing after a few minutes
Controller should continue to be emulated

Actual Behavior
After initially working just fine, GlosSI-Target crashes after a few minutes while in-game (haven't tested leaving it on the desktop).
Steps to Reproduce the Problem

Launch GlosSI-Target target from Steam
Launch Epic Games launcher
Launch Battlefront 2 (which is an Origin game)
Play for a few minutes (not consistent) before hearing the Windows "USB Disconnected" sound, and seeing GlosSI-Target has crashed

General Information

Version: <IP_ADDRESS>
Platform: Windows 10
Controller(s): 8bitdo SN30 Pro 2, used as a Switch Pro controller
Application: GlosSITarget

Logs / Additional Files
glossitarget.log
Thank you!

Note: the timestamp for the dmp file doesn't match the time in which teh crash occured
https://ufile.io/pkl6hyly

Ok, possibly found the reason.
I was running Steam as an Administrator (so I can use Desktop mode on my controller and navigate through apps that need Administrator permissions, ex. Task Manager).
To get things working I have to:

run GlosSI target
when the Steam notification appears, it says "Config Loaded" but no actual config was named
to get around this I disconnect/reconnect my controller, then the selected Steam Input config is shown

Hope this helps someone.


directing url to id page (only applies to home page)

📝 Description
Task I've done:

Creating jquery to replace page URL defined in href to its page ID (only applies to the home page). I replace all except CONTACT menu.

For example:
I keep the link (href="men.html") defined as it
On home page: the page URL "men.html" will be directed to its page ID "index.html#men"
On other page: the page URL "men.html" will directed to its original URL "men.html"
(I'll provide screenshots below)
By using jquery, you can apply the same header (tab menus section) or footer to other pages as well without changing anything (in this case, link list).

Synchronizing ID in index.html as mentioned in jquery.

Updating "Explore More, View Trending, View Best Sellers" button's link in index.html.

If you accept these changes, please make notes to contributors to use same hyperlink for header and footer in the page they'll create or they've created (in case, you wanna add header and footer to all pages to make it similar with home page's look). They can copy header or footer link from index.html.
🔧 Fixes #(issue)
#73

📷 Provide related Screenshot/Gifs if any.
The pointer will show the actual link:
HOME PAGE

TRENDING PAGE (other pages as well)

[x] I have performed a self-review of my own code.
[x] I'm a contributor in Hacktoberfest'21.

Hey, there's no need to redirect to ids, you can make all links redirect to the respective pages only, from homepage also, make the links at navbar and footer to redirect to particular pages, not ids.

I've removed the jquery to direct all links to particular pages (including home page) as you requested. Please check.

So, for the issue I've created. The tasks I've done are

Updating links on navbar and footer

Updating "Explore More, View Trending, View Best Sellers" button's link in index.html.

Sorry if I misunderstood what you asked before. I already fixed it on the last commit, so please check.

thankyou @nurulramadhona


job_start and job_stop on OpenBSD

On OpenBSD, job_stop never kills the whole process group. Since job_start is often used to run sh -c 'single command', only the sh process gets a SIGTERM and job_stop does not terminate 'single command'. The reason for this is that job_start (mch_job_start in src/os_unix.c) calls setsid (after fork and before execvp) to create a new process group, but this also creates a new terminal session. Then in job_stop (via mch_signal_job in src/os_unix.c) vim first checks if the job is group leader using getpgid, and sends SIGTERM to the whole group only if it is. But on OpenBSD, getpgid returns EPERM (-1) when a process tries to checks another process in a different session.
There are two ways to fix this:

replace setsid() by setpgid(0, 0) in mch_job_start, so that only a new process group is created. This may have surprising side effects, because the jobs now share the same terminal as the parent vim process, although this is (incorrectly) explained in vim's help for job_start.
Remove the if (job_pid == getpgid(job_pid)) check in mch_signal_job.

This changes causes Test_job_stop_immediately() to fail on Linux.
Perhaps we should both kill(-pid) and kill(pid) ?

Perhaps we should both kill(-pid) and kill(pid) ?

This might be caused by how the forking is done, and the child not being setup/configured fast enough.
I suggest doing at least the following to skip it if not needed:
diff --git i/src/os_unix.c w/src/os_unix.c
index 6387a7757..d672e99c6 100644
--- i/src/os_unix.c
+++ w/src/os_unix.c
@@ -5855,8 +5855,8 @@ mch_signal_job(job_T *job, char_u *how)
     if (job->jv_pid != 0)
     {
        // TODO: have an option to only kill the process, not the group?
-       kill(-job->jv_pid, sig);
-       kill(job->jv_pid, sig);
+       if (kill(-job->jv_pid, sig))
+           kill(job->jv_pid, sig);
     }

     return OK;

libuv uses a pipe to communicate / wait for the child to call execve: https://github.com/neovim/libuv/blob/c41b31ed3b88aacae1d8498d0f3f6bbc20d56f9a/src/unix/process.c#L469-L491 - looks very relevant.

Daniel Hahler wrote:

Perhaps we should both kill(-pid) and kill(pid) ?

This might be caused by how the forking is done, and the child not being setup/configured fast enough.
I suggest doing at least the following to skip it if not needed:
diff --git i/src/os_unix.c w/src/os_unix.c
index 6387a7757..d672e99c6 100644
--- i/src/os_unix.c
+++ w/src/os_unix.c
@@ -5855,8 +5855,8 @@ mch_signal_job(job_T *job, char_u *how)
     if (job->jv_pid != 0)
     {
        // TODO: have an option to only kill the process, not the group?
-       kill(-job->jv_pid, sig);
-       kill(job->jv_pid, sig);
+       if (kill(-job->jv_pid, sig))
+           kill(job->jv_pid, sig);
     }

     return OK;

These things always have portability issues.  Since I haven't hear users
complain, making a change here might actually cause a problem.
I'm not really afraid the pid gets reused so quickly.
--
Female engineers become irresistible at the age of consent and remain that
way until about thirty minutes after their clinical death.  Longer if it's a
warm day.
(Scott Adams - The Dilbert principle)
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


Remove the hand-made input!

Fixes the CI by removing our hand-made input!.
After all this time, I think it was not necessary.

Merging.


Provide a way for third-party recipes to specify a minimum bottle architecture, and support custom architectures

A detailed description of the proposed feature
Some software heavily depends on modern CPU features and is borderline-unusable on older systems. For example, HHVM has real-world workloads that are 19x slower when built for nehalem than when built for sandybridge.
This is really two tightly related feature requests:

allow specifying a default bottle-arch in the recipe

allow third-party recipes to define custom architectures that brew doesn't recognize out the box: for example, if I specify --bottle-arch=sandybridge, brew install errors out with "Unrecognized architecture for --bottle-arch: sandybridge`

The motivation for the feature

several of our corporate users build their own slightly different bottles, and it would be good to be able to provide a recipe that 'just works' without custom instructions.

HHVM wants to target a microarchitecture that brew doesn't recognize

How the feature would be relevant to at least 90% of Homebrew users
It probably isn't, and I'd expect it to only be used by taps. Perhaps ffmpeg could be a use case.
Opening anyway as Mike indicated on #5765 that this might be worth adding. I'm expecting this to be something I need to submit a pull request for - not for anyone else to volunteer to do, but it might be valuable to dump the ideas and motivation somewhere public anyway :)
What alternatives to the feature have been considered

specifying --bottle-arch=native: too non-portable
specifying --bottle-arch=nehalem: I was surprised to discover this isn't the default on high sierra (we're getitng penryn). It's a bit faster, but still 19x slower than sandybridge for us
specifying --bottle-arch=nehalem and monkeypatching Hardware::CPU.optimization_flags to change the -march= flag this resolves to: this is obviously a bad fix, and misleading. Currently doing it as a temporary band-ai
in the past, we did a slightly cleaner monkeypatch by setting ARGV.bottle_arch to :sandybridge and monkeypatching Hardware::CPU.optimization_flags to recognize sandybridge. This is no longer possible now that ARGV is frozen

specifying --bottle-arch=nehalem: I was surprised to discover this isn't the default on high sierra (we're getitng penryn). It's a bit faster, but still 19x slower than sandybridge for us

We'd make it default on High Sierra but unfortunately there are Penryn machines still running High Sierra.

specifying --bottle-arch=nehalem and monkeypatching Hardware::CPU.optimization_flags to change the -march= flag this resolves to: this is obviously a bad fix, and misleading. Currently doing it as a temporary band-ai

I think modifying OPTIMIZATION_FLAGS and Hardware::CPU.optimization_flags to blindly passthrough a specified bottle-arch that's unrecognised to us would make sense. The compiler will blow up if it's unsupported and kill the build process.

allow specifying a default bottle-arch in the recipe

👍 I think either something in bottle do or a separate bottle_arch :sandybridge DSL would be acceptable here. In an ideal world we'd also verify the CPU is new enough on pouring but this doesn't need to block its inclusion.

I'm expecting this to be something I need to submit a pull request for - not for anyone else to volunteer to do, but it might be valuable to dump the ideas and motivation somewhere public anyway :)

Yup, PR would be good for this. Happy to answer any questions here first, though.

Thanks - to clarify, the bad and misleading hack that I'm doing is {nehalem: "-march=sandybridge"}, not the general idea of extending OPTIMIZATION_FLAGS  - i.e. --bottle-arch=nehalem doesn't get you a nehalem HHVM bottle :p

Not sure it'll be me, but it seems fairly likely that someone at FB will end up doing at this - in case that happens, FB T41189241 is the internal reference for this issue.

Not sure it'll be me, but it seems fairly likely that someone at FB will end up doing at this - in case that happens, FB T41189241 is the internal reference for this issue.

Happy to have people go on the issue as reference if that happens but closing out until it looks like it will.


tests: use fixtures for packages

Simpler testing.

Ahh, pytest is exceeding the maximum path length on Windows. 😡

We are very close to the limit - a long test name would put it over (pytest test caps the test name to dir name at 30 chars). But using pkg for the inner folder name does keep it under the limit for now.


Issue installing code OSX and Debian

I'm having trouble getting parastell installed on either MacOS (12.7.6, x86) or on my Debian machine (GNU/Linux 12, 6.1.0-28-amd64, x86).  On the mac I tried following your intall instructions using miniconda but when I try to run the example config.yml I get the following error:
../executables/parastell config.yaml
Traceback (most recent call last):
  File "/Users/lazerson/Sims_Work/PARASTELL/Examples/../executables/parastell", line 3, in <module>
    from parastell import parastell
  File "/Users/lazerson/Sims_Work/PARASTELL/parastell/parastell.py", line 5, in <module>
    import cubit
  File "/Applications/Coreform-Cubit-2023.11.app/Contents/lib/cubit.py", line 44, in <module>
    from cubit3 import *
  File "/Applications/Coreform-Cubit-2023.11.app/Contents/lib/cubit3.py", line 12, in <module>
    import _cubit3
ImportError: dlopen(/Applications/Coreform-Cubit-2023.11.app/Contents/lib/_cubit3.so, 0x0002): Symbol not found: (_iconv)
  Referenced from: '/Applications/Coreform-Cubit-2023.11.app/Contents/lib/libCATSysTS.dylib'
  Expected in: '/Users/lazerson/miniconda3/envs/parastell_env/lib/libiconv.2.dylib'

For reference Coreform-Cubit runs fine on it's own.  This may be an OSX issue so no big deal, I have the Debian machine. However on that machine I get a different error:
WARNING:root:Mayavi not found. 3D plots will not work.
Traceback (most recent call last):
  File "/home/lazerson/src/parastell/Examples/../executables/parastell", line 3, in <module>
    from parastell import parastell
  File "/home/lazerson/src/parastell/parastell/parastell.py", line 10, in <module>
    from . import invessel_build as ivb
  File "/home/lazerson/src/parastell/parastell/invessel_build.py", line 8, in <module>
    import cadquery as cq
  File "/home/lazerson/anaconda3/envs/parastell_env/lib/python3.11/site-packages/cadquery/__init__.py", line 22, in <module>
    from .occ_impl import exporters
  File "/home/lazerson/anaconda3/envs/parastell_env/lib/python3.11/site-packages/cadquery/occ_impl/exporters/__init__.py", line 10, in <module>
    from ...cq import Workplane
  File "/home/lazerson/anaconda3/envs/parastell_env/lib/python3.11/site-packages/cadquery/cq.py", line 62, in <module>
    from .sketch import Sketch
  File "/home/lazerson/anaconda3/envs/parastell_env/lib/python3.11/site-packages/cadquery/sketch.py", line 28, in <module>
    from .occ_impl.sketch_solver import (
  File "/home/lazerson/anaconda3/envs/parastell_env/lib/python3.11/site-packages/cadquery/occ_impl/sketch_solver.py", line 3, in <module>
    from nptyping import NDArray as Array
  File "/home/lazerson/anaconda3/envs/parastell_env/lib/python3.11/site-packages/nptyping/__init__.py", line 32, in <module>
    from nptyping.ndarray import NDArray
  File "/home/lazerson/anaconda3/envs/parastell_env/lib/python3.11/site-packages/nptyping/ndarray.py", line 40, in <module>
    from nptyping.shape import Shape
  File "/home/lazerson/anaconda3/envs/parastell_env/lib/python3.11/site-packages/nptyping/shape.py", line 29, in <module>
    from nptyping.shape_expression import (
  File "/home/lazerson/anaconda3/envs/parastell_env/lib/python3.11/site-packages/nptyping/shape_expression.py", line 36, in <module>
    from nptyping.typing_ import ShapeExpression, ShapeTuple
  File "/home/lazerson/anaconda3/envs/parastell_env/lib/python3.11/site-packages/nptyping/typing_.py", line 51, in <module>
    Bool8 = np.bool8
            ^^^^^^^^
  File "/home/lazerson/.local/lib/python3.11/site-packages/numpy-2.0.0-py3.11-linux-x86_64.egg/numpy/__init__.py", line 410, in __getattr__
    raise AttributeError("module {!r} has no attribute "
AttributeError: module 'numpy' has no attribute 'bool8'. Did you mean: 'bool'?

This second issue seems to be related to the line in the environments.yml:
 - numpy=1.26.4

as the bool8 feature was depricated in 1.24, and I'm guessing removed by 1.26.4

Update the issue seems to be with pystell_uw as it's build does not specify a version of numpy and so it's installing 2.0 which then meets the needs for parastell which of course it doesn't.

Hi @lazersos, thanks for reaching out and apologies for the delay in response. Can you please confirm which version of NumPy is installed in your conda environment?

@lazersos, it doesn't seem that the output you've provided includes information about your NumPy installation. To retrieve information about your NumPy installation via conda, you can run the following command in your terminal (with the proper conda environment activated):
conda activate parastell_env
conda list numpy

@connoramoreno Sorry here you go:
conda list numpy
# packages in environment at /home/lazerson/anaconda3/envs/parastell_env:
#
# Name                    Version                   Build  Channel
numpy                     1.24.0          py311hbde0eaa_0    conda-forge

@connoramoreno Yes, from what I can tell Conda has a numpy 1.24.0 installed but pip has numpy 2.0.0 installed as well. And when the code is envoked, the pip version is the one which is picked up first.

A bit more digging and issue seems to be that cadquery which is needed by cad_to_dagmc is currently broken via pip and presumably conda (https://github.com/CadQuery/cadquery/issues/1714).

I'm unfamiliar with pip installations of CadQuery but we have not experienced any issues with conda versions yet. CadQuery is a major dependency of ParaStell and is the software toolset we use to generate all of our CAD models, so any issues with CadQuery would likely make themselves quickly apparent.
As you mentioned, it seems that the conflict is occurring because Python is importing a version of NumPy that is outside your conda environment. Here are some potential solutions:

Revert the pip installation of NumPy to an older version that is compatible with ParaStell.
Uninstall the pip installation of NumPy and rely instead on installations via conda for your computational needs. You could create other conda environments or just use the ParaStell environment.
Install ParaStell's Python dependencies via some method other than conda. This is what I would least recommend as our support for alternative dependency installation methods is limited.

@connoramoreno Doing a pip uninstall numpy and then building the conda environment again seems to fix the issue. It's odd that the pip installations outside of conda would override conda. Anyway, thanks for the help.

A bit more digging and issue seems to be that cadquery which is needed by cad_to_dagmc is currently broken via pip and presumably conda (CadQuery/cadquery#1714).

AFAICT conda installation is not broken, but if you have a reproducible problem, please open an issue.


Display browser upgrade message on older IE browsers

This resolves https://github.com/minio/doctor/issues/371

I've tested the IE comment tags.

@rushenn  I have some thoughts on this. Should we block the whole site on IE. There used to be an option to proceed anyway in unsupported mode.

@deekoder It's better to block the whole site on older IEs. Most of the CSS tech used in the site would not work in older browsers. If we give an option to proceed through, with all the style flickers, it may give a bad impression on how the way it looks.


protobuf_25: init at 25.1

Description of changes
Changelog: https://github.com/protocolbuffers/protobuf/releases/tag/v25.1
cc @tobim @jonringer

Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux
[x] x86_64-darwin
[x] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

 But seems to just be related to the test suit having issues with local imports

Unfortuantely, the protobuf_25.passthru.tests.pythonProtobuf test fails. But seems to just be related to the test suit having issues with local imports

Is it the same issue as reported in https://github.com/NixOS/nixpkgs/pull/264902#issuecomment-1838848195 ?

Yes, it is.

Yes, it is.

All right. I tried to look into it but I had no real success :/