Enable Consult to be operated from external package, and test that pr…

…edicate/rule can be consulted
Without operators the Consult function can't be called and correctly completed from an external package. The existing tests add the operators manually before calling consult. But the tests in text_test.go rely on the fact they are in the "engine" package. The SetPrologOperators is a - potentially expandable - set of default operators needed to Consult prolog files.
The rawtext map added to the VM structure is a placeholder for the next pull request (about saving files).

Hi! Thank you for the PR. Could you elaborate the problem you're solving here, please?

Without operators the Consult function can't be called and correctly completed from an external package.

I'm afraid you may have mistaken this part as a part of func TestVM_Consult(t *testing.T) which is actually a part of func TestVM_Compile(t *testing.T):
https://github.com/ichiban/prolog/blob/main/engine/text_test.go#L426-L430
Regarding rawtext, which can potentially be a separate PR, you could do the same thing within the ISO standard:
:- initialization(main).
:- dynamic(foo/1).

main :- assertz(foo(a)),
	assertz(foo(b)),
	H = foo(_), bagof((H:-B), clause(H, B), Clauses),
	write_clauses(Clauses),
	halt.

write_clauses([]).
write_clauses([(H:-true)|Cs]) :- write(H), write('.'), nl, write_clauses(Cs).
write_clauses([C|Cs]) :- write(C), write('.'), nl, write_clauses(Cs).

Of course, you can also query from Go and write them down to a file. Anyways, both can be achieved without changing this library.

Hi,
Thanks for your quick response.
The broad concept is:

Use one or more standard (i.e. already existing) prolog files.
Consult these - so they can be operated at will from the Go program(s).
Define numbers of external predicates in Go.
Using go extract information from external system (e.g. database).
Interpret the external information to create lots of additional facts.
Run a combination of standard goals, predicates and dynamic goals, predicates to analyse the information.

The raw text was conceived in order that the combined source from existing prolog files and "new dynamically created prolog" can be output to a new prolog file.
This could then be used with another prolog interpreter (e.g. swipl) either for comparative purposes or to leverage modules and facilities not currently available in ichiban/prolog system.
I'll have a look later at the solution you've kindly suggested.
Thanks

I still think you're reinventing clause/2 or listing/1. Is it a common practice in other implementations?

I felt this was hard work - and potentially fragile? And the cost of keeping/collating the Prolog rawtext inside the engine itself wouldn't be high?

I think the hard work is already done by the ISO standard authors and it's not fragile. Even if the cost is low, is it legitimized for us, the maintainers and the users, to pay it every time we asserta/1 or assertz/1 for this particular use case?

If one debugs TestVM_Consult this new test fails with the unexpectedTokenError. I chased this down and came to the conclusion that it was the lack of defined operators when the consult lead to compilation that was the problem. I believe tests to Consult files like this ought to succeed?

I see but this test must fail. We made a design decision that a zero value for VM is a sandboxed one without any built-in predicates and operators. In func TestVM_Consult(t *testing.T), we explicitly use an almost zero value VM so it doesn't recognize any operators.
In practice, you'll instantiate an interpreter with prolog.New() which has all the standard operators. So, there's no problem.
https://github.com/ichiban/prolog/blob/main/bootstrap.pl#L5-L23

Thanks... I'll consider your points in more detail..and come back to you..

Any updates?

Sorry for the delay in responding. I realise that I need to create a real scenario and populate it in order to demonstrate the "why". This will take time for me to do. If you want to close this PR in the meanwhile, I can open an issue in due course and proceed to a PR when/if we agree..

I understand. Let's close this issue for now. Please feel free to reopen it or open another!


[EventRequest] OnAfterLookupPostCode in Table 5050 "Contact"

Hello, please consider adding the following event to Table 5050 "Contact":
[IntegrationEvent(false, false)]
local procedure OnAfterLookupPostCode(var City: Text; var PostCode: Code[20]; var TerritoryCode: Code[10])
begin
end;

and raise it accordingly on field 91 "Post Code":
        trigger OnLookup()
        begin
            OnBeforeLookupPostCode(Rec, PostCode);

            PostCode.LookupPostCode(City, "Post Code", County, "Country/Region Code");

            OnAfterLookupPostCode(City, "Post Code", "Territory Code");
        end;

Thank you!

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 49203.


Remove dynamic dependency on package.json

Fixes #330

thank you for the PR, but this depends on npm and so won't be available to the packaged up binaries - i believe.

Switched it to a simple require which removes the npm dependency and allows the dependency to be discovered with static analysis.

ok thank you. i will double check this since i believe we went away from this for a reason. possibly an issue packaging up the binary - if i recollect. or it failed in certain versions of node. i think there are some caveats to require with a json file.


contributors.svg is out of date of many months

Describe the bug
The contributors.svg image don't get updated
To Reproduce
Steps to reproduce the behavior:

Go to https://opencollective.com/ankidroid/contributors.svg
Counts the number of contributors listed (243)
Look at https://github.com/ankidroid/Anki-Android
Look at number of contributors (312)
Look at the image source code.

Expected behavior
Both numbers should be equal. Or at least relatively close. After all, caching the data and being out of date from a few hours or days would be entirely acceptable for any of our purpose. Being months old, I suspect it's not the expected behavior
If a user action is required to update the cache, I'd consider making sure it can be easily found it from either the source code or the link at the end of the image. Neither seems to hold.
Also, I'd expect the indentation to be the same for every contributors. But the first one is more indented for some reason. But this one is far less important as an issue
Screenshots
From github:
From your image:
Prefix of the source code of your image:
Desktop (please complete the following information):

OS: Ubuntu
Browser Chrome
Version: 103.0.5060.53 (Official Build) (64-bit)

Additional context
I have wondered whether you only considered people who contributed to the open collective, but I doubted it, because I see we have 2543 donors (I honestly have a hard time believing it, even if it's only .1% of our user base. Thanks for allowing it). However, I saw at the start of the images some really active past contributors who have not been present with us for years and I doubt they interacted with the open Collective

Triage Template (core team only)
This template is for members of the team to triage for prioritisation. For more guidance see https://www.loom.com/share/369ab467fbc64dec848085d38ff57ca0:
P1 high frequency, high impact
P2 low frequency, high impact
P3 high frequency, low impact
P4 low frequency, low impact
Examples of high impact - a problem affects users during an essential process (expenses and payments > onboarding and registration > contributing) with no workaround.
High frequency - >10% of users affected (measured as a proportion of total potential users for this case).

It's likely that the Redis cache was stuck due to an old version of the codebase from a year ago. I just flushed the cache for ankidroid. It should be better, maybe only in a few hours due to others layers of cache on the CDN side.

Thank you. Appreciated.
Out of curiosity, did you just flush out for our project? Since it's usually not something checked very often, I could have imagined that other team have the same problems and may have missed it. So I'd humbly suggest checking whether other cache has not been updated since this old version of the codebase (admittedly, I assume some teams may have not been updated because they became inactive, I've no idea how to arrange that.)
By the way, thanks for letting us pull this image from your server. It's nice to have on our github thank's note

Closing as it seems resolve, future issues should be reported to https://github.com/opencollective/opencollective/issues.


Set-Location & Get-Item -LiteralPath Inconsistent On Files/Dirs With ?/* Characters

Steps to reproduce
Using PowerShell 7.0 on Linux or MacOS:

Create an empty directory
Create two sub directories called ":" and "?"
Attempt to change location to '?'; the directory will actually change to the ':' directory.
Attempt to Get-Item on the '?' directory; the directory information will be for the ':' directory.

Expected behavior
PS /tmp> New-Item -ItemType Directory '/tmp/dir/:' | Out-Null
PS /tmp> New-Item -ItemType Directory '/tmp/dir/?' | Out-Null
PS /tmp> Set-Location -LiteralPath '/tmp/dir/?'
PS /tmp/dir/?> Get-Item -LiteralPath '/tmp/dir/?'

    Directory: /tmp/dir

Mode                 LastWriteTime         Length Name
----                 -------------         ------ ----
d----          04/10/2020    19:45                ?

Actual behavior
PS /tmp> New-Item -ItemType Directory '/tmp/dir/:' | Out-Null
PS /tmp> New-Item -ItemType Directory '/tmp/dir/?' | Out-Null
PS /tmp> Set-Location -LiteralPath '/tmp/dir/?'
PS /tmp/dir/:> Get-Item -LiteralPath '/tmp/dir/?'

    Directory: /tmp/dir

Mode                 LastWriteTime         Length Name
----                 -------------         ------ ----
d----          04/10/2020    19:45                :

Environment data
Name                           Value
----                           -----
PSVersion                      7.0.0
PSEdition                      Core
GitCommitId                    7.0.0
OS                             Linux 4.4.0-18362-Microsoft #476-Microsoft Fri Nov 01 16:53:00 PST 2019
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

@NoMoreFood interestingly, I'm not able to repro this on my 7.1 shell (on Mac) are you still seeing this?

@JamesWTruher On the test case I originally presented, you seem to be correct.  The failure was highly dependent on how the underlying file system was enumerated in order to expose the bug.  It's possible a change elsewhere in the codebase is now requiring different preconditions or, alternatively, the bug was addressed in another way.  I will look into it.


Add NEP 29 authors to spec0

This PR adds authors who originally contributed to NEP 29 to spec0. The authors I already added I have verbal consent from or have already contributed to NEP 0 at the developer summit.
@tacaswell @rgommers @amueller and @ellisonbg would you like to be added as authors to this SPEC as well?

I don't think I'm an author, thank you though :)

@amueller Are you sure?! https://numpy.org/neps/nep-0029-deprecation_policy.html

Huh, I stand corrected. I guess it makes sense to include me for consistency then? I have no strong opinion, though.

Thanks for asking @munkm. Please include me as well ( Ralf Gommers <ralf.gommers@gmail.com>).

Ok, I added your information Ralf. Andreas, do you have a preferred email contact?

Andreas C. Mueller<EMAIL_ADDRESS>

Thanks, everyone.


Fix chat form

Victim cannot send message because "send" button isnt showing

can you give me a capture
---Original---
From: @.>
Date: Sat, May 15, 2021 20:45 PM
To: @.>;
Cc: @.***>;
Subject: [qwqdanchun/DcRat] Fix chat form (#21)
Victim cannot send message because "send" button isnt showing
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Victim cannot send message because "send" button isnt showing

Its false information. Its working in newest build, you need to download a release from auto build.

Victim cannot send message because "send" button isnt showing

Its false information. Its working in newest build, you need to download a release from auto build.

uhh lemme try

before it wasnt showing the button but the button doesnt look very good

before it wasnt showing the button but the button doesnt look very good

I know. I reported this issue to @qwqdanchun, and as you can see in newest build it's working. Not looking perfect but working 100%. In previous version send button was cut in half, it was visible but not fully. I tested it a lot 😄
So yeah it's working. @qwqdanchun Said that he is not good in design, but button is visible now in 100% so that's good


Update installation docs

Updated installation instruction in docs. I've installed the plugin recently on Sylius 1.12 and noticed some parts of instruction needs refreshing

Hey @mpysiak
Thank you for the suggested improvement.
This will be included in the next release.
Best regards.


[Admob] Project AdTest raised exception class 6.

Hello
Currently testing for iOS.
An error occurs while performing an example of admob.
The current system is iOS 15.5 / Delphi 11.1 update 1.
Error message when running the app.
Project AdTest raised exception class 6.
libc++abi NSException Process
Please check.
thank you.

I'm not seeing any exception here. Can you please provide the steps? This is what I did:

Compile/run the demo
Tap each of the Banner, Interstitial and Reward buttons, and tap the close button when each ad appears for Interstitial and Reward

No errors

I'll tell you how to proceed.
It's basically the same as what you said.
Android works fine.
Execution Environment:
iOS 15.5 and macOS Monterey(Intel) 12.3.1
Windows 10, Delphi 11.1 Update 1
The following error occurs when running:

After debugging, DW.iOSapi.Google MobileAdunit fails.
The location is as follows.

If you remove the source like this, it will run.

However, the following error occurs when running the advertisement.

Please check.
thank you.

Please check.

I did check - the demo works for me, with no alterations to the code. Please re-download the current source, and try it again.

https://github.com/DelphiWorlds/Kastri

https://github.com/DelphiWorlds/Playground
All new, but the same error occurs.

Is there any way I can test it additionally?

A fresh download is failing for me, too.. which is odd, because there aren't any changes pending. I'm looking into it.

Maybe an obvious question but have you included the skAdNetwork and other necessary entries to the info.plist file? If not that can cause an exception.
Check the Console app on the Mac connected to your phone as the iPhone console log may report the actual error associated with that crash.

One of the newer Google Mobile Ads libraries was not included in the last push, however on my machine I had the new version and git was not showing that they were different. Please update at least the Lib\iOS\UserMessagingPlatform.framework folder from the demo, and it should now work

There is no part added to info.plist because the demo is currently running.
Do I have to add it separately?

One of the newer Google Mobile Ads libraries was not included in the last push, however on my machine I had the new version and git was not showing that they were different. Please update at least the Lib\iOS\UserMessagingPlatform.framework folder from the demo, and it should now work

I received it through cocoapod from mac.
However, it does not work.

There is no part added to info.plist because the demo is currently running.
I didn't have to update the info.plist.

I received it through cocoapod from mac.
Use the one from the demo, because it is compatible with the demo code

I got a new one from git, but it doesn't work.
Is there a solution?

Sorry.. Chris was right about the info.plist file. I've now pushed the required file (in the root of the demo)

It works fine.
Thank you for your support.


Updating Release checklist

Just minor update to make the steps even more obvious. We can merge after the release.
Signed-off-by: Nishant Totla<EMAIL_ADDRESS>

LGTM


-imdb and -tvdb args don't work

[root@processor mp4-automator]# time /opt/mp4-automator/manual.py -i "/shared/video/movies/Kiss the Girls (1997).mkv" -imdb tt0119468
Traceback (most recent call last):
File "/opt/mp4-automator/manual.py", line 287, in
main()
File "/opt/mp4-automator/manual.py", line 271, in main
imdbid = int(args['imdbid'])
ValueError: invalid literal for int() with base 10: 'tt0119468'
real    0m0.555s
user    0m0.373s
sys     0m0.048s
[root@processor mp4-automator]# time /opt/mp4-automator/manual.py -i "/shared/video/movies/Kiss the Girls (1997).mkv" --imdb tt0119468
Traceback (most recent call last):
File "/opt/mp4-automator/manual.py", line 287, in
main()
File "/opt/mp4-automator/manual.py", line 271, in main
imdbid = int(args['imdbid'])
ValueError: invalid literal for int() with base 10: 'tt0119468'
real    0m0.461s
user    0m0.314s
sys     0m0.039s
So I see its complaining about int() and sure enough the IMDB number has some ascii in it, so I figure no problem remove the ascii and just send chars, the program must assume anything coming in after --imdb is a valid movieid. No go:
[root@processor mp4-automator]# time /opt/mp4-automator/manual.py -i "/shared/video/movies/Kiss the Girls (1997).mkv" --imdb 0119468
Traceback (most recent call last):
File "/opt/mp4-automator/manual.py", line 287, in
main()
File "/opt/mp4-automator/manual.py", line 278, in main
processFile(path, tagdata)
File "/opt/mp4-automator/manual.py", line 153, in processFile
tagmp4 = tmdb_mp4(imdbid)
File "/opt/mp4-automator/tmdb_mp4.py", line 14, in init
if tmdbid is False and imdbid.startswith('tt') is not True:
AttributeError: 'int' object has no attribute 'startswith'
real    0m0.500s
user    0m0.378s
sys     0m0.046s
Using -a works however.

I'm wondering if you're using an older version of the script that's causing
your current error, becasue that line imdbid = int(args['imdbid'])
doesn't seem to exist anymore and was changed for the bug you're
mentioning. Just double check that for me before I dive into this any
further.
Thanks
On Tue, Jan 6, 2015 at 3:06 AM, cod3fr3ak<EMAIL_ADDRESS>wrote:

[root@processor mp4-automator]# time /opt/mp4-automator/manual.py -i
"/shared/video/movies/Kiss the Girls (1997).mkv" -imdb tt0119468
Traceback (most recent call last):
File "/opt/mp4-automator/manual.py", line 287, in
main()
File "/opt/mp4-automator/manual.py", line 271, in main
imdbid = int(args['imdbid'])
ValueError: invalid literal for int() with base 10: 'tt0119468'
real 0m0.555s
user 0m0.373s
sys 0m0.048s
[root@processor mp4-automator]# time /opt/mp4-automator/manual.py -i
"/shared/video/movies/Kiss the Girls (1997).mkv" --imdb tt0119468
Traceback (most recent call last):
File "/opt/mp4-automator/manual.py", line 287, in
main()
File "/opt/mp4-automator/manual.py", line 271, in main
imdbid = int(args['imdbid'])
ValueError: invalid literal for int() with base 10: 'tt0119468'
real 0m0.461s
user 0m0.314s
sys 0m0.039s
So I see its complaining about int() and sure enough the IMDB number has
some ascii in it, so I figure no problem remove the ascii and just send
chars, the program must assume anything coming in after --imdb is a valid
movieid. No go:
[root@processor mp4-automator]# time /opt/mp4-automator/manual.py -i
"/shared/video/movies/Kiss the Girls (1997).mkv" --imdb 0119468
Traceback (most recent call last):
File "/opt/mp4-automator/manual.py", line 287, in
main()
File "/opt/mp4-automator/manual.py", line 278, in main
processFile(path, tagdata)
File "/opt/mp4-automator/manual.py", line 153, in processFile
tagmp4 = tmdb_mp4(imdbid)
File "/opt/mp4-automator/tmdb_mp4.py", line 14, in init
if tmdbid is False and imdbid.startswith('tt') is not True:
AttributeError: 'int' object has no attribute 'startswith'
real 0m0.500s
user 0m0.378s
sys 0m0.046s
Using -a works however.
—
Reply to this email directly or view it on GitHub
https://github.com/mdhiggins/sickbeard_mp4_automator/issues/185.

No problem I'll grab a more recent copy and try it. Thanks!

Also tried to install the subliminal piece as well, didn't work, but this is more important at the moment.


Refactor code + add support for multinode clusters

This is a big refactor that covers a few things:

Refactor to use molecule-plugins for the Docker bits, so we don't have to keep a local copy of these, relying on the Docker driver instead to do the heavy lifting for us.
Fix all of the tests so that they are all passing
Implement the ability to have multi-node KinD clusters that you can define inside platforms with different roles and labels
Add testing to make sure both old existing clusters and new multinode clusters code still works fine
Improved login support so now login can login to any of the kind nodes in multinode scenario

Indirectly, it also fixes https://github.com/retr0h/molecule-kind/issues/1 which means you can actually run against the target containers (and that's tested in the code with the ping that is being done against them all).
Thanks!  Would appreciate a release once you merge this if it is all good :)

@retr0h do you mind pressing the button to run tests if you have a second? :)

ahh, darn, the newer images of KinD don't have any python.. do you think i should install it out of the box or do it in the prepare step and then users can override that?

ahh, darn, the newer images of KinD don't have any python.. do you think i should install it out of the box or do it in the prepare step and then users can override that?

Not sure, but looks like unit tests are failing, doesn't look related to kind images not having python.

The unit tests run a whole molecule scenario 😅


Add Sprite constructor overload taking animationCache

Add Sprite constructor overload taking an explicit animationCache. This allows projects to define and use their own AnimationSet cache, which they can control the lifetime and maintenance of. This eliminates much of the need for the library to define it's own defaultAnimationCache. This change also makes the API a bit more consistent, in that AnimationSet has a constructor overload that can accept a custom imageCache.

The defaultAnimationCache is flagged when compiling with the Clang warning flag -Wglobal-constructors and -Wexit-time-destructors.
Reference: #528

Hmm, this idea may be a bit flawed, in that if we want a custom cache for Sprite to use to load AnimationSet objects, we'll probably also want the AnimationSet objects to get a custom cache to load Image objects. It would seem the two levels of caching don't compose particularly well.


backfill experiments

This would close #1435
To enable back-filling events into Marten, I'm having a play with some APIs to enable that, its very experimental, but open to feedback
example consumption:
session
    .Events.StartStream(player3, playerRegistered3)
    .WithBackfill(
        created: DateTimeOffset.FromUnixTimeSeconds(576329),
        timestamp: DateTimeOffset.FromUnixTimeSeconds(576329)
    );

I don't like the below syntax, it would be much nicer to also use .WithBackfill - however the StreamAction seems not 1->1 with events.
session.Events.Append(map1, DateTimeOffset.FromUnixTimeSeconds(646329), record4);
session.Events.Append(map1, DateTimeOffset.FromUnixTimeSeconds(656329), record5);

@Hona Hey man, can you fix up the white space issues on this? I don't even know what it is you were trying to do here

Heya, sure @jeremydmiller I can clean up the PR for you.
The main thing I wanted to do a PoC for was setting custom date times (to convert standard data into an event stream historically)
e.g. of important code from the PR is https://github.com/JasperFx/marten/pull/3358/files#diff-28a99daf7298c7f4f19399310e0b9fbbb0ce7596493d50038109fc2cda5b845cR115-R385


Include version file for Elixir binding

Hi! love this library, I discovered it last week and had my eye on the elixir bindings PR. Was happy to see it was merged in an uploaded to hex this morning, so I tried it out.
Unfortunately, it wasn't working for my project. For the Hex release, it only uploads the contents of the ./bindings/elixir directory, and that is all that is pulled down to my local machine when I add it as a dependency. See the photo of my deps folder:

In the mix.exs file, it's trying to read the current version of the project from the top level VERSION file using the path ../../VERSION, but this file doesn't exist because Hex only includes the sub-directory.
I just quickly put together this PR - I added a VERSION file in the bindings/elixir folder, I updated the mix.exs file so it reads bindings/elixir/VERSION file, and I updated the GitHub actions so that it copies the latest VERSION file from top level to the bindings/elixir folder on release. I tested the changes in mix.exs and the bindings/elixir/VERSION file and it works perfectly. I'm not sure how to test the small change in the GitHub action, since it is for uploading to Hex.
There's probably a better way to do this, so definitely open to suggestion here :)
Let me know if I can help!

Thank you for the quick fix @gworkman ! I'm going to merge this and create a new v0.1.1-alpha.18 release shortly.


The title of the image imported by OneNote is the image text recognition problem.

The file name of the picture imported into obsidian is the content of the picture after recognizing the picture. When the picture contains too long and complicated text, it will produce bad links.

I hope you can tick or unname the picture.

Could you paste the resulting markdown link here instead of a screenshot? It's not clear from the screenshot where the bad link is. Thank you.


Skip 1.31 curated packages tests for all providers

Issue #, if available:
#2399
Description of changes:
Skip 1.31 curated packages tests for all providers until we add support
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve

/lgtm


Refactor EvaluateJacobianWithImageGradientProduct and EvaluateTransformJacobianInnerProduct

Various EvaluateTransformJacobianInnerProduct member functions appear very similar. This refactoring aims to help analyzing and understandingthe metrics.
There are 17 metrics derived from AdvancedImageToImageMetric:

CostFunctions/ImageToImageMetricWithFeatures
CostFunctions/MultiInputImageToImageMetricBase
CostFunctions/ParzenWindowHistogramImageToImageMetric
CostFunctions/TransformPenaltyTerm
AdvancedKappaStatisticImageToImageMetric
AdvancedMeanSquaresImageToImageMetric
AdvancedNormalizedCorrelationImageToImageMetric
GradientDifferenceImageToImageMetric2/GradientDifferenceImageToImageMetric
NormalizedGradientCorrelationImageToImageMetric
PatternIntensityImageToImageMetric
PCAMetric2
PCAMetric
PCAMetric_F_multithreaded/PCAMetric
SumOfPairwiseCorrelationCoefficientsMetric
SumSquaredTissueVolumeDifferenceImageToImageMetric
VarianceOverLastDimensionImageMetric
CombinationImageToImageMetric

Six of them override AdvancedImageToImageMetric::EvaluateTransformJacobianInnerProduct.
The overrides of five (of those six) metrics are exactly the same:

PCAMetric
PCAMetric_F_multithreaded
PCAMetric2
SumOfPairwiseCorrelationCoefficientsMetric
VarianceOverLastDimensionImageMetric

The override of SumSquaredTissueVolumeDifferenceImageToImageMetric is different, because it takes TissueValue - AirValue into account.
AdvancedImageToImageMetric has a different implementation, for if TransformIsBSpline is true.
AdvancedTransform::EvaluateJacobianWithImageGradientProduct is just different because it uses a thread_local jacobian matrix.

This pull request should also help when analyzing the difference between #if 0 and #else in itkAdvancedMeanSquaresImageToImageMetric.hxx:
https://github.com/SuperElastix/elastix/blob/6e0316319a0542942f60e4d1bab06a53d3c75fb0/Components/Metrics/AdvancedMeanSquares/itkAdvancedMeanSquaresImageToImageMetric.hxx#L491-L503
Some notes:

this->EvaluateTransformJacobian( fixedPoint, jacobian, nzji ) appears equivalent to Superclass::m_AdvancedTransform->GetJacobian(fixedPoint, jacobian, nzji).
EvaluateTransformJacobianInnerProduct calls the implementation of AdvancedImageToImageMetric, which checks TransformIsBSpline:  https://github.com/SuperElastix/elastix/blob/6e0316319a0542942f60e4d1bab06a53d3c75fb0/Common/CostFunctions/itkAdvancedImageToImageMetric.hxx#L754-L806
EvaluateJacobianWithImageGradientProduct is implemented by AdvancedTransform, using a thread_local jacobian matrix, but has overrides by AdvancedBSplineDeformableTransformBase, EvaluateJacobianWithImageGradientProduct, and EvaluateJacobianWithImageGradientProduct.

@mstaring When doing a VS2022 Release build, the code in the #if 0 appears to make AdvancedMeanSquaresImageToImageMetric slightly faster (using TranslationTransform, NearestNeighbor interpolation, and Full sampler). I compared the following adjusted elastix version with the main version: https://github.com/SuperElastix/elastix/compare/main...Replace-0-with-1-in-AdvancedMeanSquaresImageToImageMetric. Like from 0.97 sec (main) to 0.93 sec (#if 0)


EmptyBodyStream as request body not working

There seems to be a problem when attempting to pass EmptyBodyStream as body to HTTP requests:
(new Browser())
    ->requestStreaming("GET", "https://stackoverflow.com", [], new \React\Http\Io\EmptyBodyStream())
    ->then(var_dump(...)); // never get's triggered

Whereas, empty resource works just fine:
(new Browser())
    ->requestStreaming("GET", "https://stackoverflow.com", [], new ReadableResourceStream(tmpfile()))
    ->then(var_dump(...));

I have tried other servers (URLs) as well, and for some of them I get a response immediatelly, for some there is a 10s delay. Maybe the socket isn't being closed when using EmptyBodyStream?

The full example (reason I care) is that I though this would work as a simple HTTP proxy:
$browser = new Browser();
$browser = $browser->withRejectErrorResponse(false);

$server = new HttpServer(
    new StreamingRequestMiddleware(),
    fn(ServerRequestInterface $req) => $browser
        ->requestStreaming($req->getMethod(), TARGET_URL, [], $req->getBody())
        ->then(fn(ResponseInterface $res) => new Response($res->getStatusCode(), [], $res->getBody())),
);

But it only works for requests with non-empty bodies. Otherwise $req->getBody() is EmptyBodyStream and the requests get stuck.

Hey @martin-heralecky, thanks for bringing this up :+1:
I talked with @clue about this, and it indeed seems to be a bug that has found its way into the project over the last few releases. We've already started working on a fix for this, so I will keep you updated as we make progress.
As you also may know, we're currently working on ReactPHP v3 and releasing the roadmap tickets for each component (see reactphp/event-loop#271 and others). The HTTP roadmap ticket will also be released soon, and to give you a sneak peek, we plan to release at least v1.10.0 before branching out with a 3.x branch. So, I think it makes sense to resolve this bug as part of the upcoming v<IP_ADDRESS> and thus also for the upcoming v3.

@martin-heralecky Again thank you for reporting, that was a nasty one! I've just filed #516 which fixes the problem and introduces additional tests to avoid any future regressions.
I've double-checked your gist, and I'm happy to confirm that it now works flawlessly with the applied changes. The fix may actually look surprisingly simple, but analyzing the problem and coming up with this fix took a couple of hours. If you'd like to support this development, consider sponsoring ReactPHP! ❤ Thank you!

Thanks a lot! I'll definitely consider sponsoring the project.


UI: Show IPv6 address of Instance

Description
Fixes https://github.com/apache/cloudstack/issues/4844

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[x] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[x] Minor
[ ] Trivial

Screenshots (if appropriate):

How Has This Been Tested?

@blueorangutan ui

@blueorangutan ui

@utchoang a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

UI build: :heavy_check_mark:
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/4948 (SL-JID-103)

@utchoang @rhtyd Would it be a good idea to include the ip6 address in the listview as well ?

@utchoang @rhtyd Would it be a good idea to include the ip6 address in the listview as well ?
@davidjumani @rhtyd  I think it's fine, I'll add it to the listview

@blueorangutan ui

@blueorangutan ui

@utchoang a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

UI build: :heavy_check_mark:
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/4948 (SL-JID-106)

If possible please show ipv6 link-local address as well. The one that is in range/block fe80::/10
If you have two instances on same network/subnet/broadcast domain and they both have fe80:xxx they will use it to communicate each other.
https://en.wikipedia.org/wiki/Link-local_address

@kricud are you asking the ipv6 link-local address to be shown for systemvms or user VMs? Is that nic information something that gets added internal to the VM or plugged by CloudStack? (I would image all nics/ipv4/ipv6 addresses to be returned by the API to be shown in the UI; any custom nics/setup inside the VM may not be discovered by CloudStack)

@rhtyd In case of ipv6 you will always have link-local address even if routable ipv6 is not configured. In screenshot I do not see it and its misleading. As you wrote all ip addresses must be presented.

@kricud as long as those are internal to the VM, it will be difficult for CloudStack to report those IPs (for example a custom network over L2 network, there's no IP known to CloudStack). We only see all the IPs for all the nics configured for a VM by CloudStack. As you've said, showing link-local/lo IPs won't be useful for end-users. Thanks.

@rhtyd In case of ipv6 you will always have link-local address even if routable ipv6 is not configured. In screenshot I do not see it and its misleading. As you wrote all ip addresses must be presented.
Ignore- I mis understood its not doable and user must understand that ipv6 will have link-local.
Configuring it manually for presentation in ui is nonsense.

I know it has been merged now, but let me add this:
The link-local address is a non-routable address which can't be used from the outside world. Nor does CloudStack know what it is allthough it could calculate the address based on the MAC.
CloudStack is aimed at routable addresses as that's what we use. Technically servers can communicate with each other over Link-Local as long as they are in the same L2 domain, but it's not the use-case which is used very much.
Most services use unicast addresses for the communication.
LL with IPv6 is mainly used for Router Advertisements, Neighbor Discovery, etc, etc.
I would not say it's very useful to display the LL addresses in the UI. In addition: The API does not return those.

@wido can you create a new issue tracking the new feature asked, you're right I checked the algorithm the LL address is generated using Mac address and it's certainly possible to display it in UI as mac address is already returned in API response.


Notify user when build completed

Aircoookie Yesterday at 5:24 AM
@andyshinn just a tiny suggestion for WBLD. I don't know if it would work well with the discord permission system, but it could be cool if the bot @mentioned the user that requested the binary on build completion.  Just a little quality of life improvement, don't spend too much time on it :joy:

Done.


Fix password reading from terminal. Now works in windows.

Thank for PR. After testing, I will merge it


tell users to search closed issues

tell users to also search closed github issues at bottom of tabular FAQ

Job PR-1095-1 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-1095/1/index.html


Limit report filter counters

Limit results of the report filter counter functions.
This commit might solves #1053 issue.

When clicking on the message filter options, the tooltip overflows the window at the bottom and those parts can be reached with an extra scrollbar.
This happens because of the lazy loading of its content. Maybe this can be solved by invoking an extra renderer function on the tooltip.


create an end 2 end test

This PR replaces the AnnotationBeanRegistrationIT with a full End2EndTest. The test creates an ApplicationContext, updates the configuration, and validates that the new value is loaded correctly.
It uses an S3ConfigurationRepository with localstack via testcontainers. To enable this, the S3ConfigurationRepository is refactored to allow passing the AmazonS3 client. To promote writing testable code, the old constructors are marked as deprecated.

Shouldn't be MAINTAINERS file also be updated with the name Lukas Wilhelm?

@lukasniemeier-zalando I updated the PR with a builder that should address your concerns about internals leaking to the client. Would be great if you could have another look whenever you have time.

:+1:


Release 3.0.0

This is blocked on #25.

Coverage remained the same at 100.0% when pulling d4c782cd245f28179e2c7e784e6977b0e31c0995 on final-release into 7573321b6e4e4c0804161cebc1dee5f334ad3345 on master.

Coverage remained the same at 100.0% when pulling d4c782cd245f28179e2c7e784e6977b0e31c0995 on final-release into 7573321b6e4e4c0804161cebc1dee5f334ad3345 on master.

Coverage remained the same at 100.0% when pulling d4c782cd245f28179e2c7e784e6977b0e31c0995 on final-release into 7573321b6e4e4c0804161cebc1dee5f334ad3345 on master.

#25 is fixed, but now there's #40.


Implement IPv6 capability

I've been wanting to start using linkding and installed it into my Kubernetes cluster, but couldn't access it.
My cluster runs only using IPv6 addresses and I noticed that the default configuration for uWSGI that is used in this project, defaults to binding only to an IPv4 socket and there is no config option to change this behaviour.
In the current year, with IPv4 addresses becoming more scarce and often an addon that needs to be paid extra, I think it's reasonable to change the default to also allow for IPv6 connections.
This does not break any existing configurations, as this creates a dual stack socket, which will respond to requests on both protocols (IPv4 addresses are displayed as IPv4-mapped IPv6 addresses and look like this: "::ffff:x.x.x.x".
Yet I have also provided a configuration option to change the behaviour, if a user were to desire it for whatever reason.
If you don't like changing the default, we could swap it back to <IP_ADDRESS> and only allow overwriting it, if desired, but as I have outlined I think changing the default is reasonable and doesn't harm existing setups.
I have tested these changes on my IPv6-only cluster, as well as on Docker Desktop on Windows with port mapping (which uses IPv4 addressing) and tried it on both the normal default and alpine image and it worked in all cases.

Thanks, makes sense I guess. Seems to work fine for me. If people are having issues I might revert it, then we can try the alternative option-only approach you suggested.

Reverted the changes to the env sample. LD_HOST_PORT was actually correct, it's the port that is bound on the host system, whereas LD_SERVER_PORT is the one that uwsgi runs on. This is customizable for container runtimes that use a shared port range. Also removed the new variable from the sample, people should rarely need to customize this, so no need to confuse people with that option.


Killer Joe's and Vacuum Chests are working through solid blocks.

Pretty self explanatory, I'm not sure if this is intended behaviour or not. It's quite annoying as I have items being pulled across rooms and getting stuck on doors or the middle of floors.
My Killer Joe's are also damaging me through the floors and walls - this in particular seems unintended as the ability to grief other people using this is high.

Both are code behaving as expected - whether they should behave this way is a different matter.
Personally I would say yes it should for the vacuum chest as  every other item hoover system behaves the same way, I've had my RotaryCraft Item Vacuum pick up things from 3 chunks over...
As for the Killer Joe, maybe a config option? I think by default it would be better to leave as is - because correcting it would be relatively computationally expensive, but it would be possible to do a raytrace from the Joe to the Target and check the line of sight is clear...

Fixed in https://github.com/hilburn/EnderIO/commit/0286245a5e81316cc0de997c25fb606703d6e7ed


code coverage is broken for client tests

Describe the bug
When running code converage on a PR, it fails silently.
[2024-07-25T20:48:22.149Z] ['error'] There was an error running the uploader: Error uploading to [https://codecov.io:](https://codecov.io/) Error: There was an error fetching the storage URL during POST: 429 - {'detail': ErrorDetail(string='Rate limit reached. Please upload with the Codecov repository upload token to resolve issue. Expected time to availability: 882s.', code='throttled')}
Error: Codecov: Failed to properly upload: The process '/home/runner/work/_actions/codecov/codecov-action/v3/dist/codecov' failed with exit code 255

This PR will make it fail loudly
To Reproduce
open a PR and look at the GHA workflow results

Fixed by adding the CODECOV_TOKEN to github repo secrets


Analyzer null pointer crash when combining --strong and --warnings

Analyzing a trivial file that just imports dart:html with current dev release (1.17-dev.3.0) or bleeding edge analyzer passing flags --strong --warnings, produces the following crash:
Unhandled exception:
The null object does not have a method '_setErrorState'.

NoSuchMethodError: method not found: '_setErrorState'
Receiver: null
Arguments: [Instance of 'ListResultDescriptorImpl<VariableElement>', Instance of 'CaughtException']
#0      Object._noSuchMethod (dart:core-patch/object_patch.dart:42)
#1      Object.noSuchMethod (dart:core-patch/object_patch.dart:45)
#2      CacheEntry._setErrorState.<anonymous closure> (package:analyzer/src/context/cache.dart:675)
#3      _HashFieldBase&_HashBase&_OperatorEqualsAndHashCode&SetMixin.forEach (dart:collection/set.dart:145)
#4      CacheEntry._setErrorState (package:analyzer/src/context/cache.dart:673)
#5      CacheEntry.setErrorState (package:analyzer/src/context/cache.dart:460)
#6      AnalysisDriver.performWorkItem (package:analyzer/src/task/driver.dart:270)
#7      AnalysisDriver.computeResult (package:analyzer/src/task/driver.dart:109)
#8      AnalysisContextImpl.computeResult (package:analyzer/src/context/context.dart:667)
#9      AnalysisContextImpl.computeErrors (package:analyzer/src/context/context.dart:615)
#10     AnalyzerImpl.prepareErrors.<anonymous closure> (package:analyzer_cli/src/analyzer_impl.dart:127)
#11     _PerformanceTagImpl.makeCurrentWhile (package:analyzer/src/generated/utilities_general.dart:212)
#12     AnalyzerImpl.prepareErrors (package:analyzer_cli/src/analyzer_impl.dart:125)
#13     AnalyzerImpl._analyzeSync (package:analyzer_cli/src/analyzer_impl.dart:161)
#14     AnalyzerImpl.analyzeSync (package:analyzer_cli/src/analyzer_impl.dart:120)
#15     Driver._runAnalyzer (package:analyzer_cli/src/driver.dart:575)
#16     Driver._analyzeAllImpl (package:analyzer_cli/src/driver.dart:213)
#17     Driver._analyzeAll.<anonymous closure> (package:analyzer_cli/src/driver.dart:156)
#18     _PerformanceTagImpl.makeCurrentWhile (package:analyzer/src/generated/utilities_general.dart:212)
#19     Driver._analyzeAll (package:analyzer_cli/src/driver.dart:155)
#20     Driver.start (package:analyzer_cli/src/driver.dart:136)
#21     main (file:///b/build/slave/dart-sdk-mac-dev/build/sdk/pkg/analyzer_cli/bin/analyzer.dart:11)
#22     _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:259)
#23     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:148)

Code is:
import 'dart:html';

I think there are two problems here.
The first problem is that we're throwing an exception while trying to resolve html_dartium.dart:
Mismatch in DeclarationResolver while resolving .../dart-sdk/lib/html/dartium/html_dartium.dart
Could not find element with name "_window" at 2125
Path to root: DeclaredSimpleIdentifier, VariableDeclarationImpl, VariableDeclarationListImpl, TopLevelVariableDeclarationImpl, CompilationUnitImpl
#0      RecursiveAstVisitor&ExistingElementResolver._mismatch (package:analyzer/src/generated/resolver.dart:3457:5)
#1      DeclarationResolver._findWithNameAndOffset (package:analyzer/src/generated/resolver.dart:2725:5)
#2      DeclarationResolver._findIdentifier (package:analyzer/src/generated/resolver.dart:2683:23)
#3      DeclarationResolver.visitVariableDeclaration (package:analyzer/src/generated/resolver.dart:2638:17)
#4      VariableDeclarationImpl.accept (package:analyzer/src/dart/ast/ast.dart:10679:15)
#5      NodeListImpl.accept (package:analyzer/src/dart/ast/ast.dart:7682:20)
#6      VariableDeclarationListImpl.visitChildren (package:analyzer/src/dart/ast/ast.dart:10777:16)
#7      RecursiveAstVisitor.visitVariableDeclarationList (package:analyzer/dart/ast/visitor.dart:1960:10)
#8      DeclarationResolver.visitVariableDeclarationList (package:analyzer/src/generated/resolver.dart:2655:39)
#9      VariableDeclarationListImpl.accept (package:analyzer/src/dart/ast/ast.dart:10771:15)
#10     TopLevelVariableDeclarationImpl.visitChildren (package:analyzer/src/dart/ast/ast.dart:9991:20)
#11     RecursiveAstVisitor.visitTopLevelVariableDeclaration (package:analyzer/dart/ast/visitor.dart:1918:10)
#12     DeclarationResolver.visitTopLevelVariableDeclaration (package:analyzer/src/generated/resolver.dart:2590:43)
#13     TopLevelVariableDeclarationImpl.accept (package:analyzer/src/dart/ast/ast.dart:9986:15)
#14     NodeListImpl.accept (package:analyzer/src/dart/ast/ast.dart:7682:20)
#15     CompilationUnitImpl.visitChildren (package:analyzer/src/dart/ast/ast.dart:2440:21)
#16     RecursiveAstVisitor.visitCompilationUnit (package:analyzer/dart/ast/visitor.dart:1449:10)
#17     CompilationUnitImpl.accept (package:analyzer/src/dart/ast/ast.dart:2433:15)
#18     DeclarationResolver.resolve (package:analyzer/src/generated/resolver.dart:2222:10)
#19     BuildCompilationUnitElementTask.internalPerform (package:analyzer/src/task/dart.dart:1017:33)
#20     AnalysisTask._safelyPerform (package:analyzer/task/model.dart:329:9)
#21     AnalysisTask.perform (package:analyzer/task/model.dart:229:7)
#22     AnalysisDriver.performWorkItem (package:analyzer/src/task/driver.dart:276:10)
#23     AnalysisDriver.computeResult (package:analyzer/src/task/driver.dart:109:18)
#24     AnalysisContextImpl.computeResult (package:analyzer/src/context/context.dart:667:14)
#25     AnalysisContextImpl.computeErrors (package:analyzer/src/context/context.dart:615:12)
#26     AnalyzerImpl.prepareErrors. (package:analyzer_cli/src/analyzer_impl.dart:127:17)
#27     _PerformanceTagImpl.makeCurrentWhile (package:analyzer/src/generated/utilities_general.dart:212:15)
#28     AnalyzerImpl.prepareErrors (package:analyzer_cli/src/analyzer_impl.dart:125:30)
#29     AnalyzerImpl._analyzeSync (package:analyzer_cli/src/analyzer_impl.dart:161:5)
#30     AnalyzerImpl.analyzeSync (package:analyzer_cli/src/analyzer_impl.dart:120:12)
#31     Driver._runAnalyzer (package:analyzer_cli/src/driver.dart:603:34)
#32     Driver._analyzeAllImpl (package:analyzer_cli/src/driver.dart:215:30)
#33     Driver._analyzeAll. (package:analyzer_cli/src/driver.dart:158:14)
#34     _PerformanceTagImpl.makeCurrentWhile (package:analyzer/src/generated/utilities_general.dart:212:15)
#35     Driver._analyzeAll (package:analyzer_cli/src/driver.dart:157:27)
#36     Driver.start (package:analyzer_cli/src/driver.dart:138:32)
#37     main (file:///Users/brianwilkerson/src/dart/sdk/sdk/pkg/analyzer_cli/bin/analyzer.dart:11:11)
#38     _startIsolate. (dart:isolate-patch/isolate_patch.dart:259)
#39     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:148)
This looks like it could be caused by a difference between the summaries and the file we're trying to resolve. The file defined a top-level variable named _window, while the element model we're trying to match it to defines a top-level variable named window.
@scheglov Could you look into the first problem?
The second problem is that dependencies can cross partition boundaries, but we didn't take that possibility into account. I'll look at how to fix that problem.

https://codereview.chromium.org/1988853002

Additional fix https://codereview.chromium.org/2003863002

This should solve the underlying problem: https://codereview.chromium.org/2008133002/


Add heterogeneous helper functions and restructure internals

What does this pull request do?
Provides a solution for #18 and #19 by adding helper functions based on heterogeneous. Also restructures some internals and classes to minimize the number of constraints that exist on the Formless component to save some compile times.
Unfortunately, pending some resolution in the Heterogeneous library (see: https://github.com/natefaubion/purescript-heterogeneous/pull/2), no heterogeneous constraints can be placed on the Formless component without breaking the library, so all changes directly on the component have had to be reverted.
Still, a number of new helper functions are available and the exercise cleaned up the internals of the library.
How should this be manually tested?
Verify that the example components still work as expected.
Other Notes:
This PR changes the namespaces of several functions, and if users are not importing from the main Formless module, they will see breaking changes. However, existing functionality has been maintained. If users are importing from the main Formless module, then this change will have no effect on their code.

Updates to support setAll / modifyAll and modify functions for form inputs have been merged into this branch. In addition, before merging to master, this will require the readme to be updated.

Will merge and release 0.3.0 on Monday, with a migration guide, unless there are any objections. (cc:  @whoadave @crcornwell @dariooddenino)


add config option to disable pedantic prompts

Looks good. My only real comment/question:
What are your thoughts on renaming "pedantic" to "confirmation"? That would be
more consistent with .taskrc (see man taskrc).
Minor comments:

I was going to fix the "code duplication" TODO, but realized it is not trivial because of the assignment.

I think that all bools (i.e. $burndown) should be converted to use 1/undef, like you did for $pedantic. I will try to remember to do this.

For the "TODO: get rid of the eval()", that would be nice! I agree it is a hack (which I introduced at 28450ae2) but I couldn't figure out how to do it in a better way.

I agree dat "confirmation" is a bit more neutral ;)
I also documented the option in the man page.
Should be fine to go for 1.3, now.

Regarding the eval(): are you sure it's even necessary at all?  the /e modifier in the regular expressions in replace_keycodes() seems to care care of the expansion of the KEY_F(1) &co?

squash-merged at db23fc11.

Regarding the eval(): are you sure it's even necessary at all? the /e modifier in the regular expressions in replace_keycodes() seems to care care of the expansion of the KEY_F(1) &co?

No I'm not sure. If you're confident that we can remove it, let's go ahead and make a pull and we can do that for 1.3beta. If you're not confident, let's just leave it and we can experiment with a change after 1.3 is released.


http: initializes a field of ConnectionManagerImpl::ActiveStream::State.

While debugging an issue, it became clear that this field is not properly initialized.
Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:

/retest


Polling App Specification

Demo polling app specification for functions to support blockchain (and other forms) of voting for decentralized organizations on the autom(8) platform
Motivation:
We are the autom(8) team. autom(8) is developing a decentralized serverless platform for communal microservices. Functions are deployed to the platform and shared between developers, teams, and organizations as runnable microservices that anyone can call. We are developing demo apps to demonstrate how autom(8) can be used in the blockchain space to rapidly build apps from commonly used functions. This app will be a simple polling (voting) application.
Please see the open source repository and documentation for more information on autom(8).
Product Specification
Create a polling mechanism for orgs to organize constituents and initiatives, take a poll of constituents among their initiatives, and inspect and share the outcomes of polls.
Scope of work
The steps required (in order):

Specify the Solidity smart contracts - Spec out the smart contract development work based on the Key Organizational structures in the sections below. Confirm specification with the autom(8) team.
Specify the microservices (pure functions) to be deployed to autom(8) serverless platform - Work with us to define and spec out the functions to be deployed to autom(8).
Interaction interface

CLI
(nice to have) Simple web endpoints

Build the smart contracts - Build out the Solidity contracts according to the jointly developed technical spec.
Build and deploy the microservices - Use autom(8) to host/run the functions specified according to the jointly developed technical spec.
Test and review - Test functionality with the autom(8) and participate in a debrief to give us feedback on your experience with the platform.

Key Organizational structure:

Organizations - a list of constituents and initiatives
Initiatives (measures) - Initiatives are a list of constituents, Initiative Bounds (length of vote, or number of votes allowed), a list of choices, and plaintext description of the initiative. It also has a derivative concept of a current winner, and the status (pre-voting, ongoing, concluded)
Constituents - a wallet address present on the organization’s list of constituents a
Choices - a plaintext description of an option that can be selected by a constituent on a particular initiative

Key Terms:

Status - The state of the initiative as decided by the bounds. Status can be

pre-vote - voting has yet to begin
ongoing - voting is open but the bounds have not been reached
concluded - the vote is finished and the resolution is decided

Initiative Bounds - The initiative bounds are the rules that will determine the status. Bounds can be time bound (a window of time when voting will be allowed)
Voting System - There are several imaginable types of voting systems consisting of scoring criteria and ballot type:
Scoring Criteria - Simple majority, plurality (the choice most frequently selected of a multiple choice option)
Ballot Type -  Yea or Nay; Multiple choice: First Past the Post (plurality rule)
Initiative Text - A plaintext description of what is being voted on. Interpretation and implementation of the initiative is outside of the scope. In short, this is only a polling app. It provides no means of deciding on what the outcome of a vote means, nor does it enforce the outcome of a given vote. It simply determines that a plaintext description has been voted on and which outcome has met the winning threshold of votes.

Hi, threejeez, is there a time schedule about it？

@threejeez is this still open?

I have started work on this


[SPEC] Ideal misbehavior handling

Goal
We have the most robust misbehavior system as possible, meaning the ability to report provable misbehaviors everywhere possible rather than voting/oracle reporting methods.
Overview
For all of time, we're been using offchain voting methods to agree on protocol outputs and errors (keys for DKGs, misbehaviors). This is not ideal, certainly not where we have provable ways of identifying misbehavior. Most of the DKG protocols we are interested in have provable misbehavior computations. This is precisely how validators end up in the blame vector. For offline misbehaviors this is the best we can do, but for everything else it is not.
New approach
If in the gadget we could begin with adding a catch or on_misbehavior hook to the JobBuilder that would catch errors. We would cement in the architecture the following meaning that:

pre hook is used before a job starts.
post hook is used only after a job successfully completes.
catch hook is used only after a protocol aborts or when a misbehavior is reported.

catch / on_misbehavior
MPC protocols are multi-round protocols. Often, each round has the potential for misbehavior reporting. For example, a node might be expected to send data w/ a complementary zkp, or send a commitment in 1 round and a decommit in the next, etc. To have the most robust misbehavior system, we need to hook into the messages that are being sent at each round of these protocols. This is something open source implementations usually don't provide directly, as generally they don't need to deal with verifiable misbehavior checking on a blockchain (like we want to do with Tangle). We want to map out the ideal scenario, which is that we catch misbehaviors at any round of a protocol and identify a proof of misbehavior.
Misbehavior Proofs

offline - an offline proof is not really a proof of offline behavior but rather a collection of $t+1$ signatures of a party's offline behavior.
roundX - a round X proof would be a proof of misbehavior at round X

Example proofs

A BadDecommitment error for party $i$. This means the value commitment to in the previous round and the decommitment value communicated in the following round don't match. A proof of this would be

A signed commitment message from party $i$
A signed decommitment message from party $i$
Assuming we can't forge signature's from $i$, it must hold that if $i$ signed invalid messages for a single job execution (say DKG) where $i$ provides an invalid decommitment that the two signed messages would constitute a proof we can verify onchain.

A BadSchnorrProof or BadZKPProof error for party $i$. This means that the user provided an invalid zkp over a discrete log or otherwise over a zero-knowledge statement. For example, if the MPC has a verifiable computation involved, then submitting a signed and invalid proof for the statement can also be considered a proof of misbehavior.

Related #69


Adding extra attributes for traces

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Mario Anton seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Remove deprecated ModuleIdentifier support

This class has been deprecated for many years.

IMO for this change we will need a new main branch and keep 2.x backward compatible with EAP8 @dmlloyd .

Please rebase this PR @dmlloyd

Overall this PR looks good to me @dmlloyd . I didn't merge it because it is in the draft state.

If you think it's okay to merge in the 2.x branch, then I am okay with it being merged.


Helper classes not compatible with client.create(..)

Bug description
Helper classes generated or not
Specs
Artemis version: 6.12.3-beta.2
targets:
  $default:
    sources:
      - lib/**
      - lib/graphql/**
      - lib/graphql/schema.graphql
    builders:
      artemis:
        options:
          generate_helpers: true
          schema_mapping:
            - schema: lib/schema.graphql
              queries_glob: lib/graphql/*.graphql
              output: lib/graphql_api.dart
          scalar_mapping:
            - graphql_type: MongoID
              dart_type: String
            - graphql_type: Date
              dart_type: DateTime

Why helper class is not compatible in with client.execute() params
My Code :
    final client = ArtemisClient('http://<IP_ADDRESS>:8000/graphql');
    final catmany = CategoryMany$Query();
    final response = await client.execute(catmany);

But I am getting error :

The argument type 'CategoryMany$Query' can't be assigned to the parameter type 'GraphQLQuery<dynamic, JsonSerializable>'.

What is this error ? pls help me

Hi.
Use graphq_flutter package.
I do not think we will have time to support client

Here is an article
<EMAIL_ADDRESS>

Hi.
Use graphq_flutter package.
I do not think we will have time to support client

But I took this from last part of your documentation ? What's that mean then ?

I think you have an obvious error in your code
Try this
final client = ArtemisClient('http://<IP_ADDRESS>:8000/graphql');
final catmany = CategoryManyQuery();
final response = await client.execute(catmany);

It will be great if it helps)
But you'd better use graphql_flutter it supports caching, optimistic responses, persistence

I think you have an obvious error in your code
If you look closer into the docs - there is no $ sign in sample code - https://tppr.me/lQBZo
Try this
final client = ArtemisClient('http://<IP_ADDRESS>:8000/graphql');
final catmany = CategoryManyQuery();
final response = await client.execute(catmany);

Actually I got that class from auto suggestions. It had $ symbol

Did you try to remove the $ sign?

Did you try to remove the $ sign?

Yeah After removing it's perfectly
Thanks


Some small_tweaks

I was reading the code and playing with the plugin to learn a bit more.
Made some small tweaks:

set default N_components to 6. It was zero, which made nothing happen on-run. 6 matches the example
Set the viewer to first slice upon output. The layer sliders are linked, so when the source is in the middle slice (200) then the output is missing.
cleaned up some unused imports
tried to fix NMF
This I failed. I noticed that when changing the mode the subsequent ones the image input was greyed out. I swapped the default from PCA to NMF and confirmed that NMF now works with defaults (I made minor tweak). But the issue remains: somehow the image is only passed to the default, "manual" call of _on_choice_change.
I tried a few things, but couldn't figure it out—sorry!

4. But the issue remains: somehow the image is only passed to the default, "manual" call of _on_choice_change.
I tried a few things, but couldn't figure it out—sorry!

OK, so I've been doing more testing.
It's sufficient to add or delete an image layer to make the layers combo work in the other choices.
I guess when the new widget is appended it doesn't inherit the layers list and that list needs to be modified to trigger the magicgui magic that makes the list.
I tried to pass/return the image but all the functions and events only take one parameter.
I'll keep digging—learning a lot!


[safe-condition]: Safe evaluation. Do not throws exception

Actual process of DefaultRulesEngine:

Evaluate method is not error safe
Execute method is error safe

If any rule evaluation fails and launch an exception, the engine throws an exception. If we have a complex evaluation, an exception could be thrown.
This PR avoids this problem.
Plus added onEvaluateFailure listener for this case.

Coverage increased (+0.09%) to 90.943% when pulling da86cdb998169b1da8d10f6a6de2a3ef60120a53 on sanmibuh:safe-condition into 503f5f063a070ed93d8b86cc5403d141ed05d707 on j-easy:master.

Hi,
It was intented to make the evaluate method return a boolean and not throw an exception.
The user should handle any exception and return true/false accordingly.
In contrast, the execute method was designed to throw an exception and let the engine handle it according to some configurable strategy (skipping next rules, ignore the error, etc).
The only case where an exception could be thrown in the evaluate method is when a fact is missing. To be honest, this is a design mistake from my part. It could be better to make the proxy log a warning on the missing fact and return false (instead of throwing a NoSuchFactException and adding a skipOnMissingFact parameter). This should be fixed in a future version. Do you see?
I tried to explain my thoughts about this here, but I would to rechallenge these thoughts 😄
That said, if we fix this behaviour, I'm not sure we should make the evaluate method throw an exception and intercept the evaluation error in the listener. Do you agree?
Kr
Mahmoud

Hello,
Imagine that any fact is a complex object. Maybe the evaluate method can throw any RuntimeException. In this case the rulesEngines will throw the exception, breaking the execution.
In execute method, this use case is safe
Maybe this line should catch only RuntimeException
https://github.com/j-easy/easy-rules/pull/112/files#diff-b55e6fd77e7ad5c46132938da8cd5e5dR165
Best,

Hello,
Yes I understand. But regardless of how complex the logic of the evaluate method is, the user has to catch any exception (including runtime ones if they may occurs) and return true/false accordingly. May be this should be specified in the Javadoc.
The contract of the evaluate method is clear, it is not intended to throw an exception. So any error handling is done by the user. Same for execute, it decalres that it may throw an exception, so the user can delegate error handling to the engine.
Today, the evaluate method may throw a runtime exception when a fact is missing, and as I said, this is a mistake and I'll fix it in v3.1.
Best regards
Mahmoud

Yes, I see your changes to capture NoSuchFactException.
This PR only enlarges this feature, catching all Exceptions (should be better catching only RuntimeException).
As it already exists a try/catch, there is no overhead catching also RuntimeException. And this feature avoids us to catch RuntimeException on every Rules evaluation methods.
Best,

Coverage increased (+0.3%) to 90.414% when pulling 64a61b933f183391d172002da3055ecadb0d6e9c on sanmibuh:safe-condition into a074f93e10a0893fcd37bfdbf03351268eba4934 on j-easy:master.

Coverage increased (+0.3%) to 90.414% when pulling 212a41a03ef134fbcb799963a8853bdbc7853ef7 on sanmibuh:safe-condition into a074f93e10a0893fcd37bfdbf03351268eba4934 on j-easy:master.

Rebased and modified catch on evaluate to catch only RunTimeExceptions

Thank you for the update.
I'm currently fixing the skipOnMissingFact param which is a mistake a think (it's good it is not released yet 😄). If a declared fact is missing, the rule should simply not be triggered (as suggested [here(https://gitter.im/j-easy/easy-rules?at=5934709dca6d4ae80cdf3a45)]).
The evaluate method will not throw any runtime exception (and if the user's code may throw a runtime exception, the user must handle this and not the engine) and we will have the "safe evaluation". In that case, I'm not sure this PR is relevant anymore (there is nothing to say about the quality of the PR, but more about its added value after this fix).

Hi @sanmibuh
I've removed the skipOnMissingFact parameter.  I will update the documentation to state that error handling should be done by the client code (even though it can be deduced from the method signature).
As discussed previously, I'm closing this PR since it has no added value after the fix.
Thank you for your time and effort!
Kr
Mahmoud

I will update the documentation to state that error handling should be done by the client code

doen here: 0267ba397c39ad3affbfb29df4fce314c405ceaa


OTP Password

Hello, the add-on works great so far, but I get an SMS with my OTP password every 3 minutes. I copied the 13x4 digit code from Amazon and copied it into the right place. The synchronization still works, even if I ignore the SMS.

Hi @marioir2001,
It looks like in your account, the SMS is already setup as the default method.
Try to disabled it and enable again the the 2FA with SMS:
1 - Login to Amazon https://www.amazon.com/
2 - Go to Your Account => Login & Security and click on "Manage" under 2-step verification
3 - Set the Authenticator App as the Prefer method.
4 - Set the SMS as the Backup Method.

Thank you for the explanation.
This procedure worked great. Now I no longer receive annoying OTP requests via SMS


add: added a lambda to process and update the amazon rds database with new items from the dynamo table

The usual for testing, but this time we want to test the reaction of this lambda to real wikimedia data, so test by seeding the dynamo table or adding a dummy item and see if it pops up in the amazon rds.
Too tired to write proper test instructions. Hopefully, you all got some experience working with aws and figure this one out :P

Tomorrow I will try to deal with pagination from the backend.
Hopefully, I can make this whole thing work xD


Update sysdig to 0.25 to support Linux 5.0

Issue description
Current sysdig version 0.24.2 is broken when used together with Linux Kernel 5.0. However a new version has been released.
Steps to reproduce
Use Linux 5.0 and try to enable sysdig.
boot.kernelPackages = pkgs.linuxPackages_latest;
boot.extraModulePackages = [
  config.boot.kernelPackages.sysdig
];

Technical details
I am currently attempting to upgrade the version, but ran into some issues as they have added new dependencies:
#59006

system: "x86_64-linux"
host os: Linux 4.19.33, NixOS, 19.09.git.d956f2279b8 (Loris)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.2
nixpkgs: /home/terje.larsen/src/github.com/terlar/nix-config/nixpkgs

sysdig is updated on both master and release-19.03. Closing.


Export/ Import golinks

Export: export existing golinks to a .txt file
Import: import new golinks/ overwrite existing golinks

Export

Import

can you change the file extension from .txt to .json?

Done

spacing between icon and the title was a bit off

Added some spacing, does this look good enough?


proxy config key not documented in configuration.md

https://github.com/docker/distribution/blob/master/docs/mirror.md this shows the proxy configuration key (along with config-cache.yml) but configuration.md (https://github.com/docker/distribution/blob/b78a858b5d61795ad548b501cf4a62841da625a5/docs/configuration.md) does not have it.

Closed by #929


Issues on Installing telegraf_v0.1.6_amd64 on trusty

There is an error in the newest debian package.
dpkg: error processing archive /var/cache/apt/archives/telegraf_v0.1.6_amd64.deb (--unpack):
 parsing file '/var/lib/dpkg/tmp.ci/control' near line 2 package 'telegraf':
 error in 'Version' field string 'v0.1.6': version number does not start with digit

The Version tag must be a digit according to
https://www.debian.org/doc/debian-policy/ch-controlfields.html#s-f-Version

Thanks for the report, I will fix this

This should now be fixed, please try again with the new link on the README: http://get.influxdb.org/telegraf/telegraf_0.1.6_amd64.deb
thanks again for the issue report!


The config.json of scc_net should be modified?

the config.json of the model which has been compressed as follows：
{
"_num_labels": 20,
"architectures": [
"BertForSequenceClassification"
],
"attention_probs_dropout_prob": 0.1,
"bos_token_id": null,
"directionality": "bidi",
"do_sample": false,
"early_stopping": false,
"eos_token_ids": null,
"finetuning_task": "mytask",
"hidden_act": "gelu",
"hidden_dropout_prob": 0.1,
"hidden_size": 768,
"id2label": {
"0": "LABEL_0",
"1": "LABEL_1",
"2": "LABEL_2",
"3": "LABEL_3",
"4": "LABEL_4",
"5": "LABEL_5",
"6": "LABEL_6",
"7": "LABEL_7",
"8": "LABEL_8",
"9": "LABEL_9",
"10": "LABEL_10",
"11": "LABEL_11",
"12": "LABEL_12",
"13": "LABEL_13",
"14": "LABEL_14",
"15": "LABEL_15",
"16": "LABEL_16",
"17": "LABEL_17",
"18": "LABEL_18",
"19": "LABEL_19"
},
"initializer_range": 0.02,
"intermediate_size": 3072,
"is_decoder": false,
"is_encoder_decoder": false,
"label2id": {
"LABEL_0": 0,
"LABEL_1": 1,
"LABEL_10": 10,
"LABEL_11": 11,
"LABEL_12": 12,
"LABEL_13": 13,
"LABEL_14": 14,
"LABEL_15": 15,
"LABEL_16": 16,
"LABEL_17": 17,
"LABEL_18": 18,
"LABEL_19": 19,
"LABEL_2": 2,
"LABEL_3": 3,
"LABEL_4": 4,
"LABEL_5": 5,
"LABEL_6": 6,
"LABEL_7": 7,
"LABEL_8": 8,
"LABEL_9": 9
},
"layer_norm_eps": 1e-12,
"length_penalty": 1.0,
"max_length": 20,
"max_position_embeddings": 512,
"min_length": 0,
"model_type": "bert",
"no_repeat_ngram_size": 0,
"num_attention_heads": 12,
"num_beams": 1,
"num_hidden_layers": 12,
"num_return_sequences": 1,
"output_attentions": false,
"output_hidden_states": true,
"output_past": true,
"pad_token_id": null,
"pooler_fc_size": 768,
"pooler_num_attention_heads": 12,
"pooler_num_fc_layers": 3,
"pooler_size_per_head": 128,
"pooler_type": "first_token_transform",
"pruned_heads": {},
"repetition_penalty": 1.0,
"temperature": 1.0,
"top_k": 50,
"top_p": 1.0,
"torchscript": false,
"type_vocab_size": 2,
"use_bfloat16": false,
"vocab_size": 21128
}
If I will use this model ,the config.json will be loading, and these params shoule be modified?
As I know, the num_hidden_layers is not 12.

Hi,
Sorry for the late reply. Our saved model is not compatible with the original huggingface transformers yet. But you can manually rename scc_layer to layer and modify the configure file accordingly.
We’ll provide a conversion script later.

Hi,
Sorry for the late reply. Our saved model is not compatible with the original huggingface transformers yet. But you can manually rename scc_layer to layer and modify the configure file accordingly.
We’ll provide a conversion script later.

Thank you. I have been translate these model to the tensorflow 1.x , and loading model with google bert code.


Fixed compile error

Hi, I found that there is a compile error in Uart8Transmitter.v:24 and I fixed it.
Could you please check if this commit can be merged in your repository?
It looks good to me but needs your opinion. :)
Thanks!

And it seems that the transmitter should send stop bit to 1 but the code is missing, so I fixed it.
Could you please check if it is okay?


Add Chinese docs for tutorial/testing.md

including:
tutorial/testing.md
And more will be added later.

including:
tutorial/testing.md
And more will be added later.
Are you still updating this section Bro?

It seems this was translated at some other PR, so I'll close this one, but thanks for the effort! 🍰


e2e login test

e2e login test

@devkiran
Please I need your review and below is what I'm trying to do

In docker-compose.yml I created a test database to use for the tests
In logn.spec.ts when the Sign in button is clicked. It is not redirecting to the dashboard. When the button is clicked the loading is still true therefore not navigating to the dashboard. This error as well [Failed to load resource: the server responded with a status of 401 (Unauthorized)]

Screenshot of screen
Screenshot

Thanks @emmacode for the PR. Let's review this and get back to you soon.

@emmacode Can you run and check if you have the same issue now? I have made some changes.


make HOST_NAME_MAX an alias of MAXHOSTNAMELEN

HOST_NAME_MAX is not available on Solaris/*BSD systems. However we
can use MAXHOSTNAMELEN on those platforms instead.

Sounds ok


[BUG]

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
如果删除一个带有自定义标签的主机，那么删除主机的同时，标签管理中的标签也会被删除。
Expected Behavior
no
Steps To Reproduce
no
Environment
no

Debug logs
no
Anything else?
no

fix by v1.3.1


Customize initial state of automation

Description:
Possible to set an automation to be disabled when HASS starts
Related issue (if applicable): fixes https://community.home-assistant.io/t/automation-initially-off-and-persistent/3831
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#
Example entry for configuration.yaml (if applicable):
automation:
  - alias: 'Test automation'
    initial: 'off'
    trigger:
      platform: state
      entity_id: switch.kitchen
      state: 'off'
    condition:
    - condition: state
      entity_id: switch.kitchen
      state: 'off'
    action:
      service: light.turn_off
      entity_id: light.diningtable_left

Checklist:
If the code does not interact with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] Tests have been added to verify that the new code works.

Fixed merge conflict from https://github.com/home-assistant/home-assistant/pull/3663
@balloob Could you have a look?

Can merge when name updated 🐬


feat: make CNI plugin to work with Kuma

Adjusted CNI configuration to provide argument with the type
of service mesh merbridge is working with and updated code
to make CNI plugin work with Kuma.

Adjusted tests for plugin
added mac's .DS_store files to .gitignore file

Working on style-check


User f-strings instead of str.format()

Make ruff happy

nice! this has also been on my TODO list for a while :)


Support M1 Macs in python bindings

This PR addresses the issue of using the python bindings on an M1 Mac (#100, #140).
I'm not sure on how the build process works for the python bindings (is there a CI job?), but I managed to build the necessary binaries locally and integrate them into the package.

We need to align this change with https://github.com/microsoft/BlingFire/pull/147 .
I will talk to Lei to add M1 Mac (OSX-arm64?) pipeline to create a binary dll.
His current approach is to use the same DLL/SO/DYLIB name but place them into a different directory. Your approach is to create a different file name for arm64. I am ok with either approach but 1. we need to make sure it is consistent, 2. let me know if I miss any good points why you did this way, so I talk to Lei.

Patrick, I have talked to Lei (the owner of https://github.com/microsoft/BlingFire/pull/147) he will try to add Mac Arm64 and Linux Arm64 as well (so it is all consistent), then I will update the python package to use it, if all works well then it will be ready in couple of weeks. If keeping the same name between arm64 and x64 does not work, we will use your solution.

Thank you for addressing this! I'll close this PR, as it is now a duplicate of #147.


Is there a way pass undefined params and the shell script will not FATAL and exit

Sometimes, we may pass some undefined params to shell script, is there a way(such as a flag) to ignore the undefined params, and the script will run success?
Now the script will have a FATAL error and exit when we pass undefined params

This is covered in the Quick Start, but maybe that could be improved or documented elsewhere. Where did you look (and not find) an answer?
This is a copy-and-paste from the current 1.3 docs (linked above). Let me know if you have improvement suggestions.
# parse the command-line
FLAGS "$@" || exit $?
eval set -- "${FLAGS_ARGV}"

Here is where all the magic happens. The call to FLAGS with the passing of "$@" sends all command-line arguments into the shFlags library for processing. If everything is successful, a return value of 0 (${FLAGS_TRUE}) is returned an things continue. If a false value of 1 (${FLAGS_FALSE}) or an error value of 2 (${FLAGS_ERROR}) is returned, the processing is halted, returning that value (the exit $?). If there are any command-line arguments left over that shFlags didn't know how to process, they are now present in the now updated $@ variable, which can be used as with any other script.

Thanks for your answer, but that's not my situation. In my case, for example, examples/hello_world.sh, we can run as :
$ ./hello_world.sh -n Kate
Hello, Kate!

But if I add a -c which is not defined in hello_world.sh, it will raise a error:
$ ./hello_world.sh -n Kate -c example
flags:WARN getopt: invalid option -- 'c'
 -n 'Kate' -- 'example'
flags:FATAL unable to parse provided options with getopt.

Is there a way not raise a error in this situation?


Running npm install after npx gts init

So I've done this a few times now.    This always fails with an error like this:
error TS5058: The specified path does not exist:
 '/users/beckwith/code/testy/node_modules/gts/tsconfig-google.json'.

Then I remember to run npm install, and everything is fine.  Is it possible to kick off an npm install after npx gts init modifies my package.json?

+1. Good idea. PRs welcome :).


Pub yaml file should allow tabs

When I use pub with a yaml containing tabs, I get:
Error on line 2, column 1 of pubspec.yaml: Tab characters are not allowed as indentation.
    dquery: any
^

Why not?  Seems rather arbitrary.  Tabs are convenient to line things up.  Can't you treat tabs as just some space characters?

http://stackoverflow.com/questions/19975954/a-yaml-file-cannot-contain-tabs-as-indentation

Yup http://www.yaml.org/faq.html


add unique constraint on asset_tag if asset has no owner

and move all asset's unique_together to model contratints
see #149

If assets table contains assets with duplicate asset tags and not owner, DB migration will fail with:
Applying netbox_inventory.0007_alter_asset_unique_together_and_more...
...
django.db.utils.IntegrityError: could not create unique index "unique_asset_tag"
DETAIL:  Key (asset_tag)=(1110) is duplicated.

To hopefully prevent this from happening to users, I'll hold on merging this PR till the next major version. In the meantime I'll add a management script and a notification to the release notes to the preceding releases to allow people to check and fix their data beforehand.


Feature Request: Qlab OSC passcode

Describe the feature
The ability to use the OSC passcode in Qlab.
Is this platform dependent (windows, mac, ..)?
MAC (Qlab)
If documentation is required to implement, do you know where to find it?
https://figure53.com/docs/qlab/v4/scripting/osc-dictionary-v4/
Usecases
Use passcode to prevent other than the operator to trig lab thru OSC

Module is ready for merging. then option for passcode is there

It's in there :-)


Update packages, README.md and a few other minor tweaks

Did not update infobloxopen/infoblox-go-client as it has changed substantially and will require quite a bit of rework.
Tests are currently broken.  I looked at fixing them but kubebuilder has changed substantially and I've never used it before and don't know what it was trying to do.
Took the initiative to change how versioning works for the Docker image.

Run make build to build the image then
Run make push to push the image.  This will tag the versioned image to latest and push both of them.

This looks promising, planned to test this webhook next week and also was looking on bumping versions first. Will let you know how it goes.

Hi, any update on this?  Probably needs updated again at this point.

There is #9 which takes care of some stuff.

@bb-Ricardo
OK . . but #9 also hasn't been merged.  Any plans to do that?
Are you going to make this a priority or should we continue development on our own?

Hi @sarg3nt,
This is not my repository and therefore I'm not able to merge it.
Only the owner (and other maintainers) are able to merge these PRs.

@bb-Ricardo Sorry about that.  I meant to @ the repo owner.
@luisico Any update on this?
Thank you.

Hi, I no longer have access to an environment with k8s and info lox (changed position) and don't feel I can support this repo. If you would like to maintain it, I would be happy to put a notice in the readme pointing to it's successor. Sorry.

@luisico   I'm fine with taking this over.  I'm not sure which org it's going into yet.  I have a text out to my boss to see if it should go in our corporate repo or in my personal one.   Keep this open and I'll let you know when we figure it out.

I added a deprecation notice in the readme. If you end up taking it over, please, let me know and I'll update with a link to the new repo. In the meantime, I'm closing all issues and pull requests and making the repo read only.


-unittest requires an enormous amount of memory to compile

Having a mere import argparse; in a program and compiling it with -unittest compiles and runs all of the argparse’s tests. Or at least attempts to.  I have no idea how hungry it is in reality.
In my opinion, inspecting some UDAs shouldn’t involve so much work. I’ll have a look at the code and see if I can figure out what is going on there.

As I suspected, almost all metaprogramming is done with template instantiation, although many things can be moved to CTFE. The compiler is required to cache all (successful) template instantiations for every unique combination of arguments—thatʼs why -lowmem didnʼt help.
Working on this; will send a PR as soon as I am able to run tests.


Events details summary

Adding teacher and student numbers to event details: https://github.com/isaaccomputerscience/isaac-cs-issues/issues/162

This was already approved but subsequently added several tests and a few other small changes


Advertize PostgreSQL 13 and Ubuntu Groovy support

#17170 added support for these, but not documentation.  We should add them to the list of documented versions that Zulip supports.

Ubuntu Groovy support is only for the development environment.  I guess the Postgres 13 support is probably real, but I'm not sure we've actually tested it at all?
I'm not sure how useful it is to document Groovy -- it reaches EOL in July (2-3 months from now).  I think it's reasonable to have a policy of not bothering to document Ubuntu non-LTS releases given how short their support lifetime is?
@ligmitz FYI.

Yeah, non-LTS releases have a life support of about 9 months only, so I think it is not quite useful to give production support for them (due to short life). The Zulip support for Groovy is only for development. We added support for Debian Bullseye recently but it is not documented here, should we document that as a supported platform there now?

#18271 swaps the default production PostgreSQL to 13, and thus tests it as well as upgrades to it.  Not documenting Groovy makes sense to me, given its short support schedule.  Documenting Bullseye should probably wait until it's actually officially released, I think?  But we can prep a PR that makes it easy to merge as soon as it does.

@alexmv this was fixed by #18271, right?

I think the only thing to possibly do is decide if we're advertizing Bullseye in 4.0, or holding off until it gets a release.  I think I'm in favor of "it will work when it's released, even if we don't list it; we can start officially listing it in 4.1 if it doesn't go out by 4.0 final."

Yeah, I think that policy makes sense to me, especially since it doesn't actually work without an /etc/os-release that will only be normal once it is released.


Changing all non trivial operations that use broadcasting with einsum where possible

Pull Request Summary
Provide a brief description of the PR's purpose here.
Key changes
Notable points that this PR has either accomplished or will accomplish.

[ ] Change 1

Questions

[ ] Question 1

Associated Issue(s)

[ ] Issue 1

Pull Request Checklist

[ ] Issue(s) raised/addressed and linked
[ ] Includes appropriate unit test(s)
[ ] Appropriate docstring(s) added/updated
[ ] Appropriate .rst doc file(s) added/updated
[ ] PR is ready for review

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

Project coverage is 85.41%. Comparing base (c1430f2) to head (64e4cfa).

Additional details and impacted files


Update tapir-core, tapir-json-circe, ... to 0.17.5

Updates

com.softwaremill.sttp.tapir:tapir-core
com.softwaremill.sttp.tapir:tapir-json-circe
com.softwaremill.sttp.tapir:tapir-openapi-circe-yaml
com.softwaremill.sttp.tapir:tapir-openapi-docs
com.softwaremill.sttp.tapir:tapir-swagger-ui-http4s
com.softwaremill.sttp.tapir:tapir-zio
com.softwaremill.sttp.tapir:tapir-zio-http4s-server

from 0.16.16 to 0.17.5.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.softwaremill.sttp.tapir" } ]

labels: library-update, semver-minor

Superseded by #167.

Superseded by #167.


Allow Manual Sorting of Cards Within a Column

Creating based on a discussion on #15.
It would be nice to be able to manually sort the cards within a column. Probably could only be implemented once #11 was at least partially functional.
There are some tricky pieces to this:

How to store the order (decimal field on task line, separate datastore, indented field under task, something else)
How to handle cards that exist on multiple boards

I'll give my use case and others can add theirs if different and they find value in this. I keep multiple time-based columns after my Backlog column (This Month, Next Week, This Week) where I can start moving tasks that I want to get done soon instead of just having a really long backlog. But within each of those columns I like to prioritize the tasks, moving cards to the top of the list that are more important. I can then reference the top of those very quickly if I finished up a column further to the right.
However, even with just a really long backlog I would want to prioritize a bit as part of my task review process.
My best current idea that could handle multiple boards would be to include an indented line under the task that includes the board name plus a decimal order.
So, something like this:
- [ ] This is a task #backlog
  - @order(board name, 1.1)

Not sure if board name would need to be quoted if it had spaces. But this is just to get something out there for discussion. If this were the route taken (or something similar) I would not want the @order line to be shown on the card at all.

Thank you :)
This is definitely something I want to do so am very interested in use-cases/suggestions etc.
Was thinking on it last night and I would probably aim to get the complete drag-and-drop functionality done in one release - so this would include within and between columns as (assuming there is a good understanding of how it will work) it won't be too hard to get all done if I get some of it done (I think).
I'd rather keep the discussions separate tho as there are different considerations for within and between column.
Thanks for adding !

+1 for this

+1 upvote this as well!

Will need some thought on where this order information is stored.  It could be on the task line, near it (as above), in front matter, in a special comment at the end of the page, stored in some specified separate note, in the config for the plugin, or in a database (not sure what would be available - sqlite or is indexedDB available)....

Would love this functionality :)
As a person with tasks everywhere in my vault, when working out which tasks to do next, I would like to change the order of the backlog quickly so that I can prioritize high value tasks.

+1 upvote for this
Your plugin has some nice features I like, but without drag and drop sorting within a column I probably won't use it.


One migration pending in 1.1.0

Migrations doesn't reflect the current Request#prof_file attribute, so one migration is still pending to be generated. That difficulties working with the library in some CI and automated environments

Isn't 1.1.0 a very old release?

Isn't 1.1.0 a very old release?

Hehe I know, but we have a legacy environment with django 1.7 and python 2.7...

Isn't 1.1.0 a very old release?

Hehe I know, but we have a legacy environment with django 1.7 and python 2.7...

This migration already exists in 1.1.0 https://github.com/jazzband/django-silk/blob/c9438044e59ae20da2406958b043a0f7fdce67e8/silk/migrations/0005_increase_request_prof_file_length.py

Isn't 1.1.0 a very old release?

Hehe I know, but we have a legacy environment with django 1.7 and python 2.7...

This migration already exists in 1.1.0 https://github.com/jazzband/django-silk/blob/c9438044e59ae20da2406958b043a0f7fdce67e8/silk/migrations/0005_increase_request_prof_file_length.py

Yes, migration #5 is changing Request#prof_file attribute, but the problem is with storage attribute, is not evaluated in previous migrations, so in migration #6 is taking the default value:
defaults = {
...
'SILKY_STORAGE_CLASS': 'silk.storage.ProfilerResultStorage'
}
I do not know if you want to always have previously made makemigrations before using the library, but if that is the case it makes it difficult to work in certain automated environments, requiring additional steps to have the library ready to use.
Any criticism is welcome, I hope to be useful :)

Test cases are failing, please fix them. Even if we want to merge this one we can't backport it to 1.1.0.
That being said it would only be available in 4.0.1. I'll test it once the tests are passed.

Isn't 1.1.0 a very old release?

Hehe I know, but we have a legacy environment with django 1.7 and python 2.7...

This migration already exists in 1.1.0 https://github.com/jazzband/django-silk/blob/c9438044e59ae20da2406958b043a0f7fdce67e8/silk/migrations/0005_increase_request_prof_file_length.py

Yes, migration #5 is changing Request#prof_file attribute, but the problem is with storage attribute, is not evaluated in previous migrations, so in migration #6 is taking the default value:
defaults = {
...
'SILKY_STORAGE_CLASS': 'silk.storage.ProfilerResultStorage'
}
I do not know if you want to always have previously made makemigrations before using the library, but if that is the case it makes it difficult to work in certain automated environments, requiring additional steps to have the library ready to use.
Any criticism is welcome, I hope to be useful :)

So It was fixed later in v2.0.0 https://github.com/jazzband/django-silk/commit/5ba085368b13a79f92db1b874ecf5d4ad083450e
If you want to use it with v1.1.0 Please take a fork from v1.1.0 release tag and cherry-pick this commit on top of it 5ba085368b13a79f92db1b874ecf5d4ad083450e

Thank you, I made a mistake with branches bringing master...


Add link to TidyTuesday examples point to Tidy Data paper

I would like to add text and a link to the end of the first paragraph on the first TidyTuesday example that would point to Hadley Wickham's Tidy Data paper.
Before:
We will use pyjanitor to showcase how to conveniently chain methods together to perform data cleaning in one shot. The below example shows a one-shot script followed by a step-by-step detail of each part of the method chain.
Addition would include the following:
Note: TidyTuesday is based on the principles discussed and made popular by Hadley Wickham in his paper Tidy Data.
I would be interested in working on this.

We have a pull request open that modifies the TidyTuesday examples. You may want to wait until those are merged before making any changes.

The other pull request was merged, so feel free to pull and work on this now.


Fix catkin failed build when testing is not enabled

On a clean catkin_build I get thrown this error:
Errors     << steering_functions:check /home/mcfurry/ros/melodic/system/logs/steering_functions/build.check.000.log
CMake Error at /opt/ros/melodic/share/catkin/cmake/test/tests.cmake:18 (message):
  () is not available when tests are not enabled.  The CMake code should only
  use it inside a conditional block which checks that testing is enabled:

  if(CATKIN_ENABLE_TESTING)

    (...)

  endif()

Call Stack (most recent call first):
  CMakeLists.txt:207 (catkin_add_gtest)

make: *** [cmake_check_build_system] Error 1

This fixes this.

I never experienced that error before. How about you, @Timple?

The error seemed self-explanatory. But interestingly it doesn't reproduce in a default setting. Tested using Dockerfile below:
FROM ros:noetic
RUN apt-get update -qq && apt-get install -qqy python3-catkin-tools python3-osrf-pycommon git
RUN git clone https://github.com/hbanzhaf/steering_functions src/steering_functions

# Install dependencies used by packages
RUN rosdep update && rosdep install -y -i --from-paths src

# Compile ROS package
RUN bash -c 'source /opt/ros/*/setup.bash && catkin build'

But this layout is a-typical for a ROS package (see catkin_lint output). So probably doesn't hurt to conform to the standard to prevent issues in edge-cases.

The catkin build command by default enables testing, but in several usecases testing is not enabled.
Hence it reproduces with:
FROM ros:melodic
RUN apt-get update -qq && apt-get install -qqy python-catkin-tools git
RUN git clone https://github.com/hbanzhaf/steering_functions src/steering_functions

# Install dependencies used by packages
RUN rosdep update && rosdep install -y -i --from-paths src

# Compile ROS package
RUN bash -c 'source /opt/ros/melodic/setup.bash && catkin build -DCATKIN_ENABLE_TESTING=off'

If the default is ON, it's probably safe to merge this then. No regressions for current users (default ON). And for users who have testing OFF, it failed anyway to compile.


Create new favicon

Created a new favicon which should close this issue:

https://github.com/VerzatileDevOrg/Programming_HandBook/issues/6

You can view the favicon here:

https://raw.githubusercontent.com/VerzatileDevOrg/Programming_HandBook/cfbfeac5095ad3aaa7004fe499538c81708b94f5/favicon.ico


Swap players.lua code

Describe your problem here.
Did I can make the lua function what can swap the player1 to player2?, just like an opponent mode?. Can someone tell me that its possible with lua Events/scripts?.
And I saw that is hazy river mod have Note swap Event. But of course its swapping only notes. But I want swap the player to Opponent. Can someone help?
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No response

Im too lazy to send it here but I know someone did it in the Psych Engine discord server, look it up there.

Im too lazy to send it here but I know someone did it in the Psych Engine discord server, look it up there.

There's a discord server? :0

Im too lazy to send it here but I know someone did it in the Psych Engine discord server, look it up there.

There's a discord server? :0

yea https://discord.gg/2ka77eMXDv

Im too lazy to send it here but I know someone did it in the Psych Engine discord server, look it up there.

Omg Thank you so much!. I'll check this out!.

There's a discord server? :0

yea https://discord.gg/2ka77eMXDv

Ty!

I found what I want!!, Thank you so much again MaybeMaru!.
And if the someone too what to download it:
EnemyPlay.txt