Change zoom level

Hi! Thanks for this library!
How can I change zoom level from code?  I was reading that is changing viewport, but i cant.
Thank you!

did you mean double tap zoom level ?

Try this:
final ViewPortHandler viewPortHandler = _controller.viewPortHandler;
    viewPortHandler.refresh(
      viewPortHandler.setZoom1(
        10,
        1.0,
      ),
    );

But I want to do the zooming once the chart is drawn, and I don't know how.

Thank you! It works!

controller.setScaleMinima(10, 1);


Mark this project as deprecated.

This project has a large number of design issues, doesn't conform to the OIDC spec, and does some unexpected things. I'm currently refactoring this out of a project and it's surprised me on a few occasions. I appreciate it solves some issues for some people, but in the long run, doing unexpected things creates issues with interoperability and extensions.
There are also some minor security issues (forgive me if I'm wrong about any of the particulars).

JWT Signatures from servers aren't verified. Although tokens should be retrieved from an HTTPS connection, you should still verify the token itself.
JWTs are resigned by Flask OIDC using symmetric encryption, with the Flask SECRET_KEY  as the key. This is a layer of unneeded indirection, and makes it impossible to integrate properly.
Original token exp is ignored (after token exchange)
nonce isn't checked.
Not following the spec on token exchange https://github.com/puiterwijk/flask-oidc/issues/44
There's limited point in salting if the salt isn't secret. It's hardcoded and there's no way to override it.
The oidc_id_token cookie expiry time isn't bound to the JWT. It probably should.

Given the better tools to do this like oauthlib, and pyjwt to do this job, (and probably something higher level too) perhaps the maintainers should point new projects to these tools instead.

General issues with the code:

I get this is a problem with the deprecated oauth2client. The use of JSON to configure a Python extension is odd. If you're deploying to Kubernetes you've got to base64 a group of secrets then load it up as a volume, which is simply painful. It couples all those other secrets together.

Stateful globals. I feel that having oidc_id_token_dirty global isn't needed. It's trigged by 'authenticate_or_redirect' (called by a before_request handler).  The oidc_id_token_dirty is then then picked up by an after_request handler to determine if you've logged out. Just return the 401 during authenticate_or_redirect, don't rely on callbacks and flags to do this.

It's not following the Oauth2 spec. The way to pass tokens isn't via a cookie, it's with a Authorization: Bearer jwt... header. I'm not saying that cookie authentication shouldn't be an option, but it probably shouldn't have been the default.

I recently started using it. The only reason for it was that there is a couple of examples how to make it work with Okta, e.g. like this:
from flask import Flask, url_for, redirect
from flask_oidc import OpenIDConnect

app = Flask(__name__)
app.config['OIDC_CLIENT_SECRETS'] = 'client_secrets.json'
# Contents:
# Create client_id and client_secret at https://console.developers.google.com/apis/credentials
# {
#     "web": {
#         "client_id": "123456789012-abc123hi09123.apps.googleusercontent.com",
#         "client_secret": "ab123456789ABCDEFGHIJKLM",
#         "redirect_uris": ["http://localhost:5000"],
#         "auth_uri": "https://accounts.google.com/o/oauth2/auth",
#         "token_uri": "https://accounts.google.com/o/oauth2/token",
#         "userinfo_uri": "https://www.googleapis.com/oauth2/v3/userinfo"
#     }
# }
app.config['SECRET_KEY'] = 'uq4aKjUvWXTPTIyfCz7mTtcG'
app.config['OIDC_ID_TOKEN_COOKIE_SECURE'] = False
app.config['OIDC_SCOPES'] = ["openid", "profile", "email"]
app.config['OIDC_CALLBACK_ROUTE'] = '/authorization-code/callback'
oidc = OpenIDConnect(app)

@app.route('/')
@oidc.require_login
def index():
    return redirect(url_for('personalized'))

@app.route('/personalized')
@oidc.require_login
def personalized():
    info = oidc.user_getinfo(['email', 'openid_id'])
    return 'Hello, {} ({})'.format(info.get('email'), info.get('openid_id'))

@app.route('/hello')
@oidc.require_login
def constant():
    return 'Hello'

if __name__ == '__main__':
    app.run(port=5000)

If there was an alternative that also provided a require_login decorator and works with Okta, I'd be happy to switch.

This is related to #81

And #60

I am also running into some troubles... I'm trying to use this library with the Okta implicit flow (intospect call without client secret). I'll probably make a pull-request to add this but it's weird that it's not following that spec...

@MartinThoma my advice would be to roll your own code. You really just need a decorator to do a check for the authorization cookie being valid (a few lines with pyjwt), and an OIDC callback view built on top of oauthlib and pyjwt/jose.
OIDC isn't as complex as it seems.

If you want this resolved i'm maintaining a fork of this with some added extensibility. Feel free to open a PR/issue here:

https://github.com/svintit/flask-oidc-ext/pulls


Add release notes to login design system

Proposal: Consider writing and adding release notes to design.login.gov site (Similar to USWDS' release notes)
Stories:

As a designer, I want to inform our login team members, esp. developers, of new features (and fixes), to show them that the login design team is listening to their feedback and that we are actively working to improve the login design system
As a login team member, I want to be kept in the loop with release notes. As the design system evolves over time, having a historical log of these changes, improvements, and fixes helps tell stories behind the system

Definition of Done:

[ ] Discuss this proposal with login designers, developers and product to gauge interest
[ ] If 👍, proceed to create initial draft of release notes and format
[ ] Establish a review process from drafting to publishing on design.login.gov site
[ ] Create a Release Notes page on design.login.gov site (Consider a subnav under Usage section)

migrated LG-2646


Add Perl, which is actually installed

Description
It's an improvement adding a tool that should be listed, and is not: See #2203
Related issue: #2203
Check list

[x] Related issue / work item is attached
[ ] Tests are written (if applicable)
[x] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

Hello, @JJ
Ubuntuxx04-README.md file is autogenerated. Could you please update two files SoftwareReport.Common.psm1 and SoftwareReport.Generator.ps1 to include  your fix?
As an example take a Get-RubyVersion  finction.

Sure.
El lun, 7 dic 2020 a las 6:59, Aleksandr Chebotov<EMAIL_ADDRESS>escribió:

Hello, @JJ https://github.com/JJ
Ubuntuxx04-README.md file is autogenerated. Could you please update two
files SoftwareReport.Common.psm1
https://github.com/actions/virtual-environments/blob/main/images/linux/scripts/SoftwareReport/SoftwareReport.Common.psm1
and SoftwareReport.Generator.ps1
https://github.com/actions/virtual-environments/blob/main/images/linux/scripts/SoftwareReport/SoftwareReport.Generator.ps1
to include your fix?
As an example take a Get-RubyVersion finction.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/actions/virtual-environments/pull/2204#issuecomment-739686561,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAAD5ELUAK2VK2IJ6D7T53STRVMDANCNFSM4UPRLB5A
.

--
JJ


Error: Navigating to external URL is not allowed by default. Use navigateTo (url, { external: true })

Environment

Operating System: Darwin
Node Version:     v19.8.1
Nuxt Version:     3.5.3
Nitro Version:    2.4.1
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      ssr, typescript, runtimeConfig, modules, auth, components, app
Runtime Modules<EMAIL_ADDRESS>Build Modules:    -

Reproduction
No response
Describe the bug
I setup a basic 'local' authentication using<EMAIL_ADDRESS>I can successfully login/logout etc. But everytime nuxt-auth wants to do a redirect I see two errors:
[Vue warn]: Unhandled error during execution of native event handler
Uncaught (in promise) Error: Navigating to external URL is not allowed by default. Use navigateTo (url, { external: true })
So these errors happen after using signIn() and signOut()
Additional context
My nuxt.config.ts:
export default defineNuxtConfig({
// Client side rendering only
ssr: false,
// Typescript settings
typescript: {
typeCheck: false,
strict: true,
},
runtimeConfig: {
public: {
baseURL: process.env.API_BUILDER_URL,
flowBuilderUrl: process.env.FLOW_BUILDER_URL,
downloadCloseHost:
process.env.DOWNLOAD_CLOSE_URL || "https://downloadclose.com",
},
},
modules: ["@sidebase/nuxt-auth"],
// Auth setup, see https://sidebase.io/nuxt-auth/configuration/nuxt-config
// TODO: set auth origin for production
auth: {
provider: {
type: "local",
endpoints: {
signIn: { path: "/login", method: "post" },
signOut: { path: "/logout", method: "post" },
getSession: { path: "/user", method: "get" },
},
token: {
type: "Bearer",
},
pages: { login: "/login" },
},
baseURL: ${process.env.API_BUILDER_URL}/api,
globalAppMiddleware: {
isEnabled: true,
},
},
components: [
{
path: "~/components",
pathPrefix: false,
},
],
app: {
// Page header settings
head: {
title: "Close Builder",
titleTemplate: "%s – Close",
meta: [
{ charset: "utf-8" },
{ name: "viewport", content: "width=device-width, initial-scale=1" },
{ hid: "description", name: "description", content: description },
{ hid: "og:description", name: "og:description", content: description },
],
link: [{ rel: "icon", type: "image/x-icon", href: "/favicon.ico" }],
},
},
});
Logs
No response

This seems to be because the useAuth composable of the package passed a full url with protocol by default to the navigateTo router helper function without supplying the required { external: true }. What you can do for now to prevent this is to manually supply a valid path in the signInOptions of your signIn and signOut actions where you want to redirect to after the action has finised.
Example:
signIn({ username, password }, { callbackUrl: '/' })

@tjeneis thanks, that was indeed the problem!

@omnichronous does your commit https://github.com/omnichronous/nuxt-auth/commit/30da88c899eef8183bc1bd292468c21ca24ed39b solve the issue?
If so, you are welcome to open a PR, or I can look into doing this myself!

@zoey-kaiser I didn't get around to testing it properly and I see that you already opened a PR with a fix, much appreciated ❤

signIn({ username, password }, { callbackUrl: '/' })

signIn worked, but not for signOut. I'm still getting the error.


Update tutorial notebook description

Clarify example notebook is tutorial.

Very minor doc only pr - skip review.


TypeScript definitions

Do you have any plans of supporting a TypeScript definitions file, or does one exist that is not registered on typings or not on DefinitelyTyped?
I found the Flow Types in Types.js.  This file is very easily converted into a TypeScript definition file, although I understand the pain of maintaining typing information in two separate places.  Maybe a tool could do this automatically.

If you are keen to PR - that would be great.
I reckon none of maintainers use TypeScript, so it would be near to impossible to validate if everything is good =)

I opened a repository for this here: https://github.com/mjohnsonengr/typed-react-motion
I'll fill in the d.ts file and put it in the typings registry after work tonight.

Sounds awesome, thanks, @mjohnsonengr!

I wrote one a few days ago. However, I don't have a lot of experience writing d.ts files yet... so go easy if you find errors :).  https://github.com/PhilipDavis/react-redux/blob/master/typings/react-motion/react-motion.d.ts

+1 TypeScript Defenition would be nice

Hello, i created defitions
(shameless copy from flow Types fixed for typescript)
In the future i will add docs to props

@nkbt my PR merged. Now we can use defitnitions, just call tsd
tsd install react-motion

I think, this issue should be closed

This is great =D, thanks!
I'll add a note in the README. Closing!

if typings install react-motion does not work, you have to use typings install --name react-motion


[GHSA-6v73-fgf6-w5j7] Code Injection in Spring Cloud Function

Updates

Affected products

Hey @Tsuki124, you don't happen to have a reference supporting this change do you?

@darakian
https://security.snyk.io/vuln/SNYK-JAVA-ORGSPRINGFRAMEWORKCLOUD-2436645
https://github.com/spring-cloud/spring-cloud-function/commit/0e89ee27b2e76138c16bcba6f4bca906c4f3744f

Awesome. Thank you much.


Drizzle types as Mappers not picked up

I'm trying to use this codegen with Mappers as Drizzle table types.
But these mappers are not recognized at all...
I also created a straight *Mappers interface myself and that is picked up.
Anyone knows if this is supported?

I've created this small repo, reproducing the issue.

Maybe this is related to https://github.com/eddeee888/graphql-code-generator-plugins/pull/242 ?
Are you planning a new release soon?

Hi @marceloverdijk ! Yes, I'm going to release a new patch soon. Apologies, I missed your issue in the last few weeks 🙏
Could you try this alpha version to see if it fixes your issue please?
yarn add -ED @eddeee888/gcg-typescript-resolver-files@pr242-run423-1

No worries. Not sure if I can test it the coming days, but looking forward to the new release anyway.

@eddeee888 I did a quick test with @eddeee888/gcg-typescript-resolver-files@pr242-run423-1 but I get the same result unfortunately:
@@ -1,8 +1,8 @@
 import type   { CountryResolvers } from './types.generated';
     export const Country: CountryResolvers = {
     /* Implement Country resolver logic here */
-        code: () => { /* Country.code resolver is required because Country.code exists but CountryMapper.code does not */ },
-        continent: () => { /* Country.continent resolver is required because Country.continent exists but CountryMapper.continent does not */ },
-        id: () => { /* Country.id resolver is required because Country.id exists but CountryMapper.id does not */ },
-        name: () => { /* Country.name resolver is required because Country.name exists but CountryMapper.name does not */ }
+        code: async (_parent, _arg, _ctx) => { /* Country.code resolver is required because Country.code exists but CountryMapper.code does not */ },
+        continent: async (_parent, _arg, _ctx) => { /* Country.continent resolver is required because Country.continent exists but CountryMapper.continent does not */ },
+        id: async (_parent, _arg, _ctx) => { /* Country.id resolver is required because Country.id exists but CountryMapper.id does not */ },
+        name: async (_parent, _arg, _ctx) => { /* Country.name resolver is required because Country.name exists but CountryMapper.name does not */ }
     };

Note the async part was added which is I assume another update in the alpha version.

Ah I see, thanks for trying it! I will take a look deeper into your issue tonight 👀

Hi @marceloverdijk,
I've got a PR ready: https://github.com/eddeee888/graphql-code-generator-plugins/issues/245
Could you try this alpha version please:
yarn add -DE @eddeee888/gcg-typescript-resolver-files@pr248-run448-1

Hi @eddeee888 yes that solves it 🥳
The resolver is now genereated as expected:
import type { CountryResolvers } from './types.generated';
export const Country: CountryResolvers = {
  /* Implement Country resolver logic here */
  code: async (_parent, _arg, _ctx) => {
    /* Country.code resolver is required because Country.code exists but CountryMapper.code does not */
  },
  continent: async (_parent, _arg, _ctx) => {
    /* Country.continent resolver is required because Country.continent exists but CountryMapper.continent does not */
  },
};

Thx a lot!

This fix is now released in v0.7.5
Thank you for reporting and collaborating with me through out this process! 🕺


Xnero: Watch bitcasino.com

Xnero requests the watch of the watch_keyword bitcasino\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

bitcasino\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Ryan M in Charcoal HQ


Corporate proxy and name resolution

My corporate proxy only fetches resources throw domain names. This means that try to get a resource by his IP address will throw an exception. For example I'm need to install a module called bcrypt. Tracing the process I discovered that<EMAIL_ADDRESS>is trying to download this file https://nodejs.org/download/release/v12.13.1/node-v12.13.1-headers.tar.gz and this results in the error:
gyp info it worked if it ends with ok
gyp info using<EMAIL_ADDRESS>gyp info using<EMAIL_ADDRESS>| linux | x64
gyp info find Python using Python version 3.5.2 found at "/usr/bin/python3"
gyp http GET https://nodejs.org/download/release/v12.13.1/node-v12.13.1-headers.tar.gz
gyp WARN install got an error, rolling back install
gyp ERR! configure error
gyp ERR! stack Error: connect EHOSTUNREACH <IP_ADDRESS>:443
gyp ERR! stack     at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1129:14)

I have the theory that node-gyp is resolving the name and doing the request with the IP address instead the name. Can be this true? In this case exists any way to change this behavior?

EHOSTUNREACH <IP_ADDRESS>:443 looks to me like it's not even trying to use your corporate proxy. Do you have one of the HTTP_PROXY environment variables set in your terminal so it knows what proxy to talk to? Without that it'll just blindly try and connect direct.
See https://github.com/nodejs/node-gyp/blob/master/lib/proxy.js#L70-L84 for some of the rules for detecting proxy. Use the environment variables HTTPS_PROXY, HTTP_PROXY or run npm config set proxy http..... to get this done.


Remove the javascript tag

Isn't it C? You scare away people by that tag.

My bad, looks it is available for web too.


why when the "android update available" popup on the device's screen, the scrpy cannot connect, thx

why when the "android update available" popup on the device's screen, the scrpy cannot connect, thx

Are you sure this is related?
What's the error message in the console?


Add a checkbox to hide offline hotspots

There are two more things to do:

Check the styling: I tried with the mono class and the header font color (#a0b0c2) but didn't really like it. Of course, I can still change to your preference.

Update the hotspot count: right now, the hotspot count by default includes all hotspots, even the ones that don't have a location set. Should I default to online.length + offline.length instead? If so, I probably can't get that data before /api/coverage finishes loading. That information also is not available in stats.count, and it would be weird to update it once the API call finished. At this point I haven't updated the count at all.

I can also keep the React Component stuff instead of updating it to a function in this PR in order to keep a simple PR.

(I'll merge this tomorrow morning, I don't think it'll break anything if I just merged it now but just want to err on the side of caution 😛  )

That's way better for sure!


Performance Issue: Synthesis Takes Too Long to Complete

Version
yosys 0.41+126
On which OS did this happen?
Linux
Reproduction Steps
Hello,
I encountered a performance issue while using Yosys to synthesize a Verilog file. The specific details are as follows:
I used the Verilog file design.v, and when running the Yosys synthesis, it has been over 30 minutes without completion. My system configuration is as follows: 128G RAM, and a high-performance CPU. However, the synthesis process seems to be stuck in an infinite loop or some kind of performance bottleneck.
To rule out issues with the file itself, I have checked the Verilog file for syntax errors but did not find any obvious problems. Additionally, I am using the latest version of Yosys.
Attached is the Verilog file (design.v) that triggers this issue. I hope to get the community's help and attention.
After this error, the parsing process is interrupted. I have checked the Verilog file for syntax errors but couldn't find any obvious issues. I am using the latest version of Yosys.
Attached is the Verilog file (design.v) that triggers the error.
Thank you in advance for your attention to this matter.
I look forward to hearing from you regarding this issue.
design_file.zip
Expected Behavior
Synthesis completes successfully within a reasonable time
Actual Behavior
Synthesis takes too long and does not complete

You should minimize the issue. It is a waste of everyone's time to file non-minimized fuzzer-discovered issues and personally I am leaning towards closing such issues outright.

I would agree with the previous comment on minimization. If you haven't seen it before https://blog.regehr.org/archives/2037 provides good guidelines on reporting fuzzed bugs. Additionally it would have been helpful to include the synthesis script used. I have assumed it was just synth as that causes a hang for me too.
If you aren't familiar with it, Yosys includes a testcase minimizer in the form of the bugpoint command. Running
yosys -p "read_verilog design.v; bugpoint -command \"synth\" -runner \"timeout 1\" -fast; write_verilog bug.v" results in the following testcase after only a few minutes (I've tidied the one shown here with clean -purge; write_verilog -noattr bug.v)
/* Generated by Yosys 0.40+45 (git sha1 dd2195543, g++ 13.2.1 -fPIC -Os) */

module module11(_0_);
  output [3:0] _0_;
  wire [3:0] _0_;
  wire _1_;
  wire [3:0] _2_;
  wire [39:0] _3_;
  wire [39:0] _4_;
  assign _3_ = _1_ >= _2_;
  assign _0_ = $signed(4'hx) >> _4_;
  assign _4_ = _3_ - 40'h734e475132;
endmodule

Running read_verilog bug.v; synth on this seems to go very slow at
2.10. Executing PEEPOPT pass (run peephole optimizers).
shiftadd pattern in module11: shift=$shr$bug.v:11$2, add/sub=$sub$bug.v:12$3, offset: -1313296690

2.11. Executing OPT_CLEAN pass (remove unused cells and wires).
Finding unused cells or wires in module \module11..

You can recreate this by just running wreduce; peepopt. peepopt's shiftadd converts A>>(B+D) to (A'>>D)>>(B) where D is const and A' is a zero padded version of A. The issue seems to be that where D is -1313296690, A is padded with a<PHONE_NUMBER> bit constant that is all zeros, and Yosys is understandably slow to handle such a large constant.
It seems to me like although this is a case where performance is ideal, no one would ever have a shift of -1313296690 in a non-fuzzing scenario so its not really a big deal.

Thanks @georgerennie! I would have linked the same article if I had the URL on hand.

@whitequark @georgerennie Thank you for your response. I encountered this issue during a stress test, and I also tried it on several machines, but faced the same problem where logic synthesis could not be completed within a reasonable time. I have checked #4056 and #4057, which also reported extended synthesis times due to certain padding operations. Although I discovered this performance issue during a stress test, I believe this fault is noteworthy. Thank you.

So I had a look into it a little more, and there is actually a bug happening here, but it isn't what causes the poor performance.
Although the shift is nominally by 40'h734e475132, shiftadd stores the offset in an int. This shift value overflows the int representation, becoming a shift value it shouldn't.
The poor performance is still repeatable on a testcase like the following that doesn't overflow the int. Perhaps this optimization should be disabled for large shifts to gate against this, although again I don't believe this to be a particularly noteworthy issue for real designs.
module module11(_0_);
  output [3:0] _0_;
  wire [3:0] _0_;
  wire _1_;
  wire [3:0] _2_;
  wire [31:0] _3_;
  wire [31:0] _4_;
  assign _3_ = _1_ >= _2_;
  assign _0_ = $signed(4'hx) >> _4_;
  assign _4_ = _3_ - 32'h4e475132;
endmodule

In my view, Yosys should limit both the wire size and the shift size to some reasonable amount well under 4 billion. The Verilog specification explicitly allows this (provided it's above a minimum value), and it will take care of all similar issues, which otherwise will come at a small trickle indefinitely.

While I agree, it won't stop the int casting issue here (as we can't reasonably forbid constants being over 4 billion). Ideally RTLIL::Const could have a method like representable_as_int() to gate this. #4448 is a basic fix that just disables this optimization if the constant is too wide (I chose 24 bits for now) which prevents it being incorrectly cast to int and also puts an upper bound on the size of the circuit this optimization will generate.

In my view, Yosys should limit both the wire size and the shift size to some reasonable amount well under 4 billion. The Verilog specification explicitly allows this (provided it's above a minimum value), and it will take care of all similar issues, which otherwise will come at a small trickle indefinitely.

It is specified in Verilog-2005 4.3.1 if anyone wants to look it up.
The smallest allowed upper limit on the size of a vector (number of wires/bits) is 65536 ($2^{16}$).
So something like 24 bits seems reasonable to me.

@georgerennie Thank you so much for your help!


[BREAKING] Move sparse functionality into extensions

[!WARNING]
Breaking changes:

DenseSparsityDetector requires loading SparseArrays.jl
Sparse Jacobians and Hessians require loading SparseMatrixColorings.jl
GreedyColoringAlgorithm no longer re-exported from SparseMatrixColorings.jl

DI source

Remove dependency on SparseArrays and SparseMatrixColorings
Remove sparse AD functionalities

DI extensions

Add DISparseArraysExt for DenseSparsityDetector
Add DISparseMatrixColoringsExt for sparse Jacobians and Hessians

Codecov Report
Attention: Patch coverage is 1.80180% with 109 lines in your changes missing coverage. Please review.

Project coverage is 21.78%. Comparing base (de23245) to head (8ef1f37).

Files with missing lines
Patch %
Lines

...ationInterfaceSparseArraysExt/sparsity_detector.jl
0.00%
80 Missing :warning:

...ationInterfaceSparseMatrixColoringsExt/jacobian.jl
0.00%
14 Missing :warning:

...iationInterfaceSparseMatrixColoringsExt/hessian.jl
0.00%
9 Missing :warning:

DifferentiationInterface/src/utils/traits.jl
0.00%
4 Missing :warning:

DifferentiationInterface/src/utils/check.jl
0.00%
1 Missing :warning:

DifferentiationInterface/src/utils/printing.jl
0.00%
1 Missing :warning:

:exclamation:  There is a different number of reports uploaded between BASE (de23245) and HEAD (8ef1f37). Click for more details.
HEAD has 46 uploads less than BASE

Flag
BASE (de23245)
HEAD (8ef1f37)

DIT
10
1

DI
41
4

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #448       +/-   ##
===========================================
- Coverage   98.53%   21.78%   -76.75%
===========================================
  Files         105       93       -12
  Lines        4293     4112      -181
===========================================
- Hits         4230      896     -3334
- Misses         63     3216     +3153

Flag
Coverage Δ

DI
32.80% <1.80%> (-65.88%)
:arrow_down:

DIT
0.14% <ø> (-98.08%)
:arrow_down:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


fix don't work stringify_friend_ids parameter

I tried User Stream API.
I got numeric friend ids.
But I expect stringify friend ids.
I tried changing stringify_friend_ids parameter, @"1" to @"true".
I got a result that is I expected.
When md[@"stringify_friend_ids"] = @"1";
I got numeric friend ids.

md[@"stringify_friend_ids"] = @"true";
I got stringify friend ids. 👍

At the same time other bugs...

please review 😭

thank you, sorry for the delay


List of places keeps popping up when trying to type on a different textInput

const GoogleAutocomplete = (props) =>{
    return(
    <GooglePlacesAutocomplete
            placeholder={props.placeholder}
            minLength={2}
            value = {props.value}
            keyboardAppearance={'light'}
            autoFocus = {false}
            fetchDetails={true}
            onPress = {props.onPress}
            query={{
                key: 'myKey',
                language: 'en', // language of the results
                }}
            styles={{
                textInputContainer: {
                    backgroundColor: 'rgba(0,0,0,0)',
                    borderTopWidth: 0,
                    borderBottomWidth:0,
                    paddingLeft: 50,
                    paddingRight: 50,
                    },
                loader:{
                    height:100,
                },
                textInput: {
                    fontSize: 16,
                    fontFamily:'Bellota-Regular'
                    },
                }}/>
    )
}

export default GoogleAutocomplete;

Above is my component. Each time I click on different <TextInput> and I start to type, the drop-down with all the places that I found previously opens up. How do I disable this behavior?

Please follow the issue template.  If the information in the template is provided, I can try and help you out.

@bell-steven Thank you for redirecting me to the documentation. Would this only work with a class component? What if I wanted to only use this as a functional component?

It doesn't matter what type of component you use, class or functional.  What version of the library are you using?

I am using  "react-native-google-places-autocomplete": "^1.7.1"

Can you provide a minimal code reproduction either here or in a snack?

@bell-steven Never mind, I ran an npm update on the project and it works great now!

No worries. Glad you got it sorted out.


updated readme with handy information about starting ones own app.

This adds a missing piece to the docs for getting one's own slack bot server up and running.

I actually just pushed an update to this based on slack-shellbot and slack-metabot in https://github.com/dblock/slack-bot-server/commit/954a5e4b7c087382a4dbe1394094c058af8fc186, two pet projects. The index.html has become an ERB, so you no longer need to manually edit the HTML file. And the boilerplate is much more developer friendly to work with, and does more for you like de-activating teams that were turned off server-side.
I am going to close this, let me know if there's anything else I missed, thanks!

Btw, one convenient way of getting SLACK_CLIENT_ID and SLACK_CLIENT_SECRET is to create a .env file, then foreman start will read from it. Maybe worth documenting?


Add PhyWhisperer backend.

Thanks for the review! Yeah size=0 is work-in-progress on the PhyWhisperer side, it's intended for stream capture which isn't working yet.

Finally getting around to testing the updated changes; sorry for the delay. :)

This seems to work; so merging it as a baseline. :)
Thanks!


wide actin fusion focus PMID:28410370

wide actin fusion focus
synonyms: wider actin fusion focus; increased actin fusion focus width; increased actin fusion focus diameter
requested as
name: wider actin fusion focus; def: A wider actin fusion focus an abnormal actin fusion focus assembly that results in a wider [than normal] distribution of actin fusion focus components along the shmoo tip.
include link output_of FYPO:0006108

wide actin fusion focus FYPO:0007682


[Improvement] POC for Using Style Dictionary

Description
Style Dictionary is a library that allows for a JSON-structured source file to be parsed out into a number of different formats and languages. This makes having a central source of truth for values such as tokens easy to maintain and sync out.
This Issue suggests we create a POC to test and demo how this can be used for generating tokens in Sage that can be synced from one central or set of JSON files across SCSS, Rails, JS, and React. It will also be helpful as a precursor to using Typescript well.

NOTE: As of now an implementation is set up in #837 that processes css color tokens and outputs them to packages/sage-assets/lib/stylesheets/dictionary/_tokens.scss. These are then wrapped by existing variables and functions in packages/sage-assets/lib/stylesheets/tokens/_color_combos.scss and packages/sage-assets/lib/stylesheets/tokens/_color_palette.scss.
Next steps:

[ ] Get classnames and hex values exporting in Rails and React tokens.
[ ] Evaluate additional tokens to migrate next

Now in Jira https://kajabi.atlassian.net/browse/UXD-52


allow disable api validation

Pull Request Checklist
Add option to disable api validation when deploying sensu agent
Description
Currently, when you deploy a sensu agent it makes 2 sensu API call, but I don't wan't explose the credential on agent side
Related Issue
https://github.com/sensu/sensu-puppet/issues/1322
Apply requested change in the following PR https://github.com/sensu/sensu-puppet/pull/1328
Fixes # .
Motivation and Context
I use Sensu myself and I've always deployed agents to be "standalone" and would like to be able to continue that way.
How Has This Been Tested?
Deployed it within my org, added:
sensu::validate_api: false
sensu::agent::agent_managed_entity: true

General

[x] Update README.md with any necessary configuration snippets

[x] New parameters are documented

[ ] New parameters have tests

[x] Tests pass - bundle exec rake validate lint spec

Closed in favor of #1332 1332


Test Python 3.6, 3.8, 3.11 on CI

Does what it says on the tin

I think we need to update mypy and / or black to remove the old typed-ast dependency

I think we need to update mypy and / or black to remove the old typed-ast dependency

Hmmm how does one update pip dependencies?  I don't do nearly enough Python development to know how this works...  could you possibly fix this, please? :blush:

Ahhh, thank you very much for doing this!  All looks good to me, I'll merge it now.


[redis] Update redis_exporter to latest release

What this PR does / why we need it:

Update redis_exporter to its latest release.
Update docs accordingly

/assign @carrodher

LGTM but I am not sure if we should increase the major version, what do you think @tompizmor?

I would increase the minor version. So 3.5.0

/ok-to-test
/lgtm


How to get 6dof for D435i ?

hey ,
I am working on a project related to odometry for which i need to record the 6DOF (Degree of Freedom) from intel D435i. So can someone help me that how to get this data.
Thankyou.

The RealSense T265 Tracking Camera has built-in support for accessing 6-DOF pose data but the D435 depth camera models do not.
If you want to get pose on a D435 (non-IMU), there are methods for doing so with OpenCV (Solve PNP algorithm) or ROS (Inference).
https://github.com/IntelRealSense/librealsense/issues/6038#issuecomment-598089815
There is also a D435i-compatible sample program called rs-motion that"demonstrates how to use data from gyroscope and accelerometer to compute the rotation of the camera".
https://dev.intelrealsense.com/docs/rs-motion
getting the D435i's rotation from the IMU.

Thanks fro you reply @MartyG-RealSense
Is their any document or Github material available regarding (Solve PNP algorithm) and ROS (Inference) which i can prefer to move foward?

A RealSense community member who was using the SolvePNP algorithm recommended this guide:
https://www.learnopencv.com/head-pose-estimation-using-opencv-and-dlib/
In regard to ROS, I couldn't find a good example of using RealSense to get pose with inference other than the project I linked to before.  As a potential alternative approach though, a RealSense D435 user also recommended a program called OpenPose and its ROS wrapper:
https://github.com/stevenjj/openpose_ros

I want to use d453i for making trajectory as t265 can do can it be possible to use it for trajectory.

I couldn't find an easy way to track a trajectory with D435i.  Someone did render a trajectory with a D435i in ROS though through using the RViz program.
https://github.com/rpng/R-VIO/issues/8
Also using ROS, Intel published an article on setting up SLAM with D435i, though I'm not sure that is what you are looking for.
https://github.com/IntelRealSense/realsense-ros/wiki/SLAM-with-D435i

@MartyG-RealSense
I am looking for rotation and translation matrix . like will it support RS2_STREAM_POSE (as t265 supports to get rotation and translation)

The last I heard about it was around a year ago, in a comment where @dorodnic explained the difficulties of implementing SLAM for D435i.
https://github.com/IntelRealSense/librealsense/issues/3070#issuecomment-454689295
In an earlier discussion, Dorodnic recommended looking at the D435i SLAM example (which I highlighted earlier) and also the RealSense Kinfu example linked to below.
https://github.com/IntelRealSense/librealsense/tree/master/wrappers/opencv/kinfu

This case will be closed after 7 days from the date of writing this if there are no further responses.   Thanks!


metadata.py: com_google_fonts_check_metadata_subsets_order added

Fixes #4779
Checklist

[x] update CHANGELOG.md
[x] wait for the tests to pass
[x] request a review

@felipesanches @simoncozens either of you available for a review? This test is quite important. Can squash as well.

This LGTM for now. We should combine all the subsets checks at some point, but now is not that point.


Key error on tilda

Trying to train the model on libriTTS. After successful preprocessing, it trows an error on train
KeyError: '̃'

Depending on the text phenomization you are using, you might need to update text/symbols.py file. I had similar issue when I used TTS for another language.


Add Kerbal Changelog as a dependency of Oh Scrap

I assume this won't effect old versions? Only new ones?

Changing the netkan affects the current version plus future versions.
To change older versions, edit them here:
https://github.com/KSP-CKAN/CKAN-meta/tree/master/OhScrap

That's fine, that's what I want. I don't want old versions to be touched as Changelog isn't compatible with 1.3.x.


feat(you-retriever): Support You.com News API

You.com has both a Search and News API. Currently the YouRetriever only supports Search.
This PR enables it to also support News, while maintaining backwards compatibility.
This update has been manually tested:
from dspy.retrieve.you_rm import YouRM

search_rm = YouRM(endpoint="search")
res = search_rm("Who is the most famous singer in the 90s?")

news_rm = YouRM(endpoint="news")
res = news_rm("Latest pop songs")

Thanks @christeefy !
It would also be great if you could add documentation for the YouRM to provide context as done for the RMs. Feel free to add all relevant content for the YouRM API documentation linked).


Remove Series Selector

Is your feature request related to a problem? Please describe.
Can be removed, not essential. There are other ways to select a series.
Describe the solution you'd like
Remove the component
Describe alternatives you've considered
NA
Additional context
NA

Removed in #534


support of XCode 8+

Hello
Thanks for your hard work - I can see no new version support the new XCode. I sadly have a new machine and I'm stuck with the latest XCode and UUID hack proposed does not add a menu item.
Is there an update on the way?
thanks

No, sorry, it's not really possible with Xcode 8 since they revamped the plugin API, see https://github.com/openframeworks/OFXcodeMenu/issues/46


Better -(NSString*)description for BFTask

See https://github.com/BoltsFramework/Bolts-iOS/pull/67 for the original PR.
Updated with @nlutsenko-fb comments and squashed. Also added a small test.

Build needs re-running, tests passed but then
/Users/travis/build.sh: line 41:  1181 Segmentation fault: 11  xcodebuild -project Bolts.xcodeproj -sdk macosx -scheme MacBolts

Re-enqueued build.
![Xcode]
(http://cdn.meme.am/instances/500x/58496306.jpg)

I just pushed a small changed that fixed the formatting (from -(NSString*)description to - (NSString *)description)

Looks great! Merging now.


Update interval?

Hi!
Thanks you for this card! The kids like it :)
However, I can't get it to update every day. I have the menu for current day displayed on a screen attached to the refrigerator. It shows yesterday menu until I refresh the android app.
Any update parameter I have missed?
BR/Nicklas

Hi Nicklas, sounds like my bad. I'll put in a fix.
/Anders

Should be fixed, please verify

Thanks! Until now I have used a markdown card but have switched back to your card now.


Unable to load script, make sure your bundle "index.android.bundle" is packaged correctly for release

New Version
0.66.5+
Old Version
0.63.4
Build Target(s)
Android Emulator and release build as well
Output of react-native info
System:
OS: macOS 12.6.1
CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
Memory: 170.70 MB / 16.00 GB
Shell: 5.8.1 - /bin/zsh
Binaries:
Node: 16.14.0 - /usr/local/bin/node
Yarn: Not Found
npm: 8.3.1 - /usr/local/bin/npm
Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
Managers:
CocoaPods: 1.11.3 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 22.1, iOS 16.1, macOS 13.0, tvOS 16.1, watchOS 9.1
Android SDK: Not Found
IDEs:
Android Studio: 2021.3 AI-213.7172.25.2113.9014738
Xcode: 14.1/14B47b - /usr/bin/xcodebuild
Languages:
Java: 11.0.2 - /Users/kapil.raghuwanshi/.sdkman/candidates/java/current/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 16.13.1 => 17.0.2
react-native: 0.63.4 => 0.66.5
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Issue and Reproduction Steps
We are trying to migrate our react native project in order to support android 12.
https://reactnative.dev/blog/2021/09/01/preparing-your-app-for-iOS-15-and-android-12
We are struggling to resolve this issue past few days.

We searched on the internet
https://stackoverflow.com/questions/55441230/unable-to-load-script-make-sure-you-are-either-running-a-metro-server-or-that-yo/58570426#58570426
and found a few solutions below:

adb reverse tcp:8081 tcp:8081

setting port and host configs

react-native bundle --platform android --dev false --entry-file index.js --bundle-output android/app/src/main/assets/index.android.bundle --assets-dest android/app/src/main/res

Out of which all are working fine in our local system and emulators
but once we create a release build for the production environment, no luck with any of such hacks.
Could you please help us out in resolving this issue?
@sammy-SC

If I understand correctly, you managed to solve it on local development, with adb reverse tcp:8081 tcp:8081, which is correct.
As for the release build, you might want to make sure you are using the correct version of com.android.tools.build:gradle that came with the current version of RN, else it might not able to bundle your JS code correctly.

Actually, you shouldn't even need to run react-native bundle

We are using com.android.tools.build:gradle 7.3.1 as per the https://reactnative.dev/blog/2021/09/01/preparing-your-app-for-iOS-15-and-android-12.
 classpath('com.android.tools.build:gradle:7.3.1') inside build.gradle
distributionUrl=https\://services.gradle.org/distributions/gradle-7.4-bin.zip inside gradle-wrapper.properties

@kapilraghuwanshi Based on the RN version (0.66~) that you provide, you should be using com.android.tools.build:gradle:4.2.2 instead.
https://github.com/react-native-community/rn-diff-purge/blob/version/0.66.5/RnDiffApp/android/build.gradle
I strongly advise you to use the Gradle version that comes with your current RN version, or else you might face some build issues as I mentioned earlier. Or you spend some time upgrading the RN version which supports a much higher Gradle version.
Trust me, I literally face the exact same issue where I accidentally change the Gradle version via Android studio.

Thanks for the reply bro chungweileong94
as per your shared link https://github.com/react-native-community/rn-diff-purge/blob/version/0.66.5/RnDiffApp/android/build.gradle,
targetSdkVersion = 30
we are planning to target android 12, so we must have targetSdkVersion=31.
Our build.gradle looks like below
buildscript {
    ext {
        buildToolsVersion = "30.0.3"
        minSdkVersion = 21
        compileSdkVersion = 31
        targetSdkVersion = 31
        hyperSDKVersion = "2.0.4-rc.34"
        kotlinVersion = "1.7.10"
    }
    repositories {
        google()
        jcenter()
    }
    dependencies {
        classpath('com.android.tools.build:gradle:7.0.4')
    }
}

if you can help us sammy-SC

We also face the same problem (Unable to load script, make sure your bundle "index.android.bundle" is packaged correctly for release)
Our build gradle looks like this:
ext {
        buildToolsVersion = "31.0.0"
        minSdkVersion = 21
        compileSdkVersion = 31
        targetSdkVersion = 31
        if (System.properties['os.arch'] == "aarch64") {
            // For M1 Users we need to use the NDK 24 which added support for aarch64
            ndkVersion = "24.0.8215888"
        } else {
            // Otherwise we default to the side-by-side NDK version from AGP.
            ndkVersion = "21.4.7075529"
        }
    }
    repositories {
        google()
        mavenCentral()
    }
    dependencies {
        classpath("com.android.tools.build:gradle:7.2.1")
        classpath("com.facebook.react:react-native-gradle-plugin")
        classpath("de.undercouch:gradle-download-task:5.0.1")
        // NOTE: Do not place your application dependencies here; they belong
        // in the individual module build.gradle files
    }

React native version 0.70.6

@kapilraghuwanshi Based on the RN version (0.66~) that you provide, you should be using com.android.tools.build:gradle:4.2.2 instead.

I just tried it out with a different project setup. It works fine until launching in an emulator with API level 31, regardless of gradle / react native version.
RN v 0.70.6 + gradle 7.2.1 leads to this error message,
RN v 0.66.1 + gradle 4.2.2 also leads to this error message
but only on devices with API > 31 (Android 12)

Thanks, @Fipsiflip. Yes, we tried all the above-mentioned RN and Gradle mappings which are leading us toward the same error message. Tried downgrading to RN66 and upgrading to the latest RN70 also but no luck.

RN v 0.70.6 + gradle 7.2.1 leads to this error message,

Am I reading right that this issue is happening on a empty 0.70.6 project?

@kapilraghuwanshi maybe this will also help you #35523, this solved the problem for me

@kapilraghuwanshi maybe this will also help you #35523, this solved the problem for me

Yes, sometimes you have to do this for development, but that shouldn't be the case for release build.

RN v 0.70.6 + gradle 7.2.1 leads to this error message,

Am I reading right that this issue is happening on a empty 0.70.6 project?

From what I have tested on the fresh RN 0.70.6 project, both the dev & release build work fine, at least for me.
@kapilraghuwanshi Is it possible for you to create a small example project that can reproduce this problem?

@kapilraghuwanshi maybe this will also help you #35523, this solved the problem for me

Yeah I already mentioned this in my original issue description but somehow it is not working on my final package built using react native but gets installed on seperate native android project. There it is still the same issue.

RN v 0.70.6 + gradle 7.2.1 leads to this error message,

Am I reading right that this issue is happening on a empty 0.70.6 project?

From what I have tested on the fresh RN 0.70.6 project, both the dev & release build work fine, at least for me. @kapilraghuwanshi Is it possible for you to create a small example project that can reproduce this problem?

let me give this a try.

Somehow it did work on the release build with the below combination:
gradle plugin (build.gradle): 7.0.1
gradle: 7.3.3
Kotlin: 1.7.20
RN: 0.68.5
React: 17.0.2
looks like an android native side issue not react native.

Somehow it did work on the release build with the below combination:
gradle plugin (build.gradle): 7.0.1 gradle: 7.3.3 Kotlin: 1.7.20 RN: 0.68.5 React: 17.0.2
looks like an android native side issue not react native.

How come?  the conclusion is not very obvious.


Add support for MySQL SSL connections. Uses ssl dictionary in extras …

…to take cert, ca, and key pem locations.

Tested that this works against our SSL-based MySQL server with the 'Extras' field set to:
{"ssl": {"cert": "/Path/to/cert.pem", "key": "/Path/to/key.pem", "ca": "/Path/to/ca.pem"}}

Do you guys want me to write a test for this? Seems kind of difficult given it requires an SSL-encrypted MySQL DB. I suppose I could mock the MySQLDB connection, and validate that the JSON comes in as expected, but seems painful.

:+1:


[GithubActionsWorkflowStatus] Support private repos

Implements #9189
The badge is fully backwards compatible with the tests of the current implementation, but instead of the badge.svg URL it fetches the workflow status using the GitHub API, allowing for access to private repos using a PAT.
As a bonus, the :workflow path parameter is now optional. If empty, it takes the status of the latest run of all workflows in a repository.
Depending on whether the workflow name is passed, it uses either the List runs for a repo API or List runs for a workflow API.
I made a few small changes outside of the service code. I may be able to implement this in other ways if it doesn't fit the overall design. Please let me know what you think:

In build-status.js I added a readableStatus, replacing _ with   for readability. This will show a badge message in progress instead of in_progress, which I think is nicer to read.
In validate.js, I added a way to pass options to the _validate method via _requestJson to be able to override the prettyErrorMessage. I did this to make sure the no build message is set if any validation error occurs (e.g. if there are no runs found). Another way would be to loosen the schema of the request and check the length somewhere else.

Please let me know any feedback you have. I'm not a first-time contributor to shields, but it's been a long time so I might be rusty :see_no_evil:

I suspect we're going to close this but I'm going to mark it "on hold" for the moment while we continue the conversation over in https://github.com/badges/shields/issues/9189#issuecomment-1556265808

https://github.com/badges/shields/issues/9189#issuecomment-1556265808


GetPolicyDocumentOutput panics instead of returning error

Hello!

Vote on this issue by adding a 👍 reaction
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already)

Issue details
This is a problematic line:
https://github.com/pulumi/pulumi-aws/blob/f73d82bf022793a85cdd5bf7ac55638b64740cb9/sdk/go/aws/iam/getPolicyDocument.go#L141
(expanded below)
func GetPolicyDocumentOutput(ctx *pulumi.Context, args GetPolicyDocumentOutputArgs, opts ...pulumi.InvokeOption) GetPolicyDocumentResultOutput {
	return pulumi.ToOutputWithContext(context.Background(), args).
		ApplyT(func(v interface{}) (GetPolicyDocumentResult, error) {
			args := v.(GetPolicyDocumentArgs)
			r, err := GetPolicyDocument(ctx, &args, opts...)
			return *r, err
		}).(GetPolicyDocumentResultOutput)
}

if GetPolicyDocument returns nil, <error here>, then *r will result in a panic
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x1726ecf]

Steps to reproduce
_ = iam.GetPolicyDocumentOutput(ctx, iam.GetPolicyDocumentOutputArgs{
	Statements: iam.GetPolicyDocumentStatementArray{
		iam.GetPolicyDocumentStatementArgs{
			Actions: pulumi.StringArray{
				pulumi.String("sts:AssumeRoleWithWebIdentity"),
			},
			Conditions: iam.GetPolicyDocumentStatementConditionArray{
				iam.GetPolicyDocumentStatementConditionArgs{
					Test: pulumi.String("StringLike"),
					Values: pulumi.StringArray{
						pulumi.Sprintf("repo:%s/%s:*", "mytestorg", "mytestrepo"),
					},
					Variable: pulumi.String("token.actions.githubusercontent.com:sub"),
				},
			},
			Principals: iam.GetPolicyDocumentStatementPrincipalArray{
				iam.GetPolicyDocumentStatementPrincipalArgs{
					Identifiers: pulumi.StringArray{
						pulumi.String("arn:aws:iam::12345678:oidc-provider/token.actions.githubusercontent.com"),
					},
					Type: pulumi.String("Federated"),
				},
			},
		},
	},
})

    panic: runtime error: invalid memory address or nil pointer dereference
    [signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x17240af]
    goroutine 23 [running]:
    github.com/pulumi/pulumi-aws/sdk/v4/go/aws/iam.GetPolicyDocumentOutput.func1({0x18fe240?, 0xc0005a4380})
    	/Users/wmcnamee/.asdf/installs/golang/1.18/packages/pkg/mod/github.com/pulumi/pulumi-aws/sdk/v4@v4.38.1/go/aws/iam/getPolicyDocument.go:356 +0x18f
    reflect.Value.call({0x181b940?, 0xc000580480?, 0x100ea45?}, {0x197129f, 0x4}, {0xc00050e018, 0x1, 0xc0005b8240?})
    	/Users/wmcnamee/.asdf/installs/golang/1.18/go/src/reflect/value.go:556 +0x845

Note that the line numbers don't match up with the source code (in this repo), but here's a screenshot from the vendored code:

Expected: to get a human readable error
Actual: panic and nil pointer exception

after figuring out how to attach a debugger, I can further confirm this is exactly the problem.

Paging @t0yv0 to verify that this was fixed in pulumi/pulumi v3.25.1?

Yes this is still an issue https://github.com/pulumi/pulumi-aws/blob/master/sdk/go/aws/iam/getPolicyDocument.go#L141

@ghostsquad Thank you again for noticing and reporting!
We verified it's still an issue and are working on a fix. The issue is in pulumi/pulumi codegen, which is how these files get created, and once we ship that fix, on the next release of this provider this issue should be resolved.

I'm running pulumi cli 3.37.2, and it appears this is still an issue. Do you know what's going on?

https://github.com/pulumi/pulumi-aws/blob/master/sdk/go/aws/iam/getPolicyDocument.go#L215 the fix seems to be applied on master. Mind letting us know which version of the pulumi-aws Go SDK you are using so we can repro? 🙇


delay before command runs

Sometimes I made small changes and press ctrl + s by muscle memory.
Would it be possible to add some kind of delay so that it waits n milliseconds before running the command and if before that time the document is saved again the timer starts again from n?

I think runOnSave runs only again after the first command finishes unless you are using isAsync: true.
And a delay can be done by sleep 1 && ./yourCommand (waits 1 second)

Wouldn't that still run the command as many times as I pressed save?
What I'm looking for is a delay before running the command so however many times I press save, the timer gets reset and at the end, if the timer runs out, the command is run only once.

Could be but you need to try it out :)
You could work around it by adapting a script that writes a simple lock file somewhere, like .runOnSaveLock. If that file exists, skip execution.
Looking at the state of this repo, I guess you would need to introduce it via a PR yourself, if you want it

I didn't even notice last update was 4 years ago, I'll go with the lock file.


Adding german special characters to *.ts-files breaks source maps with workspaces

Which @angular/* package(s) are the source of the bug?
compiler
Is this a regression?
Not sure
Description

Create a new project with ng new and default options (no routing, CSS)

Run with ng serve, open in Chrome or Firefox

In the developer tools, add the directory as a workspace. The local *.ts files are recognized as indicated by the little green dots:

Now go to app.component.ts and change the title so that it contains an umlaut such as ü:

The browser refreshes and looses the link to app.component.ts:

As a consequence, breakpoints in this file are no longer hit.
Expected results: Adding special characters in the utf-8 encoded file does not impact source maps.
Please provide a link to a minimal reproduction of the bug
No response
Please provide the exception or error you saw
No response
Please provide the environment you discovered this bug in (run ng version)
Angular CLI: 13.2.5
Node: 16.14.0
Package Manager: npm 8.3.1
OS: darwin arm64

Angular: 13.2.5
... animations, cli, common, compiler, compiler-cli, core, forms
... platform-browser, platform-browser-dynamic, router

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1302.5
@angular-devkit/build-angular   13.2.5
@angular-devkit/core            13.2.5
@angular-devkit/schematics      13.2.5
@schematics/angular             13.2.5
rxjs                            7.5.4
typescript                      4.5.5

Anything else?
No response

Hi @markuspalme,
I did try to look into this and it doesn't seem an issue with the sourcemaps generation when using UTF-8 characters as they are being generated correctly.
You can take a look here: https://justin.ridgewell.name/source-map-visualization-evanw/#NjQyACJ1c2Ugc3RyaWN0IjsKKHNlbGZbIndlYnBhY2tDaHVua3NtX3VuaWNvZGVfdGVzdCJdID0gc2VsZlsid2VicGFja0NodW5rc21fdW5pY29kZV90ZXN0Il0gfHwgW10pLnB1c2goW1sibWFpbiJdLHsKCi8qKiovIDEzOToKLyohKioqKioqKioqKioqKioqKioqKioqISpcCiAgISoqKiAuL3NyYy9tYWluLnRzICoqKiEKICBcKioqKioqKioqKioqKioqKioqKioqLwovKioqLyAoKCkgPT4gewoKCmNvbnN0IHggPSAnYWFhw7xhJzsKY29uc29sZS5sb2coeCk7CgoKLyoqKi8gfSkKCn0sCi8qKioqKiovIF9fd2VicGFja19yZXF1aXJlX18gPT4geyAvLyB3ZWJwYWNrUnVudGltZU1vZHVsZXMKLyoqKioqKi8gdmFyIF9fd2VicGFja19leGVjX18gPSAobW9kdWxlSWQpID0+IChfX3dlYnBhY2tfcmVxdWlyZV9fKF9fd2VicGFja19yZXF1aXJlX18ucyA9IG1vZHVsZUlkKSkKLyoqKioqKi8gX193ZWJwYWNrX3JlcXVpcmVfXy5PKDAsIFsidmVuZG9yIl0sICgpID0+IChfX3dlYnBhY2tfZXhlY19fKDIwMiksIF9fd2VicGFja19leGVjX18oMTM5KSkpOwovKioqKioqLyB2YXIgX193ZWJwYWNrX2V4cG9ydHNfXyA9IF9fd2VicGFja19yZXF1aXJlX18uTygpOwovKioqKioqLyB9Cl0pOwovLyMgc291cmNlTWFwcGluZ1VSTD1tYWluLmpzLm1hcDMwNgB7CiAgInZlcnNpb24iOiAzLAogICJmaWxlIjogIm1haW4uanMiLAogICJtYXBwaW5ncyI6ICI7Ozs7Ozs7Ozs7QUFBQSxNQUFNLENBQUMsR0FBVyxPQUFPLENBQUM7QUFFMUIsT0FBTyxDQUFDLEdBQUcsQ0FBQyxDQUFDLENBQUMsQ0FBQyIsCiAgInNvdXJjZXMiOiBbCiAgICAiLi9zcmMvbWFpbi50cyIKICBdLAogICJzb3VyY2VzQ29udGVudCI6IFsKICAgICJjb25zdCB4OiBzdHJpbmcgPSAnYWFhw7xhJztcblxuY29uc29sZS5sb2coeCk7XG4iCiAgXSwKICAibmFtZXMiOiBbXSwKICAic291cmNlUm9vdCI6ICJ3ZWJwYWNrOi8vLyIKfQo=
It also appears that breakpoints stops working when using the Filesystem feature as otherwise they are kept bound which confirms that the source-maps are being generated correctly. Therefore, this seems to be more of bug in the browser.

@alan-agius4 Thanks, that sounds reasonable. The same behavior can be observed in Chromium based browsers as well as Firefox. That makes it rather unlikely to be a browser issue though.

@markuspalme, how to do you replicate the issue in Firefox?

In that case I am closing this issue. I suggest you file an issue with Chrome.


[zh-cn]sync nodes assign-pod-node source-ip kubeadm_config_print_reset-defaults

# Changes to be committed:
#       modified:   content/zh-cn/docs/concepts/architecture/nodes.md
#       modified:   content/zh-cn/docs/concepts/scheduling-eviction/assign-pod-node.md
#       modified:   content/zh-cn/docs/reference/setup-tools/kubeadm/generated/kubeadm_config_print_reset-defaults.md
#       modified:   content/zh-cn/docs/tutorials/services/source-ip.md

/lgtm
/approve


Interleaved Two of Five

Hi,
I need to generate a bar code in the "Interleaved Two of Five" format but with 44 numbers, how can I do this?
Tks!

I would not recommend generating this long of an ITF barcode but I don't see the problem in doing it.
https://codepen.io/lindell/pen/WOQdNL

@lindell The Brazilian regulatory body for bank slips defined this standard for the barcode of normal bills.


#import <Cordova/CDVJSON.h> not found

It seems that CDVJSON is no longer supported and now it won't build on iOS.

Same thing happens to me.

Nevermind the above, I was on an older commit, the latest commit solved my problem.


fix bad early-outing behavior

also be more explicit about when we can early-out, don't hide it in RemoteAssertions()
@MarcoPolo needed this for a failure to cache bust

@patrickxb can you take a look?

LGTM pending CI fix (I think CI broken in general right now)


Test/cvutils

Codecov Report

Merging #212 into master will increase coverage by 1.12%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #212      +/-   ##
==========================================
+ Coverage   87.65%   88.77%   +1.12%
==========================================
  Files          23       23
  Lines        1077     1078       +1
==========================================
+ Hits          944      957      +13
+ Misses        133      121      -12

Impacted Files
Coverage Δ

torchbearer/cv_utils.py
100% <100%> (+81.81%)
:arrow_up:

torchbearer/callbacks/aggregate_predictions.py
58.33% <0%> (-41.67%)
:arrow_down:

torchbearer/callbacks/callbacks.py
92.2% <0%> (-1.3%)
:arrow_down:

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 7e255e0...398ee49. Read the comment docs.


Examples for everything would be really useful

There's nothing for Join, Intersect or Union (just for example). What is a "key selector method" exactly? And so on.

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

ID: a06a3c75-fe34-acfa-51ef-4296795e8f70
Version Independent ID: 5c3bdaf5-eec2-3ea2-3c5b-f53f7b80b41b
Content: Using LINQ With the debugger objects - Windows drivers
Content Source: windows-driver-docs-pr/debugger/using-linq-with-the-debugger-objects.md
Product: windows-hardware
Technology: windows-devices
GitHub Login: @DOMARS
Microsoft Alias: domars

@M-Hamilton - Sorry for the delay here and thanks for your request. It's a great idea, but unfortunately, I continue to be tasked with high-priority tasks for the foreseeable future. You might want to ask for examples on the Windows Hardware WDK and Driver Development forum and OSR.
•	Windows Hardware WDK and Driver Development
•	OSR's windbg List
Or if you do create any samples, feel free to submit it to the docs for consideration, using a commit.
Sorry I can't be of more help. Good luck in your work with LINQ and the debugger objects.


Kafka-15680 :: Calling UpdatePartitionCount method from "onPartitionsAssigned()" to handle the partition addition and revocation simultaneoulsy.

In ICR(Incremental Cooperative Rebalancing) mode, whenever a new worker, say Worker 3 joins, a new global assignment is computed by the leader, say Worker1, that results in the revocation of one task from each existing worker i.e Worker1 and Worker2.
Once the new member join is completed, ConsumerCoordinator.OnJoinComplete() method is called which primarily computes all the new partitions assigned and the partitions which are revoked and updates the subscription Object.
If it was the case of revocation which we check by checking the “partitonsRevoked” list, we call the method “invokePartitionRevoked()” which internally calls “updatePartitionCount()” which fetches partition from the assignment object which is yet not updated by the new assignment.
It is only just before calling the “invokePartitionsAssigned()” method that we update the assignment by invoking the following → subscriptions.assignFromSubscribed(assignedPartitions);

@kumarpritam863 is this the same as #12622 and https://issues.apache.org/jira/browse/KAFKA-14220? And if so, can you shed some light on why that PR and ticket were wiped and closed without comment after the initial round of review?

Yes @C0urante This is same and I closed the PR and JIRA at that time due to some privacy policy about which I was not sure at that time. Really sorry for that as that was miss from my side. Can we please precede this with new PR and JIRA.

@C0urante
It's a small but important modification, just a change in the order of calling the UpdatePartitionCount method and practically involve no code change.
I kindly request your review and approval for this PR. Your quick attention to this matter would be greatly appreciated as it will help us maintain code quality and ensure a seamless development process.
If you have any questions or need further clarification about the changes, please feel free to reach out to me. Your insights and expertise are highly valued, and your approval would be really helpful.
Thank you in advance for your time and consideration.
Best regards,
Pritam

I can take a look next week. In the meantime, do you think you could add some unit tests for this change?

@C0urante
As I mentioned this is a very small change just a change in the order of calling the same function which updates the partition count. This involves practically no code change and more than that I almost explored all the options to add UNIT or Integration test but it seems very complex adding test cases for this change. But we have tested this in our lower environments with multiple scenarios covering multiple dimensions and we have not encountered any issue. I have already spent a lot of time figuring out ways to add unit test cases.
These are the dew reasons why adding unit test is difficult for this:

Although the change is in workerSinkTask but the root cause of the issue is in ConsumerCoordinator
So the current unit test cases for worker sink task mocks the consumer assignment itself.
ConsumerCoordinator is not even a dependency for workerSinkTask and it is indirectly accessed via the kafka consumer Client.
"onJoinComplete" method is protected and can be called from that module itself.

Can we please take this without the test case or with the existing test case. This is a really small change but of utmost important to us as we heavily rely on this metrics.
Thank you in advance for your time and consideration.
Best regards,
Pritam

@kumarpritam863 I understand that the code changes are small, and I'm glad to hear that pre-prod testing indicates that this commit has the intended effect. However, I still think that unit testing coverage is warranted--we want to make sure not just that this change works now, but that we don't accidentally regress in the future.
I think it should be possible to use the MockConsumer class (possibly with some small tweaks to is rebalance method to make the ordering of steps more closely match the ordering in the KafkaConsumer class when rebalances take place) to write a new unit test in the WorkerSinkTaskTest suite that verifies that the partition-count metric is updated appropriately.
You may also notice that existing unit tests are failing now (see the CI results, or feel free to test this PR locally with ./gradlew :connect:runtime:unitTest). Can you please fix the failing tests and make sure that the build works (at least locally) before requesting review again?
Finally, can you update the title of the PR to describe what changes are actually made? You can see examples of good PR titles in the list of recently-merged commits to trunk.
Thanks!

https://github.com/apache/kafka/blob/9dbee599f13997effd8f7e278fd7256b850c8813/clients/src/main/java/org/apache/kafka/clients/consumer/MockConsumer.java Implements clients/src/main/java/org/apache/kafka/clients/consumer/Consumer.java but worker connect/runtime/src/main/java/org/apache/kafka/connect/runtime/WorkerSinkTask.java expects a clients/src/main/java/org/apache/kafka/clients/consumer/KafkaConsumer.java. The only way to use "MockConsumer" would be to create "MockWorkerSinkTask" which is not a small tweak and would be a big change to test this small code change.
Please let me know your thoughts on this and if I mis-understood something or if I am going down a wrong lane.
Thanks
Pritam

@C0urante All the tests passed locally. But I will check for the failing tests again and will update here.

I ran the following command:
./gradlew :connect:runtime:unitTest
And all the test passed locally and build is successful.

BUILD SUCCESSFUL in 3m 50s
73 actionable tasks: 6 executed, 67 up-to-date

That's fine, we can change the left-hand type for WorkerSinkTask.consumer from KafkaConsumer<byte[], byte[]> to Consumer<byte[], byte[]>.

@C0urante Did not get the above part. Do you want me to change the type of consumer from KafkaConsumer<byte[], byte[]> to Consumer<byte[], byte[]> in the WorkerSinkTask class.

Yes, exactly 👍

@C0urante I have added the test for both the scenarios:

Test failing if the ConsumerRebalanceListener of WorkerSinkTask adopts to current flow where we update the partition count before updation of Subscription Object.
Test Passing if the ConsumerRebalanceListener of WorkerSinkTask adopts to the flow added by this PR.

Can you please review and let me know if any other change or modification is required.

@C0urante I have addressed all the comments and made the changes accordingly. Can you please review once and let me know if any changes are required or I missed anything.

Thanks @C0urante


feat: navbar width toggle

Click on the logo to collapse the side nav. Click again to get it back. Invision design here: https://projects.invisionapp.com/d/main#/console/13924274/294213563/preview

(the weirdness with the light blue top bar is just an artefact from the gif screen recorder.)
License: MIT
Signed-off-by: Oli Evans<EMAIL_ADDRESS>

@fsdiogo strongly agree. I'll add something that appears when you hover over the nav.

(in another PR tho)

Awesome 👌
We can think of something cool 😎

I'm merging this. I have a way to fix the size of the action bar bellow.


chore: rm redundant pin_mut

rm extra pinmut overlooked in #2070

Codecov Report

Merging #2072 (d0c5378) into main (fa62514) will decrease coverage by 54.83%.
The diff coverage is 0.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff             @@
##             main    #2072       +/-   ##
===========================================
- Coverage   72.12%   17.30%   -54.83%
===========================================
  Files         434      434
  Lines       53123    53122        -1
===========================================
- Hits        38317     9193    -29124
- Misses      14806    43929    +29123

Flag
Coverage Δ

integration-tests
17.30% <0.00%> (-2.08%)
:arrow_down:

unit-tests
?

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

Impacted Files
Coverage Δ

bin/reth/src/runner.rs
0.00% <0.00%> (ø)

... and 310 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Add linkerd-viz-app to app catalog

Linkerd provides a dashboard and a set of grafana dashboards. Usually this is installed through linkerd viz install | kubectl apply -f -.
We suspect the cli uses this helm chart to generate the manifests.
Preliminary todo list:

[x] Add a linkerd2-viz app to the appcatalog

[x] Investigate integration with existing prometheus and grafana

[ ] Explore using the new app as a subchart in linkerd2-app
[ ] Add PSPs to allow using EmptyDirs to manifests

@ced0ps started working on https://github.com/giantswarm/linkerd-viz

linkerd-viz-app has been release and is available in the app catalog


Copy-POL-262-AWS-Unused-IP

Description
modified the tag format
Contribution Check List

[ ] All tests pass.
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable
[ ] New functionality has been documented in CHANGELOG.MD

          1 Warning

:warning:
Should this include readme changes

  Generated by :no_entry_sign: Danger


[Feature Request] Support Firefox

Hi there. I've been using this extension for the longest time.
But now that I'm using Firefox, I need to find a good alternative. Haven't been able to.
I also tried installing the extension using Foxified, which is a firefox addon that'll allow you to use "some" Chrome extensions in Firefox.
I realize that coding this extension to work with Firefox would require a ton of work and possibly even re-coding the entire extension.
So do you think you could at least make it work that way?

I've heard Firefox uses the same architecture as Chrome now, so I will investigate this.

I was able to investigate and this seems very doable with the latest version of Firefox. I will be repurposing this issue to include any tasks for supporting Firefox

Now available on firefox - https://addons.mozilla.org/en-US/firefox/addon/stylebot-web/ 🥳

Now available on firefox - https://addons.mozilla.org/en-US/firefox/addon/stylebot-web/ 🥳


Error : No such file or directory

Tried to use the docker image, and see this error in the logs.
Are any volume mappings necessary for this to work ?
Am I correct to assume that I will be able to play the game via browser if the container is up and running ?
I dont see a mention of any ports, so I assume its port 80 by default ?
Error: Os { code: 2, kind: NotFound, message: "No such file or directory" }
Error: Os { code: 11, kind: WouldBlock, message: "Resource temporarily unavailable" }
Error: Os { code: 11, kind: WouldBlock, message: "Resource temporarily unavailable" }
Error: Os { code: 11, kind: WouldBlock, message: "Resource temporarily unavailable" }
Error: Os { code: 11, kind: WouldBlock, message: "Resource temporarily unavailable" }

To play the game using the docker command you need to run the container using -it (interactive TTY) since the docker image only gives an entrypoint to the chess-tui binary, this is why you don't see any port number.
You can pull the image and directly run it using this command docker run --rm -it ghcr.io/thomas-mauran/chess-tui:main or build it locally and run it but don't forget -it

Hi! thanks for your reply @thomas-mauran !
Any chances that this will be playable over a browser in the future ?
The scenario in my mind is to have a self hosted version of the game running a home server accessible over a browser.

Hi! thanks for your reply @thomas-mauran !
Any chances that this will be playable over a browser in the future ? The scenario in my mind is to have a self hosted version of the game running a home server accessible over a browser.

Mmmh currently the game is using the library ratatui which doesn't offer a web browser portage to my knowledge, meaning that we would need to rewrite the whole UI part of the game. This will unfortunately not be done since chess-tui is a TUI project (Terminal UI) but the idea of a central game server for 2 clients to connect is good and could be done !

Thanks for the reply !
Makes sense that replacing a UI library central to the application is a lot of work.
Yes my thinking was that would be great to have 2 people play via a self hosted server.

Just came across this project that seems to enable browser access to ratatui.
[I am not a developer, so apologies if my understanding of what I read there is incorrect].

I am closing this issue since the bug might be resolved and creating a new one for the chess-tui over the browser feature you mentionned @accforgithubtest


Implement substring slice semantics from SQLite

New work in #193.

Thanks! We should also add an alias from "substring" to "substr", which is what the function is called in SQLite.

Should I add the alias before we merge?  Also, it looks like the failure was unrelated to my changes (the age function).

Would be nice to do it with this PR, but not essential. The failure with the Age function indeed seems unrelated, as does the Arrow failure. The SQLancer failure is related but is a matter of teaching SQLancer about the new error message.
Some more test cases would be nice, including triggering the negative length error and some non-inlined strings (i.e. strings with strlen > 12). Otherwise I think the PR is fine.

The alias was already there, but I added a test for the negative length.

The remaining error is caused by a change in the error message not being reflected in sqlancer.  How should we make changes like that?

Could you send a PR with the new error message to this repo? That should fix the problem.

Done. But I haven't tested it on my OSX box - is there documentation on how to do that?

I'm not exactly a Java expert, I haven't ran any of this locally either :)
The best documentation is probably the travis entry:
    - os: linux
      dist: bionic
      name: SQLancer
      if: (branch !~ ^r/.*$ AND head_branch !~ ^r/.*$) AND ((type = push AND branch = master) OR type = pull_request OR tag =~ /^v\d+\.\d+\.\d+$/)

      addons:
        apt:
          sources:
            - sourceline: 'ppa:ubuntu-toolchain-r/test'
          packages:
            - g++-10 openjdk-11-jdk openjdk-11-jre

      env:
        - JAVA_HOME='/usr/lib/jvm/java-11-openjdk-amd64'
        - CC=gcc-10
        - CXX=g++-10
        - MAKEFLAGS=

      script:
        - export PATH=$JAVA_HOME/bin:$PATH
        - mkdir -p build/release
        - python3 scripts/amalgamation.py
        - (cd build/release && cmake -DCMAKE_BUILD_TYPE=RelWithDebInfo -DJDBC_DRIVER=1 -DBUILD_UNITTESTS=FALSE -DAMALGAMATION_BUILD=1 ../.. && travis_wait 30 cmake --build .)
        - java -cp build/release/tools/jdbc/duckdb_jdbc.jar org.duckdb.test.TestDuckDBJDBC
        - git clone https://github.com/hannesmuehleisen/sqlancer.git
        - (cd sqlancer && mvn package -q -DskipTests)
        - cp build/release/tools/jdbc/duckdb_jdbc.jar sqlancer/target/lib/duckdb_jdbc-*.jar
        - java -jar sqlancer/target/SQLancer-*.jar --num-queries 1000 --num-threads 1 --random-seed 0 --timeout-seconds 600 duckdb

But in this case I think it won't be necessary to run it locally. Usually SQLancer reports on the failing queries and makes it obvious why it is failing, so it's reproducible elsewhere.


IE8 issue

Hi!
After processing background: url(image.png) no-repeat becomes background:url(image.png)no-repeat. As you see, there is no whitespace character between properties, so it causes IE8 to fail parsing.

Hello :smile:
I've seen this sort of behaviour work in Chrome; if you have a CSS rule like this:
h1 {
  border-color: rgb(255,255,255) rgb(200,200,200) rgb(150,150,150) rgb(100,100,100);
}

It can be compressed to:
h1 {
  border-color: rgb(255,255,255)rgb(200,200,200)rgb(150,150,150)rgb(100,100,100);
}

Does this fail in IE8 too? Should I remove this behaviour?

Does this fail in IE8 too? Should I remove this behaviour?

I should say that colors without a whitespace character as a separator fail in IE8-IE11 :smile:

Should I remove this behaviour?

Remove it, pls :)


Can't delete product from sidebar cart

Current behaviour
Add products to the sidebar cart.
Click the delete button from the product list.
Expected behaviour
We should delete the product from the sidebar cart normally.
Steps to reproduce the issue
See the resource, have no idea.
Can you handle fixing this bug by yourself?

[ ] YES
[x] NO

Environment details

Browser: Chrome 90.0.4430.72
OS: Windows 10, 20H2
Code Version: master, commit f7eb1b22462d2b38a623eeb5033ab5471758c155

Additional information
NO

Unable to reproduce this on develop. Closing issue.


I want to change the default topic

I want to change the topic of /scan.
This is what the current trajectory_builder_2d.lua looks like with the changes.
submaps = {
    num_range_data = 90,
    grid_options_2d = {
      grid_type = "PROBABILITY_GRID",
      resolution = 0.05,
    },
    range_data_inserter = {
      range_data_inserter_type = "PROBABILITY_GRID_INSERTER_2D",
      probability_grid_range_data_inserter = {
        insert_free_space = true,
        hit_probability = 0.55,
        miss_probability = 0.49,
        topic = "/scan",
      },
    },
  },

However, I got this error.
[cartographer_node-1] F0928 16:47:29.577988 22229 lua_parameter_dictionary.cc:410] Check failed: 1 == reference_counts_.count(key) (1 vs. 0) Key 'topic' was used the wrong number of times.
[cartographer_node-1] [FATAL] [1695887249.578198312] [cartographer_ros]: F0928 16:47:29.000000 22229 lua_parameter_dictionary.cc:410] Check failed: 1 == reference_counts_.count(key) (1 vs. 0) Key 'topic' was used the wrong number of times.

I think it would solve the problem if the default topic could be changed from /scan.
Please let me know the candidate location to change the subscribed topic.

Hello,
here you can find an example on how to change the topic names cartographer expects.


Remove embed wildcard

In https://github.com/pulumi/pulumi-aws/commit/ea37daecbb549cbf52749147da1e1cb59c2d3cbe an error was introduced invalidating the production provider build - a subtle error by moving a go:embed directory to wildcards and FS type.
Per https://pkg.go.dev/embed#hdr-Strings_and_Bytes the path to a go:embed directory is a pattern. Apparently in the case of pulumi-aws CI the binary is built twice, once for testing and once for production release. In the for-testing build environment, schema-embed.json is a file which works as expected. In the for-production build environment, schema-embed.json is a folder created by the download-artifacts action that contains a single file. Before the change, this code worked correctly in both cases:
//go:embed schema-embed.json
var pulumiSchema []byte

After the change, unfortunately the replacement code worked only for the for-test build scenario where schema-embed.json is a straight file, while failing in production.
This change goes back to using the []byte version of go:embed and makes sure the builds recompute the minimal schema as necessary for both builds to succeed.
This fixes https://github.com/pulumi/pulumi-aws/issues/4863
We still should address the root cause and refactor the builds to reduce double-building the provider, so that the exact release build is qualified by integration tests; which is tracked in https://github.com/pulumi/ci-mgmt/issues/967

/release patch


symbol lookup error

I configured a GPIO ( GPIO 25 to be exact and pulled up)
I watch the pin for state change, the following error occurs when the switch is pressed.
node: symbol lookup error: /home/pi/mytools/gateway/node_modules/onoff/node_modules/epoll/build/Release/epoll.node: undefined symbol: _ZN4node12MakeCallbackEN2v86HandleINS0_6ObjectEEENS1_INS0_8FunctionEEEiPNS1_INS0_5ValueEEE

I tried upgrading wheezy but still did not resolve the issue. Current version I am at  :
Linux version 4.1.7+ (dc4@dc4-XPS13-9333) (gcc version 4.8.3 20140303 (prerelease) (crosstool-NG linaro-1.13.1+bzr2650 - Linaro GCC 2014.03) ) #817 PREEMPT Sat Sep 19 15:25:36 BST 2015

onoff version is 1.0.3.

Was onoff installed with one version of Node.js and used with a different version Node.js? If so, this isn't a good idea. In this case please reinstall onoff with npm install onoff.
If the issue persists, please post the complete output of the following commands:
node -v
npm -v
cd
mkdir testonoff
cd testonoff
npm install onoff

Thanks!! for a quick response.
I tried re-installing onoff but the issue is not resolved.
><EMAIL_ADDRESS>install /home/pi/mytools/gateway/node_modules/onoff/node_modules/epoll
> node-gyp rebuild

make: Entering directory '/home/pi/mytools/gateway/node_modules/onoff/node_modules/epoll/build'
  CXX(target) Release/obj.target/epoll/src/epoll.o
  SOLINK_MODULE(target) Release/obj.target/epoll.node
  SOLINK_MODULE(target) Release/obj.target/epoll.node: Finished
  COPY Release/epoll.node
make: Leaving directory '/home/pi/mytools/gateway/node_modules/onoff/node_modules/epoll/build'
<EMAIL_ADDRESS>node_modules/onoff
└──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
Here is the output requested :

pi:/home/pi/mytools/test>node -v
v0.10.0

pi:/home/pi/mytools/test>npm -v
1.2.14

pi:/home/pi/mytools/test>npm install onoff
npm http GET https://registry.npmjs.org/onoff
npm http 304 https://registry.npmjs.org/onoff
npm http GET https://registry.npmjs.org/epoll
npm http 304 https://registry.npmjs.org/epoll
npm http GET https://registry.npmjs.org/bindings
npm http GET https://registry.npmjs.org/nan
npm http 304 https://registry.npmjs.org/bindings
npm http 304 https://registry.npmjs.org/nan

><EMAIL_ADDRESS>install /home/pi/mytools/test/node_modules/onoff/node_modules/epoll
> node-gyp rebuild

make: Entering directory '/home/pi/mytools/test/node_modules/onoff/node_modules/epoll/build'
  CXX(target) Release/obj.target/epoll/src/epoll.o
  SOLINK_MODULE(target) Release/obj.target/epoll.node
  SOLINK_MODULE(target) Release/obj.target/epoll.node: Finished
  COPY Release/epoll.node
make: Leaving directory '/home/pi/mytools/test/node_modules/onoff/node_modules/epoll/build'
<EMAIL_ADDRESS>node_modules/onoff
└──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

Node.js v0.10.0 and npm v1.2.14 are very very old! Where does this version of Node.js come from? I'm actually surprised that onoff appears to have installed successfully. Internally onoff depends on epoll which in turn depends on nan v2.1.x. nan v2.1.x supports Node.js 0.10.29+.
What's the output of the following commands?
cd /home/pi/mytools/test/node_modules/onoff/node_modules/epoll/test
./run-tests

Is there anything preventing you from using Node.js v4 or higher?

yep. node is old. This Pi has been running fine for a while so did not bother to update it.
Anyhow I installed new version of node and npm. Re-installed onoff. Everything good now!.
Thanks for your help. And great job with onoff.
cheers !

Thanks!
Have fun with onoff.


Help with the bot

Hi! There are a couple of questions:

Is it possible to create a bot that can walk on coordinates?

Can I replace the name of the bot device? Usually writes Windows 10, I want something of my own. I have seen many places that this can be changed.

Yes - but without mineflayer, this will be complex as you need to write your own movement code & logic. #116
In offline mode, yes, you can change username with "username" option. In online mode, the username is always the Xbox gamertag.

Not an alias, but the name of the device.
Was WIN10, will become with IOS

Is there a code example using "mineflayer"?
The essence of the connection is clear, but I don't understand how to make the code without errors:(

You'd need to modify the DeviceOS in the payload found here. https://github.com/PrismarineJS/bedrock-protocol/blob/52156a002473b9752e97bd13bf15a18645256667/src/handshake/login.js#L46
Possible values:
{
  "0": "Undefined",
  "1": "Android",
  "2": "IOS",
  "3": "OSX",
  "4": "FireOS",
  "5": "GearVR",
  "6": "Hololens",
  "7": "Win10",
  "8": "Win32",
  "9": "Dedicated",
  "10": "TVOS",
  "11": "Orbis",
  "12": "NintendoSwitch",
  "13": "Xbox",
  "14": "WindowsPhone"
}

const client = createClient(...)

client.session = {
  deviceOS: /* 1-14 */
}

Can you help with 1 question?
Example of moving by coordinates.

As extremeheat mentioned you can move the client but it's a very complex task that will be much simpler to execute once Mineflayer integration is complete. You can check the progress here https://github.com/PrismarineJS/bedrock-protocol/issues/116

OK, thanks


Add lib32gcc1 to Debian Base

lib32gcc1 is needed to run a Soldat Server

Also fixes the Typo in "libunwind8" :)

Arg use the Master :/ New PR inc


Not finding alerts when querying grafana api

I am seeing in the logs
.. backstage info ::1 - - [04/Aug/2022:10:23:04 +0000] "GET /api/proxy/grafana/api/api/alerts?dashboardTag=test-deployment-helm HTTP/1.1" 404 23 "-" "M...

but when I query directly it works
curl -s  "https://grafana.NNNN/api/alerts?dashboardTag=test-deployment-helm" -H 'Authorization: Bearer xxx' | jq .
[
  {
    "id": 143,
    "dashboardId": 216,
....
  }
]

my app-config looks like this
  '/grafana/api':
    target: https://grafana.NNNN/
    headers:
      Authorization: Bearer xxx

Don't know why..thanks!

I am seeing in the logs
.. backstage info ::1 - - [04/Aug/2022:10:23:04 +0000] "GET /api/proxy/grafana/api/api/alerts?dashboardTag=test-deployment-helm HTTP/1.1" 404 23 "-" "M...

but when I query directly it works
curl -s  "https://grafana.NNNN/api/alerts?dashboardTag=test-deployment-helm" -H 'Authorization: Bearer xxx' | jq .
[
  {
    "id": 143,
    "dashboardId": 216,
....
  }
]

my app-config looks like this
  '/grafana/api':
    target: https://grafana.NNNN/
    headers:
      Authorization: Bearer xxx

Don't know why..thanks!

any upadates on this @santinoncs ??


Crash reporting difficult to read "<couldn't map PC to fn name>"

Godot version:
3.1.1 stable official
OS/device including version:
Windows 10 Enterprise 1803
Issue description:
Upon native (c++) exception, C# crash reporting does not give descriptive names, defaulting to <couldn't map PC to fn name>"
Steps to reproduce:
I'm reproducing by generating a C++ exception via C#:
[DllImport("kernel32.dll")]
static extern void RaiseException(uint dwExceptionCode, uint dwExceptionFlags,  uint nNumberOfArguments, IntPtr lpArguments);

// Raise a native exception
RaiseException(13, 0, 0, new IntPtr(1));

Minimal reproduction project:
This type of call stack seems very common in previous bug reports, but people are usually concerned more about the crash than the lack of call stack info.
I offhand don't see a reason why the C# callstack could not be correctly dumped here, though I have not looked very deeply. Thought it worthwhile asking first before I dig deeper.

That's because official binaries don't include debug symbols, which would make the binaries several times bigger.
If you compile from source with a debug binary, you should be able to have the native backtrace.

That's very helpful Akien-mga. I'll try this.
I am also interested in getting corresponding C# call stacks (say from where the manged code entered into unmanaged code) but I don't see any support in the crash_handler_windows.cpp method CrashHandlerException. Do you think this would be impossible? I might try to put together a PR to provide this functionality.

C# call stacks should already be working normally, even in release builds with C++ debug symbols. CC @neikeq

Do you mean making the native crash handler print the C# stack trace as well?

@neikeq Yes, exactly. I'm struggling a bit to correct the missing C++ debug symbols (my project is a bit complex), but in theory I can see it can be done, however the C# stacktrace at that point would be much more helpful to me. Both would be ideal.
If you guys think it's not impossible I could try to provide a PR. But if it's not worthwhile.
Still appreciate your help on this. I'll just close it tonight pending feedback.


Is there a on_after_reset_password hook when calling /auth/reset-password successfully?

Hi great tool, thumbs up!
I wonder if there is something like a on_after_reset_password hook when calling /auth/reset-password successfully?
What I have so far looks like this but I could not figure out if there is a callback on_after_reset_password.
app.include_router(
    fastapi_users.get_reset_password_router(
        SECRET,
        after_forgot_password=on_after_forgot_password,
        reset_password_token_lifetime_seconds=3600
    ),
    prefix='/auth',
    tags=['auth'],
)

Do I miss anything?

Hi there!
Unfortunately, no, there is no on_after_reset_password hook at the moment. What would you be your use-case for this?
Cheers!

Hi thanks for your quick response. Really appreciate your work! The use case would be to send the user an email that the password was changed and if it wasn't the user he might has to contact the support. Did you got what I mean?

Yes, I totally get it! Unfortunately, there is no way to do this at the moment ; but I'll definitely think about it!

Implementation done and merged! Will be released soon 👍

Implementation done and merged! Will be released soon 👍

@all-contributors add @p3t3r67x0 for ideas


Library

Do you have the libraries used in the book?

Hello @FeFerrari87
All the code files and libraries related to the book should be present in the repository or the download links/references should be present in the book.
Thanks for getting in touch.


Just a README.md fix

Current coverage is 74.21% (diff: 100%)

Merging #202 into master will increase coverage by 0.09%

@@             master       #202   diff @@
==========================================
  Files            16         16
  Lines          1109       1109
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            822        823     +1
+ Misses          208        207     -1
  Partials         79         79

Powered by Codecov. Last update eb00b11...74fcd33

Please ignore this comment.  I've configured a commit-checking bot on this repo and am checking if a comment will nudge the bot to chime in.

Closing this as the doc appears correct to me.
@EntropyWorks: feel free to reopen if you think i've missed something!


bump istio-proxy

Please provide a description of this PR:
fix: #50698
This PR contains three parts:

bump proxy to latest
fix ambient test https://github.com/istio/proxy/pull/5472
skip opencensus https://github.com/istio/istio/issues/50814

Will this cause issues when moving the istio.io tests back to the main branch after we release 1.22.0?

it won't, c-ares on 1.30 didn't upgraded.


OIDC email not being added as a claim when I decode id_token

Describe the bug
Hi, I am trying to implement OIDC in my authorization server. I pass openid%20email when making the initial request to get the authorization_code.
After I use the authorization_code to obtain the id token. When I decode the id token, I don't see the email being added as a claim. Am I missing something? Do I need to manually add the email as a claim? I would think spring security would do this behind the scenes.
Sample
GitHub repo
To Reproduce

docker compose up - to run the postgres db

Run spring boot app using ./gradlew bootRun

This will start the authorization server as well as create the users table and insert a user with credentials: username: admin, password: admin, email<EMAIL_ADDRESS>
Paste in browser & login with above credentials, to get the authorization_code

http://localhost:8080/oauth2/authorize?response_type=code&client_id=client&scope=openid%20email&redirect_uri=http://<IP_ADDRESS>:3000/authorized&code_challenge=QYPAZ5NU8yvtlQ9erXrUYR-T5AGCjCF47vN-KsaI2A8&code_challenge_method=S256

Make a POST request in postman (MAKE SURE TO REPLACE THE authorization_code BELOW after code=)

http://localhost:8080/oauth2/token?client_id=client&client_secret=secret&redirect_uri=http://<IP_ADDRESS>:3000/authorized&grant_type=authorization_code&code=`authorization_code`&code_verifier=qPsH306-ZDDaOE8DFzVn05TkN3ZZoVmI_6x4LsVglQI

Decode id token using: https://jwt.io/

Expected behavior
Expect the users email to appear as a claim when I decode the id token.

@AbulSyed

Do I need to manually add the email as a claim?

Correct. You can add the email as a claim by configuring a custom OAuth2TokenCustomizer<JwtEncodingContext> component. Please see the reference documentation on how to implement and configure.


not setting file owner

Description
the file owner is not being set when the rip is complete.
Environment
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=20.04
DISTRIB_CODENAME=focal
DISTRIB_DESCRIPTION="Ubuntu 20.04.2 LTS"
tawhite@arm:/opt/arm$ git branch
* v2.5.0_dev
tawhite@arm:/opt/arm$ git log -1
commit 608c2ae7160dd0869ae78623e475ff2a0c904324 (HEAD -> v2.5.0_dev, origin/v2.5.0_dev, origin/HEAD)
Author: 1337-server<EMAIL_ADDRESS>Date:   Mon Feb 15 00:24:37 2021 +0000

    Update greetings.yml

it is not fully accurate. i have made some changes (see my PRs) but they are all for the web ui.
Log file
TALE_OF_TWO_CITIES_161458190889.log

EXCALIBUR.log
sorry apparently i posted the wrong log. that was for a different issue that happens when i start a bunch of jobs at the same time and i assume that is because one is locking the DB so the others cant edit it.
this log is for a successful job that did not get the owner changed and it defaulted to root. it did change the group and permissions just not the owner

ARM isnt coded to set the owner, confusingly it has config and variables for this. But unfortunatly the code for it was never finished.
Currently only CHMOD_VALUE and the  SET_MEDIA_PERMISSIONS are ever used used

well that would explain it then. i will see if i can finish that code

I looked into this and it appears that this was intentionally left out, changing owners would require root permission. ARM-ripper runs as user 'arm' so chown wouldn't work. I have added this functionality to the ARMui but its not perfect and it wont work in all cases.

ok that makes sense. if it is not implementable then it might be a good idea to remove the configuration

The latest commit has a new button on the ARMui that used to config vars to set the file permission of the so I'm not sure about removing them.

Closing as ARM now has a button to fix permissions.


[Transaction List v2] Action buttons tooltip

Previously the tooltip for the action buttons was tied to the icon, now it's related to the whole button.

Travis automatic deployment:
https://pr1884--safereact.review.gnosisdev.com/rinkeby/app

Travis automatic deployment:
https://pr1884--safereact.review.gnosisdev.com/rinkeby/app


Company Auto Billing - Take single Scheduler

Like any International Software,
We can just take one scheduler and Repeat Payment to ease 12 Scheduler Requirement
Problem -
Some Services are Monthly, and Some are Yearly
Solution -
Design Improvement can take different Scheduler for Diff Services
@vinpahilwani says-
Just change Scheduler Text to Date text, We need to stick to Current Designs As Current Staff / Users Understands it

Just change Scheduler Text to Date text


feat(asset): add calibration due date column

Pull Request
🤨 Rationale
Task: link
👩💻 Implementation
Added a new column
🧪 Testing
Modified a test+

✅ Checklist

[x] This PR has a title that follows the commit message format.

:tada: This PR is included in version 3.36.0 :tada:
The release is available on:

v3.36.0
GitHub release

Your semantic-release bot :package::rocket:


Add note to README about MAD genes

Adding note regarding #99 to README

meant to only tag @jaclyn-taroni (point reviewer for tybalt)


fix(tooltip): adding z-index

Added zIndex to Tooltip.styles.ts, and tested with yarn dev and yarn test.

@trader-dave could you regenerate the changeset as a patch? I don't think this change warrants a minor version bump


Organism Timeline

Props

#11 []
pagination(settings)

Post 1
Post 2
....
[load more]

cleanup index and break into components for more readability

done long time ago