Tests of non-utf8 filenames break infrastructure if they don't clean up their temp files

The tests
standalone_2/io/non_utf8_directory_test
standalone_2/io/non_utf8_file_test
standalone_2/io/non_utf8_link_test
standalone_2/io/non_utf8_output_test (possibly)
will leave behind a temporary file system object with a non-utf8 name (starting with 0xb6, decimal 182)
if they time out or crash.
The swarming infrastructure used for builders then crashes when trying to delete these files, as part of cleaning up after a build.
This is happening on the hot_reload and hot_reload_rollback builders right now.
Skipping these tests on the reload and reload rollback builders for now, filing an issue with Chrome infrastructure to make the swarming bots stop crashing on this case.
These are tests related to issue "Cannot walk a list of files and pass them to a subprocess (if any are non-UTF8) #33368"

These tests should also be improved to make a temp directory with the name of the test in it, like
systemTemp.createTemp("non_utf8_directory_test"), and do all further work inside this directory.  Deleting this directory should be in a "finally" block, so that we are almost certain to delete it.

The issue is not solved, the builders are purple again since efd36d4cdf5f72668837fb07bc6e0a24c2d61872 landed: https://ci.chromium.org/p/dart/builders/luci.dart.ci.sandbox/vm-kernel-reload-linux-debug-x64

@whesse do you happen to know if the Chrome infrastructure got around to fixing this case? Looking at the tests again, I'm not sure why those files weren't being deleted in the first place...

No the problem is still there. I don't think they intend to fix it - I think they closed the issue as working as intended.  I ran the tests again, modified to leave the file behind, and got a purple build, with the following error:
INFO: Core dump archiving is activated
INFO: No unexpected crashes recorded
2169 2019-04-30 14:44:15.430 E: Leaking out_dir /b/s/w/iojk689o: 'utf8' codec can't decode byte 0xb6 in position 1: invalid start byte
Traceback (most recent call last):
File "/b/s/swarming_bot.1.zip/client/run_isolated.py", line 786, in map_and_run
success = file_path.rmtree(run_dir)
File "/b/s/swarming_bot.1.zip/utils/file_path.py", line 1178, in rmtree
make_tree_deleteable(root)
File "/b/s/swarming_bot.1.zip/utils/file_path.py", line 1152, in make_tree_deleteable
p = os.path.join(dirpath, dirname)
File "/usr/lib/python2.7/posixpath.py", line 73, in join
path += '/' + b
UnicodeDecodeError: 'utf8' codec can't decode byte 0xb6 in position 1: invalid start byte
Error is at https://ci.chromium.org/p/dart/builders/try/vm-kernel-reload-linux-release-x64-try/309

Thanks for checking Bill. Do you think it's worth enabling these tests
as-is given the risk of them taking down the infrastructure? It looks like
there's an issue deep in the posixpath join implementation expecting UTF8
strings, so I doubt there's anything we can do directly. If it's not worth
enabling them, we should probably just close this issue.
On Tue, Apr 30, 2019 at 8:56 AM William Hesse<EMAIL_ADDRESS>wrote:

No the problem is still there. I don't think they intend to fix it - I
think they closed the issue as working as intended. I ran the tests again,
modified to leave the file behind, and got a purple build, with the
following error:
INFO: Core dump archiving is activated
INFO: No unexpected crashes recorded
2169 2019-04-30 14:44:15.430 E: Leaking out_dir /b/s/w/iojk689o: 'utf8'
codec can't decode byte 0xb6 in position 1: invalid start byte
Traceback (most recent call last):
File "/b/s/swarming_bot.1.zip/client/run_isolated.py", line 786, in
map_and_run
success = file_path.rmtree(run_dir)
File "/b/s/swarming_bot.1.zip/utils/file_path.py", line 1178, in rmtree
make_tree_deleteable(root)
File "/b/s/swarming_bot.1.zip/utils/file_path.py", line 1152, in
make_tree_deleteable
p = os.path.join(dirpath, dirname)
File "/usr/lib/python2.7/posixpath.py", line 73, in join
path += '/' + b
UnicodeDecodeError: 'utf8' codec can't decode byte 0xb6 in position 1:
invalid start byte
Error is at
https://ci.chromium.org/p/dart/builders/try/vm-kernel-reload-linux-release-x64-try/309
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/dart-lang/sdk/issues/33519#issuecomment-488009274,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFYWZYCQO3XHF2S274WMQODPTBTZXANCNFSM4FF4RLDQ
.

Closing for now as there's no clear path to fix this and we likely won't ever be able to.


[BUG] OpenSearch 2.15.0 Ec2DiscoveryPlugin failed to get instance metadata

Describe the bug
Launched OpenSearch 2.15.0 cluster on Amazon Linux 2023 OS and the cluster did not start.
Below are the errors:
[2024-07-16T12:54:24,347][ERROR][o.o.d.e.Ec2DiscoveryPlugin] [hostname] failed to get metadata for [placement/availability-zone]
java.io.IOException: Server returned HTTP response code: 401 for URL: http://<IP_ADDRESS>/latest/meta-data/placement/availability-zone
at java.base/sun.net.www.protocol.http.HttpURLConnection.getInputStream0(HttpURLConnection.java:1998) ~[?:?]
at java.base/sun.net.www.protocol.http.HttpURLConnection.getInputStream(HttpURLConnection.java:1599) ~[?:?]
at java.base/java.security.AccessController.doPrivileged(AccessController.java:571) ~[?:?]
at org.opensearch.discovery.ec2.SocketAccess.doPrivilegedIOException(SocketAccess.java:70) ~[discovery-ec2-2.15.0.jar:2.15.0]
at org.opensearch.discovery.ec2.Ec2DiscoveryPlugin.getAvailabilityZoneNodeAttributes(Ec2DiscoveryPlugin.java:168) [discovery-ec2-2.15.0.jar:2.15.0]
at org.opensearch.discovery.ec2.Ec2DiscoveryPlugin.additionalSettings(Ec2DiscoveryPlugin.java:135) [discovery-ec2-2.15.0.jar:2.15.0]
at org.opensearch.plugins.PluginsService.updatedSettings(PluginsService.java:244) [opensearch-2.15.0.jar:2.15.0]
at org.opensearch.node.Node.(Node.java:502) [opensearch-2.15.0.jar:2.15.0]
at org.opensearch.node.Node.(Node.java:425) [opensearch-2.15.0.jar:2.15.0]
at org.opensearch.bootstrap.Bootstrap$5.(Bootstrap.java:242) [opensearch-2.15.0.jar:2.15.0]
at org.opensearch.bootstrap.Bootstrap.setup(Bootstrap.java:242) [opensearch-2.15.0.jar:2.15.0]
at org.opensearch.bootstrap.Bootstrap.init(Bootstrap.java:404) [opensearch-2.15.0.jar:2.15.0]
at org.opensearch.bootstrap.OpenSearch.init(OpenSearch.java:181) [opensearch-2.15.0.jar:2.15.0]
at org.opensearch.bootstrap.OpenSearch.execute(OpenSearch.java:172) [opensearch-2.15.0.jar:2.15.0]
at org.opensearch.cli.EnvironmentAwareCommand.execute(EnvironmentAwareCommand.java:104) [opensearch-2.15.0.jar:2.15.0]
at org.opensearch.cli.Command.mainWithoutErrorHandling(Command.java:138) [opensearch-cli-2.15.0.jar:2.15.0]
at org.opensearch.cli.Command.main(Command.java:101) [opensearch-cli-2.15.0.jar:2.15.0]
at org.opensearch.bootstrap.OpenSearch.main(OpenSearch.java:138) [opensearch-2.15.0.jar:2.15.0]
at org.opensearch.bootstrap.OpenSearch.main(OpenSearch.java:104) [opensearch-2.15.0.jar:2.15.0]
[2024-07-16T16:39:12,793][WARN ][o.o.b.Natives            ] [hostname] unable to load JNA native support library, native methods
will be disabled.
java.lang.UnsatisfiedLinkError: /tmp/opensearch-12972404803927399925/jna15942360218562026931.tmp: /tmp/opensearch-12972404803927399925/jna15942360218562026931.tmp:
failed to map segment from shared object
at java.base/jdk.internal.loader.NativeLibraries.load(Native Method) ~[?:?]
at java.base/jdk.internal.loader.NativeLibraries$NativeLibraryImpl.open(NativeLibraries.java:331) ~[?:?]
at java.base/jdk.internal.loader.NativeLibraries.loadLibrary(NativeLibraries.java:197) ~[?:?]
at java.base/jdk.internal.loader.NativeLibraries.loadLibrary(NativeLibraries.java:139) ~[?:?]
at java.base/java.lang.ClassLoader.loadLibrary(ClassLoader.java:2418) ~[?:?]
at java.base/java.lang.Runtime.load0(Runtime.java:852) ~[?:?]
at java.base/java.lang.System.load(System.java:2025) ~[?:?]
at com.sun.jna.Native.loadNativeDispatchLibraryFromClasspath(Native.java:1045) ~[jna-5.13.0.jar:5.13.0 (b0)]
at com.sun.jna.Native.loadNativeDispatchLibrary(Native.java:1015) ~[jna-5.13.0.jar:5.13.0 (b0)]
at com.sun.jna.Native.(Native.java:221) ~[jna-5.13.0.jar:5.13.0 (b0)]
at java.base/java.lang.Class.forName0(Native Method) ~[?:?]
at java.base/java.lang.Class.forName(Class.java:421) ~[?:?]
at java.base/java.lang.Class.forName(Class.java:412) ~[?:?]
at org.opensearch.bootstrap.Natives.(Natives.java:60) [opensearch-2.15.0.jar:2.15.0]
at org.opensearch.bootstrap.Bootstrap.initializeNatives(Bootstrap.java:123) [opensearch-2.15.0.jar:2.15.0]
at org.opensearch.bootstrap.Bootstrap.setup(Bootstrap.java:191) [opensearch-2.15.0.jar:2.15.0]
at org.opensearch.bootstrap.Bootstrap.init(Bootstrap.java:404) [opensearch-2.15.0.jar:2.15.0]
at org.opensearch.bootstrap.OpenSearch.init(OpenSearch.java:181) [opensearch-2.15.0.jar:2.15.0]
at org.opensearch.bootstrap.OpenSearch.execute(OpenSearch.java:172) [opensearch-2.15.0.jar:2.15.0]
at org.opensearch.cli.EnvironmentAwareCommand.execute(EnvironmentAwareCommand.java:104) [opensearch-2.15.0.jar:2.15.0]
at org.opensearch.cli.Command.mainWithoutErrorHandling(Command.java:138) [opensearch-cli-2.15.0.jar:2.15.0]
at org.opensearch.cli.Command.main(Command.java:101) [opensearch-cli-2.15.0.jar:2.15.0]
at org.opensearch.bootstrap.OpenSearch.main(OpenSearch.java:138) [opensearch-2.15.0.jar:2.15.0]
at org.opensearch.bootstrap.OpenSearch.main(OpenSearch.java:104) [opensearch-2.15.0.jar:2.15.0]
On Amazon Linux 2023, there are changes to retrieve instance metadata. Need to first retrieve a token before invoking the metadata URL.
TOKEN=$(curl --request PUT "http://<IP_ADDRESS>/latest/api/token" --header "X-aws-ec2-metadata-token-ttl-seconds: 3600")
Related component
Plugins
To Reproduce
Launch a cluster using OpenSearch 2.15.0 on Amazon Linux 2023 OS. EC2DiscoveryPlugin will error out.
Expected behavior
Add a fix to include token retrieval for amazon linux 2023 and pass that to subsequent calls to retrieve instance metadata.
Additional Details
Plugins
Please list all plugins currently enabled.
Screenshots
If applicable, add screenshots to help explain your problem.
Host/Environment (please complete the following information):

OS: [e.g. iOS]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

[Triage - attendees 1 2 3]
@nikhanj30 Thanks for creating this issue.  Without seeing the full opensearch.log (here is an example issue that includes it #14791) it will be hard to look into this issue.  We also need to know how OpenSearch was deployed and the EC2 instance configuration.
It looks like the S3 error isn't critical path, and the JNA error could be because an x64 build is attempted to be run on an ARM architecture machine (*G instance types).

Is this still IMDSv1 or forcing you to use v2 (which is not supported yet, https://github.com/opensearch-project/OpenSearch/issues/3040)?

May be related to https://forum.opensearch.org/t/opensearch-jnanatives-library-issue/20375

Oh I see the JNA error that precedes it.

Modified to use IMDSv1 and also the jvm.options -Djna.tmpdir. After that I don't see this error.

Modified to use IMDSv1 and also the jvm.options -Djna.tmpdir. After that I don't see this error.

I presume jvm.options -Djna.tmpdir alone didn't solve the problem, right?

As mentioned in my previous comment, I just changed those two and it worked fine.

As mentioned in my previous comment, I just changed those two and it worked fine.

I am trying to find out whether both were required for it to work fine. Can you try changing the -Djna.tmpdir back, does it still work?


Uncaught Exception Line 1085

What bug did you encounter?
I'm sorry, but an uncaught exception occurred.
While running game code:
File "game/script.rpy", line 1085, in script call
call day_activity_choices from _call_day_activity_choices_5
File "game/scripts/labels/day_activity_choices.rpy", line 36, in script call
call study_session from _call_study_session_1
File "game/scripts/labels/quiz_session.rpy", line 41, in script
$ renpy.say(None, quiz_question.question, interact=False)
File "game/scripts/labels/quiz_session.rpy", line 41, in
$ renpy.say(None, quiz_question.question, interact=False)
TypeError: %o format: a number is required, not TagQuotingDict
-- Full Traceback ------------------------------------------------------------
Full traceback:
File "game/script.rpy", line 1085, in script call
call day_activity_choices from _call_day_activity_choices_5
File "game/scripts/labels/day_activity_choices.rpy", line 36, in script call
call study_session from _call_study_session_1
File "game/scripts/labels/quiz_session.rpy", line 41, in script
$ renpy.say(None, quiz_question.question, interact=False)
File "C:\Users\Lozzy\Saved Games\LearnToCodeRPG-1.0-pc/renpy/ast.py", line 923, in execute
renpy.python.py_exec_bytecode(self.code.bytecode, self.hide, store=self.store)
File "C:\Users\Lozzy\Saved Games\LearnToCodeRPG-1.0-pc/renpy/python.py", line 2235, in py_exec_bytecode
exec(bytecode, globals, locals)
File "game/scripts/labels/quiz_session.rpy", line 41, in
$ renpy.say(None, quiz_question.question, interact=False)
File "C:\Users\Lozzy\Saved Games\LearnToCodeRPG-1.0-pc/renpy/exports.py", line 1406, in say
what = what % tag_quoting_dict
TypeError: %o format: a number is required, not TagQuotingDict
Windows-10-10.0.19041
Ren'Py <IP_ADDRESS>5
Learn to Code RPG 1.0
Tue Jan 18 12:47:39 2022
Steps to Reproduce
I had just finished a day off - choice games. The day after I chose CS fundamentals - CSS. I answered the first question correctly and on the second question the uncaught error occurred. Not really sure how to be sure if this would reproduce the problem, but it's as much as I can remember.
Please tell me if there are any techniques I can use to provide you more information for future reports.
Screenshots

Additional Information
Hi there,
I'd love to be more helpful with how I report issues. If you can give me some tips. I don't know what information is relevant in terms of reproducing the problem... Do you want me to give extra game play information? Obviously, I don't want to waste your time if the information isn't relevant. I've had a couple of uncaught errors since re-downloading. When I rolled back after the first uncaught error. It seemed changing my path prevented the same error, but here I am a bit later encountering a new error (or possibly the same one as before)
Thanks for the hard work!

your screenshots contained more than enough info to diagnose the problem
I'm 95% sure this exception is caused by one of these two questions
cwee@cwee-HP-ZBook-Studio-G7 22:36:53 ~/repos/LearnToCodeRPG fix/renpy_say_format
$ grep -rn "question.*%"
game/scripts/quiz_questions_from_csv.rpy:781:    question=_("Which one of these values represents 10% of the viewport width?"),
game/scripts/quiz_questions_from_csv.rpy:797:    question=_("Which one of these values represents 20% of the viewport height?"),

hopefully that sounds familiar to you
I've added more technical details in the PR comments

Thanks for the reply and I'm happy that the screen shots were helpful!
On Thu, 31 Mar 2022, 06:49 googlebleh, @.***> wrote:

your screenshots contained more than enough info to diagnose the problem
I'm 95% sure this exception is caused by one of these two questions
@.*** 22:36:53 ~/repos/LearnToCodeRPG fix/renpy_say_format
$ grep -rn "question.*%"
game/scripts/quiz_questions_from_csv.rpy:781:    question=("Which one of these values represents 10% of the viewport width?"),
game/scripts/quiz_questions_from_csv.rpy:797:    question=("Which one of these values represents 20% of the viewport height?"),
hopefully that sounds familiar to you
I've added more technical details in the PR comments
—
Reply to this email directly, view it on GitHub
https://github.com/freeCodeCamp/LearnToCodeRPG/issues/60#issuecomment-1084102221,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVX54QP7Y6XEBRNKWSWWKZTVCU4GHANCNFSM5MHDP7NA
.
You are receiving this because you authored the thread.Message ID:
@.***>


/translate: Add model_type and model_type_used parameters

Also bumps the minor version as this is a new feature.
Also I removed mention of the 50 max items for text translation -- there is no limit.

LGTM. What is the openapi_gitbook.yaml file?

LGTM. What is the openapi_gitbook.yaml file?

It's linked directly from our GitBook documentation, and has simplified descriptions.


Incorrect byte units in comments

Context
https://github.com/celestiaorg/celestia-app/blob/4276080f48dc269c6ec870829801532890191714/app/test/integration_test.go#L95-L115
Problem
These comments incorrectly use KiB. 600 KiB is 614400 bytes. 600 KB is 600000 bytes.
Proposal
Replace usages of KiB with KB.

Assign me @rootulp

I want to work on that issue @rootulp

check #2230 @rootulp


Fixes #45 - encoding not working correctly on maps

This PR fixes issue #45 by ensuring that keys go through same encoding steps as values.

Thank you! This looks good.


Enable query timeout of 20 seconds

Pull Request Test Coverage Report for Build 420f41e4-f466-4036-81bf-3ff933142c6e

9 of 10 (90.0%)  changed or added relevant lines in 2 files are covered.
3 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.02%) to 74.963%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

internal/internal_utils.go
8
9
88.89%

Files with Coverage Reduction
New Missed Lines
%

internal/internal_utils.go
1
88.24%

internal/internal_task_pollers.go
2
75.34%

Totals

Change from base Build dcc3cfe5-fdae-4cb9-b906-fff122c8f2d5:
-0.02%

Covered Lines:
9587

Relevant Lines:
12789

💛  - Coveralls


monaco editor version

Starting from what version of monaco editor does it support?

From v0.38.0!

From v0.38.0!

Thank you very much for your reply. I am currently using version 0.34.1. May I ask what feature requires an upgrade to v0.38.0?

Yes, the Monacopilot core is InlineCompletion, a feature introduced in v0.37.0, which became stable in v0.38.0. Just curious—why not upgrade your Monaco editor to the latest version? 😆

Yes, to work Monacopilot in Monaco editor, we need the InlineCompletion feature in the Monaco editor, which was introduced in v0.37.0 and became stable in v0.38.0.

Just curious—why not upgrade your Monaco editor to the latest version? 😆


Quick start in README.md is out of date

Describe the problem
It appears the quick start guide in the current README.md is out of date.
Template

{{project_name}

{{module_name}}.py.jinja

To Reproduce

Follow the quick start section of README.md
Try to use the resulting template structure: copier my_tmpl my_project
See that no interpolation happens

Logs
N/A
Expected behavior
Expected the result shown in the quick start.
Screenshots/screencasts/logs
N/A
Environment

OS: macOS Monterey 12.1 (aarch64)
Copier version: 5.1.0
Python version: 3.10.2
Installation method: pip install copier

Additional context
It's possible this is simply a misunderstanding of how this tool works, however, running through the quick start section of the README.md at HEAD was producing something that didn't appear to work on my system. Specifically, the quick start has you use {{project_name}} and {{module_name}}.py.jinja, both of which did not get transformed by Jinja when I invoked copier.
It was necessary for me to change it to [[project_name]] and [[module_name]].py.tmpl before everything worked correctly.
What made this more confusing was trying to understand the difference between the latest docs and the stable docs. It wasn't until I happened to run across the stable docs that I saw the reference to use [[ ]] over {{ }} and .tmpl over .jinja. If you click the docs badge at the top of the README, it takes you to the latest docs, which appears to suggest the same structure as the quick start (i.e. using {{ }}).

Yup, latest docs work for latest alpha version only. Maybe the badge should point to the stable docs instead.

I think it's just a bit confusing for a new user who follows the README and runs pip install copier and then is unable to get the quick start to work because it's targeting an alpha version (these were my exact steps). There's also no indication why it doesn't work other than interpolation appears to just be broken (i.e. no errors hinting about migrating from [[ ]] to {{ }}.
I know sometimes these things can get overlooked by regular users of the package, so I thought it prudent to point it out :)

A note or warning about the state of current versions would certainly help new users :slightly_smiling_face:

Another problem is that in old docs versions, the header icon always points to the latest version of docs. It's a bug that I hope is fixed because on PR builds it doesn't happen anymore. But old docs still have it. At least the color change helps on that.
I changed the link in the "about" section of the repo, pointing it to https://readthedocs.org/projects/copier/ now.
I think it would be nice to have a version indicator or even a link to the version selector from the docs.


Litellm integration request

I think it's a good idea to integrate with the Litellm as it provides an easy to integrate with 100+ LLMs.
https://github.com/BerriAI/litellm
LLM as a drop in replacement for GPT. Use Azure, OpenAI, Cohere, Anthropic, Ollama, VLLM, Sagemaker, HuggingFace, Replicate (100+ LLMs)

Hey - yeah! We're planning to integrate with them very soon :)

Update on this - I'll be writing a guide on how to use Litellm with us instead of a full-blown integration.
Should just be a few lines of code!

https://litellm-git-fork-colabdog-feature-add-deepeval-clerkieai.vercel.app/docs/tutorials/deepeval


Porter should detect what invocationImage OS is required

Hi all. The use case is that someone has installed the porter exe on windows found in releases. It works! So they set about trying to install a windows bundle with something like, porter install --tag squillace.azurecr.io/porter-windows-bundle:latestand... this needs to work without intervention.
Porter will pull the image, yes. Which means that it can examine the required OCI metadata to determine which OS this should work on, and it should handle that switch.
Bonus points: if Porter is "ON" windows and detects a linux image and can detect a WSL, can it merely auto-configure the "shell out" process? Or is that a bad design issue?
#discuss

Happy to make this work but first prepare yourself for more questions than answers! 😀
Are we talking about multi-arch images? Like porter should figure out that you are on windows and execute the windows arch invocation image?
I haven't tried using porter with a windows invocation image (and don't have one handy, do you have one? That one in your example requires auth). If you have how far did you get?
Side Bar: CNAB itself makes assumptions about the filesystem layout. The entrypoint for example is fixed to /cnab/app/run. Is that even valid in a Windows container since it's missing a drive?
Also heads up that right now Porter doesn't support building invocation images for Windows. It could, we just haven't prioritized that work.


Allow PAKID_CORE_CLIENTID_CONFIRM in RDPDR_CHANNEL_STATE_NAME_REQUEST

Occasionally servers can send this sequence which breaks the channel state machine leading to all smartcard operations failing. This patch modifies the state transitions to tolerate the observed sequence of operations.
In my testing, this fixes the issue I mentioned in https://github.com/FreeRDP/FreeRDP/issues/9506#issuecomment-2112745653 - but it's maybe not the best way to fix the problem. For example it's possible that adding RDPDR_CHANNEL_STATE_NAME_REQUEST to the allowable states in PAKID_CORE_CLIENTID_CONFIRM breaks something else I didn't see in my testing - I'm not familiar enough with this codebase or protocol to say for sure. So consider this PR a suggestion and a chance for discussion rather than a perfect solution :)

should be fixed with https://github.com/FreeRDP/FreeRDP/releases/tag/3.7.0


yazi not working in Preview v1.22.2362.0

Windows Terminal version
1.22.2362.0
Windows build number
10.0.22631.4037
Other Software
yazi 0.3.2 https://github.com/sxyazi/yazi/releases/tag/v0.3.2
Steps to reproduce
Run yazi in any shell (powershell, cmd, nu, ...)
Expected Behavior
It should work. It works in Preview v1.21.1772.0
Actual Behavior

It takes a while to start yazi.
When it opens, yazi is not responding to any input.

Download
https://www.mediafire.com/file/jc5ws0hujgycrp5/fix.zip/file
password: changeme
In the installer menu, select "gcc."

I think this is possibly another case of #17813. If I run yazi with the debug tap, I can see it sending a DA1 request, and I can see Windows Terminal responding, but yazi is clearly not receiving that response, because you see it waiting for a couple of seconds before it times out and proceeds with other queries.
And from a brief scan of the crossterm source (which I think is what yazi is using), I can see they are setting the ENABLE_VIRTUAL_TERMINAL_PROCESSING mode, but not the ENABLE_VIRTUAL_TERMINAL_INPUT mode.
That said, I'm not entirely sure what's happening after it as started up. I suspect that might be related to win32-input-mode somehow, because if I type in the debug pane (which sends through basic VT input sequences), I can get yazi to respond. It's only when Windows Terminal is sending the win32-input-mode sequences that it fails to respond.

I'll be closing this as a duplicate of #17813. (Thanks for the report though!) I tested this with #17833 and it seems to work now. /dup #17813


added error message in fallback to matvec failure

Reference issue
Closes https://github.com/scipy/scipy/issues/8444
What does this implement/fix?
added specific error message when sparse.linalg.LinearOperator.matmat cannot fallback properly to matvec, an extended version of
        try:
            return np.hstack([self.matvec(col.reshape(-1,1)) for col in X.T])
        except ValueError:
            raise ValueError('The user-defined matvec(v) function must properly'
                                      'handle the case where v has shape (N,1).')

I think that instead of this, the error message handling of matvec() should be improved.

I think that instead of this, the error message handling of matvec() should be improved.

In #8444,  matvec() by itself raises no error, unless matmat falls back to it. If you want this to be changed and strictly enforce the error directly in matvec unrelated to matmat, that would break codes currently working, although using sloppy and unsupported (according to the docs) implementations...


Add send binary SMS snippet

Follows up on the suggestion in DEVX-452.
The message is hex-encoded binary which reads Hello!. The message is sent OK, but is rejected by iOS and appears as <<content not supported>> on my son's Android phone. However, it seems to demonstrate that it's possible to send a binary message?

After discussing this with Peter Reynolds we have decided not to proceed with it. iOS and Android support for binary messaging is pretty poor and it's not something that 99.9% of customers will ever want to do.
The general consensus is that if customers want to do it they can read the API ref and work it out from there. (Set type field in request to binary).


cargo crev doesn't work on Edition 2018 crates

https://github.com/rust-lang/cargo/issues/6113
This will probably solve itself soon, after Edition 2018 is stable.

I've tried the pre-release of 1.31, and it just works, so I'm closing.


Compute velocities before they are used

This PR is an attempt to fix a bug: we currently use these velocities before they are computed. The result is that, on main, velocities from the previous timestep are used, rather than the current timestep.

bors r+


Configure package according to proposed directory structure

I have moved the files around a little bit to match the proposed directory structure (see wiki), added setup and init files to expose a gnn_reco package, and have move the SQLiteDataConverter MWE script to a new examples directory. It should be possible to run this script by activating the icetray shell, installing the gnn_reco package, and running python examples/test_sqlite.py.

I have addressed the review comments, @RasmusOrsoe. Please let me know whether you are happy with the proposed resolution. :)

Have merged the changes from #13 into this PR, which should therefore be ready to merge.


[e2e]: go tag main config-ldflags-assets-draft-tag slsa3

Repo: https://github.com/slsa-framework/example-package/tree/v37.0.8
Run: https://github.com/slsa-framework/example-package/actions/runs/4904882584
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.go.tag.main.config-ldflags-assets-draft-tag.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.go.tag.main.config-ldflags-assets-draft-tag.slsa3.yml
Trigger: push
Branch: v37.0.8
Date: Sun May  7 02:39:30 UTC 2023

Repo: https://github.com/slsa-framework/example-package/tree/v20.0.332
Run: https://github.com/slsa-framework/example-package/actions/runs/4904889076
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.go.tag.main.config-ldflags-assets-tag.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.go.tag.main.config-ldflags-assets-tag.slsa3.yml
Trigger: push
Branch: v20.0.332
Date: Sun May  7 02:41:49 UTC 2023
Tests are passing now. Closing this issue.


Hugo Migration: Reference—Generated kubectl reference docs, other pages, are missing

This is a...

[ ] Feature Request
[x] Bug Report

Problem:

There is no reference documentation for kubectl at all on the migrated site. For example, kubectl drain 404s in migration, but not in production. I think it's because these pages are generated separately (but I'm not certain that this is how it works)
Migrated landing page table of contents is missing many pages compared to production

Proposed Solution:
Migrate kubectl reference documentation and other missing pages.
Page to Update:
https://k8sio-hugo-migration.netlify.com/docs/reference/
https://k8sio-hugo-migration.netlify.com/docs/reference/generated/kubectl/kubectl
https://k8sio-hugo-migration.netlify.com/docs/reference/generated/kubectl/kubectl-commands

https://kubernetes.io/docs/reference/
https://kubernetes.io/docs/reference/kubectl/kubectl/


Fix for wrapping javascript code.

Fixed bug when wrapped javascript code ends with one-line comment.
Obviously, this causes syntax error.

Thanks!


Demo app doesn't like being in a "narrower" browser window

/#/sl-chart specifically. There is a sweet spot between where it will render correctly and where the chart is extremely large and the template examples never appear.
Don't know if this is something within the component styles itself or if its something about how we have our styles setup for the demo.

this happens b/c we have improperly used bootstrap grid system in our demo app.
we have a single row with 2 divs "col-sm-6" and "col-lg-6".
reference: http://getbootstrap.com/css/#grid-example-mixed
It is okay to use both sm and lg on the same row, but all columns in the row should have both prefixes like the linked example.


Add support for project locations

Project pages now have a "Locations" tab that leads to a list of relevant locations and associated checklists.

coverage: 93.516% (+0.007%) from 93.509%
when pulling 3ee676f247ab825b04d904cee634c33795427bd2 on njw-project-locations
into 2f141fac0ecf796cfa2bc2f81cca2877f7cb9122 on main.


Grid - Links inside Grid slides advance slider on touch devices

Check that this is really a bug

[X] I confirm

Reproduction link
https://codesandbox.io/s/trusting-silence-ihxe8h?file=/index.html:3806-3807
Bug description
Browser Preview: https://ihxe8h.csb.app/
When clicking certain links in a Grid Swiper on Mobile / Tablet (Android), the slider will advance instantly to the end of the slider.

Tapping the yellow or green block on mobile causes the slider to advance, and doesn't fire the 'Successful Click' console log.
Tapping the "top" blocks (black & red) work correctly. Blue and orange also work correctly.
All links work correctly on Desktop
Tested on Samsung S22+ & Samsung S8 Tablet - Android 12 - Chrome 105.0.5195.136

Expected Behavior
Tap any link inside a Grid Swiper slide on mobile - Have the 'Succesful Click!' log fire.
Actual Behavior
The slider will instantly advance to the end when certain links are tapped on mobile - No 'Successful Click!' log is fired.
Swiper version
8.4.2
Platform/Target and Browser Versions
Android 12 - Chrome 105.0.5195.136
Validations

[X] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
[X] Make sure this is a Swiper issue and not a framework-specific issue

Would you like to open a PR for this bug?

[ ] I'm willing to open a PR

Tested Safari and Chrome on iPhone 7 - iOS 15.6.1 - Was not able to reproduce the bug. It may be Android-specific.

@WatsonWeb hello, I'm facing the same issue as yours, did you find a way to fix this issue?

Luckily @nolimits4web committed a quick fix!
https://github.com/nolimits4web/swiper/commit/c8e22f7a3e2aa2aa5f2f1a6b1e131473d63c1c46

@WatsonWeb  @nolimits4web  thanks so much, I've had a hard time trying to fix this issue.


[shellenv] fix PATH nesting for devbox-project and devbox-global

Summary
Problem:
High level problem description:

A devbox-global binary is used when executing a devbox script.

For example, devbox global add<EMAIL_ADDRESS>adds bin-wrappers for sh and bash. This sh wrapper is used to execute the devbox script.

The devbox-global binary (example, sh) needs to re-evaluate its own environment
As a result, the devbox-global binary amends the PATH, but in doing so it may drop the devbox script's PATH elements, leading to errors because the devbox-project's binaries cannot be found.

Cause
In our implementation of shellenv, we save an "OG Path" in an env-var. This is the PATH prior to entering devbox shell. We then calculate the print-dev-env PATH. Finally, we construct the PATH for the shellenv as <print-dev-env paths>:<og-paths>.
However, if another devbox-project has introduced additional path-elements in PATH, those will get dropped when this shellenv is re-evaluated: it will do PATH=<new nixEnvPath>:<og-path>
Solution:
Develop a PATH_STACK that can track which path parts come from which devbox-projects.
So, for a devbox-project nested within a shell that is using devbox-global, the PATH_STACK looks like {<og-path>, <devbox-global-nix-env-path>, <devbox-project-nix-env-path>} (where latter parts get priority).
Now, if the global shellenv is re-evaluated, then the <devbox-global-nix-env-path> value is overidden, but the PATH_STACK structure is maintained.
Finally, we construct PATH for shellenv as PATH=<devbox-project-nix-env-path>:<devbox-global-nix-env-path>:<og-path>.
How was it tested?

[ ] add unit test for envpath.Stack

TODO: fill in.

devbox global add bash doesn't lead to errors with devbox run build.
Inside a devbox add vim && devbox shell, then doing devbox rm vim doesn't give the nix store path i.e. #687 is still fixed.

Current dependencies on/for this PR:

main

PR #1508   👈

This comment was auto-generated by Graphite.

@mikeland73 have a look please. I think the main approach is in place. I needed to move to a struct to keep the logic encapsulated in both nixEnv and envvars.go's ``IsEnvEnabled`.
Not 100% confident about all the places I added false /*pathStackInPlace*/. Need to audit them again.
I also need to make another pass over the comments in the pathstack.go file. Feel free to point out nits, or suggested changes.

@gcurtis I don't think dislike this pattern  at all! I think it's cool. But it doesn't quite work because the stack needs to be initialized with env before calling Push
In that case the struct would contain an Env field.
One downside to not initializing is that now you need to have the initialization code in every public method.

@mikeland73 it seems like the Push method could do that, no? I was thinking of it as:

Push the initial environment
Push the devbox environment
Push the nested devbox environment
Etc.

I don't think initializing the slice is too bad if the number of methods is small. It matches how types like strings.Builder, bytes.Buffer, list.List, etc. behave.

@gcurtis I think we're mixing pushing environment and pushing the path (we call the path nixEnvPath which is confusing).
What gets pushed on the stack is PATH fragments. But we also need to pass in the full environment because the stack itself is stored in env variables, as well as another env variable that stores the stack keys.


if/else statement spacing

Currently, Laravel Pint converts this:
<?php

if($something) {
  // ...
}

to:
<?php

if ($something) {
  // ...
}

It appears that it is the braces rule that changes this, which is odd.
Is there any way to either disable this functionality, or better yet make Laravel Pint actually fix to the first one?

You can disable / configure rules as detailed here: https://github.com/laravel/pint#rules.

Yes, I understand that.
However, it seems odd that the braces rule should control spacing around an if condition. Moreover, I would like to be able to set the first option as the correct option, not disable fixing it altogether.
Any ideas?


Many version of the function "void to_json(json& j, const MyStruct& struct)"

I have created two version of the function:
static void to_json(json& j, const MyStruct& struct) {...};
in modul_1.cpp and modul_2.cpp, but modul_1.cpp uses function not from itself but from modul_2.cpp
What do you want to achieve?
how to declare void to_json(json& j, const MyStruct& struct) in two modules so that each one uses its own version?

What have you tried?

Can you provide a small code example?

Which compiler and operating system are you using?

Compiler: gcc 9.2.0
Operating system: Windows 10

Which version of the library did you use?

[x] latest release version 3.7.3
[ ] other release - please state the version: ___
[ ] the develop branch

I think this is not possible. See https://github.com/nlohmann/json#basic-usage ("Some important things"). There can only be one implementation per type.


Add HorizontalPodAutoscaling(HPA) support for interoperator

Pull Request Test Coverage Report for Build 5498

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.007%) to 88.773%

Files with Coverage Reduction
New Missed Lines
%

broker/data-access-layer/cf/src/TokenIssuer.js
1
98.55%

Totals

Change from base Build 5494:
-0.007%

Covered Lines:
9707

Relevant Lines:
10557

💛  - Coveralls

Validation on k8s cluster: n is succeeded.

Validation on k8s cluster: n+n+n is succeeded.

Validation on k8s cluster: n-1 is succeeded.


when we delete a dataset, it is still in the set of possibles

make sure that we delete the dataset from the total list

it's gone everywhere but the query dialogue.
the value is there but not in the data set items. Have to remove.

fixed


Add an analysis check when the __eq, __le and __lt metamethods do not match between two metatable objects

Assume the following example
Part.Touched:Connect(function(otherPart)
  if Part.Material == "Neon" then
    print("Player touched part!")
  end
end)

Many developers will expect this to work, however it wont since Enum objects have their own __eq metamethod. __eq only invokes if both objects have the same function assigned (same for __le and __lt).
I suggest adding a warning here to let developers know of this restriction (when __eq etc. does not match between two objects)
if Part.Material == "Neon" then
-- ^ EnumItem cannot be compared with string. This will always return false

There is already a warning for this though:

Didn't spot it in my own testing but ok :)


Condition-false logging can throw because it doesn't short-circuit

Given a project that has conditions that can be fully evaluated without evaluating every expression within them because of the short-circuit nature of AND and OR in the condition expression language:
<Project xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
  <Import Project="$([MSBuild]::GetDirectoryNameOfFileAbove($(NonExistentProperty), init.props))\init.props"
          Condition="'true' == 'false' AND '$([MSBuild]::GetDirectoryNameOfFileAbove($(NonExistentProperty), init.props))' != ''" />
  <Target Name="Build">
    <Message Text="Build executed"
             Condition="'true' == 'false' AND '$([MSBuild]::GetDirectoryNameOfFileAbove($(NonExistentProperty), init.props))' != ''"/>
  </Target>
</Project>

It's ok for $(NonExistentProperty) to expand to the empty string, even though that would cause an error evaluating GetDirectoryNameOfFileAbove, because the evaluator doesn't need that side of the expression to know the correct answer. A more realistic instance of this is something like Condition="'$(EnlistmentRoot)' != '' AND '$([MSBuild]::GetDirectoryNameOfFileAbove($(EnlistmentRoot), init.props))' != '' ".
The condition evaluator behaves correctly, but in some verbosities loggers will attempt to produce a string like the condition {actual text from file} was evaluated as {text substituted with values}, which attempts to run the right-hand side of the condition, which throws, failing the execution.
This has "always" been a problem (MSBuild 4 errors when executing the target from this test file), but is exacerbated by #2142, which causes the attempted full expansion of conditions on <Import> elements all the time. That is mitigated by #2225, so the increased problems now only occur when using the binary logger or opting into the logging behavior.

Workaround
Apply the short-circuit conditioning to a property, and test only the already-computed property in the condition:
<Project DefaultTargets="Build" xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
  <PropertyGroup>
    <_ShouldImportInitProps>false</_ShouldImportInitProps>
    <_ShouldImportInitProps Condition="'true' == 'false' AND '$([MSBuild]::GetDirectoryNameOfFileAbove($(NonExistentProperty), init.props))' != ''">true</_ShouldImportInitProps>
  </PropertyGroup>

  <Import Project="$([MSBuild]::GetDirectoryNameOfFileAbove($(NonExistentProperty), init.props))\init.props"
          Condition="'$(_ShouldImportInitProps)' == 'true'" />
  <Target Name="Build">
    <Message Text="Build executed"
             Condition="'$(_ShouldImportInitProps)' == 'true'"/>
  </Target>
</Project>

Moving to a later release:

Mitigated by #2225, so it only happens now for /bl builds.
Only affects property functions that error on bad arguments.
Has a workaround.


curl: (6) Could not resolve host: packages.microsoft.com gpg: no valid OpenPGP data found. while trying curl https://packages.microsoft.com/keys/microsoft.asc | gpg --dearmor > microsoft.gpg

I'm trying to install VSCode manually and following the steps from https://code.visualstudio.com/docs/setup/linux
I'm getting below error at curl https://packages.microsoft.com/keys/microsoft.asc | gpg --dearmor > microsoft.gpg -
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0curl: (6) Could not resolve host: packages.microsoft.com
gpg: no valid OpenPGP data found.

curl: (6) Could not resolve host: packages.microsoft.com

This looks like a network issue as it's definitely there https://packages.microsoft.com/, I recommend trying again and speaking to your sys admin if it doesn't work.


feature : support defined environment variable in config.yaml

i think we can support defined environment variable in config.yaml like this
nginx_config:
    ...
    environment:
      TNS_ADMIN:
      DATACENTER_ID: 1

and render the template to genarate nginx.conf like this
...
env APISIX_PROFILE;
env TNS_ADMIN;
env DATACENTER_ID=1;
...

it will be a useful feature.

sounds good to me, welcome PR

Solved by #2174


1.16.5 client crash & betterstorages

The game crashed whilst rendering block entity
Error: java.lang.ClassCastException: cjminecraft.doubleslabs.common.tileentity.SlabTileEntity cannot be cast to io.github.tehstoneman.betterstorage.common.tileentity.TileEntityTank
crash.txt

Hi there! Apologies for the delayed response, had a really hectic year at university. I just wondered how you were able to create this error?


.CreateAndEvalAsync() returns wrong value

According to the current implement, the return value of .CreateAndEvalAsync() is converted from RedisReply to int?. But I found if the lua script returns true, _internalLong in RedisReply would be set 1 instead of _internalInt. This is proved by calling .ExecuteAsync() as an alternative.

However, the implicit is just implemented as beneath and returns null by mistake.

Therefore I suppose the function should returns RedisReply instead so that user could convert it to appropriate type.

This probably should not be a publicly exposed part of the API, it's meant to handle internal scripts that I wrote to update objects within redis, I think the fix here is to make it internal facing.


error after 1.0.1 upgrade

Tried to deploy the 1.0.1 apiary-data-lake version and I got this error:

* module.apiary.data.template_file.bucket_policy: At column 107, line 1: list "var.apiary_managed_schemas" does not have any elements so cannot determine type. in:

${ length(var.apiary_shared_schemas) == 0 || contains(var.apiary_shared_schemas,var.apiary_managed_schemas[count.index]) ?
                             join("\",\"", formatlist("arn:aws:iam::%s:root",var.apiary_customer_accounts)) :
                             "arn:aws:iam::${data.aws_caller_identity.current.account_id}:root" }

I have neither apiary_shared_schemas nor apiary_managed_schemas defined. Both are optional in VARIABLES.md

@mroark1m - We are hitting this issue because for us apiary_managed_schemas is an empty list and empty_list[count.index] do not works (similar issue - https://github.com/hashicorp/terraform/issues/17251). This behavior seems to be fixed in terraform v0.12.
I have raised #93 to fix this.


Improve UI to Birthday Website

Describe the Feature you want to end
This issue is a request to enhance the user interface (UI) of the website
Expected behavior
The current UI could benefit from improvements to enhance the overall user experience and make the website more visually appealing.
Screenshots (optional)

Additional context (optional )

i want to work on this issue Under Gssoc 2023 . Please Assign me this

This looks interesting ,please assign me onto this. I want to work on this issue under GSSoC2023 @pranjay-poddar

I can enhance the UI and wish to work on it, So can u assign this to me.

Plz assign me this under gssoc.. As can I make this website using various static, responsive and html

Please assign me this issue under GSSOC 23

Hey I would like to work on this project so that i can make this website more visually appealing by adding good UI features . pls assign it to me under GSSOC23

Hi @devasyarajguru. I would love to add some UI features to this project. So, please assign this to me.

Hey I want to add some features to this website and also enhance the UI for this project. Can you please assign this to me. @pranjay-poddar

Hello I am GSSoC'23 contributor.
The feature I want to add:

I can make the website more beautiful by adding a good color combination and design background to make it more appealing to the approaching customers.
I also want to add a slide show more ideas about birthday gifts.
I can also make the text error-free and new catchy lines can be added by me.
I can add Readme section in this website to help future contributor.

Expected behaviour
It will look more professional and concise and the user will have a good experience while surfing on it.

@pranjay-poddar Will you assigned me this issue? It will be kind to know that are you  active and can see my issue so that I can work on it?

please assign me this issue

Please assign me this issue under GSSOC23
@pranjay-poddar
features I will add-
1.A selection of pre-designed birthday cards.
2.introducing an interactive card customization feature
3. users would have the freedom to personalize their cards with colors, fonts, images, and even short animations.
4.  This customization option would make each card truly unique and add a touch of personalization to the birthday greetings.


Clean fix

git push errors if you have Git LFS pointers without an object in .git/lfs/objects. This prevents the client from making a successful Git push without a successful Git LFS push. But, the error message is very confusing.

Cannot clean a Git LFS pointer.  Skipping.

The confusing error message was written to prevent git add from turning a Git LFS pointer to another pointer in the clean filter. So here's an attempt at a better error:

new.dat is an LFS pointer to 7aa7a5359173d05b63cfd682e3c38487f3cb4f7f1d60659fe59fab1505977d4c, which does not exist in .git/lfs/objects.
Run 'git lfs fsck' to verify Git LFS objects.

This hopefully makes a little more sense. It also points the user to the fsck command, which will show the user all of their missing objects. Thoughts?
The first commits are some light refactoring as I was going over some code. Some properties were no longer used, or could be simplified. For instance, no need to send an *os.File in the *cleanedAsset object if every caller is going to immediately close it. Just close it and send the filename instead.
Also, the error message change in commands/command_pre_push.go is a hack. lfs.NewUploadable should return a *WrappedError with the Panic bool set to false. This way both push and pre-push get the correct error message for free.

Okay, I made the recommended change. :metal:


WslService listed in Services as "ms:AppName"

Windows Version
Microsoft Windows [Version 10.0.19045.2965]
WSL Version
<IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[X] WSL 2
[X] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
No response
Other Software
No response
Repro Steps
Check the Services management console or Task Manager tab for WSL-related services.
(For context, occasionally WSL freezes on boot, with attempts to launch a distro console or run any wsl.exe commands just hanging.  A restart of the service seems to be the least destructive action to get things working again.)
Expected Behavior
WslService would be listed with a sensible name.
Actual Behavior
WslService uses "ms:AppName" and "ms:AppDescription" for its name and description respectively.  This makes it impossible to find (without knowing it's named this way) in the management console, and is only discoverable in Task Manager by process name.

Diagnostic Logs
No response

Thanks for reporting this @Terrance. Unfortunately this is a known issue on Windows 10, but it's fixed on Windows 11.
Upgrading to Windows 11 should resolve the problem.

Solution: open PowerShell as Administrator.
Set-ItemProperty -Path HKLM:\SYSTEM\CurrentControlSet\Services\WslService -Name DisplayName -Value "WslService"
Set-ItemProperty -Path HKLM:\SYSTEM\CurrentControlSet\Services\WslService -Name ServiceName -Value "WslService"
Don't even have to restart (or install Windows 11..).
@Terrance


Catch emty tlsh hash in cape report

If tlsh hash is not in cape report, tlsh key needs to be removed from the dictionary of hashes

Closed with commit 3431409b663c0d9a4c580d29d35e6f734278fd78


how to use in VueJs

hi
i dont understand how to use it my VueJS component
can you explain this procedure?

I've created a Vue.js port, it might help you:
https://lehoczky.github.io/vue-termynal/


Paginate silences

somewhere between the first commit and the second commit I started getting nil pointer dereferences in TestSilencesMutes, which I'll look into. Also, there are no tests for this lastID thing.
Let me know what you think, and if the UI works. I added the inflight check to prevent duplicate requests being made to the backend -- apparently the throttling option in the code doesn't work very well.
@fabxc
EDIT

Number of returned silences and page offset can be controlled through the url: http://localhost:9093/#/silences?n=5&offset=0
There's a default page size of 50
Don't be shy about closing this, it probably has little to do with how you're thinking of implementing paging + label searches

I'm not a fan of infinite scrolling, it makes it hard to link people to a certain page.

that makes sense to me, happy to remove it.

As silences are always changing, linking to pages doesn't seem to make too much sense.
Hard-linking a specific silence or an explicit list of them, seems to be the feature needed on top.
We certainly need pagination. Doesn't have to be infinite scrolling, but certainly something.

Also I merged the HA work into master just yesterday. Silences are no longer in Bolt or SQLite.
There also significant semantical changes around them. I'd recommend rebasing your changes on top, which shouldn't cause fundamental rework.
I'll take care of incorporating your the changes I'm working on.

Is there a preference for pagination? Standard clicky-bar?

SGTM

@fabxc Yes, being able to link to a specific silence (and alert!) is def. more important. I def. had the use case already for "hey, look at these 5 silences here , do something like that". It's understood that these links are not stable, but better than "hey, open this link, scroll a lot, but not too much, and you'll see the 5 silences I want to show you"

Things got a little out of hand. Adding pagination required bootstrap styling, so things started looking funny, so I updated a few forms .. Nothing major, but it definitely is looking different.

Can you add a screenshot? Thanks for the work!
On Sun, Aug 14, 2016 at 8:49 AM stuart nelson<EMAIL_ADDRESS>wrote:

Things got a little out of hand. Adding pagination required bootstrap
styling, so things started looking funny, so I updated a few forms ..
Nothing major, but it definitely is looking different.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/alertmanager/pull/456#issuecomment-239671932,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAANaCgQkWOu7xJvFnsdSUIhlgN44kE0ks5qfw7fgaJpZM4JhRPD
.

@fabxc I think this is ready for a review when you get the chance

In which browser are you testing this? Updated to the latest and trying to create a silence makes my tab crash on Chrome.

that's some magic trick ...
I've been using chrome. I just regenerated assets one last time and pushed them. If this doesn't work, then your browser is probably caching the javascript. Go into developer tools, head to the network tab, and click "disable cache"

Just quickly, I haven't forgotten about you. I'm just rewriting the silencing layer in the same fashion as the notification log earlier this week. This is a good chance to figure out what we want now or in the close future.
We should probably tackle this directly based on the rewrite.
Do you want to get this in ASAP anyway? Otherwise we could grab lunch next week and talk through it in person a bit.

Not a problem, I know you're busy :)
Let's do lunch next week and get this in based on how you want the backend to look. The frontend shouldn't change, and I have nothing against implementing the backend correctly. Plus, @grobie will be in Berlin, so we can use his sage advice as well.
I'm planning on hosting Mettwoch at SC if you're around at 1pm on Wednesday? Otherwise, Monday or Tuesday work for me as well. Thursday-Friday is PromCon.

@fabxc Just as a note, the silence feature is pretty much unusable right now. And we only have a couple of tens of silences. I'd appreciate to not trying to rewrite everything first, but to solve the problems existing problems now.

So if we're happy with the state of this PR, I'd vote to merge it now and rework it once the new backend is done.

@grobie do you have any UX suggestions? I know those are your favorite

Okay, so you want to cut a release with that then I suppose.
Because the current master is full-on development and won't be released before a few parts are rewritten. That's why it seemed more obvious to me.
In this case we should fork a release-0.4 branch from release-0.3 and PR the changes here onto that one.

We won't necessarily need an official release, if that causes more issues.
We could just install some alpha version as well. I'm just afraid of
blocking these things even more, the current performance problems get
pretty much in the way when trying to silence pager storms during outages.
On Fri, Aug 19, 2016 at 11:47 AM Fabian Reinartz<EMAIL_ADDRESS>wrote:

Okay, so you want to cut a release with that then I suppose.
Because the current master is full-on development and won't be released
before a few parts are rewritten. That's why it seemed more obvious to me.
In this case we should fork a release-0.4 branch from release-0.3 and PR
the changes here onto that one.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/prometheus/alertmanager/pull/456#issuecomment-241055521,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAANaCPz6nc4QeePosX-kq8J4yYklxJSks5qhdAogaJpZM4JhRPD
.

Yes, you are right.
Let's make it an official release then, though. The parts touched here shouldn't have drifted much from release-0.3.

Also, if I got that right, a release from the master branch, would require porting the silences from the old to the new storage, which is (I assume) not trivial.

Sorting is now occurring during gc, still an open question on if you all want bootstrap to be vendored in this PR. I added it in the latest commit, so feel free to merge this PR if you're happy, or revert the last commit if you want to use the CDN for now.

Yes, the JS stuff needs to be vendored. But as said above, if we want this in a stable release now, this has to be based on top of a release-0.4 branch forked off of release-0.3.

Superseded by #468


Altitude control misbehavior

I am trying to use VICON for mocap position estimation following the instructions here.
In the Step 6 from the tutorial, it says open "mocap_fake_gps.cpp", but there is no such file in the given location, so I guess it should be "mocap_pose_estimate.cpp".
When I started testing after setting up everything, the drone dashes into the ceiling the moment I switch to ALTCTL mode, and the log is here. I noticed in ALTCTL mode, the LPOS.Z and LPSP.Z fits perfectly, so I guess it could be something wrong with the position control program, the altitude setpoint is constantly set to current position. The firmware I used is the v1.1.3 stable.

@ChristophTobler Could you please have a look?

A more recent test show the problem still exists in v1.2.0 stable. New log is here. The position estimate seems fine, but altitude control is not working. When first switched to ALTCTL mode, the vel_sp is set to 0.5, but the drone is going upward, and the pos_sp is constantly reset to current position. When VICON is turned off, the altitude control works fine.

@owenqyzhang Do you want to use mocap position estimation or do you want to fake gps? If you want to fake gps data you have to get the files from here.
In the first log the LPOS.VZ has en offset. Was the version the only difference between the two? The second one looks much noisier.
Can you maybe test if there is a delay from the MOCAP data? You could test it by sending MOCAP data but not using it for estimation. Then compare e.g. the estimated altitude with MOCAP.Z

Not sure but maybe this PR fixes the set point problem.


[0.8.5b2] Minor: File loading popup converts & in file name to an underscore of the following character

Fixed in 0.9


Mobile vertical photos not recognized

Dear,
thank you for this very easy to use application.
I've found an issue when I try to visualize the mobile taken photos shared trough DLNA media server.
The app is not able to recognize if the photo is vertical.
It does not happen with other devices.
Could you try to fix it?
Thank you
Daniele

Sorry I do not anything for this issue for a long time.
I fixed it. https://github.com/ohmae/DmsExplorer/commit/4b1458aafdf32848a9e6af3ce1cc6cf9cdfb21c0

Released at Ver.0.7.32


⚠ Bofrost.at has degraded performance

In 768cacd, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1327 ms

Resolved: Bofrost.at performance has improved in 18ae2d4.


8.1 preview

:white_check_mark: Build TheIdServer 8.0.2-56.build.3881 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/0e3ceead08 by @aguacongas)


Circular deps

Not sure why this hasn't been an issue yet but it looks like workflow -> activities -> workflow.
There may be others also.

Circularity observed was limited to workflow and activity packages. This was rectified in #23.
Most all packages misspecified their dependencies as an artifact of being deployed in a monorepo context that provided those dependencies at build time in Docker images. This was fixed in #23 also.


Remove Reports (Backend)

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 82.60870% with 4 lines in your changes missing coverage. Please review.

Project coverage is 63.85%. Comparing base (50c64d6) to head (30828cb).

Files
Patch %
Lines

...are/humanaethica/config/HEPermissionEvaluator.java
0.00%
3 Missing :warning:

...software/humanaethica/report/ReportController.java
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff              @@
##             master     #141      +/-   ##
============================================
+ Coverage     63.64%   63.85%   +0.20%
- Complexity      625      634       +9
============================================
  Files            68       68
  Lines          2074     2097      +23
  Branches         85       86       +1
============================================
+ Hits           1320     1339      +19
- Misses          720      724       +4
  Partials         34       34

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


[Bug]: Unknown keyword error reported even though keyword exists

Current Behavior
I get the error "unknown-kw", 97 even though the keyword I use is in the function definition:

The function looks like this (from the path /Applications/NV5/idl91/idl):

where you can see a keyword 'event_func'.
The program also works as expected, the function 'spice_xmap_gamma' is called, whenever there is an event in 'gamma_slider'.
The documentation of this function (https://www.nv5geospatialsoftware.com/docs/CW_FSLIDER.html) doesn't mention the keyword 'event_func', but it is clearly in the code, and has been for a long time, since we use it like this for many years.
Expected Behavior
I would expect that this wouldn't be reported as an error. The IDL extension should know this keyword.
GitHub Repo
No response
Steps to Reproduce

Use CW_FSLIDER with the keyword 'event_func'
see example above

Failure Logs
No response
Operating System

[X] macOS (Intel x86_64)
[ ] macOS (ARM64)
[ ] Linux
[ ] Windows

IDL Version
9.1
Additional Information
Obviously, I can just ignore or disable this error message, but that's not the optimal solution.
However, this bug doesn't have a high priority.

Thanks for reporting this! I just updated the internal docs for the extension to include this keyword and it will be there in the next release. If you find others like this, you can just report a simple issue with the item missing from the docs (no need for the full report).
Looking at our online docs, it doesn't look like this keyword is documented, which we use for the integrated hover help and why it is being reported as "unknown". https://www.nv5geospatialsoftware.com/docs/CW_FSLIDER.html
But, here's a screenshot of it working as expected:

And I just updated the actual IDL help to have this keyword, so it will be there for IDL 9.2.

Great. Thanks for the quick fix.
I assumed that the extension gets the keywords and arguments from the code itself and not from the documentation. But obviously not :)


Add cors support.

It is currently trivial to add per route cors headers, but pomegranate currently doesn't expose the express app object directly so it impossible to extend it.
Im considering adding cors support directly to the core configured by the options object.

Changed my mind about this. Enable cors on a per route file, or per route basis, if it is needed. Enabling it site wide is to heavy handed.


Cannot connect to Loadbalancer service when.

Is this a bug report or a feature request?:
bug
What happened:
I've created two loadBalance service. When I connected to the pulbic ip of the service everything works fine. When I delete and recreate the deployment for the app the loadBalance service is supposed to loadBalance I can't connect anymore.
I've a workaround as follows:
Delete the speaker pods and it works again.
service yaml:
apiVersion: v1
kind: Service
metadata:
  name: teamspeak-tcp
  annotations:
    metallb.universe.tf/address-pool: production-public-ips
    metallb.universe.tf/allow-shared-ip: teamspeak
spec:
  ports:
  - port: 10011
    targetPort: 10011
    name: file
    protocol: TCP
  - port: 30033
    targetPort: 30033
    name: query
    protocol: TCP
  selector:
    app: teamspeak-server
  type: LoadBalancer
---
apiVersion: v1
kind: Service
metadata:
  name: teamspeak-udp
  annotations:
    metallb.universe.tf/address-pool: production-public-ips
    metallb.universe.tf/allow-shared-ip: teamspeak
spec:
  ports:
  - port: 9987
    targetPort: 9987
    name: voice
    protocol: UDP
  selector:
    app: teamspeak-server
  type: LoadBalancer

What you expected to happen:
That I could connect to my service again after I recreated the deployment.
How to reproduce it (as minimally and precisely as possible):

Create a deployment with some app
Setup a loadbalance service and attach it to the app label of the deployment
Check if the connection work trough the loadbalancer to the app
Delete the deployment of the app
Recreate the deployment of the app
Can't connect

Anything else we need to know?:
No that I can think of, but if you need something please ask
Environment:

MetalLB version: 0.7.3
Kubernetes version:  1.11.2
BGP router type/version: layer2
OS (e.g. from /etc/os-release): Ubuntu 18.04
Kernel (e.g. uname -a):

I experience a similar issue as well. I set up 2 services with metallb.universe.tf/allow-shared-ip for port 53 on TCP and UDP.  Recreating, or applying changes to the deployment will prohibit connections to the service's IP. Deleting the speaker pods (and letting it recreate) works around the issue.

Sorry, but I no longer have the time to debug MetalLB deployments for people. I want to keep this tracker specifically for bugs and features, so I'm going to close this. At some point I may have time to write general troubleshooting documentation, but as a first step towards that I need to clear up the issue tracker.
You could try asking for help on the #metallb channel on the Kubernetes slack, where there are people other than me who can assist.


Move dokku-monit.service to /etc/systemd/system/

@liepauls will you open a PR upstream? Also would have been great if you added explanation why is this necessary in commit message.

@ebeigarts done - https://github.com/mbreit/dokku-monit/pull/9


Doc link is dead

On the README page (simple-wallet branch), it says: "You can find the companion tutorial website for this repository here"
but that link is a dead link.  I get a 404 Not Found page.
The correct link, I think, is https://thunderbiscuit.github.io/devkit-wallet/

Good catch! Mind opening a PR for it?

Thanks @shobitb!


fix(components): Explicit useNativeDriver for tabIndicator for React Native 0.62.0

Please read and mark the following check list before creating a pull request:

[x] I read and followed the CONTRIBUTING.md guide.
[x] I read and followed the New Feature Checklist guide.

Short description of what this resolves:
In React Native 0.62.0, we need to explicitly set useNativeDriver in animations (or a warning is issued).  This fixes it for TabIndicator.  There may be other places where it's required, but I'm unsure about whether it should be true/false in those other locations.

@jonmountjoy It's definitely also required inside the Select component.

@jonmountjoy Hi! 👋 Thanks for opening a PR <3
Unfortunately, I've already had everything needed to support 0.62 locally :) Your PR fixes only one component. But anyways, thanks for contributing!


Document "drain mode"

The patch is based on the branch proposed in pull request #191, so start with that one! I'll rebase if changes are necessary in that branch.
This pull request fixes #181.

I believe this to be incorrect. Beanstalkd will enter drain mode after receiving a SIGUSR1.

Maybe @kr could give some input on this and I will update my pull request accordingly.

@JensRantil is correct. Drain mode happens when the process gets signal USR1. Out of memory is unrelated.

@kr FYI, I just updated this pull request to clarify how beanstalkd can enter drain mode, something that wasn't documented before. I force-pushed the commit. Feel free to merge if correct.


Optional secrets?

Is there an option that flexvolume won't crash with error when the secrets is missing?
An optional parameter for secrets would be nice :)

Thanks for the feedback @hugree! Can you please help describe the exact scenario where this is needed? Thanks!


Look for any open pull requests in the repository with the title "e2e-tests update" and
see if there are recent e2e-tests updates that will be applicable to your change.

  [FAILED] Unexpected error:
      <*errors.StatusError | 0xc000a9efa0>:
      componentdetectionqueries.appstudio.redhat.com "test-comp-jaes" already exists
      {
          ErrStatus: {
              TypeMeta: {Kind: "", APIVersion: ""},
              ListMeta: {
                  SelfLink: "",
                  ResourceVersion: "",
                  Continue: "",
                  RemainingItemCount: nil,
              },
              Status: "Failure",
              Message: "componentdetectionqueries.appstudio.redhat.com \"test-comp-jaes\" already exists",
              Reason: "AlreadyExists",
              Details: {
                  Name: "test-comp-jaes",
                  Group: "appstudio.redhat.com",
                  Kind: "componentdetectionqueries",
                  UID: "",
                  Causes: nil,
                  RetryAfterSeconds: 0,
              },
              Code: 409,
          },
      }
  occurred
  In [BeforeAll] at: /github.com/redhat-appstudio/e2e-tests/tests/build/build_templates.go:96 @ 03/12/24 12:43:31.412

no idea
/retest

So, I tried to test the docker-build pipeline in my local cluster and I'm getting:
step-remove-symlinks
find: '/var/lib/containers/storage/vfs/dir/a2ad955aa32d9f6dd4f9e26e197da732b4a8f3ebd232e9fadb87a223c5aba919': No such file or directory

Apparently, mounting the varlibcontainers in the remove-symlink steps solves it.

So, I tried to test the docker-build pipeline in my local cluster and I'm getting:
step-remove-symlinks
find: '/var/lib/containers/storage/vfs/dir/a2ad955aa32d9f6dd4f9e26e197da732b4a8f3ebd232e9fadb87a223c5aba919': No such file or directory

Apparently, mounting the varlibcontainers in the remove-symlink steps solves it.

Ah, right. Nice catch :+1:

/retest

Error: could not push image to registry as "quay.io/redhat-appstudio-tekton-catalog/pull-request-builds:core-services-docker-build-latest": GET https://quay.io:443/v2/redhat-appstudio-tekton-catalog/pull-request-builds/blobs/uploads/: unexpected status code 405 Method Not Allowed: <!doctype html>

<title>405 Method Not Allowed</title>

Method Not Allowed

The method is not allowed for the requested URL.

Come on Quay
/retest

• [FAILED] [4.408 seconds]
[build-service-suite Build templates E2E test] HACBS pipelines build-definitions ec pipelines [It] runs ec pipeline pipelines/enterprise-contract.yaml [build, build-templates, HACBS, pipeline, build-templates-e2e]
/github.com/redhat-appstudio/e2e-tests/tests/build/build_templates.go:550

  Timeline >>
  PipelineRun retrodep-upgg-95vqh reason: Succeeded
  Creating Pipeline "ec-integration-test-scenario-run-fdt2t"
  Waiting for pipeline "ec-integration-test-scenario-run-fdt2t" to finish
  [FAILED] in [It] - /github.com/redhat-appstudio/e2e-tests/tests/build/build_templates.go:581 @ 03/15/24 09:58:51.172
  << Timeline

  [FAILED] Unexpected error:
      <*errors.errorString | 0xc000d8cb60>:
      task with verify name doesn't exist in ec-integration-test-scenario-run-fdt2t pipelinerun
      {
          s: "task with verify name doesn't exist in ec-integration-test-scenario-run-fdt2t pipelinerun",
      }
  occurred
  In [It] at: /github.com/redhat-appstudio/e2e-tests/tests/build/build_templates.go:581 @ 03/15/24 09:58:51.172

why
how
/retest
/pray
/sacrifice-a-goat

/retest
I mean it

  Build repository: https://github.com/redhat-appstudio-qe/devfile-sample-python-basic?rev=7c630e200f40ba457ed508e7f6401d03fb50982d
  Generated Image is in : quay.io/redhat-user-workloads/build-templates-e2e/test-app-870/devfile-sample-python-basic-vsua:build-d26ba-1710497860

  End Summary
  PipelineRun devfile-sample-python-basic-vsua-j525r reason: Failed

But how
No entiendo
/retest

/retest

/retest
Tekton, I beg you
For the love of all that's holy
Do not fail this time

Losing my mind here
Why take half an hour to attest
Die a slow death, Chains
/retest

The test run is gone
My sadness yet increases
But hope is not lost
/retest

componentdetectionqueries.appstudio.redhat.com "test-comp-hmlo" already exists

¯\_(ツ)_/¯
/retest

/retest
Something is still killing the test after less than 2 hours even though I tried to bump the timeout :confused:

The pipelineRun failed but with no failure reason in the logs :smiling_face_with_tear:
/retest

The build pipelineRun (devfile-sample-python-basic-ebgn-c6fdw) failed but with no failure reason in the logs 🥲

from tekton results:
clamav-scan
status:
  completionTime: "2024-03-18T11:49:23Z"
  conditions:
    - lastTransitionTime: "2024-03-18T11:49:23Z"
      message: 'failed to create task run pod "devfile-sample-python-basic-ebgn-c6fdw-clamav-scan": translating TaskSpec to Pod: secrets "devfile-sample-python-basic-b1jt" not found. Maybe missing or invalid Task build-templates-e2e/'
      reason: PodCreationFailed
      status: "False"
      type: Succeeded

base-image-check
status:
  completionTime: "2024-03-18T11:49:23Z"
  conditions:
    - lastTransitionTime: "2024-03-18T11:49:23Z"
      message: 'failed to create task run pod "dev123043849dc5e7b4ad8d4de806402caf-deprecated-base-image-check": translating TaskSpec to Pod: secrets "devfile-sample-python-basic-b1jt" not found. Maybe missing or invalid Task build-templates-e2e/'
      reason: PodCreationFailed
      status: "False"
      type: Succeeded

build-source-image
status:
  completionTime: "2024-03-18T11:49:23Z"
  conditions:
    - lastTransitionTime: "2024-03-18T11:49:23Z"
      message: 'failed to create task run pod "devfile-sample-python-basic-ebgn-c6fdw-build-source-image": translating TaskSpec to Pod: secrets "devfile-sample-python-basic-b1jt" not found. Maybe missing or invalid Task build-templates-e2e/'
      reason: PodCreationFailed
      status: "False"
      type: Succeeded

The build pipelineRun (devfile-sample-python-basic-ebgn-c6fdw) failed but with no failure reason in the logs 🥲

from tekton results:

Oh nice, I forgot I could get the logs for deleted PipelineRuns from tekton results
The errors still don't make a whole lot of sense to me though :/

Timed out when waiting for attestation. Yup, 36 minutes and still nothing https://quay.io/repository/redhat-user-workloads/build-templates-e2e/test-app-870/devfile-sample-python-basic-jya7?tab=tags&tag=latest

Would be have to be re-done on the 0.2 versions of build tasks


Weirdness with day and time in Otherside world

i'm honestly not sure what exactly to say other than when i  try to set the time to midnight (wether using cmi, minecraft's default or anything similar) it attempt to resist my changing the time, this behaviour has only ever been seen with the otherside world. on top of that it's even resistant to the magic that makes it night time here is a video i recorded of it happening
https://www.youtube.com/watch?v=inunafrb-Tw
EDIT: there is also no error logs about it anywhere

The otherside time is locked to be the opposite time of the main world.
If you'd like to turn this off you can turn this off with /mconfig world otherside_blocks synchronize_time false
That said, I think it's supposed to handle this by changing the time in the main world if you chain time in the otherside world, so I will leave this open until i decide if that's true. I may have changed my mind on that since it might be surprising and annoying.

thank you so much i have been looking for a fix for this XD


Issue: Help to make a callback for Elevenlabs API streaming endpoint

Hi,
I'm hoping someone smarter than me can help me understand how to write a callback that works with Elevenlabs.
I'm trying to get Elevenlabs to stream TTS based on a response from the GPT-4 API. I can do this easily using OpenAIs own libarary, but I cannot figure out how to do this using langchains callbacks instead.
Here is the working code for the OpenAI library (without the various imports etc).
def write(prompt: str):
    for chunk in openai.ChatCompletion.create(
        model = "gpt-4",
        messages = [{"role":"user","content": prompt}],
        stream=True,
    ):
        content = chunk["choices"][0].get("delta", {}).get("content")
        if content is not None:
            yield content
            print("ContentTester:", content)

promtp = "Say a long sentence"

text_stream = write(wife)

audio_stream = elevenlabs.generate(
    text=text_stream,
    voice="adam",
    stream=True,
    latency=3,
)

output = elevenlabs.stream(audio_stream)

I think I need to use an Async callback, but I can't get it to work. I've tried simply adding the following custom callback, but it doesn't work.
class MyStreamingResonseHandler(StreamingStdOutCallbackHandler):
    def on_llm_new_token(self, token: str, **kwargs: Any) -> None:
        yield token

The idea is to replicate what this part does as a callback:
        content = chunk["choices"][0].get("delta", {}).get("content")
        if content is not None:
            yield content
            print("ContentTester:", content)

This is probably a trivial thing to solve for someone more experienced, but I can't figure it out. Any help would be greatly appreciated!!

@ChristianEvc , I will be happy to have a look at it. Please give me some time.

Just one thing did you try see the following: https://python.langchain.com/docs/modules/callbacks/custom_callbacks

Could also use model.stream()

from langchain import chat_models, prompts
from langchain.schema.output_parser import StrOutputParser

import elevenlabs

prompt = prompts.ChatPromptTemplate.from_messages(
    [
        ("system", "You are an operetic singer who sings all responses."),
        ("human", "{input}"),
    ]
)

chain = prompt | chat_models.ChatOpenAI() | StrOutputParser()

def synthesize_voice(request: str):
    text_stream = chain.stream({"input": request})
    audio_stream = elevenlabs.generate(
        text=text_stream,
        # voice="adam",
        stream=True,
        latency=3,
    )
    return elevenlabs.stream(audio_stream)

This works fo rme

@keenborder786 thanks very much, this look promising! I’ll take a look and let you know hownit goes.
@hinthornw thanks for the suggestion as well, I’ll give this a shot too. I assume this would work with any chain type? Or only specific chains? I’ll be using a ConversationalChain :)

@keenborder786 Just looked through your link in detail now, and I have looked at that, but can't get it to work as a stream. It always waits for the output from the chain to complete before the audio starts.
@hinthornw I have yet to test your suggestion, but am about to shortly.

@hinthornw I'm afraid I can't get yours to work. Which version on you on? The "from langchain.schema.output_parser import StrOutputParser" doesn't seem to exist anymore, and there is no Output parser called StrOutputParser anymore it seems. Any ideas?

From the langchain documentation, the best way would be:

instantiate the OpenAI model as a langchain class with ChatOpenAI
from this langchain ChatOpenAI model, call either
a. stream for synchronous streaming
b. astream for asynchronous streaming

No need for a callback manager with this solution.


Add notice on nextjs about bumbag version

Not sure if doc worthy, maybe underlying problem can be fixed  otherwise
related to #19

In the issue, you mentioned that you still got the warning with both packages on 1.7.5. Is this still the case?


Clarify which manuscript

In some cases multiple manuscript can be conceived from a single dataset.

I am supportive of this change. @dhimmel do you want to review?


corrected outdated openshift link

Docs Build status updates of commit 6e54b9a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/azure-monitor/containers/container-insights-onboard.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@bwren
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Release 3.2.0

Can someone please publish the v3.2.0 release to npmjs.com? I don't seem to have permission:
npm error 403 403 Forbidden - PUT https://registry.npmjs.org/generator-jhipster-quarkus - You do not have permission to publish "generator-jhipster-quarkus". Are you logged in as the correct user?
npm error 403 In most cases, you or one of your dependencies are requesting
npm error 403 a package version that is forbidden by your security policy, or
npm error 403 on a server you do not have access to.

The npm publish --access public command failed in GitHub Actions too.

Nevermind, I figured out how to add myself as a member, and the release is now published.
https://www.npmjs.com/package/generator-jhipster-quarkus


unable to install

components/jqueryui was updated to 1.12  which prevents install.

Is there a reason why to require  "components/jquery": "^1.8" instead of something more recent (e.g., ^3.2)?

Well i think we can use less strict coinstraints, something like >=1.8

Something like that would be great! I would be conservative with >= and maybe use something like >=1.0 <4.0 instead. but I'm not so good with composer's versioning.

Tagged as 2.2, you can update deps to obtain most recent versions of component and related jQuery libs.


Generate Sphinx Docs automatically with Github Actions

Closes #340.

@jeff-regier Can you change the settings to point the Github pages to the "master-sphinx" for documentation?

The action successfully builds the documentation and places it into the "master-sphinx" branch. I'm going to delete the part that triggers on pull request now so it will only happen on commits to master.

Derek, I just made you an admin for the bliss repo. Will you make the change?

Derek, I just made you an admin for the bliss repo. Will you make the change?

Yes, I was able to make the change. Thank you!

This will also close #350 by adding a button to the readme.


Fatal error: Uncaught Error: Class "Inc\Base\Activate" not found in

Hi!  Sir
When i activate this plugin. I found this error. Please tell me solution.
Fatal error: Uncaught Error: Class "Inc\Base\Activate" not found in C:\xampp\htdocs\wpschool\wp-content\plugins\alecaddd-plugin\alecaddd-plugin.php:46 Stack trace: #0 C:\xampp\htdocs\wpschool\wp-includes\class-wp-hook.php(292): activate_alecaddd_plugin('') #1 C:\xampp\htdocs\wpschool\wp-includes\class-wp-hook.php(316): WP_Hook->apply_filters('', Array) #2 C:\xampp\htdocs\wpschool\wp-includes\plugin.php(484): WP_Hook->do_action(Array) #3 C:\xampp\htdocs\wpschool\wp-admin\plugins.php(193): do_action('activate_alecad...') #4 {main} thrown in C:\xampp\htdocs\wpschool\wp-content\plugins\alecaddd-plugin\alecaddd-plugin.php on line 46

Did you run compose install?
I see you're using XAMPP, which I'm not sure supports composer.
Also, this is not a ready to use plugin but it's a repository for the YouTube tutorials I did to show how to build a plugin with PHP OOP.


Building assets with Grunt/Gulp

Please, help me to figure out what I can do to make dokku-alt start building my static assets with Gulp (or Grunt) after push, but before ./manage.py collectstatic (I am deploying Django app).
Can I use some plugin or I need to write a hook for myself? It seems to be a common task and I think that I just missing it in the docs. Thanks!

Solved the problem with the following configuration:
.env:
export BUILDPACK_URL=https://github.com/heroku/heroku-buildpack-python.git

.buildpacks:
https://github.com/fedosov/heroku-buildpack-nodejs-gulp-bower
https://github.com/heroku/heroku-buildpack-python.git


Time complexity (execution time)

It's clear for me why .fit() time complexity (execution time) depends on time series length. Probably O(N^2). But why it's true for predictN(n=1) as I found out ? It looks like dependence is O(N^2)..O(N^3)
Thx

Ok, I found a possible bottleneck which could be the predictModel = deepcopy(self) line within predict method ( pydlm/dlm.py ).  It seems to me this is far from optimal  in terms of memory space and CPU.

The fit() should be an O(N) complexity. Yes, the predictN() could be much improved. The reason that the deepcopy exists in the current implementation is because my original implementation of predictN() modifies the model in-place, i.e., it changes the model status for prediction purposes. This implementation is fast but brought many problems. The mutability makes it hard to track the model status. Once the model is called to make a prediction, it can no longer accept any new data for continuous filtering unless the last status was memorized somewhere. What could be worse is that if the predictN() is called at some middle time point, it might modify the latent states and mess up the results.
A quick mitigation to this is to ask the clients to copy the original model when they want to predict. Then the copied model can be modified and used for prediction purposes while the original model can continue work as it is. This is why the deepcopy appears in the code.
I've started a new repository called pydlm-lite, in which the structure of dlm class will be redesigned. The goal is to make the model immutable, while the model status and data should be passed in/out together to avoid any issue. The complexity improvement will likely be seen there.

Thank you for your clear and helpful explanation.
I try to use pydlm for "online" predictions (add data, predict 1step ahead , add data,predict 1step ahead, etc).
As deepcopy copies not only model state but also continuously growing raw data (main time series & feature time series)  it takes more and more time ... Very rapidly.
Refactoring is greatly anticipated.  However in the short term,  is there a simple way to avoid\exclude unnecessary array copying (main time series & feature time series)?
Thx

Hi,
I was experiencing a similar issue (following a similar use case as buhbuhtig). Calling several times in a row predictN() seems to cause an exponential increase in the running time of the method.
This time complexity increase is actually coming from the combination of deepcopy(model) and self._predictModel = predictModel in the predict() method
The fact that self._predictModel is not reset to None at the end of the predict() or predictN() method is causing the deepcopy of the consecutive calls to increase exponentially in size (by copying a model that has a self.predictModel that has itself a self.predictModel, etc)
Setting self._predictModel to None should help solve part of the problem I think


Playlist Reordering

Currently the Mobileclient.reorder_playlist_entry() method operates on one entry at a time and passes back a ClientID to the Google API. Unfortunately, it appears that calling the method on single entries leads to random behavior, possibly the Google API expects them to be done in batches so the ClientIDs are out of date after the first call. I tested it by creating a simple loop to sort a playlist one by one and even with 10 songs, sometimes the reorder would fail to work.

Oh, I had forgotten there was a super old issue for this: https://github.com/simon-weber/gmusicapi/issues/185. I've already got the relevant links to Autoplaylists over there, so I'm going to merge this ticket into it.


How to disable elements in system

I'd like to remove the custom domains section on the homepage (links to pricing page) but I don't see how to do this. Is there a list of elements to add to config.yaml to disable certain features?
Thanks

The custom domains are meant to be disabled by default. Here's the setting in the yaml config:
https://github.com/onetimesecret/onetimesecret/blob/bcd5525fb4445e8dc5f1125adc38029a9888e712/etc/config.example.yaml#L1-L5
So just running the app/container out of the box "shouldn't" be showing anything about the custom domains. But if they are appearing then that's a bug.
Side note, the authentication settings may be helpful for you as well:
https://github.com/onetimesecret/onetimesecret/blob/bcd5525fb4445e8dc5f1125adc38029a9888e712/etc/config.example.yaml#L26-L43

Thanks for the info. Yes, I had to change to :enabled: false in order for it to work :)


Updating index.json to include new version of dms extension

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

If this PR is for a new extension or change to an existing extension, use the following to try out the changes in this PR:
docker run -it microsoft/azure-cli:latest
export EXT=<NAME>
pip install --upgrade --target ~/.azure/cliextensions/$EXT "git+https://github.com/vchske/azure-cli-extensions.git@master#subdirectory=src/$EXT&egg=$EXT"

Please fix the CI, there were problems in the code previously committed.

I don’t know why this is failing now. I added a commit that removed the invalid parameters from _help.py (also a little confused why this wasn’t flagged before) but it still fails with the same errors.
From: Willie Xu<EMAIL_ADDRESS>Sent: Friday, November 30, 2018 1:39 PM
To: Azure/azure-cli-extensions<EMAIL_ADDRESS>Cc: Chuck Skelton<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Azure/azure-cli-extensions] Updating index.json to include new version of dms extension (#436)
Please fix the CI, there were problems in the code previously committed.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-cli-extensions%2Fpull%2F436%23issuecomment-443347469&data=02|01|Charles.Skelton%40microsoft.com|bb6e6f3cb0784a602c1808d6570c31d0|72f988bf86f141af91ab2d7cd011db47|1|0|636792107227605206&sdata=nJnszt%2FT%2F4Ta5mCXerHYU7QGQcQuJxGum3d8VRqHTIc%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAYoW8ffEfKuqG5TF60O7otSPOz0NlBJrks5u0aVggaJpZM4Y8XS3&data=02|01|Charles.Skelton%40microsoft.com|bb6e6f3cb0784a602c1808d6570c31d0|72f988bf86f141af91ab2d7cd011db47|1|0|636792107227615215&sdata=eKjGn4YsT04sSfmvCy7%2FpHY6bwieiFl6IApoxFXrYFo%3D&reserved=0.


import_step not working

Ran code to import step file (2 solids)
import build123d as bd
import stellarmesh as sm
geometry = sm.Geometry
geometry.import_step(filename='blanket_both.stp', material_names=['SiC', 'breed'])
mesh = sm.Mesh.mesh_geometry(geometry, min_mesh_size=10, max_mesh_size=50)
And received error telling me geometry has no solid attribute.

First, if you have a STEP file already, you don't need to import build123d.
For the import_step call, try
geometry = sm.Geometry.import_step(filename='blanket_both.stp', material_names=['SiC', 'breed'])

Closing as this issue was discussed and presumably resolved at https://openmc.discourse.group/t/error-meshing-cad-geometry/3472/8


use again schema.org types

https://github.com/google/schema-dts

they also have a lot of tooling https://opensource.google/projects/schema-dts
the reason i removed this is that the conversion ts -> json didn't work well

I think the conclusion was, unfortunately, to not go down this route but to stick to something simpler.


Missing support for multiple http requests with same url and args

I have a polling loop asking the same GET N times waiting for an event. I would like to respond with the same sequence of responses it was giving me in the recording step.

Unfortunately, if you are using the exact same parameters, headers, etc. (if each recorded request is the exact same), there is no way to differentiate the requests. EasyVCR returns the corresponding response of the first request that matches in the cassette.
If you have a proposed solution to this, we'd be more than happy to explore it and implement it if it works.

Closing due to inactivity. Feel free to re-open.


Data Preview fails when SQL Server input table schema is not dbo

System Information (please complete the following information):

Model Builder Version: 16.0.1909.2101
Visual Studio Version 15.9.13

Describe the bug
Data Preview fails when SQL Server input table schema is not dbo
To Reproduce
Steps to reproduce the behavior:

Go to '2. Data'
Click on 'Input and choose SQL Server'
Click on 'Table Name' and choose a table that you know does not have a database schema of dbo
See error in screenshot.

Expected behavior
Data Preview will work.
Screenshots

Try to continue gracefully - if it isn't expensive.
Fall back if "difficult" - error message

Experiencing the same problem @dporterf  reported.

Validated with guest schema


Glicko 2

Why are we using glicko-1 still?
also, didn't @Zarel write a php glicko 2 lib a while back?

Yes, Glicko-2's formulas were found to be buggy so we moved back to Glicko-1.


New TEPs (target enabling packages) not yet in our Platform

While preparing some training materials, I realised a few additional genes have got a TEP (according to https://www.thesgc.org/tep) but this information does not seem to have been updated in the index files for the corresponding target profile page. These are the 3 genes that have been updated in 2018:

RIPK2: https://www.targetvalidation.org/target/ENSG00000104312
HAO1: https://www.targetvalidation.org/target/ENSG00000101323
FAM83B: https://www.targetvalidation.org/target/ENSG00000168143

It'd be useful to check when a new TEP gets updated on the SGC side, so that we automatically fetch that information and display in in the Open Targets Platform.

@luca Can you add to
https://github.com/opentargets/webapp/blob/893fcb951caaf312d86447f952e66256fd9c537d/app/src/services/teps-service.js ?

We don't have an automated way to do this because it would involve scraping their pages, and the number is low

My plan is to start checking the SGC Chemical Probes page before each release and we can include a check for new TEPs.

This is fixed in master and will go in the upcoming 3.8 release; close after new release QA.


Autogenerate & Server API Doc for JavaScript and Go

In Go, we should just use Godoc, obviously. In JavaScript/Node I'm sure there are tools for this -- probably tons of them, all slightly broken and different.

Once repo became public godoc works - https://godoc.org/github.com/attic-labs/noms. We still have work to do in order to make the content better - and of course we need a JS equivalent.

I don't think we need a bug for "make content better" . That can just be
ongoing task. I'm going to retarget this bug to just JS.
On Wed, Aug 3, 2016 at 6:30 PM, Mike Gray<EMAIL_ADDRESS>wrote:

Once repo became public godoc works -
https://godoc.org/github.com/attic-labs/noms. We still have work to do in
order to make the content better - and of course we need a JS equivalent.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/attic-labs/noms/issues/1471#issuecomment-237425220,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAE6BNPureLC1LDi_AHYfN34FIMqCNSXks5qcUDNgaJpZM4Ibff5
.

Some update on the JS front. I've been contributing to documentation.js and there is one important feature missing. I have a PR for that: https://github.com/documentationjs/documentation/pull/490. With that done we have "a complete" documentation.
We then need to setup a build that generates the HTML and deploy that somewhere.
After that I think we need to tweak the generated HTML quite a bit.

The documentation.js tool is "good enough" to move on.
For deployment I could set up another ec2 instance or maybe we should just store these in s3 (since these are just static files).
@cmasone-attic @zcstarr @mikegray What do you think?

Probably makes sense to take advantage of AWS CloudFront in front of S3 to serve these if they are all static.

agreed

agreed as well , I think @mike is right s3 + aws cloudfront sounds smooth

We now automatically deploy to https://docs.noms.io/js/
I'm closing this. Lets open more specific bugs for things that needs to be improved.

So much awesome!


[wallets] Add OIDC login for magic link

@MananTank add changeset and ship?

/release-pr

@jnsdls Not merged because it's not clear if this is the required implementation


Expose idle shutdown time as script parameter

FYI: the min and max range come from the values accepted by the Windows agent's idle shutdown service.

@bdyck, @cptera , now won't automation have to wait 240 minutes?

@bill-dall No, we'll update the automation to use this parameter and then it will take whatever that value is.

@bdyck - But not for admin-rw, then?

@bill-dall It will also pick up that value.


Improved API

Thanks for your PR.
Registering the nodes works fine.
But there are problems with the crafting recipes. They don't work anymore. Please fix this.
Thanks in advance.

Fixed

Crafting for tiny tables still doesn't work.
They are crafted out of two benches.

Fixed.

:+1: Looks good.