Remove parameters not needed for eox-map config export

For example "interactions":[] creates a warning in the story editor, we currently do not need this, so let's make sure we keep only necessary information for the map-config

Some have already been removed. I think we should also get rid of the grouping, as in the export only the overall order is important as no layer control is expected once loaded into a story

We could consider also removing groups all together, but for now i will close this issue


Cask request: WD MyCloud

Cask details
Please fill out as much as possible. Before you do, note we cannot support Mac App Store-only apps.
Name: WD MyCloud
Homepage: http://wdc.com
Download URL: http://download.wdc.com/nas/WDMyCloud_mac.dmg
Description:
It was possible once before to download WD MyCloud via Caskroom? But seems like it was removed.
Original code: https://github.com/caskroom/homebrew-cask/blob/93e9e31a68bdcd7547f744836baf353d856f3b22/Casks/wd-my-cloud.rb

Migrated to caskroom/drivers: wd-my-cloud


Update Table component

Description
Update the Table, with missing states and latest changes from Figma
Documentation

The storefront
Figma
WAI-ARIA Authoring Practices 1.1

Sub tasks

[ ] Write tests
[ ] Create set of tokens (from base tokens)

Notes
#148

Add separate types for th and td, possibly by using function overload (specifically scope)
Don’t use th-tokens for thead
Rewrite sticky header example in story to use position: fixed on th (See CSS Tricks article)

Don’t use th-tokens for thead

We use th-tokens for thead because there is no tokens thead. Figma does not have this concept. We need to use the background and border from th-tokens for when the table is stretched to fit width and th not.

blocked by #830


git 2.11.0 pushing to a repo located in a windows share

After updating git from 2.10.0 to 2.11.0 when trying to push any changes to a repo located in a windows share, the following error occurs:
$ git push origin test
Counting objects: 2, done.
Delta compression using up to 8 threads.
Compressing objects: 100% (2/2), done.
Writing objects: 100% (2/2), 284 bytes | 0 bytes/s, done.
Total 2 (delta 1), reused 1 (delta 0)
remote: error: object directory /path/to/dir/objects does not exist; check .git/objects/info/alternates.
remote: fatal: unresolved deltas left after unpacking
error: unpack failed: unpack-objects abnormal exit
To //path/to/dir
 ! [remote rejected] test -> test (unpacker error)
error: failed to push some refs to '//path/to/dir'

No error occurs if pushing to the same repo (a direct copy into a local directory) using 2.11.0.
$ git push local_test test
Counting objects: 2, done.
Delta compression using up to 8 threads.
Compressing objects: 100% (2/2), done.
Writing objects: 100% (2/2), 284 bytes | 0 bytes/s, done.
Total 2 (delta 1), reused 1 (delta 0)
To C:/path/to/dir
 * [new branch]      test -> test

git remotes:
$ git remote -v
local_test      C:/path/to/dir (fetch)
local_test      C:/path/to/dir (push)
origin  //path/to/dir (fetch)
origin  //path/to/dir (push)

Using git fsck --full in both 2.11.0 and 2.10.0, it doesn't reveal any additional problems.
$ git fsck --full
Checking object directories: 100% (256/256), done.
Checking objects: 100% (2710/2710), done.

I'm not sure if it this is related but object directory /path/to/dir/objects does not exist; does not contain an expected second forward slash at the start.

Wrong place to file the bug report.


SSAOPass: Improve depth precision with WebGL 2.

should this be done here https://github.com/vanruesc/postprocessing/blob/d4576c14fa36e857ffc5f1bab2f1c28a1e79492e/src/core/EffectComposer.js#L193 ?
https://github.com/mrdoob/three.js/pull/18672

Thanks for letting me know! I'm looking into it.

The type of the depth texture has been changed to UnsignedIntType in<EMAIL_ADDRESS>


update Ddoc for root/filename.d and root/response.d

Address #Geod24 's comments in https://github.com/D-Programming-Language/dmd/pull/5608

Thanks !


[Bug]: Segmentation fault on fresh project

What happened?
I created a new project using cargo shuttle init (v0.11.2), moved into the directory and ran cargo shuttle run. After a bit of compilation I get the following:
    [...]
    Finished dev [unoptimized + debuginfo] target(s) in 54.86s

Starting <my-project> on http://<IP_ADDRESS>:8000
Segmentation fault (core dumped)

Version
v0.11.2
Which operating systems are you seeing the problem on?
Linux
Which CPU architectures are you seeing the problem on?
x86_64
Relevant log output
No response
Duplicate declaration

[X] I have searched the issues and there are none like this.

It was a rocket project, by the way.

Hey! We see this happen spuriously on linux, and more regularly on macOS. We currently compile users projects to dynamic libraries (.so files), and we need an unsafe FFI to load them. Since this has been rare on linux, I don't have a good suggestion for how you can remedy this right now.
The good news is we're refactoring our codegen and how we compile users projects this week, which means we'll be getting rid of the FFI early next week. This should be a thing of the past then.

Thanks for the response. I'm on NixOS, if that makes a difference. I'll wait for your refactoring and see if that fixes the problem.

This should be resolved after 0.12, but please re-open this if you still experience it. Thanks!


added repetition_penalty to TranscriptionOptions

CTranslate2 has a repetition_penalty argument in its model.generate method. It was however missing from WhisperModel.transcribe
Simply added the repetition_penalty argument
As it is, after each generation, the output is filtered if it's too repetitive (based on compression_ratio).
repetition_penalty can make it so repetition is penalized during the generation (decoding) itself, making some failure cases like I found disappear.
By "failure" in this case I mean taking 23 seconds to process a 13 second clip (which usually takes ~2.4 seconds) because of obvious repetition in multiple fallbacks, as seen in this image  https://files.catbox.moe/d0psud.png, where each line starting with needs_fallback shows the result of a model.generation.
repetition_penalty solves this hallucination problem I was having #394 almost perfectly, (using repetition_penalty=1.2).
Just by using repetition_penalty=1.2 it gets a non-hallucinated reply on the first try, though lower values already help on avoiding the worst cases like above. (Maybe setting repetition_penalty above one (always) is undesirable, I could see an implementation where a big hallucination is detected (the first generation has compression_ratio=10 after all!) and increasing the repetition penalty on the second generation.)

Do you know how this interacts with actual word repetition in an audio file? @bfs15 @guillaumekln


Adding kevan

This adds a customizable configuration for Kovan

i think pnpm-lock.yaml should not be in the PR.  can you exclude it?


Add "slowMo" parameter

To add artificial delay in the communication with the browser

This is now done with #61


fix/reranker_config

config was not being passed to reranker plugin

Summary by CodeRabbit

New Features

Enhanced flexibility for the reranker initialization with configuration options for improved performance.

Bug Fixes

Improved behavior of the reranker module through better configuration handling.

example
"common_query": {
      "reranker": "ovos-flashrank-reranker-plugin",
      "ignore_skill_scores": true,
      "ovos-flashrank-reranker-plugin": {"model": "ms-marco-TinyBERT-L-2-v2"}
}


[Starting view. Courier field] The ‘Select courier’ placeholder is not displayed in the Courier drop-down list after changing the language to English

Environment: Windows 11
Chrome 107.0.5304.63
Reproducible: always
Build found: last commit from https://github.com/ita-social-projects/GreenCityClient/tree/dev
E.g. "commit 147fa44"
Preconditions
Preconditions:
User is logged in as Admin.
Side bar menu :[https://www.testgreencity.ga/GreenCityClient/#/ubs-admin/orders] is oppened

Steps to reproduce

Click on the 'Uk' drop-down menu (upper right side) and choose the En item
Click on 'Tariffs' bttn in the side bar menu

Actual result
The ‘Select courier’ placeholder is not displayed in the Courier drop-down list after changing the language to English
Attached bug screenshots recorded.
Expected result
All drop-down menus are displayed with a placeholders after clicking on Tariffs bttn
User story and test case links
E.g.: "Starting view for the pricing page ("Тарифи") #3835"
Test case"
Labels to be added
"Bug", Priority ("pri: low"), Severity ("severity: low"), Type ("UI, "Functional")

")

Not reproduce


Warn in dev mode if <RouterView> is used instead of <NuxtPage>

Describe the feature
When using Vue Routers "native" <RouterView> component instead of <NuxtPage>, it can result in route desync (see e.g. this Discord thread). It would be nice to warn when using <RouterView> (and maybe <RouterLink>) instead of the idiomatic Nuxt components.
Additional information

[X] Would you be willing to help implement this feature?
[ ] Could this feature be implemented as a module?

Final checks

[X] Read the contribution guide.
[X] Check existing discussions and issues.

Maybe we can add a warning about the misuse in the documentation first, and the runtime warning can be added later.

@toto6038 PR welcome!

@manniL
Do you intend to detect the component usage in Nuxt dev runtime, or provide an eslint rule from eslint-plugin-vue?
Whichever, I think I could help.

I think this can be done during dev similar to how the layout detection works ☺

@manniL
Thank you for the reply!
If we go with dev runtime warning, we would not be able to detect RouterView usage straightforwardly, because we can't insert a logic to RouterView's setup logic, as we do for NuxtLayout (ref).
So, if I'm correct, all we can say is that "while pages are enabled (i.e. nuxt.options.pages is true (ref)[https://github.com/nuxt/nuxt/blob/44fcacba846a6cdf33e1475fc8413f6ae49d1fe4/packages/nuxt/src/pages/module.ts#L71C5-L71C23]), but NuxtPage is never used.", which may mean that developer uses RouterView instead.
So, the condition for warning message would be:

Check nuxt.options.pages
Add a flag to nuxtApp in  setup function of NuxtPage

I think it's fine to warn if developer don't use NuxtPage while they enables pages, even if they don't use RouterView.
By the way, with ESLint, we could walk through all the component tree and check their names and find RouterView.
Or, we might be able to go up the tree component by using property to ancestor instances, which I believe is too cumbersome for our goal.
Am I missing something, or do you have any better approaches?
If you are fine with my approach, I will submit a PR.

If we go with dev runtime warning, we would not be able to detect RouterView usage straightforwardly, because we can't insert a logic to RouterView's setup logic, as we do for NuxtLayout

We probably could swich out the component during dev (when auto-imported) with a version that proxies everything to the original component and have that dev message 🤔
But it could be too brittle?

I think it's fine to warn if developer don't use NuxtPage while they enables pages, even if they don't use RouterView.

I agree 👍

By the way, with ESLint, we could walk through all the component tree and check their names and find RouterView.

Indeed, that could give another DX improvement while keeping the "most basic detection" in the framework itself

Am I missing something, or do you have any better approaches?

No that sounds good to me (besides my first remark) 👍


MacOS Symlink Broken for Halide

Symlink between libHalide.16.0.0.dylib and libHalide.16.dylib seem to be broken

swang@Sams-MBA simple-app-cmake % file /Users/swang/Documents/Projects/angular-aiq/simple-app-cmake/cxx/ion-kit/ion-kit-1.7.0-arm-64-macos/lib/libHalide.dylib
/Users/swang/Documents/Projects/angular-aiq/simple-app-cmake/cxx/ion-kit/ion-kit-1.7.0-arm-64-macos/lib/libHalide.dylib: broken symbolic link to @rpath/libHalide.16.dylib

Resolved by #259


Hue Light node outputs twice

Hello,
when I send a value to my hue light node, it sends light state output twice with 3-4 seconds delay between.
Is that normal behaviour?
I´d like it to only output once. I used debug nodes to confirm I only send input once.

Is there a way to solve this?

Thank you for your notice. It should be fixed with v2.6.2 :)!

Hey, thanks for the update.
The light node does output twice occasionly now.
What's strange when that happens, is that the first output has some random brightness value, The second output has the correct value.
It doesn't matter wether it's changing color, powering on or changing the brightness.


Documentation

Hi. Thanks for your tremendous work!
I cannot seem to find the documentation for this tool although you said on another forum that it exists. Could you kindly please point me in the right direction?

Hi, there's a file that comes with the source and the releases "xtool.chm", this is the documentation, though outdated it should explain the basics of the program.

Thank you for the prompt response. Closing the issue.


Oddly non-functioning

I've had little success getting this to work and it ultimately came down to:
common.py lines 116-120 'details[artist..album..etc] = None' // record nothing for the track
and
ImportList.py lines 231-233 'skip track if details[artist..album..etc] = None' // check for the above nerf
It was an odd alignment amounting in zero functionality, at least that's what I've found. What's the go? I assume I'm missing something but if not, thought it worth a mention ...

Does ExamplePlaylist.csv import?  Double check your csv file against the example playlist to see if it matches the format.
The ExamplePlaylist.csv file has a lot of comments in it and some simple text searches at the beginning.  You can ignore those, based on the lines you narrowed in on it looks like you are attempting to import a details track. a details track line should have the following format:
title,artist,album
also, other things to try is to take your playlist you are trying to import and just a few songs before importing everything.  there could be a character or formatting issue that is throwing things off.
you should see something like the following when importing the ExamplePlaylist.csv
$ python ImportList.py ExamplePlaylist.csv
gmusic-playlist: 0.160530
gmusicapi: 10.0.0
Reading playlist... done. 51 lines loaded.
Logging into google music...

Login Successful.
Loading personal library... done. 4834 personal tracks loaded.
Searching for songs from: ExamplePlaylist
test comment (and blank track)
test fuzzy artist title search
test fuzzy title artist search
test fuzzy search for song that should return a low match

instant karma! we all shine on john lennon {s}{T} #2
Instant Karma! (We All Shine On) (2010 - Remaster),John Lennon,Power To The People: The Hits
test detailed library search (this song isn't in aa, but it's in my library)
test detailed all access search (the top fuzzy result is incorrect)
this should return a low result
test detailed search which should return song from library if you have it
test album distinction and slight artist mismatch
test slight title mismatch
test low score, mismatched title, and mistmatched artist, and comma in entry
test low score, mismatched title, mismatched artist, mismatched song, and entry comma
1o1,Chris Duarte Groop,Ronp {s}{T}{A} #11
101,Chris Duarte Group,Romp
test useless info in brackets and duplicate checks
1o1 (Live!) [In Concert] {World Tour},Chris Duarte Groop,Ronp {s}{T}{A}{D} #12
101,Chris Duarte Group,Romp
test title only search
Be Thou My Vision,Dallan Forgaill, {s}{A} #12
Be Thou My Vision,Selah,Greatest Hymns
test initial unmatched fuzzy with info in brackets
stray cats (asdfDoNotMatchMe1234) stray cat strut {s}{D} #13
Stray Cat Strut,Stray Cats,Greatest Hits

expected results
13/15 tracks imported
2 duplicate tracks
===============================================================
Adding 13 found songs to: ExamplePlaylist

ExamplePlaylist - 13/13 songs
===============================================================
13/15 tracks imported
! 0.0% of tracks could not be matched

33.3333333333% of tracks had low match scores
{T} 3 low matches were due to a song title mismatch
{A} 3 low matches were due to song artist mismatch
{D} 2 duplicates were found and skipped

66.6666666667% of tracks had high match scores

top 3 genres: [(u'Pop', 3), (u'Rock', 3), (u'Alternative/Indie', 2)]
top 3 artists: [(u'Chris Duarte Group', 2), (u'Art Of Noise', 2), (u'Stray Cats', 2)]
top 3 years: [(2010, 4), (2013, 2), (2000, 2)]
playlist playback ratio: 2.23076923077
search time: 4.41135811806


Rarity not work properly in windows version

first thank you for this great work , i wanna report a bug in current windows version , rarity not working ,   for example   if you have  3 backgrounds (yellow(common 60%) , purple(legendary 30%)
, red(rare10% )  and number of generated images 75 , you will get 25  in each color . i tested many times with different configs ( 4 folders , many images per folder ...) always same problem  rarity not working.
thank you for your help .

Hey Houssamix, thanks for making the issue, it seems to work fine for me - quick question are you seeing this behaviour when you generate the max possible combinations for your inputs?
TL;DR: It's impossible to make the rarity accurate and only output unique images, do you want the option to allow duplicates to preserve the rarity?
The reason I ask is because we guarantee uniqueness of the output images, not the rarity of components in the output images, that's determined by how many images you decide to make!
There is an inherent conflict between guaranteeing the uniqueness of the output images and preserving the rarity of the image layers/layer items. The rarity % you enter on the application is the rarity per image generation roll, and this roll can be performed several times to generate one unique image if needed (making the rarity inaccurate). This re-rolling happens more and more as the number of un generated unique image combinations goes down.
As an example, say I have an image made of 1 layer (background) and 2 rarity folders (common - 99% and legendary - 1%) with one image in each folder. There are only 2 possible outcomes, being:

Legendary background
Common background

If we tell the program we want to generate 2 images, it will guarantee image uniqueness and generate the above two combinations, making the seeming rarity be 50/50 right? But what the program has probably done, is rolled the common background on the first try and output that image, but then on the following tries it keeps rolling the common background - so it ignores it until it gets lucky and rolls the legendary background, so it outputs the image, and is done. If we wanted the 99% and 1% to be preserved, we need to create duplicate images every time a non unique roll happens, that way you get 99 common images, and 1 legendary image.
Please let me know if that was helpful at all, and if allowing duplicates is something you intended/want 😄

Hey Houssamix, thanks for making the issue, it seems to work fine for me - quick question are you seeing this behaviour when you generate the max possible combinations for your inputs?
TL;DR: It's impossible to make the rarity accurate and only output unique images, do you want the option to allow duplicates to preserve the rarity?
The reason I ask is because we guarantee uniqueness of the output images, not the rarity of components in the output images, that's determined by how many images you decide to make!
There is an inherent conflict between guaranteeing the uniqueness of the output images and preserving the rarity of the image layers/layer items. The rarity % you enter on the application is the rarity per image generation roll, and this roll can be performed several times to generate one unique image if needed (making the rarity inaccurate). This re-rolling happens more and more as the number of un generated unique image combinations goes down.
As an example, say I have an image made of 1 layer (background) and 2 rarity folders (common - 99% and legendary - 1%) with one image in each folder. There are only 2 possible outcomes, being:

Legendary background
Common background

If we tell the program we want to generate 2 images, it will guarantee image uniqueness and generate the above two combinations, making the seeming rarity be 50/50 right? But what the program has probably done, is rolled the common background on the first try and output that image, but then on the following tries it keeps rolling the common background - so it ignores it until it gets lucky and rolls the legendary background, so it outputs the image, and is done. If we wanted the 99% and 1% to be preserved, we need to create duplicate images every time a non unique roll happens, that way you get 99 common images, and 1 legendary image.
Please let me know if that was helpful at all, and if allowing duplicates is something you intended/want 😄

thank you bro for your quick replay, i tested also with 2 images in folders etc...

hello bro thank you for your quick answer, no need for duplicates, ill test it tonight and ill back to you.
can you tell me how many images you used to test ?
Thank you again for your amazing work.


The disable/enable option are not handled correctly.

The use case is: programmatically enable/disable messages at any time.
Because of the way the enable/disable option works, using set_option_value will not work as expected
A user would have to know that the “real” option is disabled, not disable
There also seems to be an issue with the processing of the enable/disable option.  They are strings from the command line, but CHECK_CODES has integers.

closed by #195


Publish frozen model?

Hi,
Are there any plans to publish a frozen model (in the .pb file)? I tried myself but failed to convert one since the code is not entirely friendly to single inference (including FIFO queue in the graph). I am wondering if there is anyone who has already frozen the model successfully and be willing to share the model or the method that you followed?
P.S. What's the input node? FIFO queue or batch? Or what else?

@chasecheng1218  can you explain more in detail what modification in python inference you need to make?

The input of frozen pb should be batch.
You need to do some modification in the inference python code to make it work.

Hi .
Please tell us what are this modification


[Bug]: Can't configurate with Modmenu and ClothConfig in fabric

Minecraft version(s)
1.19.4
Minecraft version details
No response
Mod loader
Fabric
Mod loader version
0.14.19
Fabric API version
0.80.0
In what kind of world or server did the problem occur?
Other (specify in "What went wrong?" section)
What went wrong? (Crash logs don't go here)
what the title says, you can't configurate it in the config menu, like, it doesn't give you the option like with other configurable mods
Full list of installed mods and their version numbers
sodium-fabric-mc1.19.4-0.4.10+build.24
NoFog-1.3.3_1.16.5-1.19.4-fabric
modmenu-6.2.2
logical_zoom-0.0.19
lambdynamiclights-2.3.0+1.19.4
fabric-api-0.80.0+1.19.4
cloth-config-10.0.96-forge
Did the problem cause the game to crash?
No.
Full crash report contents (if you experienced a crash)

Crash report output:

If your mods list is correct you installed the forge build of cloth config instead of the fabric one

oh that's true, my bad


Error generating code:  expected 'IDENT', found '.'

Version
1.10.0
What happened?
Trying to generate a simple query, but I get an error:
error generating code: source error: 3:9: expected 'IDENT', found '.'
Relevant log output
gionny@PC:/mnt/c/Users/gionny/Documents/Projects/Golang$ ./sqlc generate
// Code generated by sqlc. DO NOT EDIT.

package ./db/sqlc

import (

        "context"
        "database/sql"

)

type DBTX interface {
        ExecContext(context.Context, string, ...interface{}) (sql.Result, error)
        PrepareContext(context.Context, string) (*sql.Stmt, error)
        QueryContext(context.Context, string, ...interface{}) (*sql.Rows, error)
        QueryRowContext(context.Context, string, ...interface{}) *sql.Row
}

func New(db DBTX) *Queries {
        return &Queries{db: db}
}

type Queries struct {
        db DBTX
}

func (q *Queries) WithTx(tx *sql.Tx) *Queries {
        return &Queries{
                db: tx,
        }
}

# package ./db/sqlc
error generating code: source error: 3:9: expected 'IDENT', found '.'

Database schema
CREATE TABLE "accounts" (
  "id" bigserial PRIMARY KEY,
  "owner" varchar NOT NULL,
  "balance" bigint NOT NULL,
  "currency" varchar NOT NULL,
  "created_at" timestamptz NOT NULL DEFAULT (now())
);

CREATE TABLE "entries" (
  "id" bigserial PRIMARY KEY,
  "account_id" bigint NOT NULL,
  "amount" bigint NOT NULL,
  "created_at" timestamptz NOT NULL DEFAULT (now())
);

CREATE TABLE "transfers" (
  "id" bigserial PRIMARY KEY,
  "from_account_id" bigint NOT NULL,
  "to_account_id" bigint NOT NULL,
  "amount" bigint NOT NULL,
  "created_at" timestamptz NOT NULL DEFAULT (now())
);

ALTER TABLE "entries" ADD FOREIGN KEY ("account_id") REFERENCES "accounts" ("id");

ALTER TABLE "transfers" ADD FOREIGN KEY ("from_account_id") REFERENCES "accounts" ("id");

ALTER TABLE "transfers" ADD FOREIGN KEY ("to_account_id") REFERENCES "accounts" ("id");

CREATE INDEX ON "accounts" ("owner");

CREATE INDEX ON "entries" ("account_id");

CREATE INDEX ON "transfers" ("from_account_id");

CREATE INDEX ON "transfers" ("to_account_id");

CREATE INDEX ON "transfers" ("from_account_id", "to_account_id");

COMMENT ON COLUMN "entries"."amount" IS 'can be negative or positive';

COMMENT ON COLUMN "transfers"."amount" IS 'must be positive';

SQL queries
-- name: CreateAccount :one
INSERT INTO accounts (
  owner,
  balance,
  currency
) VALUES (
  $1, $2, $3
)
RETURNING *;

Configuration
version: 1
packages:
  - path: "db"
    name: "./db/sqlc"
    engine: "postgresql"
    schema: "./db/migrations"
    queries: "./db/queries"
    emit_json_tags: true
    emit_prepared_queries: false
    emit_interface: false
    emit_exact_table_names: false

Playground URL
No response
What operating system are you using?
Linux
What database engines are you using?
PostgreSQL
What type of code are you generating?
Go

I'm running using docker run --rm -v $(pwd):/src -w /src kjconroy/sqlc generate

My bad, I have mistakenly put the output path as name, and the name as path.


Dependency issue with 3.0.9

When running soda 3.0.9 scan  I run into the following exception:
(soda-core-3.0.9) $soda scan -d aws_postgres_retail sales.yml Traceback (most recent call last):  File "/Users/albin/dev/py/env/soda-core-3.0.9/bin/soda", line 5, in <module>   from soda.cli.cli import main  File "/Users/albin/dev/py/env/soda-core-3.0.9/lib/python3.10/site-packages/soda/cli/cli.py", line 24, in <module>   from soda.scan import Scan  File "/Users/albin/dev/py/env/soda-core-3.0.9/lib/python3.10/site-packages/soda/scan.py", line 28, in <module>   from soda.telemetry.soda_telemetry import SodaTelemetry  File "/Users/albin/dev/py/env/soda-core-3.0.9/lib/python3.10/site-packages/soda/telemetry/soda_telemetry.py", line 17, in <module>   from soda.telemetry.soda_exporter import SodaConsoleSpanExporter, SodaOTLPSpanExporter  File "/Users/albin/dev/py/env/soda-core-3.0.9/lib/python3.10/site-packages/soda/telemetry/soda_exporter.py", line 5, in <module>   from opentelemetry.exporter.otlp.proto.http.trace_exporter import OTLPSpanExporter  File "/Users/albin/dev/py/env/soda-core-3.0.9/lib/python3.10/site-packages/opentelemetry/exporter/otlp/proto/http/trace_exporter/__init__.py", line 40, in <module>   from opentelemetry.exporter.otlp.proto.http.trace_exporter.encoder import (  File "/Users/albin/dev/py/env/soda-core-3.0.9/lib/python3.10/site-packages/opentelemetry/exporter/otlp/proto/http/trace_exporter/encoder/__init__.py", line 19, in <module>   from opentelemetry.proto.collector.trace.v1.trace_service_pb2 import (  File "/Users/albin/dev/py/env/soda-core-3.0.9/lib/python3.10/site-packages/opentelemetry/proto/collector/trace/v1/trace_service_pb2.py", line 5, in <module>   from google.protobuf import descriptor as _descriptor  File "/Users/albin/dev/py/env/soda-core-3.0.9/lib/python3.10/site-packages/google/protobuf/descriptor.py", line 47, in <module>   from google.protobuf.pyext import _message ImportError: dlopen(/Users/albin/dev/py/env/soda-core-3.0.9/lib/python3.10/site-packages/google/protobuf/pyext/_message.cpython-310-darwin.so, 0x0002): symbol not found in flat namespace (__ZN6google8protobuf15FieldDescriptor12TypeOnceInitEPKS1_)

This can be resolved by downgrade protobuf:
pip uninstall protobuf
pip install protobuf==3.19.4

SODA-1195

https://github.com/protocolbuffers/protobuf/issues/10571

This is an upstream issue - closing this because the upstream bug seems to be fixed.


Bug:  buffers sets wrong buffer as "current buffer" if the actual current buffer is nobuflisted

RTFM Checklist

[X] I have searched exisiting issues / discussions
[ ] I have read the Wiki including the Advanced section
[ ] I have read man fzf / I am well versed in shell fzf

Operating system
Ubuntu 22.04
Shell
bash
Neovim version (nvim --version)
0.10 b3f9da952
Fzf version (fzf --version)
0.48.1
Output of :lua print(os.getenv('FZF_DEFAULT_OPTS'))
--bind alt-a:select-all,alt-d:deselect-all,alt-t:toggle-all --color=dark
Is the problem reproducible with mini.sh?

[X] My issue is reproducible with mini.sh
[ ] My issue IS NOT reproducible with mini.sh
[ ] I have not tested with mini.sh (not relevant, requires LSP, Windows, etc)

Fzf-lua configuration
No response
Describe the bug / steps to reproduce
./scripts/mini.sh
:e README.md
:new
:set nobuflisted
:FzfLua buffers

README.md is shown as the "current buffer" in the fzf window, rendering it not selectable, despite that the actual current buffer is the new empty nobuflisted buffer.

Ty @tomtomjhj for reporting this.
Seems like a somewhat contrived example, I think the proper solution here would be always add the current buffer regardless of buflisted setting.

@tomtomjhj, https://github.com/ibhagwan/fzf-lua/commit/b5e08be6835e7ef81d0b90ac58c185e276a771cd ignores buflisted for the current buffer.

Thank you! I stumbled on this issue when using a plugin that sets nobuflisted for the buffer that displays various information associated with some "main" buffer.


Localization still not working on v5.1.1

Guidelines
When I trying to use localization props I got an error i console browser, and crash of app
localization={{body: {}}}
Even when I try use like this I am getting:
Uncaught TypeError: _deepmergeTs.deepmerge is not a function
node_modules bundle.js:15370
setState vanilla.js:3
mergeLocalization LocalizationStore.js:73
node_modules bundle.js:15463
React 13
scheduleRefresh react_devtools_backend.js:6514
performReactRefresh react-refresh-runtime.development.js:298
performReactRefresh react-refresh-runtime.development.js:287
node_modules bundle.js:106576
setTimeout handler*enqueueUpdate RefreshUtils.js:86
executeRuntime RefreshUtils.js:249
$ReactRefreshModuleRuntime$ main.abd3f7bb884ddf7f46ed.hot-update.js:315
tsx main.abd3f7bb884ddf7f46ed.hot-update.js:328
factory react refresh:3
Webpack 18

Can you sahre your package json and recreate it in a sandbox?

React 18 is not supported yet


Send Max : insufficient balance error

I had 1k sats with confirmation and I was trying to send max, it says insufficient balance
https://github.com/bithyve/bitcoin-keeper/assets/107177634/a2351646-b7c6-4a4f-886e-b0a0150aac28

The error is spawning because the mainnet fee is so high that sending 1000 sats is not feasible(the fee itself is multifold of the balance).


fix: don't show warning message with Node16/NodeNext

Summary

fixes #4266
Test plan

Green CI
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information
N.A.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

6 of 6 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall first build on fix/esmodule-interdop at 96.034%

Totals

Change from base Build<PHONE_NUMBER>:
96.0%

Covered Lines:
4050

Relevant Lines:
4149

💛  - Coveralls


w2grid.request() uses wrong URLs for save and delete commands

I'm submitting a ...
[x] bug report
[ ] feature request
[ ] question about the decisions made in the repository

What is the current behavior?
On grid with remote data source methods w2grid.save() and w2grid.delete() makes request to w2grid.url.get URL.

If the current behavior is a bug, please provide the steps to reproduce and if possible a screenshots.
Try to save changes or delete record on grid with remote data source.

What is the expected behavior?
request() should use w2grid.url.save and w2grid.url.remove URLs respectively.

Other information
I think problem appeared because https://github.com/vitmalina/w2ui/commit/5f7e5986e9327ebbc8075f126fc62acb2901e475#diff-8b4d4a6961edfba8a65c282e9645abb3L2376 removed params.cmd, but not changed

if (params.cmd == 'save' && typeof edata.url == 'object')   url = edata.url.save;
if (params.cmd == 'delete' && typeof edata.url == 'object') url = edata.url.remove;

or
if (params.cmd == 'save')   ajaxOptions.type = 'PUT';  // so far it is always update
if (params.cmd == 'delete') ajaxOptions.type = 'DELETE';

or any other place where params.cmd is used in request().

Related: https://github.com/vitmalina/w2ui/issues/1837

Got corrected by https://github.com/vitmalina/w2ui/commit/670c31e80facb0e2a18db6140fe29f55a8732344 and https://github.com/vitmalina/w2ui/commit/384f163b5378059fd5d32ba08a986def37a559d2


Feature request: Add optional flag to set player width in columns

I appreciate the default small size but with the number of character columns many songs have rather long names that end up showing with ellipsis at the end.
It would be nice to have a flag allowing to set the number of columns for the player.

I might be able to add some sort of arbitrary unit, but because of the way lowfi is rendered it might make some things difficult such as with the control bar.

Resolved with https://github.com/talwat/lowfi/commit/a6f3eb034de7fa41fdf64b6cc499bbfdd2106622.


Assembly.GetCustomAttributes(Type attributeType, bool inherit) is not working

Steps To Reproduce
https://deck.net/829ec36deb1a58338f7fbab3c4a14dca
public class Program
{
    public static void Main()
    {
        var assemblies = AppDomain.CurrentDomain.GetAssemblies();
            foreach(var assembly in assemblies)
            {
                Console.WriteLine(assembly.FullName);
                var myAssemblyAttr = assembly.GetCustomAttributes(typeof(MyAssemblyAttribute), false);
            }
    }
}

[AttributeUsage(AttributeTargets.Assembly)]
public class MyAssemblyAttribute : Attribute
{
}

See Also
https://forums.bridge.net/forum/bridge-net-pro/bugs/4991-assembly-getcustomattributes-type-attributetype-bool-inherit-is-not-working

Fix merged upstream, it will be available next Bridge release (see version milestone attached to the issue -- currently 16.5.1).


Blender display Nvidia smi is not recognized as an internal or external command, an executable program or a batch file.

“nvidia-smi” is not recognized as an internal or external command, an executable program or a batch file.

ZLUDA with Blender 4.1 compatibility version used
zluda.zip

This is not a zluda issue nor blender; it is a ComfyUI-BlenderAI-node issue.

Does it blocks some functionality?
nvidia-smi is used for querying system GPU properties like GPU clock, memory clock, brand of VRAM, etc. nvidia-smi missing should not be a problem

No problem, Zluda works well, only display the error

Ok, I'm closing it then


[WIP] CI

Resolves #6.

Run tests of Result-iOS and Result-Mac for each commits.
Archive frameworks using carthage archive and upload artifacts for each git tags.

Some tasks are still not completed because I don't have admin rights.

[ ] Save developer identity at script/certificates/development.p12.
[ ] Save passphrase encrypted by travis encrypt P12_PASSWORD=.... in .travis.yml.
Ref: http://docs.travis-ci.com/user/encryption-keys/
[ ] Save GitHub API token encrypted by travis setup releases.
Ref: http://docs.travis-ci.com/user/deployment/releases/
[x] Enable CI on https://travis-ci.org.

I've tested automated release is working on my own forked repository.
@robrix Could you complete remaining tasks?

Archive frameworks using carthage archive and upload artifacts for each git tags.

Nice :+1:

I’ve enabled CI, but I’m not having a lot of success understanding the rest of the tasks. Additionally, I can’t seem to get the travis gem installed on 10.11. Does anyone know a workaround for that?

Additionally, I can’t seem to get the travis gem installed on 10.11.

Oh, really? I also use 10.11 and can install travis by [sudo] gem install travis. If you still have trouble with installation, please email me following items:

.p12 of your developer identity.
(ex. https://github.com/ishkawa/Result/tree/feature/travis-ci/script/certificates)

Passphrase for the .p12 file.

GitHub API token of this repository.

I will setup .travis.yml using them! If you hesitate to send developer identity and its password, I can provide my own developer identity for this repository (if it's OK).

If you'd like to, it's also possible to automate pushing the podspec to CocoaPods trunk during the release process too.
See https://fuller.li/posts/automated-cocoapods-releases-with-ci/

@ishkawa: Curiously, I’m still having some trouble with the install.
I don’t really understand why we’d need a certificate for CI. IIRC setting up CI for Set didn’t require that, but perhaps things have changed since then?
I don’t mind you using your own certificate (I don’t actually have one at present), if you’re comfortable with that, but it sounds to me like that’s likely to get the certificate in question revoked since it’ll be public. (It’s totally possible that I have just failed to understand this altogether.)
@kylef: That’s a splendid idea! I don’t know if @ishkawa will want to tackle that, but I’d be delighted to review a PR automating that.

Curiously, I’m still having some trouble with the install.

:sob:

I don’t really understand why we’d need a certificate for CI. IIRC setting up CI for Set didn’t require that, but perhaps things have changed since then?

A certificate is required by carthage build. Running it without any certificates causes this kind of error. If we remove auto-archiving and just run test tasks, we don't have to import a certificate. I think just testing is still valuable enough.
Now, we have to take a plan from following choices:

Run testing only (No certificates are required).
Run testing and archiving (Certificates and annual updates are required).

I'm not sure, but it might be better to start form the first choice.
@kylef Thank you for the cool guide. I'll consider whether to apply it after setup fundamental CI :smile:

To make CI tasks simpler, I've reverted some commits that requires signing items. I think automating release processes is still cool but releasing 1.0 is more important.
Now, CI is working and this PR is ready to merge!!

:heart: Thank you so much @ishkawa. I agree, it would be nice to automate releases, but having CI is a tremendous improvement! :bow:


feat(chat): reload state if tab inactive more then five minutes (Issue #784)

Description:
Add state reload if tab inactive more then 5 minutes.
Issues:

Issue: https://github.com/epam/ai-dial-chat/issues/784

Checklist:

[x] the pull request name complies with Conventional Commits
[x] the pull request name starts with fix(<scope>):, feat(<scope>):, feature(<scope>):, chore(<scope>):, hotfix(<scope>): or e2e(<scope>):. If contains breaking changes then the pull request name must start with fix(<scope>)!:, feat(<scope>)!:, feature(<scope>)!:, chore(<scope>)!:, hotfix(<scope>)!: or e2e(<scope>)!: where <scope> is name of affected project: chat, chat-e2e, overlay, shared, sandbox-overlay, etc.
[x] the pull request name ends with (Issue #<TICKET_ID>) (comma-separated list of issues)
[x] I confirm that do not share any confidential information like API keys or any other secrets and private URLs

/deploy-review

/deploy-review

/deploy-review

/deploy-review


fix bug: not working on iOS 7. Change the tinted button style from Custo...

UIButton's tintColor property seems not working well with iOS 7, so I changed the tintedButton's style from Custom to System to make it work. Reference here: http://stackoverflow.com/questions/11066008/why-is-my-uibutton-tintcolor-not-working

Awesome, thanks!


[nfc] Clean up JSG build

Spiritual successor to #2012. Also see the downstream PR.

[build][nfc] Clean up JSG build file in accordance with IWYU
[nfc] Drop support for old V8 version
This was only needed around the time of the switch from 12.0 to 12.1.

Everything looks good here but let's be sure to have an internal CI run given the build file changes.

Everything looks good here but let's be sure to have an internal CI run given the build file changes.

There is an internal PR (same branch name).

awesome, ok! internal PR stamped!


Add flag to disallow name overwrites in autoneg annotation

This PR follows up on https://github.com/GoogleCloudPlatform/gke-autoneg-controller/pull/44
We (Spotify) would like to only use generated backend service names and disallow users from setting custom service names in annotations to prevent name collision of services.

Thanks Sophy! Looks great - merging.


Contextual Help Demo

Introduce a demo that illustrates use of contextual help in forms or elsewhere. This addresses the following topics:

Use of placeholder text
Popovers and info tips
Use of tooltips

@stacymcauliffe I have created this new issue to capture the work that we agreed needed to happen coming out of yesterday's meeting on this topic. I'm assigning to you as a first step. Please post any preliminary document here. I think that ultimately this will lead to a demo that shows how these elements are used on forms and elsewhere. Let me know if you have questions.

Drafted the preliminary document and am interested to get feedback. Thanks for starting this issue, @mcarrano - Great topic!
https://docs.google.com/document/d/1RehnQ1GQLYwZqHxYjUm8Du6zNRyyv3nNrwEVHfrxSaw/edit?usp=sharing

Due to significant overlap with forms guidelines, we will be rolling this content into the PF writing standards being tracked in issue #722 in pf design repo. Closing this issue.

Closing this issue.


Debugging nock mismatches trips up every developer in my org

Nock is incredibly powerful. But debugging nock misses confuses everyone I know. Even when you turn on nock debug the best you can hope for is a body doesnt match line. But then you don't know why it doesn't match. A lot of developers end up temporarily rewriting tests at that point to intercept the body and call out to deep object checks so they can try and get a sense of whats going on. But the whole thing is a pain.
I'd suggest adding better logs around json body matching that shows a diff between exception vs received body.

Pull requests welcome 🤗


Non-DOM Backend

The pluggable backend architecture of react-dnd seemed like a possible fit for my use case, which is a 2d <canvas> dnd project, but it seems like the core of react-dnd does actually have particular ties to the DOM. This is most obvious in wrapConnectorHooks which calls throwIfCompositeComponentElement. This pretty explicitly shuts down any attempt to use react-dnd with non-DOM components:
throw new Error(
	'Only native element nodes can now be passed to React DnD connectors.' +
		`You can either wrap ${displayName} into a <div>, or turn it into a ` +
		'drag source or a drop target itself.',
)

Am I wrong in my reading that react-dnd cannot actually be used on anything other than DOM nodes? Is there a compelling reason that this needs to be the case? It seems like a significant benefit of a pluggable backend would be to encapsulate these sorts of dependencies.

I am not sure either what the issue is here. Potentially if the problem is a implementation based on a ref we could make connectDropTarget/connectDragSource accept a function:
connectDropTarget((getRef) => <Box innerRef={ref => getRef(ref)} />)

I am still new to this project so I made be completely missing how it works.

Pretty interested in the feedback here too.  Currently not seeing any widely used package for dnd in React Native, so would be intrigued to know if this is a potential solution.

Would be nice to have. Using react-native-web. Need this!


Window is too small to fit feed

The window is too small to show the feed as you can see here: https://puu.sh/wD06l/c12014aeff.png
Would be nice if that could be fixed ;-)

I confirm. Too such problem. Last kubuntu.

@vladcraft @sebastiansondergaard downgraded to 3.2.0(Issue started in 3.2.1) fixed the issue for me. Hopefully 3.2.3 will have this issue fixed.

This is a duplicate and already fixed on master. I've just released a new version right now.


Add automated workflows for greetings, autolabel, etc

Tell us about the task you want to perform and are unable to do so because the feature is not available
I would like to use github actions to design a workflow for automated greetings when user makes PR, raises issue,etc
Describe the solution/feature you'd like us to add
Use github actions
Any other feedback you would like to provide regarding the site
N/A

We don't need it as we actively engage with our contributors.


Move quota max_amount and expiration from QuotaDescriptor to QuotaParams

This change is

Jenkins job mixer/presubmit passed

Jenkins job mixer/e2e-smoketest passed

Jenkins job mixer/manager-regression passed

PTAL.

Jenkins job mixer/presubmit passed

Jenkins job mixer/e2e-smoketest passed

Jenkins job mixer/manager-regression passed

Right now, the quota manager only receives a single chunk of config. Steps
2 and 3 don't exist as far as I know.
On Sat, Apr 15, 2017 at 7:51 AM, mandarjog<EMAIL_ADDRESS>wrote:

@mandarjog commented on this pull request.
In pkg/aspect/config/quotas.proto
https://github.com/istio/mixer/pull/549#discussion_r111666505:

@@ -32,7 +33,15 @@ message QuotasParams {
// Map of quota_descriptor label name to attribute expression. At run time each
// expression will be evaluated to determine the value provided to the aspect. The
// result of evaluating the expression must match the ValueType of the label in the quota_descriptor.

   map<string, string> labels = 3;

   map<string, string> labels = 2;

   // The upper limit for this quota.

   int64 max_amount = 3;

This is the number of quota configs quota manager should get.

most specific subject within every scope
quota combine step which will be given all these quotas to combine.
dispatch those quotas

What visibility are you looking for in inside the quota manager ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/istio/mixer/pull/549#discussion_r111666505, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AVucHamBfPPh66_crXkdm0ghYTbr6NxXks5rwNmOgaJpZM4M9pl3
.

PTAL.

Jenkins job mixer/presubmit passed

Jenkins job mixer/e2e-smoketest passed


Ignore implementation_imports in non-lib files

Work towards https://github.com/dart-lang/linter/issues/4859
There are libraries outside a lib/ directory, which violate implementation_imports. This PR ignores all such violations.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

@goderbauer should these be all accompanied by TODOs or something? Not sure what the policy for violating this rule is, as I imagine it is violated a fair amount.

Can we just fix these? I am making an attempt in https://github.com/flutter/flutter/pull/143594.


Add 『✿◉●•◦  𝕱𝖗𝖔𝖘𝖙  ◦•●◉ ✿』#3293 [build]

what are you doing?

te bot is confused by your unicode i think lol

nope, it just failed to close ths issue
stop spamming my webhook pls thanks


reva_version="(Unknown)" on latest revad builds

For some reason, the git output that used to fill these variables:
https://github.com/cs3org/reva/blob/4a9be347ac29706e429d47525815e638f9770f29/Makefile#L5-L8
... is empty:
[salfagem@lxplus765 ~]$ kubectl exec -ti iop-gateway-6fbcd98bb-cgv5k -- /go/bin/revad -version
version= commit= go_version=go1.13.15 build_date=2021-01-15T14:10:42+0000

... which contrast with the version string returned by the reva client (generated on the daily release pipeline):
reva version
version=2021_01_15T150629_47119beffae215190124e98e063f1734b9e75938 commit=47119beffae215190124e98e063f1734b9e75938 go_version=go1.13.15 build_date=2021-01-15

This issue makes the reva_version exposed via revad_sys_info metric to be "(Unknown)"
# HELP revad_sys_info A metric with a constant '1' value labeled by various system information elements
# TYPE revad_sys_info gauge
revad_sys_info{reva_build_date="2021-01-15T14:10:42+0000",reva_version="(Unknown)"} 1

Therefore, some of these values are missing on the site information dashboard:

cc/ @Daniel-WWU-IT

Therefore, some of these values are missing on the site information dashboard:

cc/ @Daniel-WWU-IT

Nothing I can do about this. I use the information that is somehow built into Reva during CI, but if that is empty, I output this "(Unknown)" instead. I guess during all these recent changes to the CI, something must have broken...

Nothing I can do about this. I use the information that is somehow built into Reva during CI, but if that is empty, I output this "(Unknown)" instead. I guess during all these recent changes to the CI, something must have broken...


Anyone able to build and run with Xcode 15.2 + iOS 17.3?

I've been able to build with XCode 15.2 and run my project in iOS 16.1.
However, I cannot run it on iOS 17.3 as Xcode stops execution and throws an error as seen in the screenshot. The Unity project runs fine if I run it standalone (unless I change the Data folder's membership to UnityFramework - otherwise it gives a "IL2CPP initialization failed" error).
Did anyone encounter a similar problem?

For anyone stuck here, here's how I solved the issue:

I've upgrade my project from React Native 0.72.3 to 0.73.2 (this might not be necessary based on your experience on the second step below).
I've disabled "Thread performance checker" for the scheme in Xcode. Otherwise the app was being built but Xcode was stopping execution as soon as it started running the app.

Yess that's Perfect👍, The Issue is in the library for the latest version of Xcode and IOS. Thanks for the Solution @serdarcevher


Support for Custom Base URL for Dynamic Imports in External Widgets

Description
As a developer using Vite, I want to be able to define a custom base URL for dynamic imports in modules, so that resources are always loaded from my hosting domain (e.g., https://my-domain.com/) rather than the domain where the widget is embedded (e.g., https://example.com/).
Currently, I use Vite to create widgets that can be embedded on any webpage, for example:
<script src="https://my-domain.com/widgets/chatbot.js" type="module"></script>
<chatbot-widget key="YOUR_CHATBOT_KEY" language="LANGUAGE_CODE"></chatbot-widget>

This works perfectly as long as I do not use dynamic imports. However, when I include a dynamic import like the following:
const module = await import("./Component.tsx");

The module gets loaded from the domain where the widget is embedded (https://example.com/Component-AS23dw.js) rather than my own domain (https://my-domain.com/Component-AS23dw.js).
This creates issues since the widget and all its dependencies (including dynamically loaded modules and CSS) should always load from my domain, not the embedding domain.
Suggested solution
In the Vite configuration, it would be useful to add an option to specify a base URL for dynamic imports. This would ensure that all dynamically loaded modules reference the correct domain, regardless of where the widget is embedded.
For example:
// Vite could allow setting an option in `vite.config.ts` like this:
export default defineConfig({
  build: {
    dynamicImportBase: 'https://my-domain.com/'
  }
});

Alternatively, support could be added for the following syntax in Vite for dynamic imports:
// Using `import.meta.url` to dynamically import modules:
const module = await import(new URL('./Component.tsx', import.meta.url).href);

This is currently supported for asset imports but not for module imports. Adding support for this would allow developers to more easily specify the correct base URL without hardcoding it or relying on brittle workarounds.
Alternative
Currently, the workaround is to avoid dynamic imports, which limits flexibility and optimization (e.g., code splitting).
Additional context
This feature would be especially useful for developers building widgets that are meant to be embedded across multiple websites, ensuring that resources are always loaded from the correct domain, and avoiding potential CORS issues or broken imports.
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

Your issue sounds similar to https://github.com/vitejs/vite/discussions/18097. Can you check if the suggested solution works?
Also I'm wondering if you can just use relative base like base: "." https://vitejs.dev/config/shared-options.html#base

Thank you @hi-ogawa ,
this was what I was searching for.


"file exists" when opening a terminal

When opening the terminal, I have this error:

/home/$USER/.sdkman/bin/sdkman-init.sh:144: file exists: /home/$USER/.sdkman/var/version

The issue is fixed by running sdk selfupdate force but it appears again once the OS is restared.
Environment:

Ubuntu 16.04
shell: /bin/zsh
sdkman: latest version

This is because your shell has "noclobber" set, and the sdkman script is trying to clobber a file. You can check this by running a command like this:
set -o | grep clobber

There isn't a good solution, other than setting the "clobber" shell option for any shell that runs sdkman, prior to running any sdkman script. e.g.:
set -o clobber

Of course, you can set this in a startup file for all shells, or for a shell that you only run for sdkman.
I suppose you could modify the sdkman scripts so it uses ">|" instead of ">" when overwriting files. If this is done I'm guessing the next update to sdkman will overwrite these changes.

@jverne: I'm using zprezto and noclobber option is enabled by default. I have disabled it and everything works as expected now. Thank you!
If there is no other solution (implemented in sdkman), this issue can be closed.

Closing this issue as the problem lies with prezto being opinionated about file clobbering. I personally use ZSH with ohmyzsh and have no problems.
Worth adding that I also experienced this problem in the past with prezto which ultimately led me to switching to ohmyzsh.

This is not a zprezto issue. It is a shell issue. zprezto just also happens to complain when the shell has noclobber set.
Here is what happens on plain bash:
jverne@YKF-JVERNE:~$ echo foo > out
jverne@YKF-JVERNE:~$ echo foo > out
bash: out: cannot overwrite existing file
jverne@YKF-JVERNE:~$ sdk
bash: /home/jverne/.sdkman/var/broadcast_id: cannot overwrite existing file
bash: /home/jverne/.sdkman/var/broadcast: cannot overwrite existing file

It occurs to me that a script fix might be to use a subshell to update/clobber files. e.g., in all the places we clobber files, we do something like this:
( set + noclobber; do_the_thing_that_clobbers > some_file )

To be robust we'd probably refactor this into functions; one for Bourne-style shells that support set or setopt with the clobber option, and those shells that do not.

This is not a zprezto issue. It is a shell issue. zprezto just also happens to complain when the shell has noclobber set.

@jverne I beg to differ, and can prove it in a few lines in a terminal. Give it a try yourself:
$ docker run --rm -it ubuntu:latest bash
# echo "test" > myfile
# echo "test" > myfile
# apt-get update && apt-get --assume-yes install zsh && zsh
# echo "test" > myfile

You will see no error message in either case. Even when installing ohmyzsh, no errors. Only when using prezto these problems surface. This is due to prezto being opinionated about clobbering (and many other shell behaviours), without allowing you to override these settings. We won't be making any changes in our codebase to accommodate such a framework.

@marc0der you might be seeing a related zprezto issue, or the error being issued might be massaged by zprezto, but my example above was pure, unadulterated POSIX bash as shipped on Linux. All your example shows is that you do not have the noclobber setting enabled. But noclobber is not some special thing offered by zprezto; it is a shell option present on most modern Bourne shells. It is part of the shell environment shipped with bash, zsh, ksh.
If you use a modern Bourne shell flavour like bash, ksh, or zsh, and you have the "noclobber" option set, some utilities will fail to overwrite files. SDKMAN is one of those utilities. zprezto is another.
The underlying root cause of this report is not zprezto. It is the noclobber option that is present on most or all modern Bourne-style shells. (csh, et al, might have their own version of this, but I don't use csh.)

Yes, all distros come with noclobber disabled as sensible default. This is why we have many many happy users who never encounter this problem. If you do want to enable it, do so at your own risk, but we won't be working around it.

As I said, I don't really care if this isn't something you want to allow for. It's tricky and error prone, like most shellisms.
But we should at least document the root cause instead of pointing fingers at poor zprezto. Just this week I installed a test distro and noticed noclobber was set, and noticed someone else had it set on some build box I was inspecting. So, no, not all distros disable it. And we can't count on it being unset all the time.
In some cases it is perfectly sane to expect to run both sdkman and have noclobber enabled. At least if someone else runs into it they can get confirmation here.

I always set noclobber in my shell environments to avoid accidentally overwriting files.  To solve this sdkman issue, I edited ~/.sdkman/bin/sdkman-init.sh and disabled noclobber at the very top, and enabled it again at the very bottom:
# beginning of sdkman-init.sh file
set +o noclobber
...
# end of sdkman-init.sh file
set -o noclobber

@ironcamel Probably a better solution is to save the user's clobber state and restore it at the end of file, so that way the shell's clobber isn't changed

sdf

noclobber is a useful safety feature for interactive shells, and sdkman could easily work with it by just changing > redirections to >|. Unfortunately at the moment the only feasible options are disabling that feature, which makes normal shell use more risky, or not using sdkman. Modifying sdkman scripts manually is not a good solution because those changes would get overwritten by updates. So it would be great if sdkman could make this small change to accommodate users' preferences.
This is actually similar to the situation in Vim, where plugins are expected to be able to cope with changed options as it is normal that users want to configure Vim for their preferences.

@majutsushi is it worth creating a feature branch with this change? If all tests pass and it works on both bash and zsh, I'd be happy to merge.

I've opened PR #748 which should fix the issue. Thanks for considering it!

I guess this issue can be closed now. Thanks for merging!


tracking types of discounts

As a member of the marketing team, I'd like to be able to filter financial reports on marketing discounts, so I can focus my analysis on the transactions where marketing coupons were used.
As a member of the MITx team, I'd like to be able to filter on financial assistance discounts, so I can analyze and project how much it costs.
in xPRO we have various coupon "types." These have proven helpful for filtering data in reports
https://github.com/mitodl/mitxpro/blob/2dfd014f70abf56b3bde856160ad5d2686bfa6d3/ecommerce/models.py#L506-L515
Acceptance Criteria:

[ ] Attach a type to coupons. Suggested types:

marketing (sometimes called "B2C")
customer support (enrollment codes for medical issues, etc.)
flexible pricing / financial assistance
sales (sometimes called "B2B", for the future)

[ ] include the type in AR/Financial reports

Removing the On Hold tag as it seems we can start working on this one as it is moved to TODO.

As we are adding a new choice field that will also indicate whether a discount is for financial assistance or not. We will also need to remove the usages of the existing boolean field for_flexible_pricing. This includes writing a data migration and some refactoring to use the new field in place of for_flexible_pricing.
@jkachel WDYT how we should approach replacing for_flexible_pricing? Should all this be a part of a single PR? Or should we create a separate ticket/PR for replacing the usages and creating a data migration?
We (me & @arslanashraf7) think that we should go with a single PR.

include the type in AR/Financial reports

@pdpinch What do I need to do for this? Sorry, but I do not know about these changes. Is this something that I need to work on?

@jkachel WDYT how we should approach replacing for_flexible_pricing? Should all this be a part of a single PR? Or should we create a separate ticket/PR for replacing the usages and creating a data migration?

I think this is fine to roll into the open PR, since it's all affecting the same parts of the code.

include the type in AR/Financial reports

@pdpinch What do I need to do for this? Sorry, but I do not know about these changes. Is this something that I need to work on?

I opened a separate issue for this, and put it on hold. You can close this issue when #1390 is merged.


Build common layout containers

We need some common layout containers. These provide consistent layout styles for any components that inherit from them. These containers can also provide common hooks that could be handy for any other container, like responsive hooks etc.
The initial containers that we want is:

[x] Base Layout Container

This is the basic layout container. We can use this directly if we don't have any fancy layout, but if we want to build custom layout containers (like the sidebar container), then those layout containers should inherit from this one.
This container does not take other components like text or paragraphs etc. It's immediate children should be other layout containers, or content containers.

[x] Content Container

This is a container that is meant to carry the text content and other related content.

[ ] Toolbar Container

A toolbar that can take a horizontal list of items, like buttonsl. The toolbar should have properties that allow you to set it to absolute or fixed positioning if needed.

[x] Sidebar Container

This is the first of the more complex composite containers. This container has a sidebar with related content. Direction on the container can be specified to have the sidebar left or right. The sidebar has a property that allows us to show/hide the sidebar when on larger screens. On mobile views, the sidebar will stack either above or below the main content, depending on the direction (left = above, right = below)

I created the Toolbar component but it does not yet include the "sticky" behaviour where you can optionally have it stick to the top of it's layout component, and content scrolls underneath.


Add getEntity to the Entity wrapper

This is so that I can access net.minecraft Entity specific methods.

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

Just add a @Getter to the entity field :)

Yeah I was thinking that looking through other classes.

By the way I can't sign the CLA currently, seems to be some issues. It seemingly kept expiring the token on approval and trying to then make me re-authorize. Now GitHub has flagged it as suspicious abuse.
https://bots.are-pretty.sexy/20ba84.png

Thank you for your contribution! <3


As a Marist Student, I can select a date range and a time range of my trip so that I can plan my trip accordingly.

The driver will be able to create a destination. The Driver can select some type of slider that will show a time/date range it will be in


[Shameless show-off] Polyglot - Userscript for CodeWars

Hello all,
Recently I've created a Tampermonkey Userscript which would help me with some issues which were particularly annoying for me, especially addressing issues #790, #1162, #1272, #1641 and indirectly #727, #752. It seems to work for me and does not blow up for the other user (thanks for testing, B4B!), so maybe someone else would also find it useful. Feel free to use it, or to completely ignore it ;)
You can find the script in this repo.
Small summary of features:

Solved languages are marked:

Filtering by "Solved/not solved in [language]":

[Removed]

[Removed]

Copy code to clipboard

Solutions are tabbed by language

Spoiler flags are always displayed, so it's more difficult to accidentally flag/unflag a comment.

[Added:]
8. Solved langiages are dimmed also in language dropdowns, so you do not select the same language twice!

Hopefully, some/all of the features will be one day incorporated into CodeWars website itself. If anyone of you finds the script useful, then great. If not, well, you probably are not annoyed by these issues, so that's also great!
Have fun, and keep on CodeWarrioring!

Closing because this is not an issue.
Also, can you not provide easy access to site events? It's not made to be accessed by average user or opened frequently and it's a performance hog according to Jake.

Closing because this is not an issue.

Well, I hoped you would at least mark the thread as feature suggestion(s)  ;)

Also, can you not provide easy access to site events? It's not made to be accessed by average user or opened frequently and it's a performance hog according to Jake.

Sure thing, I did not realize that in full extent because, quite frankly, site events page was the thing I've learned on my very first day when joined CW and I thought it's some of a really common CW knowledge. Would not it be better just to remove it completely, then?
Would you want me to remove the feature no. 4 too (resolution of site event IDs into links), or can it stay for people who already know? Please mind that resolved links are persisted by the script so it would not grind through DB too much, but you're the boss, so I remove it if you request so!
Cheers!

Well, I hoped you would at least mark the thread as feature suggestion(s) ;)

If you want to request features not already done, then please fill out the issue template provided for each so it can be tracked independently. I don't think 3 and 4 will happen though.

Sure thing, I did not realize that in full extent because, quite frankly, site events page was the thing I've learned on my very first day when joined CW and I thought it's some of a really common CW knowledge.

I'm sure there's users who doesn't know about it, installs your script because they want filtering by language, then finds "Site events" link. I don't have the time to review the script and I have no idea how popular this script is or will be.
Just opening /site-events does many database queries because it's not optimized. I can remove it or optimize it, but it's extra work with low priority.

Would you want me to remove the feature no. 4 too (resolution of site event IDs into links), or can it stay for people who already know? Please mind that resolved links are persisted by the script so it would not grind through DB too much, but you're the boss, so I remove it if you request so!

I don't really care at this point. I'll do something if it's clearly affecting performance.

Okay, so I removed all functionalities related to site events. We can live without it, I think.
Regards!

Hi, Should it works?
Looks like it has no effect ....

It works for me, but I might miss something... Catch me on gitter, for example with PM, and we will try to see what's your problem.

@hobovsky Do you want to post a current version on https://github.com/codewars/codewars.com/discussions/categories/show-and-tell?

Well, I hoped to gradually remove features from the script as they get added to  Codewars itself :) but sure, I will put something there.


Package getConfig deprecated?

I could be wrong about this, but I was under the impression that storing configuration settings via $pkg->getConfig() is now considered legacy, with the Config class now being the preferred approach.
As Config uses flat files to store data it's faster too.

I think you're thinking of $pkg->saveConfig() etc...
$pkg->getConfig() is not deprecated.

Well deprecated isn't the right word, it's still available. I just thought the suggestion now was to use the non database way of storing config data.

There's likely a few things like that, that are not the v7 standard. Likely a lot of stuff from the early days of putting this together for the first client (November). So I'm open to cleanup, suggestions, and better best practices.


nheko: 0.4.2 -> 0.4.3

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Failure on x86_64-linux (full log)
Attempted: nheko
Partial log (click to expand)
building '/nix/store/2g6707fb036lxzy5hs9ldy1b009mbby9-source.drv'...

trying https://github.com/mujx/nheko/archive/v0.4.3.tar.gz
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   118    0   118    0     0    307      0 --:--:-- --:--:-- --:--:--   307
100 4356k    0 4356k    0     0  2501k      0 --:--:--  0:00:01 --:--:-- 3536k
unpacking source archive /build/v0.4.3.tar.gz
cannot build derivation '/nix/store/jvl69sv8yscxhkbhsxavgmp5ngqf9i9x-nheko-0.4.3.drv': 3 dependencies couldn't be built
error: build of '/nix/store/jvl69sv8yscxhkbhsxavgmp5ngqf9i9x-nheko-0.4.3.drv' failed

Success on aarch64-linux (full log)
Attempted: nheko
Partial log (click to expand)
glibPreFixupPhase
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/8zrd7rklz092if1gb5ylabx6i5srvb4c-nheko-0.4.3
shrinking /nix/store/8zrd7rklz092if1gb5ylabx6i5srvb4c-nheko-0.4.3/bin/nheko
strip is /nix/store/qg2agrqkf240s656d207zqhipl0bc2id-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/8zrd7rklz092if1gb5ylabx6i5srvb4c-nheko-0.4.3/bin
patching script interpreter paths in /nix/store/8zrd7rklz092if1gb5ylabx6i5srvb4c-nheko-0.4.3
checking for references to /build in /nix/store/8zrd7rklz092if1gb5ylabx6i5srvb4c-nheko-0.4.3...
postPatchMkspecs
/nix/store/8zrd7rklz092if1gb5ylabx6i5srvb4c-nheko-0.4.3

Hmm… looks like an error unrelated to nheko on x86_64-linux (one dependency of mesa-noglu couldn't be built).
Let's try again.
@GrahamcOfBorg build nheko

Success on aarch64-linux (full log)
Attempted: nheko
Partial log (click to expand)
/nix/store/8zrd7rklz092if1gb5ylabx6i5srvb4c-nheko-0.4.3


Chat settings is too wide and I can't read. (CSS bug)

I'm on mac os and if I open the chat and click in settings, I can't see anything:

Duplicate of https://github.com/danielbuechele/goofy/issues/346.

Sorry @Etheryte, I try to search as "settings" and I couldn't find.
Closing this issue, so we keep the old one.


Dpcpp port ELL and the porting script

This PR adds the dpcpp ported ELL and the porting script
TODO:

[ ] clear some unused code in script
[ ] add ConfigSelection into ELL
[ ] merge dense

format!

@upsj I think we put it here like cuda2hip. it will be hard to use on adaptive changes.
implement cuda -> apply cuda2dpcpp -> delete those porting on unchanged function -> fix the porting
maybe some script to only extract the changed function or selection help this situtation


docs: Remove the Gatsby plugin

See https://github.com/NoriSte/gatsby-plugin-perfume.js/issues/409 for more info about the deprecation choice and hence the removal from the docs included in this PR 😊

Thank you for your help :)


[SV]:Use select poll mode to implement a simple IO-Server

Use select poll mode to implement a simple IO-Server which can receive the data from endpoints and can send data to the endpoints also :-)

Can receive data from device, device-emulation or some other tools
Can show the data received in shell and save all of the data into a log file, think we should include a open source log system into the project
Maybe a loop back is also needed for us, this will be used for debug. the device will know whether it connect success or not :-)

I'm pleased to accpt the assign that the us gives to me.

Should add a connection pool now to manage all of the connections :-)

We need some classes for the connections. Client can send and receive messages through these connections directly :-)

OuterConnection class is needed now :-)

It's difficult to maintain, so close it.
We should create some much smaller story to develop it :-)


Missing data when import via Bulk API and Big Query batch mode

Environment

Deploy method: docker-compose.yml
Jitsu version:  1.38.4
OS: Ubuntu 20.04
Cloud Provider: time4vps

Description
Missing data when import using Big Query destination in Batch mode
Steps to reproduce
Steps to reproduce the behavior:
Call /api/v1/events/bulk to insert data
Data file:
demofile2.txt
Expected behavior
It should insert with full data from event

Stream mode has data_metrics_market_data_ohlcv_last_1_hour_close data
Actual behavior

Batch mode without `data_metrics_market_data_ohlcv_last_1_hour_close`

Hi @thanhlmm! I've noticed that original rows from the actual behavior screenshot in attached demofile2.txt don't have data under field ohlcv_last_1_hour. (there is null value under the key) data_metrics_market_data_ohlcv_last_1_hour_close column should be null as it is on actual behavior screenshot.
If you have any questions please let me know

Hey it seems like my mistake. I'll close the case


Update platform.txt

Added 8 new platforms in index file of search username python project

[ ] New feature (non-breaking change which adds functionality)

[ ] This change requires a documentation update

[ ] I have added the project meta data in the PR template.

[ ] I have created the requirements.txt file if needed.

Project Metadata
If there is no-file/nothing to fill the below fields with, then type: none
Example: If no requirements.txt needed/present, then type none in Requirments.
Category:

[ ] AI/ML
[ ] Social_Media

Title: Search-Username
Folder: (https://github.com/avinashkranjan/Amazing-Python-Scripts)/Search-Username/
Requirements: None
Script: Amazing-Python-Scripts)/Search-Username/search_user.py
Contributor: simar5244
Description: Added 8 new platforms in index file of search username python project

Also, the link containing the list of platforms inside readme.md is broken


Throw exception if run on headless machine without system X

Currently, if run on a headless linux machine, it results in a segmentation fault.

A silent crash creates a lot of confusion.

https://github.com/podgorskiy/ALAE/issues/25
https://github.com/podgorskiy/ALAE/issues/22
https://github.com/podgorskiy/ALAE/issues/13
https://github.com/podgorskiy/ALAE/issues/12

@podgorskiy
Hi, Thanks for the nice repository. I would like to know how to solve this exception. I was trying to work with bimpy on google Colab but it throwing the above issue.


[Tabs] Fix TabsScrollbarSize issue #38406

Fix TabsScrollbarSize issue #38406

[x] I have followed (at least) the PR section of the contributing guide.

@brentertz Can you fork the repo again and submit a new PR? I have no idea why Circle CI could not find config.yml


Feature/cvsb 19017

configure repo for development

I was thinking that if we wish to replicate this scaffold project we could leave the history until 1.0.0 which should be our starting point from releasing changes now since the configuration is done as this would help cherry picking (or not) changes if we didn't want to start with the same baseline or modifying this further? Thoughts?

I was thinking that if we wish to replicate this scaffold project we could leave the history until 1.0.0 which should be our starting point from releasing changes now since the configuration is done as this would help cherry picking (or not) changes if we didn't want to start with the same baseline or modifying this further? Thoughts?


Simplification of integer creation

Just for correctness only as the production code is not affected by this change.

Looks ok.


Change approach of device containers creation/destruction

Removed endpoints to create/remove container from udev
Removed udev rules that triggered those endpoints
Created a fsnotify watcher to listen for file changes in /dev folder, so whenever a device symlink is created/removed we in turn create/remove container for the respective device
A few small updates to get this working

Removed endpoints to create/remove container from udev
Removed udev rules that triggered those endpoints
Created a fsnotify watcher to listen for file changes in /dev folder, so whenever a device symlink is created/removed we in turn create/remove container for the respective device
A few small updates to get this working


Structs with flattened fields are serialized and deserialized as maps

This causes problems for formats that use different syntax for structs and maps. Is there some way to fix it?

Rust structs with flattened fields are not "structs" by Serde's definition of "struct". A struct in Serde means that the fields are known at compile time. This is enforced for example by Deserializer::deserialize_struct which requires a &'static [&'static str] of the field names. But in a Rust struct with flattened fields, the Serialize impl of the flattened type can potentially insert any fields it wants at runtime.
RON could work around this by allowing deserializing Serde maps from RON structs, or I would be willing to consider a PR to support a compile-time declared set of what fields the flattened type is allowed to insert:
#[derive(Deserialize, Serialize)]
struct Main {
    #[serde(flatten(first, second))]
    required: Required,
    #[serde(flatten(third))]
    optional: Optional,
    some_other_field: u32,
}

Right, this would be for flattening types that have a compile-time knowable set of fields.

Just ran into the same issue with serdebug.
@dtolnay I see there is already deserialize_identifier. Could we just add a counterpart for it in Serializer and for flattened structs continue to use serialize_map but use that new serialize_identifier for keys? I imagine that would be easier to use than enumerating flattened fields by hand.

Ideally, I would also add Token::Ident to distinguish these from regular strings, but that would be a breaking change, so I guess not an option.

Ugh, today ran into this again with yet another target that has different representations for structs and maps, resulting in incompatibilities :(
@dtolnay If you feel strongly about not fixing this issue, could we maybe soft-deprecate serde(flatten), and move current implementation to a separate proc-macro crate as you suggested for few other issues?
That would hopefully bring more attention to differences for users, and would make this less of a footgun for Serializer/Deserializer authors like me.

@dtolnay Also you said:

I would still make the choice of leaving this as a known issue

but this issue is not "known" as in "mentioned in the docs for serde(flatten)", making it really hard to debug if you don't already know what you're looking for...

I think a important thing here is that we lost struct name when using serialize_map instead of serialize_struct. is it possible to add a struct-like map to serializer
trait Serializer {
  // ...
  fn serialize_struct_map(
    self,
    name: &'static str,
    len: usize
  ) -> Result<Self::SerializeMap, Self::Error> {
    self.serialize_map(Some(len))
  }
}
trait Deserializer {
  // ...
  fn deserialize_struct_map<V>(
    self,
    name: &'static str,
    fields: &'static [&'static str],
    visitor: V
  ) -> Result<V::Value, Self::Error> {
    self.deserialize_map()
  }
}

which takes parameters like a struct, while returns a value like map.
Ideally the serialize_struct_map should return Self::SerializeStructMap, but for backward compatibility, it just returns SerializeMap.
Now https://github.com/ron-rs/ron/issues/115 is blocking on this. With this information, they could tell the different between "map because of flatten" and "map because of container", and land fix to handle with flatten


Add debug.stepInto/Out/Over to terminal.integrated.commandsToSkipShell

VSCode Version: 1.18.0
OS Version: Windows 10 FCU

Steps to Reproduce:

Install PowerShell Extention
Execute command: PowerShell: Open Examples Folder
Select DebugTest.ps1
Set a breakpoint on line 12
In Debug view, select  "PowerShell Launch Current File"
Press F5 to start debugging
When the debugger breaks, press F10 to step.

No joy, eh?  Our users are wondering why the heck F10 doesn't step the debugger. Well that's because the PowerShell Integrated Console got focus.  It's the REPL folks use even during debug sessions.
Now there is no need for PSIC to process F10/Shift+F11/F11.  So can you add these commands to the built-in commandsToSkipShell?

Reproduces without extensions:  N/A but probably - point is this impacts the PowerShell extension and any other debug extension with such a REPL.

vscodebot - go home, you're drunk.  I think this is probably best hanlded by @Tyriar  :-)

I am having an issue that F11 (enter/exit fullscreen) does not work when terminal has focus (VSCode 1.19.0, Linux).

The PR that closed this issue adds step-into (F11) to the list of commands the terminal should ignore.  We'll just have to wait for 1.20 or try it in an insiders build.


epic: enable /fine_tuning/jobs

Problem
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Success Criteria
A clear and concise description of what you want to happen.
Sub Issues
Additional context
Add any other context or screenshots about the epic request here.

My question: is this a better fit for Jan?

Nitro scope should be inference only for now


Remove default fetch of 200 documents.

Fixes: #569
fixes: #521
By default instant-meilisearch fetched 200 documents at every fetch to ensure we can provide a finite pagination (with the 1, 2, 3, at the bottom of the screen).
This impacted the speed of the search considerably.
This PR introduces a new parameter: finitePagination.
Now by default:
Each search requests a limit of hitsPerPage (default: 6) and the page number : (hitsPerPage + 1) * (pageNumber + 1)
since pageNumber starts at 0 we need to add one (as 0 * x is always 0...).  We also need to add one to hitsPerPage to ensure that there is still something to load after, as without we don't want load more to be clickable.
The pagination will stop at paginationTotalHits (default 200).
When settings finitePagination to true, one search request will ask for a limit of paginationTotalHits as limit. Thus in the default case, one search request will return 200 documents.
This number can be changed with paginationTotalHits
The following will make one request of 7 documents as hitsPerPage is at 6 by default.
instantMeiliSearch(host, key, {
   finitePagination: false // default false
   paginationTotalHits: 60 // default 200
})

You can change hitsPerPage.
The following will make one request of 200 documents
instantMeiliSearch(host, key, {
   finitePagination: true // default false
   paginationTotalHits: 60 // default 200
})

Hello! Awesome for this PR 🚀
⚠ Disclaimer: my feedback can seem huge, I'm just picky to make you improve in writing PR descriptions. This one is a good opportunity since it really interests Quentin and Morgane for the mini-dashboard, so "external" people 🤘
But your PR is really cool, the code seems clear, and you added tests! 🔥 Thank you for this! Can't wait we upgrade the mini-dashboard with this improvement!

This impacted the speed of the search considerably.

Maybe remind the more you ask the search engine to retrieve documents, the more you impact the search time. Indeed meilisearch is designed to retrieve the most relevant documents, but not all the relevant documents. The more you ask, the more the search engine has to process. There is no difference regarding the search time between retrieving 6 or 7 documents, but between 6 and 200 documents to every request, the impact can be huge.
(I don't you already know this @bidoubiwa, just to help describe the context to someone going to your PR 😇)
Also, I know the context is kind of described in the issues, but

It's always appreciated to remind the context/motivation in the PR, what you started to do, and thank you for this!
it's not really convenient to check the issues related when there are multiple ones

Now by default:
Each search requests a limit of hitsPerPage (default: 6) and the page number : (hitsPerPage + 1) * (pageNumber + 1)
since pageNumber starts at 0 we need to add one (as 0 * x is always 0...). We also need to add one to hitsPerPage to ensure that there is still something to load after, as without we don't want load more to be clickable.

You mean, when finitePagination is set to false right?
I don't understand why there is pageNumber in the calculation of pageNumber, see: "Each search requests a limit of hitsPerPage (default: 6) and the page number : (hitsPerPage + 1) * (pageNumber + 1)"
Do you mean by default only the next page was accessible? I think the context without the calculation is missing this PR description. Since the README is not updated, it's really missing.
Curisotity: why 6 for hitsPerPage?

The following will make one request of 7 documents as hitsPerPage is at 6 by default.
instantMeiliSearch(host, key, {
   finitePagination: false // default false
   paginationTotalHits: 60 // default 200
})

You can change hitsPerPage.
The following will make one request of 200 documents
instantMeiliSearch(host, key, {
   finitePagination: true // default false
   paginationTotalHits: 60 // default 200
})

The examples are clear 👌
I would say "The following will make instant-meilisearch request Meilisearch to retrieve 7 documents and not 6"
Finally, is it expected the README is not updated here? If it is, maybe you should add context in the PR description, like "the README will be updated in this PR", especially because this PR will be merged into main

Thank you again for the PR! ♥

Also, in the PR name, I would highlight the introduction of the finitePagination parameter :)

bors merge


Editor language changing issue

I use Korean for my mother tongue, and once I open another window and get back to the vscode, first input that I type (though IME setting says it's korean) is printed in English.
For Example, when I input "강조합니다", it looks like "rㅏㅇ조합니다"
(This issue is highlighted in magenta color in image)
Also, When I type in Korean in Markdown Editor, I get double cursors, front and behind the text I wrote.
Like this: when I input ㄱ, it shows like |ㄱ|.
(This issue is highlighted in skyblue color in image.)

Sorry for my short English. I hope this issue gets solved as soon as possible.

@leeminki02 could you try latest VS Code and see if you can reproduce, I'm wondering if it's a chromium issue which is already fixed upstream.

@rebornix Yeah, I think it's fixed now! Thanks for the notice! :)


When converting a Lead, if the Account has an apostrophe in its name the Lead can't be matched to existing account

POU thread: https://powerofus.force.com/0D580000029Vui7
The reason why this took a while to reproduce is that it only happens in an instance that has the managed packages. It's not reproducible in a Dev org with unmanaged code.
This issue happens regardless of whether using Visualforce override or standard Lead Convert button. In an org without NPSP 3, the Lead can be matched to the existing account.

User (and myself) can only replicate on the Standard Lead Convert page from Salesforce.  The NPSP Lead Convert Override page does not have this problem.


Index controller needs explicit /index in its routes

I can't create an action "blabla" inside my index controller and access it simply under /blabla.
get :blabla do # accessible only at /index/blabla
    'bla'
end

get :bleble, :map => '/bleble' do # this is accessible under /bleble
    'bla'
end

I've created my app since Padrino 0.10.0 and this started happening when I upgraded from 0.10.2 to 0.10.5. Originally, I had a :login action with no map that pointed to /login.

Mmm @joshbuddy ?

@igorsantos07 Just so I understand, you have a controller literally called :index and it used to be stripped out of your routes but now it's not?
I think this is a case of where us fixing a bug has led to unintended consequences. The index controller was never intended to be stripped out of routes. If you want routes in the "root" namespace, I would suggest either a) controller with no specified name b) move the routes to app.rb itself.
Feel free to re-open if I misunderstood the issue.

hahahaha yes, that's EXACTLY the problem.
I'll try with the unamed controller then, thank you (:

Same seems to be happen if you forget the slash at the beginning.
get :bleble, :map => 'bleble' do
    'bla'
end

I think this is expected. If you map the action without the beggining
slash, than you are telling Padrino to map the action inside the controller
map.
The code you pasted, inside a controller called, like, "user", would end up
with a route like "/usr/bleble".
I've opened this issue because the Index controller was being skipped in
previous versions, but it's not being skipped anymore.


PROBLEM: AIPs page loads slowly

The AIPs view page can take several minutes to load in an instance, particularly those with a lot of AIPs in the database. No further searching or browsing of AIPs can take place at this time. If you click back to the AIPs page after viewing an AIP, it will take several minutes to load again.
A potential suggested fix was to add the search box to the main page where Fetch jobs are initiated to prevent the need to load the entire AIP view. There may also be other database optimisation options.

Thanks Sarah... we'll take a look at potential solutions!

I think I might know the underlying issue and am going to start on a fix.

A pull request has been created to fix this: https://github.com/artefactual-labs/AIPscan/pull/215
I'll update this issue when it's reviewed and merged.

I've merged a fix for this.


HA Climate Group - Missing away preset mode for Heizungsgruppe

DONT'T DELETE THIS.
This is for issues only
If you have a feature request, add it as a new discussion, or add your point of view to an existing discussion.
Please answer all questions as good as possible.
Language: as you like -> english/DEUTSCH
Make sure you have read the documentation at https://github.com/danielperna84/custom_homematic#custom_homematic.
Make sure you have read the release notes at https://github.com/danielperna84/custom_homematic/releases.
Make sure you are running the latest version of the custom_component (and Home Assistant) before reporting an issue.
ENSURE that your Backend (CCU/Homegear/...) is working as expected and connected devices are controllable.
custom_component version (if applicable):
1.25.1
known last working custom_component version (if applicable):
na
Home Assistant:

Version: 2022.12.8
Installation method (HAOS, supervised, core, venv, Docker): HA supervised within Docker Synology

CCU:

Model (CCU2/3, Raspi3/4, NUC, ProxmoxVE, HA-Addon ...): CCU3
OS (Original, RaspberryMatic, debmatic ...): RasperryMatic
Version: <IP_ADDRESS>21218

Problem-relevant configuration:
Hardware:  HmIP-eTRV-E (Heizkörperthermostat), HMIP-SWDO-2 (Fensterkontakt), HmIP-WTH-1 (Wandthermostat)
Do you use tls? no
Do you use callback?no
Do you use username and password? yes
Which interfaces do you use (hmip/bidcos/wired)? hmip
Describe the bug
The "Urlaub" setting of my Thermostat-Group is not transmitted as property to Home Assistant as preset_mode: away.
To Reproduce
Steps to reproduce the behavior:
0. Setup a climate group (Heizungsgruppe) with your radiator thermostat (Heizkörperthermostat)/window contact (Fensterkontakt) /Wall thermostat (Wandthermostat)

Go in Home Assistant to 'Developer Tools", Tab "States"
In Entity, enter your "Heizungsgruppe" (climate group) name, e.g. climate.bz_heizungsgruppe
See error, e.g. my result pasted below of "climate.bz_heizungsgruppe".

Observed
'boost' and 'none' (and Profile 1/2/3) are properly declared as preset_mode
Expected
'away' is declared in preset_mode (and is the same as Urlaub/Vacation in CCU3 Web-UI)
climate.bz_heizungsgruppe:
hvac_modes:
  - auto
  - heat
  - 'off'
min_temp: 5
max_temp: 30.5
target_temp_step: 0.5
preset_modes:
  - boost
  - none
  - Profile 1
  - Profile 2
  - Profile 3
current_temperature: 22.3
temperature: 16
current_humidity: 47
hvac_action: idle
preset_mode: Profile 2
interface_id: RaspberryMatic-VirtualDevices
address: INT0000001:1
model: HmIP-HEATING
entity_type: custom
value_state: valid
friendly_name: BZ Heizungsgruppe
supported_features: 17

Screenshots (if applicable)

CCU firewall in CCU

device config in CCU

integrations view in HA

device view in HA

entity view in HA

Relevant HA log entries (press full log) VERY IMPORTANT
2022-12-25 02:12:18.471 WARNING (SyncWorker_0) [homeassistant.loader] We found a custom integration hacs which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant

2022-12-25 02:12:18.472 WARNING (SyncWorker_0) [homeassistant.loader] We found a custom integration homematicip_local which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant
2022-12-25 02:12:27.335 ERROR (MainThread) [homeassistant.components.text] Error adding entities for domain text with platform homematicip_local
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 428, in async_add_entities
    await asyncio.gather(*tasks)
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 605, in _async_add_entity
    capabilities=entity.capability_attributes,
  File "/usr/src/homeassistant/homeassistant/components/text/__init__.py", line 129, in capability_attributes
    ATTR_PATTERN: self.pattern,
  File "/usr/src/homeassistant/homeassistant/components/text/__init__.py", line 211, in pattern
    return self.entity_description.pattern
AttributeError: 'EntityDescription' object has no attribute 'pattern'

Diagnostic Information (available on every device)
{
  "home_assistant": {
    "installation_type": "Home Assistant Supervised",
    "version": "2022.12.8",
    "dev": false,
    "hassio": true,
    "virtualenv": false,
    "python_version": "3.10.7",
    "docker": true,
    "arch": "x86_64",
    "timezone": "Europe/Berlin",
    "os_name": "Linux",
    "os_version": "4.4.180+",
    "supervisor": "2022.12.1",
    "host_os": null,
    "docker_version": "20.10.3",
    "chassis": null,
    "run_as_root": true
  },
  "custom_components": {
    "hacs": {
      "version": "1.29.0",
      "requirements": [
        "aiogithubapi>=22.10.1"
      ]
    },
    "homematicip_local": {
      "version": "1.25.1",
      "requirements": [
        "hahomematic==2022.12.8"
      ]
    }
  },
  "integration_manifest": {
    "domain": "homematicip_local",
    "name": "Homematic(IP) Local",
    "config_flow": true,
    "documentation": "https://github.com/danielperna84/custom_homematic",
    "issue_tracker": "https://github.com/danielperna84/hahomematic/issues",
    "requirements": [
      "hahomematic==2022.12.8"
    ],
    "ssdp": [
      {
        "manufacturer": "EQ3",
        "manufacturerURL": "http://www.homematic.com"
      }
    ],
    "zeroconf": [],
    "homekit": {},
    "dependencies": [],
    "codeowners": [
      "@danielperna84",
      "@SukramJ"
    ],
    "iot_class": "local_push",
    "loggers": [
      "hahomematic"
    ],
    "version": "1.25.1",
    "integration_type": "hub",
    "is_built_in": false
  },
  "data": {
    "config": {
      "entry_id": "462c2c024540400223af6c5c77873e70",
      "version": 1,
      "domain": "homematicip_local",
      "title": "RaspberryMatic",
      "data": {
        "instance_name": "RaspberryMatic",
        "host": "<IP_ADDRESS>",
        "username": "**REDACTED**",
        "password": "**REDACTED**",
        "tls": false,
        "verify_tls": false,
        "callback_host": null,
        "callback_port": null,
        "json_port": null,
        "interface": {
          "HmIP-RF": {
            "port": 2010
          },
          "VirtualDevices": {
            "port": 9292,
            "path": "/groups"
          }
        }
      },
      "options": {},
      "pref_disable_new_entities": false,
      "pref_disable_polling": false,
      "source": "ssdp",
      "unique_id": "9D89BA0B6A",
      "disabled_by": null
    },
    "platform_stats": {
      "binary_sensor": 7,
      "climate": 5,
      "sensor": 8
    },
    "devices": [
      "HmIP-CCU3",
      "HmIP-HEATING",
      "HmIP-SWDO-2",
      "HmIP-WTH-1",
      "HmIP-eTRV-E"
    ]
  }
}

Expected behavior
'away' is declared in preset_mode (and is the same as Urlaub/Vacation in CCU3 Web-UI)
Additional context
See definition of Holiday/Urlaub for Homematic IP Thermostat https://www.eq-3.com/service/faq/what-are-the-auto-manu-holiday-function-operating-modes.html

away needs an end date. It will be displayed as hvac_mode but cannot be set with the default UI.
To enable away mode you have to use one of the services:
homematicip_local.enable_away_mode_by_calendar
homematicip_local.enable_away_mode_by_duration

Not an issue

Thanks for clarification @SukramJ . Didn't see this.