File paths with "+" character not identified as links in terminal

When paths to files containing the + character are printed to the terminal they do not become clickable links.
Steps to reproduce:
mkdir foo+bar foo_bar && echo "sample text" > foo+bar/foo.txt && echo "sample text" > foo_bar/foo.txt

# This prints a path that is not clickable
echo "foo+bar/foo.txt"
# This prints a path that is clickable
echo "foo_bar/foo.txt"

The "+" character in folder names is commonly used in matlab development since starting a folder name with + makes it a package.

This actually works with the new system we're working on. Try setting this in Insiders:
"terminal.integrated.experimentalLinkProvider": true

I just tried it (Windows 10, VSCode Insiders 1.45.0) and it partially works.
If the path is foo+bar/foo.txt and the pointer is hovered anywhere after the last + in the path only the part after the last + is linked, i.e. bar/foo.txt
However if the pointer is hovered over the path anywhere before the last + the full path is linked. This also applies if the pointer is hovered over the first part of the path, then moved to the last part of the path.

Are you sure you enabled the setting? It works fine for me:

Yes, the setting is enabled

And you're up to date? I'm on yesterday's build
Version: 1.45.0-insider (user setup)
Commit: 6e9a34c0de4181a99f11e106733ee3e8c9db79f1
Date: 2020-04-16T05:45:56.832Z
Electron: 7.2.1
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 10.0.18362

I'm on today's build:
Version: 1.45.0-insider (user setup)
Commit: a3083e58e2a9e77f60e531acaa32e18527ae8439
Date: 2020-04-17T08:09:47.238Z (7 hrs ago)
Electron: 7.2.1
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 10.0.18362

Weird... I updated and still don't see it. What happens when you ctrl+click the link? Does it open a file or a quick pick?

If I click the full path it opens the file as expected. If I click the partial path it opens a new instance of VSCode but doesn't open any file or folder in it.

Had this changed at all? A lot of work has gone into links in the past few days.

Tested in today's build (
If I hover over the file name, I see this:

If I hover before the last +, I get this

This might have been caused by https://github.com/microsoft/vscode/issues/95966


[Bug] [Module Name] Bug title dbgpt_server.py: error: the following arguments are required: --model_path

Search before asking

[X] I had searched in the issues and found no similar issues.

Operating system information
Linux
Python version information
3.10
DB-GPT version
main
Related scenes

[X] Chat Data
[X] Chat Excel
[X] Chat DB
[X] Chat Knowledge
[X] Model Management
[X] Dashboard
[X] Plugins

Installation Information

[X] Installation From Source

[ ] Docker Installation

[ ] Docker Compose Installation

[ ] Cluster Installation

[ ] AutoDL Image

[ ] Other

Device information
GPU:24G
Models information
LLM_MODEL=Qwen2.5-7B-Instruct
EMBEDDING_MODEL=text2vec
What happened
when start the services(python dbgpt/app/dbgpt_server.py)，it is error as follows:
Model Unified Deployment Mode!
2024-12-28 00:01:58 autodl-container-07ff4facf2-71e1d84d dbgpt.model.cluster.worker.manager[7854] INFO Register WorkerManager dbgpt_worker_manager_factory
2024-12-28 00:01:58 autodl-container-07ff4facf2-71e1d84d dbgpt.component[7854] INFO Register component with name dbgpt_worker_manager_factory and instance: <dbgpt.model.cluster.worker.manager._DefaultWorkerManagerFactory object at 0x7f123296b550>
usage: dbgpt_server.py [-h] [--model_name MODEL_NAME] --model_path MODEL_PATH [--host HOST] [--port PORT] [--daemon] [--log_level {FATAL,ERROR,WARNING,WARNING,INFO,DEBUG,NOTSET}] [--log_file LOG_FILE] [--tracer_file TRACER_FILE]
[--tracer_to_open_telemetry] [--otel_exporter_otlp_traces_endpoint OTEL_EXPORTER_OTLP_TRACES_ENDPOINT] [--otel_exporter_otlp_traces_insecure] [--otel_exporter_otlp_traces_certificate OTEL_EXPORTER_OTLP_TRACES_CERTIFICATE]
[--otel_exporter_otlp_traces_headers OTEL_EXPORTER_OTLP_TRACES_HEADERS] [--otel_exporter_otlp_traces_timeout OTEL_EXPORTER_OTLP_TRACES_TIMEOUT] [--otel_exporter_otlp_traces_compression OTEL_EXPORTER_OTLP_TRACES_COMPRESSION]
[--worker_type {llm,text2vec}] [--model_alias MODEL_ALIAS] [--worker_class WORKER_CLASS] [--model_type MODEL_TYPE] [--limit_model_concurrency LIMIT_MODEL_CONCURRENCY] [--standalone] [--register]
[--worker_register_host WORKER_REGISTER_HOST] [--controller_addr CONTROLLER_ADDR] [--send_heartbeat] [--heartbeat_interval HEARTBEAT_INTERVAL] [--tracer_storage_cls TRACER_STORAGE_CLS]
dbgpt_server.py: error: the following arguments are required: --model_path
What you expected to happen
the services can started
How to reproduce
start the services:python dbgpt/app/dbgpt_server.py
Additional context
No response
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

you start dbgpt server with auto-dl images?


cfg is different in sds and vsd

stable_diffusion_guidance.py:

stable_diffusion_vsd_guidance.py:

The two formulations are basically the same when the guidance scale is large. In VSD, we follow the same formulation as the diffusers library. We'll unify this later.


Handle classification events in OLS

Signed-off-by: Liviu Popa<EMAIL_ADDRESS>

@lcpopa Is the intent to correct this or should we close?

@planetf1  just got back from vacation. will see what checks failed and act accordingly

recheck


DP2SceneGraphWizardInstaller

hello
Am heaving issues with DP2SceneGraphWizardInstaller, i download this tool but i can't  install it, this what am facing

please can someone help me?

Are you installing the Alpha 3 version?
Did you download the EXE file?
https://github.com/rrirower/DP2SceneGraphWizard/releases/download/<IP_ADDRESS>-alpha-3/DP2SceneGraphWizardInstaller.exe
Your system must meet the following minimum requirements to use the DP2SceneGraph Wizard tool.
64-bit Windows 10 or higher.
.Net 7 framework.
30 MB Hard disk space.

Please make sure your computer meets the requirements posted for the software. You may need to install .Net 7 manually.

hello
i try to open subscription and Ads setup on DP2SceneGraphWizardInstaller but it don't open, what happened

From the release notes for the wizard:
"Add future support for RAF and Subscriptions (not implemented - future release)."
Those features are not implemented yet. They will be in a future release.

ok thanks

https://github.com/rrirower/DP2SceneGraphWizard/assets/141512200/16c7d8b4-0f2e-4da7-af38-07c9b0379af0
can this be splash screen on Roku sdk package??

This version of the code that is produced by the wizard does not play MP4 files as a splash screen. I'm not sure if you can since the manifest requires an image file. Perhaps with some custom coding of your own you can accomplish that .


Cleanup for ruby 2.7 warnings.

Signed-off-by: Ryan Davis<EMAIL_ADDRESS>

Coverage decreased (-0.3%) to 98.007% when pulling cf5d076d0baada73fc83be5b86114f510b4009d0 on zenspider:zenspider/ruby-2.7 into 8d18fbfc4b9b3ed569aeda74c4b76eaaa6f88e8e on piotrmurach:master.

Thanks ❤


[Feature Request] Generate schemas dynamically from request

I have an API that adheres to the jsonapi.org spec, and I'm attempting to describe it with Zodios. One semantic detail that's proven hard to capture is sparse fieldsets, where the set of fields returned is controlled by a query parameter. Since response schemas are static in Zodios when defining an API, I end up having to mark all fields as .optional() if I want to exclude them from the fieldset in any request context.
I haven't thought this all the way through, but I wonder if there's a way to enable schema refinement based on the input parameters to a query. I'm very new to Typescript and Zod, so I thought I'd create this issue fairly open-ended and see if it sparks any ideas (or reveals any workarounds).

Stiiiiill interested in feedback.

🏓

Last try. 🏓

Maybe using a zodios plugin
: here
In the response object, get requested fields, and return a object with filtered keys


Suggestion: Allow config to determine which biomes the mobs to spawn in

I looked through the server config in the world folder, and all it does is turn on and off existing features. I think it could be cool to allow adding biomes that the mobs can spawn in. This works nicely because then if people like me have biome mods, they can add biomes for the creatures to spawn in. This also could benefit modpack makers. I personally use oh the biomes you'll go and have a couple biomes I want to add to spawn listing.

This will be looked into in a future release
Until then you'll just have to wait :/

This is actually currently in the works for the next release, so you shouldn't have to wait too long!

I have to add (as I'm the one developing it), that there will be a system config and JSON files that will allow users to customize which biomes IE's mobs will spawn in, as well as spawnrates, minimum and maximum spawn counts for each biome.

Ok, sounds good. Also, is there a particular reason for the config to be in the world config vs the config folder? I imagine that placing it in the config folder could make this process potentially easier.

It is also being moved to the general config folder in the next release! Our config is getting a huge rework for this release, including a GUI

Oh awesome, thanks for letting me know :)


[README] Need to add Community Forum

Description

README.md is the first file one should read when starting a new project. Our community launched Discussion forum for help seekers; needs to publish it officially on README.md for convenience.
Screenshots

Current behaviour:

Desired behaviour:

Note:
Need to add only last line for Community Forum
Environment:

OS: any [e.g. Ubuntu]
Browser: any [e.g. Chrome, Safari]
Version: any [e.g. 22]
Device: any [e.g. laptop, iPhone 8]

Reference:

Take reference to this pull request

Link to the community Forum https://discuss.layer5.io/.

@sayantan1413 : Thanks for label, can you give a small shot. 😉

@amino19 I can give it a shot but would you mind if I leave this issue for a new contributor then they would feel good solving this issue in their first shot. :)
(Though I am also a newbie but I have contributed some in this organization)

I would like take this task.


Concurrency option ignored in local dev

The defer client ignores the concurrency setting when executing jobs locally, and instead immediately invokes the handler function unconditionally, regardless of how many other invocations of the handler are already running.
https://github.com/defer-run/defer.client/blob/301f78eae4eff2cc7ac72313616ad6265923f124/src/index.ts#L104-L109
Proper concurrency control is critical to my use case, and it's impossible to develop and test my code if this option does not work.
It would not be hard for the client to maintain an in-memory queue in order to ensure that only concurrency invocations of a job run at a time in dev.
This missing functionality is also not documented anywhere in the local dev docs or concurrency option docs.

Fixed in the v2.0.0!


Update API documentation for put

Not sure if anyone else had the same experience, but I was thrown off a little bit by
https://redux-saga.js.org/docs/api/#putaction

Creates an Effect description that instructs the middleware to dispatch an action to the Store. This effect is non-blocking and any errors that are thrown downstream (e.g. in a reducer) will not bubble back into the saga.

The term non-blocking somehow led me to expect put to behave like some kind of "asynchronous dispatch".  This intuition is of course, incorrect, as we can expect the store to immediately receive the action from put() unless we have some kind of asynchronous middleware that delays/prevents the action from reaching the store.
I do wonder if this is a valid concern? If it seems like a reasonable mistake to make, I'd be glad to submit a PR to suggest tweaks to the wording.

I agree that non-blocking here might be misleading - it would be better to just describe it as synchronous. In addition to that, the last part of what you have pasted in is incorrect - errors thrown in reducers are no longer swallowed - https://codesandbox.io/s/redux-saga-example-goc5o . That has changed in v1.
Would you like to prepare a documentation PR improving both things?

Sure, I'd be happy to

@superburrito

unless we have some kind of asynchronous middleware that delays/prevents the action from reaching the store
can you give an example?


Documentation and overall changes for v0.4

This is presumably the final PR into develop before develop has a PR to main, which will then be the v0.4.0 release.

@wildsm: your changes look good. As you recommended, I am self-reviewing changes I just made to the documentation branch. Further review of my changes to README is indicated before merging into main.


shipper.py's GzipLogRequest produces non-POSIX-compliant lines

POSIX defines a Line as:

3.206 Line
A sequence of zero or more non-  characters plus a terminating  character.

However, in shipper.py, a \n is added only between "lines" and not after every line. For example:
line1\nline2\nline3

This is not POSIX compliant and increases the complexity of the code (requires a counter and an additional if):
https://github.com/logzio/logzio_aws_serverless/blob/c1cf511ed9e291ec3fa9b76ed9352a0a8c51dbca/python3/shipper/shipper.py#L48-L50
Assuming logz processes POSIX-compliant lines, the body of write(self, log) should be changed to something like
self._writer.write(bytes(log+"\n", 'utf-8'))
self._logs_counter += 1

The counter needs to be kept for __len__(self) only.

Closing this issue, as the cloudwatch & kinesis solutions are deprecated. If it's still needed for the Amplify solution, please open a new issue.


update the sha256 checksum of android-studio <IP_ADDRESS>,173.4670197

the previous sha256 checksum was incorrect.
After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} reports no offenses.
[x] The commit message includes the cask’s name and version.

Duplicate of https://github.com/caskroom/homebrew-cask/pull/45284
Thanks for the PR nonetheless!


[Camel 3.12] UnmarshalTypeName should be renamed to unmarshalType

Do a
git grep unmarshalTypeName

And then rename all those to
unmarshalType

This was taken care of in https://github.com/apache/camel-quarkus/commit/c8793b83391e43760985ef62a1ae4854f7a363f6.


fix(create-note): use private constant for the default error message

I guess the default message should be a constant.
I found that PHP supports Constant Visibility.
@picklecillo Can you review it?

Cool, thanks for testing!


Changes

Overview

This PR fixes or fixes part of #[fill_in_number_here].
This PR does the following: [Explain here what your PR does and why]

Essential Checklist

[ ] The PR title starts with "Fix #bugnum: ", followed by a short, clear summary of the changes. (If this PR fixes part of an issue, prefix the title with "Fix part of #bugnum: ...".)
[ ] The linter/Karma presubmit checks have passed locally on your machine.
[ ] "Allow edits from maintainers" is checked. (See here for instructions on how to enable it.)

This lets reviewers restart your CircleCI tests for you.

[ ] The PR is made from a branch that's not called "develop".

PR Pointers

Oppiabot will notify you when you don't add a PR_CHANGELOG label. If you are unable to do so, please @-mention a code owner (who will be in the Reviewers list), or ask on Gitter.
For what code owners will expect, see the Code Owner's wiki page.
Make sure your PR follows conventions in the style guide, otherwise this will lead to review delays
Never force push. If you do, your PR will be closed.

Hi @nishantwrp, PTAL. Thanks!

Bot should assign nishant after waiting for 3 minutes

Hi @nishantwrp, PTAL. Thanks!

Hi @nishantwrp, PTAL. Thanks!

Hi @nishantwrp, PTAL. Thanks!

Hi @nishantwrp, PTAL. Thanks!

Hi @nishantwrp, I've addressed all the review comments, please take a look. Thanks.


Adjust css spacing in footer

Adjust css spacing in footer so that centering and alignment works on desktop
http://blockstack.org/

Issue non-urgent pushed to Sep. 3
https://app.asana.com/0/36491826052304/41949058386809

Closing this issue as it applied to the old site.


Update stable version in repo from 3.0.1 to 3.0.3

Think about update stable version to 3.0.3
Version: stable=3.0.1 (151 days old) => latest=3.0.3 (21 days old)
Installs: stable=4862 (56.01%), latest=1879 (21.65%), total=8680
Click to edit

done


spm: fix create binding for update with hint

What problem does this PR solve?
close https://github.com/pingcap/tidb/issues/21457
What is changed and how it works?
add one more look forward.
Check List
Tests

Unit test

Code changes

N/A

Side effects

Increased code complexity

Related changes

N/A

/cc @eurekaka

/cc @kennytm

/run-cherry-pick

/run-cherry-picker


Realtime API?

Hi,
I love your Node-RED OpenAI implementation! The speech generation feature works great. I'd like to request an update to the OpenAI-node dependency to v4.69.0, which enables the Realtime API. This would unlock new possibilities for my projects.
Could you consider upgrading the dependency to take advantage of this new feature?

You bet! Great to hear that you're finding the node useful. Look for the dependency version bump later today.


Update otelcontribcol to latest

Updating otelcontribcol to latest to include fix of https://github.com/open-telemetry/opentelemetry-collector-contrib/pull/35006 and other essential security updates in components like Prometheus exporter.
The upgrade includes a breaking change of defaulting to local host instead of <IP_ADDRESS>, the feature is now stable and feature gate has been removed, Config Sync should explicitly set endpoint to host <IP_ADDRESS> to expose port for external pod access. This applies to the metric receiver opencensus, and the exporter prometheus.

Why id this safe to re-apply now?

Why id this safe to re-apply now?

Are you referring to the upstream fix or our release schedule?

I mean, if we reverted it before, why is it now safe to reapply? What changed?

I mean, if we reverted it before, why is it now safe to reapply? What changed?

To ensure correct behavior, we've included a no_op_label in the metricstransform processor's label_set filter (explained in the description). This is a temporary solution until the upstream fix is available, but we need to update to the latest otelcontribcol regardless.

I mean, if we reverted it before, why is it now safe to reapply? What changed?

To ensure correct behavior, we've included a no_op_label in the metricstransform processor's label_set filter (explained in the description). This is a temporary solution until the upstream fix is available, but we need to update to the latest otelcontribcol regardless.

This isn't really a revert then, since new changes were added to the original commit. Suggest just creating a new commit instead of reapplying.

we need to update to the latest otelcontribcol regardless.

why?

Re @sdowell While not currently anticipated, there's a possibility that future vulnerability fixes need dependency upgrades that are incompatible with current version we use (0.103). In such cases, the version upgrade process will be similar.  Otelcontribcol versions beyond 0.104.0 will introduce breaking changes.
I could start a new PR.

Recreated the commit and message.

/blocked  https://github.com/open-telemetry/opentelemetry-collector-contrib/pull/34844
Generated files (go.mod etc.) were removed, which would break AutoVC integration. No affect to the continuous build of the internal mirror so far.
A CI can be added to restore the generated files to a new branch, along with base image update for CVE scan purpose.

@code-review-assist review


[Halide] update to 10.0.0

Library name: Halide
New version number: 10.0.0
I am happy to report that the Halide team has finally released a new major version, 10.0.0! It's a big update from the last stable release and comes with an all new CMake package config with important helpers.
Please let me know what I can do to help you update. We now use GNUInstallDirs, so the existing fix-install-path.patch should no longer be necessary; just set CMAKE_INSTALL_DATADIR=share/Halide. Regarding patches generally, we would much prefer to upstream any changes that make packaging easier, rather than have you maintain separate patches.
References
All of our CMake packaging code is in: https://github.com/halide/Halide/blob/release/10.x/packaging/CMakeLists.txt
The CI scripts we use to package on Unix / Windows are here:
https://github.com/halide/Halide/blob/release/10.x/tools/package-unix.sh
https://github.com/halide/Halide/blob/release/10.x/tools/package-windows.bat
See also #11761

Just checking in on this again... I've never written a vcpkg port file, but I'm very happy to pitch in any way I can here.


Add images to activities

Problem Description:
Activities have images but are not being rendered.
Expected Behavior/Solution:
Render the images for activities.
Requirements:

[x] Add the getImages function and component to activities in the About section

Needed for promotion:
If an item on the list is not needed, it should be crossed off but not removed.

[x] Wireframe/Mockup - Kam
[x] ~Committee discussions - Sarah~
[x] ~Feasibility/Team discussion - Sarah~
[x] ~Backend requirements - TBD~
[x] Frontend requirements - Kam
[ ] Are new regression tests required for QA - Amy
[x] ~Questions~

UAT/LUX Examples:

https://lux-front-tst.collections.yale.edu/view/activity/f7619206-f77c-4f3a-85a1-22bb091cbb1f
PRD: https://lux.collections.yale.edu/view/activity/f7619206-f77c-4f3a-85a1-22bb091cbb1f

Dependencies/Blocks:
N/A
Related Github Issues:
N/A
Related links:

Bugherd: https://www.bugherd.com/projects/284041/tasks/2433

Wireframe/Mockup:
Current:

Expected:

Looks good in prod. Closing.


Polyfill throws for older browsers not supporting getAnimations().

Another one we're seeing in the wild: c.target.getAnimations is not a function. I worked around it in my forked version by doing a check for it, but also just wrapped a try/catch around the entire animationstart event as we've hit issues on a few parts of it now for older browsers.
https://caniuse.com/?search=getAnimations
Chrome 83 and older
Safari 13 and older
Firefox 74 and older

Yeah I think that's helpful to not throw, especially with it happening on animationstart. We have tons of small animations so it saturated our logs even with a small amount of traffic exposure. I think it not working for those CSS ones when not available is reasonable, and honestly the browser versions throwing the error are outside of our support window, so it's not the biggest deal, but others might run into it as well and for us the errors in bugsnag, etc would be pretty noisy.


Cannot handle large datasets?

What is the largest dataset that can be handled? I tried putting in 8000 data points in the array and it didn't work. Limiting that same dataset to less worked though...

@denny64 I'm willing to bet that this is a bottleneck in the underlying frappe-charts library, although I could be wrong. There might be other variables at play here, such as what browser / hardware you're running it on.
If possible, please post your dataset / setup / codesandbox link so that I can investigate it further. Thanks!

I've investigated this a bit more, and here are my findings:

It appears as though there is a bit of a ceiling before performance starts to drop off (I tried with 8000+ data points).
It seems to also appear when just using frappe-charts directly.

I am going to close this because it is unrelated to this library, although I encourage you to file an issue with frappe-charts and link back to here. Keep in mind that the frappe team is working on a version 2 of that library which will hopefully fix a lot of these issues.
Feel free to file another issue if you have some examples of bottlenecks in this library.


Placing Figures into more notebook folders

Currently I need to add a link from moro/public to wherever I put figures I include in my notebooks. Ideally I like the figures somewhere close to my notebooks w/o having to add this link (because book users need to set up this link as well).

The problem is that the Play web service cannot directly access anything
outside of the "\public" path. Maybe a hint here:
https://www.playframework.com/documentation/2.0/Assets
On Tue, Oct 6, 2015 at 4:11 PM, Sebastian Riedel<EMAIL_ADDRESS>wrote:

Currently I need to add a link from moro/public to wherever I put figures
I include in my notebooks. Ideally I like the figures somewhere close to my
notebooks w/o having to add this link (because book users need to set up
this link as well).
—
Reply to this email directly or view it on GitHub
https://github.com/wolfe-pack/moro/issues/89.

Or maybe here: http://stackoverflow.com/questions/22146586/play-additional-public-assets-directory

This sounds like it's not a configurable option then, but instead hardcoded.
If all notebooks are checked in publicly, you can always add their URL from
github.
On Tue, Oct 6, 2015 at 4:48 PM, Sebastian Riedel<EMAIL_ADDRESS>wrote:

Or maybe here:
http://stackoverflow.com/questions/22146586/play-additional-public-assets-directory
—
Reply to this email directly or view it on GitHub
https://github.com/wolfe-pack/moro/issues/89#issuecomment-146036267.

I am thinking of just generally allowing access to png files using a specific route which is mapped to the same file location (the docRoot) as the notebooks themselves.

And read the docRoot from the conf file, yeah, that would work.
On Tue, Oct 6, 2015 at 5:08 PM, Sebastian Riedel<EMAIL_ADDRESS>wrote:

I am thinking of just generally allowing access to png files using a
specific route which is mapped to the same file location (the docRoot) as
the notebooks themselves.
—
Reply to this email directly or view it on GitHub
https://github.com/wolfe-pack/moro/issues/89#issuecomment-146040381.


What will happend with the channel associated with the Dataverse for Teams environment?

Documentation is not explicit about what could happen with teams channel if we do nothing to reactivate the Dataverse for Teams environment.
In case of deletion of this environment, the channel will continue to be available isn't?
Thank you in advance.

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

ID: 027010ad-198c-2ae6-f96e-ac02ff02bf08
Version Independent ID: 027010ad-198c-2ae6-f96e-ac02ff02bf08
Content: Automatic deletion of inactive Microsoft Dataverse for Teams environments (preview) - Power Platform
Content Source: power-platform/admin/inactive-teams-environment.md
Service: power-platform
Sub-service: admin
GitHub Login: @matapg007
Microsoft Alias: matgupta

Thanks for your comment, @FAC-1312. This issue has been assigned to the doc author for review.

@FAC-1312
We've recently updated this content. Please reopen this issue if your question wasn't resolved with the updates.


Use plain string matching

Hey, first of all thanks for making this plugin!
This is just a simple PR to fix a bug with Org files, by using plain matching instead of regex.
eval_code_block fails to parse the language specifier in Org files due to string.find using regex matching by default. #+BEGIN_SRC will be interpreted as a regex and thus fail to match "#+" literally.

Thank you @intr-cx


The permissions are wrong for the api call.

You all have Organization listed and it need directory. I don't know if It only needs directory and not organization or both but I was getting 403 until I added directory

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

ID: 3e0a0671-953a-7797-6e0c-6616420f4c3b
Version Independent ID: a2de95d1-67b2-bcc6-402f-f5d23c53d13a
Content: How to manage inactive user accounts in Azure AD
Content Source: articles/active-directory/reports-monitoring/howto-manage-inactive-user-accounts.md
Service: active-directory
Sub-service: report-monitor
GitHub Login: @MarkusVi
Microsoft Alias: markvi

We have verified and fixed this . It should be Directory.Read.All . The document has been updated.


Navbar Tweak Scene Name Functionality

I’m trying to figure out whether or not navbar tweaks’ functionality has actually been supplanted, as the guide suggests it has. The main thing I use it for is to keep the actual scene name visible to me while showing the navigation name to my players. Is this functionality present else where or do I still need navbar tweaks in this case?

I believe this functionality is already included, whether by core Foundry or another module I have installed, I'm not certain. Here are the options that allow me to control the "player's" view while keeping mine the same:


Awareness for Sparkplug 4.0

Hi,
I just wanted to raise awareness regarding the upcoming Sparkplug 4.0 specification mentioned here: https://github.com/eclipse-sparkplug/sparkplug/issues/453#issuecomment-1551993325
I don't know much about the new spec version apart from more Protobuf types being used for transporting payloads to combat value conversions for signed numeric types.

I have asked the sparkplug guys to get an ETA on version 4. Version 3 seems a bit premature when handking the data types if you ask me...


Code compilation doesn't work

I am a windows user. Manually performed all the setup in windows as given. Everything spawned correctly and i reached localhost:3000 too and the compiler's route too. When clicked on compile, the compilation doesn't happen and no logs in the console or terminal.
Requesting any possible fixes. Here's the screenshots:

The setup has to be done in an linux system, with all the things in the docker file properly set-up, so that the entire application will start to work.


[Docs] Add details on multi-GPU / multi-node

Addresses: https://github.com/lvwerra/trl/issues/317#issuecomment-1521454031
I think we should give minimal instructions on how to run your scripts on multiple GPUs & nodes

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Integrate Cypress in CI/CD process

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/1066
Originally created by @bencodezen on 2019-09-27 16:35:26

First thing first, we need to start a meltano service in the GitLab CI.
The way services work, they can only specify an entrypoint and a command such as the service should work with a single command.
Our current meltano/meltano docker image cannot do that, but I have a pretty simple idea how to make that work — by using an entrypoint script that can create run meltano init if a certain env variable is set.
Then we'll be able to use the meltano/meltano image built by the CI in the build step to spawn a network accessible Meltano instance that you'll have to wire to you cypress runner.
You'll have 3 containers running in this job, such as:

meltano ui from meltano/meltano:$CI_COMMIT_SHA
postgresql from postgresql (like the other tests)
cypress as the job executable from meltano/meltano:$CI_COMMIT_SHA (what will be in the job log)

Now you'll have to make sure you cypress run can run on an arbitrary URL so that we can send the URL of the running service instead of localhost:5000.
Does it makes sense @bencodezen @DouweM?
I can help you set that up.
Reference: https://docs.gitlab.com/ee/ci/docker/using_docker_images.html#what-is-a-service
cc @DouweM

View 24 previous comments from the original issue on GitLab


STAVR.toml

We are a team of professional crypto industry validators. We have been validating and testing main and test networks for over 4 years.
Decentralization, security and the highest equipment - that's us.
We help projects develop at an early stage, we develop and help the community, and we also support them after the release.
Our services include:
🔥 API / REST Endpoint
🔥 RPC Endpoint
🔥 gRPC Endpoint
🔥 Web gRPC Endpoint
🔥 SnapShots
🔥 State Sync
🔥 Archive node
🔥 Genesis file
🔥 IBC Relayers
🔥 Seed Node
🔥 Live Peers
🔥 Genesis
🔥 Addrbook file
and other.
GitHub
Explorer
Monitoring


Should helm respect InstallOrder/UninstallOrder when upgrading release?

When upgrading release, there might be some newly resources to be created, some existed resources to be updated and some existed resources to be deleted.
The present upgrade logic of Helm:

renders the templates in a chart to get the target manifests, and sort the manifests by the order InstallOrder
https://github.com/helm/helm/blob/94438b67221d4af3c9993945a42275be7ffc6baa/pkg/action/action.go#L164
https://github.com/helm/helm/blob/94438b67221d4af3c9993945a42275be7ffc6baa/pkg/action/upgrade.go#L225
iterate every resource in the target manifests
a. if the resource dose not exist in k8s, it is to be created
b. if the resource exist in k8s, it is to be updated
diff the current release manifests with the target release manifests to get the resources need to be deleted

https://github.com/helm/helm/blob/94438b67221d4af3c9993945a42275be7ffc6baa/pkg/kube/client.go#L162-L243
All the options follow the order of  the resources in the target release manifests, should the upgrade option respects InstallOrder/UninstallOrder, especially for resources to be created or deleted?
I post this issue because I face an error when upgrading a release, some resources need to be deleted but deleting order is not the same as introduced in https://github.com/helm/helm/blob/master/pkg/releaseutil/kind_sorter.go.

In general, yes, Helm should follow the install and uninstall order during an upgrade. This prevents cases where deleting an object will cause an uninstall of another to report an error because it's already been deleted from another resource's cascading delete (e.g. deleting a namespace before deleting a deployment in that namespace).
In the install case, it's important to ensure secrets and configmaps are installed before deployments to ensure the deployment has all of the expected environment variables populated.
What error are you facing?

This is the case I met:
Prerequisites

We have two CRD NginxVhost and Registration, Registration depends on NginxVhost`.
We make a patch to helm v3.2.4 to add this two CRDs toInstallOrder/UninstallOrder as the follwing:

var InstallOrder KindSortOrder = []string{
	...
        "NginxVhost",
        "Registration",
        ...
}

// UninstallOrder is the order in which manifests should be uninstalled (by Kind).
//
// Those occurring earlier in the list get uninstalled before those occurring later in the list.
var UninstallOrder KindSortOrder = []string{
	...
        "Registration",
        "NginxVhost",
        ...
}

Admin controller will deny the request if deleting NginxVhost but related Registration exists

The case
After upgrade, NginxVhost  and Registration resources are removed and should be deleted, but the deleting order is not as UninstallOrder, so an error returned.

remove /Stale


Implement armour stand marker data

API | Common

Closing as the API PR is asked to be redone as data proper.


"Unhandled error" when I try to run code

Testing #13585
I tried to run "Hello World" in my Jupyter notebook connected to a remote kernel, and I received the following:

The kernel seems valid when I "Select a different kernel:"

I can also verify the URL I'm using loads in the browser:

@rebornix helped me a ton by helping me set up a Codespace as my environment, as I couldn't get my local one working and kept seeing this error if I tried a local kernel.

With my now working local Jupyter setup, I no longer see this error! But leaving open since the error message Unhandled error was vague.

@bamurtaugh I think the problem was because your Python environment was busted, if this problem still exists with the new Virtual Environment we created in the temp folder, please do re-open this, however I suspect this was due to the old python env.


nix categories on upload

at ?action=new
I would get rid of FC categories and keep or add
Public Domain
CC licenses
Default Copyright

The next version uses these section.
Are we OK with "Default Copyright"? I wanted something relatively neutral and descriptive.


added json and cbor codecs w/ feature guards

no stress if you don't want to merge this, but i figured it would be useful to have some common codecs included in the package.
relates to #32, though i believe they are orthogonal (though perhaps it could be called SerdeBincode or something rather than just SerdeCodec ?). it may be also possible to extract some commonality for serde-based codecs if this is desired, but, i think that is a future task.

I think this is a great idea thanks. Seems like something like this would be best suited as a feature, rather than an entire crate


modifying  MODEL.MASK_FORMER.NUM_OBJECT_QUERIES can't load pretrained model when training

Thanks for your code. I find that I can save GPU memory with modifying   MODEL.MASK_FORMER.NUM_OBJECT_QUERIES when running demo.py.  But after modifying that, I can't load pretrained model when training. Can you give me Any suggestion about this?

The pre-trained model won't work if MODEL.MASK_FORMER.NUM_OBJECT_QUERIES is modified. You will need to train the model again if you want to use different MODEL.MASK_FORMER.NUM_OBJECT_QUERIES.

@bowenc0221 Thanks for your answer.
But I got this error.
RuntimeError: Error(s) in loading state_dict for MaskFormer:
        size mismatch for sem_seg_head.predictor.query_feat.weight: copying a param with shape torch.Size([100, 256]) from checkpoint, the shape in current model is torch.Size([30, 256]).

I use pre-trained model weights to train my custom dataset, and I modified MODEL.MASK_FORMER.NUM_OBJECT_QUERIES = 30 , but it dosen't work. So you mean that I can't use pre-trained model to fine tune on my custom dataset?

As the error suggests, the pre-trained model you use has 100 queries instead of 30.


Voice connection timeout

Getting the same error on both your v2 and the original eris-lavalink.
In the bot's console:
Error: Voice connection timeout
    at Timeout.pendingGuilds.(anonymous function).timeout.setTimeout [as _onTimeout] (C:\Users\chris\Desktop\baconbot\node_modules\lavalink.js\src\PlayerManager.js:354:15)
    at ontimeout (timers.js:475:11)
    at tryOnTimeout (timers.js:310:5)
    at Timer.listOnTimeout (timers.js:270:5)

Lavalink console:
[16:11:23] [ INFO ] [[/]] Initializing Spring FrameworkServlet 'dispatcherServlet'
[16:11:23] [ INFO ] [DispatcherServlet] FrameworkServlet 'dispatcherServlet': initialization started
[16:11:23] [ INFO ] [DispatcherServlet] FrameworkServlet 'dispatcherServlet': initialization completed in 17 ms
[16:11:23] [ INFO ] [AudioLoaderRestHandler] GET /loadtracks
[16:11:23] [ INFO ] [GarbageCollectionMonitor] GC monitoring enabled, reporting results every 2 minutes.
[16:11:24] [ INFO ] [AudioLoader] Loaded playlist Search results for: lo-fi
[16:11:24] [ INFO ] [SocketServer] {"op":"connect","guildId":"356197359153315840","channelId":"371378812476194838"}
[16:11:24] [ WARN ] [SocketServer] Unexpected operation: connect
[16:11:34] [ INFO ] [SocketServer] {"op":"disconnect","guildId":"356197359153315840"}
[16:11:34] [ WARN ] [SocketServer] Unexpected operation: disconnect

I'm also experiencing the same issue, however my console isn't printing the same results. I'm not getting Unexpected operation in the console, so I'm not sure why it's happening for me. I've also referenced the issue in #17, so it seems to be a lavalink issue. Hopefully someone can find a solution to this, it's been bugging me over the past few days.

@Bacon1153 your stack trace seems to show you are using lavalink.js instead of SandySounds 🤔
As for my issue, voiceServerUpdate is never getting called, causing the timeout to never be cleared/resolved. I'm not sure why this is, apparently Lavalink is supposed to automatically handle that, but it doesn't seem to be the case for me. I'd appreciate some help on this, other people seem to be experiencing the same problem, and I can't tell if it's because of SandySounds or Lavalink, or something we are doing wrong.
Here's some information; raw event log, stack trace on timeout, running latest master on SandySounds and discord.js. I've tried switching regions, checking code and everything, yet I just cannot find the source of the issue.

@zBlakee Lol sorry I shouldn't have renamed the folder. My bad.

You should also share your findings in the original eris-lavalink gist.

Ah okay, got a bit confused there 😄
I don't use Eris, so it's a bit difficult for me to test my findings over there, especially with lavalink.js not being updated for v2.

@zBlakee Still, they're all very similar. Both lavalink.js and SandySounds are just ports from Eris lavalink with very few differences.

@zBlakee Brain said that your issue with voiceServerUpdate is a result of not having the correct perms. Are you sure you have all the necessary permissions?

You get Unexpected operation: disconnect when you use the v1 because in lavalink v2, you don't need to connect.

@jtsshieh I get that both in v1 and v2.

Oh! That might be the issue. In v2, these ops were removed:

connect
disconnect
validationRes
isConnectedRes
validationReq
isConnectedReq
sendWS

Are you guys providing SandySounds a valid client? A valid client example is the default exported one. Furthermore, make sure you guys are waiting for both a voiceServerUpdate and a voiceStateUpdate before calling the voiceServerUpdate on SandySounds.

Do you have to create your own client? @MrJohnCoder

Yes.

It'd be nice to get some simple, working example code. Not to be an annoyance, but I'm still pretty lost on how I'm supposed to call voiceServerUpdate myself, and other stuff like that. 👍

I +1 this

I haven’t had the time to make docs and examples yet but I have it on my todo list

Alright, sounds good 😄 Hopefully we can see them soon, thanks for all the help

Alright, awesome news for my case, finally found out how to pass data to the voiceServerUpdate function, and I can finally hear audio! For those suffering from the same issue, here is how you pass the data from the RAW event to voiceServerUpdate (at least what worked for me);
client.on("raw", message => {
  if (message.t === "VOICE_SERVER_UPDATE") client.player.voiceServerUpdate(message.d);
  if (message.t === "VOICE_STATE_UPDATE") {
      const player = client.player.players.get(message.d.guild_id);
      if (player && player.channel.id !== message.d.channel_id) {
          player.switchChannel(message.d.channel_id);
      }
  }
});

You'll probably need to replace client.player with what you defined new SandySounds() as.
However, this still didn't fix my issue for some reason, and I wasn't sure how I'd be able to efficiently store both VOICE_SERVER_UPDATE and VOICE_STATE_UPDATE information to later combine into one object and send to voiceServerUpdate. So, I decided to modify SandySounds.js at the voiceServerUpdate function and replace the data.session_id value on any sessionIDs with client.ws.connection.sessionID (I manually passed in the client myself). Probably not the cleanest solution, and hopefully I can change the way I do it when John creates an example file/documentation, but it works, and it works fairly nicely for me.
Thanks again for all the help.

Looks great! I'll try it when I get home.

Unfortunately, it didn't work for me. My ultimate solution was using a different client which worked perfectly and didn't give me any problems at all.

Could you guys try using the Eris client that I recently added? Want to check if it's your handling of the two packets or if it's your clients.


ProcessExitedException: process 1 terminated with signal SIGKILL

I am trying to train BERT model for classification on TPU.
Here is colab link
https://colab.research.google.com/gist/talhaanwarch/3c7475bee6e1d4d032f7325ccc898a2a/urdunews-pl.ipynb?authuser=3
In this code I am defining transformer model into PL structure. This gives me ProcessExitedException: process 1 terminated with signal SIGKILL error.
If I define transformer out of PL structure, (like I can call it where I have called tokenizer), I did not get any error. but I could not specify 8 TPU cores. I have to choose one TPU core. The process becomes slow, but I can choose a large batch size.
When I define it in PL structure, I cant set large batch size and get error in small batch size even.
So what I the best place to define the model? And how can I resolve the error
GPU and CPU version of this code is working. I tried to reduce token length of BERT to 128 and batch size to 4. Because some user mention, it is because of OOM, but I could not resolve it

Kind Reminder: While using 8 TPU cores, the effective batch size is batch_size * 8

Any clue about this problem? I have similar issue.

Yes, this arises because of batch size. Reduce the batch size
On Thu, Sep 9, 2021, 11:04 AM youngrok cha @.***> wrote:

Any clue about this problem? I have similar issue.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/PyTorchLightning/pytorch-lightning/issues/7989#issuecomment-915789682,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AI5FYOZZ4XDM526IJLFVJUTUBBE75ANCNFSM46XJEXGQ
.


concurrent connections

This is the case when 2 peers are trying to connect to each other simultaneously, and it seems we have some problems in handling such connections; Before the connection limiter we were most likely "saved" by artificial (small) timeout we put in one peer based on the if (remoteId.toHex() < this.localPeerId.toHex()) condition, so it would wait till the other peer connects first; Now the order is non-deterministic and seems like it happens bit more often, so I would like to make sure we properly manage those connections (seems like some of them might stuck in connecting state and also in connection-limiter)

@egorgripasov PTAL and update as you make progress!

The progress of fixing simultaneous connections are reflected in https://github.com/dxos/dxos/pull/4037


[Bug]: a deprecated HTML tag found in 07_band.html

Problem
In the 07_band.html file, I noticed that the <marquee> tag is used(see below code snippet). According to Deque University, this HTML tag is no longer in use as it only works with most browsers, which would make it difficult for learners to view it in the lesson.
 Here is a link to BLACKPINK's website!

Their top 5 songs are:

마지막처럼 (As If It's Your Last) - 2017

뚜두뚜두 (DDU-DU DDU-DU) - 2018

How You Like That - 2020

Kill This Love - 2019

붐바야 (BOOMBAYAH) - 2016

  <marquee>Hwaiting! 🫰</marquee>

Solution
Since <marquee> tag was often used to insert a scrolling area of text, the <canvas> tag would be a good replacement as it has a similar effect and it is compatible with all sorts of browsers.

closing: https://www.codedex.io/community/monthly-challenge/submission/T3tNDfa5zFh7Vj0xxInX


Unexpected token '�', "��{  |  "... is not valid JSON

Allthough I follow the exact examples in the readme file, I receive this error:
0|mm  | [10.08.2023 12:51.58.234] [ERROR] (node:7568) UnhandledPromiseRejectionWarning: SyntaxError: Unexpected token '�', "��{
0|mm  |  "... is not valid JSON
0|mm  |     at JSON.parse ()
0|mm  |     at Response.json (file:///home/pi/MagicMirror/modules/MMM-JsonTable/node_modules/node-fetch/src/body.js:149:15)
0|mm  |     at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
0|mm  | (Use electron --trace-warnings ... to show where the warning was created)
0|mm  | [10.08.2023 12:51.58.235] [ERROR] (node:7568) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag --unhandled-rejections=strict (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
The issue is in the URL format: 'https://server.vdplas.net:8083/thuis.json' . Because when entering 'https://jsonplaceholder.typicode.com/users' it works perfectly, the table is shown, but when entering the former URL, even when there is the exact same file located, it does not work.
What is wrong in the format? Is it the : ?

Nevermind. After rebooting my pi, it has been resolved.

Without changing anything (afaik), this issue returned.
Here the module config in the config.js:
		{
			module: 'MMM-JsonTable',
			position: 'bottom_right',
			header: 'Aanwezig',
			config: {
				url: 'https://server.vdplas.net:8083/thuis.json', // Required
				arrayName: 'items', // Optional
				tryFormatDate: false,
			}
		},

And this is the (anonymized) json, found internally on my network at https://server.vdplas.net:8083/thuis.json
{
    "items":  [
                  {
                      "icon":  "fa-person-walking",
                      "name":  "XXX",
                      "aanwezig":  "is weg sinds",
                      "lastUpdate":  "10-8-2023 11:22:22"
                  },
                  {
                      "icon":  "fa-person-walking",
                      "name":  "YYY",
                      "aanwezig":  "is weg sinds",
                      "lastUpdate":  "12-8-2023 04:47:57"
                  },
                  {
                      "icon":  "fa-home",
                      "name":  "ZZZ",
                      "aanwezig":  "is thuis sinds",
                      "lastUpdate":  "12-8-2023 04:48:10"
                  }
              ]
}

What can be the cause of this issue?

Which character encoding does your file have? Run file -bi [filename] to check.

It must have been something in de file encoding.
I completely recreated the file withe the -Encoding ascii parameter
($myJson | ConvertTo-Json -Depth 2 | Out-File $filename -Encoding ascii)
Now it works!


Updating TweakScale to cope with more than one folder on the package.

See https://forum.kerbalspaceprogram.com/index.php?/topic/179030-14-tweakscale-under-lisias-management-2431-2019-0725/&do=findComment&comment=3642572 and subsequent; the default install stanza was finding Extras/TweakScale instead.

@Lisias, are any previous versions affected by this problem? Only the latest one will be fixed automatically by this PR, but we can apply the fix to older versions if needed.

No. This stunt was applied to TweakScale <IP_ADDRESS> and later only. But only <IP_ADDRESS> reached CKAN, so just know I was bitten by it.
Thank you.


Wonder if it's possible to 'force' same widths?

Firstly, I love this plugin! Im just having a ply and trying to set up a master wireframe document. However I've run into a problem… In the image below, when I resize it or change the text, sometimes the image background on the top will be smaller or bigger than the text box below?
Any ideas how I can get around this?

And here is how I have it all set up…

Hmm.

Make sure your text labels have a 'fixed' width to them

Try setting a set size of 'Rectangle 2'. By changing the name to something like Rectangle 2 [40 20 80;w=250]. That should make sure the rectangle always stays at a size of 250.

Let me know how you go 👍

Hey @Chrisedmo; this may have been an issue that existed with fixed-widths. I just pushed an update, version 1.0.2 that should fix this. Can you verify if it does indeed fix it for you?
I will close this issue in the meantime. If the problem does still persist, then please re-open this.


New tagged release version

Hello,
I apologize if this is the wrong place to ask for that but there's been a lot of commits since the last release (v 0.3.0) in April 2017. Are there any plans to release the library as-is for a new release version?
I'm just asking because a developer who uses your library for one of his WordPress plugins told me in reply to a question I asked him, that because there's no tagged release upstream with the codebase (as-is at the moment), he can’t easily reference that specific commit history version from Composer.
Thanks for reading and please keep that great project running!

The latest tag and release is actually 0.4.0 from August 2019.  Now, there has still been work since then so I'll look at cutting a new release soon.

Ok, everything seemed fine so I've also made a 0.5.0 release now.

Ok, everything seemed fine so I've also made a 0.5.0 release now.

I'd really like to thank you so much for your fast reply and release of the new version!


AAAA / ipv6 support

I am thinking about implementing AAAA / ipv6 support for this package but I'm not sure how best to go about it.
After research I'm finding that the current URL used for getting IP address is returning ipv4 only. (seems to be a current limitation with https://api.ipify.org.  Supposedly fixed, but seems to be not functioning now)
Some thoughts to get the ball rolling --
Assertions

Some hosts will only have ipv6 addresses, and some will only have ipv4 so we can't assume that one or both will exist.

If only 'A' or 'AAAA' records exist then it might be nice to intelligently make the call to get the public IP for only the required type.

Separate api calls will need to be made to obtain each ip address since a single request can't be made over both ipv4 and ipv6 protocols simultaneously

Will need to gracefully handle a failed request for ip address since one or the other might not resolve

Specific Questions

Should ipv6 address be determined by an api call or by assuming an ipv6 address is the same as the public?

If an api should be used to obtain ipv6 address, what is a preferred alternative to ipify.org?

Should the address(s) be stored as JSON in a single ipfile or as strings in separate files?

Hello @anotherZero,
Thanks for the idea and for the elaborate proposal.
At a first glance I am not sure if I want to go for a "smart" approach or for a configuration oriented one (user needs to explicitly define the protocol he wants to use and that will determine the execution flow of the application).
If we use this approach we can easily add the SMART option later on and use some technique to determine wether an IPv6 or an IPv4 is available and which one has higher priority in case they are both available.
So if we go in this direction, the new config file might become something like this:
{
  "apiKey": "",
  "secret": "",
  "domain": "example.com",
  "records": [{"protocol": "ipv6|ipv4|smart", "name": "mysubdomain", "ttl": 600}]
}

Notice that I am replacing the key type with protocol, which could have ipv6, ipv4 or smart as possible values.
As an alternative, we could keep the type key and detect the protocol from the value:

A -> ipv4
AAAA -> ipv6
smart -> smart

This is probably better for backward compatibility.
Anyway I think this still requires a bit of thinking and maybe some experimentation in order to be able to come up with a good solution.
Do you have any plan to start playing with it?

I like the idea of a smart option.  Don't forget about other possible records that one might want to continue managing such as CNAME or MX (which might better lend itself to keeping the type key.
I am interested in working on this but I have to admit that I haven't worked with bluebird or es6 much so there's a bit of a learning curve.  It might be a bit before I could produce anything.

Regarding CNAME records, I don't think this utility can really be useful (as previously discussed here).
For MX records, it is a very similar situation. With MX records you won't be specifying IP addresses as values but other domain names (which may refer either A, AAAA or CNAME records).
An example here (from this page):

Host/Domain
Address/Mail Server/MX Entries/Value
Priority

@
mx.zoho.eu.
10

@
mx2.zoho.eu.
20

@
mx3.zoho.eu.
50

As far as I am aware you can't specify IP addresses in the Mail Server column.
With this in mind  would you prefer to go with a type approach or with the protocol one?
Personally, at the moment, I am more inclined towards the type one.

I realize that there is a more limited usefulness for other types of records, but I am currently using this utility to set about a dozen records when deploying a new server.
I use saltstack to write out a config.json and run godaddy-dns once.  It's kind of nice to have all of the dns entry calls bundled into a single package as opposed to having the ip related ones done by this package, and the cname/mx done somewhere else.
That being said, I agree that type is the correct parameter to use :)

That's an interesting use case I haven't thought before.
I have to assume that for non-ip based records you just set the data option directly in the config value.
Something like the following config:
{
  "apiKey": "",
  "secret": "",
  "domain": "example.com",
  "records": [{"type": "CNAME, "name": "mysubdomain", "ttl": 600, "data": "anotherdomain.com"}]
}

I wonder if that's something we should explicitly mention in the documentation.
Also I am curious to have an example of config from your current use case.

I can provide an example config like the one I'm using for this use case.
In this case, I'm just filling in the ip addresses as well because ipv6 isn't supported yet and I'm generating the config.json anyway.
{
  "apiKey": "",
  "secret": "",
  "domain": "mydomain.com",
  "records": [
    { "type": "A", "name": "sub1", "ttl": 43200, "data": "<IP_ADDRESS>" },
    { "type": "A", "name": "smelt.sub1", "ttl": 43200, "data": "<IP_ADDRESS>" },
    { "type": "AAAA", "name": "sub1", "ttl": 43200, "data": "2001:4860:4860::8888" },
    { "type": "AAAA", "name": "smelt.sub1", "ttl": 43200, "data": "2001:4860:4860::8888" },
    { "type": "CNAME", "name": "mail.sub1", "ttl": 172800, "data": "mail.otherdomain.com" },
    { "type": "MX", "name": "sub1", "ttl": 172800, "priority": 10, "data": "mail.sub1.mydomain.com" },
    { "type": "MX", "name": "smelt.sub1", "ttl": 172800, "priority": 10, "data": "smelt.sub1.mydomain.com" }
  ]
}

Thanks a million @anotherZero! This is very helpful for me to figure out other use cases and possible improvements.

Was any progress made on AAAA records?  I'd be happy to help contribute but I don't want to repeat any work.

I think this has been implemented.  Closing
thanks @lmammino


[Android] Update input error visual cue rendering to support custom background drawables

An accident happened and #4895 got merged, redoing the PR
Description
Adds options to modify the rendering of the error visual cues for validated inputs. The original version of this feature only considered the default drawable which has a transparent background, due to this when tinting the background it could have unintended side effect such as tinting the whole background, such as in the example below

For this fix we added two new attributes called adaptiveUsingCustomTextInputs and adaptiveUsingCustomChoiceSetInputs.

adaptiveUsingCustomTextInputs: boolean flag that signalize the renderer to set the adaptive_state_input_invalid with a value of true instead of tinting the text input background.
adaptiveUsingCustomChoiceSetInputs: boolean flag to control some internal rendering for compact choice sets. If the value is set to true but no background is provided results may vary

On the other hand we added a new drawable state called adaptive_state_input_invalid which allows customers to define background drawables which define how an invalid input would look like
Finally, we also added two new drawable selector files adaptive_choiceset_compact_background.xml and adaptive_choiceset_expanded_background.xml:

compact drawable: controls the background for compact choicesets, if the adaptiveUsingCustomTextInputs flag is not set or set to false, the drawable is applied to a linear layout that encloses the spinner and is tinted in attention color.
expanded drawable:  controls the background for expanded choicesets (mono and multi select) and the background for Toggle inputs. In all cases this background is tinted with the attention color to mimic the current JS and iOS behaviours.

How Verified
The fix was verified by adding the previously mentioned properties into our app theme (which will be modified for each of the examples below)
<item name="adaptiveUsingCustomTextInputs">true</item>
<item name="adaptiveUsingCustomChoiceSetInputs">true</item>
<item name="android:editTextBackground">@drawable/layered_edit_text_styles</item>

For reference, layered_edit_text_styles is a selector which only defines a drawable for invalid state and a default drawable
<selector xmlns:android="http://schemas.android.com/apk/res/android" xmlns:adaptivecards="http://schemas.android.com/apk/res-auto" >
    <item adaptivecards:adaptive_state_input_invalid="true" android:drawable="@drawable/layered_edit_text_invalid" />
    <item android:drawable="@drawable/layered_edit_text" />
</selector>

Example 1: No custom inputs
<item name="adaptiveUsingCustomTextInputs">false</item>
<!-- <item name="android:editTextBackground">@drawable/layered_edit_text_styles</item> -->

Example 2: Having a custom drawable, but stating that is not using custom inputs
<item name="adaptiveUsingCustomTextInputs">false</item>
<item name="android:editTextBackground">@drawable/layered_edit_text_styles</item>

NOTE: the image is tinted, as it has a semi transparent background color it tints it in a lighter shade of Attention color

Example 3: Having a custom drawable
<item name="adaptiveUsingCustomTextInputs">true</item>
<item name="android:editTextBackground">@drawable/layered_edit_text_styles</item>

Example 4: Default behavior for choiceset and toggle input validations
<item name="adaptiveUsingCustomChoiceSetInputs">false</item>

Also, the default behaviour for choiceset inputs validation has changed to show a border around the inputs

Toggle Input has changed too

And the default behaviour of compact choice set is as follows

Example 5: Overriding a compact choiceset
styles.xml
<item name="adaptiveUsingCustomChoiceSetInputs">true</item>

The default background drawable for spinners can be updated by creating a file called adaptive_choiceset_compact_background.xml in the drawables folder, in this file an adaptive_state_input_invalid flag can be set in the selector to define how the compact choice set should look like
adaptive_choiceset_compact_background.xml in drawables folder
<selector xmlns:android="http://schemas.android.com/apk/res/android" xmlns:adaptivecards="http://schemas.android.com/apk/res-auto">
    <item adaptivecards:adaptive_state_input_invalid="true">
        <layer-list android:opacity="transparent">
            <item android:gravity="left" android:width="300dp" android:end="300dp">
                <shape>
                    <solid android:color="@color/colorWhite">
                    </solid>
                </shape>
            </item>
            <item android:gravity="left" android:width="100dp" android:start="300dp">
                <shape>
                    <solid android:color="@color/colorAccent">
                    </solid>
                </shape>
            </item>
            <item android:width="100dp" android:start="300dp" android:height="40dp">
                <bitmap android:src="@drawable/cat1" android:gravity="fill"/>
            </item>
        </layer-list>
    </item>
</selector>

Example 6: Overriding the rendering of expanded choicesets
The rendering for the expanded choice set and toggle inputs can also be modified (in this case no flag is required), the only consideration is that the border is always tinted with the attention color to keep consistency. As can be seen in the image below, the corners can be changed to be more rounded or change the width of the enclosing rectangle

Microsoft Reviewers: Open in CodeFlow

@almedina-ms this is running into asome Android build failures - is this needed for 20.10 Teams unblocking?

@jwoo-msft is this actually going into 20.10?


Code inspector results do not respect 'Jump to ADT first' setting #1892

enable jump to ADT and show subobjects (eg Includes) in CI result
Tested with report with include, maybe there is a different behaviour with other objects?

you are right - try to find some method / generic approach

Fixed commits
works fine now for all objetcs also with jump to line

#1892

removed commented coding


Issue with Sending Large Data Over BLE Characteristic

hi
I'm running into a problem with sending data through a BLE characteristic.
I have a characteristic with a permit of 92 and a size of 240. I need to send data that's between 750-800 bytes, but when I try to send it, I either get a "Data too long" error or the data doesn’t show up correctly on the client.
giving error when i am writing to that char
btfpy.Write_ctic(btfpy.Localnode(), characteristic, data, 0)

this is my device.txt
DEVICE = TESTDEVICE TYPE=MESH NODE=1 ADDRESS = --:--:--:--:--:-- NAME=TEST

  PRIMARY_SERVICE = 00000001-710e-4a5b-8d75-3e5b444bc3cf

    LECHAR = TEST_GET_ALL_TELEMETRY_DATA_CHARACTERISTIC  PERMIT=92  SIZE=240  UUID=00000002-710e-4a5b-8d75-3e5b444bc3cf

    LECHAR = TEST_SET_DATA_TYPE PERMIT=98  SIZE=20  UUID=00000003-710e-4a5b-8d75-3e5b444bc3cf

    LECHAR = TEST_GET_ALL_AGG_TELEMETRY_DATA_CHARACTERISTIC  PERMIT=92  SIZE=240  UUID=00000004-710e-4a5b-8d75-3e5b444bc3cf

  PRIMARY_SERVICE = 00000100-710e-4a5b-8d75-3e5b444bc3cf

    LECHAR = TEST_AWNING_CONTROL_CHARACTERISTIC  PERMIT=A6  SIZE=128  UUID=00000200-710e-4a5b-8d75-3e5b444bc3cf

    LECHAR = TEST_SEND_AWNING_MESSAGE  PERMIT=98  SIZE=16  UUID=00000300-710e-4a5b-8d75-3e5b444bc3cf

Any ideas on how to fix this?
Also how to set up the MTU negotiations?

I've tried your devices.txt and have been able to read 240 bytes from two clients: Another Pi running btferret, and NrFConnect  running on a PC, so I have been unable to reproduce your error. Both of these clients complete an MTU exchange as follows. The client should set the MTU as it does below, with the btferret replies, but to be sure, a btferret server tries to set the MTU as well (btlib.c line numbers indicate where). Set verbose print mode and see if the client is sending the same requests/responses. It may be refusing MTU requests or agreeing to a smaller value.
Edited verbose print for server showing MTU settings
Set verbose via:  set_print_flag(PRINT_VERBOSE);
or from btferret.c via k - Settings

Advertising as DC:A6:32:04:DB:56 My Pi

  *** server requests MTU = F4 (244) - expects opcode 03 reply
  *** line 6635 in btlib.c
Set MTU
  Set [1][2] handle 40 00
< CHANNEL 0004 Opcode = 02
      0000  02 40 00 07 00 03 00 04 - 00 02 F4 00

  *** server requests data length = FB (251)
  *** line 3589 in btlib.c
Set larger data length
  Set [4][5] handle 40 00
< HCI OGF=08 OCF=22
      0000  01 22 20 06 40 00 FB 00 - 00 08

  *** client replies data length = F2 (242) is OK
> Event 3E = 07 40 00 F2 00 00 08 1B 00 48...
      0000  04 3E 0B 07 40 00 F2 00 - 00 08 1B 00 48 01
Pi400 data length change T F2 R 1B

  *** client requests MTU = F7 (247) - must send opcode 03 reply
> CHANNEL 0004 Opcode = 02
      0000  02 40 20 07 00 03 00 04 - 00 02 F7 00
GOT LE server opcode 02 from Pi400

  *** server replies MTU F7 is OK
SEND MTU exchange reply
  Set [1][2] handle 40 00
< CHANNEL 0004 Opcode = 03
      0000  02 40 00 07 00 03 00 04 - 00 03 F7 00

  *** client replies to original Set MTU request MTU = F7 is OK
> CHANNEL 0004 Opcode = 03
      0000  02 40 20 07 00 03 00 04 - 00 03 F7 00
GOT LE server opcode 03 from Pi400


Reduce number of allocations in verification rotines

Don't allocate slice for object data.
Allow unmarshaling function to be provided by caller.

I think there is no point to keep both StableMarshal() and MarhsalStream() series of functions in the repository, since they are extremely similar but require a lot of tedious work to maintain it without codegen. Let's stick with one option that provides better efficiency based on our load tests and benchmarks.
To keep MarshalStream series of functions, this PR missing implementation on these types:

object.Object,
storagegroup.StorageGroup,
tombstone.Tombstone.

Needs to be reconsider after api-go -> sdk-go migration and repository structure changes.


Circular filesystem watch events cascade, including self-referential, leading to RangeError: Maximum call stack size exceeded

Describe the bug
I have encountered crash of dev script.
RangeError: Maximum call stack size exceeded

Initial investigation lead me to emitChangeEventOnDependants function in src/utils/watch.js, where I've discovered
self-referential loops, where supposed dependant was same as filename.
Unfortunately, blocking self-referential event emitting revealed another problem - circular emission of virtual change event.
It is challenging to create an isolated reproduction.
Which may or may not be related to how vite-plugin-svelte-cache choses to build dependency tree.
However, it is clear  that self-referential, and circular references may exist, and therefore may overwhelm nodejs process with storm of emited events.
While it is not as good as reproduction, I hope, you would find this information instrumental.
Reproduction URL
https://github.com/sveltejs/vite-plugin-svelte
Reproduction
No response
Logs
No response
System Info
System:
    OS: macOS 13.5.2
    CPU: (8) arm64 Apple M2
    Memory: 208.14 MB / 24.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.8.0 - ~/.asdf/installs/nodejs/20.8.0/bin/node
    npm: 10.1.0 - ~/.asdf/plugins/nodejs/shims/npm
  Browsers:
    Chrome: 117.0.5938.149
    Safari: 16.6
  npmPackages:
    @sveltejs/adapter-auto: ^2.0.0 => 2.1.0
    @sveltejs/adapter-static: ^2.0.3 => 2.0.3
    @sveltejs/kit: ^1.20.4 => 1.25.1
    svelte: ^4.0.5 => 4.2.1
    vite: ^4.4.2 => 4.4.10

Related: https://github.com/sveltejs/vite-plugin-svelte/issues/690

please provide a reproduction, I'm eager to kearn where this happens. circular dependencies in preprocess dependencies is always an error i think. we can add a guard to avoid the stack error, but would throw a "circular dependency" error instead.

@dominikg, i get it by commenting and uncommenting styles multiple times in https://github.com/andriytyurnikov/andriytyurnikov.github.io/src/routes/garage/web-font-size-calculator/+page.svelte
BTW, i've commented out "virtual update event" in my local setup.
Moreover, I have reached a conclusion, which I would like to share:
emission of such virtual events would be safe only with capability of circular dependency check before emission:
existance of emitChangeEventOnDependants outside of vite-plugin-svelte-cache looks like a fundamental mistake.
Without mechanizm to check for circular dependencies, every such virtual event is a potential circular cascade trigger.
Code is written, like there is a guarantee for cascade not to happen, while even simple references existed. I've made an accent on self-references, because it is a simplest case of circular loop, yet it is not prevented.
Or put it this way: every event emission within event processing may cause the loop. Without explicit guarantees of no loop, such solutions should be discouraged.
Have fun.

Sorry for jumping all over the place, here is a cleaner formulation:

event trigering with event processing loop is always a of circular cascade risk

which may be mitigated with some isCircular() check
or by being performed out of the loop (strongly preferred)

Having (periodic?) event emitter outside of the loop sounds like possible alternative:
instead of watcher.emit(), filesToWatchOutsideOfTheLoop.push() and periodicHartbeatEventEmitterOutsideOfTheEventProcessingLoop(filesToWatchOutsideOfTheLoop)

The fact that we even have to track these dependencies ourselves instead of relying on vite features for it is a bit unfortunate. Not looking into making this more complicated than it already is. In the long time we do this, this is only the second report of a circle happening and i still havn't seen a reproduction for it.
Discussed this with @bluwy  and 2 possible fixes are

ensure that the issues we had with vite addWatchFile are gone and switch back, removing all the custom code in v-p-s (preferred)
add a guard to prevent the event triggering the overflow

adding a separate periodic mechanism to defer the change event would introduce even more complexity and ways for it to go wrong (things happening out of order etc).

TLDR; Things are complex and there are 3 paths.

no "virtual events" (preffered)
single(!) context for dependency tree checking
fundamentally redesigned failsafe watcher, capable to withstand circular cascades
Danger! Also, there is a potential workaround hack: accumulate "virtual events" in a buffer,
and then - periodically, out of the event loop, remove duplicates, and flush that buffer by emitting those events periodically - once in a second or something. Limit the buffer size, for extra safety. Dirty-dirty-dirty.

Details:
Sad irony is that vite relies on 3rd party watchers (chokidar?), I remember reading their discussion about choosing new one.
I don't mean to be pushy, yet I would like to reiterate for clarity ( and disagreements, questions and critique are totally welcome):
My understanding is that current implementation optimistically assumes:

impossibility of self-referatial inputs from vite-plugin-svelte-cache
ability of vite's watcher (!!!) to resolve such cases
As both assumptions are proven(?) to be wrong, I am yet again advise against "virtual event emission", as every such event is a potential trigger for cascade.
Without cascade-breaker/preventer mechanism in either watcher, vite-plugin-svelte-cache or vite-plugin-svelte such behaviour is fundamentally unsafe.

Also, consider this - circular dependency prevention mechanism based on reference checks is only possible on a level, where full dependency graph is known. With this in mind - having more then one dependency-tracking context is fundamentally vulnerable to cascades as well, as such contexts would be able of triggering each-other.
It is not obvious how exactly watcher (and code triggers event on watcher, which is not the same as communicating dependency tree to vite!) is supposed resolve such an issue: watcher receives "event", and it is not aware that event is "virtual", so watcher tries hard to do his job, regardless of dependency tree - because allowing or disallowing circular references is not it's job - maybe you building a language friendly to circular behaviour.
Apparently, watchers leave issue of dependencies beyond their scope. And, again, it is not surprising, given the need of access to dependency tree for comparison-based implementation. Yet insufficient watcher resiliency is indeed surprising.

I just upgraded my entire project to svelte 4 (meaning sveltekit, vite and the rest all updated). Watcher failing and same error as described, I can't save a file without a crash that bubbles up from:
at Array.forEach (<anonymous>)
at emitChangeEventOnDependants (node_modules/@sveltejs/vite-plugin-svelte/src/utils/watch.js:22:14)
at node_modules/@sveltejs/vite-plugin-svelte/src/utils/watch.js:94:66

Here's a full stack trace:
Exception in PromiseRejectCallback:
/node_modules/vite/dist/node/chunks/dep-2b82a1ce.js:65205
    });
    ^

RangeError: Maximum call stack size exceeded
/node_modules/@sveltejs/vite-plugin-svelte/src/utils/vite-plugin-svelte-cache.js:167
                const dependants = this.#dependants.get(path);
                                   ^

RangeError: Maximum call stack size exceeded
    at VitePluginSvelteCache.getDependants (/node_modules/@sveltejs/vite-plugin-svelte/src/utils/vite-plugin-svelte-cache.js:167:22)
    at emitChangeEventOnDependants (/node_modules/@sveltejs/vite-plugin-svelte/src/utils/watch.js:21:28)
    at /node_modules/@sveltejs/vite-plugin-svelte/src/utils/watch.js:94:66
    at Array.forEach (<anonymous>)
    at FSWatcher.<anonymous> (/node_modules/@sveltejs/vite-plugin-svelte/src/utils/watch.js:94:44)
    at FSWatcher.emit (node:events:525:35)
    at /node_modules/@sveltejs/vite-plugin-svelte/src/utils/watch.js:27:13
    at Array.forEach (<anonymous>)
    at emitChangeEventOnDependants (/node_modules/@sveltejs/vite-plugin-svelte/src/utils/watch.js:22:14)
    at /node_modules/@sveltejs/vite-plugin-svelte/src/utils/watch.js:94:66

Node.js v18.12.1

Much of what you have discussed is far beyond my technical understanding of the sveltekit/svelte ecosystem so i'll pass on trying to understand for now as i'm looking to just build! However I hope the above is of some value (and proof this is happening to others too 👀)

@RowanAldean , I had same story, in your project's folder you may find file
<EMAIL_ADDRESS>on line 27
you may see
watcher.emit('change', dependant);
comment this line to avoid crashes.
P.S. I have tailwind installed. What would you say about your setup? Are you use anything beyond vanilla setup? Style preprocessors?

@andriytyurnikov Thanks, i'll try this and edit comment with result(s).
I'm using svelte-preprocessor to support a global app.scss via prependData in my svelte.config.js.
Also using tailwind for skeletonUI and recently for shadcn-svelte which is what forced me to do this whole updating everything as I had a bunch of dependency issues.
Also your solution seems painful as this would need to be done anytime I rebuild node_modules from package.json😢

@dominikg I wonder if all this jazz is connected with Tailwind recompilation or CSS purging...
... as Tailwind generates CSS classes depending on js files (configs, extensions, plugins), and basically change to any file may cause change of CSS styles, including those inside svelte components
<style></<style>

which leads to update of components, which leads to virtual event, which may lead to recompilation of tailwind.... and so on, and so forth...

yes, tailwind with it's postcss directory dependencies could cause this. I'd recommend not using their @apply feature in svelte style blocks.
in general I'd recommend unocss instead of it. But given it's popularity and use in 3rd party libraries we have to make it work somehow. still leaning towards keeping it as simple as possible which might mean just avoiding emitting the event in case it's a circle.

yeah, so whats happening here is not pretty at all...
in the project above, these are the dependencies for src/routes/+page.svelte returned by preprocess:
4:37:24 PM [vite-plugin-svelte] dependencies of andriytyurnikov.github.io/src/routes/+page.svelte
andriytyurnikov.github.io/src/pro-rata.css
andriytyurnikov.github.io/src/index.test.js
andriytyurnikov.github.io/src/lib/index.js
andriytyurnikov.github.io/src/lib/proRata.js
andriytyurnikov.github.io/src/routes/+layout.js
andriytyurnikov.github.io/src/app.html
andriytyurnikov.github.io/src/routes/+layout.svelte
andriytyurnikov.github.io/src/routes/+page.svelte
andriytyurnikov.github.io/src/routes/(app)/+layout.svelte
andriytyurnikov.github.io/src/routes/(app)/garage/+page.svelte
andriytyurnikov.github.io/src/routes/(app)/garage/fundamental/+page.svelte
andriytyurnikov.github.io/src/routes/(app)/garage/fundamental/List.svelte
andriytyurnikov.github.io/src/routes/(app)/garage/fundamental/NavBar.svelte
andriytyurnikov.github.io/src/routes/(app)/garage/fundamental/Shell.svelte
andriytyurnikov.github.io/src/routes/(app)/garage/ideas/+page.svelte
andriytyurnikov.github.io/src/routes/(app)/garage/layouts/+page.svelte
andriytyurnikov.github.io/src/routes/(app)/garage/pro-rata/+page.svelte
andriytyurnikov.github.io/src/routes/(app)/garage/ring-buoy/+page.svelte
andriytyurnikov.github.io/src/routes/(app)/garage/web-font-size-calculator/+page.svelte
andriytyurnikov.github.io/src/routes/(app)/garage/fundamental/list/+page.svelte
andriytyurnikov.github.io/src/routes/(app)/garage/fundamental/navbar/+page.svelte
andriytyurnikov.github.io/src/routes/(app)/garage/fundamental/shell/+page.svelte
andriytyurnikov.github.io/tailwind.config.js

including the file we just preprocessed in the dependencies output is obviously super wrong, so step 1 is filtering that from the returned dependencies before passing them on. This means that the direct circle from +page.svelte to +page.svelte no longer happens.
but notice how it includes all the others too? change any of those files and +page.svelte gets preprocessed/compiled again just in case the change actually affected the output.
We have no way to tell which of these dependencies are real and which are just there because tailwind postcss decided to go nuclear.

update: i cannot reproduce this issue with a clean setup of sveltekit and tailwind
pnpm create svelte@latest kit-tw
// select demo
cd kit-tw
pnpm i
pnpm dlx svelte-add@latest tailwindcss
pnpm dev

checking +page.svelte in your app shows you  have   @import "./../pro-rata.css"; with that file containing
@tailwind base;
@tailwind components;
@tailwind utilities;

so it's not tailwind going nuclear on it's own, you basically put all of it in your app.svelte style block. This is not recommended. instead import that file in the script block of your root layout (where you already import app.css which btw also includes the tailwind directives).
This explains why we don't see more reports of it happening, you really shouldn't do this.

I'm leaning towards throwing an error if we detect the file itself in preprocess dependencies, its a sign of something seriously wrong.

Well, app.css and <style></style> a processed independently, so if I want to use utility classes in
<style> @apply custom-utility; </style> - this is the way.

BTW: with asynchronous load of unnamed component in mind, I can even see the case for "update every .svelte component file".
Yet, for SvelteKit case I struggle to see how dependency between +page.svelte files can happen.

I mean if throwing an error is a first step - then sure, otherwise it would just replace RangeError with YourCustomError.

"This explains why we don't see more reports of it happening, you really shouldn't do this."

I don't think I got the resoning part. Why we should not do it?
As you see there were 2 reports in a week. It is unclear how many just give up.

@RowanAldean It looks like details about your setup and practices might be of use here.

I don't think I got the resoning part. Why we should not do it?

because <style> blocks in .svelte files are scoped to that component, you should not import global css into it. It wasn't tailwind that created this circle on it's own, it was you. tailwind generates styles based on the content of other code files, you told it to look at *.svelte files so they become a dependency of the tailwind styles. And then you go and import the tailwind generated styles into a style block in a .svelte file.
The Custom error can at least explain it better and link to a FAQ entry clarifying it (eg expand https://github.com/sveltejs/vite-plugin-svelte/blob/main/docs/faq.md#where-should-i-put-my-global-styles to explicitly mention @import as another antipattern in scoped style blocks.
The code tracking these dependencies in vite-plugin-svelte has existed for months (if not years) and it has lots of users. There are popular svelte libraries like skeleton-ui that use svelte with tailwind.  The only other report was on July 18, not last week and also did not provide a reproduction.
If this was a more general issue, there would be more reports.

Sure, I wanted Tailwind utilities in my scopes, but I get your point, thank you.

I wasted more time yak shaving and moved away from scss in favour of pure postcss (hours wasted).
I also removed the explicit dependency on this plugin (it's bundled with sveltekit anyways "@sveltejs/vite-plugin-svelte" "^2.4.1"
Since I did this (and admittedly without realising until I saw this thread) I have not had the issue.
However, yes. I was importing my top level stylesheet into my top level +layout.svelte and then making that style block <global> - this was for the following reasons:

Use the scss variables declared in my app.scss
Make the other styles in +layout.svelte apply to all children routes.

 Please say a prayer for the giant change and PR being merged soon 🙏


Mastodon support

add support for posting updates to Mastodon using the Mastodon.py module
Functionally similar to the Twitter support

Will take a look soon! 👀

@crd477 is there a reason you set test to false here? That's why the action is failing (we don't have those credentials in the test repo)
python /home/runner/work/jobs-updater/jobs-updater/.//find-updates.py update --keys url --unique url --original example/jobs-previous.yaml --updated example/jobs.yaml --deploy-mastodon
MASTODON_ACCESS_TOKEN is not set, and required when mastodon deploy is true!

No, there was no real reason.

I think it worked:
https://github.com/crd477/jobs-updater/actions/runs/3797494327/jobs/6458485807
and results:
https://mastodon.sdf.org/@crd

@crd477 thanks for your patience here! The PR here looks good - we do need one final run on your branch (with the updates here) to check that outputs / warnings have gone away (and that the toot is still successful). No worries if you want to pick up another day - we had a lot of back and forth today!

Done: https://github.com/crd477/jobs-updater/actions/runs/3797730333

Sorry - I meant the workflow where you post to mastodon? I think this is just the tests (that are here, just slack) running on your branch?
I can merge as is and we can then test after, but it's usually better to see that everything works before that.

My Mastodon account has already been censored by the server administrators for using a bot.  I was insulted enough that I simply deleted the account so I can't test it any more anyway.
I expect that it still works correctly but you can do whatever you think is best, including just closing this PR without merging it.  I'm really not happy.

oh my gosh! I'm so sorry. Let's just merge, and the update to US-RSE (I assume with some credential) can be the test, and totally OK if we need to come back and do another round! I'll hold off on drafting a release until we do that, but you should be able to use main branch. Ping me on the PR so I know when it's merged and can delete the old branch!
And seriously, thank you for your hard work today! I'm so sorry about your account, I feel partially responsible for that.

Is there someone I can ping on your server to apologize, and see if we can get your account back?

Nah, don't worry about it.  I've never been on any other social media platform before and I honestly haven't seen any redeeming qualities in Mastodon that make me want to start now.  I'm 95% sure I never would've used that account for anything other than testing this patch.

Anyway, I'm sorry you aren't happy - hopefully we can make it up in the future with a less arduous / more fun PR review!


flutter 3.10.6 uses impeller to render and iOS phones consume very high power

** Don't make a duplicate issue.
You can search in issues to make sure there isn't any already opened issue with your concern.
Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Provide us a completely reproducible code (contains the main function) in a main.dart file, it helps us to find the bug immediately.
Screenshots
If applicable, add screenshots, or videoshots to help explain your problem.
Versions

which version of the Flutter are you using?
3.10.6
which version of the FlChart are you using?
0.63

https://www.youtube.com/watch?v=pTabWoiwNWY


Laravel auth (getDateFormat() error)

Hello, I am using the native Laravel auth (from Laravel 5.2) and I am getting the following error:
BadMethodCallException in Model.php line 3001:
Call to a member function getDateFormat() on a non-object (null)
in Model.php line 3001
at Model->getDateFormat() in Model.php line 2921
at Model->fromDateTime(object(Carbon)) in Model.php line 2878
at Model->setAttribute('updated_at', object(Carbon)) in Model.php line 3420
at Model->__set('updated_at', object(Carbon)) in Model.php line 1771
...
Any ideas on where to fix?
Thanks,
David

I finally fixed that using the CammoKing pull. I had to fix some issues with the MongoDBAuthServiceProvider file that was renamed to AuthServiceProvider. And after using the following auth provider:
'users' => [
'driver' => 'mongodb',
'model' =>  Jenssegers\Mongodb\Auth\UserModel::class,
'table' => 'users',
]
The register works, login and logout works. Now I'm having issues with the reset password.
UnexpectedValueException in PasswordBroker.php line 238:
User must implement CanResetPassword interface.

Same issue here, but its fun !

For Laravel 5.3
Call to a member function getDateFormat() on nul
In the file vendor/laravel/framework/src/Illuminate/Foundation/Auth/User.php
Replace use Illuminate\Database\Eloquent\Model; for this
use Jenssegers\Mongodb\Eloquent\Model as Model;
Its Works

It is not very good idea to modify files in vendor folder. Instead you should fork package or just copy that class somewhere in your app directory, modify it for your needs and extend your user model by this copied class.


Async support

Description of the feature request:
Since we are calling an external API, adding async support will help improve latency in applications.
What problem are you trying to solve with this feature?
Calling embeddings and text generation asynchronously
Any other information you'd like to share?
No response

Added in Gemini - package version 0.3.0


FLEX-2737 ~ Initial implemenation of timestamp in EventNotificationRequest.

SonarQube analysis reported 6 issues
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 DeviceManagementService.java#L96: Method has 8 parameters, which is greater than 7 authorized.
 DeviceManagementService.java#L160: Method has 9 parameters, which is greater than 7 authorized.
 OslpChannelHandlerServer.java#L130: Remove those useless parentheses.
 OslpChannelHandlerServer.java#L184: Merge this if statement with the enclosing one.
 RegisterDevice.java#L417: Remove those useless parentheses.
 OslpChannelHandlerServer.java#L135: Complete the task associated to this TODO comment.

All is well.

SonarQube analysis reported 2 issues
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 DeviceManagementService.java#L96: Method has 8 parameters, which is greater than 7 authorized.
 DeviceManagementService.java#L160: Method has 9 parameters, which is greater than 7 authorized.

All is well.

SonarQube analysis reported 2 issues
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 DeviceManagementService.java#L96: Method has 8 parameters, which is greater than 7 authorized.
 DeviceManagementService.java#L160: Method has 9 parameters, which is greater than 7 authorized.

All is well.

SonarQube analysis reported 2 issues
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 DeviceManagementService.java#L103: Method has 8 parameters, which is greater than 7 authorized.
 DeviceManagementService.java#L167: Method has 9 parameters, which is greater than 7 authorized.

All is well.

@rlemmers your review comments have been processed.

SonarQube analysis reported 2 issues
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 DeviceManagementService.java#L103: Method has 8 parameters, which is greater than 7 authorized.
 DeviceManagementService.java#L167: Method has 9 parameters, which is greater than 7 authorized.

All is well.

Looks okay to me.


Solve bug for adding interview

Codecov Report

Merging #50 (5aed570) into master (dde4ed1) will decrease coverage by 0.09%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master      #50      +/-   ##
============================================
- Coverage     55.06%   54.96%   -0.10%
  Complexity      423      423
============================================
  Files            94       94
  Lines          1718     1721       +3
  Branches        203      205       +2
============================================
  Hits            946      946
- Misses          710      713       +3
  Partials         62       62

Impacted Files
Coverage Δ

...du/address/logic/commands/AddInterviewCommand.java
0.00% <0.00%> (ø)

.../java/seedu/address/model/interview/Interview.java
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dde4ed1...5aed570. Read the comment docs.


can't remove Marker cluster

I'm submitting a ...  (check one with "x")

[ ] question
[X ] any problem or bug report

OS:  (check one with "x")

[ ] Android
[ ] iOS
[X ] Browser

cordova information:  (run $> cordova plugin list)
com.googlemaps.ios 2.7.0 "Google Maps SDK for iOS"
cordova-plugin-console 1.1.0 "Console"
cordova-plugin-googlemaps 2.4.6 "cordova-plugin-googlemaps"
cordova-plugin-whitelist 1.3.3 "Whitelist"
insert the output from the command here
Current behavior:
Cluster are not removed, the cluster icon do not disappear and same behavior for marker inside the cluster
Expected behavior:
Cluster are removed
Related code, data or error log (please format your code or data):
No error inside console, I have used this project https://github.com/mapsplugin/marker_cluster_demo

Please share "YOUR" project files on GitHub.
https://github.com/mapsplugin/cordova-plugin-googlemaps/blob/master/src/browser/PluginMarkerCluster.js#L424-L437

https://github.com/livedota/demo-Claster.git

You just copy the files... That doesn't make sense.
I need to confirm how reproduce the problem on your actual code.
Since you say 'remove method does not work on browser', please deploy the files on GitHub as GitHub page
Like this repo.
https://mapsplugin.github.io/HelloGoogleMap/
https://github.com/mapsplugin/HelloGoogleMap
https://github.com/mapsplugin/HelloGoogleMap/blob/master/package.json#L22
If you don't know how to host HTML files on GitHub, check out here. Super easy.
https://pages.github.com

https://livedota.github.io/demo-Claster/ just changed some line to make it work, i haven't used the script beacuse i'm not on linux

Sorry for late response. I was really busy.
I fixed this bug. Please reinstall the plugin from the multiple_maps branch.
$> cordova plugin rm cordova-plugin-googlemaps

$> cordova plugin add https://github.com/mapsplugin/cordova-plugin-googlemaps#multiple_maps --variable ...


Expose create a query from a user input AST.

The use case I have for this is described in the doc comment, i.e. I use the raw literals before tokenization to fetch auxiliary information from systems outside the index.
Currently, this means that I have to parse the query twice, once in QueryParser::parse_query and then again using tantivy_query_grammar::parse_query to extract the raw literals. With this change, I can pass on the UserInputAST which I used to collect the raw terms to the QueryParser to avoid the redundant work.
Theoretically, since a UserInputAST is completely independent of a given index, it could also be used to query multiple independent indexes.

Codecov Report

Merging #2039 (f1a275a) into main (ba3a885) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main    #2039      +/-   ##
==========================================
- Coverage   94.45%   94.44%   -0.02%
==========================================
  Files         319      319
  Lines       59428    59435       +7
==========================================
  Hits        56135    56135
- Misses       3293     3300       +7

Impacted Files
Coverage Δ

src/query/query_parser/query_parser.rs
94.61% <0.00%> (-0.54%)
:arrow_down:

... and 2 files with indirect coverage changes

thank you @adamreichold


export os/exec prefixSuffixSaver and friends

What version of Go are you using (go version)?
go version go1.6 linux/amd64

This is not a bug, but a feature request:
I am using os/exec and really like Output()'s ability to capture STDERR while keeping only the head and tail of it. However, I want to apply this to STDOUT as well, to capture my own number of bytes, and I want to have all this with my own Start() and Wait() calls.
For now I have done:
cmd := exec.Command(...)
cmd.Stderr = &prefixSuffixSaver{N: 4096}
cmd.Stdout = &prefixSuffixSaver{N: 4096}
cmd.Start()
...
cmd.Wait()
stdErr := cmd.Stderr.(*prefixSuffixSaver).Bytes()
stdOut := cmd.Stdout.(*prefixSuffixSaver).Bytes()

But since prefixSuffixSaver is not exported, I've had to just copy and paste all that related code in to my own package.
prefixSuffixSaver is generally useful and non-trivial, so it would be great if this functionally was exposed publicly.

prefixSuffixSaver is not within the scope of os/exec, so I don't think we
should
export it. If you find it useful, just copy it and (optionally) make a go
gettable
package.

Yes, I could do that.
My preference would be that the author find an appropriate golang package to put it in and export it from there, so it's built in to the language, esp. since the author has commented in the code that he might have a better way of implementing it in the future.

Sorry, I have no desire to export that code. I agree you should just copy it to a new repo and then you can maintain it instead of me. :-)


Conditional key binding

Hello, Is it possible to assign key bindings to a node (for example the code block) where the key bindings only fire off if the cursor is contained within said node? I'd like to enable the Tab key (4 spaces) only within the code block. Thanks for your input.

+1

+1 update? thanks

This code goes into CodeBlockHighlight.js
This could then even be used along side List Items Tab behaviour
keys({ type }) {
    return {
      'Shift-Ctrl-\\': setBlockType(type),
      Tab: (state, dispatch) => {
        const { selection: { $from } } = state // get current selection

        // compare the selected node type against the node type 'code_block'
        if ($from.node().type === type) {
          insertText('\t')(state, dispatch) // insert a tab or something different
          return true
        }

        return false
      },
    }
  }


YiedSAFE DOI leading nowhere?

https://github.com/euraf/AF_FAIRness/blob/0c13db06d1fba1240f2d9dc012808665a2b7e6c9/catalogue/tools/yieldsafe_v2.json#L8
This DOI does not leat to the tool... Needs an update

The Cranfield Repository had DOIs when this entry was added. Now they don't seem to have them anymore. Both YieldSAFE and FarmSAFE updated to the repo handler uri (commit https://github.com/euraf/AF_FAIRness/commit/e2a6d0f858244b3cce908d671a719e3a97e19485). (This makes me wonder how permanent are the permalinks like DOI)

this should be the link  to the model: https://dspace.lib.cranfield.ac.uk/handle/1826/22344