refactor(katana): pending block provider

The motivation is to abstract how pending block data is being fetched in the rpc server. This is to support having different source for where the pending block will be provided from. Right now in sequencing mode, the pending block will be provided by the block producer's state. But in the case for a full node, where block production doesn't happen locally, the pending block information needs to be fetched from the feeder gateway (as right now we want to support syncing from fgw). ref #2724

Summary by CodeRabbit
Release Notes

New Features

Introduced the PendingBlockProvider trait to enhance functionality related to pending blocks, allowing access to pending transactions, receipts, and their statuses.
Updated the StarknetApi to utilize the new PendingBlockProvider, improving interaction with pending transactions.
Added new methods for retrieving pending block data, enhancing the API's capabilities.
Expanded the BlockProducer with methods for accessing pending transactions and their statuses.

Bug Fixes

Streamlined the fetching of pending events, improving clarity and functionality.

Documentation

Added new module for pending functionalities in the storage provider.

the pending block information needs to be fetched from the feeder gateway (as right now we want to support syncing from fgw)

This is done by polling I guess at this point, or do we have a subscription model available from the gateway?

yes, right now we're polling the gateway at a every fixed interval. the gateway doesn't provide a way to subscribe to this change.
https://github.com/dojoengine/dojo/blob/ea0973ca2f5f51ee523493076ad6d6785121be0b/crates/katana/node/src/full/tip_watcher.rs#L34-L47


[CELEBORN-826][DOC] Add storage document

What changes were proposed in this pull request?
As title.
Why are the changes needed?
As title.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
No.

cc @pan3793 @cfmcgrady @AngersZhuuuu @RexXiong

Thanks, merged to main.


Tried to install Keripy following instructions in readme.me

The installation runs smooth on a Mac OSX (see attachment for details)
But in the end I get two errors I can't decipher. See attachments.

My goal is to set up a test and demo environment and present how KERI works to others like Phil did for IIW.

What is the output of python3 --version?

(keripy) hvc@Henks-Mbp20 keripy-1 % python3 --version
Python 3.9.5

so I would start by updating to 3.9.7 this is required by keripy
https://github.com/WebOfTrust/keripy/blob/0491dad8c589d5214253941d93b81847ea11703c/setup.py#L71
we use pyenv to manage python versions, that will fix the second issue.

Thx, Kevin, the guideline said "python 3.9.1+", but I'll try and fix it.
pyenv -> do I need to install this, or just call it? Both? Any switches?

pyenv install 3.9.7
then you can set it for a specific local directory, so when inside keripy you could do:
pyenv local 3.9.7
which will leave your system python alone.

With regard to the python version, something I can't grasp is going on:
hvc@Henks-Mbp20 keri-1 % python3 -m venv keripy
hvc@Henks-Mbp20 keri-1 % source keripy/bin/activate
(keripy) hvc@Henks-Mbp20 keri-1 % python3 --version
Python 3.9.5
(keripy) hvc@Henks-Mbp20 keri-1 % brew upgrade python
Warning: python 3.9.10 already installed
(keripy) hvc@Henks-Mbp20 keri-1 %

Managing multiple installations of python can be a pain.
That is why we use pyenv.

I've installed peen, read the documentation, followed the guidelines...
(keripy) hvc@Henks-Mbp20 keripy-1 % source keripy/bin/activate
(keripy) hvc@Henks-Mbp20 keripy-1 % pyenv local 3.9.7
(keripy) hvc@Henks-Mbp20 keripy-1 % python3 --version
Python 3.9.5
It wasn't meant to be :) I don't want to be learning pyenvs, I'd like to work with keri.
Thx, for your tries, Kevin. I'll wait for a readily available test Docker container with everything tested and working?


config('value') is not working on cron

Laravel Version: 5.3.22
PHP Version: 7.0.11
OS: ubuntu 14.04.4

Description:
I was trying to send an email from a job. It was actually calls an external API and I store the URL in config/services.php file like this-
<?php

return [

    'mailgun' => [
        'domain' => env('MAILGUN_DOMAIN'),
        'secret' => env('MAILGUN_SECRET'),
    ],

    'ses' => [
        'key' => env('SES_KEY'),
        'secret' => env('SES_SECRET'),
        'region' => 'us-east-1',
    ],

    'sparkpost' => [
        'secret' => env('SPARKPOST_SECRET'),
    ],

    'stripe' => [
        'model' => App\User::class,
        'key' => env('STRIPE_KEY'),
        'secret' => env('STRIPE_SECRET'),
    ],

    'sm_api' => env('SM_API', 'api-url'),

];

I also use this sm_api value from frontend like this config('services.sm_api'). There it works like a charm. but when I try to use the same value from a laravel job class, which actually dispatched from Redis queue server, the same function call returns null. I tried to log the whole config()->all() and it returns the predefined values, except sm_api.
array (
    'mailgun' => array (
        'domain' => NULL,
        'secret' => NULL,
    ),
    'ses' => array (
        'key' => NULL,
        'secret' => NULL,
        'region' => 'us-east-1',
    ),
    'sparkpost' => array (
        'secret' => NULL,
    ),
    'stripe' => array (
        'model' => 'App\\User',
        'key' => NULL,
        'secret' => NULL,
    ),
)

It also shows mailgun's domain and secret value to null though I set it in the .env file. I also log the env('SM_API') value, and it works from the same job. So, the problem is I can bear with env('SM_API') from the job, but I can use mail, as the mailgun driver and key is null. But all this works fine from the frontend. I also tried it in php artisan tinker, and it works perfectly.
I searched on google for this and found this. This issue suggest to use php artisan config:cache to resolve this. I tried this as well. But nothing happens.

The issue you shared contains everything that can be said basically :) Not sure why you opened a new issue.

@themsaid I can't solve the problem. Any suggestion that I can do?
As it works perfectly on web end and in php artisan tinker, but not in cron. So, I thought there must be something that you can do.

When you say cron do you mean using the laravel scheduler? If so then it should work since the scheduler just call a console command in same environment as tinker

@themsaid yes, I am using laravel schedular. And I know, tinker and cron are same environment. That's the problem here. It is so weird that I can't even ask anyone to solve.
I just checked it again. In the tinker, it returns this value for config('services')-
[
     "sm_api" => "my-api-url",
     "mailgun" => [
       "domain" => "mailgun-domain",
       "secret" => "mailgun-secret",
     ],
     "ses" => [
       "key" => null,
       "secret" => null,
       "region" => "us-east-1",
     ],
     "sparkpost" => [
       "secret" => null,
     ],
     "stripe" => [
       "model" => "App\User",
       "key" => null,
       "secret" => null,
     ],
   ]

but in the cron it returns this-
[
     "mailgun" => [
       "domain" => null,
       "secret" => null,
     ],
     "ses" => [
       "key" => null,
       "secret" => null,
       "region" => "us-east-1",
     ],
     "sparkpost" => [
       "secret" => null,
     ],
     "stripe" => [
       "model" => "App\User",
       "key" => null,
       "secret" => null,
     ],
   ]

I also checked the bootstrap/cache/config.php file where these configs are cached. It looks fine. Only in the console it is not working. I am very depressed with it. I spent my last 18 hours with it find the problem. Couldn't find anything. :(

Did you try changing environment to production after caching configurations?

@lkmadushan yes, I tried. but nothing happens. And in tinker, env('value') is always return null.
For example-
>>> config('app.env')
=> "production"
>>> env('APP_ENV')
=> null

It's really weird.

@themsaid I think, in the cron the configuration is not loaded from the bootstrap/cache/config.php file. Could you please check?

@milon
It is weird. I'm working on linux mint (Ubuntu 14.04.2 LTS) on a homestead machine (Ubuntu 16.04 LTS) with PHP 7.0.8, laravel/framework v5.3.22 and I can't reproduce your problem.
Furthermore: I am sending mails – triggered from the schedule method / cron – that successfully fetch a lot of config values.
Brief tests have shown that config values are read on my system:
protected function schedule(Schedule $schedule)
{
        Log::info(config('app.env'));
        Log::info(config('services'));
}

Output for APP_ENV=production:
$ tail -f storage/logs/laravel.log
[2016-11-09 22:33:01] production.INFO: production
[2016-11-09 22:33:01] production.INFO: array (
  'mailgun' =>
  array (
    'domain' => NULL,
    'secret' => NULL,
  ),

We also have custom configuration values in services and they show up in the log properly. The output matches the output in artisan tinker:
$ php artisan tinker
Psy Shell v0.7.2 (PHP 7.0.8-2+deb.sury.org~xenial+1 — cli) by Justin Hileman
>>> config('app.env')
=> "production"
>>> env('APP_ENV')
=> "production"
>>> config('services')
=> [
     "mailgun" => [
       "domain" => null,
       "secret" => null,
     ],

php artisan tinker shows our custom services very well. No differences to the output from schedule.
Maybe something is messed up on your system?
Your example
>>> config('app.env')
=> "production"
>>> env('APP_ENV')
=> null

lets me think that env('APP_ENV') is always null on your system, because config('app.env') is resolved with fallback 'production'. You can see that in config/app.php:
'env' => env('APP_ENV', 'production'),

Maybe something on your system prevents setting environment variables?

Also, have a look at your crontab and compare it to the one, homestead set up for me:
/etc/cron.d/{appname}app <- root:root -rw-r--r--
* * * * * vagrant /usr/bin/php /home/vagrant/{appname}/public/../artisan schedule:run >> /dev/null 2>&1

The .env file is not loaded (accurate for 5.2 and 5.3) if you have cached your configuration files. This is the intended behavior. You should only call env() from your config files to help build the cached configuration, and then call config() in your code.

@sisve, I am doing that exactly. I am using this bit of code config('services.sm_api') and it's return null. Any idea why?

Yes, 1) you have an old cached configuration file, or 2) you already have a SM_API environment variable so the one in .env isn't used.
It's most probably number 1.
There are a few other common cases; like your queue being handled/executed on another machine that you havn't updated or you're censoring things in your answers to the degree that we no longer understand the problem.

@sisve, I also suspect that I have an old cached config file. But where it is located. Or how can I remove that?

You can remove it with php artisan config:clear or manually delete the file bootstrap/cache/config.php

@sisve, I removed it. then create it again with php artisan config:cache. But no luck.

@milon same as me

In my case php artisan config:clear works.

For me, php artisan config:cache worked. (config:clear didn't).
I don't know why we have these config related issues so much.

Another reason can be that your environment defines another environement variable.
Typically, I have export ALGOLIA_APP_ID=blah in my .dotfiles so when I run Laravel in console mode, this takes precedence over the .env file. In HTTP or Tinker context my config is not loaded so everything works as expected.
Hope it helps someone one day.

I had same problem when i changed alpine docker image to debian. On debian cron have specific behaviar https://manpages.debian.org/jessie/cron/cron.8.en.html
In my case i add command to container run script:
# Set env vars for cron jobs printenv > /etc/environment


YouTube Music (desktop apps and website)

Hello,
First of all thanks to @MayhemBill for his "YouTube Music Desktop" theme (from #751, here his repo, here the Dracula repo and here the Dracula webpage), it works very well.
I should add that it works for me with another desktop app for YouTube Music : "YouTube Music Desktop App" (repo ; Options > Visual Tweaks > Theme > Import custom CSS file ; and yes, this app name differ only with the "App" at the end) as much as with the YouTube Music official website via Stylus.
It might be worth mentioning these compatibilities on the draculatheme.com website, or even creating two specific repo and adding these applications to draculatheme.com (which unfortunately is beyond my skills).

I will add a list of compatible sites/apps on the YTMD repo I have, since it just works™


Added botinfo.ts

Added Bot Information Command Which Provides Multiple Information Regarding The Bot Such As The Operating System On Which It Is Running or The Architecture, etc.

This command is not neccessary for a simple boilerplate. Also, this template uses Bun, not Node.js. I will be posting contribution guidelines and feature suggestions in a bit if you wish to continue contributing. :)

I would love to contribute. Actually this was my first open source contribution so I couldn't understand the template thing. Can You just tell me what command I can add to it ?


bottom_navigation_demo can not touch when change the tab

https://github.com/flutter/flutter/blob/dev/examples/flutter_gallery/lib/demo/material/bottom_navigation_demo.dart
in this demo, I add a button on each view, but when I change the tab, the button can not be touched.

Hi @jiqimaogou
Could you please provide your flutter doctor -v?
If I understand correctly the issue is that during the animation
the icon loses gestures detection, correct?
Thank you

Hi @jiqimaogou
Could you please provide your flutter doctor -v?
If I understand correctly the issue is that during the animation
the icon loses gestures detection, correct?
Thank you

chenhjohn@chenhJohndeMacBook-Pro:~/Downloads/mediapipe% flutter doctor -v
[✓] Flutter (Channel beta, v1.10.7, on Mac OS X 10.14.6 18G87, locale zh-Hans-CN)
• Flutter version 1.10.7 at /Users/chenhjohn/Library/Android/sdk/flutter
• Framework revision e70236e36c (5 weeks ago), 2019-10-02 09:32:30 -0700
• Engine revision 9e6314d348
• Dart version 2.6.0 (build 2.6.0-dev.0.0<PHONE_NUMBER>)

replace the icon with a button, it happended.
https://github.com/flutter/flutter/blob/dev/examples/flutter_gallery/lib/demo/material/bottom_navigation_demo.dart#L61

@iapicca can you see it?

Closing this issue since it is no longer relevant here.
@jiqimaogou if you can provide a minimal complete reproducible code sample that demonstrates this on the latest version of flutter I can reopen it.


Supporting translation

Observed behavior (please include a screenshot if possible)
A few months ago translation was added to GTFS core specifications, but until now there is no support for it.
Expected behavior
The editor should allow providing translated names of records such as stops, routes, calendars, etc...
Any special notes on configuration used
In the past translation was part of Google Transit Extensions to GTFS for serval years, but now it is part of the core specifications. One issue is that the old format and the new format uses the same file translations.txt, but the content of the files has changed in a way they are not compatible. And there's still feeds that uses the old format.
Version of datatools-ui and datatools-server if applicable (exact commit hash or branch name)
82ee96632af47851c469182a3becaee5eef36c23

The translation proposal was merged on Feb 15, 2019 google/transit#180


IMPRO-1767: Improve the way we work with checksums

This PR modifies the way a user recreates the checksum file and provides the option to ignore the checksum tests if so desired.
Recreate checksums: bin/improver-test recreate_checksums
Ignore checksums: export IMPROVER_IGNORE_CHECKSUMS=true
Testing:

[x] Ran tests and they passed OK

 Here is an overview of what got changed by this pull request:

Complexity decreasing per file
==============================
+ improver_tests/acceptance/test_checksums.py  -1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity decreasing per file
==============================
+ improver_tests/acceptance/test_checksums.py  -1

See the complete overview on Codacy


Docs missing "names" event.

The docs are missing the "names" event wich is handled after a join ;)
It is correctly handled by tmi but not included in the docs.
Greetz

Merged PR tmijs/docs#1 :+1:


Improve internal naming schemes to work with autocompletion in jupyter notebooks

Summary
Some internal names like e.g. ThreeRollPass 'Pass4' are incompatible with autocompletion in jupyter notebooks.
They are not displayed in the autocompletion list.
This is at times quite comfusing when looking for specific results.
It would be better if internal names are created in a way so that they support autocompletion.

Do you refer to the string representation of units?
Where does this affect autocompletion?

Yes, I refer to the string representation of units.
I think autocompletion can't deal e.g. with . and space.
Respective units do not appear in the autocompletion list which is confusing to the user.

In the list above I would expect the ThreeRollPass units to be present in the autocompletion list.

What is this Rollpass object in your case? I do not know of any where the units were accessible by label. But this may be an idea to implement an indexer in PassSequence which allows to get units by label like sequence["Oval III"] if sequence is a PassSequence and there is a unit with label="Oval III" in.


Reducing max_seq_length

Does reducing max_sequence_length before finetuning conflict with anything that might cause a problem? not in terms of model performance

I don't think that it causes any problems.
(But in case it does, feel free to let us know that!)

I changed num of epochs to 2 and max_seq_len to 100. Training started and continued to batch 57 then stopped with the assertion error in the image. When I tried a max_seq_len less than 100, training didn't start due to a cuda error.

Note that I'm working on NER and I'm not using conll2003, I use another dataset with the exact same format, and I think the only difference is it has mention lengths greater than original conll entities.

I changed max_seq_length in this script: luke/examples/legacy/ner/main.py

By default GPU training just gives random error messages, and that makes it hard to detect the problem.
Could you run the same training run on CPU or set the environment variable like export CUDA_LAUNCH_BLOCKING=1?
I think this will show where the problem is.

@Ryou0634
This is the error appeared when I ran the script on a cpu.

Does this mean there is a conflict between the dimension of word_hidden_states of the pretrained model and the entity_start_positions created based on the max_sequence_length I used?

start_states = torch.gather(word_hidden_states, -2, entity_start_positions)
Tracing what this line depends on, I found out that the function used to create features in legacy/ner/utils.py produces some  entity_start_positions that are inconsistent with the specified max_sequence_length, but I haven't figured out why this happens yet. the whole context length every time is calculated correctly and is constrained with max_sequence_length, but it's not clear yet how some entity_start_positions are larger than max_sequence_length. I'm debugging it now, but kindly if this behaviour is expected for some obvious reason could you tell me what it is?

The issue is: according to "convert_examples_to_features function", left and right contexts are always calculated regardless of checking the original sentence length first as it might be itself => the max_sequence_length.


Discovering Python interpreters takes a long time

Type: Bug

Behaviour
Expected vs. Actual

Launch VS Code insiders after an update
The "Discovering Python Interpreters" step takes a very long time (I didn't time it), so I don't see my Python environments in the Jupyter kernel picker for a while
Close VS Code insiders
Launch VS Code insiders again (no update this time)
The "Discovering Python Interpreters" step also takes a very long time (~40 seconds for that operation alone), but since Python environments were already loaded from previous session, they still do show up in the Jupyter kernel picker

Diagnostic data

Python version (& distribution if applicable, e.g. Anaconda): 3.8.5
Type of virtual environment used (e.g. conda, venv, virtualenv, etc.): Conda
Value of the python.languageServer setting: Default

Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)

XXX

User Settings

languageServer: "Pylance"

Extension version: 2022.13.12212101
VS Code version: Code - Insiders 1.71.0-insider (dced70bbf36d3c53c08e791da1791ac7fc42519b, 2022-08-10T05:17:53.560Z)
OS version: Windows_NT x64 10.0.22000
Modes:
Sandboxed: Yes

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-1065G7 CPU @ 1.30GHz (8 x 1498)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
15.60GB (1.34GB free)

Process Argv
--crash-reporter-id 7c55b80b-a2c8-4229-8934-3646319b9311

Screen Reader
no

VM
0%

A/B Experiments
vsliv695:30137379
vsins829:30139715
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythonvspyl392:30422396
pythontb:30258533
pythonvspyt551cf:30291413
pythonptprofiler:30281269
vshan820:30294714
pythondataviewer:30285072
vscod805:30301674
bridge0708:30335490
bridge0723:30353136
vsaa593cf:30376535
pythonvs932:30404738
wslgetstarted:30449409
cppdebug:30492333
pylanb8912:30522163
vsclangdf:30492506
c4g48928:30535728
dsvsc012:30540213

Please provide the logs as mentioned in the issue template. Set python.logging.level to debug in User settings, reload window, then perform the steps to reproduce before providing the logs.

Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)

XXX

I think you have to change version

Discovering some of these envs may require activating them which can take more time. Please provide the logs as mentioned in the issue template which has this information. Set python.logging.level to debug in User settings, reload window, then perform the steps to reproduce before providing the logs.

Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)

XXX

Once Jupyter starts using proposed discovery APIs, discovering interpreters won't be auto-triggered at startup and only when the kernel/interpreter list is opened, so a lot of this problem goes away.

discovering interpreters won't be auto-triggered at startup and only when the kernel/interpreter list is opened, so a lot of this problem goes away.

Won't this mean listing interpreters will be slower? See below why i ask this.

they still do show up in the Jupyter kernel picker

Based on her comments, the issue is the python envs are not showing up in the list for a while. And now if we trigger the query only when the notebook is opened, then the list will be fetched even later in the process hencing slowing the discovery even further.

Allow me to clarify; discovery will only be auto-triggered by the Python extension the first time a workspace or scope is opened. In the following sessions, it'll up-to the individual extensions as to when it wants to trigger it. An example could be to trigger it when the picker is opened, and the picker is refreshed with any new envs it finds, while not blocking on loading the list.

Based on her comments, the issue is the python envs are not showing up in the list for a while.

Yes, but this will only be an issue the first time Python extension loads.

Yes, but the fact that discovering interpreters is slow shouldn't matter as much as it would not block the startup of the extension.
List is loaded immediately with envs founds in the previous session, and for any new envs, it'll be refreshed accordingly within those 40 seconds.

matter as much as it would not block the startup of the extension. (language servers etc.)

If it will not block the extnesion, then i don't see why we should even wait like you have suggeste

discovering interpreters won't be auto-triggered at startup and only when the kernel/interpreter list is opened, so a lot of this problem goes away.

I.e. i don't see why we need to wait till later, that would be be slower.
Anyways, just a thought. I guess Ian can look into that and figure out the pros/cons and the optimal solution.
FYI - @IanMatthewHuff will be working on integrating this API into the Jupyter extension.

If it will not block the extnesion

Thing is running discovery at startup does block starting other features in the extension, because of the single threaded nature of Node. Even though it's run in background, practically it seems to occupy so much clock that other features are not started up. That is the reason for waiting until later.
Particularly for the Python extension, refreshing the list when the picker is opened isn't so bad as it updates the list in place. But each extension is free to decide what's best for them.

Thing is running discovery at startup does block starting other features in the extension, because of the single threaded nature of Node. Even though it's run in background,

Thanks for the explanation, however I am aware of this (my comment was a question based on some assumptions, hence prefixed with if).
I've suggested using workers in the past to over come such blocking scenarios, I thought the plan was to implement such a technique, at least thats what I was informed when I demoed some perf improvements in the past https://github.com/microsoft/vscode-python/issues/15813

I think we can continue the discussion of the API in the Jupyter repo thread (the issue you created for the adoption of the new API)

using workers in the past to over come such blocking scenarios

Interesting, I think the last comment https://github.com/microsoft/vscode-python/issues/15813#issuecomment-812590160 indicated it wasn't a priority so we missed to do it. cc/ @karthiknadig


git-pulls.rb:301: syntax error, unexpected keyword_end, expecting   $end

The gem has problems with my current installation:
/Users/bodo/.rvm/rubies/ruby-1.9.3-p0/lib/ruby/site_ruby/1.9.1/rubygems/custom_require.rb:36:in `require':     /Users/bodo/.rvm/gems/ruby-1.9.3-p0/gems/git-pulls-0.3.4/lib/git-pulls.rb:288: syntax error, unexpected ':', expecting keyword_then     or ';' or '\n' (SyntaxError)
/Users/bodo/.rvm/gems/ruby-1.9.3-p0/gems/git-pulls-0.3.4/lib/git-pulls.rb:301: syntax error, unexpected keyword_end, expecting   $end
from /Users/bodo/.rvm/rubies/ruby-1.9.3-p0/lib/ruby/site_ruby/1.9.1/rubygems/custom_require.rb:36:in `require'
from /Users/bodo/.rvm/gems/ruby-1.9.3-p0/gems/git-pulls-0.3.4/bin/git-pulls:4:in `<top (required)>'
from /Users/bodo/.rvm/gems/ruby-1.9.3-p0/bin/git-pulls:19:in `load'
from /Users/bodo/.rvm/gems/ruby-1.9.3-p0/bin/git-pulls:19:in `<main>'

Same here

+1

If you are using Ruby 1.9.1 or later, you must change the colon (:) to a semi-colon (;)
This was changed in Ruby 1.9.1
http://svn.ruby-lang.org/repos/ruby/tags/v1_9_1_0/NEWS

    * Deprecated syntax
          o colon (:) instead of "then" in if/unless or case expression.

I dont know if semi-colon will work across all Ruby versions.
So the safe alternative is to replace colon (:) with the keyword "then".


⚠ Panel - Hydeos has degraded performance

In a43db60, Panel - Hydeos (https://pterodactyl.hydeos.fr) experienced degraded performance:

HTTP code: 200
Response time: 1242 ms

Resolved: Panel - Hydeos performance has improved in 7fc35b5 after 23 minutes.


Optional quest mode

It should be possible to switch the mod to a quest log and calling tasks quests.

Do you see this as an option or a key stroke switch?
Le ven. 2 nov. 2018 à 11:39, Jonas<EMAIL_ADDRESS>a écrit :

It should be possible to switch the mod to a quest log and calling tasks
quests.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/JonasJurczok/factorio-todo-list/issues/74, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAbWmwNq8mK4NeAZLHHam8brrddmUXc5ks5urCDMgaJpZM4YLazg
.

I would say an option.
I think the idea is fun but has limited use cases :)

Should the quests be repeatable? Should we stock the quests resolver instead of task assignee?

I like this idea as well. I made some changes to the language file for English to change button labels around and things. I could clone this repo and submit my changes as a pull request, though I think that way of doing things is a bad idea for this change. But it might be a useful jumping off point for discussion.

Can't realy speak in Jonas' place but I think he would prefer a clean PR for every feature.

Sorry, totally forgot to reply here.
I appreciate the pull request but I would definitely put this behind an
option. Otherwise everyone has to go full RPG mode :)


Service references should be allowed in isolated functions only if the type is a subtype of readonly

Description:
$title. Needs to be done with the service changes. May have to remove the check for services at https://github.com/ballerina-platform/ballerina-lang/blob/v2.0.0-Preview4/compiler/ballerina-lang/src/main/java/org/wso2/ballerinalang/compiler/semantics/analyzer/IsolationAnalyzer.java#L1426.
Related Issues (optional):
https://github.com/ballerina-platform/ballerina-lang/issues/25954
https://github.com/ballerina-platform/ballerina-lang/issues/26064

Fixed in #27414, merged via #27652.

Fixed in #27414, merged via #27652.


suggestion you write full namespace in your code thank you.

such as:
services.ConfigureSwaggerGen(options =>
{
options.SingleApiVersion(new Info
{});
});
some developer don't use Visual Studio,so we're unable to know the namespace of the class Info.
i had take too much  time for resolve this issue and very disappointed.
thank you.

Omnisharp - offers intellisense cross platform within Vim, Emacs, Visual Studio Code, Brackets and other text based editors.  Have you tries this add-in?


Switch bcrypt lib

jBCrypt hasn't been updated since 2015 and supports only $2a$ version.
It can't validate passwords hashed with $2y$ version (ex: generated through PHP).
I switched to another lib that is more recent, maintained and under Apache 2.0 license.

Hi @Minimoi , Im sorry but I dont handle this repo and havent seen any merge since I started using this custom user federation

Hi @Minimoi and @dasosjt  thank you all for your support, I am merging this right now and will be closing a new release with your contributions as soon as possible.


Use pytorch metrics instead of rs_metrics

Not sure whether it's always possible or not

It looks like the format is very different, so we will need to reformat everything


Could you clarify which async functionality is implemented?

Hi,
I'm a little confused which operations can be executed in async way and how to achieve it.
What I found

async reduce requires additional wrapper as global function or async with lambda https://github.com/thrust/thrust/blob/master/examples/cuda/async_reduce.cu

for_each no need it https://github.com/thrust/thrust/blob/master/examples/cuda/simple_cuda_streams.cu

There is an overload of copy function that accepts exec policy, so I can provide a stream. Will it result in async copy from host2device. Do I need to wrap that call in global function or async? I found only one  example on SO http://stackoverflow.com/questions/25064383/how-to-asynchronously-copy-memory-from-the-host-to-the-device-using-thrust-and-c it works with guaranty, but it uses a native api.

reduce, copy_if, transform_reduce, and other algorithms which return computed result are blocking. In part, because D2H copy is required before returning result to the calling thread.

for_each is non-blocking. The return value can be, and is computed on the host, and no D2H copy is required before returning control to the calling thread.

If stream is provided via the execution policy, Thrust should synchronise in this stream only. H2D, D2D and D2H copies are implemented via cudaMemCopyAsync+sync in the provided stream. Thus copies are blocking.

If asynchronous execution is desired, the best bet would be to spawn a thread, e.g. via async, which will call Thrust algorithm. Please be mindful most of the Thrust algorithms allocate temporary storage. If you use multiple threads to invoke Thrust algorithms, it is best to provide a custom thread-safe allocator (e.g. [1]) via the execution policy (e.g. [2]).
[1] https://nvlabs.github.io/cub/structcub_1_1_caching_device_allocator.html
[2] https://github.com/thrust/thrust/blob/master/examples/cuda/custom_temporary_allocation.cu

@egaburov  thank so much for detailed answers. In my case all operation don't return result, so they should work in async manner.
According to http://stackoverflow.com/questions/14093601/effect-of-using-page-able-memory-for-asynchronous-memory-copy I also should use pinned memory to get full async behavior. Is it correct?
There is a experimental pinned_allocator. Are you going to continue maintain it?
I'm asking so much questions because it's not clear is it worth to use thrust with all its limitations(and inactive development) or focus on native api.

I do not think Thrust spells out guarantees of async behaviour, hence "should work" doesn't not necessarily imply "will work". If you absolutely must have async behaviour, perhaps spawning a separate thread is what you need.
cudaMemcpyAsync requires pinned host memory buffer for non-blocking operation. thrust::copy, however, is a blocking call, whether the host buffer is pinned or not.
Hope this helps.

I'm going to use a trick from http://stackoverflow.com/questions/25064383/how-to-asynchronously-copy-memory-from-the-host-to-the-device-using-thrust-and-c

or try experimental pinned_allocator

@egaburov according documentation thrust:for_each returns  InputIterator. So it should block current thread(and only a few methods in Thrust return void). Does it mean all of them are blocking?

Let me rephrase it differently. Thrust doesn't spell out guarantees of non-blocking behaviour, and its current interface assumes blocking operation. for_each, and I think the other for_each-based algorithms, is an exception rather to this. It is not guaranteed that they will stay non-blocking.
Hope this helps.

I've finally decided to use cub. It gives more control.
Спасибо за ответы.

We are looking for user feedback to improve Thrust. May I ask you to elaborate with kind of control CUB provides you, and what would you like to have in Thrust to accomplish your tasks?
Please feel to file an RFE, either via GitHub issue or by submitting a bug via formal NVIDIA bug tracking system, to tell us what  unctionality would you like to see in Thrust.
Thank you.

First of all I don't have a lot experience with cuda and thrust, so my opinions could be really naive.

First of all it's async behavior(to mix memory copy and computation).  As you proposed I can spawn a thread and operation will block that thread, but my code becomes over complicated. I can do the same with cuda stream(s) and stream sync.

A lot of algorithms use temp data. It's worth to reuse it to eliminate cudaMaloc, cudaFree. Cub allows me to do it explicitly or with caching device allocate.

More control on algorithm. like begin_bit, end_bit for Radix sort http://nvlabs.github.io/cub/structcub_1_1_device_radix_sort.html#a31d7224d3f6c6a9309a0b84571f25eb9

More general observation. Sometimes cuda effective implementation is 'hacky' and takes into account tiny little details specific for a certain architecture. It's good to have template framework like Thrust as a starting point when you learn cuda, but when you go deeper it's easier to (at least for me) learn some native mechanism(streams, async memory copy, etc) and cub don't restrict you with that. Thrust in contrast is to abstract(I know you have to support cuda, openmp, etc) and not so transparent as cub is.

Thank you for your feedback.

A lot of algorithms use temp data. It's worth to reuse it to eliminate cudaMaloc, cudaFree. Cub allows me to do it explicitly or with caching device allocate.

Have you considered using custom allocators which can supplied via execution policy [1]

More control on algorithm. like begin_bit, end_bit for Radix sort

This is only makes sense for radix sort. But I suppose, what you imply is that it would be good if Thrust,  in addition to generic comparison sorts thrust::sort, thrust::stable_sort, would also have thrust::radix_sort with an optional argument to control bit range.
[1] https://github.com/thrust/thrust/blob/master/examples/cuda/custom_temporary_allocation.cu

Have you considered using custom allocators which can supplied via execution policy [1]
Yeap, but in my case algorithm looks like
[h2d copy][allocate tmp data][computation][d2h copy]
If I do it in parallel
[h2d copy][allocate tmp data][computation][d2h copy]
[h2d copy][allocate tmp data][computation][d2h copy]
[h2d copy][allocate tmp data][computation][d2h copy]
[h2d copy][allocate tmp data][computation][d2h copy]
it's more efficient to allocate continuous peace of memory for a temp data when you start and reuse it. So I don't event need a cub::CachingDeviceAllocator. And cub allows just provide pointer for a temp storage. For me it's easier and more readable than implement custom allocator.
This is only makes sense for radix sort. But I suppose, what you imply is that it would be good if Thrust, in addition to generic comparison sorts thrust::sort, thrust::stable_sort, would also have thrust::radix_sort with an optional argument to control bit range.
I will rephrase, a lot of algorithms cab be configured to leverage performance by using  architecture specific features. cub::BlockHistogram could be done by BLOCK_HISTO_SORT  or BLOCK_HISTO_ATOMIC for instance. It's really tremendous amount of work to support multiple device architecture(cuda, openmp, etc) and the same level of native features. Might be you should consider to focus on friction-less usage of low level libraries(like cub) inside Thrust. That will help developers to get desirable level of control and performance but keep STL like code.


add mq configurations

what
This change is to allow users to configure MQ configurations with this module
why
With current version there is no way to attach configurations to brokers
references
https://github.com/cloudposse/terraform-aws-mq-broker/issues/66

/terratest

/terratest

/terratest

@danushkaf can you update this pr?
README.md is outdated. Please run the following commands locally and push the file:
  make init
  make readme

That way we can continue to process it, otherwise it's likely to be closed.

Hi @hans-d, could we potentially revisit this PR? I don't know much about the failing tests, but it seems rather straightforward to me.


Adds basic FFaker::Address.time_zone support

Thanks for the review @marocchino. I've added both the assert_include FFaker::Address::TIME_ZONE comparison, and a more sensible regex.

@dugjason Thanks, but we don't need both because It's too verbose. >_< Could you choose one of them please?

Sure thing. I just included both examples for completeness, but I see it's one of the only tests with more than one test case.

:+1:


Allow multiple read from IPC + some refactor

The previous implementation was closing the connection right after 1 message (ligne 87) so i couldn't send multiple request with the same socket, this fix it.

that is intentional though.

Why ?

reading more can block, besides, there is no point. Just open a new socket connection?

How can it block ? the only thing that could happen is the client crash right after the data is send (so it doesn't client doesn't read hyprpaper message) and in that case it crash hyprpaper but it can also happen the old version. Also when the client disconnect it will send a message of 0 char and that is expected, with if (messageSize == 0) break; (line 61) it will close the socket connection. Nothing should block ?

the client can send a message and then do nothing, blocking hyprpaper.

its the same thing with the old one ? For the old one it will block at line 62 waiting for the data but the new one will block at line 59.
Also the old one could crash if you connect but immediately exit because  it would then try to `write(ACCEPTEDCONNECTION, m_szReply.c_str(), m_szReply.length());' having no connection to send (i didn't test it but it should do it)

true that... Fine I guess. I'll review this tomorrow. One thing I can definitely say this MR needs is a clang-format


ARROW-9283: [Python] Expose build info

There's a problem where we already generate __version__ and it ends up different, for example:
>>> import pyarrow as pa
>>> pa.__version__
'0.17.1.dev572+gf25a014ab.d20200701'
>>> pa.version
'1.0.0'
>>> pa.version_info
VersionInfo(major=1, minor=0, patch=0)
>>> pa.build_info
BuildInfo(version='1.0.0', version_info=VersionInfo(major=1, minor=0, patch=0), so_version='100', full_so_version='100.0.0', git_id='5f1800a4dfaf38754362914faf2776d6c3ec9eb9', git_description='apache-arrow-0.17.0-568-g5f1800a4d', package_kind='')

@xhochy @kou What is your advice on this?

How about using our SNAPSHOT version as the next version of pyarrow?
diff --git a/python/setup.py b/python/setup.py
index 4c264a2d7..bc3efee77 100755
--- a/python/setup.py
+++ b/python/setup.py
@@ -546,6 +546,15 @@ def parse_git(root, **kwargs):
     return parse(root, **kwargs)

+def guess_next_dev_version(version):
+    if version.exact:
+        return version.format_with('{tag}')
+    else:
+        def guess_next_version(tag_version):
+            return default_version.replace('-SNAPSHOT', '')
+        return version.format_next_version(guess_next_version)
+
+
 with open('README.md') as f:
     long_description = f.read()

@@ -595,7 +604,8 @@ setup(
         'root': os.path.dirname(setup_dir),
         'parse': parse_git,
         'write_to': os.path.join(scm_version_write_to_prefix,
-                                 'pyarrow/_generated_version.py')
+                                 'pyarrow/_generated_version.py'),
+        'version_scheme': guess_next_dev_version
     },
     setup_requires=['setuptools_scm', 'cython >= 0.29'] + setup_requires,
     install_requires=install_requires,

pyarrow.__version__ will return "1.0.0.dev..." with this change.
And how about exporting -SNAPSHOT tag as arrow::BuildInfo::version_tag? It will show that this version is a snapshot version like .devXXX in Python.
And how about adding cpp_ prefix to Arrow C++ related versions like pa.cpp_version to show that they show C++ version not pyarrow version?

@kou I don't understand. Do you mean we should have different version numbers for Arrow C++ and PyArrow? That sounds terribly confusing.

Yes.
We may release a patch version (1.0.1 for 1.0.0) that should be compatible with base version (1.0.0).
If a patch version includes fixes only in C++, source pyarrow package users don't need to upgrade pyarrow. They just need to upgrade Arrow C++. In this situation, pyarrow's version is 1.0.0 and runtime Arrow C++ version is 1.0.1.

While it may be supported to do so (upgrade only Arrow C++ DLLs), I'm not sure it's something we want to encourage, and exposing different version numbers is confusing for users.
@wesm What do you think?

I think the scenarios where the C++ library would be upgraded but not the Python library are likely to be infrequently occurring, that said I think it would be useful to be clear that the version info that's being shown was produced by the C++ library. I could imagine actually people creating multiple C++ builds like $VERSION-avx2 $VERSION-avx512

I implemented some changes:

Passing through ARROW_VERSION unmodified to arrow/util/config.h, so it will say 1.0.0-SNAPSHOT now
Add cpp_ prefix to the Python variables
Add Kou's version scheme suggestion

so now we have
In [1]: pa.show_versions()
pyarrow version info
--------------------
Arrow C++ library version: 1.0.0-SNAPSHOT
Arrow C++ git revision: b396060b1437149523248b0eb50d7101f75d7034
Arrow C++ git description: apache-arrow-0.17.0-677-gb396060b1

+1.  If the setup.py changes create problems for the release we will have some time to fix things


Auro select doesn't handle long values on a mobile device well

Describe the bug
When an extra long value is inside auro-select on a mobile device the select overflows its container.
To Reproduce

Put a really long value in the select,
Toggle on the mobile device emulator in dev tools in your browser of choice
Scroll to the side.

Expected behavior
Maybe ellipsize the value? This decision would be better made by a UX/UI designer.
Screenshots

Desktop (please complete the following information):

Browser [Firefox, mobile safari]

auro-menu: 3.9.2
auro-select: 2.5.5
auro-dropdown: 2.9.1

This issue needs to be reviewed by @leeejune and the other UI Designers in order to come up with a solution that fits the element.
We moved this from auro-select to menu as we assume that is where the issue REALLY is. Also, there is nothing to say that this same issue will not happen with auro-combobox as well.
@leeejune please consult wth @jason-capsule42 in order to come to a design resolution for this issue and then we can move this to official refinement with @Lewisfuuh

Currently @jdeeezzy is working on a mobile version of dropdown to resolve issues like this.

Moving to backlog. This issue will be reviewed once the new design spec has been completed.

@fajar-apri-alaska can you validate if this is still an issue or not?
-- Thanks!

Per latest version (2.10.1), it is currently still an issue per below video:
https://github.com/AlaskaAirlines/auro-menu/assets/124668226/73f0fc69-6f87-44c6-8860-a1db0ecf9410
Following above conversations, I only found the latest blueprint, but couldn't find mentioned case in any.
One solution maybe by:

adding position: relative to auro-dropdown host, also
adding max-width: -webkit-fill-available // or 100% maybe will work to the auro-dropdown popover wrapper

Please advise, thank you.


GH-44334: [C++] Fix S3 error handling in ObjectOutputStream

Rationale for this change
See #GH-44334. Errors from the AWS SDK are not correctly propagated onto the user of the ObjectOutputStream, not indicating an error even though there was one in some cases.
What changes are included in this PR?

Directly pass the outcome of the AWS SDK to HandleUploadUsingSingleRequestOutcome aswell as HandleUploadPartOutcome instead of wrapping it in a arrow Result class which has been constructed implictily, always indicating success.
Adjust cleanup handling in Close so that the output stream is closed if there was an error in any of the called methods. Otherwise, destructing the output stream in debug builds fails as we abort if Close() returns something else than Status::OK().

Are these changes tested?

Added assertions for catching exceptions on Close() in case delayed_open is enabled.

Are there any user-facing changes?
No.

GitHub Issue: #44334

FYI @pitrou This is related to https://github.com/apache/arrow/pull/41564


Provide ability to sort/search list of templates when copying

Kris Bolt from Harvard: It is tedious to scroll through long lists of templates, existing DMPs, etc. within the tool. A desired enhancement for these lists is the ability to search and/or sort.

Merging with #136


"az snapshot grant-access" fails with "Accessing disk or snapshot with a logical sector size of 4096 requires specifying the file format. Set the fileFormat property to VHDX."

Describe the bug
CLI cmdlet "az snapshot grant-access", using APIs equal or later than 2023-01-02, fails with "Accessing disk or snapshot with a logical sector size of 4096 requires specifying the file format. Set the fileFormat property to VHDX".

[ ~ ]$ az snapshot grant-access --duration-in-seconds 3600 --resource-group martinme_repro --name premV2Snap
(BadRequest) Accessing disk or snapshot with a logical sector size of 4096 requires specifying the file format. Set the fileFormat property to VHDX.
Code: BadRequest
Message: Accessing disk or snapshot with a logical sector size of 4096 requires specifying the file format. Set the fileFormat property to VHDX.

When trying to pass the "fileFormat" as parameter for CMDLets, an error is returned saying the 'fileFormat' isn't valid.

[ ~ ]$ az snapshot grant-access --duration-in-seconds 3600 --resource-group martinme_repro --name premV2Snap --fileFormat VHDX
unrecognized arguments: --fileFormat VHDX

Tested with RESTAPI --> Snapshots - Grant Access - REST API (Azure Compute) | Microsoft Learn
The supported api-versions are '2016-04-30-preview, 2017-03-30, 2018-04-01, 2018-06-01, 2018-09-30, 2019-03-01, 2019-07-01, 2019-11-01, 2020-05-01, 2020-06-30, 2020-09-30, 2020-12-01, 2021-04-01, 2021-08-01, 2021-12-01, 2022-03-02, 2022-07-02, 2023-01-02, 2023-04-02, 2023-10-02'

SUCCESS: I've tried it again with the RestAPI by specifying with the fileFormat set to VHDX in the body and it works with the latest API version 2023-10-02 as well: Snapshots - Grant Access - REST API (Azure Compute) | Microsoft Learn
{access: "Read",durationInSeconds: "3600",fileFormat: "VHDX"}
--

"az snapshot grant-access", using APIs equal or later than 2023-01-02, do not offer any parameter to define the "fileFormat".
RestAPI Snapshots - Grant Access - REST API (Azure Compute) | Microsoft Learn accepts the "fileFormat" as parameter. CLI cmdlet should do the same.
Related command
az snapshot grant-access --duration-in-seconds 3600 --resource-group martinme_repro --name premV2Snap
Errors
[ ~ ]$ az snapshot grant-access --duration-in-seconds 3600 --resource-group martinme_repro --name premV2Snap --fileFormat VHDX
unrecognized arguments: --fileFormat VHDX
Issue script & Debug output

Expected behavior
"az snapshot grant-access", using APIs equal or later than 2023-01-02, do not offer any parameter to define the "fileFormat".
RestAPI Snapshots - Grant Access - REST API (Azure Compute) | Microsoft Learn accepts the "fileFormat" as parameter. CLI cmdlet should do the same.
Environment Summary
Azure Portal CloudShell
Additional context
No response

Thank you for opening this issue, we will look into it.

Hi @riccom76,
currently CLI has no parameter to set the field "fileFormat". If you want CLI to support this feature, please provide the necessary info by raising a feature request using this feature request template.

hi @yanzhudd , this is not a feature request but more a fix to a misalignment between the CLI cmdlet parameters and what the RestAPI version used is expecting as parameters when invoked.
RestAPI Snapshots - Grant Access - REST API (Azure Compute) | Microsoft Learn vesion equal or later than 2023-01-02, expects the "fileFormat" to be there. This CLI cmdlet has to be updated to accommodate what the RestAPI expects as passed parameters.

Hi @riccom76
I have raised a PR to fix this issue, could you please help to verify if this private package meets your expectation?
https://dev.azure.com/azclitools/public/_build/results?buildId=140563&view=artifacts&pathAsName=false&type=publishedArtifacts
Thanks in advance!


irmin-pack.unix: refactor errors

This started as an exploration prompted by @Ngoguey42 to look into:

From the errors.ml and errors_base.ml point of view, the action after close errors gets treated a bit like RO_not_allowed so that Irmin.Closed gets raised and not Pack_error.

https://github.com/mirage/irmin/pull/1932#issuecomment-1164771862
But, as I was looking at the errors code, I saw perhaps a way to also tidy the conceptual layout of the current code. This PR does not aim to fundamentally change how we are handling errors (a larger discussion we are having). It is only a refactoring for (hopefully) better conceptual clarity.

Make a clear distinction between core errors and IO errors with errors.ml (formerly base_errors.ml) and io_errors.ml (formerly errors.ml).
Coalesce errors for operations after close to use Irmin.Closed as their exception.

Thoughts and feedback very much welcome!

@metanivek Can you change the target to main and merge when the CI is green?


Feature/unblur own cards

With this pull request we tried to tackle the issue #52
Feedback is very welcome :)

Thank you very much for your PR! Tomorrow, I will take a look at it.


fix(ci/check-size): use latest non-merge commit

We were previously using the merge commit as the baseline, leading to all commits from the pull request being rebased, rather than just the latest commit.

Codecov Report
Patch and project coverage have no change.

Comparison is base (5bb5902) 79.81% compared to head (d054676) 79.81%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #72   +/-   ##
=======================================
  Coverage   79.81%   79.81%
=======================================
  Files           3        3
  Lines         436      436
=======================================
  Hits          348      348
  Misses         65       65
  Partials       23       23

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


bug: tfsec on modules not working expected with reading template file

Describe the bug

tfsec does not give correct results when reading variable values from an external YAML file via templatefile() function.
To Reproduce
Check Example Code section.
Expected behavior

When using the templatefile() function, it incorrectly reports unset variable values in the called module.
The result should be reported after scanning for when the module has been rendered via the values in the YAML file.
Output of your tfsec command with --debug flag

System Info

> tfsec --version
v1.28.1
> tf --version
Terraform v1.3.3
on darwin_arm64

Example Code

# Module
resource "aws_iam_account_password_policy" "this" {
  minimum_password_length        = var.password_policy.minimum_password_length
  require_numbers                = var.password_policy.require_numbers
  require_symbols                = var.password_policy.require_symbols
  require_lowercase_characters   = var.password_policy.require_lowercase_characters
  require_uppercase_characters   = var.password_policy.require_uppercase_characters
  allow_users_to_change_password = var.password_policy.allow_users_to_change_password
  hard_expiry                    = var.password_policy.hard_expiry
  max_password_age               = var.password_policy.max_password_age
  password_reuse_prevention      = var.password_policy.password_reuse_prevention
}

variable "password_policy" {
  description = "(Optional) Password Policy for the AWS account."
  type = object({
    minimum_password_length        = number
    require_numbers                = bool
    require_symbols                = bool
    require_lowercase_characters   = bool
    require_uppercase_characters   = bool
    allow_users_to_change_password = bool
    hard_expiry                    = bool
    max_password_age               = number
    password_reuse_prevention      = number
  })
  default = {
    minimum_password_length        = 8
    require_numbers                = true
    require_symbols                = true
    require_lowercase_characters   = true
    require_uppercase_characters   = true
    allow_users_to_change_password = true
    hard_expiry                    = false
    max_password_age               = 0
    password_reuse_prevention      = 0
  }
  nullable = false
}

# config.yaml
account:
  password_policy:
    minimum_password_length: 8
    require_numbers: false
    require_symbols: false
    require_lowercase_characters: true
    require_uppercase_characters: false
    allow_users_to_change_password: true
    hard_expiry: false
    max_password_age: 0
    password_reuse_prevention: 5

# root-module.tf
locals {
  password_policy = {
    minimum_password_length = 14
    require_numbers = true
    require_symbols = true
    require_lowercase_characters = false
    require_uppercase_characters = false
    max_password_age = 40
    password_reuse_prevention = 5
  }
}

## INFO: It works!
module "account" {
  source  = "..."

  password_policy = local.password_policy
}

locals {
  context = { a = "hello" }
  config = yamldecode(templatefile("config.yaml", local.context))
}
## INFO: It doesn't works!
module "account" {
  source  = "..."

  password_policy = local.config.account.password_policy
}

Additional context

It works if I use file() function instead of templatefile() function.

Hi @posquit0 !
I created the issue in Trivy.
/cc @simar7


Matmul supporting pre-transposed matrix B without actually performing transpose.

GGML as an CPU/GPU inference library, uses row major layout. For performance, it pre-
transposes the weight matrix of a matrix multiplication op so GPUs can have a more linear
memory access pattern (better performance). This doesn't matter to TTNN as TTNN uses tiling
natively. But as GGML expects matrices to be pre-transposed while TTNN doesn't. Transpose of
the matrix B (in C = A @ B) is needed to convert what GGML provides into what TTNN wants.
Removing this transpose would massively improve the performance.
Currently moreh_matmul supports this functionality. However it is limited to BFP16 matrices.
TT's matmul has parameter to support it. but internally does an actual transpose causing the
same performance downgrade. Kernel level support for pre-tranposed matrix B would improve
the performance.

This is probably going to be implemented via transpose_tile flag and reader strides https://github.com/tenstorrent/tt-metal/issues/15727
@prajaramanTT could we change this issue into a feature request?


ExeName for BeyondCompare on linux should be bcompare not bcomp

Describe the bug
The command for Beyond Compare on Linux is bcompare not bcomp as currently defined: https://github.com/VerifyTests/DiffEngine/blob/6d0f011e604c86407108762fe97f777a87b147de/src/DiffEngine/Implementation/BeyondCompare.cs#L48
This can be verified via the following documentation which shows bcompare as being available on Linux and Mac but bcomp only on Mac: https://www.scootersoftware.com/v4help/command_line_reference.html
Simple validation on Debian with a fresh install of BeyondCompare from https://www.scootersoftware.com/files/bcompare-<IP_ADDRESS>397_amd64.deb
$ which bcomp
bcomp not found
                                                                                  1
$ which bcompare
/usr/bin/bcompare

Minimal Repro

Install BeyondCompare https://www.scootersoftware.com/files/bcompare-<IP_ADDRESS>397_amd64.deb
Run a test along the lines of

internal sealed class DiffToolsTests
{
    [TestCase(DiffTool.BeyondCompare)]
    [Explicit("Requires tools to be installed")]
    public void DiffTools_Resolved(DiffTool tool)
    {
        // Arrange
        // Act
        ResolvedTool[] tools = DiffTools.Resolved.ToArray();

        // Assert
        Assert.That(tools, Is.Not.Empty);
    }
}

Compared to:

    [Test]
    [Explicit("Requires tools to be installed")]
    public void DiffTools_AddTool_BeyondCompare()
    {
        // Arrange
        LaunchArguments launchArguments = new(
            Left: (temp, target) => $"--diff \"{target}\" \"{temp}\"",
            Right: (temp, target) => $"--diff \"{temp}\" \"{target}\"");
        OsSupport osSupport = new(
            Windows: null,
            Linux: new OsSettings(
                "bcompare",
                launchArguments
            ),
            Osx: null);
        DiffTools.AddTool(
            name: "BeyondCompareLinux",
            autoRefresh: true,
            isMdi: true,
            supportsText: true,
            requiresTarget: true,
            osSupport: osSupport,
            binaryExtensions: Array.Empty<string>());

        // Act
        ResolvedTool[] tools = DiffTools.Resolved.ToArray();
        string[] toolNames = tools.Select(x => x.Name).ToArray();

        // Assert
        Assert.That(toolNames, Does.Contain("BeyondCompareLinux"));
    }

Submit a PR that fixes the bug
Submit a Pull Request (PR) that fixes the bug. Include in this PR a test that verifies the fix. If you were not able to fix the bug, a PR that illustrates your partial progress will suffice.

can u submit a PR that fixes this


Include data elements in selector

Right now when you use Material UI or Office UI Fabric, or any other custom framework, you get the selectors based on the actual element you click like: div:nth-child(2) > .banner__item:nth-child(1) .news__item__social__comments:nth-child(2) .ms-Button-icon:nth-child(1).
In order to make it easier for creating tests, it is better to allow the same data attributes Cypress uses. So that the selector becomes: div:nth-child(3) > [data-testid="banner__item"]:nth-child(1) [data-testid="social__likes"]:nth-child(2) .ms-Button-icon:nth-child(1).
This has been implemented into the current PR with updating finder.

Thanks @estruyf !


"Something went wrong. Please try again." when join the waitlist

Hello, do you know how to join the waitlist of new bing?

Whatever and however you do, join it asap. They are limiting it by the day, see my other posts here.

not my problem. complain to MS


Games Played Updater for Players Not Working

Right now it displays 0 games played for everyone. Example: http://www.cfbreference.com/teams/houston/2011/players/case-keenum/7/QB/

this occurs when a team is not properly being updated. solution is to re-run game updater or update_college_year()


Landing Page Design

Design an eye-catching landing page for the project.
Requirements

Bright colorful scheme
Background Animations

I have gone through GreenSock Examples and here are some I really liked

http://www.caribana-festival.ch/en/
https://www.duroc.ma/en/
https://interview.ueno.co/

These can be taken as reference for developing the landing page design.Please feel free to share your design here or on slack channel(don't forget to create a thraed for uploaded image there) to get feedback.
Please let me know if you have any queries.

@smriti548 please free to share your idea here. Have you created any wireframes ??

I don't have any wireframes... I just have the imagination of the idea and that is:
What if as the page opens different quotes(any general quotes) just come to the page from every side and form a particular pattern or the logo of the quote app... we can make the logo accordingly.

I may be wrong but if I have imagined correctly it would look good with small words rather than sentences(quotes here). Please correct me if I am wrong.
I rather prefer some shapes (like circles) or color changes. Have you gone through the reference UI design I have shared?? What do you think about them??

My idea for the landing page is to have a random bright colour (From flatui's colour palette)  background. A random quote would be present under the play game/ start game button. The quote would have a typing effect. When a user would click on start game button the quote's text  would change into asterisks(*) and the game would start.
So this is my idea for the landing page and I have created a wireframe for the same on this link ->  https://wireframe.cc/QKazeR . Hope you like the idea and feel free to tell some improvements that could be done.

@parmeet1402 I think it would be better if we move the discussion to slack channel so that everybody can give in their feedback. Please add your idea there and start a thread by addressing @Twisha. I would like to discuss the background of the page there.

That's cool @Sahil-Dhawan. Would look great but how do you plan on implementing it??? Since we are using GreenSock, this is could be done using that.
I am a beginner in Greenock and this is what I have created so far https://codepen.io/Twisha/full/rvOgGV/.
Please Let me know what you think. Actually, there is a lot of scope for improvement but this is something very basic I created.

@twishasaraiya its nice had a look , I'll find some implementations and exemplar approach for the same till Saturday , I'll keep you posted regarding that , I'm sure this can be made pretty interactive and enticing.. My apologies for the late response

So I have made a design for Desktop and Mobile. Since I don't know UI animation, I have kept the designs static, but they can be easily animated later. I haven't seen the finalized logo or the color scheme, hence the design is completely on my own ideas. Do share your thoughts with me :slightly_smiling_face:
Keeping the whole idea of a game, I have decided to not clutter the landing page, and allow the users to directly play the game.

@blenderskool The design looks good. You can start the coding. We can tweak it as we go long.

@ayushdev Thanks, I am still not sure where and how will we include the logo of the project in my design, hence would wait for some more suggestions :slightly_smiling_face:

@blenderskool The design looks amazing. 👍  We have finalized the logo, you can have a look at it on the slack channel (#cp_quote_app). Also since our color scheme uses shades of purple/violet I think we could change the green color here.
Yes your idea of allowing the users to play the game directly is correct, that's what we will be doing. Also instead of Top Players we could add Login Option or remove it completely since this will be a single player game for now.
Let me what you think.
Once this is done you can start with the implementation. I could help with the UI animation if you are doing it on CodePen or any other online sites or we could add it later after the static site is ready. That shouldn't be a problem.

@blenderskool I would say whichever looks better go with that one.

@twishasaraiya  https://iohk.io/ have a look at this website and move around your mouse a bit this was what I was talking about but couldn't understand how to exactly implement it

The background here is dynamic as I was saying

yes lets implement something like this for later :smile: @twishasaraiya

Yes definetely.

Completed #81 . Hence closing this issue.  Can be opened again for further enhancements if required in future.


Running examples from the command line misses failing examples

Examples use AssertionFailure to mark a failure instead of TestFailure. TestFailure now mark a test as having errors.


Make updating versions easier

Don't make users delete their entire GTFO.DevTools folder as thats weird.
Maybe a way of auto-updating?

Bumped to needs-testing due to commit 80f65c1422769e697efcd4ae2635a65761195b3a, with making it easier to update assets to newer unity versions. Making auto-update for GTFO.DevTools itself will be a separate issue/problem.


[7770] Upgrade to ruby 3.3

Context
Ruby 3.3 has been out n the wild for sometime now... it be good to tame it.
Changes proposed in this pull request
Use ruby 3.3
Guidance to review
Nothing breaks
Important business

[ ] Does this PR introduce any PII fields that need to be overwritten or deleted in db/scripts/sanitise.sql?
[ ] Does this PR change the database schema? If so, have you updated the config/analytics.yml file and considered whether you need to send 'import_entity' events?
[ ] Do we need to send any updates to DQT as part of the work in this PR?
[ ] Does this PR need an ADR?

NB: Please notify the #twd_data_insights team and ask for a review if new fields are being added to analytics.yml

 Upgrade to Ruby 3.3


Add docs

This is modeled after the structure of the sverto docs, which is very clever.

@jimjam-slam Could you take a look at the workflow to auto-publish to pages? That's adapted from Svertos. What sort of config is necessary here in the repo?

You've been busy! Sorry I missed this branch — I have tomorrow off, so I'll hop in then!

No prob! There's another PR that I opened for this branch that has a bit more work.


NoSuchFileExpection looking for jakarta jars

I received the above exception trying to compile samples (both 0.6.1 and 0.7.0). Please see the full log file
output.txt

I've seen this a few times. But I can't for the life of me remember how I worked past it each time.  It is more like a issue with the scripts than native-image. Two invalid jars end up on the classpath (from your output.txt):
Warning: Invalid classpath entry: /home/user/spring-graal-native/spring-graal-native-samples/spring-petclinic-jpa/target/native-image/BOOT-INF/lib/jakarta.xml.bind-api-2.3.2.jar
Warning: Invalid classpath entry: /home/user/spring-graal-native/spring-graal-native-samples/spring-petclinic-jpa/target/native-image/BOOT-INF/lib/jakarta.activation-api-1.2.1.jar

and then later there are issues. I'm at least going to turn down the fatal error to a warning, but I don't  quite recall why the find command in the script is finding things that aren't there to put on the classpath.

Class-Path entries in jar manifests, perhaps? From time-to-time, we see similar oddities when using DevTools with certain dependencies that have "broken" manifests.

I wondered that Andy - but the script was doing just a unix find to create the classpath. Anyway, missing jars isn't a big deal but things crashing because of it is a problem. I've made the changes in master to just move on when a jar isn't around. If you try master it should behave better now - master is designed to work with the most recent Graal 20.1 dev builds - which you can grab here: https://github.com/graalvm/graalvm-ce-dev-builds/releases (below the 20.2 builds). It may still work with 20.0 release, I'm just not 100% sure.


Trying to install luiggio when having symfony 2.5.12

Im getting this error when trying to install luiggio excel bundle

What does this error means? Any workaround?

I resolved it using this composer command
composer require liuggio/excelbundle @stable


feat: add QueryInfoSecurity method

Hi and thanks for your contribution!
I'm working on migrating the code away from using the dynamic encoder (encoder.Marshal/Unmarshal functions) and instead writing custom MarshalBinary and UnmarshalBinary methods for every struct to get more control since the dynamic encoder has been a bit problematic from time to time.
So before merging this pull request I would ask you to update the code to using custom marshal/unmarshal functions for everything.
I think that you can copy must of the code from my msrrp package as I've already defined SecurityDescriptors and ACLs for DCERPC which I think are the same for this method you've implemented.
https://github.com/jfjallid/go-smb/blob/a34537bd8fced73181c8c42b9777b2237b620c10/smb/dcerpc/msrrp/structures.go#L1399
I think those structures are probably better located in the smb.go file rather than in the msrrp package so I should migrate them but for now they could be duplicated.

Hi, thank you for the quick response !
I copied the unmarshaling and structs from your DCERPC package. works fine for my use-case.

Hey! Fixed the status map. Thanks for the comment


Deploy multiple applications natively

It would be nice to be able to use a file to define multiple applications and the associated command to use it.
Something similar to pip requirements files.

Hi,
You can list required/dependencies application the 'deploy' section of the manifest
Here an example of a 'meta-package' that only regroup a list of application in one another package
https://github.com/kubespray/kpm-packages/blob/master/ant31/op-tools/manifest.yaml
---
package:
  name: ant31/op-tools
  author: Antoine Legrand
  version: 1.0.0
  description: Deploy my Kubernetes monitoring tools
  license: MIT

deploy:
  - name: kube-system/heapster
  - name: kube-system/influxdb-grafana
  - name: kube-system/kubernetes-dashboard
  - name: kube-system/kubedash
  - name: kube-system/kube-ui

this is the manifest for the package ant31/op-tools, it deploys  heaspter,grafana,dashboard,kubedash,kube-ui
the command to deploy all those 5 apps is :
kpm deploy ant31/op-tools
Similary you can create any package you want with the list of required applications
Does it answer you issue or did I misunderstood it ?


Fix image size

Last change to emqx Dockerfile separate the build of otp and emqx. However the installed package used to build otp was not removed from the otp layer, thus increasing the final image size no matter whether they're removed or not.
This pull request fix the above issue.

@thynson it was my bad.
Thanks for the PR.
It's fixed in emqx30 branch already though.
The image built from your branch is about 150MB
The image built from branch emqx30 is about 88MB.
I guess there is no easy way to split the build step without bloating image size.


Many distance feature takes long to React

Describe the bug

How to reproduce or log data

Expected behavior

Device/Version information (please complete the following information):

Device: [e.g. EON/EON Gold/comma two]
Dongle ID: [e.g. 77611a1fac303767, can be found in Settings -> Device -> Dongle ID or my.comma.ai/useradmin]
Route: [e.g. 77611a1fac303767|2020-05-11--16-37-07, can be found in my.comma.ai/useradmin]
Timestamp: [When in the route the bug occurs (e.g. 4min 30s into the drive)]
Version: [commit hash when on a non-release branch, or version number when on devel or release2 (e.g. 0.7.6)]
Car make/model: [e.g. Toyota Prius 2016]

Additional context

Please create a new issue with the bug report template filled out and a more detailed explanation of the issue.


Disable Deploy Sample API button for publisher users

Description:

Disable Deploy Sample API button for publisher users

Steps to reproduce:
Affected Product Version:

Environment details (with versions):

OS:
Client:
Env (Docker/K8s):

Optional Fields
Related Issues:

Suggested Labels:

Suggested Assignees:

I think we need to come up with a new design to handle this use case,
The scenario is, Publisher user logs into the Publisher portal when there are no APIs created. The publisher couldn't do anything at this point, So we need to gracefully inform the user how to proceed.
Publisher's can't create API products either when there are no APIs.
So this is kind of a dead-end scenario.
@erangatl Shall we come up with a UI design to cover this scenario.

@erangatl Shall we come up with a UI design to cover this scenario.

+1 will do a design

XD: https://xd.adobe.com/view/be3fc799-0d72-473d-a515-9f7665b5d08a-eb6c/screen/c74cd0bc-1cfa-4baf-81ab-3a17b6cd2180


fix padding alignment to optimize memory usage

Describe the problem you'd like to have solved
We should always verify the memory alignment of the structures.
We need to:

Inspect all structures to verify their memory alignment
Add a new check for memory alignment in our linter

Tools:
Various tools exist to check memory alignment, but aligo looks promising. It appears to be well-maintained.

after more discussion, we decided to close this issue and won't fix it.
the struct alignment is not more important than code style and clean code in Go. While ensuring proper struct alignment can improve performance, it is still important to follow best practices for writing clean and organized code. This includes proper documentation, modularization, and adherence to Go's idiomatic code style.


Set Theory: Equivalence Classes (Spelling Error)

In this section, after the first definition, you wrote "fundemental" instead of "fundamental"

Fixed


Task/survey redirect

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+1.1%) to 61.101%

Files with Coverage Reduction
New Missed Lines
%

catalyst_utils/context_processors.py
1
0%

Totals

Change from base Build<PHONE_NUMBER>:
1.1%

Covered Lines:
688

Relevant Lines:
1126

💛  - Coveralls


color sliders work and sfinv tabs are as well

To address issue #10 , if i understand the report correctly, it was to make the color sliders work with sfinv and not break the tabs?

With this PR, the color sliders aren't draggable. The purpose of form_send_job is to not refresh the form until the user has finished dragging the scroll bar. If the form gets resent to the user while the user is dragging the scroll bar, the user will lose their grip on the scrollbar.
To my knowledge everything is working fine already with the sfinv branch. Did you find something that wasn't working?

I see, my apologies for misunderstanding the previous issue (#10).
Unfortunately the scroll bar values are sent to the server immediately from the client, making dynamic refresh impossible. As you can see, when the server receives the value, the scroll box is set to the value's location on the bar once the formspec is refreshed.
What I had understood from the previous issue ticket, was the tab menu in sfinv were not showing in the skin selection menu.

Hello again,
I believe the latest commit to this pull is more in-line with the vision of this mod. The sliders are not stopped, and freely move until released, then the skin is updated.
However, I just realized that issue #10 is concerning the main branch, and as you say this branch was working fine previously. I totally misunderstood, my mistake.
I'll see if the method applied with this commit can also be used on the main branch, or try to figure a solution.

Closing stale PR


E340 & E341 completeopt+=fuzzy and i_CTRL-X_CTRL-F

Steps to reproduce

set completeopt+=fuzzy
enter insert mode and write some path
type <c-x><c-f>
E340 & E341

This demo shows:

it works correctly with vim --clean
the error occurs once the option is set

https://github.com/user-attachments/assets/8612303a-8312-4a03-ba7f-27581ae7ebd8
Expected behaviour
handle these errors
Version of Vim
VIM - Vi IMproved 9.1 (2024 Jan 02, compiled Jul 18 2024 18:4 0:28) macOS version - arm64 Included patches: 1-600 Compiled by Homebrew Huge version without GUI.
Environment
OS: macOS Sonoma 14.5
Terminal: wezterm 20240203-110809-5046fc22
value of $TERM: xterm-256color
Shell: zsh 5.9 (arm-apple-darwin23.0.0)
Logs and stack traces
E340: Internal error; if you can reproduce please report a bug
E341: Internal error: lalloc(0, )

duplicate https://github.com/vim/vim/issues/15287

closing then as duplicate


unnecessary-literal-within-tuple-call (C409) rule transforms single element list literals incorrectly

The unnecessary-literal-within-tuple-call (C409) rule detects the violation here:
tuple([1])

ruff check --fix --unsafe-fixes fails to transform the code correctly, resulting in this, which is not a tuple:
(1)

Instead it should transform the code to:
(1,)

Environment:
% ruff --version
ruff 0.3.2

% cat pyproject.toml
[tool.ruff]
lint.select = [
    "C4"
]

@zanieb Hi!
Can I take it too?

Go for it @WindowGenerator.

I don't know much about Ruff internals but it feels like it might be safer to take the approach of transforming and then rendering the AST, rather than replacing specific string indexes, since the latter is vulnerable to producing invalid or incorrect code in this fashion.

Yeah, we have a Generator struct which does exactly that. Moving the fix to use that would solve the issue as it takes one element tuple into consideration.

We should just add this case to the fix. Text-based fixes are preferable because they preserve all trivia. (This fix used to use LibCST but we intentionally changed it because it’s slow and we want to remove our LibCST dependency altogether eventually.)
I think there was just a misunderstanding on what the fix is supposed to do based on the comment above it in the code, which led to this oversight.

Fixed by #10491.


One plan, two subscriptions

Hi,
Quick question, would it be possible to specify a backend in one subscription and provision the infrastructure to another one? Any indications about how to proceed?
Many thanks,

Hey @evmimagina, I do not have a recommendation for how you could utilize the task to support this configuration. You could create the backend ahead of time outside the task. However, I believe when init runs, it will assume the backend exists in the same subscription as the service connection.
Could you provide a bit more detail on your use case? For example, do you need this configuration because a different team in your org owns and manages the backend storage than the dev/engineering teams? Any detail you can provide would help ensure that what is added to support this fits your case.

Hi,
Problem solved, the only requirement for the stages beyond the init one, wich should point to the same subscription where the backend config lives, is to include on tf scripts the storage account key for the backend configured storage account;
terraform {
backend "azurerm" {
access_key           = "storageaccountkey"
}
}
Collecting the storage account key could be done during previous steps.
Thanks for looking into this, I close the request.
Regards,

Thanks for providing the update and guidance for others that might have similar issue @evmimagina


Code duplication between buildcfg.py and geos.py

There is a significant amount of code duplication betwen _buildcfg.py and geos.py. Can we move the functions that load the shared library into it's own module, shared by both?
#327, #400, #401

I'm going to close this and mark "wontfix". These problems are going away or will be traded for other problems (that's how software works) in 2.0 and 1.8 is only about warning of upcoming changes.


Very slow search when processing big set of IP objects (> 10000)

This is a core problem. It's necessary to improve the structure of the database.

Developed in bd232096e1e6790db751c133a7f440d88665a2c2


Access logs format

Title: Access logs format is suspicious
Description:
We are seeing logs like this one:
[2020-11-26T14:25:49.160Z] "POST /istio.mcp.v1alpha1.ResourceSource/EstablishResourceStream HTTP/2" 200 - 1671 9758 300008 - "-" "grpc-go/1.24.0" "f88ece7f-e28f-454b-9bd0-2bf22b0a2260" "localhost:15019" "<IP_ADDRESS>:15019"
It looks like the format of the log is:
"[%START_TIME%] \"%REQ(:METHOD)% %REQ(X-ENVOY-ORIGINAL-PATH?:PATH):256% %PROTOCOL%\" %RESPONSE_CODE% %RESPONSE_FLAGS% %BYTES_RECEIVED% %BYTES_SENT% %DURATION% %RESP(X-ENVOY-UPSTREAM-SERVICE-TIME)% \"%REQ(X-FORWARDED-FOR)%\" \"%REQ(USER-AGENT)%\" \"%REQ(X-REQUEST-ID)%\" \"%REQ(:AUTHORITY)%\"\n"
However, if this is correct, then the duration is going to be 300 008ms, which does not sound correct. Do you have any idea what field stands behind the “300 008" value? If it’s time, is it in milliseconds, microseconds, nano, or?

According to https://www.envoyproxy.io/docs/envoy/latest/configuration/observability/access_log/usage.html?highlight=duration %DURATION% gives you "Total duration in milliseconds of the request from the start time to the last byte out."
What makes you think 300008 in milliseconds is wrong?

From what I know, this is internal communication within the K8s cluster, and 300 000ms equal to 5m, which sounds way too much.

Looks like request was closed due to a time out.

In continuation of Simeon's thought, we have researched the envoy's implementation and we figured out that the duration is returned in nanoseconds. Here are the following functions that are responsible for returning duration:

absl::optional<std::chrono::nanoseconds> duration(absl::optional<MonotonicTime> time) const {
  if (!time) {
      return {};
    }
    return std::chrono::duration_cast<std::chrono::nanoseconds>(time.value() -
                                                                start_time_monotonic_);
  }

absl::optional<std::chrono::nanoseconds> requestComplete() const override {
    return duration(final_time_);
  }

And in the test function, they compare it to nanoseconds:
https://github.com/envoyproxy/envoy/blob/105e3f1559c51f5098de1e021abefeee40844dfb/test/common/formatter/substitution_formatter_test.cc#L312
Are we looking at the wrong thing? Is the duration converted from nanoseconds to milliseconds in another piece of code?

It is converted to milliseconds here: https://github.com/envoyproxy/envoy/blob/ae7d84132858d831286a6b079fab8271faa451f5/source/common/formatter/substitution_formatter.cc#L450

How can we get the REQUEST_TX_DURATION or RESPONSE_TX_DURATION or DURATION in microseconds or even the

/reopen


Stop user from being joining a same class again and again.

At present, we have implemented so many functionalities with proper DBMS set-up for our ease, but one thing is to be done is that we don't want a user to join the same class again and again. If user has not already joined that particular class then user can join that class normally, but if user has already joined that class, he should be given a message on the screen that "You've already joined the class".

Can I take this up?

I can help with this! Please assign it to me if you can.

@arunsri7 You can also work on this issue.
You can too create  a Pull Request on this issue.
No worries! Happy Hacking!

Issue fixed by @IgnisDa
PR #19 fixes it


more options for running different cases

Codecov Report
Attention: 79 lines in your changes are missing coverage. Please review.

Comparison is base (287411a) 73.77% compared to head (934e7a2) 72.29%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #63      +/-   ##
==========================================
- Coverage   73.77%   72.29%   -1.48%
==========================================
  Files          21       21
  Lines        2040     2119      +79
==========================================
+ Hits         1505     1532      +27
- Misses        535      587      +52

Flag
Coverage Δ

unittests
72.29% <45.89%> (-1.48%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files
Coverage Δ

ocean_model_skill_assessor/featuretype.py
100.00% <100.00%> (ø)

ocean_model_skill_assessor/plot/line.py
100.00% <ø> (ø)

ocean_model_skill_assessor/plot/surface.py
92.13% <100.00%> (+0.18%)
:arrow_up:

tests/test_datasets.py
99.34% <100.00%> (+0.01%)
:arrow_up:

ocean_model_skill_assessor/plot/__init__.py
85.71% <62.50%> (-9.94%)
:arrow_down:

ocean_model_skill_assessor/utils.py
56.70% <31.42%> (-2.41%)
:arrow_down:

ocean_model_skill_assessor/main.py
64.93% <37.17%> (-2.86%)
:arrow_down:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


No longer able to Apply a FanzyZones layout when switching aspect ratio

ℹ Computer information

PowerToys version: 0.27.1
PowerToy Utility: FancyZones
Running PowerToys as Admin: Yes
Windows build number: 1809

📝 Provide detailed reproduction steps (if any)

Set up a layout with at least one zone spanning from almost top to almost taskbar using a 16:10 monitor.
Switch to a 16:9 monitor.
Layout broken, with windows going beyond the screen.
Press Win + ~ and hit Apply.

✔ Expected result
It used to unbreak the the layout on the new aspect ratio. It was a really convenient 1-click solution for when you switched between different aspect ratio monitors (say, a 16:9 laptop and a 16:10 external or visa versa).
❌ Actual result
Now nothing changes, and it stays broken.
📷 Screenshots
This is what now happens when switching from a 16:10 to a 16:9 monitor. The layout shows empty bars on the sides (not there in the original layout and undesirable):

And this is what happens with any window I then try snapping (again, undesirable):

The desirable behavior would be to have the window not be cut off at the bottom and no empty bars on the sides. The old version behaved pretty much perfectly in that regard, and I'm not sure why this was changed. The only way to make it better would've been eliminating going into layout and hitting Apply every time. But the way it is now is definitely a step back.

@github-account1111
only grid-based layouts can be applied to any resolution, canvas layouts are relative to the specific resolution on which they were created.
We will show this information in a upcoming release. This is tracked in https://github.com/microsoft/PowerToys/issues/8528

@github-account1111
only grid-based layouts can be applied to any resolution, canvas layouts are relative to the specific resolution on which they were created.
We will show this information in a upcoming release. This is tracked in https://github.com/microsoft/PowerToys/issues/8528

But then what do I do? Meticulously drag edges every time I unplug / turn off my monitor or grab my laptop? This seems like one step back, zero steps forward.
I can't be the only person using different resolution laptops / monitors.

But then what do I do? Meticulously drag edges every time I unplug / turn off my monitor or grab my laptop? This seems like one step back, zero steps forward.
I can't be the only person using different resolution laptops / monitors.

Also if the release is "upcoming" (i.e. not out yet) why have you already removed the feature? This creates sort of a limbo.

Also if the release is "upcoming" (i.e. not out yet) why have you already removed the feature? This creates sort of a limbo.

@github-account1111

Meticulously drag edges every time I unplug / turn off my monitor or grab my laptop?

Why do you consider doing that?
You will be able to assign a layout per resolution, and when the resolution changes it will apply its currently assigned layout.

@github-account1111

Meticulously drag edges every time I unplug / turn off my monitor or grab my laptop?

Why do you consider doing that?
You will be able to assign a layout per resolution, and when the resolution changes it will apply its currently assigned layout.

Because I want to use the same layout on all resolutions and I want to just press a button and have it auto-adjust whenever I switch monitors. Managing multiple layouts creates unneeded complexity and clicks.

Because I want to use the same layout on all resolutions and I want to just press a button and have it auto-adjust whenever I switch monitors. Managing multiple layouts creates unneeded complexity and clicks.


Linux build fails in workflow

Somehow the Linux build fails.. https://github.com/mipmip/mip.rs/actions/runs/5311316438/jobs/9614377483

Hey url is 404 but to compile it on my ubuntu i did:
git clone https://github.com/mipmip/mip.rs yarn cargo run sudo apt install webkitgtk sudo apt install libwebkitgtk-6.0-dev sudo apt install libgtk-3-dev sudo apt install libsoup2.4-dev sudo apt install libjavascriptcoregtk-4.0-bin echo "Types: deb\nURIs: http://br.archive.ubuntu.com/ubuntu/\nSuites: jammy noble-updates noble-backports\nComponents: main restricted universe multiverse\nSigned-By: /usr/share/keyrings/ubuntu-archive-keyring.gpg\n\nTypes: deb\nURIs: http://security.ubuntu.com/ubuntu/\nSuites: jammy-security\nComponents: main restricted universe multiverse\nSigned-By: /usr/share/keyrings/ubuntu-archive-keyring.gpg" | sudo tee /etc/apt/sources.list.d/ubuntu.sources3 //sudo -e /etc/apt/sources.list.d/ubuntu.sources sudo apt update sudo apt install libwebkit2gtk-4.0-dev cargo build --release


Fix TileSet painting options appear out of screen

Fixes #83610

Thanks! And congrats for your first merged Godot contribution :tada:

Thank you!


Not working!!

Please update...

Hi, it is working fine for me. Do you get any kind of error?

Hi, it is working fine for me. Do you get any kind of error?

Yeah, I didn't get some anime like chainsaw Man,nisekoi etc which is block on my country.

I am using brave browsers.

Sorry I don't officially support any browsers besides Firefox. Are you on Version 3.0 of the extension? This is required for the new Crunchyroll site.

Sorry I don't officially support any browsers besides Firefox. Are you on Version 3.0 of the extension? This is required for the new Crunchyroll site.

Yeah, I am using latest version.
Recently I tested on Firefox. It works but not working on chromium browsers.

I have problem with crunchyroll IFrame player. It always loading. IFrame player works when I disable cr-unblocker but I need bypass block from crunchyroll. What can I do??

I'm closing this as I kinda forgot these issues. Sorry for the missing reply - I can't really help with issues caused by other extensions as they change the browser behavior in different ways and supporting them would be more work. I'm open for contributions though, if someone wants to improve compatibility.


how to pass parameter?

how a parameter is sent to the report, for example the title of this.
the error that I get is Undefined offset: 0.
` $output = [];
$returnVar = 0;
    chdir($this->pathExecutable);
    exec($this->command, $output, $returnVar);
    if ($returnVar !== 0) {
        //throw new \PHPJasper\Exception\ErrorCommandExecutable();
        throw new \Exception("{$output[0]}", 1);
    }

    return $output;
}`

uso laravel 5.6

usually when I get this error is because one of the file locations ($input / $output) is incorrect.


linuxKernel.kernels.linux_zen: 6.10.7-zen1 -> 6.10.9-zen1; linuxKernel.kernels.linux_lqx: 6.10.6-lqx1 -> 6.10.9-lqx1

Description of changes
https://github.com/zen-kernel/zen-kernel/releases/tag/v6.10.9-zen1
https://github.com/zen-kernel/zen-kernel/releases/tag/v6.10.9-lqx1
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

@ofborg build linuxKernel.kernels.linux_lqx

LGTM so far. This can be merged when Ofborg is happy.


Naming conflict with gitea cli client

When installing tea i got cryptic error messages like Error: flag provided but not defined: -prefix.
Turns out, I already had a tea cli for Gitea https://gitea.com/gitea/tea#the-official-cli-for-gitea .
This would also be the case for any function or alias named tea.
That's fine, but the install script should be able to handle this, at least during installation. Usually that means escaping the function call in the install script \tea, or hardcoding paths to the binary, etc.
A simpler and less error prone but less magic install script would just bail if tea exists. zsh example:
(( $+commands[tea] )) && fail "command tea already found..."

yes, refs https://github.com/teaxyz/cli/issues/179
We have this as a WIP: https://github.com/teaxyz/setup/pull/85
My plan is to offer to create teaxyz as the symlink if gitea is installed since they were here first. But we won't at this point be renaming ourselves in general.

This would also be the case for any function or alias named tea.

ahh, yes we better handle that too. Thank you.

The bug is fixed. We still don't offer to name ourselves teaxyz but we'll get to that.
While working on this I noticed that generally linux distros install gitea as gitea and not tea so really this only effects brew.


How to hide update notifications?

please complete the following information:

OS version: Pop!_OS 22.04 LTS x86_64
Editor version: 1.67.0-insider
Programming language: N/A
TabNine extension version: v3.5.581

Issue Details:
TabNine outputs a lot of notifications and I'd like to disable some of them.
For example, here's a notification that I'd like to disable:

https://github.com/codota/tabnine-vscode/blob/3823b6757320243f46b6010681f66dd4f8abc436/src/preRelease/installer.ts#L41-L47
I was unable to find the setting that disables that.
gz#15414

This is a VSCode setting.
I looked it up online and found this link

You are referring to VSCode's update mechanism. I am referring to TabNine's custom notification after an update.

hey @NatoBoram, you can disable the tabnine "beta" channel  updates (tabnine.receiveBetaChannelUpdates)

Do you mean that the only way to disable this specific notification is to opt out of beta?

Do you mean that the only way to disable this specific notification is to opt out of beta?

*Sigh*

Do you mean that the only way to disable this specific notification is to opt out of beta?

Do you mean that the only way to disable this specific notification is to opt out of beta?

Do you mean that the only way to disable this specific notification is to opt out of beta?


Fix issue #144453, extension's lastUpdated time exceed 24:00:00

Signed-off-by: Quan Zhuo<EMAIL_ADDRESS>
This PR fixes #

This PR fixes #144453


Live User Filter

Project title
Live user filter
Project Description
it filters searches , from a bunch of Random users from alphabets of their name or their place.
Stack:

[x] JavaScript

/assign


Fix incorrect encoding behaviour in some cases in Modular mode.

Fixes #3717.

How did this one go undetected for so long?
Can we add some roundtrip test that would have caught this bug?
(in general, maybe not as part of regular CI, it wouldn't hurt to do some larger-scale roundtrip testing to check that lossless is actually lossless, regardless of the effort / number of threads / image dimensions / etc)

How did this one go undetected for so long?
Can we add some roundtrip test that would have caught this bug?
(in general, maybe not as part of regular CI, it wouldn't hurt to do some larger-scale roundtrip testing to check that lossless is actually lossless, regardless of the effort / number of threads / image dimensions / etc)

The bug would have been caught by any thread-sanitizer run with an image of size least 2049x2049 and effort >=5, I think.
I'd have guessed this to be caused by streaming encoding, but the fact that 0.8.2 seems to show the issue confuses me...