Best Setup

2 questions:-

The 'WalkThrough_VS21010' document indicates that i need to include 'Platform.h" ($(CPPUTEST_HOME)\include\Platforms\VisualCpp\Platform.h) but neither the file nor the folder exists. I assume this is simply some legacy requirement that is no longer needed?
I am confused about the recommended setup, with the unit tests as separate project from the application to be tested. I assume what is required is some setting that makes the unit test project include the application within it when it is built. This does not seem to be mentioned in the walkthrough document and the setup it suggests appears to be of no practical value as the unit tests can not interact in any way with the application project. As a newbie to this I have a feeling I am missing something obvious!

You typically add the CppUTest include directory to compiler include searchpath. Then you only have to #include <CppUTest/TestHarness.h> in your test implementations.
This depends a little on the build system you are using. You can take this project as example: There's a src/ folder which contains all the production code while the test/ folder contains only the tests. While the latter uses the production code under test (eg. links the code to the tests), the production code should not include any test stuff.

Hi Offa
Thanks. Your suggestion for 2. is exactly how I thought it should be done. I guess there must be  a way of telling VS201x to use the app project when building the unit test project.
Not sure that your answer 1 address my q1. :-) Everything builds nicely and nothing objects to the "Platform.h" file being omitted. Just looking for confirmation that this really is not required when using VS2013.

Just looking for confirmation that this really is not required when using VS2013.

No, it shouldn't be required anymore. The header mentioned above will take care of everything you need.

I removed the VS2010 doc and moved it to the pages repository. There someone can update it. Thanks!


Topmost item in Lorem lpsum generator text corpus list has a very narrow mouse reaction area and overlaps with the title bar

Current behavior

How to reproduce it (as minimally and precisely as possible)
Go to Lorem lpsum generator, click text corpus, navigating to the top items in the list, observe the item's reaction to mouse movement.
Expected behavior

the topmost items should have the same mouse-on reverse highlight action, just like other items.
the pop-up list should always top the app, which means holding the focus even when the mouse moves to the title bar area of the app, no overlapping.
all display language GUI should be taken into consideration, i have tried the en-us and zh-cn languages, problems exist under both.
this is a report from WIndows 10 platform, no idea what happens in a Linux and MacOS one, please give it a try, probably the same problem.
new discovery, same problem happen in "settings - language" too, i'm afraid you have to investigate into all pop-up selection box in this app.

Screenshots
see above
Workaround
No response
Affected platforms
Windows
Affected DevToys kind
DevToys (app with GUI)
DevToys Version
Version 2.0-preview.4, OS version: Windows 10 Pro, 22H2, 19045.4717
Relevant Assets/Logs
No response

Hello,
Thank you, I made a fix :)


Default Collection for Checkboxes

Motivation:

This pr add some intelligence to the form build dsl and assumes that
when a collection is not passed the form should call all on the
supplied model name.
enhancement #95

Thanks Cory!!


the patch passed

+1 :green_heart:
javadoc
0m 32s

the patch passed with JDK Ubuntu-<IP_ADDRESS>+1-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
javadoc
0m 59s

the patch passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
spotbugs
1m 45s

the patch passed

+1 :green_heart:
shadedclient
20m 48s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
190m 20s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch passed.

-1 :x:
asflicense
0m 27s
/results-asflicense.txt
The patch generated 1 ASF License warnings.

279m 31s

Reason
Tests

Failed junit tests
hadoop.hdfs.TestDFSStripedInputStream

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5888/32/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5888

Optional Tests
dupname asflicense codespell detsecrets compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle xmllint

uname
Linux 173499360b32 5.15.0-88-generic #98-Ubuntu SMP Mon Oct 2 15:18:56 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 227feac76ec559384f31a47817fbe5f048c43ab4

Default Java
Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-<IP_ADDRESS>+1-post-Ubuntu-0ubuntu120.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5888/32/testReport/

Max. process+thread count
3971 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5888/32/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

Please disscuss in https://github.com/apache/hadoop/pull/6005
This is closed because Apache Yetus always faild with once a oom file.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 20s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  1s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+0 :ok:
detsecrets
0m  1s

detect-secrets was not available.

+0 :ok:
xmllint
0m  0s

xmllint was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 2 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
31m 40s

trunk passed

+1 :green_heart:
compile
0m 42s

trunk passed with JDK Ubuntu-<IP_ADDRESS>+1-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
compile
0m 37s

trunk passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
checkstyle
0m 37s

trunk passed

+1 :green_heart:
mvnsite
0m 42s

trunk passed

+1 :green_heart:
javadoc
0m 40s

trunk passed with JDK Ubuntu-<IP_ADDRESS>+1-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
javadoc
1m  5s

trunk passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
spotbugs
1m 43s

trunk passed

+1 :green_heart:
shadedclient
20m 25s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 36s

the patch passed

+1 :green_heart:
compile
0m 34s

the patch passed with JDK Ubuntu-<IP_ADDRESS>+1-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
javac
0m 34s

the patch passed

+1 :green_heart:
compile
0m 31s

the patch passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
javac
0m 31s

the patch passed

-1 :x:
blanks
0m  0s
/blanks-eol.txt
The patch has 1 line(s) that end in blanks. Use git apply --whitespace=fix <<patch_file>>. Refer https://git-scm.com/docs/git-apply

-1 :x:
blanks
0m  0s
/blanks-tabs.txt
The patch 260 line(s) with tabs.

-0 :warning:
checkstyle
0m 30s
/results-checkstyle-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs-project/hadoop-hdfs: The patch generated 4 new + 264 unchanged - 0 fixed = 268 total (was 264)

+1 :green_heart:
mvnsite
0m 37s

the patch passed

+1 :green_heart:
javadoc
0m 31s

the patch passed with JDK Ubuntu-<IP_ADDRESS>+1-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
javadoc
0m 58s

the patch passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
spotbugs
1m 41s

the patch passed

+1 :green_heart:
shadedclient
20m 36s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
188m 43s

hadoop-hdfs in the patch passed.

-1 :x:
asflicense
0m 27s
/results-asflicense.txt
The patch generated 1 ASF License warnings.

274m 42s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5888/34/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5888

Optional Tests
dupname asflicense codespell detsecrets compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle xmllint

uname
Linux 0d24d0bd382c 5.15.0-88-generic #98-Ubuntu SMP Mon Oct 2 15:18:56 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 0909b493f8604e51e0a9797065906b034d710718

Default Java
Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-<IP_ADDRESS>+1-post-Ubuntu-0ubuntu120.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5888/34/testReport/

Max. process+thread count
4507 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5888/34/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 31s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+0 :ok:
xmllint
0m  0s

xmllint was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 2 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
47m 29s

trunk passed

+1 :green_heart:
compile
1m 19s

trunk passed with JDK Ubuntu-<IP_ADDRESS>+1-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
compile
1m 13s

trunk passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
checkstyle
1m 14s

trunk passed

+1 :green_heart:
mvnsite
1m 25s

trunk passed

+1 :green_heart:
javadoc
1m  4s

trunk passed with JDK Ubuntu-<IP_ADDRESS>+1-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
javadoc
1m 37s

trunk passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
spotbugs
3m 48s

trunk passed

+1 :green_heart:
shadedclient
38m 51s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 17s

the patch passed

+1 :green_heart:
compile
1m 20s

the patch passed with JDK Ubuntu-<IP_ADDRESS>+1-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
javac
1m 20s

the patch passed

+1 :green_heart:
compile
1m 13s

the patch passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
javac
1m 13s

the patch passed

-1 :x:
blanks
0m  0s
/blanks-eol.txt
The patch has 1 line(s) that end in blanks. Use git apply --whitespace=fix <<patch_file>>. Refer https://git-scm.com/docs/git-apply

-1 :x:
blanks
0m  0s
/blanks-tabs.txt
The patch 260 line(s) with tabs.

-0 :warning:
checkstyle
1m  6s
/results-checkstyle-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs-project/hadoop-hdfs: The patch generated 4 new + 264 unchanged - 0 fixed = 268 total (was 264)

+1 :green_heart:
mvnsite
1m 17s

the patch passed

+1 :green_heart:
javadoc
0m 54s

the patch passed with JDK Ubuntu-<IP_ADDRESS>+1-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
javadoc
1m 34s

the patch passed with JDK Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

+1 :green_heart:
spotbugs
3m 52s

the patch passed

+1 :green_heart:
shadedclient
40m 23s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
217m  7s

hadoop-hdfs in the patch passed.

-1 :x:
asflicense
0m 38s
/results-asflicense.txt
The patch generated 1 ASF License warnings.

368m  6s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5888/33/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5888

Optional Tests
dupname asflicense codespell detsecrets compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle xmllint

uname
Linux 30a4f6049ebb 5.15.0-88-generic #98-Ubuntu SMP Mon Oct 2 15:18:56 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 227feac76ec559384f31a47817fbe5f048c43ab4

Default Java
Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-<IP_ADDRESS>+1-post-Ubuntu-0ubuntu120.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_382-8u382-ga-1~20.04.1-b05

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5888/33/testReport/

Max. process+thread count
3444 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5888/33/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

@LiuGuH are you still working on this PR?

@LiuGuH are you still working on this PR?

This version is running on my production environment for long time. It is almost already done in my side. Thanks


FIX Correctly identify deprecated API in withNoReplacement

Parent issue

https://github.com/silverstripe/silverstripe-framework/issues/10705

Should target 4.12?

https://github.com/silverstripe/silverstripe-framework/blob/4.12/src/Dev/Deprecation.php
4.12 doesn't have withNoReplacement()


Fix for building on Windows with GCC

Merged. Thanks for reporting.
My tidy little header is getting a bit more ugly than originally planned (>_<)

May be even better will be write like this:
#if defined(_WIN32) || defined(_WIN64)


Include update instructions link in README

This pull request updates the README file in the following ways:

Include a link to the instructions for updating Terraform-backed providers
Remove a reference to the now-deprecated aws-serverless repo, since the functionality is now subsumed directly into the packages in this repository.

I'll open a second PR to link to additional abstractions shortly.


HDDS-4094. Support byte-level write in Freon HadoopFsGenerator

What changes were proposed in this pull request?
Teragen seems to use the byte level write method of FSDataOutputStream (write(byte) is used instead of write(byte[], int, int)).
It seems to be a good idea to extend existing ContentGenerator of ozone to support the write in smaller chunks to make it easier to reproduce performance problems.
Note: statistics from FileSystem instance:
Closing file system instance:<PHONE_NUMBER>
   write.call: 100001066
   write.allTime: 215951
   hsync.call: 1
   hsync.allTime: 3
   hflush.call: 0
   hflush.allTime: 0
   close.call: 4
   close.allTime: 62

This was a teragen test with 1GB data (and statistics from one container). write method seems to be called multiple times which means smaller write buffer.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-4094
How was this patch tested?
Executed Freon test with or without turning on the new flag:
https://elek.github.io/ozone-notes/performance/19_hcfs/

Thanks @elek for updating the patch.


Dungeons Libraries causing error; starting integrated server fail

hi i have experienced a problem with the new update 1.16.5 version 1.01.jar with having treasure 2 in the folder it causes the mods to bug out and causing an infinite loop to do with enchantments

What is treasure 2?

it is a mod that basically adds new chests thats are locked and you would have to find a key to unlock them

Closed for 2 reasons:

Not enough info provided
1.16.5 is no longer being maintained


⚠ ClamAV has degraded performance

In fa4e699, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 387 ms

Resolved: ClamAV performance has improved in f70870c.


adding file.py

We have the schema for file and fileset, however the main piece of file is integrated in the file.py file which we do not have yet. This file looks complicated and there are parts that should be changed according to our schema. Without this, it is not possible to add files, and without files it is not possible to run many wrangler scripts.

Commented out the part on aws and have a simple working version.


Support revision history

It would be apt to add support for post revisions for posts either straight away or using the modified date in the front-matter to help Medium stay in sync with the original content. Without it, users who later modify their blogs will cause the duplicative content on Medium to become out-of-sync. This issue exacerbates by the concern addressed by https://github.com/aarongustafson/jekyll-crosspost-to-medium/pull/34

I kept having Ruby/Jekyll issues when upgrading my Mac, so I have moved off of Jekyll and will not be working on this project anymore, going forward. I am going to flag this as won’t fix, but leave it open in case someone else wants to pick up the project from here.


user docs: Write user docs for the Find your Zulip organization feature

Write user guide documentation for the Find your Zulip organization feature, following Zulip's guidelines on writing user guide documentation.

Write an outline for documentation for this feature and post it to the Zulip #documentation stream to get feedback from other contributors.
Add your documentation to a new file to the templates/zerver/help/ directory. The name of the file should be a lowercase, hyphenated version of this feature like view-the-markdown-source-of-a-message for the feature "View the Markdown source of a message".
Add a link to your guide under the appropriate section in templates/zerver/help/index.md. The end result should be a nice and clear user guide for the feature, written in Markdown.
Verify your documentation looks as you intended and all links work; it's common to make small mistakes in the markdown that cause your documentation to not look right.
Add a commit and submit a pull request. The title of both the commit and the pull request should be docs: Add user guide for *feature*.

I am new to Zulip, can I take this ?
If yes can you elaborate how to proceed please

@himanshu81494 Sure! I just updated the issue description with a general outline of steps you should take to write the documentation. Let me know if you have any more questions :)

@synicalsyntax where can I find and study about this feature?

@himanshu81494 Play with the Zulip UI to learn how feature works in Zulip. If you can't find out how to do something in Zulip (or whether it's even possible), ask about it on the #documentation stream; one of the contributors can probably help you :)

This one is interesting in that it's turned off by default.  Set FIND_TEAM_LINK_DISABLED = False in zproject/dev_settings.py to test it.

@zulipbot claim

@zulipbot claim --force

I am new to Zulip, can I take this ?
If yes can you elaborate how to proceed please

@visuvasi Hello, welcome to Zulip, when you see an "in progress" tag it means someone else is already working on it, try to find some issues with beginner tag or good first issue tag. For any further help chat on Zulip.

@zulipbot abandon

@zulipbot  claim

Closing, since these are no longer good tasks for new contributors.


bug in the website when opening the website in safari like the close button is floating on the screen.

What version of Safari? v1.11 introduced styling for the ‘’ element
which is now supported in current versions of Safari. Perhaps you’re using
an older browser version?
On Wed, Oct 26, 2022 at 6:16 PM VenkataBhaskar @.***>
wrote:

[image: Screen Shot 2022-10-27 at 6 43 07 AM]
https://user-images.githubusercontent.com/111889155/198168198-34046dee-0d08-4b77-ba1f-05d36bccf352.png
—
Reply to this email directly, view it on GitHub
https://github.com/andybrewer/mvp/issues/87, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAABXNJ4FMO7U7T6HJ5QRWLWFHJWJANCNFSM6AAAAAARPRJCDU
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
Andy Brewer

Oh yes I am using version 15.1 that is why it's not supported.
Thank you.


Initial commit of work based off digital-register-frontend

All good. Seen it running

Reviewed. Looks good, left two questions.


pykalman/datasets/ not copied when pykalman is installed with pip

Hi,
There must be a problem in the setup.py script, because when (fresh) installing pykalman (on a Ubuntu machine), all the python code is copied but not the data/ and descr/ folders of the pykalman/datasets/ folder.

Just had the same problem, and the .mat file from github cannot be loaded. Error message is ValueError: Unknown mat file type, version 100, 110

2 months and no one replied yet?
Is the @pykalman project still maintained?!

looks like not. I ran into multiple issues with the example code and couldn't get it running at all.

Sadly.

Indeed, the dataset folder is not copied when installed with pip. However, if you replace the dataset folder from your pip install, by the repo's dataset folder, the example works.

Hello, are there any plans to make the installation work with PIP?


Moderation Queue

It would be nice to be able to push comments to a moderation queue.
I suspect it would work that if the user lacks a specific ROLE (for role based security) or is missing a specific Acl entry, the comment should be moderated.

:+1:

Hi, I would like to suggest that this is implemented as a separate bundle, which is quite easy to do using eventlisteners that fire when comments are posted.


Implement APIs for some threading metrics (CoreRT)

API review: https://github.com/dotnet/corefx/issues/35500
May depend on https://github.com/dotnet/coreclr/pull/22754
Fixed Timer implementation on Unixes. Previously there was only ever one timer request from the upper-level implementation and that is not the case anymore, so the lower-level "app domain timer" implementation needed to handle multiple timer requests.

There is a slightly noticeable decrease in perf of throughput of local work items queued to the thread pool in raw overhead per work item. It is very small and when a very short delay is added to the work item the difference is almost not measurable, so it should be acceptably small.

Fixed Timer implementation on Unixes. Previously there was only ever one timer request from the upper-level implementation and that is not the case anymore, so the lower-level "app domain timer" implementation needed to handle multiple timer requests.

I am not sure I understand your motivation for the change. I believe the whole point was to split the queues and process them separately in separate threads. Now you are trying to undo that at lower level. Am I missing something?

I am not sure I understand your motivation for the change. I believe the whole point was to split the queues and process them separately in separate threads. Now you are trying to undo that at lower level. Am I missing something?

Only one thread was created and the TimerQueue ID was being ignored. There isn't much benefit to using multiple threads, the goal of the original change was to split the queues such that each queue gets a separate callback on the thread pool and each queue has its own lock so they can operate in parallel, that benefit is still there. CoreCLR also uses only one timer thread to serve all timer requests from TimerQueue.

Only one thread was created and the TimerQueue ID was being ignored.

The ID was not ignored, it just was not necessary since the instance variables could be used in the pure managed code implementation. The TimerQueue objects were instantiated here:
https://github.com/dotnet/corert/blob/a8cb34b89e737806a71ca7b8e09a07febda3875a/src/System.Private.CoreLib/shared/System/Threading/Timer.cs#L46-L54
Each TimerQueue instance then already had a reference to it's own queue in the instance variables and SetTimer in Timer.Unix.cs started the thread for that queue.

There isn't much benefit to using multiple threads, the goal of the original change was to split the queues such that each queue gets a separate callback on the thread pool and each queue has its own lock so they can operate in parallel, that benefit is still there.

Ok, for reference: https://github.com/dotnet/coreclr/pull/14527.

CoreCLR also uses only one timer thread to serve all timer requests from TimerQueue.

This is the part I missed. I saw the code in AppDomainTimerNative::CreateAppDomainTimer which lead me to believe that more threads are actually created. However, ultimately it's all folded back into one thread in
https://github.com/dotnet/coreclr/blob/eb1aa094dbd32f98fd4ce5dead9f27f9c78a32c5/src/vm/win32threadpool.cpp#L4429-L4521.
I am not sure if the change back to one thread is actually beneficial, but if it matches CoreCLR then it's probably ok. Would be nice to have a benchmark, there is a simple one in https://github.com/dotnet/coreclr/pull/20302.

SetTimer in Timer.Unix.cs started the thread for that queue

It only started one thread for all TimerQueues because s_timerEvent is static, actually the starting of the thread also had race issues (which were introduced after multiple queues were introduced) that may cause multiple threads to be started, possibly multiple threads for the same queue, and very possibility not starting threads for other queues. See the test case I added in Threading.cs, which fails reliably before the fix.
It could be fixed with fewer changes and more threads (changing some static variables to instance variables, and some such), and it would still be unfortunate to have 32 timer threads on a 32-core machine all for just the purpose of a very small improvement at best. Timers can only request to tick once per millisecond, which is already long, and the resolution is actually about 15+ ms on Windows, any gain with having multiple threads serving those timer requests would be squashed just by the timer resolution.

It only started one thread for all TimerQueues because s_timerEvent is static

Ah, that is a bug I missed. It should have been changed to an instance variable. It is called from under lock (FireNextTimers -> lock (this) -> EnsureTimerFiresBy -> SetTimer), so other race conditions were not really a concern there.

it would still be unfortunate to have 32 timer threads on a 32-core machine all for just the purpose of a very small improvement at best.

I'm not convinced that the improvement is so small for a case where the timers are heavily used. However, I do believe that for the common case it will likely be overkill to have multiple threads.
There are couple other small concerns I have with the change so it would have been nice if it was split off into separate PR.

The RT implementation in latest commit is broken, will fix and retest

Updated perf numbers are here

Ping for review please

@dotnet-bot test OSX10.12 Debug and CoreFX tests

The API review was approved. Couple of pending items is:

Whether to expose PendingLocalWorkItemCount and PendingGlobalWorkItemCount. I have asked for more info on whether they would be useful, by default I'll go ahead and remove them
Whether we want to expose CompletedWorkItemCount at all.

I think that's the most expensive one to track of the lot, as it involves a bit of extra work per work item
It is possible to mostly eliminate the extra cost for managed work items in CoreCLR and CoreRT. Work items that may have some extra cost are timer callbacks in CoreRT Windows and IO completion work items in CoreCLR Windows.
The info would enable easily determining work item throughput and whether the thread pool is stalled

The perf difference from adding CompletedWorkItemCount seems to be very small, so for now I'll consider that it can reasonably be added

I have updated to remove local/global versions of PendingWorkItemCount. They can be added in the future if they would be useful. I think all three PRs (CoreCLR, CoreRT, and CoreFX) are now ready for review. @jkotas / @stephentoub, would you be able to take a look?

Rebased

shared\System\Threading\ThreadLocal.cs(457,37): error CS8600: Converting null literal or possible null value to non-nullable type.
shared\System\Threading\ThreadLocal.cs(466,31): error CS8600: Converting null literal or possible null value to non-nullable type.
shared\System\Threading\ThreadLocal.cs(466,82): error CS8600: Converting null literal or possible null value to non-nullable type.
shared\System\Threading\ThreadLocal.cs(485,41): error CS8600: Converting null literal or possible null value to non-nullable type.
shared\System\Threading\ThreadLocal.cs(493,35): error CS8600: Converting null literal or possible null value to non-nullable type.
shared\System\Threading\ThreadLocal.cs(493,86): error CS8600: Converting null literal or possible null value to non-nullable type.


Need help regrding multi-speaker longform speech generation

Could you provide some resources for a long-form speech generation code that allows for switching between multiple speakers within the same text similar to what you did in the youtube video.

Hello! Thank you for your interest in 🍵 Matcha-TTS.
This is a great idea, however I do not have a straightforward interface to do it currently. But you can pass two additional lists one of speaker IDs and a list of tuples of their phone boundaries. Later once you pass speaker IDs through the speaker encoder and obtain speaker representations, instead of, broadcasting it for all x, just broadcast only for the specific word boundaries here
https://github.com/shivammehta25/Matcha-TTS/blob/c8d0d60f87147fe340f4627b84588e812e5fbb00/matcha/models/components/text_encoder.py#L403
and in the decoder.
https://github.com/shivammehta25/Matcha-TTS/blob/c8d0d60f87147fe340f4627b84588e812e5fbb00/matcha/models/components/decoder.py#L387
Hopefully this helps :)

Hello, I am assuming you are happy with the answer for now, please feel free to reopen the issue and continue the discussions in case of any further points.


Update Next.js mention in Turbopack docs

Description
Make sure name is consistent.

Testing Instructions

#7928  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @timneutkens and the rest of your teammates on  Graphite


Add aggregator example in plugin overview

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

https://hub.steampipe.io/plugins/turbot/oci#multi-tenant-connections


Consider adding Aktienfinder as data provider

Not suitable for the purpose of this repository. Aktienfinder provides no ratings or fair value themselves, but large amounts of raw data that can be used to compute custom rankings and multiple automatically generated fair values based on different characteristics, which correlate poorly with existing fair value information.


Custom packets not suitables for packets! macro

There's 21 custom types which need to implement the Protocol trait in order for all packets to work properly. Here's the list:

[x] ClientDestroyEntities, 0x13, Client, struct
[ ] Properties, 0x20, Client, enum
[ ] BlockRecords, 0x22, Client, enum
[ ] Chunks, 0x26, Client, enum
[ ] ExplosionRecords, 0x27, Client, enum
[ ] Slots, 0x30, Client, enum
[ ] MapIcons, 0x34, Client, enum
[ ] Stats, 0x37, Client, enum
[ ] Players, 0x38, Client, enum
[ ] Matches, 0x3a, Client, enum
[ ] ScoreMode, 0x3b, Client, enum
[ ] ScoreAction, 0x3c, Client, enum
[ ] Team, 0x3e, Client, enum
[x] Event, 0x42, Client, enum
[x] WorldBorderAction, 0x44, Client, enum
[x] TitleAction, 0x45, Client, enum
[x] UseType, 0x02, Server, enum
[x] TabPosition, 0x14, Server, enum

Source

+r @fenhl @eddyb

Maybe a macro could help, but most of this packets are very different from each other.


chore:  fix bug， to avoid crashing the system when passing 0 as a par…

…ameter to UIGraphicsBeginImageContextWithOptions

Brief Description of the PR
Checklist

Demo:
Documentation:

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Migration documentations/links are missing under upgrade section in APIM 4.1.0 documentation

Description:
WSO2 APIM upgrade documentation links for only 4.0.0 to 4.1.0 and 3.2.0 to 4.1.0 (Left hand side navigation) under upgrade section in APIM 4.1.0 documentation. There should be other versions as well (2.x versions too).
Suggested Labels:

Suggested Assignees:

Affected Product Version: WSO2 API Manager 4.1.0
OS, DB, other environment details and versions:    N/A
Steps to reproduce:
Access this link - https://apim.docs.wso2.com/en/latest/install-and-setup/upgrading-wso2-api-manager/upgrading-guidelines/
and see left hand side navigation bar.
Related Issues:

Hi @isuru-udayanga ,
Thank you for reporting this issue!
These pages are actually work in progress and they will be available soon. Therefore, in order to avoid any further confusion, we added these pages to the left navigation and also added Work In Progress banners in each of these pages for the moment via PR [1].

We will update this ticket as soon as these pages are released.
[1] https://github.com/wso2/docs-apim/pull/5704
Regards,
Mariangela Hills

This is fixed now so closing this.


[WIP] add quantum shannon decomposition optimization

Summary
This adds the second optimization for the quantum shannon decomposition described in arXiv:quant-ph/0406176.
Details and comments
Adds a new decomposer class which decomposes a generic two qubit unitary, which may take 3 cnots to implement, into the product of a  diagonal gate and another unitary which takes up to 2 cnot gates. The diagonal gate of this decomposition is used to commute through neighboring gates in the QSD.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

84 of 85 (98.82%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 84.391%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit/quantum_info/synthesis/two_qubit_decompose.py
42
43
97.67%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
54613

Relevant Lines:
64714

💛  - Coveralls

@adjs Thanks for the review. Part of the reason for describing the function option is because it effects the structure of the circuit where I've made a choice about what direction to push the diagonal. It may be a choice that is less preferable depending on whether it reduces the chance for optimization of adjacent circuit elements.I haven't worked out pushing it the other way, but perhaps that could be added later.


Euro Convert USD Extension

Is your feature request related to a problem? Please describe.
Users often need to convert currency from Euro to USD for various purposes, such as travel or online shopping.
Describe the solution you'd like
Develop a user-friendly Chrome browser extension called "Currency Converter" that simplifies the process of converting currency from Euro to USD. This extension should offer quick and convenient conversion functionality, providing users with accurate results based on a fixed exchange rate.
Describe alternatives you've considered
One alternative could involve integrating the currency conversion feature into existing finance or travel-related apps that users already use regularly. Additionally, users could have the option to customize the exchange rate or access additional currency conversion options for different currencies.
Additional context
The goal of this feature is to streamline the currency conversion process for users who frequently need to convert Euro to USD. Completion of this feature can be assessed based on user feedback regarding the ease of use and accuracy of the conversion results provided by the extension. Additionally, monitoring user engagement metrics, such as the frequency of interactions with the extension, can help evaluate its effectiveness in meeting users' currency conversion needs.

@Sulagna-Dutta-Roy  PLEASE ASSIGN ME THIS ISSUE


Remove bwmf from repository

All related code has been moved to https://github.com/taskgraph/bwmf .

LGTM


fix: use std::os::fd on HermitOS

Issue https://github.com/rust-lang/rust/issues/126198 was fixed and added to 1.81.
fixes: #1803

Our MSVR is 1.70, so this will have to wait until we bump it.

Since that bump will be a long time from now, I'm closing this. Thank you @joeydewaal, we'll reopen this once we bump the MSRV.


Unable to login with CRAM-MD5 over TLS

My webserver and my dovecot are on two separate debian.
Dovecot only allow CRAM-MD5 login.

disable_plaintext_auth = yes
auth_mechanisms = CRAM-MD5

On the admin rainloop admin panel connection test is successful. When login-in on the user panel it fail every time.
The dovecot report:

Apr 30 14:38:24 imap-login: Info: Disconnected (tried to use unsupported auth mechanism): user=<>, method=PLAIN, rip=xxxx:xxxx:4ad5:2600:1::205, lip=xxxx:xxxx:4ad5:2600:1::201, TLS, session=

The rainloop log report:

[12:34:25.761][47dd8f22] IMAP[DATA]: < * OK [CAPABILITY IMAP4rev1 LITERAL+ SASL-IR LOGIN-REFERRALS ID ENABLE IDLE AUTH=CRAM-MD5] Dovecot ready.
[12:34:25.767][47dd8f22] IMAP[SECURE]: > TAG1 LOGIN "" "********"
[12:34:25.767][47dd8f22] IMAP[DATA]: < TAG1 NO [ALERT] Unsupported authentication mechanism.
[12:34:25.769][47dd8f22] IMAP[WARNING]: exception 'MailSo\Imap\Exceptions\NegativeResponseException' with message 'MailSo-Imap-Exceptions-NegativeResponseExc
eption (ImapClient.php ~ 1844)' in /var/sites/rain.nailyk.fr/rainloop/v/<IP_ADDRESS>5/app/libraries/MailSo/Imap/ImapClient.php:1844

When I enable plain in dovecot everything is going fine so I suppose the TLS configuration is ok.
I also try to set:

data/data/default/configs/application.ini:use_imap_auth_cram_md5 = On

in the lab section
Did I miss something?
Thank for your time.
P.S.: full rainloop log

Did I miss something?

No, RainLoop does not support CRAM-MD5 authentication (not ready yet :( ).
If you give me test email account, I will try to finish this functionality.

If you give me test email account, I will try to finish this functionality.

Sure! How can I contact you privately to send you the credentials?

<EMAIL_ADDRESS>

Seems there is ameliorations in <IP_ADDRESS>.
But every time password fail:

imap-login: Info: Disconnected (auth failed, 1 attempts in 2 secs<EMAIL_ADDRESS>method=CRAM-MD5, rip=xxxx:xxxx:4ad5:2600:1::205, lip=xxxx:xxxx:4ad5:2600:1::201, TLS, session=

and rainloop log:

--- FlushLogCache: WriteOnErrorOnly
[18:41:34.210][a6759238] INFO[DATA]: [DATE:28.05.16][RL:<IP_ADDRESS>][PHP:5.6.20-0+deb8u1][IP:2a02:8434:4ad5:2600:feaa:14ff:fecf:2113][PID:11987][Apache][apach
e2handler]
[18:41:34.210][a6759238] INFO[DATA]: [Suhosin:off][APC:off][MB:on][PDO:mysql][Streams:tcp,udp,unix,udg,ssl,sslv3,tls,tlsv1.0,tlsv1.1,tlsv1.2]
[18:41:34.210][a6759238] REQUEST[NOTE]: [POST] https://rain.nailyk.fr/?/Ajax/&q[]=/0/
[18:41:34.211][a6759238] AJAX[NOTE]: Action: DoLogin
[18:41:34.211][a6759238] POST[DATA]: {"Email":"nailyk@______.fr","Login":"","Password":"*******","Language":"","AdditionalCode":"","AdditionalCodeSignMe":"0"
,"SignMe":"0","Action":"Login","XToken":"c470173__________________32ed848"}
[18:41:34.213][a6759238] IMAP[NOTE]: Start connection to "ssl://kelen.nailyk.fr:993"
[18:41:34.391][a6759238] IMAP[NOTE]: Connected (success)
[18:41:34.432][a6759238] IMAP[DATA]: < * OK [CAPABILITY IMAP4rev1 LITERAL+ SASL-IR LOGIN-REFERRALS ID ENABLE IDLE AUTH=CRAM-MD5] Dovecot ready.

[18:41:34.439][a6759238] IMAP[DATA]: > TAG1 AUTHENTICATE CRAM-MD5

[18:41:34.440][a6759238] IMAP[DATA]: < + PDY5OTA4_______________________________LRUxFTj4=

[18:41:34.440][a6759238] INFO[DATA]: tiket:<PHONE_NUMBER>516772.1464461344@KELEN
[18:41:34.442][a6759238] IMAP[SECURE]: > *******

[18:41:35.972][a6759238] IMAP[DATA]: < TAG1 NO [AUTHENTICATIONFAILED] Authentication failed.

--- FlushLogCache: Trigger

Are changes needed in the rainloop configuration or something else i miss?
Are you still able to access the dovecot?

Available in the last version.

Confirmed working in <IP_ADDRESS>. (dovecot imaps authentication and postfix smtps tested)
Thanks a lot!


failure to assign invasion defense spawn request to nearest colony when recovering + colonizing a room

Issue summary
Failling to spawn defenders for a room being colonized.
Description of issue:
Looks like logistics : spawngroup is failling to find any valid spawn points for a room being colonized and/or to low level to spawn defenders.
Steps to reproduce:
Be near Tigga with weaker rooms :)
Error message:

[5:17:16 PM][shard2]ERROR   24254542 |    at SpawnGroup.init (main:26939:21)                  | Could not enqueue creep with role hydralisk in E7S54 for Overlord [invasionDefense:26ea0e>rangedDefense]!
[5:17:19 PM][shard2]WARNING 24254543 No colonies meet the requirements for SpawnGroup: invasionDefense:26ea0e>rangedDefense:SG

// Offending line:

Suggested fix (optional):
I tried to walk through the code myself, but there's not a lot of debug entry here that I can find to determine what state the object is in which tracks the valid options for valid hatchery / colonies to spawn from for a given room at the time.
Other information:
effectively running dev version with some modifications.
‣ Version:        Overmind v0.5.2
‣ Checksum:       0x0
‣ Assimilated:    No (clearance code: null) [WIP]
‣ Operating mode: automatic

If this is on the dev branch, then I broke spawn groups in some cases a week or two ago. (Currently they're constantly screaming at me in console every time overmind tries to place a distant power mining directive) I'm planning to rework them in the near future, but until then they're gonna be bugged


Missed channels EPG from Canalplus.com and Telecablesat.fr

Hello Guys,
Firstly thanks you so much for your amazing work
!!!
EPG missed for theses channels from file :
https://github.com/iptv-org/epg/blob/master/sites/telecablesat.fr/telecablesat.fr_fr.channels.xml
and
https://github.com/iptv-org/epg/blob/master/sites/canalplus.com/canalplus.com_fr.channels.xml
RMC SPORT 3
RMC SPORT 4
RMC SPORT LIVE 5
RMC SPORT LIVE 6
Especially RMC SPORT channels, we could re-add or refresh the EPG for the channels RMC SPORT 1 to RMC SPORT 15
Thanks in advance for your daily hardworking for our satisfaction

telecablesat.fr
RMC SPORT LIVE 5
RMC SPORT LIVE 6
is already on the list:
https://github.com/iptv-org/epg/blob/069b8db723beea9e804d82c7f0593dc5d841179a/sites/telecablesat.fr/telecablesat.fr_fr.channels.xml#L211-L212
Programmes for RMC SPORT 3 and RMC SPORT 4 are not on the site
canalplus.com
Missing channels are added via https://github.com/iptv-org/epg/commit/71be477a919441531a553d4b51b00368096cb2b8

Many thanks @freearhey
Really appreciated your work and keep supporting this open-source project with some dollars...
Apparently the xml link Canal.com FR don't display in the global table? Something wrong?

@freearhey apparently all 14 EPG guide for the channels RMC SPORT dont display on Telecablesat.fr and canalplus.com
Besides, canalplus.com xml link is available

The guide from canalplus.com is available again at the link:
https://iptv-org.github.io/epg/guides/fr/canalplus.com.epg.xml

The programme for the RMC Sport channels can be found in the telecablesat.fr guide:
https://iptv-org.github.io/epg/guides/fr/telecablesat.fr.epg.xml

And the canalplus.com guide does not include the RMC Sport channel programme as it is no longer available on the website: https://www.canalplus.com/programme-tv/


[WIP] support multiple registry centers

This patch will try to support multiple registry centers as described in #198 and it's work in progress, I'll continuously commit here and this is for reference and reviewing and early feedback.

Codecov Report

Merging #312 into develop will decrease coverage by 0.08%.
The diff coverage is 36%.

@@              Coverage Diff              @@
##             develop     #312      +/-   ##
=============================================
- Coverage      19.57%   19.48%   -0.09%
- Complexity       289      291       +2
=============================================
  Files             95       96       +1
  Lines           3582     3598      +16
  Branches         553      556       +3
=============================================
  Hits             701      701
- Misses          2806     2822      +16
  Partials          75       75

Impacted Files
Coverage Δ
Complexity Δ

...va/org/apache/dubbo/admin/config/ConfigCenter.java
92.15% <100%> (-0.16%)
18 <1> (+1)

...org/apache/dubbo/admin/config/AdminProperties.java
5.88% <5.88%> (ø)
1 <1> (?)

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 5bc0e33...b3a8c3a. Read the comment docs.

close it and create another draft PR


404 error while accessing client json file from php client library

Hi,
I am developing web application for accessing google reviews by using google my business API. My programming language is PHP.
PHP client library used for getting account information. But when accessing google credential.json file from php client library, I am facing 404 error. Initially json file was placed incode library folder in same location where accessing it. Same error was there.
So I placed json file inside public folder of my project. Even in this situation itself, its showing 404 error.
I am stuck on this. Can anyone give me support on this ?
What may be went wrong here.
Followup Email ID: [4-3050000031171]
Waiting for a positive response.
Sujeesh

You are using Google\Client::setAuthConfig() with the path to your credentials.json file?
First, do not place the credentials file in a public folder. This will allow anyone to view your credentials. Second, it seems as though the path you are providing to setAuthConfig() is incorrect. Since this path is specific to your application and web server, we can't tell you what it is. You will need to figure that out yourself. You can run echo __DIR__; in a PHP file to view the absolute path to that file. That should help you determine the path to your credentials file.

Hi,
Thank you so much for your comments. Defenitely I will try this.

you are using laravel
copy the JSON file into "storage/app/google-analytics/ddfdfdfdf.json"
read file
$client = new Google_Client();
$client->setApplicationName("your app");
$client->setAuthConfig(storage_path("app/google-analytics/ddfdfdfdf.json"));

Hi @sujeeshskn , I hope the above pointers helped you and thus closing this issue.
Please feel free to reopen incase you are still struck on this.


Swap forwarder interfaces.

cc @denis-tingaikin @NikitaSkrynnik

@VitalyGushin Could you also attach a picture of how it looks now?

Remote case looks good with the patch. Have you tested it with local cases?

For local cases interfaces on forwarders also will be swapped


multimaster_fkie: 0.8.10-1 in 'kinetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository multimaster_fkie to 0.8.10-1:

upstream repository: http://github.com/fkie/multimaster_fkie.git
release repository: https://github.com/fkie-release/multimaster_fkie-release.git
distro file: kinetic/distribution.yaml
bloom version: 0.6.9
previous version for package: 0.8.9-0

I'm not 100% sure why this is failing CI, but it looks like the release repository isn't setup entirely properly for the 0.8.10 release.  At least, https://github.com/fkie-release/multimaster_fkie-release/blob/debian/kinetic/xenial/default_cfg_fkie/package.xml, for example, still shows 0.8.9.  How did you do this release?

Oh, my fault. I executed bloo-release with wrong track.
I  executed bloom-release again and now https://github.com/fkie-release/multimaster_fkie-release/blob/debian/kinetic/xenial/default_cfg_fkie/package.xml seems to be ok.
Should I open a new request with bloom-release or how I can trigger travis-checks again.

Should I open a new request with bloom-release or how I can trigger travis-checks again.

You can trigger travis again by closing and opening the PR, which I'll do right now.

For whatever reason, Travis is not updating Github at the moment, but now it's failing with:
rosdistro: DEBUG: Skipped "github_manifest_provider()": specified tag "release/kinetic/node_manager_fkie/0.8.10-1" is not a git tag

At this point, I'd suggest closing this out, going through all of the steps for a new release on the upstream repository (0.8.11), and then doing a new PR to update it.  That will ensure that everything is in the correct state.

I created a new pull request and close this now.


Material-extras does not apply encoding

Material-extras does not apply encoding after #470.

Is this the same as #496?


Completion doesn't stop in function parameters for a comma (argument sparator)

Checkhealth
Put result of :checkhealth compe.
 1
 2 health#compe#check
 3 ========================================================================
 4 ## compe:snippet
 5   - OK: snippet engine detected.
 6
 7 ## compe:mapping
 8   - INFO: `compe#complete` is not mapped
 9   - INFO: `compe#confirm` is not mapped
10   - INFO: `compe#close` is not mapped
11   - INFO: `compe#scroll` is not mapped

Describe the bug
I've tried nvim-compe, but I run into a quite annoying issue for me. I'm using C here. Assume the following function:
bool cli_credentials_guess(struct cli_credentials *cred, struct loadparm_context *lp_ctx)

If I start typing the completion menu open after 3 chars for me. Once I type the ( opening paranthesis, the completion stops and the menu is closed as expected. However if I enter the first argument creds and add the , (comma) the completion doesn't stop the menu stays open.
cli_credentials_guess(cred,

The completion menu even stays open after I add a space after the comma. In combination with a signature help like lsp_signature it is annoying as you don't see the signature help window. It is overlapped by the completion menu.
Let me know if this is clear, else I could record a video.

I was wrong, an opening paranthesis doesn't stop completion too :-(

I've noticed that when using sourcekit-lsp, when I try to complete a method with signature func aMethod(param1: Int, param2: Int) -> Int { it only inserts aMethod. It would be nice if it inserted aMethod(param1: , param2: ). Not sure if its related.

How do you get your completion to only start after typing atleast 3 characters? For me it works with 0 characters which can be pretty annoying. Also, non compatibility with lsp_signature is really a huge issue imo.

require('compe').setup {
    min_length = 3,
}

And yes, it should work fine together with lsp_signature. Currently it is just an overlapping popup windows mess ;-(


Mechanism to auto-install/auto-update gem

Currently, if the neovim gem is not installed, I get this message:
$ vim
function provider#ruby#Call[8]..remote#host#Require[13]..provider#ruby#Require, line 10
Vim(if):Channel was closed by the client
rbenv: neovim-ruby-host: command not found
Failed to load ruby host. You can try to see what happened by starting Neovim with the environment variable $NVIM_RUBY_LOG_FILE set to a file and opening the generated log file.Also, the host stderr is available in messages.
Press ENTER or type command to continue

Instead, it would be nice if it did something like:
$ vim
Your ruby is missing the neovim gem.  Attempting to install...
Successfully installed the neovim gem
Press ENTER or type command to continue

My configuration files

This is a great idea, although I'm not sure we want to make assumptions about the way people install rubygems. Since this would be mostly an nvim-side change anyway, would you mind reopening this as an issue on neovim to get some of their input? Feel free to mention me.


Shorted URL nicht kopierbar/makierbar

Hey, siehe Titel.
Browser: Firefox 63.0.1 (64bit)

Thank you, for this issue. In other browsers it's selectable but in firefox not because there is a disabled field not selectable. This will be fixed in the next day.

Fixed!


Provide option to disable default mappings

Nice plugin! It would however be nice if there were a setting that disabled the default mappings, something like g:textobj_markdown_no_default_key_mappings that defaulted to 0 but could be set to 1.

Thank you for the suggestion. I'll take a look as soon as I can.￼

Also it would be nice if there were a setting one could use to easier remap for instance f/F.
It would be nice if there were a setting that could be set like so:
let g:textobj_markdown_fence_mapping = ['c', 'C']

where the first element remaps f, and the second remaps F, or something similar.
And an equivalent one could exist for "text text objects", i.e. m/M:
let g:textobj_markdown_text_mapping = ['t', 'T']

Also, I just realized that there already is a setting for disabling the default mappings: let g:__textobj_markdown_no_mappings = 1. However, it's only mentioned in the README but not in the helpfiles and that's why I missed it. Perhaps it should be mentioned in the helpfiles as well :)


Make example consistent and out of the box usable

I was checking out the examples and when using it in my own workflow I found out that the example had some copy and past errors. This PR fixes the consistency issue of the example Get all changed files using a list of files and take action base on the changes.

@cfernhout Nice

@all-contributors please add @cfernhout for docs


Please give back the ability to set Google as the default search engine.

Description
Give back the ability to set Google as the default search engine.
Who's implementing?

[ ] I'm willing to implement this feature myself

The problem
I'm frustrated when I have to use a separate search engine that is either too slow, or gives inaccurate results to what I want. Going to google.com is inconvenient for each time I want to search something on google.
This is very bad especially when I need to look up some stuff for my schoolwork. On Google, they use my previous results to provide me with relevant answers, on anything else I will get something completely irrelevant.
I understand that the point of ungoogled chromium is to get rid of google, but switching your search engine to something not on the list should be a feature.
Possible solutions
Readd or let us add our own search engines in the search engine settings page.
Alternatives
No response
Additional context
No response

Duplicate of #2730


make containerPath an optional property for DriverPreCheckInfo

This PR makes the containerPath property optional for the DriverPreCheckInfo interface in driver-definitions to help prepare for the removal of this property.
Closes https://github.com/microsoft/FluidFramework/issues/8548

i'd like to see #8508 done first

Would the simplest sequence be after https://github.com/microsoft/FluidFramework/issues/8508 is done, just to remove containerPath from here as well from here as https://github.com/microsoft/FluidFramework/pull/8436 with a single PR, without optional step ?
@scottn12 as for https://github.com/microsoft/FluidFramework/issues/8508 could we create appropriate tickets for teams working on partner repos (I'm not sure if they are all on github) ?  It's certainly key to understand partner's usage and communicate alternatives with good examples & unit tests in this repo - for current and future partners. At the same time, it may be more valuable to conclude our deprecated API efforts/understanding across the board before us diving into making changes in the partner repos. Unless of course, you have already done that, or its trivial.


A custom slimmed down version of VolumeSource

Slim down VolumeSource by removing deprecated and "removed" fields from the struct. This will slim down our nested use of ExtraMounts.
Using this version of the struct shaves 38k off the cinder-operator bundle size. Using this across the board would have a significant effect on reducing the OpenStackControlplane CRD size and little if any effect on utility of the nested ExtraMount struct.

A preview of the effect on cinder-operator is here: https://github.com/dprince/cinder-operator/commit/95e834d6f50638bcdd0f88bf623b9bb667b196a8

@olliewalsh This is similar to the StatefulSet idea you mentioned from Rabbit.

I removed zz_generated.deepcopy.go as the Makefile here doesn't seem to regenerate it. (just stale). Did something get broken recently?

@bshephar the more we can agree to remove the better.

I removed zz_generated.deepcopy.go as the Makefile here doesn't seem to regenerate it. (just stale). Did something get broken recently?

Not sure, but [1] should solve your problem here.
[1]  https://paste.opendev.org/show/bUqTCs84vk01M2PefMn0/

@dprince an additional change we need to consider across all the operators that implement extraMounts is the way we access corev1.Volumes.
We usually append( ...  a corev1.Volume to an array of the same type, and we need to patch functions like [1] to work with the Volumes struct defined in this module (but Make will tell you more about this comment).
With the goal in mind of reducing the bundle size, I think we're going in the right direction, and I can follow up w/ a glance-operator change (because it's simple) that is based on this patch.
If everyone agrees on that we can then prepare changes for all the operators implementing this new version.
[1] https://github.com/openstack-k8s-operators/cinder-operator/blob/main/pkg/cindervolume/volumes.go#L30

@dprince an additional change we need to consider across all the operators that implement extraMounts is the way we access corev1.Volumes.
We usually append( ...  a corev1.Volume to an array of the same type, and we need to patch functions like [1] to work with the Volumes struct defined in this module (but Make will tell you more about this comment).
With the goal in mind of reducing the bundle size, I think we're going in the right direction, and I can follow up w/ a glance-operator change (because it's simple) that is based on this patch. If everyone agrees on that we can then prepare changes for all the operators implementing this new version.
[1] https://github.com/openstack-k8s-operators/cinder-operator/blob/main/pkg/cindervolume/volumes.go#L30

I updated my cinder PR. Wanted to get general feedback on this before going too far but I think it should work out fine.


Improve webhook with bearer token and processing functions

[ ] support bearer token
[ ] support app-specific processing functions
[ ] payload data validation

Doublon of https://github.com/kalisio/kdk/issues/524


configure cncf cla bot

is this it? https://github.com/marketplace/cla-bot-cla-automation

https://github.com/kubernetes/community/blob/master/github-management/setting-up-cla-check.md

https://github.com/kubernetes/community/blob/master/github-management/setting-up-cla-check.md

thanks, i have configured the bot, now awaiting acceptance of the bot user

thanks, i have configured the bot, now awaiting acceptance of the bot user

All good now.


fix: remove duplicate claude-3-5-sonnet-20241022 model from VERIFIED_MODELS

End-user friendly description of the problem this fixes or functionality that this introduces

[ ] Include this change in the Release Notes. If checked, you must provide an end-user friendly description for your change below

Give a summary of what the PR does, explaining any non-trivial design decisions
Removes duplicate sonnet model from VERIFIED_ANTHROPIC_MODELS since it's already in VERIFIED_MODELS, fixing the double entry in the dropdown menu.

Link of any specific issues this addresses
Fix https://github.com/All-Hands-AI/OpenHands/issues/4653

To run this PR locally, use the following command:
docker run -it --rm   -p 3000:3000   -v /var/run/docker.sock:/var/run/docker.sock   --add-host host.docker.internal:host-gateway   -e SANDBOX_RUNTIME_CONTAINER_IMAGE=docker.all-hands.dev/all-hands-ai/runtime:5474d3e-nikolaik   --name openhands-app-5474d3e   docker.all-hands.dev/all-hands-ai/openhands:5474d3e

@xingyaoww the change removed the wrong entry, thus FE tests failing now in main?


Should values be store in their most compact representation?

Assume we have an array of int4, should each member of array store in their most compact representation?( here int1 or int2 or int4) then we have an array with index table(0x06).
or each int4 member can store in their original format? then we could store it in an array(0x02) without index table because they have same type.
should this be transparent to users or be decided by them?

VelocyPack does not distinguish between int1, int2 and int4. Instead it has the types Int and UInt. These types have a variable length encoding, so they can be used for all types of integers up to 64 bit precision. The SmallInt type offered by VelocyPack is just an optimization, and it will be used automatically when small enough values are added to a Builder.
VelocyPack is optimized for sequentially building values with a Builder object. The assumption here is that the user wants to put together values as easy as possible and does not care much about how the values are encoded internally in VelocyPack. And from the user perspective it should not make much difference if an Array or Object type with index or one without index is used.
But it may matter in terms of storage space. If this is your concern, then I suggest allowing the user to make the decision. AFAIR in the C++ version of the Builder, we always choose the index table version for types Array and Object unless some options are set in the Options object. The attributes that control the behavior are named buildUnindexedArrays and buildUnindexedObjects in the C++ version. They both default to false. But with the Options object that control a Builders behavior, developers have control whether they prefer the compact or indexed format.

@jsteemann thanks for the detailed response. currently C# encodes integral types equal to their declared type, not the length they can fit in.
For example one could use Builder to store number 15 declared as Int32 in C#(Int32 a = 15). currently it's encoded as 0x23 which requires 4Bytes space(same as its C# declared type). But since 15 is small enough it can be stored as 0x20. My question was should I encode integral types to the least length for vpack Int they can fit in?
I have already added a method that store integral types as the most compact representation, please take a look:
https://github.com/ra0o0f/velocypack.net/blob/master/src/ArangoDB.VelocyPack/VPackBuilder.cs#L109-L134

If you want to restore the original C# value type when unpacking a VelocyPack value, then you should use as much space as the C# type originally used, e.g. encode int32 using 4 bytes, int16 using 2 bytes etc. Then the convention would be to decode 4 byte integers into int32, 2 byte integers into int16 etc. This will waste some space however.
If you instead use the smallest possible VelocyPack type a number fits in, then you will save some space, but there is no way to tell the original C# number type from the VelocyPack value.
So the answer is: it depends on what your goals are. If you want users to decide whether they want most compact representation or preserve the original types, maybe making this option will help.

Thanks @jsteemann for the great help, also Slice object can now retrieve value corresponding to VPack type or convert it to C# type that user wants.
I also asked this to know that compactness and saving space is the primary goal for the VPack creators and what would be the trade offs for that.
For making .NET implementation better any comment or advice would be a great help if you look at it.


Unexpected validate behavior with classes

Short version: validating an object created from a function unexpectedly indicates a valid object, but insertion into collection fails as expected. If I take the object and deep clone it, and attempt to validate that, SimpleSchema correctly indicates an error.
Any pointers?
Sample code below.
    Test = function(a, b) {
      this.a = a;
      this.b = b;
      return;
    }

    TestCollection = new Mongo.Collection('Test');

    TestSchema = new SimpleSchema({
      a: {
        type: Number,
        min: 10
      },
      b: {
        type: Number,
        max: 10
      }
    });

    TestCollection.attachSchema(TestSchema);

    var z = new Test(1,2);

    testContext = TestSchema.namedContext('testing');
    testContext.validate(z);  // returns true
    testContext.isValid();  // returns true

    TestCollection.insert(z);
    // insert failed: Error: A must be at least 10

    var z_obj = JSON.parse(JSON.stringify(z));
    testContext.validate(z_obj);  // returns false
    testContext.isValid();  // returns false
    testContext.invalidKeys();
    // returns [{ name: "a", type: "minNumber", value: 1 }]

It intentionally only validates until it hits an object with some non-Object prototype. It assumes that if it's a Test object, you will handle validation internally within that object. I guess there could be some way of registering certain types to be fully validated if there is a use case for it.

+1
I would like to be able to validate a class instance using the schema. Most of the time the schema contains the information to check for my wrappers integrity and re-write that code (or deep clone the object) feels a bit like a stretch.
Plus, having the schema accept objects that the collection rejects sounds kind of inconsistent.

Thanks for your patience. SimpleSchema 2.0.0-rc.1 is now released and includes changes that resolve this issue.
There are a number of breaking changes when updating to 2.0, so be sure to check out the change log. If you use aldeed:collection2, you will need to use 2.10.0 or higher of that package in order to use SimpleSchema 2.0. If you use autoform, it is not yet updated to work with SimpleSchema 2.0, but hopefully soon.
SimpleSchema is now an isomorphic NPM package, so you can check out the updated readme and file issues over at the other repo. The Meteor wrapper package will exist for now but eventually I will probably deprecate support for it.
This is still a beta/RC and I do expect people will find issues, so use with caution. Production use is not yet recommended. That said, there are more and better unit tests than ever before, and the codebase should be much easier for others to read through and debug quickly.


select-layout even : C-b Esc-1

currently the Escape behave like the VI, buy how do i undo this ?
bind -n M-1 select-layout even-vertical
bind -n M-2 select-layout even-horizontal
these I believe is the standard, but the escape key no longer works
I assume i would need to make some unbind/bind in the .tmux.conf.local, but after hours of google and testing im stuck
can someone explain/help please?
-ls

I endup setting this in the local file
balance panes
bind -n F9 select-layout even-vertical
bind -n F10 select-layout even-horizontal
bind -n F11 select-layout main-vertical
bind -n F12 select-layout main-horizontal
still would be nice to know how to get the escape key back


0.74 - Continuous Arlo Errors

Home Assistant release with the issue:
0.74
Last working Home Assistant release (if known):
0.73.x
Operating environment (Hass.io/Docker/Windows/etc.):
Docker
Component/platform:
Arlo
Description of problem:
Continuous errors since update to 0.74
Traceback (if applicable):
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/concurrent/futures/thread.py", line 56, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/src/app/homeassistant/components/arlo.py", line 84, in hub_refresh
    update_base_station=True)
  File "/usr/local/lib/python3.6/site-packages/pyarlo/__init__.py", line 270, in update
    base.update()
  File "/usr/local/lib/python3.6/site-packages/pyarlo/base_station.py", line 687, in update
    self.get_ambient_sensor_data()
  File "/usr/local/lib/python3.6/site-packages/pyarlo/base_station.py", line 481, in get_ambient_sensor_data
    properties = history_event.get('properties')
AttributeError: 'NoneType' object has no attribute 'get'

Additional information:

Reported the ambient sensor issue earlier #15486 in the week during the beta release and was told it was resolved.

This was fixed in #15503 but was not included in the 0.74 release (see #15486 for more details).
Try updating homeassistant/components/arlo.py to reference pyarlo==0.2.0 - this should fix the problem in the interim. Hopefully the updated reference gets included in 0.75.0b0.
Again apologies for the inconvenience - tried to turn around the fix quickly enough to make 0.74 release, but unfortunately it did not.

Thanks, sorted, no problem - thanks for your effort anyway

On 22 Jul 2018, at 7:45 am, Luke Fritz<EMAIL_ADDRESS>wrote:
This was fixed in #15503 https://github.com/home-assistant/home-assistant/pull/15503 but was not included in the 0.74 release (see #15486 https://github.com/home-assistant/home-assistant/issues/15486 for more details).
Try updating homeassistant/components/arlo.py to reference pyarlo==0.2.0 - this should fix the problem in the interim. Hopefully the updated reference gets included in 0.75.0b0.
Again apologies for the inconvenience - tried to turn around the fix quickly enough to make 0.74 release, but unfortunately it did not.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/home-assistant/home-assistant/issues/15585#issuecomment-406830411, or mute the thread https://github.com/notifications/unsubscribe-auth/AdLnQlzVR7nH1smDqT7LCm27Mn73n37Pks5uI70ngaJpZM4VZYZe.

Just for FYI.  I run HA in a Docker container and for anyone else that might want to try the workaround until it's merged, you can just update the pyarlo version and you don't have to reference anything in the arlo.py file.    But either way, you can just uninstall 0.1.9 and install 0.2.0 and it fixes the issue.

@lukiffer I updated the arlo.py to reference that version as well as the requirements_all.txt.  I uninstalled 0.1.9 and installed 0.2.0.  Still getting these errors.

@edif30 You just need to update the arlo.py file with version 0.2.0. It fixed the errors for me.

@lukiffer I have updated that reference and seem to be still getting errors.  Not sure if this makes a difference but I rebuilt my docker container with 0.2.0 and updated the arlo.py file to reflect 0.2.0.  Still getting those errors.  By just changing the reference to 0.2.0 and not installing 0.2.0, the component wouldn't load.  So I'm not really sure what is going on here.

Is it fixed in 0.74.2?

Updated to 0.74.2, still the issue continues.  Here are some debug logs I could grad during startup
2018-07-26 12:29:54 INFO (MainThread) [homeassistant.setup] Setting up arlo
2018-07-26 12:29:54 DEBUG (SyncWorker_3) [pyarlo] Creating Arlo session
2018-07-26 12:29:54 DEBUG (SyncWorker_3) [pyarlo] Params: {'email': '<redacted>', 'password': '<redacted>'}
2018-07-26 12:29:54 DEBUG (SyncWorker_3) [pyarlo] Headers: {'Content-Type': 'application/json', 'Authorization': None}
2018-07-26 12:29:54 DEBUG (SyncWorker_3) [pyarlo] Querying https://arlo.netgear.com/hmsweb/login/v2 on attempt: 0/3
2018-07-26 12:29:56 DEBUG (SyncWorker_3) [pyarlo] Params: {'email': '<redacted>', 'password': '<redacted>'}
2018-07-26 12:29:56 DEBUG (SyncWorker_3) [pyarlo] Headers: {'Content-Type': 'application/json', 'Authorization': '<redacted>'}
2018-07-26 12:29:56 DEBUG (SyncWorker_3) [pyarlo] Querying https://arlo.netgear.com/hmsweb/users/devices on attempt: 0/3

2018-07-26 12:29:56 INFO (MainThread) [homeassistant.core] Bus:Handling <Event call_service[L]: domain=persistent_notification, serv
ice=create, service_data=title=Invalid config, message=The following components and platforms could not be set up:

 - [arlo](https://home-assistant.io/components/arlo/)

Please check your config., notification_id=invalid_config, service_call_id=140347929274688-2>
2018-07-26 12:29:56 INFO (MainThread) [homeassistant.core] Bus:Handling <Event state_changed[L]: entity_id=persistent_notification.i
nvalid_config, old_state=None, new_state=<state persistent_notification.invalid_config=notifying; title=Invalid config, message=The
following components and platforms could not be set up:

 - [arlo](https://home-assistant.io/components/arlo/)

Please check your config. @ 2018-07-26T12:29:56.802221-05:00>>

@lukiffer
2018-07-31 16:24:28 ERROR (MainThread) [homeassistant.core] Error doing job: Future exception was never retrieved
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/concurrent/futures/thread.py", line 56, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/src/app/homeassistant/components/arlo.py", line 84, in hub_refresh
    update_base_station=True)
  File "/config/deps/lib/python3.6/site-packages/pyarlo/__init__.py", line 270, in update
    base.update()
  File "/config/deps/lib/python3.6/site-packages/pyarlo/base_station.py", line 687, in update
    self.get_ambient_sensor_data()
  File "/config/deps/lib/python3.6/site-packages/pyarlo/base_station.py", line 481, in get_ambient_sensor_data
    properties = history_event.get('properties')
AttributeError: 'NoneType' object has no attribute 'get'

$ sudo docker exec -it home-assistant bash
cat homeassistant/components/arlo.py | grep REQ
REQUIREMENTS = ['pyarlo==0.2.0']
date
Tue Jul 31 16:25:16 EDT 2018
$ cat .HA_VERSION
0.75.0b0

@MartinHjelmare WOW, I didn't even notice that.  Also, I have no idea how or why the 1.9 was even in the deps.  When I run the HA container, I wipe everything except config.  Very odd.  I just rebuilt using the 0.75.0b0 and so far so good.

Where is the location of the arlo.py file for hass.io users?  I'm migrating - slowly - from an old AIO install on another pi and I can't seem to find the location of this file anywhere.  I know the issue was closed but I thought this would be the best place to ask.


New-AzureRmResource for deployments has duplicate ResourceName switches

e.g.
New-AzureRmResource -ResourceName $ResourceName -Location $ResourceLocation -PropertyObject $PropertiesObject -ResourceGroupName Default-Web-NorthEurope -ResourceType Microsoft.Web/sites/deployments -ResourceName MySite -ApiVersion 2015-08-01 -Force

The same applies to the "Powershell" tab on Azure RM Explorer

Actually, that's what this issue is referring to :)

This also happens for DataLake Store (#130)

fixed by 391f2170c9178892fe4916ec90e390fb01e76344


Fixing packaging issues as well as the Timesketch magics

Fixing the chips support for the TS magics as well as fixing issues with pip.
The issue was that the magic library was imported in picatrix/__init__ which is loaded in the setup.py file when importing version/dependencies, that causes issues, since that's before dependencies are evaluated (in order to evaluate dependencies), causing libraries to be missing. Solved by moving the logic away from __init__ to a separate file.

sorry, had to add one more change, can you review again?


Merge in access logging changes

Handled in https://github.com/coreos/alb-ingress-controller/pull/321


Fix a missing comment

This is a minor fix to Securefox to add a comment to a line which is missing one.
Thank you for maintaining this project!

Thank you


Defining JSON File Storage for Facebook Bot Causes Error

info: ** Using simple storage. Saving data to botlog.json
/var/www/html/botkit/node_modules/mkdirp/index.js:90
throw err0;
^
Error: ENOTDIR: not a directory, mkdir '/var/www/html/botkit/botlog.json/teams'
at Error (native)
at Object.fs.mkdirSync (fs.js:794:18)
at Function.sync (/var/www/html/botkit/node_modules/mkdirp/index.js:71:13)
at new Store (/var/www/html/botkit/node_modules/jfs/lib/Store.js:307:16)
at module.exports (/var/www/html/botkit/lib/storage/simple_storage.js:35:20)
at Botkit (/var/www/html/botkit/lib/CoreBot.js:978:26)
at Object.Facebookbot (/var/www/html/botkit/lib/Facebook.js:9:27)
at Object. (/var/www/html/botkit/facebook_bot.js:86:25)
at Module._compile (module.js:410:26)
at Object.Module._extensions..js (module.js:417:10)

Looks like you already had a existing file called botlog.json.
For me it was also unituitive at first, but the json_file_store value should be a directory name and not the name of a json file where it should be saved.

@djeraseit did the above solve your problem?

Closed as answered.


model-graded-factuality and model-graded-closedqa asserts using AzureOpenAI

Hi @typpo  thanks a lot for your response... (https://github.com/promptfoo/promptfoo/issues/369)
I have tried to  override the LLM grader but i am still getting error, Here you have my yaml
prompts: [prompt1.txt, prompt2.txt]
providers:
  - id: azureopenai:chat:gpt4
    config:
      apiHost: testinstance.openai.azure.com
      temperature: 0
      max_tokens: 50
      apiKey: XXXXXXXXXXX
evaluateOptions: { cache: false }
defaultTest:
  options:
    provider:
      id: azureopenai:chat:gpt4
      config:
        apiHost: testinstance.openai.azure.com
        apiKey: XXXXXXXXXXXXXXXXXX
  assert:
    - type: javascript
      value: Math.max(0, Math.min(1, 1 - (output.length - 100) / 900))
tests:
  - vars:
      query: What is the max purchase that doesn't require approval?
      context: file://docs/reimbursement.md
    assert:
      - type: contains
        value: '$500'
      - type: factuality
        value: the employee's manager is responsible for approvals
      - type: answer-relevance
        threshold: 0.8
      - type: context-recall
        threshold: 0.9
        value: Employees must submit their expense reports by the first business day of the month
      - type: context-relevance
        threshold: 0.9
      - type: context-faithfulness
        threshold: 0.9
  - vars:
      query: How many weeks is maternity leave?
      context: file://docs/maternity.md
    assert:
      - type: factuality
        value: maternity leave is 4 months
      - type: answer-relevance
        threshold: 0.8
      - type: similar
        value: eligible employees can take up to 4 months of leave

ERROR:

Error: Provider azureopenai:chat:gpt4 does not implement callEmbeddingApi for answer relevancy check, falling back to default
Error: Provider azureopenai:chat:gpt4 does not implement callEmbeddingApi for answer relevancy check, falling back to default
at getAndCheckProvider (C:\Users\XXXXXXXX\AppData\Roaming\npm\node_modules\promptfoo\dist\src\matchers.js:79:15)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
at async matchesAnswerRelevance (C:\Users\XXXXXXXX\AppData\Roaming\npm\node_modules\promptfoo\dist\src\matchers.js:297:29)
at async runAssertion (C:\Users\XXXXXXXX\AppData\Roaming\npm\node_modules\promptfoo\dist\src\assertions.js:665:17)
at async runAssertions (C:\Users\XXXXXXXX\AppData\Roaming\npm\node_modules\promptfoo\dist\src\assertions.js:73:24)
at async Evaluator.runEval (C:\Users\XXXXXXXX\AppData\Roaming\npm\node_modules\promptfoo\dist\src\evaluator.js:210:37)
at async C:\Users\francisco.j.lopez\AppData\Roaming\npm\node_modules\promptfoo\dist\src\evaluator.js:475:25
Originally posted by @FJLopezGarcia in https://github.com/promptfoo/promptfoo/issues/369#issuecomment-1881068087

removing the defaultTest: section to not override the LLM grader:
I am getting this message
You are using model-graded assertions of types factuality, answer-relevance, context-recall, context-relevance, context-faithfulness while testing an Azure provider. You may need to override these to use your Azure deployment. To learn more, see https://promptfoo.dev/docs/providers/azure/#model-graded-tests

But it seems tha works. am I doing something wrong?

Thanks for the report.  This error is occurring because the answer-relevance assertion requires both an embedding and text generation provider.  Each model is a separate deployment.
There is a bug in how promptfoo supports this right now.  After #386 lands, you should be able to run this eval with the following:
defaultTest:
  options:
    provider:
      text:
        id: azureopenai:chat:gpt-4-deployment
        config:
          apiHost: xxx.openai.azure.com
      embedding:
        id: azureopenai:embeddings:text-embedding-ada-002-deployment
        config:
          apiHost: xxx.openai.azure.com


add function to completely load scenario in memory

for comparison with #51

should be ready for review now


Add -DefaultValue to Get-Item and Get-Variable

Summary of the new feature / enhancement
PowerShell 7 introduces null-coallescing operator ?? and its compound assignement counterpart ??=. They can be used to conveniently provide default value for variables which may be null. It is however also possible for a variable to not be set at all, in which case ?? and ??= throw a StrictMode error if enabled. This is often the case when using environment variables.
The current way to write it is a quite verbose (Test-Path 'variable:x') ? $x : $y, which is similar pattern to $x -eq $null ? $x : $x which ?? replaces, and which is usable when StrictMode is not enabled. This may tempt the user to disable StrictMode to write more compact code.
Proposition is to add -DefaultValue to Get-Item and Get-Variable, which transforms the code above into much more pleasant and readable Get-Item 'variable:x' -Default $y or Get-Variable 'x' -Default $y, avoiding repetition of the variable name.
The null compound assignment ??= could be replicated by adding -SkipIfExists to Set-Variable, but this is beyond scope of this issue.
Proposed technical implementation details (optional)
No response

The idea is not quite clear in the context of this syntax. Are you suggesting something similar?:
(Get-item variable:x -ea 0) ? $x : ($x = 10)

The idea is not quite clear in the context of this syntax. Are you suggesting something similar?:
(Get-item variable:x -ea 0) ? $x : ($x = 10)

I don't follow, how is this different from the (Test-Path 'variable:x') ? $x : $y example I provided? Other than using -ErrorAction SilentlyContinue disguised under -ea 0 completely hides the intent, while my proposal is to clearly state intent of providing a default value.

while my proposal is to clearly state intent of providing a default value.

The idea is not quite clear in the context of this syntax.

I was referring specifically to this syntax:  a ? b : c
Could you give a conceptual example?

Maybe a more real life example would be clearer. In my script, I want to be able to override the default settings by setting an environment variable, as is common on Linux. Use of what is essentially a global may not be great, but in case of scripts it is usually the easiest way to go.
Currently I have written the following:
$RetryCount = (Test-Path 'env:RETRY_COUNT') ? $env:RETRY_COUNT : 3
$RetryTimeout = (Test-Path 'env:RETRY_TIMEOUT') ? $env:RETRY_TIMEOUT : 5000
./run_tests.py -- repeat $RetryCount --timeout $RetryTimeout

It's quite readable, as long as you know that Test-Path can be used for variables, but requires typing the environment variable name twice. With my proposal, this can nicely fit on a single line while being even more clear:
./run_tests.py -- repeat (Get-Item 'env:RETRY_COUNT' -Default 3) --timeout (Get-Item 'env:RETRY_TIMEOUT' -Default 5000)

Maybe a more real life example would be clearer

Yes, this explanation is quite clean. Perhaps it would be useful, but I am a supporter of the usual syntax with conditions and their checks even if it is longer.


Decklink not working under Flatpak

See this post: https://forum.lwks.com/threads/linux-flatpak-with-decklink-io.251655/

does anyone know, how to get the Decklink IO working with the Flatpak Version of Lightworks? I only get the Pulseaudio and Portaudio options, but no Decklink. In the DEB Version there is the Decklink option and it works fine.

OBS suffers the same issue, so it looks unlikely that it can be made to work without some changes from Blackmagic. I guess the Decklink drivers are supported on a limited set of platforms, so probably you need to use one of those anyway.

https://github.com/obsproject/obs-studio/issues/6004

I've created a BlackMagicLibs extension which bundles the libraries required to use the Decklink in the same way OBS added support years ago. The user with the Decklink tried to use the extension, but it did not appear to make any difference.
When I added the LD_PRELOAD=libDeckLinkAPI.so line to the Flatpak manifest, the errors went away, but no one has re-tested this with that environment variable yet.
It would be good for someone with a Decklink to try building it again and let me know if this fixes anything: https://github.com/kekkoudesu/lightworks-flatpak#decklinks-arent-supported
If we can actually get it working, these are the remaining problems:

This method of support means a Decklink-enabled Lightworks Flatpak can never be published on Flathub. Blackmagic would need to make their drivers publicly accessible before this can happen.
Building this is very cumbersome for a user with a Decklink. create-usb seems to include extensions and might be the easiest way of distributing it, but I can't get the command to work yet.

Current build and process works for a user with a decklink :)
https://forum.lwks.com/threads/linux-flatpak-with-decklink-io.251655/post-280914


VersionOne Collector

I can't get the VersionOne collector to work properly.
With the most recent commit:
I get something like this:

ERROR c.c.d.rest.RestApiExceptionHandler - null
java.lang.NullPointerException: null
at com.capitalone.dashboard.service.DashboardServiceImpl.get(DashboardServiceImpl.java:51).....
With release 2.0.0, the widget looks better, but is missing features in progress for most of the teams.

@amp5208 please take a look.

Can you provide a little more of the stack trace? Does seem to be config issue.

@j9s3 :  This appears to be a configuration issue.  Please provide your current configuration for the following property (read my notes and example below):
# In Jira, your instance will have its own custom field created for "super story" or "epic" back-end ID,
# which includes a list of information.  This field allows you to specify that data field for your instance
# of Jira.  Note:  You can retrieve your instance's epic ID field name via the following URI where your
# queried user story issue has a super issue (e.g., epic) tied to it; your custom field name describes the
# epic value you expect to see, and is the only field that does this for a given issue:
# https://[your-jira-domain-name]/rest/api/2/issue/[some-issue-name]
feature.jiraEpicIdFieldName=customfield_10400

@j9s3 Is this still an issue ?

@j9s3 Closed. Reopen if still an issue.

@satishc1 yes It is still an issue. please re-open the ticket.  I ran into the same issue. it was pulling the kanban results but no scrum results at all.


Dynamically resolve the applicant (CBV) invite form based on site_id

Ticket
Partially resolves FFS-1020. This PR only pertains to loading the various view partials. Persisting data is not within the scope of this PR. A subsequent PR will handle data persistence.
Changes

Infer the site_id from the params when loading the new invitation page, making it accessible to the responsible view handler.
Update the view to load applicable state invitation form based on the site_id

Context for reviewers

Testing instructions, background context, more in-depth details of the implementation, and anything else you'd like to call out or ask reviewers.

Testing

How did you make the database changes? Did you add all these as columns to cbv_flow_invitations table?

Please refer to the PR description. I wanted to separate the view and data persistence.

I wanted to separate the view and data persistence.

Ah my bad, sorry for missing that in your description. I don't think it will work to split it into two PRs, though, as Rails requires the database fields to be present in order to render the form fields. You could add all the database fields in the first PR, I guess, but it's probably just easiest to do it all in one PR.

@GeorgeCodes19 I just caught this now, the text under the H1 needs to be updated. See image


doc: fix example README relative links

Proposed changes

Fixes some relative links in examples/README.md.

Type of change:

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as
expected)
[ ] This change requires a documentation update

Required Checklist:

[x] I have signed the MongoDB CLA
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added any necessary documentation (if appropriate)
[ ] I have run make fmt and formatted my code
[ ] I have tested the CDK constructor in a CFN stack. See TESTING.md

Further comments

Thank you for your fix!


updates/dynamic-spm

This makes DiskCache a dynamic framework when building with SPM. Noticed that building as a static framework throws errors in the Associate app.

I've ran in to this before and I don't think concretely defining the package as dynamic is the not way to go as the issue is with the consuming app/package's interaction with SwiftPM. There are also issues around importing a dylib in to a test target.
As the docs state it is recommend to leave this argument as the default which allows SwiftPM to choose the correct linking method:
///     - type: The optional type of the library that is used to determine how to link to the library.
///         Leave this parameter unspecified to let to let the Swift Package Manager choose between static or dynamic linking (recommended).
///         If you do not support both linkage types, use `.static` or `.dynamic` for this parameter.

The issue is that Xcode is not interacting correctly with SwiftPM to allow it to choose correctly.
There are a couple of options:

Add a user defined setting called DISABLE_DIAMOND_PROBLEM_DIAGNOSTIC to Xcode build settings. This effectively instructs the compiler to ignore this error. The side effect is that the static lib will be included in the binary more the once. As far as I can tell the impact is a somewhat larger binary size.

Follow the work around instructions here. The side effect is more config time and potentially additional overhead in importing libs.

In the past I've chosen #1 as it will be easier to reverse when if/when Apple resolves this problem.


description panel

Hello
is there a way to create a "description" panel, for example for a help section or credit section
I was thinking on addMonitor with strings, but I think it's not the right way to solve it.

Hello. There is no way to create such a component. You can use a text monitor as you said, but this is not the intended use.
You can also create your own description component with the plugin system:
https://cocopon.github.io/tweakpane/plugins.html


Icon Component : Missing OnSelect Property

Icon Component does not have OnSelect

I can see the implementation here: onSelect at index.ts and also here: onClick prop on RenderButtonAs. But for some reason the prop is not present.
Used Version 1.0.19
Used on Canvas App

Hi @6gal6ler6 - You are right - the Icon component will raise the OnChange event when clicked since field components cannot raise the OnSelect event.
Simply put the code you would normally put in the OnSelect event, inside the OnChange event.
Hope this helps
Scott

HI @scottdurow , thanks for the advice. I have implemented the behaviour as you mentioned and works like a charm


feat: update radio/checkbox styles to allow full label width

TASK: #1039
Types of changes

[ ] 🛠 Tool
[ ] 🪲 Bug fix (non-breaking change which fixes an issue)
[ ] ✨ New feature (non-breaking change which adds functionality)
[ ] 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] 🧾 Documentation
[ ] 📷 Demo
[ ] 🧪 Test
[ ] 🧠 Refactor
[x] 💄 Styles

@Powerplex , we initially thought that we should only apply the flex-grow: 1 style when the orientation is set to vertical.
However, it turns out that it doesn't have any negative impact on the components, even when the orientation is horizontal. Therefore, there's no need to add a condition to our code


Jesse: Watch perfectqaservices.com

Jesse requests the watch of the watch_keyword perfectqaservices\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

perfectqaservices\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


Update (or remove) the deploy/kube/README.md text

I believe this info is out-of-date or inconsistent with other deployment docs at this point. For instance, there is repeated mention of configmaps.

I'll send a PR to delete it, I think it's been superseded by tasks on istio.io.


Using the cloudinary SDK on expo v50.0.07 causes use font hook to fail randomly

Bug report for Cloudinary React Native SDK
Describe the bug in a sentence or two.
using the most recent version of the cloudinary react native sdk, custom fonts and icons don’t work on expo, for some reason, it seems using the cloudinary package, causes the useFont and even the expo-vector/icons packages to fail and not load icons or custom fonts at all.

Hi, thanks for getting in touch.
Could you please provide us with a sample repo which demonstrates the issue?


Missing connections for Little Snitch

The About tab in the preference pane lists several addresses that are supposed to be allowed in Little Snitch, but there were two requests made that aren't listed.
curl tried to establish a connection to download.dnscrypt.org on TCP port 443
DNSCrypt Menubar via curl tried to establish a connection to download.dnscrypt.org on TCP port 80

Wrong place to report errors maybe, go here: https://www.obdev.at/support/index.html?product=LS&topic=contact

No, THIS is the correct place to report errors in DNSCrypt.
There's nothing wrong with Little Snitch, it did its job correctly.
I was simply reporting that DNSCrypt is missing two entries in the About tab.
It says

Using Little Snitch? There are a few rules you need to establish, which Little Snitch should prompt for when trying to activate this application:
Allow any outgoing connection for system processes (dnscrypt-proxy)
Allow any incoming connection for system process (dnscrypt-proxy)
Allow any outgoing connection for system processes (dig)
Allow outgoing TCP connections to port 80 (http) of www.apple.com for system process (curl)
For automatic updates, allow outgoing TCP connections to port 80 (http) of updates.dnscrypt.org and to port 443 (https) of github.com for system process (curl)

But evidently some addresses were either omitted or have changed, and need to be added to the list.

This project is dnscrypt-proxy, a headless client implementation of the protocol.
There is no "about tab" nor "preference pane" in this project.
The project you are referring to is probably https://github.com/alterstep/dnscrypt-osxclient which already lists download.dnscrypt.org in the set of things to whitelist in Little Snitch. So, it will show up when a new release of this software is available.

Oh, sorry about that.
Right site, wrong project. I wanted the dnscrypt-osxclient page.
https://github.com/alterstep/dnscrypt-osxclient
Feel free to delete this thread so people don't end up here instead of the other page I'm posting on.


Search does not work with empty search text

It just goes to a blank page.

https://github.com/yzernik/squeaknode/pull/1349


Add installation instructions

Hey Kevin,
I wrote detailed installation instructions, separating the "stable version" (published in conda/pypi) from the "development version" from github.
The instructions include how to enable the GPU on tensorflow.
If you think that the "development version" instructions should be out of the README (e.g. in a CONTRIBUTING.md file), please tell me and I will move them there.

Great, thanks!
I'm actually fine with having them in the README, so people actually find them.
We can also add them to the documentation of course.


trouble with use_frameworks! in cocoapods and swift

So the project was working fine until I upgraded to cocoapods 0.36 and set the use_framworks! flag in the podfile so that I could add the AlamoFire swift framework.

Here is my podfile:
platform :ios, '8.0'
use_frameworks!
target 'MyApp' do
pod 'Firebase', '>= 2.1.2'
pod 'Alamofire', '~> 1.1'
pod 'JSQMessagesViewController'
end
target 'MyAppTests' do
end

Thanks @mypark - There are currently some known issues with Swift.
https://github.com/jessesquires/JSQMessagesViewController/labels/swift support
I'll be working to resolve these as soon as I can.

should be fixed in v7.0

hey @mypark - let me know if v7.0 fixes this for you

just tried it and it looks like it works now.

awesome! :+1:


Add NCE on Kim CNN

I am working on this.

Kim CNN is not a siamese network. NCE design based on non-siamese network remains a problem.


Enable auto-merge capability for our repo

Since our CI pipeline is not "blazing fast", this feature can be good, when you know that everything is ready and PR can be merged:
https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/incorporating-changes-from-a-pull-request/automatically-merging-a-pull-request

Done!