Interpretation of nominal srate and mocap metadata (orientation ABCD)

Hello everyone,
I have a couple of questions about some metadata fields that have been a bit confusing for me.
It would be really awesome if someone could clarify these points so we know how to implement experiments in the future :)

How is nominal sampling rate of 0 to be interpreted?
Is this, a) necessarily a stream with no nominal srate in any sense, such as event streams OR b) can be a continuous stream with irregular sampling intervals?

In the specification, I read "If the stream has an irregular sampling rate (that is, the samples are not spaced evenly in time, for example in an event stream), this value must be 0."
Now, depending on what "irregular sample rate" means, nominal srate of 0 may also be assigned to a motion capture system of 90 Hz desirable (or nominal?) sampling rate that has some jitters in time stamps. LSL for unity appears to follow this interpretation and uses this flag to decide whether time stamps generated are perfectly regular or not.
On the other hand, xdf2fieldtrip checks nominal srate to determine whether a stream is continuous. In fact, in the group I am working in, there are data sets created using the same hardware but use either nominal srate of 0 or 90 for continuous streams depending on LSL4Unity configuration, so this has been a bit tricky to deal with.
The practical question is, is it OK to use nominal srate of 0 when there actually is some kind of nominal srate (90 Hz)? The presence of nominal srate appears to be mixed up with regularity of sample intervals or continuousness of streams, which I think all mean different things. Can we think of adding new fields for these properties?

How are the quaternion orientation units to be interpreted in motion capture metadata?

The wiki page proposes A,B,C, and D as names for quaternion elements.
How do they correspond to the X,Y,Z axes and real/imaginary elements of quaternions? I had difficulty figuring out the correct quat to eul conversion from the documentation and while it is possible to try out different configurations to visually see the correct mapping to the spatial axes, this seems to be left flexible.
Pitch, roll, and heading also makes a lot of intuitive sense but I am failing to read the order of rotations applied. Would it make sense to follow a style that is specified in for instance this proposal for extending Brain Imaging Data Structure for motion data? If there is anything I can contribute regarding this I would be more than happy to.
Thanks a lot in advance!

Hi, here's an attempt at a clarification for question (1):
Many devices (and some software applications) are deliberately designed so as to maintain (within some tolerance) a specified regular rate at which values are measured (sampled). When that is the case, one should consider declaring the specified rate as the nominal rate. When that is not the case, the stream should be declared irregular (that is, nominal_rate=0). There is a grey zone when a device exhibits deviations from some intended or specified rate that are sufficiently large (*) that it is better treated as irregular. So this means that nominal_rate=0 just means that one cannot expect there to be (even approximate) regularity. This includes events streams, but that's only a subset.
*: The remaining question is what's "sufficiently large" to qualify as irregular. Almost any real-world device or system is subject to some pervasive deviations that we'd clearly consider benign or acceptable:

no real-world system (short of an atomic clock) maintains the nominal rate anywhere near "perfectly" -- e.g. technical limitations leave even most professional-grade devices with as much as +/- 0.01% of error between the nominal (specified) rate and effective (measured) sampling rate. Devices or streams clocked off the mains frequency (or, e.g., monitor refresh rate) could have as much as 1% error. Furthermore, those deviations even change on the order of minutes to hours for various reasons (e.g. temperature), so the sampling rate is not even stable.
almost any real-world system can temporarily cease to send data (e.g. run out of battery, crash, lose connectivity), which results in occasional (usually lengthy) gaps in the time series. It helps that sufficiently long gaps are quite easy to detect if one knows the nominal rate.

Among the less benign deviations are the following:

a device that sporadically drops samples (e.g., certain cameras): these issues are extremely hard to detect -- however, in those cases more metadata is actually helpful, i.e., those devices should ideally declare both the nominal sampling rate also a (recently-proposed, not-yet-documented) flag that indicates that the stream can drop frames (synchronization/can_drop_samples).
a device whose sampling interval may be steady at times but can become quite erratic (e.g., as in video game frame rates). We currently have no flag "can_be_erratic" or the like (let alone applications using it), so we have to assume that an application reading such a stream would be unable to detect the erratic periods (they wouldn't register as gaps) and would almost certainly generate bogus outputs from them. It's also not clear that a flag would really help, or whether there should be another kind of sampling rate like the "desired_rate" or the like, which describes the behavior on a lucky day (and which can be used to detect those lucky stretches for the few (probably fairly rare) applications that want to do that).

The above should also make it clear that, just because a stream had at some point been measured to have an average sampling rate of say 53.6 Hz, that's no reason (at all) to declare that as the nominal sampling rate -- the nominal sampling rate will almost always be explicitly built into some sort of engineered process for maintaining that rate.
Lastly, it's also important to note that the time stamps themselves are measurements, and especially for hardware streams they can be jittery even though the underlying sampling process is (usually) far more regular. The regularity statement is about the sampling process (that generates the data values), not the time stamps. The time stamp jitter would usually be addressed in many applications by dejittering or linearizing the time stamps (likely after detecting and respecting any data gaps) to get improved estimates of the actual time stamps.

I was about to send my response as Christian's popped up. Some of this may be redundant with what he said but there's a useful example in there to help you think about it.
First, the idea of "continuous" is more of a scientific thing than an engineering thing. Indeed every data source in LSL is probably "continuous", depending on how you define it, otherwise you wouldn't need something like LSL. The use of 'nominal_rate' is orthogonal to the question of whether or not a stream is continuous. They don't mean the same thing, and "continuous" or not doesn't have a good definition (within the context of all LSL streams) so I don't think it merits a new flag.
The way I think about it: whether or not I set an outlet to have a non-zero nominal_rate depends on whether or not I want the XDF importers or the LSL postprocessors (if invoked by the Inlet) to try to smooth out the timestamps.
Let's say you're using a wiimote. When you are actively shaking it, its data rate is fairly consistent. However, when you set it down it ceases to transmit, so its data rate drops to 0. Now you start shaking it again, and the period at rest was less than the jitter_break_threshold_seconds. What do you want to happen? Well if you specified a non-zero nominal rate, then the xdf importer will assume that the first of the new timestamps should be 1/srate after the last received timestamp and will attempt to smooth out the discrepancies. But this is incorrect. To avoid this we could set the wiimote outlet to have a nominal_rate of 0 so all timestamps are taken at face value. This makes it impossible for the outlet to use push_chunk and for any receivers to do filtering or other signal processing because they don't know the expected rate. There may be other solutions to this particular problem, but I'm just using it to illustrate.

LSL for unity appears to follow this interpretation and uses this flag to decide whether time stamps generated are perfectly regular or not.

LSL4Unity is not a ground truth for anything. I'm sure Markus would agree.
Different software might use nominal_rate as a shortcut to say "if it's nominal_rate is 0 then this stream contains events". I usually take it a couple steps further and only assume that a stream is an event stream if its type is 'Markers', it has a single channel, and the dtype is strings.
@sjeung - please open a separate issue for the question about Mocap specifications.

Regarding question (2):
For the quaternions, the (A,B,C,D) are the 4 components of the quaternion,  following the wikipedia or MathWorld nomenclature. Game engines (Unity, Unreal) seem to often use X,Y,Z,W but when you look closely, you see that they order them as W, X, Y, Z in some of their docs (so the 3 axial components come last, and those happen to also correspond to the last three B, C, D components). However, I wouldn't be surprised if there was some MoCap system that wrote X,Y,Z,W instead of W,X,Y,Z into A,B,C,D channels. Note that there's also an alternative format in which orientations can be declared for systems that emit those, using Pitch, Heading, and Roll (PHR), but few if any system seem to use those (not surprising given gimbal lock issues etc). It's not explicitly stated in which order they'd be applied, but anything other than in the written order would be quite surprising. In theory one could also add an axis-angle format using OrientationX, OrientationY, OrientationZ, OrientationW for systems that support that (it'd be more visually intuitive but then again, most 3d software tools can already take in quaternions, and the existing LSL applications and recordings aren't going away, so then we'd be at 3 different formats to support).

Thanks a lot @chkothe and @cboulay for your prompt and detailed answers. Both answers help me get a much clearer picture. My group is indeed dealing with the less benign type of deviations (from a VR setup) that Christian described. Both nominal rate = 0 and nominal rate = 90 + can drop samples flag 'on' seem to work fairly well when we use load_xdf with default options but I will try generating and comparing data from the two configurations in the next days. I also agree with Chadwick that nominal srate is not the best way to separate different stream types.
I have created a separate issue #57 for mocap metadata, as suggested.


pretrained/pytorch_model-00001-of-00003.bin

Greetings, and thank you for this model!
Please i cant find or know what to do to find and install this file 'pytorch_model-00001-of-00003.bin'
If you can guide me please, thanks in advance

In order to download the checkpoints, fill this form: link.

In order to download the checkpoints, fill this form: link.

Thank you Sir, your effort is highly appreciated!

i filled the form on April 2nd,how long can i get the files?

i filled the form on April 2nd,how long can i get the files?

just resend the form and check your spam folder

我是4月2日填的表，多久能拿到文件？

你好，收到了下载链接吗

I have filled the form，how can I get the model，thanks！My email is<EMAIL_ADDRESS>

I have filled the form，how can I get the model，thanks！My email is<EMAIL_ADDRESS>
Have you obtained the corresponding files? Can you share them with me? My email is<EMAIL_ADDRESS>Thank you.


Updates to the help site's visual design

Main cleanup tracking issue - https://github.com/Expensify/App/issues/28497
Problem
The existing help site pages are challenging to read -- the text is close together, the headings aren't sized correctly, the background vs text color is creating readability issues.

Example help page: International Reimbursements
Slack convo: HERE

Solution(s)

Create more space between text
Create more space between paragraphs
Update how the # and ### headers are styled on the site
Alternate the backgrounds between headers to help visually separate the sections of the resource more

Added some progress here

Payment

[ ] @rushatgabhane (C+ = $500)

@rushatgabhane - was there a separate payment issue created for this one? lmk if i should be the one processing payment ty

@rushatgabhane / @dubielzyk-expensify - are we all set on this one? lmk if we're good to close 👍

If the code changes have been pushed, then yep, we're good to go.

looks like this was merged so yes, i shall close! https://github.com/Expensify/App/pull/29539

actually it's maybe not updated on prod yet, so I'll keep open.

for helpdot, merge = deploy!! we can close this @maddylewis

$500 payment approved for @rushatgabhane based on this comment.


autosave prompt

This change is

@jincypjose @Joel-C-Johnson
Thank you so much for your help

@elsylambert
I can put updated test notes in issue # 921
https://app.zenhub.com/workspaces/tc-create-workspace-5f049cdbfa88090016bf043e/issues/unfoldingword/tc-create-app/921

My test notes are here
I had two comments:
On scenario 1:
Question: at the end of the scenario, the save button is not active, even tho I have unsaved content. Which is to say, on QA DCS, my branch exists with the old version of Jude, not my changed version. If I really want to save the content, I'll have to make another change. then the button will become active, enabling me to save. Is this OK?
On scenario 3: I used a different computer to make the change instead of simulating by editing directly in DCS.
It did not prompt to discard or keep. It simply went to the latest file in my branch which I had changed on the other computer.

code looks good to.Have reviewed by me and @ancientTexts-net


Azure Logic Apps - Schedule weekday only

I'm trying Azure Logic Apps, but I can't figure how I can schedule it to run only on weekdays. The docs have a "Show advanced options" that displays a lot of configurations (including what I'm looking for), however this is not available to me, as you may see here:

...and this "Add new parameter" button only has two parameters:

Time zone
Start time

Is there any other way I can configure a recurrence schedule, for instance to run twice a day, (e.g every work day at 9am and 5pm)?
Thanks!

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: f9106dab-3f70-7388-1be0-da02b5728827
Version Independent ID: 13613736-0637-a17c-f5b4-a1476e294032
Content: Schedule and run automated tasks and workflows with Azure Logic Apps
Content Source: articles/connectors/connectors-native-recurrence.md
Service: logic-apps
GitHub Login: @ecfan
Microsoft Alias: estfan

Hi @AlissonRS, thanks for your question! You can select specific days when you set Frequency to Week as described in the Trigger details section:

And then to set the specific times of day, use At these hours and At these minutes, which appear only after you select Week or Day:

@ecfan thanks for your answer, that was exactly what I was looking for. I really appreciate it.

 I'll now ask the team to close this issue. If you have other questions, please feel free to open another issue. Thanks!
#please-close

Would it be possible to add a cronjob syntax to this at least on minutes? Writing every minute you want it to run is cumbersome. 3,6,9,12,15 etc...

Hi @boltlol, are you requesting to add this syntax as a feature? If yes, please submit your request at the Logic Apps product feedback site so the team can review:
https://feedback.azure.com/forums/287593-logic-apps
Thanks!

@ecfan  I have a similar need but a tad more complex.  We have a complex ADF pipeline run that ingests data, and at conclusion I have logicapps that fire to rebuild the Analysis Services Models.  All days except Saturday mornings, I need the logicapps to make one call to the AAS services (has a partition call), then on Saturday mornings, I need it to do a different AAS refresh call (full refresh on models).  The trick is - I don't know the time it will run, because it needs to run at the end of the ADF completion.  Is there a way within the logicapp itself (not a trigger) to do this and determine what day it is and make a loop based on the day?  Thanks

Hi @chasdso, thanks for reaching out! Could you use the dayOfWeek() function in an expression to determine the current day?

@ecfan woot!  Perfect thank you! Overlooked it -


Add plotting functions

Add plotting functions for raw spectra and DLA models
Add an optional plot_figures argument in run_bayes_select.py
Add an error handler in read_spec function due to some newly downloaded data don't have the COADD column

Examples

Usage:
For the test, simply turn on the flag:
python -c "from tests.test_selection import *; test_p_dlas(10, plot_figures=True)"

then the plots will be saved to ./images/.

Merge since it works so far.


[DO NOT MERGE] Auto PR to update dependencies of istio

This PR will be merged automatically once checks are successful.
none

@istio-testing: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
774e599f76e1f4aef384b245a0215fc4ba061cbb
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


[Code]:  DAOSpace GlobalProposals not removed after being executed or rejected.

GlobalProposals is never removed after being executed or rejected.
The vector is always growing, there may have state explosion issue.

also, need to add max limit to GlobalProposals
https://github.com/starcoinorg/starcoin-framework/blob/e82c1d32c98170e82f2145bf5a03badf8af06de7/sources/daospace/DAOSpace.move#L1467-L1470

Delete proposal after executed or rejected? Is there a need to keep them for querying history proposal info?

Delete proposal after executed or rejected directly. Is there a need to keep them for querying history proposal info?

I think they can bean deleted. The history proposal can query from history state or off-chain index DB.

ok, I will fix it.


Use slog on KubeArchive

Which Issue(s) this Pull Request Resolves

Resolves #423
Release Note

NONE

Notes for Reviewers

I removed the logger parameter forwarding in the sink and went with using the default logger. Its simpler. If we need to pass a logger for whatever reason, then we can add it again.

What does the format of the log messages look like?

2024/09/23 13:20:43 INFO Starting KubeArchive Sink version=v0.1.0 commit=023005ba461e47e5374e92da2e598b8dc1f4b4b3 built=2024-09-23T15:20:31+02:00
2024/09/23 13:20:43 INFO Successfully connected to the database


exiv2: bugfix updates

use upstream branch with backports
backport two fixes missing from it yet

Motivation for this change
I wasn't aware of work #44464 and took a different approach.  I'm not too sure which is better, so let me open another PR at least for reference :-)
Things done

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

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

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

Success on x86_64-linux (full log)
Attempted: exiv2
Partial log (click to expand)
strip is /nix/store/1hi76hr87bd1y1q1qjk0lv8nmcjip1c8-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/dx9hgbkha2zmmz2mn1wjj2isshsa4qc0-exiv2-0.26.2018.06.09/lib  /nix/store/dx9hgbkha2zmmz2mn1wjj2isshsa4qc0-exiv2-0.26.2018.06.09/bin
patching script interpreter paths in /nix/store/dx9hgbkha2zmmz2mn1wjj2isshsa4qc0-exiv2-0.26.2018.06.09
checking for references to /build in /nix/store/dx9hgbkha2zmmz2mn1wjj2isshsa4qc0-exiv2-0.26.2018.06.09...
shrinking RPATHs of ELF executables and libraries in /nix/store/q15qy4lr7q844g0kgwxnrwbgg91zgc2x-exiv2-0.26.2018.06.09-dev
strip is /nix/store/1hi76hr87bd1y1q1qjk0lv8nmcjip1c8-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/q15qy4lr7q844g0kgwxnrwbgg91zgc2x-exiv2-0.26.2018.06.09-dev/lib
patching script interpreter paths in /nix/store/q15qy4lr7q844g0kgwxnrwbgg91zgc2x-exiv2-0.26.2018.06.09-dev
checking for references to /build in /nix/store/q15qy4lr7q844g0kgwxnrwbgg91zgc2x-exiv2-0.26.2018.06.09-dev...
/nix/store/dx9hgbkha2zmmz2mn1wjj2isshsa4qc0-exiv2-0.26.2018.06.09

Success on x86_64-darwin (full log)
Attempted: exiv2
Partial log (click to expand)
rmdir: failed to remove '/nix/store/9dd89zh08f0synj21nk3lh7vpp963p6s-exiv2-0.26.2018.06.09/lib': Directory not empty
Patching '/nix/store/2f6zg73inm31kskafig630k625mrwdwl-exiv2-0.26.2018.06.09-dev/lib/pkgconfig/exiv2.pc' includedir to output /nix/store/2f6zg73inm31kskafig630k625mrwdwl-exiv2-0.26.2018.06.09-dev
gzipping man pages under /nix/store/9dd89zh08f0synj21nk3lh7vpp963p6s-exiv2-0.26.2018.06.09/share/man/
strip is /nix/store/qfxcr8c4fg7lkybrny9n2sb77bmippdx-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/9dd89zh08f0synj21nk3lh7vpp963p6s-exiv2-0.26.2018.06.09/lib  /nix/store/9dd89zh08f0synj21nk3lh7vpp963p6s-exiv2-0.26.2018.06.09/bin
patching script interpreter paths in /nix/store/9dd89zh08f0synj21nk3lh7vpp963p6s-exiv2-0.26.2018.06.09
strip is /nix/store/qfxcr8c4fg7lkybrny9n2sb77bmippdx-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/2f6zg73inm31kskafig630k625mrwdwl-exiv2-0.26.2018.06.09-dev/lib
patching script interpreter paths in /nix/store/2f6zg73inm31kskafig630k625mrwdwl-exiv2-0.26.2018.06.09-dev
/nix/store/9dd89zh08f0synj21nk3lh7vpp963p6s-exiv2-0.26.2018.06.09

Yeah, I know about that detail.

Success on x86_64-darwin (full log)
Attempted: exiv2
Partial log (click to expand)
/nix/store/9dd89zh08f0synj21nk3lh7vpp963p6s-exiv2-0.26.2018.06.09

Success on x86_64-linux (full log)
Attempted: exiv2
Partial log (click to expand)
/nix/store/dx9hgbkha2zmmz2mn1wjj2isshsa4qc0-exiv2-0.26.2018.06.09

Success on aarch64-linux (full log)
Attempted: exiv2
Partial log (click to expand)
strip is /nix/store/zrs21zqcchgyabjf4xfimncdq16njizc-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/wr9bifs9p3chc84b1f6vmdpr10avanxz-exiv2-0.26.2018.06.09/lib  /nix/store/wr9bifs9p3chc84b1f6vmdpr10avanxz-exiv2-0.26.2018.06.09/bin
patching script interpreter paths in /nix/store/wr9bifs9p3chc84b1f6vmdpr10avanxz-exiv2-0.26.2018.06.09
checking for references to /build in /nix/store/wr9bifs9p3chc84b1f6vmdpr10avanxz-exiv2-0.26.2018.06.09...
shrinking RPATHs of ELF executables and libraries in /nix/store/fzv7la79b7yly74c87j9fgf57879ayn7-exiv2-0.26.2018.06.09-dev
strip is /nix/store/zrs21zqcchgyabjf4xfimncdq16njizc-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/fzv7la79b7yly74c87j9fgf57879ayn7-exiv2-0.26.2018.06.09-dev/lib
patching script interpreter paths in /nix/store/fzv7la79b7yly74c87j9fgf57879ayn7-exiv2-0.26.2018.06.09-dev
checking for references to /build in /nix/store/fzv7la79b7yly74c87j9fgf57879ayn7-exiv2-0.26.2018.06.09-dev...
/nix/store/wr9bifs9p3chc84b1f6vmdpr10avanxz-exiv2-0.26.2018.06.09


Cut logging in release

Fixes #2237
PR checklist:

[x] make succeeds (on *nix)
[ ] make reftests succeeds
[ ] tested examples with the following backends:
[ ] rustfmt run on changed code

@grovesNL that sounds good.
bors r=grovesNL


Fix slow serialization of powerAssertContext

Given:
t.is(someObj.foo, 'bar');

The entirety of someObj is going to be serialized and sent across the wire. This is really problematic, since someObj can be arbitrarily large, and contain large loops.
In particular, test failures cause huge slowdowns if a Babel NodePath object is in the assertion.
I think the proper solution is to move powerAssert messaging back to the child processes, but that comes with a heavy dependency graph.
I know @twada is working hard on reducing that. Just let us know when it's ready so we can switch back.

@jamestalmage Sorry for my late response :bow:
I've started a new power-assert-runtime modules in a monorepo style.
First alpha version is already published and the first beta version will be released soon, maybe this weekend.


Vertical display not showing correctly

Same issue as here: https://github.com/MMM-CalendarExt2/MMM-CalendarExt2/issues/99
I also used the same method to rotate the screen:
body {
margin: 0;
position: absolute;
transform: rotate(-90deg);
transform-origin: top left;
width: 100vh;
height: 100vw;
object-fit: cover;
top: 100vh;
visibility: visible;
}
......so I can use VNC Viewer normally.
Their solution was to go with calendarext, but I prefer to stay with CalendarExt2.
 I thought rotating via CSS means the raspberry thinks the screen is still in horizontal mode, and then the magicmirror is rotated 90° in VNC.


Duplicate entries for one user

Username: Grieve4Nieve
Actual hiscore position: 64
Duplicate hiscore position: 113

Duplicate has been removed


[NavigationView] Setting AutoSuggestBox to x:Null in VisualState doesn't work

Describe the bug
When trying to hide the AutoSuggestBox of a NavigationView from a VisualState, it throws a COM exception.
Steps to reproduce the bug
NavigationViewIssue.zip
I linked a simple sample app to test it. It simply has a NavigationView containing only the suggest box, and 3 buttons to show what is the behaviour for 3 different scenario:
<VisualState x:Name="SuggestBoxChanged">
    <VisualState.Setters>
        <Setter Target="navigationBar.AutoSuggestBox">
            <Setter.Value>
                <AutoSuggestBox PlaceholderText="Changed!" QueryIcon="SolidStar" />
            </Setter.Value>
        </Setter>
    </VisualState.Setters>
</VisualState>
<VisualState x:Name="SuggestBoxNull">
    <VisualState.Setters>
        <Setter Target="navigationBar.AutoSuggestBox" Value="{x:Null}" />
    </VisualState.Setters>
</VisualState>

private void ChangeSuggestBox_Click(object sender, RoutedEventArgs e) => VisualStateManager.GoToState(this, SuggestBoxChanged.Name, false);

private void RemoveSuggestBox_Click(object sender, RoutedEventArgs e) => navigationBar.AutoSuggestBox = null;

private void RemoveSuggestBoxVisualState_Click(object sender, RoutedEventArgs e) => VisualStateManager.GoToState(this, SuggestBoxNull.Name, false);

We change the AutoSuggestBox from a VisualState => OK
We remove the SuggestBox from the code behind => OK
We remove the SuggestBox from a VisualState => FAIL

Expected behavior
What we expect here, is that all scenarios should work. We should be able to hide the suggest box froom the VisualState, as we do when we assign it a different SuggestBox.
Version Info

NuGet package version:
[Microsoft.UI.Xaml 2.4.2]

Windows 10 version
Saw the problem?

Insider Build (xxxxx)

November 2019 Update (18363)
Yes

May 2019 Update (18362)

October 2018 Update (17763)

April 2018 Update (17134)

Fall Creators Update (16299)

Creators Update (15063)

Device form factor
Saw the problem?

Desktop
Yes

Mobile

Xbox

Surface Hub

IoT

Additional context

Yes, if I set its Visibility to Collapsed, the textbox will be hidden but the presenter will still be present (as it appears when the AutoSuggestBox property is not null).
winrt::VisualStateManager::GoToState(*this, box ? L"AutoSuggestBoxVisible" : L"AutoSuggestBoxCollapsed", false /*useTransitions*/);

Hi @HunterB06 what particular scenario are you looking to drive here? We have x:Load which is used for showing and removing content on demand. Would this work in your scenario?

Hello,
We are looking to make the presenter disappear when the AutoSuggestBox is set to null, which is what it should do, as I detailed in my first message. The fact is that when we set it to null, the UI is not updated properly.
I don't know if x:Load would do the trick, but it should actually work without it. It should be fixed, not workarounded.

Hello,
We are looking to make the presenter disappear when the AutoSuggestBox is set to null, which is what it should do, as I detailed in my first message. The fact is that when we set it to null, the UI is not updated properly.
I don't know if x:Load would do the trick, but it should actually work without it. It should be fixed, not workarounded.
Thanks 🙂,

Thanks for the reply. I understand that this is not a fix for your issue, but in the meantime if this workaround is valid then it is worth highlighting while we track this issue. Unfortunately, we have no ETA for when a fix to this will come.

x:Load shouldn't do the trick since it will be applied to the AutoSuggestBox and would still leave the presenter displayed.
I've made a quick try in my app to see what happens. It throws a XamlParseException:

Failed to assign to property 'Microsoft.UI.Xaml.Controls.NavigationView.AutoSuggestBox'. [Line: 378 Position: 79]

Repro project using WinAppSDK: App9.zip
OptimizedStyle::ResolveValue needs to be updated to allow null to be assigned to properties whose type has the IsNullable flag.


Link points to old document

In the opening paragraph there are two links, one for delegates (that is pointed at the new docs.microsoft.com) and one for expression tree (which is pointed at the old MSDN site). I believe the expression tree link should be pointed to the newer documentation here: https://docs.microsoft.com/en-us/dotnet/csharp/programming-guide/concepts/expression-trees/

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 567260f8-0f94-b48c-8344-0d9780ab4448
Version Independent ID: 1bebe367-5b93-0b83-72f5-bdf31d65ef5d
Content: Lambda Expressions (C# Programming Guide)
Content Source: docs/csharp/programming-guide/statements-expressions-operators/lambda-expressions.md
Product: dotnet-csharp
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

You're right. It should go to that page instead.


Select package version on deploy

~## ⚠  Depends on https://github.com/dcos/dcos-ui/pull/2395~
Closes DCOS_OSS-1560 DCOS_OSS-1559 DCOS_OSS-1558
Checklist

[x] Did you add a JIRA issue in a commit message or as part of the branch name?
[x] Did you add new unit tests?
[ ] Did you add new integration tests?
[ ] If this is a regression, did you write a test to catch this in the future?

@weblancaster this likely needs a rebase.

retest this please

Alfred please retest this

retest this please

@MatApple please update you commit message to be inline with the contributing guidelines. Every commit messages should have a meaningful subject and the correct type. Messages like "edge cases" don't mean a thing.

The current implementation triggers a package describe loop resulting in many request. Please adjust!

@orlandohohmeier yes I will do - I pushed it last minute before I had to leave so I could follow up at home :) Will fix it up today.

Alfred retest this please

Alfred retest this please

Great Job! ⚒ 💯


Torch ngp training

Hi, nerf pytorch is really slow for training, and have you tried ngp on seven scenes?
Just wondering how would you adapt the poses? And did you get better results? Thanks.

Hi, yes instance-ngp works as well. And it's faster. I think the key insight of this paper is the proposed framework works with differentiable rendering.

I tried torch-ngp on 7scenes, it looks faster but got very blurry results. And i noticed that ngp is using different camera coordinate as Nerf. Would you like to share your idea on how to modify poses accordingly?
Many thanks:)

Hi, honestly, I have no idea cuz I didn't use the original torch-ngp code.
Maybe you could modify the PyTorch nerf with the reference code from torch-ngp or just ask the author?
Our coordinate system is the same as what original NeRF paper used.

Hi, thanks for the reply. I will try that later.
I'm closing the issue. Thanks.


[request] Test modules with 'and' without blib

For instance where modules don't work correctly/pass tests when compiled it would be useful to be able to test with panda but without the compiled units:
[nickl@db-server perl6-html-parser-xml]$ panda install XML::Query
==> Fetching XML::Query
==> Building XML::Query
Compiling lib/XML/Query/Results.pm6 to mbc
Compiling lib/XML/Query/Statement.pm6 to mbc
Compiling lib/XML/Query.pm6 to mbc
==> Testing XML::Query
t/basic.t .. Default constructor for 'Query' only takes named arguments

To check if this is the compiled unit's fault, you do:
[nickl@db-server perl6-html-parser-xml]$ panda look XML::Query
==> Fetching XML::Query
==> Entering /home/nickl/CFB/tmp/perl6-html-parser-xml/.panda-work/1416245028_1 with /bin/bash
[nickl@db-server<PHONE_NUMBER>_1]$ panda-test
t/basic.t .. ok
All tests successful.

If withp6lib was modified to include blib OR lib you could test each case individually. Furthermore another variable could be added to the reporter branch's json to report these types of failures as well as inform that user.

Panda doesn't precompile anymore, so that ceased to be a problem :)


Cannot use Globally Defined Python Environment While Inside Qtile

Describe the bug
This is almost the same problem as described in #167328, but it only occurs while inside Qtile.
Steps To Reproduce
Steps to reproduce the behavior:

Declaratively define a global Python environment in configuration.nix:

environment.systemPackages = with pkgs; [
 (let
    my-python-packages = python-packages: with python-packages; [
      matplotlib
      numpy
      pandas
      scikit-learn
    ];
    python-with-my-packages = python3.withPackages my-python-packages;
  in
    python-with-my-packages)
];

Test the Python environment

$ python -c "import pandas"

Enable Qtile:

services.xserver.windowManager = {
  qtile = {
    enable = true;
  };
};

Log into Qtile
Run the test again

$ python -c "import pandas"
Traceback (most recent call last):
 File "<string>", line 1, in <module>
ModuleNotFoundError: No module named 'pandas'

Expected behavior
I expected to be able to use the globally defined Python environment, regardless of which window manager I am currently using.
Notify maintainers
@jonringer @kamilchm
Metadata
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.15.59, NixOS, 22.05 (Quokka), 22.05.2361.45c9736ed69`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.8.1`
 - channels(root): `"home-manager-22.05.tar.gz, nixos-22.05"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

@arjan-s do you think it's possible now to go around this issue with the change you've made here? https://github.com/NixOS/nixpkgs/pull/220450

I have not tested this yet because I don't globally install Python packages and additionally I start qtile differently. But I can see whether I can make some time for it. If you could confirm whether this is still a problem with current nixos-unstable, that would be great.

I can confirm that it's still a problem. I found that the python's path is different in Qtile and tty, thus resulting in different environments. I think you can verify it by running realpath -e $(which python).

@arjan-s could you share your way of starting Qtile? If doing it your way would solve the issue it would be great :)

I'm afraid my setup has the same behaviour as yours, so that won't help...
Also I did a quick test with the solution from https://github.com/NixOS/nixpkgs/pull/146290, but that didn't work. Hopefully I can make some time soon to really try to resolve this.

Thank you so much for the involvement in this! At least now I know there might not be any way to go around it for now :)

Yeah it seems to be a widespread problem, if you look especially at the comments in the other issues.
Whether it's actually a problem depends, as it's just normal behaviour for child processes to inherit $PATH from their parent. But in this case, where Qtile is wrapped inside a separate Python environment, that means that Python occurs earlier in $PATH than the system Python.
I think you have a few options to explore:

Make your terminal emulator clean up your PATH like in https://github.com/NixOS/nixpkgs/issues/171972#issuecomment-1517324462 , although this probably doesn't work if you use something like VScode
Install the nonwrapped Qtile python package globally: add qtile and its dependencies to your my-python-packages, then remove services.xserver.windowManager.qtile and make your own session start script using services.xserver.windowManager.session (example here)
Add the system Python's site-packages directory to your $PYTHONPATH from something like .bashrc
Use something like Poetry so you can easily work with virtual environments instead of your system Python
Use su - <youruser> to get a shell with clean $PATH before using Python

Can someone share his qtile configuration? Would love to get it running under NixOS

@Rouzihiro the easiest way would be to use the qtile display manager session, for instance like this (copied from my own config):
security = {
  pam.services = {
    swaylock = { };
    login.enableGnomeKeyring = true;
  };
  polkit.enable = true;
};
hardware.graphics.enable = true;
xdg.portal = {
  enable = true;
  wlr.enable = lib.mkForce true;
  config.common.default = "wlr";
  extraPortals = [ pkgs.xdg-desktop-portal-gtk ];
};
programs = {
  xwayland.enable = true;
  dconf.enable = true;
  gdk-pixbuf.modulePackages = [ pkgs.librsvg ];
};
services = {
  displayManager.sddm.enable = true;
  xserver.windowManager.qtile = {
    enable = true;
    extraPackages = pp: with pp; [ qtile-extras ];
  };
};


Inconsistent behaviour of Value::parseValue with odd vs even number of consecutive delimiters

Initial investigation arises from the infinite loop fix in #390, which it turns out was fixed separately by #413.
I tried throwing the following at Value::parseValue:

/!;
//!;
///!;
////!;

The 1st and 3rd seemed fine, resulting in /`` and ///respectively.  The 2nd and 4th resulted inUnexpectedTokenException: Identifier expected. Got “!;”`.
In terms of the metaphor "low-hanging-fruit", this isn't even fruit.  I am just logging this so the information is captured, and that we can be mindful of the curious behaviour when revisiting that area of the code.

That's very strange (and unfruity) indeed …

Nikita Popov also reported some strange behavior he encountered using fuzzing. We'll want to check those some time (preferably after the 9.0 release - I'd like to do some big cleanups for 10.0): https://github.com/MyIntervals/PHP-CSS-Parser/issues/created_by/nikic


Annotation update/delete feature request

It would be nice to be able to update / delete annotations in the UI

I second this, how do I delete an annotation?

Hi!
When you create an annotation, it is saved in your influxDB database as a single data point.
From InfluxDB documentation:

Deletes are a rare occurrence. When they do occur it is almost always against large ranges of old data that are cold for writes.
Pro: Restricting access to deletes allows for increased query and write performance.
Con: Delete functionality is significantly restricted.

and this panel follow this design choice. When you store an annotation, you "register" an event that is surely happen.
If you want to be able to delete frequently your annotations, I suggest you to use a different datasource, e.g.: sql, or to implement by yourself a slow query like:
DELETE WHERE time =<PHONE_NUMBER>917952139
or to check new Grafana Annotation Support (build-in): link
At the moment, here, there is no support for this operation.
Greetings


How to enable HTTPS on the load balancer?

Hi Eng Hwa,
I'm trying to enable HTTPS on the load balancer. I tried your code by uncommented on...

Line 58-67
Line 90-92

...but it didn't work. Appreciate if you could provide a working example for that.
Additional information:
I'm using Cloudflare as nameserver with SSL setting sets to Full (strict) and I do not wish to point my domain to Route53. I want to keep using Cloudflare as nameserver for my domain and sub-domains.
I want https://example-app.my-domain.com to point to the Rails app's load balancer and my end users should see that URL in their web browser (not redirect).
Thanks.

Hi @zulhfreelancer .
The instructions in the README file have been updated on how to map a custom domain with SSL to an ALB, using the aws cli.
The ECS pattern ApplicationLoadBalancedFargateService used in ecs-fargate-stack.ts currently requires a hosted zone created in Route53.

Thanks @thecloudranger. Looks like we need to use AWS CLI here. Does it possible to automate this using CDK instead?
One more question regarding this code block:
aws acm request-certificate \
  --domain-name www.example.com \
  --subject-alternative-names "example.com" \
  --validation-method DNS

Let say I want to enable HTTPS on https://example-app.my-domain.com (note that it's a sub-domain), should I run the command above using example-app.my-domain.com OR my-domain.com?
Do I need to repeat the whole process for every sub-domains that I have?

u can create wildcard certs too in ACM, if u have alot of subdomain

Noted. Thanks.


⚠ Bofrost.de has degraded performance

In 9eb3074, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 2010 ms

Resolved: Bofrost.de performance has improved in 68c2ae8 after 31 minutes.


Suppress innocuous notice message

Connects to https://github.com/mapzen/vector-datasource/issues/691
@zerebubuth could you review please?

:+1:


WIP #321 play 2.6

WIP #321
I was able to compile sample project - it runs standalone, but clock request needs some more work.
I also managed to compile project code, although I am not sure if did it correctly, since test are failing, and running tomcat7 returns page as text - need some more time (or maybe someone's help) to figure out what is going on. :)
I left some // TODO #321 comments to be cleaned up, after successful migration. Most of them are about deleting commented code, which I left for comparison purposes.
PS
Any comments are more than welcome! :)

@jroper I was able to somehow plug your reactive-streams-servlet (not everywhere, though) in play2-war. Can you, please, check in some spare time?
Also, I don't know, why in result from futureResult.map { result => there is no Content-Type header, but I will have to figure it out later (at least if nobody knows out of the box... ;) ).

The content type and length are now properties of the HttpEntity (ie, the result body). You'll have to modify the code that looks for the content length too, assuming it's looking to the response headers to find it:
https://www.playframework.com/documentation/2.6.x/api/scala/index.html#play.api.http.HttpEntity

Thanks @jroper for your help. I appreciate it. :)
I was able to run sbt tomcat7 and hit http://localhost:8080/p2wsample/ - the page loads and the clock is ticking (although not every 100 millis - probably it buffers somewhere).
Sometimes I can see some logs about non-recycled responses and I have to figure out why they appear.
[info][org.codehaus.cargo.container.tomcat.Tomcat7xInstalledLocalContainer] INFO: Encountered a non-recycled response and recycled it forcedly.
[info][org.codehaus.cargo.container.tomcat.Tomcat7xInstalledLocalContainer] org.apache.catalina.connector.CoyoteAdapter$RecycleRequiredException
[info][org.codehaus.cargo.container.tomcat.Tomcat7xInstalledLocalContainer]     at org.apache.catalina.connector.CoyoteAdapter.checkRecycled(CoyoteAdapter.java:592)
[info][org.codehaus.cargo.container.tomcat.Tomcat7xInstalledLocalContainer]     at org.apache.coyote.http11.AbstractHttp11Processor.recycle(AbstractHttp11Processor.java:1814)
[info][org.codehaus.cargo.container.tomcat.Tomcat7xInstalledLocalContainer]     at org.apache.coyote.http11.Http11Protocol$Http11ConnectionHandler.release(Http11Protocol.java:140)
[info][org.codehaus.cargo.container.tomcat.Tomcat7xInstalledLocalContainer]     at org.apache.coyote.AbstractProtocol$AbstractConnectionHandler.process(AbstractProtocol.java:703)
[info][org.codehaus.cargo.container.tomcat.Tomcat7xInstalledLocalContainer]     at org.apache.tomcat.util.net.JIoEndpoint$SocketProcessor.run(JIoEndpoint.java:318)
[info][org.codehaus.cargo.container.tomcat.Tomcat7xInstalledLocalContainer]     at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
[info][org.codehaus.cargo.container.tomcat.Tomcat7xInstalledLocalContainer]     at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
[info][org.codehaus.cargo.container.tomcat.Tomcat7xInstalledLocalContainer]     at org.apache.tomcat.util.threads.TaskThread$WrappingRunnable.run(TaskThread.java:61)
[info][org.codehaus.cargo.container.tomcat.Tomcat7xInstalledLocalContainer]     at java.lang.Thread.run(Thread.java:748)

The next steps are to run sbt itt-tomcat7 - but it fails at this moment and produces some NPEs like:
[info][org.codehaus.cargo.container.tomcat.Tomcat7xInstalledLocalContainer] SEVERE: null
[info][org.codehaus.cargo.container.tomcat.Tomcat7xInstalledLocalContainer] java.lang.NullPointerException
[info][org.codehaus.cargo.container.tomcat.Tomcat7xInstalledLocalContainer]     at org.apache.tomcat.util.http.MimeHeaders.setValue(MimeHeaders.java:304)
[info][org.codehaus.cargo.container.tomcat.Tomcat7xInstalledLocalContainer]     at org.apache.coyote.http11.AbstractHttp11Processor.prepareResponse(AbstractHttp11Processor.java:1485)
[info][org.codehaus.cargo.container.tomcat.Tomcat7xInstalledLocalContainer]     at org.apache.coyote.http11.AbstractHttp11Processor.action(AbstractHttp11Processor.java:776)
[info][org.codehaus.cargo.container.tomcat.Tomcat7xInstalledLocalContainer]     at org.apache.coyote.Response.action(Response.java:174)
[info][org.codehaus.cargo.container.tomcat.Tomcat7xInstalledLocalContainer]     at org.apache.coyote.Response.sendHeaders(Response.java:348)
[info][org.codehaus.cargo.container.tomcat.Tomcat7xInstalledLocalContainer]     at org.apache.catalina.connector.OutputBuffer.doFlush(OutputBuffer.java:349)
[info][org.codehaus.cargo.container.tomcat.Tomcat7xInstalledLocalContainer]     at org.apache.catalina.connector.OutputBuffer.close(OutputBuffer.java:308)
[info][org.codehaus.cargo.container.tomcat.Tomcat7xInstalledLocalContainer]     at org.apache.catalina.connector.Response.finishResponse(Response.java:553)
[info][org.codehaus.cargo.container.tomcat.Tomcat7xInstalledLocalContainer]     at org.apache.catalina.connector.CoyoteAdapter.asyncDispatch(CoyoteAdapter.java:335)
[info][org.codehaus.cargo.container.tomcat.Tomcat7xInstalledLocalContainer]     at org.apache.coyote.http11.AbstractHttp11Processor.asyncDispatch(AbstractHttp11Processor.java:1674)
[info][org.codehaus.cargo.container.tomcat.Tomcat7xInstalledLocalContainer]     at org.apache.coyote.AbstractProtocol$AbstractConnectionHandler.process(AbstractProtocol.java:604)
[info][org.codehaus.cargo.container.tomcat.Tomcat7xInstalledLocalContainer]     at org.apache.tomcat.util.net.JIoEndpoint$SocketProcessor.run(JIoEndpoint.java:318)
[info][org.codehaus.cargo.container.tomcat.Tomcat7xInstalledLocalContainer]     at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
[info][org.codehaus.cargo.container.tomcat.Tomcat7xInstalledLocalContainer]     at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
[info][org.codehaus.cargo.container.tomcat.Tomcat7xInstalledLocalContainer]     at org.apache.tomcat.util.threads.TaskThread$WrappingRunnable.run(TaskThread.java:61)
[info][org.codehaus.cargo.container.tomcat.Tomcat7xInstalledLocalContainer]     at java.lang.Thread.run(Thread.java:748)

and some timeouts:
[info]   java.net.SocketTimeoutException: Read timed out
[info]   at java.net.SocketInputStream.socketRead0(Native Method)
[info]   at java.net.SocketInputStream.socketRead(SocketInputStream.java:116)
[info]   at java.net.SocketInputStream.read(SocketInputStream.java:171)
[info]   at java.net.SocketInputStream.read(SocketInputStream.java:141)
[info]   at org.apache.http.impl.io.AbstractSessionInputBuffer.fillBuffer(AbstractSessionInputBuffer.java:160)
[info]   at org.apache.http.impl.io.SocketInputBuffer.fillBuffer(SocketInputBuffer.java:84)
[info]   at org.apache.http.impl.io.AbstractSessionInputBuffer.readLine(AbstractSessionInputBuffer.java:273)
[info]   at org.apache.http.impl.conn.DefaultHttpResponseParser.parseHead(DefaultHttpResponseParser.java:140)
[info]   at org.apache.http.impl.conn.DefaultHttpResponseParser.parseHead(DefaultHttpResponseParser.java:57)
[info]   at org.apache.http.impl.io.AbstractMessageParser.parse(AbstractMessageParser.java:260)

I am trying to figure out what is happening right now, but if you @jroper (or anyone else involved in this project) have any hints, I'll take them all. ;)

Can you rebase on or merge play2war:develop to get your last changes on this branch? Thx!

done :)

@pwyczes which servlet version do you need?
We could maybe start only with the servlet 3.1 version and use https://github.com/jroper/reactive-streams-servlet more deeply.

Hi @yanns ,
I think, I am going to have some time to go back to this topic this (and maybe next) weekend - could you, please, find like 10 minutes to review my previous changes and maybe give me a small hint how to move forward? :)
I would really appreciate that.
No pressure, though. :)

Finally, I found some time to return to that topic.
I figured out, that those NPEs come from some concurrency issues (https://bz.apache.org/bugzilla/show_bug.cgi?id=47655#c9), since MimeHeaders is not thread safe.
I returned to using play.api.libs.streams.Execution.Implicits.trampoline (yes, I am aware of https://github.com/play2war/play2-war-plugin/issues/223) but this still did not help.
When I make one single request, like curl -i http://localhost:8080/p2wsample/sub/subdir then everything looks fine, but once I start sbt itt-tomcat7 it fails randomly with unrecognized HTTP headers like Invalid header: :21:51 GMT, or Invalid header: X-Permitted-Cross-Domain-Policiesferrer-Polimaster-only. This also suggests those concurrency issues.
I have limited experience with akka-streams, so any suggestion where to look at is more than welcome. :)

Hi @pwyczes , I gave you direct access to https://github.com/play2war/play2-war-plugin.
I guess that you're the only chance for this project to survive 😉
I can propose you the following: you can push your current PR into a new branch directly under the repo, so that we can collaboratively work on it. WDYT?

Thank you @yanns :)
I will push this PR as you suggested. :)

@yanns I merged my PR to branch 321_play_2_6.


Added BootstrapperOutputDir to config

Attempt at fixing the AV related performance issue mentioned in #3661 .
This is done by allowing OutputDir to be set in AppSettings as well, so it can be included as a permanent setting for a repository.
Since magic mode recommends using paket.exe.config for AppSettings, I've renamed the setting to BootstrapperOutputDir in AppSettings, so it is clear that the output dir relates to the Bootstrapper only, and keeps some naming overlap from the same option provided to command line already.
I've also added testing and minor doc updates.
Fixes #3661

Hmm, I've run all the failing tests locally, without error. Can anyone see why the buildenvironment behaves differently? ( @matthid , @forki ? )

Nvm, figured it out. Not used to unix filesystem.. :)

I think the advice should be that you configure a place that you can exclude from the antivirus check

Sure, but why not exclude something in the temporary directory?

I dunno. but I assume a gazillion of programs want to write there. So using a folder that only paket's config knows might be safer!?

Just make a directory within the temp directory, this one can be excluded and is paket only?

Well the issue then becomes a matter of rights, and targeted attempts. This will then be known vulnerability, and having write access to temp, can then circumvent AV by just writing to that folder. When it is configureable, it is not as simple to use it for direct attacks. :) - but yeah, would be a lot simpler to just exclude that folder, but not always as easy in enterprisy setups.. ;)

I think with the docs and typo fixes we can take it.

new bootstrapper is released. thanks


Removed GC optimization to fix #236

Note that package installation was creating default values for those options, so this has been removed as well to keep coherence.

This pull request is now managed with CodeReviewHub. Refresh.


Refactoring Merkle Mask

On the way to https://github.com/MinaProtocol/mina/issues/14669, make the library under test a little bit cleaner.
Changes include:

adding interfaces (mli) files
add most warnings back in
thanks the 2 first steps, remove dead code

Also:

sort library names alphabetically in dune
50de7034a6 uses a built-in string format specifier to achieve the same goal

@rbonichon do you think it's ready to be merged?

!ci-build-me


Add commands needed for my Ubuntu Trusty box.

Tested with docker, which needs 2 more:
apt-get update
apt-get install sudo git
I don't know where build-essential came from, but we have to draw the line somewhere.

Hum, that detailed for a specific platform seemed to be a slippery slope, but I'm fine with it.

I'm okay with this for now.


Taking a screenshot from a Windows Forms control return HTTP 500

I can take a screenshot from a Windows Forms form but if I try to take a screenshot from a child control (e.g. button or edit) I will get HTTP 500
If I do the same stuff in WPF application it works.
==========================================
GET /session/EDB053E6-9F3C-418E-8BA1-9F3C53F094B1/screenshot/42.591710 HTTP/1.1
Accept: application/json, image/png
Cache-Control: no-cache
Host: <IP_ADDRESS>:4723
User-Agent: selenium/3.11.0 (.net windows)
HTTP/1.1 500 Internal Error
Content-Length: 133
Content-Type: application/json
{"status":13,"value":{"error":"unknown error","message":"An unknown error occurred in the remote end while processing the command."}}

If I look at WebDriver specification the parameters seems to be incorrect
WebDriver spec
Should it be
GET /session/EDB053E6-9F3C-418E-8BA1-9F3C53F094B1/element/42.591710/screenshot
I tried that too but did not work.

Any news about this?

I'm experiencing the same problem. My test code:
[TestMethod]
public void TestMethod1()
{
    WindowsElement start_button = session.FindElementByAccessibilityId("50014");
    OpenQA.Selenium.Screenshot screenshot = start_button.GetScreenshot();
    screenshot.SaveAsFile("Test.png");
}

And it fails at GetScreenshot(). The corresponding message in WinAppDriver:
GET /session/22178AD8-4312-4905-A845-2AC2C0CBD743/screenshot/42.1117372 HTTP/1.1
Accept: application/json, image/png
Cache-Control: no-cache
Host: <IP_ADDRESS>:4723
HTTP/1.1 500 Internal Error
Content-Length: 133
Content-Type: application/json
{"status":13,"value":{"error":"unknown error","message":"An unknown error occurred in the remote end while processing the command."}}
As stated above by @jaska45 this doesn't seem to be the right command. This should be solved by issue #261 in version 1.0 RC2 but I'm still experiencing the problem. Am I doing something wrong or is there still a bug? Would appreciate it if somebody can help.
Version information:
WinAppDriver: v1.1(.1809.18001)
Appium.Webdriver: v<IP_ADDRESS>
Castle.Core: v4.3.1
MSTest.TestAdapter: v1.4.0
MSTest.TestFramework: v1.4.0
Newtonsoft.Json: v12.0.1
Selenium.Support: v3.10.0

I had the same issue when attempting to pass any AppiumWebElement that returned from any location method. I was able to capture an image of the application by passing the instance of the driver to this method as a work around and that worked.

I had the same issue when attempting to pass any AppiumWebElement that returned from any location method. I was able to capture an image of the application by passing the instance of the driver to this method as a work around and that worked.

Yes, that also works for me. Did you also find a way to automatically crop the screenshot?

I had the same issue when attempting to pass any AppiumWebElement that returned from any location method. I was able to capture an image of the application by passing the instance of the driver to this method as a work around and that worked.

Yes, that also works for me. Did you also find a way to automatically crop the screenshot?

Ok, I found a way. Don't know if it is the most efficient way but it works (I'm quite new to C#). I'm sharing the code here in case somebody stumbles on the same issue in the future.
using Microsoft.VisualStudio.TestTools.UnitTesting;
using OpenQA.Selenium.Appium.Windows;
using System.Drawing;

[TestMethod]
public void TestMethod1()
{
      WindowsElement start_button = session.FindElementByAccessibilityId("50014");
      OpenQA.Selenium.Screenshot screenshot_full_screen = session.GetScreenshot();

      Rectangle crop_rect = new Rectangle(start_button.Location, start_button.Size);
      Bitmap image_full_screen = Image.FromStream(new System.IO.MemoryStream(screenshot_full_screen.AsByteArray)) as Bitmap;
      Bitmap image_element = new Bitmap(image_full_screen);
      image_element = image_full_screen.Clone(crop_rect, image_full_screen.PixelFormat);

      image_element.Save("Test.png");
}

Nice solution,
Don't forget to dispose your bitmaps.
Thx


libsodium example

https://github.com/foundpatterns/torchbear/tree/master/src/lua_bindings/crypto
Example: design contest where voters' keys have witnesses and profiles connected.  profiles have reviews connected.  designs are images from a profile and have connected votes.
https://github.com/foundpatterns/lighttouch-keys

File Witness


Support API publish for Multiple environments

API needs to publish to multiple environments using API publisher.

There are few ways to publish to different environments, by default you have a production/sandbox gateway. You can add more gateways and publish your API to it, so you can deploy a gateway for production and another different for sandbox ( https://docs.wso2.com/display/AM210/Publish+through+Multiple+API+Gateways ).
Another approach is to manage the api creation out of API Publisher and deploy it with apimcli tool ( https://wso2.com/library/articles/2019/07/building-a-cicd-pipeline-for-apis/?utm_source=linkedin&utm_medium=organicpost&utm_campaign=liorgarti_aug19_api ) .
Also, if you are looking to publish the API to different stores for different consumers you should read ( https://docs.wso2.com/display/AM210/Publish+to+Multiple+External+API+Stores ).
If your are looking to publish the api to different stores

Thanks @falconmfm
Previous API Manager versions had this support through API manager Publisher. We could select multiple gateway environments and published through Publisher.

But upcoming API manager version 3.0.0 does not have this support yet. Needs to add to API Manager 3.0.0 as well.


Merge sync a82e284

Summary
Sync to latest upstream #a82e284

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 8 committers have signed the CLA.:white_check_mark: dkstyle:x: snitin315:x: draculapile:x: apathetic:x: vankop:x: chenxsan:x: Etheryen:x: dimaaan21You have signed the CLA already but the status is still pending? Let us recheck it.


Fixed water quality mass balance issue (#160)

The water quality solver was re-written to fix the mass balance issue #160. See the updated ReleaseNotes2_2.md for more details. Also the code that runs a concurrent hydraulic/quality analysis was simplified and tested using this example:
// Test of concurrent WQ analysis
#include "epanet2.h"
#include <stdio.h>
int main(int argc, char *argv[])
{
    EN_ProjectHandle ph;
    long t, tstep;
    EN_createproject(&ph);
    EN_open(ph, "net1.inp", "net1a.rpt", "");
    EN_openH(ph);
    EN_initH(ph, 1);
    EN_openQ(ph);
    EN_initQ(ph, 1);
    do {
        EN_runH(ph, &t);
        EN_runQ(ph, &t);
        EN_nextH(ph, &tstep);
        EN_nextQ(ph, &tstep);
    } while (tstep > 0);
    EN_closeH(ph);
    EN_closeQ(ph);
    EN_report(ph);
    EN_close(ph);
}

Thank you @LRossman. In the test you made above, did you compare the results to the full hydraulic and quality runs?

@LRossman Nice work!

@eladsal yes, I compared concurrent Net1.inp results against sequential results. There were some small differences in the flow values (like in the 5th - 6th digit) due to the fact that concurrent analysis reports the double floating point value of flow while sequential reads the saved single precision flow from the hydraulics file and then casts it to a double, thus losing some precision. The water quality results were the same when reported to two decimal places.

In qualroute.c, line 490, there is an extra semicolon which is causing compiling issues for me :)

@LRossman I will be making some commits to your fork to update the regression testing benchmark.

I'm not sure it's necessary but I just wanted to comment on the 4 networks that failed the CI checks.
Example1.inp (aka Net1.inp): Over the 24 hr simulation period, one node showed a concentration difference of 0.17 mg/L with the benchmark in just one time period. Otherwise for all nodes in all periods differences were no greater than 0.01 mg/L which was also the quality tolerance used in the analysis.
Sampletown.inp: The only differences were for the Newport Tank, which can't be discerned when plotted:

Example3.inp (aka Net3.inp): There were some noticeable differences in the Lake trace for this network for a subset of nodes that varied with the time period. A comparison for Node 193 (in the center of the network) looks as follows:

The comparison for Node 239 (at the bottom of the network) is shown below:

Simplenet.inp: there were no differences so I'm not sure why it failed.
Conclusion: Because it changes the order in which the nodes of a network are processed, the revised quality routine is expected to show some differences in results from the existing one. Noticeable differences only seem to appear for a small set of nodes at just a few reporting periods. Consistent, systematic differences over the entire network are not occurring, plus the revised routine is producing an error-free mass balance in every network tested so far. I believe that these findings are sufficient to adopt the updated water quality routine as the new benchmark.

Interestingly, I'm not able to reproduce these failures on my local machine. I updated the VS install and switched to 32 Bit Python to no avail.

@LRossman I'm concerned about the "example3" results. When I diff the report files there are some major differences occurring. What do you attribute this to. Do you have any other water age examples that we can look at?

@michaeltryby I think you meant to say "source tracing" and not "water age" since that is the quality analysis being made in Example3.
In my previous comment I tried to show that the differences for this network occur intermittently at just a few time points. Otherwise the time trajectories match very well. Did you find a node for which this is not the case? We can always look at more networks, but depending on their layout and flow solutions I assume we will still find some that show differences here and there. I would be more concerned if the new implementation was showing consistently different quality trajectories for a significant number of nodes.
Unlike for hydraulics where the mathematical accuracy of the solution can always be tested, there is no way to do the same for water quality (except for trivial examples). Thus you can't say that the 2.0 implementation of the LTD method is "more correct" than the new one. The new one does produce perfect mass balances (it seems like all the time) while the old one doesn't (please read the table in the Improved Water Quality Mass Balance section of the updated Release Notes included with this commit).
Because the new LTD implementation is processing the nodes (and their connected links) from "upstream" to "downstream" order, the mass inflow that a node sees already reflects the transport that occurred for nodes and links upstream of it. This is critically important for nodes at the end of zero-length links (like pumps and valves). It allows the proper amount of mass to be transferred into them, unlike in the current implementation where the mass transferred is based on whatever the upstream node's quality was at the start of the time step. The new LTD implementation is similar in spirit to the way that Kinematic Wave routing is performed in SWMM, where the links have to be processed from upstream to downstream for the results to make physical sense.

Just another note about Example3.net. When run with a 1 sec quality time step there are no longer any differences between results from the current dev version and the new commit (except for Tank 1 at hour 5: 0.92% v. 0.93%).

@LRossman I built the current epanet dev branch and I built this PR. I changed example3 to a 1 sec WQ time step and ran both versions. You can see from the attached comparison of the binary files that there are more differences than just Tank 1 at hour 5.
When examining the report files, however, once the initial conditions dissipate the results are within reasonable agreement. I will go ahead and update the benchmark and merge this most excellent PR! Great work!
report-diff.txt

@michaeltryby the diffs you report have me concerned if we are on the same page. Attached are the two EPANET Report files for Example3 at 1 sec time step that I generated: (1) is for the current dev branch build (before your new merge) and (2) is for my new commit with the modified quality routine. My diff tool only shows the one difference I mentioned previously. Does this merit further investigation?
example3-rpt.zip

@LRossman How are you building epanet?

@LRossman I have attached the example3 report files I'm looking at.
example3-tryby.zip

@michaeltryby mystery solved. I believe you were using a 0:01 (one minute) time step in your comparison instead of 0:00:01 (one second). When I ran my builds with the one minute time step the results exactly matched the report files you provided above. If you change to the 0:00:01 time step I believe you will only see one very small (0.01) diff as I noted earlier.


macOS libzstd-jni-*.dylib files are not signed and fails notarization

If this library is included on a macOS app (or maybe other Apple archive types that requires notarization) it makes the Apple Notarization process to fail because the libzstd-jni-*.dylib files are not signed (and not timestamped).
Example of an Notarization log from Apple:
{
  "logFormatVersion": 1,
  "jobId": "[REDACTED]",
  "status": "Invalid",
  "statusSummary": "Archive contains critical validation errors",
  "statusCode": 4000,
  "archiveFilename": "[REDACTED]",
  "uploadDate": "2023-10-12T12:21:27Z",
  "sha256": "[REDACTED]",
  "ticketContents": null,
  "issues": [
    {
      "severity": "error",
      "code": null,
      "path": "[REDACTED]/darwin/aarch64/libzstd-jni-1.5.5-5.dylib",
      "message": "The binary is not signed with a valid Developer ID certificate.",
      "docUrl": "https://developer.apple.com/documentation/security/notarizing_macos_software_before_distribution/resolving_common_notarization_issues#3087721",
      "architecture": "arm64"
    },
    {
      "severity": "error",
      "code": null,
      "path": "[REDACTED]/darwin/aarch64/libzstd-jni-1.5.5-5.dylib",
      "message": "The signature does not include a secure timestamp.",
      "docUrl": "https://developer.apple.com/documentation/security/notarizing_macos_software_before_distribution/resolving_common_notarization_issues#3087733",
      "architecture": "arm64"
    },
    {
      "severity": "error",
      "code": null,
      "path": "[REDACTED]/darwin/x86_64/libzstd-jni-1.5.5-5.dylib",
      "message": "The binary is not signed.",
      "docUrl": "https://developer.apple.com/documentation/security/notarizing_macos_software_before_distribution/resolving_common_notarization_issues#3087721",
      "architecture": "x86_64"
    },
    {
      "severity": "error",
      "code": null,
      "path": "[REDACTED]/darwin/x86_64/libzstd-jni-1.5.5-5.dylib",
      "message": "The signature does not include a secure timestamp.",
      "docUrl": "https://developer.apple.com/documentation/security/notarizing_macos_software_before_distribution/resolving_common_notarization_issues#3087733",
      "architecture": "x86_64"
    }
  ]
}

I am not interested in spending with Apple $99 per year for the privilege of signing the releases of a library.
If somebody is distributing the binary with an end-user application, they can sign the JAR themselves. In that case I also recommend to repackage it to remove the Windows and Linux binaries from it.


(PUP-10719) Collect gem directories from Gem::StubSpecifications

Puppet calls Puppet::Util::RubyGems::Source#directories to get a list of
candidate paths for the autoloader, so that we can load types, features, etc
from gems, see commit 2a12fdd5c. However, Gem::Specification.latest_specs
parses the full gemspec, returning Gem::Specification objects.
The Gem::Specification.stubs method only parses the "stub line" in the
gemspec, for example:
# -*- encoding: utf-8 -*-
# stub: bigdecimal 1.3.4 ruby lib

And returns Gem::StubSpecification objects. Calling the stubs method is
about 6.7 times faster than latest_specs when running from packages, i.e. not
bundler.
The stubs method also preserves the behavior of returning pre-release gems, as
the pre-release filtering happens when converting stubs to full specs.
Doing so also fixes a strange puppet/rubygems bug whereby Kernel.require can
fail to load a gem, but rubygems will load and cache stubs. If the gemspec is
later moved, but Gem.clear_paths is not called, then rubygems will fail to
convert the stub to a full spec. See PUP-10719 for details.
$ cat micro.rb
require 'benchmark/ips'

Benchmark.ips do |x|
  x.report("full specs") do
    Gem.clear_paths
    Gem::Specification.latest_specs(true)
  end

  x.report("stub specs") do
    Gem.clear_paths
    Gem::Specification.stubs
  end

  x.compare!
end

$ /opt/puppetlabs/puppet/bin/ruby micro.rb
Warming up --------------------------------------
          full specs     7.000  i/100ms
          stub specs    48.000  i/100ms
Calculating -------------------------------------
          full specs     72.956  (± 1.4%) i/s -    371.000  in   5.085961s
          stub specs    491.231  (± 0.8%) i/s -      2.496k in   5.081461s

Comparison:
          stub specs:      491.2 i/s
          full specs:       73.0 i/s - 6.73x  (± 0.00) slower

CLA signed by all contributors.

jenkins please test this with servertests


Ignore localPreferredWidth if minimumWidth is larger.

In AccordionBox review https://github.com/phetsims/sun/issues/803 we found that the following code might be buggy if someone passes in a smaller localPreferredWidth than the minimumWidth of the content is. We decided it would be best to address this in all sizable component cases by using Math.max
const preferredWidth: number = this.accordionBox.localPreferredWidth === null ? minimumWidth : this.accordionBox.localPreferredWidth;
    const preferredHeight: number = this.accordionBox.localPreferredHeight === null ? minimumHeight : this.accordionBox.localPreferredHeight;

Should be handled by the above commits! Thanks


Import posts

add posts and import them, openstates/openstates#2708

Pull Request Test Coverage Report for Build 138

75 of 82 (91.46%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.0%) to 67.913%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

scripts/utils.py
14
17
82.35%

scripts/to_database.py
34
38
89.47%

Totals

Change from base Build 132:
1.0%

Covered Lines:
908

Relevant Lines:
1337

💛  - Coveralls

al updated House posts
dc updated Council of the District of Columbia posts
ga updated House posts
id updated House posts
ma updated House posts
md updated House posts
mo updated House posts
nd updated House posts
nh updated House posts
nj updated Assembly posts
ok updated House posts
pr updated Senate posts
pr updated House posts
sd updated House posts
ut updated House posts
wa updated House posts
wv updated Senate posts
wv updated House posts


add new statistics methods and band names in ImageData object

This PR will bring some important change to the statistics method in rio-tiler

deprecate stats method
addstatistics method
add new BandStatistics model (replace ImageStatistics)
add band_names in ImageData class (statistics can be get for expression or indexes so we need a way to forward the band information from the ImageData object to the statistics method)
statistics method is only able to return stats for the full dataset (no bounds nor feature)

To Do

[ ] validate ☝ changes
[x] update base classes (deprecate stats and add statistics)
[x] update tests
[ ] update documentation (wil do in another PR)
[ ] start docs for v2 to v3 (wil do in another PR)

This is ready for review.
Main changes to review are:

new statistics class and method (matching the one we use in TiTiler). The statistics method only return stats for the whole dataset (using the preview method internally to reduce data transfer). In the documentation I'll add notes to explain how to extend the class and create custom statistics methods.

add band_names in ImageData. names are string values in form of:

COGReader: ["1", "2", "3"...] (string version of internal rasterio index names)
MultiBaseReader: ["{asset}_1", "{asset}_2", "{asset1}_3"] (asset name + string version of internal rasterio index names)
MultiBandReader: ["{band1}", "{band2}", "{band3}"]. Note we could have add index name but by definition MultiBandReader should be use for per file band data, so there should be only 1 index.

cc @kylebarron @geospatial-jeff


brod doesn't handle new partitions added to a topic via kafka-topics --alter command

Producer
ok = :brod.produce_sync(:brod_client, "dummy-data", partition_fun(key), key, value)

The partition_fun callback doesn't get the updated partition count. Calling :brod_client.get_metadata(:brod_client, :undefined) seems to update the internal cache which results in the callback getting the updated partition count, but the relevant producer for the new partitions are not started. Any attempts to write to the new partition results in {:error, {:producer_not_found, topic, partition}}. Note: we run with auto_start_producers: true
Consumer Group
I was expecting a reassignment of partitions within the group subscribers, but the addition of new partition doesn't seem to have any effect on the consumer group. The lag keeps on increasing on the new partitions.
Both of the above issues go away when we restart the service.

I noticed the same behavior. Any updates on this?

Hi! Sorry for silence. We are aware of this problem (or rather two separate problems), but we haven't got any good solution yet, other than client restart workaround.
It's definitely something to be fixed.


Call generator within another generator

Let's say I have the following generators:

module
model
view
controller

I want the module generator to internally call model, view and controller generators.
Is there an API method to achieve this?
Cheers.

currently there is no way to run a generator outside of running the CLI. Are you looking to trigger the entire prompts => actions process, or just the actions?

Just the actions. I know that I can declare the function and call them, like this:
plop.setGenerator('model', {
  description: 'app model',
  prompts: [...],
  actions: [ generateModel ]
});

plop.setGenerator('module', {
  description: 'app module',
  prompts: [...],
  actions: [ generateModule, generateModel, generateController ]
});

But i find that approach a little verbose, what if I have separate generator files? (Ex module.js, model.js), I will have to export the functions, etc.
It will be fine an API method like this:
plop.callGenerator('generatorName', { });

you could use the plop.getGenerator(name) api method to get the generator config object, then concat the actions array from there. But yeah, I see what you mean. I'm going to be working on exposing something like this for issue #22. So once that gets resolved it would probably work for this.

Excellent! You can close the issue then.

just pushed up the first pre-release version of node-plop, please have a look and let me know if you run into any issues 😄
https://github.com/amwmedia/node-plop
it's also up on npm
https://www.npmjs.com/package/node-plop


Prevent env var relaunch from extension owned terminals

We don't want to relaunch these because then I believe the replacement terminal would no longer be owned by the extension.

To clarify this issue, this is specifically what we're trying to help: The SSH extension was having its terminal relaunched which killed the connection, at which point it needed to relaunch the terminal itself, slowing the initial connection time.
As for reconnection, I think we want to keep that for extension owned terminals as that would be beneficial to extensions like:

https://marketplace.visualstudio.com/items?itemName=Tyriar.shell-launcher
https://marketplace.visualstudio.com/items?itemName=fabiospampinato.vscode-terminals

I'm sure there are cases where we may not want to reconnect to some createTerminal terminals, but it's probably in the minority and extensions can always query the terminals object to see if it owned terminals in the previous session. If this does end up being a problem for several extensions we could look into adding a new TerminalOptions API that opts out of reconnection.

It looks like PowerShell is failing hard from this because they launch a terminal on startup by default, and when it gets relaunched it never attempts to restart it https://github.com/PowerShell/vscode-powershell/issues/3227. We should bring this into the recovery build

Ah, I wondered what was going on. I'm still getting my feet wet with all this, thanks for tracking this down @Tyriar.

Verified by installing the PowerShell extension, and the terminal sample extension. Opening a PS file to launch the PS integrated console, and then triggering update environment to ensure thee warning shows up and not an automatic relaunch.


Add toggle for hiding vote counts

Codecov Report
Patch coverage is 8.33% of modified lines.

Files Changed
Coverage

election/settings/base.py
ø

vaalikoppi/forms.py
ø

vaalikoppi/views/admin/votings.py
0.00%

vaalikoppi/views/votings.py
0.00%

vaalikoppi/models.py
100.00%

:loudspeaker: Thoughts on this report? Let us know!.


Questions for iterable variables and changing variables

Hello!
I have a few simple questions about using the process.
I'm a bit confused about 'which variables are iterable and which are not' and also  'which variables are changing and which are not'. I understand that only the variables listed in lablxc (about 172 in total) can change during the code execution. However, I'm encountering a situation that I don't quite understand.
For instance, the input values ptrepv and ptripv (transport power per volume) and pradpv, pcoreradpv (radiation power per volume) are initialized to 0.0D0 in physics_variables.f90. This would mean that the total power (the sum of transport power and radiation power) should be zero, even when icc=2 is turned on.
From this, I infer that the variables listed in lablxc are used for iteration, while other variables in physics_variables.f90 are updated according to the iterating variables.
Could you please confirm if my understanding is correct? If it is, could you also explain how I can check the results of the variables listed in physics_variables.f90? (I couldn't find the result of ptrepv, ptripv, pradpv, pcoreradpv in MFILE)
Thank you!

Hi JaeBeom,
You are correct that any variable that is not an iteration variable can be updated by the code in the physics and engineering routines.  These variables are usually initialised to zero.  The iteration variables are only updated by the optimiser.
Not every variable is output - some are just intermediate quantities.  For example, ptrepv is multiplied by the plasma volume to obtain the total electron transport power ptremw, which is then output:
Electron transport (MW)                                    (ptremw)                  1.593E+02  OP

When looking at the results it is best to read through the OUT.DAT first, searching for relevant terms in English.  Looking for variables that you have seen in the code can be frustrating as they can be passed as an argument into a function (or subroutine) in which they have a different name, or used as an intermediate as in the example above.

Thanks for your reply!
.
Let me ask a one more question about constraint_equation.f90.
line 1986 in constraint_equation.f90:
!! q : safety factor 'near' plasma edge: equal to q95
I cannot understand that line because q is not same with q95 in OUT.DAT
.
And I also found some wrong descriptions about variables in constraint_equation.f90.
This issue is quite trivial but I hope it would be helpful for your code!
line 1103 in constraint_equations.f90:
!! mvalim : input real : TF coil outboard leg resistive power (MW)
should be fixed into
!! mvalim : input real : maximum TF coil MVA limit (MW)
line 1136 in constraint_equation.f90:
!! rtanbeam : input real : ratio of collision length / connection length
should be fixed into
!! rtanbeam : input real : neutral beam centreline tangency radius (m)

I am afraid the q notation is not very consistent in the code.  The OUT.DAT, however, is consistent as far as I know.  Three different q values are output:
 Safety factor on axis                                     (q0)                          2.830
 Safety factor at 95% flux surface                         (q95)                         5.816  ITV
 Cylindrical safety factor (qcyl)                          (qstar)                       3.461  OP

Thanks for pointing out the copy-paste errors in constraint_equations.f90.  I will put them into a new issue.

Thank you for your comment!


Validator::required() failed maybe to often

What about these cases for unit tests:

"" => false
0 => true
0.0 => true
"0" => true
NULL => false
FALSE => false
array() => thr InvalidArgumentException?

https://github.com/arkuuu/simple-validator/blob/dea761c93cb4dcf46ae44c3d1f3640588d27707a/src/Validator.php#L16

Thanks, there are several tests missing at the moment. I'll add them as soon as I find time.

0 => true
0.0 => true
"0" => true

You think those should return true, meaning they are valid values when "not empty" is required?

array() => thr InvalidArgumentException?

Sounds ok, as arrays may never be valid values to test.


ModuleNotFoundError: No module named 'regex._regex'

i use this code in order to build a pipeline
import re
import nltk
from textblob import TextBlob
from nltk.corpus import stopwords
from nltk.stem import PorterStemmer
from textblob import Word
from nltk.util import ngrams
import re
from wordcloud import WorldCloud , STOPWORDS
from nltk.tokenize import word_tokenize
the the error  ModuleNotFoundError: No module named 'regex._regex' come up
any suggestions on the reason ???

I'm closing this. I suspect my comment above would work, and otherwise there are many other people who have encountered this before, e.g. https://github.com/psf/black/issues/1207, which resolve it by pinning regex to a specific version.

Just had this issue popup. Does it make sense to pin the regex version to this project's requirements?

@erikc96 I'd say so. I'm willing to say that you won't have this issue again with regex>=2021.8.3. Good luck!

this issue popped up when I was importing nltk from a layer in aws lambda, the issue is with lambda detecting .cpython-xxxxx.so type of extensions and its compatibility

python-3.10.11-amd64 need -

растудыть её тудыть.


The issue about "sample" function

Dear Maulik
ThankU for your fantastic contribution, I have a little issue while performing the Testing case.
All of the procedures were done and work really well. Moreover I have tried to performing the functions in the controlDict file:
"functions
{
//#includeFunc "stressComponents"
//#includeFunc "pressureCoefficient"
//#includeFunc "sample"
//#includeFunc "sampleCp"
//#includeFunc "writeCellCentres"
//#includeFunc "wallShearStress"
}"
However, while execution, the OpenFOAM report:
"--> FOAM FATAL ERROR:
Unknown RASModel type ML_SA_CG"
Is there anything wrong about my comprehension?
Thanks a lot!

Hello! Are you sure you added the ML_SA_CG.so file in your libs information within controlDict? For example here. If you have then it may be so that the ML_SA_CG.so file is not being found by the solver at run-time. Can you confirm that this file exists in $FOAM_USER_LIBBIN?
Thanks
Romit

Much appreciation about your advice, I solved the problem by execute the following codes ahead of the mentioned codes.
"libs
(
"ML_SA_CG.so"
);"
Sorry to bother you,but I still got a little question. What does the word "yWall" stands for in your current study? I haven't found the explaination in the reference paper of yours.

Great! yWall is given by the y+ distance from the wall as computed by OpenFOAM's internal utilities.

Hi Romit，I have read carefully of the README and trying to figure out the process of ML_RANS framework. There are two questions I want to ask for your advice.
Question 1 is about the simpleFoam_ML, is the ML turbulence model only performed once in the solving？
Question 2 is about the filter, is there any detailed description about the filter that I can refer to? I was not familiar about the function of the filters.
Thank U very much!
Sanyaoshanren

Yes - simpleFOAM_ML does the turbulent eddy viscosity calculation just once prior to the start of the simple algorithm and fixes this viscosity. This is what gets you the speed up.
I think you can find the definition of the simple filter we use here. It is basically a weighted average of face-centered values (which are interpolated from the cell-centered ones) and where the weights are given by the area of the faces.

Your explanation is very clear！Thank U Romit!

Glad to help. Happy foaming!


fix import typo in _app.tsx

Hi Antonio,
I read your article and clone the repo to play. It's awesome but with a little typo cause crashing, so I sent a pull request to fix it.

Hey @ShihTingJustin, thanks for the PR.
Actually it's not a typo! The tailwind.css file is used as a base for the JIT compiler. The actual CSS that needs to be imported is in fact globals.css, which you can create by running the css:dev or css:build script in package.json.


feat(ast,parser): parse jsdoc

I see there are some demand for reading jsdoc information, let's do this in two steps:

parse jsdoc into structured data, this should be lazy.
save them into trivias and have them accessible. I haven't figured out the best way to do this, but see https://github.com/Boshen/oxc/discussions/6 for disucssion.

Prioritizing because @thepassle need this. See https://twitter.com/passle_/status/1637446645104164865

Update:
The original intent was to come up with a generic comment attachment algorithm, where we attach comments to some AST node driven by a set of rules. After some research and reading about it in the ESLint and Babel codebase, I conclude that it is incomprehensible for me right now.
So instead, let's reduce the problem space down to "attach jsdoc comments to specific AST node". This will be much more approachable.
We will build the jsdoc data structure inside the semantic analyzer. When a jsdoc targeting AST node such as a Statement or Declaration is visited, we will ask Trivias to get us a leading comment. The jsdoc data structure will save the comment span and mark the Statement to indicate it has a jsdoc. A getter will be provided to parse the jsdoc lazily in a OnceCell.

I want to help somehow. What do you need or what can I do? :)

I want to help somehow. What do you need or what can I do? :)

Hi Ema! I've forgotten everything we wrote about jsdocs, but would you be interested in getting a https://github.com/gajus/eslint-plugin-jsdoc rule to work? I can guide you on the missing pieces in the discord channel. I think we had some of the infra working, targeting a lint rule would be the easiest to fill in the gaps.

Note about problems I found are

current attaching logic fails in some cases

/** x */ // <- 2) But this is targeted and it fails because non-whitespaces are between them
/** y */ // <- 1) This should be attached,
let y;

/** y */ // <- This should be attached, but it fails because non-whitespaces are between them
// foo
let y;

/** x */
let x; // <- This is attached correctly with `/** x */`
/** y */
let y; // <- This also tries to target `/** x */` and fails

/** noop */
;
let x; // <-/** noop */ is attached but it shouldn't

no convenient way to retrieve all JSDoc comments

It depends on usecases but need to perform trivias.comment().filter(!single_line && startsWith("*")) by each rule.
I need to check lint rules how often they need to check non-attached JSDoc comments.

We talked a bit about this in #2437 ...,

There are many use cases for JSDoc, but for now, aim to implement eslint-plugin-jsdoc
To know the details, read the source of eslint-plugin-jsdoc first

So, I spent these days reading through the code.

eslint-plugin-jsdoc itself
jsdoccomment, comment-parser and jsdoc-type-pratt-parser, which are heavily dependent

(There are 3 articles on my blog if you're interested. Sorry it's in Japanese.)
As a result, although IMO, I'm not sure we should aim for 100% compatibility for this.
For a number of reasons,

The 10 year old code was very convoluted and hard to understand...

Not easy to deduce the design intent either

It depends on a lot of external libraries

https://github.com/gajus/eslint-plugin-jsdoc/blob/ab893bae6aa5f05228390cb3ce4487485360cba8/package.json#L7-L16
e.g. https://github.com/gajus/eslint-plugin-jsdoc/blob/main/src/rules/checkValues.js

Especially esquery (+estraverse), as the ability to refine the execution context with their originally invented AST seems particularly hard(but widely integrated)

https://github.com/gajus/eslint-plugin-jsdoc/blob/main/docs/advanced.md
Personally, I can't think of a use case, but it might be necessary for those who need it?
And I have no idea how to write such a dynamic process in Rust... 😢

@Boshen Sorry for the long lines, then I'd like to confirm,

Were you originally going to re-implement eslint-plugin-jsdoc with compatibility, no matter how hard it was?
Do you think it would be worthwhile to just omit the heavy functionality

and provide rules/oxc_jsdoc like rules/oxc?
or keep the label eslint-plugin-jsdoc?

(Maybe we haven't seen this yet, but) Should we leave this to external plugins and spend our time on other things?

e.g. Like TypeScript, put JSDoc in the AST?(although compatibility with ESTree would be an issue)
e.g. Just provide a generic API like getLeadingComments(node) and leave it at all

What do you think? 👀

@leaysgur Oh wow I didn't expect 3 blog posts on this topic, I thought jsdoc is a solved problem 😰
So in summary, it seems like the hardest part about jsdoc is comment attachment to AST nodes.
We can leave this part out and focus our task on just jsdoc content rules, which is quiet easy as all we need to do is finish the jsdoc parser and run rules against these parsed jsdocs.

then I'd like to confirm

The intention was to pass all the tests, but we don't really need to do it if it's not a fun task.

And if you're sick of jsdoc after looking at it for 3 days ... you may also join me on the eslint-plugin-import task.

For future reference, let me elaborate.

the hardest part about jsdoc is comment attachment to AST nodes.

This wasn't particularly difficult, at least if we trust the logic of eslint-plugin-jsdoc, or rather, jsdoccomment.

https://github.com/es-joy/jsdoccomment/blob/6aae5ea306015096e3d58cd22257e5222c54e3b4/src/jsdoccomment.js#L283

To put it simply, it was just this:
// findJSDocComment(node, sourceCode): Comment | null;
const beforeTokens = sourceCode.getTokensBefore(node, { includeComments: true });
while (let token = beforeTokens.pop()) {
  if (token.type === 'Block' && token.value.startsWith('*')) return token;
}

return null;

And I believe this behavior was already implemented in #2437 .

The part I found to be the hardest was that some(about half of) rules can:

1⃣ freely determine which astNode to execute the above logic

if (
  esquery.matches(
    astNode,
    "MethodDefinition:not([accessibility="public"]):has(JsdocBlock)"
  )
) {
  const jsdocComment = findJSDocComment(astNode, sourceCode);
}

2⃣ And also based on the Comment obtained, freely determine whether to execute rule handler

const jsdocAstNode = toESTreeLikeAST(jsdocComment);
if (
  esquery.matches(
    jsdocAstNode,
    "JsdocBlock[postDelimiter=""]:has(JsdocTypeUnion > JsdocTypeName[value="Bar"]:nth-child(1))"
  )
) {
  ruleHandler({ astNode, jsdocAstNode });
}

https://github.com/gajus/eslint-plugin-jsdoc/blob/main/docs/advanced.md

(Actually, they seemed to have a bit more complicated logic...)

Moreover, about rest half of the rules seem to simply check "only the text of all JSDoc comments in the source".
However,
for (const { astNode, jsdocAstNode } of jsdocNodesWithAttachedNode)
  // Why astNode required...??
  ruleHandler({ astNode, jsdocAstNode });
// ...
for (const { jsdocAstNode } of jsdocNodesWithoutAttachedNode)
  ruleHandler({ astNode: null, jsdocAstNode });

they are called differently for some reason.

https://github.com/gajus/eslint-plugin-jsdoc/blob/e948bee821e964a92fbabc01574eca226e9e1252/src/iterateJsdoc.js#L2279-L2328

And when I tried setting the astNode of the former to null, the tests started to FAIL...! 😇

you may also join me on the eslint-plugin-import task.

That sounds interesting as well.
Either way, I'll think a bit more about how to conclude #2437 .

@leaysgur is carrying this task 👍 . Future issues can be created separately now.


Change AddNoise to RandTransform

Address to https://github.com/fastaudio/fastaudio/issues/101
Additionally, I have to add .pre-commit-config.yaml to be able to commit. Not sure if its correct

Thanks for the review! I have made the update

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


malformed encoding leads to BlockifyError

I'm occasionally getting BlockifyError s caused by malformed encoding values set here. Here's the tail of the traceback:
Traceback (most recent call last):
    File "dragnet/blocks.pyx", line 846, in dragnet.blocks.Blockifier.blockify
    File "src/lxml/parser.pxi", line 1689, in lxml.etree.HTMLParser.__init__
    File "src/lxml/parser.pxi", line 823, in lxml.etree._BaseParser.__init__
    LookupError: unknown encoding: 'b'UTF-8,''

Looks like there's a trailing comma on "UTF-8", plus it's been incorrectly converted into unicode — possibly by calling str(b"UTF-8") instead of b"UTF-8".decode("utf-8").
I wasn't able to track down a relevant bug in blocks.pyx, so maybe this is just messy web data and 🤷♂ . Just posting in case somebody knows what's up!

Huh, that's pretty odd. Do you have an example page you can share that causes this? At a glance, I don't see anything that would cause it, but I'd be curious to poke around and see what's up with that.


build: update requirements upgrade automation

will standardize messaging to match other repos
and add PR links to resulting alerts in the description of the alert
JIRA:EDUCATOR-5925

Codecov Report

Merging #76 (5774172) into master (1c5d7cb) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master       #76   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           29        29
  Lines          723       723
  Branches        43        43
=========================================
  Hits           723       723

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 1c5d7cb...5774172. Read the comment docs.


Webstorm TSlint issue for .spec files

Bug Report or Feature Request (mark with an x)
- [X] bug report -> please search issues before submitting
- [ ] feature request

Versions.
@angular/cli: 1.0.0
node: 7.6.0
os: darwin x64

Repro steps.
After upgrading to the latest angular-cli version and splitting my TSConfig files webstorm alerts me on my spec files that it cannot find all jasmine/karma names. I guess this happens because webstorm only searches inside one of the ts config files.
Any workarounds there are? (without adding the spec types to the main app tsconfig file)

Issue for me was package.json changes, I created a fresh ng4 install and just copied em over

@scopsy I think you probably forgot to add the rootlevel tsconfig.json as described in https://github.com/angular/angular-cli/wiki/stories-1.0-update. That's the one that provides typings for IDEs.


Don't limit Options.logger to Console type

This is a port of DefinitelyTyped/DefinitelyTyped#27911
Changed Options.logger to accept any object implementing the following methods: error, warn, info.

Thanks!

Thanks


README: fixed typo (missing open quote) in FactoryGirl example

Oops - adding the quote didn't fix the example, I'll make another pull request when I actually fix it.