Running CTest rebuilds other build configuration in CMake multi generator configuration

Brief Issue Summary
CMake Tools 1.12.27
Visual Studio Code 1.72.2
I have created a project with the following CMakePresets.json:
{
    "version": 6,
    "cmakeMinimumRequired": {
        "major": 3,
        "minor": 25,
        "patch": 1
    },
    "configurePresets": [
        {
            "name": "v140_x64",
            "displayName": "Visual Studio 2015",
            "generator": "Visual Studio 14 2015",
            "architecture": "x64",
            "environment": {
                "CMAKE_CONFIGURATION_TYPES": "Debug;Release"
            }
        }
    ],
    "buildPresets": [
        {
            "name": "v140_x64_debug",
            "displayName": "Build Debug",
            "configurePreset": "v140_x64",
            "configuration": "Debug"
            },
        {
            "name": "v140_x64_release",
            "displayName": "Build Release",
            "configurePreset": "v140_x64",
            "configuration": "Release"
        }
    ],
    "testPresets": [
        {
            "name": "v140_x64_debug",
            "displayName": "Test Debug",
            "configurePreset": "v140_x64",
            "configuration": "Debug"
        },
        {
            "name": "v140_x64_release",
            "displayName": "Test Release",
            "configurePreset": "v140_x64",
            "configuration": "Release"
        }
    ]
}

If I now select the Build Debug as the active build preset and the Test Release as the active test preset, it will build the Debug configuration and run the CTest tests with the Release configuration. This is confusing behavior. Removing the configuration from the test preset will not work (assuming it takes over the configuration of the build preset), as it produces an error (Test not available without configuration).
CMake Tools Diagnostics
No response
Debug Log
No response
Additional Information
No response

I believe that this is a deficiency with the spec for test presets.  test presets are dependent on a configurePreset, not a buildPreset, so it wasn't considered when we implemented this.  Visual Studio appears to have the same problem.
It seems reasonable to me that we could filter out the test presets that don't apply to the active build preset when offering the list of presets to select since the build type is known.  We could probably also make the case where the build type is unknown work as well (assuming that it works on the command line... need to check).

@bobbrow
Could it be possible to not to build at all when running CTests? There are cases when existing binary is not necessary for test case, only CMake configure.

@MartinsSmirnovs can you open a new issue to request that?  There may not be a way for us to know beforehand that this is possible which means this might need to be controlled by a setting or by adding a new command.

Created feature request #2933

Created feature request #2933

Thank you.


Option to have only one accordion panel open at a time

As suggested by reddit user goodwid here:
https://www.reddit.com/r/webdev/comments/kcfv97/my_dad_and_i_teamed_up_to_make_a_pretty_nifty/gfrvb9v/

I personally do not like this feature as it creates UX issues.
The page can scroll up or down depending on which panel opens (or closes) and that confuses some users.
Also, the ability to open all accordion panels allows users to search content within the widget which they could not be able to do easily if only one panel could be opened at a time.

Hm, okay we should think about it a bit more.

Also, the ability to open all accordion panels allows users to search content within the widget which they could not be able to do easily if only one panel could be opened at a time.

tabpanel mode...


fix(test): test

:tada: This PR is included in version 1.0.3-rc.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Does the framework support playback of 4K 3D Videos out of the box?

When loading a 4096x4096 top-down 3D video, I can hear the sound, but the screen remains black.
When I scaled down the same video to 2880x1440 it played fine (but was a bit stretched, for obvious reasons :P)
Is this due a hardware limitation of the phone (Galaxy S6), an encoding issue?
Is there a chart or some recommendations somewhere for video compatibility?
Some specs of the video:
Codec: H264
Resolution: 4096x4096
Bitrate: 120mbps
Framerate: 30fps
Audio Bitrate 300kbps

@TimecodePete Does the video play directly with the MediaPlayer? Any exceptions in the logs?

@TimecodePete  Have you tried other resolutions? Or 2048x2048, or 2048x4096
or reduce the bitrate/framerate

@TimecodePete Please re-open if you have information you could share with us.


Add linear, record and replay rate controllers

This PR adds three additional rate controller to Caliper:

Linear rate controller for gradually increasing the TPS rate
Record rate controller for recording the timings of a rate controller
Replay rate controller for replaying an arbitrary timing trace

For details, see the documentation.
Another minor change: the clients now receive the round index in the message, which is propagated to the controllers (and soon also for the callbacks, but that's an other PR).
Signed-off-by: Attila Klenik<EMAIL_ADDRESS>

@aklenik I like the idea and I've tested the linear rate controller with 25(startingTps), 150(finishingTps) and 30(txDuration). The actual tps increased very slowly at the beginning and very quick at the end. The average sending rate was 51 tps which I thought should be nearly 90tps ((25+150)/2) before testing. Does it  match your design??
[Transaction Info] - Submitted: 15 Succ: 0 Fail:0 Unfinished:15
[Transaction Info] - Submitted: 40 Succ: 25 Fail:0 Unfinished:15
[Transaction Info] - Submitted: 65 Succ: 52 Fail:0 Unfinished:13
[Transaction Info] - Submitted: 95 Succ: 79 Fail:0 Unfinished:16
[Transaction Info] - Submitted: 120 Succ: 105 Fail:0 Unfinished:15
[Transaction Info] - Submitted: 150 Succ: 135 Fail:0 Unfinished:15
[Transaction Info] - Submitted: 179 Succ: 165 Fail:0 Unfinished:14
[Transaction Info] - Submitted: 209 Succ: 195 Fail:0 Unfinished:14
[Transaction Info] - Submitted: 240 Succ: 228 Fail:0 Unfinished:12
[Transaction Info] - Submitted: 270 Succ: 260 Fail:0 Unfinished:10
[Transaction Info] - Submitted: 305 Succ: 295 Fail:0 Unfinished:10
[Transaction Info] - Submitted: 340 Succ: 330 Fail:0 Unfinished:10
[Transaction Info] - Submitted: 375 Succ: 369 Fail:0 Unfinished:6
[Transaction Info] - Submitted: 415 Succ: 406 Fail:0 Unfinished:9
[Transaction Info] - Submitted: 455 Succ: 446 Fail:0 Unfinished:9
[Transaction Info] - Submitted: 495 Succ: 489 Fail:0 Unfinished:6
[Transaction Info] - Submitted: 540 Succ: 531 Fail:0 Unfinished:9
[Transaction Info] - Submitted: 585 Succ: 577 Fail:0 Unfinished:8
[Transaction Info] - Submitted: 632 Succ: 625 Fail:0 Unfinished:7
[Transaction Info] - Submitted: 685 Succ: 646 Fail:0 Unfinished:39
[Transaction Info] - Submitted: 735 Succ: 700 Fail:0 Unfinished:35
[Transaction Info] - Submitted: 795 Succ: 744 Fail:0 Unfinished:51
[Transaction Info] - Submitted: 855 Succ: 790 Fail:0 Unfinished:65
[Transaction Info] - Submitted: 920 Succ: 854 Fail:0 Unfinished:66
[Transaction Info] - Submitted: 991 Succ: 920 Fail:0 Unfinished:71
[Transaction Info] - Submitted: 1070 Succ: 992 Fail:0 Unfinished:78
[Transaction Info] - Submitted: 1153 Succ: 1069 Fail:0 Unfinished:84
[Transaction Info] - Submitted: 1246 Succ: 1154 Fail:0 Unfinished:92
[Transaction Info] - Submitted: 1350 Succ: 1247 Fail:0 Unfinished:103
[Transaction Info] - Submitted: 1465 Succ: 1350 Fail:0 Unfinished:115
[Transaction Info] - Submitted: 1517 Succ: 1468 Fail:0 Unfinished:49

BTW, it would be better if we can define tps schema across test rounds. For example increasing tps for each test round and stop testing when meeting some termination condition (such as finding too many failures). By doing that, performance reports for each test round can be generated. But if we want to support this, we need to support defining dynamic test rounds :)

@haojun The linear rate controller is memoryless, so it only uses the current time/Tx index to determine the current sleep time (unlike the fixed rate controller, that kind of precalculates the submit times).
So if the client doesn't reach the rate control part "often enough", it won't generate as much transaction as needed (unlike the fixed rate controller, that can "catch up" in a bursty way).
Can you increase the number of clients? It should yield a more accurate result that way.


Issue appear when i use the custom search with json

Hello Guys ,
I had this issue for 2 days and i still can't figure out the issue , if anyone had any idea it will great
the issue is this
{"error":"SearchPhaseExecutionException[Failed to execute phase [query_fetch], all shards failed; shardFailures {[ez2RjvQjSGK423_AfwIYaw][news_20150224174544][0]: SearchParseException[[news_20150224174544][0]: from[0],size[50]: Parse Failure [Failed to parse source [{\"size\":50,\"from\":0,\"query\":{\"filtered\":{\"filter\":{\"nested\":{\"filter\":{\"bool\":{\"should\":[{\"term\":{\"author.id\":\"mlDelhwO8IfeGFqdXqFNWTgwLKcSwiNo\"}},{\"term\":{\"tags.id\":null}},{\"range\":{\"published_at\":{\"gte\":null,\"lte\":\"2015-02-24 19:52:11\"}}}]}}}}}}}]]]; nested: QueryParsingException[[news_20150224174544] No field specified for term filter]; }]","status":400}

the query is like this
$contents = News::search(null, ['json' =>
            ['query' =>
                ['filtered' =>
                    ['filter' =>
                        ['nested' =>
                            ['filter' =>
                                ['bool' =>
                                    ['should' => [
                                        ['term' => [
                                            'author.id' => (Input::has('author')) ? Input::get('author') : null]
                                        ],
                                        ['term' => [
                                            'tags.id' => (Input::has('tag')) ? Input::get('tag') : null]
                                        ],
                                        ['range' => [
                                            'published_at' => [
                                                "gte" => (Input::has('db')) ? Input::get('db') : null,
                                                "lte" => (Input::has('de')) ? Input::get('de') : Carbon::now()->toDateTimeString()]
                                        ]]
                                    ]]
                                ]
                            ]
                        ]
                    ]
                ]
            ]
        ]);

What's coming from inputs? dd() the array after 'json' =>.

Input::has('db') and Input::has('de') has datetime string @halilim

Was it fixed? What was the cause?

I couldn't figure out the issue so i did that using laravel @halilim


Fix distance parameters (introduced defaults)

Short Description:
Fix distance parameters (introduced defaults)

Yes, to save from issue reporting wave.

👍


Image Transformations Via Workers fit and Format have swapped Headers

Proposed changes
The documentation on "fit" and "format" have swapped content/headers.
The .mdx file only shows the Render component, so not sure how to go about making a PR.
Subject Matter
Fit & Format
Content Location
https://developers.cloudflare.com/images/transform-images/transform-via-workers/#fit
Additional information
https://github.com/cloudflare/cloudflare-docs/blob/production/src/content/docs/images/transform-images/transform-via-workers.mdx#fit

The regression was introduced in https://github.com/cloudflare/cloudflare-docs/pull/16233, the partials of fit and format should be swapped. Also the "fit" partial is missing the details on the various options that were in the docs previously.

Thanks so much for filing this @theTyster and for the additional info @silto. Much appreciated!


Remove unneeded periods from list

Why:
Remove periods from list because list items are not complete sentences
Closes: #33638
What's being changed (if available, include any code snippets, screenshots, or gifs):

Remove periods on all list items under line 41

Check off the following:

[x] I have reviewed my changes in staging, available via the View deployment link in this PR's timeline (this link will be available after opening the PR).
[x] For content changes, I have completed the self-review checklist.

@SeanAverS Thanks so much for opening a PR with this fix! I'll update the branch and get this merged once tests are passing 🍏
If you're looking for another issue to pick up, take a look at our help wanted section to find more open issues you can work on ✨


determine whether foundation and priority are really needed

I think at least one of them can be eliminated, reducing the SDP size. @emcho, oh master of all things ICE, please help? :-)

foundation can be set to 0 -- done in a89113d0109bbd74501b32ace79b260fc96c2e99
priority could be simply increasing.

priority starting from 0 in 6704dc7888519d92e7e131fb2058b7166be3d8ed
amazingly this works instead of being rejected by chrome -- @juberti: haha :-)


poor performance attached to unreal editor by default

By default, Unreal Editor runs with this option enabled

This option, when the editor doesn't have focus, causes the editor to nearly stop updating. This will occur any time your NetImgui server app has focus. As a consequence, the frame events that UnrealNetImgui gets drawn to drop to near nothing, making interaction with the UI within the server app unusable.
Some possible fixes
Conditionally disable this option when NetImgui::IsConnected, like so
UEditorPerformanceSettings* Settings = GetMutableDefault<UEditorPerformanceSettings>();
Settings->bThrottleCPUWhenNotForeground = false;

The downside of this option is that it would basically keep this option disabled constantly, as just having the server running is almost always going to be connected. Also turning of this option keeps the editor eating a lot of CPU, as it attempts to run normally, causing your GPU/cooling to kick on as it normally does under load.
Another option potentially, rather than draw via FCoreDelegates::OnEndFrame, consider use a standalone timer callback, to fully decouple the 'framerate' of the editor, with the updaterate to netimgui, so NetImgui doesn't entirely circumvent the savings that are part of the bThrottleCPUWhenNotForeground option, and doesn't need to hijack the setting.
Perhaps the more notable bug in this situation is that the interaction with the widgets in the server window(clicks, etc) is so unreliable under this throttled situation?

Thank you. I have been meaning to explore accessing the CPUThrottle option in my next release, but the idea of not relying on FrameEnd might have some value in editor

I think there is something else going on, even with a timer based tick for editor, the click responsiveness in the server app is pretty bad.

Is there something inherent to this system that breaks down when decoupled from the framerate?

Seems like an issue with the server app itself. Is it capturing/caching the mouse click events in between updates so that it doesn't miss them? I have to long click to reliably trigger stuff. Quick clicking doesn't do anything much of the time.

I have confirmed that the timer approach calls Update at the expected rate. Tried up to 60 hz, but still the server interactivity is poor. Also confirmed that the OnEndFrame, drops to 3 fps when the editor doesn't have focus
OnEndFrame

Timer

I don't get why the interactivity is still flaky on a 60hz timer tick.

It should not be the NetImguiServer side, since it process content up to the framerate specified in its config (30fps by default) and this only arise when CPUThrottle is on. Maybe the communication thread gets throttled too somehow?
See if EndFrame() in 'NetImgui_Api.cpp' gets called at 30/60fps (should be with your timer callback change) and then check in CommunicationsHost() in 'NetImgui_Client.cpp' which runs in a separate thread to exchange with the Server.

Looks like EndFrame starvation is still occuring

In/Out is the Communications_Incoming and Communications_Outgoing

The top sections are the EndFrame counts when the app has focus, the ones below are when it doesn't have focus

Looks like the issue is related to NETIMGUI_FRAMESKIP_ENABLED, passing 0 to NetImgui::NewFrame seems to fix it

This is InputData is received from the NetImguiServer, not Unreal. When there's no pending input, it skip drawing content in the editor/game to preserve CPU since it will be discarded. With frameskip disabled, it makes sures there's always a valid imgui context to draw into, but the content is discared, never sent to the server.
Check that Communications_Incoming_Input() is called every ~33ms (or whatever fps you set on the server). This is called on the communication thread, when receiving data from the server. The server never change its speed, after receiving a draw command from the game, it waits until 33ms has passed to send a new input back to the game, triggering a new draw. If we are not receiving it at that rate, the only thing left I can think of at the moment, is that the epic TCP/IP com library also gets slowed down when CPU throttle is active?

Incoming_Input rate is steady

The hasNewInput in ProcessInputData is what drops

I'm not sure why the Incoming_Input actually doubles in rate when the editor doesn't have focus while the HasNewInput drops to 3-4, but I guess that shows the network thread isn't being starved on the unreal side.
Based on my understanding, this means the input messages double in rate when the server window has focus, but it makes no sense as to why hasNewInput would drop, bc it looks like those are tied together. Incoming_Input sets client.mPendingInputIn every call, and that's the basis for hasNewInput, so as far as I can tell, those numbers should match.
Only thing possible it looks like to me would be if a bunch of input events are coming in and just stomping each other before one can be processed. Input isn't processed as a queue, so if multiple come in they would just stomp each other.
Confirmed, lots of input stomping

I'm currently setuping myself to test this, but the question is why ProcessInputData is still called at a slower rate if it doesn't have time to process input data.

The ProcessInputData call rate stays steady at 60
Only thing I can figure is that the network buffer is filling up with a bunch of inputs, and since data is pulled out of the socket until there is nothing left in Communications_Incoming, we get a ton of input message stompage in between frames of ProcessInputData. Any reason not to push these into a queue instead?

Well, ProcessInputData is gated by FNetImguiModule::Update calling NewFrame, called by the 60 hz timer in this case(even slower without the timer when not focused), while the comm thread runs freely. That inherently creates a race condition unless there is some sort of network level message exchange that keeps the systems in some sort of lockstep.

Look how much faster the comm thread runs than the game thread

Yes, it seems like I will have to improve the input reception to buffer them until processed.
I'm currently looking at the slow frame rate.

It seems to be a queue is the right approach, in principal. Otherwise any sort of stomping or input dropping will cause unreliable input in the server app. However, the big worry with queuing is what if the app can't keep up with the input rate?
Clearly we're getting more input right now through the comms than it can process in the main loop, so it's possible the EnQueue count will outpace the Dequeue count, unless you while(Dequeue) each frame in the client side to clear the buffer each frame. That sounds proper to me, as you need to process each one through the full set of ImGui draw pipeline to pick up on all the UI state changes each individual input causes, but you don't want to send a render frame back to the server for all of them, just the last one

I believe I found the issue for the low responsiveness with the Timer method. When unfocused, the callback timing is not steady at all. Overall seems to match the requested speed, but when inspecting the Elapsed time between call, I see about 10 quick call, then a long pause (timing in ms):
| [0] | 0.000600000028 | float
  | [1] | 0.000500000024 | float
  | [2] | 0.000600000028 | float
  | [3] | 332.980194 | float
  | [4] | 0.188800007 | float
  | [5] | 0.000799999980 | float
  | [6] | 0.000300000014 | float
  | [7] | 0.000199999995 | float
  | [8] | 9.99999975e-05 | float
  | [9] | 0.000199999995 | float
  | [10] | 0.000199999995 | float
  | [11] | 0.000199999995 | float
  | [12] | 0.000199999995 | float
  | [13] | 333.130402 | float
  | [14] | 0.564000010 | float
  | [15] | 0.00200000009 | float

So, it appears something else should be used if available, or just disabling the CPU optim whiled connected. There's still the problem of better input enqueuing, but it is not the main issue observed here.

I suppose that's coming from UEditorEngine::ShouldThrottleCPUUsage
I had initially only seen the first reference to bThrottleCPUWhenNotForeground which pauses the blueprint
Looks like they have a hook to get around that, see ShouldDisableCPUThrottlingDelegates
This can check if Netimgui is connected.

That probably gets rid of the need to use the timer also?

Yes, since the timer won't work. I was pulling my hair trying to understand how I could have 30fps in the Update, 60fps in the reception of input and still have a bunch of null pending input :)

This simple code addition (executed after the engine has been properly init) does the trick. I will include it in the next release. Thank you for the pointers.
#if WITH_EDITOR
	GEditor->ShouldDisableCPUThrottlingDelegates.Add(
		UEditorEngine::FShouldDisableCPUThrottling::CreateLambda([](){ return NetImgui::IsConnected();})
	);
#endif

Where are you calling that from? In my project the GEditor isn't initialized yet(module load order)
For the sake of avoiding these issues, you might want to set that GEditor delegate inside of a FCoreDelegates::OnPostEngineInit
#if WITH_EDITOR
	FCoreDelegates::OnPostEngineInit.AddLambda([]
	{
		GEditor->ShouldDisableCPUThrottlingDelegates.Add(
			UEditorEngine::FShouldDisableCPUThrottling::CreateLambda([](){ return NetImgui::IsConnected();}));
	});
#endif

Yes, I moved some plugin init that included this new throttle callback, to a new method attached to the PostEngineInit delegate.


Add compatibility with TooManyEmotes

Right now, when using TooManyEmotes, any selected MoreCompany cosmetics will be shown floating above the dog model while emoting in third person (and maybe in other scenarios too).
To fix the above issue (at minimum), should write a postfix patch for TooManyEmotes to hide more company cosmetics when the player is a dog.

Fixed in v2.0.7


Fix #371

Done by shuffling around flag.Parse so it's done after kingpin is done but before the new client is created. Removing the init() function was necessary.
Signed-off-by: David Muckle<EMAIL_ADDRESS>

Sorry about that, had to recreate the PR to sign off.

Thanks for your PR. Here are some tips for writing a goo PR and a good commit.
The title of the commit, and the PR should describe what will happen to the codebase if this change is applied. "Fixes #371" is ok, but "move flag.Parse out of init" would be better.
The body of the PR should include the words "Fixes #371" at the start, on their own. These words are special and github will close the associated issue when the PR is merged.
The rest of the body should describe how and or why this PR should be merged. The why is simple, it's described in #371. The how could be optional in some cases, but in this case it should probably be something like

glog requires flag.Parse to be called before anything that uses glog attempts to log. In this case the glog callers are the kubernetes client, so we move he call to flag.Parse just before calling k8s.NewClient.

Sorry, amended my last commit with a correction to the comment and the commit message.

and the PR title and description :)

Ah, right, updated those too.

Pretty close. Thanks, i'll take it from here.


Light Posts (3x) are missing a power connection

Light Post 1, Light Post 2, and Light Post 3, under the 'Lights' header are all missing a power connection. Would be nice to have this added to not have to connect power in-game to then re-import into Blender again.

fixed in 1.1.9


CTC Token Passing

Hi!
I'm trying to use the Token Passing algorithm for decoding a model trained in IAM-DB. I'm using a language model built with the LOB corpus, however, there are situations in which the word that is passed to wordToLabelSeq method presents a character that is not mapped to any class, eg.: '>'. What do you advise to do in these situations?
Thanks in advance,
Dayvid.

Hi,
I use token passing only on word-level as it does not work very well when adding "words" (or entities) representing only punctuation marks such as "<" and others.
I only take real words from the corpus by using a regex expression like "\w+".
Those words are then fed into the token passing algorithm and the most probable word sequence is returned. Finally, I add whitespaces in between the words to get the text.
Coming back to your question: if your neural network does not output a character, then it also can't predict a word containing this character. Therefore, I would such words.
How about the running time of your implementation? Even with a C++ implementation I was only able to use small dictionaries because the algorithm is very slow (time depends quadratically on dictionary size).
Beam search is more flexible in my opinion: it is fast, depending on the beam width, and it can integrate a character-level language model. And of course each beam could also be checked for dictionary words, optionally giving a zero score to a beam with unknown words in it.


Revert "dbus: Switch default implementation to broker"

Description of changes
This reverts commit e827697fd3435b458a759ef0b92de7f26624b910.
We need a more elegant way to handle this as it currently breaks upgrades on some systems.
Ref:

Fixes #302771
#299812

Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Can we restrict it to a future release-24.05 branch? I'm not sure if it's worth going back and forth on dbus for unstable users, if the ultimate goal is to use dbus-broker.

Dynamic users being halfway broken, as reported in https://github.com/NixOS/nixpkgs/issues/297756 is pretty bad.
The original PR didn't notify me or any other systemd maintainer either.
I consider this a blocker for the release.
Reverting is cheap, especially if we know this breaks users on unstable currently.

Closing in favor of https://github.com/NixOS/nixpkgs/pull/307731.


Feature request: Option to include prefix & suffix spaces in android

In android resources whitespaces (prefix & suffix) are usually discarded by the xml parser.
see  https://developer.android.com/guide/topics/resources/string-resource.html#formatting-strings
There is however no flag to instruct the cli whenever to qoute these strings (or escape the starting & ending spaces).
This makes it very hard in large projects to use the cli as all strings must be inspected after pulling to ensure whitespace are preserved (if one uses it)

Thank you for the feedback and for bringing this to our attention!
I've checked the Android documentation you've attached and agree that there is room for improvement. I will forward your feedback to our product team.
In some scenarios, prefixes and suffixes may be necessary. However, for positioning alone, using margins or paddings could be more effective, especially when the spacing around the text is consistent across languages. Additionally, if including whitespace around the text is needed, whitespace Unicode characters could be a viable alternative to quotes until we provide a better solution.
Alternative:
<resources>
    <string name="example">\u0020\u0020\u0020Some message\u0020\u0020\u0020</string>
</resources>


Emit a list of term links

In GitLab by @jakebeal on Feb 1, 2019, 21:13
One of the results desired from this system is to be able to emit a list of terms that are expected to be used by an experiment request.
This will be JSON document formatted following a schema shortly TBD in the cp-request repository (https://gitlab.sd2e.org/sd2program/cp-request), and will consist of a union of all of the links to a known definition source (e.g., SynBioHub) and maybe their associated common terms?
This will probably want to be triggered by a menu selection or button.

In GitLab by @jakebeal on Mar 12, 2019, 10:56
assigned to @sumorok

In GitLab by @jakebeal on Mar 26, 2019, 10:11
https://gitlab.sd2e.org/sd2program/cp-request/blob/master/cp-identifiers-schema.json

In GitLab by @jakebeal on Mar 26, 2019, 10:18
First pass: give me a JSON to download.
Later: allow pipeline tools to directly pull from server, maybe?

In GitLab by @jakebeal on May 7, 2019, 11:08
Almost done: needs to report the set of links with the designated SynBioHub stem.

In GitLab by @sumorok on May 8, 2019, 15:37
This is now done

In GitLab by @sumorok on May 8, 2019, 15:37
closed


Feat/adm 633

Summary
...
Before
Description
Screenshots
If applicable, add screenshots to help explain behavior of your code.
After
Description
Screenshots
If applicable, add screenshots to help explain behavior of your code.
Note
Null

replace date range component for Metrics page


Test PR

Pull Request
Description
This PR created for test

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 90.65%. Comparing base (d04c45a) to head (65b8b7f).

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #448   +/-   ##
=========================================
  Coverage     90.65%   90.65%
  Complexity      488      488
=========================================
  Files            47       47
  Lines          1744     1744
  Branches        208      208
=========================================
  Hits           1581     1581
  Misses           99       99
  Partials         64       64

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Feature allow specify target git repo name for fixing git commit links

See old PR #19

Isolated the changes related to this feature. Hope the merge works better now.


Use dotnet local tools

This makes use of the new local tools that dotnet core 3.0 exposed.
paket and fake are now local tools to the repo, replacing the full framework version of paket and the global version of fake-cli.
This builds on top of #71

@demystifyfp I'd definitely for you to be aware of these changes before I merge them.


Loading Docker client from environment variables?

It would be nice if there was a method such as NewDockerClientEnv which automatically loaded the correct thing based on the environment variables such as DOCKER_HOST, DOCKER_CERT_PATH, etc.  The code would be something like:
func NewDockerClientEnv() (*DockerClient, error) {
        tlsc := &tls.Config{}
        dockerCertPath := os.Getenv("DOCKER_CERT_PATH")

        cert, err := tls.LoadX509KeyPair(filepath.Join(dockerCertPath, "cert.pem"), filepath.Join(dockerCertPath, "key.pem"))
        if err != nil {
                return nil, fmt.Errorf("Error loading x509 key pair: %s", err)
        }

        tlsc.Certificates = append(tlsc.Certificates, cert)
        tlsc.InsecureSkipVerify = true

        dc, err := dockerclient.NewDockerClient(os.Getenv("DOCKER_HOST"), tlsc)
        if err != nil {
                return nil, fmt.Errorf("Error getting Docker Client: %s", err)
        }

        return dc, nil
}

I could submit a PR if there is interest.  Interested?

I submitted #201 which does a part of this. For an example usage check out https://github.com/drone/drone-exec/pull/13
I can flesh out that PR further by adding a NewDockerClientEnv, but to match the same functionality as the official docker client is a bit more work.


Block/Mute a User

A Block/Mute user option would be nice.

So if a User is in the "Blocked users" list, his messages (messages, where he is a sender) are not even stored in the local database.

Sounds good.

I also want that if you block a user, their messages will be disappear from the timeline. For example: I have blocked a user who sent an offensive message (after the message was sent), but the message is still visible in AndStatus (not in the web UI, though).
Or should I create a separate issue for this?

@msjoberg I think we better have this as one issue.

From the conversation https://loadaverage.org/notice/8102615

As we found out recently, muting a User in a Web interface (of GNU Social) doesn't mean that the User is muted for other client applications, e.g. for AndStatus. Unfortunately. #poordesign
This means that AndStatus has to reimplement the same feature again.


Chokidar matches .js files in symlinks

Nativescript 4.1.0
nativescript-dev-sass 1.6.0
After updating from 1.3.5, my application is running node-sass dozens of times when calling tns run (platform/arguments doesn't matter). Upon investigating, I found that the promise within watcher.js would back up due to this, and the application would refresh several times for no reason once the build was complete until it caught up.
Upon further investigation, I found that the chokidar watcher is firing for every .js (its a typescript/angular project) file I have in a symlinked set of folders. I cannot explain why this is happening, but there is an issue open with chokidar:
https://github.com/paulmillr/chokidar/issues/729
While I'd prefer a solution where people could have css files in their symlinked files, I think the current behavior is worse than not allowing that. In watcher.js, adding this line to the watcherOptions:
followSymlinks: false,

solves the problem (I have no css in my symlinked files). I was going to create a .PR, but saw that there should be a corresponding issue first, and the other symlink issue was a bit different.
I can also investigate older versions of chokidar or other suggestions, the above was just the simplest to get my app back on track.
Thanks,
James

Hi  @james-criscuolo , I am sorry to hear that the problem is not resolved on your side. Unfortunately, we cannot reproduce it, so it’s hard to investigate. I have created Angular project, symlinked some plugin directories and synced files without causing the described behavior. It would be great if you can provide us with more steps or sample project to reproduce it. I suggest you log another issue and provide the details there.


ImmutableMappable example needed.

I'm testing the immutable mapping and I can't get any custom immutable objects to map. For example, if I have and object (that's Mappable) with this property defined: public var foos: [Foo]? its not mapping any Foo objects.
    public func mapping(map: Map) {

        name <- map["name"]
        foos <- map["foos"]
    }

public class Foo {

    public let name: String

    public init(map: Map) throws {
        name = try map.value("name")
    }
}

The name variable is populated as expected but I get nothing in my foo variable. Perhaps I need iterate over map["foos"] and create the objects myself?

Possibly duplicated with #668.


chore: Update yarn.lock

To resolve workspace being dirty failure in CI checks.

replaced by https://github.com/janus-idp/backstage-plugins/pull/1516 -- semantic commit messages must be used.


Unable to stream the video from my webcam to Kinesis Video Stream (on mac)

Hi,
I need to stream the video from my webcam (on mac) continuously to Kinesis Video Stream.
I have tried running the commands mentioned in this documentation.
I'm getting the following error,
ERROR: from element /GstPipeline:pipeline0/GstAVFVideoSrc:avfvideosrc0: Internal data stream error.
Additional debug info:
../libs/gst/base/gstbasesrc.c(3127): gst_base_src_loop (): /GstPipeline:pipeline0/GstAVFVideoSrc:avfvideosrc0:
streaming stopped, reason not-negotiated (-4)
[INFO ] [31-05-2022 11:19:40:359.064 UTC] stopKinesisVideoStreamSync(): Synchronously stopping Kinesis Video Stream 00007fe725037010.

ERROR: from element /GstPipeline:pipeline0/GstAutoVideoSrc:autovideosrc0/GstAVFVideoSrc:autovideosrc0-actual-src-avfvide: Internal data stream error.
Additional debug info:
../libs/gst/base/gstbasesrc.c(3127): gst_base_src_loop (): /GstPipeline:pipeline0/GstAutoVideoSrc:autovideosrc0/GstAVFVideoSrc:autovideosrc0-actual-src-avfvide:
streaming stopped, reason not-negotiated (-4)
[INFO ] [31-05-2022 11:24:29:732.792 UTC] stopKinesisVideoStreamSync(): Synchronously stopping Kinesis Video Stream 00007fd184089c10.

I have verified if my webcam is working fine and i'm able to use it for any video calls or meetings.
I have also tried the following command,
gst-device-monitor-1.0
and i see the following error,
Probing devices...

Failed to start device monitor!

I have tried connecting external webcam and I still see the same errors.
I have done the JNI setup as described in the documentation and I'm able to stream existing sample videos but unable to stream from my webcam.
Kindly help me with this, thank you in advance.

The error seems to be coming from your pipeline. Try out other sinks apart from kvssink to ensure your pipeline works.


⚠ 2 Minutes de génie (staging) has degraded performance

In 361504d, 2 Minutes de génie (staging) (https://2mdgstaging.gitlab.io/) experienced degraded performance:

HTTP code: 200
Response time: 2878 ms

Resolved: 2 Minutes de génie (staging) performance has improved in 845e3d0.


Add text "My changes" to the article

[x] I have read freeCodeCamp's contribution guidelines.
[ ] My pull request has a descriptive title (not a vague title like Update index.md)
[ ] My pull request targets the master branch of freeCodeCamp.
[ ] None of my changes are plagiarized from another source without proper attribution.
[ ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

INVALID


catch more YAML errors

Catch a more general type of YAMLExceptions when tryibg to parse TaupageConfig

:+1:

👍

Codecov Report

Merging #495 into master will not change coverage.
The diff coverage is 0%.

@@            Coverage Diff            @@
##             master     #495   +/-   ##
=========================================
  Coverage     53.47%   53.47%
  Complexity      479      479
=========================================
  Files           131      131
  Lines          2947     2947
  Branches        203      203
=========================================
  Hits           1576     1576
  Misses         1292     1292
  Partials         79       79

Impacted Files
Coverage Δ
Complexity Δ

...ullstop/jobs/common/impl/FetchTaupageYamlImpl.java
71.42% <0%> (ø)
2 <0> (ø)
:arrow_down:

.../plugin/provider/impl/TaupageYamlProviderImpl.java
89.65% <0%> (ø)
5 <0> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


map rare dust drop rate is broken

since we don't record when map doesn't drop anything, the rare dust drop rate is 100%

We know a thing is opened but since there are no items returned we don't do anything with it.
use_convertible.php is called with item_type=treasure_map_scroll_case_convertible. With no dust I only see this in the items high level object:
  "items": {
    "treasure_map_scroll_case_convertible": {
      "item_id": "921",
      "type": "treasure_map_scroll_case_convertible",
      "class": "convertible",
      "name": "Relic Hunter Scroll Case",
      "description": "This ornate scroll case has a heavy feel to it. Who knows what untold riches the scroll inside will lead to?",
      "quantity": 1,
      "thumbnail": "https:\/\/www.mousehuntgame.com\/images\/items\/convertibles\/638ae2329960ad40fce6b3b7495b919d.gif?cv=246",
      "num_owned": 11
    }
  }

Need an example when someone gets dust.

Potential clues on discord - message_model will have empty items for no dust: https://discordapp.com/channels/275500976662773761/355474934601875457/765240185369264148

We talked about a few options. The "best" would be to submit the convertible even with an empty list of IDs. This would account for anything that can be opened and sometimes drops nothing.
The changes would been to be made in the extension to stop checking for message.messageData.items.length === 0 here on line 487 -- or at least handling that case.
In convertible_intake.php  allowing empty items through and in that case inserting a row into convertible_items with a null item_id.
In the converter page's SQL to do an outer join on the item table.
The event that aggregates the data appears to be fine with a suddenly null value there. The column does need to be altered to be nullable.

We can address this and #62 at the same time!
When a scroll is opened (see above) we have to collect what type of map is opened. This is item 1 as a result of the convertible. It'll need an id so we need to see if one is provided or if we'll have to fudge them. (or fix intake to make up ids if one isn't provided and it's a new result). We can add map dust as item 2 when it exists as a result.

Db cleanup of old dust data is required


Refactoring android ktx

[x]  update dependencies - move to catalog and update the gradle and plugins
[x]  move common logic to buildsrc
[x]  update detector config
[x]  remove deprecates
[x]  add compose-ktx
[x]  describe documentation for compose-ktx

No longer relevant.
MR will be broken into subtasks: refactoring, library removal.


Installation instruction for each module

Motivation
In the following PR, the author mentioned that it is more convenient if we can install botorch directly from pip:

https://github.com/optuna/optuna/issues/5197

However, the documentation does not explain the installation.
Furthermore, the error message does not also convey the installation guide.
Suggestion
That is why I think it is necessary to provide more detailed documentation or better error messages.
Optionally, it is preferrable to be able to install the dependencies via pip, e.g. pip install optuna-integration[botorch].
Additional context (optional)
No response

This  issue was resolved:

https://github.com/optuna/optuna-integration/pull/130

So I will close this issue.


Add support for parsing revert errors from Partiy nodes

Currently revert errors from Parity nodes do not get property decoded into ExecutionError::Revert. Since the full nodes we run at Gnosis are Parity nodes, that is a problem. This PR adds support for that.
Fixed #185
Test Plan
New unit tests

Nice refactoring this logic into its own module. I was wondering if the geth specific decoding logic should be colocated with the error handling but since it also operates on successful results, this is probably the right way to do things.

Good point! I think it should move as well.


Cache generated files to speed up generation

Diagram generation can be slow, depending on the diagrams complexity. Especially with the ELK layout engine.
For example, with a simple markdown documentation:

generation will take 0.61 seconds with no d2 diagrams
generation will take 7 seconds with the following diagram included twice (embedded and included):direction: right
🧔♂ -> 🌍: 👋

(values depending on external factors, but magnitude orders are about the same)
When writing a documentation with mkdocs, the best option is to use mkdocs serve that will rebuild the documentation when the source is changed. In this case, mkdocs-d2-plugin will re-generate all diagrams, even though the source file is not updated.
A cache system, allowing to rebuild diagrams only when the source file has been changed would be great. drawio exporter plugin has such an feature for the same reason (especially since drawio requires chrome to generate images).

generation will take 7 seconds with the following diagram included twice

It's way faster on my MacBook Air M1 - building documentation of this project, with 5 diagrams (one uses ELK) takes 1.61 second - but I agree, it's too slow. The main culprit is actually calling an external process, not generation itself, at least for small to medium diagrams.
I do intent to add a cache.

M1 heh, lucky you :D
I have to work in a linux virtual machine on a clogged Windows host...
I have tested, and you are totally right:
gitpod@6a7754c6b56c$ mkdocs build
INFO    -  Cleaning site directory
INFO    -  Building documentation to directory: /workspaces/docker-mkdocs/public
True
True
INFO    -  Documentation built in 3.61 seconds
gitpod@6a7754c6b56c$ d2 docs/d2.d2
success: successfully compiled docs/d2.d2 to docs/d2.svg in 143.375124ms

On another note, I think you still have a debug leftover with the following lines, looks like a line per diagram:
True
True

:)
Thanks for your great plugin!

I just released version v1.1.0 with caching. Link to documentation
Here you can see two subsequent runs on my computer:
~/Documents/GitHub/mkdocs-d2-plugin/docs master
.venv ❯ mkdocs build
INFO    -  mkdocs-d2-plugin: Using cache at .cache/plugin/d2/db (None)
INFO    -  Cleaning site directory
INFO    -  Building documentation to directory:
           /Users/landmaj/Documents/GitHub/mkdocs-d2-plugin/docs/site
INFO    -  Documentation built in 1.22 seconds

~/Documents/GitHub/mkdocs-d2-plugin/docs master
.venv ❯ mkdocs build
INFO    -  mkdocs-d2-plugin: Using cache at .cache/plugin/d2/db (dbm.ndbm)
INFO    -  Cleaning site directory
INFO    -  Building documentation to directory:
           /Users/landmaj/Documents/GitHub/mkdocs-d2-plugin/docs/site
INFO    -  Documentation built in 0.22 seconds

I confirm the speed up when using mkdocs build:
gitpod@6a7754c6b56c$ mkdocs build
INFO    -  mkdocs-d2-plugin: Using cache at .cache/plugin/d2/db (None)
INFO    -  Cleaning site directory
INFO    -  Building documentation to directory: /workspaces/docker-mkdocs/public
INFO    -  Documentation built in 3.89 seconds
gitpod@6a7754c6b56c$ mkdocs build
INFO    -  mkdocs-d2-plugin: Using cache at .cache/plugin/d2/db (dbm.ndbm)
INFO    -  Cleaning site directory
INFO    -  Building documentation to directory: /workspaces/docker-mkdocs/public
INFO    -  Documentation built in 1.06 seconds

But it has no effect with mkdocs serve, which would be more beneficial:
gitpod@6a7754c6b56c$ mkdocs serve
INFO    -  Building documentation...
INFO    -  mkdocs-d2-plugin: Using cache at .cache/plugin/d2/db (None)
INFO    -  Cleaning site directory
INFO    -  Documentation built in 2.52 seconds
INFO    -  [08:19:53] Watching paths for changes: 'docs', 'mkdocs.yml'
INFO    -  [08:19:53] Serving on http://<IP_ADDRESS>:8000/
INFO    -  [08:20:01] Detected file changes
INFO    -  Building documentation...
INFO    -  mkdocs-d2-plugin: Using cache at .cache/plugin/d2/db (dbm.ndbm)
INFO    -  Documentation built in 2.83 seconds
INFO    -  [08:20:04] Reloading browsers

I'll try to have a look at where the cache is failing.

That's weird. I have modified the python file for the plugin, but no change in behavior. Just some logs. And now it's working fine even with mkdocs serve:
INFO    -  mkdocs-d2-plugin: Using cache at .cache/plugin/d2/db (dbm.ndbm)
INFO    -  mkdocs-d2-plugin: source not found in cache.
INFO    -  Documentation built in 2.37 seconds
INFO    -  [08:37:06] Reloading browsers
INFO    -  [08:37:18] Detected file changes
INFO    -  Building documentation...
INFO    -  mkdocs-d2-plugin: Using cache at .cache/plugin/d2/db (dbm.ndbm)
INFO    -  Documentation built in 0.72 seconds
INFO    -  [08:37:18] Reloading browsers

I can't explain that.
If I ever reproduce the issue, I'll dig into it and find a fix...
Thanks!

I tested mkdocs serve so I would be surprised if it didn't work. If you ever encounter this issue again, please open an issue and I will try to reproduce it.
I still have a few ideas for improvements but nothing major.

Yes, as is the plugin is already great. It's only missing the multilayer IMHO, but it's more on d2 side.
I have started a discussion on their Discord by the way. No answer yet, but if it happens I'll notify you: https://discord.com/channels/1039184639652265985/1039185315685998693/1171364484095426601


Updated Quickstart.

Worked through Quick Start and updated.

hey @grrlgeek !
Thank you for this update !
Were you able to make the project Y working, using this walkthrough ?


Internet Explorer 11 site bootstrap is not woking

Hii Team,
Please check below website in IE11
http://stageconnect.in/website/sbi_survey/
I am using bootswatch bootstrap.css file it is working in chrome and firefox but it is not looking same as in IE11.
What to do?

Issues are for bug reports and feature requests, not general how-to questions. For that, we suggest the official Slack channel or Stack Overflow. Both are linked from our readme.
Thanks!

thank you,,
I will put this question in stackoverflow

hii ,
any one can help me


feat: support casting FixedSizeList with new child type

Which issue does this PR close?
Closes #4426.
Rationale for this change
Makes it possible to cast between FixedSizeList arrays with different child types. We use this to be able to cast f32 vectors into f16 vectors to store them at a smaller size.
What changes are included in this PR?

Are there any user-facing changes?
Yes, though I don't think we document all possible cast option at the moment.

Thanks again @wjones127  and @tustvold

@alamb Sorry for not replying earlier. I did the casts for the other direction earlier in https://github.com/apache/arrow-rs/pull/5081. I didn't think to add a roundtrip test specifically, but I think the individual tests are specific enough to imply they will roundtrip.

@alamb Sorry for not replying earlier. I did the casts for the other direction earlier in #5081. I didn't think to add a roundtrip test specifically, but I think the individual tests are specific enough to imply they will roundtrip.

No worries -- makes sense to me.


Can you explain better what Add_Self_Loops and expand_edge_index do exactly?

Hi,
I'm trying to understand what your code do. Let's say that we have a toy dataset like this:
V;E =
v0, v0, v1, v1, v2, v2, v3
 e1, e2, e2, e1, e2, e3, e3
Which translate into this edge_index:
[[0, 0, 1, 1, 2,  2, 3],
[4, 5, 5, 4, 5, 6, 6]],
import numpy as np
 edge_index = torch.tensor([[0, 0, 1, 1, 2, 2, 3],                            [4, 5, 5, 4, 5, 6, 6]], dtype=torch.long)
 edge_index = coalesce(edge_index)
 ei = np.array(edge_index)
 num_nodes = len(np.unique(ei[0]))
 num_hyperedges = len(np.unique(ei[1]))
 data = Data(edge_index=edge_index, n_x = num_nodes, num_hyperedges=num_hyperedges)
Now, if I use your Add_Self_Loops code I obtain this edge_index:
tensor([[ 0,  0,  0,  1,  1,  1,  2,  2,  2,  3,  3]
 [ 4,  5,  7,  4,  5,  8,  5,  6,  9,  6, 10]])
Which is a little bit confusing. Shoudn't be added the same node? What are you doing in this code exactly, and why?
Same question for expand_edge_index.
tensor([[ 0,  0,  0,  1,  1,  1,  2,  2,  2,  3,  3]
 [ 5,  7,  8,  4,  6,  8,  6,  7, 10,  9, 11]])
What kind of expansion are you implementing? What is the difference between the two implementations? Why are they needed for your models AllDeepSets and AllSetTransformer?

Hi @giuliacassara ,
For Add_self_loop(), we are assigning a new "unique" hyperedge_id for each node acts as the self-loops in the standard graph case. In your example
[[0, 0, 1, 1, 2, 2, 3],
[4, 5, 5, 4, 5, 6, 6]],

since the "real" hyperedge_id is from 4 to 6, we assign node 0,1,2,3 with new hyperedges that link to each of them only, which is hyperedge_id 7,8,9,10 in the example that you pointed out. That is, the hyperedge 7 will only associate with node 0, 8 with 1, and so on. Hence, the hyperedges 7 to 10 are indeed self-loops.
For expand_edge_index(), please check our description in https://github.com/jianhao2016/AllSet/blob/0d0e399a9168829fa898dd56f3d32bee36953b04/src/preprocessing.py#L22
We will only need it if we want to "exclude" the self contribution in the message-passing in hypergraphs. his is for the formulation of the equation (6) in our paper only and it is not used in our presented experiments if I remember correctly.
Please let me know if you have further questions.
Best,
Eli

Hi,
everything clear, thank you!


Make Identity Docs JDK 20 Compatible.

Updates Identity Javadocs to compile against JDK 20 without warnings.

API change check
API changes are not detected in this pull request.


Added support for nullable FPort

This allows empty Lora messages (with neither FPort nor payload) to be correctly handled by the app server. See pull request at https://github.com/brocaar/loraserver/pull/329

I've just noticed I'm not checking for FPort being null in influxdb_handler.go. I'm trying to figure out what the code is doing in order to determine what to set f_port to when pl.FPort is null. Would 0 suffice, or the string "null"?

Please see my comment on https://github.com/brocaar/loraserver/pull/329.


Backports for 0.10.1

Ok, in the full log, it says:
09:50:54 + abort 'Error: unexpected connection between frontend and backend. wanted HTTP 403, got: HTTP 404'
09:50:54 ------------------------------------------------------------------------
09:50:54                           K8s Test Failed
09:50:54 Error: unexpected connection between frontend and backend. wanted HTTP 403, got: HTTP 404
09:50:54
09:50:54 ------------------------------------------------------------------------
09:50:54 + set +x

Ok, in the full log, it says:
09:50:54 + abort 'Error: unexpected connection between frontend and backend. wanted HTTP 403, got: HTTP 404'
09:50:54 ------------------------------------------------------------------------
09:50:54                           K8s Test Failed
09:50:54 Error: unexpected connection between frontend and backend. wanted HTTP 403, got: HTTP 404
09:50:54
09:50:54 ------------------------------------------------------------------------
09:50:54 + set +x

I think this is a false negative and wait_all_k8s_regenerated is returning early. It assumes that by the time it is called, at least one endpoint is already in state regenerating.
I will fix this to use policy revision. I think we can merge this in the meantime.

Ok, thanks for clarifying!


Generate rendered pdf from the docs source

I hjave been adding the asciidoctor-pdf package into the image, but the Ruby version is too old. Use Fedora instead, WDYT?

Fine by me, lets use the fedora image! I guess @robin-owen and @praveenkumar will also agree!

@gbraad which version of ruby it need can we start using https://access.redhat.com/containers/?tab=overview#/registry.access.redhat.com/ubi8/ruby-25 one?

@anjannath @gbraad using below I can easily able to build the image.
FROM registry.redhat.io/ubi8/ruby-25
MAINTAINER CodeReady<EMAIL_ADDRESS>
RUN gem install asciidoctor-pdf --pre

VOLUME /docs
WORKDIR /docs

ENTRYPOINT [ "asciidoctor-pdf" ]

right, so you use an alternative image... the UBI.
let's create a dockerfiel for this ...

This is what the generated doc.pdf looks like.
/cc @code-ready/crc-devel


[MEDIUM] Backwards Compatibility: "Billable no longer valid" appears in Manual but not Scan request when disabled

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

Version Number: 1.4.27-1
Reproducible in staging?: y
Reproducible in production?: n
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:
Precondition:

User is an employee of a Collect workspace that has billable option enabled by default.

Go to workspace chat.
Create a manual request and a scan request, both with billable toggled on.
As admin, disable billable option on Old Dot.
As employee, go to request details page of manual request and scan request.

Expected Result:
"Billable no longer valid" will appear under Billable section in both manual and Scan request.
Actual Result:
"Billable no longer valid" appears in the manual request but not Scan request.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/93399543/8c1bffd2-0e57-404c-b8be-9cb2d46bba33
View all open jobs on GitHub

Not a blocker as its minor and nobody is using the collect in newdot yet

Assigned to another Critical issue in this Wave so going to unassign and add the Hot Pick tag.

Picking this back up as I'm free now again!

I guess this is backwards compatibility issue given the nature of how to reproduce it, but it's not really critical functionality and the error message is only missing in one place (yet present elsewhere). I'll call this one out in the update and see what people think about priority.

Tossing a Hot Pick on this as Yuwen is working on something else right now - if anyone has bandwidth they could jump in instead!
context: https://expensify.slack.com/archives/C02MW39LT9N/p1708453585539349

I got the confirmation from @cead22 on Slack, that all violations are hidden until the SmartScanning is done. This is by design. I tested that the violation related to the Billable field appears when the SmartScanning is done, and it works well.
I'll close the issue.


fix: #1116 fastjson serialization to jackson

fix #1116

Codecov Report

Merging #1132 into develop will increase coverage by 0.03%.
The diff coverage is 64.7%.

@@             Coverage Diff              @@
##             develop   #1132      +/-   ##
============================================
+ Coverage      41.86%   41.9%   +0.03%
- Complexity      1366    1370       +4
============================================
  Files            244     245       +1
  Lines          10165   10181      +16
  Branches        1333    1333
============================================
+ Hits            4256    4266      +10
- Misses          5355    5361       +6
  Partials         554     554

Impacted Files
Coverage Δ
Complexity Δ

...ata/rm/datasource/undo/JSONBasedUndoLogParser.java
100% <ø> (ø)
3 <0> (ø)
:arrow_down:

...o/seata/rm/datasource/sql/struct/TableRecords.java
73.68% <ø> (ø)
13 <0> (ø)
:arrow_down:

...seata/rm/datasource/undo/UndoLogParserFactory.java
80% <100%> (ø)
1 <0> (ø)
:arrow_down:

...seata/rm/datasource/undo/JacksonUndoLogParser.java
62.5% <62.5%> (ø)
4 <4> (?)

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 052b432...995196e. Read the comment docs.


nested json file

hi there,
is it possible to import a json file with nested objects? how do I use them in my css?
BR
Marc

Hey!
Nested JSON objects get imported as Sass Maps that are also nested.

I think the problem was that I am using preCSS instead of real sass and they do not support maps

That sounds like it would cause some problems.
Are there any complex data structures supported by preCSS that would work for arbitrary JSON data?

I've opened issue #10 for tracking a potential rewrite of the module that would help address problems like this.


Finish Email Log Page

Resolves issue #414

Useful for email blast
Phase 1 of the email blast feature; some additional features in the future: integrate sending messages via the website

Ignore the google hangout on the bottom.

rebase the lastest dev commit; it fixes the travis problem

rebase the lastest dev commit; it fixes the travis problem

@realCalvin where we at on this homie

@realCalvin where we at on this homie

pushed changes

rebased 🦖

closing this as #561 is picking up on this. thank you @realCalvin !!!


Add RunNow implementations for Dispatcher/ParSeq

I'm kind of confused, what goal are we trying to achieve by doing this?

Mainly so you can use T: RunNow<'a> to take in any dispatcher/system instead of having to specify ParSeq or Dispatcher directly.

Failing travis should be fixed by #88


Running the command drink with no arg causes a crash.

We removed the line that seemed to cause the crash (It did not seem to do anything) it works now!

this was a regression; will check into this fix soon. The idea of the removed line is to account for fountains -- which would be room level.

Appears this is programming error -- just looking at source. The getWatersource function has an arity of 2 -- it needs to be given the command object. Sorry about that.
Room.getWatersource(roomObj, command);


⚠ Secret Website 1 has degraded performance

In bd5832b, Secret Website 1 ($SECRET_WEBSITE_1) experienced degraded performance:

HTTP code: 200
Response time: 3183 ms

Resolved: Secret Website 1 performance has improved in 5778973.


chore: :lock: Security vulnerability mitigation

PR fixes the below vulnerabilities in System.Private.Uri found by Trivy scan
dotnet: infinite loop in Uri.TryCreate leading to ASP.Net Core Denial of Service
Package: System.Private.Uri
Installed Version: 4.3.0
Vulnerability CVE-2019-0980
Severity: HIGH
Fixed Version: 4.3.2
Link: CVE-2019-0980
dotnet: crash in IPAddress.TryCreate leading to ASP.Net Core Denial of Service
Package: System.Private.Uri
Installed Version: 4.3.0
Vulnerability CVE-2019-0981
Severity: HIGH
Fixed Version: 4.3.2
Link: CVE-2019-0981
dotnet: Domain-spoofing attack in System.Uri
Package: System.Private.Uri
Installed Version: 4.3.0
Vulnerability CVE-2019-0657
Severity: MEDIUM
Fixed Version: 4.3.2
Link: CVE-2019-0657

I spent some time reading this issue and I believe this to be a false positive. This and the other 2 CVEs require System.Private.Uri to be a direct or implicit dependency. However, if you do the following you will not see it:
cd src
dotnet publish -c Release -o publish
asmspy --all publish

This PR introduces a forced dependency on this package which it otherwise would not have. Also many of these dotnet-specific CVE issues are dated.
I'll go ahead and close this issue. If I'm mistaken in some way please let me know. Otherwise, you should feel free to disregard these false positives.


go-flutter: no handler found for channel flutter/isolate

go-flutter: no handler found for channel flutter/isolate

That is because go-flutter doesn't have an implementation for that plugin. You may ask the plugin maintainer to add an implementation or you create one yourself.
The go-flutter team has implemented flutter maintained plugins: https://github.com/go-flutter-desktop/plugins
We don't plan to work on community based plugins.
Closing this issue.


FIX: Make merging of adapter weights idempotent

Right now, merging of adapters weights such as LoRA and IA³ is not idempotent. This means that if a user calls merge multiple times, the resulting weights will be different each time because the delta weights are added again and again.
This fix checks that only those adapters are merged that are not yet merged. Also, gives a more precise warning:

Say when there is nothing to merge.
If there are some adapters to merge, only mention those

This bug is more subtle than it may seem at first, since we sometimes merge implicitly without the user necessarily being aware of it (e.g. when calling merge_and_unload). Therefore, this bug can occur quite easily, even if the user does not explicitly call merge twice in a row.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


GET ../collections/{uuid}/datasets MUST document that dataset_id is required

Reported by @bkmartinj while experimenting with the API.
GET ../collections/{uuid}/datasets suggests that both curator_tag and dataset_uuid are optional - rather than one identifier is required.
With the deprecation of curator_tag, the swagger should be updated to indicate that dataset_id is now required.

Issue moved to chanzuckerberg/single-cell-data-portal #3258 via Zenhub


$route doesn't contain anything on first load in "history" mode on any other path than "/"

Version
3.0.1
Reproduction link
https://github.com/darkylmnx/Layout-system-with-vue-and-vue-router
Steps to reproduce

clone the repo
cd into it & yarn
yarn serve
go to the path "/about-us" or any other route path except the root "/"
reload the page and check the console

What is expected?

I'm expecting to see only one log, the current $route with it's informations

What is actually happening?

I see two logs, in the first log the $route contains only null values and only the second one shows the good informations

This doesn't seem to happen on the root path "/"
This doesn't seem to happen in "hash" mode

Sorry for the delay! The repro is quite an overkill. I took a look but didn't see what was wrong


serving *.mpd file using flask

Thanks for this awesome library.
But I don't know how should I serve the generated .mpd file using flask application.
I create the .mdp file and serve it as an static file in flask, but when I put the url of the route in VLC player, player never stops loading and just shows up the duration of video and nothing more.
the exposed url for .mpd file is
http://<IP_ADDRESS>:5000/stream/playback.mpd.
in the route I return this:
return send_from_directory('/stream',
'playback.mpd')
can you please elaborate how should I achieve my goal in flask?
I have no idea and I am completely new to streaming video files.

I found the answer, thank you.


lastUpload and lastDownload not getting updated

I'm seeing the lastUpload and lastDownload values being updated in my gist but they are not getting updated in my userSettings json file.

lastUpload and lastDownload has been moved from user GIST settings to settings sync global settings.
You can remove these from settings.json.
Check the readme for details


added GPU support for bertTone

Ran autopep8 (due to the inconsistent spacing around '=') - this is why you will probably see a lot of very minor changes

(the rest is only for BertTone)
2) fixed a typo for max_length (instead of max_lenght)
3) added support for dealing with batch input (you can now give a list as input)
4) added support for GPU. You can now specify device (instead of using CPU)
5) added support for parallelization using torch's dataparallel
You should probably do the pull request by @saxogrammaticus before doing this one
In general, it seems tedious to apply these changes to each function. I would suggest using a superclass for the bert classifiers.
Please note that I haven't spent the time to validate this (due to the error fixed by saxogrammaticus), however, I believe it would be more maintainable in general to create a superclass.
Best,
Kenneth

Hi @KennethEnevoldsen
Thanks for the pull request and support for GPU.
I think there is a bug with the code changes, I ran it on cpu, and it looks likes the same output from _get_pred() every time, also I can get the output out in batches when providing a list of sentences?
Good idea about a superclass.
Kind Regards Amalie

Hi Ophelia,
I will close this issue as I how now added a production-friendly wrapper of the fine-tuned Huggingface Models (at least for sequence classification) in DaCy. Feel free to import these from DaCy if you need a speed increase.
You might still be able to get a slightly faster performance using a huggingface and PyTorch only implementation, but the integration with spacy is very useful and I don't suspect that it is too big of a difference.
Best wishes


Implement Scale Encode and Decode Traits for Light Client

Implements Derived types for easy scale codec encoding and decoding
closes #19

With TryFrom we can just return errors when conversion fails.

@dharjeezy This is still not implemented

We also need the reverse case from our derived types to the ethereum consensus types.

Yes, i am on it.


Documentation/2935#dubbo rpc api

What is the purpose of the change
Added javadoc and documentation for dubbo-rpc-api filter package.
Brief changelog
Added documentation for java classes for developer, to provide high level idea what this current class suppose to do.
Verifying this change
Run build locally.
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a 2935 field for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[ ] Run mvn clean install -DskipTests=false & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #2967 into master will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2967      +/-   ##
==========================================
- Coverage   64.08%   64.03%   -0.05%
==========================================
  Files         584      584
  Lines       26059    26059
  Branches     4562     4562
==========================================
- Hits        16699    16686      -13
- Misses       7185     7194       +9
- Partials     2175     2179       +4

Impacted Files
Coverage Δ

...va/org/apache/dubbo/rpc/filter/TpsLimitFilter.java
100% <ø> (ø)
:arrow_up:

...n/java/org/apache/dubbo/rpc/filter/EchoFilter.java
75% <ø> (ø)
:arrow_up:

...org/apache/dubbo/rpc/filter/ActiveLimitFilter.java
78.12% <ø> (ø)
:arrow_up:

.../org/apache/dubbo/rpc/filter/CompatibleFilter.java
83.33% <ø> (ø)
:arrow_up:

.../org/apache/dubbo/rpc/filter/DeprecatedFilter.java
76.19% <ø> (ø)
:arrow_up:

...org/apache/dubbo/rpc/filter/ClassLoaderFilter.java
100% <ø> (ø)
:arrow_up:

...rg/apache/dubbo/rpc/filter/ExecuteLimitFilter.java
87.5% <ø> (ø)
:arrow_up:

...apache/dubbo/rpc/filter/tps/DefaultTPSLimiter.java
66.66% <ø> (ø)
:arrow_up:

...ava/org/apache/dubbo/rpc/filter/ContextFilter.java
92.59% <ø> (ø)
:arrow_up:

...apache/dubbo/rpc/filter/ConsumerContextFilter.java
100% <ø> (ø)
:arrow_up:

... and 11 more

Continue to review full report at Codecov.

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

Hi, @khanimteyaz
Very nice work, I'd like take a look.
Another thing, would u pls clear your commits? Seems like your pr has four commits.

On it. Sorry for the delay in response.
On Mon, Dec 17, 2018 at 7:50 AM 时无两丶<EMAIL_ADDRESS>wrote:

Hi, @khanimteyaz https://github.com/khanimteyaz
Very nice work, I'd like take a look.
Another thing, would u pls clear your commits? Seems like your pr has four
commits.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/incubator-dubbo/pull/2967#issuecomment-447702803,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AdXcPdW5b5DTbcqEitzRwDCSFGapxd07ks5u5v9TgaJpZM4ZUNbi
.

nice job :)


Installation instructions adapted to the requirement cftime>=1.5.

🚀 Pull Request
Description
I adapted the from source installation instructions to the requirement cftime>=1.5 which will be needed soon, as discussed in #4258.

@MHBalsmeier Awesome 🚀


Node-sass v1 incompatible with node 0.12.2

Getting an Error: libsass bindings not found. Try reinstalling node-sass?
Digging through it seams to be related to the version of node-sass specified in the npm dependancies. Have fixed this by changing node-sass dependancy version to '~2.0.1'. Not sure if I should make a pull request for this issue.

I think this issue is with our generator, not the demo site. I've just published an update that solves it, too :)


Swifft template named incorrectly

Do NOT include any confidential information, this issue will be created on a public repo
This section is completed by the issue reporter
On which version(s) do you encounter this issue?
r12.2.8
Please provide the output to 'git describe --all' for  all deployments where you have confirmed it occurs.
12.2.8
What type of issue is this? Please tick the appropriate box.

[x] bug - existing functionality that does not work for you as documented
[ ] feature - new functionality that you would like to see added

What is the expected behavior?
Swift playbook completes.
What is actually happening?
A failure occurs because roles/os_swift/tasks/swift_post_install.yml is looking for a template called "swift-container-sync-realms.conf.j2" but the template is actually called "container-sync-realms.conf.j2".
What were you attempting to do when you encountered this error (give as much detail as possible)?
Upgrading to 12.2.8
How can we reproduce this issue?
Install or upgrade swift.
How can we verify your issue is addressed?
Either rename the template or set the proper template name in roles/os_swift/tasks/swift_post_install.yml
If you are using or know of a workaround please describe the necessary steps?
I renamed the template to "container-sync-realms.conf.j2".
What is your evaluation on the importance of this issue, and why?
It important as it causes the upgrade/install to fail.
This section is completed during the issue resolution process. If you are reporting an issue, please do not modify the section below.
Which branches are impacted by this issue?
Remove branches that are not impacted.
For each pull request, update the branch line to reference it.

[ ] master
[ ] mitaka-13.1
[ ] liberty-12.2
[ ] kilo

Upgrade impact. Consider the impact on moving between major, minor and patch releases.

[ ] This change does not impact upgrades
[ ] This change addresses a potential upgrade impact
[ ] The upgrade impact is being tracked by another issue.

The issue tracking the upgrade impact is:
Testing impact

[ ] An issue/pull-request has been create or exists to track the testing gap, and is referenced below
[ ] This change includes additional testing
[ ] This change is tested by the existing setup

Additional testing links:
Release note

[ ] This change includes a release note.
[ ] This change does not require a release note.

Doc Impact
Does your change require documentation?

[ ] Yes, and I submitted an issue to Docs issues
[ ] Yes, and I contributed a documentation patch to docs-rpc
[ ] No documentation required for this change

Additional documentation links (please add links if you have ticked yes):

This is invalid since OSA does not manage  swift-container-sync-realms.conf.j2 per https://github.com/openstack/openstack-ansible/blob/eol-kilo/playbooks/roles/os_swift/tasks/swift_post_install.yml
this has been added post install, manually


Discord link in README is invalid (again)

Hi, the discord link in README is invalid. I saw there was a commit to fix it a week ago, but I wanted to join the server today and the link is invalid.

Thanks, should be fixed now

you can change the validation time of a link (so it will not expire)

Yup, I did it last time and then pasted the wrong link.


feat: Remove the Blob Container creation check as it was taking ~10s to execute

:tada: This PR is included in version 2.0.0-next.48 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 3.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Download fails with long url

I tried to download the search results from this query but the download failed.
Note the long URL; shorter queries worked fine. The celery log includes
psycopg2.errors.StringDataRightTruncation: value too long for type character varying(254)

Which is prorobably the reason

Resolved by #1283


Bump the API version for the packaging APIs and Apps

Describe the problem/challenge you have
All of the CRs provided by kapp-controller are currently versioned v1alpha1. According to the k8s API versioning guidelines, they should all actually be versioned v1beta1.
Describe the solution you'd like
Bump the api versions to v1beta1 to better align with k8s versioning practices
Anything else you would like to add:
No

Vote on this request
This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.
👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"
We are also happy to receive and review Pull Requests if you want to help working on this issue.

Something to consider here is how we would/how long we would support v1alpha1 versions of the CRS.
This would also require us to look into kubebuilder directives for generating the YAML representation of the CRDs with multiple versions (e.g. https://github.com/vmware-tanzu/carvel-kapp-controller/blob/a2236fb3c56c014a8ca10ce422208e45d21229e1/config/crds.yml#L13).

Just adding some documentation here on how k8s recommends to think about versioning CRDs: https://kubernetes.io/docs/tasks/extend-kubernetes/custom-resources/custom-resource-definition-versioning/
In general, the kapp-controller resources that exist seem to meet v1beta1 requirements. We just need to prioritize this issue and pick the right time to make the switch.


[FEATURE] Configure Cluster Search Request Slow Logs

Is your feature request related to a problem?
As of 2.12, we can configure cluster level search request slow logs. There is no support for this in the provider right now.
What solution would you like?
Add a block to opensearch_cluster_settings for search request slow logs:
resource "opensearch_cluster_settings" "example" {
search_request_slow_logs {
    slowlog_level = "TRACE"
    slowlog_threshold_warn = "10s"
    slowlog_threshold_info = "5s"
    slowlog_threshold_debug = "2s"
    slowlog_threshold_trace = "10ms"
}

What alternatives have you considered?
I do not see an alternative method using this provider. The only other way to achieve this via Terraform is via a script and the external provider, the local-exec provider running a PUT to the cluster, or via the http provider.
Do you have any additional context?
This is a relatively new feature that is by default disabled in the cluster, set to -1.

Hey @ironf1st something like the following should be even easier for the user.
resource "opensearch_cluster_settings" "slowlog" {
  cluster_max_shards_per_node = 10
  cluster_search_request_slowlog_level            = "WARN"
  cluster_search_request_slowlog_threshold_warn   = "10s"
  cluster_search_request_slowlog_threshold_info   = "5s"
  cluster_search_request_slowlog_threshold_debug  = "2s"
  cluster_search_request_slowlog_threshold_trace  = "100ms"
}

I have a PR open https://github.com/opensearch-project/terraform-provider-opensearch/pull/211 to support the Cluster Search Request Slow Logs.
Thanks
@getsaurabh02 @peterzhuamazon @rblcoder

The terraform provider 2.3.1 is released supporting Search Request Slow Logs, @ironf1st please take a look.
Thanks

Closing this issue since the terraform provider 2.3.1 is released with Cluster Search Request Slow Logs support, please feel free to add a comment or re-open if required.
Thank you
@getsaurabh02


KeyError: 'optimizer'

when training，is_finetune = True, it appears KeyError: 'optimizer'

By default, we finetune our model based on previous optimizer weights. If you are using our provided weights without optimizer, you could comment this line. https://github.com/ewrfcas/MVSFormer/blob/a635b1714a51948f6e808766e9dafa73d05ed51a/train.py#L117

Strange problem.
If you have set fp16, scaler.scale should be run in
https://github.com/ewrfcas/MVSFormer/blob/77abe395cc32844478733e00aed9f73ad40ba05f/trainer/mvsformer_trainer.py#L134-L137

the default set is fp16 = true, can it be changed to false
if self.fp16:
self.scaler.scale(loss).backward()
else:
loss.backward()
this is the same of my code, but the question appears

fp16 could be set to False, which is only used to save memory and speed up the training.


[Event Request] Report 1005 - "Job Journal - Test" - existing event "OnAfterAssignDimTableID" - change reference-type

See Reference #6568 and #6600
Please change the reference-type of the parameters "TableID" and "No" at event "OnAfterAssignDimTableID" of Report 1005 "Job Journal - Test".
OLD CODE:
[IntegrationEvent(false, false)]
local procedure OnAfterAssignDimTableID(JobJournalLine: Record "Job Journal Line"; TableID: array[10] of Integer; No: array[10] of Code[20])
begin
end;

NEW CODE:
[IntegrationEvent(false, false)]
// start kvspsa - Event Publisher - Referencetype of parameters "TableID" and "No" changed
local procedure OnAfterAssignDimTableID(JobJournalLine: Record "Job Journal Line"; var TableID: array[10] of Integer; var No: array[10] of Code[20])
// stop kvspsa
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Added task for deploying Barbican

~Not yet tested~
Tested with few CLI options (https://docs.openstack.org/python-barbicanclient/latest/cli/details.html)
, seems to work without issues

We have to wait until image is published in dockerhub

We have to wait until image is published in dockerhub

-> DONE, PR is ready for reviews

Not sure if related, but the CI failed with /bin/sh: /usr/bin/python: No such file or directory
See https://ci.suse.de/blue/organizations/jenkins/cloud-socok8s%2Fsocok8s/detail/PR-274/1/pipeline#step-25-log-1913

Not sure if related, but the CI failed with /bin/sh: /usr/bin/python: No such file or directory

I think this was related to the change of the Leap image for the deployer (which was reverted meanwhile): https://github.com/SUSE-Cloud/socok8s/pull/309


Add test pypi styles

Closes #3104

Adds sticky warning
Adds 'testing' bg image on header

I really like the background image.
At first I didn't notice the red warning because it looks just like the pre-production warning that I've become used to. Maybe it's unnecessary? We're starting to have a lot of these piling up.

Thanks for your feedback @di - I originally only had the background image, but then I realised that anyone with a visual impairment would miss this information.  Also, the notification bar is sticky, where the header is not.
I think the banner blindness that you describe should disappear once we remove the pre-production warning - e.g it won't be normal to have a big red banner at the top all the time.

Ah - one thing I just realised might be a problem - if we decide to translate the site in the future, using a word as a background image might not be ideal.  I suppose we can cross that bridge when/if we come to it.

I think we can localize the image just as easily if we do it in the template -- we'll just need to make a new image for every language.

I think images referenced in CSS is still cache busted?

I think images referenced in CSS is still cache busted?

 Sorry for the bad advice here @nlhkabu.

Could <title>PyPI - the Python Package Index · Warehouse (TestPyPI)</title> change to <title>TestPyPI - the Python Package Index · Warehouse</title> or similar, so that the truncated title in a browser tab starts with "test"?
And could the favicon be slightly different?

Yes, I can make these updates @brainwane.
@di - please advise on cache busting.  Should I remove the image altogether?  I think with the banner, combined with @brainwane's suggestions, we probably don't need it.

Here's a suggestion. Could Test PyPI, instead of saying

Find, install and publish Python packages with the Python Package Index

say something like this?

Test Python package publishing with the Test Python Package Index

@nlhkabu What you originally had (background image in CSS) was fine. I like the background image and think we should keep it, but I'll leave it up to you.

Updated:

@di or @ewdurbin - ready for re-review

looks great! I really like the repeat in the header!


NLog LogManager.Configuration.FindTargetByName doesnt seem to respect filter rules

NLog version: 4.9.0
Platform:  .NET Core 3
Current NLog config (xml or C#, if relevant)
<nlog>
  <targets>
     <target name="memory" xsi:type="Memory">
       <layout xsi:type="JsonLayout" includeAllProperties="true">
        <attribute name="test" layout="emptystr"/>
      </layout>
     </target>
  </targets>
  <rules>
    <logger name="*"  writeTo="memory">
      <filters defaultAction="Ignore">
        <when condition="equals('${environment:ENV}','test')" action="Log" />
      </filters>
    </logger>
  </rules>
</nlog>

What is the expected result?
When calling LogManager.Configuration.FindTargetByName<MemoryTarget>("memory").Logs; in a test with the above configuration I would expect if the when condition is not met for the configuration to be unavailable.
What is the current result?
It is always logging to the target.

Somehow my comment on this didn't save after I initially opened it but this ended up being a problem on my end. Thanks for the help!


Issue with P4est_jll and MPIPreferences

Let me bring up the following issue again. MWE: In a fresh environment with julia --project=., do
julia> using Pkg
julia> Pkg.add(["MPIPreferences", "P4est_jll"])
[...]
julia> using MPIPreferences
julia> MPIPreferences.use_system_binary()
┌ Info: MPI implementation identified
│   libmpi = "libmpi"
│   version_string = "Open MPI v4.1.0, package: Debian OpenMPI, ident: 4.1.0, repo rev: v4.1.0, Dec 18, 2020\0"
│   impl = "OpenMPI"
│   version = v"4.1.0"
└   abi = "OpenMPI"
┌ Info: MPIPreferences changed
│   binary = "system"
│   libmpi = "libmpi"
│   abi = "OpenMPI"
│   mpiexec = "mpiexec"
│   preloads = Any[]
└   preloads_env_switch = nothing
julia> using P4est_jll
julia> P4est_jll.libp4est
"~/.julia/artifacts/3a2ff5d867b2d3d729a84fb3f736761ae336fcbf/lib/libp4est.so"

but after leaving and restarting the REPL julia --project=.:
julia> using P4est_jll
julia> P4est_jll.libp4est
ERROR: UndefVarError: `libp4est` not defined
Stacktrace:
 [1] getproperty(x::Module, f::Symbol)
   @ Base ./Base.jl:31
 [2] top-level scope
   @ REPL[2]:1

After deleting the LocalPreferences.toml, of course, this works again.
This has not been a real issue as one usually wants to use a custom p4est version (and not the one from P4est_jll) when using a system MPI anyway, but the reason I come up with this again is that the exact same steps as above, but with HDF5_jll and libhdf5 does not give any UndefVarError for me, as I would also expect. If this would work for P4est_jll, we can think about adding a convenience function set_libraries! (or similar) that sets the preferences for P4est.jl as it is done in https://github.com/JuliaIO/HDF5.jl/pull/1115. However, with the error above using P4est does not work as long as we have MPIPreferences set and so the convenience function could not be called. Do we know why this happens with P4est_jll in the first place? Does it also happen with a local MPICH version (which is the default for P4est_jll.jl)? And even better: Can we fix this for P4est_jll?
This would also make https://trixi-framework.github.io/P4est.jl/stable/troubleshooting/ obsolete.

Good question. I'm not really familiar with the build setup of HDF5_jll. How do they handle MPI?

I think this happens because we do not build P4est_jll for OpenMPI, see
https://github.com/JuliaPackaging/Yggdrasil/blob/c26186fc6e01678668521f0be0594be6f8b81fd3/P/P4est/build_tarballs.jl#L80-L82
That means if we use P4est_jll as-is, loading a different MPI library than MPICH will cause P4est_jll to not find a suitable artifact and thus P4est_jll.libp4est remains unset. The "correct" fix would here be to enable building p4est with BinaryBuilder.jl also for OpenMPI, i.e., modifying the build_tarballs.jl and possibly some sources of p4est to play nice with OpenMPI.
@JoshuaLampert if you're willing to take a stab at this, I can guide you through using BB.jl and the build_tarballs.jl to try to fix this.

Thanks, @sloede! I was thinking something along those lines since HDF5_jll cross-compiles for all MPI implementations and P4est_jll does not.
I would definitely like to fix this properly, but I am not sure how much work this would be (especially for someone like me who has only limited experience with BB.jl). Looking at https://github.com/JuliaPackaging/Yggdrasil/pull/6551 this looks quite heavy... But if you think it is doable with your guidance @sloede, I would be happy to give it a try!

The first things are super easy:

fork Yggdrasil
clone the fork on roci and create a branch
modify the p4est build_tarballs.jl and comment the line that disables OpenMPI
create a draft PR to Yggdrasil

Now you wait and see until Yggdrasil has built all configurations and you check how many of them fail. Then, pick the "easiest" sounding config that fails (for example, Windows is not easy, whereas x86 Linux might be) and try to build it locally:

go to your Yggdrasil clone into the directory where the build_tarballs.jl is located
install BB: julia --project=. -e 'using Pkg; Pkg.add("BinaryBuilder")'
run a config that should work to verify that BB is working in general by executingjulia --project=. build_tarballs.jl --deploy=local --verbose --debug=error x86_64-linux-gnu-mpi+mpich

if that works (it should), pick a platform that does not work and modify the last argument to the command above
the build will fail at some point and BB drops you into a local sandboxed environment. it's like a Unix shell and you can now muck around and try to rebuild etc. if you want to start a fresh session, you can change it to --debug=begin to be dropped at the beginning of the build session

After that, I think it's better to ask specific questions :-)

Thanks a lot for the detailed description @sloede . I'll try it tomorrow.

Ok, a new version of P4est_jll is out (https://github.com/JuliaPackaging/Yggdrasil/pull/7500), which also builds OpenMPI (that was easier than expected :-)). So this should be fixed.
What do you think of a convenience function set_p4est_preference! (and set_sc_preference!) that sets the preference easier without the need to copy/paste the UUID of P4est? If that works, I also suggest to do something similar for T8code.jl.

If that works, it's a nice addition. It's just important to restart Julia since it's a compile-time preference, isn't it?

Yes you would still need to restart Julia.

Good idea! If HDF5 started with set_library!, what about following along with set_library_p4est! etc? Though I don't have strong preferences here (no pun intended). One should show an info message though that Julia needs to be restarted.

See my PR #88. I used a similar pattern as in HDF5.jl.


Add O/S id to the resource string

In other to easy the resources management and identification, please make Pade detect the OS Id and add it to the resource string automatically.
Thank you!

Done. Now available in version 1.0.8


Refactor middleware factory and tests

Q
A

Is bugfix?
❌

New feature?
❌

Breaks BC?
❌

👍


Upgrade to latest open release

This needs to be tested against 11.0 SP9 and upgraded if necessary.

Retarget to current open release: 11.0 SP12.

Works on V10+, V11+. (service pack not an issue due to package being SQL query)


[Bug]: Huge Error Dump When Opening F3 Screen

Describe the Bug
When opening the f3 screen it freezes for a couple seconds and dumps a massive error message to the logs.
Steps to Reproduce

Install mod
Open world
Open f3 screen
???
Profit

Expected Behavior
Shouldn't dump a massive error message to logs.
Screenshots
No response
Modloader Type
Minecraft Forge
Modloader Version
47.3.4
Mod Version
7.0.2
Log Output
Log was to big, so here is a link to a gist containing the entire error message.
https://gist.github.com/SaltSouls/932fb2a9490269ab5720d683324790da

Additional Context
No response

if you disable the fps module it wont do this error


pythonPackages.sfepy: 2019.2 -> 2019.4

Motivation for this change
Updates Sfepy to version 2019.4 and removes broken test that stops Sfepy building.
Things done

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

[ ] NixOS
[ ] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] 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)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nix-review pr 80994 1

3 package were build:

python27Packages.sfepy
python37Packages.sfepy
python38Packages.sfepy

Withdrawing this as should be made to release-20.03 which is currently breaking


AttributeError: 'ParsedHytekFile' object has no attribute 'meet'

meet.zip
Hi, the hy3 file is attached.  This is the code I am running.
import attrs
import hytek_parser
import pprint

input_file = r'.\h3file\meet.hy3'
results = hytek_parser.parse_hy3(input_file)

d = attrs.asdict(results)

pprint.pprint(d)

When I run this, I get the following error:
Traceback (most recent call last):   File "c:\Users\xyz\OneDrive\Project\hytekparser.py", line 9, in <module>     d = attrs.asdict(results)         ^^^^^^^^^^^^^^^^^^^^^   File "C:\Users\xyz\OneDrive\Project\.venv\Lib\site-packages\attr\_next_gen.py", line 198, in asdict     return _asdict(            ^^^^^^^^   File "C:\Users\xyz\OneDrive\Project\.venv\Lib\site-packages\attr\_funcs.py", line 57, in asdict     v = getattr(inst, a.name)         ^^^^^^^^^^^^^^^^^^^^^ AttributeError: 'ParsedHytekFile' object has no attribute 'meet'
I am assuming it's because the file doesn't have an attribute 'meet', how do I get around this?

I'll make a new release after #5 merges

Version 1.1.1 has been published with the fix for this issue.


Why does the extension say "Multiple function projects"?

In my solution there's only one project that relies on Microsoft.Azure.Functions.Extensions and/or Microsoft.NET.Sdk.Functions.
Granted, that project has a dependency to a nuget package that probably uses Functions too, but it's still inside one project of my solution.
Yet when I use the extension (command palette : Create Function) I get the error "Multiple function projects" from https://github.com/Microsoft/vscode-azurefunctions/wiki/Multiple-function-projects
Why?

EDIT: It turns out it works when I don't open the whole solution folder in VSCode but instead only the subfolder containing the Functions project.
So... Was that by design?

Could you provide a more detailed layout of the solution so we can see if there's something we can improve on our end here? Can you look into that dependency and verify what it used?
I'd like to know more so I can determine why we detected two Functions projects. Maybe we can fix it.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Documentation question

The documentation reads:
The structure of output.js will be as follows:

// - header
// - ordered content of the files in headerFiles
// - un-ordered content of files matched by inputFiles, but not in headerFiles or footerFiles
// - ordered content of the files in footerFiles
// - footer

Does this mean that if I have:
inputFiles: ['fileA', 'fileB']

The order of fileA and fileB in the final outputted file is not guaranteed? If so, how can I guarantee the order is as specified in the inputFiles array?

Sorry for letting this linger:

inputFiles order in 2.x is not guaranteed, if a guarantee is required please use headerFiles or footerFiles which are preserve exact order
inputFiles order in 3.x is guaranteed to be lexicographical

Why is this the case?

inputFiles allows for an array of globs and non-glob inputs, which makes an order guarantee much more complicated (although not impossible) to implement, I believe though it will be hard to reason about order when considering specificity of several complex globs. Because of this, inputFiles should be considered not order preserving, but as of 3.x the files matched by inputFiles will be lexicographically sorted based on their relativePath. Which should help with build/outputfile stability.


Mp rest client headers propagation

Fixes:

https://issues.jboss.org/browse/RESTEASY-2432
https://issues.jboss.org/browse/RESTEASY-2401
https://issues.jboss.org/browse/RESTEASY-2443

Please @kenfinnigan and @michalszynkiewicz can you review this too?

Looks okay to me. Thanks!

@asoldano the CI said it was cancelled. Any idea what happened?

@asoldano the CI said it was cancelled. Any idea what happened?

it should finish running now; there's been an error in one of the runs, so I restarted it, but forgot to also reschedule the javadoc jobs

Oh OK, great, thanks!

Thanks!