🐛 BUG: WASM program with WASI exit status doesn't propagate to http status code of worker

Which Cloudflare product(s) does this pertain to?
Workers Runtime, Wrangler core
What version(s) of the tool(s) are you using?
wrangler 0.0.0-c0d7699
What version of Node are you using?
20.5.0
What operating system are you using?
WSL 1 on Windows 11
Describe the Bug
The HTTP status code always 200 even where the WASM/WASI program exit with the system status of 1 (not zero), or where the program stopped unexpectedly.
2:19:03 AM POST / 200

✘ [ERROR] Uncaught (in promise) RuntimeError: unreachable
      at __rust_start_panic
  (wasm://wasm/008579b6:wasm-function[224]:0xe4d9)
      at rust_panic (wasm://wasm/008579b6:wasm-function[215]:0xe29a)
✘ [ERROR] Uncaught (async) CompileError: unreachable
✘ [ERROR] Uncaught (in promise) RuntimeError: unreachable

A definately success request and response case are:
curl -X POST localhost:8787 --data '{"x": 7, "y": 9}' -v

> HTTP/1.1 200 OK
> ...
> You typed: Point { x: 7, y: 9 }

Awkward request and response case are:
curl -X POST localhost:8787 --data '{"x": 7, "y ' -v

> HTTP/1.1 200 OK  <-- it should be like 404 or 500 whatever related with error code though....
> ...
>                  <-- because it was not responsed

Please provide a link to a minimal reproduction
https://play.rust-lang.org/?version=stable&mode=debug&edition=2021&gist=635c9deb39ea0db13019d6b813b189f4
Please provide any relevant error logs
✘ [ERROR] Uncaught (in promise) Error: proc_exit=1

      at WASI.proc_exit_fn (index.js:748:9)
      at __wasi_proc_exit
  (wasm://wasm/0085943e:wasm-function[252]:0x119cd)
      at _start (wasm://wasm/0085943e:wasm-function[6]:0x478)
      at WASI.start (index.js:543:9)

✘ [ERROR] Uncaught (async) Error: proc_exit=1

2:48:47 AM POST / 200

Note: Unnecessary use of -X or --request, POST is already inferred.
*   Trying <IP_ADDRESS>:8787...
* Connected to localhost (<IP_ADDRESS>) port 8787 (#0)
> POST / HTTP/1.1
> Host: localhost:8787
> User-Agent: curl/7.81.0
> Accept: */*
> Content-Length: 12
> Content-Type: application/x-www-form-urlencoded
>
* Mark bundle as not supporting multiuse
< HTTP/1.1 200 OK
< date: Wed, 02 Aug 2023 17:43:30 GMT
< report-to: {"endpoints":[{"url":"https:\/\/a.nel.cloudflare.com\/report\/v3?s=<sth_value>"}],"group":"cf-nel","max_age":604800}
< nel: {"success_fraction":0,"report_to":"cf-nel","max_age":604800}
< vary: Accept-Encoding
< cf-ray: <sth_value>
< alt-svc: h3=":443"; ma=86400
< server: cloudflare
< Connection: keep-alive
< Keep-Alive: timeout=5
< Transfer-Encoding: chunked
<
* Connection #0 to host localhost left intact

Hi @SebastiaanYN! Can you provide a minimal reproduction of your worker code? The minimal reproduction provided is only the rust code and is missing your Workers setup. Also, I noticed you're using a beta version, can you try on Wrangler's latest version please?

Hi @SebastiaanYN 👋  We haven't heard from you in a while, so we’re going to close this issue for now. If this is still a problem, please feel free to open a new issue referencing this one. Please be sure to include an updated summary and any additional requested details, followup, or context and we will revisit and investigate further. Thanks!


Unable to find Microsoft.NuGet package

Brief description of your issue
Unable to find Microsoft.NuGet package neither install it
Steps to reproduce
C:\>winget search nuget
Name                 Id                                        Version
-----------------------------------------------------------------------
NuGetPackageExplorer NuGetPackageExplorer.NuGetPackageExplorer <IP_ADDRESS>

C:\>winget install -e --id Microsoft.NuGet
No package found matching input criteria.

Expected behavior
Able to install nuget.exe or remove package https://winget.run/pkg/Microsoft/NuGet
Actual behavior
No package found matching input criteria.
Environment
C:\>winget --info
Windows Package Manager v1.0.11451
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.19042.1052
Package: Microsoft.DesktopAppInstaller v1.11.11451.0

Logs: %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\DiagOutputDir

Links
--------------------------------------------------------
Privacy Statement   https://aka.ms/winget-privacy
License Agreement   https://aka.ms/winget-license
Third Party Notices https://aka.ms/winget-3rdPartyNotice
Homepage            https://aka.ms/winget

It was removed in https://github.com/microsoft/winget-pkgs/pull/16731* and the decision to remove it was based on https://github.com/microsoft/winget-pkgs/issues/7855.

PR's description: This is not an application you can install; it's a command-line utility meant to be used within Command Prompt or PowerShell; and the Windows Package Manager currently doesn't allow downloaded files to be moved somewhere else to take advantage of it.


MISRA compliance changes for FreeRTOS_TCP_IP.c

Description
This PR adds changes related to MISRA correction in FreeRTOS_TCP_IP.c file along with config files.
Some corrections required code rework which has been done and tested Using Full_TCP test suite, Full_TCP_Extended and Full_FreeRTOS_TCP test suite.
Remaining MISRA warnings are regarding:

Pointer conversions which are essential to map data buffers to various structs.
Use of #includes after some c-code. These #includes are used in "packing" the structs.
Use of Unions.

Note to reviewers: GitHub has difficulty figuring out the block and indentation changes. While reviewing please enable the option "Hide whitespace changes" in settings or click here.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/bot run checks


ssl test not pass

Linux debian 3.16.0-4-amd64 #1 SMP Debian 3.16.36-1+deb8u2 (2016-10-19) x86_64 GNU/Linux
gcc version 4.9.2 (Debian 4.9.2-10)
OpenSSL 1.0.1t  3 May 2016
./configure --enable-ssl
make
make check
====================================
   libmill 1.16: ./test-suite.log
====================================

# TOTAL: 18
# PASS:  17
# SKIP:  0
# XFAIL: 0
# FAIL:  1
# XPASS: 0
# ERROR: 0

.. contents:: :depth: 2

FAIL: tests/ssl
===============

lt-ssl: tests/ssl.c:72: main: Assertion `ls' failed.

Can you check what the errmo is set to?

lt-ssl: tests/ssl.c:73: main: Assertion `ls' failed.
errno: 2

ENOENT - that looks like cert files are not present. Are you by chance running the tests from directory other than project's root directory?

Thank you, the pem files are not exists because I download it from [website].(http://libmill.org/download.html)
Then I make it success through git  :)
apt-get install libtool openssl libssl-dev
git clone https://github.com/sustrik/libmill.git
cd libmill
./autogen.sh
./configure --enable-ssl
make
make check
make install
ldconfig

============================================================================
Testsuite summary for libmill 1.16-12-g7766b64
============================================================================
# TOTAL: 18
# PASS:  18
# SKIP:  0
# XFAIL: 0
# FAIL:  0
# XPASS: 0
# ERROR: 0
============================================================================

Right. The files are missing in the tarball. I'll fix that.

Fixed.


feat(filter): add seata samples.

Could you please also provide a new sample using the new API?

Could you please also provide a new sample using the new API?

Meet some problems when writing this new sample. The service in this sample needs to utilize some structures and methods from seata-go, making it difficult to define the service interface using IDL. I asked my mentor, he mentioned that the non-IDL API have been consulted, but it hasn't been completed yet. Therefore, we are currently discussing the possibility of pre-writing the corresponding methods in the seata module of dubbo-go and deploying them via configuration.

I see, thanks for the clarification.

@iku50 Please create a new directory under the root directory of this project called transaction or data-consistency and move all the codes there. Then open a new pull request.
Please keep the code in compatiblity directly not changed.


FEAT: Lora multi support

Please add some screenshots and comments to explain what model and which lora(s) you used and what effect you achieved.

Also, you need to modify the releated documents: https://inference.readthedocs.io/zh-cn/latest/models/lora.html

Test for multi lora

The model gemma-it-2B, using both 1rsh/gemma-2b-it-fr and alfredplpl/gemma-2b-it-ja-poc, yields the following results:


Run prettify_attr_value on values returned from Attribute::value()

This is trickier than it should be because Attribute is Endian-less.  We should probably fix that, many of the subtypes of AttributeValue need an Endianity to be interpreted correctly.

Easy to keep track of the endian with a PhantomData, if needed.

Yes. We'd probably want to convert all the [u8]s in AttributeValue to EndianBufs too.

Yeah, that is probably the most correct thing to do.

@khuey Have you started on this yet? If not, I'll do it, because I need it for dwarfdump.

I have not.


Stride of TorchSparse Conv3d is no effect?

Hello,
I have a problem about Stride of TorchSparse Conv3d. Through experiments, I verify that stride(2) does not have scale influence on the output coordinates. In other words, STRIDE does not work in TrochSparse. For example, spnn.Conv3d(32, 64, 2, stride=2)

look

In addition, it is not the correct coordinate range output of subsample even set the k2s2. As mentioned before Conv3d with kernel_size=1 and stride=2 doesn't subsample the input tensor and Why not constraining the max coordinates during downsampling?
So, at the moment, I suspect that stride > 1 for spnnConv3d is not valid.

Hi @haiduo,
In our implementation actually stride > 1 changes the resolution of the coordinates, instead of the scale of the coordinates. After n downsampling layers, all the coordinates will be a multiple of $2^n$. Note that this is equivalent to reducing the coordinate range by 2$\times$ at each downsampling layer.
Best,
Haotian

Hi @haiduo,
In our implementation actually stride > 1 changes the resolution of the coordinates, instead of the scale of the coordinates. After n downsampling layers, all the coordinates will be a multiple of 2n. Note that this is equivalent to reducing the coordinate range by 2x at each downsampling layer.
Best, Haotian

Thank you for your reply!
I know mean what you said. Maybe I used an error word about "scale", but I want to say means what you said. That is stride > 1 changes the resolution of the coordinates, but I found is not correct about the output resolution of coordinates through experiments.
Specifically, I should use TorchSparse to replace the 3DConv layer of VoxelNet. After two times of downsampling (stride=2), it was found that the coordinate resolution did not reduce the corresponding multiple. Because there needs to be a Voxel2dense operation (for example, Torch. sparse_COO_tensor) for subsequent RPN processing. But I found that the resolution of the generated Dense tensor was not downsampled relative to the input coordinates.
I don't know if I made myself clear.
Finally, thank you again for your reply.

Since TorchSparse has been upgraded to v2.1.0, could you please attempt to install the latest version? I will now close this issue, but please don't hesitate to reopen it if the problem persists.


Implement the ByteBufferSerializer interface

This PR implements Artery's ByteBufferSerializer interface and adds test coverage. References #112.
The most optimized case is direct serialization/deserialization into/from the ByteBuffer. If one or more transformations are applied, Array[Byte] representation is used for the intermediate steps and the ByteBuffer only at the periphery.
Public API changes:

Deprecated DefaultQueueBuilder.build in favor of a future-proofed generic DefaultQueueBuilder.build[T]

Thanks in advance for your reviews and feedback.

Thanks! Do you publish snapshots and/or will release 2.2 soon?

@armanbilge We will publish an RC for 2.2. I should be able to do that today...


Display Scrollbar in Markdown Widget if needed

We have a markdown widget on our dashboard, but the content extends past the defined widget size.  I would have assumed that a scrollbar would appear when this happens, but instead it's simply cutting the overflow off.

Is there an option to display a scrollbar on the widget?

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

ID: b21c46ee-e19c-9dc4-11c2-8749c6576925
Version Independent ID: 09ad6f7f-5169-f1b5-4f71-410bd85c62df
Content: Add markdown content to a team dashboard - Azure DevOps
Content Source: docs/report/dashboards/add-markdown-to-dashboard.md
Product: devops
Technology: devops-analytics
GitHub Login: @KathrynEE
Microsoft Alias: kaelli

@DaveSlinn Consider these resources to address your question:

Azure DevOps Services
Azure DevOps Support Bot
Azure DevOps on Stack Overflow

If this is a feature that should be added to the product, create a dev issue. To make sure it gets in front of the right people, please submit your bug here.


Invalid default map group

The default map group, mp_active, is either misspelled or otherwise incorrect.
Error message emitted by CS:GO:
Setting mapgroup to 'mp_active'
ApplyGameSettings: Invalid mapgroup name mp_active

The map group is misspelled and should be mg_active

fixed in f232af0d97dd0c50477b17dfcdab87df21200a82


Let choose_action return an action and let self.args.action contain parsed actions

I think it would be nicer to keep self.args.action it's value unchanged, so it's truly always the parsed argument. We could return the chosen action from choose_action instead and return the parsed action if no return_code match is found. I feel like this way it's a bit more clear where the chosen action comes from.

No, this is by design: self.args.action should always be the go-to value for the action. If we now have to possible values for action, it's a lot easier to mix them up, but there is no advantage (as we never need the default value separately).
Additionally, having it as a field removes some boilerplate with variable passing.


Promote {rocky, almalinux, oracle, centos-stream}-9 from experimental

EL8 distros do not work on M1:

https://github.com/lima-vm/lima/issues/841

So I'm planning to promote {rocky, almalinux, oracle, centos-stream}-9 from experimental:
e.g., experimental/rocky-9.yaml -> rocky.yaml
The old rocky.yaml will be renamed to rocky-8.yaml, or deprecated/rocky-8.yaml if we want to deprecate EL8 now.

Would it be possible to get some better naming for these, that doesn't change so much ?
The yaml files can be moved around, just thinking something like rocky:9 or similar

The yaml files can be moved around, just thinking something like rocky:9 or similar

Maybe we can just ln -s rocky-9.yaml rocky so that both template://rocky-9 and template://rocky will work.
For windows support in the future, : can't be used in a template name.

I was more thinking of metadata or a mapping table, but it is a different topic so will continue elsewhere.


Generic back button

I'm building off of pull request #5093. I'm submitting this as a separate pull request because i'm changing the public API of React-Router-Native. The reason I'm changing the public API is because of React-Native 0.44 the BackHandler now handles the back button on Android AND tvOS.

This would be a breaking API change. Perhaps re-export it as BackAndroidButtonto maintain compatibility and show a deprecation warning when it's used?

Closing in favor of #5295


Update UI devkit docs for an Extension as a top-level product

Summary
Fixes #8310

Adds the necessary documentation to cover the usecase of Extensions as a top-level product
Small lint suggestion fix for RadioButton component

Note: for the docs I replaced the naming of initializing-extensions to extensions-configuration and added the new content there

Also updated the shell package and made sure the code snippets worked with the whole flow of create @rancher/app and create @rancher/pkg

Closed in favour of https://github.com/rancher/dashboard/pull/8673


Make docstring indentation consistent + some docs rewording

Docstrings are inconsistently indented, so picked one.
Is there a reason we don't use ocamlformat?

Docstrings are inconsistently indented, so picked one.
Is there a reason we don't use ocamlformat?

We do use ocamlformat. Maybe it just doesn't process doc blocks.
https://ocaml.ci.dev/github/ocurrent/ocaml-dockerfile/commit/b6706540a5917b1eda1dabf0930d5ea59c14a2f5/variant/(lint-fmt)


static binaries please

Hi,
kind request to add static binaries in the releases (via github workflow).
one cosmetic bug:
go install github.com/aydinnyunus/PassDetective@latest
go: github.com/aydinnyunus/PassDetective@latest: <EMAIL_ADDRESS>parsing go.mod:
        module declares its path as: main.go
                but was required as: github.com/aydinnyunus/PassDetective

Hi @SkyperTHC,
Thanks for your request. Can you try again ? I released v1.0.3 with Goreleaser.

@aydinnyunus sadly, the binaries aren't static:
https://github.com/aydinnyunus/PassDetective/releases/download/v1.0.3/PassDetective_1.0.3_linux_amd64.tar.gz
$ file PassDetective
PassDetective: ELF 64-bit LSB executable, x86-64, version 1 (SYSV), dynamically linked, interpreter /lib64/ld-linux-x86-64.so.2, Go BuildID=C6LEV8H6Vyw4SOhNZEyq/Olo39mRBbHGIttMs_2l3/kSyVv99_3P13ePRN3BgK/TPNPbVo5mHHDWRu55Ul9, stripped

you might have to use some flags, I can successfully build a static binary:
$  pushd "$(mktemp -d)" && git clone "https://github.com/aydinnyunus/PassDetective" && cd "./PassDetective"
$  CGO_ENABLED=0 go build -v -ldflags="-s -w -extldflags '-static'"
$  mv "./PassDetective" "$HOME/bin/passdetective" ; popd ; go clean -cache -fuzzcache -modcache -testcache

$ file PassDetective
PassDetective: ELF 64-bit LSB executable, x86-64, version 1 (SYSV), statically linked, Go BuildID=2Fqw3y4cO8WOVN2wTfCJ/Tsoh7SliuTsnqppcVsg6/D0Ypu2k_ekjoIkubPhXV/Atj9C2waK2g1Dp6mj9et, stripped

Hi @Azathothas ,
Thank you for your advice. I add .goreleaser.yaml file with provided ldflags and unset CGO_ENABLED.  I think now it is fixed.
➜  Downloads file PassDetective
PassDetective: ELF 64-bit LSB executable, x86-64, version 1 (SYSV), statically linked, Go BuildID=VcZRjmVvK4i41z8hLHe_/mU_B8eo97qqlldVEJDA8/cQAgQFnmith10cXa0E6J/vPG_baHrXB4GRnZ-ZQOA, stripped


Shaking angle of view in 3D scene, model with ghosts

Godot version:
godot3.1 alpha3
OS/device including version:
manjaro gnome x64
Intel® Core™ i3-2350M CPU @ 2.30GHz × 4
Intel® Sandybridge Mobile
OpenGL ES 3.0 Renderer: Mesa DRI Intel(R) Sandybridge Mobile
intel hd 3000
Issue description:
The ground model（a built-in box model in the meshinstance） has ghosts when shaking angle of view  in this scene.
Also, the ghosts  appears to have a collision volume that controls the character to move over it until it disappears and does not fall.
I tried to take a screenshot, but the picture was normal.
The following pictures show some differences from the actual effect：

Steps to reproduce:
Create a 3D scene, place the model, and then shaking the angle of view
Minimal reproduction project:
Test.zip

The RigidBody is not meant to be rotated / moved manually. You should control its position with forces. Check the documentation to get more info about it

The RigidBody is not meant to be rotated / moved manually. You should control its position with forces. Check the documentation to get more info about it

Although I would like to use torque and force to achieve role steering and movement, but they are currently unable to achieve direction control based on their own axes(Not similar to add torque or force on local )


www/nginx Load naxsi core rules / Learningmode working

Added an option in Global HTTP settings to load naxsi core rules and fix the problem with learning mode not working.
TODO: check if the core rules exists / make it work with some routine that update this rules.

I have to reject it for license reasons - the core rules are GPL. That's the reason why they cannot be included. See #891 for my alternative.

The GPL rules are not linked to a binary, are they? Or do you mean GPLv3?

On 7. Oct 2018, at 08:48, Fabian Franz BSc<EMAIL_ADDRESS>wrote:
I have to reject it for license reasons - the core rules are GPL. That's the reason why they cannot be included. See #891 for my alternative.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@fichtner File is under LGPL: https://github.com/nbs-system/naxsi/issues/423#issuecomment-394407953

LGPL is perfectly fine. If you are worried we can have a separate nginx-naxi-rules plugin so it's more clear to the end-user?

@fichtner this one is in my opinion obsolete because #891 loads the core rules into the configuration and after assignment to the location you have the same result in this issue with the exception that my version is configurable and more flexible. It also does not require to hardcode any rules.

Ok, closing due to #891.

Thank you guys!


Scripting: Download full-time recordings where at least one motion recording was made

I know this might be more scripting related, but I've playing around and right now I have TBs of video but most of them are "static images on video" ( no movement whatsoever) so I was wondering if there a way to only download the full-time videos where there has been at least one motion recording for that time period.
Let's say we have 3 full-time recording videos:

8:00AM to 8:10AM
8:11AM-8:20AM
8:21AM-8:30AM

If there is a motion recording in the 3rd time period I will only download that video.
The other way around is to locked those videos and then filter through it... but let see which ideas might come to you :D

Yes, this does sound like something that should be handled on the application side, i.e. I don't think it makes sense to include anything specific to this in unifi_video.
To address the question outright, I would simply get the start times for motion recordings and then fetch fulltime recordings and only act on those that that span over any of the motion recordings:
from unifi_video import UnifiVideoAPI
from datetime import datetime

def fulltime_recs_with_motion(uva, since=None):
    '''Get full time recordings that span one or more motion recordings
    '''

    motion_start_times = [
        r.start_time for r in uva.get_recordings(
            'motion', start_time=since)]

    for fulltime_rec in (uva.get_recordings('fulltime', start_time=since)):
        if [m_start_time for m_start_time in motion_start_times
                if m_start_time >= fulltime_rec.start_time \
                    and m_start_time <= fulltime_rec.end_time]:
            yield fulltime_rec

uva = UnifiVideoAPI(...)

for ft_rec in fulltime_recs_with_motion(uva, datetime(2020, 9, 1)):
    print(ft_rec)

(The above is for master. Still no PyPI release that includes get_recordings().)
Personally though, I would just download the motion recordings :) Maybe with a minute or two of margin using UnifiVideoCamera.recording_between() if I was worried about something getting left out.

I'm dealing with this in a different way. I only enable motion-recording on the NVR, and then filter the recordings list to remove overlapping videos.

I'm assuming you got this sorted out @EddieDee. If not, leave a comment.


Deduplicate redundant logging to fix #24

This PR deduplicates the following redundant logging. The messages are shown in console every FilterPageOutput objects creation. They are redundant.
2017-05-17 05:28:35.871 -0700 [WARN] (0001:transaction): JDBC parameter 'caseSensitive' is implicitly set to false as default in
2017-05-17 05:28:35.872 -0700 [WARN] (0001:transaction): embulk-filter-calcite 0.1 but, it's scheduled to change default with true
2017-05-17 05:28:35.872 -0700 [WARN] (0001:transaction): in 0.2. Please use 'options' option to set 'caseSensitive' to false.

@dmikurube Please take a look?

@dmikurube Thank you for reviewing. Merging.


DX: enforce imports to top by default

Temporary fix for #269

This is done already seems like?

See https://github.com/astral-sh/ruff/issues/13025

This could be fixed with https://github.com/ComPWA/policy/pull/392, but it's not the most ideal because you have to list all excepted notebooks with excluded.

CI should be fixed again once https://github.com/ComPWA/compwa.github.io/pull/294 is through. @shenvitor could you review that PR?


Support for Private Terraform Modules

Overview of feature request
Radius can work with publicly-hosted Terraform modules across a few different module sources. We don't support working with privately-hosted Terraform modules, and there's no way to configure authentication.
This is important because organizations write their own Terraform modules and store them in privately-accessible sources. In order for us to support serious use, we need to enable private sources.
The challenge is that each module source has its own authentication mechanism with its own style of configuration. For example the git module source relies on the environment hosting Terraform to provide the credentials.
For right now the set of required sources is TBD.
Acceptance criteria
Other than the "glue" to wire up credentials, the Terraform Recipe driver should behave the same way for a private module. Given the correct configuration of credentials, the Terraform driver should behave the same for a public vs private module.

Radius API allows users to register credentials for some set of Terraform module sources.
TBD set of Terraform module sources are supported.
Documentation updated.

Additional context
It may be helpful to pair this with the work to isolate the Terraform driver in its own container. Some of the source configuration makes changes to the execution environment (for example logging into git).
AB#10652

Triage Team: creating this issue to track roadmap work.

reopening as functional tests and docs still need to be created


Improve travis configuration

testing PHP 7
remove testing against Symfony 2.4/2.5
add testing against newer versions of symfony
switch to the faster container-based infrastructure on Travis
persist the composer cache between builds

Thanks man, you rock!


[Awaiting payment May 14th] [Wave Collect] [Xero] [Import Flow] Create the ConnectionLayoutComponent

You need to build the ConnectionLayoutComponent which will be reused across the Import flow, following the steps outlined here.
This will be worked on by SWM engineers, see Slack discussion here.
Until the Xero authorization flow (NewDot issue)  is done (which entails NewDot, Web-E and IS changes), we won't be able to work on this yet.

@lakchote should someone from SWM be assigned here, and do you need me assigned?

@lakchote should someone from SWM be assigned here, and do you need me assigned?

Double-checking with internal engineers, some might be interested to work on this.

@lakchote 👋 just checking in on this.

@lakchote 👋 just checking in on this.

Was done in this PR by @hungvu193

Deployed to prod two days ago.

Cool, pending payment next week

Payments for the C+ guys on this will be handled centrally when we wrap the project, we can close this now the regression period is over.


feat: alpine support

fixes #490

Provides a node.napi.musl.node for aarch64 & x86_64 musl variants
Prepackaged in pact-js-core, so end users don't need to do anything to enable, will autodetect if running under musl or glibc
Pact ruby standalone, is still glibc based, and still included in the package

In order to use in alpine, users will be required to install bash gcompat libc6-compat for full functionality of the included cli tools (only if using cli tools)

Content-type matching (related to #447) resolved in linux by installed shared-mime-info
Re-enabled all plugin tests in this repo

Plugin tests passing on alpine

Fixes an issue with can-i-deploy output being returned as a buffer array with a single entry with the entire response body, rather than a buffer array with each line.

Amazing - will try and look today Yousaf! 👏

No rush at all dude!

switching this to a draft, as there are a couple of additional things I want to do

switch to musl *.so (smaller)

but few changes I want to make in pact-ref with regards to some linker errors and content-type issues
https://github.com/pact-foundation/pact-reference/issues/416#issuecomment-2113121731
https://github.com/YOU54F/pact-reference/pull/3


请教: 登录成功但list出错，error code 31066

Authorizing, please be patient, it may take upto 300 seconds...
Authorizing/refreshing with the OpenShift server ...
OpenShift server failed, authorizing/refreshing with the Heroku server ...
Successfully authorized
 [01:32:39] ----------------------------------------------------------------
Error 31066
osmc@osmc:~$ python -m bypy list
 [01:33:09] ----------------------------------------------------------------
Error 31066
osmc@osmc:~$ python -m bypy info
Quota: 2.005TB
Used: 849.024GB
osmc@osmc:~$ python -m bypy list
 [01:36:06] ----------------------------------------------------------------
Error 31066
osmc@osmc:~$ python -m bypy mkdir /app/bypy
osmc@osmc:~$ python -m bypy list
 [01:37:07] ----------------------------------------------------------------
Error 31066
osmc@osmc:~$

I have the save question, and don't know how to solve it.Maybe the file is too big?

{u'error_code': 31066, u'error_msg': u'file does not exist'}
maybe some help

在 apps目录下手动创建个bypy文件夹

在 apps目录下手动创建个bypy文件夹

As @7449 mentioned - you need to manually create this bypy folder in your My App Data folder. I can't remember since when Baidu started to restrict App access to their own folders in the /apps folder. So all manipulation that bypy can do, is within this folder [/apps/bypy].
hth.


Add font files

@rubillionaire I've uploaded the font files and deployed it. When I got to something like http://welcomegrad.risd.edu/static/fonts/541402/1C72577AB7AD828D6.css, the file exists.
When I go through the steps typography.com outlines, the last step involves them confirming the installation of the fonts, but I get this error:

Not too sure why this is happening.

This could be because welcomegrad.risd.edu is only available on the on
campus network. OIT needs to make the CNAME record public, so that you can
visit the site from outside of RISD. I noticed this when I tried visiting
last night from home.
On Wed, Feb 1, 2017 at 9:59 AM, mmarol<EMAIL_ADDRESS>wrote:

Assigned #6 https://github.com/risd/risd-graduate-welcome/issues/6 to
@rubillionaire https://github.com/rubillionaire.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/risd/risd-graduate-welcome/issues/6#event-944871948,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAaiMXmbig0ZhQJbeLYr-vVT7q_vFMJRks5rYJ3MgaJpZM4LzSOR
.

@room421 is this a thing I can get in contact with them about, or will you be doing that?


Need to review for OpenCDSS release

The documentation needs to be reviewed for OpenCDSS release, including adding open source license and also make work for MkDocs 1.0.4.

I updated the README for open source license.  I copied many images for commands where image file matched command name.   I updated the build scripts to work with GCP upload and also MkDocs 1.0.4.  Close the issue.


Remove DO_COV

We have test coverage by way of coveralls now. Remove the old stale DO_COV stuff.
Fix: #1853

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
19221

Relevant Lines:
22812

💛  - Coveralls


Duplications due to pluralization of "Selected Cell Type"

The "Selected Cell Type" facet (cell_suspension.selected_cell_types.text) in the Data Browser "Tissue Type" dropdown contains pluralization of many values, resulting in duplications:

astrocytes vs astrocyte
common myeloid progenitor cell vs common myeloid progenitor
dendritic cells vs dendritic cell
hematopoietic stem cells vs hematopoietic stem cell
monocytes vs monocyte
natural killer cells vs natural killer cell
neurons v neuron
peripheral blood mononuclear cells vs peripheral blood mononuclear cell
Plasma cell vs plasma cell (capitalization should be removed)

Acceptance criteria for the task:

Remove pluralization from all terms thus removing the duplications

Needs triaging

@ESapenaVentura doesn't ingest check the label to the id? Why is it giving a pass with the plural version if the looked-up term isn't plural?

@willrockout I don't think ingest checks label --> ontology ID, but I will check with the developers. In any case, the .text is definitively not checked, because we agreed that that is a free string text where the contributor can input the data as they please
@theathorn I thought the browser only took the ontology label field when available? (And ignored the .text) Was it not available for these datasets?

@ESapenaVentura Azul first checks for the cell_type_ontology.ontology_label and, if not present, falls back to the cell_type_ontology.text field. So either the ontology_labels are inconsistent or are not present.

This is not really stalled. It has simply not been prioritised just yet. Moving to the Backlog


add To-do list webpage Project using html, css, js

Develop an To-do list project using HTML, CSS , JS

with Add new task and tracking completed , pending Task

@trishan9 please assign me this issue I would like to work on it !!

@trishan9 please assign me this issue I would like to work on it !! and please add Hacktoberfest labels to it

Sure, I assign this issue to you!

Issue has been assigned to you under Hacktoberfest!

Happy contribution!


Write or improve intros for getting started map topics

What article on docs.github.com is affected?

[ ] "Quickstart"
[ ] "Learning about GitHub"
[ ] "Signing up for GitHub"
[ ] "Using GitHub"
[ ] "Exploring projects on GitHub"
[ ] "Getting started with Git"
[ ] "Using Git"

What part(s) of the article would you like to see updated?
These map topics (pages that group articles) either don't have intros or have intros that could be improved to be friendlier to our audience of new users. Here are some guidelines on writing good intros.

Intros are displayed in search results and provide contextual information to help readers choose a result. Within a page, intros allow readers to quickly decide if the category, map topic, or article is relevant to them.
Map topic and category intros should be one sentence long, and general and high-level enough to scale with future changes without needing to be frequently updated. Avoid lists of features or article content, and focus on the more evergreen overall task the category or map topic describes.

Additional information
This issue is for a special project and is not currently available for contributors to work on.

I would like to work on this issue. :)

Thanks @codewithdev! ⚡


[5.4] Update dirty attributes when soft deleting Eloquent model.

Add protected variable updateDirtyOnDelete to track if dirty attributes should be updated when deleting.
Add function deleteWithUpdate() to update dirty attributes when deleting.
Modify runSoftDelete() to update dirty attributes when updateDirtyOnDelete == true.
Update unit tests for new functionality.
With this change, one can make modifications to their model and call deleteWithUpdate() to persist any changes to the database.
Before:
$model->deleted_by = $user->id;
$model->save();
$model->delete();

After:
$model->deleted_by = $user->id;
$model->deleteWithUpdate();

The original delete() function will still behave the same way as before so no existing applications will break with this change. The only instance where this will cause a breaking change is if a class implementing the SoftDeletes trait has the following variable:
protected $updateDirtyOnDelete = false;

No plans on adding this.

This is great feature... you don't think add that in future ?


Added timeout functionality on Server

Added:

Parameter 'timeout' (Optional of type int) which signifies the request time for attestation and assertion operations to class Fido2Server.
Added a timestamp in the 'state' value to handle timeout
Added errors for request timed out on register_complete() and authenticate_complete() functions in server.

Closing this PR coz merging is blocked. Will open another PR


Table with thead tag produces auto-scrolling to the bottom of the page on Firefox

On Firefox version 66+ (not tested before), if the StickyTableHeaders-Table in on top of the page and has a thead tag, after leaving the sticky header table, the browser continues to scroll by it self until the bottom of the page and "stays" there, no more scrolling up possible.
Here is a jsfiddle to reproduce this: https://jsfiddle.net/vLx4z3n5/1/

It does seem to be a new thing with firefox - we've been using this for many years and just started getting odd complaints of issues with the page scrolling to the bottom - have tracked it down to this - disabled all stickytableheaders on firefox as a work around - I couldn't see what on earth is going wrong

This is firefox bug and I have created a ticket on bugzila
https://bugzilla.mozilla.org/show_bug.cgi?id=1543599
Still waiting for them to fix it

I'm looking into fixing that Firefox bug and got here just by chance, that jsfiddle with the reduction would've been extremely useful!
Anyhow, thank you for reporting it. You can work around it by using overflow-anchor: none on the container. But I'll try to get it fixed next week.

Should be fixed in Nightly, 68ESR and Beta, fwiw.
Relatedly, this plugin creates an infinite scroll event bug in Chrome, which I fixed on Firefox too, and which is tracked in https://bugs.chromium.org/p/chromium/issues/detail?id=988991. See my comment there for how to reproduce it.


nixos/river: add xdg.portal.config

Description of changes
Add xdg.portal.config, needed since xdg-desktop-portal 1.17
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Priorities
Add a :+1: reaction to pull requests you find important.

@lheckemann This should be backported to 23.11.


A.05.IR13.5.char.enc

This issue has been extracted from the issue list on:https://ies-svn.jrc.ec.europa.eu/issues/2685
Comment
Testability: The test should clarify that this only works if the resource can be retrieved online and is a textual resource.
Ambiguity: The test should clarify for each possible resource how the character encoding of a resource can be verified. If the character encoding is not included in the retrieved resource, no claim can be made about the used character encoding. For example, in XML, the character encoding prolog is not mandatory (i.e. ).

This element is mandatory only if an encoding is used that is not based on UTF-8.
If the dataset is not available online, we cannot know whether this element is needed and what value it should have.
(Question: what are the allowed values to describe the characterset? E.g. 8859, or ISO8859P1...)
However, if the dataset is not available online for recognized reasons of limitations to public access, we could flag this test as passed, since interoperability is already out of the question anyway.
If the dataset is not available online because no download service is documented or available, the outcome of this test is undetermined.
If the dataset can be retrieved, we need to verify that the declaration corresponds to what is present in the dataset.
It seems unreasonable to download the full dataset every time the metadata is validated.
It is also practically valueless since by the time all the dataset has been validated, it might have changed and become invalid.
However, it is feasible and in general useful to "probe" and test only the first bytes/kilobytes/megabytes of a dataset. For xml this is enough to determine if the character encoding, the spatial object type, the encoding, the transfer rate and so on, correspond to what is declared and/or required.
If the encoding declared in the corresponding metadata element does not use XML, this outcome of this test is undetermined.
XML is intended as XML 1.0 and XML Schema as XML Schema 1.0
From XML 1.0:
Definition: An XML document is valid if it has an associated document type declaration and if the document complies with the constraints expressed in it.
Parsed entities which are stored in an encoding other than UTF-8 or UTF-16 MUST begin with a text declaration (see 4.3.1 The Text Declaration) containing an encoding declaration.
Data Spec. TG Requirement 6  requires that "Data instance (XML) documents shall validate without error against the provided XML schema".
In order to determine the actual character encoding we can refer to
From XML 1.0:
F Autodetection of Character Encodings (Non-Normative)
These rules are already implemented in several software libraries.
I assume each spatial object is valid against the corresponding INSPIRE schema.


Undo feature?

Any plans to add an undo feature? Really handy to have

There is an undo feature. No need to add it ;-)

Holding down SHIFT+CTRL while looking at a blocks will show you the last blocks you placed with a green border around them. SHIFT+CTRL+Right clickking any of them will undo the operation, giving you all the items back.


Roku entities

Relevant information
Re-submitting this after some time. We have decided to split into two entities - one for the video telemetry and one for the system/app level telemetry.
Checklist

[X ] I've read the guidelines and understand the acceptance criteria.
[ X] The value of the attribute marked as identifier will be unique and valid.
[ X] I've confirmed that my entity type wasn't already defined. If it is I'm providing an
explanation above.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.3 out of 4 committers have signed the CLA.:white_check_mark: naxhh:white_check_mark: srvaroa:white_check_mark: cjmccarthy:x: asllopYou have signed the CLA already but the status is still pending? Let us recheck it.

Hi folks! I'm having trouble understanding the entityBot requested changes - are these query formats not allowed?

Hi @cjmccarthy, exactly, those queries must to be compatible with the format expected by the Golden Metrics Harvester, please check those rules and create your queries accordingly.

hi @cjmccarthy , just a callout, there are not tags specified as part of the def, is it ok?

No tags is ok for now! After initial customer testing we will see which tags might be useful and come back to add them.

Hi @cjmccarthy , exactly!!, they should appear in STG.


Optional compression

Sometimes software (minisat for example) uses optional compression: it checks if provided data file is correct gzip archive. If yes then it unpacks and reads it, if not --- just reads file as is. Zlib even has gzopen/gzdopen functions that supports this behaviour.
GzDecoder::new consumes stream in case of gzip header parsing error, so we are forced to reopen in case of file or to have big toubles in case of non-file. It would be nice to support optional compression in flate2 or at least get stream and readed bytes back in case of header parse error.

Hm it does seem that zlib has some... interesting behavior with gz files. I've historically also seen how they support concatenated gzip streams being interpreted as one stream as well. I personally prefer to be more explicit about what's going on rather than hide that under the covers...
Along those lines, it looks like zlib mainly only supports this sort of operation with a file descriptor as a file as the backing storage? Along those lines couldn't this be done by using seek to go back to the beginning of a file in the case that the header wasn't parsed? You shouldn't need to relinquish ownership as you can hand in &mut R instead of just R.
It may also be prudent though to add the ability to get back out the initial stream when trying to parse a gz header

Seek will work for files, although it is even uglier because something simple with seek instead of reopen, like:
    let mut reader = io::BufReader::new(try!(fs::File::open(path)));
    match GzDecoder::new(&mut reader) {
        Ok(gz) => { parse(gz) }

        Err(ref e) if e.kind() == io::ErrorKind::InvalidInput => {
            try!(reader.seek(SeekFrom::Start(0)));
            parse(reader)
        }

        Err(e) => { Err(e) }
    }

will not pass borrow check.
Also, it would be nice to be able to read in such way from pipes or stdin (like minisat do with zlib)

Yeah you won't be able to seek on pipes but you can perhaps use some buffering to try to decode just the header of the stream to see if it's a gzip stream or not (the first two bytes should tell I believe).
You don't need to an an explicit layer of buffering yourself (as this library already does so), and this should also work fine with pipes as it just requires R: Read

Ok I'm going to close this for now as I'd prefer the types in this library do what they say on the tin, and otherwise I think all the bits and pieces are there now to recover this if need be.


Improve mailing lists description

There were mailing lists without description. Picked text from http://docs.qgis.org/testing/en/docs/user_manual/preamble/help_and_support.html

just merge it after the test check.


Adds variable loot from creatures, adds Healing Potions, additional g…

…ame balance

Closes #16
Closes #38


TypeError: INSTRUCTOR._load_sbert_model() got an unexpected keyword argument 'token'

This is the command line lauched:
python generate.py \
    --base_model=h2oai/h2ogpt-4096-llama2-7b-chat --prompt_type=llama2 \
    --use_gpu_id=True --gpu_id=-1 --max_seq_len=8192 \
    --user_path=/opt/myDocuments/arsys.es/all/ --langchain_mode='UserData' --max_quality=True \
    --add_chat_history_to_context=True --keep_sources_in_context=True --enable_ocr=True --enable_doctr=True \
    --answer_with_sources=True --show_link_in_sources=True --append_sources_to_chat=True \
    --hf_embedding_model="hkunlp/instructor-base" \
    --memory_restriction_level=0 --score_model=None --verbose=True --debug=True

This is the error found:
Traceback (most recent call last):
  File "/root/REPOSITORIES/aramirez/void_h2ogtp/generate.py", line 20, in <module>
    entrypoint_main()
  File "/root/REPOSITORIES/aramirez/void_h2ogtp/generate.py", line 16, in entrypoint_main
    H2O_Fire(main)
  File "/root/REPOSITORIES/aramirez/void_h2ogtp/src/utils.py", line 75, in H2O_Fire
    fire.Fire(component=component, command=args)
  File "/root/miniconda3/envs/h2ogpt/lib/python3.10/site-packages/fire/core.py", line 143, in Fire
    component_trace = _Fire(component, args, parsed_flag_args, context, name)
  File "/root/miniconda3/envs/h2ogpt/lib/python3.10/site-packages/fire/core.py", line 477, in _Fire
    component, remaining_args = _CallAndUpdateTrace(
  File "/root/miniconda3/envs/h2ogpt/lib/python3.10/site-packages/fire/core.py", line 693, in _CallAndUpdateTrace
    component = fn(*varargs, **kwargs)
  File "/root/REPOSITORIES/aramirez/void_h2ogtp/src/gen.py", line 2015, in main
    model=get_embedding(use_openai_embedding, hf_embedding_model=hf_embedding_model,
  File "/root/REPOSITORIES/aramirez/void_h2ogtp/src/gpt_langchain.py", line 556, in get_embedding
    embedding = HuggingFaceInstructEmbeddings(model_name=hf_embedding_model,
  File "/root/miniconda3/envs/h2ogpt/lib/python3.10/site-packages/langchain_community/embeddings/huggingface.py", line 167, in __init__
    self.client = INSTRUCTOR(
  File "/root/miniconda3/envs/h2ogpt/lib/python3.10/site-packages/sentence_transformers/SentenceTransformer.py", line 287, in __init__
    modules = self._load_sbert_model(
TypeError: INSTRUCTOR._load_sbert_model() got an unexpected keyword argument 'token'

System:

SO: Ubuntu 22.04 LTS utterly updated.
Commit: 7435b4bc (the last one at the moment this issue is written)
Environment utterly updated and clean after execution of script bash docs/linux_install_full.sh

Observations:

This only happens with embedding models based on BERT
Embedding models as "intfloat/multilingual-e5-small" does not report any problem.

Thanks for everything.

did you use fresh env?   new packages are required for instructor

677bf0817d3e342ffc32a31a94cf63cc05096660

Building new image with above fix to see if order helps.  I saw in jenkins that earlier requirements.txt file triggered instructorembedding install, so needs to be early and not as late as langchain one.

Seems to work.  Thanks for reporting!

It´s working right now, thanks.


Node-API on Windows: window asking how to execute .node file

Running npx prisma generate on Windows with set PRISMA_FORCE_NAPI=true set shortly opens weird window asking how to execute .node file (but generates Client successfully anyway and windows goes away again):

Is this trying to somehow execute this file in a way that does not actually work, like with the binary?

Possibly related https://github.com/prisma/prisma/issues/4308#issuecomment-844344765

"Debugged" this via process.exit(1) in the built Client...
And here is the reason:
https://github.com/prisma/prisma/blob/ad8f073b83ddd9fd6a57c01fa7ac16d75a69f0a7/src/packages/sdk/src/engine-commands/getVersion.ts#L17-L28
This goes into the wrong arm of the conditional because the string it is comparing to does not match the Windows Node-API library engine file name:
https://github.com/prisma/prisma/blob/ad8f073b83ddd9fd6a57c01fa7ac16d75a69f0a7/src/packages/fetch-engine/src/download.ts#L35-L41
The value of binaryName is query-engine here for some reason - although it should be libquery-engine-napi.
It has that value, as that is the default of the function if only 1 param is supplied:
https://github.com/prisma/prisma/blob/ad8f073b83ddd9fd6a57c01fa7ac16d75a69f0a7/src/packages/sdk/src/engine-commands/getVersion.ts#L12-L15
And we are indeed only calling it with 1 param here:
https://github.com/prisma/prisma/blob/9c287b3d45ae721f463b682651bd778cfab9d05a/src/packages/client/src/generation/generateClient.ts#L283-L287

Fixed by https://github.com/prisma/prisma/pull/7898


Add support for more lifecycle methods

Since we use purescript-react now, this is no longer necessary in the 0.11 branch, where we can compile to a ReactSpec which can then be updated with new lifecycle methods.


[Keyvault] Fix#21487:az keyvault key create:Fix keyvault createkey links issue

Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

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

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Keyvault


[Enhancement]: UI improvement according to UI/UX principles.

Do you want to have the enhancement of existing game ? 😀 Describe yourself..
Proposals:
- Improvement in overall UI.
- Add margin and Padding wherever required.
- Change the color scheme.
- Add sufficient whitespace for better user experiene.

Describe the solution you'd like
Pros:

Fully optimized website for all devices.
Increase in SEO ranking.

Cons:

Can take some time to implement.
Some properties needs to be fully changed to optimize some particular sections including font units etc.

Select program in which you are contributing
No response
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @shaurya35 !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

Hi @kunjgit
Thanks for the information.
I request you to assign the gssoc and level labels so that I can start working according to the guidelines provided by Gssoc

/assign

Hello @shaurya35, Time's up!⏰
Sorry for closing your issue!
But it's more than a week since we haven't received anything from your side 😢 .
Come up with new ideas, create a new issue and make sure you finish it within a week! 🔥
All the best! 🚀
Happy Hacking! 💗

Hey @shaurya35 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


Adding unnecessary fields doesn't trigger an error.

This actually creates hard to track bug when what we want to load is actually missing fields, but the file we load looks fine and has that field and there is no error on load.

That's not a RON issue. Serde does give you a warning, we can't do more about this.

If there's a warning by serde, then its okay. I opened the issue for someone else. Said person probably forgot to check the log :)
thanks


Command remove with extra space

I think it's a typo

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Skype/Skype-For-Business-Bot-Framework/docs/Overview.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


More compact 'Global working directory' toolbar

From<EMAIL_ADDRESS>on 2011-01-25T17:52:59Z
I have made a small patch that removes the (to my opinion) quite verbose description string in the "Global working directory" toolbar:
--- a/spyderlib/plugins/workingdirectory.py	Wed Jan 19 16:40:56 2011 +0100
+++ b/spyderlib/plugins/workingdirectory.py	Sat Jan 22 13:59:17 2011 +0100
@@ -159,13 +159,13 @@
self.setWindowTitle(self.get_plugin_title()) # Toolbar title
self.setObjectName(self.get_plugin_title()) # Used to save Window state

   label = QLabel(self.tr("Global working directory:")+" ")

   label.setToolTip(self.tr("This is the working directory for newly\n"

                            "opened consoles (Python interpreters and\n"

                            "terminals), for the file explorer, for the\n"

                            "find in files plugin and for new files\n"

                            "created in the editor"))

   self.addWidget(label)

+#        label = QLabel(self.tr("Global working directory:")+" ")
+#        label.setToolTip(self.tr("This is the working directory for newly\n"
+#                                 "opened consoles (Python interpreters and\n"
+#                                 "terminals), for the file explorer, for the\n"
+#                                 "find in files plugin and for new files\n"
+#                                 "created in the editor"))
+#        self.addWidget(label)
     \# Previous dir action
     self.history = []

@@ -192,7 +192,12 @@
# Path combo box
adjust = self.get_option('working_dir_adjusttocontents', False)
self.pathedit = PathComboBox(self, adjust_to_contents=adjust)

   self.pathedit.setToolTip(self.tr("Working directory"))

   self.pathedit.setToolTip(self.tr("This is the working directory for newly\n"

                            "opened consoles (Python interpreters and\n"

                            "terminals), for the file explorer, for the\n"

                            "find in files plugin and for new files\n"

                            "created in the editor"))

+#        self.pathedit.setToolTip(self.tr("Working directory"))
self.connect(self.pathedit, SIGNAL("open_dir(QString)"), self.chdir)
self.pathedit.setMaxCount(self.get_option('working_dir_history', 20))
wdhistory = self.load_wdhistory( workdir )
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=537

From<EMAIL_ADDRESS>on 2011-03-01T00:36:16Z
I like the patch.  I think the tooltip is enough for people to figure out what the directory means. (There is a folder icon directly to the right of it.)  Maybe overlay an Earth icon on top of that folder icon to signify "Global" working directory.
Without the patch, I can't see what directory is being used (or rather, I can only see 5 pixels of the global working directory path).  Please patch.  :)

From ccordoba12 on 2011-02-16T11:46:18Z
Hi Christoph,
I agree with you that "Global working directory" is just too long to fit on the toolbar, especially for people using netbooks. But your patch simply removes the name and leaves the user the responsibility to find what the toolbar is about through a tooltip, which could be another (serious?) usability problem too.
We need to think more carefully about it. What if we change the name for an icon? Something that can represent visually the idea of "working directory"?
Carlos
Status: Accepted
Labels: -Type-Defect -Priority-Medium Type-Enhancement Priority-Low

From pierre.raybaut on 2011-03-19T05:40:33Z
This issue was closed by revision 4142c2fb4091 .

From pierre.raybaut on 2011-03-18T00:08:29Z
This issue was closed by revision 32a01181a585 .
Status: Fixed

From pierre.raybaut on 2011-04-24T07:38:37Z
This issue was closed by revision 32a01181a585 .


Cannot find modsecurity in /usr/local/modsecurity/include and /usr/local/modsecurity/lib/

Hi guys, so i try to build nginx with mod security with command look like this :
export TENGINE_VERSION=3.1.0
export YAJL_VERSION=2.1.0
export MODSEC_VERSION=v3.0.12
export MODSEC_NGX_VERSION=v1.0.3
export TEMP_DIR=/mnt/data/ehe/wrapper-tengine/install
export MODSECURITY_INC="/usr/local/modsecurity/include/"
export MODSECURITY_LIB="/usr/local/modsecurity/lib/"

mkdir -p ${TEMP_DIR} \
&& curl -fsSL https://github.com/openresty/luajit2/archive/${LUAJIT_VERSION}.tar.gz -o luajit.tar.gz \
&& tar zxf luajit.tar.gz -C ${TEMP_DIR} \
&& cd ${TEMP_DIR}/luajit2-${LUAJIT_VERSION#v} \
&& make -j4 && make install -j4 \
&& export LUAJIT_INC=/usr/local/include/luajit-2.1 \
&& export LUAJIT_LIB=/usr/local/lib \
&& ln -sf luajit /usr/local/bin/luajit \
&& cd .. \
&& curl -fsSL https://github.com/lloyd/yajl/archive/refs/tags/${YAJL_VERSION}.tar.gz -o yajl.tar.gz \
&& tar zxf yajl.tar.gz -C ${TEMP_DIR} \
&& cd ${TEMP_DIR}/yajl-${YAJL_VERSION} \
&& ./configure && make && make -j4 install -j4 \
&& ldconfig /usr/local/lib \

# rm -Rf ${TEMP_DIR}/modsecurity-${MODSEC_VERSION} \
git clone --depth 1 -b v3/master --single-branch https://github.com/owasp-modsecurity/ModSecurity ${TEMP_DIR}/modsecurity-${MODSEC_VERSION} \
&& cd ${TEMP_DIR}/modsecurity-${MODSEC_VERSION} \
&& git submodule init \
&& git submodule update \
&& ./build.sh \
&& ./configure \
&& make -j4 \
&& make install -j4

cd ${TEMP_DIR}/tengine-${TENGINE_VERSION} \
&& MODSECURITY_INC="/usr/local/modsecurity/include/" MODSECURITY_LIB="/usr/local/modsecurity/lib/" ./configure --with-ld-opt="-lpcre" --prefix=/etc/nginx --sbin-path=/usr/sbin  \
        --error-log-path=/var/log/nginx/error.log \
        --conf-path=/etc/nginx/nginx.conf --pid-path=/run/nginx.pid \
	--add-dynamic-module=${TEMP_DIR}/modsecurity-nginx-${MODSEC_NGX_VERSION} \
	--with-compat \
&& make -j4 && make install -j4

but it always give me this error
checking for ModSecurity library in "/usr/local/modsecurity/lib/" and "/usr/local/modsecurity/include/" (specified by the MODSECURITY_LIB and MODSECURITY_INC env) ... not found
        ./configure: error: ngx_http_modsecurity_module requires the ModSecurity library and MODSECURITY_LIB is defined as "/usr/local/modsecurity/lib/" and MODSECURITY_INC (path for modsecurity.h) "/usr/local/modsecurity/include/", but we cannot find ModSecurity there.

i already follow the steps in repository and even execute steps that suggested like :
libtool: finish: PATH="/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/var/lib/snapd/snap/bin:/sbin" ldconfig -n /usr/local/modsecurity/lib
----------------------------------------------------------------------
Libraries have been installed in:
   /usr/local/modsecurity/lib

If you ever happen to want to link against installed libraries
in a given directory, LIBDIR, you must either use libtool, and
specify the full pathname of the library, or use the '-LLIBDIR'
flag during linking and do at least one of the following:
   - add LIBDIR to the 'LD_LIBRARY_PATH' environment variable
     during execution
   - add LIBDIR to the 'LD_RUN_PATH' environment variable
     during linking
   - use the '-Wl,-rpath -Wl,LIBDIR' linker flag
   - have your system administrator add LIBDIR to '/etc/ld.so.conf'

See any operating system documentation about shared libraries for
more information, such as the ld(1) and ld.so(8) manual pages

and this is the directory of modsecurity looks like :
/usr/local/modsecurity
├── bin
│   └── modsec-rules-check
├── include
│   └── modsecurity
│       ├── actions
│       │   └── action.h
│       ├── anchored_set_variable.h
│       ├── anchored_set_variable_translation_proxy.h
│       ├── anchored_variable.h
│       ├── audit_log.h
│       ├── collection
│       │   ├── collection.h
│       │   └── collections.h
│       ├── debug_log.h
│       ├── intervention.h
│       ├── modsecurity.h
│       ├── reading_logs_via_rule_message.h
│       ├── rule.h
│       ├── rule_marker.h
│       ├── rule_message.h
│       ├── rules_exceptions.h
│       ├── rules.h
│       ├── rules_set.h
│       ├── rules_set_phases.h
│       ├── rules_set_properties.h
│       ├── rule_unconditional.h
│       ├── rule_with_actions.h
│       ├── rule_with_operator.h
│       ├── transaction.h
│       ├── variable_origin.h
│       └── variable_value.h
└── lib
    ├── libmodsecurity.a
    ├── libmodsecurity.la
    ├── libmodsecurity.so -> libmodsecurity.so.3.0.12
    ├── libmodsecurity.so.3 -> libmodsecurity.so.3.0.12
    ├── libmodsecurity.so.3.0.12
    └── pkgconfig
        └── modsecurity.pc

any other suggestion regarding this issues?

Found something regarding this issue. It's look like issue only appear when i using build from source modsecurity. all working well if i using package from repo.
Any other suggestion to solve this issue in case install and download from repos cannot be done some times?
Best Regards,
bijancot

Hi @bijancot,
you should try to pass CFLAGS when you build Tengine and ModSecurity connector:
export CFLAGS="-I/usr/local/modsecurity/headers -L/usr/local/modsecurity/lib"

Thank's for the suggestion. Would try it soon

Have the same issue on alpine-3.21 when configure nginx from source:
./configure --with-compat --add-dynamic-module=/opt/ModSecurity-nginx
#33 1.229 checking for OS
#33 1.241  + Linux 5.15.0-1068-gcp x86_64
#33 1.241 checking for C compiler ... found
#33 1.289  + using GNU C compiler
#33 1.291  + gcc version: 14.2.0 (Alpine 14.2.0)
...
#33 5.232 adding module in /opt/ModSecurity-nginx
#33 5.232 checking for ModSecurity library ... not found
#33 5.252 checking for ModSecurity library in /usr/local/modsecurity ... not found
#33 5.275  ./configure: error: ngx_http_modsecurity_module requires the ModSecurity library.

when I configure nginx on alpine-3.20 it ends up successfully:
./configure --with-compat --add-dynamic-module=/opt/ModSecurity-nginx
#28 1.250 checking for OS
#28 1.257  + Linux 5.15.0-1068-gcp x86_64
#28 1.257 checking for C compiler ... found
#28 1.294  + using GNU C compiler
#28 1.296  + gcc version: 13.2.1 20240309 (Alpine 13.2.1_git20240309)
...
#28 4.287 adding module in /opt/ModSecurity-nginx
#28 4.287 checking for ModSecurity library ... not found
#28 4.302 checking for ModSecurity library in /usr/local/modsecurity ... found
#28 4.335  + ngx_http_modsecurity_module was configured


[KYUUBI #4352] Support System.gc() with periodic GC interval

Why are the changes needed?

How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

Codecov Report

Merging #4382 (a9eb126) into master (0dce65d) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #4382      +/-   ##
============================================
- Coverage     53.72%   53.71%   -0.01%
  Complexity       13       13
============================================
  Files           563      564       +1
  Lines         30877    30892      +15
  Branches       4164     4164
============================================
+ Hits          16588    16594       +6
- Misses        12735    12745      +10
+ Partials       1554     1553       -1

Impacted Files
Coverage Δ

...in/scala/org/apache/kyuubi/config/KyuubiConf.scala
97.45% <100.00%> (+<0.01%)
:arrow_up:

.../scala/org/apache/kyuubi/server/KyuubiServer.scala
53.84% <100.00%> (+1.03%)
:arrow_up:

...a/org/apache/kyuubi/server/PeriodicGCService.scala
100.00% <100.00%> (ø)

.../kyuubi/server/mysql/constant/MySQLErrorCode.scala
13.84% <0.00%> (-6.16%)
:arrow_down:

...ache/kyuubi/server/mysql/MySQLCommandHandler.scala
77.77% <0.00%> (-4.05%)
:arrow_down:

...apache/kyuubi/engine/JpsApplicationOperation.scala
77.41% <0.00%> (-3.23%)
:arrow_down:

...in/spark/authz/ranger/SparkRangerAdminPlugin.scala
64.47% <0.00%> (-2.64%)
:arrow_down:

...ache/kyuubi/server/mysql/MySQLGenericPackets.scala
76.59% <0.00%> (-2.13%)
:arrow_down:

...rg/apache/kyuubi/ctl/cmd/log/LogBatchCommand.scala
59.09% <0.00%> (-1.52%)
:arrow_down:

...n/scala/org/apache/kyuubi/engine/ProcBuilder.scala
79.01% <0.00%> (+0.61%)
:arrow_up:

... and 2 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

@turboFei

thanks, merged to master and branch-1.7


Added users data

Added new users

Task linked: CU-8697359pc Deploy node js project to AWS AWS Elastic Beanstalk


Add style block syntax check

we should support check the css block syntax of vue and svelte template.

Yes please :) !!
Was just about to open an issue for this.
Also, it would be great to have intellisense features in .css files.

/cc @ThaddeusJiang


Outgoing Image message sending 2 times

How it can be fix.

Facing same issue... Steps to reproduce

Create new channel
Send message
SBDMessageCollection Delegate send insert action twice which in result show 2 messages in ChatView of sender user.

Please provide some solution to this I am using SendbirdSDK v.3.0.156 and SendbirdSyncManager v.1.1.17

@famous90  Could you please tell me why this error is occurring in iOS 13. But it is working in iOS 12 . can you please give me a solution end of the day otherwise I will uninstall this worst sdk and will choose another one.

i am facing the similar issue

I am also facing same issue. So, Please give me solution

I am also facing same issue. So, Please give me solution


enhancement/message-request

Potentially scope this function so it becomes RequestFrom(group state.GroupID, peer state.PeerID, id state.MessageID)


use_sync_to_async Hook

This hook is best way I can think of developing ORM query support, in a way that Django developers would understand.
Potentially closes the following

#79
#80

@rmorshea Let me know what you think about this implementation.


changes for localization issue

Fixes # for issue 1795 e.changeLanguage is not a function

#1795 fix for this issue

Hi @kumargulshan91 did you have a read about the contributing guidelines: https://github.com/covid19india/covid19india-react/blob/master/CONTRIBUTING.md ? Please do not fix an issue unless one of the mods has a look at the issue.

Patched by #1862


[Question] Double replies using Skype for Business on phone client

System Information

SDK Language: .NET
SDK Version: 3.9
Development Environment: Azure Bot Service

Bot Info

Connected channels: Skype, Web Chat, Skype for Business, Cortana

Issue Description
I'm seeing double responses from the bot when using Skype for Business on a Windows Phone, for almost every response issued by the bot. For example, the bot will send a reply prior to doing a search, then send the results of the search shortly after. Almost invariably, the please wait and the results are sent, followed after a delay then the same please wait and the results again.
The iOS client seems to be have the same issue, but rather less often.
I've not yet seen this behaviour in Skype for Business on Windows, or indeed other channels in general.
I'm wondering if it relates to the bot 'waking up' after a lull in conversations, as sometimes a question following up a previous double response works correctly.
Reason for Question
My query is really to determine if this is a issue under my control (i.e. is it coding issue that needs solving), or is it a quirk of the channel or client software (i.e. not under my control).
The behaviour occurred while showing a business sponsor our progress, so hoping to feedback with an explanation.
Expected Behavior
To get exactly one response in the client application, for each response sent from the bot.

Is there any way you could share your code so we can take a look?

Jason, thanks for replying.
Even this simple code will reply with 'You selected....' twice:
    [LuisIntent("Root.IWantTo")]
    public async Task IWantTo(IDialogContext context, LuisResult result)
    {
        IMessageActivity activity = context.MakeMessage();

        activity.Text = "You selected the IWantTo intent";

        await context.PostAsync(activity);

        context.Wait(MessageReceived);
    }

Mark

I am unable to repro this as I do not have a windows phone to test with.  Looking at your code the only thing I can think of trying would be to maybe use CreateReply() instead of  MakeMessage()

I am unable to test this scenario, but looking at your code you may be able to use CreateReply() instead of MakeMessage() . this may or may not help though.  I'll try to ping someone from S4B to take a look.

We have encounter this problem.
If the client is using Skype on iOS and send one message, our micro-service would receive that message twice. Therefore, our microservice would replay (twice) and the client would see two response from our microservices.
One hypothesis is that the MS bot is waiting for the sendTyping signal immediately after sending the message to the microservice. Since it doesn't receive anything in a time frame, it timeout and resend the message.
If ensuring that sendTyping flag is send to every message received from the MS bot, immediatly, do not solve the problem then there could be another solution. This solution would be to treat this problem as the button bouncing (aka. contact bounce) issue. When you receive two identical messages from the MS bot, check the time frame between them. If the time frame between both messages is short, then ignore the second message.
 I will post a solution if I find it.

Thank you for opening an issue against the Bot Framework SDK v3. As part of the Bot Framework v4 release, we’ve moved all v3 work to a new repo located at https://github.com/microsoft/botbuilder-v3. We will continue to support and offer maintenance updates to v3 via this new repo.
From now on, https://github.com/microsoft/botbuilder repo will be used as hub, with pointers to all the different SDK languages, tools and samples repos.
As part of this restructuring, we are closing all tickets in this repo.
For defects or feature requests, please create a new issue in the new Bot Framework v3 repo found here:
https://github.com/microsoft/botbuilder-v3/issues
For Azure Bot Service Channel specific defects or feature requests (e.g. Facebook, Twilio, Teams, Slack, etc.), please create a new issue in the new Bot Framework Channel repo found here:
https://github.com/microsoft/botframework-services/issues
For product behavior, how-to, or general understanding questions, please use Stackoverflow.
https://stackoverflow.com/search?q=bot+framework
Thank you.
The Bot Framework Team


Editing Bonds have no /year argument in UG

Bug in UG found where the adding and editing of bonds do not match in parameters
Should also include the fact that years can only be bigger than the prev one.

I will fix this in my next PR


Adds Call Of Duty

Description
Closes #1941
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes:  #1941

Type of change

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

Checklist:

[x] I have made this from my own
[x] I have taken help from some online resourses
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK

Added all the index, styles, script and manifest files.

Added all the index, styles, script and manifest files.


OneDrive cannot connect or sign in

I have used your host file to set up adblock list in DNSMasq on my DDWRT router. However, if I turn on my VPN to a remote server (TorGuard VPN), onedrive can re-establish the connection.

@tienhuynh5312 do you know which domain is being blocked?
Anyone else experiencing this?

Weird, I'm always up-to-date (using Steven's default hosts list + other hpHosts sources) on my Win8.1Pro and onedrive syncs peacefully without any errors. Even on my Windows Phone and Note 4.

I have spotted out the error causing this. Beside using DNSmasq adblock host, I turned on Encrypt DNS that use Adguard DNS (maybe other DNS in the ddwrt setting). After I disable Encrypt DNS, everything works just fine now. @StevenBlack

@tienhuynh5312 that's interesting.  Thanks.
Closing.


Multi normal derivatives

Summary
This shows that multi_normal should just take the cholesky decomposition and call multi_normal_cholesky.
I just copied all the multi_normal_cholesky_lpdf code over. I wasn't sure how to just call it. That should be updated. The unit tests that call multi_normal_log fail.
Question: Would it be better to form the cholesky decomposition using ldlt instead of cholesky_decompose()?
Tests
No new tests.
Here's a benchmark vs the current multi_normal_lpdf

Side Effects
None
Release notes
multi_normal_lpdf now takes the Cholesky decomposition of the covariance matrix and calls multi_normal_cholesky_lpdf under the hood
Checklist

[ ] Math issue #2544

[x] Copyright holder: Sean Pinkney
The copyright holder is typically you or your assignee, such as a university or company. By submitting this pull request, the copyright holder is agreeing to the license the submitted work under the following licenses:
- Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
- Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

[x] the basic tests are passing

unit tests pass (to run, use: ./runTests.py test/unit)
header checks pass, (make test-headers)
dependencies checks pass, (make test-math-dependencies)
docs build, (make doxygen)
code passes the built in C++ standards checks (make cpplint)

[x] the code is written in idiomatic C++ and changes are documented in the doxygen

[x] the new changes are tested

@SteveBronder or @bob-carpenter , everything is working well except that the expression_tests fail for multi_normal_log.
I assume that if the expression_tests got to multi_normal_lpdf it would fail also. However, if I replace those with the test in multi_normal_cholesky_lpdf (but calling `multi_normal_lpdf' at the end of the test) then the test pass.
I think the issue is that there are some legacy signatures in multi_normal_log which just calls multi_normal_lpdf. So should we add those to `multi_normal_cholesky'? Or can we remove them?

The one issue is the failing expression test. The issue is that the multi normal is testing the expression vector, array[] vector, matrix whereas the cholesky version is testing array[] vector, array[] vector, matrix. So now that the multi_normal calls the cholesky version, it is failing.
@rok-cesnovar is this a stanc required change?

The expressions are now handled properly. @bob-carpenter would you have time to review this? You definitely know more about these multi normal lpdf than me.

This is ready, @bob-carpenter will you have time to look prior to the feature freeze?

I'm heading out of town on vacation for a few days, so I won't be able to review until next Tuesday at the earliest.

No worries, enjoy your vacation!

@spinkney mind merging in develop here? We should get this in for the next release.

@rok-cesnovar can you restart the CI/jenkins process? Once that finishes I can merge in

@rok-cesnovar (or @bob-carpenter or @bgoodri) I updated this to use the ldlt decom rather than the cholesky as it's more numerically stable. I attempted to keep the ldlt factor type and use it in the mdivide_*_ldlt for the derivatives but I was getting a bunch of type errors. I resorted to creating the cholesky factor from the L and sqrt(D) multiplication.

Taking the square root is where most of the precision gained from using LDLT gets lost. There should be a way to differentiate with respect to the diagonal elements of D directly.

Taking the square root is where most of the precision gained from using LDLT gets lost. There should be a way to differentiate with respect to the diagonal elements of D directly.

Ok, then I may as well have kept it simple by taking the cholesky decomposition and passing that to multi_normal_cholesky_lpdf. The issue with that is the numerical precision of the cholesky decomp is sometimes going to fail and users will have to add a jitter to the diagonal.
The way I think this should be done is to create a multi_normal_ldlt_lpdf() that doesn't get exposed outside of stan-math. Autodiff through the ldlt then pass that into the multi_normal_ldlt function and utilize the LDLT functions to get the derivatives. My one worry is that we need the inverse of the cholesky factor L. Can we just use mdivide_left_ldlt(Sigma_ldlt) for this? Or will this introduce the sqrt problem all over again?

@spinkney, I didn't realize this was a draft. Let me know what you think.

The issue is that the cholesky decomposition is less stable than the ldlt so I'm waiting until tuples get in the language or once someone implements ldlt with derivatives.


⬆ Upgrade openssl~=1.1.1o -> openssl~=1.1.1q

User Story
As a… Cloud Ops Admin
I need/want/expect to… package versions to be up to date
Value / Purpose
No response
Useful Contacts
No response
Additional Information
No response
Definition of Done

[ ] tests passing successfully
[ ] PR created and merged, approved to production

This came up when investigating this issue: https://github.com/ministryofjustice/network-access-control-integration-tests/issues/44


Update package.json

Thanks for the PR! I see this addresses #7 filed by @victorteokw.
Can you explain why this is helpful? I would understand wanting to add a require property pointing to a CommonJS module, but since ./dist/index.mjs is an ESM bundle, how does adding a default fallback help?
(Aside: The trailing comma makes this invalid JSON, which needs to be fixed before landing.)

@Asgarrrr @victorteokw Any idea on what behavior this is actually changing and why? I'd be happy to make this change, but would first like to understand why this helps and confirm that it actually solves whatever specific tooling problem you're facing.

I can't explain the why, but it does look like something in the Vercel pipeline is trying to load this package as cjs.
cache miss, executing 887572bb27302b2a
$ next lint --format json
node:internal/modules/esm/resolve:304
  return new ERR_PACKAGE_PATH_NOT_EXPORTED(
         ^

Error: No "exports" main defined in /code/node_modules/oniguruma-to-es/package.json
    at exportsNotFound (node:internal/modules/esm/resolve:304:10)
    at packageExportsResolve (node:internal/modules/esm/resolve:594:13)
    at resolveExports (node:internal/modules/cjs/loader:634:36)
    at Module._findPath (node:internal/modules/cjs/loader:724:31)
    at Module._resolveFilename (node:internal/modules/cjs/loader:1211:27)
    at /code/node_modules/next/dist/server/require-hook.js:55:36
    at Module._load (node:internal/modules/cjs/loader:1051:27)
    at Module.require (node:internal/modules/cjs/loader:1311:19)
    at mod.require (/code/node_modules/next/dist/server/require-hook.js:65:28)
    at require (node:internal/modules/helpers:179:18) {
  code: 'ERR_PACKAGE_PATH_NOT_EXPORTED'
}

Node.js v20.18.0

🤔 the proper fix is probably reaching out to someone on the Turborepo or Next.js team about it

It appears to happen if you name your config file next.config.ts, and does not trigger if you name it next.config.mjs. So definitely a Next.js issue

Thanks for sharing that workaround, @IanMitchell. At https://github.com/shikijs/shiki/issues/809 people also mentioned working around variations of the same thing by upgrading to the latest vite or svelte.
My problem with this PR is that, short of a more detailed explanation of what exactly happens that makes this work, it sounds like adding a "default" export actually makes some pipelines treat the ESM bundle as CJS, which I suspect might cause other issues even if it works in some cases.
Most likely, adding an actual CJS bundle (and a "require" property, instead of "default") would resolve this. But I'm not sure I want to go down that road if there are simple workarounds, since a CJS bundle hasn't actually been requested thus far, and CJS would require long-term support plus changes to how types are exported.
I'm not very knowledgeable about CJS and build pipelines. Very open to feedback/PRs from people who understand this more deeply.

My guess is this package being listed is a symptom and not the core issue - if you patch something, the next package down the list will throw the same error.

Agreed. Okay, I'm going to close this PR since it seems like it's not the right fix. But hopefully the workarounds described here are helpful for anyone encountering the same issue.

My file is named next.config.mjs and I'm still getting the same error.

@msiric Are you using Vite, and if so have you upgraded to 5.x? Other people in the issue I linked fixed it it by upgrading Vite or Svelte. I'm guessing that some build tool that is a common dependency of them is the source of the problem, and needs to be updated to a newer version.

@slevithan I'm using Webpack with my Next.js app. No Svelte or Vite.

FWIW, Node.js recommends including "default":

When using environment branches, always include a "default" condition where possible. Providing a "default" condition ensures that any unknown JS environments are able to use this universal implementation, which helps avoid these JS environments from having to pretend to be existing environments in order to support packages with conditional exports. For this reason, using "node" and "default" condition branches is usually preferable to using "node" and "browser" condition branches.

Source: https://nodejs.org/api/packages.html#conditional-exports

Good insight, @wereHamster. However, the "environment branches" that quote refers to are things like "node-addons", "node", and "browser". oniguruma-to-es is not using environment branches in its conditional exports, so that doesn't apply in this case.
I'd be fine with accepting this PR if I understood (or someone could explain) why it helps avoid the issue a few people are facing, so I could have more confidence that adding "default" in addition to "import" (despite only offering an ESM bundle) wouldn't break things or lead to harder to debug issues for other setups that are incorrectly trying to load oniguruma-to-es as CJS.
Alternatively, if someone could provide a simple/reduced demo repo that is easy to clone for testing and reproduces the problem, I could try to play around with other package.json properties that might work around the issue.
However, as @IanMitchell has pointed out, the error a few people are seeing when loading this package is almost certainly a symptom and not the cause of whatever the underlying issue is in some build pipelines.

People who use next.js with ts config, when Shiki is used for code coloring, it requires this package to have a "default".


Jkt/autofill top popup

Task/Issue URL:
Tech Design URL:
CC:
Description:
Steps to test this PR:
1.
1.

OS Testing:

[ ] iOS 13
[ ] iOS 14
[ ] iOS 15
[ ] macOS 10.15
[ ] macOS 11

Internal references:
Software Engineering Expectations
Technical Design Template

Closing in favour of https://github.com/duckduckgo/BrowserServicesKit/pull/67


Fix hint message

First of all thank you for making this tool! I discovered it today and it just feels like I imagine EB CLI should look like.
I noticed that in few places there are hints pointing to old config command.

@maciejwalkowiak Thanks so much for fixing that!  And the kind words.  Merged and released: https://github.com/tongueroo/jack/blob/master/CHANGELOG.md

👍


[CHECK] Adapter info needs verification

Adapter info needs verification:
Repository iobroker-community-adapters/ioBroker.info is archived
Adapter info is currently installed 30690 times

[x] verify that repository https://github.com/iobroker-community-adapters/ioBroker.info is really archived
[x] check if adapter has been migrated but links are not yet updated at repository
fix links at repository if this is causing the problem
[x] decide if adapter should (and can) be moved to community-adapters (current usage 30690 installs)
[ ] move adapter to iobroker-community-adapters

or

[x] create deprecation news id required
[x] remove from stable
[ ] remove from latest

reminder 1.4.2024
remove from latest


fix:allow users to customize Logging fluentbit inputTail Buffer_Max_Size

Q
A

Bug fix?
yes

New feature?
no

API breaks?
no

Deprecations?
no

Related tickets
fixes #1036

License
Apache 2.0

What's in this PR?

Allow users to customize Logging fluentbit inputTail Buffer_Max_Size
Checklist

[x] Code meets the Developer Guide
[x] User guide and development docs updated (if needed)
[x] Related Helm chart(s) updated (if needed)

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

My first issue with this PR is that it is not doing what it says it does. Users can already set the input tail's Buffer_Max_Size to a custom value through the resource. If no value is provided, Fluentbit will use its own default, which is the same value.


All SWaP and Patch for MST2_EU_VW_PQ_PO369T

I would like to have support for this version in Toolbox if possible. Patch for ZR not work.
support.zip

Added https://github.com/olli991/mib-std2-pq-zr-toolbox/archive/refs/heads/master.zip

Спасибо за оперативность. Все работает.


feat: add support for dynamic imports in CJS

Summary
Since we no longer use compileFunction, we can now implement this.

Test plan
Test added

Should this work on Node.js 14.15.0 with Jest?
I have a CJS function like this one:
"use strict";

exports.default = import_;

function import_(filepath) {
  let res = import(filepath);
  throw new Error("Ok 2");
  return res;
}

When I call it with a valid filepath, Jest exits without reporting any failure (I would expect it to throw "Ok 2").
I thus tried adding require("fs").writeFileSync("/tmp/linkModules-called", "ok"); as the first line of the linkModules function (which in this PR is directly used by importModuleDynamically) to check if it's called and it seems that it is never called :thinking:

Should this work on Node.js 14.15.0 with Jest?

Definitely, as can be seen from the test here, this runs successfully on<EMAIL_ADDRESS>|| >=13.7.0

EDIT: It's a segmentation fault (core dumped), I'll try creating a smaller example and open a new issue.

That doesn't sound good 🙈 Ping me when/if you're able to put together a repository. If it's in Babel I'm happy to take a look at some branch

Does this mean that babel-plugin-dynamic-import-node wouldn't be necessary anymore, or am I misunderstanding this PR?

correct. If you use preset-env it will be disabled by default, but if you explicitly add it you can remove it


graceful failure on missing credentials

if there is an error, give the user the choice to retry or exit the session instead of bailing out

Are you talking here specifically about the lack of the config file? The current workflow is, if it does not have the config files, it tells you to re-run bedrust with the --init flag.
I will implement the ask to run init straight from the failure. 👏

no if e.g. you don't have working credentials (maybe you haven't selected the profile, etc.)
I think in general, an error that might work on retry should probably just go back into the loop and ask the user if they want to quit (so I don't have to retype my message)


Establish single connection for running commands on container

Which functionality do you think we should update/improve?
occlient.ExecCMDInContainer executes commands on the container by opening a remote shell connection for each command. Instead open a connection to a bash shell at the start of odo and re-use it for each command we need to rn.
Why is this needed? `
Performance - opening a connection (especially with SSL) can take longer than running the command itself.

Replaced by concurrency fixes in #1879


Add spm support

Another great package manager: http://spmjs.io
Documentation: http://spmjs.io/documentation
http://spmjs.io/package/list.js

It is a package manager based on Sea.js which is a popular module loader in China.
spmjs focus in browser side. We supply a complete lifecycle managment of package by using spm.

If you need ownership of list.js in spmjs, I can add it for your account after signing in http://spmjs.io.

I'm sorry but I'm not interested in this, and in the next version (1.2.0) I'll remove all external dependencies and move them into List.js itself. So I guess that some of this PR won't be needed.


Reading multiple reports (USB-HID-KBE)

Hi,
I have a device which is configured as USB-HID-KBE. It has two reports with report Ids as  0x01 and 0x02. 0x01 is designated as keyboard report ,0x02 is output report which i use to move data out from and to the device. Using  "HidDevices.Enumerate(VendorId_KBE, ProductId_KBE).FirstOrDefault();" i can only read report 0x00 which actually is not present , I cannot write to or read from report Id 0x02. what might have been the reason?
This is the report structure ;
/* Keyboard page here
HID_Usage(0x06),
HID_ReportID (0x01),
/
HID_UsagePageVendor(0x00                    ),
HID_Usage         (0x01                     ),
HID_Collection    ( HID_Application          ),
HID_ReportID 			(0x02),
HID_LogicalMin   	( 0                        ),  / value range: 0 - 0xFF /
HID_LogicalMaxS  	( 0xFF                     ),
HID_ReportSize   	( 8                        ),  / 8 bits */
HID_Usage					(0x1A),
HID_ReportCount		(HID_DECODE_STATUS_BYTE),
HID_Input					(HID_Data | HID_Variable | HID_Absolute ),
HID_Usage					(0x1B),
HID_ReportCount		(HID_OUTPUT_REPORT_BYTES_VENDOR),
HID_Output				(HID_Data | HID_Variable | HID_Absolute | HID_Array),
HID_EndCollection

@mikeobrien Can chime in if he wants, but I don't believe this library is intended to be used with a keyboard, as those are handled differently by the operating system.


Fix macOS download URL mismatch

The download URL mismatched with the architecture for macOS builds.

Good catch!


Preview Builds.

I know this project is still very early im development, but would it be possible to release weekly builds or something so users don't have to download visual studio and compile it themselves?
Anyways keep up the good work☺

This could be done via a CI like Jenkins or something, but would mean I'd have to rent a virtual machine in the cloud.
To answer your suggestion yes, automatic builds will be a thing once we have a broader feature set :)

It would be a good idea, it will be qn would be possible for you compile a version of the project and level up in a download site? I believe we have many users eager to do some testing in this gorgeous design!

I will see if I can get travis CI working

@azhang66 Travis CI doesn't support building in Windows which is required to compile windows applications, ovbs some unit testing could be done in Linux though.

@filfat  https://www.appveyor.com/ appears to support Windows build enviornments

@filfat I've added a build script that makes packages in #19


If DNS is encrypting, how is my ISP sending me cease pirating warnings?

Even running through VPN. Shouldn't DNScrypt encrypt my activity data stream back to provider?
Hope this is a suitable place to ask. Thanks

Yes it does, but only DNS requests, those are not even used when you connect to other peers say in a torrent, since those are direct IP connections.
Eg, you want to conntect to myfavouritepiratesite dot com (pictures of pirate hats that is):

request is encrypted to DNS -> ISP sees only a DNS request, not what site you asked for
you PC gets the site IP
your PC connects to that IP -> ISP sees IP
ISP has IP is does a reverse DNS and sees that IP used for myfavouritepiratesite dot com

Now, if torrents are involved, usually when an ISP send you warnings it's because they got that info from another entity, one that was usually getting the torrent and taking notes of IPs that seed/peer, again that has nothing to do with DNS.
/mark as invalid and close

Thank you so much for your response.
cheers

"your PC connects to that IP -> ISP sees IP"
What stops this? Is this still the case when you're using a third-party DNS server that is not your ISP's?
OR routing internet traffic through a VPN?

VPN yen, it will always see the VPN IP.


Saving Images byte array format instead of base64

Hi everybody,
is it possible to take a picture saving directly a byte[] or an image jpg without converting it from base64 ?
Thanks a lot!

Unfortunately, base64 is the only way to get the image out of Adobe Flash into JavaScript land, which is why that conversion is done.  However, you can get access to the underlying canvas yourself, if you want to grab pixels from that.  Instructions are here:
https://github.com/jhuckaby/webcamjs#snapping-a-picture


Update dependencies

Updated dependencies to use the latest and greatest as of this moment.

Closing the PR accidentally made here instead of my own fork.


Update README.rst

With windows 10 need to run pip install --upgrade setuptools. Haven't tried this with Linux but I figure it wouldn't hurt to make things work on both windows/linux minus the sudo apt-get

This seems okay. It shouldn't hurt anything if it is not needed.


drivers: gpio: sifive: Update to use the available common IRQ API and remove obsolete code.

Fixes issue: #20338

@henrikbrixandersen , @mnkp  would you be able to take a look?