Notifying mailing lists when there's a new release (or a major request for comment)

We released a new version v1.2.2 (#405) recently and in our recent steering group meeting we discussed curating a list of mailing lists to share this news! 🎊 🎉 ✨
Please comment below with suggestions of communities to contact.
General updates
This list will be extended with mailing lists / communities that would like to hear about a broad set of different updates to BIDS.

Freesurfer
FSL
SPM
OHBM (via their newsletter?)

Specific update communities
This list will be extended with mailing lists / communities that would like to hear about specific updates to BIDS, eg MEG only, or genetics communities etc.

For electrophysiology I would like to communicate to the communities of FieldTrip, SPM (also for imaging, see above), MNE-Python, BrainStorm, EEGLAB.

General updates should also go to bids-discussion (i.e. the google group) and to the news section on the website.

@robertoostenveld - can you check the grouping for the ones you've suggested? I wonder if MNE-Python would want to hear about more general updates?

For MEG community also may be worth considering<EMAIL_ADDRESS>

@robertoostenveld - can you check the grouping for the ones you've suggested? I wonder if MNE-Python would want to hear about more general updates?

With "electrophysiology" I meant to refer to that part of the neuroscience community, not that part of the specification.
I.e. I did not mean that those lists would only be interested in updates limited to electrophys-specific aspects, the community on these electrophys-focussed lists should also be informed about  general changes. A lot of researchers on these lists use a combination of neuroimaging methods (e.g. you don't do iEEG without MRI and CT).

for easier editing and additions I have created a google sheet for capturing this information - https://docs.google.com/spreadsheets/d/16SAGK3zG93WM2EWuoZDcRIC7ygPc5b7PDNGpFyC3obA/edit#gid=0

Thanks Franklin! I'll add :)


Fingerprint update request _TZ3000_cfnprab5/TS011F

Hi,
this device is a powerboard with 4 switchable plugs and 1 group of switchable USB ports (1 switch for 4 USB).
Currently when I add it, it is recognised by your Zigbee Mult Switch Mc driver, but only 4 switches in total are added (main+3).
Can you please update the fingerprint to make this a 5 switch plug.
Manufacturer

_TZ3000_cfnprab5
Model:
TS011F

Thanks
Ian

Hi  Ian
Do the 4 switches shown work independently for you?
I was trying to fix it and it was impossible, I couldn't find the reason why they all turn on and off at the same time.
I don't have access to the CLI now, a couple of days I will.
with what you tell me I do it

Hi Mariano,
sorry, I did not explain very clearly. This is the Zigbee powerboard image: https://www.aliexpress.com/i/4000504463608.html
The four existing switches for the four power outlets all work correctly (each works independetly of each other).
I just need you to add one more switch. This will switch the four USB outlets. There board has only one endpoint that controls all four USB outlets.
Thank you in advance.
Ian

I'm glad they work independently.
A few days ago to another user both worked at the same time and I left it as impossible.
https://community.smartthings.com/t/beta-zigbee-edge-driver-compatible-with-lidl-ecosmart-osram-smartthings-others/231689/2317?u=mariano_colmenarejo
I will add the 5th switch as son is posible and you try it

Hi Mariano,
thanks, that works great. All 5 outlets are switchable independently.

One question: Is it possible to edit the name of the Switch-2, Switch-3 etc? I would like to change them to TV, DVD, etc

Sorry, user can not change the labels of the components.
It has been requested and I don't know if they will

Thanks

Can you also add the following:
Manufacturer:

_TYZB01_vkwryfdr
Model:
TS0115

This is physically identical to the device above. It has 4 power outlets and 1 USB.
However, when I built the Groovy DTH, the USB endpoint is 7.
i.e.
EP1 = Power outlet
EP2 = Power outlet
EP3 = Power outlet
EP4 = Power outlet
EP7 = USB outlets
Thanks
Ian

I need to see the clusters list to match the profile

Hi Mariano,
here are the clusters:

Thanks

Hi
Give me some time I am modifying this driver to add other functions

Hi
Added to this driver version
┌─────────────┬──────────────────────────────────────┐
│ Name        │ Zigbee Multi Switch Mc               │
│ Version     │ 2022-09-22T21:13:14.703540351        │
└─────────────┴──────────────────────────────────────┘

and this other driver with childs devices
┌─────────────┬──────────────────────────────────────┐
│ Name        │ Zigbee Multi Switch and Childs Mc    │
│ Version     │ 2022-09-22T21:13:20.327906873        │
└─────────────┴──────────────────────────────────────┘

  - id: "TS0115/_TYZB01_vkwryfdr"
    deviceLabel: TS0115 Switch
    manufacturer: _TYZB01_vkwryfdr
    model: TS0115
    deviceProfileName: five-outlet

Hi Mariano,
thanks for adding TS0115/_TYZB01_vkwryfdr.
The 4 outlets (Main, switch2, switch3, switch4) = EP1, EP2, EP3, EP4 are working fine.
The USB (switch5) is not working. This is EP7 (not EP5 which is more logical).
Can you please check this for me.
Thanks
Ian


New protcore container

I added a new protcore container as an intermediate before building all proteomics packages, as the later always times out.
@lshep - could you add it to the docker hub, or tell me how to do it. Thanks!

Is there a reason these extra packages couldn't have been added into the already existing mscore or protmetcore container?

mscore contains a minimal set of manually curated mass spectrometry packages and protmetcore the packages common to proteomics and metabolomics biocViews. The extra packages are not relevant for metabolomics; I'm not sure it would be useful to bloat the metabolomics containers. @sneumann, what do you think?

I set up and am trying to build the devel version now. If that is successful I will work on the release version

The new containers are up on docker hub :   bioconductor/devel_protcore2  and bioconductor/release_protcore2.


Global Brewfile would be better placed somewhere under ~/.config or ~/.local as per XDG

The global Brewfile is somewhat inconveniently placed as ~/.Brewfile (macos) with no configuration or environment variables allowing to change its location. This is doubly inconvenient given the additional generation of .Brewfile.lock.json in the home folder.
Instead of adding to polluting the home folder with gazillion other dotfiles with no structure it would be better to place the files in a subfolder as per XDG.
The Brewfile should almost certainly be used as ~/.config/homebrew-bundle/Brewfile but I'm uncertain of whether the lockfile should be better treated as configuration or data.
ref: XDG Base Directory Specification

You can toggle where the file is dumped to and read from using --file of you'd like it somewhere else.

macOS does not follow the XDG. homebrew-bundle was originally macOS only. We cannot change the default location without breaking all existing users.

Operating systems don't follow the XDG, applications do. Many applications can look up configuration files from multiple locations in order of preference, using the first one that's found. Changing default location with a fallback to previous shouldn't break anything.
I wouldn't necessarily object to using ~/Library/Application Support like macos apps are meant to but for terminal apps I think XDG is more appropriate.

Changing default location with a fallback to previous shouldn't break anything.

We'll review a PR to fallback to XDG locations if the necessary variables are set.


Delayed jobs leaves an idle postgreSQL process on DEALLOCATE

Laravel Version: 5.4.15
PHP Version: 7.0.15-0ubuntu<IP_ADDRESS> (cli)
Database Driver & Version: postgreSQL 9.5
Queue driver: redis

Description:
Every time a delayed job is dispatched, when it is executed, I have new idle connection (not correctly closed).
I explained this with detail here: http://stackoverflow.com/questions/42838840/laravel-jobs-leaves-an-idle-postgresql-process-on-deallocate
Here is the original post:
Every time a Delayed Job is has run on my server I can see a new idle process in postgreSQL. Running select * from pg_stat_activity; I can see:
DEALLOCATE pdo_stmt_00000018

I tried to understand, and one more line (and one more process in htop) appears each time a delayed queued job had just ran.
The last line of my Job is:
$this->log->info("Invitation {$this->invitation->uuid} sent");

And I can see this in my logs, so everything is alright BUT it does not clean up after. I have a idle process every time with "DEALLOCATE pdo_stmt_00000xxx".
What can I do to avoid this problem? What is causing this?
Here is my supervisor config:
[program:laravel-queue-worker]
process_name=%(program_name)s_%(process_num)02d
command=php /path/to/my/site/artisan queue:work --queue=invitation,default --sleep=3 --tries=3
autostart=true
autorestart=true
user=www-data
numprocs=2
redirect_stderr=true
stdout_logfile=/path/to/my/logs/worker.log

Side note: the idle processes disappear when I run php artisan queue:restart
Steps To Reproduce:
Dispatch a delayed job on a ubuntu server 16.04 using redis queue, supervisor and postgreSQL. Check in postgreSQL, you will see idle process on DEALLOCATE which never(?) disappear.

I found a (quick and dirty) workaround. Adding this at the end of my Job handle function:
\DB::disconnect();
sleep(1);

I was inspired by this: https://github.com/laravel/framework/issues/17785
Not sure it's the same problem though. And I still don't understand why I have to do that.

I'm not a postgresql user myself, and I only did some quick googling. It seems that the query you see is the last executed one, and not something that is stuck. The worker process itself is sleeping, waiting to check the database again in a few seconds for any new jobs.

The query column now means "current or last query" --- there's a separate column to check for the idle status.  Those DEALLOCATEs are evidently what your app does before going idle.

Source: https://www.postgresql.org/message-id/20442.1350597286%40sss.pgh.pa.us

Closing for lack of activity.

+1, i have this issue too

+1

+1

@Katerou22 Have you read my previous answer and the link to the documentation? If so, what's the actual problem you're having?

+1

@Katerou22 Have you read my previous answer and the link to the documentation? If so, what's the actual problem you're having?

I have same issue with Postgres, after few days of running queues with horizon those idle processes rise up to number of maximum connections database can have and this is a problem as standard app connections will be then refused. For now we need to restart queues every day to make it running smoothly.

@themsaid Not sure why this was closed? Can confirm this is still a bug that is happening and impacting numerous people.

there is a solution? i have the same issue, my DB  collapses with a few users

I had this problem using sqs as queue driver
A hacky solution could be closing the connection explicit at the end of job
with:
DB::disconnect('foo');

We use AWS t2.micro, with some limitation, the connection can only handle 50 at once.
Basically, we make our own migration for each company on its own schema (you get the idea), we iterate through companies and do migrate, but from each iteration, the idle connection counter keep there and not (supposedly) releasing (?), and keep going up until it hit max connection
We use a workaround to close the connection after each iteration, we also use Migrator to execute companies migration folder, is this how we supposedly handle this?
->each (function () {

$generator = new Generator($companyDatabase);
$engine = $generator->engine();
$connection = $generator->newConnection();
$migration = new Executor($connection);

$engine->assign($engine->config('username'), $engine->config('database'));

$migration->setOutput($command->output);

$migration->migrate();

$connection->disconnect(); //we add this

});

Same issue there. Quick & dirty workaround :
SELECT pg_terminate_backend(pid) FROM pg_stat_activity WHERE datname = 'dbname' AND pid <> pg_backend_pid()  AND state in ('idle');

Out of curiosity, does everyone experiencing this issue call DB::connection at some point in their jobs/application logic?

@atrauzzi I've got this issue, did a quick project search and i'm not seeing any calls to DB::connection so the answer would be no

check if you did beginTransaction without commit or without rollback, properly way is that
\DB::beginTransation();
try{
//some matter stuff
\DB::commit();
} catch(Exception $e) {
\DB::rollback();
}

I've screenshotted the only bit of code in our code base which is using transactions.
Transactions should always be committed unless i'm missing something blatantly obvious.

The problem is how do you use transactions, so you should move last 3 lines to
try {
// current code
// 3 last lines with
DB::commit()
} catch (){
DB::rollback();
}

DB::commit always place to the end of try block, but DB::rollback to the beginning of catch block.
Your case - If last 3 lines fail (for example, DB connection broken or Model's validation error) you will get Uncommited transaction.

I was under the impression i had to commit the rollback aswel. I update the code and see if this solves our issue! Thanks in advance!

I have tried moving the commit & apiUpdateStatus code within the try block. This has not solved our problem, i will have a look at the code you added to the AppServiceProvider today.

Any suggestions for this issue?

We ended up closing idle connections manually through the use of a cron job.
$result = DB::select(DB::raw("SELECT count(*) FROM pg_stat_activity WHERE state = 'idle' AND datname = :database"), ["database" => config("DB_DATABASE")]);
if($result[0]->count > 5){
            DB::select(DB::raw("SELECT pg_terminate_backend(pid) FROM pg_stat_activity WHERE datname = :database AND pid <> pg_backend_pid()  AND state in ('idle')"), ["database" => config("DB_DATABASE")]);
}

+1 here. We are on Vapor and SQS. We are using CockroachDB (with Postgres driver) and we see a lot of DEALLOCATE queries.

+1 here. We are on Vapor and SQS. We are using CockroachDB (with Postgres driver) and we see a lot of DEALLOCATE queries.

It seems that the connection might relate to the amount of workers, is that possible?
So it least, even if a problem, it's accountable: use the ::disconnect workaround, increase the pgsql connection count or reduce workers.
It also might be an option to use something like https://github.com/pgbouncer/pgbouncer
I can confirm I see a lot of those idle connection with DEALLOCATE from the prepared statements, but I never had a problem (basically I wasn't even aware until I came across this issue), but that might simply because I have enough connections ATM and using pgbouncer.

It seems that the connection might relate to the amount of workers, is that possible?
So it least, even if a problem, it's accountable: use the ::disconnect workaround, increase the pgsql connection count or reduce workers.
It also might be an option to use something like https://github.com/pgbouncer/pgbouncer
I can confirm I see a lot of those idle connection with DEALLOCATE from the prepared statements, but I never had a problem (basically I wasn't even aware until I came across this issue), but that might simply because I have enough connections ATM and using pgbouncer.

Leaving a note here that I've have been experiencing similar issues as above.
I'm trying out the below solution where it is possible to delete all idle connections at a set interval.
alter system set idle_in_transaction_session_timeout='5min';
https://stackoverflow.com/questions/12391174/how-to-close-idle-connections-in-postgresql-automatically/52704477#52704477

I've recently commented on this thread where I was suggesting pgbouncer.
In my case I was:

running pgbouncer since ages due to other reasons with PHP and many connections to the DB (laravel unrelated)
recently deploying a much more active job/queue change to one of my application, encompassing > 60 queues and > 300 workers

Here is what I observed using:

Postgres 9.6 (I know, old)
pgbouncer 1.12 (also outdated)

using in pool_mode=session
This is important, as it means connections will only be reclaimed by pgbouncer if the clients is/gets disconnected. Other modes require explicit transaction which isn't always feasible in my case

number of workers: 300 (make it easy number)

Without pgbouncer:

300 connections to pgsql server, constantly, also constantly requiring memory on pgsql (each connection has an overhead)

With pgbouncer:

300 connections to pgbouncer, which held 300 connections to pgsql, had the same effect as above

Because Laravel workers are not concerned by themselves about disconnecting database connections, as long as a worker is running, it consumed/blocks a connection, either to pgbouncer or to pgsql serer.
In case the worker is idle, the connection is still open. pgbouncer in such case (pool_mode=session) does not know it can be re-used (and: shouldn't) will also hold the same amount of laravel worker connections to the pgsql server too.
I then used the following worker events and: JobExceptionOccurred, JobFailed and JobProcessed and implemented a simple disconnect logic in the listener, basically like this:
        $connectionNames = array_keys($this->config->get('database.connections'));

        foreach ($connectionNames as $name) {
            DB::disconnect($name);
        }

With pgbouncer and manually disconnecting:

~75 connections to pgbouncer

After deploying, the impact was instantly measurable in my case; likely ridiculously measurable; the graph shows the connections pgbouncer manages (sorry can't expose more details on the Graph):

As can be seen, the necessary connections drop from ~300 to < ~75 which is very significant (for us, at least).
Any gotchas?
Yes!
At least one, I found. I was having code like this to directly work with the PDO object of a model for performance reasons:
$pdo = (new SomeModel)->getConnection()->getReadPdo();

This broke when a worker, which previously disconnected, executed this and getReadPdo was returning null.
Because this bypasses the inner workings of the Connection, it misses logic like calling reconnectIfMissing https://github.com/laravel/framework/blob/91dfbd0c3823069eb8e8d65752a6c85e81e5b4d8/src/Illuminate/Database/Connection.php#L662-L668
For those few manual cases I had to adapt the code and check if its null and ->reconnect() on the connection manually and then retrieve the PDO again.
HTH & YMMV


Jenkins Wrapper Failing with "anscii" error on 2016

Starting polling for our job to start
 . Waiting for job to start because : In the quiet period. Expires in 4.8 sec
 . Waiting for job to start because : In the quiet period. Expires in 2.6 sec
 . Waiting for job to start because : In the quiet period. Expires in 0.42 sec
 . Waiting for job to start because : Finished waiting
 . Waiting for job to start because : Traceback (most recent call last):
  File "/hack/jenkins_bootstrap.py", line 177, in <module>
    myrigger.main()
  File "/hack/jenkins_bootstrap.py", line 170, in main
    job_number = self.waiting_for_job_to_start(queue_url)
  File "/hack/jenkins_bootstrap.py", line 97, in waiting_for_job_to_start
    print " . Waiting for job to start because :", queue_request.json().get("why")
UnicodeEncodeError: 'ascii' codec can't encode character u'\u2018' in position 39: ordinal not in range(128)

/area sig/test-infra
/area jenkins

/sig/test-infra


CLP-000: Remove release workflow

As we use tusd as a library, we remove the release workflow.

If we use it as library we can get rid of the fork all together, don't we?

If we use it as library we can get rid of the fork all together, don't we?

unfortunately not yet as we have made some adjustments which didn't land finally in the upstream repo. the maintainer doesn't pass the request context to the hooks but tbh haven't checked the code state for quite some while. momentchen ...

If we use it as library we can get rid of the fork all together, don't we?

unfortunately not yet as we have made some adjustments which didn't land finally in the upstream repo. the maintainer doesn't pass the request context to the hooks but tbh haven't checked the code state for quite some while. momentchen ...

no, still that weird behaviour. the maintainer argues that the network connection could drop and all bytes uploaded from the last chunk call could get lost: https://github.com/tus/tusd/pull/740

If we use it as library we can get rid of the fork all together, don't we?

unfortunately not yet as we have made some adjustments which didn't land finally in the upstream repo. the maintainer doesn't pass the request context to the hooks but tbh haven't checked the code state for quite some while. momentchen ...

no, still that weird behaviour. the maintainer argues that the network connection could drop and all bytes uploaded from the last chunk call could get lost: tus#740

but we can get rid of this repo when we have async video processing in place. the context topic could have been worked around with some ugly hack but more important - and which is actually blocking us from removing our little fork - is this one line change pr, which directly writes to the response object. but the maintainer argues that the callback should not block too long. so he dictates how tusd should be used as a library in that regard. but it is free lunch so we don't complain and accept it.

If we use it as library we can get rid of the fork all together, don't we?

unfortunately not yet as we have made some adjustments which didn't land finally in the upstream repo. the maintainer doesn't pass the request context to the hooks but tbh haven't checked the code state for quite some while. momentchen ...

no, still that weird behaviour. the maintainer argues that the network connection could drop and all bytes uploaded from the last chunk call could get lost: tus#740

but then we need the release action. Otherwise we cannot release with the change we made.

but then we need the release action. Otherwise we cannot release with the change we made.

the release action creates a docker image. for golang you just need that github repo ...

but then we need the release action. Otherwise we cannot release with the change we made.

the release action creates a docker image. for golang you just need that github repo ...

Ah ok, got it! :+1:


Vide.js breaking the ui-router links in Angular applications

If you try to use Vide.js in an Angular application with ui-router, you will see that putting a link inside a video container won't actually render as a working link.
More specifically, in the following example the link within the video container is not clickable:

     ...
     Inactive link
     ...

 Regular link

In this example vide-block has been initialized in a controller via

$("#vide-block").vide({
 ....
});

and the video plays nicely in the background.
Additionally, here is the error message displayed in the console on page load by ui-router when you try to put an ui-sref link inside a video container:
TypeError: Cannot read property 'toUpperCase' of undefined

Is it a bug of Vide.js? How could this be solved?

Demo, pls. I don't see any possible bugs here.
Vide does not call toUpperCase anywhere.


Research/Prototype LMS Generation Tool

Requirement: generate mock student activity data (including outcomes, participation, grades, etc.) to populate testing and production instances of Canvas LMS

(from Cole:) Will explore existing solutions to see if anyone has addressed this before. If not, I'll initiate the prototype phase, starting with Canvas and potentially extending to other LMS platforms in the future.


ClientEncryption : Add decryption functionality for change feed iterator and decouple encryption code

Description

Refactor code to decouple encryption logic from client SDK core code
Add decryption functionality for change feed iterator
Add capability to handle decryption failure

Type of change

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

Closing issues
closes #1406

@j82w how about a contract generation through the GenAPI tool once for high level walkthrough?


Explicit code license

It's unclear what the license is for these code snippets. My very limited understanding of copyright law suggests that source code without a license implies a copyright by the author and does not allow sharing / modifications / distribution / etc.
I would love to use these patterns, but am hesitant to do so until I understand the IP implications. It would  be great if the example source files had a license header, if there was a LICENSE file, README.md was updated, or similar.

Look good in my opinion.

This issue can be closed I believe?

Yep. Thanks!


List copy / paste missing ul / ol tag

Steps to reproduce:

In an empty document, create a new numbered list.
Create two list items with text.
Delete trailing space after the list.
Select all and copy.
Delete entire document.
Paste. Note that bullets appear instead of numbers. Inspect the source and note that only li tags were pasted and not the ol tag that wraps them.

I found the same problem in v1.6.0.

Thanks @neilj !


Unable to auth via cloud

Receiving the error below (with a correct email address)

Logger: aiohttp.server
Source: custom_components/dyson_cloud/config_flow.py:71
Integration: dyson_cloud (documentation, issues)
First occurred: 10:00:31 PM (3 occurrences)
Last logged: 10:00:54 PM

Error handling request
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/aiohttp/web_protocol.py", line 422, in _handle_request
    resp = await self._request_handler(request)
  File "/usr/local/lib/python3.9/site-packages/aiohttp/web_app.py", line 499, in _handle
    resp = await handler(request)
  File "/usr/local/lib/python3.9/site-packages/aiohttp/web_middlewares.py", line 119, in impl
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/security_filter.py", line 60, in security_filter_middleware
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/forwarded.py", line 197, in forwarded_middleware
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/request_context.py", line 24, in request_context_middleware
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/ban.py", line 78, in ban_middleware
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/auth.py", line 144, in auth_middleware
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/view.py", line 135, in handle
    result = await result
  File "/usr/src/homeassistant/homeassistant/components/config/config_entries.py", line 155, in post
    return await super().post(request, flow_id)
  File "/usr/src/homeassistant/homeassistant/components/http/data_validator.py", line 63, in wrapper
    result = await method(view, request, *args, **kwargs)
  File "/usr/src/homeassistant/homeassistant/helpers/data_entry_flow.py", line 109, in post
    result = await self._flow_mgr.async_configure(flow_id, data)
  File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 202, in async_configure
    result = await self._async_handle_step(flow, cur_step["step_id"], user_input)
  File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 260, in _async_handle_step
    result: FlowResult = await getattr(flow, method)(user_input)
  File "/config/custom_components/dyson_cloud/config_flow.py", line 71, in async_step_email
    self._verify = await self.hass.async_add_executor_job(
  File "/usr/local/lib/python3.9/concurrent/futures/thread.py", line 52, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/local/lib/python3.9/site-packages/libdyson/cloud/account.py", line 127, in login_email_otp
    response = self.request(
  File "/usr/local/lib/python3.9/site-packages/libdyson/cloud/account.py", line 119, in request
    raise DysonInvalidAuth
libdyson.exceptions.DysonInvalidAuth

Yes this has something to do with changes in Dyson API. I'm a little busy recently. It'll be good if someone can somehow find out how the new login process works.

gotta hate it when dyson changes the login process again, this is like the 2nd time they did this within this year.

I just installed this addon and am having the same authentication issue with the Cloud

Getting the same error trying to set up Dyson Cloud too 😞
They keep changing that login api a lot.

I think I found a fix, the Dyson login changed, and requires a "2FAish" setup where they send you a code and then you use your password and code to sign. Log out of the mobile app, close the app, then log back in using the code provided. I did that and my integration was able to login automatically  without me manually logging on . I found this answer in a different dyson repository. I guess their 2FA is a one time thing

For anyone still dealing with this Dyson login issue, I've got a fix implemented over in a PR over here: https://github.com/libdyson-wg/libdyson-neon/pull/6. If anyone wants to help test it, you can clone https://github.com/dotvezz/libdyson-neon/tree/task/fix-account-login and run the get_devices.py script!
Short version I did some reverse-engineering of the Android mobile app. I discovered a mysterious GET request it was making to a provisioning API. Turns out, just by making that request (and even throwing away its response - no session tokens or cookies to hold onto), it unlocks something on the server side, and allows the rest of the ogin requests from your IP address. I've included more thoughts about this in the code comments and PR description for anyone who is curious.
If it proves robust enough in testing, I'll get it merged and tag a new release of https://github.com/libdyson-wg/ha-dyson-cloud to use the updated new login flow. To use it, you'll need to migrate to the new repositories in your HACS setup, there's a simple how-to in the readme here: https://github.com/libdyson-wg/ha-dyson#migration-from-shenxnha-dyson. Users have reported painless migrations so far!


Upgrade tool and ownership are wonky in Singleplayer

When i used the matter upgrader on an EMC link in SP world, and tried to interact with it afterwards the game told me 'You do not own this,  does.'

It seems like loading an SP world jumbles up the ownership data of things. When I place a flower, I can upgrade it just fine, but after loading the world, attempting to upgrade a flower that was there tells me 'You do not own this'. It still shows my name when right clicked though.

For EMC links, the bug appears to happen immediately, and not on world reload.
Steps to reproduce:

put down a collector
set an export item
upgrade the collector
right click the collector to retrieve items
And you'll get the ownership error message.

Why it's giving the user hell is a mystery to me, as the data looks the same, but I do know what I'm missing
there's a desync somewhere in here I feel
It seems the client knows the correct data (even after a world reload), but the server sees it differently
dunno, something's funky here

Data matches yet the first errors
I'll have a release to fix that soon

Fixed in latest releases
1.15: 1.0.33
1.16: 1.0.34
1.18: 1.0.14
1.19.2: 1.0.2


Remove the "is a work in progress" notice in the README

The README currently contains:

This library is a work in progress.  Please give feedback!

I think that this is an incorrect statement. While any non-dead project will always be a work in progress, such a statement suggests the library is immature, which Click (entering 5.x) is most certainly not.
Just my 2 cents... :smiley:
PS: Any plans on moving over to a README.md?

I feel like Click is not yet ready to go out of beta, regardless of what the version numbers imply.
I'll accept a PR that removes this though.

Yay! :tada:


Property actionFilter is not allowed.

What happened:
Property actionFilter is not allowed.

How to reproduce it:
json example:
{
	"id": "simpleadd",
	"name": "simpleadd",
	"version": "0.1.0",
	"specVersion": "0.8",
	"functions": [{
		"name": "Increment Count Function",
		"operation": ".count += 1 | .count",
		"type": "expression"
	}],
	"start": "Initialize Count",
	"states": [{
			"name": "Initialize Count",
			"type": "inject",
			"data": {
				"count": 0
			},
			"transition": "Increment Count"
		},
		{
			"name": "Increment Count",
			"type": "operation",
			"actions": [{
				"functionRef": "Increment Count Function",
				"actionFilter": {
					"toStateData": "${ .count }"
				}
			}],
			"end": true
		}
	]
}

Anything else we need to know?:
When I save the json

@whuanle That's because it's actionDataFilter instead of actionFilter

Aouch, seems there's indeed a typo in the spec. Can u please report it?


No sound when using Text to Speech in Steam Deck

From forum's post: https://forums.unrealengine.com/t/free-azspeech-plugin-text-to-speech-speech-to-text-and-more-with-microsoft-azure-cognitive-services/495394/107?u=lucoiso

BeehiveBob
Hello!
Thanks for this amazing plugin, epic should hire you! I recently added it to my project and it works fine on windows. But not on steam deck it seems, no sound. I’m using the TTS function for voice. Any ideas what could be the cause? (the game: FlipScapes 1 :))
Thanks!

Changed the Sound Wave generation, but I don't have a way to test in Steam Deck because I don't have one.
Ref.: https://github.com/lucoiso/UEAzSpeech/commit/f4e3199c0bc0fab7504afc56c83441d601b1fa69

Closing until get more feedbacks about the changes :)


FR: allow an alternative way to add a user ssh key using GitHub username

I suggest adding a second way to add a user ssh key to the profile manager, which would be faster, and developer friendly.

a direct UI integration is that we could have a toggle button next to the ssh field to allow switching between entering the actual ssh key and GitHub username.
Another way which less explicit is that, without adding extra UI elements, we could validate the user entry and in case it wasn't an ssh key, we could use it as a GitHub username and try to grab the key.

For gripping the ssh keys for GitHub users, it's known that Github exposes the public ssh keys:

e.g. https://github.com/[USER-NAME].keys
or using the API
e.g. https://api.github.com/users/[USRE-NAME]/keys

I don't think we should support a specific platform e.g specific vendor
1- there're other platforms e.g gitlab, gitea (while the popularity of github is out of question), but the other users will be having to copy/paste
2- lots of the users don't even have an sshkey too


[MISC] - Pypi web page installation update.

I come to you to let you know that the pypi web page needs to be updated involving the installation process. (To use the proper name)
Currently says do this to install: pip install -U discord-py-slash-command
You no longer use that name as far as I can see and it conflicts with the name of the package.
Shouldn't it be pip install -U discord-py-interactions instead?
Either way, the name of what we are being told to install is different from the name of the package/library.

Ignore it, discord-py-interactions is the correct one to be installing. We will be releasing version 4.0.0 today, so this won't be seen by most.


More writing on the testing section

Some more edits, cut the eBPF C testing section until I have figured out how to cleanly implement it.

This can be reviewed, I'll likely have time to work on it again on Wednesday/Thursday


Fix failed UUID dataclass_factory test

@Dronablo, hello, thank you for your PR. I updated the version of dataclass-factory, in the new version you do not need to use uuid_schema, it works by default. https://github.com/Tishka17/dataclass_factory/blob/master/dataclass_factory/schema_helpers.py#L77


Feature to choose SMTP SERVER instead of AWS

Hey there,
Your work looks pretty awesome but why is it required to use it only with AWS ?!
Is it possible for you to add a setting to use with another smtp server ?
Thank you so much in advance :)

I think this might be considered. But for now, the whole tool is modeled around AWS services like SNS, SQS, and SES.
If you ask why ? It's all because SES is dirt cheap. Like, 10,000 mails for $1 is quite cheap. That's why it's all modeled around SES

Yes you right it's cheap but the rules of AWS are really stricts with high volumes.
That's why it's good to manage his own smtp server.
Thank you for your answer, hope it's possible in the futur to have this possibility because the client looks very good :)

@AlienFX Yes, I have seen some people trying to use it for high volumes. But AWS is only strict about spams and stuff. You can always justify your reason and ask for a limit increase (emails/day) and send rate (emails/second) increase if all your mails are sent to people who are wanting to receive mails - that is subscribed users or known customers in case of transactional mails or something. In any other case, you cannot use AWS SES, like, especially in Email Marketing - it's quite tough and it's basically a spammer's game, because most people hate getting Ads and marketing mails and they report it as spam and the email clients like Gmail mark those emails as spam and in large scale, it will also black list the mail server. That's why AWS restricts spam because in any other case their server will get blacklisted

I totally agree with you, but sadly people who registered to your list/service are the same who are reporting you at spam too (ofc not all of them but on high volume it's mean a lot). That's why it's most easier to manage his own smtp server and IPs. Thanks for your time

Mail for Good provides a feature to put an Unsubscribe link in the mails so that users can unsubscribe with just a click

Yes but people are lazy and sometime prefer to click on the spam button of their client instead of the unsubscribe link... It's a real issue.
I already saw the README and looks so impressive. I really want to use it :))

Okay..
And oh, nice. Actually, you can test it using SES with AWS Free Tier

@karuppiah7890 I sent you an email :)

Yes, I just read it

Hi @AlienFX,
It would be fully possible to use other SMTP services, but each addition would require a hefty amount of client and server side code to accommodate them. A modular SMTP lib could be used, with the configuration of each SMTP relay plugged in prior to a campaign send.
We're using AWS' API as HTTP is quite a bit faster that SMTP. An SMTP approach would therefore first need to be configured for this use case.
Other elements of the app are dependant on AWS services however, and these would need to be decoupled. The complexity of integrating multiple relays becomes apparent when trying to keep your own application's state (in terms of bounce rate etc) in sync with data from the relay.
It's not an impossibility though.

open source email server compatible with Amazon SES APIs https://github.com/manjeshpv/emailq works with SMTP


Autocomplete ignores leading pipe symbol (|) for namespaced symbols

Describe the bug
When a symbol from a different namespace starts with a pipe symbol, the autocomplete ignores it when you type it, causing duplication when you select the autocomplete item.
To Reproduce
Steps to reproduce the behavior:

Create a namespace called foo, with a symbol called |bar in it.
In another namespace, require foo and type foo/, then trigger autocomplete
Type |ba and select |bar option on the autocomplete list
It'll result in foo/||bar

Expected behavior
Consider the pipe symbol for the search when typed, removing it when it's replaced with the auto-completed value, avoiding duplication.
Desktop (please complete the following information):

OS: macOS 13.6.6
Intellij version: IntelliJ IDEA 2023.3.5 (Community Edition)
Plugin version: 2.3.8

Fixed on 420dd7b


Add property to allow passing a helm chart readme file

When the helm chart is deployed to harbor it has a summary page displays the contents of the helm chart readme.
More info in https://helm.sh/docs/topics/charts/#chart-license-readme-and-notes

Can you provide more information, please?

I think this is already covered as part of the changes in https://github.com/quarkiverse/quarkus-helm/pull/72.
If not, feel free to reopen this issue.


Run metadetect on HSC

Run metadetect on HSC
The title says it all
Contacts: @arunkannawadi @chihway @empEvil
Day/Time:
Main communication channel:
GitHub repo:
Zoom room (if applicable):
Goals and deliverable
Get a cookie at the SLAC sprint day. Stretch goal: Run metadetect on HSC.
Resources and skills needed
NERSC/USDF access.
Detailed description
[add additional details  about the project]

We are unlikely to get anything done here during the sprint week since 1) the environment to run this is planned for, but not available at USDF (HSC is not available via the new middleware at NERSC AFAIK) 2) there are other pressing sprints to work on. But if you are interested in doing anything this week, let me know.

When do you think USDF will have the environment ready? If this is soonish it might be worth it to go through and figure out what configuration choices should be picked for the next steps, but otherwise we can also wait till later.

I head TQ brought some HSC data over to NERSC, though I don't know whether it's the data that would be needed for this

I don't know the timeline and it's beyond my payscale. The environment will be ready whenever DM-40695 gets done. And yes, what TQ has on NERSC will be processed data and won't be in cell-based coadd format. I don't think it adds value to run metadetect on those images, and doing validation because I am sure it'll look bad.


Don't display text questions that haven't been answered

Checking one thing before this is ready.

That's it! I'll redo this change to hide text entries with no data in the template, but still make them editable.


Devise doesn't play well with Apartment after password failure

Environment

Ruby 2.4.2
Rails 5.1.5
Devise 4.4.3
Apartment 2.1.0

Current behavior
I am pretty sure that everybody know about apartment gem. However, for completeness sake, apartment gem (https://github.com/influitive/apartment) provides multitenancy for Rails and ActiveRecord.
I am using both devise and apartment gem. They play nicely with each other. However, I just stumbled on one use case, when they don't.

I open a sign in page

HTTP GET
SessionsController#new is executed.
Apartment works correctly and puts into a correct tenant.

I enter a password and click Sign in

HTTP POST
SessionsController#create
Apartment works correctly and puts into the correct tenant

The password is incorrect

SessionsController#create is "Completed 401 Unauthorized"
SessionsController#new is called (One immediately strange thing that it's called without a new request . As I understand devise intercepts something and calls two controllers for one request in this case
Apartment isn't executed for this SessionsController#new and code isn't placed into a correct tenant. (as result, any tenant related functionality throws).

I found couple of things:

Devise on a failure calls another controller method.
This is implement in file /lib/devise/failure_app.rb inFailureApp.respond
it looks like there are couple of methods to force it to redirect (vs recall). The way how I did this is subcall SessionController and override

  def auth_options
    super.without(:recall)
  end

That being said recall method doesn't work with apartment gem.


Properties whitelist not working

Probably because https://github.com/spring-cloud-stream-app-starters/header-enricher/blob/master/spring-cloud-starter-stream-processor-header-enricher/src/main/java/org/springframework/cloud/stream/app/header/enricher/processor/HeaderEnricherProcessorProperties.java#L32 needs to be header-enricher
If you use --headers , you get an NPE.  Currently must be --header.enricher.headers.

Looks like #2 has fixed that already.
Plus NPE has been fixed with the  https://github.com/spring-cloud-stream-app-starters/header-enricher/commit/fa45b72038d758216147d43dd61a5e41ea575294 As well.


[INLONG-8447][Manager] Optimize paging logic

Prepare a Pull Request
(Change the title refer to the following example)

Fixes #8447

Motivation
Explain here the context, and why you're making that change. What is the problem you're trying to solve?
Modifications
Describe the modifications you've done.
Verifying this change
(Please pick either of the following options)

[ ] This change is a trivial rework/code cleanup without any test coverage.

[ ] This change is already covered by existing tests, such as:
(please describe tests)

[ ] This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a follow-up issue for adding the documentation

@vernedeng Good job. Please add more detail in the PR's description box, thanks.


Allow setting PagedAttention KV cache allocation from context size

@oldgithubman @mcm007 this implements allocating KV cache allocation for a specific number of tokens, when using CUDA.
@oldgithubman:

The problem with dynamically-allocated cache (assuming no limiter functionality), is usually, if I'm targeting a certain performance level (for example, RAM), I don't want the cache to grow beyond RAM and even using all my RAM is bad. So, I think a limiter is still useful. Again, it could just be a matter of adapting to a new workflow though. Being able to prevent cache from using swap is very important though, since this adds wear and tear on expensive nvme's

I will add this in a future PR!


widget_info loosing focus

When using the widget_info extension in Omniverse Create,

I create a mesh cone and try to use the slider. When I select a value with the slider the widget just looses focus and often deselects the selected item, selecting what I guess to be in the selection square that spans up in the background maybe?
After skipping through the sources my guess is a problem somewhere within the classes _ViewportLegacyDisableSelection, _DragPrioritize and _DragGesture in file widget_info_maipulator and disabling user selection.
How can that be fixed?

@VictorYudin ?


Setup to run E2E tests

Currently just calls an endpoint and checks a successful response.
Will update to check records when I have the database schema.

Did you want to get the setup merged in first? And then another PR to update the tests?

@Renny4Real  Yes, I can't write the tests without the database schema so I thought I would get this merged in first. I'm also going to add some of the setup back into the base api


SLF4J, Logback and MySQL library upgrades.

Should also include Rhino R3, but not tested. Less 1.1.5 is not working (Rhino detection code seems fucked).

Already in the 'Update dependencies' pull request.


Add all plugins in one Player

Hi, :smile:
I want to ask how can I add all plugins like

Loop
Auto Translation
Backlight
YouTube

in one MediaElementPlayer video :question:
Can you show me an example code.
Thanks for your help.
Yours,
Suriyaa Kudo :octocat:

Nobody answered. :-1:


feat: Recents Widget

This adds the initial recents widget.
Outstanding issues:

[x] Display unread indicator
[x] UI Styles
[ ] Extended events
[ ] Docs
[ ] Fetching avatars for spaces and rendering them
[ ] Add support for displaying shares
[ ] Handle Space Name change from server
[ ] Handle Space Avatar change
[ ] Incoming Call scenarios

Round one review complete 👍

Ok, ready for 2nd round of reviews @adamweeks.
If things look good and tests pass then this should be ready for our initial release.
The following items still need to be addressed, but that can happen over the weekend in another PR.

[ ] Snapshot tests for redux
[ ] Fetching avatars for spaces and rendering them
[ ] Handle Space Name change from server
[ ] Handle Space Avatar change
[ ] Incoming Call scenarios
[ ] Handle active user being added to a room
[ ] Handle events appearing for spaces that haven't been loaded


Proper handling of web content filtering

Some of our customer are using our website from an entreprise network.
Those networks often have a Web Content Filtering policy, and sometimes the payline website is blocked.
Using the kit, the call to the payline website will be filtered, probably return a HTTP 200 page with HTML content (allowing the end user to see that the target page has been blocked), but usually not contain an 'errorCode' part.
The result is that the HTML content from the Web Content Filtering policy is sent to MangoPay API, producing a HTTP 500 response from your side.
If think some improvement could be the use of a regular expression, or at least a more strict error handling to ensure the process only continues if the payline response contains a valid registration data.

Hello @outofrange-consulting
We're deprecating this SDK as of today (24-July). You should upgrade to the newest version of our Vault SDK, which provides the same functionality of card-registration.
If you still experience the same issues with our new SDK, please reach out to our team via the Hub.
Thanks!


1.20.2?

It does not install on 1.20.2, but all I think you have to do is set the depends to >=1.20.1 or 1.20.2. I tried it, by editing the fabric.mod.json to >=1.20.2 and it worked for me.

I assume this will work for 1.20.4 too?


[SUPPORT] Running --continuous mode with HoodieMultiTableDeltaStreamer seems to only ingest first table

Describe the problem you faced
New to Hudi so very well could be some configuration issue on my side. I'm trying to set up a continuous multi-table ingestion job. I can successfully ingest multiple tables when running without --continuous flag so properties seem set up correctly. When I add the --continuous flag, the job seems to be ingesting data for the first table in continuous mode and never proceeds to ingest data from other tables.
To Reproduce
Steps to reproduce the behavior:

Set up multitable ingestion with --continuous flag
Run hudi job
Observe first table processing new incoming data, but other tables not progressing

Expected behavior
Multitable ingestion with --continuous flag on should process data continually cycling through all tables
Environment Description
Using AWS EMR Release 6.6

Hudi version : 0.10.1-amzn-0

Spark version : 3.2.0

Hive version : 3.1.2

Hadoop version : 3.2.1

Storage (HDFS/S3/GCS..) : S3

Running on Docker? (yes/no) : No

Additional context
spark-submit --class org.apache.hudi.utilities.deltastreamer.HoodieMultiTableDeltaStreamer --packages org.apache.hudi:hudi-utilities-bundle_2.12:0.10.1,org.apache.spark:spark-avro_2.12:2.4.5 --master yarn --deploy-mode cluster --conf spark.serializer=org.apache.spark.serializer.KryoSerializer --conf spark.sql.hive.convertMetastoreParquet=false /usr/lib/hudi/hudi-utilities-bundle_2.12-0.10.1-amzn-0.jar --table-type COPY_ON_WRITE --source-class org.apache.hudi.utilities.sources.ParquetDFSSource --payload-class org.apache.hudi.payload.AWSDmsAvroPayload --source-ordering-field ts --base-path-prefix s3://{hudi_root} --target-table dummy_table --transformer-class org.apache.hudi.utilities.transform.AWSDmsTransformer --props s3://{properties_file} --config-folder s3://{config_directory} --continuous --min-sync-interval-seconds 60 --source-limit<PHONE_NUMBER>

Stacktrace

@ddai-shippo This looks like a legit bug where the continuous running of the first table blocks the rest in the HoodieMultiTableDeltaStreamer.  I'll work on a fix.

@yihua @ddai-shippo  the only way the multitable deltastreamer job moves to the second table is if there is any error on the first table. Or else, the multitable deltastreamer keeps running only on the first table.

@yihua  any ETA for the fix?

@brskiran1 I'll try to put up a simple fix this week so that at least the HoodieMultiTableDeltaStreamer can iterate through all tables in the continuous mode.

@yihua : if you have a tracking jira, can you post it here.

https://issues.apache.org/jira/browse/HUDI-1881
Lets track the updates from jira.
thanks for reporting.

I added a workaround for this issue in my local fork of Hudi. Small tweaks to HoodieAsyncService.java and HoodieMultiTableDeltaStreamer.java were required and now it's working as expected. The executor shutdown timeout being 24 hours was causing the job to hang, so I changed it to 10 seconds with no negative consequences. I also enabled the config --post-write-termination-strategy-class org.apache.hudi.utilities.deltastreamer.NoNewDataTerminationStrategy in MultiTableDeltaStreamer, so it can move on to the next table in MultiTableDeltaStreamer after N number of retries with no new data (max.rounds.without.new.data.to.shutdown) instead of being stuck on the first one until there is an error.
It may not fully solve all use case, such as if you aren't expecting the No New Data condition in all the tables in the multi-job. Maybe a new PostWriteTerminationStrategy would be required for some use cases, or a refactor of how the loop functions.
It does not go back to the beginning of the tables and continuously loop over the set of MultiTables, because after the last one has NoNewData the Spark job will end. So I am handling that within the job orchestration.

After more testing, I believe one more code change is required to stop the async services in other threads when NoNewDataTerminationStrategy is reached. Otherwise the background threads remained alive and left the spark job running. If you change it to forcefully terminate, it threw sleep interrupted errors when the Delta Sync thread was shut down.


GEOMESA-882 RemoveSchema not deleting empty z3 tables

fixed issue where removeschema left empty z3 tables on accumulo
added a unit test specific to this case

Signed-off-by: Andrew Annex<EMAIL_ADDRESS>

Merged as d749747837a7d6c8332b074a472f4a6eb8c6b00c on master.


Bug: Duplicate Menu Items

Are you experiencing an issue with...
shields.io
🐞 Description
While scrolling through the https://shields.io/ I found that some menu items are coming twice. Their URLs are like:

https://shields.io/badges/git-hub-issues
https://shields.io/badges/git-hub-issues-1

Possible Problem
I think someone has made the duplicate or the page is rendering twice.
🔗 Link to the badge
No response
💡 Possible Solution
No response

Hi.
I am going to close this as a duplicate of https://github.com/badges/shields/issues/9393
If you have a look at all these, you'll notice that these are not actually duplicate pages. They are actually slightly different variants of the same badge. They just have the same title in the menu, which I know is unhelpful.
We're aware of the issue. Basically, in order to launch the new frontend, we auto-converted all the old content as a first step, rather than manually reviewing it. I am gradually working through the process of reviewing every single documentation page. One of the things I am doing as part of that process is making sure every page on the site has a globally unique title. You can see the (long) list of PRs in https://github.com/badges/shields/issues/9285#ref-pullrequest-1774983388
I've made a lot of progress, but also there is a lot more to do. I'll reach these eventually.


unit_of_measurement not using right temperature C* vs. F*

Home Assistant release with the issue:

0.101.3
Last working Home Assistant release (if known):
Operating environment (Hass.io/Docker/Windows/etc.):

ubuntu, docker
Integration:

Description of problem:
entity values being stored as F* when the unit_of_measurement is set to C*.
raw MQTT values are in C:
octoPrint/temperature/tool0 {"_timestamp":<PHONE_NUMBER>, "actual": 200.0, "target": 200.0}
octoPrint/temperature/bed {"_timestamp":<PHONE_NUMBER>, "actual": 45.56, "target": 45.0}
from states page:
sensor.bed_temp | 114.01 | unit_of_measurement: °F friendly_name: Bed Temp
sensor.bed_temp_target | 113.0 | unit_of_measurement: °F friendly_name: Bed Temp Target
sensor.nozzle_temp | 392.0 | unit_of_measurement: °F friendly_name: Nozzle Temp
sensor.nozzle_temp_target | 392.0 | unit_of_measurement: °F friendly_name: Nozzle Temp Target
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
  - platform: mqtt
    state_topic: "octoPrint/temperature/bed"
    unit_of_measurement: '°C'
    name: "Bed Temp"
    value_template: "{{ value_json.actual }}"
  - platform: mqtt
    state_topic: "octoPrint/temperature/tool0"
    unit_of_measurement: '°C'
    name: "Nozzle Temp"
    value_template: "{{ value_json.actual }}"
  - platform: mqtt
    state_topic: "octoPrint/temperature/bed"
    unit_of_measurement: '°C'
    name: "Bed Temp Target"
    value_template: "{{ value_json.target }}"
  - platform: mqtt
    state_topic: "octoPrint/temperature/tool0"
    unit_of_measurement: '°C'
    name: "Nozzle Temp Target"
    value_template: "{{ value_json.target }}"

Traceback (if applicable):

Additional information:

@stoneobscurity I cannot reproduce this issue. Have you updated your system to the newest version and tested whether or not this issue persists?

(i purposely keep one version behind latest)
i have about 10 other F temps from sensor: platform: mqtt, these are the only C. perhaps it's getting stuck on F.
or is it using homeassisant: unit_system: imperial ?

i have about 10 other F temps from sensor: platform: mqtt, these are the only C. perhaps it's getting stuck on F.

Could be. Are the F sensors the correct unit of temperature and the correct value?

or is it using homeassisant: unit_system: imperial ?

Glanced over the code and it does not look like that to me.

all the F are reporting correctly.

okay, if you leave out the sensor: platform: mqtt unit_of_measurement: '°C'
the state saves correctly as a number (200.xx).
then you have to go into customize, and set the unit_of_measurement: °C there.
kinda retarded, if you ask me.

@stoneobscurity I have done some more digging and now I am pretty sure this is intended behavior.
Home Assistant lets users pick one unit system either metric or imperial. In addition, Home Assistant automatically converts all values that do not have the correct unit of measurement for the chosen unit system.
It is a really rare case that someone wants to view some sensors in Fahrenheit and others in Celsius since they originate from two completely different unit systems. Therefore, I do not believe that this edge case will be implemented.
Hope you understand and are okay with your workaround for this rare issue.

to me, what i set in the sensor: platform: mqtt should be what it goes off of, not unit_system: imperial. but i'm okay since there's a workaround. albeit a non-obvious, non-intuitive one.

I understand your point of view just as good as the implemented one. Yeah the workaround is definitely not very pretty but it does work. Since your issue is fixed, please be so kind to close this issue. Thank you very much :)

so i restarted my docker and it reverted back to F. and the C is still in customize.yaml.


Updated CONTRIBUTERS.MD

What type of PR is this? (check all applicable)

[yes ] 🚀 Added Name
[x] ✨ Feature
[x] 🌟 stared the repo
[ ] 🐛 Grammatical Error
[ ] 📝 Documentation Update
[x] 🚩 Other

Description : updated contributors.md
Add Link of GitHub Profile : https://github.com/malyabansalper

please kindly label it as accepted for hacktoberfest2021 .I would be really grateful


Language Service API should allow to remove newline before interface declaration opening curly bracket

I'm using typescript-formatter, which says that any issues regarding TS formatting should be opened here.
It should be possible to convert
export interface AdminForDetails
{
  UserID: number;
  AdminFor: number;
  Hash: string;
}

to:
export interface AdminForDetails {
  UserID: number;
  AdminFor: number;
  Hash: string;
}

Do you have placeOpenBraceOnNewLineForFunctions option set to true? if so set it to false.

I've set it to false, but nothing changes

I'm sorry, It seems like the formatter was never actually run. Everything works now the way it's expected


🐛 BUG: wrangler pages dev doesn't provide a user-friendly error message on node imports

Which Cloudflare product(s) does this pertain to?
Wrangler
What version of Wrangler are you using?
2.19.0
What operating system are you using?
Mac
Describe the Bug
Run wrangler pages dev on a pages project which imports from a standard nodejs module without specifying the nodejs_compat flag generates a non user friendly error:

Compared to what we get for example with wrangler pages functions build or wrangler pages publish:

This error message should be aligned and made user friendly like the other ones.

Note: I've tested which with wrangler pages dev I am not sure wrangler dev might have the same issue

(original discussion)

hi @dario-piotrowicz ,
I have tested this in latest wrangler v 3.52.0 and this is what I'm seeing:

Are the above warning/error messages what you'd expect? Or what is the expected messaging you were hoping to see/find useful?

@CarmenPopoviciu yeah that looks much better, you get that with wrangler pages dev right?

@dario-piotrowicz yes, that is correct. If you are happy with that error message, are we good to close this out?

Yes, let's close the issue then 🙂
Thanks a bunch 🙏


Update direct-federation-adfs.md

Learn Build status updates of commit 8352417:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/active-directory/external-identities/direct-federation-adfs.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@msmimart
Thanks for reviewing the proposed changes! IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Moved this change to the azure-docs-pr repository: https://github.com/MicrosoftDocs/azure-docs-pr/pull/217994.
#please-close


fix(looker): removes Tag owner

This is removing the datahub owner for the Measure, Dimension and Temporal tags emitted by Looker and LookML source connectors.
Looker connector is too much opinionated on the ownership of these tags and not aligned with how other connectors work, which just emit the tag with no owner.
Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[x] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

There's a small lint failure, but we'll be able to merge once that's fixed
+ flake8 --count --statistics src/ tests/ examples/
Skipped 1 files
src/datahub/ingestion/source/looker/looker_common.py:78:1: F401 'datahub.metadata.schema_classes.OwnerClass' imported but unused
src/datahub/ingestion/source/looker/looker_common.py:78:1: F401 'datahub.metadata.schema_classes.OwnershipClass' imported but unused
src/datahub/ingestion/source/looker/looker_common.py:78:1: F401 'datahub.metadata.schema_classes.OwnershipTypeClass' imported but unused
3     F401 'datahub.metadata.schema_classes.OwnerClass' imported but unused
3

> Task :metadata-ingestion:lint FAILED


Not playing nicely with startify

First of all, this may be a problem in my config, but so far it doesnt look like it ...
The problem is that when I start nvim and the startify buffer is presented. If I popup a telescope picker and pick a file (eg: oldfiles) then the TelescopeNormal highlight and window options (nonumber, etc) are applied to main window where the file is opened.
@tjdevries since you use Startify as well, could you please try to reproduce the issue and see if its something on my end? Thanks!

I tracked this down with startify buffer being not listed here
This can be reproduced by setlocal nobuflisted in any buffer and then running a Telescope picker that opens a new buffer
@mhinz  In the meantime, do you think it is possible to unset this option on startify buffer since its gonna be wiped out anyway?


Change date handling code so that frontend doesn't assume order

Done and merged - closing this ticket.
Ref:  https://github.com/VirusTrack/COVIDvu/blob/954c187f0afd51d4ee260c08cc22d3fb8cd1fc28/work/covidvu/pipeline/jsonpack.py#L41
This was done a day or two ago.


typo: specfied -> specified

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

api/Access.Form.AfterLayout(even).md
:white_check_mark:Succeeded
View

api/Access.ImportExportSpecification.Execute.md
:white_check_mark:Succeeded
View

api/Excel.PivotTable.ChangePivotCache.md
:white_check_mark:Succeeded
View

api/Excel.Range.CalculateRowMajorOrder.md
:white_check_mark:Succeeded
View

api/Excel.Workbook.SendFaxOverInternet.md
:white_check_mark:Succeeded
View

api/Outlook.Conversation.SetAlwaysDelete.md
:white_check_mark:Succeeded
View

api/Word.Document.SendFaxOverInternet.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[WIP] Commitment Scheme Contract to implement acs6

:white_check_mark: Build AElf <IP_ADDRESS>71 completed (commit https://github.com/AElfProject/AElf/commit/29961ddc04 by @EanCuznaivy)


More names with problematic epithets

These are all accepted species names from COL2021-07-29.  I don't think any of them are database artifacts.
Here's another one for the "nudum" whitelist:
Microsiphum nudum
Names with short epithets that are interpreted as part of the author data:
Selenops ab Logunov & Jäger, 2015
Ruteloryctes bis Dechambre, 2006
Antaplaga dela Druce, 1904
Baeolidia dela (Er. Marcus & Ev. Marcus, 1960)
Dicentria dela Druce, 1894
Epeloria dela Chamberlin, 1946
Eulaira dela Chamberlin & Ivie, 1933
Paralvinella dela Detinova, 1988
Scoparia dela Clarke, 1965
Tortolena dela Chamberlin & Ivie, 1941
Semiothisa da Dyar, 1916
Gnathopleustes den (J.L. Barnard, 1969)
Agnetina den Cao, T.K.T. & Bae, 2006
Desmoxytes des Srisonchai, Enghoff & Panha, 2016
Meteorus dos Zitani, 1998
Stenoecia dos Freyer, 1838
Sympycnus du Curran, 1929
Bolitoglossa la Campbell, Smith, Streicher, Acevedo & Brodie, 2010
Leptonetela la Wang & Li, 2017
Nocaracris van Ünal, 2016
Zodarion van Bosmans, 2009
Malamatidia zu Jäger & Dankittipakul, 2010
Names with ser/subser epithets that are interpreted as rank information or end up in an unparsed tail:
Helophorus (Lihelophorus) ser Zaitzev, 1908
Serina ser Gredler, 1898
Serina subser Gredler, 1898
Names with strange epithets that end up in an unparsed tail:
Seleuca chûjôi Voss, 1957
Peperomia non-alata Trel.
Hyacinthoides non-scripta (L.) Chouard ex Rothm.
Monocelis non-scripta Curini-Galletti, 2014
Macromitrium st.-johnii E. B. Bartram, 1940
Names with seemingly innocuous epithets that nevertheless end up in an unparsed tail:
Profusulinella оblопgа Potievskaya, 1964
Tetrataxis toгosus Postoyalko, 1975
Anomalina badkhyseпsis Kuryleva, 1973
Bigenerina iпfrapaleogenica Suleymanov, 1963
Carpelimus (Trogophloeus) rougemoпti Gildenkov, 2014

Names with seemingly innocuous epithets that nevertheless end up in an unparsed tail:

Profusulinella оblопgа Potievskaya, 1964
Tetrataxis toгosus Postoyalko, 1975
Anomalina badkhyseпsis Kuryleva, 1973
Bigenerina iпfrapaleogenica Suleymanov, 1963
Carpelimus (Trogophloeus) rougemoпti Gildenkov, 2014

All of these contain characters that not suppose to be in scientific names: п, г, so they are not parsed correctly

Selenops ab Logunov & Jäger, 2015
Ruteloryctes bis Dechambre, 2006
Antaplaga dela Druce, 1904
Baeolidia dela (Er. Marcus & Ev. Marcus, 1960)
Dicentria dela Druce, 1894
Epeloria dela Chamberlin, 1946
Eulaira dela Chamberlin & Ivie, 1933
Paralvinella dela Detinova, 1988
Scoparia dela Clarke, 1965
Tortolena dela Chamberlin & Ivie, 1941
Semiothisa da Dyar, 1916
Gnathopleustes den (J.L. Barnard, 1969)
Agnetina den Cao, T.K.T. & Bae, 2006
Desmoxytes des Srisonchai, Enghoff & Panha, 2016
Meteorus dos Zitani, 1998
Stenoecia dos Freyer, 1838
Sympycnus du Curran, 1929
Bolitoglossa la Campbell, Smith, Streicher, Acevedo & Brodie, 2010
Leptonetela la Wang & Li, 2017
Nocaracris van Ünal, 2016
Zodarion van Bosmans, 2009
Malamatidia zu Jäger & Dankittipakul, 2010

these are tricky, I have to think what to do

ambiguous names should be parsed now without conswquences for other names (hopefully). The solution is a bit hackish, but should work

Is chûjôi a valid epithet? I don't think accents are accepted in latin or latinized names at all.

Seleuca chûjôi Voss, 1957

All of these contain cyrillic characters that not supposed to be in scientific names: п, г, so they are not parsed correctly
I bet whoever entered them was in the middle of replying to their instant message :D

Those cyrillic characters do so much resemble n and r Latin characters. Very difficult to detect them when looking at an excel file. Would it be possible for gnparser to automatically detect and replace them for their Latin equivalents, and rise a quality warning?


Artifact Registry migration

Migration from Container Registry to Artifact Registry due to the product deprecation.

Thank you!


changed all the whitespace to match standards according to flake8

This commit removes all the unnecessary whitespace as mentioned in issue #47
Please do add comments/ feedback. Thanks

Hi @JoJo10Smith -- thanks for this pull-request.  The files your suggesting changing are actually ignored by flake8, as seen in setup.cfg.  For that reason, I'm going to close this PR without merging it.  However, thank you for your interest in this project.


[CI] Cancel stale runs not working as expected

we have to rewrite the CI code because most configurations have changed in the latest version.


fix(api): api location cache fix

Overview
The location cache is used regardless of which pipette is moving. If left moves immediately after right, the left move's cached location will be used. This can cause jerky motion.
closes #7156
Changelog

update protocol api version to 2.10
maintain the last mount moved.

Review requests
Test on a robot using example protocols in #7156
Risk assessment
High.

Unable to reproduce pipettes issue on my machine. I don't really think missing pipettes makes sense for this branch; this code doesn't get anywhere near GET /pipettes


Chart.js Line Chart Background

Expected behavior

Current behavior
Wrong background color in buy & sell position.
I wanna create something like Kucoin table and show symbol volume but I have a problem with the background when I update data.
Both solutions didn't work.
animation: false

animation: {
	duration: 1000,
	easing: 'easeInElastic'
}

}
	label: "buy",
	data: [],
	borderColor: "rgb(36, 174, 100)",
	backgroundColor: "rgba(36, 174, 100, 0.2)",
	fill: true,
	borderWidth: 1,
	pointRadius: 0,
	pointHoverRadius: 2,
	pointHoverBorderWidth: 2,
	parsing: {
		xAxisKey: 'volume',
		yAxisKey: 'price'
	}
}

chart.js version
v3.9.1
Browser name and version
Chrome, 114.0.5735.199

As stated in the issue template:

Chart.js versions lower then 4.x are NOT supported anymore, new issues will be disregarded.

Also a reproducible sample is required. So please add a v4 reproducible sample

"chartjs-chart-financial" doesn't work with v4.3.0

For now we can't do anything about it. If you can reproduce it without the financial package we can take a look at it. Otherwise you will need to ask if the maintainers of the financial package can update it to be compatible with chart.js v4

@etimberg @kurkle @tannerlinsley @benmccann @simonbrunel @nnnick @panzarino @nagix

Is anyone in here? -_-

The bug was Fixed, I just download the main source from Github and put into the project files

But I have the same issue with<EMAIL_ADDRESS>
I'm getting data from Websocket so I can't give you a sample!


Worldborder module (rebased)

Rebased world border module, to work in 1.9, tested in only 1 xml (without region files) and worked as expected

I remember having issues with this the last time I checked it out (a few months ago).
If it works as docs describes then it shouldn't be an issue.

@iPGz the problem is that docs aren't always too specific, but it should work as the docs say


Header config causes CLI to throw "invalid rule: cannot contain anything after *"

Describe the bug
The generated _headers file contains a /*/_next/static/chunks/* rule, which throws Error: invalid rule (A path rule cannot contain anything after * token) at line: 1 when netlify dev is run.
https://github.com/netlify/next-on-netlify/blob/e1f037865c170941f70c8db731ef2802d2e07c15/lib/steps/setupHeaders.js#L22-L27
This rule is disallowed by the CLI logic:
https://github.com/netlify/cli/blob/425e2c6eeb6a3b589370a78407b18c53f4b850bc/src/utils/headers.js#L65-L73
Does anyone have any suggestions as to how to remedy this?
To Reproduce
Steps to reproduce:

Set up project
Run next build to build the Next project
Run next-on-netlify to build the Netlify output
Run netlify dev to start dev server
See error

Versions

Next: v10.0.5
next-on-netlify: 1.22.5

There is an issue matching this one in the CLI repo. Please let me know if it would be best to add to that thread and I can close this.
https://github.com/netlify/cli/issues/1148

There is an issue matching this one in the CLI repo. Please let me know if it would be best to add to that thread and I can close this.
https://github.com/netlify/cli/issues/1148

@ctjlewis thanks for finding this and opening! at first i thought the leading * was necessary because our original syntax wasn't working but that syntax ended up being separate from the *. opening a PR now! <3

@ctjlewis thanks for finding this and opening! at first i thought the leading * was necessary because our original syntax wasn't working but that syntax ended up being separate from the *. opening a PR now! <3

@lindsaylevine No worries at all!
Am I right in understanding that this is actually a valid path, and that the CLI should be patched? I started working on a PR in that repo.

@lindsaylevine No worries at all!
Am I right in understanding that this is actually a valid path, and that the CLI should be patched? I started working on a PR in that repo.

@ctjlewis 🙏
no worries for the CLI questions! happy to help if i can, especially because this is directly related to the issue you opened! that said, you're asking a great question, lol. as evidenced by the several changes i've made now to NoN's _headers logic, it's not exactly my area of expertise. the strange things for me are: 1. the rule/path is respected in and works in production just fine 2. people on the CLI issue you linked claim these rules work just fine if configured in the .toml file. i'd pause that work/PR until you/we can get confirmation on https://github.com/netlify/cli/issues/1148, though i love that you started one already!!!!! i'll be sure to ping people on slack tomorrow about it

@ctjlewis 🙏
no worries for the CLI questions! happy to help if i can, especially because this is directly related to the issue you opened! that said, you're asking a great question, lol. as evidenced by the several changes i've made now to NoN's _headers logic, it's not exactly my area of expertise. the strange things for me are: 1. the rule/path is respected in and works in production just fine 2. people on the CLI issue you linked claim these rules work just fine if configured in the .toml file. i'd pause that work/PR until you/we can get confirmation on https://github.com/netlify/cli/issues/1148, though i love that you started one already!!!!! i'll be sure to ping people on slack tomorrow about it

It's already pretty much done, so what I will do is send it in as a draft and you guys can edit it if you need, toss it if not. Like you said, it depends on that clarification - if that rule really is allowed, the PR can get merged, and if not, someone might to update documentation / deploy logic.
Anyway, no rush at all, TYSM for your help! Mind if I CC you on that PR draft when I send it?

It's already pretty much done, so what I will do is send it in as a draft and you guys can edit it if you need, toss it if not. Like you said, it depends on that clarification - if that rule really is allowed, the PR can get merged, and if not, someone might to update documentation / deploy logic.
Anyway, no rush at all, TYSM for your help! Mind if I CC you on that PR draft when I send it?

@ctjlewis ah okay! sounds good. i just sent this message in slack:
hello! i have a next-on-netlify user who wants to open a PR that addresses https://github.com/netlify/cli/issues/1148. however, he’s/we’re not sure what in this issue is actually considered a valid path and whether it’s safe for him to “remove” this check https://github.com/netlify/cli/blob/425e2c6eeb6a3b589370a78407b18c53f4b850bc/src/utils/headers.js#L65-L73. i’m not entirely sure why these rules seemingly work in production and when configured in .toml files, as the issue commenters suggest. lmk!
feel free to tag me in the PR draft and i'll update my slack message with the PR link!

@ctjlewis ah okay! sounds good. i just sent this message in slack:
hello! i have a next-on-netlify user who wants to open a PR that addresses https://github.com/netlify/cli/issues/1148. however, he’s/we’re not sure what in this issue is actually considered a valid path and whether it’s safe for him to “remove” this check https://github.com/netlify/cli/blob/425e2c6eeb6a3b589370a78407b18c53f4b850bc/src/utils/headers.js#L65-L73. i’m not entirely sure why these rules seemingly work in production and when configured in .toml files, as the issue commenters suggest. lmk!
feel free to tag me in the PR draft and i'll update my slack message with the PR link!

@ctjlewis morning! don't see a draft PR from you but here's what my team has said:
Looks like the implementation was based on the docs:
https://github.com/netlify/cli/pull/754#issuecomment-599325288 which don’t have that information, but if it works in production it’s most likely the CLI’s fault.
I would suggest the user to try with the experimental -t  flag (e.g. ntl dev -t) .
That would delegate headers parsing to the traffic mesh local agent.

my teammate that works most on the CLI said you "can open a draft PR fixing the specific use case that doesn't work in the CLI (but works in production) and they'll comment on the PR"!

@ctjlewis morning! don't see a draft PR from you but here's what my team has said:
Looks like the implementation was based on the docs:
https://github.com/netlify/cli/pull/754#issuecomment-599325288 which don’t have that information, but if it works in production it’s most likely the CLI’s fault.
I would suggest the user to try with the experimental -t  flag (e.g. ntl dev -t) .
That would delegate headers parsing to the traffic mesh local agent.

my teammate that works most on the CLI said you "can open a draft PR fixing the specific use case that doesn't work in the CLI (but works in production) and they'll comment on the PR"!


Bootloop after flash Xperia 1 II (PDX203)

As the title reads. Bootloop continues before the "xperia" logo appears. Reflashing a custom rom to get out of the bootloop is possible but then the device has no imei number and therefore unable to read sim card for calls/network data.

HI, this is all unrelated to newflsaher! Further questions please ask on xda forum in your device model subforum thank you


[Snyk] Security upgrade golang from 1.11 to 1.19.7

This PR was automatically created by Snyk using the credentials of a real user.Keeping your Docker base image up-to-date means you’ll benefit from security fixes in the latest version of your chosen image.
Changes included in this PR

go1.11/Dockerfile

We recommend upgrading to golang:1.19.7, as this image has only 120 known vulnerabilities. To do this, merge this pull request, then verify your application still works as expected.
Some of the most important vulnerabilities in your base image include:

Severity
Priority Score / 1000
Issue
Exploit Maturity

714
Exposure of Resource to Wrong Sphere SNYK-DEBIAN10-CURL-3065760
No Known Exploit

714
Directory Traversal SNYK-DEBIAN10-DPKG-2847944
No Known Exploit

714
Out-of-bounds Read SNYK-DEBIAN10-LIBTASN16-3061094
No Known Exploit

714
Buffer Overflow SNYK-DEBIAN10-OPENSSL-1569403
No Known Exploit

714
Credentials Management SNYK-DEBIAN10-PYTHON27-349027
No Known Exploit

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Directory Traversal

Thanks for sharing this information regarding the App and the PR. It's really important to keep the Docker base image up-to-date to ensure the security of the App. I hope the upgrade to golang:1.19.7 will help in resolving the known vulnerabilities.


Heredoc flagged with Layout/HeredocIndentation creates infinite loop in rubocop

Prism version: v0.29.0
Rubocop version: 1.64.0
With this file:
# frozen_string_literal: true

{
  "content" => <<~XML.strip,
    <section>
      Hello
    </section>
  XML
}

This error is flagged:

And attempting to fix it with ruby-lsp (rubocop) yields:
LSP[ruby_lsp] Formatting error: Infinite loop detected in ./prism_bug.rb and caused by Layout/ExtraSpacing, Layout/SpaceAroundMethodCallOperator
Hint: Please update to the latest RuboCop version if not already in use, and report a bug if the issue still occurs on this version.

Using bundle exec rubocop -a prism_bug.rb results in a step-by-step breakdown of the states:
prism_bug.rb:4:23: C: [Corrected] Layout/ExtraSpacing: Unnecessary spacing detected.
  "content" => <<~XML.    strip,
                      ^^^
prism_bug.rb:4:23: C: [Corrected] Layout/HeredocIndentation: Use 2 spaces for indentation in a heredoc.
  "content" => <<~XML.strip, ...
                      ^^^^^^
prism_bug.rb:4:23: C: [Corrected] Layout/SpaceAroundMethodCallOperator: Avoid using spaces around a method call operator.
  "content" => <<~XML.    strip,
                      ^^^^

0 files inspected, 3 offenses detected, 3 offenses corrected
Infinite loop detected in .prism_bug.rb and caused by Layout/ExtraSpacing, Layout/SpaceAroundMethodCallOperator
Hint: Please update to the latest RuboCop version if not already in use, and report a bug if the issue still occurs on this version.
Please check the latest version at https://rubygems.org/gems/rubocop.

This is now fixed on main and will be released soon

Thanks - keep up the great work :)


How to set the swagger Summary with Tags?

Swagger  used in ApiController can has summary from the class name like :
/// <"summary"">
/// Just For Test
/// </"summary">
public class TestController : BaseController
How to set this summary in FastEndpoints as it has no controller but tags?

upgrade to v5.2 and enable xml documentation like this
or use Summary(...) method: https://fast-endpoints.com/docs/swagger-support#swagger-documentation

/// <summary>
/// this is the summary
/// </summary>
/// <remarks>this is the description</remarks>
public class Endpoint : EndpointWithoutRequest
{
    public override void Configure()
    {
        Get("test");
    }

    public override async Task HandleAsync(CancellationToken c)
    {
        await SendOkAsync();
    }
}

public override void Configure()
{
    Get("test");
    Summary(s =>
    {
        s.Summary = "this is the summary";
        s.Description = "this is the description";
    });
}

public class EndpointSummary : Summary<Endpoint>
{
    public EndpointSummary()
    {
        Summary = "this is the summary";
        Description = "this is the description";
    }
}

upgrade to v5.2 and enable xml documentation like this
or use Summary(...) method: https://fast-endpoints.com/docs/swagger-support#swagger-documentation

I would want a summary about the Tags, not the Endpoint.
What you said is about the summary about the endpint ,  but the endpoint "abc/getabc/{id}"  has a router "abc", I want swagger to show  router "abc" is use for what purpose, thanks.
public override void Configure()
{
Get("abc/Getabc/{id}");
Options(x => x.WithTags("abc"));
Summary(s =>
{
s.Summary = "this is  the Getabc summary";
s.Description = "this is Getabc the description";
s.Params[id]="this is param id description";
//how to set the description about abc ?
});
}

with v<IP_ADDRESS>-beta you can add tag descriptions for a document like this:
builder.Services.AddSwaggerDoc(s =>
{
    s.TagDescriptions(
        new OpenApiTag() { Name = "my-tag", Description = "my tag description" },
        new OpenApiTag() { Name = "another-tag", Description = "another tag descrptions" });
});

so if an endpoint is tagged with a matching tag name (such as my-tag), it will show up under the document tags you add as above.


0.71 Update to Open Source

This updates the open source repo to the latest 0.71 release of Dragon.

Thanks for getting the updates in! I understand you verified an open source package both single and multi-node.


file modified indicator overlaps filename

X-icons are only shown on hover, with a subtle shadow behind them to prevent overlap with filenames. However, the modified indication dot is always visible but lacks the shadow.

This has already been fixed in #144.


feat(single-instance): add MacOs unix domain socket impl

Hi! My Tauri project needs single-instance support on macOS as it'll mainly launch through the CLI. Last week during my research I ran into this PR. Since back then little movement had happened, I initially copied the solution to tweak it to v2, but FIFO files were giving some problems for my (rather complex) use case. I decided to rewrite it using unix domain sockets.
I now read this comment and wanted to share my solution as a possible alternative. You can see it in action in the video below.
https://github.com/tauri-apps/plugins-workspace/assets/7761005/52b57523-3675-47d5-8bd9-1d704d3c9bee

Thanks for contributing :)
Did you ever try to change the order to first try to bin the UnixStream? I've been doing that and couldn't spot any issues though my use of unix sockets on macos wasn't tested that much... It would simplify the implementation quite a bit as you could get rid of the whole retry logic.

Thank you! On first glance (i'll do a proper review later, busy day) the only thing left to do is to replace println/eprintln with log::debug since most of them are part of normal operation. Maybe use log::warn and/or log::error for those that aren't, like the single_instance failed to listen to other processes - launching normally: {} one.
Oh and we'll also need a changefile like https://github.com/tauri-apps/plugins-workspace/blob/v2/.changes/fix-shutdown-timing.md :)

Hey again! I just found some time to wrap up this PR. I pushed the log changes and added a changefile. Please check it out and let me know if I can somehow help.


Cannot run deptrac on PHP8.1

Version: qossmic/deptrac-shim  0.23.0
Box Requirements Checker
========================

> Using PHP 8.1.7
> PHP is using the following php.ini file:
  /etc/php/8.1/cli/php.ini

> Checking Box requirements:
  E...

 [ERROR] Your system is not ready to run the application.

Fix the following mandatory requirements:
=========================================

 * The application requires the version "^7.2.5 || 8.0.*" or greater.

Never mind. Our script run older version from cache.


Allow for non-read-only queries in the Queries UI

Currently, the Queries UI only permits read-only queries. This was a limitation put in place to prevent users from taking destructive actions (by accident). However, it turns out that it's quite useful to be able to make "write" statements in the editor, for management actions that's not yet available in the UI.
Consider adding a "non-read-only mode" in the UI (and backend). I'm thinking a toggle with a warning perhaps.

This PR: https://github.com/mergestat/mergestat/pull/544 adds a param to the execSQL graphql query to toggle readonly/non-readoly executions. We can use that.
This ticket should be for the UI implementation of that toggle.


Code rectangle is extremely large when used with opengl.

Description of bug / unexpected behavior

While working on #1735 I noticed that the Code mobject looked mostly fine, except for the very large background rectangle around it:
(will need to zoom out)
class CodeFromString(Scene):
    def construct(self):
        code = '''from manim import Scene, Square

        class FadeInSquare(Scene):
            def construct(self):
                s = Square()
                self.play(FadeIn(s))
                self.play(s.animate.scale(2))
                self.wait()
        '''
        rendered_code = Code(code=code, tab_width=4, background="window",
                            language="Python", font="Monospace", scale_factor=0.3)
        self.add(rendered_code)

        self.interactive_embed()

This odd behaviour only exists with the opengl renderer.
Expected behavior

Running the exact same code in cairo:

Logs
N/A
Terminal output

N/A
```
PASTE HERE OR PROVIDE LINK TO https://pastebin.com/ OR SIMILAR
```
System specifications

OS windows 11
RAM: 8 GB
Python version 3.9

System Details
Additional comments

I tried debugging this, and I think i've pinpointed the cause of the issue to be at gen_line_chars  for Paragraph starting from gen_colored_lines in Code.
from my initial attempts at debugging this:

i took the sum of the widths of everything in the vgroups for both opengl/cairo and they're both the same, it's just that when i get the vgroup as a whole that the widths are mismatched

relevant lines
https://github.com/ManimCommunity/manim/blob/main/manim/mobject/svg/text_mobject.py#L514-L530

I was looking at this and found that when the line below gets called only the submobjects of self.chars get scaled but the parent self.chars itself which is an OpenGLVGroup object doesn't scale with them so the width doesn't change. In cairo renderer both the submobjects and the parent's width is scaled, so maybe that could be causing it?
https://github.com/ManimCommunity/manim/blob/bc93a2e99afb3f77c18a0c7e56d148aca10683f8/manim/mobject/svg/text_mobject.py#L509

I doubt it's an actual fix but it seems to scale on both cairo and opengl when changing the above line to
            self.chars.scale(TEXT_MOB_SCALE_FACTOR)


Refs #31670 -- Used allowlist_externals in tox.ini.

Tox has deprecated whitelist_externals in favour of allowlist_externals
https://tox.readthedocs.io/en/latest/config.html#conf-allowlist_externals

Thanks both :+1:


All endpoints become unreachable after publishing specific endpoint

1.
2. To Reproduce

Expose the production orders by an API v2 endpoint:
`page 50088 "API - Production Order"
{
PageType = API;
APIVersion = 'v2.0';
APIPublisher = 'trds';
APIGroup = 'standard';
EntityCaption = 'ProductionOrder';
EntitySetCaption = 'ProductionOrders';
EntityName = 'productionOrder';
EntitySetName = 'productionOrders';
SourceTable = "Production Order";
SourceTableView = where(Status = filter(Released));
ODataKeyFields = Status, No.;
Extensible = false;
DelayedInsert = true;
layout
{
area(Content)
{
repeater(data)
{
field(status; Rec.Status)
{
Caption = 'Status';
}
field(no; Rec."No.")
{
Caption = 'No.';
}
}
}
}
`

Publish to server.

Try to reach any endpoint, it does not matter which one. It fails with
"Error 500 - Internal Server Error
Sequence contains more than one matching element"

3. Expected behavior
I can read the endpoint and other endpoints too.
4. Actual behavior
All endpoints become unreachable until the endpoint is removed.
5. Versions:

AL Language: 8.1
Visual Studio Code: 1.87.2
Business Central: Business Central 2021 release wave 2
List of Visual Studio Code extensions that you have installed:  AL Language Extension
Operating System:

Windows

This repository is only dedicated to bug reports for AL compiler and developer tools. This is an issue with the runtime and is not covered by the repository.
Runtime issues should follow the support process indicated below.

If you are a Microsoft Partner, follow the documentation here to escalate a customer issue
If you are a Business Central customer, read about reporting a problem
Open a bug in Collaborate
Contact your Service Account Manager (SAM) in your local subsidiary to understand what is included in your contract as of support incident and PAH (Partner Advisory Hours). Your SAM might also direct you "step by step" how to open a support request or how to get credentials, if this is the first time for you or your company.

You can also use Viva Engage for asking questions and providing feedback around the runtime behavior of the product.


[Idea] Config file to certainly not use a lot of arguments

A posible idea to make it can be a config file because a lot of users don't know anything about arguments or complicated arguments

Hmm I don't think this is currently on scope, the idea of this project is to be an interface, and for example to be use later by another project to create a GUI, Web app or whatever.
So for now I guess we'll have to stick to arguments :(

i can create a WPF interface :)


[Catch2] Wrong libdir in Catch2Main.pc

Is your feature request related to a problem? Please describe.
Catch2Main.pc libdir ist set to prefix/lib but the catch2main.lib was moved to manual-link
Proposed solution
change the libdir var to prefix/lib/manual-link
Describe alternatives you've considered
No response
Additional context
No response

I assume what was meant is catch2-with-main.pc.


Show Browser Release Date / Add age-aligned mode

Hi,
Think about extending the UA info adding the release date.
Example, when you see feature X is not working with android 2.3 would be very very handy to quickly find when android 2.3 was public released.
What do you think?

+1

+1
Agreed. Browser such as Firefox and Chrome have very short release cycles while Safari and IE/Edge have much longer cycles. As such, whether a feature was supported, say, two versions ago, has a different meaning for these browsers.
So for example, a feature which was implemented in versions, say, 1 year ago, would be be safer to use than features implemented 2 versions ago.

+1

@NKjoep: Could you please edit your first comment to engage peoples to vote for #202?
Vertical aligned for the current main web browsers for instance..
Versionitis is a serious disease of todays web browser manufacturers.
https://upload.wikimedia.org/wikipedia/commons/7/74/Timeline_of_web_browsers.svg

@peterdd I edited the first comment, I agree on versionitis being a serious illness nowadays.

Browser release dates are now included in the tooltip when you hover/tap on a version.
Have started working on the UI for a "release date relative mode".

New UI is now available! See http://caniuse.com/feed/157 for details.

How about some horizontal year lines in the date relative view?


Highlight search result match?

Hi,
Is there a way to highlight matches in the search results returned by pg_search?
Thanks

Highlights are now present in the latest versions. See the README.


Handle multiple series in aggregateLine

The aggregateLine function should be able to handle multiple series lists being passed to it.
Also adjusts the result name so that when calling with aggregateLine(foo-*.bar) it will return each match, instead of aggregateLine(foo-*.bar, 42), aggregateLine(foo-*.bar, 45), ...

@iain-buclaw-sociomantic The tests are still failing, can you fix this up and we'll get it merged in?

Sure, I'll look into it.

Hmm, would the desired behaviour be to ignore aggregated None values here?  Maybe not...

[Current coverage][cc-pull] is 65.30% (diff: 100%)

Merging [#1439][cc-pull] into [master][cc-base-branch] will increase coverage by 0.03%

@@             master      #1439   diff @@
==========================================
  Files            52         52
  Lines          5842       5848     +6
  Methods           0          0
  Messages          0          0
  Branches       1131       1133     +2
==========================================
+ Hits           3813       3819     +6
  Misses         1827       1827
  Partials        202        202

Powered by Codecov. Last update [093999f...b1adfc8][cc-compare]
[cc-base-branch]: https://codecov.io/gh/graphite-project/graphite-web/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/graphite-project/graphite-web/compare/093999f4f5ee573370d45a3c3d77af162c897bbd...b1adfc8024666fe4cebc17caf9b0363e7f2eda5f?src=pr
[cc-pull]: https://codecov.io/gh/graphite-project/graphite-web/pull/1439?src=pr

Would you mind also updating the comment to reflect that it supports multiple series?

OK, done.

Thanks!


Safari tabs

Thoughts tabs could be more native-y, so I made them look like Safari tabs. No biggie if you don't like this one—just messing with stuff.

One thing that would be nice to do is toggle a class when the app loses/gains focus. I know there are events for it, but I couldn't get them to work.

Looks dope.


Densmap supervised (issue 609)

Coverage increased (+0.5%) to 88.944% when pulling fdc9b72700b03bf565c1ccc718da0c4220a5adca on rxjx:densmap_supervised into b1223505ca56ae104feb35e4196227277d1e8058 on lmcinnes:master.

Could you run it supervised on Fashion-MNIST and post the resulting plot? I just want to be sure it "mostly works" in reasonable cases.

Yeah, I think I can do it over the weekend.

Looks good! I was worried there might be enough weird interactions to make it go screwy, but it seems good enough for now. Thanks for taking the time and trouble to look into this. The test is also greatly appreciated.

No problem at all. Thank you for UMAP and all the work you've put into it.


[action] gradle: Update for GRADLE_OUTPUT_JSON_OUTPUT_PATH and GRADLE…

…_ALL_OUTPUT_JSON_OUTPUT_PATHS in Android studio 4+
The output.json file has been rename to output-metadata.json in Android studio 4+
Checklist

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

Motivation and Context
I need to read meta data from this file, but this one has renamed
Description
The old shared value (GRADLE_OUTPUT_JSON_OUTPUT_PATH || GRADLE_ALL_OUTPUT_JSON_OUTPUT_PATHS) is outdated, need to update with new file name
Testing Steps

@googlebot I signed it!

@googlebot I consent.