Add feature to require participants to complete their profile before participating

Changes proposed in this pull request:

Modified participateInChallenge to handle the error case when required fields are empty for participating in a challenge.

Some minor css changes

Link to live demo: http://pr-198-evalai.surge.sh

Screenshot:

Codecov Report

Merging #201 into master will decrease coverage by 0.04%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master     #201      +/-   ##
==========================================
- Coverage   51.47%   51.43%   -0.05%
==========================================
  Files          66       66
  Lines        3629     3636       +7
  Branches      410      411       +1
==========================================
+ Hits         1868     1870       +2
- Misses       1668     1673       +5
  Partials       93       93

Impacted Files
Coverage Δ

src/app/services/challenge.service.ts
44.04% <33.33%> (-0.68%)
:arrow_down:

Impacted Files
Coverage Δ

src/app/services/challenge.service.ts
44.04% <33.33%> (-0.68%)
:arrow_down:

Continue to review full report at Codecov.

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


 It's obnoxious.
Compare that to https://github.com/nhooyr/websocket/tree/067b40e227d0d6af9e399c1efe0dd80efae1b79f where the Go module has been moved to the subdirectory ./mod inside the repository.
See https://github.com/nhooyr/websocket/pull/136
So I want to move the Go module to the subdirectory ./mod inside the repository and serve that subdirectory for the nhooyr.io/websocket import path but it doesn't look like there is an easy way to do that.
The go-import meta tag only allows me to specify the import path of the repository, which in this case would make it nhooyr.io/websocket/mod which is nasty. I want to serve the subdirectory directly under nhooyr.io/websocket. Looks like I can do this with the new mod vcs to the go-import tag but then I have to run my own module server which I want to ideally avoid.
Is this something that would be considered or is already possible?

For now I'm redirecting go get to https://github.com/nhooyr/websocket-mod where I've moved the module to the root of the repository. Unideal for sure but works well for now.

I might also just switch to moving the real code into the subdir mod but have a package in the root that uses type aliases to alias directly to the mod subpkg. Would involve a lot of duplicate code and be harder to maintain though.

So unfortunately people were getting confused that the mod path didn't resolve directly to the root of the repo and I tried the type alias thing but in the docs it links directly to the other package instead of inlining the docs so I'd have to copy a lot of code.

It's unclear what bug is being raised in this issue and/or what proposal to the go command is being suggested. I'm going to close this bug, but check out https://github.com/golang/go/wiki/Questions as it may provide a good avenue to continue this discussion.

Apologies for the confusion. I'm trying to serve a subdir of my repository https://github.com/nhooyr/websocket under the import path nhooyr.io/websocket but this doesn't seem feasible with the current tooling without my own module proxy and event then GoDoc wouldn't work as it doesn't support modules yet.

I've got same issue. I have project with migrations, protobuf files and other infrastracture files.
So I decide to put my project under
https://vcs/vendor/project/src/.
But i could not import in go modules.

I have the same issue. @katiehockman Please reopen, this is a legitimate issue that a lot of people are experiencing.
A brief proposal to fix the issue would be to add an optional fourth parameter to go-import, so if you wanted to specify a subdirectory you could do something like this:
<meta name="go-import" content="nhooyr.io/websocket git https://github.com/nhooyr/websocket mod">

This would import the mod subdirectory of the github.com/nhooyr/websocket git repository.

/cc @bcmills @jayconrod

See previously #33562.

I have the same issue. @katiehockman Please reopen, this is a legitimate issue that a lot of people are experiencing.
A brief proposal to fix the issue would be to add an optional fourth parameter to go-import, so if you wanted to specify a subdirectory you could do something like this:
<meta name="go-import" content="nhooyr.io/websocket git https://github.com/nhooyr/websocket mod">

This would import the mod subdirectory of the github.com/nhooyr/websocket git repository.

The subdirectory rule is not working. Does anyone find it working with "go version go1.13.6 darwin/amd64"?
Thanks

We (Apache Thrift) are bitten by this issue as well.
For Apache Thrift project, the only part that would be imported by others is github.com/apache/thrift/lib/go/thrift. If we put the go.mod file under lib/go/thrift, the importing still works, but that comes with caveats for users importing this library:

They have to import github.com/apache/thrift/lib/go/thrift instead of github.com/apache/thrift in their go.mod files
They no longer get the git tagged versions. The version of github.com/apache/thrift/lib/go/thrift in their go.mod files will always be v0.0.0-timestamp-commitsha1.

If we move go.mod file to the top level, that comes with the problem that we have test code running in the repo in other directories that would affect the now shared go.mod file, causing more problems (the most notable one is that people only work on the library code cannot run go mod tidy because some of the go code used by other test code in the repo are generated by thrift compiler and they need to run the whole automake process to generate them).

We (Apache Thrift) are bitten by this issue as well.
For Apache Thrift project, the only part that would be imported by others is github.com/apache/thrift/lib/go/thrift. If we put the go.mod file under lib/go/thrift, the importing still works, but that comes with caveats for users importing this library:

They have to import github.com/apache/thrift/lib/go/thrift instead of github.com/apache/thrift in their go.mod files
They no longer get the git tagged versions. The version of github.com/apache/thrift/lib/go/thrift in their go.mod files will always be v0.0.0-timestamp-commitsha1.

If we move go.mod file to the top level, that comes with the problem that we have test code running in the repo in other directories that would affect the now shared go.mod file, causing more problems (the most notable one is that people only work on the library code cannot run go mod tidy because some of the go code used by other test code in the repo are generated by thrift compiler and they need to run the whole automake process to generate them).

Adding to proposal minutes. The basic problem is to be able to use a meta tag to redirect not just to a Git repo but to a subdirectory within a Git repo where the module root lives. That could be an extra field in the meta tag form, but of course it would not be safe to use until the change trickled to older versions of Go (or you assume those older versions are all using the proxy).

This proposal has been added to the active column of the proposals project
and will now be reviewed at the weekly proposal review meetings.
— rsc for the proposal review group

For Apache Thrift project, the only part that would be imported by others is github.com/apache/thrift/lib/go/thrift. If we put the go.mod file under lib/go/thrift, the importing still works, but that comes with caveats for users importing this library:

They have to import github.com/apache/thrift/lib/go/thrift instead of github.com/apache/thrift in their go.mod files

Note that this proposal wouldn't help with github.com paths, as those don't use go-import metadata.

They no longer get the git tagged versions. The version of github.com/apache/thrift/lib/go/thrift in their go.mod files will always be v0.0.0-timestamp-commitsha1.

You can make tags for the module by calling the tags lib/go/thrift/v1.2.3. Not ideal, just FYI.
See https://golang.org/ref/mod#vcs-version.

In general, I support this proposal. When using go-import metadata there is already indirection between the import path and where the source is stored, and not requiring it to be at the root of a repository makes sense to me.
A question is what the applicable tags would be. I think people would expect and want unprefixed tags to work judging from the described use cases, but since different modules might point to different subdirectories, prefixed tags would probably be more consistent.

since different modules might point to different subdirectories, prefixed tags would probably be more consistent.

I agree. I think prefixed tags are the only viable option.

While I can see the prefixed tags could be useful in some use cases, for Apache Thrift we would really prefer to be able to use the global git tag.
The reason is that for Apache Thrift the releases are always global: it includes the compiler, and libraries for all the supported languages. So for go library, people really need to use the matching version with the compiler (e.g. always use v0.14.0 compiler with v0.14.0 go library). If we only support prefixed tags, that means whenever we do a release we would need to tag it twice, once with v0.14.0 and again with lib/go/thrift/v0.14.0.
May I suggest that we use the global tag as a "fallback"? e.g. if there's no prefixed tags matching the go.mod path, fallback to use the global tag instead? I can see there could be a potential issue of a project used to not use prefixed tags, so we fallback to global tag, then at some point they started to use prefixed tags, and then we could have two, conflicting v1.0.0. For that issue, I think maybe we can add a suffix to the fallback global tag, similar to how we auto add -incompatible suffix. The suffix could be something like -global, -repo, etc. (so when we use fallback tag, the version tag in user's go.mod file would be v0.14.0-global or something like that).

@fishy, fallbacks are not viable because they could result in time-dependent meanings of versions for a given repo.
If we start out with only a v0.14.0 tag, then users who require … v0.14.0 will use that tag. But then what happens if a lib/go/thrift/v0.14.0 tag is added, on a different commit, down the road? Proxies will continue to cache and serve the original v0.14.0, but new users with GOPROXY=direct will instead see the nested tag and get checksum errors.
Given that it's straightforward to for module authors to duplicate tags with prefixes if desired, it seems better to avoid introducing that ambiguity.

@bcmills did you see the part that I suggested to add suffix for fallback global tags to resolve that issue?
tag it twice is "trivial", sure, but it's also some special thing we need to do for go library. thrift is a project supporting tens of languages, and no other language requires this special treatment like go does (if we move forward to only support prefixed tags).

If we start out with only a v0.14.0 tag, then users who require … v0.14.0 will use that tag. But then what happens if a lib/go/thrift/v0.14.0 tag is added, on a different commit, down the road? Proxies will continue to cache and serve the original v0.14.0, but new users with GOPROXY=direct will instead see the nested tag and get checksum errors.

I am not saying I like fallbacks, but this is not actually that much of a blocker. git tags can change, so we can treat adding a prefixed tag for an existing unprefixed tag the same way we treat changing a tag: it's a mistake and it will be blocked by the Checksum Database.
A bigger problem is unprefixed tags leaking into all modules which might not be intended, and inconsistency with the current behavior of modules in subdirectories which we can't change.

A bigger problem is unprefixed tags leaking into all modules which might not be intended, and inconsistency with the current behavior of modules in subdirectories which we can't change.

I think an auto added suffix can mitigate that issue to some degree?

The original proposal was about being able to make a custom domain like nhooyr.io/websocket be able to redirect to a subdirectory of a Git repo. That's still something we can do and it doesn't seem too bad to do it.
Thoughts, @bcmills, @jayconrod, @matloob?
It is definitely the case that we can't make github.com/apache/thrift somehow redirect to github.com/apache/thrift/lib/go/thrift, both due to ambiguity issues and because there is no control over that rule. (Maybe if GitHub let you edit the <meta> tag we could relax the rule and respect the meta tag, but right now the hard-coded rule in cmd/go matches GitHub's hard-coded <meta> tag.)
But is the custom domain version still worth doing?

This seems feasible. Two thoughts about remaining issues.
First, what should the <meta> tag look like? Adding another space-separated field was suggested earlier:
<meta name="go-import" content="nhooyr.io/websocket git https://github.com/nhooyr/websocket mod">

This would work: the go command ignores tags if they don't have exactly three fields, so older versions of Go would act as if the meta tag weren't present at all. They wouldn't be able to handle the subdirectory anyway, so that's fine. golang.org/x/tools/go/vcs would need to be updated, too.
Second, what should the version tags look like. I agree with @bcmills: we should avoid letting multiple tags represent the same version; it leads to too much confusion. #32945 also introduces to this problem, and if we accepted both, we'd have four possible tags for each version. I'm not really convinced that creating a Go-specific tag is that much of a burden; I'd expect each language would have its own idiosyncrasies, like creating and uploading a release artifact.
We already have the convention that if a module is rooted in a subdirectory of a repository, its version tags must have that subdirectory as a prefix (not including a major version subdirectory if any). So for the module above, the tags would be:
mod/v1.0.0
mod/v2.0.0

If there were another module nhooyr.io/websocket/example in a subdirectory mod/example, those tags would be:
mod/example/v1.0.0
mod/example/v2.0.0

Feel free to yell and throw chairs at me:

what if some form of aliasing "magic" go.mod could be devised, one that could be placed at the root of the repository to point at the actual root of the module, similar to how the replace directive already works for 3rd party modules?

I went through all messages  :neckbeard:.
I would like to clarify that this issue points to this scenario

We follow golang "standard" layout

packages are located under pkg
services are located under cmd

With years we have a lot of modules under pkg and some of them are used very extensively in our ecosystem.
So the versioning of submodules started to conflict with versions of services cmd as git's tags also are used for versions in our build system and  as  tags of docker containers.
We decided to move most general packages to the separate repository.  Till it's clear that one repository for one module would be ideal. We don't want to have 30 additional repositories :flushed:. From other side to keep one go.mod at the root is totally ok but doesn't seem right .

P.S.
While writing this I was decide that to have the go.mod in the root is not incorrect.
I don't see any benefits with go.mod in every subdirectory in this scenario except of the inner satisfaction  :relieved: .

We follow golang "standard" layout

Apologies, but this is not actually a standard layout. If you look at any of the github.com/golang repos, they have no "pkg" directory. The link https://github.com/golang-standards/project-layout is in the github.com/golang-standards org, which I've never heard of before today. (We do tend to put commands in cmd/, but that's not relevant here.)
It's fine if you want to put Go code in pkg/. It just means that pkg will appear in your import path.
Moving general packages to a separate repo sounds like a reasonable solution.

OK, so it sounds like people are on board with adding a new space-separated field in the metadata, and then having the module version tag include the subdirectory name. So:
<meta name="go-import" content="nhooyr.io/websocket git https://github.com/nhooyr/websocket mod">

tagged in the repo as mod/v1.0.0. That is, go get<EMAIL_ADDRESS>would fetch the same code you'd get from go get<EMAIL_ADDRESS>(but fetching the latter would reject because the go.mod would say module nhooyr.io/websocket).
Do I have that right?
Does anyone object to this?

Based on the discussion above, this proposal seems like a likely accept.
— rsc for the proposal review group

While the current proposal seems to have popular support, another less impactful option; if the only goal is not having code at the repo root, not using an arbitrary path name[1]; is extending the major subdirectory spec for modules to allow use of v0 and v1 directories. I can also confirm that this does not work today, since I was initially unsure.
0] Credit to @wozz for the suggestion.
1] This could be important for polyglottal repos that want java, go, etc.

No change in consensus, so accepted. 🎉
This issue now tracks the work of implementing the proposal.
— rsc for the proposal review group

Any news on this proposal ?
I don't see it reflected in the go.dev/ref/mod#vcs-version so I'm assuming not yet, right ?

No update at the moment, but I'm planning to do some work on the git module fetch path for #47308 and #56881. I'll keep an eye out to see whether I can fit this in easily too.

Any update on this?

Looks like it is planned for go 1.24

Unfortunately we didn't make it in time for the 1.24 freeze and we will have to move this to 1.25


Upgrade SLF4J to version 1.7.33

2022-01-13 - Release of SLF4J 1.7.33
• SLF4J now ships with the slf4j-reload4j module delegating to the reload4j backend. Reload4j is a drop-in replacement for log4j version 1.2.17.
• SimpleLogger now prints the thread Id if instructed to do so. This fixes SLF4J-499 requested by Michael Osipov

2022-01-13 - Release of SLF4J 1.7.33
• SLF4J now ships with the slf4j-reload4j module delegating to the reload4j backend. Reload4j is a drop-in replacement for log4j version 1.2.17.
• SimpleLogger now prints the thread Id if instructed to do so. This fixes SLF4J-499 requested by Michael Osipov

FTR, I don't think mvnd is impacted by those two issues as the backend used is logback.

FTR, I don't think mvnd is impacted by those two issues as the backend used is logback.

Yes, we have not used log4j 1.x, but we do use slf4j-simple in the client.

Right, I forgot about that.  That makes we wonder if the following file is used at all:
https://github.com/apache/maven-mvnd/blob/79137867e51e04e24ae1e924afbd2e4f77e2d860/dist/src/main/distro/conf/logback-client.xml

makes we wonder if the following file is used at all:
https://github.com/apache/maven-mvnd/blob/79137867e51e04e24ae1e924afbd2e4f77e2d860/dist/src/main/distro/conf/logback-client.xml

Yes, it is probably a left-over from earlier times when we used or wanted to use logback on the client.


Docs fix spelling issues

Hello
I found and fixed several spelling issues
Hope it helps
Br, Elias.

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

Rejecting PR - we do not want to change CHANGELOG.md as it is automatically generated and manual fixing can cause issues.
And the rest of typo fixes are insignificant in code comments or in unused code.


Suggestion: minify gopherjs code

I think the gopherjs build command should have an -m command to produce minified code; the gopherjs tool is better at minifying its out than any other minifier, so this is preferable to minifying the resulting code manually.

Thanks for the suggestion, I'll think about how to set it up so the require would do minify if you wish to (probably by setting some env var).

In order to generate minified javascript code, first set the following environment variable:
NODE_GO_REQUIRE_MINIFY=TRUE


Speed up power algorithms

[ ] Benchmark the performance before and after. Nothing crazy, just time how long it takes to run calc_power() on a relatively large Activity dataframe.
[ ] Move the power package into a separate repo / pypi package.

The functionality extends beyond what I even need here. I can re-introduce it as needed.

[ ] Clean up dashboard_activity.py, removing old functionality and cleaning up what I've got. Just NGP to start, since it's compatible with TSS. Create an issue to re-introduce the commented-out stuff.

All the slowness comes in right here:
https://github.com/aaron-schroeder/distilling-flask/blob/0d875caeb199cf15576100d7b97a0b9be76ab8cb/application/plotlydash/dashboard_activity.py#L59-L92
Relating to the top-level power package:
https://github.com/aaron-schroeder/distilling-flask/tree/master/power
And heartandsole

Benchmarking
I made a big DataFrame, 3 hours' worth of data at 1-second intervals.
To start, calc_power takes anywhere from 75 to >100 seconds to run. Let's fix that.


Change QM's supervisor to Captain

About the PR

This makes the QM more inline with the other department heads.
P.S. it's 2am I should probably sleep.
Media

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

Changelog

:cl: crazybrain

tweak: QM is now officially supervised by the Captain.

Cargo Technicians and Salvage still say "You answer to the quartermaster and head of personnel". If QM being under the HoP is changed, both of their lines should be changed too.


Splitting too long messages into multiple messages

Which version are you using?
3.5.0
Splitting too long messages into multiple messages like this library does: https://github.com/php-telegram-bot/core

Hi!
Thanks for the idea, however I don't really know what you are referring to as I don't use the library you refer to.
Are you referring to the fact that the text of a message has to be less than 4096 bytes long or do you want an arbitrary stop? In that case; why does that belong in this library and is not implemented by the user?
Greetings.

I'm saying that it would be useful to insert a logic in your library that: once the character limit is exceeded, it automatically splits into several messages.
I sent you an example of a Telegram library that does this.


initialize() never resolves Future.

I'm upgrading from v2.8.0 to v2.9.1.
When running the app, the method for initialize never resolves. It only does this if I hot reload the app.
While debugging the code. iOS logs do show it's "initialized" but the Future never resolves on the Dart portion of the code.
await ZendeskMessaging.initialize(
  androidChannelKey: kAndroidChannelKey,
  iosChannelKey: kIosChannelKey,
);

Android does work without problems. At this point only iOS is giving us issues.

Any ideas?

Please try version 2.9.3 .


Fix pfor eagerness

I noticed pfor is not actually eager, though the docs say so.
Add a test and fix it.

Thanks!


Hi @KentAVP,  currently the kong.Nginx.SetCtx() method doesn't support "deep" keys.  Calling it as kong.Nginx.SetCtx("balancer_address.host", "<IP_ADDRESS>") is equivalent in Lua to ngx.ctx["balancer_address.host"] = "<IP_ADDRESS>", and not ngx.ctx.balancer_address.host = "<IP_ADDRESS>"
On a separate note, I'm not sure that setting the ctx is a documented behaviour of the balancer; it looks more like abusing some private variables, and wouldn't be surprised if a future release breaks this use.   Can it be done via documented methods, like kong.Service.SetUpstream() or kong.Service.SetTarget()?

hi @javierguerragiraldez ! ok, i got it. i already found solution. at Access method i made my own http client and use corporate proxy inside this client.
upstream and target doesn't helps to me


please check each commit with valgrind

Since this repo contains a lot of unsafe code, I suggest to check each commit with valgrind.
Note: valgrind shows some memory leaks / corruption on master branch, I will try to fix that later.

Hi @huachaohuang
Seem we can use sanitizer now. See https://github.com/rust-lang/rfcs/issues/670

Looks good to me, when can we use it?

@huachaohuang
You can try it ASAP.


KAFKA-13435; Static membership protocol should let the leader skip assignment (KIP-814)

KIP: https://cwiki.apache.org/confluence/display/KAFKA/KIP-814%3A+Static+membership+protocol+should+let+the+leader+skip+assignment
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@hachikuji @showuon Thanks for reviewing. I have updated the PR based on your comments. I have also bumped to version of the JoinGroup API to v9. I thought that we could lean on v8 which is new in 3.2 but it seems safer to bump it again just in case someone would run with trunk.


not letting me like anything.

Include version of extension. Make sure you're using last version before submitting a bug report! Check releases page before reporting anything!
i am using the latest version of the github and its not letting me like things :(

In my case Old Tweetdeck 3.6.4 will not load my Likes column. All the other columns load fine.


Doesn't the model have a TabNet regressor or maybe a multi-class classifier?

Thanks for the answer. Sorry for the delay.
You can easily add a TabNet regressor by only modifying one line in the current implementation (the head on top of the TabNet embedding). I might add an implementation later or feel free to submit a PR.
https://github.com/ostamand/tensorflow-tabnet/blob/c4caa8f48f86d55b54b6816818047f12e9dc6bf6/tabnet/models/classify.py#L63
As for multi-class classifier it should already be supported thru the n_classes argument.
https://github.com/ostamand/tensorflow-tabnet/blob/c4caa8f48f86d55b54b6816818047f12e9dc6bf6/tabnet/models/classify.py#L15


Fix race shutdown logic MPExecutor

Description
This PR fixes a race condition that causes Process.close() to be called twice, which results in a ValueError the second time. The regular shutdown logic races with invocation of the connection_lost() finalizer of the FinalizingIPCClient.

        This method releases resources held by the Process object.  It is
        an error to call this method if the child process is still running.
        '''
        if self._popen is not None:
            if self._popen.poll() is None:
                raise ValueError("Cannot close a process while it is still running. "
                                 "You should first call join() or terminate().")
            self._popen.close()
            self._popen = None
>           del self._sentinel
E           AttributeError: 'Process' object has no attribute '_sentinel'

/usr/lib64/python3.11/multiprocessing/process.py:185: AttributeError

Self Check:

[ ] Attached issue to pull request
[x] Changelog entry
[x] Type annotations are present
[x] Code is clear and sufficiently documented
[x] No (preventable) type errors (check using make mypy or make mypy-diff)
[x] Sufficient test cases (reproduces the bug/tests the requested feature)
[x] Correct, in line with design
[ ] End user documentation is included or an issue is created for end-user documentation
[ ] If this PR fixes a race condition in the test suite, also push the fix to the relevant stable branche(s) (see [test-fixes](https://internal.inmanta.com/development/core/tasks/build-master.html#test-fixes) for more info)

Will be fixed in https://github.com/inmanta/inmanta-core/pull/7299#pullrequestreview-1914355275


fix!: consider non-files as pruned officially.

Fixes #1729.
Tasks

[x] experiment with NonFile + Pruned
[x] change it to NoneFile + default status handling

I think the learning is that NonFile is a file type downstream has to deal with officially, just pruning these leads to unexpected results.
Git will officially say that it won't diff or read or add non-files, and to do that, such entries must not be pruned, but they must be part of the normal entry handling that should always or most of the time reject non-files.

That's great to hear, thanks for checking!

I wonder if there is a clearer name than NonFile.

I was wondering the same thing, but also believe to remember that name in Git somewhere.

I was wondering the same thing, but also believe to remember that name in Git somewhere.

I didn't find that, or not yet. I have a bit more detail in the #1734 description, though that PR does not change the name.

It's true, I also couldn't find that NonFile name anymore. However, I found the alternative of Untrackable so fitting as to go with that instead.


Inconsistent task description in rampup/reversingChanges across languages

I noticed that at least in some languages the task description in rampup/reversingChanges is inconsistent. Solving the task, it is clear that one commit on local and one commit on pushed needs to be reset or reverted, respectively. The English description points that out correctly: "You will revert two commits total (one per branch)."
However, at least in some other languages, this is not clear:
de_DE:
"Um diesen Level abzuschließen musst du sowohl auf local also auch auf pushed jeweils die zwei letzten Commits zurücknehmen."
-> This says to revert the last two commits on local as well as the last two commits on pushed.
I'd propose to reword this to:
"Um diesen Level abzuschließen musst du sowohl auf local also auch auf pushed jeweils den letzten Commit zurücknehmen."
The same issue seems to be present in gl, es_ES, es_AR and pt_BR, while it is apparently correct in fr_FR and sl_SI. Unfortunately, I cannot asses the other languages…

It's because the english description was changed:
https://github.com/pcottle/learnGitBranching/commit/e61a71223b3c3024d39543a76d1bf4d6f39e0440
after the translations were completed. sadly we don't have a very robust i18n system overall -- otherwise we would "trigger" a refresh of translations whenever the english source is changed.
feel free to update them! this is why I try to resist all english content updates, given that the other languages are then left with outdated versions


Important thing for people making issues

When making an issue, please double check to make sure your issue hasn't been reported already!
Here's how you can check:
In the issues tab, type your issue here.

Any issues similar to what you typed will show up. This is useful and 99% of the time can help you find the solution you're looking for. If you're ABSOLUTELY sure that nothing pops up when you type your issue in the search bar, THEN you can make the issue.

I think this is something everyone should know, but unfortunately they don't. One question, is there any way to pin this to the "Issues" tab? I know the max is 3 posts.

i'd have to unpin one of the already pinned issues in order to pin this one.


Synchronizer ergonomics

Pull request checklist
Please check if your PR fulfills the following requirements:

[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been reviewed and added / updated if needed (for bug fixes / features)
[x] Build (yarn build) was run locally and any changes were pushed
[x] Lint (yarn lint --check) has passed locally and any fixes were made for failures

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, renaming)
[x] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Closes #308
What is the new behavior?

This pull request improves the performance/stability of the synchronizer. When the unirep contract is deployed an attester with the 0 address is created. This event is used as the starting point for synchronization. The synchronizer now uses a more simple polling strategy. In the browser unirep applications tended to leak memory and become very slow after a while. I suspect this was a result of the websocket subscription approach. Now the synchronizer loads events in batches of (up to) 1000 blocks.
The circuits package has a new class CircuitConfig that can be used to specify a config for unirep circuits. This class has a static member CircuitConfig.default that contains the default config.
The UserState class no longer inherits from Synchronizer. Instead UserState has a property sync that represents the synchronizer. A UserState object can be initialized with a synchronizer that already exists. e.g. new UserState(synchronizer, identity). A single synchronizer can be used to power many user state instances.
The synchronizer class support global sync (e.g. loading events for all attesters).
The synchronizer class has fewer required arguments in the constructor. Now a synchronizer can be made like new Synchronizer({ prover, provider, unirepAddress }). By default an in memory database will be used.
Does this introduce a breaking change?

[x] Yes
[ ] No

Other information

Needs lots of docs updates

some SNARK_SCALAR_FIELD in tests still imported from @unirep/circuits but not from CircuitConfig

Yes, the index.ts in the circuit package re-exports it like export * from './CircuitConfig'

if we change the field we should use another elliptic curve? will it be another precompiled contract?

Yes, if we switch to BLS for example we would change the field constant. But for now it's the same as before.


Fixed bug with null file field

I have a null value for one of my ParseFile fields.  This results in an null pointer exception here if this remains as is.  It looks as though the current version of Parse returns null for this field, so that's what I've updated here.

:+1:


Add inputs for resource usage in submit notebook/script

Fixes #1429. Adds 3 number fields to the submit notebook / script dialogs for cpu, gpu, and ram.
The notebook/script submission dialog looks like this with the new changes:

On hover, each field has up and down arrows:

With values:

Developer's Certificate of Origin 1.1
   By making a contribution to this project, I certify that:

   (a) The contribution was created in whole or in part by me and I
       have the right to submit it under the Apache License 2.0; or

   (b) The contribution is based upon previous work that, to the best
       of my knowledge, is covered under an appropriate open source
       license and I have the right under that license to submit that
       work with modifications, whether created in whole or in part
       by me, under the same open source license (unless I am
       permitted to submit under a different license), as indicated
       in the file; or

   (c) The contribution was provided directly to me by some other
       person who certified (a), (b) or (c) and I have not modified
       it.

   (d) I understand and agree that this project and the contribution
       are public and that a record of the contribution (including all
       personal information I submit with it, including my sign-off) is
       maintained indefinitely and may be redistributed consistent with
       this project or the open source license(s) involved.

@marthacryan overall the code and the UI look good, but could we also have some basic validation similar to the ones available in node properties? It would be ok to set the ok button as disabled if negative values are added to the resources.

@lresende Looks like there isn't currently any validation happening in the file submission dialogs, so maybe we should make that a separate PR?


Stop duplicating datasources

There is a constraint on the SQL side, but it was introduces after quite big set of duplicates was created. To make the clean up process manageable - we should prevent users from creating more duplicated definitions of tables and druid datasources.
Partially helps https://github.com/airbnb/caravel/issues/1306.
Reviewers:

@mistercrunch
@ascott
@vera-liu

LGTM

Thanks !


Fix ellpack transform iter

Rework: https://github.com/dmlc/xgboost/pull/6181
I tested it with chunk size greater than MAX_INT on Linux with CUDA 11.  I also compiled it on Windows with CUDA 10.2 (but no test).

Still having issue.


Allow additional security groups for EC2 Instances

Discussion in https://github.com/hashicorp/terraform-aws-nomad/issues/13

https://github.com/hashicorp/terraform-aws-nomad/releases/tag/v0.0.5


RC1 start SSH systemd services after HDD mount

As mentioned by @Emzy in https://github.com/rootzoll/raspiblitz/issues/1785#issuecomment-730476628 it makes sense to start SSH systemd servcies after hdd mounts are done. TODO: Prepare this of RC2

Testing of RC2 works great. Closing issue for v1.6.2 release.


Add --isolate-clients option

Uses the ap_isolate option in hostapd config to disable communication between clients.

Thanks. I will test it tomorrow.

Thanks!


Shardcache not closing if shard byte size is divisible by read byte size

I've been having issues cacheing datasets pulled from S3, where some shard caches are not triggering self.close(complete=True). When I look at the cache folder, the shards are all 100% there but a few will still have their temp name with suffix ~...~. Manually removing the suffix from those few shards allows me to read the webdatset from cache just fine.
I think this behavior is because the last call to CacheStream.read(n) is not happening if the shard byte size is perfectly divisible by the read byte size n. So there is no final call (where data is expected to be None) and this block is not getting triggered:
https://github.com/webdataset/webdataset/blob/d616fe4eac6715c939d1511e6c9348319dc5e057/webdataset/shardcache.py#L76-L79
I was able to fix the issue locally by just removing all self.close(complete=True) calls inside of CacheStream, and just manually calling close() at the end of the iterator, inside def cache_shards(...):
def cache_shards(urls, cache_dir="./data", cache_size=1e15, cache_name=guess_shard, verbose=False):
...
    if not os.path.exists(cache_path):
        _cache = CacheStream(cache_path, stream, verbose=verbose)
        yield dict(url=url, stream=_cache)
        _cache.close(complete=True)

I'm not 100% sure if this is a good fix though... would love to get a second opinion / merge a stable fix!

Thanks. This code has been rewritten in v2 (soon to go live).


REPLACE statement

This adds the REPLACE statement, which as an unimplemented TODO in the source:
REPLACE $str-expr FROM $str-expr WITH $str-expr IN $str-var

example:
DATA:
string IS text
PROCEDURE:
STORE "Please reject this Pull Request." IN string
REPLACE "reject" FROM string WITH "accept" IN string
DISPLAY string crlf
REPLACE "Pull Request" FROM string WITH "lizard" IN string
REPLACE "accept" FROM string WITH "admire" IN string
REPLACE "." FROM string WITH ": 🦎" IN string
DISPLAY string crlf

output:
Please accept this Pull Request.
Please admire this lizard: 🦎

Aw yeah! This is so so so so so wonderful, thank you very much for your awesome contributions!

Please do not forget to document and implement tests for this new feature! (if possible, if you cannot do it just tell me and I'll do it) ;)

@lartu :+1: https://docs.ldpl-lang.org/text-statements/replace-from-with-in

Haha I love the example! Thank you very much!


Update DevShell to write entities via the new storage stack

/gcbrun

Running the cloud build again to see if the timeouts are flakey

Running the cloud build again to see if the timeouts are flakey

Thanks; they are. There is no way this PR could be causing any tests to fail...

Hmm, I probably wasted a bit of cloud time then. Ah well.
Thanks Myk
Joshua Pratt | Arcs |<EMAIL_ADDRESS>On Fri, May 1, 2020 at 4:01 PM mykmartin<EMAIL_ADDRESS>wrote:

Merged #5240 https://github.com/PolymerLabs/arcs/pull/5240 into master.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/PolymerLabs/arcs/pull/5240#event-3292330162, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIRUHXV2OSWBVG5IM5DSJDRPJQTNANCNFSM4MW3ZM3Q
.


Add egigeozone2 v1.0.5 to stable

Adding egigeozone2 v1.0.5 to stable

I don't understand issue E407.
In addition it would be preferred to jump directly to v1.0.6 as stable (minor changes), but v1.0.6 isnt available in latest yet.

E407 is caused by an error in LATEST repo file.
I'll fix it.
If you want to wait for 1.0.6 it's no problem. Simple create a PR when 1.0.6 has been available at latest for app 2 weeks and has no major new problems. I will close this PR then to wait for 1.0.6.


Voice calls are logged as "failed" and no calls are made

@lamby : Please use this phone number: +44 1233 800896


Icon/arrow keyboard up and filmstrip

Description

Adds arrow-keyboard-up icon to accompany the existing arrow-keyboard-down arrow icon
Vertically flips the filmstrip-layout icon so it more accurately reflects that layout
Closes #223

Pull Request Checklist

[X] Ask the contributors if a similar effort is already in process or has been solved.
[X] Review the contribution guidelines.
[X] Use staging as your pull request's base branch. (All PRs using production as its base branch will be declined).
[X] Ensure all gulp scripts successfully compile.
[X] Update, remove, or extend all affected documentation.
[X] Provide a description what is being changed, extended, or removed.
[X] Link to any issue(s) this request is resolving.
[X] Ensure no private Dialpad links or info are in the code or pull request description (Dialtone is a public repo!).
[X] Request a review from Drew Chandler, Joshua Hynes, or Ted Goas

Obligatory GIF

@vcheng-dialpad I'm adding you here so you can see arrow-keyboard-up is being added per your request.

Arrow, Keyboard Down and Arrow, Keyboard Up are interesting ways to label these. Are chevrons the same thing?

Yes, chevrons are the same, just pointed left or right. I agree the naming is wonky (should probably just be chevron-up, chevron-right, chevron-down, and chevron-left), but that's the naming pattern that exists. Changing Arrow, Keyboard Up to chevron-up is a breaking change I can't really support at the moment.


Getting error after install php

Hi Team,
I have cloned this repository. After this i created sample php configuration file with my parameters
<?php

/**
 * This demonstrates how to authenticate with LinkedIn and send api requests
 */

/*
 * First you need to make sure you've used composers auto load. You have is probably
 * already done this before. You usually don't bother..
 */
require_once "vendor/autoload.php";
//require_once "src/HappyR/LinkedIn/LinkedIn.php";

$linkedIn = new HappyR\LinkedIn\LinkedIn('mykey', 'mykey');

if ($linkedIn->isAuthenticated()) {
    //we know that the user is authenticated now. Start query the API
    $user=$linkedIn->api('v1/people/~:(firstName,lastName)');
    echo "Welcome ".$user['firstName'];

    exit();
} elseif ($linkedIn->hasError()) {
    echo "User canceled the login.";
    exit();
}

//if not authenticated
$url = $linkedIn->getLoginUrl();
echo "Login with LinkedIn";

After this when i open this file in browser i am getting error as
Fatal error: Call to a member function get() on a non-object in /usr/share/nginx/www/lnk/vendor/illuminate/support/Illuminate/Support/Facades/Facade.php on line 208

Can any one please help me to get this problem fixed

How about when I do not wish to use Composer?

I don't understand the question. If you don't use composer, composer will not download the vendors and you will not have the problem in the first place.
Would you mind elaborate your question?

You could manually download and require_once the needed files for inclusion instead of using Composer and its autoloader to do so. Which I'm doing for instance. I'm running into this problem as well. Now I simply excluded the Illuminate Session thingy, but there must be some reason you prefer to use this lib. Thats why I would like to know how to include this lib manually without using Composer or its Autoloader.
// Use the Illuminate Session storage if it is available
        /*if (class_exists('\Illuminate\Support\Facades\Session')) {
            $this->storage = new IlluminateSessionStorage();
        } else {
            $this->storage = new SessionStorage();
        }*/
        $this->storage = new SessionStorage();

This line if (class_exists('\Illuminate\Support\Facades\Session')) { checks if the class exists. If it does (then you are using Illuminate already) we use the special IlluminateSessionStorage. Otherwise, just use the normal SessionStorage.
Since you manually download the files you will not download the get the Illuminate library.
It seams like you already have downloaded the Illuminate lib and not really using it. Could that be the case?

if (class_exists('\Illuminate\Support\Facades\Session')) I do have the class but I guess its the wrong one because I'm getting Fatal error: Call to a member function get() on a non-object in <$path> vendor/illuminate/support/Illuminate/Support/Facades/Facade.php on line 208 as well. And actually the Facade.php and Session.php class neither have the get() function.
I do have \Illuminate\Support\Facades\Session installed. It came from this repo: https://github.com/illuminate/support but i'm guessing its not the same one?

That's the one.
I can see that it is a problem here. You have the class loaded but you don't want to use the IlluminateSessionStorage. Can you see a good solution to this problem?

Well I would like to use that one, since you have your reason to do so. But when I do I get this error:
Fatal error: Call to a member function get() on a non-object in <$path> vendor/illuminate/support/Illuminate/Support/Facades/Facade.php on line 208
As above a fix is to remove the vendor folder and reinstall all dependencies by using composer. But I do not use Composer. I need to manually include the files needed. So how can I get this to work when i'm not using composer.

Thank you for highlighting a problem. You have loaded the Illuminate\Support\Facades\Session class but you do not which to use the IlluminateSessionStorage. Do you have a solution to this problem?
If you want to use the Illuminate\Support\Facades\Session for your sessions (like the Laravel framework does) I suggest you to read its documentation. It is out of scope for this lib and I'm not familiar with it.


Fix a few issues for AOT compilation

Fix the logging in SpmdTrainer when devices list is present.
In flash attention, obtain the backend type from physical_mesh instead of jax.default_backend().
Allow custom data_dir for AOT compilation.

Oops I didn't have the chance to address your comments before it got auto-merged. I can create another PR for the stylistic changes or leave it as is. WDYT? @markblee

No worries, they were nitpicks -- feel free to ignore/address in the future when you have spare cycles.


Request: Speed up the uniform mesh converter

The uniform mesh converter can sometimes take a very long time. However the majority of what it does is loop over all the assemblies in the core and assign parameters back and forth between blocks.
Calls to setAssemblyStateFromOverlaps are all completely independent and could benefit from parallel execution. Based on looking at the uniform mesh generator, this should cover the majority of the work in that class and I think could speed up the conversion process a ton.

Can we at least run the operation through the profiler first to see exactly what the slow bottlenecks are with precision instrumentation? Then we can see if there's any superlow hanging fruit before parallelizing.

Yes, a very good idea! This was just based on the keckler-profiler, aka my eyeballs looking at the exploding runtimes.

Can we at least run the operation through the profiler first to see exactly what the slow bottlenecks are with precision instrumentation? Then we can see if there's any superlow hanging fruit before parallelizing.

I did this months ago but didn't write seem to write it down. It is slow on getting and setting number densities for each block in the core. I feel like this suggest keeps getting made. @john-science did timing tests of this already a couple months ago too.

A good process for speeding up code goes something like:

Profile the code.
Review the code profile and see where the slow-downs are and what expensive operations are being called inside nested loops that can easily be improved.
See what data structures are being used.
See if we can lean more on NumPy versus pure Python
and on and on and on...
The very last act of desperation is deciding you have sped up the code as much as humanly possible and you need to parallelize it.

It is not appropriate to say "We need to parallelize this code" before you've done any work to improve it's performance.

Also, and not for nothing, you want to parallelize the uniform mesh converter?  Even though the ARMI model is already run in parallel?
You can't spawn parallel code INSIDE code that's already parallel.
Is the uniform mesh converter run outside the already parallel code in the ARMI interface loop?

I'm still experimenting with the profiler, but here are some preliminary results. I ran a script that loads up a reactor, does a uniform mesh conversion and then de-conversion. 50% of the overall run time is spent in makeAssemWithUniformMesh, with most of that being spent in this block.deepcopy():

I don't think we really need a deepcopy here. We just want a new block that has the same name and as the original. We're going to overwrite everything else (height, number densities, parameters, etc.) We could probably cut down a good chunk of this overhead if we avoid a deepcopy.

Great progress @mgjarrett !
Removing a deepcopy() is usually a win.  In most languages, doing a "deep copy" of an object is an expensive operation.  So, that makes good sense.
My first though on __getitem__() was "that's a system library, you're not going to speed that up". But I'm glad to see that it all comes down to one line that couple be optimized.
You're burying the lead though.  I've always here it was the number densities that were the slowest part of ARMI, specifically the meshing.  So... you did an actual profile and found otherwise.  That's great!  Way to use analytical data, instead of just trusting the standard wisdom.  You rock.


Update searching.rst

https://github.com/mattermost/mattermost-server/pull/11196

I wanted to confirm whether the statement under "PostgreSQL" ("Terms containing a dash return incorrect results as dashes are ignored in the search engine.") is still correct? It seems to contradict the addition of the "-" feature but I may be interpreting it incorrectly.

@justinegeffen I just tested the exclusion on a server with Postgres and can confirm it is working as expected.  I also tested searching returning the word "sign-up" and there still seems to be some unexpected behavior - ie: "sign up" returns when search term is "sign-up".  Probably best to keep that section in the docs.


Localization language change clears QueryString

I use some query params for user registration or other purposes:
http://localhost:5000/Controller/Action?queryParam=value
After user changed localization language to different, SelectLanguage.cshtml posts
var returnUrl = string.IsNullOrEmpty(Context.Request.Path) ? "/" : $"{Context.Request.Path.Value}";
back, page will be re-rendered, but without query params:
http://localhost:5000/Controller/Action
Can be easy fixed by changing SelectLanguage.cshtml to this one:
var returnUrl = string.IsNullOrEmpty(Context.Request.Path) ? "/" : $"{Context.Request.Path.Value}{Context.Request.QueryString}";
I can create PR with fix, just not sure if it's good idea.

@zvasilius - thank you for reporting this issue, for sure, please you can make a PR. 👍


Setup codacy for automated code reviews

Feature Request
@Lavanyagaur22 can you go ahead and setup codacy for java and kotlin files in the project?
That should help us with automated code reviews for the code we push.
P.S. this shouldn't require you to push any code to the repo; just link the project from the codacy dashboard.

@harshithdwivedi Would not it include adding codacy.yml to the repo?

+1 for badge. Let's try to use all badge from https://shields.io/

Ah yes, you will need to add the badge.
I'm not sure about the yaml, but if the steps require you to do so, feel
free to do so.
On Mon, Jun 3, 2019, 1:51 AM Daniel Passos<EMAIL_ADDRESS>wrote:

+1 for badge. Let's try to use all badge from https://shields.io/
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/aerogear/voyager-android/issues/17?email_source=notifications&email_token=ALLWCVHQZCTVTRFT7TCIUCDPYQTUTA5CNFSM4HSCZ4AKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWX5HQI#issuecomment-498062273,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALLWCVGCEBUZLCUZEJ5ZCMTPYQTUTANCNFSM4HSCZ4AA
.

Let me know if you will need any admin rights for attaching service to repo

@wtrocki I've configured the Codacy for this repo already. I mean gave the Codacy access to add the webhooks


nuxt generate produces really unexpected result when using */index.html URLs

Version
v2.3.4
Reproduction link
https://github.com/nuxt/codesandbox-nuxt
Steps to reproduce

git clone https://github.com/nuxt/codesandbox-nuxt/
cd codesandbox-nuxt
add code to pages/about.vue component:  mounted() { console.log('JS CHECK') }
npm i
npm run generate
npm install http-server
npx http-server dist
open http://localhost:8080/about/index.html

What is expected ?
Working app or error message.
What is actually happening?
Nothing!
Additional comments?
I spent a 1.5 day trying to get a dependency for some dynamic functionality to work. When everything else was working fine, including npm run dev. One error message in console would be enough...

This bug report is available on Nuxt community (#c8299)

Is this problem solved？

Why js dose not work ？

Why js does not work ？

Is this problem solved？

No, absolutely the same behavior. Steps to reproduce from the beginning are actual.


Tag directive added to stitched schema with EnableTag = false, by default (EnableTag = true) it causes an exception

Is there an existing issue for this?

[X] I have searched the existing issues

Product
Hot Chocolate
Describe the bug
13.6.0 introduced a tag directive which caused an issue described in https://github.com/ChilliCream/graphql-platform/issues/6634.
However, the workaround of EnableTag = false doesn't appear to work as intended on the Stitching gateway.
Without the EnableTag=false on the Stitching gateway it will generate the same exception despite all downstream graphs having EnableTag = false and no tag directive is present there
With EnableTag=false on the Stitching gateway it will work, however the stitched schema will still have a tag directive present even though you specifically asked to disable it.
Steps to reproduce

Define two subgraphs both with EnableTag = false , one having an entity and the other pointing to that entity by the id
Define a stitching gateway without EnableTag = false with an extension similar to: extend type DefaultEntity { secondary: SecondaryEntity @delegate (schema: "TestRemoteSchemaSecondary", path: "mySecondaryEntityBatched(id: $fields:secondaryId)") }
Perform a stitched query through the gateway, it will cause an exception
Change the stitching gateway to have  EnableTag = true
Perform a stitched query through the gateway, it will not cause an exception but the tag directive is present

Relevant log output
No response
Additional Context?
Like this it won't be possible to stitch two stitched gateways, and out of the box a stitching gateway won't work despite no tag directives in downstream graphs.
Version
13.6.0

We will be not investing any time anymore into stitching. If you are depending on stitching stay on the latest version that worked for you ... its really EOL and we will remove it from the build by the end of December.


Adds golang support

This adds go support for ifsc for the following features:

Language
Validation
API Client
Sublet Support (Custom)
Bank Constants

Go
✅
✅
✅ ( ✅ )
✅

Items:

[x]  Test coverage.(81.8%)
[x]  Readme updated
[x]  UPI-enabled bank list updated. (there is some conflict, added comment below.)

Let me know if I missed something.

While adding bank codes for new upi enabled banks, there were few duplicate names. Please help to resolve which one to go forward with:

Citizen Co-operative Bank Ltd - Noida:

"CZCX": "Citizen Co-operative Bank",(current choice)
"CCBX": "Citizen Co-operative Bank”

Satara District Central Co-operative Bank

"SDSX": "Satara District Central Co-operative Bank”,
"SDCE": "Satara District Central Co-operative Bank”, (current choice)

Please add go build to wercker pipeline.
Also, go tests are not written in tests/go dir since go test doesn't give coverage report if test and source files are present in different dir.

there were few duplicate name

both stay. We keep old bank codes and names across the repo.

Closing this for #245


Improve Authentication Flow for User Registration

User story
As a user, I want to be able to login to Keycloak after IDP settings reconfiguration.
The problem we have now is that Keycloak is restarting the registration process after IDP settings updates.
The current configuration for the User Registration process does not allow re-submitting registration or updating existing user entry using the Browser flow.
Tasks
core:

[ ] Investigate alternative Authentication Flow methods
[ ] Implement the custom user registration flow

tasks:

[ ] Update source code to reflect new settings

Definition Of Done

[ ] Tasks are done
[ ] (Unit) Tests are performed
[ ] (Architecture) Design Record(s) have been added as adr/*.md and appended to list in adr/_index.md
[ ] Specs and demo files have been updated to reflect code changes
[ ] Documentation has been updated
[ ] Functionality, code, and/or documentation has been peer reviewed
[ ] Relevant team members have been notified

@rawc0der is it still valid ?

@rawc0der is it still valid ?

Somehow I was still able to reproduce the issue on my local, regardless of the previous "update" we did. We can keep this open imo.

Somehow I was still able to reproduce the issue on my local, regardless of the previous "update" we did. We can keep this open imo.

The current spec is probably broke for some Admin REST api calls, although the part of the API for AuthenticationFlowManagement is working. Currently I did not manage to extract all proper calls to create an AuthenticationFlow from scratch.
Closing this issue as I did not make a lot of progress without an officially provided openAPI spec / official npm package or examples.

The current spec is probably broke for some Admin REST api calls, although the part of the API for AuthenticationFlowManagement is working. Currently I did not manage to extract all proper calls to create an AuthenticationFlow from scratch.
Closing this issue as I did not make a lot of progress without an officially provided openAPI spec / official npm package or examples.


fix multiple private typo in script tag

fix https://github.com/intercom/intercom-rails/issues/197

🐑


Expand documentation on HF ticking

Closes #211

Almost forgot. #211 mentions that we should have unit tests for extendToSlot. Are we still going to do that?


Cannot require() starting with version 4

Starting with version 4.0.0, it is not possible to import d3-time-format in a script using require() syntax, for example:
const { timeParse, timeFormat } = require('d3-time-format');

The error is:
internal/modules/cjs/loader.js:1102
      throw new ERR_REQUIRE_ESM(filename, parentPath, packageJsonPath);
      ^

Error [ERR_REQUIRE_ESM]: Must use import to load ES Module: /path/to/myprogram/node_modules/d3-time-format/src/index.js
require() of ES modules is not supported.
require() of /path/to/myprogram/node_modules/d3-time-format/src/index.js from /path/to/myprogram/myprogram.js is an ES module file as it is a .js file whose nearest parent package.json contains "type": "module" which defines all .js files in that package scope as ES modules.
Instead rename index.js to end in .cjs, change the requiring code to use import(), or remove "type": "module" from /path/to/myprogram/node_modules/d3-time-format/package.json.

Could you restore support for using require(...) syntax instead of requiring import ... syntax? Note that developers can still opt to use import ... syntax if they like, this would just restore support for require(...) which is quite convenient when quickly writing utility scripts.

I'm sorry that you don't see this as an issue and have simply closed it. I'll just provide one article worth reading, paying special attention to section How to Create a Good “Dual Package” Containing Both CJS and ESM: https://redfin.engineering/node-modules-at-war-why-commonjs-and-es-modules-cant-get-along-9617135eeca1

I deliberately dropped support for CJS. I have no interest in maintaining multiple module formats and encourage everyone to adopt ES modules. If you disagree, you are free to fork this module, to use an earlier version, or simply to not use it. Thank you for your feedback.

I understand. I appreciate the response.


iPhone X, iPhone SE and Samsung Skins

Hi guys,
Will the iPhone 8 skin suffice for the iPhone X email client preview? If not, are there any plans to add it in to here?
Thanks again :-)

Also, I've notice there is now the iPhone SE too, along with the Samsung Android. Should I just use the iPhone 5s for the SE and the previous android device for the Samsung or is there specific skins for those too?

Hey Lloyd,
Sorry no one has gotten back to you. Let me see if I can track down someone to create the art for it. The Samsung and iPhone SE vs 5S are a design decision on our team, we usually try to represent the device and not re-use to avoid customer confusion.
Thanks,

Ed

@adnan959 looks like you last helped us with this. I'm going to create a JIRA in our board. Can you just update this git issue when the work has been completed or handed off to someone else? Thank you!

Hey @lloydpowell88,
I've just added the missing device samples, while we're re-using the same skins for the iPhoneSE and Samsung Android, the iPhone X one is significantly different (https://github.com/litmus/litmus-mobile-samples/commit/cedd5d3d2aa81511fb317ea7aa0480223ea6ffa7).
I hope this helps, if you have any additional questions -- just let us know!
Thanks,
Adnan


Website: Broken subscription form

Status: In progress
Issue Description -
Any email shared through the form is currently going nowhere.
No email gets sent to any new subscribers.
We are unsure where the emails get stored.

Link -
https://www.zesty.io/mindshare/customer-stories/
Fix -
Form should be fixed for lead capture. Every lead should be captured and stored in CRM as a ‘Mindshare MQL’.
An automated welcome email should be sent to every new subscriber/MQL.
Form should be registered as a component that marketers can drag and drop, and use in any landing page.

Upon checking looks like the Cloudfunctions has issues, will debug further on the issues


Check if sshpass is available

Withouth sshpass installed I get this error:
Usage: mccli [OPTIONS] COMMAND [ARGS]...

Error: No such command 'ssh-oidc-demo.data.kit.edu'.

Which makes sense, if you know the cause :)
Maybe adding a short "Prerequisites" section to installation.rst:
Prerequisites:
=============
Required dependencies:
- `sshpass`: This package is available on most linux distributions as `sshpass`
- `oidc-agent`: This package is available on most linux distributions as `oidc-agent` (use `oidc-agent-cli` on systems without GUI)

sshpass is not required by mccli, it is only an alternative to mccli, and --dry-run gives you the sshpass command after deploying the local account.
What command did you run to get this error?
It seems like you might be missing the ssh in mccli ssh ssh-oidc-demo.data.kit.edu.


New Release

Please tag current master as 1.4.1.

Done :shipit: :tada:
https://github.com/clue/php-stream-filter/releases/tag/v1.4.1


computed_field: incorrect return_type from of wrapped method should raise an error

Initial Checks

[X] I have searched Google & GitHub for similar requests and couldn't find anything
[X] I have read and followed the docs and still think this feature is missing

Description
Hi there, the computed_field decorator currently accepts a return_type argument and raises an error if an incorrect type is returned. The computed_field decorator can also infer the return type from the wrapped method, however it logs a warning when an incorrect type is returned.
I have a use case where computed fields in my schema need to be strict with the types they return. Extracting the return type of a computed field is also useful for us.
For example this code would log a warning:
@computed_field
def get_id(self) -> int:
    return 'hello world!'

Whereas this code would raise an error:
@computed_field(return_type=int)
def get_id(self):
    return 'hello world!'

We could use both, but this feels repetitive and the method and the decorator can accept different types. For example this passes without any errors:
@computed_field(return_type=str)
def a_filled_with_b(self) -> int:
    return 'hello world!'

It would be good if these two would behave in the same way for consistency. This would be more consistent with the pydantic schema syntax.
Affected Components

[ ] Compatibility between releases
[ ] Data validation/parsing
[ ] Data serialization - .model_dump() and .model_dump_json()
[ ] JSON Schema
[ ] Dataclasses
[ ] Model Config
[ ] Field Types - adding or changing a particular data type
[ ] Function validation decorator
[ ] Generic Models
[X] Other Model behaviour - model_construct(), pickling, private attributes, ORM mode
[ ] Plugins and integration with other tools - mypy, FastAPI, python-devtools, Hypothesis, VS Code, PyCharm, etc.

Seems reasonable, but might have to be a change that waits for V3

On my side both of the examples you shared just raise warnings;
This warning:
/usr/local/Caskroom/miniconda/base/envs/jamble-be/lib/python3.11/site-packages/pydantic/main.py:441: UserWarning: Pydantic serializer warnings:
  Expected `int` but got `str` with value `'haha'` - serialized value may not be as expected
  return self.__pydantic_serializer__.to_json(
Out[6]: '{"user_id":"22","coupon_id":"hh","amount":23,"sss":"haha"}'

From this code:
from pydantic import BaseModel, ConfigDict, Field, computed_field, field_serializer, model_validator

class CreateCouponInput(BaseModel):
   ...:     user_id: str
   ...:     coupon_id: str
   ...:     amount: int
   ...:
   ...:     @computed_field(return_type=int)  # type: ignore
   ...:     def sss(self):
   ...:         return "haha"

bb = CreateCouponInput(user_id="22", coupon_id="hh", amount=23)
bb.model_dump_json()

I am expecting a TypeError to be raised as per the docs but that never happens, am I doing something wrong? @sydney-runkle

@aelamran,
I think you can do:
from pydantic import BaseModel, ConfigDict, Field, computed_field, field_serializer, model_validator

class CreateCouponInput(BaseModel):
    user_id: str
    coupon_id: str
    amount: int

    @computed_field(return_type=int)
    def sss(self):
        return "haha"

c = CreateCouponInput(user_id="1", coupon_id="2", amount=3)
print(c.model_dump(warnings='error'))
"""
pydantic_core._pydantic_core.PydanticSerializationError: Pydantic serializer warnings:
  Expected `int` but got `str` with value `'haha'` - serialized value may not be as expected
"""


Mast: Watch topcomputersciencedegree.com

Mast requests the watch of the watch_keyword topcomputersciencedegree\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

topcomputersciencedegree\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Makyen in Charcoal HQ

Approved by Makyen in Charcoal HQ


Update plugin version in readme

Pasting from the readme into my random build.gradle and running the checkNewVersions task helpfully points out:
Plugin upgrades available for project 'random-project'
- com.markelliot.versions:com.markelliot.versions.gradle.plugin:{0.1.11 -> 0.42.0}


Better instructions for spacemacs

I had difficulty getting this to work in spacemacs with the instructions that were provided. I did ultimately get it working, so I wanted to share my config. I ultimately had to add a bunch of stuff to dotspacemacs/user-config:
(defun dotspacemacs/user-config ()
  "Configuration for user code:
This function is called at the very end of Spacemacs startup, after layer
configuration.
Put your configuration code here, except for variables that should be set
before packages are loaded."
  (defvar bootstrap-version)
  (let ((bootstrap-file
         (expand-file-name "straight/repos/straight.el/bootstrap.el" user-emacs-directory))
        (bootstrap-version 5))
    (unless (file-exists-p bootstrap-file)
      (with-current-buffer
          (url-retrieve-synchronously
           "https://raw.githubusercontent.com/raxod502/straight.el/develop/install.el"
           'silent 'inhibit-cookies)
        (goto-char (point-max))
        (eval-print-last-sexp)))
    (load bootstrap-file nil 'nomessage))
  (use-package copilot
    :straight (:host github :repo "zerolfx/copilot.el" :files ("dist" "*.el"))
    :ensure t)
  (setq copilot-node-executable "/home/<username>/.nvm/versions/node/v16.16.0/bin/node")
  (with-eval-after-load 'company
    ;; disable inline previews
    (delq 'company-preview-if-just-one-frontend company-frontends))
  (define-key copilot-completion-map (kbd "<tab>") 'copilot-accept-completion)
  ;; (define-key copilot-completion-map (kbd "TAB") 'copilot-accept-completion)

  (add-hook 'prog-mode-hook 'copilot-mode)

  (define-key evil-insert-state-map (kbd "C-<tab>") 'copilot-accept-completion-by-word)
  (define-key evil-insert-state-map (kbd "C-TAB") 'copilot-accept-completion-by-word)  )

This looks a little hack to me, but it works. If there's a better way to handle that, I'd like to know and help update the instructions for others.

Thanks for sharing your configuration. Good to know you can use straight.el in spacemacs.
Spacemacs allows adding packages directly from GitHub by changing the dotspacemacs-additional-packages variable. Doesn't it work?

I just tried setting it up and it (almost) works. I had to wrap the define-key calls in (with-eval-after-load 'copilot ...) as it would not find the completion map otherwise.
I do have a few problems: the autocompletion doesn't trigger automatically for copilot - I have to invoke copilot-complete, and copilot-mode isn't automatic in programming modes (I use LSP, maybe that complicates things). Not sure how to fix these, so any help is appreciated. But otherwise things do work, I get completions and can accept them.

#60 may help you

I'm getting the following w/ @milanmitrovic 's suggestion:
Cloning https://github.com/melpa/melpa.git to /Users/jleonard/.emacs.d/.cache/quelpa/melpa
Fetcher: github
Source: zerolfx/copilot.el

Cloning https://github.com/zerolfx/copilot.el.git to /Users/jleonard/.emacs.d/.cache/quelpa/build/copilot/
/Users/jleonard/.emacs.d/.cache/quelpa/build/copilot/copilot.el -> /var/folders/qk/8w2j623n6hj3yhm2xb959gd8y93h_4/T/copilotig9WPE/copilot-20221114.1757/copilot.el
/Users/jleonard/.emacs.d/.cache/quelpa/build/copilot/dist => /var/folders/qk/8w2j623n6hj3yhm2xb959gd8y93h_4/T/copilotig9WPE/copilot-20221114.1757/dist
Parsing tar file...done
Error getting PACKAGE-DESC: (wrong-type-argument arrayp nil)

Is there any better (up to date) guidance on getting this working w/ spacemacs?

First, many thanks to @zerolfx for making this! I'm officially a copilot addict and will build side projects around new AI tools as a result. life-changing.
Second, here's my Spacemacs config, in the hope it helps someone.
   dotspacemacs-additional-packages '(org-super-agenda git-auto-commit-mode prettier-js
                                                       (copilot :location (recipe
                                                                           :fetcher github
                                                                           :repo "zerolfx/copilot.el"
                                                                           :files ("*.el" "dist")))
                                                       )

And top of user-config:
  (with-eval-after-load 'company
    ;; disable inline previews
    (delq 'company-preview-if-just-one-frontend company-frontends))

  ;; The with-eval-after-load prevents initial loading errors. Without this I had to comment out the two define-keys, load, uncomment, reload.

  (with-eval-after-load 'copilot
    (define-key copilot-completion-map (kbd "<tab>") 'copilot-accept-completion)
    (define-key copilot-completion-map (kbd "TAB") 'copilot-accept-completion)
    )
     (define-key evil-insert-state-map (kbd "<backtab>") 'copilot-next-completion)

  (add-hook 'prog-mode-hook 'copilot-mode)
  (define-key evil-insert-state-map (kbd "C-<tab>") 'copilot-accept-completion-by-word)
  (define-key evil-insert-state-map (kbd "C-TAB") 'copilot-accept-completion-by-word)

My Spacemacs is up to date, GUI OSX, and this all works well for me.

@jleonard-r7 Using TAB to accept completion is the default behavior of VSCode, so I tried to make them consistent.

Ah yea. Makes sense. My keybinding space in general is starting to get pretty crowded so it could be particularly bad in my case vs a barebones install.

For mac users,
change this path to
"/home/<username>/.nvm/versions/node/v16.16.0/bin/node"

this. (found by which)
"/usr/local/bin/node"

The path for node is very much user and system specific. I don’t think we can make any prescriptions here.

First of all, thanks for all the work here.
Does this only work in evil mode? I use spacemacs with emacs keybindings (so no insert mode) and I'm not seeing the autocompletion appear until I explicit run copilot-complete in the buffer

@jleonard-r7 still not working for me.
To be clear I have this in dotspacemacs-configuration-layers
     (auto-completion :variables
                      auto-completion-enable-snippets-in-popup t)

This for dotspacemacs-additional-packages
   dotspacemacs-additional-packages
   '((copilot :location (recipe
                         :fetcher github
                         :repo "zerolfx/copilot.el"
                         :files ("*.el" "dist"))))

And this in user-config
  (with-eval-after-load 'company
    ;; disable inline previews
    (delq 'company-preview-if-just-one-frontend company-frontends))

  (add-hook 'prog-mode-hook 'copilot-mode)
  (define-key evil-insert-state-map (kbd "M-<right>") 'copilot-accept-completion-by-line)
  (define-key evil-insert-state-map (kbd "M-<return>") 'copilot-accept-completion)
  (define-key evil-insert-state-map (kbd "M-<tab>") 'copilot-next-completion)

Now when I open a buffer and enable company-mode no auto completion prompts are popping up.


problem with homebridge 1.3.4

Hello till last version all was ok, with the update to 1.3.4 now I get this problem.
Any idea?
many thanks
[6/5/2021, 00:32:36] [homebridge-withings-air-quality] This plugin generated a warning from the characteristic 'Battery Level': characteristic value expected valid finite number and received "undefined" (undefined). See https://git.io/JtMGR for more info.
(node:21526) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'value' of undefined
at r.getTemperature (/usr/local/lib/node_modules/homebridge-withings-air-quality/dist/index.js:4:2582)
at i.updateTemperature (/usr/local/lib/node_modules/homebridge-withings-air-quality/dist/index.js:6:5207)
at /usr/local/lib/node_modules/homebridge-withings-air-quality/dist/index.js:6:3615
at i. (/usr/local/lib/node_modules/homebridge-withings-air-quality/dist/index.js:6:4575)
at /usr/local/lib/node_modules/homebridge-withings-air-quality/dist/index.js:6:1529
at Object.next (/usr/local/lib/node_modules/homebridge-withings-air-quality/dist/index.js:6:1634)
at /usr/local/lib/node_modules/homebridge-withings-air-quality/dist/index.js:6:547
at new Promise ()
at t (/usr/local/lib/node_modules/homebridge-withings-air-quality/dist/index.js:6:295)
at CurrentTemperature. (/usr/local/lib/node_modules/homebridge-withings-air-quality/dist/index.js:6:4505)
(node:21526) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag --unhandled-rejections=strict (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 3)
(node:21526) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'value' of undefined
at r.getCarbonDioxide (/usr/local/lib/node_modules/homebridge-withings-air-quality/dist/index.js:4:2513)
at i.updateAirQuality (/usr/local/lib/node_modules/homebridge-withings-air-quality/dist/index.js:6:4891)
at /usr/local/lib/node_modules/homebridge-withings-air-quality/dist/index.js:6:3409
at i. (/usr/local/lib/node_modules/homebridge-withings-air-quality/dist/index.js:6:4575)
at /usr/local/lib/node_modules/homebridge-withings-air-quality/dist/index.js:6:1529
at Object.next (/usr/local/lib/node_modules/homebridge-withings-air-quality/dist/index.js:6:1634)
at /usr/local/lib/node_modules/homebridge-withings-air-quality/dist/index.js:6:547
at new Promise ()
at t (/usr/local/lib/node_modules/homebridge-withings-air-quality/dist/index.js:6:295)
at AirQuality. (/usr/local/lib/node_modules/homebridge-withings-air-quality/dist/index.js:6:4505)
(node:21526) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag --unhandled-rejections=strict (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 4)

we all kinda seem to be experiencing the same issue 🤷♂ i hope it gets fixed soon; i would love to help but i'm a total noob - would love to invest my time into this if anybody happens to know how to point me in the right direction though

My guess is that they Withings changed something regarding the Data available via their API.  When I looked whether or not I could try and help with fixing this plugin, I found that there is a Home Assistant plugin that integrates with the Withings API via OAuth.  However this plugin is not able to access the Temperate sensor data.  So I don't know if there is much hope of a resolution to this.

thanks, do you happen to have a link to that?

This is the plugin page on HA: https://www.home-assistant.io/integrations/withings/ and the source is https://github.com/home-assistant/core/tree/dev/homeassistant/components/withings :)

Hi, as stated in the link returned by the warning (https://git.io/JtMGR), this message should not prevent the plugin from working as Homebridge will correct the value to a valid one.  This happens because the newest versions of Homebridge appears to do more data validation, and the plugin probably doesn't set a valid value for the characteristic before retrieving the data from Withings API.
As I'm not actively maintaining this plugin, I can't provide you with an estimated timeframe for a fix, but I'll gladly accept pull requests that resolves this issue.

Thank you very much. Tested and working!

Many thanks tested and working!


Add eurijon as a Reviewer

As a pull request owner and reviewers, I checked that:

[x] Appropriate testing is done and actually running

/verify-owners

/approve

/lgtm


[Azure Search] Add -InstanceName parameter to catalog2azuresearch and db2azuresearch

This means we need to update the version of NuGet.Services.Validation.Common.Job in NuGet.Services.AzureSearch and then update the command line parameters for the jobs.


Added Machine Learning Folder with Readme

Hey, I created a Machine Learning folder with a Readme. Please check and merge,

@amitrakshar01 You don't need to give PR to main for now. Please work on the corresponding branch only


[kubernetes] Complete log solution

I'm submitting a ...

[ ] bug report
[x] feature request
[ ] support request

Do you want to request a feature or report a bug?
Feature

What is the current behavior?
We don't have a solution for storing and searching logs

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem

What is the expected behavior?
A complete solution for storing and filtering logs

What is the motivation / use case for changing the behavior?
To store and to filter logs

Please tell us about your environment:

Version: 0.2.x
Environment: [kubernetes, docker-compose, standalone]
Operating system: [Ubuntu 16.04, Fedora 28, Ubuntu on VirtualBox (Windows 10), etc.]

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, gitter, etc)

Linked to dojot/dojot#1483


aba: init at 0.7.0

Description of changes
Initialize aba, an address book for the aerc mail client, at 0.7.0.

Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] 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

[ ] 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
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 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.

@GrahamcOfBorg eval

@grahamcofborg eval


Filename change in Hugo 0.20.3 binaries

They are now prefixed with "v" ... and I did not notice it.
See https://github.com/goreleaser/goreleaser/issues/210

Attempting to deploy 0.20.5 with Netlify throws an error due to the renamed binary, even with a cleared build cache - is this related to #3385? Apologies if not. Log below.
8:50:51 PM: Installing Hugo 0.20.5
8:50:53 PM: Cleaning up docker container
8:48:09 PM: Error running command: Build script returned non-zero exit code: 1
8:50:54 PM: Error running command: Build script returned non-zero exit code: 1
8:50:54 PM: Finished processing build request
8:50:53 PM: Error during Hugo 0.20.5 install: error renaming /opt/build/cache/.binrc/binaries/spf13/hugo/v0.20.5/binrc-302612769/hugo_0.20.5_linux_amd64/hugo_0.20.5_linux_amd64 to /opt/build/cache/.binrc/binaries/spf13/hugo/v0.20.5/hugo: rename /opt/build/cache/.binrc/binaries/spf13/hugo/v0.20.5/binrc-302612769/hugo_0.20.5_linux_amd64/hugo_0.20.5_linux_amd64 /opt/build/cache/.binrc/binaries/spf13/hugo/v0.20.5/hugo: no such file or directory

@tombennet I have notified Netlify about the change, but they haven't got to updating their scripts. I kind of did not think about this side effect, but we had to do this to get away from the unmaintained goxc as a build system. And it is much cleaner this way.
So from Hugo >= 0.20.3, the binary inside the archive is just named hugo (or hugo.exe on Windows.)
/cc @biilmann

Ok cool, makes sense. Thanks @bep @biilmann


Unexpected warnings in Local Mode (chef-client -z) when cookbook_path has multiple entries

When you pass in multiple entries to cookbook_path using chef-client -z things get screwy.
Key Info

Version: Chef Client 11.14.0.rc.2 (as shipped with ChefDK)
Platform: OSX 10.10 DP5

Seems to be reproducible on other platforms so I'm not sure this is specifically due to the above?
Attempted Flow
Very simply what I want is chef-client -c etc/client.rb -z -o recipe[baa] and to load recipe[baa] from cookbooks/ and then have that depend on recipe[sheep] which is located in site-cookbooks/. Obviously you cannot Baa without having a Sheep.
Expectation
I would expect to be able to pass in multiple things to cookbook_path and to not result in that causing entries in nodes/ (or other directories) to be evaluated more than once, and to not see warnings.
I've spelunnnked as much as possible but I'm getting lost in ChefFS and MultiplexedDir with regards to this bug, there's just too much metaprogramming going on there, it's significantly over my head. Everyone on IRC suggested @jkeiser is the most experienced at understanding this stuff.
Observations / Steps to Reproduce
I have made the following observations which may help you nail down the specific bug.
Providing an invalid cookbook_path
If I update the entries in client.rb to specify /root/chef/cookbooks and /root/chef/site-cookbooks then the entire run fails, because chef-client tries to look in /root/chef/nodes for the JSON data.
stone:chef agh$ chef-client -c etc/client.rb -z -o recipe[baa]
Starting Chef Client, version 11.14.0.rc.2
[2014-08-07T22:34:31+01:00] ERROR: #<Errno::ENOENT: No such file or directory @ dir_s_mkdir - /root/chef/nodes>
/opt/chefdk/embedded/apps/chef/lib/chef/chef_fs/file_system/chef_repository_file_system_root_dir.rb:57:in `mkdir'
/opt/chefdk/embedded/apps/chef/lib/chef/chef_fs/file_system/chef_repository_file_system_root_dir.rb:57:in `block in create_child'
/opt/chefdk/embedded/apps/chef/lib/chef/chef_fs/file_system/chef_repository_file_system_root_dir.rb:55:in `each'
/opt/chefdk/embedded/apps/chef/lib/chef/chef_fs/file_system/chef_repository_file_system_root_dir.rb:55:in `create_child'
/opt/chefdk/embedded/apps/chef/lib/chef/chef_fs/chef_fs_data_store.rb:406:in `get_dir'
/opt/chefdk/embedded/apps/chef/lib/chef/chef_fs/chef_fs_data_store.rb:395:in `with_dir'
/opt/chefdk/embedded/apps/chef/lib/chef/chef_fs/chef_fs_data_store.rb:68:in `create'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-zero-2.2/lib/chef_zero/data_store/v1_to_v2_adapter.rb:56:in `block in create'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-zero-2.2/lib/chef_zero/data_store/v1_to_v2_adapter.rb:168:in `fix_exceptions'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-zero-2.2/lib/chef_zero/data_store/v1_to_v2_adapter.rb:55:in `create'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-zero-2.2/lib/chef_zero/rest_base.rb:92:in `create_data'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-zero-2.2/lib/chef_zero/endpoints/rest_list_endpoint.rb:30:in `post'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-zero-2.2/lib/chef_zero/rest_base.rb:29:in `call'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-zero-2.2/lib/chef_zero/rest_router.rb:23:in `call'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-zero-2.2/lib/chef_zero/server.rb:443:in `block in app'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/rack-1.5.2/lib/rack/handler/webrick.rb:60:in `call'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/rack-1.5.2/lib/rack/handler/webrick.rb:60:in `service'
/opt/chefdk/embedded/lib/ruby/2.1.0/webrick/httpserver.rb:138:in `service'
/opt/chefdk/embedded/lib/ruby/2.1.0/webrick/httpserver.rb:94:in `run'
/opt/chefdk/embedded/lib/ruby/2.1.0/webrick/server.rb:295:in `block in start_thread'

================================================================================
Chef encountered an error attempting to load the node data for "stone.local"
================================================================================

Unknown Server Error:
---------------------
The server had a fatal error attempting to load the node data.

Server Response:
----------------
Exception raised!  #<Errno::ENOENT: No such file or directory @ dir_s_mkdir - /root/chef/nodes>
/opt/chefdk/embedded/apps/chef/lib/chef/chef_fs/file_system/chef_repository_file_system_root_dir.rb:57:in `mkdir'
/opt/chefdk/embedded/apps/chef/lib/chef/chef_fs/file_system/chef_repository_file_system_root_dir.rb:57:in `block in create_child'
/opt/chefdk/embedded/apps/chef/lib/chef/chef_fs/file_system/chef_repository_file_system_root_dir.rb:55:in `each'
/opt/chefdk/embedded/apps/chef/lib/chef/chef_fs/file_system/chef_repository_file_system_root_dir.rb:55:in `create_child'
/opt/chefdk/embedded/apps/chef/lib/chef/chef_fs/chef_fs_data_store.rb:406:in `get_dir'
/opt/chefdk/embedded/apps/chef/lib/chef/chef_fs/chef_fs_data_store.rb:395:in `with_dir'
/opt/chefdk/embedded/apps/chef/lib/chef/chef_fs/chef_fs_data_store.rb:68:in `create'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-zero-2.2/lib/chef_zero/data_store/v1_to_v2_adapter.rb:56:in `block in create'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-zero-2.2/lib/chef_zero/data_store/v1_to_v2_adapter.rb:168:in `fix_exceptions'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-zero-2.2/lib/chef_zero/data_store/v1_to_v2_adapter.rb:55:in `create'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-zero-2.2/lib/chef_zero/rest_base.rb:92:in `create_data'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-zero-2.2/lib/chef_zero/endpoints/rest_list_endpoint.rb:30:in `post'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-zero-2.2/lib/chef_zero/rest_base.rb:29:in `call'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-zero-2.2/lib/chef_zero/rest_router.rb:23:in `call'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-zero-2.2/lib/chef_zero/server.rb:443:in `block in app'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/rack-1.5.2/lib/rack/handler/webrick.rb:60:in `call'
/opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/rack-1.5.2/lib/rack/handler/webrick.rb:60:in `service'
/opt/chefdk/embedded/lib/ruby/2.1.0/webrick/httpserver.rb:138:in `service'
/opt/chefdk/embedded/lib/ruby/2.1.0/webrick/httpserver.rb:94:in `run'
/opt/chefdk/embedded/lib/ruby/2.1.0/webrick/server.rb:295:in `block in start_thread'

[2014-08-07T22:34:31+01:00] FATAL: Stacktrace dumped to /Users/agh/.chef/local-mode-cache/cache/chef-stacktrace.out
Chef Client failed. 0 resources updated in 2.006429 seconds
[2014-08-07T22:34:31+01:00] ERROR: 500 "Internal Server Error "
[2014-08-07T22:34:31+01:00] FATAL: Chef::Exceptions::ChildConvergeError: Chef run process exited unsuccessfully (exit code 1)

cookbook_path is valid and contains two entries
As described in the Attempted Flow, this has one cookbook in each directory and we pass in two directories to cookbook_path which is supposedly supported according to documentation.
# This setting enables SSL validation for HTTPS connections between
# chef-client and chef-server (but does not affect other HTTPS connections).
verify_api_cert true

cookbook_path [ "/Users/agh/src/agh/chef/cookbooks", "/Users/agh/src/agh/chef/site-cookbooks" ]

This does work but it emits an unexpected warning:
stone:chef agh$ chef-client -c etc/client.rb -z -o recipe[baa]
Starting Chef Client, version 11.14.0.rc.2
[2014-08-07T22:42:48+01:00] WARN: Child with name 'stone.local.json' found in multiple directories: /Users/agh/src/agh/chef/nodes/stone.local.json and /Users/agh/src/agh/chef/nodes/stone.local.json
[2014-08-07T22:42:48+01:00] WARN: Run List override has been provided.
[2014-08-07T22:42:48+01:00] WARN: Original Run List: []
[2014-08-07T22:42:48+01:00] WARN: Overridden Run List: [recipe[baa]]
resolving cookbooks for run list: ["baa"]
Synchronizing Cookbooks:
  - baa
  - sheep
Compiling Cookbooks...
Converging 0 resources
[2014-08-07T22:42:48+01:00] WARN: Skipping final node save because override_runlist was given

Running handlers:
Running handlers complete
Chef Client finished, 0/0 resources updated in 2.136351 seconds

The unexpected warning is [2014-08-07T22:42:48+01:00] WARN: Child with name 'stone.local.json' found in multiple directories: /Users/agh/src/agh/chef/nodes/stone.local.json and /Users/agh/src/agh/chef/nodes/stone.local.json and suggests that something is incorrectly making the assumption cookbook_path should be singular.
cookbook_path with three entries
I've now created another directory (mkdir fake-cookbooks) and added it to cookbook_path, however it is completely empty. This makes things get very shouty indeed.
# This setting enables SSL validation for HTTPS connections between
# chef-client and chef-server (but does not affect other HTTPS connections).
verify_api_cert true

cookbook_path [
  "/Users/agh/src/agh/chef/cookbooks",
  "/Users/agh/src/agh/chef/site-cookbooks",
  "/Users/agh/src/agh/chef/site-cookbooks" ]

Output from the convergence -
stone:chef agh$ chef-client -c etc/client.rb -z -o recipe[baa]
Starting Chef Client, version 11.14.0.rc.2
[2014-08-07T22:46:34+01:00] WARN: Child with name 'stone.local.json' found in multiple directories: /Users/agh/src/agh/chef/nodes/stone.local.json and /Users/agh/src/agh/chef/nodes/stone.local.json
[2014-08-07T22:46:34+01:00] WARN: Child with name 'stone.local.json' found in multiple directories: /Users/agh/src/agh/chef/nodes/stone.local.json and /Users/agh/src/agh/chef/nodes/stone.local.json
[2014-08-07T22:46:34+01:00] WARN: Run List override has been provided.
[2014-08-07T22:46:34+01:00] WARN: Original Run List: []
[2014-08-07T22:46:34+01:00] WARN: Overridden Run List: [recipe[baa]]
resolving cookbooks for run list: ["baa"]
[2014-08-07T22:46:34+01:00] WARN: Child with name 'sheep' found in multiple directories: /Users/agh/src/agh/chef/site-cookbooks/sheep and /Users/agh/src/agh/chef/site-cookbooks/sheep
[2014-08-07T22:46:34+01:00] WARN: Child with name 'sheep' found in multiple directories: /Users/agh/src/agh/chef/site-cookbooks/sheep and /Users/agh/src/agh/chef/site-cookbooks/sheep
[2014-08-07T22:46:34+01:00] WARN: Child with name 'sheep' found in multiple directories: /Users/agh/src/agh/chef/site-cookbooks/sheep and /Users/agh/src/agh/chef/site-cookbooks/sheep
[2014-08-07T22:46:34+01:00] WARN: Child with name 'sheep' found in multiple directories: /Users/agh/src/agh/chef/site-cookbooks/sheep and /Users/agh/src/agh/chef/site-cookbooks/sheep
[2014-08-07T22:46:34+01:00] WARN: Child with name 'sheep' found in multiple directories: /Users/agh/src/agh/chef/site-cookbooks/sheep and /Users/agh/src/agh/chef/site-cookbooks/sheep
[2014-08-07T22:46:34+01:00] WARN: Child with name 'sheep' found in multiple directories: /Users/agh/src/agh/chef/site-cookbooks/sheep and /Users/agh/src/agh/chef/site-cookbooks/sheep
[2014-08-07T22:46:34+01:00] WARN: Child with name 'sheep' found in multiple directories: /Users/agh/src/agh/chef/site-cookbooks/sheep and /Users/agh/src/agh/chef/site-cookbooks/sheep
[2014-08-07T22:46:34+01:00] WARN: Child with name 'sheep' found in multiple directories: /Users/agh/src/agh/chef/site-cookbooks/sheep and /Users/agh/src/agh/chef/site-cookbooks/sheep
[2014-08-07T22:46:34+01:00] WARN: Child with name 'sheep' found in multiple directories: /Users/agh/src/agh/chef/site-cookbooks/sheep and /Users/agh/src/agh/chef/site-cookbooks/sheep
[2014-08-07T22:46:34+01:00] WARN: Child with name 'sheep' found in multiple directories: /Users/agh/src/agh/chef/site-cookbooks/sheep and /Users/agh/src/agh/chef/site-cookbooks/sheep
[2014-08-07T22:46:34+01:00] WARN: Child with name 'sheep' found in multiple directories: /Users/agh/src/agh/chef/site-cookbooks/sheep and /Users/agh/src/agh/chef/site-cookbooks/sheep
[2014-08-07T22:46:34+01:00] WARN: Child with name 'sheep' found in multiple directories: /Users/agh/src/agh/chef/site-cookbooks/sheep and /Users/agh/src/agh/chef/site-cookbooks/sheep
Synchronizing Cookbooks:
  - baa
  - sheep
Compiling Cookbooks...
Converging 0 resources
[2014-08-07T22:46:34+01:00] WARN: Skipping final node save because override_runlist was given

Running handlers:
Running handlers complete
Chef Client finished, 0/0 resources updated in 2.091054 seconds

cookbook_path is valid and contains a single entry
If I move the sheep cookbook over, and adjust the configuration file -
# This setting enables SSL validation for HTTPS connections between
# chef-client and chef-server (but does not affect other HTTPS connections).
verify_api_cert true

cookbook_path [ "/Users/agh/src/agh/chef/cookbooks" ]
#cookbook_path [ "/Users/agh/src/agh/chef/cookbooks", "/Users/agh/src/agh/chef/site-cookbooks" ]

Then this causes everything to work as I expect, with no errors or warnings.
stone:chef agh$ chef-client -c etc/client.rb -z -o recipe[baa]
Starting Chef Client, version 11.14.0.rc.2
[2014-08-07T22:41:41+01:00] WARN: Run List override has been provided.
[2014-08-07T22:41:41+01:00] WARN: Original Run List: []
[2014-08-07T22:41:41+01:00] WARN: Overridden Run List: [recipe[baa]]
resolving cookbooks for run list: ["baa"]
Synchronizing Cookbooks:
  - baa
  - sheep
Compiling Cookbooks...
Converging 0 resources
[2014-08-07T22:41:41+01:00] WARN: Skipping final node save because override_runlist was given

Running handlers:
Running handlers complete
Chef Client finished, 0/0 resources updated in 1.908445 seconds

Which PR is that? I am experiencing this issue with Chef 12.3.0 running in local mode.

Same issue with chef-client 12.3.0 running in zero/local mode (-z)

I see the same with 12.4.1 using zero/local mode and specifying chef_repo_path. I do not specify cookbook_path in my /etc/chef/client.rb file.

I found workaround.
In my environment, this warn message does not appear when .chef/knife.rb includes node_path configuration. FYI.

I think this will be fixed when f4ad4af6 is released

Hopefully fixed in 12.5.0

Closing since this should be fixed in master and the forthcoming 12.5 release. If that's not the case, please feel free to reopen.

I had come across this problem where when trying to use chef-zero, with Chef 12.4.3-1 at the time.
And read the comments above where it should be fixed in 12.5 release.