The "route:list" command displays/output all attached Middleware.

Laravel Version: 7.13.0
Laravel Sanctum: 2.3.3
PHP Version: 7.4.6

Description:
This issue/behavior is introduced on Laravel Version 7.13.0 and specific on commit (https://github.com/laravel/framework/commit/7ebd21193df520d78269d7abd740537a2fae889e).
The "route:list" command will output all the attached Middleware per route after installing Laravel Sanctum.
Steps To Reproduce:
laravel new app

List all routes

php artisan route:list

All Good and Nice :)

Now lets install Laravel Sanctum

composer require laravel/sanctum

List all the routes again

php artisan route:list

Related commit (https://github.com/laravel/framework/commit/7ebd21193df520d78269d7abd740537a2fae889e)

i am not using laravel/sanctum, but it happen after the composer update
Loading composer repositories with package information
Installing dependencies (including require-dev) from lock file
Package operations: 0 installs, 23 updates, 0 removals
  - Updating doctrine/cache (1.10.0 => 1.10.1): Loading from cache
  - Updating doctrine/inflector (2.0.1 => 2.0.2): Loading from cache
  - Updating doctrine/lexer (1.2.0 => 1.2.1): Loading from cache
  - Updating guzzlehttp/guzzle (6.5.3 => 6.5.4): Loading from cache
  - Updating voku/portable-ascii (1.4.10 => 1.5.1): Loading from cache
  - Updating vlucas/phpdotenv (v4.1.5 => v4.1.6): Loading from cache
  - Updating symfony/event-dispatcher-contracts (v2.0.1 => v2.1.2): Loading from cache
  - Updating symfony/service-contracts (v2.0.1 => v2.1.2): Loading from cache
  - Updating opis/closure (3.5.1 => 3.5.3): Loading from cache
  - Updating symfony/translation-contracts (v2.0.1 => v2.1.2): Loading from cache
  - Updating nesbot/carbon (2.34.0 => 2.35.0): Loading from cache
  - Updating monolog/monolog (2.0.2 => 2.1.0): Loading from cache
  - Updating league/flysystem (1.0.68 => 1.0.69): Loading from cache
  - Updating laravel/framework (v7.11.0 => v7.13.0): Loading from cache
  - Updating cakephp/chronos (2.0.4 => 2.0.5): Loading from cache
  - Updating laravel/horizon (v4.3.2 => v4.3.3): Loading from cache
  - Updating spatie/laravel-permission (3.11.0 => 3.13.0): Loading from cache
  - Updating backpack/crud (4.1.4 => 4.1.6): Loading from cache
  - Updating backpack/generators (v3.0.1 => v3.1.0): Loading from cache
  - Updating facade/ignition (2.0.2 => 2.0.5): Loading from cache
  - Updating laravel/telescope (v3.4.0 => v3.5.0): Loading from cache
  - Updating mockery/mockery (1.3.1 => 1.4.0): Loading from cache
  - Updating phpunit/phpunit (8.5.4 => 8.5.5): Loading from cache

i am not sure which packages contributing to this issue, just giving my output for whoever looking in this issue...
the middleware list become looooong expanded list...

I have the same problem..

This is fixed with https://github.com/laravel/framework/pull/32993 which will be in the next patch release.

@TiBianMod  @plai2010
do you have barryvdh/laravel-debugbar installed? removing that fixed it for me. Related issue:
https://github.com/barryvdh/laravel-debugbar/issues/1046

Thanks @JoshuaDoshua for sharing your experience. I didn't install barryvdh/laravel-debugbar however; I cannot find it in my composer.lock or vendor either.
You have Sanctum installed, and removing DebugBar helps? Just trying to nail down the scenario.

Hi @JoshuaDoshua, I don't use barryvdh/laravel-debugbar, but like I explained before, you can reproduce this issue by installing Laravel Sanctum after creating a new Laravel app and you will see the route:list what outputs.

Hi @JoshuaDoshua, I don't use barryvdh/laravel-debugbar, but like I explained before, you can reproduce this issue by installing Laravel Sanctum after creating a new Laravel app and you will see the route:list what outputs.

Yes, I agree with you. By installing laravel/sanctum, route:list comes with all route middlewares expanded. That is really annoying :)

I actually don't think #32993 is the same issue as this; I would like to see this issue re-opened.

any update on this issue?

Outputting all the middleware is intended.

I think it would be ideal to leave the format as it was before, and if you really want to show the current format, then do it through an optional parameter in the command. Like php artisan route:list --expand-middleware

Kind of inclined to agree with you there @leo95batista. You could always attempt a PR to see if Taylor would approve it.

Outputting all the middleware is intended.

I noticed something to that effect in the check-in history of some code when I was looking into it. I was puzzled however that Laravel without Sanctum did not expand the middleware classes.
Either way, the behavior seems inconsistent.

I have created a pull request to solve this problem, I hope it will be accepted

I have created a pull request to solve this problem, I hope it will be accepted

I have created a pull request to solve this problem, I hope it will be accepted

I have created a pull request to solve this problem, I hope it will be accepted

My pull request was rejected, I just have to wait for someone to give a solution...

having the same proble, it was solved after removing  barryvdh/laravel-debugbar

having the same problem. but you can filter the column with flag. php artisan route:list -c

I don't know how the maintainers managed to mess this one up!
@kenxu5500 gave the cleanest solution, but here's a cross-platform one-liner if you need to see all of the columns:
php artisan route:list | perl -pe 's/\x1b\[[0-9;]*[mG]//g' | perl -0777 -pe 's/ +\|(\r\n?|\n)/\n/g' | perl -0777 -pe 's/(\r\n?|\n)(\| +){6}/, /g'

to break it down:
php artisan route:list | \
perl -pe 's/\x1b\[[0-9;]*[mG]//g' | \  # strip control characters: https://superuser.com/a/561105
perl -0pe 's/ +\|(\r\n?|\n)/\n/g' | \  # strip trailing whitespace and pipe character, substituting newline with \0 during processing via -0
perl -0pe 's/(\r\n?|\n)(\| +){6}/, /g' # strip leading pipes and whitespace before middleware column, substituting newline with \0 during processing via -0

Remember to change the {6} portion if you change the number of columns with the --columns[=COLUMNS] option, see php artisan route:list -h.
I haven't tested it on Windows yet, but my understanding is that perl will match any line ending with (\r\n?|\n) and then substitute the platform-specific line ending when the \n replacement is written to the output stream.
I messed around with sed and a few other methods, but unfortunately there is no cross-platform multiline search and replace regex command that isn't burdened with quirks. So I've decided to always use perl moving forward since it's available in most environments.
A better method might be to use php artisan route:list --json | jq but I haven't figured out the magic incantation to output a table yet.
https://stedolan.github.io/jq/
Other workarounds:
https://stackoverflow.com/questions/62244648/laravel-route-list-php-artisan-routelist-displaying-middleware-on-new-lines


Buster deprecation: easyinstall.sh

This ticket is to track modifications to easyinstall.sh required for the Buster deprecation

At this moment I have identified one required update: The removal of the locking of clang to v11. This was required on Buster since it shipped with a too old version of clang (v7) as the default, and we had to specifically use the v11 package.
On Bullseye, v11 is the default, and on Bookwork the default is v14 which has no issues with our codebase in my testing.
PR that removes the locking to v11 and defaults to the distro default https://github.com/PiSCSI/piscsi/pull/1194

@rdmark Please take note of my changes to easyinstall in the issue_1179 branch. What about adding any required changes to easyinstall to this branch?

@uweseimet Normally I prefer to make smaller stand-alone changes in separate PRs. But since you already have changes in your branch I can discard my branch.
BTW, what do you think about locking to a newer version (such as clang-13 in your branch) vs. letting it float with the default for the distro (as in my branch)? I can see pros and cons, but my choice would be to let it float since it removes the maintenance overhead of managing the package and compiler version.

@rdmark If the default compiler works fine I agree that there is no need for using a newer version, unless it turns out that there is an issue with the old compiler.
In case you test with the default compiler, can you please also test whether the -flto option produces correctly working binares?

@uweseimet I tested Raspbian bullseye default clang (11.0.1-2+rpi1) with -flto using your work branch. The resulting piscsi and scsictl binaries seem to work well, although I did only cursory dynamic testing.
I'll test on Raspbian bookworm once it's released.

@rdmark Is there still anything to be done for this ticket, or can we close it?

Nothing else that I can think of.


feat: Add Heurist API Integration as New Model Provider

Relates to:

Risks
Low - This change adds a new model provider integration (Heurist) with proper API key handling and follows existing patterns for model providers.
Background
What is Heurist?
Heurist is a DePIN (Decentralized Physical Infrastructure Network) protocol for AI inference that:

Aggregates compute resources from GPU owners and data centers to provide cost-effective API access to open source models

Reduces costs by 60-95% compared to closed source alternatives

Has processed over 1 billion AI inference requests

Has 11k unique addresses contributing GPUs since April 2024

Powers multiple production applications including:

Heurist Imagine (image generation)
Pondera (chatbot)
Heurist Search (AI search engine)
Integration customers: GenLayer, Chasm Network, Haptic AI, DecentAI

Partners with key infrastructure providers like Aethir, Exabits, Spheron, Gateway Network

What does this PR do?
This PR adds support for the Heurist API as a new model provider. Specifically:

Adds HEURIST_API_KEY configuration
Implements Heurist model provider integration
Adds Heurist model configurations
Updates API key handling logic to support Heurist provider
Adds model settings configuration including temperature, tokens, and other parameters

The integration allows us to leverage Heurist's cost-effective infrastructure for accessing open source LLMs while maintaining our existing architecture patterns.
What kind of change is this?
Features (non-breaking change which adds functionality)
Documentation changes needed?
My changes require a change to the project documentation to document:

New HEURIST_API_KEY configuration option (free API key can be applied here )
Available Heurist models and their capabilities
Model configuration parameters

Testing
Where should a reviewer start?

Review the model configuration in packages/core/src/models.ts
Check API key handling in .env.example
Review the generateText implementation for Heurist in packages/core/src/generation.ts

Detailed testing steps

Configure HEURIST_API_KEY in environment
Test text generation using configured model by pnpm start

Screenshots
N/A - Backend API integration changes only
Deploy Notes

Requires setting up HEURIST_API_KEY in production environment
No database changes required

This is awesome, would you mind adding an "integrations" section and docs with all the Heurist info and stuff (like an integrations dir)? Basically whatever is clean and helps users understand the value prop, get setup and onboarded


Project Recovery and Autosave Feature.

Task1 -
Project Recovery not working:-
Currently, the project is recovered only on page-reload but it is not working.
Just page reload is not enough. The user might accidentally close the browser tab or Power Off.
in simulator/src/project.js
window.onbeforeunload = function () {
	...
};

Task2 -
Issue #34
AutoSave feature for CircuitVerse Simulator.
Autosave feature will automatically save the project offline with every update made to it.
Task3 -
When the user opens a project :-
1. offline - It will keep autosaving as before (maybe we can prompt the user to save it online in intervals)
Once user saves it online - we remove that project from local storage.
2. online - We check if there is a more recent update of that project saved offline.
We prompt the user that there are recent unsaved edits made to this project.
Difference between Recovery and Autosave

Saving the project in local storage under the key 'recover' until the project is first saved online/offline manually by user. Once project is saved online/offline.

Then autosave feature will work for those projects whose instances are already present in local storage. Autosave will keep updating the offline saved instance of that project in local storage.

Recover Project will work only for the currently opened unsaved(offline and online) project. All previous recover projects will be cleared.

Autosave will work for all projects whose instance is already present.

@tachyons @gr455 @ayan-biswas0412 @abstrekt  can you please suggest any changes you feel necessary.
Are you working on this? (Yes/No)
Yes, already solved Task 1 and 2 according to the discussion in slack.

@devartstar great research go ahead 🎉

@gr455 As of now I planned to autosave offline only.
But if we want to autosave online we can do it in this way:-
In the autosave logic - there are two intervals small and big.

We can autosave online every one big interval (5 mins...can be changed)
autosave offline if it passes the check (changeCounter > changeCounterLimit) after every small interval.
We can alter with the three variables smallInterval, bigInterval, changeCounterLimit to get a proper combination.

Calling save online every small interval might lead to Rate Limit Exceeded.
Is it possible to know the number of requests that can be made per minute per user.

@gr455 As of now I planned to autosave offline only. But if we want to autosave online we can do it in this way:-
In the autosave logic - there are two intervals small and big.

We can autosave online every one big interval (5 mins...can be changed)
autosave offline if it passes the check (changeCounter > changeCounterLimit) after every small interval (5 sec...can be changed)
We can alter with the three variables smallInterval, bigInterval, changeCounterLimit to get a proper combination.

Calling save online every small interval might lead to Rate Limit Exceeded. Is it possible to know the number of requests that can be made per minute per user.

Extremely sorry for such a late reply. Yes, you can know the limits https://github.com/CircuitVerse/CircuitVerse/blob/6362a9eb4a9fd6b5391c8e4ef0e3b82525e722de/config/initializers/rack_attack.rb#L35


[jss-cli] expose jss options

jss createStyleSheet currently has named: bool but the jss cli has no way of using it.

Thats because jss-cli isn't using jss js api. Its output is jss json api. You can use after that js api with any options you want.


SAM not working.

I was using SAM for a while now but yesterday it just went completely blank randomly. It only shows spacewar and nothing else. What can I do with this? I read that maybe the servers are down and if that's the case how long is it to make them work again? I wanna use the program again.

For the time being you can search for your game on (SteamDB.info)[https://steamdb.info/] and use the App ID from that game's page to manually add it in SAM.

Thank your so much for your help!

See #152.


Can libdispatch/u subdirectory be removed?

Under libdispatch we have a sub directory called "u":
/home/ed/tmp/netcdf-c/libdispatch/u:
 total used in directory 80 available 53602068
 drwxrwxr-x 2 ed ed  4096 Nov 12 07:36 .
 drwxrwxr-x 4 ed ed 12288 Dec  3 04:23 ..
 -rw-rw-r-- 1 ed ed 29959 Nov 12 07:36 utf8proc.c
 -rw-rw-r-- 1 ed ed 29500 Nov 12 07:36 utf8proc.h

It is not used in the build.
Can this directory and contents be removed?

@DennisHeimbigner I believe this is something you created; any reason not to remove it?

These were I think the old versions of utf8proc. They can be removed.

I'll roll this into the next pull request I process whatever it may be. Thanks Dennis!


Implement FromValue trait for byte arrays.

(small nit on the side and I feel bad for commenting on formatting, but could you check in a .rustfmt.toml that matches your style? I accidentally ran fmt on the repo by reflex... :hide: but I would also feel more confident in the consistency of my formatting if I could leave it to the robots)

hey, we use stock rustfmt, and the CI shouldn't pass without rustfmt, so go ahead :)
could you please add a small unit test somewhere, please? otherwise lgtm :)
Thanks!

hey, we use stock rustfmt, and the CI shouldn't pass without rustfmt, so go ahead :)

Apologies and glad to hear I was wrong. I was suffering from a case of nested workspace syndrome :hide:

could you please add a small unit test somewhere, please? otherwise lgtm :)

:+1: I didn't find a fitting place, so I added a new one. Happy to move it around.
Thanks

thanks!


stream::ForEach can no longer block indefinitely

This PR removes the loop from stream::ForEach. Instead, the future
makes at most 1 attempt to poll its interior, and if it can make more
progress, it immediately calls wake_by_ref(), then returns Pending.
In this way, under a fair executor, it can't accidentally starve the
task queue.

I'm skeptical of this approach-- see my the related discussion between myself, @jonhoo , and @carllerche in this thread: https://github.com/rust-lang/futures-rs/pull/2049#issuecomment-577780571
I also anticipate that this PR would have significant negative performance implications if we adopted this approach throughout the ecosystem: yielding back up to the top-level executor with every step seems like it would interact poorly with locality, and could be very inefficient depending on the top-level executor's queuing implementation.


Performance

Thank you for your work on this library.
Would you have any feedback on the performance difference in Postgres DB for primary key on a table with 1 million records for read, write sorting between:
BIGINT
UUID v7
ULID

I have no idea about Postgres' implementation, but I don't see a plausible reason why the read/sort performance differs between UUIDv7 and ULID as long as they are stored in the 128-bit binary form because the proposed UUIDv7 produces a quite similar byte sequence to ULID.
As for the write performance, UUIDv7 and ULID have several flavors in monotonicity guarantee, and the performance characteristics vary depending on the throughput of random number generator and the cost of mutex and other synchronization mechanisms. But if you only need one million records, then such a difference will be negligible (unless you wish to generate one million IDs every second).


Update core components

In this PR we update base and core-java dependencies. The new version is 1.0.0-pre6.
Changes:

getter style (from getSomething() to something());
removed deleted options (events) and (rejections);
fixed some Javadoc warnings.

@armiol, PTAL.


telegram bot not sending?

hi
when i click "test send message" the image* arrives in telegram chat ✅
but when i generate images it doesn't send automatically 🤷
these are my settings:

am i missing something?
*but the prompt is missing

*generate forever is support

generate image with batch size 1 or more is support
now discord is support. maybe just update try try~

hi
i use forge webui (installed with stability matrix.) ✅
i tested discord bot as well now. like telegram bot it worked with "test send". I enabled "SD INFO" so the prompt is included. ✅ (but "text-prompt" checkbox doesn't work. And "PNG INFO" only includes filename: ▣ImgFile-Info:output\txt2img-images\2024-08-20\00112-300339626.png)
but i still can only trigger manually. i cannot get these to work: 2.1 SD-Image-generated count or 2.2 Timer-Countdown

cheers

2.1 its most important function. Are u say this function works on discord but telgram wont trigger?
2.2 NEED click start timer button.

both telegram and discord don't trigger with 2.1 and 2.2.
i can only manually trigger (test send).

okay, i got it. forge not imprelement _js as method, its seems like autom1111 does.
updated.

ComfyUI online....
Auto prompt by LLM and LLM-Vision (Trigger more details out inside model)
sd-web-ui: https://github.com/xlinx/sd-webui-decadetw-auto-prompt-llm
ComfyUI: https://github.com/xlinx/ComfyUI-decadetw-auto-prompt-llm
Automatic1111 | Forge extension (LINE | Telegram | Discord)
sd-web-ui:https://github.com/xlinx/sd-webui-decadetw-auto-messaging-realtime
ComfyUI: https://github.com/xlinx/ComfyUI-decadetw-auto-messaging-realtime


App crashes on Android 6.0.1 on Samsung S7 when run in release version but works on dev version

Is this a bug report?
Yes
Have you read the Installation Instructions?
Yes
Environment
react-native: 0.59.4
react: 16.6.3
react-native-maps: 0.24.0
Samsung Galaxy S7
Android 6.0.1
Steps to Reproduce

Create react native app.
Put a map view inside of a view.
Generate a release APK.
Run it on a Samsung S7 device with Android 6.0.1

Expected Behavior
Working like it would work on develop mode (works).
Actual Behavior
Crashes immediatly when run on the screen
with this:

or with an out of memory error:

Reproducible Demo
/

Same, PlayStore gives me some memcpy and tgkills backtrace on why the app restarted.

Fwiw, I just tested a signed APK with a Galaxy S7 running Android 8.0 and didn't come across this issue.
I'll try to get my hands on a device running Android 6 to test further.

Fwiw, I just tested a signed APK with a Galaxy S7 running Android 8.0 and didn't come across this issue.
I'll try to get my hands on a device running Android 6 to test further.

Tried it on an Galaxy S7 as well with Android 7.0 and din't get the error either.

I think this is related to last RN versions (58 & 59). They added two CPU architecture type (64 bit cpu).
Looks similar: https://github.com/facebook/react-native/issues/24261

@Merikw could you test it on RN 0.59.10? it should be fixed by new JSC

reopen this issue facing the same problem with my s5.

reopen this issue facing the same problem with my s5.

Seems to still be an issue. Device model is Galaxy J7 Prime


Add dependency on Emacs 26.1

This adds an explicit dependency on Emacs 26.1. According to package-lint-current-buffer in go-mode.el:
1930:18: error: You should depend on (emacs "26.1") if you need `make-nearby-temp-file'.
1956:43: error: You should depend on (emacs "26.1") if you need `file-local-name'.
1960:36: error: You should depend on (emacs "26.1") if you need `file-local-name'.
2004:46: error: You should depend on (emacs "26.1") if you need `file-local-name'.

There is also a dependency on xref, which was first included in Emacs 25.1. Thanks.
EDIT: I also see that you now intend to depend on Emacs 26.1 (https://github.com/dominikh/go-mode.el/commit/32cbd78c0af29837ace3db04a224d6d01ec6851e).

Thanks.


Added New Features Cleaned Old Code

This pull request:

Removes deprecated Predict function
Makes some dependencies optional
Makes Brain_Data iterable

Coverage decreased (-3.5%) to 33.367% when pulling 7733f5154ee0f9cfd4d7a522eef7944f99809378 on dev into 1803ed3c192de0e4730918b9d6dc9e2335d145c5 on master.

this pull request resolves #52, #49, #25,


This library is copyright Google

Why?

Thanks for the info @willnorris. I remember now those terms of the Google contract. They really got ahead of the law there, though I believe the same to be true of any open source work performed while under similar employment contracts—I'm just not aware of any precedent set yet (maybe you know?)
Anyway, thanks for creating this library. If you're able to dual license this in the future and release the copyright to GPL or WTFPL please do comment here so I may pursue using this library.
Cheers.

Thanks for the info @willnorris. I remember now those terms of the Google contract. They really got ahead of the law there, though I believe the same to be true of any open source work performed while under similar employment contracts—I'm just not aware of any precedent set yet (maybe you know?)
Anyway, thanks for creating this library. If you happen to know of a similar work written in Go and not copywritten by Google I'd love to get my hands on it.
Cheers.

I'm just curious, why does the Google copyright matter?  It's still under an open source license, so your rights as a user of the library are exactly the same.
(Not saying you don't have your reasons, I'm just curious what they are.)

I disagree with the GNU's definition of free software. To me free software is simply one which is free of burden, thereby maximizing its potential for reuse. Burden, in this sense, means software which is difficult to use because of onerous licensing requirements (like the GNU) or limitations of warranty and liability. But that's just my turn off with the BSD. Google is a different matter entirely.

I disagree with the GNU's definition of free software. To me free software is simply one which is free of burden, thereby maximizing its potential for reuse. Burden, in this sense, means software which is difficult to use because of onerous licensing requirements (like the GNU) or limitations of warranty and liability. But that's just my turn off with the BSD and the like. Google is a different matter entirely and one I've been concerned about since the day Google purchased DoubleClick.


Fix #194 : Split implementation of IConfigurationBuilder and IConfiguration

cc @Eilon, @divega, @muratg, @troydai

:shipit: after the small comments.

:shipit: despite my different opinion on whether to throw exception, i found it is not significant enough to block this PR.


M5Display::setBrightness will it be implemented ?

Hello,
in M5Display.cpp I see that various functions including M5Display::setBrightness and M5Display::sleep are not implemented (wrapped in an "#if 0" cpp directive). Is there any hope for those to be implemented ?
Thank you very much!
giovanni

Great. That works. Thank you very much!


Filament - Fix gltf viewer build on macOS

Jira ticket URL (Why?)
N/A
Short description (What? How?) 📖
Filament's build system is somewhat flaky. It sometimes builds gltf viewer fine, but sometimes it fails to build on macOS. This PR should fix these build failures on macOS.
On macOS >= 11, we convert file extensions to UTIs for showing the file open dialog. This wasn't available on previous versions of macOS, so only a string conversion is needed. The choice between the two is done at runtime, but the UTI-based function cannot be built with previous SDKs, so an unresolved external issue pops up.
Testing
Design review 🎨
N/A
Affected areas 🧭
Building gltf viewer on macOS
Special use-cases to test 🧷
N/A
How did you test it? 🤔
Manual 💁♂
It builds on every try

Note: googling just UTI is not a good idea...

I can confirm that googling UTI is not a good idea.


CSS overflow-y: auto drags container rather scroll in Firefox

I just added more content to one of the Demo's widget. I see a y scroll bar.
I can't scroll content, it is triggering drag and whole is dragging.

@venkata82 This is actually normal behavior. As you can see, you may drag any of the panels by clicking and dragging anywhere on the panels. However, you may set a specific drag handle so that only a particular place will be draggable.


Remove VS Code tasks.json to run all tests in parallel

VS Code now has a command to run all tests in parallel, so we don’t need this anymore.

@swift-ci Please test


Unable to establish WebSocket connection with separated code

I am experiencing an issue when trying to separate my WebSocket code into different parts.
The initial code, where everything is in a single block, works fine.
import { Elysia, t } from "elysia"

const app = new Elysia()
  .group("inbox", (app) =>
    app.ws("join", {
      body: t.Object({
        inboxId: t.String(),
      }),
      message(ws, { inboxId }) {
        ws.send(inboxId)
      },
    })
  )
  .listen(3000)

However, when I split the code into separate parts, the WebSocket connection to ws://localhost:3000/inbox/join fails with the error message: WebSocket connection to 'ws://localhost:3000/inbox/join' failed.
const ws = new Elysia().group("inbox", (app) =>
  app.ws("join", {
    body: t.Object({
      inboxId: t.String(),
    }),
    message(ws, { inboxId }) {
      ws.send(inboxId)
    },
  })
)

const app = new Elysia().use(ws).listen(3000)

Elysia Version: 0.7.29
Bun Version: 1.0.13
Thank you for your assistance, and I apologize if I am using the framework incorrectly. Your guidance is highly appreciated.

I ran into this issue too, I think currently websocket connections only work with one segment at the root of your application. This probably needs to go into a feature request (and should really be documented on the website).

I've been encountering this issue as well, and came to report it after testing. It seems like WebSocket connections don't work as intended when behind a .group that comes off another instance of Elysia than the root instance.
Testing this URI: ws://localhost:3000/group/ws
Works:
const websocketApp = new Elysia().ws("/ws", {});
const app = new Elysia().group("/group", app => app.use(websocketApp)).listen(3000);

Doesn't work:
const websocketApp = new Elysia().group("/group", (app) => app.ws("/ws", {}));
const app = new Elysia().use(websocketApp).listen(3000);

Notice how the .group is simply moved to the other instance.
The server ends up responding with a 400 Bad Request.

this still happens, related issue https://github.com/elysiajs/elysia/issues/553
it's really confusing too since the client console just logs this:

This issue have been fixed by previous subsequent patches of Elysia as it's now unable to reproduce the bug in the latest version of Elysia (1.1.9 by the time of writing) with the following code:
import { Elysia, t } from '../src'

const ws = new Elysia().group('inbox', (app) =>
	app.ws('join', {
		body: t.Object({
			inboxId: t.String()
		}),
		message(ws, { inboxId }) {
			ws.send(inboxId)
		}
	})
)

const app = new Elysia().use(ws).listen(3000)

Feel free to reopen the issue if the bug still persist, thanks


[C-Api/pipeline] clear element callback

Clear element callback before changing the pipeline state - receiving the EOS event.
Fix element lock when an error occurs in internal process.

Signed-off-by: Jaeyun<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #162. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.

@gichan-jang plz check this changes.

:octocat: cibot: @jaeyun-jung, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nnstreamer-api/ci/repo-workers/pr-checker/162-202204252003200.7461268901825-0858edb64f007c217ae9f108399bae77b205a752/.


refactor: rename references from hacbs-contact to enterprise-contract

The EC team is removing the hacbs name from the quay org used. This update also updates the tag used for the ec task to the latest.

Closing as it only changes version that are no longer in use


Transition to supporting JAWS with Chrome rather than Firefox

Due to the performance of Firefox Quantum with JAWS 2018, we would like to transition to support JAWS with Chrome because Chrome is much much faster with JAWS. We should spot check JAWS with Chrome to make sure that there aren't too many odd issues on that platform. Then, if all seems OK, we should update the QA documentation to reflect the change.

Here are the sims I would like to test with Latest chrome and JAWS:

[ ] john-travoltage
[ ] balloons-and-static-electricity
[ ] resistance-in-a-wire

I am going to do some testing on my machine to see how it all sounds on Chrome, but it would be very helpful to see what others find too. @ariel-phet could you please assign priority to this issue?

Notes from @jessegreenberg testing (I will edit this comment with things I find):

I noticed that when reading with the virtual cursor PhET's focus highlight doesn't appear. But once an element is clicked it appears. But I found that this is happening in FF now too. So by default JAWS is no longer synchronizing DOM focus with virtual focus when reading.

In https://github.com/phetsims/a11y-research/issues/105, it was mentioned that Chrome does not (and probably will never) support MathML, if that impacts the decision here.
But the positive experience I am seeing in https://github.com/phetsims/QA/issues/217#issuecomment-436455512 makes me think that we are much better off supporting Chrome.

I feel confident now that this is the correct transition.
But over slack, @KatieWoe said

I know I accidentally tested with chrome instead of firefox once and saw some pretty odd issues.

@KatieWoe if you have any specifics could you please list in this issue? No worries if you don't remember. It would still be helpful for someone to double check the list in https://github.com/phetsims/QA/issues/217#issuecomment-436455512 if there is time.
While testing, JAWS released version 2019 last month so please make sure that JAWS is up to date on the versions that are being tested.
Removing my assignment.

@jessegreenberg If I recall correctly, It appeared that text was being highlighted as the scene description was read.  No such text was visible (which is good).  I may be remembering NVDA instead of Jaws though.  I've been doing some other testing, so I don't know if it is happening with Jaws.  There might have been a few other things that I don't remember.

@phet-steele is this some testing you are also available for?

We already got good testing in https://github.com/phetsims/QA/issues/219, and I don't think we will get much more out of spending time on john-travoltage because teh a11y components are so similar to RIAW. We are going to proceed with supporting Chrome with JAWS.

@KatieWoe I updated places in QA documentation that refer to this in the above commits. The only other place that I could find was tester-book.tex, does that need to be updated too? Looks like almost same content as the qa-book.md so I was wondering that was generated or something. Is there anything else we need to document for this change?

That document is the old version of the QA book that would be turned in to a pdf.  It does not need to be updated as we are switching to the version that you updated.  The issue templates would also need to be updated, as I believe they list the supported platforms.  I'm not sure though, so I can make the checks and updates there myself.  Thanks @jessegreenberg

OK, thanks @KatieWoe! In that case I think this issue can be closed.


i have not adapter_config.json

I have every checkpoint model ,but I have not adapter_config.json

add "model.save_pretrained(output_dir)" create adapter_config.json and adapter_model.bin


[Feature]-[445]:Menu Button widget - dynamic Menu items

Is there an existing issue for this?

[X] I have searched the existing issues

Summary
I'd like to have an option to add the menu options dynamically, using JS.
Why should this be worked on?

Use case -
I am trying to create an action column to set the status of the current row. I had to manually add all the items and the onClick of each one and it is really time-consuming. It would've helped if I could use my JS snippet to generate these.

+1, would be greate to have it.
I tried to somewhat reverse-engineer the js objects behind the curtains. I first only tried to update a single MenuButton widget. I printed all the records in the MenuButton1 object, and tried to find and update all related records in it. However for some unknown reason, the changes are not updated on the UI.
Let emphasise, that is still for a MenuButton widget, not a MenuButton widget integrated in a Table, which actually has a completely different structure.
updateMenuButton1: async () => {
  MenuButton1.menuItems["menuItem2"] = {
        "label": "Mö",
        "id": "menuItem2",
        "widgetId": "",
        "isVisible": true,
        "isDisabled": false,
        "index": 1
      };
  MenuButton1bindingPaths["menuItems.menuItem2.isVisible"]  = 'TEMPLATE';
  MenuButton1bindingPaths["menuItems.menuItem2.isDisabled"]  = 'TEMPLATE';
  MenuButton1bindingPaths["menuItems.menuItem2.label"]  = 'TEMPLATE';
  MenuButton1reactivePaths["menuItems.menuItem2.label"]  = 'TEMPLATE';
  MenuButton1reactivePaths["menuItems.menuItem2.label"]  = 'TEMPLATE';
  MenuButton1reactivePaths["menuItems.menuItem2.label"]  = 'TEMPLATE';
  MenuButton1reactivePaths["menuItems.menuItem2.onClick"]  = true;
  MenuButton1validationPaths["menuItems.menuItem2.isVisible"]  = { "type": "BOOLEAN"};
  MenuButton1validationPaths["menuItems.menuItem2.isDisabled"]  = { "type": "BOOLEAN"};
  MenuButton1validationPaths["menuItems.menuItem2.label"]  = { "type": "TEXT"};
}

Job stories -

When working with dynamic data like tables or lists, I want the menu options to be dynamic so that I can only show specific actions.
When adding dynamic menu options, I don't want to transform query data so that I don't have to learn something new.

This looks achievable actually, within the menu button we can add a parent field for dynamic. When dynamic is selected, similar to List we allow user to enter the source data which can be linked to any query like the list widget.
The field config will show a generic menu button where the user can add the text for label by referencing currentItem.
To allow onClick to be dynamic we need to expose a selectedButton property which will have all the details like List.selectedItem. Will work with design on a POC.
CC @Nikhil-Nandagopal

Stats

Stat
Values

Reach
445

Effort (months)
0.5

This would be great as this could be used for headers on pages with menu buttons to control what the logged in user has access to instead of manually updating each page I could use this to get the information from the database and populate through JS function.  At least until you have multi page widgets working.

Adding PRD here -
Job stories -

When working with dynamic data like tables or lists, I want the menu options to be dynamic so that I can only show specific actions.
When adding dynamic menu options, I don't want to transform query data so that I don't have to learn something new.

Acceptance criteria

People should be able to select menu button type as static or dynamic
When static is select the menu button behaves exactly as it does today.
When dynamic is selected they should bind a source data of arrays (Tolu to check if only string arrays will work or only arrays with objects)
The menu buttons in dynamic has all properties same as static menu button
Binding in the dynamic menu button can be done by referencing currentItem
There is a max no. of dynamic menu buttons that can be added (Andrew to update us on the limit)
Standard flow like labels and onClicks work dynamically as expected.

Scope of widgets

Menu button
Table column > Menu button
Button group > Menu button

Action items before picking -

@andrewdietekoki @Tooluloope @vivekverma2312  please read the above items
@andrewdietekoki please add the link to figma below
@andrewdietekoki @Tooluloope please discuss point 6 in acceptance criteria and update in the thread
@andrewdietekoki make sure you discuss with @vasanth-appsmith on the updated property pane in the widgets and propose newer versions of property pane.

Figma Design: https://www.figma.com/file/Dwng9laAPen5uNYArmMt9z/Menu-Button-Widget---Add-Dynamic-Menu-Items?node-id=222%3A2798

For the limit of Max No. of Dynamic Menu Button items, it can be quite tricky. But I would suggest a maximum of 10 items.
Though we could give a number based on usage by users.

Hey Team,
is there an ETA when this function will get implemented?

Hey @vmatt we are planning to pick this up soon in the next few weeks. We will keep you posted

@dhruvikn can you create a new issue for doing the same change in button group?

Two more issues for adding dynamic menu items inside the table widget (#17955) and in the button group widget have been created (#17956).


How to configure offline persistence for firestore

please tell me how to make my app to save downloaded data and load in offline  for firestore flutter .

Isn't this the default for firestore?

@zoechi sorry. I commented it by mistake .Do you know how enable offline capability for firestore. ?. There is great documentation for web . but didn't find for flutter .

@dileepxdn Offline persistence is enabled by default for android and iOS [and so the same applies for the flutter plugins that use those native sdks].
From: https://firebase.google.com/docs/firestore/manage-data/enable-offline#configure_offline_persistence

For Android and iOS, offline persistence is enabled by default. To disable persistence, set the PersistenceEnabled option to false.

@xqwzts I have designed an app using flutter framework and used Firestore plugin and my app doesn't store data it loaded when I opened the app. when hit flight mode and restart app then the data is not showing.

Hi, I have the same issue here. As the persistent is enabled by default, but I try to add data to subcollection and it not working. So first I run the app with internet connection (adding data works) and then use flight mode -> adding data not working.
And I try to explicit call Firestore.instance.enablePersistence(true) and still not working.
Flutter Doctor :
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel beta, v0.7.3, on Linux, locale en_US.UTF-8)
[✓] Android toolchain - develop for Android devices (Android SDK 27.0.3)
[✓] Android Studio (version 3.1)
[✓] VS Code (version 1.26.1)
[✓] Connected devices (2 available)
• No issues found!
here the library i use :
firebase_core: ^0.2.5
firebase_auth: ^0.5.20
cloud_firestore: ^0.8.0
firebase_messaging: ^2.0.0

Same issue

@apinprastya @joaovic  It seems to work by default for me. If I put the phone into flight mode and add/update data then it changes it locally but the future never resolves which might give the appearance of it not working. When I turn flight mode off it will resolve the future and update the firestore db though sometimes it takes half a minute.
I am using cloud_firestore 0.8.1 and tested on Flutter dev (v0.10.0) and beta (v0.8.2).

@apinprastya @joaovic It seems to work by default for me. If I put the phone into flight mode and add/update data then it changes it locally but the future never resolves which might give the appearance of it not working. When I turn flight mode off it will resolve the future and update the firestore db though sometimes it takes half a minute.
I am using cloud_firestore 0.8.1 and tested on Flutter dev (v0.10.0) and beta (v0.8.2).

Same issue for me.  await Future never resolves. I'm using cloud_firestore 0.7.4.

Same issue for me, the Future will resolve only when connection it's back. In my case I'm trying to store data. Any update about this issue?

@bygrace1986 : checking on my side the future will never resolve if the device is offline. And checking on other bug about firestore on Android library, it is intended behavior.

@bygrace1986 here the link

@apinprastya Thanks for the link. I looked through the API docs and couldn't find any explanation about what the future represents.  I just assume that the local write always works and is synchronous. Honestly I'm not sure what I would do if the remote write failed because I assumed that it would retry till it worked. A bit of documentation would go a long way.

Hello, I just tested with: cloud_firestore: ^0.9.13
When I create new documents in offline mode, then they're not saved to firestore after coming back online.
It only works when working in online mode.
Any ideas what could be wrong?

_configureFirestoreOfflinePersistence() {
Firestore.instance.settings(persistenceEnabled: true);
}

In case in 2019 who's looking for persistence in flutter .
Firestore _db = Firestore.instance;
// TODO :
_db.settings(persistenceEnabled: true);

Can someone point me to a example which is using the offline handling. I have trouble to get it reliable working. Thanks.

@ride4sun Alright, from my experience, here is what I know to be true..
First of all.. Offline handling is enabled by default!

When you create a new document offline, Firesstore gives the document a local id, and once online, Firestore will push the new document and any cloud triggers related to the update will run.
When you delete an existing document, Firestore will remove the document from the local cache history, and once online Firestore will remove the document from the database and any cloud triggers related to the update will run.

The trouble happen when attempting to update a document offline! Unexpected random behavior happens ESPECIALLY if the document was created or removed offline!
In my app, once the user creates or deletes a document offline, I IMMEDIATELY block user interaction on the created or deleted document, or else the app will crash!
Possibley a bug or another GitHub issue to create, but I'm not sure.

@sssalib42 Thx for th great description. I am looking more for a flutter code example which works offline and online. I run into 10 second delays when I set data and because I can not execute Transactions becaus the calls have to be  synchronous and block the UI thread which is not acceptable in my case.(not sure anybody can accept that)  I am getting really annoyed with Cloud Firestore and the lag of good examples. I created an adapter to just write all my data to shared preferences to avoid the DB and keep going with my implementation. How I understand the synchronous offline handling makes cloud firestore just not useable for me. But I might miss something critical besides the great promotial videos. There are  not even example code for flutter.

Oh brother!
@ride4sun
So, my app is completely written in flutter.
The same Firestore code which works online is the same Firestore code which works offline.
Just want to clarify something real quick..
Do you want your app to completely run offline? If this is the case, then you will need to use Sqflite, to manage local database implementation.
My app offers free offline support, and paid online support.
The offline persistence for Firestore is meant to handle rare cases when the user's network is not available. Firestore uses cached transactions at 10 MB of memory, and literally starts removing old transactions from memory even if they haven't been synced up online, once the new transactions push the 10 MB limit.
So I hope you can see, it's not meant to run completely offline, but rather it's an attempt to supplement the online experience, adding protection from rare network glitches.
It may seem there isn't enough Flutter examples, but what I notices is there is not need for it in most cases, since there isn't really much they can add to the basics. StackOverflow has the answer to any question you will have, this I know for sure.
Good luck!

@dileepxdn
This issue has been moved to https://github.com/FirebaseExtended/flutterfire/issues/680. Any further collaboration will be done there.

if i create a file offline and then removed the app from memory before going online again, can firestore persist this files?


Possible context leak in code

What changed?
A marker has been added to the code. This is not an actual fix, but a marker denoting that there may be a bug.

Why?
In file: taskpoller.go, there is a function createContext that contains a cancellation function returned by context.WithTimeout. However, the cancellation function is ignored using _. I suggested that the cancellation function must be called or the new context will remain live until the parent context is cancelled. I did not propose a fix since I am not familiar with the business logic. However, the codebase appears to be following that prescribed convention in other parts.
Sponsorship and Support:
This work is done by the security researchers from OpenRefactory and is supported by the Open Source Security Foundation (OpenSSF)(https://openssf.org/): Project Alpha-Omega(https://alpha-omega.dev/). Alpha-Omega is a project partnering with open source software project maintainers to systematically find new, as-yet-undiscovered vulnerabilities in open source code - and get them fixed – to improve global software supply chain security.

How did you test it?
Not reproduced.

Potential risks
Cannot ascertain.

Release notes
Appears not a release candidate.

Documentation Changes
Appears no change is needed.

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

I have prepared an updated patch. However, I am traveling now and cannot access the GitHub account used to create the pull request. I will address this in about a week.

While I agree with this in general, I don't think it's true for context.Background().
The leak would be coming from the parent's list of children to propagate cancels, but a background context both does not have that list, and it has a nil done chan so this branch is taken when making a cancel-context derived from it: https://cs.opensource.google/go/go/+/refs/tags/go1.21.5:src/context/context.go;l=461
So these should be cleaned up as soon as they're garbage-collectable.  Which is fine.
That said, these kinds of funcs mean we can't easily add context data (no parent-ctx arg), and this is a potential landmine if we did (because at that point it'd likely leak).  So as long as we've got a clear path to cancel correctly, it seems like an improvement.

Pull Request Test Coverage Report for Build 018da900-96a5-4f49-9b26-1a6ebd52dce3
Details

-63 of 131 (51.91%)  changed or added relevant lines in 6 files are covered.
107 unchanged lines in 21 files lost coverage.
Overall coverage decreased (-0.04%) to 62.745%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

host/esutils/client_v6.go
18
21
85.71%

host/taskpoller.go
41
59
69.49%

host/esutils/client_os2.go
0
21
0.0%

host/esutils/client_v7.go
0
21
0.0%

Files with Coverage Reduction
New Missed Lines
%

common/persistence/sql/common.go
2
72.73%

common/persistence/sql/sqlplugin/mysql/db.go
2
83.33%

common/persistence/sql/sqlplugin/mysql/task.go
2
73.68%

common/persistence/sql/sqlplugin/postgres/db.go
2
85.0%

common/persistence/sql/sqlplugin/postgres/task.go
2
73.4%

service/history/task/transfer_standby_task_executor.go
2
87.42%

service/matching/db.go
2
73.23%

common/log/tag/tags.go
3
50.18%

common/persistence/nosql/nosqlplugin/cassandra/workflow.go
3
59.1%

common/task/fifo_task_scheduler.go
3
84.54%

Totals

Change from base Build 018da81a-4d02-4ee6-9401-7a60db74410e:
-0.04%

Covered Lines:
92605

Relevant Lines:
147589

💛  - Coveralls


Fix option syntax in Influx CLI example

The Create a task using raw Flux example for Create a task using the influx CLI shows options instead of the correct option.

[x] Signed the InfluxData CLA
(if necessary)
[ ] Tests pass (no build errors)
[ ] Rebased/mergeable

thank you, @douglaswth!


sap.m.ComboBox - jQuery(...).control is not a function

OpenUI5 version: 1.65.1
Browser/version (+device/version): Chrome v74.0.3729.169 (64-bit)
Any other tested browsers/devices(OK/FAIL): Firefox Quantum v67.0.2 (64-bit)
Steps to reproduce the problem:

I'm using ComboBox like in example on website https://sapui5.hana.ondemand.com/#/entity/sap.m.ComboBox/sample/sap.m.sample.ComboBox/code

<ComboBox                             id="idComboBoxState"                             items="{decisionModel>/decisionStates}"                             change="onChangeState"                             placeholder="{i18n>selectState}"                             width="100%"                             textAlign="End">                         <core:Item text="{decisionModel>text}" key="{decisionModel>key}"/>                     </ComboBox>

When I select the combo item from app, and I get error message:

ComboBox-dbg.js:1310 Uncaught TypeError: jQuery(...).control is not a function
at f.k._handleItemTap (ComboBox-dbg.js:1310)
at c (Element-dbg.js:248)
at f.b._handleEvent (Element-dbg.js:261)
at constructor.U._handleEvent (UIArea-dbg.js:921)
at HTMLDivElement.dispatch (jquery-dbg.js:4737)
at g (jquery-mobile-custom-dbg.js:1881)
at HTMLDivElement.r (jquery-mobile-custom-dbg.js:2000)
at HTMLDivElement.dispatch (jquery-dbg.js:4737)
at HTMLDivElement.c3.handle (jquery-dbg.js:4549)
at Object.trigger (jquery-dbg.js:7819)

Any other information? (attach screenshot if possible)

What kind of runtime are you using when the issue occurs? A self-contained build for your app or the cloud version of openui5?

I'm using self-contained build, not cloud.

Hello @FilipMucka,
Since the issue is not reproducible in the mention sample:  https://sapui5.hana.ondemand.com/#/entity/sap.m.ComboBox/sample/sap.m.sample.ComboBox/code
In order to support you, could you please provide us with an isolated example where the issue could be reproduced?
You could use JSBin for that purpose: https://jsbin.com/?js,output
Regards,
Niya

Since 549f08ff1df8a4b10ccfb29d72b1b12e283af0d0, sap.m.ComboBox indeed uses sap/ui/dom/jquery/control without declaring a dependency to it. However, UIArea also depends on it and there shouldn't be a path where a control is rendered without a UIArea.
I wonder a bit about the stack trace. The last line Object.trigger() sounds a bit as if this happens in a test case? Is that true? If so, how is the view rendered then? Anything special?

Since there is no response from the author for more than 4 weeks I'm closing the ticket.


GitHub Actions - Pro Login - Password?

Prerequisites

[X] I am running the latest version. (up upgrade)
[X] I searched to see if the issue already exists.
[X] I inspected the verbose debug output with the -v, --verbose flag.
[X] Are you an Up Pro subscriber?

Description
I am trying to setup a github action to deploy on push. I am a pro subscriber and it seems when up team login is entered it sends an email. Is there a way to pass a password? The login works but I have to respond to the email. I want to make it automated. I can't find anything in the docs.
Here is the script I am trying to run in the github action.
curl -sf https://up.apex.sh/install | BINDIR=. sh
./up team login --email $UPEmail --team $UPEmail
./up upgrade
./up production

Ahh yep! For running in CI you can use up team ci which gives you an auth token you can set as an environment variable UP_CONFIG. Then all you should need is:
curl -sf https://up.apex.sh/install | BINDIR=. sh
./up upgrade
./up production

Here's the full guide: https://apex.sh/docs/up/guides/#deploying_applications_from_continuous_integration

Sorry I thought I read all the docs in that section.  Somehow I missed that.   Thanks for pointing out I am blind!  :-)


Dynamically-assembled require() calls in lib/widget.js break Browserify support

Looks like the refactoring introduced by 8d11075b891ca2dd7aaeb2777a4e6e632d3875e8 means those require calls can't be resolved by bundling tools like Browserify.
Wasn't too hard for me to fix myself, but I imagine I'm not the only one using this with Browserify -- there was some talk in chjj/tty.js#106 about it.

Ah, I see:
[user@node blessed]$ node bundle.js
module.js:338
    throw err;
          ^
Error: Cannot find module './widgets/node'
    at Function.Module._resolveFilename (module.js:336:15)
    at Function.Module._load (module.js:278:25)
    at Module.require (module.js:365:17)
    at require (module.js:384:17)
    at s (/home/user/work/node_modules/blessed/bundle.js:1:176)
    at /home/user/work/node_modules/blessed/bundle.js:1:367
    at /home/user/work/node_modules/blessed/bundle.js:13553:33
    at Array.forEach (native)
    at Object.<anonymous> (/home/user/work/node_modules/blessed/bundle.js:13551:16)
    at s (/home/user/work/node_modules/blessed/bundle.js:1:316)

I guess we could require them all at once again instead of using a forEach.

Just made it a bit less insane, in case widget.js is ever changed.

Refactored it a bit. Should work now. Trying browserifying it and let me know.

Amazing, thanks!
Unfortunately, looks like the widget API isn't compatible with Browserify -- looks like Screen implicitly depends on tput, even if it's passed an existing Program initialized tput: false (at https://github.com/chjj/blessed/blob/master/lib/widgets/screen.js#L61) -- couldn't figure out a sensible way to get around this.

Yeah, tput is going to have to read the terminfo file, however, it will fallback to a builtin vt102 termcap, or, if browserify can manage it: an xterm terminfo file. We could include this in tput.js as a fallback if necessary I suppose.

I can't seem to convince tput to use the fallback -- even if I call tput._useVt102Cap() it still seems to want to read the terminfo from the disk, which again breaks the Browserify bundle.
It should be doable to inline the xterm terminfo with https://github.com/substack/brfs -- but like the widget requires, it doesn't look like the way readFileSync is called in tput.js is statically resolvable.

@lostfictions, if you give me a PR that's relatively unobtrusive, I'll glady merge it.

@lostfictions, I added a fix for tput.js to compile in the xterm terminfo for the browser. Let me know how it works.

It works! Amazing! Thanks so much.

Awesome. Good to know we got that squared away. Just curious, if I may ask: what are you working on for the browser?

It's a bit frivolous -- I'm working on a (browser-based) game about a sort of byzantine old terminal interface, so blessed+term.js seemed like a natural fit!
I was getting really excited looking at blessed-contrib's extra widgets, too. It looks like they don't render correctly in term.js though -- it just outputs ? instead of certain characters. (Doesn't seem to be a font/styling issue, either -- ? is the actual character it writes to the DOM. I have no idea how to diagnose or debug this...)

wrt blessed-contrib, possibly you do not have braille fonts.
do components that do not use braille font render correctly? (e.g. bar
chart)
if you are using windows, can you try do it from an ubuntu VM which
natively supports braille?
On Tue, Jul 28, 2015 at 12:28 PM, s<EMAIL_ADDRESS>wrote:

It's a bit frivolous -- I'm working on a (browser-based) game about a sort
of byzantine old terminal interface, so blessed+term.js seemed like a
natural fit!
I was getting really excited looking at blessed-contrib's extra widgets,
too. It looks like they don't render correctly in term.js though -- it just
outputs ? instead of certain characters. (Doesn't seem to be a font/styling
issue, either -- ? is the actual character it writes to the DOM. I have
no idea how to diagnose or debug this...)
—
Reply to this email directly or view it on GitHub
https://github.com/chjj/blessed/issues/158#issuecomment-125519214.

--
I'm on Twitter (@YaronNaveh http://twitter.com/#!/YaronNaveh)

also try to render a blessed regular box with content which contains a braille character:
var box = blessed.box({
   content: String.fromCharCode(0x2812)
});

Could be an issue with term.js. @lostfictions, do the blessed-contrib characters render correctly in your native terminal?

I'm running from Ubuntu. @yaronn, that blessed box just outputs a ? character, yeah. On the other hand, if I do this:
  var div = document.createElement("div");
  div.innerHTML = String.fromCharCode(0x2812);
  div.style.fontFamily = '"DejaVu Sans Mono", "Liberation Mono", monospace';
  document.body.appendChild(div);

...the Braille character shows up just fine.
@chjj, running the same examples in my native terminal works just fine too.

running the same examples in my native terminal works just fine too.

Looks like an issue with term.js then. Will investigate. That's very strange though since those braille characters are still within the BMP.

@lostfictions, can you try adding LANG=en_US.UTF-8 to the term.js env and see if it makes a difference? I'm guessing your native terminal already has that env variable set, but when you run your term.js server, there is no LANG, and since the regular xterm terminfo has no U8, it assumes you don't support utf8.

Also, yes, after looking at term.js, the only reason you would get ? in place of a BMP unicode character is if blessed cannot detect unicode either through the terminfo or LANG env variable. So it really just looks like a problem with configuration of your term.js server to me.

Hmmm, wait a second, since blessed is looking at the browserified process.env, try doing:
process.env.LANG = 'en_US.UTF-8';

Before starting up your blessed app.

@lostfictions, try the latest commit and add forceUnicode: true to the blessed.screen() options.

I've added a commit which forces unicode support when browserified.

That gets the braille characters rendering, thanks!
Unfortunately, the layout is still messed up:

I suspected this was down to a styling issue, so I poked around in Devtools. It looks like the braille characters are actually being rendered in the non-monospace version of the font, so it's using DejaVu Sans instead of DejaVu Sans Mono. I tried using Ubuntu Mono and had the same problem.
FreeMono fares better and can render the braille characters, fixing the alignment issues, but unfortunately its braille glyphs make it unsuitable for drawing graphics:

Not sure why this is happening -- my terminal font is Ubuntu Mono, so it must have those Braille characters, right...?

As far as I know, braille characters are not double-width (term.js should have no problem rendering them), so that is undoubtedly a browser font issue.
Why does DejaVu Sans Mono not work in the browser for you? That really seems like the solution.

Not sure why this is happening -- my terminal font is Ubuntu Mono, so it must have those Braille characters, right...?

Not necesarilly. Having written a terminal in C for X11, I can tell you, usually, if a font doesn't contain certain glyphs, the application will do a lookup on your fonts and iterate through them until it finds a scalable or compatible (for BMP fonts) containing those glyphs: so you're actually seeing two fonts.
This is why unicode support is harder than it looks for native applications (which is the whole reason why rxvt was forked to rxvt-unicode - the unicode part is non-trivial).
Ubuntu Mono may not contain Braille glyphs, and maybe your terminal just does a better job of finding them elsewhere than your browser does.

Why does DejaVu Sans Mono not work in the browser for you? That really seems like the solution.

I'm honestly not sure -- it seems like it must be a browser styling issue? I tried explicitly embedding DejaVu Sans Mono as a web font (making sure it wasn't subsetted) and referring to it under a different font-family name to ensure it was used, but no dice.

@lostfictions, you could try embedding a font that works with @font-face. That might be the best solution here.

That's what I just tried with DejaVu Sans Mono! I'm really confused as to why it's not working -- I even opened it up in FontForge and double-checked that the Braille glyphs were there just to be safe.
I'm at a total loss -- maybe there's something in the browser style engine that's causing it to be overridden? And yet FreeMono seems to work...


Jesse: Watch quickproadvisor.us

Jesse requests the watch of the watch_keyword quickproadvisor\.us. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by Jeff Schaller in Charcoal HQ

quickproadvisor\.us has been seen in 1 true positive, 0 false positives, and 0 NAAs.


AudioStreamPlayer2D Doesn't Move Proportional to Parent ParallaxLayer

Godot version
4.1
System information
Godot v4.1.stable - Arch Linux #1 SMP PREEMPT_DYNAMIC Sat, 10 Jun 2023 00:35:35 +0000 - Vulkan (Compatibility) - AMD Radeon RX 7900 XT (gfx1100, LLVM 16.0.0, DRM 3.52, 6.3.7-arch1-1) () - AMD Ryzen 9 3950X 16-Core Processor (32 Threads)
Issue description
I have an AudioStreamPlayer2D as a child node of a background element. The background element, being on a ParallaxLayer, moves at a small fraction of the camera's (and AudioListener2D's) movements. Unfortunately, the AudioStreamPlayer does not honor this fractional moving so it does not stay aligned with the background element.
Steps to reproduce
Put AudioStreamPlayer2D on parallax background with some graphic and make the layer move at some small fraction of the distance as the camera with the AudioListener2D. Move the camera and AudioListener2D. The visual object will still be on the screen but the AudioStreamPlayer2D will be inaudible with a great enough distance.
Minimal reproduction project
AudioStreamPlayer2D Bug.zip

I doubt this is intended to work in the first place. I'd probably document the limitation in the meantime.

Yeah, I'm sure it is just an unforeseen consequence of how the parallax layers work (and how nobody really thought to consider AudioStreamPlayer2D interaction with it), but I would imagine expected behavior would be for an AudioStreamPlayer2D placed on a background element to always sound like it's coming from that background element. Not a big deal in my particular case, and I'm working around it. I just thought I would report it.

I just encountered this same issue on my own, been driving me crazy trying to figure out what was going on.
Running 4.1.3 stable

I suggest testing the new Parallax2D node in 4.3.dev builds (make a backup of your project before upgrading). It may address the issue outlined here.

Just tested the MRP converting to Parallax2D and works as expected. CanvasLayer strikes again!


Fix Resource Group Locks so to define the lock type

Is there an existing issue for this?

[X] I have searched the existing issues

Community Note

Please vote on this issue by adding a :thumbsup: reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description
The new lock option on a resource group  is based on
lock_resource = true
this will result in a CanNotDelete lock been applied
this should be changed to read as
lock_resource = null | CanNotDelete | ....
and the module in modules\resource_groups\module.tf code should be updated to use this structure
New or Affected Resource(s
azurerm_resource_group
Potential Configuration file
No response
References
No response

Lock levels: CanNotDelete, ReadOnly or NotSpecified


Use MQTT in a usefull way for app

e.g: the temperature sensor publishes the temperature every 2 min to the homonymous topic; another part of the system listens to this topic and if the temperature is  2 deg less than the temperature set by the user it turns on the heating system.

already done this with snoring (sound) - we can use it with temperature too


dunePackage: add support function and use it in a few packages

Motivation for this change
This takes advantage of the uniformization of the OCaml package ecosystem around the dune build system (and often the accompanying dune-release) to simplify the way usual OCaml derivations are prepared in nixpkgs.
cc @vbgl
Things done
I have ported a few packages which allowed me to refine the design of dunePackage but not all of them yet.

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

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

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

I can certainly backtrack of the src refactoring but I would find this pretty sad given that virtually all the dune packages that I refactored took their sources from GitHub. Furthermore, this setup makes it simpler for package maintainers to use the dune-release tarball if it is available (just set dune-release = true). In the previous setting, switching from one setup to the other was significantly more complicated. Of course, I can also keep only the shared unpackPhase when dune-release = true is set.
I don't see any issue in changing the hash when doing this refactoring (switch from fetch unpacked to fetch). It's not bringing any complexity after the refactoring.

@Mic92 I still do not know what you meant by "build hook" but the composability aspect might not be so important given how similar all dune packages are (and anyways, the current setup allows to override any given phase).

Here is an example for cmake: https://github.com/NixOS/nixpkgs/blob/master/pkgs/development/tools/build-managers/cmake/setup-hook.sh
and mesonbuild: https://github.com/NixOS/nixpkgs/tree/master/pkgs/development/tools/build-managers/cmake

OMG, this is horrible :scream:  Don't count on me to imitate that :wink:

Where are these hooks coming from? Where is this documented? How is it different from pre and post phases?
About the need to document this, I agree and I will take care of this.

Build phases are described in the stdenv documentation: https://nixos.org/nixpkgs/manual/#sec-stdenv-phases

OK, I knew about this part of the doc but I had missed that this had to be run from the build phases themselves. Will update.

Refactoring complete. Doc remains.

The PR is ready now, and it includes the doc. It was quite time that OCaml got its section in the nixpkgs manual!

CI had passed but an update to odoc meant that I had to rebase the PR and fix the conflicts. Can this PR get merged now? I would like to avoid having to rebase it once more.

I’m very surprised you did not find any interesting issues with the zmq packages when testing this PR…

I built all affected packages on Linux. Are there problems on other architectures?

This might have been not included in the list as ocamlPackages are not recursed.
$ nix-shell -p acgtk -p eff -p flow -p fstar -p google-drive-ocamlfuse -p jackline -p libbap -p liquidsoap -p ocaml-top -p ocamlformat -p patdiff -p pyre -p reason -p satysfi -p stog -p trv -p virt-top

But also that one seems to work:
$ nix-shell -p ocamlPackages.zmq
these derivations will be built:
  /nix/store/scsiz53z9wyypkrrgndnmp747f477nd5-source.drv
  /nix/store/kaxk9z132zkg7mn9j9yd5ah3f7v0d295-ocaml4.06.1-zmq-20180726.drv
these paths will be fetched (0.21 MiB download, 1.06 MiB unpacked):
  /nix/store/5217m8ssa8n7x0vpw8ndxplwf72wqrxs-czmq-4.1.1
copying path '/nix/store/5217m8ssa8n7x0vpw8ndxplwf72wqrxs-czmq-4.1.1' from 'https://cache.nixos.org'...
building '/nix/store/scsiz53z9wyypkrrgndnmp747f477nd5-source.drv'...

trying https://github.com/issuu/ocaml-zmq/archive/d312a8458d6b688f75470248f11875fbbfa5bb1a.tar.gz
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   157    0   157    0     0    371      0 --:--:-- --:--:-- --:--:--   370
100 30169    0 30169    0     0  30259      0 --:--:-- --:--:-- --:--:-- 30259
unpacking source archive /build/d312a8458d6b688f75470248f11875fbbfa5bb1a.tar.gz
building '/nix/store/kaxk9z132zkg7mn9j9yd5ah3f7v0d295-ocaml4.06.1-zmq-20180726.drv'...
unpacking sources
unpacking source archive /nix/store/24j11nwkl20f4v6rm3x8sgb8h51638b7-source
source root is source
patching sources
applying patch /nix/store/kg1vs1b0azdk8i5hyzwrw5z9d7bazifh-ocaml-zmq-issue43.patch
patching file zmq/src/caml_zmq_stubs.c
configuring
no configure script, doing nothing
building
installing
Processing file zmq.install as zmq.
post-installation fixup
moving /nix/store/dr8qcfxiphjpmvjszjnmg9y1zdjv0i8i-ocaml4.06.1-zmq-20180726/doc to /nix/store/dr8qcfxiphjpmvjszjnmg9y1zdjv0i8i-ocaml4.06.1-zmq-20180726/share/doc
shrinking RPATHs of ELF executables and libraries in /nix/store/dr8qcfxiphjpmvjszjnmg9y1zdjv0i8i-ocaml4.06.1-zmq-20180726
shrinking /nix/store/dr8qcfxiphjpmvjszjnmg9y1zdjv0i8i-ocaml4.06.1-zmq-20180726/lib/ocaml/4.06.1/site-lib/stublibs/dllzmq_stubs.so
shrinking /nix/store/dr8qcfxiphjpmvjszjnmg9y1zdjv0i8i-ocaml4.06.1-zmq-20180726/lib/ocaml/4.06.1/site-lib/zmq/deferred/zmq_deferred.cmxs
shrinking /nix/store/dr8qcfxiphjpmvjszjnmg9y1zdjv0i8i-ocaml4.06.1-zmq-20180726/lib/ocaml/4.06.1/site-lib/zmq/zmq.cmxs
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/dr8qcfxiphjpmvjszjnmg9y1zdjv0i8i-ocaml4.06.1-zmq-20180726/lib
patching script interpreter paths in /nix/store/dr8qcfxiphjpmvjszjnmg9y1zdjv0i8i-ocaml4.06.1-zmq-20180726
checking for references to /build in /nix/store/dr8qcfxiphjpmvjszjnmg9y1zdjv0i8i-ocaml4.06.1-zmq-20180726...
bash: /tmp/env-vars: cannot overwrite existing file

This one is broken: ocamlPackages_latest.zmq. It was already broken. Now it is harder to fix.

In any case, if zmq is not well served by buildDunePackage, feel free to revert the change on this file only.

PR to fix zmq: #49921. Suggestions welcome.


Add a LedgerSpecifier class to replace ledgerIndex and ledgerHash in request objects

Adds a new immutable class called LedgerSpecifier, which will allow developers to specify ledger versions in XrplRequestParams more clearly.
This PR does not actually change any of the XrplRequestParams and is strictly additive. Another PR will alter all XrplRequestParams extensions by removing ledgerIndex and ledgerHash fields and replacing them with a LedgerSpecifier field.

Originally, I split all of these changes into this PR and #126 to minimize the size of each PR. My intention was to eventually merge both and release a new major version.
However, after chatting with @theotherian in #126 , I think it's actually better to merge this PR, cut a v2.1 release, and save #126 for a later release. I will add deprecation notices wherever the old LedgerIndex class is used to give developers an obvious heads up that it will go away soon.

I actually decided to add all of these features in a non-breaking way, making a lot of this PR moot. Closing this PR and rebasing #126 to main


Add metadata support and unify transaction calls

Issue/Task Number: T116
Overview
Describe what your Pull Request is about in a few sentences. This is a global description that should tell people what your PR is all about. Don't go too much into details here.
Changes

Add metadata / encrypted_metadata for transactions & minted token
Allow use of all with a provider_user_id parameter to scope down transactions
Updated version to 0.9.6

ping


Homebrew-installed Pipenv defaults to Homebrew-created virtualenv Python

As following log, it will use /usr/local/Cellar/pipenv/2018.5.18/libexec/bin/python3.6, instead of use system installed python
🚀  pipenv install --dev
Warning: the environment variable LANG is not set!
We recommend setting this in ~/.profile (or equivalent) for proper expected behavior.
Creating a virtualenv for this project…
Using /usr/local/Cellar/pipenv/2018.5.18/libexec/bin/python3.6 (3.6.5) to create virtualenv…
⠋Already using interpreter /usr/local/Cellar/pipenv/2018.5.18/libexec/bin/python3.6
Using real prefix '/usr/local/Cellar/python/3.6.5/Frameworks/Python.framework/Versions/3.6'
New python executable in /Users/aweimeow/pipenv/.venv/bin/python3.6
Also creating executable in /Users/aweimeow/pipenv/.venv/bin/python
Installing setuptools, pip, wheel...done.

Virtualenv location: /Users/aweimeow/pipenv/.venv
Installing dependencies from Pipfile.lock (ae475c)…
 This is however especially problematic in Homebrew’s case because Homebrew always installs each package version in a virtualenv, and the old one will break when you upgrade and do cleanup (brew purge).
This is not a Pipenv bug per-se, but we’ll probably want to improve the Homebrew experience if we support it has a first-class installation method. One simple solution would be to remove the auto-Python-detection outright, and requires the user to specify a Python when initialising an environment.

@uranusjr I just pushed up a bunch of changes to https://github.com/sarugaku/pythonfinder which simply use the user path order to find python, unless a version is supplied.  We could swap to that (which has been the plan all along) once we are happy with it

@nathanielford This is not related to the top issue. It is a long-standing Homebrew problem around Python formulae. Due to how they manage Python installations, every Python package formula would break every time you upgrade Homoebrew Python, until a new revision is released to correct the path. You should try brew update and brew upgrade pipenv, and if that doesn’t work, submit a bug report to Homebrew. There is nothing Pipenv can do here.
Side note: Personally I strongly discourage using Homebrew to install Python packages, or install any Python packages against the Homebrew Python using pip (or otherwise). It is just not a good idea.

@uranusjr I see! That makes a lot of sense; apologies for the noise. Thanks for the pointers!

@nathanielford , did you fixed this problem with brew?

What method for managing installed python versions do you recommend (especially Mac OS)? Homebrew is not as flexible with this as I had hoped/imagined. Some projects do not yet support 3.7, which is the homebrew default at this time, so being able to easily specify a particular version is becoming a requirement for me.

Side note: Personally I strongly discourage using Homebrew to install Python packages, or install any Python packages against the Homebrew Python using pip (or otherwise). It is just not a good idea.

@QQsha I uninstalled pipenv from homebrew and installed it explicitly with python3.

@uranusjr have you tried pipx? It’s similar to pipsi but is actively developed.  It’s straightforward to install. https://packaging.python.org/guides/installing-stand-alone-command-line-tools/
pip install pipx —user
pipx install pipenv

Closing this issue as I believe the documentation is corrected these days, I just checked:

It’s possible to install Pipenv with Homebrew on MacOS, or with Linuxbrew on Linux systems. However, this is now discouraged, because updates to the brewed Python distribution will break Pipenv, and perhaps all virtual environments managed by it. You’ll then need to re-install Pipenv at least. If you want to give it a try despite this warning, use:


Question: opening the browser only once

Might be a dumb question, but do you happen to know of a trick to open a tab only once? I'm using nodemon to watch multiple servers:
nodemon server.js
So every time I change the file and hit save, a new tab is opened. I am using nodemon because it is convenient and helps me restart all the servers, so I don't have to figure out which one to restart.
Thanks!

@st32lthx  I have the same problem too. Do you figure it out?
@pwnall Could you help us?


CompactArray could use a constructor from slice

Similar to https://docs.rs/sucds/latest/sucds/int_vectors/compact_vector/struct.CompactVector.html#method.from_slice.
Currently constructing it from existing data requires some overhead.

Ok, what about an extend method that takes a general iterator? The codegen should be the same.
Ofc there with an ExactSizeIterator we could pre-allocate everything and concurrently build the compact_array by doing compare_exchanges.

Yeah taking an interator sgtm.

Actually, if you construct it from fresh it would be nice to have a from_iterator() -> Self that also computes the required bit-width.

I'm just adding to common_traits the UnsignedInt.len() method for that purpose (because I need it for another type)!

Actually, if you construct it from fresh it would be nice to have a from_iterator() -> Self that also computes the required bit-width.

that would require either allocating, or iterating twice

Right, that makes more sense for from_slice where you already know you can iterate twice.
(Or you could take a Iterator + Clone but that's somewhat unconventional I think.)

Implemented in the compact branch,


Implement role field in AuthorizationPolicy

Add the option to use role instead of roleRef in AuthorizationPolicy.
The value of role refers to the ServiceRole at the local namespace or at the root namespace (currently using forward slash to refer to the root namespace, e.g. /service-role-name. This may be changed in the future).
Note: if we can't find a ServiceRole at the local namespace, we don't look for the ServiceRole at root namespace. In another words, the user must specify / if they want to use ServiceRoles at root namespace.
Will update e2e tests once https://github.com/istio/istio/pull/12814 is checked in.
For https://github.com/istio/istio/issues/12394

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: pitlv2109
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: rshriram
If they are not already assigned, you can assign the PR to them by writing /assign @rshriram in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@pitlv2109: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
ad1e7a03607fa5a2d307541435c73ccb3a887ba2
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@pitlv2109: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
ad1e7a03607fa5a2d307541435c73ccb3a887ba2
link
/test istio-pilot-multicluster-e2e

prow/istio-integ-k8s-tests.sh
ad1e7a03607fa5a2d307541435c73ccb3a887ba2
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@pitlv2109: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
ad1e7a03607fa5a2d307541435c73ccb3a887ba2
link
/test istio-integ-k8s-tests

prow/istio-pilot-multicluster-e2e.sh
30c65b42cb3e54f20fa3812f8758f89cbf86c2d9
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@pitlv2109: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
30c65b42cb3e54f20fa3812f8758f89cbf86c2d9
link
/test istio-pilot-multicluster-e2e

prow/istio-integ-k8s-tests.sh
30c65b42cb3e54f20fa3812f8758f89cbf86c2d9
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@pitlv2109: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
30c65b42cb3e54f20fa3812f8758f89cbf86c2d9
link
/test istio-integ-k8s-tests

prow/istio-pilot-multicluster-e2e.sh
7a4a512fcef2501f9bd3110fdd80d31c5d866c3f
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@pitlv2109: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
7a4a512fcef2501f9bd3110fdd80d31c5d866c3f
link
/test istio-pilot-multicluster-e2e

prow/istio-integ-k8s-tests.sh
7a4a512fcef2501f9bd3110fdd80d31c5d866c3f
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@liminw @yangminzhu Resolved comments. Ready for review, thanks.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: pitlv2109, yangminzhu
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: rshriram
If they are not already assigned, you can assign the PR to them by writing /assign @rshriram in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@pitlv2109: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
7a4a512fcef2501f9bd3110fdd80d31c5d866c3f
link
/test istio-integ-k8s-tests

prow/istio-pilot-multicluster-e2e.sh
3336adb82e10f2242de0eb499c7d3de84161ce03
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@pitlv2109: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
3336adb82e10f2242de0eb499c7d3de84161ce03
link
/test istio-pilot-multicluster-e2e

prow/istio-integ-k8s-tests.sh
3336adb82e10f2242de0eb499c7d3de84161ce03
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: diemtvu, pitlv2109, yangminzhu
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS [diemtvu]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@pitlv2109: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
3336adb82e10f2242de0eb499c7d3de84161ce03
link
/test istio-integ-k8s-tests

prow/istio-pilot-multicluster-e2e.sh
ab9902a24957ff528a74e0313f8941a5bc084857
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@pitlv2109: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
ab9902a24957ff528a74e0313f8941a5bc084857
link
/test istio-pilot-multicluster-e2e

prow/istio-integ-k8s-tests.sh
ab9902a24957ff528a74e0313f8941a5bc084857
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Disabling of compiler optimizations needed to prevent non-null pReserved when invoking C_Initialize().

Environment
Reproducible with Rust pkcs11 crate version v0.5.0 on:

SoftHSM 2.6.1-2 on Ubuntu 21.10 installed with apt install softhsm, or
SoftHSM 2.6.1 on Alpine Linux 3.12 installed with RUN apk add softhsm from a Dockerfile

Steps to reproduce
Calling Ctx::Initialize(Some(args)) where args is either CK_C_INITIALIZE_ARGS::default() or:
let mut args = CK_C_INITIALIZE_ARGS::new();
args.CreateMutex = None;
args.DestroyMutex = None;
args.LockMutex = None;
args.UnlockMutex = None;
args.flags = CKF_OS_LOCKING_OK;
ctx.initialize(Some(args)

(note that CK_C_INITIALIZE_ARGS::new() sets args.pReserved = ptr::null_mut() and that CK_C_INITIALIZE_ARGS::default() calls CK_C_INITIALIZE_ARGS::new())
Works in debug builds
When built with cargo build my application works without error.
Fails in release builds
When built with cargo build --release my application fails with error CKR_ARGUMENTS_BAD and in /var/log/ via Syslog logging from SoftHSM I see error message SoftHSM.cpp(436): pReserved must be set to NULL_PTR.
Disabling compiler optimizations helps
Adding this to Cargo.toml and rebuilding my application with cargo build --release "solves" the problem:
[profile.release.package.pkcs11]
opt-level = 0

I encountered a similar issue today, but more severe.  C_Initialize ends up getting called with garbage parameters.  Here's the portion of memory where the CK_C_INITIALIZE_ARGS should be, right before C_Initialize is called:
(gdb) x/6a $rdi
0x7fffffffd238: 0x7ffff7b7ee30 <C_OpenSession>  0x7ffff7c29020
0x7fffffffd248: 0x7fffffffd560  0x7fffffffd300
0x7fffffffd258: 0x7ffff7dba670 <__memmove_avx_unaligned_erms>   0x555555692ba0 <_ZN4clap3app6parser6Parser7add_arg17h6a546dc72bff8a9cE>

Not only that, right before C_Initialize gets called, we've already clobbered the stack:
#0  0x00005555556d8f80 in pkcs11::Ctx::initialize ()
#1  0x00007ffff7b7ede0 in ?? () from /usr/lib/softhsm/libsofthsm2.so
#2  0x00007ffff7b7ee30 in ?? () from /usr/lib/softhsm/libsofthsm2.so
#3  0x00007ffff7c29020 in ?? () from /usr/lib/softhsm/libsofthsm2.so
#4  0x00007fffffffd560 in ?? ()
#5  0x00007fffffffd300 in ?? ()
#6  0x00007ffff7dba670 in ?? () at ../sysdeps/x86_64/multiarch/memmove-vec-unaligned-erms.S:221 from /lib/x86_64-linux-gnu/libc.so.6
#7  0x0000555555692ba0 in ?? ()
#8  0x0000555555977b70 in ?? ()
#9  0x00007fffffffd5d8 in ?? ()

Things seemed to be fine before Ctx::initialize, so I'm suspecting that function has some UB somehow.
This is also on x86_64 Linux, and also only happens with optimizations on.

Adding this to this issue as well: please switch to the cryptoki crate: https://github.com/parallaxsecond/rust-cryptoki


Use generic types for choice and shuffle methods

When using any[] for argument type, Typescript throws an error when readonly array is passed in which can be only fixed by casting it into any[]. The resulting type is also any thus losing the certainty of what type will be returned.
Using a simple generic, we can preserve types and also get rid of Typescript error.

ping @chrisakroyd


Overwrite logging format

I am running an eventhub trigger azure function in python in Docker. I want to ship the azure function logs to my logging pipeline. The logs are written to /tmp/Functions/Function/EventhubTrigger/.
The log format is Timestamp Level Message. The format I need is '%(message)s'. How could I overwrite the log format to only keep message without Timestamp and level ahead.
Example:
The log I got now is
"2019-04-02T01:10:59.843 [Information] Executing 'Functions.EventhubTrigger' (Reason='', Id=29d82fae-719c-4109-98a5-79d1d83628e3)"
The log I need is
"Executing 'Functions.EventhubTrigger' (Reason='', Id=29d82fae-719c-4109-98a5-79d1d83628e3)"
Investigative information
Please provide the following:

Timestamp:
Function App name:
Function name(s) (as appropriate):
Core Tools version:

Repro steps
Provide the steps required to reproduce the problem:

Expected behavior
Provide a description of the expected behavior.

Actual behavior
Provide a description of the actual behavior observed.

Known workarounds
Provide a description of any known workarounds.

Related information
Provide any related information

Links to source
Contents of the requirements.txt file
Bindings used

You should be able to use regular expressions to accomplish what you need here. All languages should have regex support and you can use that to pull out the relevant parts of the message that you need. Once you have that, you can rewrite the relevant logs to a separate file and then export that to whatever logging pipeline you have.

Thanks. I tried with config logging format in the python like this
logging.basicConfig(
stream=sys.stdout,
level=logging.DEBUG,
format='%(message)s',
)
However, the log output still contains timestamp and level ahead. I wonder whether azure function defines the log format I cannot override like this way.

You should be able to use regular expressions to accomplish what you need here. All languages should have regex support and you can use that to pull out the relevant parts of the message that you need. Once you have that, you can rewrite the relevant logs to a separate file and then export that to whatever logging pipeline you have.

The Timestamp and Level are not controlled by the worker but are added on by the Host process. Formatting them from within the Python function itself will not change the addition of these entries. @brettsam, might be able to suggest if there is a way to do this directly using some configuration etc. I don't believe we have direct support for scenarios like this one. My suggestion would be to read the logs from the file destination, format them and then right them to wherever you need them.

Closing due to inactivity. Please feel free to re-open if the solution proposed above does not work for you.

Is there anything we can do about logging format? When azure function is deployed in AKS and set to send logs to Log Analytics - all sysout lines are treated separately and log format that insists logging result in two lines is quite unhelpful.
Having logs output in single line is what we are after.


[scan] Skip unavailable os_versions from simclt output

Stop processing simctl output immediately after first "Unavailable" OS as simctl aggregates them at the end of the output

Checklist 🔑

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Motivation and Context
If your machine had already booted simulators that are not available in current's Xcode toolchain, these booted simulators get parsed as usually, but with some invalid os_version (like com.apple.CoreSimulator.SimRuntime.iOS-12-1). This breaks scan command which tries to transform "com.apple.CoreSimulator.SimRuntime.iOS-12-0" into Gem::Version.
Fixes:

https://github.com/fastlane/fastlane/issues/12734

Description
Since simctl always returns all the unavailable OSes below available ones, once we recognize first unavailable OS (using ^-- Unavailable regexp) we don't have to parse simctl output anymore.
Test spec adds a custom simctl output with already booted unavailable device that previously was considered as a valid simulator even it should not.
I was also to reproduce #12734 by given configuration which now succeeds:
xcversion(version: "10.1")
run_tests(device: "iPhone XR")
xcversion(version: "10.0")
run_tests(device: "iPhone XR")

I signed it!

Thanks so much @polac24, this is great work


View: Toggle Zoom Into Focused Editor in Group - tmux-like maximizing of only one editor

This Feature Request is about either adding new setting to Zen Mode, or preferably a new command that would work similarly to tmux zoom feature.
The main difference would be that currently it is only possible to maximize editor group and or toggle Zen Mode thus hiding panels sidebard etc.
The reason why I believe this feature would be useful it a scenario like this:

someone has multiple file or terminal editors opened, maybe even sidebar or panel opened
it is becoming hard to see a specific file in "full context" (sure someone could maximize it, but this would rearange other windows and still left the visible parts of other editors on the edges of the editor area)
one wants to make changes to a file that is opened in really small editor, lets say file3.txt

command or keyboard shortcut View: Toggle Zoom Into Focused Editor in Group is used when focused on file3.txt editor
temporarily other editors, both file and terminal ones are hidden, and the file3.txt editor is displayed as if it was the only one open at this time

potentially it could be also possible to adjust whether sidebar/panel etc. are also hidden on entering this mode, or if this mode stays open after restart
one makes changes to the file and wants to go back to original view state
command View: Toggle Zoom Into Focused Editor in Group is ran once again (alternative to it could be a looking glass icon nexxt to file name or something like this)
original view state from first image is restored without any changes to editor layout

This behaviour could be repeated in such a way that someone always works on "full-screen" editor while also preseving the layout of open editors. This feature is present in tmux, and enables a very efficient way of maanging multiple so called panes inside single window, but in tmux it is possible to create multiple windows.
In VSCode there is no such option which is understandable (multiple windows), but the ability to zoom in and out of specific editor (aka tmux pane) inside a VSCode instance (aka tmux window), in my opinion, would be more that enough.
Lastly, there could be multiple option that modify the behaviour of this zoom view, similarly to existing Zen Mode.

/duplicate https://github.com/microsoft/vscode/issues/16927

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


CLI not selecteable on Windows CMD and Powershell

This is how it looks on Cmd. Probably not intended!

same as me.  i also tired it on my friend's laptop and the result was like this

Workaround is to select the project you intend to use. Example:
flutterfire configure --project=project-name

With the --yes flag, the workaround is to specify all the configurations in the command prompt.
Closing with workaround.