Fix README

Function batchGeocode's callback function's first param is the error message. Results array is second param.

Thanks for the correction 👍


How can we automate the sql database to azurr sql database migrations

I have about 200 databases that I need to migrate how can I go about automating this so that, I haven't going to log in to the on prem vm  ?

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

ID: 088ad5a0-569b-e8ea-7cce-83757c37c427
Version Independent ID: 52097b3c-58cc-a3e3-9511-e8f9d88b2f69
Content: Migrate SQL Server DB to Azure SQL Database using DMA
Content Source: articles/sql-database/sql-database-migrate-your-sql-server-database.md
Service: sql-database
GitHub Login: @sachinpMSFT
Microsoft Alias: sachinp

@Anmolgan81 Thanks for your interest in Azure products and services. We are actively investigating and will get back to you soon. Thank you, ~Mike

@Anmolgan81 I will continue to look at all the options that are available but it appears that there is a PowerShell library to automate a bulk migration task.
Automating the end-to-end migration of SQL Server to Azure SQL Database using the Azure Database Migration PowerShell module

@Anmolgan81 See the Data Migration Service @ https://azure.microsoft.com/en-us/services/database-migration/

@Mike-Ubezzi-MSFT @CarlRabeler  How can I automate this? The guide that @Mike-Ubezzi-MSFT has shared is not very helpful are there ways using which I an automate this like the Azure seRviCes automation account?

@Anmolgan81 let me see what else I can find for you - stay tuned

@CarlRabeler ok, it will be helpful, I will wait for your reply.

@Mike-Ubezzi-MSFT So there are no detailed documentation as to how can this be done?

There is but to clarify....how much do you need automated? Do you need to manually kick off a script that runs as a foreground process or do you need it to be scheduled (like a cron job) and to be run as a background task. Thanks!

@Mike-Ubezzi-MSFT I want it to be a one time process and without being able to login onto the onpremise servers I will be able to Copy the data to Azure SQL Database, Its like deploying the Onpremise database to Azure SQL Database by exporting the data tier, can something like that be done?

@Mike-Ubezzi-MSFT lets say there are 200 databases in 1 SQL Server and 100 in another one I want a script or something that will get executed by passing in onpremise details and that will export the datatier and deploy the database in azure for each of them SQL Servers.

@Anmolgan81 Have you looked at Azure Database Migration Service? It offers several source to target scenarios with new scenarios actively being developed and released fairly often. In the case of an on prem SQL Server to Azure SQL it offers a one-time migration as well as continuous sync. In a nutshell, once you set up the service you create tasks to migrate 1 to N databases to a specified target. You can also use PowerShell or Azure's Python CLI to set up the details and kick off the tasks. More information can be found here: https://azure.microsoft.com/en-us/services/database-migration/. If you would like guidance on how to prepare for the migration you can visit the Database Migration Guide at https://datamigration.microsoft.com/. If you have further questions please ask and if I can't answer I can connect you to someone who can.

@vchske We want to build a solution for one of out customer who wants to automate the migration of various databases attached with different SQL Server, Currently he do not want us to access the VM, although we have the access, how can we do this remotley deploying a script which will run on the VM and it will be executed externally and not on the VM and will export the data to azure SQL Databases?
Can you help me on that?

Can I ask if it's integral to deploy a script to their VM? Is their VM on Azure? If their VM can connect to a VNet then would DMS be a good candidate for them? You wouldn't need to access their VM and once the task is kicked off it just runs until the data is migrated. It's very straightforward (provided their data is ready for Azure SQL DB as there are a couple differences but they are usually minor if coming from SQL Server).

@vchske yes the sql server is inside the azure vm, what can be the best simplified approach to use the method in order to migrate the data to azure sql database?
Do data migration service provisions the azure sql databases?
How can we use this without getting on to the vm?
Get Outlook for Androidhttps://aka.ms/ghei36

From: vchske<EMAIL_ADDRESS>Sent: Friday, September 28, 2018 2:35:52 AM
To: MicrosoftDocs/azure-docs
Cc: Anmol Ganju; Mention
Subject: Re: [MicrosoftDocs/azure-docs] How can we automate the sql database to azurr sql database migrations (#15689)
Can I ask if it's integral to deploy a script to their VM? Is their VM on Azure? If their VM can connect to a VNet then would DMS be a good candidate for them? You wouldn't need to access their VM and once the task is kicked off it just runs until the data is migrated. It's very straightforward (provided their data is ready for Azure SQL DB as there are a couple differences but they are usually minor if coming from SQL Server).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/15689#issuecomment-425243016, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AhHKpxOn3djxP6Bt72MMpEeyPgnUiV_kks5ufT2wgaJpZM4W7ewe.
This message contains confidential information and is intended only for the individual named. If you are not the named addressee you should not disseminate, distribute or copy this e-mail. Please notify the sender immediately by e-mail if you have received this e-mail by mistake and delete this e-mail from your system. E-mail transmission cannot be guaranteed to be secure or error-free as information could be intercepted, corrupted, lost, destroyed, arrive late or incomplete, or contain viruses. The sender therefore does not accept liability for any errors or omissions in the contents of this message, which arise as a result of e-mail transmission. If verification is required please request a hard-copy version. IFI Techsolutions Pvt. Ltd. : www.ifi.tech

@Anmolgan81 DMS has a scenario currently in private preview that allows for migrating the schema which provisions the database. That's fairly automated from soup to nuts. If there is some urgency, they can use DMA to generate a schema script and provision the Azure SQL database (that can be done via PS or Python) and then use DMS to migrate the data. DMA is handy in that it will point out any discrepancies between their current schema and what Azure SQL demands. None of these scenarios require anyone to be on their VM. It just uses a SQL connection (which means SQL ports will need to be open).
I'm trying to find out how one onboards with a private preview and will get back on this point as soon as I can.
Are all these ~300 databases using the same schema? All different?
(more info on DMA can be found here https://docs.microsoft.com/en-us/sql/dma/dma-overview?view=sql-server-2017)

@Anmolgan81 You should be able to onboard with the schema scenario. For that, I would direct you to talk to Raj at<EMAIL_ADDRESS>He will be able to walk you through the process. There will be a bit of a delay to onboard. That is why I say, if the timeline is urgent then using DMA to generate the schema(s) would be the most expedient approach.

@vchske so How can we automate the DMA mechanism?

@Anmolgan81 Unfortunately you cannot automate the DMA process at this time. This is why I was asking if they have 300 schemas or 300 databases using the same schema. At any rate, I highly recommend you reach out to Raj<EMAIL_ADDRESS>at this point. He is probably best qualified to walk you through all the various options. Feel free to let me know if I can help the process move forward for you.

@vchske I have mailed this to viraj, currently we have 1000 databases on which we have 724 db's that uses the same schemas and other of the databases uses some different schemas. How can we migrate these db's to azure sql database's so that sql databases should also be created in azure and migrated as well. Also they want to test the SQL Databases migration into the Managed Instances, how can we go about this here?
They have a Datawarehouse that they want to migrate to Azure Datawarehouse how should we approach this here?

@vchske hi I was automating the migration of 700+ databases using the same schema, I have automated this to an extent, I have built a script which which will hse the DMS to migrate those databases but currently it's failing, I have reached out to raj here as well, but I want to get this done today, let me know if anyone wants to help me on this and I will share the script which I am using to automate this.


[FORMATTING] Params not working when dialect is postgresql.

Input data
select $1

with params:
["a"]

Expected Output
select a

Actual Output
select

Usage

How are you calling / using the library?

console.log(
  formatDialect("select $1", {
    dialect: postgresql,
    params: ["a"],
  })
);

What SQL language(s) does this apply to?
Postgres
Which SQL Formatter version are you using?
12.0.4

Parameter placeholders seem to count from zero.
Case 1
Input
select $0

["a"]

Output
select a

Case 2
Input
select $1

["a","b"]

Output
select b

For the numbered placeholders one should really use an object like {1: "a", 2: "b"} instead of an array. It's pretty clearly documented.
I agree it's not the most intuitive approach, but then again, making $1 correspond to index 0 is similarly not quite intuitive. For now the API is like so by design.

Thank you for your kind answer.


[documentation] Documentation for NakadiClient.publish is missing

The publish function should be documented, especially as it is not obvious, a camelCase -> snake_case conversion is applied on the events.

This is funny. I didn't even know this existed.


Re generate queries after applying a migration

So, this is the first time I'm in need to run a migration on SQLDelight.
The library is truly a delight to use btw. Thank you so much for your hard work!
Here's my problem. I first create a database:
CREATE TABLE IF NOT EXISTS users (
    id                      TEXT PRIMARY KEY,
    username                TEXT NOT NULL,
);

CREATE TABLE IF NOT EXISTS albums (
    id                      TEXT PRIMARY KEY,
    user                    TEXT NOT NULL,
    FOREIGN KEY(user) REFERENCES users(id)
);

getAlbums:
SELECT * FROM albums INNER JOIN users ON users.id = user WHERE albums.id = ?;

So far so good. Now I decide to run a migration, to do that I create 1.sqm with the following statement
ALTER TABLE albums ADD COLUMN some_column INTEGER DEFAULT 0 NOT NULL;

Now, the getAlbums query is still not returning this new column for me...
I assume the migration was run...? At least calling Database.Schema.version returns 2.
And the database implementation generated code reads
override fun migrate(
            driver: SqlDriver,
            oldVersion: Int,
            newVersion: Int
        ) {
            if (oldVersion <= 1 && newVersion > 1) {
                driver.execute(null,
                        "ALTER TABLE albums ADD COLUMN whatever INTEGER DEFAULT 0 NOT NULL;",
                        0)
            }
        }

So, it looks OK. However, the getAlbumsImp generated code doesn't reflect this new field. I think it should, shouldn't it?
environment info:
Multiplatform project
ext.kotlin_version = '1.3.41'
ext.sqldelight_version = '1.1.4'
 ext.android_gradle_version = '3.4.2'

Have you updated the original .sq files?
I think you need to update CREATE statements and SELECT statements as well.

Thanks @yshrsmz! I tested this and it works perfectly, I think I had the wrong idea of how migrations worked. Thanks for the clarification!


crash floating point exception

Describe the bug
crash on ubuntu20.04
[20:53:35] Lumos Engine - Version 0.3.2
[20:53:35] Initialised JobSystem with [16 cores] [15 threads]
[20:53:35] Initialising System
[20:53:35] Editor Ini not found
[20:53:35] Creating Editor Ini Editor.ini
[20:53:35] No saved Project file found ExampleProject/Example.lmproj
[20:53:35] [SceneManager] - Enqueued scene : Empty Scene
[20:53:35] Serialising Application NewProject/NewProject.lmproj
[20:53:35] VSync : False
[20:53:35] Creating window - Title : App, Width : 1200, Height : 800
[20:53:35] Initialised GLFW version : 3.3.4 X11 GLX EGL OSMesa clock_gettime evdev
[20:53:35] Vulkan : 1.1.182
[20:53:35] GPU : llvmpipe (LLVM 12.0.0, 256 bits)
[20:53:35] Vendor ID : 65541
[20:53:35] Device Type : CPU
[20:53:35] Driver Version : 0.0.1
[20:53:35] Selected physical device has 54 extensions
WARNING: lavapipe is not a conformant vulkan implementation, testing use only.
[1]    1352178 floating point exception (core dumped)  ./LumosEditor

Describe the bug crash on ubuntu20.04
[20:53:35] Lumos Engine - Version 0.3.2 [20:53:35] Initialised JobSystem with [16 cores] [15 threads] [20:53:35] Initialising System [20:53:35] Editor Ini not found [20:53:35] Creating Editor Ini Editor.ini [20:53:35] No saved Project file found ExampleProject/Example.lmproj [20:53:35] [SceneManager] - Enqueued scene : Empty Scene [20:53:35] Serialising Application NewProject/NewProject.lmproj [20:53:35] VSync : False [20:53:35] Creating window - Title : App, Width : 1200, Height : 800 [20:53:35] Initialised GLFW version : 3.3.4 X11 GLX EGL OSMesa clock_gettime evdev [20:53:35] Vulkan : 1.1.182 [20:53:35] GPU : llvmpipe (LLVM 12.0.0, 256 bits) [20:53:35] Vendor ID : 65541 [20:53:35] Device Type : CPU [20:53:35] Driver Version : 0.0.1 [20:53:35] Selected physical device has 54 extensions WARNING: lavapipe is not a conformant vulkan implementation, testing use only. [1] 1352178 floating point exception (core dumped) ./LumosEditor

I find this issue disappeared after restart my computer.


Can't create account/log in with google

Describe the bug
Cannot create an account on https://admin.leddepartureboard.com/login
The login screen opens very briefly, then closes
To Reproduce
Steps to reproduce the behavior:
0. On google chrome/safari (macbook pro)

Go to https://admin.leddepartureboard.com/login
Click on "sign in with google"
Wait, the dialog box will exit.
See error

Expected behavior
Should be able to log in using google.

https://github.com/benfl3713/DepartureBoard/assets/32208841/da5e1504-dbb1-4826-83a1-c6ecd0dad570
Here's a video

Hi there @aronhegedus
Thanks for letting me know about this problem
I think it was a configuration issue with my authentication provider.
I've updated the config to hopefully solve the problem.
Hopefully if you try again now it should work for you
Cheers
Ben


Climate sensor

It would be nice if we could add climate sensors like DHT22 or similar...

Digging through my sensors bin, I don't seem to have any of these sensors. I'll order some from aliexpress but it's going to take a while to ship...
I might get my hands on a couple of Dallas sensors sooner though.

First of all thanks for this great project.  But could you please add the bme280 sensor for heating control.  So I don't need any additional hardware to use the display better for smart room control

Adding a sensor not the hard part. But to what endpoint/topic does the sensor data need to be sent and in what format?
On the receiving end there has to be a controller or some logic that processes these messages, like the Custom Component.
If I know what the message requirements are, it will be a lot easier to provide the data in a way that can be processed.

(Not) One data model fits them all.
I'm sure if I ask nine people in wich form the collected data of a bme280 for example should be transmitted i will get ten answers :-).
The Temperature for example:

temp: "19.6 °C" -> will be nice to show it on a screen but for math you need to parse the value first.
temp: 19,600 -> great for most european math. In USA ... oops ... you'll need a lot of ice. And by the way. °C or °F?
temperature: 19.6, tempUnit: "Celsius" -> show on screen as it is ... no way ... but concatenating and localising may be easier then parsing ... hmmm

Going further:

how should the data be structured if you have .. lets say two temp units to measure a delta?
So what suggestion for a format is the right one?
People out there all have their own preferences which sensors or other hardware they want to attach and what automatic behavior they need. If you take a short look at the Tasmota Firmware or ESP Easy you can see which dimensions the support of devices and ruled behavior can reach easily.

Lets see some more docs:

Amazon Alexa
Google Smarthome
EnOcean EEP
Tasmota firmware
ESP Easy - BME280

Wow, keep this horse in check!
As I remember there are several requests (Issues and Discussions) to support alternative hardware within OpenHASP the last 2 years. But I think it will take some "glue" between the CUSTOM Interface and the differnt hardware implementations to get that managed.
Maybe:

some more advanced examples on howto get managed a build and deploy pipeline for the firmware.
some examples on how to create a "controller" on other Smart Home Systems (e.g. openHAB).
some guidelines for the "glue".
it's all there and i don't know the www links (if true, please tell me)

But to end the philosophizing. This is my suggestion for the data format.
{
  "custom": {
    "<topic compliant string>": {
      "customId": "<topic compliant string>",
      "meta": {
        "deviceType": "eg. BME208",
        "pressureUnit": "hPa",
        "tempUnit": "C"
      },
      "properties": {
        "lastUpdate": "<iso8601 utc zulu time>",
        "temperature": 21.7,
        "humidity": 66.6,
        "pressure": 988.6
      }
    }
  }
}

Remarks:

Just read the json properties in chain as the custom topic (eg. /custom/CUSTOM_ID/properties/temperature)
populate meta less more then properties and not in the "get sensor data" callback
don't use an array's in the "custom" property or numeric indexes. Stay with the custom_id. It will make trouble if you want to do things like a JSON Merge or JSON Patch on more advanced "controllers" in the backend. And, be honest, what is "/plate_46/state/custom/8/xyz"?

Regards to all the lovely work on OpenHASP.

Adding a sensor not the hard part. But to what endpoint/topic does the sensor data need to be sent and in what format? On the receiving end there has to be a controller or some logic that processes these messages, like the Custom Component.
If I know what the message requirements are, it will be a lot easier to provide the data in a way that can be processed.

For me, it would bei enough when in see im my mqtt Broker (i use iobroker with mqtt Adapter) an additonal datapoint, like for example a Display Button, but with the temperature , so i can use the state to control my heating valve and also send it back to the Display

I just finished my_custom.ccp which features a motion sensor and a dht22 temp/humidity sensor. I'm fairly new to c++, arduino and openHasp so comments are welcome. I thought some folks here would be interested...
my_custom .cpp.txt

There some other critical aspects to adding sensors into a device. You really must also include an identifier of exactly what part/chip/transducer is providing the data so that you can know the accuracy and range of the sensor.
We often think that getting the temperature or humidity is enough, but without understanding the accuracy or the part or where the reading is within it's operational range, your code could be making bad choices in controlling things.

Sure, for my application which is not a critical one it suffices to double check the numbers with another device I know is accurate. I had to change my design a couple of times and ended up with an external case for the sensor because the board gives off too much heat. But that’s the responsibility of the diy’er to do…

@fvanroie I added the arduino-sht library in platform-override.ini and wrote few lines of code in my_custom.cpp to read from an SHT31 sensor, but I found that Wire (first I2C port) is already initialized (Wire1 is used by the touchscreen) and I had to call Wire.end() before re-initializing it with my custom pins. In this way it works, but is there something I broke?
I have a WT32-SC01 Plus with 3ab88a3.


Can't run Netflix

I'm submitting a ...

[x ] bug report
[ ] feature request
[ ] support request

General infomration
Prerequisites

[x ] Are you running the latest version?

Expected behavior: [What you expected to happen]
Show the netflix list of Movie and serie
Actual behavior: [What actually happened]
Show that is a error and then just show
-Files
-Playlists
-Video add-ons

[x ] I installed the plugin via zip from the Releases page
[ ] I´m using the Netflix Repo
[ ] I´m using a different source (Please tell which)

Operating System

[x] Linux (ARM) raspberry Pi 3

Additional informatin on the environment
Run LibreElect the latest night build from milhouse
Debug log
https://paste.ubuntu.com/p/HWK2ZtSF9X/
I have write my email address and password (it ask for that when I inside "Video add-ons"
But same error
I also have Viaplay on same machine and it also use Libwidevine and Inputstream.adaptive and it running fine.

Here
https://paste.ubuntu.com/p/QWsVmprYXW/

OK. The initial error is list index is out of range. And the only element referenced is react_context[0]. So does that mean react_context = None? If so, why did line 146 of NetflixSession.py fail to find "netflix.reactContext" in "content"?
I can monitor the contents of "content", so if the error would occur for me (which it doesn't for 0.13.5 or 0.13.6) I might get a better idea of whats going on.
Is this issue the same as #373?

A Little update
I did notice it was a new version of Netflix Add-on so I did install the older one 0.13.5 and now it works :)
I guess it upgrade to 0.13.6 when I did not notice it and that version does not my system like at all

@TheCazz can you pls. install 0.13.7, and if working close the issue? Thx

It working :)


Swarms created on Hub discovery service should be linked to Hub accounts

At some point, we may have some kind of UI for managing Swarms created with the Hub discovery service. To be able to do this, we should connect Swarms that have been created to Hub accounts.
When running swarm create, it should first ask you to provide user/pass to log in with, then the Swarm will be attached to this user.
In the future when we allow you to use custom token servers, this could also work in a similar way with them, like you can provide auth details to a third-party image registry. Perhaps there could be some way of making it optional for third party token servers?

This is being worked on #695

Closing this issue since Hub discovery service is now deprecated: https://github.com/docker/swarm/pull/2743

Oh wow! I remember this. So pleasing to see that after all these years the intent behind this has now actually been implemented in Cloud and D4M. 🎉

(Congrats!)

@bfirsh much has changed too 😸


Send mail using Sendgrid API key not with SMTP in Codeigniter

Summary
I need to send mail through API key details in codeigniter and not with the SMTP details which is shown in the example below
application/config/email.php

/* application/config/email.php */

if ( ! defined('BASEPATH')) exit('No direct script access allowed');

/*
|--------------------------------------------------------------------------
| SendGrid Setup
|--------------------------------------------------------------------------
|
| All we have to do is configure CodeIgniter to send using the SendGrid
| SMTP relay:
*/
$config['protocol']	= 'smtp';
$config['smtp_port']	= '587';
$config['smtp_host']	= 'smtp.sendgrid.net';
$config['smtp_user']	= 'yourusername';
$config['smtp_pass']	= 'yourpassword';

application/controller/Welcome.php

defined('BASEPATH') OR exit('No direct script access allowed');

class Welcome extends CI_Controller {
	public function index()
	{
		$this->load->library('email');

<EMAIL_ADDRESS>'John');
		$this->email->to('test123@gmail.com');
		$this->email->subject('Test Email using SendGrid');
		$this->email->message('This email was delivered by your friends at SendGrid');

		$this->email->send();

		echo $this->email->print_debugger();

		$this->load->view('welcome_message');
	}
}

Technical details:

sendgrid-php version: 3
php version: 7.2
codeigniter version: 3

It doesn't appear you'll be able to use CodeIgniter's built-in email library if you want to use an API key for authorization. To use an API key, you have to install the sendgrid-php library and use as regularly documented.

Yes. I accept your proposal. Because it is only way to proceed further.


Hardwired schema urls for model parameters

All schema files include a field called meta_schema_url, which is the url of the metaschema in the github repository.
The url points to the main branch, which is not ideal as any changes to directory or file names means that older versions don't work anymore. Discuss if there is a better solution.

To explain this a bit better. We have in the header of each schema:
%YAML 1.2
---
title: Schema for array_coordinates model parameter
version: 0.1.0
meta_schema: simpipe-schema
meta_schema_url: https://raw.githubusercontent.com/gammasim/simtools/main/src/simtools/schemas/model_parameter_and_data_schema.metaschema.yml

In case of parameter renaming, removal, or move of the schema file this url becomes invalid for previously release versions. Not an issue right now during development, but possibly later (especially as we require reproducibility)
Only solution which comes to my mind is to add the release number for each release of simtools, e.g., then we have
meta_schema_url: https://raw.githubusercontent.com/gammasim/simtools/refs/tags/v0.8.2/src/simtools/schemas/model_parameters/array_coordinates.schema.yml

Not nice, but would work.
Again, this is a bit of an issue for the pedantic ones.


ci: run the tests with Podman

@aureamunoz here's a first try to have Podman in CI, but that's an old version here: podman version 3.4.4

@aureamunoz there's a bit of progress

Note: this branch is going to be used for dirty tests, proper PRs will be extracted by cherry-picking relevant commits 🤣

Superseded by #14


Issues section partially covers Flutter app

See screenshot in https://master-api.flutter.dev/flutter/material/BottomNavigationBar-class.html

I think you meant to link to https://github.com/dart-lang/dart-pad/issues/1743 here

I'm a little worried by how much it may limit vertical space in these use cases

Me too, but at least it would be better than what we currently have...


do not expose global "yue"

I noticed that using require("yue") not only returns the module, but also sets global yue variable:
> print(yue)
nil
> require"yue"
> print(yue)
table: 0x00698928

Is that intended?

Yes, it's intended and done by the Lua C-API from here for Lua version above 5.1
https://github.com/pigpigyyy/Yuescript/blob/b3d021e2bbb08d01961d2a597715faff5e78b715/src/yue.cpp#L47
and here for Lua 5.1
https://github.com/pigpigyyy/Yuescript/blob/b3d021e2bbb08d01961d2a597715faff5e78b715/src/yuescript/yuescript.cpp#L382
So you think it is more of polluting the global env instead of offering convenience?

So you think it is more of polluting the global env instead of offering convenience?

Yes, exactly, that is my opinion.

OK, got this issue fixed by lastest commit. After all, it won't cost much effort to manually expose yue as global variable.


ARROW-13155: [C++] MapGenerator should optionally forward reentrant pressure

Added a basic mapping generator that does not queue incoming jobs.  This allows it to forward async-reentrant pressure to the source.  Fixed some issues in the CSV reader that were preventing it from running truly parallel.  Performance is now significantly better but still not quite the same as the threaded reader.  For the NY taxi dataset the streaming read time went from ~7 seconds to ~1.6 seconds.  However, the file reader is still at ~0.8 seconds.  I'll do more investigation later.
Leaving in draft as I want to extract a thread spawning generator I created into an independently tested thing.

@ursabot please benchmark


ale doesn't respect .prettierignore file

Ale prettier fixer doesn't respect .prettierignore file rules.
Example:
.prettierignore
foo.js

Running this commend on the CLI will respect .prettierignore
$ prettier --write foo.js # won't be formatted

While running the same command through ale will modify the file, I assume that's because ale has no knowledge about the paths in .prettierignore & it just modifies the buffer.  I dunno how this can be fixed to be honest, any ideas?

Does prettier support fixing files via stdin with flag like --stdin-filename yet? That's the reason why things like this don't work so well.

To support this, we'll probably have to read the ignore file and apply the globs ourselves to ignore things.

Does prettier support fixing files via stdin with flag like --stdin-filename yet? That's the reason why things like this don't work so well.

I don't think so.

To support this, we'll probably have to read the ignore file and apply the globs ourselves to ignore things.

Yeah I thought so, but then it might be too much work especially that paths in .prettierignore mostly are relative & you can have multiple of these files inside multiple directories etc...

At the very least you can use g:ale_pattern_options to change the fixers for particular files.

Something like this for example to disable the fixer?
  let g:ale_pattern_options = {
  \   '\.foo\.js$': {
  \       'ale_fixers': [],
  \   },
  \}

Replace [], with something like {'javascript': []}.

Thanks!

As far as I understood from the :help, the regex will use match() to match agains the absolute path of the buffer.

I tried that too, still didn't work
let g:ale_pattern_options = {
      \   '\.min\.(js\|css)$': {
      \       'ale_linters': { 'javascript': [], 'css': [] },
      \       'ale_fixers': { 'javascript': [], 'css': [] },
      \   },
      \   'jquery.*': {
      \       'ale_linters': { 'javascript': [] },
      \       'ale_fixers': { 'javascript': [] },
      \   },
      \   'node_modules': {
      \       'ale_linters': { 'javascript': [], 'css': [], 'scss': [] },
      \       'ale_fixers': { 'javascript': [], 'css': [], 'scss': [] },
      \   },
      \   'Sites/forks': {
      \       'ale_fixers': { 'javascript': [], 'css': [], 'scss': [] },
      \   },
      \}

Could you share the filename you're trying to ignore, and the series of steps you followed?

Sure, the rules are the following:

ignore any file that ends with .min.(js|css). <--- this seems to be working fine
ignore any file that has this pattern jquery.js, jquery.somename.js, etc...
ignore any file inside node_modules folder.
ignore any file inside Sites/forks folder.

The first one seems to be working fine, the other 3 are not working at all. I tried with the following files

A .js file inside node_modules directory called bulk.js (doesn't get ignored)
A .js file inside Sites/fork directory called index.js (doesn't get ignored)
A .js file called jquery.tipsy.js (doesn't get ignored)
A .min.css file (works fine)
A .min.js file (works fine)

You might be interested in this issue I just opened: #1108

Will keep track of that issue & will close this now. Thanks for the help :)

I'll keep this open. I want to see if there's anything wrong with the patterns.

Right now I managed to do what I need doing this, not so elegant but it works for now :)
if (!empty(matchstr(expand('%:p'), '\(node_modules\|Sites/forks\|\(\.min\|jquery.*\).\(js\|css\)$\)')))
  let b:ale_fix_on_save = 0
endif

I have changed the behavior for g:ale_pattern_options now so all matched patterns will be applied instead of the first match. Let me know if that makes a difference for you. The patterns are sorted alphabetically, so the patterns will always be applied in a consistent order.

I tried it but it's not working for me still

This is my ale settings if you are interested, maybe I'm doing something wrong?

You might need to set let g:ale_pattern_options_enabled = 1 also, as you are defining the Dictionary after ALE is loaded.

This working now, thanks very much 👍
let g:ale_pattern_options_enabled = 1
let g:ale_pattern_options = {
      \   '\.min\.(js\|css)$': {
      \       'ale_linters': [],
      \       'ale_fixers': []
      \   },
      \   'jquery.*': {
      \       'ale_linters': [],
      \       'ale_fixers': []
      \   },
      \   'node_modules/.*': {
      \       'ale_linters': [],
      \       'ale_fixers': []
      \   },
      \   'Sites/forks/.*': {
      \       'ale_fixers': []
      \   },
      \}

Okay, sounds good. :+1:

FYI the orininal issue can be solved with this Prettier argument:
--stdin-filepath <path>  Path to the file to pretend that stdin comes from.

Can we reopen the issue?

Sure. I have since added support for chaining commands for fixers, but I haven't used it yet. We can pass that argument for the versions that support it.

I can modify the fixer to use the --stdin-filepath option, but I found out that prettier doesn't read the ignore file if you use that option. I'll start using that option and open an issue for prettier.

I opened an issue here: https://github.com/prettier/prettier/issues/3308

ALE will now use the --stdin-filepath argument for prettier when the version is 1.6.0 or greater. 1.6.0 has a bug where the configuration file isn't read, and 1.6.1 fixes that. No version respects the .prettierignore file at the moment with --stdin-filepath or the working directory set to the file's directory. That bug will have to be fixed in prettier. I made the command switch to the file's directory in case some part of prettier uses the working directory for something.

I removed the --parser argument. That should be determined automatically by the filename and the configuration file.

Looks like the Prettier bug is being fixed: https://github.com/prettier/prettier/pull/3309
@w0rp thank you for your great work!

Now --stdin-filepath will be automatically used for prettier-eslint when prettier-eslint-cli is 4.4.0 or newer. I removed the old legacy option now support for version detection is in. Now --eslint-config-path will only be added for versions 4.2.0 and up. I imagine at some point the configuration file detection will start working like it does for eslint in its latest version. At which point we can remove --eslint-config-path and use only --stdin-filepath for prettier-eslint.

@w0rp thanks!

The issue in prettier has been resolved. Update to the next version when it's out, and this should work.


Issue43511: Update System Default Domain tooltip href

Rationale
Previously, the System Default Domain tooltip pointed to the configAdmin page. Now, it correctly points to authenticationModule#dom.
Related Pull Requests

https://github.com/LabKey/platform/pull/2327

Changes

Update href

I'm guessing we don't have a TypeScript helper function to generate help links? The equivalent of org.labkey.api.util.HelpTopic on the Java side? It's nothing fancy, but it helps us avoid hardcoding the URL prefix in lots of different places

There is the method helpLinkNode in ui-components if that helps.


Update environment.yml with new dependencies.

v0.1.13 will include a new dependency, TinyDB, which will be the database system for BioProv.
Must update environment.yml with the following packages:

TinyDB
nb_conda_kernels

TinyDB wouldn't be necessary since it's already in the install_requires of the package, so it would get downloaded anyway when developing.
I'll submit a simple PR soon to add nb_conda_kernels.

Closed by #21


Deploy a VampIR circuit to Ethereum

We'd like to have an entry point for ZK developers to get familiar with VampIR. Having a way to easily deploy a VampIR circuit to Ethereum would be a great way to show how VampIR can be used.
In order to do this, VampIR must compile to some backend ZK proof system library which can create a proof, and there must be a verifier matching this proof system as a smart contract on Ethereum.

VampIR currently compiles to zk-garage/plonk and Halo2, so the easiest path to EVM would be to find a verifier contract written in Solidity for Halo2.
Scroll appears to have one here: verifier.sol I'm not sure if there are any others.

Alternatively, we could write a synthesis to a new backend that is known to have a solidity verifier. Snarkyjs/Circom or Bellman would be good choices here. Snarkyjs can supposedly generate a solidity Verifier for both Plonk and Groth16. We might be able to save some time writing the synthesis (and get better efficiency) by mapping VampIR aliases directly to gadgets in those libraries.

ZKSync has a Plonk verifier in Solidity: https://github.com/matter-labs/solidity_plonk_verifier/blob/master/Verifier.sol


Add airline icao to subfleet selection in flights

Just add the airline wich belongs the subfleets in the selectable menu, and add the airline logo or icao in the table.
i managed to do it editing the file FlightController.php and the subfleets.blade.php, but my coding skills are rubbish so could fail
 $retval[$ac->id] = '['.$ac->airline->icao.'] '. $ac->type.' - '.$ac->name;   instead of
 $retval[$ac->id] = $ac->type.' - '.$ac->name;

Having images at admin... I am not sure that I liked the idea, I mean it is only visible to us (va admins) and we already know how our logo looks like.
On the other hand, displaying airline ICAO Code or Name is indeed useful (for both the table and the dropdown). Even only airline icao would be more than enough for admins to identify subfleets easily. (If they have more than one airline and their subfleet names are not easy to separate from each other)
Maybe we can implement this as a pr.

the image is just for looks..... used as an icon is much faster to identify as text, but we could have both AIRLINE ICAO --- LOGO.
As i stated i do not dare to pr, but as you told me dispo that the boss will have a look prior to anything, so could try to do it, my side is working ok but i`m not that tidy at coding as you or nabeel.

I am not the boss :) I can make your a little bit nicer maybe but final
decision is not mine. Nabeel would have the final word on changes.
On 1 Apr 2021 Thu at 20:50 Maximiliano Fallico @.***>
wrote:

the image is just for looks..... used as an icon is much faster to
identify as text, but we could have both AIRLINE ICAO --- LOGO.
As i stated i do not dare to pr, but as you told me dispo that the boss
will have a look prior to anything, so could try to do it, my side is
working ok but i`m not that tidy at coding as you or nabeel.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/nabeelio/phpvms/issues/1106#issuecomment-812069938,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARXKVMMBSXBO53BUK2T6FN3TGSW5DANCNFSM4Z5VOOVA
.

--
B. Fatih KOZ
Sent from mobile device / Mobil cihaz ile gonderildi.

yep was refering to nabeel, the styling at admin that not worries me much..... the problem is touching that stuff in the controller......

I'm ok with images there. Throw a PR in, I can clean it up if needed.

Was this PR'd? I can't remember

yep nabeelio can be closed


fix concurrency

Hi, I change the check from database to base executor to ensure each task_instance will get the consistent view of running instances in memory. Coding style may be not very good. But I will appreciate if you can review the logic, thanks a lot!

Sorry this doesn't work for us as the DB is the source of truth, the executor's state can get out of sync and ultimately we trust what is in the DB...

@mistercrunch
Trust DB as the source of truth is the root of cause in such case.
As I mentioned in https://github.com/airbnb/airflow/issues/1085
If the concurrency works by checking how many running task instances in database, which means it's possible all tasks can start running at the same time because when they query the database, they all get a count of 0.
And I'm wondering how the executor's state can get out of sync? By multiple schedulers?.
Maybe a better solution is using curator/zookeeper's semaphore to solve this problem(each of the task get a resource from the semaphore).
In a word, by simply counting the number of running tasks can lead to concurrency issues when multiple task_instance is checking the database at the same time. We need to solve it using other mechanism

Right, it's a tricky situation as we want to support multiple schedulers in the future, and we certainly don't want to make zookeeper a hard requirement. Having support for multiple executors (and even more in the future), that only leaves the DB as the source of truth, and means we'd have to use it as a message queue almost, which isn't great.

@mistercrunch Just curious how you are going to solve this issue using DB though? thanks

Yeah I think that's the idea, it may take a little while though. I'm going to write a design proposal and submit it out for review soon.

@lucafuji I am not sure we have a clear design for this on our end, but if you still want to be involved feel free to reply. If we do not hear from you, we might close this PR for now as part of a cleanup effort for long standing PRs.

@artwr Feel free to abandon this one, this PR also has race condition issue though.
Looking forward to your new design


On the Metrics page the cards for the user-tag-based metrics get focus when using tab navigation

make sure you have at least 1 user-tag-based metrics configured;
navigate to /administrator/projects/<project-id>/metrics
tab through the page, notice that focus is given to the entire metrics card

:+1:


Fix Seed bootstrap

How to categorize this PR?

/kind bug
/kind regression
/priority normal
What this PR does / why we need it:
Fix Seed bootstrap which is failing because of the APIServerSNI feature gate.
Which issue(s) this PR fixes:
Fixes #2492
Special notes for your reviewer:
Release note:

NONE

/assign @mvladev

We were discussing to enhance the chartApplier with configurable set of errors to ignore on deletion. Something similar would be a more elegant solution as currently all resources need to be enumerated and explicitly deleted.
/hold

I'll suggest to close this in favor of https://github.com/gardener/gardener/pull/2496

@ialidzhikov do you agree?

/close


Windows version currently crashes

I've yet to find where the issue may lie but on Windows 10 the tutorial seems unwilling to run anymore. I do not know if this is mostly related to recently upgrading to Windows 10 and if it still runs on older version of the OS or if there is some other cause.

Resolved, turned out to be mainly a driver issue after upgrading to Windows 10. Finally managed to manually install the latest drivers and all good.


Plan of supernode's golang SDK refactoring

Golang SDK
Dragonfly's supernode needs a Golang SDK which is stored in https://github.com/dragonflyoss/Dragonfly/tree/master/client. Then we achieve the following goals:

every caller of supernode can take advantages of this SDK by integrating it
golang sdk encapsulates of the details of HTTP communication
it could provide scalability and portability for other system to integrate supernode.

In project Dragonfly, it will be used in component dfget and the dashboard. The refactoring plan of golang sdk is described in.
It is one part of Plan of supernode's refactoring plan.
Details
System Related Endpoints
GET /_ping: https://github.com/dragonflyoss/Dragonfly/blob/master/apis/swagger.yml#L34-L46

[x] Pull Request: https://github.com/dragonflyoss/Dragonfly/pull/373
[x] Mock Test: https://github.com/dragonflyoss/Dragonfly/pull/373

Preheats Related Endpoint
POST /preheats: https://github.com/dragonflyoss/Dragonfly/blob/master/apis/swagger.yml#L336-L359

[x] Pull Request: https://github.com/dragonflyoss/Dragonfly/pull/210
[ ] Mock Test:

GET /preheats:  https://github.com/dragonflyoss/Dragonfly/blob/master/apis/swagger.yml#L361-L379

[x] Pull Request: https://github.com/dragonflyoss/Dragonfly/pull/210
[ ] Mock Test:

GET /preheats/{id}:  https://github.com/dragonflyoss/Dragonfly/blob/master/apis/swagger.yml#L382-L404

[x] Pull Request: https://github.com/dragonflyoss/Dragonfly/pull/210
[x] Mock Test: https://github.com/dragonflyoss/Dragonfly/pull/365

Peer Related Endpoints
POST /peers: https://github.com/dragonflyoss/Dragonfly/blob/master/apis/swagger.yml#L49-L68

[x] Pull Request: https://github.com/dragonflyoss/Dragonfly/pull/384
[ ] Mock Test:

GET /peers: https://github.com/dragonflyoss/Dragonfly/blob/master/apis/swagger.yml#L70-L107

[x] Pull Request: https://github.com/dragonflyoss/Dragonfly/pull/384
[ ] Mock Test:

GET /peers/{id}:  https://github.com/dragonflyoss/Dragonfly/blob/master/apis/swagger.yml#L109-L129

[x] Pull Request: https://github.com/dragonflyoss/Dragonfly/pull/384
[ ] Mock Test:

DELETE /peers/{id}: https://github.com/dragonflyoss/Dragonfly/blob/master/apis/swagger.yml#L131-L151

[x] Pull Request: https://github.com/dragonflyoss/Dragonfly/pull/384
[ ] Mock Test:

Task Related Endpoints
POST /tasks:  https://github.com/dragonflyoss/Dragonfly/blob/master/apis/swagger.yml#L154-L174

[ ] Pull Request:
[ ] Mock Test:

GET /tasks/{id}:  https://github.com/dragonflyoss/Dragonfly/blob/master/apis/swagger.yml#L177-L197

[ ] Pull Request:
[ ] Mock Test:

PUT /peers/{id}: https://github.com/dragonflyoss/Dragonfly/blob/master/apis/swagger.yml#L200-L228

[ ] Pull Request:
[ ] Mock Test:

PUT /peers/{id}: https://github.com/dragonflyoss/Dragonfly/blob/master/apis/swagger.yml#L230-L251

[ ] Pull Request:
[ ] Mock Test:

GET /tasks/{id}/pieces: https://github.com/dragonflyoss/Dragonfly/blob/master/apis/swagger.yml#L253-L294

[ ] Pull Request:
[ ] Mock Test:

@allencloud it mean task?

Yes，I think you are right. @zgfh


[SVG] max-width should have units

Line 2175 should have + 'px' at the end, since Fixed() does not include units itself.
Originally from mathjax/mathjax-node#399.

==> Merged.

==> In testsuite
MathMLToDisplay/issue1950.html


New api function: Rename a container

The latest API allows containers to be renamed.
http://docs.docker.com/reference/api/docker_remote_api_v1.17/#rename-a-container
Example usage:
$container = $manager->find('mycontainer');
$ret = $manager->rename($container, 'mynewcontainer');

:+1:

LGTM, just missing unit test and documentation :)

Test added and runs fine. However, it leaves a container lying around on the system and does not cleanup. (So added second commit to delete the test container)
In fact it looks like running the tests leave many containers around, why is there no cleanup after tests??

About tests not cleaning up: that can be considered a bug if anyone wants to work on it :)

Bump, why has this not been merged, could it be done soon please?
It works and the tests are clean?

Sorry about that, merged now thanks!


Anim Jitter Hold Note

What is your question?
I've seen a lot of character that dosen't matter the bpm or anything they jitter when holding a note but, is a little bit crazy, but I like that the characters jitter when holding a note. So can you say to me if is there anything to do for that the animations jitter no matter the bpm or anything, pls???

That how it supposed to work
and no one’s argues about it

That how it supposed to work and no one’s argues about it

So I cannot do anything like in adobe animate to make them jitter???

for the char editor add the anim-hold for each pose to 1 frame

for the char editor add the anim-hold for each pose to 1 frame

How???
I just put in the advanced (animations indices) the number 0 that will be the first frame???

for the char editor add the anim-hold for each pose to 1 frame

So I just put the 0 in the animation indices (advanced) that wil be the first frame???

yes

yes

Ok, let me try it

yes

It didn't work

yes

Nvm I fixed it. I just need to remove one frame in adobe animate. Thank you anyway for the help


trailingSlash option to never is not in line with $page.params

Describe the bug
When settings the following config in svelte.config.js, I expect all paths to contain no trailing slashes.
const config = {
  kit: {
    trailingSlash: "never",
  }
};

But when looking at the  $page.params store, this is not always the case.
This happens when navigating on the website with links and looking at rest parameters.
Reproduction
Reproduction repo: https://github.com/oganexon/sveltejs-kit-trailing-slash-page-store
Stackblitz: https://stackblitz.com/edit/sveltejs-kit-template-default-vlxa2g

Go to /test/foo/bar/

See that $page.params.path == "/test/foo/bar"

Click on "Go up" (href=".")

$page.params.path == "/test/foo/" There's a trailing slash

Note: The URL doesn't update in the search bar in stackblitz. That's not part of the issue.
Logs
No response
System Info
__Local:__

  System:
    OS: Windows 10 10.0.19044
    CPU: (12) x64 Intel(R) Core(TM) i7-10750H CPU @ 2.60GHz
    Memory: 4.36 GB / 15.75 GB
  Binaries:
    Node: 16.14.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.17 - ~\AppData\Roaming\npm\yarn.CMD
    npm: 8.3.1 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: Spartan (44.19041.1266.0), Chromium (104.0.1293.47)
    Internet Explorer: 11.0.19041.1566
  npmPackages:
    @sveltejs/adapter-node: ^1.0.0-next.85 => 1.0.0-next.85
    @sveltejs/kit: ^1.0.0-next.405 => 1.0.0-next.405
    svelte: ^3.49.0 => 3.49.0
    vite: ^3.0.3 => 3.0.3

__StackBlitz:__

  System:
    OS: Linux 5.0 undefined
    CPU: (4) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 0 Bytes / 0 Bytes
    Shell: 1.0 - /bin/jsh
  Binaries:
    Node: 16.14.2 - /usr/local/bin/node
    Yarn: 1.22.10 - /bin/yarn
    npm: 7.17.0 - /bin/npm
  npmPackages:
    @sveltejs/adapter-auto: next => 1.0.0-next.64
    @sveltejs/kit: next => 1.0.0-next.405
    svelte: ^3.46.0 => 3.49.0
    vite: ^3.0.0 => 3.0.5

Severity
serious, but I can work around it
Additional Information
No response

This is super tricky - we're sort of tied here. The trailingSlash option is determined by the route that is loaded. beforeNavigate and $navigating contain params which are updated before that route's nodes are loaded, therefore before the trailingsSlash value is determined. We could modify params afterwards so that it's correct for $page.params and for the load functions, but it would be different to the value in beforeNavigate and $navigating, which sounds like a bad solution. I'm tempted to mark this as "won't fix" since it's such an edge case:

you need a <a href=".">
you need a [..rest] route
you need someone to navigate upwards within that rest route

Btw the URL in the browser bar is wrong, too, since . apparently triggers this behavior. The best way to avoid this is therefore to not use href="."


JSON Variation are notified twice

Description
JSONVariation are notified twice that result in an error in the data export.
Changes include

[x] Bugfix (non-breaking change that solves an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking changes (change that is not backward-compatible and/or changes current functionality)

Checklist

[x] I have tested this code
[ ] I have added unit test to cover this code
[ ] I have updated the documentation (README.md and /docs)
[x] I have followed the contributing guide

Coverage decreased (-0.003%) to 96.535% when pulling 8e1a6fb29cb032e6a48cd8daa975e6c9934bce05 on fix/notify-variation-twice into 858f573949244a3db54896a528a137bc4c6a9c8c on main.


docs(jwt) key_claim_name payload vs. header logic

Summary
Update key_claim_name documentation to reference first checking the payload claims
and then checking the header claims
Relevant Links
From Kong PR #3313

@brycehemme Thank you for the note! We'll schedule this for the next release!

This note was added by https://github.com/Kong/getkong.org/commit/be618513430d94b25ba0d4b6228175c4b571d7f0, sorry that it superseded this PR!


Add pipelined correlated reweighting

The reweighting method described in this paper seems can be integrated with stim quite well. Would you consider adding the reweighting option to stim in the future?

I don't think this makes sense as a feature of Stim. It's certainly possible to implement code that takes a DEM and the detection events from a shot and outputs a reweighted DEM, but it wouldn't be fast enough.
In order for a method like that to be fast, it has to be applied directly to the internal representation of the decoder. If the reweighting was implemented in Stim, then that would require the decoder to convert the DEM (into whatever the decoder's internal representation of the match graph is) once per shot. That is really expensive! Too expensive to put on the hot path of batch sampling or streaming.


update .gitignore

I noticed an inconsistent line gap
Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #XXXXX

I'm happy to contribute to this project


Update office-365-management-activity-api-schema.md

Updates to the ATP and Threat Intelligence Schema based on the enhancements announced at Ignite - i.e. including URL time of click events for ATP Safe Links and malware detections for files in SharePoint, OneDrive and Teams.

OPS Build status updates of commit 299bf05:
:clock10: Preparing: average preparing time is 49 sec(s)

OPS Build status updates of commit 299bf05:
:clock10: Incremental building: average incremental building time is 19 sec(s)

OPS Build status updates of commit 299bf05:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

office-365-management-api/office-365-management-activity-api-schema.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.

OPS Build status updates of commit d992e77:
:clock10: Incremental building: average incremental building time is 19 sec(s)

OPS Build status updates of commit d992e77:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

office-365-management-api/office-365-management-activity-api-schema.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.


SlideMenu: Slide to open bug

If you use gesture to open the menu, clicks on menuItems renders the page with the menu still open.

Hi @rafazundt
Sorry to hear.
Unfortunately I've tried cloning and running on 2 separate computers and can't replicate this. I have since just released v1 of this project, which includes Redux (and changes SideMenu to work with Redux), so I'd suggest giving it a try now.


Minimize refs fetched during 'git clone' for users with unstable connections

Perform a shallow clone of the monero repo & all submodules (only gets the refs required to build). Speeds up the clone & makes it more reliable for users with unstable connections (tor, vpn, or just a low-quality ISP).

Thank you for the PR, looks great and had always wanted to circle back to the git step and now I don't have to :)


Decompiler should further simplify not(equals(a, b))

At the moment, the decompiler will convert not(equals(a, b)) in an ARM expression to: (!(a == b)). We should instead simplify further to (a != b)

@anthony-c-martin  Happy to pick this up

Btw, something similar could also be done for =~ and !~ operators in Bicep. These also compile into a pair of functions: https://github.com/Azure/bicep/blob/main/docs/spec/expressions.md


Add tests for Gitlab

When test coverage in #50 is added we will see the need for those tests.

Blocked by #132 (Formulate a testing strategy)

PR #146 includes some tests for gitlab

PR #146 was merged, closing


Set the document clone iframe container's size via the style attribute

Summary
This PR fixes/implements the following bugs/features

[x] Bug: CSS rules can override the iframe's size which compromises the screenshot

Some sites I used html2canvas on have a CSS rule like
body>iframe {
    position: absolute;
    width: 0;
    height: 0;
    overflow: hidden;
}

This CSS rule takes precedence over the the width and height attributes of the html2canvas iframe container. As a result, the document clone is rendered with a width and height of 0, so I get a blank, all-white screenshot.
This PR also sets the width and height of the iframe within the style attribute. The style attribute will take precedence over any CSS rules.
Test plan (required)
When I tried to run npm test locally I got this error:
Traceback (most recent call last):
  File "/Users/joey/.nvm/versions/node/v12.13.1/lib/node_modules/npm/node_modules/node-gyp/gyp/pylib/gyp/common.py", line 499, in <module>
    class OrderedSet(collections.MutableSet):
AttributeError: module 'collections' has no attribute 'MutableSet'

I'm on macos Ventura 13.4.1 using npm 6.12.1 and node v12.13.1

For anyone else who might have come across this issue - a good workaround is to use the onclone callback:
      const canvas = await html2canvas(document.body, {
        x: window.scrollX,
        y: window.scrollY,
        width: document.documentElement.clientWidth,
        height: document.documentElement.clientHeight,
        onclone: (document, _element) => {
...
          // This is a hack to fix a bug with html2canvas. When html2canvas is
          // rendering the element to screenshot, it actually adds the element
          // to the DOM inside an invisible iframe. The bug is that the
          // html2canvas iframe container uses the width and height attributes
          // instead of the style attribute to set the iframe's size.  Some
          // sites have a CSS rule that sets iframes' width and height to zero,
          // which takes precedence over the width and height attributes. To fix
          // this bug, we just copy the html2canvas iframe container's width and
          // height attributes into the style attribute.
          const container = document.querySelector('iframe.html2canvas-container');
          container.style.width = container.width + 'px';
          container.style.height = container.height + 'px';
...
        },
...
      });

Can be tested in @cantoo/html2canvas


chatterbox-qt background colors for different users

Apparently not supported with textarea, which we want for copy-paste:
https://phanective.wordpress.com/2011/08/20/text-bgcolor-qml/. Could we pick some other form of emphasis? Name color as in Pidgin?


1.16.2 Wrong class loader used to load "lombok.javac.apt.Processor"

Migrated from Google Code (issue 791)

I have experienced the same problem with version 1.16.4

You found a bug in lombok; lombok.javac.apt.Processor is not available. Lombok will not run during this compilation: java.lang.ClassNotFoundException: lombok.javac.apt.Processor

Some more details

Apache Maven 3.0.4 (r1232337; 2012-01-17 09:44:56+0100)
Maven home: /home/tprzybyl/softwares/maven
Java version: 1.7.0_79, vendor: Oracle Corporation
Java home: /usr/lib/jvm/java-7-openjdk-amd64/jre
Default locale: en_US, platform encoding: UTF-8
OS name: "linux", version: "3.16.0-44-generic", arch: "amd64", family: "unix"

But the funny thing is that it appears only in some projects, while others work just fine. I am reverting version to 1.14.8. Good luck hunting this bug down!

Hello
same probleme
mvn -V clean package -e
Apache Maven 3.3.3<PHONE_NUMBER>791599e205a5524ec3e0dfe41d4a06; 2015-04-22T13:57:37+02:00)
Maven home: /Users/clalleme/opt/apache-maven-3.3.3
Java version: 1.8.0_51, vendor: Oracle Corporation
Java home: /Library/Java/JavaVirtualMachines/jdk1.8.0_51.jdk/Contents/Home/jre
Default locale: fr_FR, platform encoding: UTF-8
OS name: "mac os x", version: "10.10.5", arch: "x86_64", family: "mac"

We've added some debug information in the latest edge release. Can you give this one a try?

I tried but sounds the same
Where are the debug information: stdout ? or a file ?
[WARNING] /Users/clalleme/IdeaProjects/demog/src/main/java/org/icroco/haura/knxip/KnxHeader.java:[5,26] [deprecation] Builder in lombok.experimental has been deprecated
[WARNING] /Users/clalleme/IdeaProjects/demog/src/main/java/org/icroco/haura/knxip/KnxHpai.java:[5,26] [deprecation] Builder in lombok.experimental has been deprecated
[WARNING] /Users/clalleme/IdeaProjects/demog/src/main/java/org/icroco/haura/knxip/EKnxCode.java:[11,7] You found a bug in lombok; lombok.javac.apt.LombokProcessor is not available. Lombok will not run during this compilation: java.lang.ClassNotFoundException: lombok.javac.apt.LombokProcessor
[WARNING] 356)
/Users/clalleme/IdeaProjects/demog/src/main/java/org/icroco/haura/knxip/KnxHeader.java:[5,26] [deprecation] Builder in lombok.experimental has been deprecated
[WARNING] /Users/clalleme/IdeaProjects/demog/src/main/java/org/icroco/haura/knxip/KnxHpai.java:[5,26] [deprecation] Builder in lombok.experimental has been deprecated
[INFO] 5 warnings

By chance I experienced the same error. NetBeans created my "Maven -> Java Application" project with a pom.xml that used the maven-compiler-plugin version 2.3.2. After updating that version number to the current 3.5.1 it compiled fine. I use JDK 1.8.0_72 with Maven 3.3.9 and NetBeans 8.1.

For us this problem has occurred all versions of Lombok since 1.14.8 when used with maven-compiler-plugin version 2.3.2. Updating to 3.5.1 according to @lathspell s suggestion resolved it.

Several reports suggest that using the 3.5.1 maven-compiler-plugin fixes this problem.


fix some typos in tests

Merry Christmas to you as well!
I assume this should not be marked as a draft?


Aggregate collection as a Target for SimpleUpdate

This pull request adds an ability to pass Aggregate collection as one of the Targets for SimpleUpdate action -
this action will be invoked for each eligible element of the given Aggregate.
Example:
POST  https://ip:port/redfish/v1/UpdateService/Actions/UpdateService.SimpleUpdate
{
  "ImageURI":"{image_uri}",
  "Targets": [
		"/redfish/v1/AggregationService/Aggregates/{aggregate_uri}"
	]
}

This PR should be merged after #61 and #62

Accepted on OCP-DM community meeting 27.06.2023


Filter authorizations by status of authorization

Note: MVP task moved over from previous requirements document.
Description:
Filter existing authorizations by status of authorization (default to active vs. inactive, all) on text search page
Definition of Ready:

[x] User role and type are identified
[x] Wireframes completed
[x] Design / Solution is accepted by Product Owner

Done in #43


site broken with pretty urls

In 8.0.3 and RC2,
I  had pretty urls on,
then screen turned white,
the site never accept any operation.
I had no other way except deleting my site.
This does not happen in <IP_ADDRESS>.
I guess something had been changed in ver.8.
I am now afraid turning pretty urls on,
but love to use it in the near future.
I am working with apache 2.4.6.

@Kitayamasugi
I was unable to reproduce this issue using 8.0.3.
With Pretty URL issues, I believe you can delete your .htaccess file and then edit application\config\generated_overrides\concrete.php, removing 'url_rewriting' => true.

Please enable debug mode, enable pretty urls, and please provide any errors that appear on screen.
If no errors appear on screen, please supply the PHP error log from the server.
Also please provide the content of the environment page.

I can verify I had a similar problem in 8.1.
I enabled pretty URLs, but forgot to enable mod_rewrite. I received no warning, Concrete5 throught all was well, but of course it did not work. I don't know if this is the same issue though, so I will open it as a new one since I have specific details


Add Researcher Classes API endpoint

https://www.pivotaltracker.com/story/show/187185134
This PR adds a controller that provides API for a Researcher Classes form components. It's based on: https://github.com/concord-consortium/rigse/blob/master/rails/app/controllers/api/v1/report_users_controller.rb
Of course, there's quite a few of updates. Classes query is new. Everything is first filtered by project_id. Filtering assumes the following relationship between models: Project -> Cohort -> Teacher -> Class -> Resource. This was not necessarily the case in the User Report API. Queries that filter out cohorts based on runnables, and runnables based on cohorts, are rewritten because of that. There is a few more changes related to that.
Also, I had to modify cohort policies a bit, so the scope allows researchers to list cohorts. They still won't be able to edit them.
This is still missing specs and hopefully bigger refactoring that we discussed today. But it let me implement working UI and probably it could work just fine if refactoring becomes too tedious. @scytacki, so your idea would be to leave only classes_query and derive runnables, cohorts, and teachers from result of this query, right?

Yes, right about the classes query. I would try to have the query return an active record query not an actual result. Then I think you can join that with additional conditions to get each list.
If that doesn't work I'd look at sub query support. Then you can join with the sub query.
Either approach should prevent requiring two round trips to the DB.


can't hide/show n-form-item-gi with v-show

TuSimple/naive-ui version (版本)
2.34.3
Vue version (Vue 版本)
3.2.47
Browser and its version (浏览器及其版本)
Chrome Dev 113.0.5668.0
System and its version (系统及其版本)
Windows 11 Pro 22H2
Node version (Node 版本)
19.6.0
Reappearance link (重现链接)
https://github.com/FangDingli/naive-ui-grid-bugreport
Reappearance steps (重现步骤)
In initialize, this component is not show with v-show="false" and it will show with v-show="true".
When you want to hide it again, it's got error in console
Expected results (期望的结果)
n-form-item-gi can work fine with v-show
Actual results (实际的结果)
got error in console
Remarks (补充说明)

Same question. The utilization of the :span=0 directive, similar to the conditional statementv-if, hinders the usage of the keep-alive feature for component persistence. While v-show allows the component to remain without triggering a re-render, it cannot be used simultaneously with :span=x.

By employing the "layout-shift-disabled=true" attribute, it is possible to use "v-show" without issues. However, it should be noted that this approach may compromise the responsiveness of the layout, although no responsiveness-related issues have been encountered thus far.

#4422

This issue does not have any recent activity. If you are still experiencing similar problems, open a new error, including a minimal copy of the problem


Conditional Parameters

This PR introduces conditional parameters.
What's the problem?
In some cases, we want parameters to take different values depending on the value of other parameters.
One example is the WIMP-mass-dependent signal efficiency uncertainty. Here, we typically have larger uncertainties for lower masses. With conditional parameters, we can now simply add the following to our parameter definition in the config file:
wimp_mass:
    nominal_value: 50
    ...
signal_efficiency:
    conditioning_parameter_name: wimp_mass
    uncertainty:
      5: 0.3
      10: 0.2
      50: 0.15
      100: 0.1
      500: 0.1
    ...

Depending on the value of wimp_mass this will result in a different value of the signal_efficiency uncertainty.
The ConditionalParameter class
Here's an example how the ConditionalParameter class works:
from alea import load_yaml, Parameters

config_path = "%SOMEPATH%/alea/alea/examples/configs/unbinned_wimp_statistical_model_mass_dependent_efficiency.yaml"
config = load_yaml(config_path)

params = Parameters.from_config(config["parameter_definition"])

#  If you directly access a property of the conditional parameter, it will just return the
#  Parameter under nominal conditions (i.e. using the uncertainty for wimp_mass = 50)
print(params.signal_efficiency.uncertainty)
# >>> 0.15

#  The same happens if you call the conditional parameter without any arguments:
print(params.signal_efficiency().uncertainty)
# >>> 0.15

#  If you provide another value though, it will return the parameter for another wimp_mass:
print(params.signal_efficiency(wimp_mass=5).uncertainty)
# >>> 0.3

#  Another way to get there is to change the nominal value of wimp_mass:
params.wimp_mass.nominal_value = 5
print(params.signal_efficiency.uncertainty)
# >>> 0.3

Caveat
So far, this doesn't work "on-the-fly" i.e. if I have a model instance and just change the wimp_mass value it won't correctly update the treatment of the signal efficiency since the likelihood is already initialized with the constraint terms. However, the structure introduced in this PR should allow introducing this feature in the future.
What I tested so far
I tested this in a notebook for a few different WIMP masses (this won't work with the config in alea since we don't have any other WIMP mass templates yet, but you can ask me for an example notebook using actual templates). So far, everything looked good and behaved as expected but heavier testing with large toyMCs will follow.
Please have a look, play around with the new class, and let me know if something unexpected happens. Also, if you have any suggestions on how to implement this better, please let me know! In the meantime, I'll write some unittests and do some first checks with toyMCs.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Details

41 of 88 (46.59%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-2.3%) to 89.905%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

alea/parameters.py
41
88
46.59%

Totals

Change from base Build<PHONE_NUMBER>5:
-2.3%

Covered Lines:
1603

Relevant Lines:
1783

💛  - Coveralls

Hi thanks @hammannr. How urgent is this one? Is it OK if I come back to it this Friday?


S3 Action: Fix retrieving ipa info containing embedded app bundle

Fixed retrieving .ipa info that contains embedded app bundle like WatchOS application.

Thanks for the pull request, I added a comment :+1:

Thanks for the pull request, I added a comment :+1:

@KrauseFx Thanks for feedbacks, I've sent PR on fastlane_core: https://github.com/fastlane/fastlane_core/pull/66
I'll rewrite this PR using FastlaneCore::IpaFileAnalyser.fetch_info_plist_file[...] once new fastlane_core version is released.

Great, thanks @ngs. Do you want to update the s3 action to use the class from fastlane_core?

@KrauseFx Updated the S3 action using FastlaneCore::IpaFileAnalyser.fetch_info_plist_file.
I'll revert Gemfile and update fastlane.gemspec after new version of fastlane_core becomes available.

@ngs New fastlane_core is available :+1:

@KrauseFx Updated, thanks!

Cool, thanks for squasing the commit. Did you test everything locally?

Yes, I'm referencing this branch from my working project: https://github.com/ngs/ci2go/blob/master/Gemfile#L11
https://circleci.com/gh/ngs/ci2go/tree/master

Perfect, thank you so much @ngs :rocket:


Quest-boardgame Initial submit

Changes / Comments
Initial submit, new sheet.
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[ ] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[X ] If this is a new sheet, did you follow Building Character Sheets standards ? Yes.

If you do not know English. Please leave a comment in your native language.

Not sure why there is a failed check. Is it possible to confirm this PR is OK and get a sense for when it might be merged? This sheet at the request of someone planning to use it for GaryCon.


THORN-2229 AbstractMethodError when building using Maven 3.6.0

Motivation
Thorntail projects fail to build with Maven 3.6.0.
Modifications
Dependecies on org.eclipse.aether:aether-:1.1.0 were replaced by org.apache.maven.resolver:maven-resolver-:1.3.1. Class RepositorySystemSessionWrapper implements missing method getFileTransformerManager().
Result
Build works with Maven 3.6.0, 3.5.2 and 3.3.9.

[x] Have you followed the guidelines in our Contributing document?
[x] [v2] Have you created a JIRA and used it in the commit message?
[ ] [v4] Have you created a GitHub Issue and used it in the commit message?
[x] Have you checked to ensure there aren't other open Pull Requests for the same issue?
[x] Have you built the project locally prior to submission with mvn clean install?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

I just rebased the branch, because it seems we now have a "PR must be up to date with master" check enabled :-)

I just merged this, so I'm pretty sure it will make it to 2.3.0. I'm not aware of any plans to do 2.2.2.


Worse  performance  with multiple GPU  training with Keras

The  network efficient net B3 used for Image Classification get worse performance  with multiple GPU  training.   The single GPU training get valid dataset accuracy at 0.94.  After multiple training,  the valid dataset accuracy fluctuates  between  0.49 to 0.53 .And  the loss value and accuracy of training dataset  are hard to change with bad metrics value. Just like the model  is hard to learn from training data and gives a  classfication result randomly . But the GPU are  still running for the network.  May the optimizer run the problem?
`
450/650 [===================>..........] - ETA: 10s - loss: 8.1307 - acc: 0.4956 - categorical_crossentropy: 8.1307
456/650 [====================>.........] - ETA: 10s - loss: 8.0944 - acc: 0.4978 - categorical_crossentropy: 8.0944
462/650 [====================>.........] - ETA: 9s - loss: 8.0590 - acc: 0.5000 - categorical_crossentropy: 8.0590
468/650 [====================>.........] - ETA: 9s - loss: 8.0590 - acc: 0.5000 - categorical_crossentropy: 8.0590
474/650 [====================>.........] - ETA: 9s - loss: 8.0931 - acc: 0.4979 - categorical_crossentropy: 8.0931
480/650 [=====================>........] - ETA: 8s - loss: 8.0926 - acc: 0.4979 - categorical_crossentropy: 8.0926
486/650 [=====================>........] - ETA: 8s - loss: 8.0922 - acc: 0.4979 - categorical_crossentropy: 8.0922
492/650 [=====================>........] - ETA: 8s - loss: 8.1246 - acc: 0.4959 - categorical_crossentropy: 8.1246
498/650 [=====================>........] - ETA: 7s - loss: 8.0590 - acc: 0.5000 - categorical_crossentropy: 8.0590
504/650 [======================>.......] - ETA: 7s - loss: 8.0910 - acc: 0.4980 - categorical_crossentropy: 8.0910
510/650 [======================>.......] - ETA: 7s - loss: 8.0274 - acc: 0.5020 - categorical_crossentropy: 8.0274
516/650 [======================>.......] - ETA: 6s - loss: 7.9966 - acc: 0.5039 - categorical_crossentropy: 7.9966
522/650 [=======================>......] - ETA: 6s - loss: 8.0590 - acc: 0.5000 - categorical_crossentropy: 8.0590
528/650 [=======================>......] - ETA: 6s - loss: 8.0590 - acc: 0.5000 - categorical_crossentropy: 8.0590
534/650 [=======================>......] - ETA: 6s - loss: 8.0590 - acc: 0.5000 - categorical_crossentropy: 8.0590
540/650 [=======================>......] - ETA: 5s - loss: 7.9994 - acc: 0.5037 - categorical_crossentropy: 7.9994
546/650 [========================>.....] - ETA: 5s - loss: 8.0295 - acc: 0.5018 - categorical_crossentropy: 8.0295
552/650 [========================>.....] - ETA: 5s - loss: 8.0590 - acc: 0.5000 - categorical_crossentropy: 8.0590
558/650 [========================>.....] - ETA: 4s - loss: 8.0590 - acc: 0.5000 - categorical_crossentropy: 8.0590
564/650 [=========================>....] - ETA: 4s - loss: 8.0305 - acc: 0.5018 - categorical_crossentropy: 8.0305
570/650 [=========================>....] - ETA: 4s - loss: 7.9742 - acc: 0.5053 - categorical_crossentropy: 7.9742
576/650 [=========================>....] - ETA: 3s - loss: 7.9471 - acc: 0.5069 - categorical_crossentropy: 7.9471
582/650 [=========================>....] - ETA: 3s - loss: 7.9483 - acc: 0.5069 - categorical_crossentropy: 7.9483
588/650 [==========================>...] - ETA: 3s - loss: 7.9768 - acc: 0.5051 - categorical_crossentropy: 7.9768
594/650 [==========================>...] - ETA: 2s - loss: 8.0048 - acc: 0.5034 - categorical_crossentropy: 8.0048
600/650 [==========================>...] - ETA: 2s - loss: 8.0053 - acc: 0.5033 - categorical_crossentropy: 8.0053
606/650 [==========================>...] - ETA: 2s - loss: 7.9793 - acc: 0.5050 - categorical_crossentropy: 7.9793
612/650 [===========================>..] - ETA: 1s - loss: 7.9800 - acc: 0.5049 - categorical_crossentropy: 7.9800
618/650 [===========================>..] - ETA: 1s - loss: 7.9808 - acc: 0.5049 - categorical_crossentropy: 7.9808
624/650 [===========================>..] - ETA: 1s - loss: 7.9816 - acc: 0.5048 - categorical_crossentropy: 7.9816
630/650 [============================>.] - ETA: 1s - loss: 8.0335 - acc: 0.5016 - categorical_crossentropy: 8.0335
636/650 [============================>.] - ETA: 0s - loss: 8.0590 - acc: 0.5000 - categorical_crossentropy: 8.0590
642/650 [============================>.] - ETA: 0s - loss: 8.0088 - acc: 0.5031 - categorical_crossentropy: 8.0088
648/650 [============================>.] - ETA: 0s - loss: 8.0342 - acc: 0.5015 - categorical_crossentropy: 8.0342
650/650 [==============================] - 39s 61ms/step - loss: 8.0590 - acc: 0.5000 - categorical_crossentropy: 8.0590 - val_loss: 6.6775 - val_acc: 0.5857 - val_categorical_crossentropy: 6.6775`

This can be due to: the reduction strategy that you specified (https://www.tensorflow.org/api_docs/python/tf/distribute/Strategy#reduce). It's quite tricky to determine how to fuse the results of the GPUs sometimes.
Sharing your code can help us figuring out.

This can be due to: the reduction strategy that you specified (https://www.tensorflow.org/api_docs/python/tf/distribute/Strategy#reduce). It's quite tricky to determine how to fuse the results of the GPUs sometimes.
Sharing your code can help us figuring out.
from keras.utils import multi_gpu_model self.model=multi_gpu_model(self.model,gpus=self.gpu_counts) self.model.compile(loss=categorical_crossentropy,                            optimizer=Adam(lr=self.lr),                            metrics=['acc', categorical_crossentropy])

Which version of tensorflow/keras are you using ?
I cannot find keras.utils.multi_gpu_model in the documentation (https://www.tensorflow.org/api_docs/python/tf/keras/utils).

Keras  version  is  2.3.1 ，with python =3.6.6， tensorflow-gpu=1.14.0   on Ubuntu  18.04

Can you re-run the training with self.model=multi_gpu_model(self.model,gpus=self.gpu_counts, cpu_relocation=True) please. Maybe upgrading the model on CPU can overcome the problem.
https://github.com/tensorflow/docs/blob/r1.14/site/en/api_docs/python/tf/keras/utils/multi_gpu_model.md
cpu_relocation: A boolean value to identify whether to create the model's weights under the scope of the CPU. If the model is not defined under any preceding device scope, you can still rescue it by activating this option.

I have tried the resolution ,but there is  no improvement. The metrics such as  loss and acc  are still  same as the description  firstly

@lyw615 ,
Can you please try to execute the code in latest stable version of tensorflow and keras form here and let us know if you are facing same issue.Thanks!


need change of set_local_description constructor

Describe the Bug
set_local_description requires a &RtcSessionDescriptionInit to be executed
I can't see the method restartIce neither
basically a lot the current js method and properties don't exist or are incomplete
Steps to Reproduce
look at the rustwasm doc and look a the MDN doc
Expected Behavior
The method should also be callable without argument.
Actual Behavior
the description doesn't get set automatically.

set_local_description requires a &RtcSessionDescriptionInit to be executed

I'm not sure what happened there, but the WebIDL was defined incorrectly in wasm-bindgen, the parameter should be optional: https://www.w3.org/TR/webrtc/#ref-for-dom-peerconnection-setlocaldescription-4.
Unfortunately this can only be fixed in the next breaking change release.

I can't see the method restartIce neither

This should be easy to add. Would you like to make a PR?
wasm-bindgen Guide to add new Web APIs.


Improve handling of dispersion transformed variables

Notice that this fix is restricted to dispersion parameters, transformation of other variables is still missing. But this should make simple to deal with those. Additionally, this prevents the transformed dispersion parameter to be added to the idata.

not sure why this is failing, but it seems to be unrelated to the changes. Also all test and pre-commit checks are passing locally.

Seems that tests were only failing because of a PyPI issue, now fixed.


Pre-mapinit lights don't work so you can't actually see anything

Description

Makes it harder to map. Visually, the bulbs are gone and when you try to simulate the lighting it only shows tech glow. Turns out it also edits the file itself, but doesn't have an affect on gameplay after mapinit.
Screenshots

Just to clarify, is there actually some bug with mapping mode & mapinitialization, or is this a feature request that lights should be on while mapping?

Just to clarify, is there actually some bug with mapping mode & mapinitialization, or is this a feature request that lights should be on while mapping?

I assume it is a bug because this has never happened before

NVM I think I misunderstood what you mean. It does seem like theres a bug, I'll work on fixing it tomorrow

space-wizards/RobustToolbox/pull/5063 should fix it, though I'll also add a test that the mapping command works as it should to #27207, which we really should've had a long time ago

It was not fixed

To clarify, yes his PR did fix adding new light entities, but it did not fix already existing entities from before the change was merged. This means that basically the whole map is still having this problem, and considering the number of lights it would take a million years to fix manually. Ideally, it should be automatic.


perlPackages.XMLRPCLite: init at 0.717

Motivation for this change
Add missing cpan package.
Things done

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

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

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

@grahamcofborg build perlPackages.XMLRPCLite

@GrahamcOfBorg build perlPackages.XMLRPCLite

@GrahamcOfBorg build perldevelPackages.XMLRPCLite

See darwin log.

@GrahamcOfBorg build perldevelPackages.XMLRPCLite

@GrahamcOfBorg build perldevelPackages.XMLRPCLite

Thanks team :tada:


Include github-traffic-stats and use it to run GitHub Actions cron job

The code of interest is available here.
A simple bash execution may suffice.

The ncah code generates CSV for each stats. However, the scripts that merge contents in such a way that doesn't help separate the different sources, thus, I recommend modifying their merge_csv.sh script as a starting point.


V2 - Iterating on Documentation to match new changes / interface

Documentation site
https://cds-snc.github.io/forms-api/
User story
As an infrastructure developer, I want the latest documentation know what I need to do to integrate with the GC Forms API so that I can retrieve form submissions programmatically, without reaching out for help.
Context
As part of the self-serve version of the API release, we need open, published, bilingual documentation to help guide developers through the process. This version should be fully bilingual.
What's involved
Update content for full supported bilingual experience

[ ] Confirm up-to-date with Design Doc 17: GC Forms data retrieval API
[ ] Iterate on content based on testing and interface changes - where will the API key be created?
[ ] Entry paths from the application UI identified and linked
[ ] Create high-level guidance page that links to API documentation
[ ] Get content back from translation (sent Oct 9)
[ ] Get content inputted, linked and styled in FR in Markdown
[ ] Published online in French and English

Further iterations

Styling to match GC Forms / get closer to GC Notify Documentation
Submenu structure to make navigation more scannable within API requests
Explore dedicated domain and parallel structure to Notify docs VuePress site

Related https://github.com/cds-snc/forms-api/issues/104

Should be able to get this done pretty quickly upon my return (major change is to the API key creation / deletion process)

https://github.com/cds-snc/forms-api/pull/112

To be merged in with the release

Complete


Heap-error calling FirebaseData::setExternalClient()

Hello,
I just updated your library from version 4.3.10 to 4.3.12.
I am using an external client (TinyGsm) (ESP32; PlatformIO) which worked fine in version 4.3.10.
With the new version 4.3.12 I get the following heap-error when calling fbdoRtdb.setExternalClient(...):
CORRUPT HEAP: Bad head at 0x3ffd1368. Expected 0xabba1234 got 0x8c8fc300
assert failed: multi_heap_free multi_heap_poisoning.c:253 (head != NULL)

Backtrace: 0x40084055:0x3ffd90e0 0x400963d5:0x3ffd9100 0x4009c811:0x3ffd9120 0x4009c427:0x3ffd9250 0x40084519:0x3ffd9270 0x4009c841:0x3ffd9290 0x401da25d:0x3ffd92b0 0x40109a07:0x3ffd92d0 0x40203e62:0x3ffd92f0 0x4010dd65:0x3ffd9310 0x40109e30:0x3ffd9340 0x4013c096:0x3ffd9360 0x400d640a:0x3ffd9380 0x400d4f12:0x3ffd93c0

  #0  0x40084055:0x3ffd90e0 in panic_abort at /Users/ficeto/Desktop/ESP32/ESP32S2/esp-idf-public/components/esp_system/panic.c:408
  #1  0x400963d5:0x3ffd9100 in esp_system_abort at /Users/ficeto/Desktop/ESP32/ESP32S2/esp-idf-public/components/esp_system/esp_system.c:137
  #2  0x4009c811:0x3ffd9120 in __assert_func at /Users/ficeto/Desktop/ESP32/ESP32S2/esp-idf-public/components/newlib/assert.c:85
  #3  0x4009c427:0x3ffd9250 in multi_heap_free at /Users/ficeto/Desktop/ESP32/ESP32S2/esp-idf-public/components/heap/multi_heap_poisoning.c:253
      (inlined by) multi_heap_free at /Users/ficeto/Desktop/ESP32/ESP32S2/esp-idf-public/components/heap/multi_heap_poisoning.c:245
  #4  0x40084519:0x3ffd9270 in heap_caps_free at /Users/ficeto/Desktop/ESP32/ESP32S2/esp-idf-public/components/heap/heap_caps.c:361
  #5  0x4009c841:0x3ffd9290 in free at /Users/ficeto/Desktop/ESP32/ESP32S2/esp-idf-public/components/newlib/heap.c:39
  #6  0x401da25d:0x3ffd92b0 in operator delete(void*) at /builds/idf/crosstool-NG/.build/HOST-x86_64-w64-mingw32/xtensa-esp32-elf/src/gcc/libstdc++-v3/libsupc++/del_op.cc:49
  #7  0x40109a07:0x3ffd92d0 in FB_Custom_TCP_Client::~FB_Custom_TCP_Client() at lib/MsFirebaseESPClient/src/./wcs/custom/FB_Custom_TCP_Client.h:50
  #8  0x40203e62:0x3ffd92f0 in Firebase_Signer::freeClient(FB_Custom_TCP_Client**) at lib/MsFirebaseESPClient/src/signer/Signer.cpp:852 (discriminator 1)
  #9  0x4010dd65:0x3ffd9310 in Firebase_Signer::setTCPClient(FB_Custom_TCP_Client*) at lib/MsFirebaseESPClient/src/signer/Signer.cpp:1605
  #10 0x40109e30:0x3ffd9340 in FirebaseData::setExternalClient(Client*) at lib/MsFirebaseESPClient/src/session/FB_Session.cpp:83

MsFirebaseESPClient is a fork of your library.
The only relevant difference between 4.3.10 and 4.3.12 I found is the elimination of
    if (noClient)
        freeClient(&tcpClient);

in bool Firebase_Signer::reconnect().
With re-inserting these two lines version 4.3.12 works for me with an external client again.
Regards, Martin

Thanks for reporting this bug,
This is because I'm trying to avoid the annoying and unnecessary verbose debug message from ssl_client class "Cleaning the SSL connection" when its reconstructor was called even it does not use. I will fix and inform you soon for the update.

I update the library with bug fixes in v4.3.13.
This version is based on your modification in the fork for header files inclusion/exclusion for reduce program space.
If no filesystems macro defined in FirebaseFS.h, the FS.h and SPI.h will not include.
If external client macro was defined in FirebaseFS.h, the ETH.h will not include in ESP32.
FirebaseJson's readClient function is also updated.


Add i18n support for percent value

The percentage value on the initial Vim buffer did not support i18n before. This particular value is not simple to internationalize, since the positioning is absolute, and this causes problems for languages like Turkish that uses the %100, %10, %1 percentage format, leading to outputs like % 10 and/or %  1. This commit adds a helper to make i18n process accurately.
I don't think this is the most elegant solution however. Any comments to further improve the logic is most welcome.

There are several problems with this solution.  One of which is that memory leaks.
I would think only the printf string "%d%%" needs to be translated.  The result can be right-aligned
in the available space, e.g. by first calling vim_snprintf(), getting the length and then when it's too short
prepend a space and repeat.  Or compute the percentage, and when it smaller than 10 insert a space.
Translation should never change the decimal number value.

I would think only the printf string "%d%%" needs to be translated. The result can be right-aligned in the available space, e.g. by first calling vim_snprintf(), getting the length and then when it's too short prepend a space and repeat. Or compute the percentage, and when it smaller than 10 insert a space. Translation should never change the decimal number value.

Thanks for the tips! Indeed the updated solution turned out to be much more concise. I am still not sure about memory management though, since I haven't used C for anything other than snippets like these. Please let me know if it's not okay!

I would think only the printf string "%d%%" needs to be translated.
The result can be right-aligned in the available space, e.g. by
first calling vim_snprintf(), getting the length and then when it's
too short prepend a space and repeat. Or compute the percentage, and
when it smaller than 10 insert a space. Translation should never
change the decimal number value.

Thanks for the tips! Indeed the updated solution turned out to be much
more concise. I am still not sure about memory management though,
since I haven't used C for anything other than snippets like these.
Please let me know if it's not okay!

The new version doesn't allocate memory, thus that's taken care of.
We can put the space in in the code, so that there is no need for two
translated text entries.  I'll make it like that, please check.
--
Hanson's Treatment of Time:
There are never enough hours in a day, but always too
many days before Saturday.
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


Feature/dupstreams batch

Implements a HLS op for duplicating streams, useful for nonlinear topologies.
Yaman, with the addition of rtlsim_multi_io, I could modify the addstreams_batch code to use that and revert rtlsim to its original form (single input). Let me know what you prefer.

Thanks Lucian, I've moved the rtlsim_multi_io int utils. I'll merge this as-is for now and open a separate issue to migrate all HLSCustomOps to use rtlsim_multi_io so that we can get rid of the other variants.


⚠ Fortnite has degraded performance

In 357078f, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1660 ms

Resolved: Fortnite performance has improved in 44c9023.


Access violation when running simulation

Hi guys,
I am trying to simulate some CBCT projection data with XCIST recently and I started with the Adult_Male_Standard_Lung_Phantom provided in xcist/phantoms-voxelized. Much to my disappointment, I got some issues with memory access. I'm not quite sure if there is something wrong with my code or XCIST itself.
Here is my simulation code
import os
import catsim as xc
##--------- Initialize
ct = xc.CatSim("FI_sim/cfg/Phantom_malelung")

##--------- Make changes to parameters or load other cfg files (optional)
output_folder = "./FI_sim/output/test_malelung/"
if not os.path.exists(output_folder):
    os.mkdir(output_folder)
ct.resultsName = output_folder + "result"

##--------- Run simulation
ct.run_all()  # run the scans defined by protocol.scanTypes

Phantom configuration
# Phantom
phantom.callback = "Phantom_Voxelized"      # name of function that reads and models phantom
phantom.projectorCallback = "C_Projector_Voxelized" # name of function that performs projection through phantom
phantom.filename = './phantoms-voxelized/Adult_Male_Standard_Lung_Phantom/adult_male_standard_lung.json'
phantom.centerOffset = [0.0, 0.0, 0.0]      # offset of phantom center relative to origin (in mm)
phantom.scale = 1.0                         # re-scale the size of phantom

Terminal output
Airscan
Loading GEANT4/EDLP97 cross section database...compton..rayleigh..photo..scatter..form..done
Scan sim time: 4.1 s
Offset scan
Scan sim time: 1.0 s
Phantom scan
Backend TkAgg is interactive backend. Turning interactive mode on.
in C> Preparing to allocate memory for material volume data...
in C> Allocated memory for image volume for material  1
in C> Copying data for material  1 into C memory...in C>  done.
in C> Allocated memory for image volume for material  2
in C> Copying data for material  2 into C memory...Traceback (most recent call last):
  File "C:\Users\Administrator\anaconda3\envs\catsim\lib\runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "C:\Users\Administrator\anaconda3\envs\catsim\lib\runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "c:\Users\Administrator\.vscode\extensions\ms-python.python-2021.10.1336267007\pythonFiles\lib\python\debugpy\__main__.py", line 45, in <module>
    cli.main()
  File "c:\Users\Administrator\.vscode\extensions\ms-python.python-2021.10.1336267007\pythonFiles\lib\python\debugpy/..\debugpy\server\cli.py", line 444, in main
    run()
  File "c:\Users\Administrator\.vscode\extensions\ms-python.python-2021.10.1336267007\pythonFiles\lib\python\debugpy/..\debugpy\server\cli.py", line 285, in run_file
    runpy.run_path(target_as_str, run_name=compat.force_str("__main__"))
  File "C:\Users\Administrator\anaconda3\envs\catsim\lib\runpy.py", line 263, in run_path
    pkg_name=pkg_name, script_name=fname)
  File "C:\Users\Administrator\anaconda3\envs\catsim\lib\runpy.py", line 96, in _run_module_code
    mod_name, mod_spec, pkg_name, script_name)
  File "C:\Users\Administrator\anaconda3\envs\catsim\lib\runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "e:\Tianling Lyu\CatSim\code\FI_sim\scripts\simulate_super0_malelung.py", line 25, in <module>
    ct.run_all()  # run the scans defined by protocol.scanTypes
  File "C:\Users\Administrator\anaconda3\envs\catsim\lib\site-packages\catsim\CatSim.py", line 28, in run_all
    cfg = self.phantom_scan(cfg)
  File "C:\Users\Administrator\anaconda3\envs\catsim\lib\site-packages\catsim\CatSim.py", line 52, in phantom_scan
    cfg = one_scan(cfg)
  File "C:\Users\Administrator\anaconda3\envs\catsim\lib\site-packages\catsim\OneScan.py", line 39, in one_scan
    cfg = feval(cfg.phantom.callback, cfg)
  File "C:\Users\Administrator\anaconda3\envs\catsim\lib\site-packages\catsim\CommonTools.py", line 30, in feval
    return eval("md."+funcName)(*args)
  File "C:\Users\Administrator\anaconda3\envs\catsim\lib\site-packages\catsim\Phantom_Voxelized.py", line 59, in Phantom_Voxelized
    set_voxelized_volume(cfg, volumeData, volumeDims, offsets, voxelsize, xyMask, materialIndex, numberOfMaterials)
  File "C:\Users\Administrator\anaconda3\envs\catsim\lib\site-packages\catsim\Phantom_Voxelized.py", line 89, in set_voxelized_volume
    offsets[0], offsets[1], offsets[2], voxelsize[0], voxelsize[2], xyMask, materialIndex, numberOfMaterials)
OSError: exception: access violation writing 0x000001945B5C8000

Is there any advice to run simulations on those lung phantoms? Thanks a lot in advance.
Best regards,
Tianling Lyu

This issue seems to be related to the integer upper bound.
// clib_build/src/voxelized_projector.cpp: line 196-197
unsigned long phantom_num_voxels_this_material  = (unsigned long)dims[0]*(unsigned long)dims[1]*(unsigned long)dims[2];
unsigned long required_memory_size_this_material  = phantom_num_voxels_this_material *(unsigned long)sizeof(float);

For material 2 in the male lung phantom, the voxel number is 1,444,060,800 and the required memory size would be 5,776,243,200 which exceeds the upper bound of unsigned long on my PC.
The code works after I changed those unsigned long to unsigned long long.

Thank you for the advice. I have updated the code with "long long".


Logo Change

The logo doesn't look appealing, so I need to redesign it with a more modern and attractive look.

plz @SurajPratap10 assign to the task to me

@adityakr0660 you can start working on this


Daft Sparse Tensor Logical Type

Enable an in memory format for sparse tensors like COO
This should also work with writing out to parquet.
We can probably export to something like arrows sparse tensor type:

https://arrow.apache.org/docs/cpp/api/tensor.html#sparse-tensors

Hi, I am on it :)

@michaelvay let us know how progress is so far! Happy also to pair up and walk you through any parts of the codebase that you might have questions on.