Does it only work for projects created in C# language?

I'm from https://stackoverflow.com/a/72021827/13792395 and I don't see a similar description on the project homepage on github (about the languages it supports)
//I don't know if this is because I don't have the components needed for c# development installed, I'm installing

I installed .NET desktop development via Visual Studio Installer
But I still can't get it to work

I re-read it and I found myself using this plugin in a wrong way, I should have followed the instructions at https://stackoverflow.com/a/72021827/13792395 exactly,
Add VSCodeExtension to VisualStudio
Add VSCode NugetPackage reference in your project.
I don't have a problem now, what I'm looking for is a solution that can be used by a c++ project.

I'm not smart enough to use C++. 😉
I know it works with C# and I don't see any reason it won't work with VB.Net and F#.
In theory, it works with any language that uses .NET and would call System.Console.* to output to the console or System.Diagnostics.Debug.* to write debug output.
If it doesn't work with another programming language, it should be possible to build a simple integration by creating a library that can create output that ends up in the VIsual Studio Debug window.

Would you like to implement a library for C++ console programs to provide the same functionality! I swear I'll advertise it to a lot of people

Do you know IDEA, when I finish writing a program and press the run button, it will output the result to a sub-window of the IDE instead of a separate window that pops up.
I wish there were plugins to achieve the same in visual studio.

Do you know IDEA, when I finish writing a program and press the run button, it will output the result to a sub-window of the IDE instead of a separate window that pops up. I wish there were plugins to achieve the same in visual studio.

I don't know IDEA.
Are you now asking about something else, rather than making this work from C++?

I mean, for a simplest program like
int main()
{
cout<<"hello,spice";
}

When I create a Console App and copy the code to the file, and try to run it, a Microsoft Visual Studio Debug Console will pop up, in which it will display hello, spice, the function I want is, hello, spice will appear in some sub-window of the IDE, such as output-window instead of Debug Console

This project does try and provide a solution for the scenario you describe, but it does it by creating a separate version of the static class (System.Console) that .NET apps use to write to the console.
As C++ uses a different approach to write to the console window it is not possible/practical to create a version of this library for use within a C++ codebase.


Adding support for WLS 12.1.3 Update 1 ZIP Distribution

Support to install WLS 12.1.3 Update 1, Checksum included, and .gitignore updated.

Resolved in pull request #11.


Clean up pure pursuit

Mainly getting the types straight and making the waypoints and segments namedtuples, but I've also cleaned up the maths in the find_intersections method a bit.

Codecov Report

Merging #23 into master will increase coverage by 0.95%.
The diff coverage is 93.02%.

@@            Coverage Diff             @@
##           master      #23      +/-   ##
==========================================
+ Coverage   78.44%   79.39%   +0.95%
==========================================
  Files          18       18
  Lines         988      995       +7
  Branches      101      101
==========================================
+ Hits          775      790      +15
+ Misses        190      183       -7
+ Partials       23       22       -1

Impacted Files
Coverage Δ

tests/test_pure_pursuit.py
100% <100%> (ø)
:arrow_up:

autonomous/autonomous.py
64% <100%> (-0.36%)
:arrow_down:

autonomous/test.py
68.18% <50%> (+8.18%)
:arrow_up:

utilities/pure_pursuit.py
84.61% <92.42%> (+1.28%)
:arrow_up:

pyswervedrive/chassis.py
99.28% <0%> (+3.57%)
:arrow_up:

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 740fc74...3051827. Read the comment docs.


Exec tachikoma update<PHONE_NUMBER>5748

:hamster::hamster::hamster:   Strategy david
Powered by David Update as a Service - Tachikoma.io Announcement: 2015-06-28 Updated permissions - Tachikoma.io

Coverage decreased (-2.2%) to 75.87% when pulling 45c581b0d49afbaceb0d2cc691d83cb2e76d82cc on pazjacket:tachikoma/update-20170101165748 into 561aeec19d8836578f0838fe4cbff883050bb66d on festivals-tech:master.

Coverage decreased (-2.2%) to 75.87% when pulling 45c581b0d49afbaceb0d2cc691d83cb2e76d82cc on pazjacket:tachikoma/update-20170101165748 into 561aeec19d8836578f0838fe4cbff883050bb66d on festivals-tech:master.


Error in extendsPage

Getting this error trying to add the plugin to our site:
⠹ Initializing and preparing dataerror error in hook extendsPage from vuepress-plugin-open-graph
✖ Initializing and preparing data - failed in 9.22s
RangeError: Invalid time value
    at Date.toISOString (<anonymous>)
    at Object.extendsPage [as hook] (file:///Users/forrestevans/Documents/GitHub/docs/node_modules/vuepress-plugin-open-graph/dist/server/index.js:19:62)
    at Object.process (file:///Users/forrestevans/Documents/GitHub/docs/node_modules/@vuepress/core/dist/index.js:681:37)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async createPage (file:///Users/forrestevans/Documents/GitHub/docs/node_modules/@vuepress/core/dist/index.js:568:3)
    at async Promise.all (index 296)
    at async resolveAppPages (file:///Users/forrestevans/Documents/GitHub/docs/node_modules/@vuepress/core/dist/index.js:580:17)
    at async appInit (file:///Users/forrestevans/Documents/GitHub/docs/node_modules/@vuepress/core/dist/index.js:603:15)
    at async file:///Users/forrestevans/Documents/GitHub/docs/node_modules/@vuepress/cli/dist/index.js:487:7
    at async file:///Users/forrestevans/Documents/GitHub/docs/node_modules/@vuepress/utils/dist/index.js:106:20

Using the latest build of 2.0.0-beta.67

It seems like this line assumes every page without a layout specified will have a date entry.  We have some pages with Frontmatter but may not have a date.  Am I on the right track?
https://github.com/azat-io/vuepress-plugin-open-graph/blob/44cbebec134a9ca4e1ab5f4daf40b6ba6a37c234/server/index.ts#L37


Use AppDesigner special file handler in AbstractSpecialFileProvider

Customer scenario
Opening project properties, choosing the Resources page and choosing This project does not contain a default resources file. Click here to create one. fails with:
---------------------------
Microsoft Visual Studio
---------------------------
Could not find the file 'c:\users\davkean\documents\visual studio 2017\Projects\ConsoleApp49\ConsoleApp49\Resources.resx'.
---------------------------
OK
---------------------------

Bugs this fixes:
https://github.com/dotnet/roslyn-project-system/issues/1078
Workarounds, if any
Manually adding a resource file to the project file.
Risk
Low.
Performance impact
Low
Is this a regression from a previous update?
No
Root cause analysis:
We didn't miss this - we simply hadn't implemented support, other bugs that were recently fixed prevented us from hitting this path.
Added unit tests - but we still don't have integration tests yet to test the full path.
How was the bug found?
Ad hoc testing.
Developer Notes
Changed AbstractSpecialFileProvider to call through the AppDesigner special file handler instead of handling/creating itself.

tag @dotnet/project-system @srivatsn

I am surprised there were no test changes here. Don't we have tests for any of these classes?

See above:

Fixing tests to come...

Aha! Missed it

tag @MattGertz for review.

Shiproom Approved to Merge


Open Source Request

Hello @benborgers,
Awesome work - not just including this project! Was creeping through your other repositories as well!
I stumbled upon this as I needed it for an upcoming project instead of other solutions. (such as converting the text file manually to JSON key in client)
I was curious if you would be willing to open source / license this project as open source. So there is no confusion about the uses for hosting this & possibility get more involvement in contributors to fix some errors such as range value errors etc.
Thank you in advanced & thank you for hosting this freely!

Yeah definitely! Do you mean by including a license file in the repository?

@benborgers correct!
Right now almost all your work does not have license's attached - which is fine.
But keep in mind if you are looking to Open Source your work - assuming you are.
If a repository has no license, then all rights are reserved and it is not Open Source or Free. You cannot modify or redistribute this code without explicit permission from the copyright holder.

Meaning - no commercial business can host this software LEGALLY as you own all rights to it by not including licenses dictating what they can and can't do with it
So other's know what the copyright stance is / the way your work can be legally.  Assuming you are familiar with the process.
Debating on the license you choose - would define those rights to your work.
MIT License for example - would wave all rights to the work and let anybody & anyone do as they please with the work.


Choice and BProgram to prism conversion added

TODO:

Define execution behavior for Choice statements
Generate docs?

Hi @zisserj, thanks!
There's no need to generate the docs, I'll do it later.
Just make sure all classes and relevant methods have docstrings.
Additionally, it would be great if you can add comments on the monty example. It will improve the readability and people will better understand it.


ModuleNotFoundError: No module named 'blessed'

I'm on macOS Catalina and receive the above error when attempting to launch pg_activity. Funny thing is that I do have blessed installed in my env -- version 1.18.0.
I'm running pg_activity version  2.1.2.
Full traceback:
Traceback (most recent call last):
  File "/Users/******/.pyenv/versions/data-3.9.1/bin/pg_activity", line 35, in <module>
    from blessed import Terminal
ModuleNotFoundError: No module named 'blessed'

Can you try to install as pip install https://github.com/dalibo/pg_activity/archive/entry-points.zip? This corresponds to #198; if it works for you, I'll merge it and ship a release. Thanks.

Unfortunately, that did not fix my issue:
pip install https://github.com/dalibo/pg_activity/archive/entry-points.zip
Collecting https://github.com/dalibo/pg_activity/archive/entry-points.zip
  Downloading https://github.com/dalibo/pg_activity/archive/entry-points.zip
     \ 296 kB 1.0 MB/s
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
    Preparing wheel metadata ... done
Requirement already satisfied: attrs in ./.pyenv/versions/3.9.1/envs/data-3.9.1/lib/python3.9/site-packages (from pg-activity==2.1.2) (20.3.0)
Requirement already satisfied: humanize in ./.pyenv/versions/3.9.1/envs/data-3.9.1/lib/python3.9/site-packages (from pg-activity==2.1.2) (3.2.0)
Requirement already satisfied: psutil>=2.0.0 in ./.pyenv/versions/3.9.1/envs/data-3.9.1/lib/python3.9/site-packages (from pg-activity==2.1.2) (5.8.0)
Requirement already satisfied: blessed in ./.pyenv/versions/3.9.1/envs/data-3.9.1/lib/python3.9/site-packages (from pg-activity==2.1.2) (1.18.0)
Requirement already satisfied: six>=1.9.0 in ./.pyenv/versions/3.9.1/envs/data-3.9.1/lib/python3.9/site-packages (from blessed->pg-activity==2.1.2) (1.15.0)
Requirement already satisfied: wcwidth>=0.1.4 in ./.pyenv/versions/3.9.1/envs/data-3.9.1/lib/python3.9/site-packages (from blessed->pg-activity==2.1.2) (0.2.5)
Requirement already satisfied: setuptools in ./.pyenv/versions/3.9.1/envs/data-3.9.1/lib/python3.9/site-packages (from humanize->pg-activity==2.1.2) (54.1.2)
(data-3.9.1)[debarrosa ~]$ pg_activity -h wsj-data-journalism-db-1.chthw1jg4rom.us-east-1.rds.amazonaws.com -d us_fed_contracts -U wsj_scraper --rds
Traceback (most recent call last):
  File "/Users/debarrosa/.pyenv/versions/data-3.9.1/bin/pg_activity", line 35, in <module>
    from blessed import Terminal
ModuleNotFoundError: No module named 'blessed'

  File "/Users/******/.pyenv/versions/data-3.9.1/bin/pg_activity", line 35, in <module>
    from blessed import Terminal
ModuleNotFoundError: No module named 'blessed'

That file should not be that long (35 lines here, only 10 lines in a normal installation); it seems that the installation did nothing, probably with a "Requirement already satisfied" message.
Can you try pip install -I https://github.com/dalibo/pg_activity/archive/entry-points.zip?

That worked and fixed the issue!

Just released version 2.1.3 with the fix; you can now pip install -U pg_activity.


Network changes for k_timeout_t

Initial set of patches for networking related code. This is still very much WIP and more patches will be added to this set after conversions are done.

Converted most of the networking code to use k_timeout_t.
Still missing:

lwm2m
mqtt
canbus related stuff
modem
I am hoping volunteers for converting above.

@jukkar I can handle MQTT.

@jukkar, so does this have to go together with another PR, as in: if another PR is merged, and this isn't, we have broken build? If so, can you link to this PR in the description?
And if not, the usual comment applies: can we please avoid gigantic, hard to review PRs? Can this be split into a few smaller PRs, e.g. per-sub-subsystem to allow for attentive review? (And actually, could have been done gradually, like convert one part of code, see it working well (or break), use that experience to convert next part of code).

so does this have to go together with another PR, as in: if another PR is merged, and this isn't, we have broken build? If so, can you link to this PR in the description?

This PR should go in first as we are not disabling legacy timeout api yet.

And if not, the usual comment applies: can we please avoid gigantic, hard to review PRs? Can this be split into a few smaller PRs, e.g. per-sub-subsystem to allow for attentive review? (And actually, could have been done gradually, like convert one part of code, see it working well (or break), use that experience to convert next part of code).

There are lot of commits but they are small and should be easy to review.
I am not going to start to split this one, the amount of commits would not be lower anyway.
The sanity passes so I am hoping I did not make a mistake, but of course more testing is needed after this is merged.

Updated according to comments. Removed K_MS_TIMEOUT and introduced NET_WAIT_FOREVER to be used in those networking APIs that use millisecond timeouts.

Updated according to comments.


flash msg not working

follow the wiki and testing the flash msg and it seem like not working.
I belive it should be after $app->flash('key','msg) then we can use $flash[key] in the template, but currently on dev branch it doesn't really work.

Have you enabled a PHP session, or have you added the Slim_Middleware_SessionCookie middleware to your app?

yup tested both also can't, nothing in the $_SESSION

Hmm. Can you send me code that demonstrates this not working? I'm testing the development branch right now and it works as expected for me. If I can see your code I may be able to see what is causing this not to work. This example app does work for me:
require 'Slim/Slim.php';

$app = new Slim();

$app->add('Slim_Middleware_SessionCookie');

$app->get('/foo', function () use ($app) {
    $app->flash('info', 'Hello there');
    $app->redirect('/bar');
});

$app->get('/bar', function () use ($app) {
    var_dump($_SESSION);
});

$app->run();

Josh

Also, an important bug fix was pushed yesterday that fixed a bug in Slim_Session_Middleware. If you are using that middleware, be sure you pull down the latest code.

ya i m the one reporting that bug : ) , k will try to post it later when I have time.

haha. Sorry about that. It's early and I've yet to have my coffee :P

Any luck? I'm unable to duplicate. Let me know tonight. Otherwise I'll close this out and we can continue troubleshooting in a forum thread at http://help.slimframework.com.

sorry I was rather busy for the past 2 day, just did another round testing and found the problem.
found the problem after I flash and redirect, i redirect to a url without ending slash and my controller is the one with ending slash, so it redirect one more time, thats why the session is gone.
closing this issue.

sorry just a follow up base on the Doc ,
The message in the example below will be available in the variable flash['info'] in the current request’s View templates.
does it mean i should be able to use $flash['info'] variable on the render() template ?

@imknight Yes, but how you access the variable in your templates depends entirely on the template system you are using. If you are using the default Slim_View, you will access it with $flash['info']. If you are using Twig, you will access it with flash.info, and so on.


Bern Features: Tool tip for categories belonging to an event/series always in english.

Tool tip (or hover over) for the new option to change between categories belongig to an event or a series is always in english. Please implement german translation.
I suggest the following wording, as "Event" does not really fit in this circumstances:
Serie: "Kategorie gehört zur ganzen Serie"
Event: "Kategorie gehört nur zum Video"

Looks fine and can be closed.


A question about deconfounder_tutorial

Thank you for providing this detailed tutorial.
I want to ask a question. After training the factor model, you use the following codes to generate samples (x_generated).
 with ed.interception(replace_latents(w_sample, z_sample)):
            generate = ppca_model(
            data_dim=data_dim, latent_dim=latent_dim,
            num_datapoints=num_datapoints, stddv_datapoints=stddv_datapoints)
 with tf.Session() as sess:
        x_generated, _ = sess.run(generate)

However, in the ppca_model, the mean matrix is masked by "1-holdout_mask":
x = ed.Normal(loc=tf.multiply(tf.matmul(z, w), 1-holdout_mask),
                scale=stddv_datapoints * tf.ones([num_datapoints, data_dim]),
                name="x")

Should we remove the mask "1-holdout_mask" when generating samples and then mask it with "holdout_mask" instead?

Yes. I just pushed an update. Thank you!


Problems about the async_hooks module

Hi. I'm using GraalVM 21.1.0 with Node.js installed (by gu install nodejs) on macOS 11.4. I want to acquire triggerAsyncId and executionAsyncId using the async_hooks module for indicating the relationship between async contexts, but the execution results of Node.js and $GRAALVM/bin/node are different.
Code
const fs = require('fs');
const async_hooks = require('async_hooks');

async_hooks.createHook({
    init(asyncId, type, triggerAsyncId, resource)
    {
        fs.writeSync(1, `init: ${triggerAsyncId} -> ${asyncId}\n`);
    },
    before(asyncId)
    {
        fs.writeSync(1, `before: ${async_hooks.triggerAsyncId()} -> ${async_hooks.executionAsyncId()}\n`);
    },
}).enable();

let globalVar = 1;

async function setTimeoutAsync(timeout)
{
    return new Promise(resolve =>
    {
        setTimeout(() => resolve(), timeout);
    });
}

async function asyncFunc()
{
    globalVar = 2;
    await setTimeoutAsync(10);
    fs.writeSync(1, `${async_hooks.triggerAsyncId()} -> ${async_hooks.executionAsyncId()}\n`);
    globalVar = 3;
    await setTimeoutAsync(10);
    fs.writeSync(1, `${async_hooks.triggerAsyncId()} -> ${async_hooks.executionAsyncId()}\n`);
    globalVar = 4;
}

asyncFunc();

Output of Node.js 14.16.1
init: 1 -> 2
init: 1 -> 3
init: 1 -> 4
init: 1 -> 5
init: 3 -> 6
before: 1 -> 3
init: 4 -> 7
before: 1 -> 5
before: 4 -> 7
before: 3 -> 6
3 -> 6
init: 6 -> 8
init: 6 -> 9
init: 6 -> 10
init: 8 -> 11
before: 6 -> 8
init: 9 -> 12
before: 6 -> 10
before: 9 -> 12
before: 8 -> 11
8 -> 11

Output of $GRAALVM/bin/node (also 14.16.1)
init: 1 -> 2
init: 1 -> 3
init: 1 -> 4
init: 1 -> 5
init: 4 -> 6
before: 1 -> 5
before: 4 -> 6
0 -> 0
init: 0 -> 7
init: 0 -> 8
init: 0 -> 9
init: 8 -> 10
before: 0 -> 9
before: 8 -> 10
0 -> 0

The problem is that if async_hooks.triggerAsyncId() or async_hooks.executionAsyncId() returns 0, it's impossible for me to get the relationship between async contexts.

And it seems that the difference only appears with async/await code?

@Soulike Thanks for reporting the issue
we will look into it and get back to you


UnboundLocalError: local variable 'dt' referenced before assignment

Netbox version (v3.2.8)
When run ./nb-dt-import.py I get error message:
# python3 ./nb-dt-import.py
Package devicetype-library is already installed, updating /opt/netbox_device_type_import/repo
Pulled Repo https://github.com/netbox-community/devicetype-library.git
No Vendors Specified, Gathering All Device-Types
88 Vendors Found
1882 Device-Types Found
[{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{"slug":["manufacturer with this slug already exists."]},{},{}]
{"manufacturer":["Related object not found using the provided attributes: {'name': '3Com', 'slug': '3com'}"]}
Traceback (most recent call last):
  File "./nb-dt-import.py", line 853, in <module>
    main()
  File "./nb-dt-import.py", line 826, in main
    createDeviceTypes(deviceTypes, nb)
  File "./nb-dt-import.py", line 680, in createDeviceTypes
    dt.id, nb)
UnboundLocalError: local variable 'dt' referenced before assignment

Also get error when use Netbox v.3.3.5
# python3 ./nb-dt-import.py
Package devicetype-library is already installed, updating /opt/netbox_device_type_import/repo
Pulled Repo https://github.com/netbox-community/devicetype-library.git
No Vendors Specified, Gathering All Device-Types
88 Vendors Found
1882 Device-Types Found
Manufacturer Exists: SNR - 20
Manufacturer Exists: Extreme Networks - 9
Manufacturer Exists: Dell - 12
Manufacturer Exists: MikroTik - 10
Manufacturer Exists: Panduit - 24
Manufacturer Exists: Huawei - 7
Manufacturer Exists: Eltex - 22
Manufacturer Exists: Cisco - 5
Manufacturer Exists: Arista - 8
Manufacturer Exists: Vertiv - 18
Manufacturer Exists: Juniper - 1
[{},{},{},{},{},{},{},{},{},{},{},{},{},{},{"slug":["manufacturer with this slug already exists."]},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{"slug":["manufacturer with this slug already exists."]},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{},{}]
{"manufacturer":["Related object not found using the provided attributes: {'name': '3Com', 'slug': '3com'}"]}
Traceback (most recent call last):
  File "./nb-dt-import.py", line 853, in <module>
    main()
  File "./nb-dt-import.py", line 826, in main
    createDeviceTypes(deviceTypes, nb)
  File "./nb-dt-import.py", line 680, in createDeviceTypes
    dt.id, nb)
UnboundLocalError: local variable 'dt' referenced before assignment

Workrounded adding manually the manufacturer's name (Devices > Manufacturers) and then pulling just that brand with
--vendors. Using NB version 3.3.6 here.

Hello @blackbicbiro, the NetBox Community will be taking over this repository. Due to the fact that this issue has been around for a very long time and we have updated the requirements within this repository as well as the new codebase for NetBox and DeviceType-Library, I am going to close it. Should you encounter any further issues, please feel free to contact us. Thank you!


missing app/assets/javascript/application.js

Hi!
When I create the app, Rails doesn't generate the app/assets/javascripts directory, nor the related manifesto.
What I have to do?

@Marialuisaclaro - This is more than likely because you're using rails 6 in your app. If this is the case you will just have an app/javascript directory and you will need to install jQuery in a webpack using Yarn.

this app should probably be upgraded to Rails 6
would be very useful for a lot of people


ISSUES: Rails google analytics api error

Using the gem 'google-api-client', require: 'google/apis/analytics_v3'
However, i am getting this error
ArgumentError in WelcomeController#analytics
Missing token endpoint URI.

(Error highlighted on Analytics method on @account_summaries)
Controller file
class WelcomeController < ApplicationController

    def redirect
      client = Signet::OAuth2::Client.new({
        client_id: 'APP ID',
        client_secret: 'APP SECRET',
        authorization_uri: 'https://accounts.google.com/o/oauth2/auth',
        scope: Google::Apis::AnalyticsV3::AUTH_ANALYTICS_READONLY,
        redirect_uri: url_for(:action => :oauth2callback)
      })

      redirect_to client.authorization_uri.to_s
    end

    def oauth2callback
        client = Signet::OAuth2::Client.new({
        client_id: 'APP ID',
        client_secret: 'APP SECRET',
        token_credential_uri: 'https://accounts.google.com/o/oauth2/token',
        redirect_uri: url_for(:action => :oauth2callback),
        code: params[:code]
      })

      response = client.fetch_access_token!

      session[:access_token] = response['access_token']

      redirect_to url_for(:action => :analytics)

    end

    def analytics
        client = Signet::OAuth2::Client.new(access_token: session[:access_token])

        access_token = Accesstoken.new 'client'

        service = Google::Apis::AnalyticsV3::AnalyticsService.new

        service.authorization = access_token

        @account_summaries = service.list_account_summaries(options: { authorization: client })

    end

end

I did follow an advice stated here - Missing token endpoint URI while using a valid access_token - but this still does not work.
class Accesstoken < ActiveRecord::Base
  attr_reader :token
  def initialize(token)
    @token = token
  end

  def apply!(headers)
    headers['Authorization'] = "Bearer #{@token}"
  end

end

Where is the issue??
Please note -
If in @account_summaries I put in authorization: access_token instead of client, it would then return Unauthorized. What is going on?

The error means exactly that -- you're not configuring the oauth client with enough information.
I suggest using https://github.com/google/google-auth-library-ruby/ instead of signet directly.

@sqrrrl this worked a while ago, and now i get same issue. So something changed.

I'm not using OAuth2 for authentication (I'm using server credentials), but I recently updated the gem after a while and also got the Missing token endpoint URI. error. My solution was to use the #make_creds method.
I changed
Google::Auth::ServiceAccountCredentials.new(
  json_key_io: path_to_json_file,
  scope: ['https://www.googleapis.com/auth/analytics.readonly']
)

to
Google::Auth::ServiceAccountCredentials.make_creds(
  json_key_io: path_to_json_file,
  scope: ['https://www.googleapis.com/auth/analytics.readonly']
)

and everything works fine now.

@arjunmenon Did you get this to work? I've had the exact same problem for 2 days now. Getting "Missing token endpoint URI". @23tux Would you mind showing an example with the rest of the code for that?

You can follow this excellent guide here. This would surely get things started - http://blog.jieck.com/server-side-integrates-google-analytics-api-with-ruby/


Add attestation clients

closes #3444
Pull Request Checklist
Please leverage this checklist as a reminder to address commonly occurring feedback when submitting a pull request to make sure your PR can be reviewed quickly:
See the detailed list in the contributing guide.

[x] C++ Guidelines
[x] Doxygen docs
[x] Unit tests
[x] No unwanted commits/changes
[x] Descriptive title/description

[x] PR is single purpose
[x] Related issue listed

[x] Comments in source
[x] No typos
[x] Update changelog
[x] Not work-in-progress
[x] External references or docs updated
[x] Self review of PR done
[x] Any breaking changes?

/azp run azure - core

/azp run cpp - core

/azp run cpp - core

/azp run cpp - keyvault


you better update some names (i needted to do this because glow api)

will resolve these changes later since it requires updating our other plugins


Is there a way to monitor connexion ?

Hey,
is there a way to monitor connexion loss/re-establish. (without sending a request)
Thank you,

You mean network state? Eg. getting info when you lose wireless, cellular-data, or both, and when you regain them?

yes !

That's not question of this library, use the official way, documented here:
https://developer.android.com/training/monitoring-device-state/connectivity-monitoring.html

Oops sorry, didn't know that we could check it by broadcast receiver ! Thank you !

You're welcome :)


Plugin stops without error after multiple checks

Hello there,
I am running the cachet-monitor on a CoreOS cluster. I've setup the fleet unit file and config file. Everything is running smoothly except the monitor docker container appears to just stop running after 9 checks. No errors in any logs.
Docker logs
core@farm-3 ~ $ docker logs platform-monitor
2015/08/18 14:34:04 /go/src/github.com/castawaylabs/cachet-monitor/main.go:12: System: platform-monitor, API: https://technicstatus.pagefortress.com/api/v1
2015/08/18 14:34:04 /go/src/github.com/castawaylabs/cachet-monitor/main.go:13: Starting 1 monitors:
2015/08/18 14:34:04 /go/src/github.com/castawaylabs/cachet-monitor/main.go:15:  Technic Platform: GET http://www.technicpack.net & Expect HTTP 200
2015/08/18 14:34:04 /go/src/github.com/castawaylabs/cachet-monitor/main.go:17:  - Logs lag to metric id: 1
2015/08/18 14:34:04 /go/src/github.com/castawaylabs/cachet-monitor/main.go:21:
2015/08/18 14:34:05 /go/src/github.com/castawaylabs/cachet-monitor/cachet/monitor.go:84: http://www.technicpack.net 0.00% Down at<PHONE_NUMBER>. Threshold: 80.00%
2015/08/18 14:34:06 /go/src/github.com/castawaylabs/cachet-monitor/cachet/monitor.go:84: http://www.technicpack.net 0.00% Down at<PHONE_NUMBER>. Threshold: 80.00%
2015/08/18 14:34:07 /go/src/github.com/castawaylabs/cachet-monitor/cachet/monitor.go:84: http://www.technicpack.net 0.00% Down at<PHONE_NUMBER>. Threshold: 80.00%
2015/08/18 14:34:08 /go/src/github.com/castawaylabs/cachet-monitor/cachet/monitor.go:84: http://www.technicpack.net 0.00% Down at<PHONE_NUMBER>. Threshold: 80.00%
2015/08/18 14:34:09 /go/src/github.com/castawaylabs/cachet-monitor/cachet/monitor.go:84: http://www.technicpack.net 0.00% Down at<PHONE_NUMBER>. Threshold: 80.00%
2015/08/18 14:34:10 /go/src/github.com/castawaylabs/cachet-monitor/cachet/monitor.go:84: http://www.technicpack.net 0.00% Down at<PHONE_NUMBER>. Threshold: 80.00%
2015/08/18 14:34:11 /go/src/github.com/castawaylabs/cachet-monitor/cachet/monitor.go:84: http://www.technicpack.net 0.00% Down at<PHONE_NUMBER>. Threshold: 80.00%
2015/08/18 14:34:12 /go/src/github.com/castawaylabs/cachet-monitor/cachet/monitor.go:84: http://www.technicpack.net 0.00% Down at<PHONE_NUMBER>. Threshold: 80.00%
2015/08/18 14:34:13 /go/src/github.com/castawaylabs/cachet-monitor/cachet/monitor.go:84: http://www.technicpack.net 0.00% Down at<PHONE_NUMBER>. Threshold: 80.00%

Journal logs
Aug 18 14:34:00 farm-3 systemd[1]: Stopped Technic Platform monitor service for Cachet.
Aug 18 14:34:04 farm-3 systemd[1]: Starting Technic Platform monitor service for Cachet...
Aug 18 14:34:04 farm-3 docker[24066]: platform-monitor <-- killed
Aug 18 14:34:04 farm-3 docker[24074]: platform-monitor <-- removed
Aug 18 14:34:04 farm-3 systemd[1]: Started Technic Platform monitor service for Cachet.
Aug 18 14:34:04 farm-3 docker[24081]: 134f51c3542c545d0ae0bdd6badee21e300756b604a3ed1e46d75eb <-- run
Aug 18 14:34:14 farm-3 docker[24102]: platform-monitor <-- stopped unexpectedly

Monitor config.json
{
  "api_url": "https://technicstatus.pagefortress.com/api/v1",
  "api_token": "<redacted>",
  "monitors": [
    {
      "name": "Technic Platform",
      "url": "http://www.technicpack.net",
      "metric_id": 1,
      "component_id": 1,
      "threshold": 80,
      "expected_status_code": 200,
      "strict_tls": false
    }
   ],
   "insecure_api": true
}

Hmm that's strange. Tried running it interactively?

Yeah, running the docker container without the fleetctl overhead results in no crash. Must be something within fleetctl that is causing the stop.

Hmm thats interesting, i'll take a look.
Is using docker compose a viable alternative for you? (i havent tried it though)

@GenPage let us know if this is still an issue


Miscellaneous development

This pull fixes a lot of the formatting issues.

Oh, I actually didn't think of that. I definitely should. Thanks!


fixing incorrect English

Hi, thanks for contribution. Can you please also edit same language in https://github.com/pleabargain/js-slides/blob/317c555269c9649e80629bc88cc136f3fbc54295/npm/index.md ? the index.html file is generated from markdown.

Outdated as of #49


Add tmpexport.*.apk.idsig file to cleanup

[!WARNING]
Not tested yet.

This PR adds .idsig files to the Android cleanup phases.
Fixes #100101

Superseded by #100103. Didn't see the PR before doing mine, sorry.


ssl: add kctl engine for linux

Under Linux, cryptodev and afalg can be used to implement the offload of cryptographic calculations, unfortunately, neither of them supports asymmetric encryption.  In order to support asymmetric encryption algorithms, I implemented this kctl engine using keyutils.
I used the speed command to benchmark rsa2048. My platform is a 8-cpu virtual machine with a pci pass-through intel QAT card(C62xx), here are the benchmark result:
openssl speed -engine kctl -elapsed -multi 85 rsa2048 :

openssl speed  -elapsed -multi 85 rsa2048 :

We are going to use the kctl engine and nginx together, so there is currently no adaptation to openssl3.0. If this pr has the possibility of being merged, I am very happy to add tests/documentation, and make the code compatible to openssl3.0.
Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

Signed-off-by: Lei He<EMAIL_ADDRESS>

Unfortunately this is a feature PR against the 1.1.1 branch. This is not going to be accepted as we do not allow new features in stable branches. However even if this was rebased and submitted against the master branch it would not be merged as this adds a new functionality in terms of a new ENGINE. However ENGINEs are deprecated as of 3.0 release and we are not going to include any new engine.
I would like to suggest to you to create a separate project where your engine would be maintained and released.

Thank you very much for your suggestion. As far as I know, nginx does not support OpenSSL 3.0 now. Thanks for the dynamic loading capability provided by OpenSSL, so that I can try to maintain this engine in an separate repo.
In addition, maybe I can try to abstract it into a provider in OpenSSL 3.0? I think it should be a good idea to offload cryptographic operations through the standard system calls provided by the Linux kernel.

Unfortunately this is a feature PR against the 1.1.1 branch. This is not going to be accepted as we do not allow new features in stable branches. However even if this was rebased and submitted against the master branch it would not be merged as this adds a new functionality in terms of a new ENGINE. However ENGINEs are deprecated as of 3.0 release and we are not going to include any new engine.
I would like to suggest to you to create a separate project where your engine would be maintained and released.

Yes, creating a provider would certainly make sense for OpenSSL 3.0. However I cannot promise we would include such provider in the OpenSSL source tree either. The intention is to allow third party providers and the provider API is supposed to be stable so the third party providers should not require arbitrary OpenSSL release to work with.

Intel already provides a QAT engine, and my understanding is that Intel wants to create a QAT provider.

This PR has the label 'hold: cla required' and is stale: it has not been updated in 30 days. Note that this PR may be automatically closed in the future if no CLA is provided.  For CLA help see https://www.openssl.org/policies/cla.html

This PR has the label 'hold: cla required' and is stale: it has not been updated in 61 days. Note that this PR may be automatically closed in the future if no CLA is provided.  For CLA help see https://www.openssl.org/policies/cla.html

This PR has been closed.  It was waiting for a CLA for 90 days.

This PR has been closed.  It was waiting for a CLA for 91 days.

This PR has been closed.  It was waiting for a CLA for 92 days.

This PR has been closed.  It was waiting for a CLA for 92 days.

This PR has been closed.  It was waiting for a CLA for 93 days.

This PR has been closed.  It was waiting for a CLA for 94 days.

Why do we get autoclose notifications every day?

Why do we get autoclose notifications every day?

Because the bot does not have sufficient permissions to really close the issue.


v4, Can't use MartenIgnore attribute

The solution for #1891 added more uses to the MartenIgnoreAttribute, but it still can't be accessed outside of Marten. #1894 attempts to resolve this naively, but perhaps a broader rename is appropriate if the attribute is not just intended for the CodeGeneration namespace.

Fixed by https://github.com/JasperFx/marten/pull/1894
@willhausman should be available in the next RC or the major v4 👍
If you need it sooner, please ping us on Gitter, then we'll release an RC more shortly.


Sync docker install script with upstream

Check diff between our install-docker scripts and upstream from https://get.docker.com.
Also check client/server versions installed.

Next minor version we should just move to using the source (https://github.com/docker/docker-install/blob/master/install.sh) and setting the environment variable VERSION. Possibly we can also upstream the rancheros logic as that is the last part which differs from upstream.

This is done in 20.10.0 and up


Nick: Watch xrptousd.net

Nick requests the watch of the watch_keyword xrptousd\.net. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

xrptousd\.net has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by tripleee in Charcoal HQ


ng-model and ng-change not worked when element's input type is number

input type="text" name="bankCardNo" ng-model="bankData.bankCardNo"
required="true" placeholder="Bank Card" maxlength="20" ng-change="bankCardChange()"
this code  is worked good, but when I change to <input type="number" ... ,  a error will happen:
TypeError: undefined is not a function
at Scope.$scope.bankCardChange
Maybe it is a angularJs bug.

I am getting the same issue with 'email' and 'tel' also.
Ionic: 1.0.0-beta.14 Angular: 1.3.6

ng-change will work on anything whether it is "number" or "text". But make sure the entered value is valid one after all your validations. Without any validation check if you still have doubts.


Create Room without joining (RTCMultiConnection 3.0)

Can I create rooms without access and store rooms on server?
Thanks


Adding kochava links

Proposed changes
In this PR I have added links to Kochava in our docs. This change has been made upon their request in this ticket https://segment.zendesk.com/agent/tickets/441026. Let me know if you're against adding these links and you want me to push back on them.
Merge timing
No urgent. Feel free to merge once approved.

Thanks @gezgblack. I don't have an issue with including links. I will tweak the language a little bit and also add a link to Tune where applicable, so we do not appear to favor one over the other.


Fix some errors and add some default parameters

hey @viagram, made some changes based on these suggestions. Let me know if it addresses issues you've raise in this PR. Since a lot of people disable systemd-resolved to get this working, I introduced a secondary method to get the public IP through DNS. I also overhauled the whole flag system with POSIX ones which support short flags (-c).

hey @viagram, made some changes based on these suggestions. Let me know if it addresses issues you've raise in this PR. Since a lot of people disable systemd-resolved to get this working, I introduced a secondary method to get the public IP through DNS. I also overhauled the whole flag system with POSIX ones which support short flags (-c).

👍👍👍 Very nice


GitHub Actions nightly build triggered on forks

It turns out nightly builds have been running on my fork for 5 months, so since commit eb3717130a probably. I only recently started noticing, because I happened to have the most recent commit on master being one that was failing. I suspect nightly builds are running on all forks.
I'm not yet very familiar with GitHub Actions, but it looks like we're missing something important here. Any ideas @andyfaff, @tylerjereddy?

Hmm, I do see these kinds of notifications sometimes for my fork too I think. I  always get confused because we were using the "nightly build of Python 3.9 for CI" but GitHub actions also allows tests to run "nightly," which is two different uses of "nightly" of course.

I was seeing those notifications on my fork as well, when I asked about it on scipy-dev Andy suggested:
https://github.community/t/stop-github-actions-running-on-a-fork/17965/2
which I setup on my fork and I haven't seen the nightly builds on my fork email since.
Admittedly the emails are infrequent but that was almost 2 months ago.

I suspect nightly builds are running on all forks.

At the moment the Github Actions run on all public forks of public repos. I think GH is going to change that in the near future so that doesn't happen. However, it's a cool thing as it allows you to try out changes on your own fork before making an official PR. This kind of thing is also possible with travisCI, where the scipy defined CI matrix also runs on my fork. In that case the individual user has to opt in.

I always get confused because we were using the "nightly build of Python 3.9 for CI" but GitHub actions also allows tests to run "nightly," which is two different uses of "nightly" of course.

GH Actions does have the ability to run in a cron fashion. Our use of nightly is confusing, Tyler is correct in his explanation. In our context we're testing against the nightly builds of Python, rather than running a CI test nightly.  The CI run using GH Actions is triggered on a particular scipy fork when a commit is made to it. There are in fact two CI scripts on GH Actions, one that tests Nightly builds of Python (bleedingedge.yml) and one that tests on macOS 3.6/3.7/3.8 (testing.yml). The top of those scripts configure which events trigger the scripts, e.g. push to master, pull request, etc.
CI runs are not happening on all scipy forks every night, but only when forked repos have changes made to them.
As Lucas points out it's possible to disable the CI run on your own fork. I'm not sure if it's possible to do that globally, but I like using that CI run to try out experimental changes before making a PR. I don't know if it's possible to disable email notifications.

Sorry, I didn't remember correctly, what I said was totally incorrect. The "nightly build" runs every night against the nightly build of python 3.9, https://github.com/scipy/scipy/blob/02563b2c873f042f377ec19b08ac6c681b826897/.github/workflows/bleedingedge.yml#L10.
How did we want to change this? It was designed tonwork

It seems like we are missing this conditional in the workflows:
if: github.repository == ‘scipy/scipy’

The default should be to not run anything on forks, that is super wasteful for no good reason.

is super wasteful for no good reason

Agree. Perhaps the schedule line isn't needed at all.


Update amplicon sample sheet with currently used values

In [Header]:
Change value associated with "Assay" from "Amplicon" to "TruSeq HT"
Change value associated with "Chemistry" from "Default" to "Amplicon"
In [Settings]:
Add "Adapter" field and appropriate value as DNA
Add "AdapterRead2" field and appropriate value as DNA
(Scope of this issue includes FINDING OUT from wet lab what the appropriate values are and figuring out where to get the in Labman db).
In [Data]:
Check with Greg whether I5_Index_ID and index2 fields should be added to output (after index column)
Note: do NOT change "Well_Description" back to "Description" : Gail Ackermann requested this change as part of PR 268.
Raised in amplicon walkthrough 07/30/2018

Per Greg Humphrey: "The adapters are always the same and are located on our primers" and "the adapters are the same for 16S, 18S, and ITS".  Thus, the values for Adapter and Adapter2 will be the same in every amplicon sample sheet, regardless of primer set, etc.

Hi @AmandaBirmingham I have tackled this issue. See PR #318 . I talked with Greg and understood their requirement in the sample sheet format.


Data Logging while using simple_vehicle_control in OpenSCENARIO

Hi Team, I would like to log the ego and target vehicle location, speed and so on in a log file. Earlier I used to run the ego vehicle manually or using autopilot and have provide the file write in manual_control.py. Now since Iam running using simple_vehicle_control, I would like to know where I can provide the logging commands. Can i utilise the scenario runner python script. If yes can you kidnly let me know where I could place the scripts. Thanks in advance.
Desktop (please complete the following information):

OS: [e.g. Ubuntu 18.04]
CARLA Version [e.g. CARLA 0.9.11]
Python version [e.g. 2.7]
Open Scenario Version [e.g. 0.9.11]

@fabianoboril Hi Do you any insights on this please

Hi @fabianoboril  Would be great to hear your insights on this. Iam actually trying to log the data as well as try to interact with xosc file with a python script so that I can corporate my coordinate transformations and so on. I would like to know if an interaction between .py and xosc is possible and if yes where can I find the helpful steps for that. Thanks in advance

Data logging , I could do by inserting commands inside simple_vehicle_control python script , but the interaction with the openSCENARIO xosc file and the scenario runner python script seems very vague. Would be helpful to see any documentation regarding this

Thanks. Will look into how to do that . Any inputs regarding any documentation about interaction between scenario runner and openSCENARIO xosc file would also be very helpful

Hi   @fabianoboril
Iam getting this error while running python metrics_manager.py --metric srunner/metrics/examples/distance_between_vehicles.py --log srunner/metrics/data/DistanceBetweenVehicles.log
I have added /home/schemba/carla/scenario_runner-0.9.11/srunner/metrics/tools to the python path
Traceback (most recent call last):
File "metrics_manager.py", line 28, in
from srunner.metrics.tools.metrics_log import MetricsLog
ImportError: No module named metrics.tools.metrics_log
Can you kindly provide your insights on the same


add python 3.11 and github actions for tests

Hi,
I can split in 2 different PR if needed.
Travis-ci does not work anymore if I'm not wrong or should be configured again to work.
Johnny

Codecov Report

Merging #128 (45b16f4) into master (d639973) will decrease coverage by 1.42%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #128      +/-   ##
==========================================
- Coverage   95.53%   94.11%   -1.43%
==========================================
  Files           1        1
  Lines         852      850       -2
==========================================
- Hits          814      800      -14
- Misses         38       50      +12

Impacted Files
Coverage Δ

polib.py
94.11% <0.00%> (-1.43%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Merged, many thanks for this one !


Generate Security Group using ACK Runtime v0.15.2

Issue #, if available: https://github.com/aws-controllers-k8s/community/issues/489
Description of changes:

Adds Create/Delete functionality and tests for Security Group resource using ACK runtime v0.15.2 at this commit

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

It appears SecurityGroup by itself is not that useful. It really needs support for ModifySecurityGroupRules, right? This will be supported in a following PR probably by having a list of rules as a field on the spec?

That's correct; wanted to get Create/Delete out here in case other contributors were eagerly waiting.

/lgtm


Rounding in print summary

Two decimals places please

Any specific summaries or all of them?

Specifically the way we present them in print_summary methods, this is the rule I'd like to implement: https://twitter.com/jaimiegradus/status/1075000614499901440

Ok thanks, I'll have a stab


Update LayoutEditor.cshtml.

Wrapped strings in T delegate.
Fixes #6269

Good catch, I seem to have forgotten. Thank you.


concurrent connects crash + solution

I have used a previous version of the  driver to connect to two remote databases over slow connection, therefore setting up the connection was done in parallel. This crashed now and then with SEGV's as the malloc() in the client library are not necessarily thread safe.
The solution is to use different calls to setup the initial environment,
and to set the OCI_THREADED option:
Here is the diff:
@@ -70,22 +72,19 @@ func (d *OCI8Driver) Open(dsn string) (driver.Conn, error) {
        token := strings.SplitN(dsn, "@", 2)
        userpass := strings.SplitN(token[0], "/", 2)

-       rv := C.OCIInitialize(
-               C.OCI_DEFAULT,
+       rv := C.OCIEnvCreate(
+               (**C.OCIEnv)(unsafe.Pointer(&conn.env)),
+               C.OCI_THREADED,
+               nil,
                nil,
                nil,
                nil,
+               0,
                nil)
        if rv == C.OCI_ERROR {
                return nil, ociGetError(conn.err)
        }

-       rv = C.OCIEnvInit(
-               (**C.OCIEnv)(unsafe.Pointer(&conn.env)),
-               C.OCI_DEFAULT,
-               0,
-               nil)
-
        rv = C.OCIHandleAlloc(
                conn.env,
                &conn.err,

Should be fixed in latest.


Update workflow.yaml with a subPath

This reflects the move of this accelerator into this new repo

Hi, we have updated all workload.yaml files to only have a place-holder for the Git repo URL


add/remove on fantom not synced

add/remove events only synced up to april 20  (getEvents)
maybe other chains affected?

Events behind:
"POLYGON"	"SWAP"<PHONE_NUMBER>
"FANTOM"	"JOIN"<PHONE_NUMBER>
"FANTOM"	"EXIT"<PHONE_NUMBER>
"FANTOM"	"SWAP"<PHONE_NUMBER>
"AVALANCHE"	"SWAP"<PHONE_NUMBER>

Is that a timestamp or number of events?

timestamp

The syncing issue is caused by the missing block number in the existing data. To resolve this, I cleared the table of records where blockNumber = 0 for Fantom join/exits. All of the swap data for Fantom is missing the block number as well, to keep them i'll change syncing to be based on timestamps.


import_fields attribute

I did not find a way to specify import_fields - this might be helpful to export columns (e.g. for analytics) but only import a sub set of fields.
It might work overwriting get_import_fields() to check for import_fields attribute being set, use it if so and otherwise use get_fields.
If thats something you like, I can provide a PR.

@jensneuhaus yes, I've doing this myself, it would be great if you could provide a PR with tests!

Closing due to inactivity


[SPARK-37589][SQL] Improve the performance of PlanParserSuite

What changes were proposed in this pull request?
https://github.com/apache/spark/pull/34543 provides the parse method parseQuery. So we can reduce the time spent on executing PlanParserSuite by replace parsePlan.
Why are the changes needed?
Improve the performance of PlanParserSuite.
Does this PR introduce any user-facing change?
'No'.
Just update tests.
How was this patch tested?
Jenkins test.

Test build #146023 has started for PR 34841 at commit 3b08e4a.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50498/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50498/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/50498/

Test build #146023 has finished for PR 34841 at commit 3b08e4a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/146023/

ping @cloud-fan

Maybe it not worth.

Thank you for closing, @beliefer .


Add sautorun.require

Would allow for people to actually structure their code in modules rather than a massive single file. Safety is getting pretty huge
Still not sure about adding this kind of stuff because I'm afraid some might leak these to the global table where the functions could be abused.

When this is added I'll make it look local to a sautorun-rs/scripts dir rather than absolute

@Vurv78 wake


adding STARRED PSF fitting notebook

This adds a new notebook demonstrating how to use the STARRED PSF reconstruction module, within lenstronomy.
This is not directly comparable with the time_delay_cosmography_with_uncertain_psf.ipynb notebook because here, Starred reconstructs the PSF on super-sampled grid. Subsampling the PSF leads to a better accuracy on Ddt, even though this is not really significant.
Final model with Starred reconstruction (point_source_supersampling_factor = 3, supersampling_factor: 1):

Same config, with iterative PSF reconstruction:

Yes, if you turn 'verbose' to False in the kwargs_starred dictionary, STARRED is not printing anything in the terminal. I can make it False by default if needed.

View entire conversation on ReviewNB

This is the same thing in the original notebook! Gamma is set to 1.95 in the simulation but the fit converges to 1.99

View entire conversation on ReviewNB

The DDt is actually more accurate, but I think this is because you subsample the PSF.

View entire conversation on ReviewNB

And true Fermat potential is not plotted, so I can't tell if this is off or not. Probably not if Ddt is unbiased.

View entire conversation on ReviewNB

ah, we should plot the true Fermat potential (I thought it is outside). I will also do it for the other notebook

View entire conversation on ReviewNB

Yes, this can be merged!

I left a minor request on the notebook before merging. Can you please address it and then it's good to merge

This should be fixed in my latest commit.


Congatudo does not find segments if there are maps pending to be saved

Hello.
Sometimes, when the conga automatically goes out to do a cleanup and, for whatever reason, it detects that the map could have changed (when it's not the case, but there were things in the way anyway and it went crazy)
So in the application what I show you in image 1 appears. The issue is that as long as I do not assign YES or NO to the newly detected map, the remaining automations no longer work. This is the error that the log spits out:
[ERROR] MQTT: Error while handling MapSegmentationCapability/clean/set {
  payload: '{   "segment_ids": [ "12" ],   "iterations": 1 }',
  error: Error: Segment ID does not exist, or map was not loaded: 12
      at PropertyMqttHandle.setter (/snapshot/app/backend/lib/mqtt/capabilities/MapSegmentationCapabilityMqttHandle.js:48:35)
      at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
}

I'd say it's more than the "map hasn't loaded".
Does anyone know if there is a way to prevent this from happening? Thank you!
To Reproduce
Cause your conga to detect a map change.
Next, clean up a segment.
I do it with home assistant + MQTT sending the segment id to be cleaned.
Screenshots

Vacuum Model
Conga 4090
Congatudo Version
2023.05
Expected behavior
That, despite detecting a map change, it does not remain as if the map had not loaded and respects "the previous one".

More news. Today I noticed that the Conga did not go out to clean.
Again it was in this state.
I attach a picture of the whole screen so that you can see how "the map" loses all sense, it is half-finished, without segments, colors or anything.

How do I have to solve it?

I don't want to save the supposed new map.
2a) Reloading the page sometimes is sufficient.
2b) if the reloading does not work, a reset of the docker, does

Hello.
It happens the same to me. I have done a full cleanup to generate a new map, but it has no segments.
Now I can't even register the Conga with the Cecotec application.
How can I leave the Conga in the initial state to connect with the Cecotec application and create a new map with segments?

I would like at least to be able to use the vacuum cleaner again with its original application.

I would like at least to be able to use the vacuum cleaner again with its original application.

You have to undo all the changes you made to your conga in order to use congatudo.

I would like at least to be able to use the vacuum cleaner again with its original application.

You have to undo all the changes you made to your conga in order to use congatudo.

Hello.
If I'm not mistaken, the only thing I did was access the Conga via USB/ADB and add the access to the Home Assistant in the hosts file:
$ echo " cecotec.das.3irobotix.net cecotec.download.3irobotix.net cecotec.log.3irobotix.net cecotec.ota.3irobotix.net eu.das.3irobotics.net eu.log.3irobotics. net eu.ota.3irobotics.net cecotec-das.3irobotix.net cecotec-log.3irobotix.net cecotec-upgrade.3irobotix.net cecotec-download.3irobotix.net" >> /etc/hosts
If I manage to delete that line (let's see how), will I get control back?

Hola.
Si no estoy equivocado lo unico que hice fue acceder al Conga via USB/ADB y añadir el acceso al Home Assistant en el fichero hosts:
$ echo " cecotec.das.3irobotix.net cecotec.download.3irobotix.net cecotec.log.3irobotix.net cecotec.ota.3irobotix.net eu.das.3irobotics.net eu.log.3irobotics.net eu.ota.3irobotics.net cecotec-das.3irobotix.net cecotec-log.3irobotix.net cecotec-upgrade.3irobotix.net cecotec-download.3irobotix.net" >> /etc/hosts
Si consigo borrar esa linea (a ver si descubro como), ¿recuperare el control?

If need to remove this line from your /etc/hosts:
" cecotec.das.3irobotix.net cecotec.download.3irobotix.net cecotec.log.3irobotix.net cecotec.ota.3irobotix.net eu.das.3irobotics.net eu.log.3irobotics.net eu.ota.3irobotics.net cecotec-das.3irobotix.net cecotec-log.3irobotix.net cecotec-upgrade.3irobotix.net cecotec-download.3irobotix.net"

Muchas gracias Juan.
Mis conocimientos de linux son limitados. Si la explicacion de como editar el hosts desde la consola es sencilla, ¿me la podrias dar?
Si no lo busco, que tampoco me importa.

Creo que en el grupo de telegram hay algún tutorial al respecto, si no en la wiki me parece que hay algo también. Si no lo ves allí, vente aquí y vemos si te puedo poner algunos pasos.

Ok. Lo busco. Prefiero currar un poco antes de andar preguntando.
Gracias.

No hay nada como leer.
He editado el host con el comando 'vim' de linux y he quitado la ip de mi HA de la ultima linea del fichero. Ya puedo volver a emparejarlo con la app de Cecotec.
Imagino que antes o despues volveré a Congatudo, pero cuando me documente mejor.
Gracias por la ayuda Juan.


Latest Version Unavailable

The latest version for Hyperconnectivity published in CocoaPods is 1.1.0. However, the 1.1.1 was tagged in March of 2022. Is this something that was forgotten or expected?

Hi @BrunoMiguens, thanks for flagging - I've just published 1.2.1 to Cocoapods which should address this 🙏


[T4A7][T09-A2] Tham Shi Yuan

Ready for Review.

Accepting this PR without a review because another of your PRs for this tutorial was reviewed in detail.


How to get an unownedRouter for RedirectionRouter?

This is the example in the project readme:
class ParentCoordinator: NavigationCoordinator<ParentRoute> {
    /* ... */

    override func prepareTransition(for route: ParentRoute) -> NavigationTransition {
        switch route {
        /* ... */
        case .child:
            let childCoordinator = ChildCoordinator(parent: unownedRouter)
            return .push(childCoordinator)
        }
    }
}

class ChildCoordinator: RedirectionRouter<ParentRoute, ChildRoute> {
    init(parent: UnownedRouter<ParentRoute>) {
        let viewController = UIViewController()
        // this viewController is used when performing transitions with the Subcoordinator directly.
        super.init(viewController: viewController, parent: parent, map: nil)
    }

    /* ... */

    override func mapToParentRoute(for route: ChildRoute) -> ParentRoute {
        // you can map your ChildRoute enum to ParentRoute cases here that will get triggered on the parent router.
    }
}

But what do you do when viewController needs be passed an UnownedRouter<ChildRoute> so it can trigger routes? RedirectionRouter doesn't have an unownedRouter like actual coordinators do.

Hey @kevinrenskers,
UnownedRouter is actually a typealias for UnownedErased<StrongRouter>, so you should actually be able to do the following:
extension RedirectionRouter {
    var unownedRouter: UnownedRouter<RouteType> {
        UnownedRouter(self) { $0.strongRouter }
    }
}

Let me know, if this helps - it would probably be a nice addition for the framework as well!

I'm closing this due to inactivity and a given solution that should work as expected - if you have further questions or the given solution does not work, feel free to add further comments or open additional issues.


Cant authenticate with basic Auth credentials in /login/

Hi! I i am new to django-rest-auth and apis.
Its the first time i build a rest auth and i am not very familiar with Authorization headers and Content Types.
I am trying to understand why when i try to authenticate a user in /login/ with Basic Authorization like this:
curl -X POST -H "Authorization: Basic dXNlcm5hbWU6cGFzc3dvcmQ="  'https://myurl.com/rest-auth/login/' --insecure
i got this error message:
{"password":["(This field is required"]}
When passing the username and password in the body like this:
curl -X POST -H "Content-Type: application/x-www-form-urlencoded" -d 'username=myuser&password=mypassword' 'https://myurl.com/rest-auth/login/'  --insecure
I got the key:
{"key":"b5c0f3a9c7b2fc2f58a74b25f816e2968c64712f"}
Why this is happening?
I also wonder why when trying the same in /user/ it didnt throw me any error and give me my user model serialized
curl -X GET -H "Authorization: Basic dXNlcm5hbWU6cGFzc3dvcmQ=" -H "Cache-Control: no-cache" 'https://myurl.com/rest-auth/user/' --insecure
The only difference i can understand is that in /login i am using POST and in /user/ is GET
Can anybody explain this to me?
Thanks for reading!

you dont need to send the Authorization  header on /login
otherwhise in /user/ the Authorization header is how the user is retrieved from the database

You don't use /login in case of basic authentication type.
Authorization: Basic dXNlcm5hbWU6cGFzc3dvcmQ= are already your valid credentials that you use to access any protected endpoints (that's exactly why your second endpoint worked)
Please refer to DRF's authentication section for more details.


fix:updated SQLAlchemy=1.3.21 is incompatible in setup.py file.

SUMMARY
Marked SQLalchemy version 1.3.21 release as incompatible in setup.py. Which created a database connection issue for the oracle database. For more details --> 5784
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Codecov Report

Merging #12191 (51559b4) into master (93af329) will increase coverage by 0.26%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #12191      +/-   ##
==========================================
+ Coverage   63.06%   63.33%   +0.26%
==========================================
  Files         994      484     -510
  Lines       49075    29748   -19327
  Branches     4984        0    -4984
==========================================
- Hits        30950    18841   -12109
+ Misses      17925    10907    -7018
+ Partials      200        0     -200

Flag
Coverage Δ

javascript
?

python
63.33% <ø> (-0.93%)
:arrow_down:

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

Impacted Files
Coverage Δ

superset/db_engines/hive.py
0.00% <0.00%> (-85.72%)
:arrow_down:

superset/databases/commands/create.py
32.65% <0.00%> (-59.19%)
:arrow_down:

superset/sql_validators/postgres.py
50.00% <0.00%> (-50.00%)
:arrow_down:

superset/db_engine_specs/hive.py
54.61% <0.00%> (-29.62%)
:arrow_down:

superset/views/database/mixins.py
59.64% <0.00%> (-22.81%)
:arrow_down:

superset/db_engine_specs/presto.py
70.56% <0.00%> (-12.78%)
:arrow_down:

superset/db_engine_specs/mysql.py
79.59% <0.00%> (-12.25%)
:arrow_down:

superset/databases/commands/update.py
85.71% <0.00%> (-8.17%)
:arrow_down:

superset/databases/dao.py
94.11% <0.00%> (-5.89%)
:arrow_down:

superset/views/database/validators.py
78.94% <0.00%> (-5.27%)
:arrow_down:

... and 531 more

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 93af329...51559b4. Read the comment docs.


Workspace chat - Incorrect amount shown in preview card after payment.

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: v9.0.12-0
Reproducible in staging?: Y
Reproducible in production?: N
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:
Prerequisite: A workspace is created and 2 expenses are submitted in the workspace chat.

Go to [staging.new.expensify.com].
Navigate to the workspace chat created.
Go to the expense details page.
Click on the first expense.
Click on the header, then select "hold."
Write any reason and then click on "hold."
Turn off the connection.
Click on the "pay with expensify" dropdown and then select "pay elsewhere."
Pay the amount.
Navigate back to the main workspace chat by clicking on the subtitle link.

Expected Result:
After paying ETB 300, the preview card on the workspace chat page should show that ETB 300 is paid.
Actual Result:
After paying ETB 300, the preview card on the workspace chat page incorrectly shows that ETB 400 is paid.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

https://github.com/user-attachments/assets/4ab664f7-f409-4794-84fc-145c3457f066

View all open jobs on GitHub

Production
https://github.com/user-attachments/assets/b5b0ecfc-73fe-4d6b-9d8b-5b9cd0070279

Proposal
Please re-state the problem that we are trying to solve in this issue.
After paying ETB 300, the preview card on the workspace chat page incorrectly shows that ETB 400 is paid.
What is the root cause of that problem?
If we paid partial the expense report, we don't update the total of the expense report in optimistic data
https://github.com/Expensify/App/blob/afebc66349e21a5a7aa8ce6b04ab3c75e6e21620/src/libs/actions/IOU.ts#L6446
And then the ReportPreview show the amount is the old total amount
What changes do you think we should make in order to solve the problem?
We should update the total of the expense report here if the paid partial
{
    onyxMethod: Onyx.METHOD.MERGE,
    key: `${ONYXKEYS.COLLECTION.REPORT}${iouReport.reportID}`,
    value: {
        ...iouReport,
        lastMessageText: ReportActionsUtils.getReportActionText(optimisticIOUReportAction),
        lastMessageHtml: ReportActionsUtils.getReportActionHtml(optimisticIOUReportAction),
        hasOutstandingChildRequest: false,
        statusNum: CONST.REPORT.STATUS_NUM.REIMBURSED,
        pendingFields: {
            preview: CONST.RED_BRICK_ROAD_PENDING_ACTION.UPDATE,
            reimbursed: CONST.RED_BRICK_ROAD_PENDING_ACTION.UPDATE,
            partial: full ? null : CONST.RED_BRICK_ROAD_PENDING_ACTION.UPDATE,
        },
        total: full ? iouReport.total : total
    },
},

https://github.com/Expensify/App/blob/afebc66349e21a5a7aa8ce6b04ab3c75e6e21620/src/libs/actions/IOU.ts#L6472
What alternative solutions did you explore? (Optional)
NA

@madmax330 Since it's a blocker, I can raise a PR quickly.

Hi @nkdengineer, with any deploy blocker, can you please also link to the PR from the current checklist that caused the regression? Thanks!

@francoisl It comes from this PR https://github.com/Expensify/App/pull/42573.

Proposal
Please re-state the problem that we are trying to solve in this issue.
Preview card showing incorrect amount after payment while offline.
What is the root cause of that problem?
When one payment was on hold, payment amount correctly reflects in "Pay XXX Elsewhere". But the same does not reflect in card amount. It still shows total amount before going offline.
Onyx is not updated while making part payments. Hence while offline, when we do not receive API response, we fetch data from onyx and display, which still contains the total amount.
What changes do you think we should make in order to solve the problem?
Update onyx on every payment irrespective of online or offline.
What alternative solutions did you explore? (Optional)
N.A.

Updated proposal

Add the fix for approving money request

Hi all 👋 I'm the author of the https://github.com/Expensify/App/pull/42573
We decided not to update the value of the old (original) report because we can have expenses in different currencies.
Example:

report (A) has two expenses (A1, A2)

A1 - 10 USD
A2 - 30 some other currency which is 10x cheaper

A2 has value = 13 USD
we put A2 on hold
we can't perform conversion on FE so we set A value to 13 - 30 = -17 which is of course wrong

That's why we can't do it and behaviour is like that

Ok, so this seems to be the expected behavior then. @war-in can you confirm so we can close this issue?

@luacmartins Yes, I confirm :)

Thanks! Gonna close this issue.


User/chheilma/experiment source order viewer

Added the Source Order Viewer Experiment

Docs Build status updates of commit a9e1a67:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/devtools-guide-chromium/experimental-features.md
:white_check_mark:Succeeded

microsoft-edge/devtools-guide-chromium/media/experiments-source-order-viewer.png
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Any news on this? Can we merge this now?

Docs Build status updates of commit 4d00ac9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/devtools-guide-chromium/experimental-features.md
:white_check_mark:Succeeded

microsoft-edge/devtools-guide-chromium/media/experiments-source-order-viewer.png
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit f2b94a7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/devtools-guide-chromium/experimental-features.md
:white_check_mark:Succeeded

microsoft-edge/devtools-guide-chromium/media/experiments-source-order-viewer.msft.png
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Looks great to me!
#sign-off

signed-off and merged.


Improve performance of MetricsHeartbeatContextTest

The test currently runs for almost 3 minutes due to client retry
configuration. This change makes the PollingMasterInquireClient
respect the USER_RPC_RETRY* configuration values which
subsequently speeds up the test from ~3min to ~5sec

@gpang FYI more test improvements

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/7998/
Test FAILed.

jenkins, test this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8001/
Test PASSed.

These are the difference in maven build times when running mvn test -Dsurefire.forkCount=12 before and after this change:
Before:
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] Alluxio Core - Client 2.2.0-SNAPSHOT ............... SUCCESS [  2.902 s]
[INFO] Alluxio Core - Client - File System ................ SUCCESS [03:31 min]
[INFO] Alluxio Core - Client - HDFS 2.2.0-SNAPSHOT ........ SUCCESS [ 10.336 s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 03:45 min
[INFO] Finished at: 2020-02-05T19:32:54-08:00
[INFO] ------------------------------------------------------------------------

After:
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] Alluxio Core - Client 2.2.0-SNAPSHOT ............... SUCCESS [  2.711 s]
[INFO] Alluxio Core - Client - File System ................ SUCCESS [ 40.926 s]
[INFO] Alluxio Core - Client - HDFS 2.2.0-SNAPSHOT ........ SUCCESS [ 11.513 s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 55.316 s
[INFO] Finished at: 2020-02-05T19:27:15-08:00
[INFO] ------------------------------------------------------------------------

hrmmm, on the PR build, it is still 3+ minutes. Do you know why that would be?
[INFO] Alluxio Core - Client - File System ................ SUCCESS [03:15 min]

It's possible that the time to build the module is still high because during that phase of the build I believe there are 6-7 modules running concurrently which are probably in contention for CPU time.
Specifically, if you look for the test I set out to fix, it is much lower now:
Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 9.575 sec - in
alluxio.client.metrics.MetricsHeartbeatContextTest

jenkins, test this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8019/
Test PASSed.

@gpang any more comments?

alluxio-bot, merge this please


Introduce ranking of suggestions

Rank the suggestions using the following list of criteria

is pre-selected
grey star
remaining suggestions by priority of the source of the respective signal

Note that attributions should for now be ignored for suggestions. In the future these might be taken into account. Ones with golden stars might play a special role.

I won't do the "pre-selected" attribute for now as I'm not sure how much value it really adds since most suggestions come from signals which can never be pre-selected. On the other hand, we could rank signals there were pre-selected somewhere in the folder content higher. Just not sure if this would rather confuse than help since a resource's child may not be directly related to its parent.


[libiconv] Build error on x64-windows

First time reporting an issue on github.
I've noticed a few other people had a similar issue a few months back.
However, changing permissions, running in administrator mode, and updating Visual Studio 2022 results in the same error. Am I missing something?
Thank you,
error: building libiconv:x64-windows failed with: BUILD_FAILED
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:112 (message):     Command failed: "C:/Users/Skyler Crank/Documents/vcpkg/vcpkg/downloads/tools/msys2/52c8bb2635a5bae2/usr/bin/bash.exe" --noprofile --norc --debug -c "V=1 CPP='compile cl.exe -E' CC='compile cl.exe' CC_FOR_BUILD='compile cl.exe' CPP_FOR_BUILD='compile cl.exe -E' CXX_FOR_BUILD='compile cl.exe' CXX='compile cl.exe' RC='windres-rc rc.exe' WINDRES='windres-rc rc.exe' AR='ar-lib lib.exe' LD='link.exe -verbose' RANLIB=':' STRIP=':' NM='dumpbin.exe -symbols -headers' DLLTOOL='link.exe -verbose -dll' CCAS=':' AS=':' ./../src/v1.17-84a471678e.clean/configure --build=x86_64-pc-mingw32 \"--enable-extra-encodings\" \"--without-libiconv-prefix\" \"--without-libintl-prefix\" \"--enable-relocatable\" \"ac_cv_prog_ac_ct_STRIP=:\" \"gl_cv_double_slash_root=yes\" \"ac_cv_func_memmove=yes\" \"--disable-silent-rules\" \"--verbose\" \"--enable-shared\" \"--disable-static\" \"--prefix=/C/Users/Skyler Crank/Documents/vcpkg/vcpkg/installed/x64-windows/debug\" \"--bindir=\\${prefix}/../tools/libiconv/debug/bin\" \"--sbindir=\\${prefix}/../tools/libiconv/debug/sbin\" \"--libdir=\\${prefix}/lib\" \"--includedir=\\${prefix}/../include\" \"--datarootdir=\\${prefix}/share/libiconv\""     Working Directory: C:/Users/Skyler Crank/Documents/vcpkg/vcpkg/buildtrees/libiconv/x64-windows-dbg     Error code: 2     See logs for more information:

Call Stack (most recent call first):   scripts/cmake/vcpkg_configure_make.cmake:862 (vcpkg_execute_required_process)   ports/libiconv/portfile.cmake:29 (vcpkg_configure_make)   scripts/ports.cmake:170 (include)

This issue was fixed by building without any spaces in the directory

Please provide the full log of console output, thank you.

We don't have enough information to solve this issue, unfortunately; if you have any more information to help us solve this, please reopen!


fix: force esm to reload the new module instead of the cached one

Without this change, config files stored as esm (the ones that use the dynamicImport loader) won't be updated. Esm returns the previous loaded state of the module.
How to visualize the bug

create a config file using .mjs.
Write a simple index.js which attaches a file watcher to your config file.
Inside the file watch handler, load or search the config again. Do not recreate the lilconfig object. Use a singleton approach.
Do some changes in your config file, then save.
Verify that the watcher handler reloaded the config file, but the the state has not changed. Add a debugger inside dynamicImport to better visualize it.
Apply these PR changes
Rerun the above steps and now verify that the new state of the module was reloaded.

@antonk52 could you take a look, please.

Hey, thanks for the PR. I am hesitant to merge this in the current state. Since I last checked it is a known limitation of ESM in node that you cannot clear the module cache. Not sure if something changed in this regard. Currently we pass the module id as is to the loader which can result in an error if the id already contains any query params. So my main questions are:

Is there a recommended nodejs approach to clear module cache instead of an arbitrary timestamp query param?
If there is no other way, we need to ensure that it does not conflict with existing query params if any
Adding this change will introduce a discrepancy to how modules load in sync vs async loaders. A similar functionality is needed for loaders that use require

@antonk52 I totally forgot that bundlers preprocessors could also add query params to the id to force invalidation for browser builds. Is there a lib to parse an esm module Id? Like an url parser? Or can it be done with an url parser?

I only found solutions for browsers. In a static site build, for example, when we configure our bundlers to keep original file names, we have to append some sort of build id or timestamp to the file name at build time, to force the browser to invalidate cached modules. When file names don't change between builds, we use query params in

I will research if an url parser can do the job. Maybe there is a module Id parser too.

There is a built in Url and SearchParams that can utilised for this reason.

this change will only focus on the "dynamicImport" loader, for async loads, and for esm modules. Can we think about require in a another pr?

I don't want to add discrepancy between async and sync loaders. For this reason adding a cache invalidation PR should target both loaders.

As there is currently no recommended way to do this in nodejs can this be a user provided loader instead? We can add this recommendation to the package documentation

@antonk52 I agree with that being a custom loader. Where would be the best place to put this example?

Where would be the best place to put an example? Right after the yamls loader or in the section where you talk about ESM?

You can add a section about reloading modules and provide an example how to defined a loader that appends ?tm=Date.new to a filepath. Something along these lines.
import {lilconfig} from 'lilconfig'

function jsLoader(filepath)
  return import(`${filepath}?_lilconfig_ts=${Date.now()}`)
}

const options = {
    loaders: {
        '.js': jsLoader,
        '.mjs': jsLoader,
        '.cjs': jsLoader,
    }
}

lilconfig('myapp', options).search()
lilconfig('myapp', options).load(filepath)

If I provide a custom loader for dynamic imports, will the "id" of the module be available in the loader's scope? Or do we need to change something to make it available?

Overriding loaders is already available as an api. In your project you can provide a custom function for any extension and lilconfig will use it to load the content, this function takes filepath and filecontent and returns the module.


add keystone support

Hi Gnosis Safe team, I'm a developer from Keystone. This PR adds support for Keystone. 😄
Changes:

Upgrade bnc-onboard to latest version which supports Keystone. But we tested some bugs when we do the integration with Gnosis Safe, so we forked bnc-onboard v1.33.0 and fixed the bug on it, we use the forked github link in this PR just for test. At the same time, we have raised PR to Onboard.js: https://github.com/blocknative/onboard/pull/656. After bnc-onboard release a new version we can change the version number to the right way.
Add supports for Keystone, including some ui and logic changes.

Current change works well with Keystone Multi-Coin V1.3.0(which need user to prepare ABI info in their microSD-card, we will post a tutorial later.), And we are preparing a new release to do a deep integration with Gnosis Safe, including internal ABI support and functionality refinement for nested contract interactions.
Sincerely, I found Gnosis Safe a really great product when I'm doing the integration job.
Please have a review when you have time. 😃

You can follow this tutorial to upgrade the Keystone device to the latest firmware.: https://support.keyst.one/getting-started/firmware-upgrading

Hi @mikheevm , Onboard has already released 1.34.0, so I update the bnc-onboard version. Please have a review when you get time.

@soralit thanks for the PR, I was about to test this with one of the keystone wallets you sent over. It has firmware 1.0.1 though. Will this PR work with 1.0.1?
If not, is there a way to upgrade it without a micro SD card?

@soralit thanks for the PR, I was about to test this with one of the keystone wallets you sent over. It has firmware 1.0.1 though. Will this PR work with 1.0.1?
If not, is there a way to upgrade it without a micro SD card?

Hey @tschubotz I am sorry you have to use a microSD card to upgrade to the latest frimware.
Here is the tutorial - https://support.keyst.one/getting-started/firmware-upgrading
We will also share a draft tutorial later today which may ease your work.

Hi @tschubotz @katspaugh @mikheevm , sorry for the late reply. Here is a quick guide for how to use Keystone with Gnosis Safe: https://docs.google.com/document/d/1VYOUcP2gBvbQ7UNbw3QaMiVuM6pmmprB56AbF3n2xIw/edit?usp=sharing
Please kindly note that we also include the process for how to put ABI into the Keystone to fully verify the multi-sig transaction, which will help close some huge attack surfaces with ease of use.
If you have an estimate date for new version release, please let us know.

@BitcoinLixin @soralit

I am sorry you have to use a microSD card to upgrade to the latest frimware.

Okay, we need to order a micro sd card then first before actually testing it.
2 comments on the already though:
(1)
You added Keystone to the default options. Could you please move it to "show more"?

(2)
I am unable to show the QR code. I get errors on both Firefox and Chrome after clicking "sync keystone". I'm on MacOS.

Hey @tschubotz , thanks for your info.

Yes, I can move it to show more.
Please check the Camera permission of Chrome and Firefox, you can check it by click the icon before URL both in Chrome and Firefox.

@soralit

Yes, I can move it to show more.

thanks!

Please check the Camera permission of Chrome and Firefox, you can check it by click the icon before URL both in Chrome and Firefox.

I can't see anything related to camera.

We actually use the camera inside the Safe web app to scan addresses. It works without issues there.
I used the deployment in https://github.com/gnosis/safe-react/pull/2651#issuecomment-899742858 for testing.

@tschubotz I'm looking into this problem.
May I ask what version of Chrome and Firefox are you using? Is there another application using the camera?

Seems browsers cannot find the camera device (maybe the driver is not working), could you please restart your Mac and retry?

This didn't fix the issue. When we use the camera inside the Gnosis Safe web app, I get this permission popup:

It is not showing up for keystone.

If restart not working, may I ask what version of Chrome and Firefox are you using?
Firefox 89.0.1 (64-bit)
Chrome Version 92.0.4515.159 (Official Build) (x86_64)

Is there another application using the camera?

no

Could you also try the camera to scan addresses in this deployment?

Yes, it worked without issues.

@tschubotz That's really odd...
I only met this error when I connected my mac to a screen while my mac was folded.
Could you please try it again by this way:

Use Chrome anonymous mode, and open the testing website.
Then connect to MetaMask or other wallet, select an account, import a safe, then scan an address for requesting the Camera permission.
Change the wallet to Keystone, then try to sync.

@tschubotz That's really odd...
I only met this error when I connected my mac to a screen while my mac was folded.

You are actually right, I had my Mac book closed. I use it with an external monitor. Seems I had opened it between testing keystone qr code scanner and our qr scanner.
All good then, perhaps the error message could be improved? ;)
Either way, we can continue once we got our mico sd card.

Hi @tschubotz , we have released a new firmware version, you can download it here: https://github.com/KeystoneHQ/Keystone-cold-app/releases/download/V1.4.0Multi-Coin/V1.4.0-Multi_Coins.zip
This version add internal GnosisSafe support as mentioned above. Please have a look if you have time.

Hi @tschubotz , we have released a new firmware version, you can download it here: https://github.com/KeystoneHQ/Keystone-cold-app/releases/download/V1.4.0Multi-Coin/V1.4.0-Multi_Coins.zip
This version add internal GnosisSafe support as mentioned above. Please have a look if you have time.

With this firmware, Gnosis Safe's ABI is embedded into Keystone's firmware so that unsigned transaction can turn into human readable format. Related code -
https://github.com/KeystoneHQ/Keystone-cold-app/blob/V1.4.0-release/coinlib/src/main/java/com/keystone/coinlib/coins/ETH/GnosisHandler.java#L19
https://github.com/KeystoneHQ/Keystone-cold-app/blob/1a667ec8218d0bd82574c263851501c29709582b/coinlib/src/main/java/com/keystone/coinlib/coins/ETH/ABIReader.java#L76

I’ve updated the deployment branch, here’s a link to test on Rinkeby: https://pr2651--safereact.review.gnosisdev.com/rinkeby/app/
Cc @liliya-soroka

What networks are supported by the keystone? May Rinkeby be used or it will be available only on mainnet ?

Great, thank you @liliya-soroka!
@soralit let's fix them tests and update the branch and merge the PR. 🚀

Hey @katspaugh , branch was updated. I ran the tests and CI locally and all tests passed.

Nothing critical was found.
Just a few suggestions to improve:

Hard to close the window in case of a laptop with small resolution ( 13")

The error "Invalid scheme" is not informative ( scanning wrong QR code)
there is no way to go back and scan the code again if "Scan keystone" was clicked

Thanks for the feedback
all the feedback have been logged on our development backlog we will improve these in the coming release :)

Merged, thank you folks!
It will be in the next release (hopefully this week).


bug: item-right no longer places icon on the right side

Type: bug
Ionic Version: 2.x
Platform: all
After the release of Alpha.53, item-right stopped placing icons on the right side of both ion-item and buttons.

What does your markup look like when it's not working?

<ion-card *ngIf="!search">
    <button ion-item>
      <ion-icon primary item-right name="bookmark"></ion-icon>

Bookmarks

    </button>
  </ion-card>

This use to place the icon on the right side before .53.

For anyone that runs into this issue, if you have a theme directory like this:
│   ├── theme/                         * App theme files
│   │   ├── app.core.scss              * App Shared Sass Imports
│   │   ├── app.ios.scss               * iOS Sass Imports & iOS Variables
│   │   ├── app.md.scss                * MD Sass Imports & MD Variables
│   │   └── app.variables.scss         * App Shared Sass Variables

You need to make sure the ionic.config Sass src looks like this:
    sass: {
      src: ['app/theme/app.+(ios|md).scss'],
      dest: 'www/build/css',
      include: [
        'node_modules/ionic-framework',
        'node_modules/ionicons/dist/scss'
      ]
    },

Leaving this issue open until we can fix the starters.

All of the starters have been updated to use a theme directory & alpha.53:
https://github.com/driftyco/ionic2-starter-tutorial/commit/14cd14cbcdd37a2a0711f6f72945cefa66b5da01
https://github.com/driftyco/ionic2-starter-tabs/commit/3912d7dfc702e8d478a75ba3e22b52e5c07d949c
https://github.com/driftyco/ionic2-starter-blank/commit/3c44de6d07047ac5cea6910bb0795bad173b830f
https://github.com/driftyco/ionic2-starter-sidemenu/commit/f4c8f7aaf1acb9235f22f03955ca4cb841d50eb4
https://github.com/driftyco/ionic2-app-base/commit/64647cf027b8583ea7ced657b4ab29e162c5587b


ERA5 monthly means by hour of day: time not monotonically increasing

For the reanalysis-era5-land-monthly-means:monthly_averaged_reanalysis_by_hour_of_day dataset (and, probably, the similarly structuredreanalysis-era5-single-levels-monthly-means:monthly_averaged_ensemble_members_by_hour_of_day and reanalysis-era5-single-levels-monthly-means:monthly_averaged_reanalysis_by_hour_of_day), data are sometimes returned with the time not monotonically increasing, causing them to fail cube validation.

While simply sorting the datasets along the time axis would technically fix this bug, there is a more significant problem with the data structure, described in Issue #6 .

I will close this issue by blacklisting the offending datasets so that they are no longer presented by the xcube CDS store plugin. If there is demand for these datasets in future, further work on restructuring and properly presenting the data can be covered uner Issue #6.


[Cupertino icons]Glyphs for car and car_fill from CupertinoIcons are broken

Steps to Reproduce

Open Cupertino Icons documentation
Find car or car_fill constant

Expected results:

Actual results:

@xster I don't know much about CupertinoIcons, could you help provide some context for where the font comes from, how to generate it, etc.?

Here's how the icon is supposed to look
Screenshot from SF symbols

@maheshmnj   Can you tell me the address of the website in the screenshot?

@maheshmnj Can you tell me the address of the website in the screenshot?

@jahnli https://fonts.google.com/icons

@jahnli The screenshot is from SF symbols app https://developer.apple.com/sf-symbols/

@maheshmnj  It seems that there is only a material icon and no Cupertino icon

@maheshmnj  thank you

Should be fixed in CupertinoIcons 1.08


tokenizer.seperator should ideally be spelled separator (with an A)

Hi,
Thanks for all the great work put into this library. I'm merely creating this issue in hope that it will show up in someones search results if they are as careless as I was.
Today https://github.com/olivernn/lunr.js/blob/master/lib/tokenizer.js#L30 reads seperator, with a second e. Whereas a quick google will tell you that it is more often spelled separator, with an a in this position.
Ideally, maybe this should be made into a setter function, so as to avoid spelling mistakes on something like this.
 Thanks for the very delicate way you put it also...

it is more often spelled separator

:laughing:
As annoying as having something spelled wrong esoterically is, I'd rather not change the name of this, it would be a breaking change, and for something that others are likely to have incorporated into their applications. As far as I know there is no good way of aliasing in JavaScript, though if you can think of a good solution then do let me know.
I'm going to close this for now, just to keep the issues queue slightly more under control, but it is not a nonsense issue.
Thanks,


WIP: Rewrite typechecker related functions using CheckM monad

Solves #110

Nice work, but I believe you should also change eval to use the environment in the reader context, so you also have to change the calls to eval that modifies the environment to use the withReader function

Can you give me an example?


HTTP request failing with Beyla enabled

Hey,
We've been having an issue where http requests to<EMAIL_ADDRESS>fail for a service when Beyla is enabled and running as a Daemonset in Kubernetes. The Kubernetes cluster it is running on is a GKE cluster running on preemtible nodes
Here's some test code we were able to reliably reproduce the issue with:
package main

import (
	"context"
	firebase "firebase.google.com/go/v4"
	"firebase.google.com/go/v4/auth"
	"fmt"
	"google.golang.org/api/option"
	"log"
	"net/http"
	"strings"
)

func main() {
	AUTH_SERVICE_ACCOUNT_ID :=<EMAIL_ADDRESS>	AUTH_SERVICE_ACCOUNT_TOKEN := "< Secret >"
	FIRESTORE_PROJECT_ID := "< ProjectID >"
	client, _ := newFireAuthClient(context.Background(), AUTH_SERVICE_ACCOUNT_ID, AUTH_SERVICE_ACCOUNT_TOKEN, FIRESTORE_PROJECT_ID)
	ss := SimpleServe{client}
	http.HandleFunc("/test", ss.verify)
	http.HandleFunc("/healthy", ss.health)
	if err := http.ListenAndServe(":8090", nil); err != nil {
		panic(err.Error())
	}

}

type SimpleServe struct {
	*auth.Client
}

func (ss *SimpleServe) health(w http.ResponseWriter, r *http.Request) {
	w.WriteHeader(http.StatusOK)
	return
}
func (ss *SimpleServe) verify(w http.ResponseWriter, r *http.Request) {
	token := r.Header.Get("Authorization")
	if token == "" {
		w.WriteHeader(http.StatusUnauthorized)
		return
	}
	token = strings.ReplaceAll(token, "Bearer ", "")
	jwtToken, err := ss.VerifyIDToken(r.Context(), token)
	if err != nil {
		panic(fmt.Sprintf("Error verifying jwt token for user with message %s ", err.Error()))
	}
        subject := jwtToken.Subject
	log.Printf("validated user: %s\n", subject)
	w.WriteHeader(http.StatusOK)
	return
}

func newFireAuthClient(ctx context.Context, AuthServiceAccountID, AuthServiceAccountToken, FirestoreProjectID string) (authClient *auth.Client, err error) {
	conf := &firebase.Config{ProjectID: FirestoreProjectID, ServiceAccountID: AuthServiceAccountID}
	options := []option.ClientOption{option.WithCredentialsJSON([]byte(AuthServiceAccountToken))}

	fire, err := firebase.NewApp(ctx, conf, options...)
	if err != nil {
		return nil, err
	}

	authClient, err = fire.Auth(ctx)
	if err != nil {
		return nil, fmt.Errorf("error failed creating Auth client: %w", err)
	}

	return authClient, nil
}

With Beyla enabled and port-forwarding to the service this hangs:
curl --location --request POST 'http://localhost:8090/test' --header 'Authorization: Bearer $token'
I have managed to find traces (not using the code above - a timeout is needed to find them):

I believe the code that makes the HTTP request is here - https://github.com/firebase/firebase-admin-go/blob/master/auth/token_verifier.go#L414
We have managed to make this request succeed if the HTTP request is made immediately and repeatedly on app startup, possibly before Beyla has started instrumentation?

Thanks so much for reporting this @firthh! We'll take a look and see if we can reproduce. I wonder if we need to be selective where we inject the headers, maybe the google api is signing them or they don't like that we add an additional header. We'll report back here once we investigate more.

I just wanted to report back, I've reproduced the issue and I'm looking at how to fix this. Thanks again!

Hi @firthh, we just merged a fix for this issue. We'd really appreciate if you could give it a go and let us know if this resolves the timeouts you were seeing. It's merged into main, build should become available in 30 mins or so and I'll be backporting the fix to 1.3 too.

Closing for inactivity.


Louis feature sidebar

Desktop view

Mobile view

Sorry for taking so long to review this @louisan42 , Have not had a good chance to sit down and look at this until now! Absolutely great work on it however. It looks really good! Let me know if you would like any help in relation to this review 👀

@ryanolee
I pushed a new update with review changes.

[x] Drawer has close button
[x] set global styling with default font Walter Turncoat