fix rollbacks

Rebased, squashed, and improved commit message from @kderme .

Still testing this. Will keep on rebasing as things are merged to master.

Have had one crash running this branch (following the chain). I was not able to see the actual error because it kept on retrying and failing and the real error was not in the scrollback.

Ok, this thing crashed again:
[2021-04-29 02:07:11.39 UTC] runDBThread: DbInsertException "Tx: 32"
  ( SqlError {sqlState = "23503"
  , sqlExecStatus = FatalError
  , sqlErrorMsg = "insert or update on table \"tx\" violates foreign key constraint \"tx_block_id_fkey\""
  , sqlErrorDetail = "Key (block_id)=(5899482) is not present in table \"block\"."
  , sqlErrorHint = ""
  })

Something is really wrong in this PR. I highly suspect this is due to trying to implement rollbacks at the source end of the TBQueue instead of the destination end.

Indeed there is a race condition. drainQueue is not an good enough check to access the db from the main thread.

I'm thinking about changing or adding a new lock, but I havent found a good solution yet.


Has anyone  installed Dispatch successfully lately? I get build errors

Describe the bug
install.sh run fails with buster-pgdg Release error
$ ./install.sh
Checking minimum requirements...
./.env already exists, skipped creation.
./requirements.txt already exists, skipped creation.
Creating volumes for persistent storage...
Created dispatch-postgres.
Leaving existing SECRET_KEY...
Leaving existing DISPATCH_JWT_SECRET...
Pulling, building, and tagging Docker images...
[+] Pulling 1/1
✔ postgres Pulled                                                                                0.7s
[+] Building 92.8s (14/16)
=> [core internal] load build definition from Dockerfile                                                                                0.0s
=> => transferring dockerfile: 4.51kB                                                                                0.0s
=> WARN: FromAsCasing: 'as' and 'FROM' keywords' casing do not match (line 1)                                                           0.0s
=> [core internal] load metadata for docker.io/library/python:3.11.4-slim-buster                                                        0.6s
=> [core internal] load .dockerignore                                                                                0.0s
=> => transferring context: 400B                                                                                0.0s
=> [core internal] load build context                                                                                0.2s
=> => transferring context: 120.94kB                                                                                0.2s
=> [core sdist 1/6] FROM docker.io/library/python:3.11.4-slim-buster@sha256:c46b0ae5728c2247b99903098ade3176a58e274d9c7d2efeaaab3e0621  0.0s
=> CACHED [core stage-1 2/7] RUN groupadd -r dispatch && useradd -r -m -g dispatch dispatch                                             0.0s
=> CACHED [core stage-1 3/7] RUN apt-get update && apt-get install -y --no-install-recommends     ca-certificates     wget gnupg     &  0.0s
=> CACHED [core stage-1 4/7] RUN echo "deb http://apt.postgresql.org/pub/repos/apt buster-pgdg main" > /etc/apt/sources.list.d/pgdg.li  0.0s
=> ERROR [core stage-1 5/7] RUN wget --quiet -O - https://deb.nodesource.com/setup_12.x | bash -                                       87.3s
=> CACHED [core sdist 2/6] RUN apt-get update && apt-get install -y --no-install-recommends     ca-certificates     wget     && rm -rf  0.0s
=> CACHED [core sdist 3/6] RUN wget --quiet -O - https://deb.nodesource.com/setup_16.x | bash -     && apt-get install -y nodejs --no-  0.0s
=> CACHED [core sdist 4/6] RUN echo "DISPATCH_LIGHT_BUILD="                                                                             0.0s
=> [core sdist 5/6] COPY . /usr/src/dispatch/                                                                                0.4s
=> CANCELED [core sdist 6/6] RUN YARN_CACHE_FOLDER="$(mktemp -d)"     && export YARN_CACHE_FOLDER     && pushd /usr/src/dispatch       89.3s

[core stage-1 5/7] RUN wget --quiet -O - https://deb.nodesource.com/setup_12.x | bash -:
0.518
0.518 ================================================================================
0.518 ================================================================================
0.518
0.518                               DEPRECATION WARNING
0.519
0.519   Node.js 12.x is no longer actively supported!
0.519
0.519   You will not receive security or critical stability updates for this version.
0.519
0.519   You should migrate to a supported version of Node.js as soon as possible.
0.519   Use the installation script that corresponds to the version of Node.js you
0.519   wish to install. e.g.
0.519
0.519    * https://deb.nodesource.com/setup_16.x — Node.js 16 "Gallium"
0.519    * https://deb.nodesource.com/setup_18.x — Node.js 18 LTS "Hydrogen" (recommended)
0.519    * https://deb.nodesource.com/setup_19.x — Node.js 19 "Nineteen"
0.519    * https://deb.nodesource.com/setup_20.x — Node.js 20 "Iron" (current)
0.519
0.519   Please see https://github.com/nodejs/Release for details about which
0.519   version may be appropriate for you.
0.519
0.519   The NodeSource Node.js distributions repository contains
0.519   information both about supported versions of Node.js and supported Linux
0.519   distributions. To learn more about usage, see the repository:
0.519     https://github.com/nodesource/distributions
0.519
0.519 ================================================================================
0.519 ================================================================================
0.519
0.519 Continuing in 20 seconds ...
0.519
20.52
20.52 ================================================================================
20.52 ▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓
20.52 ================================================================================
20.52
20.52                            SCRIPT DEPRECATION WARNING
20.52
20.52
20.52   This script, located at https://deb.nodesource.com/setup_X, used to
20.52   install Node.js is deprecated now and will eventually be made inactive.
20.52
20.52   Please visit the NodeSource distributions Github and follow the
20.52   instructions to migrate your repo.
20.52   https://github.com/nodesource/distributions
20.52
20.52   The NodeSource Node.js Linux distributions GitHub repository contains
20.52   information about which versions of Node.js and which Linux distributions
20.52   are supported and how to install it.
20.52   https://github.com/nodesource/distributions
20.52
20.52
20.52                           SCRIPT DEPRECATION WARNING
20.52
20.52 ================================================================================
20.52 ▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓
20.52 ================================================================================
20.52
20.52 TO AVOID THIS WAIT MIGRATE THE SCRIPT
20.52 Continuing in 60 seconds (press Ctrl-C to abort) ...
20.52
80.53
80.53 ## Installing the NodeSource Node.js 12.x repo...
80.53
80.54
80.54 ## Populating apt-get cache...
80.54
80.54 + apt-get update
80.65 Get:1 http://deb.debian.org/debian buster InRelease [122 kB]
80.70 Get:2 http://deb.debian.org/debian-security buster/updates InRelease [34.8 kB]
80.70 Get:3 http://deb.debian.org/debian buster-updates InRelease [56.6 kB]
80.91 Get:4 http://deb.debian.org/debian buster/main amd64 Packages [7909 kB]
80.97 Ign:5 http://apt.postgresql.org/pub/repos/apt buster-pgdg InRelease
81.13 Err:6 http://apt.postgresql.org/pub/repos/apt buster-pgdg Release
81.13   404  Not Found [IP: <IP_ADDRESS> 80]
81.28 Get:7 http://deb.debian.org/debian-security buster/updates/main amd64 Packages [610 kB]
81.91 Get:8 http://deb.debian.org/debian buster-updates/main amd64 Packages [8788 B]
84.16 Reading package lists...
85.52 E: The repository 'http://apt.postgresql.org/pub/repos/apt buster-pgdg Release' does not have a Release file.
85.52 Error executing command, exiting

failed to solve: process "/bin/bash -o pipefail -o errexit -c wget --quiet -O - https://deb.nodesource.com/setup_12.x | bash -" did not complete successfully: exit code: 1
Cleaning up...
To Reproduce
Steps to reproduce the behavior:

Run install.sh as directed in the install steps.

Expected behavior
Dispatch is deployed
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Ubuntu
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Tried this on a Amazon Linux 2 host also and got the similar results.

Same issue. Is there an already built docker image or helm chart? I didn't found any and this would be handy (trying to do a kube deploy)

Same issue. Is there an already built docker image or helm chart? I didn't found any and this would be handy (trying to do a kube deploy)

Considering the complex build steps involved I wish Dispatch is released in Docker image form as well. The users don't gain anything by building it locally.
I could build and deploy it somehow after making few changes to the docker resource files but not sure if that's the right way.


Update Specific Columns on certain rows with values?

We have a use case where we want to update specific columns on some given rows of a google sheet.
We are already reading from a google sheet using the sheets API but finding it very cumbersome on trying to actually update cells. Essentially we are reading from a google sheet, automating some tasks with other 3rd parties using sheet information and then coming back to update the google sheet on some cells depending how the automated jobs ran and whether or not they succeeded
Can this library support this functionality?

@LeoJHarris.  Yes, you can definitely do that with this library.  One thing to be aware of.  Google Sheets API does throttle how many calls you can make against the API, so if you are doing a lot of calls, you may need to build in some wait logic to avoid hitting the throttle limits.
I updated the Sample Project in this source code to do something similar to what you are asking here => https://github.com/SteveWinward/GoogleSheetsWrapper/blob/main/src/GoogleSheetsWrapper.SampleClient/Program.cs
The main logic is here below.  Please read this projects main README.md to see how to implement strongly typed models to do this.
var records = respository.GetAllRecords();

foreach (var record in records)
{
    try
    {
        Foo(record.TaskName);
        record.Result = true;
        record.DateExecuted = DateTime.UtcNow;

        var result = respository.SaveFields(
            record,
            r => r.Result,
            r => r.DateExecuted);
    }
    catch (Exception ex)
    {
        record.Result = false;
        record.ErrorMessage = ex.Message;
        record.DateExecuted = DateTime.UtcNow;

        var result = respository.SaveFields(
            record,
            r => r.Result,
            r => r.ErrorMessage,
            r => r.DateExecuted);
    }
}

The SampleRecord class looks like this,
public class SampleRecord : BaseRecord
{
    [SheetField(
            DisplayName = "Task",
            ColumnID = 1,
            FieldType = SheetFieldType.String)]
    public string TaskName { get; set; }

    [SheetField(
            DisplayName = "Result",
            ColumnID = 2,
            FieldType = SheetFieldType.Boolean)]
    public bool Result { get; set; }

    [SheetField(
            DisplayName = "Error",
            ColumnID = 3,
            FieldType = SheetFieldType.String)]
    public string ErrorMessage { get; set; }

    [SheetField(
            DisplayName = "DateExecuted",
            ColumnID = 4,
            FieldType = SheetFieldType.DateTime)]
    public DateTime DateExecuted { get; set; }
}

The SampleRepository class looks like this,
namespace GoogleSheetsWrapper.SampleClient
{
    public class SampleRepository : BaseRepository<SampleRecord>
    {
        public SampleRepository() { }

        public SampleRepository(SheetHelper<SampleRecord> sheetsHelper)
            : base(sheetsHelper) {}
    }
}

Thanks you for the comprehensive and sample provide! 👍I did run into a simple error I think, that is that on the sheetHelper.Init("jsonCredentials")
System.InvalidOperationException: 'Error creating credential from JSON or JSON parameters. Unrecognized credential type .'
I have credentials.json file that was working previously and directly against the google API NuGet:
UserCredential credential;
        using (FileStream stream = new("C:\\.......\\credentials.json", FileMode.Open, FileAccess.Read))
        {
            const string credPath = "token.json";
            credential = GoogleWebAuthorizationBroker.AuthorizeAsync(
                GoogleClientSecrets.FromStream(stream).Secrets,
                _scopes,
                "user",
                CancellationToken.None,
                new FileDataStore(credPath, true)).Result;

            Console.WriteLine("Credential file saved to: " + credPath);
        }

        _sheetsService = new(new BaseClientService.Initializer()
        {
            HttpClientInitializer = credential,
            ApplicationName = "Discovery Sample",
        });

From your sample I tried this:
// Get the Google Spreadsheet Config Values
        var serviceAccount = #########;
        var documentId = _appSettings?.GoogleSheetsID;
        var jsonCredsPath = ""C:\\.......\\credentials.json"";

        // In this case the json creds file is stored locally, but you can store this however you
        // want to (Azure Key Vault, HSM, etc)
        var jsonCredsContent = System.IO.File.ReadAllText(jsonCredsPath);

        // Create a new SheetHelper class
        var sheetHelper = new SheetHelper<GoogleSheetProfile>(documentId, serviceAccount, "Staging");
        sheetHelper.Init(jsonCredsContent);

The credentials.json is the same but throwing error in the library 😕 is there something I am doing wrong?

Might be worth pointing out the format of the json is:
{
    "installed": {
        "client_id": "################.apps.googleusercontent.com",
        "project_id": "#####",
        "auth_uri": "https://accounts.google.com/o/oauth2/auth",
        "token_uri": "https://oauth2.googleapis.com/token",
        "auth_provider_x509_cert_url": "https://www.googleapis.com/oauth2/v1/certs",
        "client_secret": "###########",
        "redirect_uris": [
            "http://localhost"
        ]
    }
}

@LeoJHarris.  For this library you need to create a service account and authenticate as the service account.  This is why one of the parameters of the SheetsHelper class is the service account identity.
Take a look at the notes I put together on authentication here,
https://github.com/SteveWinward/GoogleSheetsWrapper?tab=readme-ov-file#authentication
There's also a good summary of how to configure this and grant the service account access to the Google spreadsheet in this article below,
https://robocorp.com/docs/development-guide/google-sheets/interacting-with-google-sheets#create-a-google-service-account

@LeoJHarris  I’m going to close this issue out. If you still have issues please reopen this one or create a new one.

Great thanks! Working now, will test the sample code you provided but thanks heaps this is a really great library! 👍  perfect fit for out application 🥇

Awesome. If you have any feedback for the library feel free to share. We also accept pull requests too for library updates.


include nanosecond precision for write_points method

According to the documentation, the new 0.9.0 release will include support for nanosecond time precision:
http://influxdb.com/docs/v0.9/concepts/reading_and_writing_data.html#time-format
the write_points method will need to support a new time_precision parameter (maybe with 'n' for nanosecond?)

It looks like n is already accepted.

Ah, I see it in the current code. I was looking at the code for release 2.3.0 (which does not have it).


How to re-train Segnet ?

I saw you have released Classification/Recognition (ResNet-18) and Object Detection (SSD-Mobilenet), but how about re-training of Segmentation Networks, I want use the segnet to train my own dataset, how should I do?

Hi @Jingchensun , I recommend checking this post:   https://github.com/dusty-nv/jetson-inference/issues/724#issuecomment-694832441
Segmentation networks are heavy and the datasets typically large, so may need to train on PC / server with discrete GPU instead.  If you use a small custom dataset like in that post I linked to, you may be able to do it on Jetson though.

Hi @Jingchensun , I recommend checking this post: #724 (comment)
Segmentation networks are heavy and the datasets typically large, so may need to train on PC / server with discrete GPU instead. If you use a small custom dataset like in that post I linked to, you may be able to do it on Jetson though.

Hi, thank you for your suggestion, I re-trained the segmentation model on my server successfully with your instruction.

Hi,Can you teach me how to do that,thanks! @Jingchensun

Please what are the steps to retrain the segnet model

Hi @Yhaw, please see this post - https://www.highvoltagecode.com/post/edge-ai-semantic-segmentation-on-nvidia-jetson

Thank you
On Sun, Mar 26, 2023 at 5:41 AM Dustin Franklin @.***>
wrote:

Hi @Yhaw https://github.com/Yhaw, please see this post -
https://www.highvoltagecode.com/post/edge-ai-semantic-segmentation-on-nvidia-jetson
—
Reply to this email directly, view it on GitHub
https://github.com/dusty-nv/jetson-inference/issues/730#issuecomment-1484001663,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMOOJ4PHDRUZ5G4H5HXJZ7TW57JJTANCNFSM4RS35TKQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


[v6] Feature: disable use of remote audit by default

Request Description
For npm, the remote audit is enabled by default to avoid false negatives. Let's make this in an opt-in in v6 to prefer offline-only first.
Additional Information
How do we deal with the fact that we might miss legitimate malware since the vdb is rebuilt only every x hours or so, and users might forget to refresh the database periodically?

@prabhu I had started on a config file for depscan that would include allowing the user to set a setting to periodically update vdb. My thought was that we could also store the date/time of the last update and log that info at the beginning of every scan so that the user would be aware.
How about I return to this and add to v6... there are other things I want to incorporate into it when I have time, but this part is quite easy.

Thanks @cerrussell. Adding this to the config file is a good idea.


Incompatibility of listen with Ruby < 2.2

The dependency "listen" in version >= 3.1 isn't compatible with Ruby < 2.2.
See https://github.com/guard/listen/pull/371

@tdreyno Now the build is 💚

Did they really just remove support for an entire version of ruby in a minor version number bump? :(

@tdreyno -

Did they really just remove support for an entire version of ruby in a minor version number bump? :(

An "entire" MINOR version bump of Ruby.
People mindlessly update gems (without upgrading Changelogs) but ironically they're very conservative when upgrading backward-compatible Ruby versions.
Long story short: according to semver it's fine. Impact: 0.
@danielbayerlein - wasn't compatible "de facto", so this is more of a bugfix (in dependency declaration).
Also, Ruby 2.2. and 2.3 are recommended. Also, Ruby 2.1 is potentially buggy with no support:
"This means that after the release of 2.1.9 we will never backport any bug fixes to 2.1 except security fixes."
Please don't make me the enemy of the community all because clueless users are frustrated about upgrading to a recommended version of Ruby.

The Ruby community is broken and needs patching ;)
Please consider branching of for any (unreasonable IMHO) backports to Ruby 2.1 and switching to Ruby 2.2.

Amen.
FYI: I'm using Ruby 2.3 but not all users of Middleman do this.


Show warning when no indices set

Closes #72
This PR adds a warning to the php bin/console meili:import command when no indices are specified.

bors try


Fix init-env.php to account for upstream changes

Right now the container image cannot be rebuilt because a function signature changed upstream, and init-env.php is not calling it correctly. Let's fix that so I can rebuild the container...

For context:
In acquia_cms_headless 1.3.0-beta3, createHeadlessSite() took no arguments: https://git.drupalcode.org/project/acquia_cms_headless/-/blob/1.3.0-beta3/src/Service/StarterkitNextjsService.php#L188
As of 1.3.0-beta4 (released August 25th), it does: https://git.drupalcode.org/project/acquia_cms_headless/-/blob/1.3.0-beta4/src/Service/StarterkitNextjsService.php#L209
So, init-env.php needs to pass those arguments.


Fix bug where getModel return undefined because deviceId is undefined

Description
Can't find a related issue, but when testing the RC I found that deviceId in getModel would be undefined because of a inconsistency of the deviceId const in getModel.
Compatibility

OS
Implemented

iOS
✅

Android
✅

Windows
✅

Checklist
All others seem unrelated ;)

[x] I have tested this on a device/simulator for each compatible OS
[ ] I added the documentation in README.md
[ ] I mentioned this change in CHANGELOG.md
[ ] I updated the typings files (deviceinfo.d.ts, deviceinfo.js.flow)
[ ] I updated the dummy web/test polyfill (default/index.js)
[ ] I added a sample use of the API (example/App.js)

Obsoleted by https://github.com/mikehardy/react-native-device-info/commit/a7129c15dcdd79480eec23011db31154cd1ca7df (I wanted to keep the different variable name, but obviously I needed to use it) - will be released as rc.5 shortly - thanks for reporting this @rnnyrk !


Added GamepadEx class with mapping functionality for actions

The GamepadEx class adds the ability to map actions to specific buttons, and automatically runs those actions when the buttons are pressed.
Further, it exposes useful information for the buttons, joysticks, and triggers such as whether it was just pressed (rather than just whether it's down or not, like Qualcomm gives you) or just released.
It also allows users to increase deadzones for joysticks and triggers, which is something I have personally desired (on rare occasions) and is a useful feature (and doesn't get in the way if it's not desired, as it defaults to the same as what Qualcomm does).

Thanks for the patch!
I'll go into a deeper review in a bit, but some quick overall changes I noticed: instead of creating its own action, this should integrate with the existing ActionRunner class.
I also would love to see some example usage of this (I'm struggling to understand how it's used right now)
Sorry about the confusion with the merge and then unmerge, there was a minor error on our part; we're still interested in your contribution!

Thanks for the patch! I'll go into a deeper review in a bit, but some quick overall changes I noticed: instead of creating its own action, this should integrate with the existing ActionRunner class. I also would love to see some example usage of this (I'm struggling to understand how it's used right now) Sorry about the confusion with the merge and then unmerge, there was a minor error on our part; we're still interested in your contribution!

Sounds good! I'll take a look at integrating it with ActionRunner, and I'll write an example usage for it.
No worries about the merge&unmerge, it's an easy mistake to make!


Add rl/height-weight-analysis into spine-generic:master

Hi @jcohenadad, here is the requested PR. I am still working on help/documentation texts, but the code might be ready.
If you would like to replicate the graphs and results, pseudocode of 4 necessary steps would be as follows:
1. process_data.sh
Extended list of outputs
extract_measures.sh file might be deleted as everything is included in the updated process_data.sh
2. Download:
fs_measurements.xlsx, sg.lh.aparc.stats.postcentral.csv, sg.lh.aparc.stats.precentral.csv, sg.rh.aparc.stats.postcentral.csv and sg.rh.aparc.stats.precentral.csv files into the ${PATH_RESULTS} folder

fs_measureemnts.xlsx table was provided by @mbondy023 who visually inspected each segmentation prior adding numbers of each participant into the table
.csv files were additionally generated from FS result files with extract_fs_measures.sh script in the root folder (after the @mbondy023's visual inspection)
Pitfall: FS pipeline is not automated into the spine-generic pipeline now. Do we want and need to improve this? UMN preference was to rahter share just the table files, as FS is stochastic and leading to slightly different numbers per each run.
BTW all my ${PATH_RESULTS} including the 4 above listed same files

3. Open MATLAB and execute:
cd('~/spine-generic/matlab')
stat = sg_structure_versus_demography(path_results,path_data)

In bash pseudocode you need to execute the function with following settings:
cd('`which process_data.sh`/matlab')
stat = sg_structure_versus_demography('${PATH_RESULTS}','${PATH_DATA}')

PATH_DATA is necessary to find the exclude.yml file which is not copied into the PATH_RESULTS folder during the process_data.sh
I tested MATLAB code on version R2021b, because MathWorks introduced quite inteligent automated axis label orientation and saved me time and nerves with it... :-)
4. Bunch of fig_*.png files should appear in the ${PATH_RESULTS} folder.
From the matlab stat variable, you might be interested in stat.tbl

Please let me know, if something is not clear.
Good luck,
Rene

we probably don't need these:

matlab/Tests/Data/test_import/~temporary.yaml
matlab/Tests/Data/test_import/selftest_report.html
matlab/Tests/selftest_report.html

and other temporary and non-code data.

@jcohenadad Yes, you are right some files from the Tests folder might be deleted. I tried to delete the whole Tests folder but the yamlmatlab toolbox stopped to work then. I will try the old-school Windows 95 optimization method, alá: "Try everything what can be deleted from the C:\WINDOWS and the system is still working..."
But note, that after deleting of these files we will not have in the code the exact copy of the yamlmatlab toolbox.

But note, that after deleting of these files we will not have in the code the exact copy of the yamlmatlab toolbox

how about adding as a dependency this toolbox (with its version) in the README for people who would like to reproduce these results?  The philosophy is not to include the code of others (which could conflict with our license), but to instruct how to download it

how about adding as a dependency this toolbox (with its version) in the README for people who would like to reproduce these results? The philosophy is not to include the code of others (which could conflict with our license), but to instruct how to download it

Yes, we can do it this way, not a big deal.  I will delete it and include description and URL link into README.md
Originally, I thought that as the yamlmatlab toolbox is under MIT license, it is fine to extend it into our source code when their copyright/license is acknowledged. I also thought that ANTs registration is similarly included into the SCT.

how about adding as a dependency this toolbox (with its version) in the README for people who would like to reproduce these results? The philosophy is not to include the code of others (which could conflict with our license), but to instruct how to download it

Yes, we can do it this way, not a big deal. I will delete it and include description and URL link into README.md
Originally, I thought that as the yamlmatlab toolbox is under MIT license, it is fine to extend it into our source code when their copyright/license is acknowledged.

Indeed, the MIT license would be compatible so it should be fine to include it, but good practice is to mention external dependencies instead of including them.

I also thought that ANTs registration is similarly included into the SCT.

Actually, ANTs code is not included in SCT's repository but it is downloaded upon installation of SCT by the install script. So this is exactly what I am suggesting to you here: having instructions (or installation script if you package it) that fetches a specific version of YAMLMatlab to install it upon installation of the spine-generic project. What complicates things here, however, is that we are dealing with two different languages (python and Matlab), and packaging becomes a bit of a nightmare. Which is why instructions (vs. install script) seems to me the most reasonable approach.

How will we name the current analysis in the documentation? "Body size versus the structure of the central nervous system" ? For headings, sub-headings, references, etc.

yup! sounds good

"Fetch" sounds me like somethin github related. I am not sure if it is possible for the yamlmatlab toolbox as only place where I found it stored is google archive: https://code.google.com/archive/p/yamlmatlab/downloads

Sorry I'm not sure I understand the question. My suggestion was to write in the instructions something like this:
Section: Body size versus the structure of the central nervous system
Dependencies: YAMLMatlab_0.4.3

@jcohenadad UPDATE:

New file: matlab/sg_example_script.m explaining how to set YAMLMatlab toolbox and other variables
Deleted YAMLMatlab toolbox from the source code
New doc file: docs/body-size-pipeline.rst explaining how to replicate Results (I have shortened the title due to limited space in the website left sidebar)
Edited: docs/index.rst

Please let me know, what else needs to be revised. I hope the updated docs stayed compatible and functional.

Looks good @renelabounek ! I only made minor suggestions. It can be merged afterwards

@Igor-Nestrasil has also approved final code and docs version. So @jcohenadad, you can approve the review and let the branch to be merged. I do not have the right to approve the review.
Thanks, Rene

@jcohenadad Can you please force merge of this PR? Again, it stopped on the build python checks. Like previously here https://github.com/spine-generic/spine-generic/pull/253#issuecomment-1005115882 and here https://github.com/spine-generic/spine-generic/pull/255#issuecomment-989120707
I have no  clue why it is happening when I have commited zero to the python code. It seems that you did not also identified the cause before.


Update Settings (fixes#14)

انبسط

Please create another branch and pull request and do the following changes:
First of all before creating the branch update your main branch (you should always update main to get new changes before creating a new branch from it)
Name of branch is: feature/14-update-settings
Always check you changes for example in this pull request you added a package-lock.json file in the root directory. you shouldn't add this file
You need to add media folder to .gitignore
For static root make it like this:
STATIC_ROOT = os.path.join(BASE_DIR, "assets")
and add that folder (assets) to gitignore
انبسط أنت هيهيهيه


state continuous changes in calendar view

something is causing MakeActive state to change continuously, potentially due to returned useEffect when component is unmounted.

I was not able to replicate this in production, seems to have only occured on locally run server.  This may not be an issue.  Can anyone else confirm this behavior locally or on production? (locally, easiest way is to check if MakeActive dispatch runs continuously in console of app while on calendar (view events) page)

closing this, as it doesnt seem to happen in production or locally anymore

reopening, as I was able to replicate consistently on production and local branch.
steps to replicate issue: (must have 2+ events created on a single month)

open calendar view tab (view-events)
select an event
quickly select another event on the calendar view

Seems if second event is selected before redux dispatches and/or axios post requests finish for first event, it starts a loop of dispatch requests on front end, and is visually seen as the light green selector switching to events that were not selected. console shows MAKEACTIVE and SINGLE_EVENT_FETCH_SUCCESS dispatches running multiple times, before stopping. (not endless loop).
Seems to happen more often when first visiting page (before all events have been fully loaded from database), and the loop seems to run longer if you select more events to start the issue. (ex. select 5 different events in a row instead of just 2)


Prepare for releasing v1.1.3

Updated POM and release notes for releasing the v1.1.3 of the plugin

Coverage remained the same at 55.823% when pulling 913be611c4dba52ab9e0b255de25c6fad691626f on GerritForge:master into e30cd98453c698849a0c60a0bc696bb261a1df79 on jenkinsci:master.


Fix email preview and sending.

I'm not familiar with that flow so I'll leave the review to @malemburg


How to setup WAF?

The documentation here (https://docs.microsoft.com/en-us/azure/frontdoor/front-door-overview#application-layer-security) says that

Azure Front Door allows you to author custom web application firewall (WAF) rules for access control to protect your HTTP/HTTPS workload

But I can't find where I can configure the WAF on the portal like I usually do using Azure Application Gateway (with WAF).

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

ID: dbadca96-7341-36d0-8a38-1c6a5c96b22d
Version Independent ID: 2fad91dc-25f8-0c7a-c21d-93986ec51145
Content: Azure Front Door Service
Content Source: articles/frontdoor/front-door-overview.md
Service: frontdoor
GitHub Login: @sharad4u
Microsoft Alias: sharadag

Hi @frankfuu Thank you for your feedback! We have assigned this issue to the author to investigate further and take the right course of action.

hi @mike-urnun-msft ,
Just FYI, I raised a support ticket on this and the response given was that WAF functionality on Azure Front Door is limited to REST, Powershell, and CLI while it is still in Preview. Portal access will supported before the service is in GA.
Here is the docs to implement a WAF in Powershell - https://docs.microsoft.com/en-us/powershell/module/azurerm.frontdoor/new-azurermfrontdoorfirewallpolicy?view=azurermps-6.13.0

@teresayao - Can you take a look at this?

@mike-urnun-msft - Please work with @KumudD and @teresayao to resolve this.

#reassign: amitsriva

@frankfuu: Thanks for providing feedback that helps improve our documentation. This issue has been resolved and we are closing the issue. #please-close


There was an error with your request

Describe the bug
It use to be working, but today doesn't work and shown either "There was an error with your request" or " OpenAI error.." how can i de-bug it?
Steps to reproduce
It use to be working, but today doesn't work and shown either "There was an error with your request" or " OpenAI error.." how can i de-bug it?
Screenshots or additional context
It use to be working, but today doesn't work and shown either "There was an error with your request" or " OpenAI error.." how can i de-bug it?

I also encountered this problem

https://vercel.com/new/clone?repository-url=https%3A%2F%2Fgithub.com%2Fhahahumble%2Fspeechgpt&env=VITE_OPENAI_API_KEY,VITE_ACCESS_ CODE&envDescription=If%20you%20do%20not%20want%20to%20provide%20a%20value%2C%20use%20REPLACE_WITH_YOUR_OWN.&project-name=speechgpt &repository-name=speechgpt
Now after redeployment, entering the question on the page still prompts this:There was an error with your request


Design question: require a local npm install of wct in project under test?

In wct 6.0 we're moving to requiring that the project under test has a local install of the client-side code for wct. In the current bower world that means that they need to have run bower install web-component-tester. This reduces magic and gives the project under test more control over the versions of their test-time dependencies.
Should we do the same for the node side of wct, and require that the project under test have wct installed in their node_modules directory? Already the global wct command defers to any local installation, if any. This change would reduce the impact of any future breaking changes in wct, but it would increase the friction of getting started with wct, and require users to use both bower and npm in all of their projects under test.

Given how important it is that tests be repeatable, and that if they pass on one machine they should pass on another, we're moving forward with this requirement.

On further reflection, we're moving this to just be a warning, as it's not yet common for elements to have a package.json file, and it would be really disruptive to throw an error here, especially as we haven't actually broken backcompat with any server-side features, provided that the project under test has the web-component-tester bower package installed.


don't use input quality when resizing images

Scenario:

site has images displayed at 100x100 but they're actually 200x200
images are stored on the server as jpegs at quality 50
without pagespeed this doesn't look that bad, because the browser retains all available info when resizing the image to 100x100 for display
pagespeed will substitute 100x100 images at quality 50
with pagespeed this looks very bad, because with 75% fewer pixels quality 50 now looks terrible

The main difference between what the browser is doing and what pagespeed is doing is that on the resize pagespeed is setting output quality to min(ImageRecompressionQuality, input quality) while the browser is setting output quality to ∞.  To fix this, pagespeed should set output quality on resize to ImageRecompressionQuality, ignoring input quality, and then as usual only accept the resize if we made the image take fewer bytes.
Mailing list discussion prompting this: https://groups.google.com/forum/#!topic/mod-pagespeed-discuss/abtCwJ08gdY

If we're feeling more ambitious (and some of Maks' image studies might provide the data we would need for this) we can look at scaling up the quality as we scale down the image, on the principle that there's a continuum of badness here (and that we'll tend to just do a lot of work that we throw away if we peg things at 85%).
This ties in to quality settings for responsive images – that is an untapped TODO on the responsive images filter that might provide big wins for bandwidth in practice.

Yes, ideally if we would say "this 200x200 image at quality N displayed at 100x100 looks like what you would get if you resized to 100x100 with quality M" and then use min(M, 85) for the output quality.  But I don't think in general there's a nice way to find which (quality, DPI) tuples are equivalent?

A few years ago (2012?) people were talking about how you could sometimes get better looking images by increasing pixels while decreasing quality [1][2].  I vaguely remember Pat Meenan saying that if you do your experiments properly it's not actually an improvement, but I'm not finding that now.  If we understood the tradeoffs around size vs quality level better it's possible that we could do some interesting things here.
[1] http://www.netvlies.nl/blog/design-interactie/retina-revolution
[2] https://www.filamentgroup.com/lab/compressive-images.html

Yes, this is pretty much what the original reporter was doing on their site.
On Wed, Jul 22, 2015 at 2:53 PM, Jeff Kaufman<EMAIL_ADDRESS>wrote:

A few years ago (2012?) people were talking about how you could sometimes
get better looking images by increasing pixels while decreasing quality
[1][2]. I vaguely remember Pat Meenan saying that if you do your
experiments properly it's not actually an improvement, but I'm not finding
that now. If we understood the tradeoffs around size vs quality level
better it's possible that we could do some interesting things here.
[1] http://www.netvlies.nl/blog/design-interactie/retina-revolution
[2] https://www.filamentgroup.com/lab/compressive-images.html
—
Reply to this email directly or view it on GitHub
https://github.com/pagespeed/mod_pagespeed/issues/1112#issuecomment-123826325
.

After talking to @huibaolin I no longer think "stop using input quality to determine output quality on resize" is right in all cases.  Most of the resizing we do currently is by a smaller amount, like 200x200 to 195x195, and in those cases if the input is q=75 we don't want to be saying that at 195x195 we require q=85.  Ideally here we would consult a table and determine that the correct output quality is, say, q=76 for 195x195 and q=82 for 150x150.
With the current architecture of image.cc, making a quality decision based on resize ratio is pretty awkward.

After talking to @huibaolin, I don't think it makes sense to work on this right now.  While this was something that some people advocated for performance, it never caught on especially and is still very rare.  Fixing it would make our image resizing code more complex for very small benefit, and there are bigger things to work on for now.


Make a pipeline for pushing the VsCode extension

As part of migrating from ADO to GitHub Actions, we now need to reimplement a pipeline for automatically pushing the .VSIX file to VsCode Marketplace.
While doing that, It is also a good idea to consider moving the extension under the Microsoft publisher.

Done


Sdk resolution should emit some events to help with debugging

On diagnostic verbosity, whenever the Evaluator asks the SdkResolverService for an sdk, it should emit an event stating:

referenced sdk
actual sdk resolved (actual version)
sdk path
resolver that resolved it (type simple name and assembly path). Also mention whether it resolved from the cache or not

@nguerrera @dsplaisted


RTP implementation

Issue #, if available:
RTP implementation.
Description of changes:
Add STUN serializer/deserializer and make them standalone. The change also adds codec packetizer and depacketizer for H.264, G.711, Opus and VP8 codec.
Testing -

RTP tests run with the wrapper layer for serialization and deserialization.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice

Need to close this and open new PR
As this PR was not getting updated.


Enable non-youtube videos

Currently, we only have the ability to embed youtube videos, so important videos get missed.  here is a example of one video we could not accommodate: http://newsone.com/3203425/footage-of-matthew-ajibades-struggle-with-police-released-during-trial/

Although we have now enabled Vimeo videos like in this case: http://ebwiki.org/articles/george-golden, I will leave this issue open since there are likely many other types of videos we will want to embed.

Or you can close this one, and open a ticket for each additional type of
embedded video
On Apr 27, 2016 8:02 AM, "campgurus"<EMAIL_ADDRESS>wrote:

Although we have now enabled Vimeo videos like in this case:
http://ebwiki.org/articles/george-golden, I will leave this issue open
since there are likely many other types of videos we will want to embed.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/BOWiki/BOW/issues/255#issuecomment-215061317

whate'r you think is best

ok closing this. We will add issues as we find types of videos we want to enable.


deprecate FormGroup component

I don't think this really adds much value. I think we should just remove this all together. Thoughts?

Yeah if all it's doing is adding spacing, a component seems like overkill.

While I am not a fan of FormGroup we do have a very similar component downstream. Knowing this we could keep it so that there is some parity with the design updates across consumers, but I think it's perfectly acceptable to handle this on a per-project basis than host this upstream.

addressed in #14

Are we keeping this then, or removing it?

I have another PR coming up that will address this. had to address a few other needs that will result in a v 2.0

Cool. Looks like we're set to remove the style folder too.
On Apr 20, 2017, 10:01 AM -0600, Jake Rios<EMAIL_ADDRESS>wrote:

I have another PR coming up that will address this. had to address a few other needs that will result in a v 2.0
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub (https://github.com/TWExchangeSolutions/es-components/issues/11#issuecomment-295792099), or mute the thread (https://github.com/notifications/unsubscribe-auth/AAwpkrdkRMwEosEo4nGDg-jWpDWyGzjxks5rx4FjgaJpZM4NAoMt).

Nice. I'll pull that out as well. 👍

Now addressed in #16


Update 1.21

Have you tried it? Both GlowingEntities and GlowingBlocks?

Have you tried it? Both GlowingEntities and GlowingBlocks?

I tried GlowingEntities, but I did not try GlowingBlocks since I don't have a Paper setup.
I will update when I do.

Works great on paper 1.21

Awesome! Thank you very much for this PR :)


Is it possible to instruct envoy transcoder to return the errors triggered at its level as a JSON and not as a plain string ?

Hi,
Is it possible to instruct envoy transcoder to return the errors triggered at its level in the following JSON format, and not just as a plain string message?
Current response:
project.isDone: invalid value false for type type.googleapis.com/test.api.YesNo
What I would like to get is a JSON like this:
{   "code": 3,   "message": "project.isDone: invalid value false for type type.googleapis.com/test.api.YesNo",   "details": [] }
The reason why I want this is because I want to have a standard error message structure across my transcoded endpoints.
Thanks!
FYI I'm triggering this error by passing a Boolean value in a filed that expects an enum value.

cc @qiwzhang @lizan

Any feedback on this one ? Thanks!

@ghevge you may do this with existing local reply mapper: https://www.envoyproxy.io/docs/envoy/latest/configuration/http/http_conn_man/local_reply

@lizan  Thank you! I will give it a try.


arm target

I'd like to bench fft codes on an armv8 target
what options should I set to get the best results with KFR vs fftw ?
with my best regards

Done.


Any suggestion on connecting gopro through wifi & internet at the same time.

Hi,
I am new on gopro here, I can connect my computer to gopro by connecting gooro wifi.
However, i lost my internet connection so is here any way to connect both gopro & internet at the same time?

connect an Ethernet cable

There is a way but it's a bit technical. You will need two WiFi routers and both should have DD-WRT firmwares on them to be able to tweak the configuration.
It is an interesting project if you like to try it. I wrote a detailed guide here.
Feel free to ask any questions.
Thanks


Button in the Deploy a model menu to deploy the model doesn't work

Module: 02
Lab/Demo: Explore Automated Machine Learning in Azure ML
Task: Deploy a predictive service
Step: 03
Description of issue
Button in the Deploy a model doesn't work/do anything after pressing it while trying to deploy a model. Not the only one encountering issue with this step.  https://www.reddit.com/r/AzureCertification/comments/13cy5no/ai900_first_lab_does_not_create_the_azure/

Repro steps:

Follow the lab instructions

Hello,
I followed all steps and couldn´t reproduce the issue. The Deploy button worked and I could finish the lab without issues.
Have you tried going through a new deployment to see if the error continues? How about a different browser?
Best regards,
Mihai

Hi,
I have run this through the Skillable Learn on Demand labs which only contain Microsoft Edge as browser to my knowledge. I have tried it two times just to be sure I will try a different lab provider if it makes a difference.

Thanks @teriaavibes and @Mihai-Ac. I will share this bug with the Azure ML product team as it is not a lab-specific issue.


Fail to launch Spyder - updated the spyder output #8211

Sir,
This issue is related to issue #8211
I have updated the spyder output above for your inspection.
I have not able to launch python.


Noncached answers, returning 500 internal server error.

This link produces a 500 internal server error:
http://cheat.sh/psql/convert+numeric+column+to+integer
Wheras this works as expected:
http://cheat.sh/python/write+to+file
Again this produces a 500 internal server error:
http://cheat.sh/python/write+to+foobar
screenshots
https://i.imgur.com/ON0X66o.png
https://i.imgur.com/q4C5T2z.png
Why is your server showing me variables that link to brightcove and something.oreilly.com and techbus.safaribooks, to fetch courses then blasting me in the face with "The hostname changed, so you can't do that."
Caution, Rogued Robots?
https://youtu.be/iHwIFGi3WPU?t=112

Please try now.
What is for JS code? We have no JS in the project

It's working now.  The brightcove and techbus errors were clientside cascade errors from a naughty browser plugin that was examining your web page, and when it tried to talk to its mothership server, it caused a cascade of errors.  today I learned.
Another gentle reminder for everyone everywhere to review all browser plugins that can examine site data, they can and will examine your visited-site data and transmit observed information back to its whatever.mothership_data_collector49783.com then puke on the carpet when the original site produces a malformed response during chrome V8 engine html/javascript render.
Thanks.

Now it makes sense. Eric, thank you for reporting!

Reopening because this bug crops up every 2 to 5 months.
This query works because serverside has cached the response:
http://cheat.sh/psql/convert+numeric+column+to+integer
This query produces an internal server error because serverside has not cached the response:
http://cheat.sh/psql/convert+numeric+column+to+double


[Server-Worker-6/WARN]: Exception loading blockstate definition

If I try to harvest your plants they simply disappear, even if they plantet on soil or are naturally grown
It dosn't work with empty hand and/or any tool
https://paste.ee/p/Wg7JG
the log is from a actually running game.
If mor logs need please tell.

These exceptions are unrelated to actually harvesting. These are because I haven't gotten around to implementing an actual model for soybeans yet.


UDPConn.ReadFromUDP blocked in syscall when underlying socket option is set

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
1.11.1
Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
linux, centos 7, amd4
What did you do?
I have a program need to create large number of listening UDPConn, basically around 30k, each UDPConn will run in its own go routine waiting incoming packets, so that's 30k go routines, however I hit the wall of 10000 system threads limitation; after search around, I found it is because I set socket option to use ESPinUDP encap: "syscall.SetsockoptInt(conn_fd, syscall.IPPROTO_UDP, 100, 2)" , with the socket option set, each go routine is blocked in syscall, thus consuming a lots of system threads:
101 @ 0x47b915 0x47b163 0x47841f 0x49039f 0x4f735b 0x516dba 0x514bad 0x6fd1bf 0x45c741
#	0x47b914	syscall.Syscall6+0x4			/usr/local/go/src/syscall/asm_linux_amd64.s:44
#	0x47b162	syscall.recvfrom+0xa2			/usr/local/go/src/syscall/zsyscall_linux_amd64.go:1648
#	0x47841e	syscall.Recvfrom+0xae			/usr/local/go/src/syscall/syscall_unix.go:262
#	0x49039e	internal/poll.(*FD).ReadFrom+0xfe	/usr/local/go/src/internal/poll/fd_unix.go:215
#	0x4f735a	net.(*netFD).readFrom+0x5a		/usr/local/go/src/net/fd_unix.go:208
#	0x516db9	net.(*UDPConn).readFrom+0x69		/usr/local/go/src/net/udpsock_posix.go:47
#	0x514bac	net.(*UDPConn).ReadFromUDP+0x6c		/usr/local/go/src/net/udpsock.go:109
#	0x6fd1be	main.listensocket+0x3ae			/root/gowork/src/test/test.go:55

if I don't set the socket option, then number of thread is far less, and go routine no longer use syscall:
101 @ 0x42f3cb 0x42a709 0x429db6 0x48f1aa 0x48f2bd 0x490408 0x4f735b 0x516dba 0x514bad 0x6fd0b7 0x45c741
#	0x429db5	internal/poll.runtime_pollWait+0x65	/usr/local/go/src/runtime/netpoll.go:173
#	0x48f1a9	internal/poll.(*pollDesc).wait+0x99	/usr/local/go/src/internal/poll/fd_poll_runtime.go:85
#	0x48f2bc	internal/poll.(*pollDesc).waitRead+0x3c	/usr/local/go/src/internal/poll/fd_poll_runtime.go:90
#	0x490407	internal/poll.(*FD).ReadFrom+0x167	/usr/local/go/src/internal/poll/fd_unix.go:219
#	0x4f735a	net.(*netFD).readFrom+0x5a		/usr/local/go/src/net/fd_unix.go:208
#	0x516db9	net.(*UDPConn).readFrom+0x69		/usr/local/go/src/net/udpsock_posix.go:47
#	0x514bac	net.(*UDPConn).ReadFromUDP+0x6c		/usr/local/go/src/net/udpsock.go:109
#	0x6fd0b6	main.listensocket+0x2a6			/root/gowork/src/test/test.go:55

so is there way to use socket option without consume so many system threads?
What did you expect to see?
consume less system threads
What did you see instead?
hit 10000 system threads limitation

How are you getting the value conn_fd that you mention?  My first guess would be that you are doing something that causes the file descriptor to go into blocking mode.  If so, the fix will be to call the SyscallConn method and call the Control method of the returned syscall.RawConn value.
Closing on the assumption that that is what you are looking for.  Please comment if you disagree.  In general, please ask questions on a forum, not the issue tracker; see https://golang.org/wiki/Questions .  Thanks.

Thanks, what you suggested works.
I was using File().Fd() to get the fd, and using syscall.SetsockoptInt
btw: I knew this is not right place to ask questions, but I have posted this question on go-nuts a few days ago, did't get any reply ...

File().Fd() to get the fd, and using syscall.SetsockoptInt

And forgot to call FilePacketConn for turning the socket descriptor into the runtime-integrated network poller again.

but I have posted this question on go-nuts ... did't get any reply

Probably the provided information is not sufficient, as @iant had to "guess" for your "question." Having a look at https://github.com/golang/go/wiki/HowToAsk and https://github.com/golang/go/wiki/How-to-ask-for-help would help you next time.


Add PowerOffTime Settings

Add PowerOffTime Settings
0: Never turn off
else: power Off after setting hours.

#105 : fixed


Navigation issues in V1.0 updated setup sequence.

Version
V1.0.0

Android 9
Moto G7

Steps to reproduce
Multiple navigation issues in the startup flow.
By fast the worst is when you get to the HA screen.  Having entered your HA, you then have to go "back" to go forward.  And the back button is in the top left hand corner, which doesn't match any of the navigation patterns the user has seen up to that point.
Vedha didn;t spot that back button, so tried the Android back button, and at this stage, that just closes the app.
I think most users will fail to get past this stage without closing the app themselves or hitting the back button.
There are a number of other navigation issues around this initial flow.  None so critical, but in combination they do make things even worse.

You can't go back on any of the opening screens.  That was OK before as they were mostly just messages, with no config, but now e.g. the location screen is directly after tha HAs.  Totally legitimate that you might want to go back & tweak your HA settings, but there is no back button.  Android back button closes the app.
If you do close the app (whether using Android back button, or by other means), there are some odd things in the flow:
You have to accept the Icense agreement again, which feels odd.
You don't have to enter an HA again, if you entered it already.  But the prompt tells you that you have to.  There is no adjustment to the text to reflect the fact that you have done thisstep already - I guess the design didn't anticipate the user having to come bak a 2nd time.  So the text says: "... you will need to subscribe to a health authority" even tough you already have.
When you have not yet entered a Health Authority, I would suggest the "Next" button be labelled "Later" (or "Skip").
It is inconsistent that you can skip HA setup, but not skip location setup (even though you can turn location off again in-app later!).

Expected Behavior
Expectations are:

A seamless setup experience, where I keep moving forward and don't have ot move back
If I do need to move back, the back controls should be clerly visible and the ANdroid Back button should also work.
HA & Location screens should be consistent with regard to navigation (can I go back?) and whether I can skip.
If I end up repeating after closing the app, we should have a consistent paradigm on whether config /choices need to be re-entered.  My preference should be that choices doon't need to be re-entered, but I value consistency most of all.

Some screenshots.
Here is the HA selection screen.  The back button is in the top left.  This made sense previously when you had navigated to the screen via the dashboard link in the top right of the app, but in this new context it makes no sense at all.

Here's the previous screen.  No "Back" button, and "Next" would be better as "Later" or "Skip" until you have set an HA.  Then when you have set an HA, it should no longer say "you need to set an HA".  But even better would be not having to come back to this screen again after setting your HA.

Here's the location screen.  No back button on here.

One further thought:

To me the ordering of HAs vs. Locations is odd.  Location is something everyone using the app should set up, whether they have an HA available or not.  WHereas HA is something that to start with most users of teh app won't be able to set up.

Therefore HA to me belongs logically after Location.  I think people who don't have an HA available in their area will be confused by the HA setup page (we stil l haven't done enough to help it look OK to people that they don't have any HAs to register with yet).  So let's at least get the location setup (which is not confusing) done before we confuse them...!

Hi all! Thoughts:

You can't go back on any of the opening screens. That was OK before as they were mostly just messages, with no config, but now e.g. the location screen is directly after tha HAs. Totally legitimate that you might want to go back & tweak your HA settings, but there is no back button. Android back button closes the app.
-To my knowledge, there is only one HA that's selectable at the moment. Am I wrong? That would lead me to believe this is relatively okay for now.

If backgrounding the app:
-You have to accept the Icense agreement again, which feels odd.
-The implementation is currently a step and not a recorded DB value
-We should think about this in the future. But I'm not sure this is a blocker

-You don't have to enter an HA again, if you entered it already. But the prompt tells you that you have to. There is no adjustment to the text to reflect the fact that you have done thisstep already
-I guess the design didn't anticipate the user having to come bak a 2nd time. So the text says: "... you will need to subscribe to a health authority" even tough you already have.
-When you have not yet entered a Health Authority, I would suggest the "Next" button be labelled "Later" (or "Skip").
-It is inconsistent that you can skip HA setup, but not skip location setup (even though you can turn location off again in-app later!).
-Will think about this more...

Seems like we can close this?

closing.   Health authority selection was remove from the onboarding sequence in #679


ruby version dependence

Need to add the language version dependence
Because in a new feature we start to use the method .try() included in ruby 2.3.0

It's already set on gemspec (currently >=2.6.0)


Send headers in Sendgrid Adapter

Implements #445. Allows arbitrary setting of Sendgrid email headers through Bamboo.Email's headers field.
Detailed documentation on Sendgrid's support for email headers here: https://sendgrid.com/docs/API_Reference/Web_API_v3/Mail/index.html

@maymillerricci could I get a review on this?

@maymillerricci any updates on this? anything I should change?

@michaelm244 and @maymillerricci I attempted to add the suggested test, and ended up having to do the following:

d584ca1 rebases thoughtbot/bamboo:master into this branch
079ce95 fixes some code formatting issues I created fixing merge conflicts
b825220 fixes a test that broke after rebasing with upstream

a test was introduced which added a "Reply-To" header with a Tuple value, which breaks Jason.Encoder
i noticed that put_headers/2 was already handling something with the "reply-to" header, and guessed that I'd need to do the same for "Reply-To"
i then renamed the variable from prepared_headers to headers_without_tuple_values to reflect that removing Tuple values is necessary for adding headers

541af34 finally adds the requested test for removing "reply-to" and "Reply-To" headers

I think this may be ready to merge now, what do you think?

Thanks for taking this to the finish line @stratigos 🙂


Add tests

Can we close this @ruffle1986 ?

https://github.com/purposeindustries/react-key-value/pull/10


Write a function to exclude EE and TB

Should take a list of dates per station for which to drop the daily averages

Started in branch eetf. Currently takes a data frame of id's and dates, but could optionally take a named list of id's and date ranges

Or a data frame like:

id
start
end

a
2016-08-12
2016-08-21

a
2016-09-01
2016-09-03

b
2016-08-15
2016-08-19


one large README instead of a multi-page documentation site

I believe, it would be more convenient for users, if you generate the entire documentation as a single long Markdown document and then inject it into the README.md on every push, making a pull request with the changes. Then, you will merge the pull request and the project face will be updated. We do this, for example, in this repo: https://github.com/yegor256/micromap/blob/master/.github/workflows/benchmark.yml
Motivation:

The navigation will be easier
Formatting will be GitHub native (no need to get used to your own style)

I assume your comment is about our site (link).

I believe,  it would be more convenient for users, if you generate the entire documentation as a single long Markdown document

GH Markdown makes section names at certain level so small that they're almost indistinguishable from bold text. The site  does have pages with nested sections. Hence, the text structure in a single Markdown file may be non-obvious. Example:
Section name
Section body
Additionally, the site contains sections that are of no interest for a user, e.g., Contributing. I don't think users would like to scroll through them in the README.

We do this, for example, in this repo: https://github.com/yegor256/micromap/blob/master/.github/workflows/benchmark.yml

Do you suggest that we make pipeline results more prominent?
We currently have a link to the report (link) in the Introduction section on our site (link).
I can agree that this link is not very discoverable. To fix this, I can additionally put it into the README.
However, I can't agree that we should include the whole report into the README.

The report is large. It's not convenient to scroll through it in the README.
It's impossible to make the report interactive and well-formatted in the README due to GH limitations for JS and CSS.
I believe the full report in the README is of no interest for a user. So, I prefer to have a dedicated page that

Explains (almost) everything about the results
Provides the summary of results
Provides the results

I believe, it would be more convenient for users

Why do you believe so?

The navigation will be easier

What is not easy about the current navigation?

@deemp I can't prove it: https://stackoverflow.com/questions/78519596/what-is-better-a-single-long-readme-file-or-a-set-of-smaller-cross-linked-pages but it seems that one long README is a good practice now

We do have TOC.

@maxonfjvipon @volodya-lombrozo what do you think?

@yegor256, @maxonfjvipon here's some evidence that a README shouldn't contain the whole documentation. So, I vote for a short README with the most important links.

link

@deemp currently, there is nothing in README. If you move "the information necessary for developer to get started and contributing" -- 50% of your documentation site will end up being in the README. Then, the question is: why not 100%? Where is the line?
I believe that the links above are mostly about Javadoc (or similar) documentation generators, which indeed are very verbose and should be placed in README. If this project is planning to have such an API doc -- definitely let's keep it in a separate site. However, currently, your documentation site contains what usually people keep in the README.

the information necessary for developer to get started and contributing

I assume something like this.
https://github.com/haskell/haskell-language-server

@deemp yes, this is what looks wrong, if you ask me. And the authors of the links you quoted above would agree with you :)

However, currently, your documentation site contains what usually people keep in the README.

@yegor256, I'm not convinced. I think I can agree or accept your suggestion in two cases.

You show me:

The data on the number of repos with:

docs sites only
README only
both

That repos with a README only prevail

You set a clear policy for the objectionary organization what should go into a README

The policy should explain what to do about:

interactive elements
custom styling
math notation
blobs of data like our report
acceptable section nesting level
etc.

@deemp how about we just stick to the formula suggested above by yourself: "README should be a short overview and quick start" (it seems that most people agree, in the poll). Once it's done, you will see how much is left for the auto-generated static site.

@yegor256, I don't mind to try, so I created https://github.com/objectionary/normalizer/issues/368.


[Snyk] Security upgrade langchain-core from 0.1.23 to 0.1.31

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `pip` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

vizro-ai/snyk/requirements.txt

⚠ Warning
openai 0.28.1 requires aiohttp, which is not installed.

Vulnerabilities that will be fixed
By pinning:

Severity
Priority Score (*)
Issue
Upgrade
Breaking Change
Exploit Maturity

888/1000  Why? Proof of Concept exploit, Recently disclosed, Has a fix available, CVSS 9.9
Path Traversal SNYK-PYTHON-LANGCHAINCORE-6370598
langchain-core:  0.1.23 -> 0.1.31
No
Proof of Concept

(*) Note that the real score may have changed since the PR was raised.
Some vulnerabilities couldn't be fully fixed and so Snyk will still find them when the project is tested again. This may be because the vulnerability existed within more than one direct dependency, but not all of the affected dependencies could be upgraded.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

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

Duplicate of https://github.com/mckinsey/vizro/pull/369


Minor Makefile adjustment to ease the dev/test workflow in docker-compose / swarm scenarios [skip ci]

This minor change simplifies the workflow when working with docker-compose and docker-swarm tools, where a 'runtime' flag is not either available (docker-swarm), or matured enough (docker-compose). In this environment it is handy to set sysbox-runc as the default runtime, while continuing to use regular runc for building/testing sysbox.
Signed-off-by: Rodny Molina<EMAIL_ADDRESS>

LG, but we should probably not "skip CI" for this change, just to make sure all is good in the CI machine.

Not needed in this case, but done anyways :-)


NetworkManager + fortisslvpn can no longer connect after update

Describe the bug
After recent updates (I'm not sure which component is here to blame exactly, NM, pppd or the plugin) it can no longer connect to VPN. I can still connect fine when going back to generation made few days ago. I'll reboot to older generation to get versions of those components and post that in comments later.
Steps To Reproduce
Steps to reproduce the behavior:
1.Create Forti SSL VPN connection in NetworkManager
2. Try to connect
Expected behavior
Established connection
Screenshots
N/A
Additional context
Output from journalctl -b -u NetworkManager
maj 10 11:03:47 BigPC NetworkManager[3895]: INFO:   Connected to gateway.
maj 10 11:03:47 BigPC NetworkManager[3895]: INFO:   Authenticated.
maj 10 11:03:47 BigPC NetworkManager[3895]: INFO:   Remote gateway has allocated a VPN.
maj 10 11:03:47 BigPC pppd[3896]: Plugin /nix/store/brlk8rk7wj8h7jyaxkf1zx8nklx161by-NetworkManager-fortisslvpn-gnome-1.4.0/lib/pppd/2.5.0/nm-fortisslvpn-pppd-plugin.so loaded.
maj 10 11:03:47 BigPC NetworkManager[3896]: Plugin /nix/store/brlk8rk7wj8h7jyaxkf1zx8nklx161by-NetworkManager-fortisslvpn-gnome-1.4.0/lib/pppd/2.5.0/nm-fortisslvpn-pppd-plugin.so loaded.
maj 10 11:03:47 BigPC pppd[3896]: pppd 2.5.0 started by root, uid 0
maj 10 11:03:47 BigPC pppd[3896]: Using interface ppp0
maj 10 11:03:47 BigPC NetworkManager[3896]: Using interface ppp0
maj 10 11:03:47 BigPC NetworkManager[3896]: Connect: ppp0 <--> /dev/pts/0
maj 10 11:03:47 BigPC pppd[3896]: Connect: ppp0 <--> /dev/pts/0
maj 10 11:03:47 BigPC NetworkManager[3896]: {/tmp/.CJZA41} {/tmp}
maj 10 11:03:47 BigPC NetworkManager[3896]: {/tmp/.CJZA41} {.CJZA41}
maj 10 11:03:47 BigPC NetworkManager[3895]: INFO:   Got addresses: [OBFUSCATED]
maj 10 11:03:47 BigPC NetworkManager[3895]: INFO:   Negotiation complete.
maj 10 11:03:49 BigPC NetworkManager[3895]: INFO:   Negotiation complete.
maj 10 11:03:49 BigPC NetworkManager[3896]: Peer refused to agree to his IP address
maj 10 11:03:49 BigPC NetworkManager[3896]: Connect time 0.1 minutes.
maj 10 11:03:49 BigPC NetworkManager[3896]: Sent 1101 bytes, received 1081 bytes.
maj 10 11:03:49 BigPC pppd[3896]: Peer refused to agree to his IP address
maj 10 11:03:49 BigPC pppd[3896]: Connect time 0.1 minutes.
maj 10 11:03:49 BigPC pppd[3896]: Sent 1101 bytes, received 1081 bytes.
maj 10 11:04:47 BigPC NetworkManager[3895]: ERROR:  Timed out waiting for the ppp interface to be UP.
maj 10 11:04:47 BigPC NetworkManager[3895]: INFO:   Cancelling threads...
maj 10 11:04:47 BigPC NetworkManager[3895]: INFO:   Cleanup, joining threads...
maj 10 11:04:47 BigPC pppd[3896]: Hangup (SIGHUP)
maj 10 11:04:47 BigPC NetworkManager[3896]: Hangup (SIGHUP)
maj 10 11:04:47 BigPC NetworkManager[3896]: Modem hangup
maj 10 11:04:47 BigPC pppd[3896]: Modem hangup
maj 10 11:04:47 BigPC NetworkManager[3896]: Connection terminated.

Notify maintainers
@jtojnar @vcunat @maxeaubrey
Metadata
-> % nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.27, NixOS, 23.05 (Stoat), 23.05.20230509.9524f57`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.13.3`
 - channels(root): `"nixos"`
 - channels(dominik): `""`
 - channels(dkl): `"nixpkgs"`
 - nixpkgs: `/home/dkl/.nix-defexpr/channels/nixpkgs`

On older generation, NetworkManager succesfully connects and here's the log
maj 10 11:19:08 BigPC systemd[1]: Starting Network Manager...
maj 10 11:19:08 BigPC systemd[1]: Started Network Manager.
maj 10 11:19:33 BigPC NetworkManager[3197]: INFO:   Connected to gateway.
maj 10 11:19:33 BigPC NetworkManager[3197]: INFO:   Authenticated.
maj 10 11:19:33 BigPC NetworkManager[3197]: INFO:   Remote gateway has allocated a VPN.
maj 10 11:19:33 BigPC pppd[3201]: Plugin /nix/store/icwbvs7hzxrxc6ch7sj61hi3fn2k6ik4-NetworkManager-fortisslvpn-gnome-1.4.0/lib/pppd/2.4.5/nm-fortisslvpn-pppd-plugin.so loaded.
maj 10 11:19:33 BigPC NetworkManager[3201]: Plugin /nix/store/icwbvs7hzxrxc6ch7sj61hi3fn2k6ik4-NetworkManager-fortisslvpn-gnome-1.4.0/lib/pppd/2.4.5/nm-fortisslvpn-pppd-plugin.so loaded.
maj 10 11:19:33 BigPC pppd[3201]: pppd 2.4.9 started by root, uid 0
maj 10 11:19:33 BigPC pppd[3201]: Using interface ppp0
maj 10 11:19:33 BigPC NetworkManager[3201]: Using interface ppp0
maj 10 11:19:33 BigPC NetworkManager[3201]: Connect: ppp0 <--> /dev/pts/0
maj 10 11:19:33 BigPC pppd[3201]: Connect: ppp0 <--> /dev/pts/0
maj 10 11:19:33 BigPC NetworkManager[3201]: {/tmp/.X5VA41} {/tmp}
maj 10 11:19:33 BigPC NetworkManager[3201]: {/tmp/.X5VA41} {.X5VA41}
maj 10 11:19:33 BigPC NetworkManager[3197]: INFO:   Got addresses: [OBFUSCATED], ns [OBFUSCATED, <IP_ADDRESS>], ns_suffix [OBFUSCATED]
maj 10 11:19:33 BigPC NetworkManager[3197]: INFO:   Negotiation complete.
maj 10 11:19:35 BigPC NetworkManager[3197]: INFO:   Negotiation complete.
maj 10 11:19:35 BigPC pppd[3201]: local  IP address OBFUSCATED
maj 10 11:19:35 BigPC NetworkManager[3201]: local  IP address OBFUSCATED
maj 10 11:19:35 BigPC NetworkManager[3201]: remote IP address OBFUSCATED
maj 10 11:19:35 BigPC pppd[3201]: remote IP address OBFUSCATED
maj 10 11:19:35 BigPC NetworkManager[3201]: Can't execute /etc/ppp/ip-up: Permission denied
maj 10 11:19:35 BigPC pppd[3201]: Can't execute /etc/ppp/ip-up: Permission denied
maj 10 11:19:35 BigPC NetworkManager[3197]: INFO:   Interface ppp0 is UP.
maj 10 11:19:35 BigPC NetworkManager[3197]: INFO:   Tunnel is up and running.

-> % nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.26, NixOS, 23.05 (Stoat), 23.05.20230429.3dcff81`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.13.3`
 - channels(root): `"nixos"`
 - channels(dominik): `""`
 - channels(dkl): `"nixpkgs"`
 - nixpkgs: `/home/dkl/.nix-defexpr/channels/nixpkgs`

Uh, I don't know... perhaps updating to latest commit instead of just pulling the patches?  For example: https://github.com/NixOS/nixpkgs/compare/master...vcunat:nixpkgs:p/networkmanager-fortisslvpn

Uh, I don't know... perhaps updating to latest commit instead of just pulling the patches? For example: master...vcunat:nixpkgs:p/networkmanager-fortisslvpn

Unfortunately, the same result
maj 14 00:34:38 BigPC NetworkManager[3257]: INFO:   Connected to gateway.
maj 14 00:34:38 BigPC NetworkManager[3257]: INFO:   Authenticated.
maj 14 00:34:38 BigPC NetworkManager[3257]: INFO:   Remote gateway has allocated a VPN.
maj 14 00:34:38 BigPC pppd[3261]: Plugin /nix/store/9d7mdq5vhqmc1w6vyxg1hb13c13nimhh-NetworkManager-fortisslvpn-gnome-unstable-2023-03-07/lib/pppd/2.5.0/nm-fortisslvpn-pppd-plugin.so loaded.
maj 14 00:34:38 BigPC NetworkManager[3261]: Plugin /nix/store/9d7mdq5vhqmc1w6vyxg1hb13c13nimhh-NetworkManager-fortisslvpn-gnome-unstable-2023-03-07/lib/pppd/2.5.0/nm-fortisslvpn-pppd-plugin.so loaded.
maj 14 00:34:38 BigPC pppd[3261]: pppd 2.5.0 started by root, uid 0
maj 14 00:34:38 BigPC pppd[3261]: Using interface ppp0
maj 14 00:34:38 BigPC NetworkManager[3261]: Using interface ppp0
maj 14 00:34:38 BigPC NetworkManager[3261]: Connect: ppp0 <--> /dev/pts/0
maj 14 00:34:38 BigPC pppd[3261]: Connect: ppp0 <--> /dev/pts/0
maj 14 00:34:38 BigPC NetworkManager[3261]: {/tmp/.A04D51} {/tmp}
maj 14 00:34:38 BigPC NetworkManager[3261]: {/tmp/.A04D51} {.A04D51}
maj 14 00:34:38 BigPC NetworkManager[3257]: INFO:   Got addresses: [10.x.x.x], ns [x.x.x.x, <IP_ADDRESS>], ns_suffix [blablabla]
maj 14 00:34:38 BigPC NetworkManager[3257]: INFO:   Negotiation complete.
maj 14 00:34:40 BigPC NetworkManager[3257]: INFO:   Negotiation complete.
maj 14 00:34:40 BigPC NetworkManager[3261]: Peer refused to agree to his IP address
maj 14 00:34:40 BigPC NetworkManager[3261]: Connect time 0.1 minutes.
maj 14 00:34:40 BigPC NetworkManager[3261]: Sent 1101 bytes, received 1081 bytes.
maj 14 00:34:40 BigPC pppd[3261]: Peer refused to agree to his IP address
maj 14 00:34:40 BigPC pppd[3261]: Connect time 0.1 minutes.
maj 14 00:34:40 BigPC pppd[3261]: Sent 1101 bytes, received 1081 bytes.
maj 14 00:35:39 BigPC NetworkManager[1339]: <warn>  [1684017339.5801] vpn[0x19144c0,0ba639c5-a719-44ba-8e06-1c519231cac6,"Unity VPN"]: connect timeout exceeded
maj 14 00:35:39 BigPC nm-fortisslvpn-[3234]: Connect timer expired, disconnecting.
maj 14 00:35:39 BigPC NetworkManager[3257]: INFO:   Cancelling threads...
maj 14 00:35:39 BigPC NetworkManager[3257]: INFO:   Cleanup, joining threads...
maj 14 00:35:39 BigPC NetworkManager[3261]: Hangup (SIGHUP)
maj 14 00:35:39 BigPC NetworkManager[3261]: Modem hangup
maj 14 00:35:39 BigPC pppd[3261]: Hangup (SIGHUP)
maj 14 00:35:39 BigPC NetworkManager[3261]: Connection terminated.
maj 14 00:35:39 BigPC NetworkManager[3261]: unable to delete pid file /var/run/pppdppp0.pid: Permission denied
maj 14 00:35:39 BigPC pppd[3261]: Modem hangup
maj 14 00:35:39 BigPC pppd[3261]: Connection terminated.
maj 14 00:35:39 BigPC pppd[3261]: unable to delete pid file /var/run/pppdppp0.pid: Permission denied
maj 14 00:35:39 BigPC pppd[3261]: Exit.
maj 14 00:35:39 BigPC NetworkManager[3257]: INFO:   pppd: The link was terminated by the modem hanging up.
maj 14 00:35:39 BigPC NetworkManager[3257]: INFO:   Terminated pppd.
maj 14 00:35:39 BigPC NetworkManager[3257]: INFO:   Closed connection to gateway.
maj 14 00:35:39 BigPC NetworkManager[3257]: INFO:   Logged out.

I'll try to downgrade individual packages to see what helps.

I'd guess that ppp update to 2.5.0 is what triggered the change/failure.

I finally made it to build it with ppp 2.4.9 (it triggered a lot of compilation), but sadly looks like that didn't help
I used such overlay
self: super:
{
  ppp = super.ppp.overrideAttrs (old: {
    version = "2.4.9";
    sha256 = "sha256-8+nbqRNfKPLDx+wmuKSkv+BSeG72hKJI4dNqypqeEK4=";
    configureFlags = [
      "--with-openssl=${super.openssl.dev}"
    ];
  });
}

And the output
kwi 20 15:29:27 BigPC pppd[17805]: pppd 2.4.9 started by root, uid 0
kwi 20 15:29:27 BigPC pppd[17805]: Using interface ppp0
kwi 20 15:29:27 BigPC NetworkManager[17805]: Using interface ppp0
kwi 20 15:29:27 BigPC NetworkManager[17805]: Connect: ppp0 <--> /dev/pts/2
[...]
kwi 20 15:29:29 BigPC NetworkManager[17805]: Can't execute /etc/ppp/ip-up: Permission denied
kwi 20 15:29:29 BigPC pppd[17805]: remote IP address <IP_ADDRESS>
kwi 20 15:29:29 BigPC pppd[17805]: Can't execute /etc/ppp/ip-up: Permission denied
kwi 20 15:29:29 BigPC NetworkManager[17801]: INFO:   Interface ppp0 is UP.
kwi 20 15:29:29 BigPC NetworkManager[17801]: INFO:   Tunnel is up and running.
kwi 20 21:07:48 BigPC NetworkManager[17801]: INFO:   Cancelling threads...
kwi 20 21:07:48 BigPC NetworkManager[17801]: INFO:   Cleanup, joining threads...
kwi 20 21:07:48 BigPC NetworkManager[17801]: INFO:   Setting ppp0 interface down.
kwi 20 21:07:48 BigPC pppd[17805]: Hangup (SIGHUP)
kwi 20 21:07:48 BigPC NetworkManager[17805]: Hangup (SIGHUP)
kwi 20 21:07:48 BigPC NetworkManager[17805]: Modem hangup
kwi 20 21:07:48 BigPC NetworkManager[17805]: Connect time 338.4 minutes.
kwi 20 21:07:48 BigPC NetworkManager[17805]: Sent 986135184 bytes, received<PHONE_NUMBER> bytes.
kwi 20 21:07:48 BigPC NetworkManager[17805]: ioctl(SIOCSIFFLAGS): Operation not permitted (line 2664)
kwi 20 21:07:48 BigPC NetworkManager[17805]: ioctl(SIOCSIFADDR): Operation not permitted (line 2816)
kwi 20 21:07:48 BigPC NetworkManager[17805]: Can't execute /etc/ppp/ip-down: Permission denied

I don't know if that can be related to permission denied while accessing /etc/ppp/ip-{up,down}? There were some other differences in the update patch and I didn't reverse everything. I'll leave it like that and try to also roll back NetworkManager fortisslvpn plugin

Trying to connect directly using openfortivpn also doesn't work and I already installed previous version (1.20.2)
It ends up with pretty much the same result
Peer refused to agree to his IP address
Connect time 0.1 minutes.
Sent 1101 bytes, received 1081 bytes.
ERROR:  Timed out waiting for the ppp interface to be UP.
INFO:   Cancelling threads...
INFO:   Cleanup, joining threads...
Hangup (SIGHUP)
Modem hangup
Connection terminated.
ERROR:  pppd: The link was terminated by the modem hanging up.
INFO:   Terminated pppd.
INFO:   Closed connection to gateway.
INFO:   Logged out.

For me ipcp-accept-remote was already there. A simple downgrade to the version in nixos 22.11: openfortivpn v1.19.0 resolved the issue

Same issue here.
Both of the following workarounds worked for me:

Using overlay with ppp 2.4.9.
Adding ipcp-accept-remote to /etc/ppp/options.

just echoing that for me the solution was also
{
  environment.etc.ppp.options.text = "ipcp-accept-remote";
}

Should be fixed upstream by https://github.com/adrienverge/openfortivpn/issues/1111 in upcoming release 1.20.4.

networkmanager-fortisslvpn  on Arch Linux (extra-testing repo which has been upgraded ppp to 2.5.0 and networkmanager-fortisslvpn has been rebuilt with that) still has the problem, on openfortivpn latest commit (which includes ipcp-accept-remote patch).
Executing openfortivpn directly works though.
There are some commits like this https://gitlab.gnome.org/GNOME/NetworkManager-fortisslvpn/-/merge_requests/27 regarding ppp 2.5.0, but it hasn't made through a tag though.

Just adding ipcp-accept-remote to /etc/ppp/options doesn't seem to be solving it for me.
Now I no longer get the Peer refused to agree to his IP address error, but the Can't execute /etc/ppp/ip-up: Permission denied error.
Downgrading to ppp 2.4.9 isn't an option since the build time is immense.
How do I apply the support-ppp-2.5.0.patch?

Then it seems that the problem might lay elsewhere. Any ideas on how to debug this?
$ cat /etc/ppp/options
ipcp-accept-remote

Logs journalctl -b -u NetworkManager:
Jul 04 13:28:26 nixos NetworkManager[897670]: INFO:   Connected to gateway.
Jul 04 13:28:26 nixos NetworkManager[897670]: INFO:   Authenticated.
Jul 04 13:28:26 nixos NetworkManager[897670]: INFO:   Remote gateway has allocated a VPN.
Jul 04 13:28:26 nixos pppd[897671]: Plugin /nix/store/7hpjmywg9skxfxl8fyzmjflss67f43w5-NetworkManager-fortisslvpn-gnome-1.4.0/lib/pppd>
Jul 04 13:28:26 nixos NetworkManager[897671]: Plugin /nix/store/7hpjmywg9skxfxl8fyzmjflss67f43w5-NetworkManager-fortisslvpn-gnome-1.4.>
Jul 04 13:28:26 nixos pppd[897671]: pppd 2.5.0 started by root, uid 0
Jul 04 13:28:26 nixos pppd[897671]: Using interface ppp0
Jul 04 13:28:26 nixos NetworkManager[897671]: Using interface ppp0
Jul 04 13:28:26 nixos NetworkManager[897671]: Connect: ppp0 <--> /dev/pts/1
Jul 04 13:28:26 nixos pppd[897671]: Connect: ppp0 <--> /dev/pts/1
Jul 04 13:28:26 nixos NetworkManager[897671]: {/tmp/.LE0G71} {/tmp}
Jul 04 13:28:26 nixos NetworkManager[897671]: {/tmp/.LE0G71} {.LE0G71}
Jul 04 13:28:26 nixos NetworkManager[897670]: INFO:   Got addresses: [<IP_ADDRESS>], ns [<IP_ADDRESS>, <IP_ADDRESS>]
Jul 04 13:28:26 nixos NetworkManager[897670]: INFO:   Negotiation complete.
Jul 04 13:28:26 nixos NetworkManager[897671]: Failed to create /etc/ppp/resolv.conf: No such file or directory
Jul 04 13:28:26 nixos pppd[897671]: Failed to create /etc/ppp/resolv.conf: No such file or directory
Jul 04 13:28:26 nixos pppd[897671]: local  IP address <IP_ADDRESS>
Jul 04 13:28:26 nixos NetworkManager[897671]: local  IP address <IP_ADDRESS>
Jul 04 13:28:26 nixos NetworkManager[897671]: remote IP address <IP_ADDRESS>
Jul 04 13:28:26 nixos NetworkManager[897671]: primary   DNS address <IP_ADDRESS>
Jul 04 13:28:26 nixos NetworkManager[897671]: secondary DNS address <IP_ADDRESS>
Jul 04 13:28:26 nixos pppd[897671]: remote IP address <IP_ADDRESS>
Jul 04 13:28:26 nixos pppd[897671]: primary   DNS address <IP_ADDRESS>
Jul 04 13:28:26 nixos pppd[897671]: secondary DNS address <IP_ADDRESS>
Jul 04 13:28:26 nixos pppd[897671]: Can't execute /etc/ppp/ip-up: Permission denied
Jul 04 13:28:26 nixos NetworkManager[897671]: Can't execute /etc/ppp/ip-up: Permission denied
Jul 04 13:28:26 nixos NetworkManager[897670]: INFO:   Interface ppp0 is UP.
Jul 04 13:28:26 nixos NetworkManager[897670]: INFO:   Tunnel is up and running.

And my configuration.nix file:
  environment.systemPackages = with pkgs; [
    networkmanager-fortisslvpn
  ];

Along with system info:
 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.34, NixOS, 23.05 (Stoat), 23.05.1532.aed4b19d312`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.13.3`
 - channels(root): `"nixos-23.05, nixos-unstable"`

I am trying to figure out what else might be causing this problem.
(P.S. IP Addresses have been obfuscated)

Unfortunately, I had to revert the change in 1.20.4 in 1.20.5, because it breaks macOS and NetworkManager-fortivpnssl with pppd 2.4.9, until I can find time to understand and solve the problem.

What exactly doesn't work? Your log shows that the tunnel is up, so I guess routing is broken.
Routing is probably broken because of Can't execute /etc/ppp/ip-up: Permission denied, which raises the question of who or what created /etc/ppp/ip-up. And what is the output of ls -l /etc/ppp/ip-up?
As for message Failed to create /etc/ppp/resolv.conf: No such file or directory suggests directory /etc/ppp might be missing, when it shouldn't be missing.

It might be that the internal DNS servers have been taken into account, but are not reachable, because of a routing issue. Hence all DNS requests are slow, as slow as the DNS client timeout. Are pings to the IP address associated to google.com faster?
I suspect NetworkManager-fortivpnssl attempts to create /etc/ppp/ip-up, but fails. I am afraid I have no clue, except that NetworkManager-fortivpnssl is not correctly installed.
If I had to guess, I'd say NetworkManager-fortivpnssl needs to run as root or run a helper as root, but fails to do so. I don't know enough about NetworkManager-fortivpnssl to help much. It would probably help to run tests on a Linux distribution where NetworkManager-fortivpnssl and pppd are already properly installed, fix any issues there, and then resolve remaining issues specific to NixOS.
Who or what installed NetworkManager-fortivpnssl?

Actually, google.com is a bad example, because it has both IPv4 and IPv6 addresses, and openfortivpn does not support IPv6 (https://github.com/adrienverge/openfortivpn/issues/112):

Only IPv4 traffic is routed through the tunnel.
IPv6 traffic cannot be routed through the tunnel, in your case it's probably (poorly) routed outside the tunnel after some timeout.
Because of that, there's a risk of creating a bridge the protected internal IPv4 network and IPv6 internet.

In any case, experiment with IPv4 addresses to start with.

Hi,
I've added this to the configuration.nix, and issue solved for me.
 environment.etc."ppp/options".text = "ipcp-accept-remote";

Hi, I've added this to the configuration.nix, and issue solved for me.  environment.etc."ppp/options".text = "ipcp-accept-remote";

Hello I've had this in my config ever since this issue came up and it fixed openfortivpn via CLI but not NetworkManager. Can you confirm that this is working in NetworkManager?

Hi, I've added this to the configuration.nix, and issue solved for me.  environment.etc."ppp/options".text = "ipcp-accept-remote";

Hello I've had this in my config ever since this issue came up and it fixed openfortivpn via CLI but not NetworkManager. Can you confirm that this is working in NetworkManager?

Can confirm that this workaround works for me with NetworkManager (VPN configured and started via networkmanagerapplet)

I also debugged this further, therefore it creates bogus routes which lead to a disfunctional tunnel. It creates two routes to the tunnel gateway via the ppp0 interface. If I remove those routes, the tunnel gets functional.
It may be a specific Forti configuration, since @dminuoso and I are working at the same company.

same problem with openfortigui & ppp 2.5.0, under gentoo
all resolved adding into /etc/ppp/options:
ipcp-accept-remote
plugin /lib64/pppd/2.5.0/pppoe.so

Same problem with openfortivpn & ppp 2.5.0, under Fedora 39. This workaround works fine

same problem with openfortigui & ppp 2.5.0, under gentoo all resolved adding into /etc/ppp/options:
ipcp-accept-remote plugin /lib64/pppd/2.5.0/pppoe.so

The new version of openfortivpn 1.21.0 will fix that, at the expense of breaking openfortivpn (by default only) on platforms with pppd < 2.5.0.

Folks. Its better to use openconnect instead of openfortivpn:

CLI way:

echo $vpn_password | sudo openconnect --prot=your-gateway.domain.net:443 -u<EMAIL_ADDRESS>--passwd-on-stdin

GUI way:

Folks. Its better to use openconnect instead of openfortivpn:

CLI way:

echo $vpn_password | sudo openconnect --prot=your-gateway.domain.net:443 -u<EMAIL_ADDRESS>--passwd-on-stdin

GUI way (NetworkManager): chose openconnect

This is working perfectly!
Thanks @sgremyachikh !

Any progress on it?

Folks. Its better to use openconnect instead of openfortivpn:

Indeed after updating to ubuntu 24.10 my work VPN stopped working. I just switched to running that oneliner instead of the broken NM plugin.


Proxy Authentication for Google Speech Api Credential

I am trying to configure google speech api to work in my project within the company's firewall. I have configured the SpeechSettings as following:
InputStream is = this.getClass().getResourceAsStream("/my-service-account.json");
CredentialsProvider credentialsProvider = FixedCredentialsProvider.create(ServiceAccountCredentials.fromStream(is));
SpeechSettings.Builder builder = SpeechSettings.newBuilder();
builder.setTransportProvider(SpeechSettings.defaultTransportProvider());
builder.setCredentialsProvider(credentialsProvider);
SpeechSettings settings =  builder.build();
SpeechClient.create(settings);

Describe the issue
Using this works outside proxy network correctly and starts a speech recognize session. But it fails to transcribe anything under a proxy authenticated network and times out without giving any error. Is it possible to authenticate with the proxy settings while creating the speech client session? Thanks.

Yes, I have tried that solution but that didn't work for me.
The only solution that worked with the proxy issue was setting environment variable: GRPC_PROXY_EXP=":" in the $PATH.

Yes, I have tried that solution but that didn't work for me.
The only solution that worked with the proxy issue was setting environment variable: GRPC_PROXY_EXP="hostname:port" in the $PATH.

this solve my problem， thank you

This issue appears to have been resolved, if that is not the case. Please feel free to reopen. Thanks! :)


Plays same song

When I put different URLs as a parameter of the play function of the plugin, it plays the same song.
So even I tap different songs, my app plays the same song.

It was my mistake. I solved the problem.


[BUG] Container fluid prop doesn't work

Description
There is a problem with the fluid property of the Container component, the problem is that when placing this property, it does not fulfill its function of occupying 100% of the width in all the breakpoints. From 650px it stops occupying 100% of the width.
I tried the solutions said on the discord server like putting gap={0} but that still doesn't solve the problem.
The only solution I found is to place css={{ mw: '100%' }}, but of course, this should be the default behavior when placing the prop fluid
Finally, and this is more of a personal opinion, I think that fluid should be the default property, since in html we are used to containers occupying 100% of the width by default
Codesandbox example
https://codesandbox.io/s/nextui-container-fluid-bug-4v9x7p
Steps to Reproduce the Bug or Issue

Use the Container component
Put the fluid property and add some background color to see the container size
Resize the viewport size above 650px and see how the fluid property is not working

Expected behavior
The container should have 100% width
Screenshots or Videos
Here is a screenshot of the result when we use fluid prop

Does not occupy 100% of the width
Operating System Version
Windows
Browser
Chrome

@aKus-dev
I dont know if its intended or not, maybe in the documentation example it is missing, but that is because the responsive tag is default to true a quick fix using the provided props would be to set it to false
Container fluid responsive={false}
but its most likely to be a bug
maybe this is a fix
main/packages/react/src/container/container.styles.ts
line-7
variants: {     fluid: {       true: {         maxWidth: "100%",         responsive: false,       },

Im trying to use Grid.Contaimer and its need to be fluid but this property doesnt seem to exist. any solution? since Grid is a responsive-true element, above solution wont work for my case.

Hey please upgrade to V2 since V1 will no receive more updates.
Use TailwindCSS container


made --sentencepiece-vocab flag more clear

Before submitting

[x] Was this discussed/approved via a Github issue? (doc change, so not necessary)
[x] Did you read the contributor guideline?
[x] Did you make sure to update the docs?
[x] Did you write any new necessary tests?  n/a

What does this PR do?
Addresses #2319 by making it clear that the --sentencepiece-vocab requires a path to a sentencepiece model.

Thanks! Since there's only a few callers of this, let's just rename from --sentencepiece-vocab -> --sentencepiece-model. I'll make the change before merging.

Fixed by commit ffecb4e3496379edf5ecae1483df5b7e0886c264


Upgrade libsodium to 1.0.12

Benches show that this speeds up crypto_stream_xor_instance by 2x \o/


requires cupy >=9.5.0,<11.0.0a0, but none of the providers can be installed

I tried building the environment (mamba env create -f environment.yml) today on SMSL, and got:
package rapids-22.06.00-cuda11_py38_ged167d1_108 requires cupy >=9.5.0,<11.0.0a0, but none of the providers can be installed

Thanks for the report. When I cloned this repo on SMSL and ran the same command just now things worked as expected.
Could you share the full output from your command, the output of running nvidia-smi at the terminal, and your version of mamba in the base/studiolab environment from which you're running the command?
cc @taureandyernv

and ... as is typical, after I post on an issue I remember something ... I had channel_priority: strict in my .condarc. I recall seeing in the RAPIDS docs that you shouldn't do this, so I commented it out and re-ran the install of the nightly build ... voila, no error!

Hooray!  Thanks for the update @ktyle .  Closing, and hoping others who are facing a similar challenge find this solution :)


Update feature-policy.json

Adding links to:

MS Edge feature suggestion
Feature-Policy Tester (Chrome DevTools Extension)

Thanks!


Adding vertical slider for showcasing Meshery Extension

Description
This pull request fixes the vertical slider instead of the current screenshot. Below is the video representation of the changes made.
https://github.com/meshery/meshery.io/assets/68152307/c4a613be-ff59-4b78-9b79-236c5d8982e8
This PR fixes #1477
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

This screen recording is no longer accurate, right?

The current approach doesn't seem to work in Safari.

Hey @yash37158 Let's discuss this on website call tomorrow at 6:30 PM IST / 7 AM Central time. Please add this as an agenda item in the meeting minutes.

Hey @yash37158 Let's discuss this on website call tomorrow at 6:30 PM IST / 7 AM Central time. Please add this as an agenda item in the meeting minutes.


swagger classifier name

https://github.com/kongchen/swagger-maven-plugin/blob/9ff0630508ba8ff7f391a5119f3f25de6792aa2c/src/main/java/com/github/kongchen/swagger/docgen/mavenplugin/ApiDocumentMojo.java#L121
Hi, the behavior was changed here, and also the documentation is no longer in line with the code:
Documentation:
attachSwaggerArtifact | If enabled, the generated swagger.json file will be attached as a maven artifact. The swaggerDirectory's name will be used as an artifact classifier. Default is false.
However, we have specified both the artifactFilename and artifactDirectory. However, now that we stepped our swagger version it no longer takes the artifactDirectory name as classifier.
We suggest to add an additional parameter  and if that one is not set, default back to the old behavior (e.g. artifactDirectory).

thank you!


Fix portrait tree drag drop bugs

Maybe Fixes: #2372
Added check to make sure something is selected when trying to create drag preview in the Character editor's Portrait tree. Also renamed some function arguments to avoid shadowing builtin position variable.

I'm actually wondering if instead of using get_selected() if we should check for an element under the position and get that. After testing this fix, if you have an item selected, but you drag in the empty space at the bottom, it can reorder that item, which is likely not intended, or expected by the user.

Not sure if it fixes #2372, but the changes look good.


Conversion from reader spreads to printer spreads

Description
Typst has page order for reader. Known as reader spreads. If I want to print a booklet with eg. 18 pages I need a printer spread.
See also https://smartpress.com/support/printing-basics/readers-spread-vs-printers-spread#printer-s-spread
Use Case
Very useful for printing small booklets on ordinary printer at home.

Duplicate of https://github.com/typst/typst/issues/187