MQTT server with Websockets and TCP together

I am reading the usage of MQTTnet server but it is not clear how to configure a server to run Websocket and TCP clients together. In our use case TCP messages needs to forwarded to websocket clients (webapp). Your examples with aspnet core websockets leaves me confused if TCP clients will be able to connect or not?

I m also interested in usage of MQTTnet server with TCP and WebSocket together.
I try.... but i cant integrate TCP client with Websocket....  anyone help us?

i found a solution.
inject a MqttServerAdapter(new MqttNetLogger())

@mircotamburini Is't better to use the broker server as a console application or in asp.net core 2 application to use it for a web client dashboard?

I use it in a asp.net core2 because i need to have integrated mqtt server tcp and websocket in a single bundle

My idea is to have the broker server as a console application for scalability and manutenabily.
I’ve to build a multi customer dashboard. So my problem is to build the client into asp.net core 2. How to? Or Is better to use a mqtt js client?

Hey guys,
I found this issue and tried to do the same.
Is it possible to share the code samples with us or prepare an example using MQTTnet with both TCP and WebSockets?

Hola chicos,
encontré este problema e intenté hacer lo mismo.
¿Es posible compartir los ejemplos de código con nosotros o preparar un ejemplo usando MQTTnet con TCP y WebSockets?

The first thing is to develop a server with websocket. Did you get to that point?
Note: Go to https://github.com/chkr1011/MQTTnet/wiki/Tips-and-Tricks (End of page)

Hola chicos,
encontré este problema e intenté hacer lo mismo.
¿Es posible compartir los ejemplos de código con nosotros o preparar un ejemplo usando MQTTnet con TCP y WebSockets?

The first thing is to develop a server with websocket. Did you get to that point?
Note: Go to https://github.com/chkr1011/MQTTnet/wiki/Tips-and-Tricks (End of page)

Well, basically I don't know where to start :/
I implemented a "standard" mqtt-server based on the code snippets provided here which is running fine. Now my task would be to add additional support for websockets which I tried to do based on the test application but there is no point to refer to an existing/running instance of the mqtt-server.
Do you know what I mean? Or do I need to start from a different point?


Update dependencies

Coverage remained the same at 96.25% when pulling 006dd402a6fe7112a59c2c455150de2fae0c3bfb on update-dependencies into 3d1c03033d9d92e7dfa284f31cebea929202d3b9 on master.


Failed to retrieve file names from artifact

Hello,
I have problem adding your action to my CI workflows:
I have 2 workflows
First go CI, which builds app, run tests and upload test results and coverage as artifacts. This one is complete with success state and uploads artifacts:

After workflow [CI] is completed my second workflow is triggered, which should process tests and coverage reports:
name: 'Test Report and Code Coverage'

on:
  workflow_run:
    workflows: ["CI"]
    types: [completed]

jobs:
  report:
    name: Workflow test
    runs-on: ubuntu-latest
    steps:
    - name: .NET Tests Results
      uses: dorny/test-reporter@v1
      with:
        artifact: test-results
        name: .NET Tests results
        path: '**/**/*.trx'
        reporter: dotnet-trx
    - name: Code Coverage
      uses: zgosalvez/github-actions-report-lcov@v1
      with:
        artifact-name: code-coverage-report
        name: Code Coverage
        coverage-files: '**/TestResults/coverage.info'
        minimum-coverage: 0
        github-token: ${{ secrets.GITHUB_TOKEN }}

While tests reported is working as expected (so artifacts seems to be working), code coverage isn't.
It is failing with following error:

I was playing witch changing .info file retrieving patterns, but doesn't help.
What is interesting is that when I had code coverage inside the same workflow as running tests it was working.
Any idea what's the problem here?
Links:

Tests and coverage processing action log: https://github.com/Stanislaw000/ONIX-Data/runs/3244608588?check_suite_focus=true
Build, test and artifacts upload action log: https://github.com/Stanislaw000/ONIX-Data/runs/3244213227?check_suite_focus=true
Coverage processing workflow: https://github.com/Stanislaw000/ONIX-Data/blob/NetStandard/.github/workflows/test-report.yml (default branch version, because this one is always working as being triggered by workflow complete)
CI workflow: https://github.com/Stanislaw000/ONIX-Data/blob/Test-Report-and-Code-Coverage-CI-improve/.github/workflows/ci.yml

Hi @Stanislaw000! You'll see at https://github.com/zgosalvez/github-actions-report-lcov/blob/49af65b0586c274a625a41a2dc8aaaae7d00568b/src/main.js#L60-L65 that it's expecting inputs right after /usr/bin/genhtml, before the --output-directory, which you can see working here.
Please try modifying the coverage-files input like so: TestResults/coverage.info, assuming that the TestResults folder is generated in the root of your working directory.

Hello,
Thank you for your answer! I have figured out that I have miss read action parameters and I was assuming that  was input field instead of output.
Thanks for making it clear!


Add appropriate project badges to this repo's README

Each of the Meshery repos include a set of project badges.
Please add the applicable set of badges to this repo's readme.
Example badges from main meshery repo:

@arturba would you like to grab this one?

Addressed by @gunjanmimo. Thanks!


[stable/prometheus-blackbox-exporter] prometheusrule additionalLabels have too many indents/spaces

Is this a new chart
No
What this PR does / why we need it:
Too may spaces before the label value is causing this error:
Error: UPGRADE FAILED: YAML parse error on prometheus-blackbox-exporter/templates/prometheusrule.yaml: error converting YAML to JSON: yaml: line 9: mapping values are not allowed in this context

Because of this error I can't add labels to integrate with prometheus-operator.
Special notes for your reviewer:
Checking a similar template for another project, moving the templated values four spaces to the left should solve it. See:
https://github.com/helm/charts/blob/master/stable/airflow/templates/prometheus-rule.yaml
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[ ] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/assign @desaintmartin

Thanks!
/ok-to-test
/lgtm


Fix request schema on send-ping

Codecov Report

Merging #290 into master will increase coverage by <.01%.
The diff coverage is 85.71%.

@@            Coverage Diff             @@
##           master     #290      +/-   ##
==========================================
+ Coverage   81.64%   81.64%   +<.01%
==========================================
  Files         232      232
  Lines       10785    10789       +4
==========================================
+ Hits         8805     8809       +4
  Misses       1980     1980


Not support Django 4.0

https://docs.djangoproject.com/en/4.0/releases/4.0/#:~:text=django.utils.encoding.force_text() and smart_text() are removed.
https://github.com/jazzband/django-pipeline/blob/86283262bbbeb9c55bbf265456c8444a122c681a/pipeline/compressors/__init__.py#L10
There is an invalid function that is removed in django4.0..
We must replace django.utils.encoding.force_text() to django.utils.encoding.force_str()
Hope to see a fast upgrade for django4.0 support

There's an approved PR https://github.com/jazzband/django-pipeline/pull/760 for this, just waiting for it to be merged and released (maybe @cyberdelia or @TheBuky will have some time?)

Thanks for your reply.
I am using django-pipeline package in our company projects. We started to
upgrade django4.0. We are waiting for new release version for django4.0
support.
Hope to get news as soon as possible.
Regards,
Joe
On Mon, Jan 10, 2022 at 11:40 PM Kevin Marsh @.***>
wrote:

There's an approved PR #760
https://github.com/jazzband/django-pipeline/pull/760 for this, just
waiting for it to be merged and released (maybe @cyberdelia
https://github.com/cyberdelia or @TheBuky https://github.com/TheBuky
will have some time?)
—
Reply to this email directly, view it on GitHub
https://github.com/jazzband/django-pipeline/issues/766#issuecomment-1009005851,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVBCEC6XA52FFVYTCRJJYD3UVL4YHANCNFSM5LQY4ZTQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

Thank for the PR and the support of Django 4.0
I have start to work on it last week, but I get some trouble documented as #765 with an old python package to minify JS. I will drop the support for slimit, we should past tests and the packages will be release.

All tests past after slimit removed but the automated task to push the release 2.0.8 failed... File already exists.
I report the issue.

All tests past after slimit removed but the automated task to push the release 2.0.8 failed... File already exists.
I report the issue.
Many thinks have change in Jazzband, now it's the project lead to see that. Can you look why an error @cyberdelia happen during the publishing process?

@TheBuky Looks like it's conflicting with the previously unreleased 2.0.8 you did a few days ago, I removed that upload and re-released.

Oh my bad, I didn't think the previous unreleased version could block this one. Thank for fixing it!
The package is now on Pypi @markflorence728


flutter_localizations depends on outdated version of material_color_utilities

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce

Create a Flutter project that declares Flutter 3.16 and material_color_utilities 0.8.0 as dependencies.
Try to compile it

Expected results
It compiles without issues
Actual results
It fails to compile, producing the following message:
Note: material_color_utilities is pinned to version 0.5.0 by flutter_localizations from the flutter SDK.
See https://dart.dev/go/sdk-version-pinning for details.
Because gofuntime depends on flutter_localizations from sdk which depends on material_color_utilities 0.5.0, material_color_utilities 0.5.0 is required.
So, because gofuntime depends on material_color_utilities ^0.8.0, version solving failed.
You can try the following suggestion to make the pubspec resolve:

Consider downgrading your constraint on material_color_utilities: flutter pub add material_color_utilities:^0.5.0

Code sample
Code sample
name: gofuntime
description: Go Fun Time!

# The following line prevents the package from being accidentally published to
# pub.dev using `flutter pub publish`. This is preferred for private packages.
publish_to: 'none' # Remove this line if you wish to publish to pub.dev

# The following defines the version and build number for your application.
# A version number is three numbers separated by dots, like 1.2.43
# followed by an optional build number separated by a +.
# Both the version and the builder number may be overridden in flutter
# build by specifying --build-name and --build-number, respectively.
# In Android, build-name is used as versionName while build-number used as versionCode.
# Read more about Android versioning at https://developer.android.com/studio/publish/versioning
# In iOS, build-name is used as CFBundleShortVersionString while build-number used as CFBundleVersion.
# Read more about iOS versioning at
# https://developer.apple.com/library/archive/documentation/General/Reference/InfoPlistKeyReference/Articles/CoreFoundationKeys.html
version: 3.2.0

environment:
  sdk: ">=2.17.1 <4.0.0"
  flutter: ^3.16.0

# Dependencies specify other packages that your package needs in order to work.
# To automatically upgrade your package dependencies to the latest versions
# consider running `flutter pub upgrade --major-versions`. Alternatively,
# dependencies can be manually updated by changing the version numbers below to
# the latest version available on pub.dev. To see which dependencies have newer
# versions available, run `flutter pub outdated`.
dependencies:
  flutter:
    sdk: flutter

  flutter_localizations:
    sdk: flutter

  material_color_utilities: ^0.8.0

dev_dependencies:

  # The "flutter_lints" package below contains a set of recommended lints to
  # encourage good coding practices. The lint set provided by the package is
  # activated in the `analysis_options.yaml` file located at the root of your
  # package. See that file for information about deactivating specific lint
  # rules and activating additional ones.
  flutter_lints: ^3.0.0

# For information on the generic Dart part of this file, see the
# following page: https://dart.dev/tools/pub/pubspec

# The following section is specific to Flutter packages.
flutter:
  generate: true

  # The following line ensures that the Material Icons font is
  # included with your application, so that you can use the icons in
  # the material Icons class.
  uses-material-design: true

  assets:
    - google_fonts/

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.16.0, on macOS 14.1.1 23B81 darwin-arm64, locale en-DE)
    • Flutter version 3.16.0 on channel stable at /Users/tilo/development/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision db7ef5bf9f (11 days ago), 2023-11-15 11:25:44 -0800
    • Engine revision 74d16627b9
    • Dart version 3.2.0
    • DevTools version 2.28.2

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /Users/tilo/Library/Android/sdk
    • Platform android-33, build-tools 33.0.0
    • ANDROID_SDK_ROOT = /Users/tilo/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)
    • All Android licenses accepted.

[!] Xcode - develop for iOS and macOS (Xcode 15.0.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15A507
    ✗ Unable to get list of installed Simulator runtimes.
    • CocoaPods version 1.13.0

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)

[✓] IntelliJ IDEA Community Edition (version 2023.2.5)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 76.3.4
    • Dart plugin version 232.10248

[✓] VS Code (version 1.83.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.74.0

[✓] Connected device (3 available)
    • Pixel 7 (mobile) • adb-2B041FDH2002Y6-M6ucHs._adb-tls-connect._tcp. • android-arm64  • Android 14 (API 34)
    • macOS (desktop)  • macos                                            • darwin-arm64   • macOS 14.1.1 23B81 darwin-arm64
    • Chrome (web)     • chrome                                           • web-javascript • Google Chrome 119.0.6045.159

[✓] Network resources
    • All expected network resources are available.

Thanks for the report @tiloc
I tried the same using latest stable version by running flutter pub add material_color_utilities which used ^0.5.0 by default and showed below in the log:
< material_color_utilities 0.5.0 (was 0.8.0) (from transitive dependency to direct dependency) (0.8.0 available)
so I think if we specifically try to add 0.8.0 version, then it'll show the error as you reported, but if we add that plugin using the command as mentioned above, it takes the required version with which it compiles properly.
Keeping the issue open for team's input.

 stable, master flutter doctor -v
[!] Flutter (Channel stable, 3.16.0, on macOS 12.2.1 21D62 darwin-x64, locale
    en-GB)
    • Flutter version 3.16.0 on channel stable at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision db7ef5bf9f (12 days ago), 2023-11-15 11:25:44 -0800
    • Engine revision 74d16627b9
    • Dart version 3.2.0
    • DevTools version 2.28.2
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

[!] Flutter (Channel master, 3.17.0-15.0.pre.29, on macOS 12.2.1 21D62
    darwin-x64, locale en-GB)
    • Flutter version 3.17.0-15.0.pre.29 on channel master at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision c45f583799 (19 minutes ago), 2023-11-27 00:04:28 -0500
    • Engine revision 292a92166a
    • Dart version 3.3.0 (build 3.3.0-152.0.dev)
    • DevTools version 2.30.0-dev.4
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/dhs/Library/Android/sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for
      more details.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 13C100
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] IntelliJ IDEA Ultimate Edition (version 2021.3.2)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 65.1.4
    • Dart plugin version 213.7228

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.29.0

[✓] Connected device (3 available)
    • Darshan's iphone (mobile) • 21150b119064aecc249dfcfe05e259197461ce23 • ios
      • iOS 15.3.1 19D52
    • macOS (desktop)           • macos                                    •
      darwin-x64     • macOS 12.2.1 21D62 darwin-x64
    • Chrome (web)              • chrome                                   •
      web-javascript • Google Chrome 109.0.5414.119

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

Looks like this is already fixed on the main branch:
https://github.com/flutter/flutter/blob/60d5c8abc5d362c2fc57d5623a58ca68cf836738/packages/flutter_localizations/pubspec.yaml#L16
I am going to close this as there isn't anything else we can do. You can wait for the release or upgrade to the main brnach.


Large gitea-actions stops running at ~3hours

Description
I'm running gitea-actions for some linux image building task. It takea hours to finish on a regular machine. The gitea-action job stops at around 3 hours after begin, without any reported error on the webui. Build a smaller image, if the job is able to complete within 3 hours, it runs successfully. I already allocate an 12-core VM on a modern CPU to run the actions, there's almost no way I could increase the performance.
Is there a hard limit on how long a single job can run? Is there any way to configure the limit?
(not sure if this is the right place to report bugs about actions / runners...)
Thanks
Gitea Version
1.21.6
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
No response
Screenshots

Git Version
No response
Operating System
Debian bullseye
How are you running Gitea?
Official docker image
Database
MySQL/MariaDB

I think you have kept the default actions timeout
Take a look at the config file and update the ENDLESS_TASK_TIMEOUT
https://docs.gitea.com/administration/config-cheat-sheet?_highlight=config#actions-actions

That is the solution. Thanks

Will add more context, because I has been hit by same behavior. In my case increasing ENDLESS_TASK_TIMEOUT was not enough because runner has it's own runner.timeout setting


Add a License

Thank you for creating and sharing this great project! I noticed that this repository does not currently have a license. Adding a license would clarify how others can use, modify, and distribute the code. It would also help potential contributors know the terms under which they can contribute.

Thanks for your feedback @Naveenraj006
Do you recommend what license should i apply to this project? Yes, I'd like to share to everyone and they can use this to whatever purpose they want

@Naveenraj006 added MIT license

thanks.


Make VMWare user credentials

What happened:
I note that with the make vmware scripts the vms do not auto-login
What you expected to happen:
Former make kvm they auto-login
How to reproduce it (as minimally and precisely as possible):
make vm
load ova in vmware
Anything else we need to know
I feel like I am doing something wrong, is there a default cred, should I be passing credentials in the script, is there a default?
Environment:
esxi 7.0 Update 2

there are NO credentials et all :-D ... i mean for safety, but it usually uses client tool of the hyperscaler / hypervisor to get the credentials propagated from outside.
If you build with the _dev feature it adds an autologin - either on the console drop dead stupid, or with a generated key for ssh.

I am installing VCenter Server, I am assuming this would give me the ability to propagate credentials.

This is a user error, when creating the vm I only used the ova, and did not include vmdk and raw file


Got 'generator' object is not subscriptable chainlit when do streaming call

chainlit version: 0.7.1
langchain version: 0.0.301
below is the content of app.py which will get 'generator' object is not subscriptable chainlit  if  explicitly set stream=True .
from langchain import PromptTemplate, LLMChain
import chainlit as cl
import os
import dotenv
from langchain.llms import AzureOpenAI

dotenv.load_dotenv()
openai_key = os.getenv("OPENAI_API_KEY")

template = """Question: {question}

Answer: Let's think step by step."""
llm = AzureOpenAI(
    temperature=0,
    deployment_name="openai-text-davinci-003",
    model_name="openai-text-davinci-003",
    openai_api_base="https://m1nlp-chatgpt-resources.openai.azure.com/",
    openai_api_version="2022-12-01",
    openai_api_key=openai_key,
    openai_api_type="azure",
    stream=True,
)

@cl.on_chat_start
def main():
    # Instantiate the chain for that user session
    prompt = PromptTemplate(template=template, input_variables=["question"])
    # llm_chain = LLMChain(prompt=prompt, llm=OpenAI(temperature=0), verbose=True)
    llm_chain = LLMChain(
        prompt=prompt,
        llm=llm,
        verbose=True,
    )

    # Store the chain in the user session
    cl.user_session.set("llm_chain", llm_chain)

@cl.on_message
async def main(message: str):
    # Retrieve the chain from the user session
    llm_chain = cl.user_session.get("llm_chain")  # type: LLMChain
    res = await cl.make_async(llm_chain)(
        message, callbacks=[cl.LangchainCallbackHandler()]
    )
    await cl.Message(content=res["text"]).send()
    return llm_chain

Was able to reproduce. Seems that Langchain changed how to import llms.
I fixed it by changing the import from from langchain.chat_models to from langchain.chat_models and setting streaming=True instead of stream=True.
from langchain.llms import OpenAIChat

template = """Question: {question}

Answer: Let's think step by step."""
llm = OpenAIChat(temperature=0, streaming=True)

when are you going to release a new version? I want to try some new features that you merged recently

Target is early next week

Sounds good. It would be great if you can release the new version this
Friday.  BTW, can we run chainlit app on mobile device (like: Android
phone)?
Thanks
Xiaopei
On Thu, Oct 5, 2023 at 9:32 AM Willy Douhard @.***>
wrote:

Target is early next week
—
Reply to this email directly, view it on GitHub
https://github.com/Chainlit/chainlit/issues/444#issuecomment-1748909395,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEK6EW3UWX2UJYKCSCBE7BLX52ZIFAVCNFSM6AAAAAA5QJXFBSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONBYHEYDSMZZGU
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Xiaopei   Wu

Target is early next week

@willydouhard  Will you release a new version on Monday@10/9 ?

Target is early next week

@willydouhard Will you release a new version on Monday@10/9 ?

Probably tomorrow (Tuesday 10/10)


Currently doesn't compile using Clang/GCC

I will be fixing issues with these compilers, if need be. I am unsure if the compiler I tried when receiving  build errors was the latest version or not, but nonetheless I will test this again shortly.

Confirmed compilation using GCC. A lot of the files were restructured so, just in case, there's a backup branch from before this change was put into affect.

Confirmed compilation using GCC. A lot of the files were restructured so, just in case, there's a backup branch from before this change was put into affect.


freerdp: 2.3.2 -> 2.4.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/FreeRDP/FreeRDP/releases.
meta.description for freerdp is: "A Remote Desktop Protocol Client"
meta.homepage for freerdp is: "https://www.freerdp.com/"
meta.changelog for freerdp is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

/nix/store/wnqnsp3l4zd59rg1hy7v83sj36djfpmx-freerdp-2.4.0/bin/winpr-makecert passed the binary check.

Warning: no invocation of /nix/store/wnqnsp3l4zd59rg1hy7v83sj36djfpmx-freerdp-2.4.0/bin/winpr-hash had a zero exit code or showed the expected version

Warning: no invocation of /nix/store/wnqnsp3l4zd59rg1hy7v83sj36djfpmx-freerdp-2.4.0/bin/xfreerdp had a zero exit code or showed the expected version

Warning: no invocation of /nix/store/wnqnsp3l4zd59rg1hy7v83sj36djfpmx-freerdp-2.4.0/bin/wlfreerdp had a zero exit code or showed the expected version

/nix/store/wnqnsp3l4zd59rg1hy7v83sj36djfpmx-freerdp-2.4.0/bin/freerdp-shadow-cli passed the binary check.

/nix/store/wnqnsp3l4zd59rg1hy7v83sj36djfpmx-freerdp-2.4.0/bin/freerdp-proxy passed the binary check.

1 of 6 passed binary check by having a zero exit code.

3 of 6 passed binary check by having the new version present in output.

found 2.4.0 with grep in /nix/store/wnqnsp3l4zd59rg1hy7v83sj36djfpmx-freerdp-2.4.0

found 2.4.0 in filename of file in /nix/store/wnqnsp3l4zd59rg1hy7v83sj36djfpmx-freerdp-2.4.0

directory tree listing: https://gist.github.com/67558c336c0006d94273d984bd0e35e6

du listing: https://gist.github.com/b66bfb70c1065d2638e511253c0b5b6c

Rebuild report (if merged into master) (click to expand)

61 total rebuild path(s)

20 package rebuild(s)

20 x86_64-linux rebuild(s)
18 i686-linux rebuild(s)
5 x86_64-darwin rebuild(s)
18 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
brutespray
crowbar
freerdp
freerdpUnstable
gnome-connections
gnome.gnome-boxes
gnome.gnome-control-center
gnome.gnome-remote-desktop
gnome.gnome_control_center
gtk-frdp
libsForQt5.krdc
libsForQt512.krdc
libsForQt514.krdc
libsForQt515.krdc
medusa
phosh
plasma5Packages.krdc
remmina
virt-manager-qt
weston

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/wnqnsp3l4zd59rg1hy7v83sj36djfpmx-freerdp-2.4.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A freerdp https://github.com/r-ryantm/nixpkgs/archive/9730596ace05443a9f271b8188e978c5c8ac8d68.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/wnqnsp3l4zd59rg1hy7v83sj36djfpmx-freerdp-2.4.0
ls -la /nix/store/wnqnsp3l4zd59rg1hy7v83sj36djfpmx-freerdp-2.4.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package marked as broken and skipped:

libsForQt512.krdc

15 packages built:

brutespray
crowbar
freerdp
gnome-connections
gnome.gnome-boxes
gnome.gnome-control-center
gnome.gnome-remote-desktop
gtk-frdp
krdc (libsForQt5.krdc)
libsForQt514.krdc
medusa
phosh
remmina
virtmanager-qt (virt-manager-qt)
weston

Maintainer pings
cc @peterhoeg @lheckemann for testing.

Result of nixpkgs-review pr 131919 run on x86_64-linux 1

1 package marked as broken and skipped:

libsForQt512.krdc

15 packages built:

brutespray
crowbar
freerdp (freerdpUnstable)
gnome-connections
gnome.gnome-boxes
gnome.gnome-control-center (gnome.gnome_control_center)
gnome.gnome-remote-desktop
gtk-frdp
libsForQt5.krdc (libsForQt515.krdc ,plasma5Packages.krdc)
libsForQt514.krdc
medusa
phosh
remmina
virt-manager-qt
weston

Result of nixpkgs-review pr 131919 at 9730596a run on aarch64-linux 1

3 packages skipped due to time constraints:

gnome.gnome-boxes
libsForQt512.krdc
virt-manager-qt

11 packages built successfully:

crowbar
freerdp (freerdpUnstable)
gnome-connections
gnome.gnome-control-center (gnome.gnome_control_center)
gnome.gnome-remote-desktop
gtk-frdp
libsForQt5.krdc (libsForQt515.krdc ,plasma5Packages.krdc)
libsForQt514.krdc
phosh
remmina
weston


fix(editissuecomment, textinput): Fix height rendering for Android

For #423
onContentSizeChange doesn't seem to fire when the component mounts @jouderianjr so adding a height property seems to fix it

Going to merge this in as part of 1.3.1 (please feel free to comment if I should have done it differently however)


Get streamer identity from Twitch token

twitch token should include login of who it belongs to. Use this in place of streamer configuraiton key.
Alternatively it might be possible to fall back on [irc] channel and strip the hash prefix.

Done.


Uncaught Error: ENOENT, open 'undefined//Users/StruckRug67/Library/Mobile Documents/com~apple~CloudDocs/Projects/project.json'

[Enter steps to reproduce below:]

Open atom
Go to File > Open... > Select "projects" folder > Open random file from tree
The file opens, but several red error balloons show up as well.

Atom Version: 0.166.0
System: Mac OS X 10.10.1
Thrown From: Atom Core
Stack Trace
Uncaught Error: ENOENT, open 'undefined//Users/StruckRug67/Library/Mobile Documents/com~apple~CloudDocs/Projects/project.json'
At fs.js:75

Error: ENOENT, open 'undefined//Users/StruckRug67/Library/Mobile Documents/com~apple~CloudDocs/Projects/project.json'
  at Error (native)

Commands
 11x -0:00.0 core:close (atom-text-editor.editor)

Config
{
  "core": {
    "disabledPackages": [
      "linter",
      "linter-puppet"
    ],
    "projectHome": "/Users/StruckRug67/Projects"
  },
  "editor": {
    "fontFamily": "source code pro",
    "fontSize": 12,
    "showIndentGuide": true
  }
}

Installed Packages
# User
language-docker, v1.1.2
language-dockerfile, v0.1.0
language-puppet, v0.10.0
linter-puppet-lint, v0.2.3
save-session, v0.11.2

# Dev
No dev packages

This is a duplicate of https://github.com/atom/atom/issues/4854

mpeterson2/save-session#43


Add category (Remap a multiple combination of keyboard keys to a single keyboard key) in PowerToys / keyboard manager.

[###] Description of the new feature / enhancement
Add category (Remap a multiple combination of keyboard keys to a single keyboard key) in PowerToys / keyboard manager.
To (assign) a custom multiple combination of this type of keys:
Alphabet Keys
Number Keys
Punctuation Keys
Function Keys
Navigation Keys
to make a word, sentence or custom text format to a single key
example text format:
(Ds12/*./  rgD_)
(Zr85/-, /_)
(Ze1//)
(Zx12/32/23)
location example:
C:\Windows\7z1900.exe
Scenario when this would be used?
On the job when you need to insert or copy a few different text formats or folder locations as a text to a program or email...
So that there is no need to open notepad and copy long texts of different specific formats to a program that doesn't have history for last few copied inputs or programs that doesn't allow copying text and you need to input text by manually tipping.
I can simply assign custom text to a specific keyboard key and enter custom text to any program that i need with a single keyboard key...
In PowerToys / Keyboard Manager currently i can only remap a:
Remap a key
Remap keys to other keys or shortcuts
(remap single key for a single key)

Remap a shortcut
Remap shortcuts to other shortcuts or keys for all or specific applications

version

Supporting information

Yes this would be great and I have been looking for this too!
Can I add my use case to this?
I am a Vim user on both Linux and Windows.
When working with Vim / Neovim it's very useful to remap the Caps Lock key to something else, like Escape.
However, this would completely disable Caps on/off functionality.
Therefore you also want to have another way of actually doing this.
Therefore, on Linux I am using the following command to both map Caps Lock to Escape as well to map the combination Shift + Caps Lock to the regular caps on/off behaviour: setxkbmap -option caps:escape_shifted_capslock
I am looking for a compatible solution on Windows as well.
Remapping Caps Lock to Escape already works in PowerToys. So that's already great. 🙂
However, Shift + Caps Lock is a combined keybinding, with only a signle key as it's target: Caps Lock.
It would be really great if this was possible with Power Toys.
Thanks in advance!

Thank you so much for this option:)


Fix OSGI package exports

We do not properly set package exports for our osgi bundles
a good example could be:
<plugin>
  <groupId>org.apache.felix</groupId>
  <artifactId>maven-bundle-plugin</artifactId>
  <extensions>true</extensions>
  <version>3.0.1</version>
  <configuration>
    <instructions>
      <Export-Package>io.tracee.*</Export-Package>
    </instructions>
  </configuration>
</plugin>

However, we should restrict the export to those packages who are acutally public api of each module.

@SvenBunge
Wondering if the mentioned IntelliJ warnings are not false positives. The generated bundle descriptors do not look to bad. example:
Manifest.MF from tracee-core
Manifest-Version: 1.0
Bnd-LastModified:<PHONE_NUMBER>021
Build-Jdk: 1.8.0_45
Built-By: daniel
Bundle-Description: Please refer to https://github.com/tracee/tracee.
Bundle-DocURL: http://www.holisticon.de/
Bundle-License: http://en.wikipedia.org/wiki/BSD_licenses#3-clause_licen
 se_.28.22Revised_BSD_License.22.2C_.22New_BSD_License.22.2C_or_.22Modif
 ied_BSD_License.22.29
Bundle-ManifestVersion: 2
Bundle-Name: tracee-core
Bundle-SymbolicName: io.tracee.core
Bundle-Vendor: Holisticon AG
Bundle-Version: 1.1.3.SNAPSHOT
Created-By: Apache Maven Bundle Plugin
Export-Package: io.tracee.backend.slf4j;version="1.1.3";uses:="io.tracee
 ,io.tracee.spi",io.tracee;version="1.1.3";uses:="io.tracee.configuratio
 n,javax.xml.namespace",io.tracee.configuration;version="1.1.3",io.trace
 e.transport;version="1.1.3";uses:="javax.xml.soap,javax.xml.transform,o
 rg.w3c.dom",io.tracee.transport.jaxb;version="1.1.3";uses:="javax.xml.b
 ind.annotation"
Import-Package: io.tracee.spi;version="[1.1,2)",javax.xml.bind,javax.xml
 .bind.annotation,javax.xml.namespace,javax.xml.soap,javax.xml.transform
 ,org.slf4j;version="[1.6,2)",org.w3c.dom
Require-Capability: osgi.ee;filter:="(&(osgi.ee=JavaSE)(version=1.6))"
Tool: Bnd-<IP_ADDRESS>501161923

I've opened the module with the warnings again on different computer and the warnings are not shown. So I close the issue for now.

I think the IDE is just unhappy because the package exports do not include the -SNAPSHOT modifier. They should be okay once we release a proper version without suffix.


Possible flickering menu fix

Fix for flickering menu (1920x1080 resolution problem)
This problem cant be replicated on the local server for some reason... But I tried to fix it using User CSS (browser extension) and it seems that editing this part of CSS solved the problem. (Tested several times)

.deploy

Thanks for the PR, but this doesn't seem to fix the issue to me.

.deploy

.deploy main


Handle Tab Groups when opening a paper in a new tab

The current implementation opens a new tab outside of any group (rightmost position). I would expect to open a paper in the current group. Or at least make it an option in the popup menu.

I may have misunderstand
https://github.com/julien-blanchon/PaperMemory/tree/178-Handle-Tab-Groups


add i8 support

Hi there, thanks for an excellent lib :+1:
I'm building an xmlrpc interface to rTorrent and it sends i8 values on top of i4, which I patched in.
More PR's may come as I go.

Hi SchizoDuckie,
Thank you for your support and your PR. It have been merged into the project. :)

Excellent, thanks!


Suggestion - Move up and move down buttons

Hi,
Theses buttons ease on edit mode for example. What do you think about it?

Hey @andersonfelix,
i don't think that this fits to this package.

Ok. It makes sense.
What do you recommend? A fork?

Yes a fork would be a solution


[Premier League] Update Standings, metadata

Description

Update Standings view
Update metadata

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Hi @pernielsentikaer
Thanks for merging. I just have a question: Is it possible to have more than 1 dropdown in the search bar accessory? It will be made this extension better


feat: add custom method toJSON for _LocalStream

The main purpose of this update is to make better work with LocalStream and smoother debugging for webex-js-sdk. Mainly it should get rid of custom overrides for objects and give more useful information in better structure. See current overrides here https://github.com/webex/webex-js-sdk/pull/3148/commits/4089648179ee2566cab1e7e718c6ac5908c83eb7
This PR consists of the following updates:

update for _LocalStream object and add custom toJSON method
added new test cases for _LocalStream

Code looks fine, but I'll defer to @marcin-bazyl to see if the output of the toJSON function matches his expectations. I noticed the current workaround in the SDK includes label, readyState, and enabled (though I'd argue we should use muted instead for the last one).

yeah, good point, it would be good to include label, readyState, enabled and muted from the inputTrack

@marcin-bazyl @bbaldino @brycetham here is an example from the console of how it looks like for the test-app. Please let me know @marcin-bazyl  if it is what you need.

Thanks, guys, for your review. Merge it 👍


Compatibility problems of Java8

I suggest to not use Java8 syntax which will cause compatibility problems in many old apps.
https://github.com/flutter/engine/blob/master/shell/platform/android/io/flutter/view/AccessibilityBridge.java
https://github.com/flutter/engine/blob/master/shell/platform/android/io/flutter/plugin/platform/PlatformViewsController.java

Hello @SupSaiYaJin !
Thank you for posting.
Can you please provide a bit more information? Some example that demonstrates this issue would be very helpful. E.g. self-contained repository with this compatibility issue.
Thank you.

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Could everyone who still has this problem please file a new issue with the exact descriptions what happens, logs and the output of ' flutter doctor -v'  please.
All system setups can be slightly different so its always better to open new issues and reference related issues.


Restyle ScrollViewer in menus.

What does the pull request do?
Restyle the ScrollViewer that appears in menus. Now appears as up and down button at the top and bottom of the menu when needed. This follows WPF's menu scrolling behavior.

Note
Only added to the default theme - @danwalmsley should i update the fluent theme now, or will you do it when you fix menus in the fluent theme?

@grokys I will manually port it across to fluent, so dont worry about fluent theme.


Tailwind CSS v4.0 support

with the Tailwind CSS v4.0 beta, this error is given regardless of using the config file or the new CSS-first approach to import this plugin, appreciate anyone could take a look!
[plugin:@tailwindcss/vite:generate:serve] Package subpath './lib/util/flattenColorPalette' is not defined by "exports" in /Users/.../Documents/.../node_modules/.pnpm/tailwind-scrollbar<EMAIL_ADDRESS>

https://github.com/tailwindlabs/tailwindcss/pull/15318 this should fix the bug

tailwindlabs/tailwindcss#15318 this should fix the bug, needs to confirm once it's out

This error is resolved at<EMAIL_ADDRESS>but at the same time the following different errors occur
Package subpath './lib/util/toColorValue' is not defined by "exports" in repo/node_modules/.pnpm/tailwind-scrollbar<EMAIL_ADDRESS>
This toColorValue function does not exist in the tailwind v4 code, so it may need to be addressed on the tailwind-scrollbar side

toColorValue is the following function. If the argument is a string, it is considered to work without this function.
export default function toColorValue(maybeFunction) {
   return typeof maybeFunction === 'function' ? maybeFunction({}) : maybeFunction
}

The following two locations are used in this library.
Since both pass strings, they probably do not need to be used.

https://github.com/adoxography/tailwind-scrollbar/blob/42cae713a0fe3e1aae8dd38907f9193b9441d443/src/utilities.js#L125-L128
In this case, there is no difference between colors and themeColors because flattenColorPalette returns Record<string, string>.

https://github.com/adoxography/tailwind-scrollbar/blob/42cae713a0fe3e1aae8dd38907f9193b9441d443/src/utilities.js#L130-L134
In this case, there is no difference between color and value because the object of the first argument of matchUtilities has the value of type (value: string, extra: { modifier: string | null }) => CssInJs | CssInJs[].

thanks!

So, for full transparency here, I'm going to drag my heels on officially supporting v4 until I can get the package's test suite running using their new architecture. The way Tailwind is actually invoked has changed in v4 (i.e. it's not just a postcss plugin anymore), and that means that none of the tests are presently able to run.
Although it may seem like a simple little plugin, it's shockingly difficult to target pseudoclasses of pseudoelements (think ::-webkit-scrollbar-thumb:hover) in Tailwind. As a result, this plugin has to do some manipulation on the postcss AST that Tailwind generates, and that's a little dangerous; it's easy to create scenarios like #101 where that manipulation inadvertently messes up the CSS output from completely unrelated packages. I have tests in the test suite that should prevent any regressions, and I want confirmation from them before I go about releasing anything.
I'll finally have some down time at work this month, so I'm hoping to find time to do some reading on how the authors intend Tailwind to be invoked now


underwater environment in isaac sim.

Hello, I am trying to simulate the underwater environment and underwater robot on isaac sim. Can you kindly share the files and some instructions on how to accomplish this as well (as shown in https://www.youtube.com/watch?v=qnOuPrKCIss)?

yes, all files are on this package now


feat(errorHandler): async error handling for watchers (fix #9625 #10009)

Added async error handling for watchers and immediate watchers
What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[x] Feature
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:

[x] It's submitted to the dev branch for v2.x (or to a previous version branch), not the master branch
[x] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[x] All tests are passing: https://github.com/vuejs/vue/blob/dev/.github/CONTRIBUTING.md#development-setup
[x] New/updated tests are included

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:
Fixes #9625
Fixes #10009

I'm waiting for this.
@hello-brsd you should squash your commits into one

no need to squash your commits, we already do when merging 😉

@posva Thank you for the review, i added some extra test case to the errorCaptured.spec.js.

@posva - Is there an ETA for this merge ?

Great work @hello-brsd! Until this is merged, I'm having to write an interim solution using axios interceptors along with vuex. 😬
@posva Thanks for approving the PR. Any updates on merge ETA?

Sorry, I cannot provide any ETA so please don't ping me for it 😓

I would also like to know what the status is on this because I've been bitten by this multiple times by now, and the fix seems very straightforward. I see Vue 2.x hasn't had a release with anything besides bug fixes for a while, so is this something you would even consider merging anymore? Or is all the focus on Vue 3 now?

Any updates on this? Our error handling would be greatly improved by this because we use a lot of async watches in our code. I'd really like to avoid having to resort to workarounds!

Can this please be reviewed?

@OzCroot Looks like this is already approved a while ago, @posva any newer ETA for this to land?
Documentation for errorHandler says:

Assign a handler for uncaught errors during component render function and watchers.

and there is a 2.6.0+ update section that mentions added async support:

In addition, if any of the covered hooks or handlers returns a Promise chain (e.g. async functions), the error from that Promise chain will also be handled.

Overall one would expect from documentation that async watchers would work with global error handler, but currently it is not so 🙂

Here`s a fix I came up with until this PR is merged.
// Fix "Vue.config.errorHandler and errorCaptured should capture rejected promised in watchers"
// https://github.com/vuejs/vue/issues/10009
// Ugly monkey patching until https://github.com/vuejs/vue/pull/9484 is merged
const realWatch = Vue.prototype.$watch;
Vue.prototype.$watch = function $watchDetour(expOrFn, callback, options) {
  if (typeof callback === 'function') {
    if (this.$_watchCount === undefined) {
      this.$_watchCount = 0;
    }

    // Reroute $watch to $on which handles promise rejections properly
    const watchDetourEvent = `watchDetour_${this.$_watchCount}`;
    this.$_watchCount += 1;
    const realCallback = callback;
    this.$on(watchDetourEvent, (args) => { return realCallback.apply(this, args); });
    callback = (...args) => { this.$emit(watchDetourEvent, args); };
  }

  // realWatch will call back into $watchDetour with a function if callback wasn't one
  return realWatch.call(this, expOrFn, callback, options);
};

Here`s a fix I came up with until this PR is merged.
// Fix "Vue.config.errorHandler and errorCaptured should capture rejected promised in watchers"
// https://github.com/vuejs/vue/issues/10009
// Ugly monkey patching until https://github.com/vuejs/vue/pull/9484 is merged
const realWatch = Vue.prototype.$watch;
Vue.prototype.$watch = function $watchDetour(expOrFn, callback, options) {
  if (typeof callback === 'function') {
    if (this.$_watchCount === undefined) {
      this.$_watchCount = 0;
    }

    // Reroute $watch to $on which handles promise rejections properly
    const watchDetourEvent = `watchDetour_${this.$_watchCount}`;
    this.$_watchCount += 1;
    const realCallback = callback;
    this.$on(watchDetourEvent, (args) => { return realCallback.apply(this, args); });
    callback = (...args) => { this.$emit(watchDetourEvent, args); };
  }

  // realWatch will call back into $watchDetour with a function if callback wasn't one
  return realWatch.call(this, expOrFn, callback, options);
};


[v1.4] Update documentation

Refactored File Management and Suggestion implementations.

Coverage remained the same at 66.466% when pulling 47330ba74aaca819f899398a648121d8fe588cf4 on JiaHaoLim:update-documentation into 84cd0d203eb64f2c6d3a0e6dda8663dedd27fde4 on CS2103-AY1819S2-W17-2:documentation.


Cannot set object keys from const

After the last update, I have a pretty odd failure, which I cannot repro on a smaller codebase.
Essentially, I am importing a const that defines a key of the object.
import {
  ADVERTISING_AUTHORIZATION_CREDENTIALS,
  ADVERTISING_AUTHORIZATION_REDIRECT_FINAL_URI,
} from '../../infra/constants'

Then, I am using it to construct an object:
    console.log(
      'ADVERTISING_AUTHORIZATION_REDIRECT_FINAL_URI',
      ADVERTISING_AUTHORIZATION_REDIRECT_FINAL_URI,
    )

    const enviro = {
      [ADVERTISING_AUTHORIZATION_REDIRECT_FINAL_URI]: 'http://example.com',
    }
    console.log('enviro', enviro)

This produces the following output:
  console.log
    ADVERTISING_AUTHORIZATION_REDIRECT_FINAL_URI ADVERTISING_AUTHORIZATION_REDIRECT_FINAL_URI

      at Object.<anonymous> (...test.ts:39:1)

  console.log
    enviro { undefined: 'http://example.com' }

      at Object.<anonymous> (...test.ts:47:1)

Notice that ADVERTISING_AUTHORIZATION_REDIRECT_FINAL_URI does have a value, as evident from the first console log.
Yet when using in the context of object key, it seems to be undefined.
Any tips on how to repro this better are appreciated!
Cheers!

First, try to run jest with --no-cache flag: npx jest --config ./jest.config.js --no-cache PATH_TO_ERROR_SPEC
If the error still appear, try to log the transformed file while running the test:

enter the @swc-node/jest dist file in<EMAIL_ADDRESS>Add log to source file:

- return core_1.transformJest(src, path, transformOptions);
+ const { code, map } core_1.transformJest(src, path, transformOptions);
+ console.log(code)
+ return { code, map }

Record the log:

npx jest ./PATH_TO_JEST_CONFIG --no-cache ./PATH_TO_SPEC_FILE > log.js

Seems to be resolved now... Never got around to actually digging into it. But after upgrading to latest, the issue is gone. I'll close this for now. Thanks!


Stretch goals: owner can edit qrcode

Decision made to not do QR code names, this story is thus irrelevant


Build fix

Apologies previous PR didn't quite build correctly. This fixes that mistake.

I should've caught it. Thanks for the quick fix.


fix for the ComboBox positioning issue

This PR modifies how <ComboBox> calculates its offset.
It uses el.getBoundingClientRect() to detect the difference between the selected item and the ComboBox's button and moves the menu by this difference.
This does not fix the other issue when the ComboBox will be displayed out of the screen's bounds. I left a proposed solution commented out, because I think it's not the most optimal way of doing it.
Doesn't yet close #6, because some display issues still remain.

Note that to compare how my solution works with the current ComboBox implementation, you will need to create more than 14 items in the ComboBox. I didn't want to edit the 'test' page, so I decided not to include it in the commit

Thank you! LGTM.

It seems like some tool removed the requestAnimationFrame call in the updateOffset function. This call is critical to the fix, but maybe we can replace it with a polyfill if the requestAnimationFrame isn't supported for compatibility reasons.

There's an interesting issue with my implementation: it doesn't work with scroll-behavior: smooth. It seems like it's records the selected item's position before it was actually scrolled to.
To fix this, we just need to put scroll-behavior: auto on the <ul> element. I guess I should make another PR for this, since this one was already merged?


Update CLOMonitor badge url

Signed-off-by: Cintia Sanchez Garcia<EMAIL_ADDRESS>Please ensure your pull request adheres to the following guidelines:

[x] For first time contributors, read Submitting a pull request
[x] All code is covered by unit and/or runtime tests where feasible.
[x] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[x] All commits are signed off. See the section Developer’s Certificate of Origin
[x] Provide a title or release-note blurb suitable for the release notes.
[x] Thanks for contributing!

This PR updates the CLOMonitor’s badge url, which has changed recently
Update CLOMonitor badge url

Thanks for noticing @cynthia-sg.
I think the link you are looking for is actually  from https://clomonitor.io/projects/cncf/cilium

Sorry, my bad, will fix it right away 😇

Thanks! Can you just resolve the conflicts?

I think I created this PR from a fork that wasn’t up to date, hence the conflicts. I will close it and create a new one.


FR: in review queue, load more revisions before and after the timeWindow

Blocks #109, #108

Step 1: load a few more latest Revisions, i.e. revisions after current timeWindow
Step 2: load a few more revisions before current timeWindow
Step 3: sort all local revisions
Step 4: update entire nextWikiRevIds with sorted result.

A key design decision: shall we use Revision Id revId or Timestamp timestamp for the timeWindow?
Timestamp timestamp is more intuitive for a timeWindow, and they key benefit is a timestamp is global, so it works for multiple wikis or in other environment if we have other dependencies. However the key disadvantages are that

If we use startTimestemp and endTimestamp to make query, we might not necessarily have fetched all revisions within the timeWindow, because of API limits to certain amount of revisions to be returned in each time. (e.g. 50 for normal client, 500 for approved bots)
we have to convert wikiRevId to timestamp all the time and we don't necessarily have access to such information. We need to store this information somewhere or request it from the server all the time.

Using revId as starting and ending point of the timeWindow avoids these caveats above.
Now the next question is, is it possible to query with revId as start and end?

Now the next question is, is it possible to query with revId as start and end?
Unfortunately, according to the MediaWiki Documentation for Query:Revisions
Using revision id is only available for given set of pages or a single plage


RTL support

make tables fit in rtl, mainly css adaption

Put an x in the boxes that apply_

[ X] Fix
[ X] Feature

Checklist

Put an x in the boxes that apply._

[X ] Title as described
[ ] Add unit test by vitest if necessary

asked for in issue #303

how to use rtl?


[CLOSED] Make words match examples

 Issue by edds
Tuesday 18 March 2014 at 14:14 GMT
Originally opened as https://github.com/alphagov/specialist-publisher/pull/47

edds included the following code: https://github.com/alphagov/specialist-publisher/pull/47/commits

 Comment by elliotcm
Tuesday 18 March 2014 at 14:43 GMT

We could even go so far as to say "first item" rather than "first 1 items".
Or turn this into a 2 of 4 instead to make it more full an example?


feat: dynamoDb options is able to accept sessionToken

As you advice to me in this issue: https://github.com/bboure/serverless-appsync-simulator/issues/76 I have implemented and tested the connection with my dybamoDb with session token and It work fine because amplify-appsync-simulator is just able to accept the sessionToken parameter!

Awesome @Scale93
LGTM
Great job!

@all-contributors please add @Scale93 for code and documentation

:tada: This PR is included in version 0.12.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Feature Request: Change the icon to comply with macOS specifications

The macOS app icons are a rounded rectangle after Monterey, as shown in the figure below:

Why so they are!  I'll fix that before the next release once I figure out the appropriate size and corner radius.  I see that the GeoTag icon is a bit bigger than others, too.


Feature request: regex-style capture groups.

If my (very limited) understanding of jsonpath and jsonpath-rw is correct -- BIG "if" there -- a .find() returns the entire path through the object, root to leaf, that satisfies the given expression. So with an "un-rooted" expression likefield1.field2 on a complicated object, the thing you get back is yet another complicated object. You know that field1.field2 is in there somewhere, but you have no idea where. Retrieving, say, "everything above field1.field2" means doing guess-and-check on the returned path. This is repeated work.
regex engines solve a similar problem by capturing parts of matched string during their search. "Capture groups" (also sometimes called "match groups") refer to the parts of the string that should be kept if matched. There are various mechanisms for then referencing the match groups on success, including by name and by number.
Something similar to regex capture groups would be highly useful in jsonpath.

Real life use-case that I'm dealing with: $.paths.*.get,put,post,options,delete,head,patch.. where I would like to capture the * and the get,put,post,options,delete,head,patch matches.
Possible capture syntax for this use-case using {} and #:
$.paths.{*}#url.{get,put,post,options,delete,head,patch}#method

where capture groups populate match.url and match.method after .find(...).

Started trying to implement this myself in earnest. So far I have the lexer accepting '{' and '}' correctly. In jsonpath.py I defined:
class Capture(JSONPath):
    def __init__(self, subexpr, callback):
        self._subexpr = subexpr
        self._callback = callback

    def find(self, datum):
        datum = DatumInContext.wrap(self._subexpr)
        self._callback(datum)
        return [datum]

    def __str__(self):
        return '{' + str(self._subexpr) + '}'

    def __repr__(self):
        return '%s(subexpr=%r)' % (self.__class__.__name__, self._subexpr)

The callback gets set on the parser as follows:
class JsonPathParser(object):
    ...
    def p_jsonpath_braces(self, p):
        "jsonpath : '{' jsonpath '}'"
        p[0] = Capture(p[2], self._register_capture)

I think this all makes sense so far but what exactly Capture.find is supposed to do is a little fuzzy in my mind. Am I on the right track?


serialization of arrays didn't work with non-pod types. fixed.

I did pull out some code of which the first half (related to Endianess) did just look wrong and the other half didn't work properly.

What's wrong with the endianess-code? if the endianess of the target does not match the one of current locality it falls back to non-optimized serialization, otherwise it invokes the optimized code path.

The Endianess-code was the same for every Endianess (or I and @sithhell are blind, which is quite possible), so that made no sense.
The fallback code invoked bitwise serialization regardless of the type of the cargo. That's not suitable for vectors (see new unit test). I only noticed this because the previous code was causing a segfault in LibGeoDecomp (because two vectors were both calling d-tors on the same memory, but the second one was doing this after the first one had already freed that memory.

On 08/12/2015 06:25 PM, Hartmut Kaiser wrote:

What's wrong with the endianess-code? if the endianess of the target
does not match the one of current locality it falls back to
non-optimized serialization, otherwise it invokes the optimized code path.

That's exactly what's wrong: We are in the function which is supposed to
load/save non bitwise serializable objects, but call the functions for
bitwise serialization if the endianness matches.

—
Reply to this email directly or view it on GitHub
https://github.com/STEllAR-GROUP/hpx/pull/1716#issuecomment-130362041.

That's exactly what's wrong: We are in the function which is supposed to  load/save non bitwise serializable objects, but call the functions for bitwise serialization if the endianness matches.

Ok, but where should this code go, then? If the endianess is not matching we shouldn't do optimized (bitwise-copy) serialization for that type.

Am 12.08.2015 7:46 nachm. schrieb "Hartmut Kaiser"<EMAIL_ADDRESS>
:

That's exactly what's wrong: We are in the function which is supposed to
load/save non bitwise serializable objects, but call the functions for
bitwise serialization if the endianness matches.

Ok, but where should this code go, then? If the endianess is not matching
we shouldn't do optimized (bitwise-copy) serialization for that type.

In that case, I'd say the archive has to disable array optimization
globally, that saves a lot of otherwise duplicated checks.

—
Reply to this email directly or view it on GitHub.

I think the removed code needs to be re-added to both optimized overloads, here: https://github.com/gentryx/hpx/blob/master/hpx/runtime/serialization/array.hpp#L51-L61. You were right, as it was the code was broken.

I'm testing a nicer way of doing this. (Nicer than re-adding the Endianess-code twice)

@hkaiser Like this?

@gentryx yes, looks good! Thanks!


Dead link in side bar at https://docs.helm.sh

If you visit https://docs.helm.sh and then click on "Helm Commands" followed by "Helm List" it doesn't work. The link is to https://docs.helm.sh/helm/#helm-rep-listo which doesn't exist.
I can't figure out how this is generated, not sure it is possible for me to fix it with a PR. I can't find any instance of the string "listo" anywhere

https://github.com/helm/helm-www is the repository that generates the site, if you want to take a shot at fixing this bug.

Thanks. See https://github.com/helm/helm-www/pull/62


Add Greek language

Hello.
I bought a Cat S22 phone, (unlocked from E-bay) and I need to write in Greek as I could on the older Nokia ones and the LG Wine smart.
I'M totally blind and prefering these devices because of the traditional alphanumeric keypad which have.
I've already switched the display language by using another app but not the keypad yet.
Can you pleas add the Greek language also or tell me what to do because I don't know how to program?
So far, haven't found any similar app for such cases. There are for older Android versions which are discontinued.
Thanks for diveloping this project and waiting for your responce.
Best regards.

Can you pleas add the Greek language also or tell me what to do because I don't know how to program?

If you have no technical skills, it would take more time for me to explain, instead of doing it. Anyway, you could still check the Adding a new language guide.

@AgentGolder, I checked my old Nokia 3310, that I always use for reference. The Greek layout there is:

Space, Enter, Math
Punctuation
αβ
δεφ
ηγι
κλ
ομνω
σςπρψ
θτ
χζυξ

Is this correct? It looks out of order, but I guess, it may be like Hebrew, where the layout is not in alphabetical order for convenience.

Thanks for your responce.
The correct is:

Panctuation
αβγά2
δεζέ3
ηθιήίϊΐ4
κλμ5
νξοό6
πρσς7
τυφύϋ8
χψωώ9
Please add it and let me know where is the .apk file to get it.
Best regards.

Forgot to mention 0 is space, new line and 0

is * and math simbles. and # is case and T9 change.
Please include it.
Thanks in advance!

@AgentGolder, I am trying to add the language, but the dictionaries I've found (including the one in Android OSP), contain letter variants with diacritics that you haven't noted above, such as: "ὐ", "ΰ", "ῆ", "ἦ", "ῷ" and many more. I am not sure if all of them are valid and in use today.
Could you please provide the full valid character list?

You probably found the ancient greek. The only valid character today from those you mentioned is "ΰ".

Ok, then what do with the words that contains such characters? Shall I replace with the respective with non-accented variants or throw the words away?
Here are some examples:
αὐτῷ
τοὺς
ἦν
οὐκ
ούσαν
κεφαλῆς

You should replace them with the respective non-accented variants.

Task completed! I tried typing some text in Greek, using a random Wikipedia article for reference and I think it is working very good.
It will be officially available in the upcoming v23.0. Until then, if you are very eager to try it out, you could build an APK using the latest master.

Thank you very much! You're very awsumn!
Can you please attach the .apk build for me here or send it to<EMAIL_ADDRESS>somehow?
God bless you.

Sorry, I don't have the capacity to make personal APKs. You will have to wait for the next stable release. Just keep an eye on the releases page.

You can use a service such as sendspace.com, mediafire.com. Google drive, wetransfer, etc to upload the file on.
Thanks for your corporation.

Hello.
Because I¨M totally blind )using a screen reading program= and don¨t have enough disc space for all Android studio¨s components to be downloaded, can you please upload an .apk file for me on one of the file hosting services above_
I¨ve already donated you 15 euros, more to follow soon.
Please understand that I really need it as soon as you can.
The problem is also that, I can¨t actually switch to Greek language when writing with the external bluetooth keyboard )Hoco S55 transparent).
Waiting for your responce, thanks and best regards.

Please understand that I really need it as soon as you can...

Please, understand this is a hobby project that I work on voluntarily, in my free time. There are no deadlines and no guarantees about anything and I would like it to stay that way. Hundreds of people are using this keyboard and they all want something. Even if I was working full-time on it, it wouldn't have been possible to grant everyone's wishes immediately.
The current state of the code is NOT ready for a public release. Once it is ready, I'll make an APK available to everyone. Please, have some more patience until then.

@AgentGolder, v23.0 is out. Check it out here.

Thank you very much for your understanding, you are a great person! I'll donate to support you again in the next days.


Add dropdown documentation

I would like to add dropdown documentation

Hey @prachigodse58, your contribution is welcome, would you like some help?

https://kitchn.tonightpass.com/docs/components/dropdown


fix: add documentation for rules engine actions

fix: add documentation for rules engine actions
fix: removed unused module import in the rules engine

Created #195 to continue on the logic of decoupling rules engine dependencies

General comments for the md while it is being modified: (line numbers based on modified file)

Line 3:  comes from the @o3r/rules-engine package
Line 13: add to the application (suggestion: we have to add the npm packages created in otter library (and not only) to the application.)
Line 14: replace "." by ":" at the end of the sentence
Line 57: At each fact change**,** (add a comma between "change" and "the")
Line 57: More information about facts OR More details about facts
Line 59: in the rules-engine otter package
Line 80: For example purposes
Line 82: integrated the following facts modules
Line 103: later in the flow
Line 103: More information about OR More details about
Line 107: the facts have to be registered in the rules engine
Line 149: in the rules engine scope
Line 171: referenced in the
Line 185: reference to the
Line 192: references ?
Line 197: example


After running :done on all visible notifications,  the repl is stuck

Apply a filter, select all notifications, run done, press q.
At this point you can't go on filter mode again.

This was fixed also by #182


Add pull_request for next build

We don't have any tests at all at the moment :grimace:
But at least we can make sure PRs don't totally break the build...

Hmm, this didn't run on this PR as I expected it would. Did I get my syntax wrong, or will this not actually run on PRs until this change is merged?

Think it has to be merged?

I'll merge and we'll see how it works!


Upgrading from v1 to v2 made special characters appear wrong

After upgrading, some texts were saved/converted in wrong char encoding, for example:
https://qsnapnet.com/snaps/lcsr9frkjmtxzux
Is there a way to fix it retroactively or to apply fix in code so that it does not happen when somebody else upgrades from v1 to v2?

I saw same characters problem in codeforphilly.org
https://qsnapnet.com/snaps/5yoqs8pc1lymbo6
https://qsnapnet.com/snaps/kbvzyv3gpjw3tyb
All snaps: https://qsnapnet.com/folders/lzgzdbb1ezbbj4i

https://digwp.com/2011/07/clean-up-weird-characters-in-database/ This may help in cleaning up. Hoping to take a look at this shortly

@bennovack thanks for the tip!
I'm testing workaround by replacing characters on database level, seems to be working:
e.g. UPDATE projects SET Title = REPLACE(Title, 'Ä‡', 'ć');

@bennovack thanks for the tip!
I'm testing workaround by replacing characters on database level, seems to be working:
e.g. UPDATE projects SET Title = REPLACE(Title, 'Ä‡', 'ć');

@schlos are you able to input extended characters on the site now and they show up correctly on display and on edit?

Hi @themightychris
Problem was not saving special characters in v2, but historic, already present characters that were stored in v1, somehow after upgrading to v2 they were scrambled in db.
Any new text with special characters are displayed good after saving in v2.
So only I will do now (after testing on staging db) - is to replace all scrambled characters with correct ones.
I've seen that this happened also on codeforphilly.org ((example, issuesâ€”a)[https://codeforphilly.org/project-buzz/accessibility_in_an_app]), so just heads up - maybe you will need to run replace characters script. This will be reproducible on all Laddr instances that are or will be upgraded from v1 to v2, the script has to be part of upgrade procedure.
I will share script once I've checked all db tables.

This is script I've used:
replace_characters_script.sql.zip


[vwc-datepicker & vwc-accordion clash]

Describe the bug
Accordion close by itself after using date picker.
To Reproduce
Go to User Admin -> Schedules -> https://itg-test.nvmtest.net/CallCentre/portal/useradmin/schedules/
Open "Create schedule" or edit existing one
Expand "Rules" section
Click on a time to show time selector
Choose some time and click outside the selector/press Esc/press Enter
Expected behavior
Accordion should still be expanded, because after choosing the time user wants to choose the day/days
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):
Windows 10
Chrome Version 97.0.4692.71 (Official Build) (64-bit)
Additional context
Use the same credentials for UserAdmin as before - user testUserVivid.

This is caused due to both component sharing the exact same event firing name of closed
https://github.com/Vonage/vivid/blob/31c06c0afb665c0139f0f9f677b7489891267257/components/expansion-panel/src/vwc-expansion-panel-base.ts#L46
@Vonage/vivid-core as accordion, expansion-panel & datepicker are all in alpha state, it's possible to apply namespace to custom fired events as in #1199
@jfedoruk in the meantime can you workaround this with
{datepicker}.addEventListener('closed', (e) => {
  e.stopPropagation();
});

and
{expansion-panel}.addEventListener('closed', function(e) {
  if (e.target !== this) {
    console.log("datepicker closed");
  } else {
    console.log("expansion panel Closed")
  }
});

@yinonov thanks - the workaround works fine.

Thanks. closing as it's a symptom of #1199


Update checklist and add in more COPY-files

Bonus would be to create a script in this repo and that we can do something like:
\curl script | bash

which will copy the COPY_* to the current folder that ran this command.

Actually we can delete COPYME and remove all COPY- prefix. Then write a script to get all files to current folder?

COPYME is still good for copying it to be pasted as an issue in a repo.

The files you add in this Pull Request are templates, and this repo is Open Source Checklist. Should they live here or a new repo like Open Source Template?

I see them as part of the checklist, so I would keep them here for now, for convenience.


does not handle circular nested objects

I found your module in stackoverflow and I thought this one will solve my problem in modifying data in redis cache.
Basically I want to modify a field within an object with a specific id in redis cache using graphql.
so I have a data like:
const data: {
projects: [
{
name: 'sample',
id: 1
photos: [
id: 12,
project: {
id: 1
name: 'sample'
}
]
}
]
}
So i want to modify the name of a project with id 1.
using the changeProps it should have also changed the project inside photos but it doesnt

Thank you for discovering a bug! Fixed in 1.1.3


How to load scripts in rendered html

I'm trying to load external html inside my lit-element, I'm using lit-html to render the template with couple 'script' and 'link' tags, but no GET request are triggered.
I found out that if you set the innerHTML, scripts won't be executed but script-inside-shadow-dom-not-working solution will work.
I'a looking for better and robust solution for dynamic and unknown HTML so I can load it in run time.
render() {
    return html`

I'm a Web Component

    `
  }

You are trying to load html in a script tag, that won't work. There is no browser-native way to load HTML on the web. There was HTML Imports, but it's now deprecated: https://developer.mozilla.org/en-US/docs/Web/Web_Components/HTML_Imports
There is a proposal for HTML modules but it's not there yet: https://github.com/w3c/webcomponents/issues/645
For loading javascript dynamically, there is a new browser standard called dynamic imports. It's only implemented in chrome, but all modern build tools use it for code splitting. See: https://developers.google.com/web/updates/2017/11/dynamic-import

Thanks.
I could use build tools like webpack and probably my problems will be solved, the problem is that I have to load on run time and I don't really know the content of the html (but I can trust it).
As I don't have good solution, I trying to use jQuery load (yeh, I know) or manually find and parse any script that I'll find on the retrieved html.

If you load your HTML via fetch, you can render it with the unsafeHTML and until directives:
import {unsafeHTML} from 'lit-html/directives/unsafe-html.js';
import {until} from 'lit-html/directives/until.js';

const someHTML = fetch('some-index.html').then((response) => unsafeHTML(response.text()));

render() {
    return html`

I'm a Web Component

      ${until(someHTML)}
    `;
  }

This should work, but combining directives can be tricky. Let me know if there's an issue.

Great answer! Thanks

@justinfagnani https://stackblitz.com/edit/unsafe-html-lit-element-dqpy64?file=index.js
What I did wrong? :sweat:

@justinfagnani Yes, but that still does not execute script tags inside the HTML inserted with unsafeHTML (probably because under the hood it uses innerHTML), which is what my cool-o system for composing apps needs to do.

@rtm: Correct, script tags need to be handled specially to be able to run them. You'll have to create a directive to document.createElement('script') and set the inner text on that node, then insert the node into the DOM.
See https://stackblitz.com/edit/lit-html-script-tag for a working example.


[Question] How access other services via VPN's ips

I wanted to know if it's possible to access other services, like web GUI from other container without using the server's private network.
Server's and client's private network is <IP_ADDRESS>/24.
I didn't change the vpn's configuration, it's still on <IP_ADDRESS>
How can I access the other container via <IP_ADDRESS>, is it impossible because of the fact that the VPN is in a docker container ?

I have the same problem: it's impossible to connect to port 80 to the host <IP_ADDRESS> (OpenVPN server host) from the VPN client from the same network. How to fix this?

I have created a network with docker network create and added both the service and the openvpn service to this external network. Then I added the route to the subnet into my client config file. I guess it might also work if you add it to the server:
route <IP_ADDRESS> <IP_ADDRESS>
I can ping and access the internal services this way but I would also like to add a dns service so that the the dns requests are also forwarded through the vpn to the internal dns server. This way I can use the service names as hostname instead of the IPs.

Something like this works:
docker-compose.yml
version: '3.7'
services:
  openvpn:
    cap_add:
     - NET_ADMIN
    image: kylemanna/openvpn
    container_name: openvpn
    ports:
     - "1194:1194/udp"
    restart: always
    volumes:
     - /var/docker/volumes/openvpn:/etc/openvpn
    networks:
      vpnnet:
        ipv4_address: <IP_ADDRESS>
  dns:
    restart: always
    image: strm/dnsmasq
    volumes:
      - ./dnsmasq.conf:/etc/dnsmasq.conf
    expose:
      - "53/udp"
      - "53/tcp"
    cap_add:
      - NET_ADMIN
    networks:
      vpnnet:
        ipv4_address: <IP_ADDRESS>

networks:
  vpnnet:
    external: true

The in the openvpn.conf (I have mounted the volume on my system)
### Route Configurations Below
route <IP_ADDRESS> <IP_ADDRESS>
route <IP_ADDRESS> <IP_ADDRESS>

### Push Configurations Below
push "block-outside-dns"
push "dhcp-option DNS <IP_ADDRESS>"
push "comp-lzo no"

In your client configuration this has to be appended for Linux:
script-security 2
up /etc/openvpn/update-resolv-conf
down /etc/openvpn/update-resolv-conf

I also have route command in the client configuration but I don't know if it is necessary.

dnsmasq.conf

What you have in your dnsmasq.conf?

Sorry, I gave up doing it this way and simply run openvpn server on the host. Way easier for the setup I want to achieve. Dnsmasq was setup in a way that it only acts as a dns relay and I added all static hosts there iirc.


[CI] Enabled automatically deploy to gh-pages by GitHub Actions.

@huningxin @anssiko PTAL,thanks.

No objections. Let me merge it and deploy the latest webnn-polyfill.

The latest webnn-polyfill was deployed successfully: https://github.com/webmachinelearning/webnn-polyfill/tree/gh-pages.


Supports XMLHttpRequest interception in a browser

GitHub

Follow-up to #93

Changes

Adds in-browser tests for XMLHttpRequest interception and response mocking.
Uses the env-agnostic bufferFrom utility instead of Buffer.from() that's available only in Node.js.
Adds tests for the createEvent function.

We don't need to use the native window.Event because it forbids setting the target, which in the case of XMLHttpRequestOverride should be the overridden class. That's one of the reasons we have the EventPolyfill because it allows setting a custom event target in the class constructor.

@msutkowski, thank you for the review, Matt! I've addressed your comments and elaborated on the Buffer changes made here. Hope it's better now.


Extensions list should fill ActionBar like a menu

This debt item consists of the following

Extensions list should use the ActionBar like all other menus. Thus hidden actions should not be part of that ActionBar
We should remove the flag focusOnlyEnabledItems in ActionBar that was introduced just for extensions. Also any call to action enablment should be removed from the ActionBar

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!


raw HTML in a component with SSR

Problem
Using raw HTML in Yew is not so simple but thankfully the inner_html example provides a decent solution.
However, this doesn't work with SSR as the web_sys/gloo packages are not available in that context.
So, how can raw HTML be used with SSR?
Steps To Reproduce
Steps to reproduce the behavior:

Use the inner_html example
Switch to the SSR renderer
Doesn't work

Expected behavior
The inner_html example was a workaround anyway so it would be good if there was an official way to handle raw HTML which also works with SSR.
Environment:

Yew version: master
Target, if relevant: any non-WASM target
Build tool, if relevant: trunk

Questionnaire

[ ] I'm interested in fixing this myself but don't know where to start
[ ] I would like to fix and I have a solution
[x] I don't have time to fix this right now, but maybe later

Currently, inner_html is set with Html::VRef.
We cannot support this in SSR as we cannot instantiate web_sys::Node in non-browser environments.
However, this may change if https://github.com/yewstack/yew/issues/182 is implemented.
If you don't need the content in the inner html to be indexed by a search engine, you can delay the rendering logic that uses Web APIs with a use_effect hook so they are excluded from the server-side rendering process.

Currently, inner_html is set with Html::VRef. We cannot support this in SSR as we cannot instantiate web_sys::Node in non-browser environments.
However, this may change if #182 is implemented.
Cool, it seems like that's exactly what we need :)

If you don't need the content in the inner html to be indexed by a search engine, you can delay the rendering logic that uses Web APIs with a use_effect hook so they are excluded from the server-side rendering process.

It needs to be indexed, unfortunately.


[BugFix] Parametric collectors

I have added the test (if it s too long feel free to cut some parts of it).
It uses an inproved version of NestedCountinEnv which will help us test nested envs with an added dimensionality for nested keys (just like MARL)

I have added more general tests for nested envs and specific collector tests

Ok mergable for me

Let's reduce the test time and we're good to go!

Is it still too high after the commit named slimmer test?


Upgrade action in main.yml to no use deprecated environment

This should be fixing:

@peterhuene  the build are failing due to

And it is independent of those changes


[bug] Actions stop working after second connect attempt to same room flutter livekit client

Describe the bug
Hello! When I try to disconnect from a room in my Flutter app, I've noticed that it works the first time, but if you connect again and then try to disconnect a second time, it doesn't do anything and no error is returned. Additionally, other actions like setCameraEnabled and setMicrophoneEnabled also stop working after the second connection to the same room.
To Reproduce

Connect to a room.
Disconnect from the room.
Connect to the same room.
Disconnect from the room (This step doesn't work for some reason).
Try to use setCameraEnabled or setMicrophoneEnabled (These actions also stop working SOMETIMES).

Expected behaviour
The app should successfully disconnect from the room on the second attempt as it does on the first attempt, and actions like setCameraEnabled and setMicrophoneEnabled should continue to work.
Platform information

Flutter version: 3.22.1
Plugin version: livekit_client: ^2.2.0
Flutter target OS: Android/iOS/Web (Only tested on Android Emulator running android 14 & Samsung Galaxy S22+ running Android 14)
Flutter target OS version: Android 14
Flutter console log:

I have the same issue in version 2.2.0.
In version 2.1.6 it's working fine.


Use white background for scarabs, incubators and emblems

Same color used in-game and on trade. closes #998

Looks good to me, I would have prefered to just default to the currency view model if we don't explicitly define one.
Discussion here:
Are we sure it's safe to default to currency hover? What about if new types get added in the future?
_Originally posted by @Stickymaddness in https://github.com/render_node/MDIzOlB1bGxSZXF1ZXN0UmV2aWV3VGhyZWFkMTYzNDgzMTI1OnYy/pull_request_review_threads/discussion

@aydjay Your link seems to load a blank page.

How about returning currency if (item.MaxStackSize > 0) and return white (Rarity.Normal) for the rest.


Just some small typos

Visualizations of all channel are available in the appendix. (channels)
This article focusses on feature visualization. (focuses)
In the following sections we’ll examine techniques to get diverse visualizations, understand how neurons interact, and avoid high frequency artefacts. (artifacts)
How can we chose a preconditioner that will give us these beneﬁts? (choose)
used in adverserial work (adversarial)
As long as a correlation is consistent across spatial positions — such as the correlation between a pixel and its left neighbor being the same across all positions of an image — the Fourier coefﬁcients will be independant variables. (independent)

I'll add a few others I found :)
The objectives we’ve mentioned only scratch the surface of possibile objectives. (possible)
Additionally, the optimisation may make examples be different in an unnatural way. (optimization, for consistency)
Gradient blurring[15]  is equivalent to gradient descent in a different paramaterization of image space, (parameterization)

Yay for all your work and attention! Will fix on Monday. :-)


Add Avatar component

Closes #66.

@matthieuh the changes you suggested have been applied! Now the Avatar component will show itself on a dark or light variant according to the theme, but if you want to override it, you can pass a variant prop with the value dark or light.

@matthieuh it's done! 😄


Update use of libc::timespec to prepare for future libc version

In a future release of the libc crate, libc::timespec will contain private padding fields on *-linux-musl targets and so the struct will no longer be able to be created using the literal initialization syntax.
Update places where libc::timespec is created to first zero initialize the value and then update the tv_sec and tv_nsec fields manually. Many of these places are in const fns so a helper function zero_init_timespec() is introduced to help with this as std::mem::MaybeUninit::zeroed() is not a const function.
Some matches on libc::timespec are also updated to include a trailing .. pattern which works when libc::timespec has additional, private fields as well as when it does not (like for
x86_64-unknown-linux-gnu).
See also https://github.com/rust-lang/libc/pull/2088

@wesleywiser it looks like mem::transmute only became const in Rust 1.56.1.  That means we can't backport this PR.  Can you come up with a version that doesn't rely on mem::transmute?  CC @rtzoeller .

Sorry for the delay! I'll post the backports tomorrow.


Redis class (Cache library driver) throws exception when Redis server is down

When the Redis server is down, and you try to use the redis driver like :
$this->load->driver('cache');
$this->cache->redis->save('foo', 'bar', 10);

It returns an exception:
Type: RedisException
Message: Redis server went away
Filename: E:\codeigniter-app\system\libraries\Cache\drivers\Cache_redis.php
Line Number: 140

The line throwing the error is from the constructor of CI_Cache_redis class:
$serialized = $this->_redis->sMembers('_ci_redis_serialized');
Is this expected behavior or should the Redis cache driver handle this exception in some way? (like currently it handles RedisException while doing connect/auth).
This was found while using the latest version of CI : 3.1.9

workaroud
$this->load->driver('cache');

$ok = true;
try {
	// @$this->cache->redis->save('foo', 'bar', 10);
	$this->cache->redis->save('foo', 'bar', 10);
} catch (\Exception $e) {
	$ok = false;
	log_message('error', 'redis server is down! ' . $e->getMessage());
}

The on-connect checks exist so that the library can fallback to another driver if configured so.
After a connection is established, it is assumed that everything related to it is supposed to be fine, and your particular issue shouldn't even be possible - you have connected to the server literally a millisecond ago. Such crashes are not just rare and unexpected ... I didn't even know they were possible, since method calls (in this case to sMembers()) are supposed to just return FALSE in case of failure.
I suppose we could add checks for that, although they'd be some annoyance in our code, but whether that's expected behavior or really a moot point, as it was never envisioned. The question is whether CI handling this particular exception is beneficial or not. If it objectively helps users - then it's good, but if it just hides debugging information, then it wouldn't be such a good idea.
P.S. Please don't post screenshots of code. It's annoying, literally doesn't help with anything and in this particular case nobody needed to see that code anyway.

Thanks @narfbg for your thoughts.
Sorry I am replying so late.
Regarding the issue that happened with me, it was due to some outage at the cloud provider and the redis server went completely down (which is really rare). So due to this even the connection establish call had failed as I could see errors in the logs for that. Hence it wasn't the case that connection was established and the sMembers() call failed a millisecond later.
According to me, CI handling the exception would be beneficial, as I too was expecting redis calls to just return false in case if they fail. A simple log_message like the one's for connection establish seems good for debugging. However I can understand other developers might expect different behavior. So the decision is up to the community & contributors. Feel free to close if this issue if needed.
Out of curiosity, I tried modifying the redis cache driver myself so that the sMembers() call in question doesn't throw an exception.
But then any call to redis like $this->cache->redis->save('foo', 'bar', 10); throws an exception on line 193 $this->_redis->set($id, $data, $ttl); and line 325 $this->_redis->close();.
So for CI to handle all exceptions, along with the sMembers call it would be required to put each function called using _redis in the cache driver to be in try-catch.
Or alternatively, users can just put their calls to the driver like $this->cache->redis->save() in the try-catch block.
About the screenshot of code, will keep in mind.

I see ... Well, the most debug-friendly thing we could do is to actually remove the current catch block for the connection establishment. If the existing log message saying that the connection failed didn't help you, I don't see how countless others saying that e.g. save() fails would.
That way, one could at least do with just a try ... catch during initialization as opposed to every other call.