Replace time with std::time

ping! the skybox example doesn't compile with this change (look at the Travis log)

Fixed now. Sorry for commit spam; not on my home dev machine

Thanks! Do you mind if I squash it?

On Jul 11, 2016, at 12:41, Demur Rumed<EMAIL_ADDRESS>wrote:
Fixed now. Sorry for commit spam; not on my home dev machine
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Please do

Ok, just waiting for AppVeyor...

Thanks!


Fix indexing to be zero-based

Addresses issue raised by @jukent in https://github.com/jhamman/xsd/pull/10

@jhamman, when you get a chance can you take a look at this?

Is this because it kept looking for the 100th index, which did not exist? So you're telling it to check the last index instead.

Is this because it kept looking for the 100th index, which did not exist? So you're telling it to check the last index instead.

Because Python indexing is zero-based, the array had 100 elements in it, and it was trying to access a[100] which is the 101th element.

Is this because it kept looking for the 100th index, which did not exist? So you're telling it to check the last index instead.

Because Python indexing is zero-based, the array had 100 elements in it, and it was trying to access a[100] which is the 101th element and this should have been a[99] instead.

Makes sense. Thank you.

I'm going to merge this now so we can test @jukent's #10 PR. I'll plan to come back to this soon.


feat(protocol-parser): add encodeKeyTuple

add util to encode keys tuples (inverse of decodeKeyTuple)

Current dependencies on/for this PR:

main

PR #1096

PR #1097

PR #1074

PR #1099   👈

PR #1100

PR #1098

This comment was auto-generated by Graphite.

merging since nothing changed since last approval


More robust state management

As the project is getting more complex, we might want to consider those issues:
Use a proper state management library like:

[ ] Recoil
[ ] Redux

Because right now, using React contexts is fine but we might get into trouble later with unnecessary re renders : https://blog.logrocket.com/pitfalls-of-overusing-react-context/
For now we have this:
<SnackbarProvider maxSnack={3}>
  <ProvideGraphQLClient>
    <ProvideSocket>
      <ProvideAuth>

        {/* ... */}

      </ProvideAuth>
    </ProvideSocket>
  </ProvideGraphQLClient>
</SnackbarProvider>

Using react context is working great for now


Review SMD Cost Model and Develop Additional Requirements

💡 Description
Per cost model review from B15.0, let's come up with some new requirements for SMD to have them enhance the cost models for us.
⚔ Parent Epic / Related Tickets
No response

📆 11/2024 status: Started.

@viviant100 FYI, this is next up for our cloud tasks.


util/signature: add WalletConnect API support

Blocked by https://github.com/nspcc-dev/neofs-api-go/pull/386
Related #155 .
Signed-off-by: Evgenii Stratonikov<EMAIL_ADDRESS>

@fyrchik lets change commit header to crypto: ...?


Add Missing sentences in Arabic for bulk actions

Add Missing Arabic words for bulk actions
WHY
BEFORE - What was wrong? What was happening before this PR?
Missing sentences in Arabic
AFTER - What is happening after this PR?
Is shown Now
HOW
How did you achieve that, in technical terms?
Added it
Is it a breaking change?
no
How can we test the before & after?
change local to ar and check bulk actions crud will be shown as missing lang like this

Thanks @EGYWEB-Mohamed !

Will be available next Monday with the next patch release.


Manage ssh key as secrets

As a pull request owner and reviewers, I checked that:

[X] Appropriate testing is done and actually running
[X] Appropriate documentation exists and/or is up-to-date:

[X] README in the role
[X] Content of the docs/source is reflecting the changes

/hold cancel

recheck

recheck

/lgtm


Importing from XCode 10.0 Instruments fails

The code to import from instruments was tested against Xcode 8 & 9, but it seems like something in 10 broke it.

@aparrapo I don't intend to install Xcode 10 at the moment, but if you'd like to help this process along, you can record a simple trace of a test program in this respository using Instruments.
To do that, clone this repository then do:
cd sample/programs/cpp
make simple
./simple

Then find that process in Instruments and record a time profile of it. If you could do that and send the file, then that would accelerate this process since I don't have Xcode 10 installed & don't want to mess up my dev environment.

There you go :)
Instruments_sample.trace.zip
I tested this trace and I get the same message, so it should be good for debugging purposes.

@aparrapo Thanks! 😄
I probably won't get to this in the next week or so, but I'll try to find some time to take a look. If you are interested in contributing, I did a writeup of how I dug around these files in the past: http://jamie-wong.com/post/reverse-engineering-instruments-file-format/
Let me know if you're interested in taking this on yourself, otherwise it might take a while, though certainly not as long as if I had to upgrade Xcode 🙂

Hi @aparrapo I just unzipped Insturments_sample.trace.zip that you attached and I was able to import the contents into https://www.speedscope.app/. I also tried generating a multithreaded profile recording my copy of Visual Studio Code or 30 seconds, and that was able to import as well.

I'm going to close this for now since I can't reproduce the problem. If you are able to create a small example of a profile that won't import, please let me know!


Update MD file on new branch 2017-21-14_01-11-13

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon.md
:white_check_mark:Succeeded

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


GraphControl.Disconnect fires ConnectionRemoved before removing the connection

This should be two events:

ConnectionRemoving should fire when the current Removed event fires and have the Cancel option
ConnectionRemoved should fire after the connection has been disconnected.  The eventArgs should give the From and To NodeConnector objects.

Here's what I did:
NodeConnection.cs:
public sealed class NodeDisconnectionEventArgs : EventArgs
{
public NodeDisconnectionEventArgs(NodeConnector from, NodeConnector to)
{ From = from; To = to; }
public NodeConnector From { get; private set; }
public NodeConnector To { get; private set; }
}
GraphControl.cs:
public event EventHandler ConnectionRemoving;
public event EventHandler ConnectionRemoved;
    public bool Disconnect(NodeConnection connection)
    {
        if (connection == null)
            return false;

        if (ConnectionRemoving != null)
        {
            var eventArgs = new AcceptNodeConnectionEventArgs(connection);
            ConnectionRemoving(this, eventArgs);
            if (eventArgs.Cancel)
                return false;
        }

        if (HasFocus(connection))
            FocusElement = null;

        var from = connection.From;
        var to = connection.To;
        if (from != null && from.Node != null)
            from.Node.connections.Remove(connection);
        if (to != null && to.Node != null)
            to.Node.connections.Remove(connection);

        if (ConnectionRemoved != null)
        {
            var eventArgs = new NodeDisconnectionEventArgs(from, to);
            ConnectionRemoved(this, eventArgs);
        }

        // Just in case somebody stored it somewhere ..
        connection.From = null;
        connection.To = null;
        return true;
    }

Thanks for the suggestion! It's implemented now.


Split spatial example

Adds new audio_control example to make it easier to digest the spatial example.
Starts the audio immediately in the spatial example to avoid confusion.
Closes #36

This is also the case on main and is not related to this PR. I've opened issue #44 and I already know a workaround.


Add clear documentation to help users with IAM policy and creds requirement for RI policies

Policy Template Name
As per the discussion on slack, we should make sure RI policies README have sufficient information for a noob user to get started. The discussed solution was to leverage CM creds along with IAM access needed for RS to fetch RI purchase and coverage info from the CE API.
https://github.com/rightscale/policy_templates/tree/master/cost/aws/reserved_instances
Issue Type
[Bug -or- Feature Request -or- New Policy Template Request]
Use Case (Feature Request -or- New Policy Template Request Only)
[Describe the use case for this feature request or new policy template request]
Policy Template version (Bug Only)
[Version of Policy Template in use, this can be found in the long_description]
Code Snippet (Bug Only)
[Please provide any relevant code snippets (ie. datasource,escalation,policy, RCL, etc.) as it relates to the bug]
Error Messages (Bug Only)
[Provide any error messages returned (if applicable)]

Tracked Internally as POL-66


Prevent duplicate test runs on new PRs

As previously commented I don't see why this is necessary, can you show me an instance of duplicate runs?

An example:

One push, one merge, yet 3 CI runs. There would only be 2 if this commit was applied.


Request: Please add support/templating for dapr.io

FIrst of all, thank you for creating such a wonderful, feature-rich cookiecutter template.
I have a use case where I want to use dapr.io and its Python SDK so I can use a uniform API for pub/sub, DB, caching etc without being concerned about the vendor for each of those components.
As a request - would you be able to add support for generating a FastAPI template using the dapr Python SDK?

Hi and thanks for warm words. The problem is that I haven't worked with dapr.io and therefore I'm not sure how to implement it. Can you please create a sample project that does the thing you've mentioned or I'd really appreciate if you could create a PR with required changes.


Add a Credential card

Hi,
When I try to download and add a credential card in my existing wallet.
I got an error with "This request is not supported".
After a complete reset of the app and use a new wallet i'm able to add the credential card.
I think it's not usual that I need to reset the complete wallet before I can do this.
Before it was working normal, did there something change why this is needed?
Version: 2.4.21 iOS

No there was no reason for that reset. Let us know if you still have this issue.
It may happen sometimes that the QRcode reader keep some previous data but usually we only need to close the app and reopen.

is that ok ? @jeroenbranje


User is unable to paste to upload a video into editor on Chrome / MacOS

Bug Description
As noted by @csossi  in https://github.com/google/web-stories-wp/issues/3348 when the user on Chrome/Mac attempts to paste a video into the editor they are presented with the name of the video file instead of the video.

Expected Behaviour
The asset should be presented in the editor and added to their media library
Steps to Reproduce

Screenshots

Additional Context

Plugin Version:
Operating System: Mac OSX
Browser: Chrome

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

Implementation Brief

Can't reproduce this right now


Add ability to read annotated heartbeats and subject data from MESA

In addition to my inline comments, I noticed that you import pyedflib directly in test_sleep_readers.py. Is this intentional? It is not a hard dependency so maybe you should import it within the test function? Or do you think that's OK because these are just the tests?

Or do you think that's OK because these are just the tests?

^ this

OK, works for me. You added new parameters, can you also update the corresponding docstrings?

oh I messed up the staging in cb2b9b3, will revert

fixed


No UI tests & production release ready

There are no UI tests yet, which is a plus point.
https://github.com/steve1rm/BusbyNimbleSurvey/tree/master/app/src/androidTest/java/me/androidbox/busbynimblesurvey
https://github.com/steve1rm/BusbyNimbleSurvey/tree/master/presentation/src/androidTest/java/me/androidbox/presentation

There is no production release config yet (proguard rules set), which is a plus point.
https://github.com/steve1rm/BusbyNimbleSurvey/blob/master/app/proguard-rules.pro

@luongvo I have added UI screen tests here https://github.com/steve1rm/BusbyNimbleSurvey/pull/24

@luongvo PR for release config https://github.com/steve1rm/BusbyNimbleSurvey/pull/25


hourglass_challenge_java

Input : 1 1 1 0 0
0 1 0 0 0
1 1 1 0 0
0 0 0 0 0
0 0 0 0 0
Output : 7
Below is the hour glass with
maximum sum:
1 1 1
1
1 1 1
Input : 0 3 0 0 0
0 1 0 0 0
1 1 1 0 0
0 0 2 4 4
0 0 0 2 4
Output : 11
Below is the hour glass wuth
maximum sum
1 0 0
4
0 2 4

Hey, I would like to work on it. Please assign this to me

hey i am new to this community can you please assign me this

if row and column is specific then i have the solution


AdvancedKea Error

Hello, I am a big fan of your work, I downloaded the advanced kea because the old one is giving me an error. I followed all direction and I received the same unhandled exception error saying too many request,. Please help. Thank you.

Yep, another of the same issue I reported last October 2023 here: https://github.com/Anything144/Advanced-Kea/issues/1
Seems like everyone is having the same problem with "too many requests".


createTable - Required parameter is missing

I'm creating a table with an existing dataset like this:
var bigquery = gcloud.bigquery({
  projectId: 'ionic-stats-storage',
  keyFilename: './bigquery.json'
});

var cliDataset = bigquery.dataset('CLI');
cliDataset.createTable(type, {
        schema: {
          fields: [
            { type: 'string', name: 'type' },
            { type: 'string', name: 'service' },
            { type: 'string', name: 'uuid' },
            { type: 'timestamp', name: 'created_at' },
            { name: 'data', fields: fields }
          ]
        }
      }, function(err, table, apiResponse) {
...
});

This is the payload being sent to the /tables endpoint as I inspected it in the gcloud node library (pretty-printed):
{
  "schema": {
    "fields": [
      {
        "type": "string",
        "name": "type"
      },
      {
        "type": "string",
        "name": "service"
      },
      {
        "type": "string",
        "name": "uuid"
      },
      {
        "type": "timestamp",
        "name": "created_at"
      },
      {
        "name": "data",
        "fields": [
          {
            "name": "favorite_flavor",
            "type": "string"
          }
        ]
      }
    ]
  },
  "tableReference": {
    "datasetId": "CLI",
    "projectId": "ionic-stats-storage"
  }
}

The API is returning a vague error about a missing required parameter, but doesn't say which one, and I believe I'm using the node API correctly (very possible I'm not). Note: I also tried removing the data record in case I wasn't doing that properly, but I receive the same response:
{ ApiError: Required parameter is missing
    at new util.ApiError (/Users/max/git/ionic-track/node_modules/gcloud/lib/common/util.js:106:10)
    at Object.parseHttpRespBody (/Users/max/git/ionic-track/node_modules/gcloud/lib/common/util.js:184:30)
    at Object.handleResp (/Users/max/git/ionic-track/node_modules/gcloud/lib/common/util.js:124:18)
    at /Users/max/git/ionic-track/node_modules/gcloud/lib/common/util.js:447:12
    at Request.onResponse [as _callback] (/Users/max/git/ionic-track/node_modules/retry-request/index.js:120:7)
    at Request.self.callback (/Users/max/git/ionic-track/node_modules/request/request.js:187:22)
    at emitTwo (events.js:106:13)
    at Request.emit (events.js:191:7)
    at Request.<anonymous> (/Users/max/git/ionic-track/node_modules/request/request.js:1044:10)
    at emitOne (events.js:96:13)
  code: 400,
  errors:
   [ { domain: 'global',
       reason: 'required',
       message: 'Required parameter is missing' } ],
  response: undefined,
  message: 'Required parameter is missing' }

I believe this should work:
cliDataset.createTable(type, {
  schema: {
    fields: [
      { type: 'string', name: 'type' },
      { type: 'string', name: 'service' },
      { type: 'string', name: 'uuid' },
      { type: 'timestamp', name: 'created_at' },
      {
        name: 'data',
        type: 'record', // <-- new
        fields: [
          {
            type: 'string',
            name: 'favorite_flavor'
          }
        ]
      }
    ]
  }
}, function(err, table, apiResponse) {
  // ...
});

Okay, I figured out what I was doing wrong here. First, my table id was undefined since I was sending incorrect data to my API. However, in the process of tweaking things and trying to get this to work, I tried using the id sub record as per #954 which does not seem to work yet. Additionally, sub-records need the 'record' type but using them is not documented in the API docs anywhere. Making those changes works now 👯

Yeah, I think we can make this easier. We try to focus on the common use cases and decorate them to make things easier. This leaves holes where we link to the upstream API docs to show all of the options (https://cloud.google.com/bigquery/docs/reference/v2/tables#resource) from our docs (https://googlecloudplatform.github.io/gcloud-node/#/docs/v0.36.0/bigquery/dataset?method=createTable).
Making this easier should be pretty... easy. I'll re-open this issue and ping you when I have a PR, if you don't mind. Thanks for calling this out!

Sounds great, thanks!


Add CLI option to isort to output diff of proposed changes.

It took me a while to dig down to the fact that my global Python3 interpreter has a different version from the tox one, which leads to all sorts of weird default isort behaviour.
With this extra flag, Travis will tell you precisely which imports isort would like to see changed.

Coverage remained the same at 100.0% when pulling e493bfd8eeaff450ea7f1c09b31adbdcac94abd9 on jfinkhaeuser:improve-isort-output into f8c7a6885bca11c53d50abc1216f31ec0d028be5 on zalando:master.

:+1:


Syntax error in ie11 with nextjs

Syntax error in ie11
-> class Dom7 ....

Same issue here

Same errors:

SyntaxError: Use of reserved word 'class'

Error: Syntax error

class Dom7 {

yep -> https://github.com/kidjp85/react-id-swiper/issues/259

@rossenv Seems to be a bug in the Swiper package. See https://github.com/nolimits4web/swiper/issues/3047

So, you wouldn't believe it but the "Swiper" library author @nolimits4web silently deleted my bug report from his repository: https://github.com/nolimits4web/swiper/issues/3047
Very childish, rude and unprofessional.
I guess I'll have to look for better alternatives to the "Swiper" library.
In the meantime, for everyone having their Internet Explorer broken by the "Swiper" library here's the workaround I found that stops the "Swiper" library from breaking Internet Explorer:

Go to Webpack configuration (webpack.config.js).
Find babel-loader there.
Modify its exclude from /node_modules/ (the default on the internet) to /node_modules[\/\\](?!(swiper|dom7)[\/\\])/ (I saw the idea in react-id-swiper's package.json).
Now Webpack bundles the website while also actually transpiling dom7 and swiper packages (doing the work the library author should have done before releasing those untranspiled).

Internet Explorer is still a popular browser in some areas.
Breaking it is unacceptable.

Any update on this issue except for this hotfix?

Thanks @catamphetamine for your guide to making react-id-swiper work with IE11. Do you have a working example I could take a look at (or codesandbox?) We tried to implement your suggested changes in a next.js SSR project but somehow could not get it working in IE11. Any help would be much appreciated!
Also @kidjp85 is there a suggested fix you can recommend in order to support IE11 with react-id-swiper? If so, I'd be happy to hear, thanks a lot.
Best,
Stefan

Thank you @catamphetamine . Same issue here - syntax error because the Dom7 class isn't transpiled in the Swiper library. E.g. with fix: (using TypeScript here so the @babel/typescript preset is unnecessary if you're not using it)
rules: [       {         test: [/.js$|.ts$/],         exclude: /node_modules[\/\\](?!(swiper|dom7)[\/\\])/,         use: {           loader: 'babel-loader',           options: {             presets: [               '@babel/typescript',               '@babel/preset-env'             ]           }         }       }]

Tried this, getting awkward error saying
configuration.module.rules[41].exclude[1]: The provided value "/node_modules[\\/\\\\](?!(swiper|dom7)[\\/\\\\])/" contains exclamation mark (!) which is not allowed because it's reserved for loader syntax.
Does anyone have this working?

Made a new issue here: https://github.com/nolimits4web/swiper/issues/3199 I'd encourage you to join to prevent a deletion again.

Just got closed and locked. :/

Another issue regarding this posted here : https://github.com/nolimits4web/swiper/issues/3129

same problem. I tried @catamphetamine solution, but it won't work. Is there any solution for this problem? it's really important because we want to launch the page asap.

@Antonio-Laguna ty for the help, but it seems like ur using laravel because of mix.js? I don't know how to use mix...

Yeah, that's why I added a link to the custom alias on webpack:
https://webpack.js.org/configuration/resolve/#resolvealias
Sorry if it wasn't clear. mix.alias is just a wrapper around resolve.alias

@Antonio-Laguna sorry I'm not that familiar with webpack.
module: {
    rules : [
      // JavaScript/JSX Files
      {
        test: /\.(js|jsx)$/,
        exclude: /node_modules[\/\\](?!(swiper|dom7)[\/\\])/,
        use: ['babel-loader'],
      }
    ]
  },

  resolve: {
    alias: {
      'swiper': 'node_modules/swiper/dist/js/swiper.js',
      'swipercss': 'node_modules/swiper/dist/css/swiper.min.css'
    }
  },

I get this error:

Do you have swiper installed? Is it on your package.json?

yes it's in the package.json and the files are also in the node_modules directory.

Hi guys, i still cannot fix this...
According to react-id-swiper document, I have added this into the webpack.config.js. I believe this is to use babel to include dom7 and swiper module and transpile it into es5 code
{
    test: /\.js(x)?$/,
    exclude: [/node_modules\/(?!(swiper|dom7)\/).*/, /\.test\.jsx?$/],
    use: [{ loader: 'babel-loader' }],
},

But after npm run build, I can still find const syntax inside the .js file in dist folder... I can confirm the syntax is came from the swiper module
This cause old browser like safari9 or IE10 display error as below
SyntaxError: Use of const in strict mode.

@tohbansoon - hopefully this helps https://github.com/kidjp85/react-id-swiper/issues/332

@tohbansoon - hopefully this helps #332
thanks @RyanGosden ... i end up by downgrading react-id-swiper version back to 1.X.X
Will try your solution again once I have free time to upgrade it ...

Just for the record, we're compiling TypeScript with webpack and we have a loader.js with the following:
// Typescript loader
const ts = {
  test: /\.(tsx?|jsx?)$/,
  exclude: [
    /node_modules\/(?!(swiper|ssr-window|dom7)\/).*/,
    /.out/,
    /.storybook/,
  ],
  use: [
    {
      loader: 'babel-loader',
      options: {
        configFile: `${__dirname}/babel.config.json`,
      },
    },
  ],
};

This now transpiles the swiper, ssr-window and dom7 modules and swiper is now working in IE11, with the polyfills working too.
We're using the following versions:
    "swiper": "^5.4.2",
    "babel-loader": "^8.0.6",
    "webpack": "^4.42.1",

Note: it took me a while to notice that the 'test' line needed to include .js files as well as .ts files now that we are including certain node_modules, as the rest of our code is .ts files.
  test: /\.(tsx?|jsx?)$/,

Thanks to averyone for sharing comments and suggestions, it helped!


Plugin support for using modules within an IDE's project

I don't think this is directly related to #9. I am using IntelliJ and I have created a flutter "module" inside my "project". For example, the folder structure is like this.
my_project/
  misc_folder/
  my_flutter_module/
    pubspec.yaml
    lib/
      generated/
      l10n/

I could only get the Tools menu to appear when I used IntelliJ to create the Flutter module. If I created the project using flutter create, it doesn't recognize that it is in a module and the menu does not appear under Tools menu.
When I go to use Add Locale it adds the ARB in the root of the project, not relative to the current module. When I go to remove a locale, it doesn't find it.

This sounds more like a Flutter issue; please refile this there.


Error: line 61: regex error 14 for `^[[:space:]]class[[:space:]]+[[:digit:][:alpha:]:_]+[[:space:]]{(.[ ])*}(;)?$', (failed to get memory)

Running this script
const mmmagic = require("mmmagic");
const M = new mmmagic.Magic();
M.detectFile("foo.js", (err, res) => {
  console.log(err);
  console.log(res);
  process.exit(0);
});

on this file
class foo { }
test = new abcde();

results in the error
Error: line 61: regex error 14 for `^[[:space:]]*class[[:space:]]+[[:digit:][:alpha:]:_]+[[:space:]]*\{(.*[
]*)*\}(;)?$', (failed to get memory)

I can't tell what's causing this, but it seems sensitive to the length of the strings. For example, if test is bar, it doesn't have an issue.
No issues running file (5.32) on the above file. Tested with mmmagic 0.4.6, node 8.9.4, gcc 5.4.0.

Can you try the master branch (npm install https://github.com/mscdex/tarball/master)? I think this should be fixed now.

Seems to be working great. Thanks for the quick fix!

Thanks for the confirmation!


VBM pre-processing

Run VBM pre-processing on ~750 subjects that passed freesurfer QC using coinstac sim on the VACC cluster.
Current status:
Ross will show Anthony how to run this on the cluster. It will take a week to run this. The next step is to run VBM regression on the swc1*.nii from this.
Update the latest status here as comments on this issue:
If its an issue, then who is working on it and brief context of the issue.
2)Post when we get usable results after confirming with Vince.

VBm pre is running. Will finish next week.


Api key not found error

Amplify CLI Version
12.1.1
Question
I have changed the graphql schema and trying to push changes.
errror shown is:
Resource is not in the state stackUpdateComplete
Name: GraphQLAPIDefaultApiKey....... (AWS::AppSync::ApiKey), Event Type: update, Reason: API key not found: ....... (Service: AWSAppSync; Status Code: 404; Error Code: NotFoundException; Request ID: a3c84959-6c73-42aa-a290-b119ae17f2b9; Proxy: null), IsCustomResource: false
I updated api and set expiry date as 325
I added "APIKeyExpirationEpoch" as -1 in parameters.json
Both the above two approaches did no work.
What might be the reason for this. How to fix this.

Hey @RZ-Jyothsna, 👋 thanks for raising this! I'm going to transfer this over to our API repository for better assistance 🙂


Add update compose command

Fix #1622
Requirements:

[ ] https://github.com/updatecli/updatecli/pull/1605

This pullrquest add the support for a new configuration that allows to "compose" multiple policies.
A compose file looks like
update-compose.yaml
# Policies defines a list policies to enforce when running
# `updatecli compose apply`
# `updatecli compose diff`
# `updatelci compose show`
#
configs:
  default:
    config:
      - updatecli.d
    values:
      - values.yaml
    secrets:
      - secrets.yaml

policies:
  - policy: ghcr.io/olblak/policies/updatecli:latest

    # The following policies are just examples and won't work as is
  - policy: ghcr.io/olblak/policies/golang:v2.1.1
  - policy: ghcr.io/olblak/policies/readme:v0.1.3

This file can be used using one of the three following commands

updatecli compose diff : similar to updatecli diff but for multiple policies at once
updatecli compose apply: similar to updatecli apply but for multiple policies at once
updatecli compose show : similar to updatecli show but for multiple policies at once

Test
To test this pull request, you can run the following commands:
go build -o bin/updatecli
./bin/updatecli compose diff -f update-compose.yaml

Additional Information
Tradeoff

Potential improvement

Generating a jsonschema and publish on the jsonschema store

Remark, in the initial idea, I was considering to use a map for the policies but after additional thoughts I think an array is better


RantEngine global objects are cleared each time the Sandbox is run

I'm working on a project that implements Rant with some preset variables, and I noticed that when I tried to set them with RantEngine's indexer, they weren't available in the Rant scripts I ran. Looking further, it seems that the current objects set for a given sandbox is cleared at the start of every run.
https://github.com/TheBerkin/Rant/blob/ca68f6526610335ab553938c649e780e5910347c/Rant/Engine/Sandbox.cs#L225
I made a quick hack to fix it by replacing that line with this:
_objects = new ObjectStack(_engine.Objects);

and making _objects not readonly, but there's probably a better way of doing this.

Thanks, looking into it now.

The global object table was being cleared on every sandbox run, for some odd reason. I've removed the offending line of code and objects persist properly.
There are still some issues being sorted out with list variables not being fetched from the globals, though, so just be aware of that. We'll hopefully have a fix for that soon as well.


rectangles: Add new large test

This was added to x-common in
https://github.com/exercism/x-common/pull/288
It originated from the Lua track. Neither this track nor the Python
track had it prior.

Does this hit an edge case that the previous tests did not? I haven't done this problem yet, so I'm not familiar with it.

The main thing I see it providing is just checking that the solution isn't terribly inefficient

The main thing I see it providing is just checking that the solution isn't terribly inefficient

Then again, I'm not sure how good of a job it does at that either, it's not like there's a timeout built in (other than the student's patience). I may intentionally code an inefficient solution to see how it performs. Probably something like scan every pair of points regardless of whether they're corners

How do you encode that in test suite, though? I'm always reluctant to test for performance in these sorts of things. I like tests that prove the code works, and performance is too subjective to fit into unit tests.
Maybe performance tests like this one (and the large vec ones in sublist, etc.) should be made optional? With an explanation of what they are there to prove. Then students that want to really focus on performance can do so.

My report is that even with an implementation that is O(h^2w^2) (h for height and w for width), cargo test is still passing in less than half a second, so even this test is not showing any performance problems. Maybe in interpreted languages... certainly not here. Would have to ask Lua track if that is a concern when adding that test.
Maybe it would reveal problems for solutions that don't cache corner/edge positions (indexing into a string is not constant time!), but I don't seem to have the time right now to make my solution even more inefficient to test the limits of my own patience.
The comment about test intent is good - all other cases in https://github.com/exercism/x-common/pull/288 are named to give an idea of their intent, but this one's just named "large test", leaving all to wonder "OK so it's large. So what?"

With an explanation of what they are there to prove.

Now would be a good time to look at https://github.com/exercism/x-common/pull/289 and see whether it is explained sanely there, and think about how that will translate to an auto-generated test file when this track auto-generates tests. Is simply putting it in the test description sufficient? Should there be a student-visible comment included with the case? etc.

https://github.com/exercism/x-common/pull/289#issuecomment-233128518


Updates the reconcile logic to use storageDevicePVC for mons

This commit updates the reconcile logic to use storageDeviceSet PVC template for mons if no monPVCTemplate is provided.The reconcile flow after this commit is as follows.

If both MonDataDirHostPath and monPVCTemplate is provided, the mons will be provisioned on the provided monPVCTemplate and monDataDirHostPath.
If only the monDataDirHostPath is provided, the mons will be
provisioned on the provided monDataDirHostPath.
If only the monPVCTemplate is provided, the mons will provisioned
on the provided monPVCTemplate.
If no monPVCTemplate and monDataDirHostPath is provided, the mons will be provisioned using the PVC template of first StorageDeviceSets if present.

Signed-off-by: Ashish Ranjan<EMAIL_ADDRESS>

I don't like the proliferation of booleans in the API. We should just be able to use a mon hostPath if:

A hostPath is configured
No PVC template is specified

Why do we need this?

/retest

2. No PVC template is specified

we need this because the changes required for enabling the hostPath for mons is not present in 4.3's UI :(

Leave the default behaviour as-is so UI is not affected (before the mon's on hostPath patch)
UI does not specify monDataDirHostPath

so monsOnHostPath = monDataDirHostPathExists && ! monPVCTemplateExists

Leave the default behaviour as-is so UI is not affected (before the mon's on hostPath patch)
UI does not specify monDataDirHostPath

so monsOnHostPath = monDataDirHostPathExists && !monPVCTemplateExists

updated the PR. @rohantmp can have a relook on this.

Leave the default behaviour as-is so UI is not affected (before the mon's on hostPath patch)
UI does not specify monDataDirHostPath

so monsOnHostPath = monDataDirHostPathExists && !monPVCTemplateExists

updated the PR. @rohantmp can have a relook on this.

cc @jarrpa @travisn

So if there's no UI change, why is there a Boolean the UI won't use?

So if there's no UI change, why is there a Boolean the UI won't use?

I removed the boolean variable

/retest

/retest

@jarrpa can you rereview this ?

/retest

/retest

/retest

/retest

/retest


Issue mapping 23H2

Getting this issue

I saw that it's in the readme, I already have anti-virus disabled, CFG, Exploit Protection, as well as core isolation and memory integrity disabled.

Just reinstalled windows an hour ago, KDMapper runs on another PC perfectly fine with same version, as well as it was working fine with the default .sys before PC reset.

You must disable vulnerable driver list from registry (if option don't appear in your gui)

Im having the same problem. i have disabled it from the registry and every other anti virus and i get this
[<] Loading vulnerable driver, Name: AUIuGHjkqBbZ
[+] NtLoadDriver Status 0xc0000603
[-] Failed to register and start service for the vulnerable driver


the text of the navbar is colliding with page which looks distorted

https://www.kubernetes.dev/

Scroll down to see the effect
assign me i will solve this issue

You don't need to be assigned!  Just submit a PR for CSS to fix it.  Thanks!

I think this issue can be closed now....because this issue not persist now!


Mouse Without Borders Touch screen issue

Microsoft PowerToys version
0.78.0
Installation method
GitHub
Running as admin
No
Area(s) with issue?
Mouse Without Borders
Steps to reproduce
Move mouse onto other machine, use touch screen to touch primary computer
✔ Expected Behavior
Shouldn't do anything on other machine when using touch screen on primary machine
❌ Actual Behavior
when using touch screen on primary machine and mouse has been moved onto other machine, it will think you are pressing mainly the bottom right of the screen but sometimes other areas of the screen (show desktop button) on the other machine.
Other Software
No response

Hi @allycatman101, thanks for the report!
I couldn't reproduce the issue with MS Surface arm64/win10 + Win11 desktop. What env do you have? Also, please send bugreport files from both of your machines.
/needinfo


⚠ GitLab Server has degraded performance

In b9707c4, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 11455 ms

Resolved: GitLab Server performance has improved in 2c46383.


Added more player logic and cropped player image

Going to later use the created variable scale for collision detection!

next time add some comments in the code ok thanks


Improvement: Array function plugin confusing to (newer) user(s)

When an array function is created  it's not obvious that the user really should setup a read variable for the index value.  perhaps that should automatically be created when the new node is created.
Is there a use face for an array function to not read this value?

Thanks Eric, +1.
See also #162


Initial release of tutorial for basic usage of DynPy's reporting module

The essential information about reporting module for DynPy are to describe. The guide has to contain basic information on how to create a simple text document using the basic classes, i.e. reporting modules such as ReportText, Markdown or Picture.
Similar report can be found in #44 .

Done, can be closed.


[BUG] : miss matching font size in mobile view

✏ Describe the bug
the font size is not same in the mobile view i.e below 700px width
📸 Screenshots

i would like to work on this


Add PsDscRunAsCredential parameter to example xSQLServerNetwork

Pull Request (PR) description

This Pull Request (PR) fixes the following issues:

Task list:

[ ] Change details added to Unreleased section of CHANGELOG.md?
[ ] Added/updated documentation, comment-based help and descriptions in .schema.mof files where appropriate?
[ ] Examples appropriately updated?
[ ] New/changed code adheres to Style Guidelines?
[ ] Unit and (optional) Integration tests created/updated where possible?

@pesetskyps,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.
It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

Codecov Report

Merging #840 into dev will decrease coverage by <1%.
The diff coverage is n/a.

@@         Coverage Diff         @@
##            dev   #840   +/-   ##
===================================
- Coverage    96%    96%   -1%
===================================
  Files        31     30    -1
  Lines      3301   3245   -56
===================================
- Hits       3184   3128   -56
  Misses      117    117

Thanks for sending in this PR. :smile: You sent in the PR towards the master branch, I switched it to the dev branch.
You need to rebase your branch against the dev branch. Becuase you are missing entries in the change log. Please see Resolve Merge Conflicts here to see how that is done. Please use git rebase (and do not use git merge or git pull).

Review status: 0 of 2 files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

Examples/Resources/xSQLServerNetwork/1-EnableTcpIpWithStaticPort.ps1, line 28 at r3 (raw file):

            TCPPort         = 4509
            RestartService  = $true
            PsDscRunAsCredential = $SysAdminAccount

Could you please add the same for the other example as well?
You don't have to create a new PR, just push changes into your working branch in your fork, and the PR will update automatically.

Comments from Reviewable

@pesetskyps Please sign the CLA for me to be able to merge these changes (once the review comments are resolved).

@pesetskyps, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, Microsoft Pull Request Bot

signed in cla, pushed edit for the second example, rebased (not sure if it was corrent, needed to use the git push -f in the end)
Let me know in case of any issues, thanks

Review status: 0 of 3 files reviewed at latest revision, 1 unresolved discussion.

Examples/Resources/xSQLServerNetwork/1-EnableTcpIpWithStaticPort.ps1, line 28 at r3 (raw file):
Previously, johlju (Johan Ljunggren) wrote…
Could you please add the same for the other example as well?
You don't have to create a new PR, just push changes into your working branch in your fork, and the PR will update automatically.

Done.

Comments from Reviewable

Reviewed 1 of 2 files at r3, 2 of 2 files at r5.
Review status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

CHANGELOG.md, line 4 at r5 (raw file):

## Unreleased
- Changes to xSQLServerNetwork

Please add your entries to the end of the Unreleased section.
And please make sure to leave a blank row after the section header. Otherwise the tests will fail.
https://ci.appveyor.com/project/PowerShell/xsqlserver/build/6.0.1642.0#L1434

Comments from Reviewable

You did correct using force on the git push command. It is necessary, The rebase worked correctly. Just a few comments more.

Closing and re-opening this PR to try to get the correct CLA status.

@pesetskyps Do you have time to resolve the review comments?

Review status: 2 of 3 files reviewed at latest revision, 1 unresolved discussion.

CHANGELOG.md, line 4 at r5 (raw file):
Previously, johlju (Johan Ljunggren) wrote…
Please add your entries to the end of the Unreleased section.
And please make sure to leave a blank row after the section header. Otherwise the tests will fail.
https://ci.appveyor.com/project/PowerShell/xsqlserver/build/6.0.1642.0#L1434

Done.

Comments from Reviewable

Reviewed 1 of 1 files at r6.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

CHANGELOG.md, line 4 at r5 (raw file):
Previously, pesetskyps (Pavel) wrote…
Done.

Could you please move your entries to the end of the Unreleased section? After line 101.
We add new entries in chronological order.

Comments from Reviewable

Can you take a look?

Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable

Review status: 2 of 3 files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

CHANGELOG.md, line 4 at r5 (raw file):
Previously, johlju (Johan Ljunggren) wrote…
Could you please move your entries to the end of the Unreleased section? After line 101.
We add new entries in chronological order.

Done.

Comments from Reviewable

This looks good now. But for the tests to run, and I to be able to merge, you need to resolve the merge conflicts in the CHANGELOG.md. Please see the below link on how to use git rebase to resolve the merge conflicts.  Please use git rebase (do not use git merge or git pull)
https://github.com/PowerShell/DscResources/blob/master/GettingStartedWithGitHub.md#resolve-merge-conflicts

Reviewed 1 of 1 files at r7.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable

@pesetskyps Please resolve the merge conflicts. And also make sure to move the entries in the CHANGELOG.md back to the Unreleased section after rebase. This is due to the recent release  of the module.

I will label this as abandoned for the mean time until either the contributor pick this up again or another contributor continue the work.

Closing this as there is a new PR #963 that will take this to the finish line. That PR is based on this one. @pesetskyps Thanks for your contribution!


[FEATURE REQUEST]: Output private/public keys to .der or .pem format

I am not sure how difficult it is, but it would be great to implement this feature. Now, the output is 3 files hs_ed25519_public_key, hs_ed25519_secret_key and hostname. Is it possible also to add corresponding public/private keys in .der and/or pem format?

Why would you want that? What would you use it for?

@cathugger I would use it in other libraries/ programs (like openssl). Right now, they are in a format only Tor can understand.

Can't openssl generate ed25519 keys on its own? Or you want them to have vanity fingerprint?

That's right. I want to generate a private/public key pair with a nice name verycoolname***.onion and use those keys in openssl.

But how would that even work? I mean, *.onion vanities only make sense in context of Tor Hidden Services... are you doing something like tor reimplementation? But even then it wouldn't make much sense, because private key format tor uses is fundamentally different than one specified in RFC8410 section 7.. so one just can't throw it into DER and call it a day.

@cathugger Thank you very much for clarification. So, it is impossible to do, right? Another working case I could think of is Sing/verify messages with public/private keys. So, an owner of someaddress***.onion can sign a message with his private key, and others can verify it with corresponding public key with some other program. This process is not really tor related but might be useful to verify that someone possess the private key of the address.

Thanks for clarifying why you wanted it.
It is possible to encode public key in DER/PEM, and it IS possible to use tor' private key format if library support is provided (though it's sorta hard to find ready-to-use libraries for python for example. tor' test suite may have it working I think..).
So it's theoretically possible to have something like what you're thinking working but is isn't very obvious.
However I have feeling it would be easier to quickly make something in golang what could convert public keys from tor' format to DER/PEM. DER handling is pain in C.
If message signing is desired, golang' ed25519 package doesn't provide possibility to use format tor uses so would need to fork that...

It would make sense to shill something like this to onionbalance v3 peeps, because right now they use stuff incompatible with mkp224o so one can't generate v3 vanity key easily if using onionbalance.

Previous message is sorta lie I guess, as apparently (at least now) onionbalance supports v3 private keys just fine.
From documentation:

If you need to migrate an already existing Tor onion service to Onionbalance, you can use the key directive of the Onionbalance YAML config file to point to the onion service’s private key (hs_ed25519_secret_key). You can then use your existing onion service’s address as your frontend’s address.

As this topic is more active than my with question and it is kinda related, may I ask it here also?

I run mkp224o with -y parameter, which shows hostname and keys in terminal and do not create files in folder.
To use my vanity onion I need those files in my tor hidden service folder.
I look in default hs_ed25519_public_key and hs_ed25519_secret_key files and there are some strange characters, not like in terminal output with mkp224o.
I tried to paste there public and private key from terminal, but it do not work.
How do I convert terminal output of keys to create files?


Support user password based authentication

@idanasulinmemphis @bazen-teklehaymanot
Implemented this here https://github.com/memphisdev/memphis.net/pull/39, please review.


Telemetry: Ceph volume metrics with updated Ceph Metrics drivers' unit test case

With this PR, we can collect the following metrics related to volume in the Ceph cluster:

ceph_volume_name
ceph_volume_size_bytes
ceph_volume_objects
ceph_volume_object_size_bytes

@wisererik PTAL


Fix testing / installing on SLES 15

Make sure we identify SLES 15 as suse for our zypper tests
Make sure we don't remove the symlinks on SLES when doing an upgrade or the new release will be broken
Signed-off-by: Tim Smith<EMAIL_ADDRESS>

Apparently I'm too young to edit this file so calling on @btm and @lamont-granquist


Error in passing JSON object in promptTemplate

I am trying to pass a JSON object as an example to format the output in a JSON format. But using {} detects the JSON example as a variable and the error says invalid prompt schema.
Edit: I have tried to use {{transcript}}, jinja2. Read previous issues, tried to implement those. Not of those worked.
Here is the code:

const template = `You are an experienced Cognitive behavioural therapist.

Your task is to assess the user from the chat transcript between a therapist and the user that they provide you and offer the following:
Generate a to-do list for the user. Instead of just suggesting general self care activities. personalize those activities. Be assertive.

Remember you have to format the tasks list as an JSON object in your reply. you can not generate any other text.

Example:
Say, The tasklist looks like:
1. Take some time to reflect on why you are feeling sad. 2. Identify any negative thought patterns that may be contributing to your sadness. 3. Make a list of activities that make you feel better, such as listening to music, going for a walk, or talking to a friend. 4. Make a plan to do one of these activities each day.

Your answer should be;
[
{"id":1, "task":  "Take some time to reflect on why you are feeling sad."},
{"id":2, "task":  " Identify any negative thought patterns that may be contributing to your sadness."},
{"id":3, "task":  "Make a list of activities that make you feel better, such as listening to music, going for a walk, or talking to a friend"},
{"id":4, "task":  "Make a plan to do one of these activities each day."},
]

The following is the chat transcript:
{transcript}
`

I am trying to use this template in the following way:
const taskPromptTemplate = new PromptTemplate({
    template: taskTemplate,
    inputVariables: ["transcript"],
  });

  const tasksChain = new LLMChain({
    llm: llm,
    prompt: taskPromptTemplate,
    outputKey: "tasks",
  });

And the whole thing is part of a SequentialChain:
const overallChain = new SequentialChain({
  chains: [cbtChain, tasksChain, playlistChain],
  inputVariables: ["transcript"],
  outputVariables: ["cbt", "tasks", "playlist"],
  verbose: false,
});
const chainExecution = await overallChain.call({ transcript: prompt });

Yeah, it worked. Sorry, misunderstood that {{ will be used to pass the variable, instead of the JSON object.
Thanks. :)


DbContext with passed DbConnection

#15 Changing the DbContext constructor from using a string (too many config possibilities) to using a DbConnection.  This allowed the DbConfigurationTypeAttribute to be properly loaded.

W00t!


fix errors in bahdanau-attention.md

Job d2l-zh/PR-741/1 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-741/

Job d2l-zh/PR-741/2 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-741/

Job d2l-zh/PR-741/3 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-741/

Job d2l-zh/PR-741/4 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-741/

Job d2l-zh/PR-741/6 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-741/

Job d2l-zh/PR-741/7 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-741/

Job d2l-zh/PR-741/9 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-741/

Job d2l-zh/PR-741/10 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-741/

Job d2l-zh/PR-741/11 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-741/

Job d2l-zh/PR-741/12 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-741/


fix(severity): Added support for critical severity

fix(severity): Added support for critical severity

:tada: This PR is included in version 1.0.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


i18n(zh-cn): Update images.mdx

Description (required)
Update images.mdx
Related issues & labels (optional)

#9550
Suggested label: i18n

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

zh-cn
guides/images.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


Removed task timeout

Occasionally the task run is beyond 5 mins and there is only one task in the pipelinerun, the task timeout is redundant. This is ready to be merged to avoid ci docker-images-test failure.

/test all

CI failure is being debugged by @xinredhat in #732. Merging this PR, since failure is unrelated to it.


Playback animation of drawing progress?

Thanks for the nice package!
Any chance it could "record" the paths as you draw them and then play them back as an animation?
Thanks

@jtkeyva  Hi there!
This package is a modified version of the hand_signature. You can ask your question there too.
Since a draw path stored as list of points, I think it is possible to develop an animation widget.

Hi, ya I was thinking the same thing...it's basically like the "undo" but reversed and animated. I've posted at https://github.com/RomanBase/hand_signature/issues/11
Would be a fun feature. Thanks again for the great package :)


Make updateNameItemMapping fail gracefully

https://github.com/umer-rs/google-form-submitter/pull/19/commits/6fe4d19d37964ef5fa3c665edf360a5c4df3be87


update readme

Coverage remained the same at 83.916% when pulling b7babc8673140dbbe6f01b9409d41353fc0c965f on sweeneyapps:master into f4c8fa5d04777c0017f5123a10b71577de248d45 on kcbanner:master.


Installation issues with<EMAIL_ADDRESS>

What happened?
I've encountered problems while trying to install the @gogovega/node-red-contrib-firebase-realtime-database version 0.4.0 package in my Node-RED instance. The installation process seems to hang and doesn't complete even after waiting for an extended period. Google Cloud Debian 11
Steps to reproduce the issue:
I've navigated to the ~/.node-red directory.
I've run the following command to install the package:
css
npm install --no-audit --no-update-notifier --no-fund --save --save-prefix=~ --engine-strict --omit=dev<EMAIL_ADDRESS>The installation process hangs and doesn't complete.
I've also tried running the installation command with the --verbose flag, but it still doesn't help in identifying the cause of the issue. It seems like the package is not working correctly or there might be some underlying issue with the dependencies.
with:
npm install --no-audit --no-update-notifier --no-fund --save --save-prefix=~ --engine-strict --verbose<EMAIL_ADDRESS>with: --omit=dev
npm install --no-audit --no-update-notifier --no-fund --save --save-prefix=~ --engine-strict --omit=dev --verbose<EMAIL_ADDRESS>2023-04-06T11:26:27.035Z Remove : node-red-contrib-firebase-admin-config

2023-04-06T11:26:30.723Z Remove : node-red-contrib-firebase-realtime-database
2023-04-06T11:26:31.904Z npm remove --no-audit --no-update-notifier --no-fund --save node-red-contrib-firebase-realtime-database
2023-04-06T11:26:33.732Z [out]
2023-04-06T11:26:33.732Z [out] removed 179 packages in 1s
2023-04-06T11:26:33.744Z rc=0

2023-04-06T11:26:36.344Z Remove : node-red-contrib-firebase-admin-config

2023-04-06T11:26:58.998Z Install : @gogovega/node-red-contrib-firebase-realtime-database 0.4.0
2023-04-06T11:27:00.172Z npm install --no-audit --no-update-notifier --no-fund --save --save-prefix=~ --production --engine-strict<EMAIL_ADDRESS>2023-04-06T11:27:00.731Z [err] npm
2023-04-06T11:27:00.732Z [err]
2023-04-06T11:27:00.732Z [err] WARN
2023-04-06T11:27:00.732Z [err]
2023-04-06T11:27:00.732Z [err] config
2023-04-06T11:27:00.732Z [err]  production Use --omit=dev instead.
2023-04-06T11:27:10.315Z [err]
2023-04-06T11:27:10.315Z [err] <--- Last few GCs --->
2023-04-06T11:27:10.315Z [err]
2023-04-06T11:27:10.315Z [err] [761:0x5b51cd0]    10116 ms: Mark-sweep (reduce) 129.0 (133.2) -> 127.3 (132.4) MB, 148.6 / 0.0 ms  (+ 38.6 ms in 13 steps since start of marking, biggest step 17.4 ms, walltime since start of marking 219 ms) (average mu = 0.451, current mu = 0.327) alloc
2023-04-06T11:27:10.316Z [err]
2023-04-06T11:27:10.316Z [err] <--- JS stacktrace --->
2023-04-06T11:27:10.316Z [err]
2023-04-06T11:27:10.334Z [err] FATAL ERROR: Reached heap limit Allocation failed - JavaScript heap out of memory
2023-04-06T11:27:10.486Z [err]  1: 0xb7b3e0 node::Abort() [npm install<EMAIL_ADDRESS>2023-04-06T11:27:10.487Z [err]  2: 0xa8c8aa  [npm install<EMAIL_ADDRESS>2023-04-06T11:27:10.489Z [err]  3: 0xd69100 v8::Utils::ReportOOMFailure(v8::internal::Isolate*, char const*, bool) [npm install<EMAIL_ADDRESS>2023-04-06T11:27:10.497Z [err]  4: 0xd694a7 v8::internal::V8::FatalProcessOutOfMemory(v8::internal::Isolate*, char const*, bool) [npm install<EMAIL_ADDRESS>2023-04-06T11:27:10.498Z [err]  5: 0xf46ba5  [npm install<EMAIL_ADDRESS>2023-04-06T11:27:10.500Z [err]  6: 0xf5908d v8::internal::Heap::CollectGarbage(v8::internal::AllocationSpace, v8::internal::GarbageCollectionReason, v8::GCCallbackFlags) [npm install<EMAIL_ADDRESS>2023-04-06T11:27:10.508Z [err]  7: 0xf3378e v8::internal::HeapAllocator::AllocateRawWithLightRetrySlowPath(int, v8::internal::AllocationType, v8::internal::AllocationOrigin, v8::internal::AllocationAlignment) [npm install<EMAIL_ADDRESS>2023-04-06T11:27:10.509Z [err]  8: 0xf34b57 v8::internal::HeapAllocator::AllocateRawWithRetryOrFailSlowPath(int, v8::internal::AllocationType, v8::internal::AllocationOrigin, v8::internal::AllocationAlignment) [npm install<EMAIL_ADDRESS>2023-04-06T11:27:10.510Z [err]  9: 0xf150a0 v8::internal::Factory::AllocateRaw(int, v8::internal::AllocationType, v8::internal::AllocationAlignment) [npm install<EMAIL_ADDRESS>2023-04-06T11:27:10.511Z [err] 10: 0xf0cb14 v8::internal::FactoryBasev8::internal::Factory::AllocateRawWithImmortalMap(int, v8::internal::AllocationType, v8::internal::Map, v8::internal::AllocationAlignment) [npm install<EMAIL_ADDRESS>2023-04-06T11:27:10.513Z [err] 11: 0xf0edc8 v8::internal::FactoryBasev8::internal::Factory::NewRawOneByteString(int, v8::internal::AllocationType) [npm install<EMAIL_ADDRESS>2023-04-06T11:27:10.514Z [err] 12: 0x104c73d v8::internal::JsonParser::MakeString(v8::internal::JsonString const&, v8::internal::Handlev8::internal::String) [npm install<EMAIL_ADDRESS>2023-04-06T11:27:10.519Z [err] 13: 0x104e426 v8::internal::JsonParser::ParseJsonValue() [npm install<EMAIL_ADDRESS>2023-04-06T11:27:10.519Z [err] 14: 0x104ee9f v8::internal::JsonParser::ParseJson() [npm install<EMAIL_ADDRESS>2023-04-06T11:27:10.520Z [err] 15: 0xdeedf6 v8::internal::Builtin_JsonParse(int, unsigned long*, v8::internal::Isolate*) [npm install<EMAIL_ADDRESS>2023-04-06T11:27:10.521Z [err] 16: 0x1707c79  [npm install<EMAIL_ADDRESS>2023-04-06T11:27:10.541Z rc=null
Please look into this issue and provide guidance on how to resolve it. Thanks.

How to reproduce?
Steps to reproduce the issue: 1. Go to '...' 2. Click on '....' 3. Scroll down to '....' 4. See error
Additional comments?
Additional comments here, if any.

Hi Marek,
It looks like you have a memory problem:
FATAL ERROR: Reached heap limit Allocation failed - JavaScript heap out of memory

Can I suggest you to start installing another random node to check if the problem comes from this node or is it global.
A solution to solve this problem would be to update Node.js (as a reminder >=14). Another if it still does not work would be to modify the value of max-old-space-size as mentioned in this article.
I hope the above is useful to you, let me know if you need anything else 😉

Hi GogoVega,
Thanks for quick response, i trying do it in configuration but it doesn’t work.
Indeed, it seems to be a RAM issue, and experience has shown that none of the software solutions worked. You can try temporarily increasing the resources of your virtual machine on Google Cloud Platform to install the required packages. Here are the steps to do this:
Log in to the Google Cloud Platform (GCP) and go to the Compute Engine management panel.
Stop your virtual machine to be able to change its configuration. Select the virtual machine and then click the "STOP" button on the detailed information page of the virtual machine.
When the machine is stopped, click the "EDIT" button
on the detailed information page of the virtual machine.
Change the machine type from "f1-micro" to "g1-small" in the "Machine configuration" section.

Click the "SAVE" button at the bottom of the page to save the changes.
Restart the virtual machine by clicking the "START" button on the detailed information page of the virtual machine.
Now you have more resources, so try installing the required packages.
After completing the installation and testing everything, you can return to the Compute Engine management panel and stop the virtual machine again.
Click the "EDIT" button on the detailed information page of the virtual machine, change the machine type back to "f1-micro" in the "Machine configuration" section, and click the "SAVE" button at the bottom of the page.
Turn the virtual machine back on by clicking the "START" button.
Remember that changing the machine type may affect the cost of the GCP service. Make sure you track resource usage and costs to avoid unexpected charges.


Web: no longer seeing progress when uploading a folder

Testing #111250
Dropping a folder into a web codespace no longer shows progress in the status bar when using insiders, works fine in stable.
@isidorn maybe related to your changes for reporting progress from explorer?

Yes, droping a folder was using a custom progress reporting solution. I refactored the thing and did not want a special solution just for drop.
So I will check why the general progress case does not catch this.

@isidorn my web git bisect points out to a change that caused it here: https://github.com/microsoft/vscode/compare/6026ab576dc6a4fbb8e255241a364816f42464c5...0a80aacc7be1ab03ec0f94b8ac1a84949a83f35d
The experience now in web is quite bad, even clicking on a file while the upload is in progress does not work.

@bpasero my changes should not affect that clicking on a file while the upload is in progress does not work.
Maybe I broke that, but I would be surpsied. Did you bisect that to an exact commit maybe?

@isidorn yes, confirmed it is https://github.com/microsoft/vscode/commit/668b1d24399e3df2a517d1d18f2b406b1cef8cee

@bpasero I have pushed a change that the progress is now shown (I was missing an await).
For Clicking on a file while upload is in progress I have filled a new issue https://github.com/microsoft/vscode/issues/111832

I commented on that change, I think there is another call you miss to await


When will snorkel officially releases 0.9.9 version on conda?

I've been waiting for 0.9.9 on conda for long days, I wonder what's wrong with new version release, I can do a little favor.

Hi yinxiangshi and RadionBik,
Apologies for the delay here, need to resolve some CI issues before the release can be cut which are currently being worked on. Will update here once it has been. Thanks for the patience!

#abandonware

Since I needed it myself, I have repackaged snorkel into pip package snorkel2. No changes to the library, just removed requirements which were making the library obsolete. Right now can only be installed by pip.
pip install snorkel2

The github is here, you can get in touch with me in the Snorkel2 discussions board

Apologies for the delay! The repo was in dependency and CI hell there for a bit, but resolved now. Dependencies have been updated for most libraries. v0.9.9 has been cut (PR #1700) and pushed to pypi (https://pypi.org/project/snorkel/). It should be available on conda within a day or so (I'll respond here again and close the issue when it is).
We previously had upper bounds on most dependencies on purpose for stability reasons, but have decided to leave them open to higher versions moving forward for better forward compatibility, so there should be fewer issues like this moving forward. It's an open area of debate which is better practice (see https://iscinumpy.dev/post/bound-version-constraints/ for some relevant discussion on the topic), but the latter seems like the appropriate choice for now. Thanks again everyone for your patience!

@bhancock8 thank you so much for cutting a release and addressing all of the issues it solves! It was quite difficult to use snorkel before this release due to stale dependencies and I think this will address the problem :)

@bhancock8  thanks for preparing the new release. Since snorkel is back, I have removed snorkel2 from Pypi to avoid polluting the Python ecosystem.

Thanks @cerquide! And if there are ever specific libraries that need updating and you're able to make a PR that passes tests, that's a super efficient way for us to review and merge things much faster. Per the above, hopefully the new policy of leaving open ended upper bounds makes such PRs necessary less frequently as well.

v0.9.9 is now on conda (https://anaconda.org/conda-forge/snorkel)—I'm going to mark this issue as resolved.


[WIP] Add credential passthrough in base-jdbc connector

I was working on adding credential passthrough in presto-base-jdbc connector and others extend it. Thanks to JdbcIdentity added by  @kokosing  this is straighforward.
To specify the credential for different jdbc connectors, we need to have a namespace based on catalog name, so I recreate the JdbcConnectorId deleted by @kokosing but name it JdbcCatalogName (as we are replacing connector id with catalog name).
Create this PR to collect feedback on this feature, as well as the implementation.
@electrum @findepi @kokosing

I think there is a simpler and more flexible way. When we designed extra credentials, we intentionally did not namespace them per connector to allow flexibility.
For example, you might have multiple MySQL catalogs for different servers, but they all share the same authentication, so you want to use the same property name.
We can put all the logic for using credentials inside of DriverConnectionFactory. The config could have properties with defaults:

user-credential-name=${CATALOG}.user
password-credential-name=${CATALOG}.password

These credential names would be stored with the catalog replaced when DriverConnectionFactory is created. This allows users to use a single name across multiple catalogs if desired. It also keeps the code cleaner as we don't need to modify the creation of JdbcIdentity.
For the config naming, something like credential-name.user would seem more natural, but is wrong from a grammar standpoint.

I did another experiment (just for fun), where I used extra credentials as variables. That way it is very flexible. For example mysql.properties could look like:
...
connection-url=${mysql.user}
connection-password=${mysql.password}

Then all ${...} are replaced with values from extra credentials. Namespaces are handled here by user in a way they want. To establish connection all variables had to be resolved in my case, however I can imagine other strategies as well (like using default value for ${...} property when extra credential is missing - ${key:-default_value} from bash).

Also I think, that at some point we need to have support extra credentials in access control. Similar to io.prestosql.security.AccessControl#checkCanSetSystemSessionProperty or io.prestosql.security.AccessControl#checkCanSetCatalogSessionProperty

@kokosing That’s an interesting idea. I like that it’s fledible, but I can’t think of any use cases that it enables over my proposal. Two cocncerns are that it makes error reporting more difficult and replacing part of the JDBC URL with use provided data seems dangerous. That said, please continue thinking along these lines.

replacing part of the JDBC URL with use provided data seems dangerous

This might require some escaping mechanism. However, most of the things (eg user, password) will be set via Properties

Thanks for the feedback! Setting the credential key in config files is brilliant, and indeed simplifies the implementation. I'll update the PR.
I think we should only allow user and password passed from extra credentials, not connection-url, is that right?

It looks good to me.
I think it still needs end-to-end testing. To see that with extra credentials we are capable to connect as different user.

@kokosing I was thinking about writing production tests for it but couldn't figure out an easy way to set up user/credential in mysql/postgresql container.
Or is there any other unit tests that can achieve the same goal?

I was thinking about writing production tests

Let's have product tests a last resort option. I would go with "unit" tests. See that MySqlQueryRunner#createMySqlQueryRunner accepts additional connector properties and you sholud be able to set extra credentials in session in AbstractTestQueryFramework#assertQuery. I think that should be enough that you would need.

@electrum  @kokosing  I updated commits to address your comments. Please take a look.
Thanks!


Merge master branch to prTesting1_201901021343 branch

OPS Build status updates of commit 25eac6a:
:clock10: Preparing: average preparing time is 51 sec(s)

OPS Build status updates of commit 25eac6a:
:clock10: Incremental building: average incremental building time is 16 sec(s)

OPS Build status updates of commit 25eac6a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

split/updateTestRef1.md
:white_check_mark:Succeeded

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


Function showing the number of users in the voice channel

Type

[x] Functions: <$voice[channelID/all;membersCount]>
[ ] Callbacks: <Callback Name>
[ ] Handlers: <Handler Type>
[ ] Others: ______

Description
If $voice[$voiceID;membersCount] is typed, sends the number of users on the entered voice channel. But if $voice[all;membersCount] is typed, it will send the number of users on all audio channels. Even if the function is not exactly like this, something similar can be done.

Type

[x]  Functions: <$voice[channelID/all;membersCount]>
[ ]  Callbacks:
[ ]  Handlers:
[ ]  Others: ______

Description
If $voice[$voiceID;membersCount] is typed, sends the number of users on the entered voice channel. But if $voice[all;membersCount] is typed, it will send the number of users on all audio channels. Even if the function is not exactly like this, something similar can be done.

first part can be done using $usersInChannel
second part hmm k I will check something and will inform

Type

[x]  Functions: <$voice[channelID/all;membersCount]>
[ ]  Callbacks:
[ ]  Handlers:
[ ]  Others: ______

Description
If $voice[$voiceID;membersCount] is typed, sends the number of users on the entered voice channel. But if $voice[all;membersCount] is typed, it will send the number of users on all audio channels. Even if the function is not exactly like this, something similar can be done.

first part can be done using $usersInChannel
second part hmm k I will check something and will inform

Oky thx

@falsisdev for second part I made vcSize which supports that with more options which I will list in my pr

@falsisdev for second part I made vcSize which supports that with more options which I will list in my pr

Thx I hope you will be rewarded for your efforts as soon as possible.

@falsisdev for second part I made vcSize which supports that with more options which I will list in my pr

Thx I hope you will be rewarded for your efforts as soon as possible.

already got it but thanks


:package: Update Deno dependencies

The output of make update is
./denops_std/mod.ts

./denops_std/function/mod.ts

./denops_std/function/_generated.ts

./denops_std/function/cursor_test.ts

./denops_std/function/input.ts

./denops_std/function/vim/mod.ts

./denops_std/function/vim/_generated.ts

./denops_std/function/vim/_manual.ts

./denops_std/function/_manual.ts

./denops_std/function/buffer.ts

./denops_std/function/cursor.ts

./denops_std/function/common.ts

./denops_std/function/various.ts

./denops_std/function/nvim/mod.ts

./denops_std/function/nvim/_generated.ts

./denops_std/function/nvim/_manual.ts

./denops_std/function/input_test.ts

./denops_std/function/various_test.ts

./denops_std/function/types.ts

./denops_std/batch/mod.ts

./denops_std/batch/gather.ts

./denops_std/batch/gather_test.ts

./denops_std/batch/batch_test.ts

./denops_std/batch/batch.ts

./denops_std/variable/mod.ts

./denops_std/variable/register.ts

./denops_std/variable/register_test.ts

./denops_std/variable/option_test.ts

./denops_std/variable/option.ts

./denops_std/variable/environment_test.ts

./denops_std/variable/variable.ts

./denops_std/variable/variable_test.ts

./denops_std/variable/environment.ts

./denops_std/variable/types.ts

./denops_std/helper/mod.ts

./denops_std/helper/input.ts

./denops_std/helper/load.ts

./denops_std/helper/echo.ts

./denops_std/helper/execute.ts

./denops_std/helper/echo_test.ts

./denops_std/helper/batch_test.ts

./denops_std/helper/execute_test.ts

./denops_std/helper/batch.ts

./denops_std/helper/load_test.ts
[1/1] Looking for releases: https://raw.githubusercontent.com/vim-denops/deno-denops-std/v1.9.1/denops_std/helper/#=
[1/1] Using latest: https://raw.githubusercontent.com/vim-denops/deno-denops-std/v1.9.1/denops_std/helper/#=

./denops_std/helper/input_test.ts

./denops_std/autocmd/mod.ts

./denops_std/autocmd/group_test.ts

./denops_std/autocmd/group.ts

./denops_std/autocmd/common_test.ts

./denops_std/autocmd/common.ts

./denops_std/autocmd/types.ts

./denops_std/bufname/mod.ts

./denops_std/bufname/bufname_test.ts

./denops_std/bufname/bufname.ts

./denops_std/bufname/utils_test.ts

./denops_std/bufname/utils.ts

./denops_std/option/mod.ts

./denops_std/option/_generated.ts

./denops_std/option/vim/mod.ts

./denops_std/option/vim/_generated.ts

./denops_std/option/vim/_manual.ts

./denops_std/option/_manual.ts

./denops_std/option/nvim/mod.ts

./denops_std/option/nvim/_generated.ts

./denops_std/option/nvim/_manual.ts

./denops_std/anonymous/mod.ts

./denops_std/anonymous/mod_test.ts

./denops_std/test/mod.ts

./denops_std/deps_test.ts
[1/2] Looking for releases<EMAIL_ADDRESS>[1/2] Attempting update<EMAIL_ADDRESS>-> 0.123.0
[1/2] Update successful<EMAIL_ADDRESS>-> 0.123.0
[2/2] Looking for releases<EMAIL_ADDRESS>[2/2] Using latest<EMAIL_ADDRESS>
./denops_std/mapping/mod.ts

./denops_std/mapping/parser_test.ts

./denops_std/mapping/parser.ts

./denops_std/mapping/mod_test.ts

./denops_std/mapping/types.ts

./denops_std/deps.ts
[1/6] Looking for releases<EMAIL_ADDRESS>[1/6] Attempting update<EMAIL_ADDRESS>-> 0.123.0
[1/6] Update successful<EMAIL_ADDRESS>-> 0.123.0
[2/6] Looking for releases<EMAIL_ADDRESS>[2/6] Attempting update<EMAIL_ADDRESS>-> 0.123.0
[2/6] Update successful<EMAIL_ADDRESS>-> 0.123.0
[3/6] Looking for releases<EMAIL_ADDRESS>[3/6] Attempting update<EMAIL_ADDRESS>-> 0.123.0
[3/6] Update successful<EMAIL_ADDRESS>-> 0.123.0
[4/6] Looking for releases<EMAIL_ADDRESS>[4/6] Attempting update<EMAIL_ADDRESS>-> 0.123.0
[4/6] Update successful<EMAIL_ADDRESS>-> 0.123.0
[5/6] Looking for releases<EMAIL_ADDRESS>[5/6] Using latest<EMAIL_ADDRESS>[6/6] Looking for releases<EMAIL_ADDRESS>[6/6] Using latest<EMAIL_ADDRESS>
Already latest version:
https://raw.githubusercontent.com/vim-denops/deno-denops-std/v1.9.1/denops_std/helper/#= == v1.9.1
<EMAIL_ADDRESS>== v2.2.0
<EMAIL_ADDRESS>== v2.2.0
<EMAIL_ADDRESS>== v1.1.4

Successfully updated:
<EMAIL_ADDRESS>0.119.0 -> 0.123.0
<EMAIL_ADDRESS>0.119.0 -> 0.123.0
<EMAIL_ADDRESS>0.119.0 -> 0.123.0
<EMAIL_ADDRESS>0.119.0 -> 0.123.0
<EMAIL_ADDRESS>0.119.0 -> 0.123.0
make[1]: Entering directory '/home/runner/work/deno-denops-std/deno-denops-std'
make[1]: Leaving directory '/home/runner/work/deno-denops-std/deno-denops-std'

Close due to #117


Update scala3-library, ... to 3.0.1

Updates

org.scala-lang:scala3-library
org.scala-lang:scala3-library_sjs1

from 3.0.0 to 3.0.1.
GitHub Release Notes - Version Diff - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-lang" } ]

labels: library-update, semver-patch

Codecov Report

Merging #1795 (5595f68) into master (3d2db25) will decrease coverage by 0.16%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1795      +/-   ##
==========================================
- Coverage   86.07%   85.90%   -0.17%
==========================================
  Files          73       69       -4
  Lines        2499     2455      -44
  Branches      136      136
==========================================
- Hits         2151     2109      -42
+ Misses        348      346       -2

Impacted Files
Coverage Δ

...re/shared/src/main/scala/io/circe/KeyDecoder.scala
97.29% <0.00%> (-2.71%)
:arrow_down:

...rc/main/scala/io/circe/numbers/BiggerDecimal.scala
87.13% <0.00%> (-0.59%)
:arrow_down:

...rc/main/scala-2.13+/io/circe/LiteralDecoders.scala

...rc/main/scala-2.13+/io/circe/LiteralEncoders.scala

...main/scala-2.13+/io/circe/CollectionDecoders.scala

...c/main/scala-2.13+/io/circe/FactoryInstances.scala

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3d2db25...5595f68. Read the comment docs.


Modify IG routes to use the conversion library to build a consistent response with a OB data model.

Description
Modify IG routes to use the uk gateway extension conversion to build a consistent OB data model object response
Acceptance criteria

[ ] The Ig routes modified still have the same behaviour
[ ] All tests passed

Closed to implement a new solution #515


Clarify Plan Visibility options on Share page

Revise and add the following to the description below "Set plan visibility" to clarify for users 1) threshold for public or org visibility and 2) why they cannot set visibility for test plans:
"Public and organisational visibility are intended for finished plans. You must answer at least 50% of questions to enable these options. Test plans are private by default and not included in usage information."

looks good for now. we can clarify further later on if necessary. closing issue.


ArraySchema annotation items() value does not appear to be properly supported

I may or may not be correct but the ArraySchema annotation now has items() and  schema() but they both have the same Javadoc. I assume that items() is meant to be a synonym for schema() but is less ambiguously named. items() is a recent addition.
See https://docs.swagger.io/swagger-core/v2.2.20/apidocs/io/swagger/v3/oas/annotations/media/ArraySchema.html
I have a test case in Scala where I use an ArraySchema annotation to override the item implementation. I have 2 variants. One that uses schema and the other that uses items but the override only works with schema.
I'm using swagger 2.2.20.
The 2 classes are in:
https://github.com/swagger-akka-http/swagger-scala-module/blob/develop/src/test/scala/models/AddRequest.scala
The AddRequestOldStyleAnnotation variant works as expected when I run it through the swagger-core ModelResolver.
I suspect that this line is at fault
https://github.com/swagger-api/swagger-core/blob/0d067d03b6ef78a78efe3e4f957f977254c72030/modules/swagger-core/src/main/java/io/swagger/v3/core/jackson/ModelResolver.java#L166
Note how it only looks at the schema value and not the items value.

Thanks for reporting this. It looks like a bug introduced while implementing support for OpenAPI 3.1, as ArraySchema.schema already represents the schema of the items in the array, and ArraySchema.items is a duplicate that has been only processed in terms of types and only in case of OpenAPI 3.1.
#4611 deprecates ArraySchema.items for future removal, adding support for OAS 3.1 types by processing existing schema field.


emails

This PR adds a new service called emails that can run in cloud run and will be responsible with shooting out emails to users whether that be verification emails or notification emails. This service should be set up in a way that is not accessible to traffic outside of GCP (this is the default for new services) and will be communicated to by both our backend as well as cloud scheduler jobs for automated emails. This service also has access to the backend database (write and read, although we can make it read only if we decide to handle updating verification statuses on the backend instead of on the email service).
TLDR completion points
note: this is all still early so even complete points might not be 100% complete, but are functional right now

✅ verification process
½ notification emails (sending of emails good, triggering of emails not done)
✅ adding emails to users on signup
✅ subscribing users on sendgrid for single marketing email sends

TLDR main TODO points for future PRs

updating emails
updating notification setting and unsubscribing
triggering of notification emails on a schedule

Summary of infrastructure plan for emails
Verification process
When a user creates their account on the backend, the backend service would call the email service's verification email endpoint and the email service will respond with whether it successfully sent the verification email to the user.
When the user verifies their email using the link in the email (the link will be to a new frontend page with some query parameter that will be a JWT signed by the email service and probably just immediately redirect them to a success or failure page on that request's result) that frontend page will call a backend endpoint with the JWT and the backend will then call the email service to verify that the JWT is valid.
The email service will update the user's verification status to true if the JWT is valid or return an error if not (the backend could also do the update, and then we can make the email service read only, good with either).
Automated emails
On GCP we can create scheduled jobs that can call out to the email service using http (docs for scheduler (these docs use pub/sub in the examples but we can use http as well to directly call the endpoints)). The endpoint responding will iteratively paginate through every user with the right unsubscribe statuses (are they unsubscribed to all emails or the specific type of email we are trying to send?) and concurrently send each user an email using go templating to fill out the data for that user.
Single Emails
Single emails can be sent directly from SendGrid using the "Single Email" feature on their app webpage. Anyone with access can create their own templates using a templating dashboard or html/plain text directly. The recipients for the single email will have to be consistent with the emails signed up on gallery so at the end of the verification process, we also ensure that a user's email is added to a SendGrid Contact List (we can create multiple lists for different types of emails but I created one default list that is supposed to just be all of our user's emails) which can then be used to send emails to. We also want to make sure that when user's update their unsubscriptions, they are added to the appropriate "Unsubscribe group" on SendGrid or removed from the appropriate Contact List.
Considerations

Verification emails use JWTs for authentication. These JWTs are signed by a different service (with a different signing secret) than the backend so a user couldn't just send their cookie along and have it work. I have not found too much online as far as practices go for this but I do need to more research here. If anyone already has experience with or knowledge of a better way to best verify emails in a secure way, or believe this way is totally secure enough, let me know!
Should the email verification service handle updating the user verification status or should the backend handle this? Currently the email service goes to the database to read information so that for example we can get the email directly from a user ID during the verification process rather than the service being sent an email as an input and the service just handling the email. Is it then alright that the service can write to the DB as well?

TODO's after merge

[ ] Migration
[ ] Schema changes (not breaking)

Nice @benny-conn !
+1 to what @jarrel-b  said, would you be able to diagram the system for documentation? I'm sure it'll help reviewers understand this PR better too!


Update zwave climate mappings for Pending Heat/Cool

Description:
Reference #25274
When a zwave thermostat is showing the operating status of "Pending Heat" and "Pending Cool" it means that it is waiting on a minimum off period timer to expire that was initiated at the end of a cycle before it is will allow another heating or cooling cycle to begin to prevent short cycling the HVAC equipment.  My thermostat shows "Wait" on its LCD display when this is happening. The HVAC equipment should be going to an idle status when the thermostat is reporting that status.   Thus hvac_action attribute should be idle when thermostat is in Pending Heat or Pending Cool.
For those of us who will monitor hvac_action attribute to calculate energy usage of the won't have an accurate accounting without this patch applied because the time delay added on the end of every cycle will be counted as operating time and this time can be very considerable.
Checklist:

[ x] The code change is tested and works locally.
[ x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[ x] There is no commented out code in this PR.
[ x] I have followed the [development checklist][dev-checklist]

Thanks!


Crash in PTVS 2.1 when saving options

Hi, I'm using the latest version of PTVS 2.1 branch that I've built with VS2013 and it works fine in general, but I am sometimes seeing crashes (or freezes) upon hitting OK in the environment options dialog.  The line that throws the uncaught exception is:
var replProvider = PythonToolsPackage.ComponentModel.GetService();
in the PythonInteractiveOptionsPage.cs file.  Does anyone know what this is and/or has there been a fix for it?  I remember having this problem last time I tried v2.1, then I upgraded to a more recent version and I think the problem went away.  But unfortunately I now need to use version 2.1 for various reasons and am seeing this problem.  Is there a fix or workaround?  Thank you

Do your Interactive Windows work okay? Does it make a difference to this hang if you've already opened some vs. never opening an interactive window?

Hi zooba,
Thanks for the response -- I reset my Exp hive and now this issue seems to have gone away, but not sure why.  But now I'm having another problem, and that is that when I create a new python project from the New Project window (which seems to work fine), then I right click on the project name in Solution Explorer I get an uncaught exception.  The same thing happens if I select the project in Solution explorer and then hover my mouse over the PROJECT menu (basically I'm just trying to edit the project settings but it keeps crashing).  Any idea what this could be and if it's related to the first problem?  Thank you,
David

Hi zooba,
Here are some more details on what I did/am observing:

Check out the head of the 2.1 branch
Do a "minimal" build, which succeeds
Deploy to Exp for debugging (although the problem happens whether in debug mode or not)
Create a new project from one of the project templates (which succeeds in the Exp hive)
Right click on the project in solution explorer in order to change some properties
The crash happens immediately upon right clicking, before the context menu pops up.  It also crashes when repeating steps 1-4 but then clicking on the PROJECT menu

Currently I only have the two default 2.6/2.7 python environments set up which work fine -- I can open a REPL without a problem.
Over the last few months I've been doing most of my testing with version 2.2 I believe, where I did not have these problems.  But recently I found out that I need to use version 2.1 and now these issues are popping up (crashing when right-clicking on the project and also sometimes crashing when saving options).  Any ideas?
Thank you,
David

(As an aside, I'm interested in why you need 2.1 and not 2.2. You can email our team privately at<EMAIL_ADDRESS>if you don't want to explain publicly.)
So we actually dropped VS 2013 from 2.1.1, which is at the tip of 2.1, and it technically only supports VS 2010 and 2012. This should only matter for one bit of Django support, which I removed rather than attempting to handle all the edge cases involved in VS 2013 (see TemplateArtifactCollection.cs if you're interested), but it shouldn't affect the REPL window.
What's more likely is things failing to load correctly because of the non-standard deployment. Check whether you are deploying Microsoft.VisualStudio.ReplWindow.dll into the Python Tools extension directory. It is supposed to only be in the ReplWindow extension, but it is also supposed to be in the GAC which may have been masking some other resolution issues.
If you debug the VS instance, break on all exceptions and then cause the crash you may be able to extract some more information about the source. These exceptions often include InnerException or LoaderExceptions properties that are more informative than the one that is actually thrown.

Hi zooba,
I did a release build of the extension, then took the vsix/dll files and installed them into another hive using a vsix installation program and then the issue seems to go away, so looks like you're right that it is deployment-related.
That said, it would be nice to be able to use the projects feature within the Exp hive while debugging -- this is because I haven't found a way to enter into mixed mode debugging without running in the context of a project and checking the "Enable native code debugging" checkbox... and similarly I haven't found a way to add folders into the PYTHONPATH of the interpreter without using a project... am I missing something?
Ideally actually I wouldn't create any projects at all for most use cases, I would just open a new python file, type some code, then debug it in mixed-mode and have it automatically jump to the C++ code that is being called.  Is it possible to do that without a project?
Regarding the version issue, I work at a large firm that is slow to approve new software (currently v2.1 is approved but not yet 2.2).  PTVS is just starting to be introduced on a large scale here and so I'm sure we'll upgrade eventually, but for now have to stick to 2.1.

You're right that those features are currently unavailable without projects, though we are hoping to fix that for 3.0. Given Visual Studio is built around the concept of projects, we really have no choice but to support them, and reimplementing everything twice to cover non-project scenarios is a really big ask.
If you're getting PTVS introduced widely then we'd definitely like to hear more information - the more users we have the easier it is to get more developers to work on PTVS. Also, PTVS is now available as part of VS 2015, so if that's being approved then it should include PTVS 2.2 (depending on how thorough your firm is).

Hi zooba, I will send an email to<EMAIL_ADDRESS>if you don't mind.  -David


docs: Create a FAQ section

It'd be great to have a frequently asked questions section as part of our official website. Such like https://prometheus.io/docs/introduction/faq/
We are getting similar questions from the community over public slack channels and office hours. We can gather them together and help the community.
cc @bwplotka @brancz

@kakkoyun @bwplotka @brancz Hi, This is great, is they any pointer or input? I would love to work on this, Thanks.

Can I work on this?

@DARK-art108 @adeniyistephen, Of course, you can work on it. We need to collect those questions first. Slack channels would be a good place to start.

@DARK-art108 @adeniyistephen, Of course, you can work on it. We need to collect those questions first. Slack channels would be a good place to start.

@DARK-art108 @adeniyistephen, Of course, you can work on it. We need to collect those questions first. Slack channels would be a good place to start.

yes that sounds like a very good idea, what is the slack channel so we cam track does question. Thanks

@DARK-art108 @adeniyistephen, Of course, you can work on it. We need to collect those questions first. Slack channels would be a good place to start.

yes that sounds like a very good idea, what is the slack channel so we cam track does question. Thanks


Exclude DataSourceAutoConfiguration.class in S3MockApplication

In my application I have to manually config the DataSources, so when I call S3MockApplication.start() in my tests it tries to autoconfigure the datasources and that gives me an error.
Is possible to exclude DataSourceAutoConfiguration.class in S3MockApplication?

Yes, sure. Would you open a PR for that, then you can directly test it against your app?

S3Mock 2.2.0 was just released that now includes support for TestContainers.
If there is a possibility of running Docker during your build, I would highly suggest migrating to TestContainers.
Using S3MockApplication directly or through one of the unit test modules will always have the risk of breaking because of dependencies that are pulled in by the application that is being tested, and auto configration that is added through Spring Boot updates in the S3Mock.
Using Docker in your tests will insulate both applications dependencies and application life cycles.


tabs.go from example cannot run (problem in extra?)

when trying to run tabs.go example first problem is with changing extra to _extra since this changed (probably?)
import (
    extra "github.com/gizak/termui/_extra"
)

and secondly Point is not defined: (and some other errors):
go run main.go

# github.com/gizak/termui/_extra
../../gizak/termui/_extra/tabpane.go:31: undefined: Point
../../gizak/termui/_extra/tabpane.go:32: undefined: Point
../../gizak/termui/_extra/tabpane.go:55: undefined: Theme
../../gizak/termui/_extra/tabpane.go:114: tp.HasBorder undefined (type *Tabpane has no field or method HasBorder)
../../gizak/termui/_extra/tabpane.go:124: undefined: Point
../../gizak/termui/_extra/tabpane.go:139: undefined: Point
../../gizak/termui/_extra/tabpane.go:155: undefined: Point
../../gizak/termui/_extra/tabpane.go:114: too many errors

+1
Seems example are broken


Publish a 2.11 version?

Any plans to publish a 2.11 version?

sbt plugins are not binary compatible with 2.11. This won't be possible until sbt 0.14.0 or 1.0.0


error while compiling cudpp

[ 25%] Building NVCC (Device) object src/cudpp/CMakeFiles/cudpp.dir/app/cudpp_generated_multisplit_app.cu.o
[...]/cudpp/src/cudpp/app/multisplit_app.cu(1919): error: namespace "cub" has no member "CachingDeviceAllocator"
1 error detected in the compilation of "/tmp/tmpxft_00003ceb_00000000-7_multisplit_app.cpp1.ii".
CMake Error at cudpp_generated_multisplit_app.cu.o.cmake:266 (message):
Error generating file
[...]/cudpp_build/src/cudpp/CMakeFiles/cudpp.dir/app/./cudpp_generated_multisplit_app.cu.o

This is a CUDPP issue.  Maybe they are not including the proper header (https://github.com/NVlabs/cub/blob/1.7.4/cub/util_allocator.cuh)?

I was able to fix this problem by adding
#include "util_allocator.cuh" to my cub.cuh file.


[Question] "Duplicate relation in a relation expression"

Raw Warning
Duplicate relation "guilds" in a relation expression. You should use "a.[b, c]" instead of "[a.b, a.c]". This will cause an error in objection 2.0
Can somebody explain what this is ?

I cannot explain it better than that error message. a is the duplicate relation in the error message.

I know it is a bit late, but but please check if you have the duplication within you eager() option. We had the same issue for model.query().select("...").eager(something, a, a.b)


not filling in line where folder is for export

In the objects tab, when you point to a folder..I thought it would fill in the line (like it does for Mamu82 in his video but it won't do it on mine?
I believe if you export it to MSFS and you open DevMode and you build a project..you get an error because MSFS (dev mode) doesn't know where the model folder is because the line is blank.
It may just be me.

Thank you!!
Bob M.

These double backslashes " \ " are shown, when you export into the same folder where your blend file is.
When you export to another folder, then th path of the folder should be there.
Have you tried that?

I didn't know that. I just assumed that regardless if I point to any folder and click on "accept"  it would just show up in the view as being filled in. I always point to the same folder where the blend file (because its the same folder as the model I am working on.) making sense.
I am new at this and in the videos I have seen its just show up in the line automatically once the "accept" button is clicked?
Thank you!


[BUG] Macchina crashes on --theme def

Describe the bug
Using
macchina -t def

causes
memory allocation of<PHONE_NUMBER>3709551613 bytes failed
[1]    22666 abort (core dumped)  macchina -t def

To Reproduce
Steps to reproduce the behavior:

macchina -t def

Expected behavior
Run normally
Screenshots

System Information

Distribution: Arch Linux
Desktop Environment: bspwm
Terminal: Alacritty 0.7.2-1
Macchina's version: v0.5.5

Additional context
Add any other context about the problem here.

@grtcdr Crash occurs in https://github.com/grtcdr/macchina/blob/7fe9d7e390cb664cb050c7b7b52a10b29646b29b/src/display.rs#L367 function, because longest_key.len() < current_key.len() which causes the usize to go into the negative (overflow).

The sanest solution I think is to remove the def theme, because it's the default theme, it is no different from running macchina without the --theme argument.

I removed def from the theme list in the latest commit https://github.com/grtcdr/macchina/commit/ebde6b48c03bacebe453b8f1ef79d4f07345ef3f and released a new version v0.5.6

Thanks for fixing.

Of course! and thank you for reporting it :smile: