I miss more documentation

Good job, I'm really happy with this library, I like much more than RAC 3.0. I am new to the FRP world and I miss some more documentation and simplest example.The GitHubSignup example is quite clear but WikipediaImageSearch see it a little complicated.
I miss documentation in the code.

Perhaps this example is too simple, but if I can understand the complete library I would like to make a number of easy to difficult examples of use.

¿If it's not too much trouble and is not too much you could implement this example or something similar with your library?

I do think that the project needs a lot more documentation and examples.
I'm interested in hearing your pain points while using this project or reading the code.
Since this kind of feedback has significant value to this project, and this is probably inefficient way of collecting all of the pain points, feel free to contact me on my email and setting up some kind of meeting.
I'm really interested in collecting this valuable feedback more efficiently and providing you an explanation for all of the confusing parts.
Kruno

Although the documentation is great, I ran across a point of confusion for several people in the comparison to RAC section.
I've seen a discussion where people interpreted this section as stating that Rx described sequences of events while RAC was implemented as signals, and RxSwift as sequences.
They argued that RAC was actually implemented as sequences of events while the original Rx was about signals (the other way around).
I'm not familiar enough with RAC nor RxSwift to know better, but that got me interested as my understanding of Rx is that it deals with sequences of events, and I understood the aforementioned documentation section to state that Rx was about sequences, as opposed to the concept of signals, but I don't see it stating explicitly RAC is implemented as signals either.
So I'm curious of which it is, and of the concrete differences with RAC in general, and think that the project would benefit greatly from making this clearer :)

In the "Introduction to Rx" book, the word "signal" appears a grand total of 5 times whereas the word "sequence" appears several hundred times...
I think Rx is about sequences, not signals.

Hi @desseim,
@dtartaglia is correct. From what I understand, the abstraction underlying Rx is sequence.
    I also didn't see word signal mentioned anywhere in Rx.Net source code repository.

    The main class in RAC is called signal :) and sequences aren't mentioned anywhere. I totally understand your confusion, and has the same doubts some time ago, but it looks like RAC signals have some different properties then sequences.

    According to RAC 3.0-beta6, signals can change ordering of values (otherwise observeOn operator has bug for concurrent schedulers). RAC 3.0-beta6 also has a concept of interrupted signal, which is really hard to fit in abstraction of sequences.

    Unfortunately, I don't know if observeOn has a bug or it's possible for signals to change ordering of elements. I couldn't find documentation anywhere specifying properties of signals, so you will need to ask RAC project maintainers for more info about this.

    Hope this clears things up for you.

Thanks, this matches with my understanding as well (regarding Rx).
What confused me was a RAC maintainer stating that the allegation about signal values arriving out of order was "not true" and a misunderstanding of how RAC works (it was on the slack RAC channel in case anyone is interested).
As someone new to both RAC and RxSwift I want to understand the difference better and I think it would be nice to make it clearer for anyone else in this situation. As RAC seems to be confusing a few terms, anyone looking for FRP or reactive extension on iOS or Cocoa is likely to end up there, so I think it'd make sense to strengthen that README part about the differences between RAC and RxSwift.
I could certainly see a clearer attribution of this signal and ordering examples to RAC (see my original post) and links to sources to back up these assertions beneficial to the clarity of the README.
It may also just be me and anyone with a good experience of RAC wouldn't care as much though, so just take it as a suggestion :)

Hi, I agree about a need to strengthen the difference between RAC and RxSwift, and I do believe it's confusing right now.
The more I think about it, I think that best thing to do would be to write a couple of examples of parallel RAC and RxSwift code.
I never intended to say that RAC signals can swap element orderings for sure, I only meant to say that it's a confusing concept and that they don't define properties of signal anywhere, so I have no idea can signal elements actually do that, and is that a valid signal. The fact that you've had to contact one of the team members proves that the confusion is real.
However, I will remove those sentences about changing orderings from documentation because I don't want to mislead people. I've definitely learned a lot from RAC team, and wouldn't want to cause any damage to their reputation in this way. This change will be released in a day or two with next release 1.5
Parallel examples will have to wait until 1.6 or 1.7.
@desseim appreciate the feedback
Kruno

@desseim Sry, I'm now reading my last response, I've actually said that they change ordering values. That was a mistake of mine.

@kzaher Wholeheartedly agree with each one of your points.
And thanks for planning on making things clearer !

Hi guys,
this issue was opened a long time ago :) I feel comfortable closing it now. If we still feel there are some specific areas that need specific documentation attention, please nominate those areas in issues.
Since it was opened lot of things have happened.

We've been featured on http://reactivex.io/, and since we were trying to consolidate our efforts with broader Rx community anyway, we'll reference that portal as our official documentation portal, and unify all naming.
I see this as an excellent way or defining documentation/naming source of truth.
The only thing this project still needs to do is document peculiarities of Swift integration.
But I think that huge steps forward have been made toward unification. RxSwift operator names are also being added to that portal, so documentation is improving significantly (@carlosypunto has also helped us with that).

Since there were some complaints regarding the way we compared ourselves with ReactiveCocoa, I've removed that comparison to avoid any further confusion, and accidentally cause any damage to that project.
For now, our README.md contains basic difference between those two projects, and other similar projects.
There will be further improvements to documentation in the future.
As far as for more detailed description of differences, I'm hoping that with time community will create enough materials to help people more easily decide which one to use.
This project's documentation will focus in the future on communicating the strengths of RxSwift in a better and clearer way :)
Unfortunately, we'll just need some time to do it properly.


Sending a zero-valued transaction panics

This is due to the fact that a state object whose value doesn't change, will not cause a tree update in updateStateObject, but the AccessWitness will contain the key and so an attempt will be made to create a proof for this leaf.

Fixed after the move to IPA.


in init setup- offer to add repo key to Cargo.toml for those missing it

the number one error i see with end users is that they realize they are missing the repo key in their Cargo.toml or that it is incorrect. in general, cargo-dist has the info it needs to update this (as it is almost certainly running in a local git checkout) - for situations where we can safely and confidently offer to update Cargo.toml- i think we should do so in init
current:
✔ enable Github CI and Releases? · yes

  × Github CI support requires you to specify the URL of your repository
  help: Set the repository = "https://github.com/..." key in your Cargo.toml

proposal:
✔ enable Github CI and Releases? · yes

  ? You haven't specified a repository key in your Cargo.toml Set it to <repourl from local checkout>? · yes

whoever does this should duplicate the logic to also fill in the "authors" (maybe actually prompt for this like we do for npm scopes), that's the #1 error with msi


Middleware class must be registered as singleton if it has terminate method

If middleware is not registered as singleton within container and it has terminate method, HttpKernel will create a new instance of your middleware and call terminate on that instance, losing your previous instance properties.
https://github.com/laravel/framework/blob/5.1/src/Illuminate/Foundation/Http/Kernel.php#L143
Have not tested this on Laravel, only on Lumen, but pretty sure it's the same.
To fix, either make a note about this in documentation so users can take care of this by themselves or register all passed middleware as singletons for users during bootstrap.

Not sure how we can actually fix this?

register all passed middleware as singletons for users during bootstrap.

That would be a bad idea, as it could destroy existing registrations.

It's up to users to deal with the correct bindings them selves.

Yes I agree with you. But can we at least update the documentation to let users know that if they want terminate to work correctly not to forget to bind the middleware as singleton. Not everyone has knowledge and time to dig through source code to figure out why on Earth it's not working.
If you give me thumbs up for this I'll send PR to laravel/docs.

If you give me thumbs up for this I'll send PR to laravel/docs.

Not really up to me, though I don't think anything needs changing on the docs.

Argh.. this is a big problem for me. Thanks for pointing it out in the docs. Googled that exact line in the docs, took me to stackoverflow, then took me here.
Looking at the Kernel.php.. there doesn't seem to be an easy way to fix this.


PowerShell Beta 9 causes a regression in the PSRP omi provider due to the name change to pswh.

startCoreCLR in coreclrutil.cpp (https://github.com/PowerShell/psl-omi-provider/blob/master/src/coreclrutil.cpp#L191) finds powershell's coreclr binary using the powershell binary. The change in name to pwsh breaks this logic.
$PSVersionTable
Name                           Value

PSVersion                      6.0.0-beta.9
PSEdition                      Core
GitCommitId                    v6.0.0-beta.9
OS                             Linux 4.4.0-97-generic #120-Ubuntu SMP Tue Sep 19 17:28:18 UTC 2017
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

NOTE: Closing this is pending a PSRP release; currently blocked due to 3 issues in libMI.


[CARBONDATA-1900][Core,processing] Modify loadmetadata to store timestamp long value(in ms) instead of formatted date string for fields "loadStartTime" and "timestamp"

Problem:
If  the table is moved to environment having different timezone or we change the system current timezone, after IUD operation some of the blocks are not treated as valid blocks.
[{"timestamp":"15-12-2017 16:50:31:703","loadStatus":"Success","loadName":"0","partitionCount":"0","isDeleted":"FALSE","dataSize":"912","indexSize":"700","updateDeltaEndTimestamp":"","updateDeltaStartTimestamp":"","updateStatusFileName":"","**loadStartTime**":"15-12-2017 16:50:27:493","visibility":"true","fileFormat":"COLUMNAR_V3"}]
part-0-0_batchno0-0-1513336827493.carbondata
If timezone is different than at the time load was done, the value calculated from loadStartTime "15-12-2017 16:50:27:493" will not match to the time stamp extracted from block file name.
Solution:
We should stop writing the loadStartTime and timestamp in "dd-MM-yyyy HH:mm:ss:SSS" format.
We should write the long value of the timestamp as given below.
[{"timestamp":"1513336827593","loadStatus":"Success","loadName":"0","partitionCount":"0","isDeleted":"FALSE","dataSize":"912","indexSize":"700","updateDeltaEndTimestamp":"","updateDeltaStartTimestamp":"","updateStatusFileName":"","loadStartTime":"1513336827493", "visibility":"true","fileFormat":"COLUMNAR_V3"}]

[X] Any interfaces changed?

[X] Any backward compatibility impacted?
For old loaded if string to Long parse fail, will fall back for date parsing.

[X] Document update required?
For old loaded table we can add limitation that the table movement should be done
across the same timezone environment.

[X] Testing done
Please provide details on
- Whether new unit test cases have been added or why no new tests are required?
- How it is tested? Please attach test report.
- Is it a performance related change? Please attach the performance test report.
- Any additional information to help reviewers in testing this change.
Manually tested

[X] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.
NA

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/2005/

Build Success with Spark 2.2.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/779/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/2325/

This PR is ok, no code change after first push only pushed the commit message change
@ravipesala Please review

Build Success with Spark 2.2.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/796/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/2338/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/2029/

Failed Test in SDV build is unrelated, its randomly failing with other PR #1659,
but retested got paased.

retest this please

Build Success with Spark 2.2.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/801/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/2032/

Build Success with Spark 2.2.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/860/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/2085/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/2382/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/2383/

Build Success with Spark 2.2.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/864/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/2089/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/2387/

retest this please

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/2117/

Build Success with Spark 2.2.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/893/

LGTM

LGTM


Add phar files

Hi @beberlei ,
Why are you uploading tar.gz file. May be upload phar itself as you did in first time ?

Github says phar is not allowed, so i wrap it.


Async problem while passing property into child component

I have two component (grid) is the parent which contain an array of emotions which use v-for to initiate multiple (emotion) component.
fight is a computed property that depends on data from the data property using AJAX

        <emotion :emotion=item></emotion>

this is the emotion component
<template id="emotion">

</template>

module.export = {
  props: ['emotion']
}

errors I receive in console
[Vue warn]: Error when evaluating expression "emotion.name".
VM54101:3 Uncaught (in promise) TypeError: Cannot read property 'name' of undefined(…)

I require the emotion component (child) in the parent component, which seems to be rendered before it has data to work with.
Thanks in advance.

Hmm, <emotion> should not be rendered if list is empty. Can you add {{ list | json }} and see what value it has?

Better yet - create a repro if possible.

Thanks for the prompt response
The problem was a typo
module.exports instead of module.export
Thanks so much


Prepare a batch of releases.

[x]<EMAIL_ADDRESS>[x]<EMAIL_ADDRESS>[x]<EMAIL_ADDRESS>[x]<EMAIL_ADDRESS>[x]<EMAIL_ADDRESS>[x]<EMAIL_ADDRESS>[x]<EMAIL_ADDRESS>[x]<EMAIL_ADDRESS>[x]<EMAIL_ADDRESS>[x]<EMAIL_ADDRESS>[x]<EMAIL_ADDRESS>[x]<EMAIL_ADDRESS>[x]<EMAIL_ADDRESS>

No updated version of polyserve? I think it includes fixes to NPM warnings.

No updated version of polyserve? I think it includes fixes to NPM warnings.

Oh good point, I'll add the packages with NPM warnings. Was only looking at CHANGELOGs.

Awesome thanks!

All releases done! @TimvdLippe how are we looking with the NPM audit issues now?

@hastebrot wct-browser-legacy and web-component-tester still have vulnerabilities, I am working on fixing them.
@aomarks Audit issues in all other packages are resolved :tada: However, the lockfiles are broken again, as the npm run nuke script is not working correctly. See https://github.com/Polymer/tools/pull/448#discussion_r198787654 for the fix.

@TimvdLippe can I ask you to also look at Polymer/polymer-modulizer#452 later if you have time?
It also contains a fix for some vulnerabilities issues in modulizer.

@TimvdLippe that's a valid point. I recall some PR to import polymer-workspaces (used by modulizer) which has been postponed.
At least it would be nice to have another modulizer release before importing it here, as there are at least 2 merged but not released fixes.


update dependencies

The repo is outdated with respect to aiida-core v2.6
I need this update to perform some performance test for aiida-firecrest
The main changes in this PR are as follows:

$\to$ aiida-core v2.6

orm.Group.objects.get $\to$ orm.Group.collection.get
orm.Group.objects.get_or_create $\to$ orm.Group.collection.get_or_create

$\to$ pydantic v2.8.2:

@validator $\to$ @field_validator
~allow_reuse=True~  not needed anymore

/examples/add_in_batches.py

added load_profile()
a condition to break the loop once everything is calculated.

Thanks @khsrali! I had made similar changes locally, but still wanted to have a v1.X-compatible release before updating everything, but just never got around to it.

Thanks @mbercx, connection on the glacier seems to be really good, ha.
why not just tag the current status and move one?

Btw, there is also the pydantic-v2 branch by @unkcpz. So based on the number of people who have already implemented these changes themselves, I agree that we should push to get this merged 😄

Thanks again @khsrali! If you're ok with it, I'm going to split this PR up in 3 commits:

Upgrade pydantic to ~=2.8 (I don't see a reason to be compatible with a patch version?)
Upgrade aiida-core to ~=2.5 (I think only from this version aiida-core uses pydantic v2)
Upgrade Python versions to 3.9 -> 3.13 (3.8 is EOL and 3.13 is released).

Note that if we merge this PR, anyone using aiida-core<2.5 will no longer be able to use the aiida-submission-controller because of the pydantic conflict. So I'd be inclined to still release a pydantic v1 version first.

@mbercx all good, do whatever you think is convenient :+1:


New logo?

I've designed a new logod. The gear icon is taken from the rust logo
Before

After

Don't worry about the size diff

UPDATE: I changed the logo a little so that the W is a little lower

I now made the final version


has text relocations

Caused by: java.lang.UnsatisfiedLinkError: dlopen failed: /data/app/com.qd.videorecorder-1/lib/arm/libavcodec.so: has text relocations

I have a same problem with u  Have u solove this problem yet. If u do Pls inform me tks

try change targetSdkVersion to 22


Error LNK2005 & Fatal Error LNK1169 when packaging with other plugins

I am currently using your plugin alongside 2 others, OVRLipSync as well as JsonCsvToDataTableToStruct and they managed to package fine without your plugin in the project.
But when I include your plugin into the project it fails to do a successful package. Attached is the error log from when it failed to package.
Would you have any idea why this is happening? I'd appreciate all the help I can get!!
LogUObjectHash: Compacting FUObjectHashTables data took   1.13ms
LogMainFrame: Project requires temp target (SunPosition plugin is enabled)
LogLauncherProfile: Unable to use promoted target - ../../../../../../Users/MyComputer/Desktop/TestingProject/Binaries/Win64/UE4Game-Win64-Shipping.target does not exist.
UATHelper: Packaging (Windows (64-bit)): Running AutomationTool...
UATHelper: Packaging (Windows (64-bit)): Parsing command line: -ScriptsForProject=C:/Users/MyComputer/Desktop/TestingProject/MotionTrackTest.uproject BuildCookRun -nocompileeditor -installed -nop4 -project=C:/Users/MyComputer/Desktop/TestingProject/MotionTrackTest.uproject -cook -stage -archive -archivedirectory=C:/Users/Dex-lab/Desktop/3-Combo-Plugin -pac
kage -ue4exe="C:\Program Files\Epic Games\UE_4.27\Engine\Binaries\Win64\UE4Editor-Cmd.exe" -clean -compressed -ddc=InstalledDerivedDataBackendGraph -pak -prereqs -nodebuginfo -targetplatform=Win64 -build -clientconfig=Shipping -utf8output
UATHelper: Packaging (Windows (64-bit)): Setting up ProjectParams for C:\Users\MyComputer/Desktop/TestingProject/MotionTrackTest.uproject
UATHelper: Packaging (Windows (64-bit)): MotionTrackTest.uproject requires a temporary target.cs to be generated (SunPosition plugin is enabled)
UATHelper: Packaging (Windows (64-bit)): ********** BUILD COMMAND STARTED **********
UATHelper: Packaging (Windows (64-bit)): Running: C:\Program Files\Epic Games\UE_4.27\Engine\Binaries\DotNET\UnrealBuildTool.exe MotionTrackTest Win64 Shipping -Project=C:\MyComputer/Desktop/TestingProject/MotionTrackTest.uproject -Clean -NoHotReload C:\MyComputer/Desktop/TestingProject/MotionTrackTest.uproject -NoUBTMakefiles  -remoteini="C:
\Users\Dex-lab\Desktop\IslandHealth_POC" -skipdeploy -nobuilduht -log="C:\Users\MyComputer\AppData\Roaming\Unreal Engine\AutomationTool\Logs\C+Program+Files+Epic+Games+UE_4.27\UBT-MotionTrackTest-Win64-Shipping.txt"
UATHelper: Packaging (Windows (64-bit)):   Cleaning OKI_AnimationTest binaries...
UATHelper: Packaging (Windows (64-bit)): Took 0.8157848s to run UnrealBuildTool.exe, ExitCode=0
UATHelper: Packaging (Windows (64-bit)): Running: C:\Program Files\Epic Games\UE_4.27\Engine\Binaries\DotNET\UnrealBuildTool.exe OKI_AnimationTest Win64 Shipping -Project=C:\Users\Dex-lab\Desktop\IslandHealth_POC\OKI_AnimationTest.uproject  C:\Users\Dex-lab\Desktop\IslandHealth_POC\OKI_AnimationTest.uproject -NoUBTMakefiles  -remoteini="C:\Users\Dex-lab\Desk
top\IslandHealth_POC" -skipdeploy -Manifest=C:\Users\Dex-lab\Desktop\IslandHealth_POC\Intermediate\Build\Manifest.xml -NoHotReload -log="C:\Users\Dex-lab\AppData\Roaming\Unreal Engine\AutomationTool\Logs\C+Program+Files+Epic+Games+UE_4.27\UBT-OKI_AnimationTest-Win64-Shipping_2.txt"
UATHelper: Packaging (Windows (64-bit)):   Using 'git status' to determine working set for adaptive non-unity build (C:\Users\Dex-lab\Desktop\IslandHealth_POC).
UATHelper: Packaging (Windows (64-bit)):   Parsing headers for OKI_AnimationTest
UATHelper: Packaging (Windows (64-bit)):     Running UnrealHeaderTool "C:\Users\Dex-lab\Desktop\IslandHealth_POC\OKI_AnimationTest.uproject" "C:\Users\Dex-lab\Desktop\IslandHealth_POC\Intermediate\Build\Win64\OKI_AnimationTest\Shipping\OKI_AnimationTest.uhtmanifest" -LogCmds="loginit warning, logexit warning, logdatabase error" -Unattended -WarningsAsErrors
-abslog="C:\Users\Dex-lab\AppData\Roaming\Unreal Engine\AutomationTool\Logs\C+Program+Files+Epic+Games+UE_4.27\UHT-OKI_AnimationTest-Win64-Shipping_2.txt" -installed
UATHelper: Packaging (Windows (64-bit)):   LogInit: Display: Loading text-based GConfig....
UATHelper: Packaging (Windows (64-bit)):   Reflection code generated for OKI_AnimationTest in 4.8656366 seconds
UATHelper: Packaging (Windows (64-bit)):   Writing manifest to C:\Users\Dex-lab\Desktop\IslandHealth_POC\Intermediate\Build\Manifest.xml
UATHelper: Packaging (Windows (64-bit)):   Building OKI_AnimationTest...
UATHelper: Packaging (Windows (64-bit)):   Using Visual Studio 2019 14.29.30137 toolchain (C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.29.30133) and Windows 10.0.19041.0 SDK (C:\Program Files (x86)\Windows Kits\10).
UATHelper: Packaging (Windows (64-bit)):   [Upgrade]
UATHelper: Packaging (Windows (64-bit)):   [Upgrade] Using backward-compatible build settings. The latest version of UE4 sets the following values by default, which may require code changes:
UATHelper: Packaging (Windows (64-bit)):   [Upgrade]     bLegacyPublicIncludePaths = false                 => Omits subfolders from public include paths to reduce compiler command line length. (Previously: true).
UATHelper: Packaging (Windows (64-bit)):   [Upgrade]     ShadowVariableWarningLevel = WarningLevel.Error   => Treats shadowed variable warnings as errors. (Previously: WarningLevel.Warning).
UATHelper: Packaging (Windows (64-bit)):   [Upgrade]     PCHUsage = PCHUsageMode.UseExplicitOrSharedPCHs   => Set in build.cs files to enables IWYU-style PCH model. See https://docs.unrealengine.com/en-US/Programming/BuildTools/UnrealBuildTool/IWYU/index.html. (Previously: PCHUsageMode.UseSharedPCHs).
UATHelper: Packaging (Windows (64-bit)):   [Upgrade] Suppress this message by setting 'DefaultBuildSettings = BuildSettingsVersion.V2;' in OKI_AnimationTest.Target.cs, and explicitly overriding settings that differ from the new defaults.
UATHelper: Packaging (Windows (64-bit)):   [Upgrade]
UATHelper: Packaging (Windows (64-bit)):   Building 42 actions with 8 processes...
UATHelper: Packaging (Windows (64-bit)):     [1/42] optical_flow_field_data.pb.cc
UATHelper: Packaging (Windows (64-bit)):     [2/42] anchor.pb.cc
UATHelper: Packaging (Windows (64-bit)):     [3/42] classification.pb.cc
UATHelper: Packaging (Windows (64-bit)):     [4/42] face_geometry.pb.cc
UATHelper: Packaging (Windows (64-bit)):     [5/42] SharedPCH.Core.cpp
UATHelper: Packaging (Windows (64-bit)):     [6/42] MediaPipe.init.gen.cpp
UATHelper: Packaging (Windows (64-bit)):     [7/42] geometry_pipeline_metadata.pb.cc
UATHelper: Packaging (Windows (64-bit)):     [8/42] MediaPipeModule.cpp
UATHelper: Packaging (Windows (64-bit)):     [9/42] MediaPipeObserverComponent.gen.cpp
UATHelper: Packaging (Windows (64-bit)):     [10/42] MediaPipePipelineComponent.gen.cpp
UATHelper: Packaging (Windows (64-bit)):     [11/42] MediaPipeLandmarkObserverComponent.gen.cpp
UATHelper: Packaging (Windows (64-bit)):     [12/42] MediaPipePresenceObserverComponent.gen.cpp
UATHelper: Packaging (Windows (64-bit)):     [13/42] matrix_data.pb.cc
UATHelper: Packaging (Windows (64-bit)):     [14/42] DynamicTexture.cpp
UATHelper: Packaging (Windows (64-bit)):     [15/42] environment.pb.cc
UATHelper: Packaging (Windows (64-bit)):     [16/42] image_file_properties.pb.cc
UATHelper: Packaging (Windows (64-bit)):     [17/42] MediaPipeClassificationObserverComponent.cpp
UATHelper: Packaging (Windows (64-bit)):     [18/42] locus.pb.cc
UATHelper: Packaging (Windows (64-bit)):     [19/42] mesh_3d.pb.cc
UATHelper: Packaging (Windows (64-bit)):     [20/42] rasterization.pb.cc
UATHelper: Packaging (Windows (64-bit)):     [21/42] MediaPipeClassificationObserverComponent.gen.cpp
UATHelper: Packaging (Windows (64-bit)):     [22/42] rect.pb.cc
UATHelper: Packaging (Windows (64-bit)):     [23/42] MediaPipeObserverComponent.cpp
UATHelper: Packaging (Windows (64-bit)):     [24/42] time_series_header.pb.cc
UATHelper: Packaging (Windows (64-bit)):     [25/42] image_format.pb.cc
UATHelper: Packaging (Windows (64-bit)):     [26/42] landmark.pb.cc
UATHelper: Packaging (Windows (64-bit)):     [27/42] MediaPipePresenceObserverComponent.cpp
UATHelper: Packaging (Windows (64-bit)):     [28/42] Default.rc2
UATHelper: Packaging (Windows (64-bit)):     [29/42] OKI_AnimationTest.cpp
UATHelper: Packaging (Windows (64-bit)):     [30/42] MediaPipeFaceMeshObserverComponent.gen.cpp
UATHelper: Packaging (Windows (64-bit)):     [31/42] detection.pb.cc
UATHelper: Packaging (Windows (64-bit)):     [32/42] location_data.pb.cc
UATHelper: Packaging (Windows (64-bit)):     [33/42] MediaPipePipelineComponent.cpp
UATHelper: Packaging (Windows (64-bit)):     [34/42] SharedPCH.Engine.cpp
UATHelper: Packaging (Windows (64-bit)):     [35/42] MediaPipeLandmarkObserverComponent.cpp
UATHelper: Packaging (Windows (64-bit)):     [36/42] Module.JsonCsvToDataTableToStruct.gen.cpp
UATHelper: Packaging (Windows (64-bit)):     [37/42] Module.OVRLipSync.gen.cpp
UATHelper: Packaging (Windows (64-bit)):     [38/42] Module.OVRLipSync.cpp
UATHelper: Packaging (Windows (64-bit)):     [39/42] MediaPipeFaceMeshObserverComponent.cpp
UATHelper: Packaging (Windows (64-bit)):     [40/42] Module.JsonCsvToDataTableToStruct.cpp
UATHelper: Packaging (Windows (64-bit)):     [41/42] OKI_AnimationTest-Win64-Shipping.exe
UATHelper: Packaging (Windows (64-bit)):     libprotobuf.lib(libprotobuf.dll) : error LNK2005: "void __cdecl google::protobuf::internal::VerifyVersion(int,int,char const *)" (?VerifyVersion@internal@protobuf@google@@YAXHHPEBD@Z) already defined in webrtc.lib(common.obj)
UATHelper: Packaging (Windows (64-bit)):     libprotobuf.lib(libprotobuf.dll) : error LNK2005: "public: void * __cdecl google::protobuf::internal::ArenaImpl::AllocateAlignedAndAddCleanup(unsigned __int64,void (__cdecl*)(void *))" (?AllocateAlignedAndAddCleanup@ArenaImpl@internal@protobuf@google@@QEAAPEAX_KP6AXPEAX@Z@Z) already defined in webrtc.lib(arena.obj
)
UATHelper: Packaging (Windows (64-bit)):     libprotobuf.lib(libprotobuf.dll) : error LNK2005: "private: void __cdecl google::protobuf::Arena::OnArenaAllocation(class type_info const *,unsigned __int64)const " (?OnArenaAllocation@Arena@protobuf@google@@AEBAXPEBVtype_info@@_K@Z) already defined in webrtc.lib(arena.obj)
UATHelper: Packaging (Windows (64-bit)):     libprotobuf.lib(libprotobuf.dll) : error LNK2005: "void __cdecl google::protobuf::internal::OnShutdownRun(void (__cdecl*)(void const *),void const *)" (?OnShutdownRun@internal@protobuf@google@@YAXP6AXPEBX@Z0@Z) already defined in webrtc.lib(common.obj)
UATHelper: Packaging (Windows (64-bit)):     libprotobuf.lib(libprotobuf.dll) : error LNK2005: "protected: void __cdecl google::protobuf::internal::RepeatedPtrFieldBase::Reserve(int)" (?Reserve@RepeatedPtrFieldBase@internal@protobuf@google@@IEAAXH@Z) already defined in webrtc.lib(repeated_field.obj)
UATHelper: Packaging (Windows (64-bit)):     libprotobuf.lib(libprotobuf.dll) : error LNK2005: "private: void __cdecl google::protobuf::internal::RepeatedPtrFieldBase::MergeFromInternal(class google::protobuf::internal::RepeatedPtrFieldBase const &,void (__cdecl google::protobuf::internal::RepeatedPtrFieldBase::*)(void * *,void * *,int,int))" (?MergeFromInte
rnal@RepeatedPtrFieldBase@internal@protobuf@google@@AEAAXAEBV1234@P81234@EAAXPEAPEAX1HH@Z@Z) already defined in webrtc.lib(repeated_field.obj)
UATHelper: Packaging (Windows (64-bit)):     libprotobuf.lib(libprotobuf.dll) : error LNK2005: "void __cdecl google::protobuf::internal::InitSCCImpl(struct google::protobuf::internal::SCCInfoBase *)" (?InitSCCImpl@internal@protobuf@google@@YAXPEAUSCCInfoBase@123@@Z) already defined in webrtc.lib(generated_message_util.obj)
UATHelper: Packaging (Windows (64-bit)):     libprotobuf.lib(libprotobuf.dll) : error LNK2005: "void __cdecl google::protobuf::internal::DestroyMessage(void const *)" (?DestroyMessage@internal@protobuf@google@@YAXPEBX@Z) already defined in webrtc.lib(generated_message_util.obj)
UATHelper: Packaging (Windows (64-bit)):     libprotobuf.lib(libprotobuf.dll) : error LNK2005: "public: class std::basic_string<char,struct std::char_traits<char>,class std::allocator<char> > * __cdecl google::protobuf::internal::ArenaStringPtr::MutableNoArena(class std::basic_string<char,struct std::char_traits<char>,class std::allocator<char> > const *)" (
?MutableNoArena@ArenaStringPtr@internal@protobuf@google@@QEAAPEAV?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@std@@PEBV56@@Z) already defined in webrtc.lib(arenastring.obj)
UATHelper: Packaging (Windows (64-bit)):     libprotobuf.lib(libprotobuf.dll) : error LNK2005: "public: void __cdecl google::protobuf::internal::ArenaStringPtr::DestroyNoArena(class std::basic_string<char,struct std::char_traits<char>,class std::allocator<char> > const *)" (?DestroyNoArena@ArenaStringPtr@internal@protobuf@google@@QEAAXPEBV?$basic_string@DU?$
char_traits@D@std@@V?$allocator@D@2@@std@@@Z) already defined in webrtc.lib(arenastring.obj)
UATHelper: Packaging (Windows (64-bit)):     libprotobuf.lib(libprotobuf.dll) : error LNK2005: "public: static unsigned __int64 __cdecl google::protobuf::internal::WireFormatLite::Int32Size(class google::protobuf::RepeatedField<int> const &)" (?Int32Size@WireFormatLite@internal@protobuf@google@@SA_KAEBV?$RepeatedField@H@34@@Z) already defined in webrtc.lib(w
ire_format_lite.obj)
UATHelper: Packaging (Windows (64-bit)):     libprotobuf.lib(libprotobuf.dll) : error LNK2005: "public: __cdecl google::protobuf::internal::ExtensionSet::ExtensionSet(void)" (??0ExtensionSet@internal@protobuf@google@@QEAA@XZ) already defined in webrtc.lib(extension_set.obj)
UATHelper: Packaging (Windows (64-bit)):     libprotobuf.lib(libprotobuf.dll) : error LNK2005: "public: __cdecl google::protobuf::internal::ExtensionSet::~ExtensionSet(void)" (??1ExtensionSet@internal@protobuf@google@@QEAA@XZ) already defined in webrtc.lib(extension_set.obj)
UATHelper: Packaging (Windows (64-bit)):     libprotobuf.lib(libprotobuf.dll) : error LNK2005: "public: void __cdecl google::protobuf::internal::ExtensionSet::Clear(void)" (?Clear@ExtensionSet@internal@protobuf@google@@QEAAXXZ) already defined in webrtc.lib(extension_set.obj)
UATHelper: Packaging (Windows (64-bit)):     libprotobuf.lib(libprotobuf.dll) : error LNK2005: "public: void __cdecl google::protobuf::internal::ExtensionSet::MergeFrom(class google::protobuf::internal::ExtensionSet const &)" (?MergeErrorFrom@ExtensionSet@internal@protobuf@google@@QEAAXAEBV1234@@Z) already defined in webrtc.lib(extension_set.obj)
UATHelper: Packaging (Windows (64-bit)):     libprotobuf.lib(libprotobuf.dll) : error LNK2005: "public: void __cdecl google::protobuf::internal::ExtensionSet::Swap(class google::protobuf::internal::ExtensionSet *)" (?Swap@ExtensionSet@internal@protobuf@google@@QEAAXPEAV1234@@Z) already defined in webrtc.lib(extension_set.obj)
UATHelper: Packaging (Windows (64-bit)):     libprotobuf.lib(libprotobuf.dll) : error LNK2005: "public: bool __cdecl google::protobuf::internal::ExtensionSet::IsInitialized(void)const " (?IsInitialized@ExtensionSet@internal@protobuf@google@@QEBA_NXZ) already defined in webrtc.lib(extension_set.obj)
UATHelper: Packaging (Windows (64-bit)):     libprotobuf.lib(libprotobuf.dll) : error LNK2005: "public: unsigned __int64 __cdecl google::protobuf::internal::ExtensionSet::ByteSize(void)const " (?ByteSize@ExtensionSet@internal@protobuf@google@@QEBA_KXZ) already defined in webrtc.lib(extension_set.obj)
UATHelper: Packaging (Windows (64-bit)):     libprotobuf.lib(libprotobuf.dll) : error LNK2005: "public: static unsigned __int64 __cdecl google::protobuf::internal::WireFormatLite::UInt32Size(class google::protobuf::RepeatedField<unsigned int> const &)" (?UInt32Size@WireFormatLite@internal@protobuf@google@@SA_KAEBV?$RepeatedField@I@34@@Z) already defined in w
ebrtc.lib(wire_format_lite.obj)
UATHelper: Packaging (Windows (64-bit)):        Creating library C:\Users\Dex-lab\Desktop\IslandHealth_POC\Binaries\Win64\OKI_AnimationTest-Win64-Shipping.lib and object C:\Users\Dex-lab\Desktop\IslandHealth_POC\Binaries\Win64\OKI_AnimationTest-Win64-Shipping.exp
UATHelper: Packaging (Windows (64-bit)):     C:\Users\Dex-lab\Desktop\IslandHealth_POC\Binaries\Win64\OKI_AnimationTest-Win64-Shipping.exe : fatal error LNK1169: one or more multiply defined symbols found
UATHelper: Packaging (Windows (64-bit)): Took 57.1001483s to run UnrealBuildTool.exe, ExitCode=6
UATHelper: Packaging (Windows (64-bit)): UnrealBuildTool failed. See log for more details. (C:\Users\Dex-lab\AppData\Roaming\Unreal Engine\AutomationTool\Logs\C+Program+Files+Epic+Games+UE_4.27\UBT-OKI_AnimationTest-Win64-Shipping_2.txt)
UATHelper: Packaging (Windows (64-bit)): AutomationTool exiting with ExitCode=6 (6)
UATHelper: Packaging (Windows (64-bit)): BUILD FAILED
PackagingResults: Error: Unknown Error

that's because mediapipe and some other plugin use different libs of protobuf:
libprotobuf.lib(libprotobuf.dll) : error LNK2005: "void __cdecl google::protobuf::internal::VerifyVersion(int,int,char const *)" (?VerifyVersion@internal@protobuf@google@@YAXHHPEBD@Z) already defined in webrtc.lib(common.obj)
possible solutions:
a. in case you don't using webrtc try to disable it
b. configure both plugins to use same lib
c. build mediapipe plugin in different project and use precompiled binaries in your project

Thank you for the suggestions. I tried the first one and I managed to get it past the first error. Then there was another error which I took some time to solve but thank you!!


Support generic calls, e.g., foo(1.2)

This improves our parsing statistics from 86% to 90.3%
on our 500K Kotlin corpus
test plan:
test file included

cc @fwcd we reach 90.3% of correct parsing with this!

:+1: Thanks!


DBZ-2870 Using --release instead of --source/--target

https://issues.redhat.com/browse/DBZ-2870

Shouldn't that trigger a build workflow for all modules?

The main pom.xml is not among paths of module worflows.

Like God above everything? 😅
Do we act on it?

I'll create a Jira issue and push a commit to this PR so it is tested as well

@gunnarmorling I believe the --release works only for JDK 9+ compilers so probably jdk11 profile should be introduced back and the property set in it.

@gunnarmorling I believe the --release works only for JDK 9+ compilers so probably jdk11 profile should be introduced back and the property set in it.

How about we simply mandate JDK 11+ to be used for the build? Seems that's happening anyways as per the error report by the user on chat triggering this?

@gunnarmorling No, we are not. I've received the same error report independently and after further investigation I've found that it was Confluent's build

Ok, thanks for checking! So in that light, let's keep that change for 1.5.

Closing for now, as we'll defer that upgrade. We still build with JDK 8 for the time being.

Closing for now, as we'll defer that upgrade. We still build with JDK 8 for the time being.


Implemented Navigation Bar

Ok I will comment after making the changes

for some changes you can use directly the ui

@eduardconstantin Done 👍


code completion should show indicate when there are no results

When the user explicitly asks for code completion results (ctrl-space), we should indicate when there are none available. Either something like a beep, or flashing some ui element.

I'll deal with this. I think for a first answer just putting in a box with '---- no completions available -----' should do the job.
I'm a bit puzzled as to what conditions would give no completions though.....

Trying to complete in main will show this behavior:
void main() {
  for (int i = 0; i < 5; i++) {
    print('hello ${i + 1}');
  }
}

Fixed by https://github.com/dart-lang/dart-pad/pull/196


Allow comment of a "else[ if]" block to be dedented

Description
When writing if...else block with 1TBS style, it's hard to comment the else (or else if) comment as the comment for the else block is indented at the same level than the if block content.
Input
The code looked like this before beautification:
// In this case
if (foo) {
    // Do something

// Else do something else
} else {
    // Do something else
}

Expected Output
The code should have looked like this after beautification:
if (foo) {
    // Do something

// Else do something else
} else {
    // Do something else
}

Actual Output
The  code actually looked like this after beautification:
if (foo) {
    // Do something

    // Else do something else
} else {
    // Do something else
}

Environment
OS: Linux Mint 17
Settings
{
    "indent_size": 4,
    "indent_char": " ",
    "eol": "\n",
    "indent_level": 0,
    "indent_with_tabs": false,
    "preserve_newlines": true,
    "max_preserve_newlines": 5,
    "jslint_happy": false,
    "space_after_anon_function": false,
    "brace_style": "collapse-preserve-inline",
    "keep_array_indentation": true,
    "keep_function_indentation": true,
    "space_before_conditional": true,
    "break_chained_methods": false,
    "eval_code": false,
    "unescape_strings": false,
    "wrap_line_length": 130,
    "wrap_attributes": "auto",
    "wrap_attributes_indent_size": 4,
    "end_with_newline": true,
    "operator_position": "before-newline"
}

Because of #200, the // Else do something else comment is printed before the beautifier knows about the close brace and unindent.  We could solve this one case by look ahead, but it would leave so many others - big bug farm.
There might be a more general hack/rule, where we special case any group of comments that occur on the lines directly before an element.
/* block comment */
if (foo) {
    do_something();
    // this comment set has a blank line after it
    // so it stays here
    /* block comments follow the same behavior */

// Else do something else
// this set, has no blank lines after it so it sticks to the element below
} else if (bar) {
    // this set, already behaves correctly
    do_something_else();
// this set, has no blank lines after it so it sticks to the element below
/* block comments
 * are not special
 */
// even though there isn't a blank line above.  Seems reasonable.
} else {
    /* still correct */
    do_something_else();  // same line comment stays here

// This is odd, but is what the rule would make happen.
// Of course, this is also an odd place to put comments
}
next_step();

/* let's talk about unbraced */
if (foo)
    do_something();
// DIFFERENT
// without a brace, this unindents because the if block ended at the semicolon.
// should the example above also follow this rule?  Maybe.

// Else do something else
// this set, has no blank lines after it so it sticks to the element below
else if (bar)
    // this set, already behaves correctly
    do_something_else();
// this set, has no blank lines after it so it sticks to the element below
/* block comments
 * are not special
 */
// even though there isn't a blank line above.  Seems reasonable.
else
    /* still correct */
    do_something_else();  // same line comment stays here

// This no longer looks odd. And is a reasonable place for a comment.
next_step();

Gah, the problem is achieving this without better parsing.  So many places that can go wrong.
Wishful thinking.

I disagree with the original request, at least if it is not configurable. I think indented end-of-block comments make a lot of sense in many situations.
if (modulus === 2) {
    // i might be odd here
    i += (i & 1);
    // now i is guaranteed to be even
} else {
    // rounding up using integer arithmetic only
    if (i % modulus)
        i += modulus - (i % modulus);
    // now i is divisible by modulus
}

Here I'm using an end-of-block comment to state a post-condition. The comment doesn't directly describe any particular statement or block, but instead describes the state of some object at a specific point during the execution of the code. As such, it should be indented like a statement, not like the closing parenthesis.
If you can find a way to distinguish between these two use cases, that might be fine. I can imagine a global switch somewhere. Making the distinction based on the presence or absence of empty lines somewhere is bound to confuse people, though, so I'd rather avoid that.

@gagern - Yeah, this is a particularly difficult problem to land universally.
I agree with counter-examples. Those should remain where they are.
I think about it as answering the question "What element is this block of comments attached to?"
So, we need to define "block of comments" and then what factors we can use to answer what element each block attaches to.   Those factors include, type of previous and following elements, whether a statement/expression ended after the previous or before the following element, and what newlines were present between each block and the previous/following elements.
No matter what answer we choose it will always before wrong in some cases.  Indentation of other element should not change based on presence or absence of comments except where newlines at the end of comments change semantics.  Beyond that the best we can do is make the behavior predictable and consistent.
So, what do you think of this:

Your example looks good from the point of reading the code, but if code written that way got actually indented to this style, I as a writer of code vould still be very surprised. Personally I'd not expect anything in {…} to get dedented, nor would I expect comments trailing an unbraced statement to maintain the indentation. I guess you'd make the distinctions between these cases based on empty lines, but I consider that again to be a source of a lot of confusion.
If you are looking at empty lines, you are essentially looking at the whitespace before beautification. Can't you look at original indentation, too? Note that if the comment is indented like the closing brace, it should stay that way even if both of them now get indented differently. And if the comment is indented like some of the levels of unbraced control constructs, then it can stay with that level. That way, users could control correct indentation simply by indenting correctly, which is a very intuitive approach I think. And js-beautify would know that in some situations several indentations are possible, so it checks all of them in a relative context but picks a preferred one if none of the alternatives matches the input.

Using indents is an interesting idea. Thanks!


Audio issues using Ubuntu Trusty 14.04 with KDE

Hi,
I was excited at the prospect of [this article about a DAW on a chromebook] (https://github.com/dnschneid/crouton/wiki/Full-Blown-Audio-Video-Workstation-using-KDE-chroot-and-KXStudio) but after following the example point by point, I am still having issues where KDE will start complaining about the drivers on my Chromebook - [ASUS C300] (http://www.asus.com/us/Notebooks/ASUS_Chromebook_C300/specifications/). I saw a few forum posts that suggested changing the order of the audio drivers in the KDE settings, prefering the 'defaults' so I did that but still no sound...
Any suggestions? I wonder if this is the best place to ask or if I should jump on the ubuntu trusty forums...
Thanks!

V

Possible duplicate of #2015

Please re-open if this is still an issue.


⚠ BCR Bot has degraded performance

In cb48264, BCR Bot (https://bcr.alternativasinteligentes.com/BCRApi/web/chatbot) experienced degraded performance:

HTTP code: 200
Response time: 2923 ms

Resolved: BCR Bot performance has improved in f2b10f7.


Add subchapters to Week 4

Thanks @johnshea  -- just a heads up that they now have split the Ch 4 into a/b/c parts. I haven't checked out all the changes yet so if you have, and you think the table-of-contents itself needs to be updated, let me know.
And thanks for the awesome update.


add golang video-tutorials

הוספתי קטגוריה חדשה של מדריכים בשפת GO

Thank you so much for the contribution, the build failed because there was not added an icon for the new language.
I'll sort it out later, or you can do it if you'd like too... the icons are from this site:  and they should be added to this file (without the <svg> element - ontly the <path>'s). thanks again!

Is there a way to use the logo from another site.
Because the site you wrote has the old logo.
And it's better to put the new logo that looks better.


Add documentation for Citadel

Hi @AaronDewes very sorry I missed this pull request until now, it seems I have disabled notifications for all my repo's.


chore: prepare for 3.8.0-beta release

Pull Request Test Coverage Report for Build 1626

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 89.748%

Totals

Change from base Build 1625:
0.0%

Covered Lines:
4412

Relevant Lines:
4916

💛  - Coveralls


Add pull_request_target as valid event

Would it be possible to support this event? I would like to use this in my (protected) environment where there are no drawbacks to pull_request_target and it secret handling.

Sure it possible. I will add support for it very soon.

Awesome, thank you!

I add support for this, you can use:
- name: Pytest Coverage Comment
  uses: MishaKav/pytest-coverage-comment@main

or use a specific version
- name: Pytest Coverage Comment
  uses<EMAIL_ADDRESS>


Added linux/ppc64le and linux/s390x

Some more additional architectures for linux.

Is anything blocking merging this?  Would test reports be helpful?  Would be great to have ppc64le support merged.

Could we get this merged?  We're currently looking for a new GIT solution and are considering Gitea, but would need to see upstream is willing to support our host server architecture (in this case ppc64le).
Thanks!

As a point of larger concern, the last PR merged was back in February, with no other activity in the source history from this year.
Is xgo maintained any more?


Update message_create_options.py

There was missing attribute sendFrom in init, so sendFrom could not be changed. This changes fexed it.

@barkipeto We've put this in, in a slightly different order to avoid any breaking changes: https://github.com/mailosaur/mailosaur-python/commit/bc01d56baa8fe39a7db1e04ce4346b6e6d36963a
This is now released under version 7.15.0


It's not possible to create a CDN Endpoint with a DeliveryRule that matches "Any"

It's not possible to create a CDN Endpoint with a DeliveryRule that matches "Any". Pulumi validation requires a Match List but azure does not allow a match list.
 var endpoint = new Endpoint(n.CdnEndpointName(name), new EndpointArgs
            {
                ResourceGroupName = resourceGroupName,
                ProfileName = cdnProfile.Name,
                OriginHostHeader = storageAccount.PrimaryWebHost,
                Origins = new List<EndpointOriginArgs>
                {
                    new EndpointOriginArgs
                    {
                        Name = "blobstorage",
                        HostName = storageAccount.PrimaryWebHost
                    }
                },
                DeliveryRules = new EndpointDeliveryRuleArgs
                {
                    Name = "deeplinkredirect",
                    Order = 1,
                    UrlFileExtensionConditions = new EndpointDeliveryRuleUrlFileExtensionConditionArgs
                    {
                        MatchValues = new InputList<string>(),
                        Operator = "Any",
                        NegateCondition = true,
                    },
                    UrlRewriteAction = new EndpointDeliveryRuleUrlRewriteActionArgs
                    {
                        SourcePattern = "/",
                        Destination = "/"
                    }
                }
            });

Updating (test):
Type                            Name                  Status                  Info                                      pulumi:pulumi:Stack             Test.Proj.Infra-test  failed              1 error                               +   ├─ azure:cdn:Endpoint           app-spa-cdn-ep        creating failed     1 error                               +   └─ azure:appservice:AppService  app-app-api           created
Diagnostics:
azure:cdn:Endpoint (app-spa-cdn-ep):
error: Error waiting for CDN Endpoint "app-spa-cdn-ep78bd5ab" (Profile "app-spa-cdn290c7b0" / Resource Group "rg-appef85df78") to finish creating: Code="BadRequest" Message="{"ErrorMessage":"Invalid RulesEngineConfig: (Match value(s) must not be provided for Operator 'Any')."}"
...
Previewing update (test):
Type                   Name                  Plan       Info                                                            pulumi:pulumi:Stack    Test.Proj.Infra-test             1 error                                                     +   └─ azure:storage:Blob  index.html            create
Diagnostics:
pulumi:pulumi:Stack (Test.Proj.Infra-test):
error: Running program 'C:\repos\prft\at-rest-test\Test.Proj.Infra\bin\Debug\netcoreapp3.1\Test.Proj.Infra.dll' failed with an unhandled exception:
System.ArgumentNullException: [Input] Pulumi.Azure.Cdn.Inputs.EndpointDeliveryRuleUrlFileExtensionConditionArgs._matchValues is required but was not given a value (Parameter '_matchValues')
...
Previewing update (test):
Type                   Name                  Plan     Info                                                              pulumi:pulumi:Stack    Test.Proj.Infra-test                                                                             └─ azure:cdn:Endpoint  app-spa-cdn-ep                 1 error
Diagnostics:
azure:cdn:Endpoint (app-spa-cdn-ep):
error: azure:cdn/endpoint:Endpoint resource 'app-spa-cdn-ep' has a problem: delivery_rule.0.url_file_extension_condition.0.match_values: attribute supports 1 item as a minimum, config has 0 declared

I can't be sure but according to this question on stackoverflow "not any" is not supported for Extension matches. So this may be a non-issue. https://stackoverflow.com/questions/58914446/azure-cdn-microsoft-standard-rules-engine-rewrite-url-for-single-page-applicatio

Thank you for reporting!
This is an issue from the upstream provider, let's track it here: https://github.com/terraform-providers/terraform-provider-azurerm/issues/8770
We'll get the fix as soon as it's released upstream, so I'll close this issue here


Doesn't play well with UIMenuController under iOS 5.1(b)

Heads-up. Under iOS 5.1b, once the HUD has appeared and then disappeared, invocations elsewhere in the UI of UIMenuController will fail (ie the menu will simply not appear), both in custom views but also in system-standard text views.
It's something to do with the interactions in window ordering, though I'm not sure whether it's a beta-only bug that Apple's introduced, or whether this is going to be a production problem.  I'm not well enough versed in the UIWindow stuff to understand what's happening exactly, but possibly the UIMenuController lives on its own window and that's getting shuffled behind when the HUD dismisses?
Opening another modal view controller (e.g. the mail compose controller, or any custom form sheet, etc) "clears" the condition and the menu will function again.
Workaround: If I turn the HUD into a UIView subclass (rather than UIWindow) and simply add it as a subview to my current known window, I find it no longer exhibits this interaction.

Thanks @bzotto, I will look into this shortly.

I'm currently away from home and don't have access to the 5.1 SDK, but you can start by checking the value of previousKeyWindow at the end of the show method and then in the dismiss method. My guess would be that your UIMenuController instance isn't assigned to it (maybe because of its windowLevel property, haven't looked at the 5.1 doc yet), so it's not made the key window again upon the HUD's dismiss.

Hey Sam,
Thanks for the comment. It looks like this issue doesn't appear with your
stock SVProgressHUD. I've been playing with a modified fork from BarrettJ,
and the HUD's UIWindow seems to linger on in the window list after it
dismisses there, which looks suspicious. I'll keep poking at it and let him
(and you) know if I root-cause it.
Thanks, sorry for the confusion.
Ben
On Sun, Dec 11, 2011 at 3:49 AM, Sam Vermette <
<EMAIL_ADDRESS>
wrote:

I'm currently away from home and don't have access to the 5.1 SDK, but you
can start by checking the value of previousKeyWindow at the end of the
show method and then in the dismiss method. My guess would be that your
UIMenuController instance isn't assigned to it (maybe because of its
windowLevel property, haven't looked at the 5.1 doc yet), so it's not made
the key window again upon the HUD's dismiss.

Reply to this email directly or view it on GitHub:
https://github.com/samvermette/SVProgressHUD/issues/35#issuecomment-3096786

--
Cocoa Box -- http://www.cocoabox.com
Makers of fine applications for iPad and iPhone
Follow us on Twitter at http://twitter.com/cocoabox

OK, nevermind. Not only was this not an issue with core SVProgressHUD, it was not an issue with Barrett's fork, either. I'd introduced a subtle behavior change in the window search when I retrofitted it for iOS 3.x compatibility. This one was all me. Sorry guys, and thanks.

Hah, ok. Thanks for the follow-up!


fix: logger

Another approach for #683 with a refactor and skipping tests that always presume logger would be winston-like

Codecov Report

Merging #684 into master will decrease coverage by 1.19%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master     #684     +/-   ##
=========================================
- Coverage   91.95%   90.76%   -1.2%
=========================================
  Files           4        4
  Lines         199      195      -4
  Branches       31       30      -1
=========================================
- Hits          183      177      -6
  Misses         16       16
- Partials        0        2      +2

Impacted Files
Coverage Δ

src/phantom.js
87.75% <100%> (-1.65%)
:arrow_down:

Continue to review full report at Codecov.

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

Aside from the code coverage going slightly down, I think I am ready to merge.

Merged. If all goes well, I will release tonight.

Thanks @laggingreflex


ISPN-15277 Default ConcurrentMap methods should ensure they close Ite…

…rators
https://issues.redhat.com/browse/ISPN-15277
Note all methods changed will close the iterator if fully iterated upon, this only ensures it is closed if there was an exception in processing somewhere.

I assume we'll want a backport ?

I assume we'll want a backport ?

I was debating, it only happens on an exception while iterating, so I would say it isn't a huge chance. But I see no harm in doing so.

Restarting CI as the last run failed due to a hung SIFS test:
[2023-10-27T16:01:58.888Z] [TestSuiteProgress] Test failed: org.infinispan.persistence.sifs.SoftIndexFileStoreRestartTest.clearContent
[2023-10-27T16:01:58.888Z] Dumping threads of process 4121961 to /home/infinispan/workspace/Infinispan_PR-11443/core/threaddump-SoftIndexFileStoreRestartTest-20231027-1201-4121961.log
[2023-10-27T16:01:58.888Z] Interrupted thread testng-SoftIndexFileStoreRestartTest (50).
[2023-10-27T16:02:00.355Z] Killed processes 4121961

Thanks @wburns


valueChange on keyDown in datetime time input

I have a requirement to trigger a valueChange on keyDown when the user manually types in a time in a datetime flatpickr instance. Currently the typed time input only updates when it is blurred.
Is there any practical way to trigger an event like this which in return triggers a flatpickr.setDate() or something the likes?

added a custom keyup event to the minute and hour input and works good enough


Amazon Linux Kernel 5.4.91-41 Missing from Build Process

**Amazon Linux Kernel 5.4.91-41 is missing from the driverbuilder/builder/amazonlinux.go **

How to reproduce it

Expected behaviour

Screenshots

Environment

Falco version:

System info:

Cloud provider or hardware configuration:
OS:

Kernel:

Installation method:

Additional context

Sysdig encountered a problem when attempting to build an AmazonLinux2 kernel probe.  Falco will encounter the same problem.  This is the guidance provided by Amazon:
“The 5.4 kernel is in an “extras” repository rather than “core”.
The corresponding URL would look like this:
http://amazonlinux.us-east-1.amazonaws.com/2/extras/kernel-5.4/latest/x86_64/mirror.list
Falco will find this problem with this go code file: https://github.com/falcosecurity/driverkit/blob/92eabdd6adaef6135ce96810ab83055b3478e21d/pkg/driverbuilder/builder/amazonlinux.go
The section of code that needs to be changed is:
var baseByTarget = map[Type]string{
TargetTypeAmazonLinux:  "http://repo.us-east-1.amazonaws.com/%s",
TargetTypeAmazonLinux2: "http://amazonlinux.us-east-1.amazonaws.com/2/core/%s/%s",
}
func fetchAmazonLinuxPackagesURLs(kv kernelrelease.KernelRelease, arch string, targetType Type) ([]string, error) {
urls := []string{}
visited := map[string]bool{}
amazonlinux2baseURL := "http://amazonlinux.us-east-1.amazonaws.com"
for _, v := range reposByTarget[targetType] {
	var baseURL string
	switch targetType {
	case TargetTypeAmazonLinux:
		baseURL = fmt.Sprintf("http://repo.us-east-1.amazonaws.com/%s", v)
	case TargetTypeAmazonLinux2:
		baseURL = fmt.Sprintf("%s/2/core/%s/%s", amazonlinux2baseURL, v, arch)  <<<<< This also needs to handle extras/kernel-5.4/latest
	default:
		return nil, fmt.Errorf("unsupported target")
	}

Sysdig uses a python script for the repository URL hunting.  The Sysdig change for AmazonLinux2 was to use these substitutions:
'core/2.0', 'core/latest', 'extras/kernel-5.4/latest'


Keyvault App Setting integration does not use the outbound connections from the web app

This feature does not guarantee the use of the outbound IPs of the web app for the integration. I'd recommend documenting this to avoid customers creating firewall rules that may break this functionality.

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

ID: 7feaa3d0-7233-a3cb-cb9b-a9a55b1f0d7a
Version Independent ID: f41f9d51-52f8-1f45-1043-1344a3b74458
Content: Key Vault references - Azure App Service
Content Source: articles/app-service/app-service-key-vault-references.md
Service: app-service
GitHub Login: @mattchenderson
Microsoft Alias: mahender

Thanks for the feedback! We are currently investigating and will update you shortly.

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

In a similar vein, I'm seeing my Key Vault deny an inbound connection from my App Service. In this case, I've attached the App Service (Function App) to a subnet via the new Vnet join functionality, and then added the subnet as an allowed network on the Key Vault. My logs are showing an error of "Client address (X.X.X.X) is not authorized and caller is not a trusted service"

@droessmj The doc has been updated to indicate this isn't supported.
Key Vault references are not presently able to resolve secrets stored in a key vault with network restrictions.

@jcbrooks92, what is the correct path forward for this? Will this likely be resolved by Key Vault's "allow trusted Azure services"? Or will this require further change on our side?

@maletor sorry i missed you reply. There's a feedback post tracking this request. There isn't an ETA on the implementation.
https://feedback.azure.com/forums/355860-azure-functions/suggestions/38817385-allow-key-vault-references-to-access-secrets-behin

Does this work when the keyvault is already created in another resource group?

Is this still unsupported? I ran into a very similar issue today, but the app has been running like this for the past three year without incident.
Ip restrictions on the key vault, and access policy through managed identity.

The doc currently has a section called: Access network-restricted vaults. For any issues based on the current doc, please open a new issue. Close this legacy issue.
#please-close


add info block to intro page for PAT generation

closes #1868
closes #2063

Coverage increased (+0.03%) to 79.117% when pulling 68defe5477b236cf9ffad4cd1b8c784540706453 on abemassry:docs-intro-review into 9c2d6f47f2886af6b0ec43e5b6d44cb6c58f8900 on linode:develop.

Tested and looks good, minor adjustments made for manager info alerts: https://github.com/abemassry/manager/pull/24
@abemassry

Personally, I'd really prefer we left these alert changes alone (both color and size).

@eatonphil are we introducing a change that has a negative impact in some way? re: color, blue is pretty common for non-warning, non-error states. We could maybe improve the text contrast

I'm just saying I would personally prefer we not make the color / size change. I put some work into how they were before because I did not feel the Bootstrap default fit our L/F.

Coverage increased (+1.6%) to 80.668% when pulling bbf1cfa6175e456d59a5d67615bf5f6928dfbb71 on abemassry:docs-intro-review into 9c2d6f47f2886af6b0ec43e5b6d44cb6c58f8900 on linode:develop.


NFFS: Handle unexpected Power Off

Can NFFS handle unexpected power failures (either brownout reset - NRF52832, or loss of VCC)?
We encountered a corrupt nffs file system (see attachment).
It is not possible to unlink existing files (error code 3 - Invalid offset).
@nvlsianpu Can you please comment on this
nffs_file_system_corrupt.bin.gz

Apparently looks like not. As far i know there is any routine for checking integrity of file in nffs while the FS starts. This should be easy to add, also recovery previous files sate should be easy to do (as only need to restore previous version of the file inode).
Unfortunately what you encountered is not only nffs issue we have.
https://github.com/apache/mynewt-nffs/issues/10

@nvlsianpu Thanks for the reply.
I think this would be a good feature to have.


VLC extensions fail to load

This is probably already known but I want to report that extensions fail to load in VLC. This is the terminal output:
$ ./VLC-3.0.0.gitfeb851a.glibc2.17-x86-64.AppImage
/tmp/.mount_fa3H50
/tmp/.mount_fa3H50/usr/bin/vlc.wrapper
/tmp/.mount_fa3H50/vlc.desktop
VLC media player 3.0.0-git Vetinari (revision feb851a)
[00000000018c50c8] core interface error: no suitable interface module
[00000000017f7178] core libvlc error: interface "globalhotkeys,none" initialization failed
[00000000017f7178] core libvlc: Running vlc with the default interface. Use 'cvlc' to use vlc without interface.
Unable to load library icui18n "Cannot load library icui18n: (libicui18n.so.48: cannot open shared object file: No such file or directory)"
[00000000018c50c8] qt interface error: Unable to load extensions module

Yes, I know, but so far I don't know the reason. Do you have any clue what would need to be changed? Thanks

No clue, just wanted to open this issue in case someone who does eventually sees it. I've been using the VLC AppImage quite a bit but the inability to use the VLsub extension has been the one thing holding me back from uninstalling the Launchpad PPA version.

What distro is being used?

Any I have tried so far. Not sure what is causing this, but if someone with some spare time could have a look, this would be fantastic.

Results of my investigation so far:

Use --verbose 2
Install libicui18n.so.48 from http://security.debian.org/debian-security/pool/updates/main/i/icu/libicu48_<IP_ADDRESS>-12+deb7u5_amd64.deb
Main issue seems to be "core generic debug: no extension modules"
usr/lib/vlc/plugins/control/ seems to be missing
core generic debug: looking for extension module matching "any" SHOULD say: 1 candidates, lua generic debug: Opening Lua Extension module but usr/share/vlc/lua seems to be missing - this is essential for playing YouTube

Can anyone fix the vlc 3 recipe?


Retroactively resend past follows by a newly bridged user

As far as I understand, if a Bluesky user follows a bridged Mastodon account before they bridge their own account in the reverse direction, the follow notification is never received even after the mutual bridge is made (unless they manually unfollow & refollow), retaining discrepancy. I wish this situation could be “retconned” automatically by resending follows to currently followed accounts once their account is bridged. Is that feasible?

This was filed a while ago, in #1102.
(In theory it's feasible, and Bridgy Fed does track the necessary data to do this already, but it doesn't seem to be a high priority.)


Cherry-pick : Upgrade kserve version to v0.12.1 in odh-model-controller

Description
Cherry-pick : Upgrade kserve version to v0.12.1 in odh-model-controller
https://github.com/opendatahub-io/odh-model-controller/pull/210
How Has This Been Tested?

Merge criteria:

[ ] The commits are squashed in a cohesive manner and have meaningful messages.
[ ] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[ ] The developer has manually tested the changes and verified that the changes work

/retest

/lgtm

/approve


Wav2vec2

Added demo-notebook 'wav2vec2-quantize'.
To launch the notebook openvino-dev ver. 22.1+ is expected, not working with earlier versions.

@DmitryMalishev that you for submitting this PR. We will start reviewing it soon, but can you add one additional step to run inference, decoding and print output for a sample audio file?
You can see how we show the ASR output in a similar notebook: https://github.com/openvinotoolkit/openvino_notebooks/blob/main/notebooks/211-speech-to-text/211-speech-to-text.ipynb

@DmitryMalishev that you for submitting this PR. We will start reviewing it soon, but can you add one additional step to run inference, decoding and print output for a sample audio file?
You can see how we show the ASR output in a similar notebook: https://github.com/openvinotoolkit/openvino_notebooks/blob/main/notebooks/211-speech-to-text/211-speech-to-text.ipynb

@ryanloney, do you mean to add the following at the end of the notebook: [1] take one audio from the dataset [2] play it back  [3] inference with quantized only model [4] print the recognized text ?

@DmitryMalishev that you for submitting this PR. We will start reviewing it soon, but can you add one additional step to run inference, decoding and print output for a sample audio file?
You can see how we show the ASR output in a similar notebook: https://github.com/openvinotoolkit/openvino_notebooks/blob/main/notebooks/211-speech-to-text/211-speech-to-text.ipynb

@ryanloney, do you mean to add the following at the end of the notebook: [1] take one audio from the dataset [2] play it back [3] inference with quantized only model [4] print the recognized text ?

@DmitryMalishev yes. If we use this notebook for a webinar, workshop or developer training program, the presenter will need to show sample input and output to the quantized model.

Overall it looks good to me. Thanks, @DmitryMalishev!

@ryanloney @KodiaqQ @AlexKoff88 Thanks for you comments, I took into account all of them and just submitted new version with changes.
Please note, Inference Example won't work until the dynamic shape issue isn't fixed.

Thanks @DmitryMalishev. Is this the dynamic shape issue?
ValueError: get_shape was called on a descriptor::Tensor with dynamic shape (at step 6)
Is there a solution? Or do we need to wait until a future release like 2022.2?

Thanks @DmitryMalishev. Is this the dynamic shape issue? ValueError: get_shape was called on a descriptor::Tensor with dynamic shape (at step 6)
Is there a solution? Or do we need to wait until a future release like 2022.2?

@ryanloney,
The error you've faced is a part of the dynamic shape issue. This part is already fixed and available through manual installing OpenVino from the github's head. By doing this you can go further with the notebook.
Another part affects making prediction at the Model Usage Example section. Fixing this part is pending and not available yet.
The release with both fixes and thus fully workable notebook will be announced once it becomes available.

@paularamo @ryanloney This notebook is ready for merge, no other changes are planned. Could you please review and merge?

@ryanloney thanks for corrections, I updated directory structure and root 'readme'
@paularamo @adrianboguszewski please launch checking workflows

Failures in recent nbval checks are caused by notebook-105 and not relevant to current notebook.

@ryanloney Cannot merge it due to random CI errors and too strict merging rules or missing admin permissions.


Calculation of lastOffset in Apple/VisHiRes is wrong

(problem reported by @Overld; see pull req #93)
The GenerateBitmap() and GenerateGrid() functions in the Apple II hi-res visualizer attempt to catch overruns ahead of time, by computing the last referenced location in the bitmap data.  The formulas used are incorrect, and yield incorrect results in some situations.  For example, in a bitmap grid with the row stride of 256 and cell stride of 1 (e.g. a font in which the first 1x8 glyph is at +000000/+000100/+000200...+000700), it incorrectly computes a last offset that is well past the end of the data.
The problem is that the current code is trying to compute the full size and then back up a little.  It's more reliable to compute the offset of the last row of the bitmap (or, in a grid, of the last bitmap), and then compute the offset of the last byte in that row.

That makes sense.  Let me redo that calculation, and I will resubmit the
change.
On Mon, May 17, 2021 at 2:08 PM Andy McFadden @.***>
wrote:

(problem reported by @Overld https://github.com/Overld; see pull req #93
https://github.com/fadden/6502bench/pull/93)
The GenerateBitmap() and GenerateGrid() functions in the Apple II hi-res
visualizer attempt to catch overruns ahead of time, by computing the last
referenced location in the bitmap data. The formulas used are incorrect,
and yield incorrect results in some situations. For example, in a bitmap
grid with the row stride of 256 and cell stride of 1 (e.g. a font in which
the first 1x8 glyph is at +000000/+000100/+000200...+000700), it
incorrectly computes a last offset that is well past the end of the data.
The problem is that the current code is trying to compute the full size
and then back up a little. It's more reliable to compute the offset of the
last row of the bitmap (or, in a grid, of the last bitmap), and then
compute the offset of the last byte in that row.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fadden/6502bench/issues/94, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AC3BLQOCBVJOXLMF575WGYTTOGATLANCNFSM45BGZODQ
.

Fix pushed, with a pair of regression tests whose layout mimics the Empire II EWS3 font, and the Lode Runner sprite sheet.


Generate and store the deployment manifest in the repo

/kind user-story
User Story
As a cluster administrator, I want to deploy the operator easily without having to clone the repo, So that I can get started and try it out quickly.
This is a follow-up to a discussion during one of the Working Group calls.
As suggested in https://github.com/janus-idp/operator/pull/242#discussion_r1504433943, generating and storing the deployment manifest (outcome of make deployment-manifest) would allow people to install the operator with a one-liner command, like so:
kubectl apply -f https://raw.githubusercontent.com/janus-idp/operator/[branch|tag]/deploy/rhdh-operator.yaml

This also makes it easier to refer to the deployment manifests in the AKS/EKS/GKE docs.
Acceptance Criteria

[ ] Just like with the bundle manifests (https://github.com/janus-idp/operator/tree/main/bundle/manifests), it should generate and store the deployment manifest in the repo, for example under ./deploy/rhdh-operator.yaml
[ ] Just like with the bundle manifests (https://github.com/janus-idp/operator/blob/main/.github/workflows/pr-bundle-diff-checks.yaml), it should be enforced in CI that the deployment manifest is always updated
[ ] (For later) We could generate a similar manifest for downstream RHDH and store those manifests under .rhdh/deploy/, such that it references the right downstream images

Links

Related to https://github.com/janus-idp/operator/issues/241

/kind user-story

This is what we discussed indeed ( https://issues.redhat.com/browse/RHIDP-1612 ) and approach is fine.
My concern is mostly about "just like bundle manifest approach"
Deployment manifest is a way to install operator w/o needs to clone project, alternative to OLM, Helm (which we do not have, just to mention).
So it has a real value for users which rather need to install "predictable/supportable" version than "arbitrary" state.
Also, as a maintainer of Operator I would prefer to deal with issue reports/requests containing concrete (supported) version and not arbitrary state downloaded from main branch any time.
I.e:
I am fine with storing deployment manifest files in github repo (this or some other) and have it downloadable using direct github url (no clone needed) as an installation option but it should be "released" manifests, i.e. the link like
https://raw.githubusercontent.com/janus-idp/operator/${VERSION}/deploy/rhdh-operator-[${VERSION}].yaml
is stable and points to released ${VERSION} of operator

Instead of supporting non-OLM installs of the operator, we can simply update the docs to clearly articulate that OLM is a prereq, with a link to where / how to install OLM. See RHIDP-2707
I suggest we close this and the associated JIRA https://issues.redhat.com/browse/RHIDP-1612 as "won't do" and simply focus on the doc change above.

Instead of supporting non-OLM installs of the operator, we can simply update the docs to clearly articulate that OLM is a prereq, with a link to where / how to install OLM. See RHIDP-2707

From a downstream product perspective, it makes sense to focus on OLM as install method. But for the long-term vision of an upstream Backstage operator, I think we should not limit ourselves to only OLM.
But I agree we can close this one, since its goal was clearly for downstream RHDH usage. We can later on create specific issues if need be.
/close


Apply cgroup constraints once pod overhead is fully supported

once pod overhead is fully supported and mandatory. sandbox_cgroup_only must be set to true by default and all the cgroups in the host honoured.  check createCgroupManager.

https://github.com/kata-containers/kata-containers/pull/972 replaces this issue


kvm: Use 'ip route show default <IP_ADDRESS>/0' to find the default gateway

If a host has many routes this can be a magnitude faster then printing
all the routes and grepping for the default.
In some situations the host might have a large amount of routes due to
dynamic routing being used like OSPF or BGP.
In addition fix a couple of loglines which were throwing messages on
DEBUG while WARN and ERROR should be used there.
Signed-off-by: Wido den Hollander<EMAIL_ADDRESS>Description

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[ ] 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)

Screenshots (if appropriate):
How Has This Been Tested?

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2483

@blueorangutan test

@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-3250)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 24446 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr3080-t3250-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_internal_lb.py
Intermittent failure detected: /marvin/tests/smoke/test_multipleips_per_nic.py
Intermittent failure detected: /marvin/tests/smoke/test_vpc_redundant.py
Intermittent failure detected: /marvin/tests/smoke/test_hostha_kvm.py
Smoke tests completed. 68 look OK, 2 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

test_nic_secondaryip_add_remove
Error
32.75
test_multipleips_per_nic.py

test_04_rvpc_network_garbage_collector_nics
Failure
495.27
test_vpc_redundant.py


v1.1.1 pre-built binaries ready?

I'm running v1.1.1 af39bd0dcf on Ubuntu for Windows.
$ ~/duckdb_111/duckdb

INSTALL h3 FROM community;

HTTP Error: Failed to download extension "h3" at URL "http://community-extensions.duckdb.org/v1.1.1/linux_amd64_gcc4/h3.duckdb_extension.gz" (HTTP 403)

Candidate extensions: "s3", "http", "https", "httpfs", "motherduck"

Community extension are being build at this moment.
We have a plan in place to improve this and build also community extensions in advance, but as of today it's a process kicked off after release is tagged, and a few hours for all extensions to be there.

Great, got it installed here. Thank you.


cannot stat ‘build-server/server/scrcpy-server.jar’

When I compiled it ， scrcpy-server.jar  no generation.

Did you execute ninja as root? If so, don't.

Did you execute ninja as root? If so, don't.
See 4c49b27.

Thank you very much.


Install kraken2, bracken and mapseq data managers

TODO: Please replace this header with a description of your pull request.
Installation sequence for tool-installers

[ ] Test using @galaxybot test this
[ ] Inspect CI output for expected changes
[ ] Deploy using @galaxybot deploy this if test install was successful
[ ] Merge this PR

@galaxybot deploy this


Installation Failure "UnifiedNlp (no GAPPS)"

Hello!
I have installed CM13 (13.0-20160605-NIGHTLY-ks01lte) from scratch w/o GApps.
Threema support adviced to use microG to overcome some issues related to polling.
Based on this I started installation with F-Droid and succeeded for some packages.
But the installation of UnifiedNlp (no GAPPS) fails with unknown error:

At this point I'd like to understand why this installation fails.
Furthermore it would be helpful if somebody could explain which packages should be installed:
UnifiedNlp.apk
NetworkLocation.apk
LegacyNetworkLocation.apk
This is not clear, but my understanding was that I should have UnifiedNlp (no GAPPS) installed on devices w/o Google services.
THX

You don't need that package anymore if you've installed the latest GmsCore, as it got integrated with that. The "UnifiedNLP (no GAPPS)" is just a "legacy app" for those still using the GmsCore predecessor "NOGAPPS".

Great! Thanks, Izzy.
I'll continue testing Android w/o Google and may come back to you

Anytime, @74cmonty ! Be welcome to see the linked site for more privacy tips as well :)


Allow GHA testing on the dev branch

This gives a contributor a fixed branch they can use for development
& testing on their fork without abusing the main branch which should
track upstream (tidymodels/main).
This is a QOL PR, it is tested here.

Codecov Report

Merging #37 (4924542) into main (dea8303) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #37   +/-   ##
=======================================
  Coverage   93.61%   93.61%
=======================================
  Files          27       27
  Lines         736      736
=======================================
  Hits          689      689
  Misses         47       47

Continue to review full report at Codecov.

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


Upload .tar.gz source distribution with wheels

As seen on https://pypi.org/project/sqloxide/#files, sqloxide does not have any source distributions uploaded to PyPI. This makes it impossible to use PyPI with sqloxide for a given os and architecture unless the necessary wheel has been built.
We can include this source distribute by creating the job source-build in ci.yml. A separate job is preferable here to prevent duplication and collisions from multiple matrix configurations.
By running python setup.py sdist we can generate sqloxide-0.1.30.tar.gz in dist/.

@adavis444 thanks for both of these fixes, I can merge them and do some testing and cut a release. We are overdue on a few versions.

@wseaton happy to help with some CI updates! Thanks for your work on sqloxide!


The title is not align center in vertical.

The title "Test" is not align center in vertical. How do i fix?

nav version: 1.2.2
RN version: 0.20.0

Here are the changes i made to styles.js to center the buttons and title vertically.
const NAV_BAR_HEIGHT = 44;
const STATUS_BAR_HEIGHT = 20;
const NAV_HEIGHT = NAV_BAR_HEIGHT + STATUS_BAR_HEIGHT;
module.exports = {
navBarContainer: {
backgroundColor: 'white',
},
statusBar: {
height: STATUS_BAR_HEIGHT,
},
navBar: {
height: NAV_BAR_HEIGHT,
flexDirection: 'row',
justifyContent: 'space-between',
},
customTitle: {
position: 'absolute',
left: 0,
right: 0,
bottom: 14,
alignItems: 'center',
},
navBarButton: {
marginTop: 10,
},
navBarButtonText: {
fontSize: 17,
letterSpacing: 0.5,
marginTop: 10,
},
navBarTitleText: {
fontSize: 17,
letterSpacing: 0.5,
color: '#333',
fontWeight: '500',
position: 'absolute',
left: 0,
right: 0,
bottom: 14,
textAlign: 'center',
},
};

@spietrek  Thank you. I changed the styles for chinese.
const NAV_BAR_HEIGHT = 44;
const STATUS_BAR_HEIGHT = 20;
const NAV_HEIGHT = NAV_BAR_HEIGHT + STATUS_BAR_HEIGHT;
module.exports = {
navBarContainer: {
backgroundColor: 'white',
},
statusBar: {
height: STATUS_BAR_HEIGHT,
},
navBar: {
height: NAV_BAR_HEIGHT,
flexDirection: 'row',
justifyContent: 'space-between',
},
customTitle: {
position: 'absolute',
left: 0,
right: 0,
bottom: 14,
alignItems: 'center',
},
navBarButton: {
marginTop: 12,
},
navBarButtonText: {
fontSize: 17,
letterSpacing: 0.5,
marginTop: 12,
},
navBarTitleText: {
fontSize: 17,
letterSpacing: 0.5,
color: '#333',
fontWeight: '500',
position: 'absolute',
left: 0,
right: 0,
bottom: 14,
textAlign: 'center',
},
};
This works for me.

I have a ultimate solution for this problem, regardless language or font size of the title.
please review the PR #151

I have problems with title not centering too. I managed to center left and right button. But title seems to take on a different styling from the other 2 since I cannot apply the same styles on all 3 of them to arrive at similar results. I have observed that using a custom component for title restricts the position of it.
Please review @xing-zheng xing-zheng's PR. Thanks (:


S3 API compatibility

I want to choose another S3 vendor and set my own key, secret, region, bucket name and endpoint url.

What you are describing is on the infrastructure side. The packages in this repo do not interact with s3 other than via presigned urls returned from the uploadthing service. You could write your own compatible service if you wanted this functionality 👍


Add support for adding assets from CDN

It will be nice if I can set all the dependencies on the webpack config file, instead of having to handle this in my Server side code.
For example I am using the flask-webpack plugin so as my webpack packaged files are being loaded to the flask app. At the moment, in the Flask app I have added a script tag that loads jQuery from CDN and then in the webpack config file I add it to the externals so as I can use it in the code. Although this makes me to have to update the code in two places.
Wouldn't be great if I can add a list of full URLs to be added in the manifest file, and then having flask-webpack or other similar plugins parse it from there?

My concern is this isn't really a responsibility of this plugin but I am not opposed to the general idea.
What you could do for now is pass in a custom function reporter (just copy/paste the default one and modify it to do what you want). This wouldn't require forking this plugin. I'd be curious to see what you come up with.

I thought the same at the begging, although this plugin is the only one which can talk with my Flask app.
I will try to do what you suggest this week, thanks.

Yeah I currently just enter in the CDN addresses in my server side templates.
Although if you use a CDN locally you can configure webpack to use that so your URLs are configured with the CDN address automatically.
This project is configured to do this: https://github.com/nickjj/build-a-saas-app-with-flask (check the config/ folder as well as the Dockerfile which is where the assets get built).
Also I do want to point out the flask-webpack template tags do understand URLs directly. If you dropped in the jquery CDN URL instead of passing in an asset name, it knows to use the URL straight up with the javascript_tag helper.

Thanks a lot for your quick responses.
Indeed, if everything is served from same location (CDN) all is fine, but what happens if I prefer to use another location (CDN) for some libraries? e.g. jQuery.
Thanks for you note that the javascript_tag helper already understands full URLs, haven't checked this yet ;)

Going to close this. Sorry I missed your last reply for so long. In the case where you want to use multiple CDNs (ie. jQuery), the javascript_tag won't touch the URL so it'll take it as is.


Server "lua_ls" is not a valid entry in ensure_installed.

I get below warning when I open neovim:
[mason.nvim] [mason-lspconfig.nvim] Server "lua_ls" is not a valid entry in ensure_installed. Make sure to only provide lspconfig server names.

and this is my config:
return {
  "neovim/nvim-lspconfig",
  dependencies = {
        "williamboman/mason.nvim",
        "williamboman/mason-lspconfig",
  },

  config = function()
    local servers = {
      "lua_ls",  -- lua
      "clangd",  -- c cpp
      "pylsp",   -- python
    }

    local capabilities = require('cmp_nvim_lsp').default_capabilities()
    require("mason").setup({
      ui = {
        icons = {
          package_installed = "√",
          package_pending = "→",
          package_uninstalled = "×"
        }
      }
    })
    require("mason-lspconfig").setup({
      ensure_installed = servers,
    })
    require("lspconfig").lua_ls.setup {
      capabilities = capabilities,
    }
    require("lspconfig").clangd.setup {
      capabilities = capabilities,
    }
    require("lspconfig").pylsp.setup {
      capabilities = capabilities,
    }
  end
}

I have the same issue.
I also tried intelephense, to confirm that the issue isn't specific to the Lua language server, and encountered the same error.


Support tmux session titles with non alphanumeric characters

Attach session with a non alphanumeric name is not working.
For example : if I select a tmux session named 'private_dotfiles', I get this error :
.tmux/plugins/tmux-fzf/scripts/session.sh' returned 123
I'll submit a PR soon, it's a regexp pb.

Thanks for your fix! This PR has been merged.


Code in Form docs usage doesn't import useForm

There should be an import import { useForm } from "react-hook-form"; in https://ui.shadcn.com/docs/components/form#define-a-form

Yes it is missing in the docs

This issue has been automatically closed because it received no activity for a while. If you think it was closed by accident, please leave a comment. Thank you.


Problem following the guides.

Hey, I seem to be having a problem following the guides on creating a bot, unsure whether i'm just being dumb or things no longer match.
The docs say that DiscordSocketClient is in Discord.WebSocket but all I have is Discord.Net.WebSockets. The nuget also apparently doesn't support .NET core 1.1 or 1.0, so really unsure whether I've just done something wrong.
Using VS 2017, unsure whether this is a problem
Thanks for helping, I've been looking into it for about an hour and can't even get the guide working 😢

Did you download Discord.NET 0.9.6 by accident?

The Discord.WebSocket namespace is found in the Discord.Net.WebSockets package. Make sure to check show prereleases to get access to the 1.0.0-rc build, which supports .NET Core.

That'll be it, I downloaded the latest stable release off Nuget, I'll get it a different way and try again

PS, these types of questions are more suitable for the discord support server (click on #dotnet_discord-net).

Everything seems fine now, thanks ^_^


Redirect loop for /pages/platform/serving-pages/

Example URL(s)
https://developers.cloudflare.com/pages/platform/serving-pages/
Actual Behavior

Expected Behavior
No loop
Additional information
No response

Resolved in #7345


Missing appealDraws make canRule always falsy in getDataForDispute

Causes kleros/kleros-juror-front#12

If there are no draws for the juror canRule has to be false. appealDraws are currently being populated normally. This could have been an issue with old disputes missing appealDraws. There is no way to fetch the draws for a dispute from a previous session. Closing this issue


Adding Rust benchmarks

Description
Rust benchmarks using the bencher crate (without nightly). I tried the nightly bencher, but the Rust Bazel rules were buggy. Also fixed a small Go typo.
Affected Dependencies
Added bencher crate dependency.
How has this been tested?
N/A
Checklist

[x] I have followed the Contribution Guidelines and Code of Conduct
[x] I have commented my code following the OpenMined Styleguide
[x] I have labeled this PR with the relevant Type labels
[x] My changes are covered by tests

@Daniel-Liu-c0deb0t , LGTM aside from the conflicts in the Go benchmarks. Feel free to merge when these are resolved.


Old version installed in Loki?

I just installed Dropbox with the script, and it worked great.
The only thing is that it installed version 9.4.49, and from dropbox.com I can apparently download version 10.28:
https://www.dropbox.com/download?dl=packages/ubuntu/dropbox_2015.10.28_amd64.deb

Correct me if I'm wrong, but I think the version "2015.10.28" corresponds to the package "nautilus-dropbox", which is a helper that installs the "dropbox" binary, whose version is "9.4.49".

I'm sure you are correct. Thank you!


Expose register method to manually subscribe new user

Thank you for this plugin!
What do you think about this? I would like to design my own dialog so I exposed three methods:
PerfectyPush.register(): Promise<boolean> ask for permission, return promise fulfilled by user action
PerfectyPush.unregister(): Promise<boolean>
PerfectyPush.isRegister(): boolean check if this browser have permission and is registered

Please check the failing CI job, I've allowed it to run


Permission for camera use?

Do I need to install the camera plugin to get the camera permission?

Hi,
Could you give a little more context to your issue?

Are you trying to run this on android/iOS/web?
Which version of the OS/browser?
By "plugin" you mean cordova plugin?

To be more clear about this repository's purpose:

This sample repo is meant to be a PWA so a website with a native like experience.
I did not test it as an hybrid app, even though I use Ionic (and Cordova underneath).
As you'll see, there is no config.xml file, and no Cordova Platform in this repo.
This is a pure web app. I wanted to test how to achieve a QR Scanner from a pure web site to see if this feature could be done in a PWA. My conclusion is.. not yet.

Would you think updating the README with this be of any help for other github users?


Big value handling

Some claims are very big so a bigger field is needed

@mirceanis this is a potential breaking change for anyone with existing databases. Should we mark it as a breaking change and increment the version nr?

@mirceanis this is a potential breaking change for anyone with existing databases. Should we mark it as a breaking change and increment the version nr?

Why? I think typeorm manage to upgrade the database schema gracefully

I'm not well versed enough in TypeORM to know if this is breaking or not.
How would one check?
I imagine create a daf-based app, use some claims, upgrade daf to this PR, see if everything still works?
If it doesn't work out of the box, then a migration is necessary and if DAF handles that migration, it doesn't count as a breaking change, because the DB schema is internal to DAF.


Update coreclr-debug packages for 1.3 release.

@rajkumar42 @gregg-miskelly @caslan

We should do some sanity testing before we merge this, but certainly looks otherwise good.

This addresses #581


Refactor Mend CLI Folder

Performed the following:

Structure Mend CLI folder similar to UA
Added Readme to Mend CLI
Removed Conan from UA/GH Readme
Updated name convention from WS to Mend in GH Actions for UA

A simpler unified CLI example can be found here, which we could use instead. Goal is to reduce number of files needed and complexity: https://github.com/lukebrogan-mend/Mend/blob/main/mend-azure-devops.yml

A simpler unified CLI example can be found here, which we could use instead. Goal is to reduce number of files needed and complexity: https://github.com/lukebrogan-mend/Mend/blob/main/mend-azure-devops.yml

Looks great, Luke.  My only suggestion would be to add another comment beneath the trigger recommending to scan on push to main and push to release branches since scanning release branches and the main branch are a best practice for pipeline scans.  Then we should do this per technology in this refactor instead of having so many files.
Example below, feel free to reword
# Define your own pipeline trigger here:
# Below is defined a trigger for scanning on a push to the main branch and scanning on a push to release branches.  This is considered an ideal time to scan so that an SBOM can be created for software releases and inventory is managed for released software.
trigger:
- main
- release*

The only downside with these examples is the need to "touch" the examples when having a call with a prospect. Therefore, created 4 examples per each use case.
Let me know how to you want to proceed in order to update the PR.

Updated the structure except for Github.com as we have a README with plenty of good examples for the pre-steps.
How do you want to implement it? if we do it in one file, the example for GH will be very long.

Updated the structure except for Github.com as we have a README with plenty of good examples for the pre-steps. How do you want to implement it? if we do it in one file, the example for GH will be very long.

What README are you referring to?  The changes look really good so far.  Just trying to understand what you are referring to about GitHub.  In the future, please squash your commits before submitting a pull request as it is incredibly difficult to review right now.

Updated the structure except for Github.com as we have a README with plenty of good examples for the pre-steps. How do you want to implement it? if we do it in one file, the example for GH will be very long.

What README are you referring to? The changes look really good so far. Just trying to understand what you are referring to about GitHub. In the future, please squash your commits before submitting a pull request as it is incredibly difficult to review right now.

This is the README (from UA space): https://github.com/mend-toolkit/mend-examples/blob/main/Unified Agent/CI-CD/GitHub/README.md

@tabac-mend , I'm going to merge this PR.  If you if you want to clean up the UA folder, please create a new PR.


Fix CollectionConstraint to allow uniqueItems to be false

According to the latest json-schema-validation.
uniqueItems should be either a boolean or if not present, set to false

+1