assistant => Error: EventSource stream error: Invalid status code: 400 Bad Request exit 1

Thanks for this awesome project and it works super fast  and lightweight.
Just found an error occurs several times as below and subject:
"assistant => Error: EventSource stream error: Invalid status code: 400 Bad Request exit 1"
and the program will be quit after the above error, the way to solve it is just launch the program again.
btw: My network is stable and confirmed no problem when the error occurred.

This error is from the server-side. Today I've also found the API service of OpenAI seems quite unstable.
But I agree that we should handle this error in a better way, rather than quiting immediately. I'll try to fix it.

Now it will print and error and allows you to continue the conversation.

i have pull the new code and rebuilt the app but no luck still got the problem as below:
still need to quit the current interactive conversation prompt and re-launch the appliction.

When the input length exceeds the max token, a 400 error will be returned.
If that's not the case, can you please post the input here? So that I can try to reproduce it.


fix: remove console warnings

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Moving Start and Stop of Continuous Reader to Run / Pause of Stream

Moved start and stop of continuous reader to the correct location, the run / pause of stream. Gives more stable operation.

Integrated first in multiple instances, extracted and pushed to this pull request to improve current driver and validate function.


Invalid command: -d

Brand new Debian 11 install. Have run the following commands:
apt install gcc build-essential git
cd /usr/local/src
wget https://docs.phreaknet.org/script/phreaknet.sh
chmod +x phreaknet.sh
./phreaknet.sh make
phreaknet update

The above works fine, but then:
phreaknet -d install
Invalid command: -d, type phreaknet help for usage.

-d is an option to install, not phreaknet, so it should be phreaknet install -d


Amélioration de la documentation des commandes CLI

Something like this ?

@foucsi Yes, but we don't need the overview.md file; its content should be included in the README.md after the usage example.
And please do not open new issue (use #411)


Production agent couch replication is not working.

All workqueue replication is failing now.

It is due to the cmsweb couchserver crash. closing the issue.


AsyncIterator tests fails

After dependency update #440 tests fails
See: https://travis-ci.org/metarhia/metasync/jobs/619382154
@nechaido

That is due to the way metatests changed the handling of tests that return Promise (and therefore async).
I'll push to the branch the fix.

https://travis-ci.org/metarhia/metasync/jobs/622009735
@lundibundi please see, is it related bug?


Added missing VoteCollector error message: not licensed.

@jwinzer Are you allowed to merge PRs?

I have no permission to merge PRs.

Ok, I'll merge then


Terminal: ANSI escape sequences no longer display correctly on Windows

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.66.0
OS Version: Windows 10 Version 10.0.19041.1415

Steps to Reproduce:

Set "terminal.integrated.minimumContrastRatio": 1:
All terminal colors and most ansi sequences would be expected, but this is not the case.
Get this file: https://gist.github.com/rewrking/f797bd19e9ecc44fa1c646b3f7b795c0

Steps for Git Bash inside of VS Code:

Run cat print_term.txt where the file in step 3 was saved

Steps for Powershell inside of VS Code:

Run Get-Content -Path .\print_term.txt where the file in step 3 was saved

Right now, my VS Code and Windows Terminal setup use the same color scheme. Both Git Bash and Powershell display the same way in each. In order to test this correctly, all 16 colors need to be unique.
This is what displays in VS Code on Windows:

This is what displays in Windows Terminal:

I would expect underlined, strikethrough, dim and italic to be consistent, but also it seems the ansiBlack background is ignored entirely. blink never worked in VS Code, so I have no expectations there. I also tested this on MacOS and Linux (zsh) and the output seems correct on those platforms.
Maybe relates to: https://github.com/microsoft/vscode/issues/146406

I noticed this issue with oh-my-posh module which changes the way powershell looks and it seems to look fine in the terminal and not on vs code integrated pwsh

I don't think https://github.com/microsoft/vscode/issues/146406 relates to this. This looks like the older version of conpty that you're on due to being on Windows 10 just lacks those features that Windows Terminal added later (which bundles the latest version of conpty). See my generic response to these types of issues below:

This issue seems to originate from an upstream component and is not something VSCode can fix by making a change to our code. As such, I will go ahead and close it. Please try to reproduce in Windows Terminal and report to microsoft/terminal with repro steps.
If you cannot reproduce in Windows Terminal, chances are it's fixed in a later version of Windows. You can read more about how we consume conpty here. Thanks for understanding.
/label upstream terminal-conpty windows

???
I have the latest Windows 10 updates and Windows Terminal updates. What do I need to do to fix the problem?

@Tyriar

Please try to reproduce in Windows Terminal

Did you read my post? It's not a problem in Windows Terminal at all.
I'm on Windows 10 Version 10.0.19041.1415.
Windows Terminal Preview Version 1.13.10733.0 (again, this works perfectly)
Why would I need to open the issue with that team? At a loss as to what I can actually do here. You're literally asking me to open an issue with them and say there's a problem with VS Code. Sounds like a MS process issue to me.

Please re-read what I said:

If you cannot reproduce in Windows Terminal, chances are it's fixed in a later version of Windows. You can read more about how we consume conpty here. Thanks for understanding.

I followed the first paragraph with my generic response to conpty issues to save time. You can't do anything until you update Windows to get the later version of conpty we use, it's a system utility that is provided by Windows but Windows Terminal bundles the latest version in their package.

@rewrking I think you misunderstood the comment by @Tyriar. Windows Terminal seems to bundle a version of ConPTY while VS Code uses the version that ships with the OS which has a bug.

You can't do anything until you update Windows to get the later version of conpty we use, it's a system utility that is provided by Windows but Windows Terminal bundles the latest version in their package.

That makes 10x more sense than your canned response... but it's still frustrating. Why can't VS Code bundle its own version too? Why can't that component be updated when Windows Terminal gets installed? Why can't it get updated when VS Code gets installed? This kind of thing burdens users with the responsibility of figuring out why unrelated software has issues, when the problem at at hand (terminal colors/styles) is so small by comparison.

@rewrking I get the frustration. The reasons are:

When we evaluating this possibility we determined it would involve more effort than we're willing to invest.
It would be more risk than it's worth, IIRC when I reached out to their team they recommended against it.

If their team provided an easy way for us to consume and bundle the latest version it would be a different story, but the model they went for was to ship it in Windows and Windows Terminal is basically their platform for testing early versions before they hit Windows.
So currently you need to update to Windows 11 and we sometimes have to wait up to a year before a fix lands for VS Code in the Windows 11. Luckily most issues at this point are non-critical like this one.

Got it. Thanks for explaining that part. Guess I'll sulk and consider upgrading my PC for a better terminal experience... 😂

Yeah, I was forced to upgrade my CPU recently. Greatly improved my overall coding experience jumping 8 Intel generations though 🤯

For anyone else looking for a "fix", this is what you have to do right now:
Copy conhost.exe and ConhostV1.dll (in System32) from a Windows 11 machine (version >= 10.0.22000) to the Windows 10 machine (version < 10.0.22000). Obviously not for everybody, but it seems to work so far.

Hi @Tyriar, I wanted to chime in on this issue as I believe I'm experiencing the same thing, but unlike @rewrking, I'm on Windows 11 (specifically v21H1 OS Build 22000.556). If, as you say, this is an issue with an older version of conpty that is bundled with Win10, why am I facing the same issue in Win11?
Can this be explained by something else or is there a later build of conpty in insider builds of Win 11 that aren't in public version I'm using?


Timezone is missing

Hello,
Is there any way to get the timezone.
I and using google calendar and I want the time zone when we access the token.
Thanks,

It might be available in $user->toArray().
If you can point me to the documentation that shows where the timezone information would be made available?

I use like this and this is working
public function GetUserCalendarTimezone($access_token) {
$url_settings = 'https://www.googleapis.com/calendar/v3/users/me/settings/timezone';
	$ch = curl_init();
	curl_setopt($ch, CURLOPT_URL, $url_settings);
	curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1);
	curl_setopt($ch, CURLOPT_HTTPHEADER, array('Authorization: Bearer '. $access_token));
	curl_setopt($ch, CURLOPT_SSL_VERIFYPEER, FALSE);
	$data = json_decode(curl_exec($ch), true); //echo '<pre>';print_r($data);echo '</pre>';
	$http_code = curl_getinfo($ch,CURLINFO_HTTP_CODE);
	if($http_code != 200)
		throw new Exception('Error : Failed to get timezone');

	return $data['value'];
}

That looks like a perfect solution for your need!
Including such code in this package could be useful, as a separate piece of functionality. I would welcome a PR for it.


Fix the Remix Jokes App tutorial docs

While following the Remix Jokes App tutorial, I have spotted some filename mistakes that arise my confusion. The following filename issues have been addressed by this PR

Also, in one of the code snippets, one of the dependencies has not been imported properly causing error when pasting into a code editor without adding the missing dependency.

Thanks for this fix! I turned off GitHub during the holiday so I didn't see your contribution, but I made this fix while I was away. Thanks for the contribution anyway!


New Chat - Type box is not auto focused to search for an user when creating a new chat

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Expected Result:
Search should be auto focused so the user doesn't have to manually select the box to search for an account.
Actual Result:
Type box is not auto focused to search for an user when creating a new chat
Action Performed:

Log in to e.cash
Click on the + button
Click on new chat

Workaround:
User has to manually select the box to start typing.
Platform:
Where is this issue occurring?
Web ✔
iOS
Android
Desktop App
Mobile Web
Version Number: 1.0.53-0
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:

Expensify/Expensify Issue URL:
View all open jobs on Upwork

From @mountiny https://expensify.slack.com/archives/C01GTK53T8Q/p1621954471186900

When I click on the green plus button and then "New chat" option, the Name/email/phone search input in the modal is not focused so I cannot type straightaway.
Additionally, pressing Tab focuses element "underneath" the New chat modal and not the search input inside of the modal making it inaccessible for anyone relying on these accessibility features. Using Chrome on macos

Triggered auto assignment to @johncschuster (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

Triggered auto assignment to @francoisl (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

Interesting, looks like we already try to set focus on the input here.
It sounds like the cleanest way to fix this would be to use the autoFocus property of InputText – I did a quick test and added it to the <TextInput> in TextInputWithFocusStyles, but it didn't change anything.
Also tried replacing that TextInput with a TextInputFocusable, but that didn't fix the focus, and also we'd need to do more changes for the focus styles I imagine.
The only thing I found that worked was to use a delay before trying to apply focus in the componentDidMount of OptionsSelector
diff --git a/src/components/OptionsSelector.js b/src/components/OptionsSelector.js
index 341127cdb..d6239aaba 100755
--- a/src/components/OptionsSelector.js
+++ b/src/components/OptionsSelector.js
@@ -95,7 +95,7 @@ class OptionsSelector extends Component {
     }

     componentDidMount() {
-        this.textInput.focus();
+        setTimeout(() => this.textInput.focus(), 250);
     }

     /**

but that sounds super hacky, so I imagine there must be a better way to fix this.
cc @marcaaron if you have any ideas, looks like you worked on these components.

This issue will be automatically resolved when https://github.com/Expensify/Expensify.cash/pull/2956 this is merged. But we have  a issue on IOS for which I have not found any straight solution except using InteractionManager for native devices.

Ah, good point, looks like it's working now!


[PL-48563] SMP 0.14.6 patch release notes

SMP 0.14.6 patch release notes
For reviewers, a preview is available:
https://6602f1928fb0ec57aa8fbc5b--harness-developer.netlify.app/release-notes/self-managed-enterprise-edition#march-26-2024-patch-version-0146
What Type of PR is This?

[ ] Issue
[ ] Feature
[x] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[x] Tested Locally
[ ] Optional Screenshot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6602d6a65c335e3bda3da156--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6602e3d80cdeb149f626683e--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6602f11c5c335e57093d9a53--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6602fa043e4bc85fb09e6ac9--harness-developer.netlify.app


Unable to Open/Read Book

I clicked on "AWS CSA Ian Berry" Book & it shows me the below message
" Unable to render code block "
there are some more books, which show the same message

Thanks for submitting the issue, I will re-upload the same.


[usd] Update to 23.02

[x] Changes comply with the maintainer guide
[x] SHA512s are updated for each updated download
[ ] The "supports" clause reflects platforms that may be fixed by this new version
[ ] Any fixed CI baseline entries are removed from that file.
[ ] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

From ci.baseline.txt:
# USD has set official policy that they will not update to be compatible with TBB in the near term (https://github.com/PixarAnimationStudios/USD/issues/1600)
usd:arm64-windows=skip
usd:arm-uwp=skip
usd:x64-uwp=skip
usd:x64-windows=skip
usd:x64-windows-static=skip
usd:x64-windows-static-md=skip
usd:x64-linux=skip
usd:x64-osx=skip
usd:x86-windows=skip

Is this problem fixed in this change?

Is this problem fixed in this change?

No, I still need to downgrade TBB to version 2020_U3 in my manifest file.

OK. Other reviewers, that means CI isn't testing this for you, please keep eyes peeled.

Is this problem fixed in this change?

No, I still need to downgrade TBB to version 2020_U3 in my manifest file.

Is there any kind of check and warning during configuration? If not, it would be good to add it. Maybe it can be done without patching, e.g. using CMAKE_PROJECT_INCLUDE. Facilitates triage on https://github.com/microsoft/vcpkg/issues

Is this problem fixed in this change?

No, I still need to downgrade TBB to version 2020_U3 in my manifest file.

Is there any kind of check and warning during configuration? If not, it would be good to add it. Maybe it can be done without patching, e.g. using CMAKE_PROJECT_INCLUDE. Facilitates triage on https://github.com/microsoft/vcpkg/issues

There's a message during the configuration (see portfile.cmake):
message(STATUS [=[
The usd port does not work the the version of Threading Building Blocks (tbb) currently chosen by vcpkg's baselines,
and does not expect to be updated to work with current versions soon. See
https://github.com/PixarAnimationStudios/USD/issues/1600

If you must use this port in your project, pin a version of tbb of 2020_U3 or older via a manifest file.
See https://vcpkg.io/en/docs/examples/versioning.getting-started.html for instructions.
]=])

@Adela0814 Please review carefully.

Thank you for the reminder, my mistake. This PR is ready to merge.

Thanks for the review @Adela0814 @dg0yt 👍

Guys How do i run my executable.
Concerned Issue:
https://github.com/PixarAnimationStudios/USD/issues/1788
To fix this issue, this is how i structured for executable to run

However these Lines (from ./ports/usd/portfile.cmake L91) changes the structure
foreach(PLUGINFO ${PLUGINFO_FILES} ${PLUGINFO_FILES_DEBUG})
    file_replace_regex(${PLUGINFO} [=["LibraryPath": "../../([a-zA-Z0-9_]+).dll"]=] [=["LibraryPath": "../../../bin/\1.dll"]=])
endforeach()

can anyone help me with this

Hello @lalishansh
Did you perhaps find a solution?
I'm having the same problem but with port usd 23.5. My exe doesn't find the plugins at runtime after an install via vcpkg manifest on Visual Studio.


Do build tools checks before build

Make sure that cmake and all necessary build tools are installed:

[ ] - mingw32-make
[ ] - Download cmake if not found (can be installed to C:\Program Files\Unity\Editor\Data\Tools)
[x] - Android SDK/NDK (Ideally from the same path in the Unity preferences)
[x] - Visual studio
[ ] - Xcode
[ ] - g++/gcc
[x] - Emscripten

Is it possible to build WebGL with visual studio on windows?


Switch internal color type to avh/elm-color?

As far as I know, avh/elm-color is supposed to become kind of a community standard when it comes to color. This would make interop of packages much easier. What are your thoughts on this?
It would of course add an alpha channel, but palette could just ignore that, as far as I'm concerned (I personally need support for it, but I don't need it in the functions palette provides)

I think one issue with this could be that avh/elm-color currently doesn't have native support for an hsl color space. I like the idea of treating color spaces differently, so maybe I'll ask there if the plan is to add that at some point.

Hi! Thanks for bringing this up!
It's definitely something that I'm considering. In the short term, I'll think about adding helpers for going from an avh4/elm-color Color to a tesk9/palette Color and back again. I worry a bit about becoming more dependent than that on avh4/elm-color, since I think there are still lots of discussions happening around how color should be modeling in the library, and what the API will be..
Are you currently using both avh4/elm-color and tesk9/palette? I guess that would be naming-collision prone... 🤔

Currently I'm using neither. I'm writing a wrapper around elm-ui for our application that allows runtime skinning, and at the same time I'm creating a small element library. In some of these parts I want the ability to automatically select good-contrast colors for text etc, and in the process of looking these things up I decided to wrap the elm-ui color values completely (since I'm already doing a wrapper).
palette seems to have very nice accessibility functions, but at the same time it doesn't have alpha support (which would at least be necessary for full elm-ui "compatibility") and if there's already an effort to standardize color in Elm, I would not want to go a different route.
There is apparently elm-color-extra which also has some contrast helpers. It doesn't have the nice blending options of palette though (which I'm planning to use for disabled buttons etc.), so I'm torn^^

That makes sense! I probably won't have much time to work on this for about a week or so, but I'll come back to it after that.
At minimum, I think the naming collision is an issue that needs to be resolved. Plus some level of support for avh4/elm-color should be on the roadmap. And of course transparency.
Thanks again for bringing this up!

Thanks for reacting so quickly!
For now I've decided to use avh4/elm-color because I didn't want to make the decision to drop transparency support from my wrapper. Plus, the maximumContrast function in elm-color-extra is pretty handy.
I think what I'd like to see in the future is just more specialized packages? For example I don't think avh4/elm-color needs to include a color palette. Although of course it can be exhausting to have to install 5 packages just to get the funcitonality you expected from one, so I don't know^^
I really like palettes functions for testing accessibility though. And its explicit support for different color spaces.

Thank you for following up! It's really helpful to hear what you ended up doing & why.

I ran into an unfortunate circumstance relating to this issue.
I really like the functions relating to the color circle from color theory in this package. In a test project I wanted to do color animation on an svg object with help from those functions. My go-to package for svg is elm-community/typed-svg.
But not only does typed-svg have avh4/elm-color as a dependency, many of its functions uses its opaque Color type as arguments. Since the color type from your package is also opaque, in combination with both packages having a module called Color, conversions between the types becomes impossible.
So, in short, if I want to use typed-svg, I can't use palette and vice versa.

I'm working on changing the names to avoid this conflict -- the next major version of palette should be usable.
Thanks for bringing this up!

The renames in https://github.com/tesk9/palette/pull/18 should allow you to use this package along with avh4/elm-color (although you'll need to write your own conversion helpers to go from the elm-color implementation to the palette implementation).
I want to take some time to get API reviews and thoughts from folks before I publish, so it may be a while yet before these changes are available.
Please feel free to (gently 🙏) share your thoughts on the PR.

What are your current thoughts on this? It looks like the Color -> Colour name change got reverted, which I think makes sense but leaves the question of compatibility/interoperability with avh4/elm-color open.
(I do like the design of tesk9/palette for use in ianmackenzie/elm-3d-scene, but I'm trying to weigh that against the interop benefits of using avh4/elm-color which currently seems more widely used. Happy to chat on Slack if that's easier!)

Ian and I talked on slack and over zoom and we pulled some other perspectives in too.. I've drafted a PR https://github.com/tesk9/palette/pull/26 to resolve the naming collisions.
Let me know if you have thoughts on the changes!
I'm waffling a bit on "SolidColor" and "TransparentColor." Perhaps "OpaqueColor" and "TranslucentColor"?

I decided to stop waffling on getting the best-possible fix here and just went ahead and published what I had.
Please let me know if there are any issues with the new names. If it's a pain to upgrade, let me know. I'll try to put something together to help.

Plus, the maximumContrast function in elm-color-extra is pretty handy.
I really like palettes functions for testing accessibility though. And its explicit support for different color spaces.

Speaking of @ianmackenzie / @tesk9 y'all may find the widen function I wrote interesting :) I'm using in a ui-framework hobby project I'm working on (super super alpha) that's built on theming & accessibility guaranteed to be correct by construction ala "Make Impossible States" and all...
https://ellie-app.com/bFd9x3KcPVQa1
Not sure if I'd call it a fold on Colors but widen is how I think of the behaviour :man_shrugging:

That's very neat! Thanks for the link


[WIP] Classification optimization

Dependencies:

[x] #18409
[ ] #19523
[x] https://travis-ci.org/kbrock/manageiq-cross_repo/builds/612422470 - failures unrelated:

azure: #19522 https://github.com/ManageIQ/manageiq-providers-azure/pull/369
api sporadic: https://github.com/ManageIQ/manageiq-api/pull/711

Reduce the number of queries for classification updates:
c = Classification.last
c.update_attributes(:name => "managed")          # name

ms
bytes
objects
query
qry ms
rows
comments

84.1
1,504,436*
19,330
14
14.7
7
before-name-1

63.8
1,178,814*
15,670
7
11.9
3
after-name-2

24%
22%
19%
50%
19%
57%
diff

* Memory usage does not reflect 113 freed objects.
* Memory usage does not reflect 100 freed objects.
c = Classification.last
c.update_attributes(:description => "Migrated")  # desc

ms
bytes
objects
query
qry ms
rows
comments

85.4
1,496,034*
19,243
14
15.8
9
before-desc-1

46.8
159,206*
1,924
4
7.7

after-desc-1

45%
89%
90%
71%
51%
100%
diff

* Memory usage does not reflect 112 freed objects.
* Memory usage does not reflect 5 freed objects.
c = Classification.last
c.save                                           # nop

ms
bytes
objects
query
qry ms
rows
comments

72.6
1,436,927*
18,756
13
8.9
9
before-nop-1

37.2
65,917*
837
2
0.7

after-nop-2

49%
95%
96%
85%
92.1%
100%
diff

* Memory usage does not reflect 112 freed objects.
* Memory usage does not reflect 2 freed objects.
of note, the 2 "queries" for the no-op save is "BEGIN" ; "END"

fixed cops

rebasing to get a more recent version of rails (so cross testing will run) ==> https://travis-ci.org/kbrock/manageiq-cross_repo/builds/612422470

@kbrock @bdunne what's holding back this PR from being merged?

WIP: When revisiting the number of queries, it looks like uniqueness is no longer checked like it did before. (tricky to get validation on another object to propagate here) It must have been lost over the multiple rebases.
I'll find a way to make sure that the tag validation will force the uniqueness over here.

Seems part of this was merged in #20678 ?


[QoL][Relax] Use SeqExpr in IR types when SeqExpr is required

The Relax IR requires the FunctionNode::body, IfNode::true_branch, and IfNode::false_branch to be instances of relax::SeqExpr. If these Relax requirements are violated, correctly-implemented transformations may raise an exception
(e.g. from Downcast in Downcast<SeqExpr>(func->body)->blocks), or even segfault (e.g. when .as returns a nullptr in func->body.as<SeqExprNode>()->blocks).  Debugging these failure modes is also difficult, as even the TVMScript printer relies on the body of the function being a SeqExprNode.
This commit updates the C++ type of FunctionNode::body, IfNode::true_branch, and IfNode::false_branch to be relax::SeqExpr instead of relax::Expr.  This does not impact any well-formed Relax IR, and allows this type of ill-formed Relax IR type to be checked at compile-time.  A large number of checks applied during TVM runtime can now be removed, as they duplicate the new compile-time check.
To maintain backwards compatibility, this commit adds a new constructor to relax::SeqExpr, which accepts a single Expr body argument.  This constructor provides either an additional reference to the same underlying relax::SeqExprNode, if body already contains a relax::SeqExprNode, and otherwise wraps the body in a relax::SeqExpr.  For implementations that previously produced well-formed Relax IR, this change has no effect.  For implementations that previously produced ill-formed Relax IR, this change results in the equivalent well-formed Relax IR.
Alternate implementations considered:

Perform the backwards-compatibility wrapping within the relax::Function and relax::If constructors.  While this would provide the intended conversion when these constructors are used, Relax transforms make frequent use of copy-on-write (e.g. func.CopyOnWrite()->body = new_body), which does not use the constructor.  Maintaining backwards compatibility for this usage requires the implicit conversion constructor that was chosen for this PR.

Remove the Relax IR requirement for these expressions to be SeqExpr.  While this would make Relax more internally consistent, such a change would break backwards compatibility that relies on SeqExpr being present.  While the callsites within TVM could be updated to resolve this breakage, callsites outside of TVM (e.g. MLC-LLM) could not.  Exposing the special case within the C++ type, as done in this PR, maintains backwards compatibility.

This PR is currently marked as a draft, to allow time for community discussion.  A forum thread (link) for discussion has also been started for this change.


txsource: delegation workload

Part of: #2506
Delegation workload continuously makes escrow and reclaim escrow transactions between accounts. (note: the escrow balance is only funded at start which effectively means reclaim is also tested as a part of this workload).
Also fixes a bug in delegation queries (extracted into #2756)
TODO:

[x] probably should add unit tests for the delegation queries

thanks for review!


Write MatLab Dataset with MatFileHandler

I am wondering how to write a MatLab dataset or perhaps a table variable to a .mat file. An example or guidelines would be much appreciated.

Hi,
unfortunately, neither datasets nor tables are supported at the moment. Datasets seem to be deprecated, so I don't think it makes sense for MatFileHandler to deal with them. It would be nice to have support for tables, and this is in the plans, but I can't give you any time estimates right now.


[BEAM-5577] Update MongoDB

Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run Java MongoDBIO Performance Test

Run Java PostCommit

retest this please

retest this please

Run Java PostCommit

Hey, @iemejia could you take a look? Thanks
R: @iemejia


output unmount error when unmount fail in diff.Apply

Signed-off-by: yason<EMAIL_ADDRESS>When unmount fail we should output unmount err and return error.

LGTM

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1877   +/-   ##
=======================================
  Coverage   48.87%   48.87%
=======================================
  Files          86       86
  Lines        8332     8332
=======================================
  Hits         4072     4072
  Misses       3586     3586
  Partials      674      674

Flag
Coverage Δ

#linux
52.34% <ø> (ø)
:arrow_up:

#windows
44.1% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

LGTM


Upgrade to Flux version v0.23

Issue #, if available:
#600
on /hold till ship the v0.6 patch release
Description of changes:

Add default namespace to eks-a resources pushed to the Git repo, for both create and upgrade cases
Delete old flux-syste secret before flux bootstrap, to always use ECDSA as the default SSH key algorithm

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jiayiwang7
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jiayiwang7]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jiayiwang7, mdsgabriel
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jiayiwang7,mdsgabriel]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/lgtm


collapsible sidebar sections / Visible scroll

Describe the feature you'd like
On main page with all pages sidebar with “My Recently Viewed”, “Recently Updated Pages”, “Recent Activity” should be collapsible to not view whole list at once. Or maybe show scrollbar to scroll through sidebar. So users with no mouse (with touchpad) can access it easier. I thought about something like that (collapsed):

After clicking them they should expand:

Describe the benefits this would bring to existing BookStack users
Website would be more accessible for those who can't scroll in different places of website (they are using scrollbar on the right of the screen etc)
Can the goal of this request already be achieved via other means?
Adding css to make scrollbar show would be the simplest solution. Collapsing list is also easy task with some JS i think.
Have you searched for an existing open/closed issue?

[X] I have searched for existing issues and none cover my fundemental request

How long have you been using BookStack?
0 to 6 months
Additional context
No response

Hi @Skydem,
Please see #3789 for a previous discussion in relation to the scroll bar, and a possible workaround you can add to your instance if desired.
Collapsing elements has been touched upon in #2277 although I'm not too keen for collapsible sections as a solution this described issue, since scroll could still be present with a single sidebar section. I'd stick to showing the scrollbar as above if you really to display it.

Thank you, code mentioned in #3789 solves my problem.


Parsing isolated and quarantined for ZH

Thanks @dominikgehl for the parser.
@tlorusso and I decided to create a separate file for the respective ZH data: https://github.com/openZH/covid_19/blob/master/COVID19_Fallzahlen_Kanton_ZH_isolated_quarantined.csv
While our colleagues @GD-ZH are updating https://github.com/openZH/covid_19/blob/master/fallzahlen_kanton_total_csv_v2/COVID19_Fallzahlen_Kanton_ZH_total.csv daily, we will run a script, which is adding the current_isolated and current_quarantined to that file after each update.


Add allowed values to metadata

As quick fix okay. But shouldn't we add it in kuksa.val.v1/v2 as well as message in types?

Should not be merged to https://github.com/boschglobal/kuksa-databroker/tree/prep-rel-0.5.0?

As quick fix okay. But shouldn't we add it in kuksa.val.v1/v2 as well as message in types?

This adds it to kuksa.val.v1 and sdv.databroker.v1. kuksa.val.v2 is not a thing (yet)

yes, in code but why only add message Allowed in types.proto of sdv.databroker.v1?

Okay, my fault. It is nested in ValueRestriction here https://github.com/eclipse-kuksa/kuksa-databroker/blob/7951c62702e3eb3052bcf8f4e2c3235cd5adb6f5/proto/kuksa/val/v1/types.proto#L127


Maximize Window covers Start Menu on Windows

This only happens when HasSystemDecorations = false

This can be seen just... by setting XAML Test app to HasSystemDecorations = false, removing width and height and WindowState=Maximized, you will see some of the app is behind start menu.
In AvalonStudio which has a Control Template with a border overriding the normal Window control template, which may be the cause. So I suspect that SetWindowDecorations win32 implementation is not taking into account the windows control template.

This was recently resolved via #805


fix: file input overflows

Input elements were flowing out of the parent wrapper.

The issue was related to shadow elements in safari, have fixed in a separate commit, hence closing this.


feat(type): major rewrite of typography section

co-authored: @janchild
This PR is for the new guidance for expressive and productive type sets.
Issue for component work: https://github.com/carbon-design-system/carbon/issues/8934
Notes from Scott:

I took the typeScale and typeSets JSON and moved them into their own individual files to make TypesetStyle more readable.
Lots of renaming style tokens to match the spec. Pretty much everything got a renaming.
Refactored TypesetStyle to allow for a boolean prop to turn the old header off and on. Even though there's no use-case for the old header, I didn't have the heart to remove the code, thinking we might bring it back someday. Added defaults of breakpointControls={false} & header={false} to make the component easier to implement.

Testing
Head to /guidelines/typography/type-sets and make sure everything is in order.

@sstrubberg Just fixed a few little things in each of the three pages. Will check after the build, but we should be good after this.

@janchild This is missing something IDL about it. This current intro text could be the language used for anyone but what can make a touch more about our approach?
What about something like?
The approach to our typographic system uses IBM Plex as the typeface. It has been carefully engineered with suitable scales and weights to help create clear hierarchies and organize information that guides users through IBM products or experiences.
or
Our approach to the typographic system uses IBM Plex as its typeface. It has been carefully engineered with suitable scales and weights to help create clear hierarchies and organize information that guides users through IBM products or experiences.
or feel free to stab at this further.

@janchild This is missing something IDL about it. This current intro text could be the language used for anyone but what can make a touch more about our approach?
What about something like?
The approach to our typographic system uses IBM Plex as the typeface. It has been carefully engineered with suitable scales and weights to help create clear hierarchies and organize information that guides users through IBM products or experiences.
or
Our approach to the typographic system uses IBM Plex as its typeface. It has been carefully engineered with suitable scales and weights to help create clear hierarchies and organize information that guides users through IBM products or experiences.
or feel free to stab at this further.

Love it! Thanks, @mjabbink! Will do!

Maybe better to add “styles” in the there too.
... scales, styles, and weights...

There is a spacing issue here.
https://carbondesignsystem-git-fork-sstrubb-19b965-carbon-design-system.vercel.app/guidelines/typography/overview/#scale
Should be more space between image and text

I’ve mention before to update the login image with a more recent illustration style. That illustration is not really IDL so not a good example of illustration. Maybe @thyhmdo or @kingtraceyj can help replace this with something more current. See Cloud or Security designs which use some recent illustration designs.

@janchild in https://carbondesignsystem-git-fork-sstrubb-19b965-carbon-design-system.vercel.app/guidelines/typography/type-sets
current intro:
Carbon uses type tokens to manage typography, and these tokens sit within two type sets: expressive and productive.
Maybe something like...
Carbon uses clear naming approach and type tokens to manage typography across complex and layered design experiences, and these tokens sit within two type sets: expressive and productive.
or
Carbon uses clear naming approach and type tokens to manage typography across complex and layered layouts and patterns, and these tokens sit within two type sets: expressive and productive.

In “Typesets” the outer columns with specs  within the table is off Grid. All of it is off grid but @alisonjoseph has a fix under way to sort that out but even with that upcoming fix that last spec typography is not on the column.

@janchild in “Code” intro again.
current:
Type is a core part of any offering. Use the Carbon type package to leverage IBM Plex across your product.
maybe something like...
Type is a core part of any offering and critical to how brands express and communicate throughout any experience. Use the Carbon type package to leverage IBM Plex and create effective typography across your product more easily.
or
Type is a core part of any offering and critical to how brands express and communicate throughout any experience. Use the Carbon type package to leverage IBM Plex and create effective typography for your users and across your products.

@sstrubberg Let me know if you want to pair up on the grid alignment for this, but after applying my fix for overall grid, looks like these two styles below need to get removed.

also looks like bx--typeset-example-specs and bx--typeset-example-description are not on the grid. I don't have the spec in front of me, but looks like its 7.5 cols and 4.5 cols right now. Do we want 7/5 or 8/4?
.

One last thing to make sure we don't forget because we are removing / changing urls people may have bookmarked.

[ ] Add redirect from Productive and Expressive pages

@janchild need to add “a” in “Carbon uses a clear naming approach...”
I had left it out on my copy suggestions above my accident.
Carbon uses a clear naming approach and type tokens to manage typography across complex and layered layouts and patterns, and these tokens sit within two type sets: expressive and productive.

@jeanservaas and @theiliad Should we not include any details for the condensed which is used in Carbon Charts?


Seemed to have dupilicate options

Would you mind tell the diffrences between the two options? They looks like the same.

It's a bug. Should be fixed in the latest version.


just adjust some writing syntax

there is a little suggestion that self should be used in closure, which explicitly remind us to avoid circle reference, so i just warmly remove that, and hope you have a consideration. :]

Codecov Report

Merging #4 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master       #4   +/-   ##
=======================================
  Coverage   86.79%   86.79%
=======================================
  Files          15       15
  Lines         765      765
=======================================
  Hits          664      664
  Misses        101      101

Impacted Files
Coverage Δ

Sources/Schedule/Task.swift
79.06% <100%> (ø)
:arrow_up:

Sources/Schedule/WeakSet.swift
78.94% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Haha~ good point! Thank you for your contribution!


AFNetworking nullability warning

Why i am getting this warning?
pointer is missing a nullability type specifier (__nonnull or __nullable) [-Wnullability-completeness]

What version of xcode are you using?

And which version of AFNetworking?

Xcode 6.4


hnix: Fix build

Motivation for this change
The hnix build was broken because the added patch could no longer be applied.
I believe it is included now.
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[X] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[X] Fits CONTRIBUTING.md.

I ran nox-review wip but it cannot complete because of #44156

@GrahamcOfBorg build haskellPackages.hnix

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: haskellPackages.hnix
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: haskellPackages.hnix
Partial log (click to expand)
strip is /nix/store/a3nk8z2i7m7wa3jdckgv710n7j3yx4b5-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/i7smyjp4fi4yxk1r2spcs8i3dv9l6fms-hnix-0.5.2/lib  /nix/store/i7smyjp4fi4yxk1r2spcs8i3dv9l6fms-hnix-0.5.2/bin
patching script interpreter paths in /nix/store/i7smyjp4fi4yxk1r2spcs8i3dv9l6fms-hnix-0.5.2
checking for references to /build in /nix/store/i7smyjp4fi4yxk1r2spcs8i3dv9l6fms-hnix-0.5.2...
wrong ELF type
shrinking RPATHs of ELF executables and libraries in /nix/store/xbb4qz8pj384mh75zxfj5zy4drmbw3vv-hnix-0.5.2-doc
strip is /nix/store/a3nk8z2i7m7wa3jdckgv710n7j3yx4b5-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/xbb4qz8pj384mh75zxfj5zy4drmbw3vv-hnix-0.5.2-doc
checking for references to /build in /nix/store/xbb4qz8pj384mh75zxfj5zy4drmbw3vv-hnix-0.5.2-doc...
/nix/store/i7smyjp4fi4yxk1r2spcs8i3dv9l6fms-hnix-0.5.2

Success on x86_64-darwin (full log)
Attempted: haskellPackages.hnix
Partial log (click to expand)
/nix/store/y06mk1hjnpy13g3ya775g6ylqb8g949c-hnix-0.5.2/lib/ghc-8.4.3/x86_64-osx-ghc-8.4.3/libHShnix-0.5.2-GRObQNgD4K53WKtuuoSKVH-ghc8.4.3.dylib: fixing dylib
/nix/store/y06mk1hjnpy13g3ya775g6ylqb8g949c-hnix-0.5.2/lib/ghc-8.4.3/x86_64-osx-ghc-8.4.3/libHShnix-0.5.2-GRObQNgD4K53WKtuuoSKVH-ghc8.4.3.dylib: fixing dylib
/nix/store/y06mk1hjnpy13g3ya775g6ylqb8g949c-hnix-0.5.2/lib/ghc-8.4.3/x86_64-osx-ghc-8.4.3/libHShnix-0.5.2-GRObQNgD4K53WKtuuoSKVH-ghc8.4.3.dylib: fixing dylib
/nix/store/y06mk1hjnpy13g3ya775g6ylqb8g949c-hnix-0.5.2/lib/ghc-8.4.3/x86_64-osx-ghc-8.4.3/libHShnix-0.5.2-GRObQNgD4K53WKtuuoSKVH-ghc8.4.3.dylib: fixing dylib
/nix/store/y06mk1hjnpy13g3ya775g6ylqb8g949c-hnix-0.5.2/lib/ghc-8.4.3/x86_64-osx-ghc-8.4.3/libHShnix-0.5.2-GRObQNgD4K53WKtuuoSKVH-ghc8.4.3.dylib: fixing dylib
/nix/store/y06mk1hjnpy13g3ya775g6ylqb8g949c-hnix-0.5.2/lib/ghc-8.4.3/x86_64-osx-ghc-8.4.3/libHShnix-0.5.2-GRObQNgD4K53WKtuuoSKVH-ghc8.4.3.dylib: fixing dylib
/nix/store/y06mk1hjnpy13g3ya775g6ylqb8g949c-hnix-0.5.2/lib/ghc-8.4.3/x86_64-osx-ghc-8.4.3/libHShnix-0.5.2-GRObQNgD4K53WKtuuoSKVH-ghc8.4.3.dylib: fixing dylib
strip is /nix/store/sixx1lfy97125f6hxymbwn64cb8z62p3-cctools-binutils-darwin/bin/strip
patching script interpreter paths in /nix/store/0jp95ab7ssd96af9nbl7gsnrmapcbqn9-hnix-0.5.2-doc
/nix/store/y06mk1hjnpy13g3ya775g6ylqb8g949c-hnix-0.5.2

Thanks!


Examples don't run because v2.x google provider is required

The examples don't run because they pin to v2.x terraform google provider.
for example, https://github.com/terraform-google-modules/terraform-google-cloud-operations/blob/26efc88d929012d17e17e3f86c61f77861eba0b7/examples/agent_policy_simple_example/main.tf#L18

Thanks, I think it's a good idea to upgrade the examples to v3.x. PRs welcome.


Error in diskcapacity check. 'Assert-DiskSpace' is unknown

Bug Report
General Troubleshooting steps

[x] Verified running the latest release of dbachecks?
V2.0.3
Does (Find-Module dbachecks).Version match (Get-Module dbachecks).Version.ToString()
yes

[x] Can duplicate in new/clean PowerShell session (clean = powershell -NoProfile)?
yes

Version Information

Operating System (Name|Version):  Windows Server 2016
PowerShell Version:  5.1
SQL Server (Edition|Version):
2017

Steps to Reproduce
<--
PLEASE include as much information as possible if this is a bug report.
The more you include the faster we can identify the problem and get it fixed
-->

[x] Attach output from PowerShell console (if possible/allowed)
Invoke-DbcCheck -Check diskcapacity -ComputerName xXxXxX

Description of Bug
Starting discovery in 1 files.
Discovery finished in 2.09s.
[-] Disk Space.Testing Disk Space on xXxXxX.D:\ with 64.49% free should be at least 20% free on xXxXxX 386ms (254ms|133ms)
CommandNotFoundException: The term 'Assert-DiskSpace' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.
at , C:\Program Files\WindowsPowerShell\Modules\dbachecks\2.0.3\checks\Server.Tests.ps1:36
[-] Disk Space.Testing Disk Space on xXxXxX.K:\ with 83.28% free should be at least 20% free on xXxXxX 41ms (39ms|2ms)
CommandNotFoundException: The term 'Assert-DiskSpace' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.
<--
PLEASE include as much information as possible if this is a bug
Explain what it does that you would like it not to do
or
What it doesnt do that you would like it to do :-)
-->

@laric I encountered this same problem when I tried installing dbachecks for the first time last week. I got some help from someone in the Slack channel... I believe the issue is with the Pester module.  When you run the command to install dbachecks, it installs all of the dependent modules, including the most recent version of Pester.  I had to remove Pester and reinstall it, limiting the version to 4.10.1.  This error (and several others I was getting) all went away.

I will give it a go.

That works fine if you force uninstall of pester 5. But this leaves this still as a bug. Not compatible with pester 5.

Agreed. I think this is the same issue reported in 774.
Duplicate of #774


Omit self in constructors and other method signatures

Currently self is included when rendering both constructor (__init__) and standard method signatures. It'd be nice if this parameter was omitted, as is done by other documentation tools like sphinx.
Examples from the quartodoc docs:
Inclusion of self in __init__
See quartodoc.MdRenderer

MdRenderer(self, header_level=1, show_signature=True, show_signature_annotations=False, display_name='relative', hook_pre=None, render_interlinks=False)

Inclusion of self in other methods
See quartodoc.MdRenderer.render_annotation

MdRenderer.render_annotation(self, el)

Thanks for raising--I can take a quick stab at this!
It looks like..

sphinx just checks inspect.ismethod(), and uses that to drop the first parameter
mkdocstrings looks for parameters named "self" or "cls" inside functions on classes

We'll likely need to follow mkdocstrings lead here, since we're also using griffe for static analysis

Should be addressed in at least v0.6.3


Fallback to default when requested language is missing, rather than panicking

I'm fairly new to fluent, and in my application I'm setting the requested language id from the user's locale settings, but there's only the one localization at the moment, and it doesn't seem right to have the application panic at runtime if the user's locale differs from the fallback locale.
At the very least I think it should return an error rather than panic, so that it's recoverable, but it seems to me that the requested language not being available is also a situation where the fallback language should be used.
#15 also would help with avoiding the panic, but I think that for most situations it would be sufficient just to use the fallback language, and simpler from the perspective of using the library.

Thank you for your PR, and congrats on your first contribution! 🎉

This has beenn released as 0.5.12

No problem, and thanks for the quick response!


A larger default marginLeft for y axes

The current default of 40px is often too small, requiring a custom marginLeft. Since we don’t automatically increase the left margin based on the tick labels #383, we should increase the default, say to 60px. And likewise marginRight if the y axis uses the right anchor.
We should also consider an even larger default if the y scale is ordinal, since ordinal names tend to be longer than numbers. Perhaps 80px. The same applies to the fy axis.

Another consideration is that ordinal scales tend to use the center labelAnchor, which consumes additional margins. So perhaps the default margin could adjust based on whether there is a centered label, too.


Week 2 Hacks

Edwin Abraham, Mr. Mortensen Period 4

[ ] Lists and Dictionaries link
[ ] Changing Theme link
[ ] Time Box link

Grade: 2.7
Thoughts:
Edwin did an amazing job at all of the hacks that he was supposed to do. He included very clear comments on what all of the commands did and did a good job at organizing his tags. Additionally, he gave a clear explanation and detailed observations of what happened when he tried to change the minima theme. While Edwin did everything he was supposed to very well, I think he could have added on to it a little, such as using user input to add information to a dictionary. Overall, Edwin did an outstanding job on the hacks for Week 2! :)


Translate May 15 covid report into Turkish

Happy weekend @nextstrain/turkish,
The situation report from 2020-05-15 is ready to be translated to Turkish. This week's report is short and focuses on mutations. It's targeted more at the public than some of our previous reports and is intended to answer some of the many questions we've been receiving on the subject. I hope it will be useful for Turkish speakers.
Sadly, this will be our last weekly Situation Report for awhile although we'll issue periodic sit-reps as needed. Thank you, thank you, thank you for translating these reports. It's been an amazing collaborative effort to be involved in. I'll miss working with you on them.  I've personally derived a lot of global solidarity from our work together. Although we will all probably enjoy having more time on the weekend!
Branch to translate on: sit-rep_tr
File to translate: https://github.com/nextstrain/translations/blob/sit-rep_tr/narratives/translations_ncov_sit-rep_tr_2020-05-15.md You can translate directly in this file, but please remember to push often as other people may be working on the translation!
Link to preview: https://nextstrain.org/community/narratives/nextstrain/translations@sit-rep_tr/ncov/sit-rep/tr/2020-05-15
Twitter post to translate:
“Our latest situation report focuses on viral mutations: why they occur, when they matter, and how they can be useful in tracking outbreaks. The report is now available in Turkish.
Since repeated patterns of viral introduction and spread across the world have become clear, we'll be pausing weekly situation reports for a while. We will publish periodic situation reports as the data allows.
Thank you to @, @, @ for their rapid translation of this report. Thank you to @, @ for their work on previous translations as well."
Regards,
Cassia
Logistics
Please update the translations board as you work on the translation: https://github.com/orgs/nextstrain/projects/7
Please assign yourself to the translation if you’re working on it this week.
When a translation is ready for publication -- at minimum meaning two people have worked on it and both agree that it’s ready -- please create a pull request to merge to master and tag me and the issue in it!
Discussion board for your team is here: https://github.com/orgs/nextstrain/teams/turkish
Standard rules for translation:
Note that the file to translate is in Markdown. In general, it is important not to change any formatting or links. (Although it is okay to update Wikipedia or CDC links to the appropriate language version.) Comments to help with translation are noted in comment symbols like <! -- Comment -->.
Please add your name under translators: with new line, tab, and dash, formatted just like authors are. Please also add a link to your professional page (can use Twitter) under translatorLinks:.
Note: You may know of other great resources available in Turkish; please don’t add them to the situation report. We need to keep content the same across all translations, so we can be confident in the accuracy of what we publish. Thanks for understanding.
At minimum, I require two sets of eyes on every translation; one of those people should have a background in phylogenetics. Although if you have more, that’s fine too.
Feel free to use a live, online editor like hackmd (sign in with GitHub) to translate together.

Hello,
I think translation is done so I will be proofreading it first thing in the morning tomorrow if it is okay for everyone.

That would be great @zeynephrc!

Hello,
I've finished the proofreading and @cassiawag it is ready for publication.
Twitter post:
“Son durum raporumuz, viral mutasyonlara  ve neden ortaya çıktıklarına, ne zaman önemli olduklarına ve salgınları izlemek için nasıl kullanılabileceğine odaklanıyor. Şimdi Türkçe haliyle de yayında!
Hızlı çevirileri için @zharcanoglu @benuraa ve @cagataytarhan'a çok teşekkür ediyoruz. Ayrıca @erenada 'ya da önceki çevirileri için çok teşekkür ediyoruz.

Published and tweeted: https://nextstrain.org/narratives/ncov/sit-rep/tr/2020-05-15
Thank you all, really been great to work with you on this. Zeynep, still mad props for getting Turkish translations started; keep up the leadership!

Thank you @cassiawag and all the Turkish translation team. It's been really really amazing to work with you. @benuraa @erenada and @Cagataytarhan it was a privilege for me to collaborate with you and contribute to Nextstrain in these times especially.
And Cassia, I'll definitely continue to work passionately in these fields. Thank you for giving me this chance!


Cannot work with anuvyklack/windows.nvim

Not gonna fix this currently because I don't use it anymore.


Upgrade solutions to target 2.2.207 SDK

Kindly upgrade the projects to the Azure Service Fabric 2.2.207 SDK.

Done.


Create doc pages for APIM Extension with PingIntelligence

Description:
Create official doc pages for APIM Extension with PingIntelligence.
Reference content: https://github.com/wso2-extensions/apim-handler-pingai/blob/master/README.md
Affected Product Version:
3.0.0

APIM Traffic Tool will be another tool used with the Ping Extention. Needs to create Traffic Tool docs too. Created issue for it: https://github.com/wso2/docs-apim/issues/548


slight rewording of mac instructions

I think we should leave this as-is. I'm on an Intel MacBook Pro, and without running pip install tensorflow-macos tensorflow-metal Polymath crashes with the following error:
ModuleNotFoundError: No module named 'tensorflow'


RDF Mapping

As stated in https://github.com/Onto-Med/top-frontend/issues/335, we want to provide content of the TOP Framework in RDF format to enable better interoperability and as a step to implement the FAIR principles.

[ ] decide, which types of resources to map
[ ] find suitable vocabularies and properties
[ ] create mapping
[ ] decide on a namespace

Types of resources

organisation
repository
phenotypes (see https://github.com/Onto-Med/COP)

We will stick to phenotypes for now, leaving out text related concepts.

Organisation

TOP   property
RDFS equivalent
value

id
dcterms:identifier

name
dcterms:title

description
dcterms:description

createdAt
dcterms:created

updatedAt
dcterms:modified

superOrganisation
dcterms:isPartOf
superOrganisation.id

permission
-

dcterms:publisher
framework base URL

foaf:page
URL of organisation, without /api

rdf:type
http://purl.org/dc/dcmitype/Collection

rdf:type
foaf:Organisation

Repository

TOP   property
RDFS equivalent
value

id
dcterms:identifier

name
dcterms:title

description
dcterms:description

primary
dcterms:accessRights
true=http://publications.europa.eu/resource/authority/access-right/PUBLIC       false=http://publications.europa.eu/resource/authority/access-right/RESTRICTED

createdAt
dcterms:created

updatedAt
dcterms:modified

organisation
dcterms:isPartOf
organisation.id

repositoryType
dcterms:type
cop:Phenotype

dcterms:publisher
framework base URL

foaf:page
URL of repository, without /api

rdf:type
http://purl.org/dc/dcmitype/Collection

Phenotype

TOP   property
RDFS equivalent
value

id
dcterms:identifier

entityType
rdf:type
single_phenotype=cop:BasicPhenotypeClass     composite_phenotype=cop:DerivedPhenotypeClass     single_restriction=cop:BasicPhenotypeRestriction     composite_restriction=cop:DerivedPhenotypeRestriction

titles
dcterms:title

synonyms
dcterms:alternative

descriptions
dcterms:description

version
pav:version

author
dcterms:creator

createdAt
dcterms:created

codes
skos:closeMatch
code URI

repository
dcterms:isPartOf

equivalentEntities
skos:exactMatch

superCategories
dcterms:isPartOf

subCategories
dcterms:hasPart

superPhenotype
single_restriction=cop:basicRestrictionOf     composite_restriction=cop:derivedRestrictionOf

dataType
sd:hasDataType
map to ONTODT or xsd data types?

itemType
cop:hasRepresentationAttribute

restriction

expression
cop:hasRepresentationAttribute
expression.toString()

unit
iao:has measurement unit label
map UCUM to UO concepts?

dcterms:publisher
framework base URL

foaf:page
URL of phenotype, without /api

dcterms:replaces
URL of previous version

dcterms:isReplacedBy
URL of next version


Model lists are very slow to load

Since the change to a unified model listing, the list seems very slow to load. We need to do some optimisation.

I think this is the offending line, it's doing a query for every tag:
https://github.com/Floppy/van_dam/blob/109f523881b0d581b92cc3e24271ceaf7b40ff97/app/views/application/_tag.html.erb#L5
Thing is, Tag keeps a count in taggings_count so we don't need to do this.

Ahh, oops.    That also explains why I wasn't seeing the problem, I trimmed
my tags down significantly to useful ones.
On Mon, Mar 13, 2023, 5:50 PM James Smith @.***> wrote:

I think this is the offending line, it's doing a query for every tag:
https://github.com/Floppy/van_dam/blob/109f523881b0d581b92cc3e24271ceaf7b40ff97/app/views/application/_tag.html.erb#L5
Thing is, Tag keeps a count in taggings_count so we don't need to do this.
—
Reply to this email directly, view it on GitHub
https://github.com/Floppy/van_dam/issues/1092#issuecomment-1467082544,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAS526OFXRDPZKG2QGQDA6TW36QETANCNFSM6AAAAAAVYHEOCE
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Failures: flows/AddProjectChatter-1.flow(AddProjectChatter-1):Post_Comment_on_Project (Action Call) - We can't find an action with the name and action type that you specified.

Please Help
Getting this error while trying to deploy
Failures:
flows/AddProjectChatter-1.flow(AddProjectChatter-1):Post_Comment_on_Project (Action Call) - We can't find an action with the name and action type that you specified.

Sorry for the late reply @amrit-sharma, this action is defined in the Flow correctly. However i am wondering if you are deploying to an org that does not have Chatter enabled?


Why 'colors' node module is restricted?

See here:
https://github.com/sindresorhus/eslint-config-xo/blob/master/index.js#L118
What is the reason for it being restricted?

See the intro text here: https://github.com/chalk/chalk
I suggest using chalk or cli-colors instead.


Pin dev/test dependencies

Currently the CI installs the latest dev and test dependencies, and this might cause troubles when patching or hot-fixing older releases due to breaking changes in the latest dependencies update.
Pin dev and  test dependences to avoid the CI to fail when patching or hot-fixing older releases.
Addresses: https://github.com/frequenz-floss/frequenz-sdk-python/issues/243

Also please use Fixes #... in the PR body, so the PR gets associated with the issue (and the issue closed when it's merged)


Hawkular-sink updates to definition cache handling

disablePreCache parameter set to true will prevent initial fetch of definitions from the metrics server (instead building it from the scratch when storing datapoints)
Each time datapoint is stored, the tags are compared to see if there's an update (such as labels)
Increase concurrency of registerLabeledIfNecessary with changes the mutexes (they only prevent concurrent cache modification now)

cc @mwringe

This change is

@mwringe can you do a review?

And what happened to this..?

Merging, since this is from the sink owner and seems to have a generally positive review otherwise.


Amazing Library!

Hey, I just want to say thank you for this library! We have been using LinqToTwitter but recently our users have had very inconsistent results when Tweeting with video media. Some posts would go out others would not attach the video. I found your library and noticed you wrote a custom chunking method that works with Twitter's new requirements for chunking all video uploads.
Anyway, its working perfect now with your library! Thanks again we will recommend to anyone looking for a C# Twitter library.

Hello,
I would like to say that I really appreciate this message. It is really nice to receive positive feedback from the community but even more when someone just open an issue to say it.
So thank you very much for this message, you have made my day :)
Cheers,
Thomas


CalledProcessError: Command 'lake build Lean4Repl' returned non-zero exit status 1.

Description
I tried setting up LeanDojo (and running the demo-lean4.ipynb file) in vscode, and ran into the error in the header. I also tried running it in Jupyter Notebook, and got the same error. Is there an installation I'm missing?
Detailed Steps to Reproduce the Behavior
I ran the following commands:
conda create -n "lean4test" python=3.10
conda activate lean4test
jupyter notebook
I set the GitHub Access Token locally within the notebook, and then ran the cells. I ran into an issue in the first cell under the "Interact through Tactics" section. The line causing the error:
dojo, state_0 = Dojo(theorem).enter()
Logs in Debug Mode
`023-12-04 21:30:48.768 | WARNING  | lean_dojo.interaction.dojo:__init__:172 - Using Lean 4 without a hard timeout may hang indefinitely.
2023-12-04 21:32:06.550 | INFO     | lean_dojo.utils:execute:115 -
2023-12-04 21:32:06.553 | ERROR    | lean_dojo.utils:execute:116 - error: unknown target `Lean4Repl`

---------------------------------------------------------------------------
CalledProcessError                        Traceback (most recent call last)
Cell In[28], line 5
      2 theorem = Theorem(repo, "Mathlib/LinearAlgebra/Basic.lean", "pi_eq_sum_univ")
      4 # For some theorems, it might take a few minutes.
----> 5 dojo, state_0 = Dojo(theorem).__enter__()

File /Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/lean_dojo/interaction/dojo.py:293, in Dojo.__enter__(self)
    291 os.chdir(self.origin_dir)
    292 shutil.rmtree(self.tmp_dir)
--> 293 raise ex

File /Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/lean_dojo/interaction/dojo.py:237, in Dojo.__enter__(self)
    235     cmd = f"./build/release/stage1/bin/lean {self.file_path}"
    236 else:
--> 237     self.container.run(
    238         "lake build Lean4Repl",
    239         mts,
    240         as_current_user=True,
    241         capture_output=True,
    242         work_dir=f"/workspace/{self.repo.name}",
    243         cpu_limit=None,
    244         memory_limit=None,
    245         envs={},
    246     )
    247     cmd = f"lake env lean {self.file_path}"
    249 self.proc = self.container.run_interactive(
    250     cmd,
    251     mts,
   (...)
    256     envs={},
    257 )

File /Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/lean_dojo/container.py:193, in NativeContainer.run(self, command, mounts, envs, as_current_user, capture_output, cpu_limit, memory_limit, work_dir)
    190         work_dir = Path.cwd() / work_dir.relative_to(work_dir.root)
    192 with working_directory(work_dir):
--> 193     execute(cmd, capture_output=capture_output)
    195 self._unmount_files(mounts)

File /Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/lean_dojo/utils.py:117, in execute(cmd, capture_output)
    115         logger.info(ex.stdout.decode())
    116         logger.error(ex.stderr.decode())
--> 117     raise ex
    118 if not capture_output:
    119     return None

File /Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/lean_dojo/utils.py:112, in execute(cmd, capture_output)
    102 """Execute the shell command ``cmd`` and optionally return its output.
    103
    104 Args:
   (...)
    109     Optional[Tuple[str, str]]: The command's output, including stdout and stderr (None if ``capture_output == False``).
    110 """
    111 try:
--> 112     res = subprocess.run(cmd, shell=True, capture_output=capture_output, check=True)
    113 except subprocess.CalledProcessError as ex:
    114     if capture_output:

File /Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/subprocess.py:524, in run(input, capture_output, timeout, check, *popenargs, **kwargs)
    522     retcode = process.poll()
    523     if check and retcode:
--> 524         raise CalledProcessError(retcode, process.args,
    525                                  output=stdout, stderr=stderr)
    526 return CompletedProcess(process.args, retcode, stdout, stderr)

CalledProcessError: Command 'lake build Lean4Repl' returned non-zero exit status 1.`

Platform Information

OS: macOS
Browser Safari
LeanDojo Version 1.4.0

I also tried it with LeanDojo versions 1.2.6, 1.3.0, and 1.4.2. Let me know if some other version might be better for this. Thank you!

Hi,
This looks similar to https://github.com/lean-dojo/LeanDojo/discussions/99? I'll be able to take a look around Christmas holidays.

@shubhramishra07 Other cells was fine before dojo, state_0 = Dojo(theorem).enter()?

I can reproduce the error! It will be solved very soon.

Thank you very much!

I fixed it in https://github.com/lean-dojo/LeanDojo/pull/112. Could you try the latest main branch *version 1.4.4rc) and see if the problem is resolved? It can be installed by:
git clone https://github.com/lean-dojo/LeanDojo
cd LeanDojo
pip install --ignore-installed .

I have released it as a new version (v1.4.4)

Feel free to re-open if it doesn't work.


Defered State

Hi, so after poking around the source code, I can't see evidence that anyone considered using DBT's defer flag. I wonder if anyone tried incorporating it or if this was considered a bad idea.
I'd be happy to make a PR to include it, but I want to get some context first; previous attempts/discussions I may have missed. Also, I realize you are using dbt internals, so it might not be a good practice to make code here more dependent on dbt's code, and in this case, I think this would be required.

Hey @ce-tcholewik thanks for raising an issue!
I hadn't heard of the defer CLI flag in dbt before. Looks very useful for development workflows. However I am struggling to see what problem it would solve by adding it to the dry runner though? Can you elaborate on your use case?
If you wanted to dry run a dbt project that had been compiled with defer I think that would be possible:
# compile the manifest with deferred references
dbt compile --defer
# Run the dry runner
dbt-dry-run

Although the dry runner works best when you run it against your production environment if you are using it in CI.


How to implement the plugin within a Moodle plugin

Hi there,
I'm writing a plugin in Moodle which uses Chart JS - specficially the Doughnut/Bar chart - however, the charts lack any kind of accessibility features. Looking to use your plugin within Chart JS as the keyboard navigation is exactly what I'm looking for which would help meet our accessibility requirements.
Without having to go down the NPM route to install this, and having tried the Codepen example (https://codepen.io/chart2music/pen/ZEMyLVZ) - which unfortunately doesn't work for me - is there a way to get this running in Moodle?
Currently, trying to get this running using the Codepen example gives me the following console log errors:
Uncaught SyntaxError: Unexpected token 'export' (at chartjs-plugin-a11y-legend@0.0.2:15:1)

A couple clarifying questions -

Are you seeing the errors in CodePen itself, or are you seeing the errors when you copy the code somewhere else?
What operating system and browser combination are you using?
Are you referring to the keyboard navigation within the chart, or are you looking for keyboard navigation for the chart legend? The chartjs-plugin-a11y-legend repo only manages the keyboard navigation of the legend, whereas the chartjs2music repo handles the keyboard navigation for the chart data.

All that said, if you are getting a syntax error on the keyword "export", you might need another version. CodePen supports ES6 imports and exports, but if your build environment doesn't, you might want to try linking to this URL:
https://cdn.jsdelivr.net/npm/chartjs-plugin-a11y-legend
If you add that script, it should initialize a global variable named chartjsPluginA11yLegend, which you would use to register the Chart.js plugin. (Chart.js plugin documentation).

Hi Julianna - thanks for getting back so quickly.

Not in CodePen itself, it was when using the import statement 'import a11yLegend from "https://cdn.skypack.dev/chartjs-plugin-a11y-legend@0.0.2";' in my script. Not sure how familiar you are with how Moodle does things, but it's an AMD module that is getting compiled using Grunt.
Running this using Multipass on a Mac - basically, Docker containers set up to run web/db servers via Ubuntu. Browsers used are Chrome/FF/Opera.
Ideally it would/could be for both. I've managed to get the legends and tooltips running with their own onclick handlers - the plugin that you have developed will be ideal for adding keyboard navigation ^to^ the legends essentially - I have therefore not used the chartjs2music code as I'm happy to see this working just for the legends to begin with.

I will try your suggested link in the meantime.
Thanks again.

Hi Julianna - that link didn't work I'm afraid.
That would be great if you could open the PR when you get a chance.
Thanks again!

Just to set expectations, it will probably be another week or 2 until that comes through, but this is on my list.

No problems - thanks again for accommodating this request.

Had some unexpected free time, so I took a stab at this. I'm now outputting AMD format in v0.2.0.
Here is a direct link to a minified, AMD-formatted js file. Here's the unminified version.
I haven't used anything in an AMD format in a long time, so I'm not really sure how to test it... Do you mind giving that file a try, and let me know how it works?

That seems to have worked perfectly - thanks again for making those changes.
I'll get some end users to check that all is working as expected, but for now that change seems ideal.

Awesome! I'm glad that worked.
Note, I've opened an issue in the other plugin to support AMD formats, so that it'll be there for you when you need it.


Stuck on "Quality" window when creating icon

When trying to create an icon (either import or create) I get to the select quality box and don't have an OK button to press
To Reproduce
Steps to reproduce the behavior:

Click on 'import icon'
Select image in explorer pop up
Select desired quality in Quality pop up window
Unable to proceed - no OK button visible

Expected behavior
I'd expect there to be some way to accept the selection
Screenshots

Additional context
Add any other context about the problem here.
Log file
Please upload the latest log file (Help -> Export latest log)
2022-04-26.log

This is a glitch because I think you're using a small, really high resolution display with a scale of >100%. You can temporarely fix this following these steps:

Navigate to C:\Program Files\Macro Deck (Or the installation directory of Macro Deck) in the Windows Explorer
Right click Macro Deck 2.exe and click "Properties"
Click "Compatibility" and then "Change high DPI settings"
Now check "Override high DPI scaling behaviour" and select "System (Enchanced)"
Then you can close the dialoges with "Ok"


Update/Option Menu

💬 Description
Adds the OptionMenu component.

Updates DropdownContent with our new GCDS border, shadow and border radius
Adds OptionMenu component

Check it 👉 https://share.getcloudapp.com/KouqljJ0
🔗 Links

GCDS Designs: https://www.figma.com/file/66iSurBAx32DWrJymo9NKL/GCDS-–-Web-UI?node-id=321%3A668
Formatting Toolbar Designs: figma.com/file/nYWpuGuUnmpPMm0E4x0cKY/Static-Toolbar?node-id=266%3A0

📹 GIF (optional)
A short GIF of the changes in action.
🚪 Start Points
Where is a good place to start the review? If there are multiple changes it may be worth listing multiple files.
👫 Related PRs (optional)
Any PRs that relate to the changes.
✅ Checklist

[ ] Tests written
[ ] Browser tested
[ ] Added to documentation
[ ] Added to storybook

 Add Dropdown Option Menu to GUI from GCDS


Optimized_thetas Issue

Why getting this error?
FileNotFoundError: No such File or Directory: Optimized_thetas/model_2017-06-3_21-08-44_10.0_h50.pkl

Hi @Noorulain15, This error is caused by the absence of the optimized_thetas directory. The commit 048d983 fixes the issue.


[minor] Migrating from Mint to Tesla

This will give more freedom to the consumer to configure it's own adapter, so now the SSL and Pool stuff are delegated to the adapter itself.
Sadly this is a breaking change, and it breaks streamming because Tesla does not support it yet :(.

Pull Request Test Coverage Report for Build 919f18ef4c2343209eb9f1410bb9a235889d24e2-PR-30

106 of 128 (82.81%)  changed or added relevant lines in 10 files are covered.
19 unchanged lines in 8 files lost coverage.
Overall coverage decreased (-0.6%) to 86.145%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/rql/query.ex
4
6
66.67%

lib/connection/executor/supervisor.ex
17
20
85.0%

lib/connection/connection_state_manager.ex
7
11
63.64%

lib/connection/executor/request_executor.ex
39
52
75.0%

Files with Coverage Reduction
New Missed Lines
%

lib/connection/models/node_selector.ex
1
83.33%

lib/connection/models/server_node.ex
1
92.31%

lib/rql/query.ex
1
78.13%

lib/connection/connection_state_manager.ex
2
79.07%

lib/connection/executor/supervisor.ex
2
82.86%

lib/documents/session/session.ex
2
90.91%

lib/documents/commands/stream_query_command.ex
4
0%

lib/documents/session/session_manager.ex
6
87.67%

Totals

Change from base Build 6540da7a22d07272ac59deb072ebb7a18f451511:
-0.6%

Covered Lines:
628

Relevant Lines:
729

💛  - Coveralls


terragrunt plan-all is flakey from 0.19.25

Hi
We recently noticed that terragrunt plan-all is flakey and would randomly fail with files not found errors, sometimes reporting that a terraform backend hasn't been configured, despite being configured correctly:
.terragrunt-cache/yZpa9oXGPFA9yBeYJLKvMDWHQLA/nOHeCNZgRObX4kF5VK1jboCrx_w/.terragrunt-module-manifest: no such file or directory

I wrote a test to reproduce the problem and if the subtest runs 100 times I get to reproduce the problem at least twice (this can be run in integration_test.go on the master branch):
func TestDependencyPlanAllFlake(t *testing.T) {

	cleanupTerraformFolder(t, TEST_FIXTURE_GET_OUTPUT)
	tmpEnvPath := copyEnvironment(t, TEST_FIXTURE_GET_OUTPUT)
	rootPath := util.JoinPath(tmpEnvPath, TEST_FIXTURE_GET_OUTPUT, "localstate", "live")

	stdout := bytes.Buffer{}
	stderr := bytes.Buffer{}
	err := runTerragruntCommand(
		t,
		fmt.Sprintf("terragrunt apply-all --terragrunt-non-interactive --terragrunt-working-dir %s", rootPath),
		&stdout,
		&stderr,
	)
	logBufferContentsLineByLine(t, stdout, "stdout")
	logBufferContentsLineByLine(t, stderr, "stderr")
	require.NoError(t, err)
	for i :=0; i<100; i++ {
		t.Run(fmt.Sprintf("take %d", i), func(t *testing.T) {
			stdout := bytes.Buffer{}
			stderr := bytes.Buffer{}
			err := runTerragruntCommand(
				t,
				fmt.Sprintf("terragrunt plan-all --terragrunt-non-interactive --terragrunt-working-dir %s", rootPath),
				&stdout,
				&stderr,
			)
			logBufferContentsLineByLine(t, stdout, "stdout")
			logBufferContentsLineByLine(t, stderr, "stderr")
			require.NoError(t, err)
		})
	}
}

The test output for the failed cases looks like:
            	            	Encountered the following errors:
            	            	open /var/folders/cf/nhvbzztn3yn08kfswy1rqkmh0000gn/T/terragrunt-test981814188/fixture-get-output/localstate/live/parent/.terragrunt-cache/yZpa9oXGPFA9yBeYJLKvMDWHQLA/nOHeCNZgRObX4kF5VK1jboCrx_w/.terragrunt-module-manifest: no such file or directory

We downgraded to 0.19.24 where it appears to be stable.
Would it be possible to have a look and understand where the race condition is coming from, please?

@ekini Do you have any idea what could be causing this?

I haven't been able to reproduce it so far. Will try to investigate more tomorrow.

I'm having the same problem as @dvulpe said, and on version 0.19.24 everything seems to work fine.
Example error output at the end of terragrunt plan-all command using 0.20.4 version:
[terragrunt] 2019/10/15 07:42:32 Encountered the following errors:
Hit multiple errors:
error downloading 'file:///Users/rmitula/Desktop/Company/infrastructure/terraform': open /Users/rmitula/Desktop/Company/infrastructure/live/global/dev/.terragrunt-cache/XM0YtL2xdz7j-Sq9CHfbzNBuR90/rXfWpL2tPlCaJpLebr06_0ly1Lg/global/modules/generic-config/files/.terragrunt-source-manifest: no such file or directory
Hit multiple errors:
error downloading 'file:///Users/rmitula/Desktop/Company/infrastructure/terraform': open /Users/rmitula/Desktop/Company/infrastructure/live/global/dev/.terragrunt-cache/XM0YtL2xdz7j-Sq9CHfbzNBuR90/rXfWpL2tPlCaJpLebr06_0ly1Lg/data-providers/provider3/modules/process-1/files/.terragrunt-source-manifest: no such file or directory
[terragrunt] 2019/10/15 07:42:32 Unable to determine underlying exit code, so Terragrunt will exit with error code 1

The commits in question must be here https://github.com/gruntwork-io/terragrunt/pull/866 as it's the only difference between 0.19.24 and 0.19.25.
However, I don't see anything wrong with it.
I still can't reproduce the issue. I've tried:

Fresh repo on Mac with the test TestDependencyPlanAllFlake. Even with 500 passes it succeeds.
Fresh repo in Docker. Also no issues.
I used Terraform v0.12.10 and 0.12.7.

Noticing the same issue. It's really intermittent
terragrunt version v0.20.4
Terraform v0.12.12
OSX + Ubuntu
``
[terragrunt] 2019/10/24 11:55:13 Encountered the following errors:
open /Users/peter.odonovan/projects/xxxxxxxx/deploy/xxxxxxx/vpc/.terragrunt-cache/eBDA0HQj6jnPLe2W4IcRuLq4eII/SmNP8PUyhYl3r9vRGSS8PFqGa5I/ci-runner.tf: no such file or directory
[terragrunt] 2019/10/24 11:55:13 Unable to determine underlying exit code, so Terragrunt will exit with error code 1

Hi @yorinasub17
I'll give the alpha version a go this week and let you know how it goes.
Thanks,
Dan

@dvulpe thanks! I would also appreciate it if you can give https://github.com/gruntwork-io/terragrunt/releases/tag/v0.21.2-alpha.2 (https://github.com/gruntwork-io/terragrunt/pull/937) a try as well. Unlike the previous fix which rolls back the change, this will do proper synchronization of the terragrunt output call (and cache the results) so that we can keep the original change that is causing problems. If this works, we will most likely go with this version of the fix.

@yorinasub17 ive been having flaky *-all that seemed to be around having multiple modules depending on the same module (they all need to read its outputs). I have no idea if it's related to this issue, but I tested the latest alpha and so far the flakiness seems to be resolved.

this also resolved flaky in our gitlab CI/CD pipeline
example error from our terragrunt demo project pipeline:
[terragrunt] 2019/10/31 18:06:13 Encountered the following errors:
open /builds/heb-engineering/projects/heb-eng-sandbox/heb-ecom-sbxdemo/heb-ecom-sbxdemo-terraform/accounts/heb-ecom-sbxdemo-alpha/environments/bld/account-bootstrap/gitlab/group/.terragrunt-cache/FY9QfprjjTbanXO3jXY6Y6yNypU/Pr91Q2cIsjPsXNWIjAHf_EvC9aQ/outputs.tf: no such file or directory
[terragrunt] 2019/10/31 18:06:13 Unable to determine underlying exit code, so Terragrunt will exit with error code 1
ERROR: Job failed: exit code 1

Released the latest alpha fix as v0.21.2 (https://github.com/gruntwork-io/terragrunt/releases/tag/v0.21.2). Binaries will show up shortly. The original alpha releases will be deleted once the binaries for v0.21.2 show up.
I will keep this open until I have confirmation that this fixes the root issue.

Great work!
I can confirm the latest release v0.21.2 completely solves the flakiness we were experiencing with plan-all.
Thank you so much for fixing it!

Thanks for confirming! Will close this out.


added fake icclim module to access icclim related functions directly

Documentation now includes an ICCLIM page where we'll find ICCLIM indicators under their own ICCLIM standard name.

Pull Request Test Coverage Report for Build 103

5 of 5 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.7%) to 52.632%

Totals

Change from base Build 98:
0.7%

Covered Lines:
150

Relevant Lines:
285

💛  - Coveralls


dep: update libxml2 to v2.10.3 (backport to v1.13.x)

What problem is this PR intended to solve?
Backport of #2666 to the v1.13.x branch.
Also see parent issue #2665.

Hey @flavorjones Would it be okay if I put a PR for backporting this to 1.12.x?
Curious if there is any strong reason for not doing so 😄

@EduardoGHdez Thanks for asking this question.
The v1.12.x branch is not maintained and we are not planning on making any more releases on that branch. Can you say more about what you're hoping to achieve with a backport? Why aren't you instead upgrading to v1.13.x?
If your primary interest is security, then I'm obligated to point out there are published vulnerabilities in the v1.12.x code that will remain unaddressed (see here and here).

Why aren't you instead upgrading to v1.13.x?

It's in our plans to upgrade to 1.13.x, but the app I'm working on hasn't been upgraded to ruby 2.6.x (Nokogiri 1.13 requires using it)
I was just looking for a temp solution meanwhile we work on the ruby upgrade.
Perhaps this is not but one more reason to prioritize that upgrade work 😅
Thanks for answering and pointing to the other unaddressed vulnerabilities 🙌🏽 , this is very helpful 🙇🏽


Start a stack: no space left on device

If I try to start a stack with a link like
https://labs.play-with-docker.com/?stack=https://raw.githubusercontent.com/t3easy/docker-typo3/11.x/.docker/pwd/stack.yml
I get an error:
Error response from daemon: manager stopped: can't initialize raft node: failed to create WAL: no space left on device

/dev/sdb  is running out of space on https://labs.play-with-k8s.com
kubadm init is failing due to this.

This is still an issue. Keeps running out of space during init. Which makes the playground unusable.
It also ends up refusing connection, like its not starting up even if the init command is successfull. Running systemctl status kubelet gives me an error saying it cant create temporary file: no space left on device. So cant verify that its running.

This is still an issue. Keeps running out of space during init. Which makes the playground unusable.

can you try again please? Just forced some cleanup :pray:


Uncaught Error: ENOENT: no such file or directory, open '/Users/igorikeda/.atom/packages/sync-set...

Duplicate of https://github.com/Hackafe/atom-sync-settings/issues/194.
In the above error report you have shared your personalAccessToken. You must revoke it to prevent abuse.


SignalR issue with  '@' in path

Visual Studio 2019
When using 'Add Client-Side Library' within Visual Studio to add the SignalR JS Library's the following path is created by default:
<EMAIL_ADDRESS>When this is used in a Razor Page (I guess all of them but I have not tested) Visual Studio will not compile.
Error: 'The name 'aspnet' does not exist in the current context'
It is the '@' that is the issue. I don't need a work around as its easy enough to fix but I don't see why its there in the first place. I also know I can change the install path in the 'Add Client-Side Library' (NOW) as well.
We as programmers have enough to learn and remember with all the goal posts you keep changing but silly things like this are becoming to the point I'm seriously debating dropping everything .NET etc.
Why would you ever put a symbol in the path that your own products complain about. Do you not test this stuff?

@mkArtakMSFT This looks like a razor issue.

This is an NPM standard pattern, so SignalR is not unique here. For example, all the Angular packages would have the same problem. Sounds like this is actually an issue in the "Add Client-Side Library" logic in VS, can you file a VS Feedback item about it (using the feedback button in the top-right corner of Visual Studio)

Agree with @anurse.
This looks like a VS tooling issue. Please use VS Feedback to report this issue.


allow to specify different base DN for group lookups

I have multiple wordpresses, which use a single ldap base.
Users have different rights in different WP installations.
I have a ou for each WP with user groups in those.
so i need to set the group-search base-dn for each WP installation to be different from the user-dn.
This patch adds a option to specify the search base in the ldap->search function. if none is specified, the search base is taken from the ldap-uri (same as before this patch).

Thanks for your contribution.
I'll have a look at it later today!

I've notuced that you gave removed a whole bunch of files and changed some. code that is not releva t to the functionality you addressed. I'd be really grateful if you could provide a PR with just the relevant changes. Thanks

sorry about that, i make a new PR with only the relevant merges.


connected/disconnected operations mode

Ability to turn should_receive into should_call with an env variable to enable running same set of tests as unit tests or integration tests

Won't be implementing this for flexmock, maybe do it in testutils..


Polyline with height height greater than 0 breaks Datasources hierarchy

The bug occurs when we add to a polyline inside a Datasource a 'height' property with a value greater than zero. When creating polyline with 'height' greater than 0, the polyline ignores the DatasourceCollection Hierarchy and goes above all entities.
The required view to reproducing the bug is 2D.
The following code creates two Datasources, one that contains cesium Entity with polyline and one that contains cesium Entity with a billboard.
The first example works as expected, the polyline will be under the billboard (same as the Datasources hierarchy).
In the second example, bug reproduced, the polyline will go above the billboard regardless of Datasources Hierarchy.
Note that the only difference between the two examples is that in the second example one of the polylines coordinates as a height of 1.
"positions: [Cesium.Cartesian3.fromDegrees(-75, 35), Cesium.Cartesian3.fromDegrees(-125, 35, 1)],"
I am assuming this is a bug because in other graphic entities like BillboardGraphic when creating an entity with a height greater than zero the entity will not break the Datasources hierarchy.
Billboard with height under another billboard (same hierarchy as Datasources).

Cesium view: 2D
Browser: Chrome
Operating System: Windows

Thanks @Lampent, this is a known issue.  I have added a comment to https://github.com/CesiumGS/cesium/issues/3647 so we can notify you when the issue is fixed.


Response -> validateNumAssertions() logic error.

I think the logic of validateNumAssertions() function from Response of SAML2 is wrong because Assertion or EncryptedAssertion can have Advice element, that inside can contain more than 1 another Assertion's or EncryptedAssertion's. The result of function will be false even the Response is valid.
According the Documentation

You are right, but this toolkit only support 1 Assertion.

Toolkit tries to follow saml2int profile
Assuming a successful response, the <saml2p:Response> message issued by an Identity Provider MUST contain exactly one assertion (either a <saml2:Assertion> or an <saml2:EncryptedAssertion> element).

Inside saml2int there is no any word about Advice element.
From saml2int also, inside 10 Normative References I found [SAML2Core] which has

Advice [Optional]
Additional information related to the assertion that assists processing in certain situations but which
MAY be ignored by applications that do not understand the advice or do not wish to make use of it.

The Advice element contains any additional information that the SAML authority wishes to provide.
This information MAY be ignored by applications without affecting either the semantics or the validity of the assertion.
The Advice element contains a mixture of zero or more Assertion, EncryptedAssertion, AssertionIDRef, and AssertionURIRef elements, and namespace-qualified elements in other non-SAML namespaces.

Hi @pitbulk.
Could you reopen this issue?

+1

Sorry but we don't plan to support Advice element.

Even if I made a PR #113 with Advice validation implemented?

If the Service Provider gonna ignore the content of the Advice element, (99% of use cases) I don't see any point of supporting that element.
Support the scenario of more than 1 Assertion only implies possible security issues.
If you have this requirement for your specific protect, you always can extend the toolkit and adapt it to your needs.


Property 'self.newVideo' not initialized at super.init call

Hi @BrianyinIOS , could you provide some more details about this issue?
What version of XCode are you using?
Is this occurring when using the framework through CocoaPods or if you try to build the source?

The Xcode version I use is 10.2.1. This problem has been solved, because the cached ASPlayer in my cocopods is not the latest one, I just need to clear it and reinstall it. Finally, thank you for your reply！


New Workbook

Required items, please complete
Change(s):

Cleaned folder structure and added a new Workbook

Reason for Change(s):

See guidance below

Version Updated:
Testing Completed:

In running environment

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

yes

Hello @FelixBueltmannBarracuda I upgraded the api version due to which the arm ttk check was failing but now this new content validation check is failing please look into this

Looking into this

Hello @FelixBueltmannBarracuda please update on the following

Hello @FelixBueltmannBarracuda please provide your reply on the above comments

Hello @FelixBueltmannBarracuda please provide your feedback on this

Hello @FelixBueltmannBarracuda its been more than a month please reply on the above comments
or this pr will be closed

@FelixBueltmannBarracuda, Since we have not received a response from quite a long, we are closing your PR #<7443> as per our standard operating procedures. If you still need support for this issue, feel free to re-open at any time. Thank you for your co-operation.


insane memory use?

When I use npm-check it uses 10gb of ram for some reason. Does anyone haappen to know why this happens?

That could explain why it's slow.

I'm getting the same thing... extremely slow, and very high memory usage.

I just tried running npm-check for the first time, noticed my computer was running super slow, and stopped it via Ctrl+C. I then re-tried it a few minutes later with npm-check --skip-unused, noticed slow performance, and opened activity monitor... to see over "50GB" of memory in use.

I have several large files in the folder I'm running npm-check. A dependency depcheck appears to load into memory any file it comes across. This behavior seems to be triggered by using any specials parsers. By default depcheck uses all available specials.
This is combined with the fact that --skip-unused doesn't stop the check from being run, it just doesn't report it. There needs to be a return after this line.

@AaronAsAChimp good catch!
On a side note, I ended up using yarn upgrade-interactive, it's impressive.

same here on linux, unusable!


field_display not seem to work properly

From what I understand by reading the documentation, setting field_display to false should make invisible the field and don't occupy the space as before, instead it seems to work exactly as field_visible.

Hi, I checked the issue, it works as expected. Please show me the steps to reproduce the issue.

Example code:
#include <nana/gui/wvl.hpp>
#include <nana/gui/widgets/button.hpp>
#include <nana/gui/place.hpp>
int main()
{
	using namespace nana;
	form    fm;
	place pl{ fm };
	button btn1{ fm, "Button 1" };
	button btn2{ fm, "Button 2" };
	button btn3{ fm, "Button 3" };
	button removebtn{ fm, "Remove Button 1" };
	pl.div("vert <<btn1><btn2><btn3>><<><button><>>");
	pl["btn1"] << btn1;
	pl["btn2"] << btn2;
	pl["btn3"] << btn3;
	pl["button"] << removebtn;
	pl.collocate();
	removebtn.events().click([&pl] {
		pl.field_display("btn1", false);
	});
	fm.show();
	exec();
}

Start screenshot

After "Remove Button 1" click

What I expected

    removebtn.events().click([&pl] {
        pl.field_display("btn1", false);   //the change of layout will take effect after collocate()
        pl.collocate(); //so don't forget this.
    });

Ah ok, thanks for the explanation!